cmd/compile/ssa: panic: unexpected nil Type in LongString

Compiling this function panics during init, in LongString:
package main

func f_ssa() (int, int) {
	var x, y int
	x = 1
	y = 2
	return x, y
}

This looks fixed. Not sure what did it, but the example code works now.


docs: add image depicting the API generation pipeline

Just adds an overview image to get the idea of the pipeline across

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


pfe-cta reduce padding in narrow spaces

Perhaps by adding attribute pfe-size="small"
but even better would be to use element queries to detect container width.

This feels like a dupe of https://github.com/patternfly/patternfly-elements/issues/437


Tokenomics - Staking home page

Description
This change brings in the first PR for merging into the base branch for tokenomics. It is very simply in allowing the PoolsTable to get ready to show staking information
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Dependency changes
[ ] Code refactor / cleanup
[ ] Documentation or wording changes
[ ] Other

How should this be tested?
Simply test that you can see the 'Stake' section under the primary pools table under 'My Investments'. The integrations and cleanups that come with, will come in future pull requests (already exist at time of this being converted to a non-draft PR)
Visual context
https://www.loom.com/share/43e159a6a71b442f9f3492a077c3db79
Checklist:

[X] I have performed a self-review of my own code
[X] I have requested at least 2 reviews (If the PR is significant enough, use best judgement here)
[X] I have commented my code where relevant, particularly in hard-to-understand areas
[X] If package-lock.json has changes, it was intentional.
[X] The base of this PR is master if hotfix, develop if not

@1saf I don't think the primary investment pools list should be using the label 'My APR' unless I'm missing something?

@1saf I don't think the primary investment pools list should be using the label 'My APR' unless I'm missing something?

Fixed


[FEAT] Parse Int96 timestamps and coerce to different precisions

Add an option int96_timestamps_coerce_to_unit=<"ns"|"us"|"ms"> which defaults to "ns"
This option now works for our legacy Parquet reader
This option is propagated to arrow2's infer_schema and column_iter_to_arrays functions in our native downloader. Changes have to be made in the arrow2 repository to add this read option.

Issue filed in arrow2: https://github.com/jorgecarleitao/arrow2/issues/1527


module layoutparser has no attribute EfficientDetLayoutModel

Hi! I run the code of VILA PDFPredictor but find the error as follows. Could you please help me with it? Thank!

Perhaps you might want to try to upgrade layoutparser to the latest version  -- pip install -U layoutparser

Alright close it first and feel free to reopen it if you have more questions!


Mention complementary browser blocking

Alternative: https://www.eff.org/privacybadger

Ghostery does a good job and complementing adblock.

On that topic, I use both ghostery and umatrix. I think that it's somehow redundant with adblock and I don't want to let hosts’ filtering on the browser side when it can be achieved (I guess faster) by the system. What are your thoughts about that?

https://github.com/MattiSG/adblock#is-it-enough  :wink:

It's interesting when you use LittleSnitch to see that Slack channels are tracked via GAnalytics too ;)


axes

I'll create a separate branch for this, as it's complex enough.
Creating three lines of different colors is easy.
...but we probably also want to transparent-fade the rest of the world while we are showing axes, because otherwise the axes are very likely to be occluded by some other geometry.
Also we probably want to add the X,Y,Z writing (and multiple ones since they would be invisible "sideways") and add a nice arrow at the top of the lines (we don't have such shape yet) and we should make the axes scalable and since lines don't scale in thickness that means that the lines are really boxes.
Since we are implementing thick-lines, then thick lines can be minimally made with 2 rectangles and 2 triangles, and they should be flat-filled (no lights should apply), so we should do that.
Also ideally we'd like to back-scale the lines of the axes so they are always the same size no matter what the current scale of the world is - there are a couple of ways to do that (keeping the scale vector around or extracting it from the world matrix) but either method needs implementation.

(in theory instead of writing multiple "X" "Y" "Z" near the axes we could present the "X,Y,Z" always front-facing by extracting the Euler angles (any of the standard 12 common orders) from the world matrix as well. If someone has a better idea let me know.

no wait it's probably easier, I can just take each end-point of the line, find the world coordinate of that point, and move a pristine unrotated letter (starting from the identity matrix) right there. Should work?

draft version of front-facing text is tracked here: https://github.com/davidedc/livecodelab/issues/259


How to generate a test with backward branches?

Please refer to the riscv_rand_instr_test
https://github.com/google/riscv-dv/blob/master/yaml/testlist.yaml#L76
You can increase the ratio of loop instructions to generate more backward branch instructions.
The loop instruction is generated by below class, you can enhance it if you want to cover other backward branch scenarios.
https://github.com/google/riscv-dv/blob/master/src/riscv_loop_instr.sv

Thanks, I got this one generated, how do I increase probability of the loops?
From: taoliug<EMAIL_ADDRESS>Sent: Wednesday, September 4, 2019 5:48 PM
To: google/riscv-dv<EMAIL_ADDRESS>Cc: Alexander Grobman<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [google/riscv-dv] How to generate a test with backward branches? (#112)
CAUTION: This email originated from outside of Western Digital. Do not click on links or open attachments unless you recognize the sender and know that the content is safe.
Please refer to the riscv_rand_instr_test
https://github.com/google/riscv-dv/blob/master/yaml/testlist.yaml#L76
You can increase the ratio of loop instructions to generate more backward branch instructions.
The loop instruction is generated by below class, you can enhance it if you want to cover other backward branch scenarios.
https://github.com/google/riscv-dv/blob/master/src/riscv_loop_instr.sv
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/google/riscv-dv/issues/112?email_source=notifications&email_token=AK5G545BCC24XTS5LVHQPETQIA3LDA5CNFSM4ITW4NAKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD55IARA#issuecomment-528121924, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AK5G545IZXVDBO4K4Z4JVBTQIA3LDANCNFSM4ITW4NAA.

Just increase the number 4 (this is the knob for the possibilities)
+directed_instr_1=riscv_loop_instr,4
You can also remove the other directed tests from the command line options if you don't want to include them in your test.

Thanks a lot again.
From: taoliug<EMAIL_ADDRESS>Sent: Wednesday, September 4, 2019 5:53 PM
To: google/riscv-dv<EMAIL_ADDRESS>Cc: Alexander Grobman<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [google/riscv-dv] How to generate a test with backward branches? (#112)
CAUTION: This email originated from outside of Western Digital. Do not click on links or open attachments unless you recognize the sender and know that the content is safe.
Just increase the number 4 (this is the knob for the possibilities)
+directed_instr_1=riscv_loop_instr,4
You can also remove the other directed tests from the command line options if you don't want to include them in your test.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/google/riscv-dv/issues/112?email_source=notifications&email_token=AK5G546OPIAZUNJYRIY3RCLQIA33PA5CNFSM4ITW4NAKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD55IIEQ#issuecomment-528122898, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AK5G543EGUSRR3QQU423PL3QIA33PANCNFSM4ITW4NAA.

BTW, are you using it for swerv core? I have done some work locally, the ISS/RTL sim comparison part as the tracer from swerv core doesn't have the rd value information. Let me know if you need help on this.

Yes, we want to use riscv_dv  as  a 2nd source test generator for next generation of the swerv cores. I’m not sure what did you mean by “rd value information”. Internally (in house) we are comparing RTL CPU state with ISS for every executed instruction ..
From: taoliug<EMAIL_ADDRESS>Sent: Wednesday, September 4, 2019 5:55 PM
To: google/riscv-dv<EMAIL_ADDRESS>Cc: Alexander Grobman<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [google/riscv-dv] How to generate a test with backward branches? (#112)
CAUTION: This email originated from outside of Western Digital. Do not click on links or open attachments unless you recognize the sender and know that the content is safe.
BTW, are you using it for swerv core? I have done some work locally, the ISS/RTL sim comparison part as the tracer from swerv core doesn't have the rd value information. Let me know if you need help on this.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/google/riscv-dv/issues/112?email_source=notifications&email_token=AK5G545CRXH2TWOSYP54NI3QIA4FZA5CNFSM4ITW4NAKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD55IM2Q#issuecomment-528123498, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AK5G5445DS7M5UQRB72IPITQIA4FZANCNFSM4ITW4NAA.

I mean the external trace interface doesn't provide the result of the instruction. Maybe there'e some other way to get the complete trace info?
https://github.com/westerndigitalcorporation/swerv_eh1/blob/master/design/swerv.sv#L34

I need to talk to management. To get full CPU state you need to integrate our UVM tracer/whisper(ISS).
From: taoliug<EMAIL_ADDRESS>Sent: Wednesday, September 4, 2019 6:06 PM
To: google/riscv-dv<EMAIL_ADDRESS>Cc: Alexander Grobman<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [google/riscv-dv] How to generate a test with backward branches? (#112)
CAUTION: This email originated from outside of Western Digital. Do not click on links or open attachments unless you recognize the sender and know that the content is safe.
I mean the external trace interface doesn't provide the result of the instruction. Maybe there'e some other way to get the complete trace info?
https://github.com/westerndigitalcorporation/swerv_eh1/blob/master/design/swerv.sv#L34
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/google/riscv-dv/issues/112?email_source=notifications&email_token=AK5G544DEJGY3VWVH7OEEKTQIA5LZA5CNFSM4ITW4NAKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD55I6YI#issuecomment-528125793, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AK5G546RFVIIAVJMTLIUIJ3QIA5LZANCNFSM4ITW4NAA.

Sure, let me close this issue for now, we can discuss with a separate thread.
I actually have a plan to open-source an integrated TB with swerv-eh1.  Adding whisper(ISS) should be straightforward, but I failed to install in my local (got core dump). If you get it work, it will be great to add this support in the riscv-dv repository. Let me if you question with ISS integration.

Our ISS guy is going to vacation soon, I’ll ask if he can help you with whisper .. Still whisper need to be integrated to TB …
From: taoliug<EMAIL_ADDRESS>Sent: Wednesday, September 4, 2019 6:20 PM
To: google/riscv-dv<EMAIL_ADDRESS>Cc: Alexander Grobman<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [google/riscv-dv] How to generate a test with backward branches? (#112)
CAUTION: This email originated from outside of Western Digital. Do not click on links or open attachments unless you recognize the sender and know that the content is safe.
Sure, let me close this issue for now, we can discuss with a separate thread.
I actually have a plan to open-source an integrated TB with swerv-eh1. Adding whisper(ISS) should be straightforward, but I failed to install in my local (got core dump). If you get it work, it will be great to add this support in the riscv-dv repository. Let me if you question with ISS integration.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/google/riscv-dv/issues/112?email_source=notifications&email_token=AK5G545HCA7MI4VGZ6Q4RRLQIA7A3A5CNFSM4ITW4NAKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD55JW5A#issuecomment-528128884, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AK5G546NTUOZCOX4DDJSAWLQIA7A3ANCNFSM4ITW4NAA.


StringError("stream did not contain valid UTF-8")

I'm very new to rust so forgive me if this is a poor question. However all I am trying to do is get the data from a url and read it into a Document however I keep getting this error
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Custom { kind: InvalidData, error: StringError("stream did not contain valid UTF-8") }'

I took a look at this issue however my code still isn't too happy. All I'm doing is this..
let url = "http://www.google.com";
let resp = reqwest::get(url).unwrap();
let document = Document::from_read(resp).unwrap();

It works on other websites, but no all (including google). Am I doing something wrong? Could someone point me in the right direction?

Sometimes websites don't send the bytes of their body in UTF-8 encoding, but some other encoding. I'm not certain what Document is exactly, but likely when you call Document::from_read, it reads the response body and tries to decode it as UTF-8. You could probably do a few different things to fix your issue:

If the pages are never that big, or you don't mind reading the full page into memory, you could make use of Response::text to try to auto-decode the body into UTF-8. Then you would feed that new string into Document.
let mut resp = reqwest::get(url).expect("reqwest GET");
let text = resp.text().expect("response text");
let doc = Document::from_read(::std::io::Cursor::new(text.into_bytes())).expect("Document from_read");

You'd need to check the documentation for Document, but there may be other constructors that don't require an impl Read, so you may not need to wrap the string in a Cursor.

Look into using something like the encoding_rscrate to decode the body *while* reading. This *might* result in reducing a copy, and *may* help not read the entire body into memory, depending on whatDocument::from_read` actually does.


Added icicle block

Adds Icicle block

There are a few issues that I found that I would like fixed before merge:

Remove unused fields in IcicleBlock
Annotate appropriate methods in IcicleBlock with @Override
Remove unnecessary casts in IcicleBlock
Remove unused methods in IcicleBlock

Still a few issues:

There are still a few unused methods: tryGrow, getSupportingPos, isHeldByPointedDripstone, getDripPos for example.
Annotate overridden methods with @Override


Allow uniform download links in JCR Compare

Fix for 1836.

Pull Request Test Coverage Report for Build 4404

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 52.774%

Totals

Change from base Build 4400:
0.0%

Covered Lines:
13423

Relevant Lines:
25435

💛  - Coveralls


If the cluster label is already set, we don't overwrite it

We're setting the cluster variable with the external_labels section of prometheus.remote_write. That appears to insert the label, but it does not upsert it.

One thing worth mentioning here is that some users may have dependencies on the "broken" method. For example, elasticsearch, kafka and zookeeper community charts set the "cluster" label which folk then select in their dashboards.
In our estate we set a unique "k8s-cluster" label to identify clusters given how "cluster" can mean different things to many people. Might be worth covering that in docs somewhere.

Fair point! I'll close this issue as a "known quirk".
We wont overwrite the cluster label.


[R-package] Convert to S3; some new DMatrix methods

Addresses #1045

moved everything related to xgb.Booster methods into xgb.Booster.R
moved everything related to xgb.DMatrix methods into xgb.DMatrix.R
converted the S4/S3 class system mix to S3
added XGDMatrixNumCol_R function to xgboost_R.cc
added dim.xgb.DMatrix method
removed explicit nrow method for xgb.DMatrix (it is now handled implicitly by dim)
added dimnames.xgb.DMatrix (getter & setter), which enables colnames to work with xgb.DMatrix
added [.xgb.DMatrix as an alias for slice
added print.xgb.DMatrix
brushed up the xgb.DMatrix documentation
added xgb.DMatrix unit tests

I was trying to preserve the existing user-exposed functionality, and so far my limited test runs were doing good. But some more testing wouldn't hurt.

@hetong007 can you review this PR?

Thank you @khotilov , I appreciate the modification on xgb.DMatrix.
Can you update it with the changes on the master branch? Then I will merge it. Thanks.


feat: introduce decider-agent

Thanks, looking good. Specifically about the retries, I think it's good to handle this on a per-tool basis. E.g., BLAST can take a while until computation completes, that's why a couple more retries are needed until the result can be fetched. I would increase the number for those tools, but 3 is a good default for APIs that should immediately return something.

@fengsh27 can you try again?

@slobentanzer No it seems it doesn't work

Is it the ci.yaml in this repository? You should have write access...
Do you have a more detailed error description?

You should be able to see the error information:

Here is error message:
Switched to branch 'coverage'

##[info] Updating branch coverage
##[info] Removing all files from target branch
##[info] Copying all files from /home/runner/work/biochatter/biochatter/docs/coverage
##[info] Pushing
remote: Permission to biocypher/biochatter.git denied to github-actions[bot].
fatal: unable to access 'https://github.com/biocypher/biochatter.git/': The requested URL returned error: 403

Error: Process exited with code: 1[28](https://github.com/biocypher/biochatter/actions/runs/10150872160/job/28068937469?pr=183#step:8:29):
remote: Permission to biocypher/biochatter.git denied to github-actions[bot].
fatal: unable to access 'https://github.com/biocypher/biochatter.git/': The requested URL returned error: 403

    at ChildProcess.<anonymous> (/home/runner/work/_actions/s0/git-publish-subdir-action/develop/action/dist/index.js:11966:20)
    at ChildProcess.emit (node:events:519:28)
    at maybeClose (node:internal/child_process:1105:16)
    at ChildProcess._handle.onexit (node:internal/child_process:[30](https://github.com/biocypher/biochatter/actions/runs/10150872160/job/28068937469?pr=183#step:8:31)5:5)

@slobentanzer I think I don't have permission to repo biocypher/biochatter. I forked the repo, and can only submit the changes by raising PR.

Ah, these are not your permissions, but the GitHub bot. I recently introduced coverage to the CI, but it does not seem to work on PRs (this is the first PR that has the change I think). This is the changed CI:
      - name: Run tests and generate coverage report
        run: poetry run coverage run -m pytest test --ignore=./volumes
        shell: bash

      - name: Generate coverage badge
        run: poetry run coverage-badge -f -o docs/coverage/coverage.svg
        shell: bash

      - name: Commit changes
        uses: s0/git-publish-subdir-action@develop
        env:
          REPO: self
          BRANCH: coverage
          FOLDER: docs/coverage
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

I guess we cannot update the coverage branch from a PR (nor do we want to). Maybe this can be fixed by simply running the last step only on main, not on PRs. Let me try.

@fengsh27 now it works :)

@slobentanzer is this PR ready to merge?

@fengsh27 have not had time for thorough review, is next on my list. Looking into it now (after dealing with some issues on the biochatter-server Docker).


Wrong checksums on several (transitive) dependencies

Hi everyone,
I'm trying out to migrate our relatively large pom.xml at the moment, using coursier resolverType and I'm getting several checksum errors such as:
java.lang.RuntimeException: we could not download the artifact commons-httpclient:commons-httpclient:3.1 to compute the hash for digest type Sha1 with error com.github.johnynek.bazel_deps.CoursierResolver$FileErrorException: wrong checksum: /Users/haikalpribadi/Library/Caches/Coursier/v1/https/jcenter.bintray.com/commons-httpclient/commons-httpclient/3.1/commons-httpclient-3.1.jar
	at com.github.johnynek.bazel_deps.CoursierResolver$$anonfun$2.apply(CoursierResolver.scala:98)
        ...

I tried explicitly declaring commons-httpclient:commons-http-client:3.1 in input yaml file, and the error went away.
However, I keep on getting more of these errors. Do we know what the real cause is, and what is the best way to resolve these checksum errors?
This is the configuration of my dependency list (yaml input) file to bazel-deps:
options:
  languages: ["java"]
  resolverType: coursier
  resolvers:
  - id: mavencentral
    type: default
    url: "https://jcenter.bintray.com"
  strictVisibility: true
  transitivity: runtime_deps
  versionConflictPolicy: highest
  thirdPartyDirectory: "dependencies/maven/"

ps: I don't get these errors when resolverType: aether, but the documentation says "aether is the default, but it is slower and seems to silently miss some dependencies for reasons we don't yet understand" - and this makes me nervous when using it - are we able to trust it?

Thanks, @johnynek.

so, you are downloading from bintray there...
Why is that? could they be silently changing the jars?
It produced less "missing" warnings/errors than the other repositories, and seem to be more complete.

it may be that the SHA you downloaded does not match the SHA when you fetch
How is the above scenario possible? And how do I avoid it?

A difference with aether vs coursier is that coursier, tries to fetch the declared sha
Does that mean if aether passes fine (ignoring the SHA checks for now), I should be able to trust the generated bazel dependencies will contain all the required artifacts to build and run the system?

Maybe you should check manually if they do actually match?
How do you do that?

At the moment, the reason why I can't move forward with the aether generated dependencies (even though it completes with no errors) is because of issue #184.
Do you know a workaround that I could for now?

#184

so, maven jars declare the sha, e.g.:
https://repo1.maven.org/maven2/commons-httpclient/commons-httpclient/3.1/
see: https://repo1.maven.org/maven2/commons-httpclient/commons-httpclient/3.1/commons-httpclient-3.1.jar.sha1
964cd74171f427720480efdec40a7c7f6e58426a
So, coursier, fetches that value and verifies the hash. You can see if the URL you are downloading has that sha1 value.
If it does, there is some problem with coursier and/or bazel-deps.
if it does not, that means that bintray is not transparently copying the file but is reprocessing it in some way.
This could happen if bintray recompresses the jars, or does any kind of transformation. You can check this using curl and shasum to check the values. Let's try to figure out the above so we can see where the issue is.
Also, you should be able to use maven central, so, could you also open an issue about the problems you see if you use maven central?
We use an internal nexus server that proxies to several remote maven servers, but I don't recall an issue exactly like this.

I've reverted back to url: https://repo.maven.apache.org/maven2/ (and `resolverType: "coursier").
And now it complains:
[main] ERROR MakeDeps - resolution and sha collection failed
java.lang.RuntimeException: no artifacts for servletapi:servletapi:2.3 found

While we can see at maven central that servletapi:2.3 exists: https://repo.maven.apache.org/maven2/servletapi/servletapi/2.3/
Any ideas?
And I suppose I should open a new issue for this too?

is there a config to disable the sha check for coursier, @johnynek ?

interestingly, servletapi is not named as files normally are, note it is servletapi-2.3.jar not servletapi-servletap-2.3.jar
This may be a bug with coursier. We could try bumping to the latest coursier or seeing if we can reproduce the bug just using the coursier command line tool:
https://github.com/coursier/coursier#command-line
it looks like there is a later version of coursier if you want to open a PR to see if we can use the latest in this repo and see if it possibly unblocks you?

Sorry for the late reply on this, @johnynek. I will try to upgrade coursier once I'm done with our current migration from Maven to Bazel, and will let you know my findings here. For now, I've just removed it from our dependency.


在文章中添加“我的更改”文字

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

在文章中添加“我的更改”文字


SyslogEncryptedTcpSender MessageLength is not correct

Hi,
Below is the message received from SyslogNet library for SyslogEncryptedTcpSender.
108 <54>1 2019-06-26T09:48:06.979058+05:30 MD1WU1QC Source_TCP TestRfc5424 TestRfc5424 - This is 5424 Message
If you see above message actually the Message length is 106 but it is sent as 108 which is incorrect.
Assembly Version Used : <IP_ADDRESS>.

Hi,
I think it might be correct. As per RFC5424 the message is encoded in UTF8 and 2 UTF preamble bytes are included before the message text. You don't see them, but they are there ;).


[Bug]: S3 datasource - Save button disabled on page refresh,

Is there an existing issue for this?

[X] I have searched the existing issues

Description
The S3 cred fields are automatically filled with data, Was unable to edit the same. So I had to refresh and the cred fields were editable but save button is disabled.
Steps To Reproduce
Prerequisite

Datasource
Select S3
Refresh the page
Save button is disabled even on entering the cred reqd.

Public Sample App
No response
Environment
Release
Issue video log
No response
Version
Appsmith Business v1.9.24-SNAPSHOT

Closing it as this issue is not reproducible on prod:
https://github.com/appsmithorg/appsmith/assets/1757421/7611ccec-2bd6-4d37-a8d5-a4f24f28d86d


[Feature] Ansible

I just noticed that you provide the option to install Ansible as a provisioner for the Unbuntu boxes. Would it be possible for you to add Ansible for the Centos boxes as well?

PRs accepted ;-)


About total_loss in prognn

total_loss = loss_fro
+ args.gamma * loss_gcn
+ args.alpha * loss_l1
+ args.beta * loss_nuclear
+ args.phi * loss_symmetric
In prognn, total_loss should be
"loss_diffiential
+ args.alpha * loss_l1
+ args.beta * loss_nuclear " ?

Hi songshuhan,
args.phi is usually set to 0; for gamma you can refer to Eq (9) of the paper.

Thanks for your reply！But I mean why "args.lambda_ * loss_smooth_feat" does not appear in total_loss

Hi,
total_loss is not used in backward propagation. The variable loss_differential in line 176 is the one used in backward propagation.

Thanks for your reply！Can I have your wechat? I want to consult  some questions

Hi, sorry for the late reply. You can email me at<EMAIL_ADDRESS>and I will email you the wechat number.


fix: Add network body response in terraform errors for equinix_fabric_connection resource

Add errors method to output err and network response body together
Update equinix_fabric_connection resource to include the errors method in response to errors on network requests

Before:
Do you want to perform these actions?
  Terraform will perform the actions described above.
  Only 'yes' will be accepted to approve.

  Enter a value: yes

equinix_fabric_connection.fcr2azure: Creating...
╷
│ Error: 400 Bad Request
│
│   with equinix_fabric_connection.fcr2azure,
│   on main.tf line 14, in resource "equinix_fabric_connection" "fcr2azure":
│   14: resource  "equinix_fabric_connection" "fcr2azure"{

After:
Do you want to perform these actions?
  Terraform will perform the actions described above.
  Only 'yes' will be accepted to approve.

  Enter a value: yes

equinix_fabric_connection.fcr2azure: Creating...
╷
│ Error: 400 Bad Request;
│  [{"errorCode":"EQ-3142601","errorMessage":"Error while validating Authorization Key","details":"Authorization key provided is not valid.","correlationId":"pd6cVulZP2&JMxPsfDtIoiG#3","additionalInfo":[{"property":"/zSide/accessPoint/authenticationKey"}]}]
│
│   with equinix_fabric_connection.fcr2azure,
│   on main.tf line 14, in resource "equinix_fabric_connection" "fcr2azure":
│   14: resource  "equinix_fabric_connection" "fcr2azure"{

Codecov Report
Attention: 48 lines in your changes are missing coverage. Please review.

Comparison is base (26b2f3d) 60.11% compared to head (027c7ae) 60.08%.
Report is 41 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #449      +/-   ##
==========================================
- Coverage   60.11%   60.08%   -0.04%
==========================================
  Files          99       99
  Lines       20045    20043       -2
==========================================
- Hits        12051    12043       -8
- Misses       7691     7694       +3
- Partials      303      306       +3

Files
Coverage Δ

equinix/config.go
66.48% <100.00%> (+3.19%)
:arrow_up:

equinix/fabric_cloud_router_schema.go
100.00% <100.00%> (ø)

equinix/helpers_device.go
82.26% <97.14%> (-2.65%)
:arrow_down:

equinix/fabric_mapping_helper.go
0.77% <0.00%> (ø)

equinix/data_source_metal_device.go
88.33% <89.47%> (-0.71%)
:arrow_down:

equinix/resource_fabric_cloud_router.go
9.69% <0.00%> (-0.05%)
:arrow_down:

equinix/resource_metal_device.go
78.00% <87.50%> (-1.08%)
:arrow_down:

equinix/errors.go
80.64% <67.85%> (-5.72%)
:arrow_down:

equinix/resource_fabric_connection.go
5.86% <0.00%> (+0.01%)
:arrow_up:

... and 7 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


uniqueness document name per category, validation messages

@nuvanda  - I tested your patch. What we need is as follows :
Continue to support /docs/name-of-document.
But additionally also have the ability to create /docs/category_name/name-of-document.
We would love to see your PR for supporting multiple slugs. Default slug will be /docs/name-of-document for all unique document names. Whenever there is a name collision, we will offer /docs/category-name/name-of-document. We could validate that this is not a unique name and offer to create the /docs/category-name/name-of-document slug instead. Feel free to propose a checkbox in the UI admin if needed. This would support #251 comprehensively.

Closing this PR for now.


Open the website

How to open the website for the first time after the installation?
You said "Go to the admin panel http://host/backend/web/ and change the username and password" but there was no "/web" folder in the backend & frontend.

Sorry, I forgot to update readme.md.
Frontend http://host/public
Backend http://host/public/admin/
Thanks, I'll update the readme soon.


Link to the shared secret generation documentation

Issue #, if available: N/A
Description of changes:
Add a link to the documentation on how to create the shared secret key
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Hi @shinglyu, thanks for the suggestion on how to improve our README. The update you've provided looks good after reviewing.


fix(dt-scroller): fix dynamic scroller not passing fieldKey properly

PR Title

:hammer_and_wrench: Type Of Change

[x] Fix
[ ] Feature
[ ] Refactoring
[ ] Documentation

:book: Description
Field key was still being assigned to id for the dynamic scroller
https://dialpad.atlassian.net/browse/DIALTONE-1081
:bulb: Context

:pencil: Checklist

[x] I have reviewed my changes
[ ] I have added tests
[ ] I have added all relevant documentation
[ ] I have validated components with a screen reader
[ ] I have validated components keyboard navigation
[ ] I have considered the performance impact of my change
[ ] I have checked that my change did not significantly increase bundle size
[ ] I am exporting any new components or constants in the index.js in the component directory
[ ] I am exporting any new components or constants in the index.js in the root

:crystal_ball: Next Steps

:camera: Screenshots / GIFs

:link: Sources

hmm... seems like its a bug in the core-scroller ... to be specific, is in the _updateVisibleItems function in line code 448
const key = keyField ? item?.[keyField] : item;

    if (key == null) {
      throw new Error(`Key is ${key} on item (keyField is '${keyField}')`);
    }

I did a check on the vue-virtual-scroller library and I set it up locally to replicate this, and it seems its a issue in they side.
This is not an issue on our modifications or some change we did...
So I guess this is a valid fix.
However I suggest to do it in this way:
result.push({
         item,
         [keyField]: id,
         size,
       });


Error in the log: "SamsungTV Smart - Detected fake power on, status not updated"

Describe the bug
Always get this error in the log:
Logger: custom_components.samsungtv_smart.media_player
Source: custom_components/samsungtv_smart/media_player.py:643
Integration: SamsungTV Smart (documentation)
First occurred: 14:11:23 (1 occurrences)
Last logged: 14:11:23
SamsungTV Smart - Detected fake power on, status not updated
Expected behavior
If applicable, a clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment details:

Environment (HASSIO, Raspbian, etc): HA in Docker
Home Assistant version installed: Latest 0.115.2
Component version installed: latest
Last know working version: don't know
TV model: Samsung Q65

Output of HA logs
Paste the relavant output of the HA log here.
Logger: custom_components.samsungtv_smart.media_player
Source: custom_components/samsungtv_smart/media_player.py:643
Integration: SamsungTV Smart (documentation)
First occurred: 14:11:23 (1 occurrences)
Last logged: 14:11:23

SamsungTV Smart - Detected fake power on, status not updated

Additional context
Add any other context about the problem here.

This is not an error but a warning. Do you also have issue with the TV status that is not properly detected?

Status update a little bit late, but it is ok. Only this warning in the log.

So just ignore it, is warning you that the TV network is on but is not a real status. I will lower this message to info in the future.

Ok, thank your good work!


Add missing docs for DFS and BFS

Coverage remained the same at 77.96% when pulling f50d63ac02b62e7a15ed6d75961bd8ba864b4196 on shsr04:docs into c6c6489ab2557b8a861b76667bf8f1a7b4f8bab1 on thm-mni-ii:master.


Missing Alert Action on LoginVc and ChatVc

Actual Behaviour
No Alert Action on loginVc and ChatVc
If Internet goes down or disconnected then user need to close the app.
Expected Behaviour
Alert Action should be available in order to avoid this problem.
Screenshots of the issue

Would you like to work on the issue?
Yes

Why do we need this? I suppose after internet connection is available, this alert goes away.

@jogendra , for better user experience , we can use notification banner .
Here are the GIF .

What's your opinion about this ?

@baquer Yeah it is better idea, I was thinking about this one!

@baquer Yes please go with it.


Properties on Error are non-enumerable

Errors returned by Firebase can't be printed to the screen using {{ error }} because the properties are non-enumerable. Maybe we should wrap these in something Angular could understand before rejecting our promises?

:disappointed:

We will consider returning our own error objects that contain just a message, code, and stack.

What about doing an (angular filter)[https://docs.angularjs.org/api/ng/filter/filter] so you could print them out with  {{ error | fb_error }}

I looked into this, and it isn't a Firebase / Angularfire specific issue. Error's properties are not enumerable according to the spec (V8 team discussion on the topic here).
I vote close this as won't fix. Or alternately write a filter like I suggested above, but my thinking is that it's a general problem not specific to angularfire and would be useful released as a separate lib.

I looked into this a bit myself, and I agree that we should won't fix this one as it is beyond the scope of AngularFire. Thanks for the info @jamestalmage!

I don't really agree with this outcome. While the properties on Error are in fact non-enumerable (that was the title of the issue), and this is indeed a problem that is not specific to AngularFire, it's still one that tripped me up while dogfooding AngularFire. I suspect, therefore, it's going to trip others up while using our lib. If we're going to return error objects in our promises, they should probably be something that can be used in a view.
If you're still not swayed, I can defer here. But I feel like this is going to cost dev time with something we could remedy.

Yeah I'm on the fence on this. I kind of group this with the "async bug trap" issue that people run into with things like $asArray() and $loaded(). It's annoying, but it's not really our fault per se. It's just JavaScript gotchas. We could throw non-Error objects but I actually don't really like that solution since the Error object exists for a reason. If anything, Angular should add parsing capabilities so that doing {{ error }} actually prints something other than {}. AngularFire just seems like the wrong place to fix this quirk.


Responsive dialog

Is it possible to make a shown dialog responsive when the browser is resized?
Actually the behaviour is like as shown in this video: https://app.screencast.com/wjwqpGEmcmBUg
I wish to have the dialog responsive (moved to the center of the browser when resized)
Thanks

@radiolondra
You can try specifying the option fixChromeBlur: false.
For example, modify
xdialog.open({
    title: 'Hello, xdialog',
    body: 'This is a message.'
});

to
xdialog.open({
    title: 'Hello, xdialog',
    body: 'This is a message.',
    fixChromeBlur: false
});


Change to use 1d array in traj file samples

Lgtm, it looks like the split range is inclusive on both ends but double check if you haven't already.

What do you mean by this?


EmailBuilder possibly getting wrong relativePath value

On emailBuilder.js line 70 options.relativePath is set to the buffer's base, which the Gulp docs say is

Used for relative pathing. Typically where a glob starts.

This means EmailBuilder will end up trying to load CSS relative to the base path of the glob rather than each HTML file.  So for example, if you glob for src/**/*.html and if one of those HTML files — say src/foo/bar.html — links to ../foo.css, Gulp will try to load src/../foo.css rather than src/foo/../foo.css.  Replacing file.base with path.dirname(file.path) on at least line 70 (and possibly also line 38) results in the second behavior — relative CSS links are relative to the HTML file.  Currently, the same set of CSS & HTML files can be processed with one glob but fail with another; making this change would give more predicable behavior.

Pretty sure I'm running into this issue as well. If I glob ./Templates/Emails/src/**/*.htm I get "Not found, skipping" messages and the css is not embeded. But if I use ./Templates/Emails/src/NameOfFolder/*.htm it works fine


fix an error when viennaCL is used

@karlnapf
let's see whether this PR can fix the error when viennaCL is used.

Let me know when to merge this

Good to go from travis side

Seems good to me, Ill merge

@karlnapf  @lambday
It seems there is still some error when viennaCL is used.
Could you tell me how to configure cmake so that eigen3 is used in GP module while viennaCL is used in linalg

@yorkerlin if both eigen3 and viennacl is there, presently it is not possible to set the latter as default with cmake. If you want to reproduce the error (and if you need both eigen3 and viennacl for that, with viennacl being default), you can locally switch their preferences in linalg.h and test. You need to change their order in the enum Backend.
enum class Backend
{
#ifdef HAVE_VIENNACL
    VIENNACL,
#endif
#ifdef HAVE_EIGEN3
    EIGEN3,
#endif
    NATIVE,
    DEFAULT = 0
};

BTW the cmake that the trusty-libshogun-viennacl builder uses is
cmake -DENABLE_TESTING=ON -DLinAlgBackend=VIENNACL ..


What is the correct version of Python?

I am sure you know that different version of Python has a different way of running things. Please add which version of python is required to run PosterPy. Thank you

It works with python 3.x


Make sure all rednose prints are logged

When creating an EKF class we need to be able to (optionally) pas in a logger object for all the internal error messages.

Fixed by https://github.com/commaai/openpilot/commit/d9c1561d3722c067b0dec5e4c63d09b00399a133


Use UN Population forecasts in demographics.py

The current demographics.py code only takes recent data and generates its own series to compute the steady state and projections. It would be desirable to use the UN models for population, fertility, mortality, and immigration.
UN Population publishes forecasts through 2100. UN Population forecasts are now available in 1-year increments and by age throughout the forecast period: World Population Prospects 2022.
UN Data API (implemented in #4).

@rickecon I'm still interested in the calculation of the population steady state and transition path. The current approach seems to be predicated on not having much data and the need to create a SS as well as the pathway. However, with the transition to using the UN Population data and annual forecasts until 2100, would it be better to simply use this data?
It seems to me that it would be possible to use the last year (2100) as the SS population distribution, and to use the annual data (2021-2099) as the pathway to the SS. This would replace the function get_pop_objs() and generate the pop_dict directly from data instead of computing it. Maybe for a longer term project?

This is a bigger demographics refactoring project than the current PR #14. I am posting two comments made by @SeaCelo in PR #14.
In this comment, @SeaCelo said:

To help understand the South African population profiles, here are the detailed profiles prepared by UN Population: https://population.un.org/wpp/Graphs/DemographicProfiles/Line/710

In this comment, @SeaCelo said:

I looked at the population trends for South Africa and the data is correct. South Africa experienced a big change in fertility and mortality due to a few causes. My quick search of the literature give some clues (preferences, risk aversion by women, HIV epidemic and mortality, etc). When we analyze the results, we will need to keep these factors in mind as the population structure has been affected by the large changes in the 2000s.

demographics.py is now part of OG-CORE (#51). Closing


Add helpers to upload to Koji

Currently, only password auth is supported and it is hooked up to osbuild-composer. But the test and the container should be fully functional.
Based on #149

The container part is here:
https://github.com/osbuild/containers/tree/ghci/ghci-koji/rc1/ghci/containers/ghci-koji

Oh, I wanted to add a short README how to run the koji test locally, hold on with merging. :)

I added a small README about the koji test and bumped the container version from rc1 to v1. Now it's imho ready to be merged.


Convert to Safari extension for MacOS and iOS?

Can you convert this from Chrome to Safari for both desktop and iOS?
https://developer.apple.com/documentation/safariservices/safari_web_extensions/converting_a_web_extension_for_safari
Seems like there's a fairly straight forward path. Would love to see this implemented for Safari users!

Thanks for the suggestion. I will look into this.


Update 03_classification.ipynb

I added some comments to the code in the json format for a much easier readability

Hi @ShehanLiyanaarachchi ,
Thanks for your PR, but it creates an error in the ipynb file: they cannot contain comments like that.
Cheers


Complete support for DataFrame Add

Is your feature request related to a problem? Please describe.
As a experimental operation for DataFrame, DataFrameAdd has support adding two DataFrames (#351) and implement radd in #506 .
A complete support also including:

[ ] add a scalar
[ ] add a tensor
[ ] add a series

See #451 , the implementation of adding a scalar may be different from tensor, AddConstant op is not needed.


Superclass constructor call missing in Event Handler examples?

The wiki examples on how to create an event handler does not call the superclass constructor - is this just missing or is it done on purpose?
In the latter case, I think it should be explained why the constructor is not called.

Hello,
I think this is explained here but of course feel free to edit the wiki page if you want to improve it.


C++ refactoring: ak.flatten

completely_flatten is a method on Content now:
https://github.com/scikit-hep/awkward-1.0/blob/82fda7394418bcae2e4c967f58f9fd5196660803/src/awkward/_v2/contents/content.py#L1231-L1233
It's because it only affects one array. Things that operate on only one array should be methods with overloads on each subclass (the OOP way), rather than functions that if .. elif .. else them all. We only resort to if .. elif .. else style for functions of more than one array, because then we have to deal with the fact that you can get different combinations of node types. With one array, single-dispatch works well.
So completely_flatten should not be reintroduced to ak._v2._util. Thanks!


Add logo to the site.

your site does not have logo which is quite odd so allow me to add logo.

Thank you for suggesting the addition of a logo, but at this point, we’ve decided not to move forward with it. The current design serves the project’s needs, and a logo isn’t a priority right now. I appreciate your input, though


Models 2

Marc and Ford progress with imagersite after adding album and photo models

Changes Unknown when pulling 5884baf703f193ad9719c7ee9e64ed2b922f6050 on models-2 into ** on master**.


fio: 3.5 -> 3.6, enable on darwin

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Success on x86_64-linux (full log)
Attempted: fio
Partial log (click to expand)
stripping (with command strip and flags -S) in /nix/store/6k825pjy829qnj0f9jlz40w4lnw2a8d7-fio-3.6/bin
patching script interpreter paths in /nix/store/6k825pjy829qnj0f9jlz40w4lnw2a8d7-fio-3.6
/nix/store/6k825pjy829qnj0f9jlz40w4lnw2a8d7-fio-3.6/bin/fio_generate_plots: interpreter directive changed from "/bin/sh" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/sh"
/nix/store/6k825pjy829qnj0f9jlz40w4lnw2a8d7-fio-3.6/bin/fio2gnuplot: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/l9j2jsc9flrbmpf799nw9wdq59gpwms8-python-2.7.14/bin/python2.7"
/nix/store/6k825pjy829qnj0f9jlz40w4lnw2a8d7-fio-3.6/bin/genfio: interpreter directive changed from "/usr/bin/bash" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/bash"
/nix/store/6k825pjy829qnj0f9jlz40w4lnw2a8d7-fio-3.6/bin/fiologparser.py: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/l9j2jsc9flrbmpf799nw9wdq59gpwms8-python-2.7.14/bin/python2.7"
/nix/store/6k825pjy829qnj0f9jlz40w4lnw2a8d7-fio-3.6/bin/fiologparser_hist.py: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/l9j2jsc9flrbmpf799nw9wdq59gpwms8-python-2.7.14/bin/python2.7"
/nix/store/6k825pjy829qnj0f9jlz40w4lnw2a8d7-fio-3.6/bin/fio_jsonplus_clat2csv: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/l9j2jsc9flrbmpf799nw9wdq59gpwms8-python-2.7.14/bin/python2.7"
checking for references to /build in /nix/store/6k825pjy829qnj0f9jlz40w4lnw2a8d7-fio-3.6...
/nix/store/6k825pjy829qnj0f9jlz40w4lnw2a8d7-fio-3.6

Success on aarch64-linux (full log)
Attempted: fio
Partial log (click to expand)
stripping (with command strip and flags -S) in /nix/store/ghd2hrf2v5gc5z0ih7v7zdip3a65vwrs-fio-3.6/bin
patching script interpreter paths in /nix/store/ghd2hrf2v5gc5z0ih7v7zdip3a65vwrs-fio-3.6
/nix/store/ghd2hrf2v5gc5z0ih7v7zdip3a65vwrs-fio-3.6/bin/fio_jsonplus_clat2csv: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/pmsj5p5hj9ylj9lnpd0n1vr37m6kl4sa-python-2.7.14/bin/python2.7"
/nix/store/ghd2hrf2v5gc5z0ih7v7zdip3a65vwrs-fio-3.6/bin/fiologparser_hist.py: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/pmsj5p5hj9ylj9lnpd0n1vr37m6kl4sa-python-2.7.14/bin/python2.7"
/nix/store/ghd2hrf2v5gc5z0ih7v7zdip3a65vwrs-fio-3.6/bin/fiologparser.py: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/pmsj5p5hj9ylj9lnpd0n1vr37m6kl4sa-python-2.7.14/bin/python2.7"
/nix/store/ghd2hrf2v5gc5z0ih7v7zdip3a65vwrs-fio-3.6/bin/genfio: interpreter directive changed from "/usr/bin/bash" to "/nix/store/adw9jx59wnrh5659wz43nbjya3m4b3gl-bash-4.4-p19/bin/bash"
/nix/store/ghd2hrf2v5gc5z0ih7v7zdip3a65vwrs-fio-3.6/bin/fio2gnuplot: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/pmsj5p5hj9ylj9lnpd0n1vr37m6kl4sa-python-2.7.14/bin/python2.7"
/nix/store/ghd2hrf2v5gc5z0ih7v7zdip3a65vwrs-fio-3.6/bin/fio_generate_plots: interpreter directive changed from "/bin/sh" to "/nix/store/adw9jx59wnrh5659wz43nbjya3m4b3gl-bash-4.4-p19/bin/sh"
checking for references to /build in /nix/store/ghd2hrf2v5gc5z0ih7v7zdip3a65vwrs-fio-3.6...
/nix/store/ghd2hrf2v5gc5z0ih7v7zdip3a65vwrs-fio-3.6

Success on x86_64-darwin (full log)
Attempted: fio
Partial log (click to expand)
strip is /nix/store/kdff2gim6417493yha769kh00n63lnrw-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/k6mfaw1qa67gmipc2nkf8zar29pcvp85-fio-3.6/bin
patching script interpreter paths in /nix/store/k6mfaw1qa67gmipc2nkf8zar29pcvp85-fio-3.6
/nix/store/k6mfaw1qa67gmipc2nkf8zar29pcvp85-fio-3.6/bin/fio2gnuplot: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/10f4qxb00wk9j6cicd9h1qzwqgnc36ji-python-2.7.14/bin/python2.7"
/nix/store/k6mfaw1qa67gmipc2nkf8zar29pcvp85-fio-3.6/bin/fio_generate_plots: interpreter directive changed from "/bin/sh" to "/nix/store/r8bx3qf1bpncb14i9gzma4vr089pc3pv-bash-4.4-p19/bin/sh"
/nix/store/k6mfaw1qa67gmipc2nkf8zar29pcvp85-fio-3.6/bin/fio_jsonplus_clat2csv: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/10f4qxb00wk9j6cicd9h1qzwqgnc36ji-python-2.7.14/bin/python2.7"
/nix/store/k6mfaw1qa67gmipc2nkf8zar29pcvp85-fio-3.6/bin/fiologparser.py: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/10f4qxb00wk9j6cicd9h1qzwqgnc36ji-python-2.7.14/bin/python2.7"
/nix/store/k6mfaw1qa67gmipc2nkf8zar29pcvp85-fio-3.6/bin/fiologparser_hist.py: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/10f4qxb00wk9j6cicd9h1qzwqgnc36ji-python-2.7.14/bin/python2.7"
/nix/store/k6mfaw1qa67gmipc2nkf8zar29pcvp85-fio-3.6/bin/genfio: interpreter directive changed from "/usr/bin/bash" to "/nix/store/r8bx3qf1bpncb14i9gzma4vr089pc3pv-bash-4.4-p19/bin/bash"
/nix/store/k6mfaw1qa67gmipc2nkf8zar29pcvp85-fio-3.6

Success on x86_64-linux (full log)
Attempted: fio
Partial log (click to expand)
/nix/store/6k825pjy829qnj0f9jlz40w4lnw2a8d7-fio-3.6

Success on aarch64-linux (full log)
Attempted: fio
Partial log (click to expand)
/nix/store/ghd2hrf2v5gc5z0ih7v7zdip3a65vwrs-fio-3.6

Success on x86_64-darwin (full log)
Attempted: fio
Partial log (click to expand)
strip is /nix/store/kdff2gim6417493yha769kh00n63lnrw-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/k6mfaw1qa67gmipc2nkf8zar29pcvp85-fio-3.6/bin
patching script interpreter paths in /nix/store/k6mfaw1qa67gmipc2nkf8zar29pcvp85-fio-3.6
/nix/store/k6mfaw1qa67gmipc2nkf8zar29pcvp85-fio-3.6/bin/fio_jsonplus_clat2csv: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/10f4qxb00wk9j6cicd9h1qzwqgnc36ji-python-2.7.14/bin/python2.7"
/nix/store/k6mfaw1qa67gmipc2nkf8zar29pcvp85-fio-3.6/bin/fio_generate_plots: interpreter directive changed from "/bin/sh" to "/nix/store/r8bx3qf1bpncb14i9gzma4vr089pc3pv-bash-4.4-p19/bin/sh"
/nix/store/k6mfaw1qa67gmipc2nkf8zar29pcvp85-fio-3.6/bin/fiologparser_hist.py: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/10f4qxb00wk9j6cicd9h1qzwqgnc36ji-python-2.7.14/bin/python2.7"
/nix/store/k6mfaw1qa67gmipc2nkf8zar29pcvp85-fio-3.6/bin/genfio: interpreter directive changed from "/usr/bin/bash" to "/nix/store/r8bx3qf1bpncb14i9gzma4vr089pc3pv-bash-4.4-p19/bin/bash"
/nix/store/k6mfaw1qa67gmipc2nkf8zar29pcvp85-fio-3.6/bin/fio2gnuplot: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/10f4qxb00wk9j6cicd9h1qzwqgnc36ji-python-2.7.14/bin/python2.7"
/nix/store/k6mfaw1qa67gmipc2nkf8zar29pcvp85-fio-3.6/bin/fiologparser.py: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/10f4qxb00wk9j6cicd9h1qzwqgnc36ji-python-2.7.14/bin/python2.7"
/nix/store/k6mfaw1qa67gmipc2nkf8zar29pcvp85-fio-3.6

Success on x86_64-linux (full log)
Attempted: fio
Partial log (click to expand)
stripping (with command strip and flags -S) in /nix/store/lp8vr5bx92afya7brdfw4daxgixnjd04-fio-3.6/bin
patching script interpreter paths in /nix/store/lp8vr5bx92afya7brdfw4daxgixnjd04-fio-3.6
/nix/store/lp8vr5bx92afya7brdfw4daxgixnjd04-fio-3.6/bin/fio_generate_plots: interpreter directive changed from "/bin/sh" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/sh"
/nix/store/lp8vr5bx92afya7brdfw4daxgixnjd04-fio-3.6/bin/fio2gnuplot: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/l9j2jsc9flrbmpf799nw9wdq59gpwms8-python-2.7.14/bin/python2.7"
/nix/store/lp8vr5bx92afya7brdfw4daxgixnjd04-fio-3.6/bin/genfio: interpreter directive changed from "/usr/bin/bash" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/bash"
/nix/store/lp8vr5bx92afya7brdfw4daxgixnjd04-fio-3.6/bin/fiologparser.py: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/l9j2jsc9flrbmpf799nw9wdq59gpwms8-python-2.7.14/bin/python2.7"
/nix/store/lp8vr5bx92afya7brdfw4daxgixnjd04-fio-3.6/bin/fiologparser_hist.py: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/l9j2jsc9flrbmpf799nw9wdq59gpwms8-python-2.7.14/bin/python2.7"
/nix/store/lp8vr5bx92afya7brdfw4daxgixnjd04-fio-3.6/bin/fio_jsonplus_clat2csv: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/l9j2jsc9flrbmpf799nw9wdq59gpwms8-python-2.7.14/bin/python2.7"
checking for references to /build in /nix/store/lp8vr5bx92afya7brdfw4daxgixnjd04-fio-3.6...
/nix/store/lp8vr5bx92afya7brdfw4daxgixnjd04-fio-3.6

Success on aarch64-linux (full log)
Attempted: fio
Partial log (click to expand)
stripping (with command strip and flags -S) in /nix/store/5zh0g5n3xmxwlbm5zb614d4a5pv3v1fq-fio-3.6/bin
patching script interpreter paths in /nix/store/5zh0g5n3xmxwlbm5zb614d4a5pv3v1fq-fio-3.6
/nix/store/5zh0g5n3xmxwlbm5zb614d4a5pv3v1fq-fio-3.6/bin/fio_jsonplus_clat2csv: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/pmsj5p5hj9ylj9lnpd0n1vr37m6kl4sa-python-2.7.14/bin/python2.7"
/nix/store/5zh0g5n3xmxwlbm5zb614d4a5pv3v1fq-fio-3.6/bin/fiologparser_hist.py: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/pmsj5p5hj9ylj9lnpd0n1vr37m6kl4sa-python-2.7.14/bin/python2.7"
/nix/store/5zh0g5n3xmxwlbm5zb614d4a5pv3v1fq-fio-3.6/bin/fiologparser.py: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/pmsj5p5hj9ylj9lnpd0n1vr37m6kl4sa-python-2.7.14/bin/python2.7"
/nix/store/5zh0g5n3xmxwlbm5zb614d4a5pv3v1fq-fio-3.6/bin/genfio: interpreter directive changed from "/usr/bin/bash" to "/nix/store/adw9jx59wnrh5659wz43nbjya3m4b3gl-bash-4.4-p19/bin/bash"
/nix/store/5zh0g5n3xmxwlbm5zb614d4a5pv3v1fq-fio-3.6/bin/fio2gnuplot: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/pmsj5p5hj9ylj9lnpd0n1vr37m6kl4sa-python-2.7.14/bin/python2.7"
/nix/store/5zh0g5n3xmxwlbm5zb614d4a5pv3v1fq-fio-3.6/bin/fio_generate_plots: interpreter directive changed from "/bin/sh" to "/nix/store/adw9jx59wnrh5659wz43nbjya3m4b3gl-bash-4.4-p19/bin/sh"
checking for references to /build in /nix/store/5zh0g5n3xmxwlbm5zb614d4a5pv3v1fq-fio-3.6...
/nix/store/5zh0g5n3xmxwlbm5zb614d4a5pv3v1fq-fio-3.6

Success on x86_64-darwin (full log)
Attempted: fio
Partial log (click to expand)
strip is /nix/store/kdff2gim6417493yha769kh00n63lnrw-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/6r43cvjasksj287rmwya1hczb2qgl219-fio-3.6/bin
patching script interpreter paths in /nix/store/6r43cvjasksj287rmwya1hczb2qgl219-fio-3.6
/nix/store/6r43cvjasksj287rmwya1hczb2qgl219-fio-3.6/bin/fio2gnuplot: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/10f4qxb00wk9j6cicd9h1qzwqgnc36ji-python-2.7.14/bin/python2.7"
/nix/store/6r43cvjasksj287rmwya1hczb2qgl219-fio-3.6/bin/fio_generate_plots: interpreter directive changed from "/bin/sh" to "/nix/store/r8bx3qf1bpncb14i9gzma4vr089pc3pv-bash-4.4-p19/bin/sh"
/nix/store/6r43cvjasksj287rmwya1hczb2qgl219-fio-3.6/bin/fio_jsonplus_clat2csv: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/10f4qxb00wk9j6cicd9h1qzwqgnc36ji-python-2.7.14/bin/python2.7"
/nix/store/6r43cvjasksj287rmwya1hczb2qgl219-fio-3.6/bin/fiologparser.py: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/10f4qxb00wk9j6cicd9h1qzwqgnc36ji-python-2.7.14/bin/python2.7"
/nix/store/6r43cvjasksj287rmwya1hczb2qgl219-fio-3.6/bin/fiologparser_hist.py: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/10f4qxb00wk9j6cicd9h1qzwqgnc36ji-python-2.7.14/bin/python2.7"
/nix/store/6r43cvjasksj287rmwya1hczb2qgl219-fio-3.6/bin/genfio: interpreter directive changed from "/usr/bin/bash" to "/nix/store/r8bx3qf1bpncb14i9gzma4vr089pc3pv-bash-4.4-p19/bin/bash"
/nix/store/6r43cvjasksj287rmwya1hczb2qgl219-fio-3.6

Success on x86_64-darwin (full log)
Attempted: fio
Partial log (click to expand)
strip is /nix/store/kdff2gim6417493yha769kh00n63lnrw-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/1ivn2rvs8qs3qqa06n9h1qindsva3cjm-fio-3.6/bin
patching script interpreter paths in /nix/store/1ivn2rvs8qs3qqa06n9h1qindsva3cjm-fio-3.6
/nix/store/1ivn2rvs8qs3qqa06n9h1qindsva3cjm-fio-3.6/bin/fio_jsonplus_clat2csv: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/10f4qxb00wk9j6cicd9h1qzwqgnc36ji-python-2.7.14/bin/python2.7"
/nix/store/1ivn2rvs8qs3qqa06n9h1qindsva3cjm-fio-3.6/bin/fio_generate_plots: interpreter directive changed from "/bin/sh" to "/nix/store/r8bx3qf1bpncb14i9gzma4vr089pc3pv-bash-4.4-p19/bin/sh"
/nix/store/1ivn2rvs8qs3qqa06n9h1qindsva3cjm-fio-3.6/bin/fiologparser_hist.py: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/10f4qxb00wk9j6cicd9h1qzwqgnc36ji-python-2.7.14/bin/python2.7"
/nix/store/1ivn2rvs8qs3qqa06n9h1qindsva3cjm-fio-3.6/bin/genfio: interpreter directive changed from "/usr/bin/bash" to "/nix/store/r8bx3qf1bpncb14i9gzma4vr089pc3pv-bash-4.4-p19/bin/bash"
/nix/store/1ivn2rvs8qs3qqa06n9h1qindsva3cjm-fio-3.6/bin/fio2gnuplot: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/10f4qxb00wk9j6cicd9h1qzwqgnc36ji-python-2.7.14/bin/python2.7"
/nix/store/1ivn2rvs8qs3qqa06n9h1qindsva3cjm-fio-3.6/bin/fiologparser.py: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/10f4qxb00wk9j6cicd9h1qzwqgnc36ji-python-2.7.14/bin/python2.7"
/nix/store/1ivn2rvs8qs3qqa06n9h1qindsva3cjm-fio-3.6

Success on aarch64-linux (full log)
Attempted: fio
Partial log (click to expand)
stripping (with command strip and flags -S) in /nix/store/7lrhayznq84kd5xp6rwzjx2xlshkazg7-fio-3.6/bin
patching script interpreter paths in /nix/store/7lrhayznq84kd5xp6rwzjx2xlshkazg7-fio-3.6
/nix/store/7lrhayznq84kd5xp6rwzjx2xlshkazg7-fio-3.6/bin/fio_jsonplus_clat2csv: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/pmsj5p5hj9ylj9lnpd0n1vr37m6kl4sa-python-2.7.14/bin/python2.7"
/nix/store/7lrhayznq84kd5xp6rwzjx2xlshkazg7-fio-3.6/bin/fiologparser_hist.py: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/pmsj5p5hj9ylj9lnpd0n1vr37m6kl4sa-python-2.7.14/bin/python2.7"
/nix/store/7lrhayznq84kd5xp6rwzjx2xlshkazg7-fio-3.6/bin/fiologparser.py: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/pmsj5p5hj9ylj9lnpd0n1vr37m6kl4sa-python-2.7.14/bin/python2.7"
/nix/store/7lrhayznq84kd5xp6rwzjx2xlshkazg7-fio-3.6/bin/genfio: interpreter directive changed from "/usr/bin/bash" to "/nix/store/adw9jx59wnrh5659wz43nbjya3m4b3gl-bash-4.4-p19/bin/bash"
/nix/store/7lrhayznq84kd5xp6rwzjx2xlshkazg7-fio-3.6/bin/fio2gnuplot: interpreter directive changed from "/usr/bin/python2.7" to "/nix/store/pmsj5p5hj9ylj9lnpd0n1vr37m6kl4sa-python-2.7.14/bin/python2.7"
/nix/store/7lrhayznq84kd5xp6rwzjx2xlshkazg7-fio-3.6/bin/fio_generate_plots: interpreter directive changed from "/bin/sh" to "/nix/store/adw9jx59wnrh5659wz43nbjya3m4b3gl-bash-4.4-p19/bin/sh"
checking for references to /build in /nix/store/7lrhayznq84kd5xp6rwzjx2xlshkazg7-fio-3.6...
/nix/store/7lrhayznq84kd5xp6rwzjx2xlshkazg7-fio-3.6

Success on x86_64-linux (full log)
Attempted: fio
Partial log (click to expand)
these paths will be fetched (0.35 MiB download, 1.54 MiB unpacked):
  /nix/store/z5bzhpwxh4rnrvv8a0zc6mjpw0v0g2r4-fio-3.6
copying path '/nix/store/z5bzhpwxh4rnrvv8a0zc6mjpw0v0g2r4-fio-3.6' from 'https://cache.nixos.org'...
/nix/store/z5bzhpwxh4rnrvv8a0zc6mjpw0v0g2r4-fio-3.6


Replace "internal" metrics.Factory usage with direct calls to expvar

Some of Jaeger components utilize expvar to report their configuration options for introspection. However, instead of using expvar directly they are going about it in a round-about way:

they create a combined metrics factory via fork.New which takes the primary factory and an expvar-backed metrics. Example: https://github.com/jaegertracing/jaeger/blob/a7a7308c2da52a90ebf75313ce322f82cd43b59d/cmd/agent/main.go#L64
the secondary factory is only used by Fork when the namespace starts with "internal". Example: https://github.com/jaegertracing/jaeger/blob/a7a7308c2da52a90ebf75313ce322f82cd43b59d/cmd/agent/app/builder.go#L135

We can simplify all of this by letting those few places that create internal namespace to use expvar directly:

./cmd/agent/app/builder.go:132
./plugin/storage/memory/factory.go:127
./plugin/storage/factory.go:342

For example, this code https://github.com/jaegertracing/jaeger/blob/a7a7308c2da52a90ebf75313ce322f82cd43b59d/plugin/storage/memory/factory.go#L126-L130
can be replaced with
func (f *Factory) publishOpts() {
	v := expvar.NewInt("jaeger_storage_memory_max_traces")
	v.Set(int64(f.options.Configuration.MaxTraces))
}

Steps

[ ] track down the use of fork.New factory and remove them
[ ] identify which internal components use "internal" namespace to report settings and replace with expvar directly
[ ] remove internal/metrics/fork/*
[ ] remove internal/metrics/expvar/*
[ ] fix tests

hii @yurishkuro, after removing the  internal/metrics/fork, what could be the necessary way to return the factory ??!

return from where?

return from where?

in the https://github.com/jaegertracing/jaeger/blob/a7a7308c2da52a90ebf75313ce322f82cd43b59d/cmd/agent/main.go#L64
to be used afterwards

we just use the base factory directly


RabbitMQ: channel not initialized, even if channel exists

Expected Behavior
Dapr to send messages, since channel exists
Actual Behavior
Dapr complains about.
error in subscriber for scraper-scraper in ensureSubscription: channel not initialized"
Steps to Reproduce the Problem
I'm not quite sure? It started happening since I've updated to 1.5.1

Release Note

RELEASE NOTE:

@WhoAteDaCake , could you provide instructions on how to reproduce this issue?

/assign

Sorry, but I couldn't catch the exact scenario. I'll try to replicate over the weekend. It started happening as soon as I updated from 1.3.1 -> 1.5.1. We reverted back now and the issue has stopped

@WhoAteDaCake , I have a fix for #1372.
Could you try it with your test case by changing daprd image from daprio/daprd:1.5.1 to dmitsh/daprd:dev-rmq-linux-amd64 ?

It seems to be working as expected now. Thank you for your work @dmitsh

This not resolve my problem.
I create this branch repository: https://github.com/andreatosato/dapr-cloudchampion/tree/rabbitmq-error
time="2021-12-14T21:22:11.8985009Z" level=error msg="rabbitmq pub/sub: error in subscriber for cloudchampion-web-orderstatus in ensureSubscription: channel not initialized" app_id=cloudchampion-web instance=c821022bbc95 scope=dapr.contrib type=log ver=edge

time="2021-12-14T21:22:11.8985535Z" level=warning msg="rabbitmq pub/sub: subscriber is reconnecting in 3s ..." app_id=cloudchampion-web instance=c821022bbc95 scope=dapr.contrib type=log ver=edge

time="2021-12-14T21:22:14.9033483Z" level=info msg="rabbitmq pub/sub: connectionCount: current=0 reference=0" app_id=cloudchampion-web instance=c821022bbc95 scope=dapr.contrib type=log ver=edge

time="2021-12-14T21:22:14.9135839Z" level=info msg="rabbitmq pub/sub: connected with connectionCount=1" app_id=cloudchampion-web instance=c821022bbc95 scope=dapr.contrib type=log ver=edge

time="2021-12-14T21:22:14.9270754Z" level=info msg="rabbitmq pub/sub: declaring queue 'cloudchampion-web-orderstatus'" app_id=cloudchampion-web instance=c821022bbc95 scope=dapr.contrib type=log ver=edge

time="2021-12-14T21:22:14.9488794Z" level=info msg="rabbitmq pub/sub: binding queue 'cloudchampion-web-orderstatus' to exchange 'orderstatus'" app_id=cloudchampion-web instance=c821022bbc95 scope=dapr.contrib type=log ver=edge

time="2021-12-14T21:22:14.9622449Z" level=info msg="rabbitmq pub/sub: subscribe to topic/queue 'orderprocessed/cloudchampion-web-orderprocessed'" app_id=cloudchampion-web instance=c821022bbc95 scope=dapr.contrib type=log ver=edge

time="2021-12-14T21:22:15.0300956Z" level=info msg="rabbitmq pub/sub: declaring queue 'cloudchampion-web-orderprocessed'" app_id=cloudchampion-web instance=c821022bbc95 scope=dapr.contrib type=log ver=edge

time="2021-12-14T21:22:15.0657686Z" level=info msg="rabbitmq pub/sub: binding queue 'cloudchampion-web-orderprocessed' to exchange 'orderprocessed'" app_id=cloudchampion-web instance=c821022bbc95 scope=dapr.contrib type=log ver=edge

time="2021-12-14T21:22:15.0776415Z" level=info msg="dapr initialized. Status: Running. Init Elapsed 40594.8337ms" app_id=cloudchampion-web instance=c821022bbc95 scope=dapr.runtime type=log ver=edge

/cc @dmitsh

Reopened for now.
cc @artursouza

@andreatosato , could you elaborate what is the problem? From the log I can see that your application has connected to rabbitmq pub/sub.

When application start, DAPR must create topic on rabbit.
Then this is the error:
rabbitmq pub/sub: error in subscriber for cloudchampion-web-orderstatus in ensureSubscription: channel not initialized" app_id=cloudchampion-web instance=c821022bbc95 scope=dapr.contrib type=log ver=edge

time="2021-12-14T21:22:11.8985535Z" level=warning msg="rabbitmq pub/sub: subscriber is reconnecting in 3s ..." app_id=cloudchampion-web instance=c821022bbc95 scope=dapr.contrib type=log ver=edge

Right, but after that the component is reconnected:
time="2021-12-14T21:22:11.8985535Z" level=warning msg="rabbitmq pub/sub: subscriber is reconnecting in 3s ..." app_id=cloudchampion-web instance=c821022bbc95 scope=dapr.contrib type=log ver=edge
Does your application work as expected?

My sample don't work fine, this is my error application:
 Dapr.DaprException: Publish operation failed: the Dapr endpoint indicated a failure. See InnerException for details.
       ---> Grpc.Core.RpcException: Status(StatusCode="Internal", Detail="error when publish to topic orderstatus in pubsub pubsub: channel not initialized")

could you provide the steps to reproduce the error?

I create 3 topic in my sample.
I have clean rabbit (empty configuration, no exchange, no queue, no topic).
When start application for the first time, Dapr raise error.
In my sample, run docker compose, start web application.
I have only index page.
Insert number, than text and click button... Application crash

could you be more specific with the steps?

From visual studio 2022:

run docker compose
open web container on index
compile form

number

text

click button

Error appear

docker-compose  -f "C:\Users\AndreaTosato\Source\Repos\dapr-cloudchampion\docker-compose.yml" -f "C:\Users\AndreaTosato\Source\Repos\dapr-cloudchampion\docker-compose.override.yml" -f "C:\Users\AndreaTosato\Source\Repos\dapr-cloudchampion\obj\Docker\docker-compose.vs.debug.g.yml" -p dockercompose2803362638168428963 --ansi never up -d --build --remove-orphans
This is command generated from VS 2022

docker-compose -f "C:\Users\AndreaTosato\Source\Repos\dapr-cloudchampion\docker-compose.yml" -f "C:\Users\AndreaTosato\Source\Repos\dapr-cloudchampion\docker-compose.override.yml" -f "C:\Users\AndreaTosato\Source\Repos\dapr-cloudchampion\obj\Docker\docker-compose.vs.debug.g.yml" -p dockercompose2803362638168428963 --ansi never up -d --build --remove-orphans

This command starts docker containers, but does not trigger the application flow.
Do you have to run your application on Windows? Is there a way to isolate the issue and reproduce it in a platform-agnostic way?

I think I found it. Could you try this image: dmitsh/daprd:dev-rmq1-linux-amd64 ?

Works now!
Thanks @dmitsh
Can you release dapr 1.5.2 this change?

Can you release dapr 1.5.2 this change?

Most likely it will be released in 1.6 next month.

Works now! Thanks @dmitsh
Can you release dapr 1.5.2 this change?

@dmitsh is correct, this will go into 1.6.

I also encountered this problem. It is easy to reproduce it, such as restart or replace rabbitMQ, and there is a high probability of the above error occurring in applications with subscriptions.


Apex driver failure on Ubuntu 20.04 on Raspberry Pi Compute Module 4.

Hello,
I am trying to use the "Mini PCIe Accelerator" on the Raspberry Pi Compute Module 4 running Ubuntu Mate 20.04 64-bit. I am able to install the drivers and necessary runtimes required. On the device, I have libedgetpu1-std=15.0, python3-pycoral=1.0.1, and gasket-dkms=1.0-14, these are able to run the "USB Accelerator". But when I go to use the PCIe Accelerator, I am unable to get past ValueError: Failed to load delegate from libedgetpu.so.1 when running...
from pycoral.utils import edgetpu

# Works on usb accelerator
tpu = edgetpu.make_interpreter("ssd_mobilenet_v2_coco_quant_postprocess_edgetpu.tflite")

And for debugging purposes, I ran dmesg and found...
apex 0000:01:00.1: Couldn't reinit interupts: -524
apex 0000:01:00.1: Permission checking failed.

This is concerning as the Getting Started page for the M.2/Mini PCIe accelerators say that Ubuntu 16.04 or later is supported on ARMv8. I have the apex driver setup with the proper udev rules and added my user to the group.

Closing in order to follow issue at https://github.com/google-coral/edgetpu/issues/280


👩⚕ Feature Request: Display Doctor Experience and Minimum Fee

Patients often face challenges in identifying doctors with suitable experience within their budget. Therefore, it would be beneficial to display the experience level and minimum fee charged by every doctor on the platform.
Feature Points:

Implement a feature to showcase the experience level of each doctor, including years of practice or relevant qualifications.
Display the minimum fee charged by each doctor for their services, providing transparency and helping patients make informed decisions.
Enhance user experience by allowing patients to filter and sort doctors based on their experience level and minimum fee.

Program: GSSoC24
Code of Conduct:
I will adhere to the CONTRIBUTING GUIDELINE of this project and ensure that all contributions align with the project's standards and requirements.
"I'm excited about the potential of this feature to enhance the user experience and streamline the process of finding the right doctor for each patient.
Kindly assign this issue to me...
I already installed it into my local system and want to contribute it.
Looking forward to contributing to the implementation of this feature!"

Assigning to you...go ahead

@nidhaahmed  Assigned! Any updates on this?

@nidhaahmed  Assigned! Any updates on this?

Hey @varshith257 ,
I have already started working on it but my pc crashed and it is now in repair. It may take two days approximately to be alright. I'm really sorry for the situation but I promise I will raise the pr ASAP. Thank you.

Ok, take your time and add this feature 🚀

ANY UPDATES?

I'm interested to work on this issue . I request you @Anishkagupta04 to assign this issue to me .
thank you!

Unassigned! No activity

@varshith257 please assign this issue under GSSOC

@varshith257 please assign this issue to me . I will try to complete this issue within  less time limit

Assigning to the one who commented first after unassign

@varshith257 please check below video this the idea that is described in the issue , i made a rough layout , if that okay , i will add styling's
https://github.com/Anishkagupta04/RAPIDOC-HEALTHCARE-WEBSITE-/assets/154777864/d9186d5f-58e1-44ac-9066-160f9a61b973

Go ahead

Okay

@varshith257 please see the Below preview , if it's okay i will create a PR
https://github.com/Anishkagupta04/RAPIDOC-HEALTHCARE-WEBSITE-/assets/154777864/a444f658-79ea-48b1-8f84-1ca083ca69ce

@varshith257 close this issue.


License

Can't find a license description - without our company does us not allow to use it

APEX Visualizer License is set to MIT License


sweetchicks.net fuckmebook.com

Scam adults sites. #NSFW
FROM https://linkfly.to/WhatsAppaPPDating opening some blocked by adguard sites. Anyway if allow go next opens scams whos is not blocked. Thats why I made this PR.
Link shared on facebook groups.

Thanks T!  Merging.


Unable to fetch push certificate. Request ID [...], message: Internal error occurred. Please try again in a few minutes or contact support

I keep getting this error while trying to generate ios standalone app. I've tried clearing my credentials several times using exp build:ios -c and I've tried deleting the iOS distribution certificate generated in the previous build attempts from my apple developer account.

Same error here. I revoked all my certificates and this again.
[exp] Error validating credentials. You may need to clear them (with `-c`) and try again.
[exp] Unable to validate credentials. Request ID 250e29d0-8753-11e7-b518-8b3e547ff520, message: Internal error occurred. Please try again in a few minutes or contact support.

@doc-phily as requested in the issue template, please open issues against https://github.com/expo/expo. Also make sure to provide your request/build IDs for us to troubleshoot.
@franckmaurin it looks like this was due to a temporary issue on our build machines, can you try again?

@dikaiosune Yep, it works now. Thanks.


[Bug]: HasOne overrides nullability of BelongsTo

Describe the bug
Hi,
I am trying to integrate CycleORM into a Symfony application, in order to replace Doctrine, and I encountered a strange behavior with relations.
I will show you the example first.
I trimmed all the other tables and managed to reproduce on following 4 entities:
#[Entity(database: 'main')]
class Question
{
    #[Column(type: 'primary')]
    private int $id;

    #[BelongsTo(target: QuestionTextual::class, nullable: true)]
    private ?QuestionTextual $questionTextual = null;

    #[BelongsTo(target: QuestionChoice::class, nullable: true)]
    private ?QuestionChoice $questionChoice = null;

    #[BelongsTo(target: QuestionMultichoice::class, nullable: true)]
    private ?QuestionMultichoice $questionMultichoice = null;
}

#[Entity(database: 'main')]
class QuestionChoice
{
    #[Column(type: 'primary')]
    private readonly int $id;

    #[HasOne(target: Question::class)]
    private Question $question;
}

#[Entity(database: 'main')]
class QuestionTextual
{
    #[Column(type: 'primary')]
    private readonly int $id;

    #[HasOne(target: Question::class)]
    private Question $question;
}

#[Entity(database: 'main')]
class QuestionMultichoice
{
    #[Column(type: 'primary')]
    private readonly int $id;

    #[HasOne(target: Question::class)]
    private Question $question;
 }

Those entities result in following database schema:
create table question_choices
(
    id int auto_increment
        primary key
);

create table question_multichoices
(
    id int auto_increment
        primary key
);

create table question_textuals
(
    id int auto_increment
        primary key
);

create table questions
(
    id                     int auto_increment
        primary key,
    questionChoice_id      int not null,
    questionTextual_id     int null,
    questionMultichoice_id int null,
    constraint questions_foreign_questionchoice_id_66745e0c2f565
        foreign key (questionChoice_id) references question_choices (id)
            on update cascade on delete cascade,
    constraint questions_foreign_questionmultichoice_id_66745e0c2f6fc
        foreign key (questionMultichoice_id) references question_multichoices (id)
            on update cascade on delete cascade,
    constraint questions_foreign_questiontextual_id_66745e0c2f607
        foreign key (questionTextual_id) references question_textuals (id)
            on update cascade on delete cascade
);

create index questions_index_questionchoice_id_66745e0c2f53b
    on questions (questionChoice_id);

create index questions_index_questionmultichoice_id_66745e0c2f6e5
    on questions (questionMultichoice_id);

create index questions_index_questiontextual_id_66745e0c2f5ef
    on questions (questionTextual_id);

The strange thing this issue is about is the nullability of questionChoice_id column. For some reason the column is declared as NOT NULL, while others are nullable - and all the columns have identical attribute declaration.
In this usecase the question has 3 possible types and the tables include some additional settings. In the base table the relation is nullable, because only one of the 3 columns is filled. However in the extending tables the column is not null, because it must have a base table paired to it.
There might be some sort of misunderstanding of the CycleORM relations on my side, but it seems strange to me that one column has different nullability than the others. When the HasOne attribute is removed from the entitiy, the nullability is corrected.
Is this a regression?
I cannot tell as I am a new user of Cycle ORM.
To Reproduce
Use the default setup and sync a database with entitiy declaration above.
Expected behaviour
The column should be nullable.
Media prove
No response
Database
MySQL
Your environment

OS: PopOS
PHP version: 8.3.7
Package version:

annotated: 4.1
database: 2.11
orm: 2.8
schema-builder: 2.8

Additional context
No response

Hello. Good catch. Looks like a bug.

Hi @roxblnfk ,
I tried to dig into the internals but the learning curse seems very steep, as I am not very familiar with the whole system. Would you give me some pointers where to look in order for me to provide a fix? Or is there something else I can do to accelerate the resolution? I am also open to bounty this bug if you had spare time to prioritize things.
Thanks,
VP


[CT-2952] Add tracking on plugin.get_nodes calls

Track the number of models injected, and number of associated packages per plugin.
iglu schema: https://github.com/dbt-labs/iglu.sinter-collect.com/pull/77

Closed by: https://github.com/dbt-labs/dbt-core/pull/8336


Check for regular passwords

There are lists online which contain the most regular passwords.
When entering a new password, one can check that it is contained.

@niccokunzmann The idea would be to check against common passwords to enforce security by disabling those common ones? We already enforce at least (a number OR a special char) and a length of 8.
@butlerx What's your opinion on this?

@Wardormeur Yes, this is the idea. I would argue if there are passwords with a number or a special character among those most popular passwords, the enforcement does not help in those cases.
I found this website. http://www.passwordrandom.com/most-popular-passwords 12345678 is among them There should be a list somewhere on the web.

Found one: https://raw.githubusercontent.com/danielmiessler/SecLists/master/Passwords/10_million_password_list_top_1000000.txt

Interesting to see it comes from the OWASP projejct . :+1:
I'm a bit concerned in term of priority:  as much as I'd want security to be a priority, this one is about prevention of creation of a weak password, considering we already have a unique salt per password.
Any recommendation for the user message to display in case of weak password?
"This password has been recognized to be used as one of the 10 000 most common password, please change it to a more secure one or use a password manager (LastPass, PassPack)" ?

id agree with @niccokunzmann that you'd be unlikely to see any password with special char on that list but youd probably find some mix of numbers and letter password.
i think its a good idea, but we need to be careful of removing possible passwords as doing that means the pool of passwords a attacker would have to test is smaller

I've asked on twitter a couple of people and it seems it's a decent recommendation
ref (https://pages.nist.gov/800-63-3/sp800-63b.html) :
`When processing requests to establish and change memorized secrets, verifiers SHALL compare the prospective secrets against a list that contains values known to be commonly-used, expected, or compromised. For example, the list MAY include (but is not limited to):
Passwords obtained from previous breach corpuses.
Dictionary words.
Repetitive or sequential characters (e.g. ‘aaaaaa’, ‘1234abcd’).
Context specific words, such as the name of the service, the username, and derivatives thereof.`
So definitely something we should do, thanks @niccokunzmann  :+1:


Fixed license block to specify license; made links relative.

Thanks @mbrukman!
/CC @erjohnso
LGTM


Easy RAG: strategies to control whether data should be ingested or not

When using the default (in-memory) embedding store, it is irrelevant, but when running against a persistent store, it is necessary to be able to avoid re-ingesting all data when the application is started multiple times against the same store.
This is relevant not exclusively to Easy RAG, but for Easy RAG it is most important, because the ingestion is executed automatically from extension code, not by any application code, so the user currently has no way to control it.
I propose introducing a property quarkus.langchain4j.easy-rag.ingestion-strategy with the following values:

on: always ingest all documents from the data directory
if-empty (default): ingest if the embedding store is detected to be empty
overwrite: delete everything in the store and re-ingest (this is a little tricky because there is no unified delete operation in dev.langchain4j.store.embedding.EmbeddingStore, but we can provide it at least for some of our implementations)
off: never ingest anything - assume that the embedding store is already populated

@langchain4j, or is there perhaps any common way that we should reuse here - how does one usually detect whether ingestion is necessary or the store is already populated?

@jmartisk there is nothing in langchain4j yet, but you can check out langchain and llamaindex, I am sure they have something.
I was thinking about an option to calculate a checksum of all documents and re-ingest them (or only the difference) if it changes.

I was thinking about an option to calculate a checksum of all documents and re-ingest them (or only the difference) if it changes.

This would be awesome if we could do.

I propose introducing a property quarkus.langchain4j.easy-rag.ingestion-strategy with the following values

This absolutely makes sense in the same way that Quarkus provides a configuration option for whether Hibernate should create the schema.


Improve mono SQLite driver

This should detect the versions of SQLite installed and use the appropriate one.

https://github.com/mono/mono
This repository has the SQLite driver source code, in the path:
mcs/class/Mono.Data.Sqlite

This can be used to find it's previous versions.

The plan is to use a static method to determine (and then cache) the version of mono SQLite that is installed, by trying a number of potential assemblies in priority order (based on previous versions found).
As soon as one is found we remember it as the one we want to use, and cache it in the application in order to avoid scanning for assemblies again.
Then, once we have found an assembly, we can reuse it via it's strong name.


Updating 6to5 to v2.9.0

Not sure what impact this will have... I have no idea what version the minified file was before. Maybe @sebmck can give us pointers on this one as well.

API has remained completely the same and backwards compatible. You can check out the changelog for more info.


Support "rediss://..." URI

Solve the error below
$ redis-dump -u "rediss://example.com" -D
#1565767946.1320: redis_uri: redis://rediss://example.com (0..15)
#1565767946.1366: Process: 1675; Memory: 23844
#1565767946.1367: CONNECT: redis://rediss://example.com/0
#1565767946.1369: CONNECT: redis://rediss://example.com/1
#1565767946.1370: CONNECT: redis://rediss://example.com/2
#1565767946.1371: CONNECT: redis://rediss://example.com/3
#1565767946.1371: CONNECT: redis://rediss://example.com/4
#1565767946.1372: CONNECT: redis://rediss://example.com/5
#1565767946.1372: CONNECT: redis://rediss://example.com/6
#1565767946.1374: CONNECT: redis://rediss://example.com/7
#1565767946.1375: CONNECT: redis://rediss://example.com/8
#1565767946.1375: CONNECT: redis://rediss://example.com/9
#1565767946.1376: CONNECT: redis://rediss://example.com/10
#1565767946.1376: CONNECT: redis://rediss://example.com/11
#1565767946.1377: CONNECT: redis://rediss://example.com/12
#1565767946.1378: CONNECT: redis://rediss://example.com/13
#1565767946.1378: CONNECT: redis://rediss://example.com/14
#1565767946.1379: CONNECT: redis://rediss://example.com/15
#1565767946.1379:  filter
Error connecting to Redis on rediss:6379 (SocketError)}

I do not see where it switches to the TLS/SSL mode... (rediss is for encrypted communication). Also, the URL should be rediss://:@:/ where user is entirely optional, but token is quite important. Same for the redis scheme.

Sorry, but it was an inappropriate explanation.
I'm using a Google translate, so it might be poorly explained.
The argument URL is passed as it is to Redis::Dump.
The URL is also used for Redis.new, and Redis works because it supports rediss://.
This change removed the process of disabling rediss://.
The actual connection test is performed in this way.
$ bin/redis-dump -u "redis://:password@example.com:6379" -D
#1000000000.0000: redis_uri: redis://:password@example.com:6379 (0..15)
#1000000000.0000: Process: 21016; Memory: 20964
#1000000000.0000: CONNECT: redis://:password@example.com:6379/0
#1000000000.0000: CONNECT: redis://:password@example.com:6379/1
#1000000000.0000: CONNECT: redis://:password@example.com:6379/2
#1000000000.0000: CONNECT: redis://:password@example.com:6379/3
#1000000000.0000: CONNECT: redis://:password@example.com:6379/4
#1000000000.0000: CONNECT: redis://:password@example.com:6379/5
#1000000000.0000: CONNECT: redis://:password@example.com:6379/6
#1000000000.0000: CONNECT: redis://:password@example.com:6379/7
#1000000000.0000: CONNECT: redis://:password@example.com:6379/8
#1000000000.0000: CONNECT: redis://:password@example.com:6379/9
#1000000000.0000: CONNECT: redis://:password@example.com:6379/10
#1000000000.0000: CONNECT: redis://:password@example.com:6379/11
#1000000000.0000: CONNECT: redis://:password@example.com:6379/12
#1000000000.0000: CONNECT: redis://:password@example.com:6379/13
#1000000000.0000: CONNECT: redis://:password@example.com:6379/14
#1000000000.0000: CONNECT: redis://:password@example.com:6379/15
#1000000000.0000:  filter
Connection timed out

$ bin/redis-dump -u "rediss://:password@example.com:6379" -D
#1000000000.0000: redis_uri: rediss://:password@example.com:6379 (0..15)
#1000000000.0000: Process: 21042; Memory: 20548
#1000000000.0000: CONNECT: rediss://:password@example.com:6379/0
#1000000000.0000: CONNECT: rediss://:password@example.com:6379/1
#1000000000.0000: CONNECT: rediss://:password@example.com:6379/2
#1000000000.0000: CONNECT: rediss://:password@example.com:6379/3
#1000000000.0000: CONNECT: rediss://:password@example.com:6379/4
#1000000000.0000: CONNECT: rediss://:password@example.com:6379/5
#1000000000.0000: CONNECT: rediss://:password@example.com:6379/6
#1000000000.0000: CONNECT: rediss://:password@example.com:6379/7
#1000000000.0000: CONNECT: rediss://:password@example.com:6379/8
#1000000000.0000: CONNECT: rediss://:password@example.com:6379/9
#1000000000.0000: CONNECT: rediss://:password@example.com:6379/10
#1000000000.0000: CONNECT: rediss://:password@example.com:6379/11
#1000000000.0000: CONNECT: rediss://:password@example.com:6379/12
#1000000000.0000: CONNECT: rediss://:password@example.com:6379/13
#1000000000.0000: CONNECT: rediss://:password@example.com:6379/14
#1000000000.0000: CONNECT: rediss://:password@example.com:6379/15
#1000000000.0000:  filter
#1000000000.0000: Memory after loading keys: 20000kb
#1000000000.0000:  dumping 947 (947) from redis://example.com:6379/0
^C
Exiting...

excuse me.
I was misunderstanding.
This change was completely meaningless and originally corresponded to rediss://.


Update 'type' requirement: line 82

Edit line 82 to stipulate a 'type' value is required as solutioned here:
https://datadog.zendesk.com/agent/tickets/887791
What does this PR do?

Edits line 82 to stipulate a 'type' value is required as solutioned in referenced support ticket
Motivation

https://datadog.zendesk.com/agent/tickets/887791
Additional Notes

This may also mean that our public docs on this integration needs to be updated as well:
https://docs.datadoghq.com/integrations/openmetrics/
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

Hi @TSmith1992 , Openmetrics/Prometheus normally shouldn't require a type because it looks for the type listed in the # TYPE column before the actual metric. Sample Prometheus output:
# TYPE kong_datastore_reachable gauge
kong_datastore_reachable 1

This will let our check recognize that the type of kong_datastore_reachable should be gauge. I'm going to close this PR, but feel free to ask any questions about this here!


invalid CLI Args should fail gracefully

Right now entering invalid arguments causes the tool to throw a panic when there should ideally be some error handling

is a little scary looking. A better message would be something like
Unknown argument "ddfgdg".
<output usage banner>

What version of the tool are you using (theme version). On v0.2.7 I see some 406 errors pop up, but the application still runs correctly.
If config.yml is missing I get a gross error, but it's different than this one.

➜  ~  theme version
Theme Kit v0.2.7

Its actually giving me the 406 errors now too. Using the wrong config.yml gives me issue https://github.com/Shopify/themekit/issues/87
Running the command sans config.yml file gives me a different error (probably the one you got too)
➜  Simple git:(master) ✗ ls
assets         config         config.yml_gem config.yml_kit layout         locales        snippets       templates
➜  Simple git:(master) ✗ theme replace kdhfkdfdf
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x10 pc=0xe1e72]

goroutine 10 [running]:
net/http.(*Client).doFollowingRedirects(0x0, 0xc208034c30, 0x480c10, 0x0, 0x0, 0x0)
	/usr/local/Cellar/go/1.4.2/libexec/src/net/http/client.go:304 +0x62
net/http.(*Client).Do(0x0, 0xc208034c30, 0x0, 0x0, 0x0)
	/usr/local/Cellar/go/1.4.2/libexec/src/net/http/client.go:174 +0xa4
github.com/Shopify/themekit.ThemeClient.query(0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
	/Users/csaunders/development/golang/src/github.com/Shopify/themekit/theme_client.go:230 +0x28b
github.com/Shopify/themekit.func·020()
	/Users/csaunders/development/golang/src/github.com/Shopify/themekit/theme_client.go:95 +0x5e
created by github.com/Shopify/themekit.ThemeClient.AssetList
	/Users/csaunders/development/golang/src/github.com/Shopify/themekit/theme_client.go:112 +0x18b

goroutine 1 [chan receive]:
main.main()
	/Users/csaunders/development/golang/src/github.com/Shopify/themekit/cmd/theme/main.go:127 +0x3aa

goroutine 5 [runnable]:
github.com/Shopify/themekit.func·002()
	/Users/csaunders/development/golang/src/github.com/Shopify/themekit/error_reporter.go:50 +0x52
created by github.com/Shopify/themekit.SetErrorReporter
	/Users/csaunders/development/golang/src/github.com/Shopify/themekit/error_reporter.go:56 +0x69

goroutine 7 [chan send]:
github.com/Shopify/themekit.func·003()
	/Users/csaunders/development/golang/src/github.com/Shopify/themekit/error_reporter.go:62 +0x55
created by github.com/Shopify/themekit.func·004
	/Users/csaunders/development/golang/src/github.com/Shopify/themekit/error_reporter.go:63 +0x73

goroutine 8 [chan receive]:
github.com/Shopify/themekit.func·025()
	/Users/csaunders/development/golang/src/github.com/Shopify/themekit/theme_client.go:189 +0x9a
created by github.com/Shopify/themekit.ThemeClient.Process
	/Users/csaunders/development/golang/src/github.com/Shopify/themekit/theme_client.go:198 +0x1d8

goroutine 9 [chan receive]:
github.com/Shopify/themekit/commands.func·002()
	/Users/csaunders/development/golang/src/github.com/Shopify/themekit/commands/common.go:142 +0xbf
created by github.com/Shopify/themekit/commands.mergeEvents
	/Users/csaunders/development/golang/src/github.com/Shopify/themekit/commands/common.go:148 +0xd8

goroutine 11 [runnable]:
github.com/Shopify/themekit/commands.drainErrors(0xc20803e4e0)
	/Users/csaunders/development/golang/src/github.com/Shopify/themekit/commands/common.go:127
created by github.com/Shopify/themekit/commands.Replace
	/Users/csaunders/development/golang/src/github.com/Shopify/themekit/commands/replace.go:28 +0x20c

goroutine 12 [runnable]:
github.com/Shopify/themekit/commands.removeAndUpload(0xc20803e480, 0xc20803e360)
	/Users/csaunders/development/golang/src/github.com/Shopify/themekit/commands/replace.go:53
created by github.com/Shopify/themekit/commands.Replace
	/Users/csaunders/development/golang/src/github.com/Shopify/themekit/commands/replace.go:29 +0x236

goroutine 13 [runnable]:
main.func·001()
	/Users/csaunders/development/golang/src/github.com/Shopify/themekit/cmd/theme/main.go:110
created by main.main
	/Users/csaunders/development/golang/src/github.com/Shopify/themekit/cmd/theme/main.go:126 +0x382

I'm not longer able to replicate the original issue, but a missing config file should fail gracefully too. I wish i knew what happened to cause that slice error originally though.

Thanks!

When running theme without arguments, it throws an error on v0.3
➜  Simple git:(master) ✗ theme
panic: runtime error: index out of range

goroutine 1 [running]:
main.SetupAndParseArgs(0xc20800a000, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0)
	/Users/csaunders/development/golang/src/github.com/Shopify/themekit/cmd/theme/main.go:295 +0x11d
main.main()
	/Users/csaunders/development/golang/src/github.com/Shopify/themekit/cmd/theme/main.go:120 +0xba

goroutine 2 [runnable]:
runtime.forcegchelper()
	/usr/local/Cellar/go/1.4.2/libexec/src/runtime/proc.go:90
runtime.goexit()
	/usr/local/Cellar/go/1.4.2/libexec/src/runtime/asm_amd64.s:2232 +0x1

goroutine 3 [runnable]:
runtime.bgsweep()
	/usr/local/Cellar/go/1.4.2/libexec/src/runtime/mgc0.go:82
runtime.goexit()
	/usr/local/Cellar/go/1.4.2/libexec/src/runtime/asm_amd64.s:2232 +0x1

goroutine 4 [runnable]:
runtime.runfinq()
	/usr/local/Cellar/go/1.4.2/libexec/src/runtime/malloc.go:712
runtime.goexit()
	/usr/local/Cellar/go/1.4.2/libexec/src/runtime/asm_amd64.s:2232 +0x1

goroutine 5 [runnable]:
github.com/Shopify/themekit.func·002()
	/Users/csaunders/development/golang/src/github.com/Shopify/themekit/error_reporter.go:48
created by github.com/Shopify/themekit.SetErrorReporter
	/Users/csaunders/development/golang/src/github.com/Shopify/themekit/error_reporter.go:56 +0x69

Invalid arguments work great though


The diff coverage is 77.33%.

@@            Coverage Diff             @@
##           master     #723      +/-   ##
==========================================
+ Coverage   88.05%   88.09%   +0.03%
==========================================
  Files         100      100
  Lines        9896     9920      +24
  Branches     2009     2014       +5
==========================================
+ Hits         8714     8739      +25
+ Misses        939      938       -1
  Partials      243      243

Impacted Files
Coverage Δ

devito/ir/iet/analysis.py
93.38% <ø> (ø)
:arrow_up:

devito/operator.py
88.65% <ø> (ø)
:arrow_up:

devito/mpi/routines.py
100% <100%> (ø)
:arrow_up:

devito/dimension.py
92.18% <100%> (ø)
:arrow_up:

devito/ir/support/basic.py
91.97% <65.78%> (+1.34%)
:arrow_up:

devito/ir/support/space.py
87.03% <72.72%> (-0.61%)
:arrow_down:

devito/mpi/halo_scheme.py
94.53% <95%> (-0.39%)
:arrow_down:

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 98909dd...b5d9edc. Read the comment docs.

Merged.