ci: build prqlc on PRs

Addresses #1910

It seems working.

After a struggle I gave up on linux arm64. Could it be a psm problem?
$ cargo build -p prqlc --release --target=aarch64-unknown-linux-musl
   Compiling cfg-if v1.0.0
   Compiling once_cell v1.17.1
   Compiling bitflags v1.3.2
   Compiling linux-raw-sys v0.1.4
   Compiling libc v0.2.140
   Compiling lazy_static v1.4.0
   Compiling memchr v2.5.0
   Compiling concolor-override v1.0.0
   Compiling concolor-query v0.3.3
   Compiling utf8parse v0.2.1
   Compiling adler v1.0.2
   Compiling gimli v0.27.2
   Compiling anstyle v0.3.5
   Compiling serde v1.0.157
   Compiling psm v0.1.21
   Compiling pin-project-lite v0.2.9
error: failed to run custom build command for `psm v0.1.21`

Caused by:
  process didn't exit successfully: `/workspaces/prql/target/release/build/psm-bd89755e99a601de/build-script-build` (exit status: 1)
  --- stdout
  OPT_LEVEL = Some("3")
  TARGET = Some("aarch64-unknown-linux-musl")
  HOST = Some("x86_64-unknown-linux-gnu")
  cargo:rerun-if-env-changed=CC_aarch64-unknown-linux-musl
  CC_aarch64-unknown-linux-musl = None
  cargo:rerun-if-env-changed=CC_aarch64_unknown_linux_musl
  CC_aarch64_unknown_linux_musl = None
  cargo:rerun-if-env-changed=TARGET_CC
  TARGET_CC = None
  cargo:rerun-if-env-changed=CC
  CC = None
  RUSTC_LINKER = None
  cargo:rerun-if-env-changed=CROSS_COMPILE
  CROSS_COMPILE = None
  cargo:rerun-if-env-changed=CFLAGS_aarch64-unknown-linux-musl
  CFLAGS_aarch64-unknown-linux-musl = None
  cargo:rerun-if-env-changed=CFLAGS_aarch64_unknown_linux_musl
  CFLAGS_aarch64_unknown_linux_musl = None
  cargo:rerun-if-env-changed=TARGET_CFLAGS
  TARGET_CFLAGS = None
  cargo:rerun-if-env-changed=CFLAGS
  CFLAGS = None
  cargo:rerun-if-env-changed=CRATE_CC_NO_DEFAULTS
  CRATE_CC_NO_DEFAULTS = None
  DEBUG = Some("false")
  CARGO_CFG_TARGET_FEATURE = Some("neon")
  cargo:rustc-cfg=asm
  cargo:rustc-cfg=switchable_stack
  running: "aarch64-linux-musl-gcc" "-O3" "-ffunction-sections" "-fdata-sections" "-fPIC" "-Wall" "-Wextra" "-xassembler-with-cpp" "-DCFG_TARGET_OS_linux" "-DCFG_TARGET_ARCH_aarch64" "-DCFG_TARGET_ENV_musl" "-o" "/workspaces/prql/target/aarch64-unknown-linux-musl/release/build/psm-9cc78c5eb0f1a15f/out/src/arch/aarch_aapcs64.o" "-c" "src/arch/aarch_aapcs64.s"

  --- stderr

  error occurred: Failed to find tool. Is `aarch64-linux-musl-gcc` installed?

Thank you for reviewing this.
I considered cross, but did not try it at this time due to the need for a completely different workflow to use Docker and concerns about slowdowns due to QEMU emulation.
Attaching these to the release would be easy as it is just one (?) additional step.

Thanks to the advice I received in rust-lang/stacker#80, I am now able to build for Linux arm64.
(I have confirmed that the prqlc binary works on my Raspberry Pi 4)
Once this PR is merged I will work on attaching these to the releases.


[proofing] Fully clean up search/replace UX

These changes build on @kvchitrapu's prior work and simplify it slightly.
Specifically, this makes the following changes:

collapse the UX from 4 phases (search, check, confirm, submit) to 3 (search, check, submit) -- I felt that the "confirm" stage was not adding much as compared to the complexity it added.
create a single _find_replacements utility function that does the regex searching, etc. By using dataclasses, we can reuse this one function for the full flow.
add unit tests to increase test coverage

Proposed flow:

Sorry @akprasad . I meant get back to regular routine last week, but could not squeeze time to unblock.

@akprasad , go forward with this PR and land it. Consider adding an assert to catch lines that escape odd/even structure. Record user's regex in the log message.


CI for Windows Subsystem for Linux

Following #2313 and given that @cfhammill has been working on getting us fully working on Windows 10 -- via WSL -- we should add WSL to Travis.

The work is being done by @lrouhier, I just noticed some numerical instability. At any rate, windows is in beta at the moment https://docs.travis-ci.com/user/reference/windows/, the installed package lists WSL, but I have no idea what that means. The default shell is git-bash, with the chocolatey package manager providing a very minimal package set. Not sure how to open a WSL subshell, and what distro(s) are available via WSL. I'll investigate a bit more

This link gives you the official command-line installation for the various Linux distros available in WSL. I gave the users instructions for installing Ubuntu 18.04 LTS.  I don't know if travis will open a PowerShell or directly the ubuntu 'VM'...


AppliedSuccefully doesn't set condition properly

 - lastTransitionTime: "2022-10-26T10:35:55Z"
    message: 'Applied successfully: awesome-repo/f368f993dc74ca0f5a6588e24b356f518c8b7ce8'
    reason: TerraformAppliedSucceed
    status: Unknown
    type: Ready

status should be True

https://github.com/weaveworks/tf-controller/blob/main/api/v1alpha1/terraform_types.go#L550
explicitly sets unknown

Hi @k0da
Thank you for that.
We're working on fixing this and other related issues in a different PR.

Fixed by #438 and #440 (v0.13.0)
Please feel free to re-open it if you still see the error.


swagger-ui.html show nothing

Hey springfoxer:
I can visit url '/v2/api-docs' and 'swagger-ui.html'. The json document seems ok. But in swagger-ui.html, I cannot find any api here.
So I check the browser and find some error in swagger-ui-min.js:
Uncaught TypeError: Cannot read property 'toLowerCase' of undefined(anonymous function) @ swagger-ui.min.js:13Handlebars.templates.param_list.Handlebars.template.main @ swagger-ui.min.js:1e @ handlebars-2.0.0.js:27SwaggerUi.Views.ParameterView.Backbone.View.extend.render @ swagger-ui.min.js:13SwaggerUi.Views.OperationView.Backbone.View.extend.addParameter @ swagger-ui.min.js:13SwaggerUi.Views.OperationView.Backbone.View.extend.render @ swagger-ui.min.js:12SwaggerUi.Views.ResourceView.Backbone.View.extend.addOperation @ swagger-ui.min.js:13SwaggerUi.Views.ResourceView.Backbone.View.extend.render @ swagger-ui.min.js:13SwaggerUi.Views.MainView.Backbone.View.extend.addResource @ swagger-ui.min.js:12SwaggerUi.Views.MainView.Backbone.View.extend.render @ swagger-ui.min.js:12window.SwaggerUi.Backbone.Router.extend.render
Could you help to check this issue? Thanks.

Thats not much information to figure out the problem. See guidelines for posting issues. Plus thats an issue with swagger-ui (unless of course springfox is generating a bad service description)

I think I may miss some configuration, because the console shows error 'validatorUrl' is not found.
2015-12-21 10:26:58,312 [http-bio-8080-exec-8] WARN  [org.springframework.web.servlet.PageNotFound] - No mapping found for HTTP request with URI [/security/validatorUrl] in DispatcherServlet with name 'springmvc'
I already configure the component scan '<context:component-scan base-package="springfox.documentation"/>' in spring context file. So where should I do more configure?

@dilipkrish Here is the problem
the interface ‘/v2/api-docs’ is ok.

the interface '/swagger-ui.html' is available, But it seems there is no api here?

And I also saw there is a js error here:

So is there any steps I missed?
Thanks

@dilipkrish
ok, I know why this error happens.
I have one method using 'httpservletResponse' as a parameter. So this could impact the js. But I think this situation should commonly be used in 'download' function.
So any adverse on how to solve this issue? I also try to add @ApiIgnore on 'HttpServletResponse', but it seems un-work.
waiting you response.
Thanks.

Instead of using @ApiIgnore you need to use
//In your docket configuration add...
docket.ignorableTypes(HttpServletResponse.class)


[Translator] Option to throw an exception instead of returning the message as is when the message is not found.

Q
A

Bug fix?
no

New feature?
yes

Issues
Fix #2183

License
MIT

Now I consider my PR ready @Kocal. Let me know what you think.

Thanks for your work on this new feature!


[Bug][devLake:UI] Error: Request failed with status code 502 , while login to devLake

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
It was configured last week and it has been working fine till yesterday, when i try to log in, i am getting a 502 error since morning.I did not modify anything.
What do you expect to happen
I don't know why suddenly getting this error
How to reproduce
when i try access datalake from browser i.e IP.address:4000 i am getting below ereor

Even i click on continue , it is not moving.
Anything else
No response
Version
V1.0.0
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@ravisanrjy check the backend log and see what happened

When i run the docker-compose up -d, i am getting below output
WARN[0000] The "HTTPS_PROXY" variable is not set. Defaulting to a blank string.
WARN[0000] The "GOPROXY" variable is not set. Defaulting to a blank string.
[+] Running 7/8
⠦ Network incubator-devlake_default                                    Created                                                                                2.6s
✔ Container incubator-devlake-authproxy-1                              Started                                                                                0.4s
✔ Container incubator-devlake-mysql-1                                  Started                                                                                0.7s
✔ Container incubator-devlake-postgres-1                               Started                                                                                0.7s
! authproxy Published ports are discarded when using host network mode                                                                                0.0s
✔ Container incubator-devlake-devlake-1                                Started                                                                                1.4s
✔ Container incubator-devlake-grafana-1                                Started                                                                                1.3s
✔ Container incubator-devlake-config-ui-1                              Started

Can you share the backend logs by using the command docker-compose logs devlake.

@klesh -Please find the attached logs. Till last week it was working fine, suddently i start getting 502 error since yesterday. I would appreciate your quick response
devLake_log.txt
Thankd
Ravi

Seems like the mysql service is down.
time="2024-08-06 06:55:24" level=info msg="\x1b[35m/app/core/runner/db.go:154\n\x1b[0m\x1b[31m[error] \x1b[0mfailed to initialize database, got error dial tcp <IP_ADDRESS>:3306: connect: connection refused"

Hi Klesh,
I have checked the docker logs and found below details
time="2024-08-07T07:51:03Z" level=info msg="no [.env] file, please make sure you have set the environment variable."
Version:
time="2024-08-07 07:51:03" level=info msg="\x1b[35m/app/core/runner/db.go:154\n\x1b[0m\x1b[31m[error] \x1b[0mfailed to initialize database, got error Error 1049 (42000): Unknown database 'lake'"
How set the version in .env . PFA .env file
env.txt

Hi Klesh, I have checked the docker logs and found below details time="2024-08-07T07:51:03Z" level=info msg="no [.env] file, please make sure you have set the environment variable." Version: time="2024-08-07 07:51:03" level=info msg="\x1b[35m/app/core/runner/db.go:154\n\x1b[0m\x1b[31m[error] \x1b[0mfailed to initialize database, got error Error 1049 (42000): Unknown database 'lake'"

Are you using the mysql service in the docker-compose.yml file? If you do, the lake database should be created automatically.
Or is it a standalone MySQL? if so, create the database and user/pass then update the DB_URL accordingly.

How set the version in .env . PFA .env file env.txt

What do you mean by version? what version are you referring to?

@klesh -> I fixed the issue, i removed all docker images and volumes and created again and it's working now. Thanks

Nice. Closing issue now.


Incorporate Identity SDK 2.x extensions API UX study feedback

Action items from 8/4 meeting:

[x] Replace the term "extension" with something else. Maybe "feature"?
[x] Update token cache persistence code snippets to use the enabled property instead of name.
[x] Update the VS Code credential constructor to mention that the extension package is required.
[ ] Add the package-specific npm install command, useIdentityExtension function call snippet, and an aka.ms link (to the README) to the exception message.
[x] Replace the persistence variable reference with cachePersistenceExtension in the useIdentityExtension function's JS docs.

[x] tokenCachePersistenceOptions API ref page changes:

[x] Remove the stray bullet at the bottom of the page.

[x] Update the name property's "Windows 8" text to "Windows 8+".

[ ] README file changes:

[x] Move the useIdentityExtension function call snippet from the "Key concepts" section to the "Getting started" section.
[x] Swap the order of the "Install the package" and "Prerequisites" sections.
[ ] Add a link from the change log to a breaking changes guide (this is what Scott Schaab mentioned the .NET SDK does).
[x] Update the VS Code credential package’s npm install command to use --save-dev instead of --save.

I’m going to break “Add a link from the change log to a breaking changes guide” Into a separate issue and mark it as a GA blocker for Identity 2.x.


missing data

in paper
[
Missing values
No transformation is done for missing values present in the data.
We let the model learn the distribution of the missing values.
This strategy gives us the ﬂexibility to let the model impute or generate missing values during the sampling process
]
but error occur by missing data
how do i have to?

Hello @limhasic , could you please share more detail about the error you are getting?
The model should be able to handle NaN values in the raw dataset and you will also have the option to impute or generate NaN values in the synthetic data as well.
To impute, you just need to pass the token id of the NUMERIC_NA_TOKEN to the sample() method.
from realtabformer.data_utils import NUMERIC_NA_TOKEN

model = <REaLTabFormer Model>
model.fit(...)

data = model.sample(..., suppress_tokens=[model.vocab["decoder"]["token2id"][NUMERIC_NA_TOKEN]])

I also know that this model needs to learn Nan values as well.
Your also option contradicts the model.
And from the "model.fit" stage, an error occurs due to the presence of Nan values.


don't reset expense form on load

this fixes submitExpense issue

why?

form was being rest after being loaded from Link in react-router.

Why was that reset there in the first place?
When you click on "Submit a new expense", does it still reset it properly?

yes. not sure why it was there. this code is over 6 months old.


Clarify error message when modifying read-only resources

User driven: https://discord.com/channels/905144809105260605/1131288083078328471
User was trying to submit a PUT and PATCH request to a resource with a "read-only" access policy and received a 403: Forbidden error. Our docs didn't not explicitly define what the expected result would be.

coverage: 94.367%. remained the same when pulling 0f66808841a123e730afa9a3e5de3daa3d2645a1 on rahul-documentation-clarify-read-only-error-message into db1906131df88fdbd1dca01a67841b73a6851457 on main.


Compare iAA1259 with iKS1317

After iAA1259 being published we should include the information from this model. From reading the paper my impression is that main improvements are:

Updated biomass reaction
Massive increase in annotation of genes, reactions, metabolites
New / curated pathways:

Polysaccharide degradation
biosynthesis of yCPK
biosynthesis of gamma-butyrolactones
futalosine pathway
curation of phosphorylative oxidation

Tasks:

[ ] Compare reactions in iAA1259 with iKS1317 (and SCO4)
[ ] Compare the content of the phosphorylative oxidation pathway, and include these corrections
[ ] Add reactions in iAA1259 to iKS1317 (in correct namespace)
[ ] Use the updated Biomass function?
[ ] Adopt annotations from iAA1259

All details of updates in model
12864_2018_4905_MOESM1_ESM.pdf


Super minor change for format consistency

I removed a space... Call me nitpicky if you want...

I like the space.

Lol.


Suggest a potential alternative/fix
We should either reduce the data test size or even exclude them from the non numba tests. The time limit is  30 mins, we will get to the point the overnight tests take more than the night to run!
=========================== short test summary info ============================
FAILED aeon/distances/tests/test_sklearn_compatibility.py::test_multivariate[task0-3-dist14] - Failed: Timeout >1800.0s
FAILED aeon/regression/distance_based/tests/test_time_series_neighbors.py::test_knn_neighbors[twe] - Failed: Timeout >1800.0s
Additional context
No response

closing,duplicate of #1991


Possibly incorrect WARN on monospaced build width consistency - some marks have zero advance width definitions

Glyphs source file -> static instance TTF font build with fontmake v2.2.1 pipeline.  This is a weight axis monospaced typeface.
Observed behaviour
fontbakery raises a WARN level report that the combining marks *comb glyphs in our glyph set have inconsistent widths in a monospaced typeface.
The report is:
* WARN: Font is monospaced but 9 glyphs (3.91304347826087%) have a different width. You should check the widths of: ['uni0308', 'gravecomb', 'acutecomb', 'uni0302', 'uni030C', 'uni030A', 'tildecomb', 'uni0304', 'uni0327'] [code: mono-outliers]

and this appears to come from this block of the fontbakery source:
https://github.com/googlefonts/fontbakery/blob/42e872c92947569f5972a76bb52175e95666ed08/Lib/fontbakery/profiles/shared_conditions.py#L152-L173
I reviewed the Glyphs source files and the advance width for all of the reported glyphs is set at the expected value for the typeface (600).
Here is what I see in the binaries:
>>> for glyphname, metrics in glyph_metrics.items():
...     if metrics[0] != 600:
...         print(f"{glyphname}: {metrics}")
...
uni0308: (0, 140)
gravecomb: (0, 213)
acutecomb: (0, 213)
uni0302: (0, 146)
uni030C: (0, 146)
uni030A: (0, 190)
tildecomb: (0, 126)
uni0304: (0, 161)
uni0327: (0, 200)

And a visual of dieresiscomb:

It looks like a subset of the marks (maybe all combining marks?) are compiled with a zero advance width value and should be excluded from this check.
Expected behaviour
This is not reported as a warning for monospaced typefaces if these metrics are valid.

@chrissimpkins please let me know if this fix is good for you. Otherwise, consider reopening the issue.

earlier this week @chrissimpkins commented on the merged PR at https://github.com/googlefonts/fontbakery/pull/3062#discussion_r505990549:

I'm not sure that this influences the issue report if I understand the change correctly Felipe. This looks like it is a "is it a monospaced font" side definition change. The fonts that I tested were designed as monospaced fonts. We successfully passed that test and the result was correct. The issue that I found was that when it is a monospaced font, there are glyphs outside of the ASCII set that are detected as inappropriately defined in a monospaced font b/c they have zero advance widths. These were in the combining mark glyph set which are compiled with zero advance widths. Please let me know if I can provide any additional information.

I've been re-evaluating this today and I think my changes in that PR are actually good. But I also think that the problem observed by Cris may be a corner-case occurrence. My theory is that perhaps the most common advance for glyphs on Chris' font may be zero, which would cause that bad log message. To address that, I think I may have to make one additional change so that only non-zero advance widths are considered when calculating the most common advance. In other words, we are actually interested in the most common non-zero advance value.
In order to confirm if that's the actual case we're dealing with here, I'd need access to a sample of your font files. Can you please send me some of those files, @chrissimpkins?

It would also be good to know if @chrissimpkins fontbakery includes the PR from #3036 — somewhat related issue possibly, no?
Combing marks should be 0 width also in monospace fonts afaik.

I've been re-evaluating this today and I think my changes in that PR are actually good.

No criticism about the changes.  I think that they do not address my issue.

Combing marks should be 0 width also in monospace fonts afaik.

This is the root of the issue.  FB appropriately detects our fonts as "monospaced" based on the FB assessment of the adv width of the ASCII set + the included FB glyph filters.  FB is then in "monospaced" check mode and flags combining marks as not appropriate for a monospaced build because they do not share the adv width metrics of the ASCII set calculation.  Combining marks are compiled with a zero advance width definition by fontmake so they will not have the adv width calculated by the ASCII set routine.
I think that the pseudo code to address it is ~
if MONOSPACED:
   for GLYPH in GLYPHSET:
       if GLYPH.advWidth != MONOSPACED_ADVWIDTH and GLYPH not in FILTER_LIST:
           FONTBAKERY_WARN/FAIL

I believe that the combining marks simply need to be added to the filter so that they are either not checked or are confirmed to have a zero adv width that can differ from the global adv width definition for the font.

Can you please send me some of those files, @chrissimpkins?

Done!  Thanks Felipe!

I've just verified that indeed commit 42e872c92947569f5972a76bb52175e95666ed08 (as suggested by @kontur) fixed the problem last month and was included in the last two stable releases, 0.7.30 and 0.7.31
@chrissimpkins please update your fontbakery installation to benefit from the fix.


Allow reading files

Looking at ZORK as my example program it actually uses both:

15 / F_OPEN
33 / F_RANDOMREAD

So I guess this will be annoying.  Opening seems simple enough.  But random reading seems annoying given the flags for offsets and a lack of good documentation in my simple reference:

https://www.seasip.info/Cpm/bdos.html

Can I just be smug for a moment?
 $ go build . &&  ./cpmulator ZORK1.COM
ZORK I: The Great Underground Empire
Copyright (c) 1981, 1982, 1983 Infocom, Inc. All rights
reserved.
ZORK is a registered trademark of Infocom, Inc.
Revision 88 / Serial number 840726

West of House
You are standing in an open field west of a white house, with
a boarded front door.
There is a small mailbox here.

>help
I don't know the word "help".

>foo
I don't know the word "foo".

>quit
Your score is 0 (total of 350 points), in 0 moves.
This gives you the rank of Beginner.
Do you wish to leave the game? (Y is affirmative): >y


documentation: add notes on debugging IE

@FreHu

I'm gonna turn it into a blog post, when it's published I will link it in the documentation.

This can be closed


Added 'seek to position in currently playin track'

Adding seek.
I can attempt to write some tests for it as well, but I don't have much experience with it!

I'm closing this PR. I have decided to create a PR combining the contributions from several pending PRs that tried to include the functionality but were either missing functionality (eg optional parameters) or tests.
Thank you very much for your contribution and we¡ll try to provide feedback and act on this earlier next time.


Remove Xdebug in the PHP master definition as it does not support PHP7

See the discussion in https://github.com/travis-ci/travis-ci/issues/2480#issuecomment-74901214

:+1:

@rogeriopradoj @loicfrering any change needed here ?

@stof, I think it's ok for merge. The link you provided cites Travis. @loicfrering , any thoughts about it?

Merged, thanks, @stof!


Upgrade to PHP 8.1

Waiting for https://github.com/filamentphp/filament/pull/6453 to be merged first.

Hmm I'm not too sure about wrapping, @pxlrbt.
It's already readable to me.
Will let @danharrin decide.


Made IBAnimatable a shared scheme

That makes the framework Carthage compatible (#6).
Also, if you merge this, you'll probably want to tag the commit to a version number.

Thank you very much.

Thanks again, I have tagged it as V1.0.1 and will update readme to support Carthage.

@delannoyk Sorry to bother you, I am having some issues with Carthage support. I created an example project and added a Cartfile at https://github.com/JakeLin/IBAnimatable-PackageManagementExamples/blob/master/IBAnimatable-Carthage/Cartfile
When I ran carthage update, then I got some errors like
*** Fetching IBAnimatable
*** Checking out IBAnimatable at "V1.0.1"
*** xcodebuild output can be found in /var/folders/r9/2nsd0w6d2mn87jkg75njd3md14_q5m/T/carthage-xcodebuild.lQfGCo.log
*** Building scheme "IBAnimatable" in IBAnimatableApp.xcodeproj
/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator9.2.sdk/usr/include/sys/cdefs.h:707:2: error: Unsupported architecture
...
...
...

The following build commands failed:
	CompileSwift normal arm64 /Users/jake.lin/dev/Jake/IBAnimatable-PackageManagementExamples/IBAnimatable-Carthage/Carthage/Checkouts/IBAnimatable/IBAnimatable/AnimatableButton.swift
	CompileSwift normal arm64 /Users/jake.lin/dev/Jake/IBAnimatable-PackageManagementExamples/IBAnimatable-Carthage/Carthage/Checkouts/IBAnimatable/IBAnimatable/RotationDesignable.swift
	CompileSwift normal arm64 /Users/jake.lin/dev/Jake/IBAnimatable-PackageManagementExamples/IBAnimatable-Carthage/Carthage/Checkouts/IBAnimatable/IBAnimatable/AnimatableLabel.swift
	CompileSwift normal arm64 /Users/jake.lin/dev/Jake/IBAnimatable-PackageManagementExamples/IBAnimatable-Carthage/Carthage/Checkouts/IBAnimatable/IBAnimatable/AnimatableStackView.swift
	CompileSwift normal arm64 /Users/jake.lin/dev/Jake/IBAnimatable-PackageManagementExamples/IBAnimatable-Carthage/Carthage/Checkouts/IBAnimatable/IBAnimatable/MaskType.swift
	CompileSwift normal arm64 /Users/jake.lin/dev/Jake/IBAnimatable-PackageManagementExamples/IBAnimatable-Carthage/Carthage/Checkouts/IBAnimatable/IBAnimatable/AnimatableTableView.swift
	CompileSwift normal arm64 /Users/jake.lin/dev/Jake/IBAnimatable-PackageManagementExamples/IBAnimatable-Carthage/Carthage/Checkouts/IBAnimatable/IBAnimatable/Animatable.swift
	CompileSwift normal arm64 /Users/jake.lin/dev/Jake/IBAnimatable-PackageManagementExamples/IBAnimatable-Carthage/Carthage/Checkouts/IBAnimatable/IBAnimatable/AnimatableTextField.swift
	CompileSwiftSources normal arm64 com.apple.xcode.tools.swift.compiler
(9 failures)

I followed the doc from https://github.com/Carthage/Carthage#resolve-build-failures
I cd to /Users/jake.lin/dev/jake/IBAnimatable-PackageManagementExamples/IBAnimatable-Carthage/Carthage/Checkouts/IBAnimatable and executed xcodebuild -scheme IBAnimatable -project IBAnimatableApp.xcodeproj -configuration release build and succeeded to build the the framework target like below:
=== BUILD TARGET IBAnimatable OF PROJECT IBAnimatableApp WITH THE DEFAULT CONFIGURATION (Release) ===

Check dependencies

** BUILD SUCCEEDED **

That's my first time to support Carthage, can you give me some hint when you have time?
Thanks
Jake

I find something interesting. Once I add -sdk iphonesimulator to xcodebuild like
xcodebuild -scheme IBAnimatable -project IBAnimatableApp.xcodeproj -sdk iphonesimulator -configuration release build
Then I got this:
/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator9.2.sdk/usr/include/sys/cdefs.h:707:2: error: Unsupported architecture #error Unsupported architecture
Something wrong with -sdk iphonesimulator

This is interesting, that's a problem I didn't have using my fork.
Anyway, I tried with a Cartfile containing:
github "JakeLin/IBAnimatable"
And I don't have the same error you have:
*** Building scheme "IBAnimatable" in IBAnimatableApp.xcodeproj
xcodebuild: error: Unable to find a destination matching the provided destination specifier:
		{ platform:iOS Simulator, id:A13AE2E7-4EB7-4165-9DFC-332BB80C8E24 }
A shell task failed with exit code 70:
xcodebuild: error: Unable to find a destination matching the provided destination specifier:
		{ platform:iOS Simulator, id:A13AE2E7-4EB7-4165-9DFC-332BB80C8E24 }

I'm at work right now but I'll definitely take a deeper look at it when I'll come home later today!

I'm not having any issues on my personal laptop. Weird.
~/D/t ❯❯❯ carthage update                                                     ⏎
*** Fetching IBAnimatable
*** Checking out IBAnimatable at "V1.0.1"
*** xcodebuild output can be found in /var/folders/ws/xc68hw2j7kb5l6hqtmz0kh7m0000gn/T/carthage-xcodebuild.BDvtiE.log
*** Building scheme "IBAnimatable" in IBAnimatableApp.xcodeproj
~/D/t ❯❯❯

And the framework is built successfully.

@JakeLin  Which version of carthage do you have ? you should have the 0.11.
I just tried to built your framework from carthage and it works.
Be sure to have the latest version.

@delannoyk  @lastMove Thanks guys, I installed Carthage using Carthage.pkg long time ago, and used brew install carthage to install again. But didn't notice I had old version of Carthage. I have fixed this issue by uninstalling the older version.


services/horizon: Add missing tx result codes

PR Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[ ] This PR's title starts with name of package that is most changed in the PR, ex.
services/friendbot, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.
[ ] I've updated any docs (developer docs, .md
files, etc... affected by this change). Take a look in the docs folder for a given service,
like this one.

Release planning

[ ] I've updated the relevant CHANGELOG (here for Horizon) if
needed with deprecations, added features, breaking changes, and DB schema changes.
[ ] I've decided if this PR requires a new major/minor version according to
semver, or if it's mainly a patch change. The PR is targeted at the next
release branch if it's not a patch change.

Added a new Protocol 18 and missing codes from previous protocol upgrade. Added a test to ensure return codes of all specific operation types are implemented.
Close #3858.

@stellar/horizon-committers bump. I rely on this in integration tests. Can we merge?


fileListingWorker restarting

Tried out alm in the cloud shell on gcloud. It complains repeatetly about FSWatcher. I guess that
the cloud shell doesn't support that. But that should not cause alm to fail.
Error: watch /home/erik_martino/angular-start/node_modules/sntp/package.json ENOSPC
    at exports._errnoException (util.js:1022:11)
    at FSWatcher.start (fs.js:1429:19)
    at Object.fs.watch (fs.js:1456:11)
    at createFsWatchInstance (/usr/local/nvm/versions/node/v6.9.2/lib/node_modules/alm/node_mo
dules/chokidar/lib/nodefs-handler.js:37:15)
    at setFsWatchListener (/usr/local/nvm/versions/node/v6.9.2/lib/node_modules/alm/node_modul
es/chokidar/lib/nodefs-handler.js:80:15)
    at FSWatcher.NodeFsHandler._watchWithNodeFs (/usr/local/nvm/versions/node/v6.9.2/lib/node_
modules/alm/node_modules/chokidar/lib/nodefs-handler.js:228:14)
    at FSWatcher.NodeFsHandler._handleFile (/usr/local/nvm/versions/node/v6.9.2/lib/node_modul
es/alm/node_modules/chokidar/lib/nodefs-handler.js:255:21)
    at FSWatcher.<anonymous> (/usr/local/nvm/versions/node/v6.9.2/lib/node_modules/alm/node_mo
dules/chokidar/lib/nodefs-handler.js:473:21)
    at FSReqWrap.oncomplete (fs.js:123:15)
fileListingWorker worker restarting. Don't know why it stopped with code: 1
projectServiceWorker worker restarting.


Document viewer not presenting in iOS

Document viewer not presenting in iOS

This issue was due to the viewController taking inside this plugin by this code
UIViewController *viewController =     [UIApplication sharedApplication].delegate.window.rootViewController;
which wont work in my case as my project is native app + flutter as plugin..

I changed the viewController instance. and things working for me


(BUG) The "Invite" page isn't loaded after clicking the "Invite+" button

Precondition:
The "Home" screen is displayed
Steps to reproduce:

Click the "Invite+" button
Pay attention to the blank page instead of the  content according to the requirement

Actual result: The "Invite" page isn't loaded after clicking the "Invite+" button [only on the desktop. On the same account but on the mobile the "Invite" page is displayed]
Expected result: The "Invite" page should be displayed after clicking the "Invite+" button
Checked on PROD V0.24.0
Reproduced on:
Desktop: Windows 10 // Google Chrome 83.0.4103.97
Video: https://www.screencast.com/t/74Rs9GTIH
Errors from the console:

As i remember, invites was etoro-only stuff.
So this button shouldn't be even displayed on current prod (because REACT_APP_ETORO is set false)
@sirpy is it right ?

we have invites. for me it works. can you reproduce this?

@AnastasiiaOdnoshevna

@AnastasiiaOdnoshevna bugs blocking functionality are critical

@sirpy @AnastasiiaOdnoshevna
this issue comes from 'https://w3.gooddollar.org'. it's not the GoodDapp issue.

@yaroslav-fedyshyn-nordwhale please elaborate what is the error
how to reproduce? i dont get this error.
does it happen with specific user?
please provide details in slack to reproduce

@AnastasiiaOdnoshevna @sirpy
Actually I am not able to reproduce this issue again. @AnastasiiaOdnoshevna could you try?
Maybe it was fixed since my last comment.

@yaroslav-fedyshyn-nordwhale we didnt do anything, so it cant be fixed.
we need to understand how we can reproduce it

This issue still reproduced. I checked this in the incognito tab and this was reproduced only at the first time immediately after login after opening the invite page. But  in the ordinary tab, this issue was reproduced quite often
Video: https://www.screencast.com/t/mihbyR61gyb
Checked on PROD V0.24.0
Reproduced on:
Desktop: Windows 10 // Google Chrome 83.0.4103.97

@sirpy @yaroslav-fedyshyn-nordwhale
Also, I checked on 'https://w3.gooddollar.org' and I can say that this issue reproduced here too
I don't know related it to this issue but I noticed when the issue isn't reproduced you can close the console tab and refresh the page again a few times and then screen getting blank and errors appear in the console (or  vice versa, for example, the console  is closed then you should  open and refresh the page a few time)
the https://www.screencast.com/t/agEE6T5N

@AnastasiiaOdnoshevna please supply user credentials

@sirpy one of those accounts where it had been reproduced<EMAIL_ADDRESS>/Qwert_321 but I recently have checked and this issue isn't reproduced anymore

the issue isn't reproduced
Checked on PROD V0.24.0:
Desktop: Windows 10 // Google Chrome 83.0.4103.116


Bunch of cleaning up in tests

Remove unused tests
Get rid of old tests specification in some functions
Remove remnant of old CMake script that are now useless

si c'est juste raw(acos) c'est pas grave : il est prévu pour ne pas donner de (très) bons resultats près de 1.
2 manières de gérer

utiliser une erreur absolue
réduire l'intervalle de test entre -1 et 0.9

note qu'il y a ça aussi :
35/80 Test #674: unit.real.core.gather.exe ...........................***Failed    1.95 sec
[EVE] - Target: eve::avx512_ - Assertions: Enabled  - PRNG Seed: 18102008
[SCENARIO] - Check eve::gather behavior with 32 bits indexes (with T = eve::avx512_abi_v0::wide<float, eve::fixed<1> >)
[SCENARIO] - Check eve::gather behavior with 32 bits indexes (with T = eve::avx512_abi_v0::wide<float, eve::fixed<2> >)
[SCENARIO] - Check eve::gather behavior with 32 bits indexes (with T = eve::avx512_abi_v0::wide<float, eve::fixed<4> >)
[SCENARIO] - Check eve::gather behavior with 32 bits indexes (with T = eve::avx512_abi_v0::wide<float, eve::fixed<8> >)
[SCENARIO] - Check eve::gather behavior with 32 bits indexes (with T = eve::avx512_abi_v0::wide<float, eve::fixed<16> >)
[SCENARIO] - Check eve::gather behavior with 32 bits indexes (with T = eve::avx512_abi_v0::wide<float, eve::fixed<32> >)
[SCENARIO] - Check eve::gather behavior with 32 bits indexes (with T = eve::avx512_abi_v0::wide<double, eve::fixed<1> >)
[SCENARIO] - Check eve::gather behavior with 32 bits indexes (with T = eve::avx512_abi_v0::wide<double, eve::fixed<2> >)
[SCENARIO] - Check eve::gather behavior with 32 bits indexes (with T = eve::avx512_abi_v0::wide<double, eve::fixed<4> >)
[SCENARIO] - Check eve::gather behavior with 32 bits indexes (with T = eve::avx512_abi_v0::wide<double, eve::fixed<8> >)
[SCENARIO] - Check eve::gather behavior with 32 bits indexes (with T = eve::avx512_abi_v0::wide<double, eve::fixed<16> >)
[SCENARIO] - Check eve::gather behavior with 32 bits indexes (with T = eve::avx512_abi_v0::wide<signed char, eve::fixed<1> >)
[EVE] - Assertion 'is_aligned<alignment()>(p)' failed !
In file /home/falcou/dev/actions-runner/legend-1/_work/eve/eve/include/eve/memory/aligned_ptr.hpp, line 78
Because:
0x7ffd4c587670 is not aligned on 64 elements (or 64 bytes).

note qu'il y a ça aussi : 35/80 Test #674: unit.real.core.gather.exe ...........................***Failed 1.95 sec [EVE] - Target: eve::avx512_ - Assertions: Enabled - PRNG Seed: 18102008 [SCENARIO] - Check eve::gather behavior with 32 bits indexes (with T = eve::avx512_abi_v0::wide<float, eve::fixed<1> >) [SCENARIO] - Check eve::gather behavior with 32 bits indexes (with T = eve::avx512_abi_v0::wide<float, eve::fixed<2> >) [SCENARIO] - Check eve::gather behavior with 32 bits indexes (with T = eve::avx512_abi_v0::wide<float, eve::fixed<4> >) [SCENARIO] - Check eve::gather behavior with 32 bits indexes (with T = eve::avx512_abi_v0::wide<float, eve::fixed<8> >) [SCENARIO] - Check eve::gather behavior with 32 bits indexes (with T = eve::avx512_abi_v0::wide<float, eve::fixed<16> >) [SCENARIO] - Check eve::gather behavior with 32 bits indexes (with T = eve::avx512_abi_v0::wide<float, eve::fixed<32> >) [SCENARIO] - Check eve::gather behavior with 32 bits indexes (with T = eve::avx512_abi_v0::wide<double, eve::fixed<1> >) [SCENARIO] - Check eve::gather behavior with 32 bits indexes (with T = eve::avx512_abi_v0::wide<double, eve::fixed<2> >) [SCENARIO] - Check eve::gather behavior with 32 bits indexes (with T = eve::avx512_abi_v0::wide<double, eve::fixed<4> >) [SCENARIO] - Check eve::gather behavior with 32 bits indexes (with T = eve::avx512_abi_v0::wide<double, eve::fixed<8> >) [SCENARIO] - Check eve::gather behavior with 32 bits indexes (with T = eve::avx512_abi_v0::wide<double, eve::fixed<16> >) [SCENARIO] - Check eve::gather behavior with 32 bits indexes (with T = eve::avx512_abi_v0::wide<signed char, eve::fixed<1> >) [EVE] - Assertion 'is_aligned<alignment()>(p)' failed ! In file /home/falcou/dev/actions-runner/legend-1/_work/eve/eve/include/eve/memory/aligned_ptr.hpp, line 78 Because: 0x7ffd4c587670 is not aligned on 64 elements (or 64 bytes).

I know. I probably replaced a T with v_t instead of wide somewhere.


grpc connection breaks with Reannounce method

when reannounce job in cron is enabled, grpc connection breaks with error "found protocol version 21 but expected 1"
Logs
master - https://hastebin.com/ucacifipan.apache
worker - https://hastebin.com/omaqekodop.http

concurrent errors. need grpc refactoring or migration on ws(s)
time.Sleep in this block resolves the problem

not fixed
4:10PM DBG running 1min jobs... func=app.(*cron).runCronTasks
4:10PM DBG reannounces is not locked; running job... func=app.(*cron).reannounce
4:10PM DBG running 5min jobs... func=app.(*cron).runCronTasks
4:10PM WRN abnormal response from worker service error_code=DeadlineExceeded error_message="context deadline exceeded" func=master.(*worker).getRPCErrors
4:10PM ERR got an error in force reannounce request error="rpc error: code = DeadlineExceeded desc = context deadline exceeded" func=app.(*cron).reannounce.func1
4:10PM WRN abnormal response from worker service error_code=DeadlineExceeded error_message="context deadline exceeded" func=master.(*worker).getRPCErrors
4:10PM ERR could not get torrents from the given worker id; skipping... error="rpc error: code = DeadlineExceeded desc = context deadline exceeded" func=app.(*deploy).getWorkersTorrentsV2 worker_id=70e40133-4c0c-4859-a04d-e1c7e5f2c74c
4:10PM ERR got an error in deploy failed_announces request error="could not continue the delpoy process because one of workers errors" func=app.(*cron).reannounce.func1
4:10PM DBG deploy updates is not locked; running job... func=app.(*cron).redeploy
4:10PM DBG mask after switch cron_tasks=1 func=app.(*cron).runCronTasks
4:10PM DBG Called getResponse. func=anilibria.(*ApiClient).getApiResponse
4:10PM INF Correct response api_method=/getUpdates func=anilibria.(*ApiClient).getApiResponse
4:10PM ERR ATTENTION!!! Ignoring title with id 3996 or 8452; please, fix this shit immediately func=app.(*deploy).getAnilibriaUpdatesTorrents
4:10PM DBG worker reply accepted, authorizing... func=master.(*worker).authorizeSerivceReply worker_id=70e40133-4c0c-4859-a04d-e1c7e5f2c74c
4:10PM DBG the worker's reply has been authorized func=master.(*worker).authorizeSerivceReply worker_id=70e40133-4c0c-4859-a04d-e1c7e5f2c74c
4:10PM DBG got reply from the worker with torrents list func=master.(*worker).getTorrents torrents_count=45
4:11PM DBG mask before switch cron_tasks=0 func=app.(*cron).runCronTasks
4:11PM DBG running 1min jobs... func=app.(*cron).runCronTasks
4:11PM DBG reannounces is not locked; running job... func=app.(*cron).reannounce
4:11PM DBG mask after switch cron_tasks=2 func=app.(*cron).runCronTasks
4:11PM DBG worker reply accepted, authorizing... func=master.(*worker).authorizeSerivceReply worker_id=70e40133-4c0c-4859-a04d-e1c7e5f2c74c
4:11PM DBG the worker's reply has been authorized func=master.(*worker).authorizeSerivceReply worker_id=70e40133-4c0c-4859-a04d-e1c7e5f2c74c
4:11PM DBG worker reply accepted, authorizing... func=master.(*worker).authorizeSerivceReply worker_id=70e40133-4c0c-4859-a04d-e1c7e5f2c74c
4:11PM DBG the worker's reply has been authorized func=master.(*worker).authorizeSerivceReply worker_id=70e40133-4c0c-4859-a04d-e1c7e5f2c74c
4:11PM DBG got reply from the worker with torrents list func=master.(*worker).getTorrents torrents_count=45
4:11PM DBG torrent marked as failed func=app.(*deploy).searchFailedTitles torrents_hash=06f93ea97
filter=id%2Ccode%2Cnames%2Cupdated%2Clast_change%2Cstatus%2Ctype%2Ctorrents&search=Golden+Kamuy+4
4:11PM DBG api reqiest has been completed with response 200 OK api_method=/searchTitles func=anilibria.(*ApiClient).getApiResponseV2
4:11PM WRN got a problem in searching failed titles; there are none, two or more titles in the result; manual search required func=app.(*deploy).searchFailedTitles title_name="Golden Kamuy 4" titles_count=4 torrent_hash=06f93ea97
4:11PM WRN failed torrent will be skipped and not deleted func=app.(*deploy).searchFailedTitles title_name="Golden Kamuy 4" torrent_hash=06f93ea97
4:12PM DBG mask before switch cron_tasks=0 func=app.(*cron).runCronTasks
4:12PM DBG running 1min jobs... func=app.(*cron).runCronTasks
4:12PM DBG reannounces is not locked; running job... func=app.(*cron).reannounce
4:12PM DBG mask after switch cron_tasks=2 func=app.(*cron).runCronTasks
4:12PM WRN abnormal response from worker service error_code=DeadlineExceeded error_message="context deadline exceeded" func=master.(*worker).getRPCErrors
4:12PM ERR got an error in force reannounce request error="rpc error: code = DeadlineExceeded desc = context deadline exceeded" func=app.(*cron).reannounce.func1
4:12PM WRN abnormal response from worker service error_code=Internal error_message="found protocol version 107 but expected 1" func=master.(*worker).getRPCErrors
4:12PM ERR could not get torrents from the given worker id; skipping... error="rpc error: code = Internal desc = found protocol version 107 but expected 1" func=app.(*deploy).getWorkersTorrentsV2 worker_id=70e40133-4c0c-4859-a04d-e1c7e5f2c74c
4:12PM ERR got an error in deploy failed_announces request error="could not continue the delpoy process because one of workers errors" func=app.(*cron).reannounce.func1

the problem occurs when the eth0 link is 100% busy


Refactor StockSnap to use ProviderDataIngester

Fixes
Fixes #595 by @stacimc
Description
Moves StockSnap provider API script to use the new ProviderDataIngester class, and updates a couple of the url and title details. (More detail in #299 and https://github.com/WordPress/openverse-catalog/pull/555 ).
Testing Instructions
Check out the branch, and run just test as one piece.
You can also go to the Airflow page and manually kick off the stocksnap_workflow dag, and take a look at the resulting tsv file. The Airflow log will give you the exact file name, and you can see it by running just shell and more /var/workflow_output/stocksnap_image_v001_20220713202334.tsv (or whatever the file is called). My only hesitation with running it that way is wanting to avoid abusing / overwhelming the stocksnap API.
Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[x] I added or updated tests for the changes I made (if applicable).
[x] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

I don't seem to be able to run this with real data locally (at least, when I do just down -v, just recreate, and then go to my local airflow site and click run, it runs twice "successfully" but skips writing to or pulling from S3 both times, and I can't find any tmp directory with tsv files on my local machine after a very quick look), but that might be about separate issues with my local set-up, which I'm going to delve into more with @sarayourfriend later this week.
I think everything else is resolved?

Thanks so much @zackkrida , @krysal  and @stacimc!

💯 This looks great! Thanks so much for your perseverance with local testing issues :) Everything looks good, and running locally works great.

Thanks so much! I didn't see it had been approved when I just edited the overview. Oh well. Either way-- Yay! Onward!


Use crayfish commons

GitHub Issue: Part of https://github.com/Islandora-CLAW/CLAW/issues/578

Depends on https://github.com/Islandora-CLAW/Crayfish-Commons/pull/1

What does this Pull Request do?
Adapts hypercube to use the param converter from Crayfish Commons, preparing it for use with API-X.
What's new?
Using the param converter now, so the controller and its tests had to be touched up a little.
How should this be tested?

Get a JWT.  I added a dsm here and then performed a write operation on a FedoraResource to print it to the page.
Use an HTTP client like Postman, making sure to add your token as an Authorization header for both of the following requests.
Add a tiff to Fedora as a NonRdfSource at http://localhost:8080/fcrepo/rest/some/crazy/path
Run the builtin webserver with php -S localhost:8088 -t /path/to/Hypercube/src
Request the OCR for the tiff sending a GET request to Hypercube at http://localhost:8088/some/crazy/path

Interested parties
Tag (@ mention) interested parties or, if unsure, @Islandora-CLAW/committers

This'll fail until https://github.com/Islandora-CLAW/Crayfish-Commons is on packagist.

@ruebot yes.  the resource must be in fedora at that point.  we'll use camel to ccontrol the order of operations so things get put into fedora before derivatives are attempted.

Just tested this and it does what it says. I made a PR to your branch @dannylamb to add some documentation since I ran into some configuration issues when testing with Apache.
Looks like travis is failing because the lock file needs to be update. Running composer update should fix it. Once Travis is happy I'm happy. 👍🏻

IMO we should leave the lock in version control for the reasons listed here:
https://getcomposer.org/doc/01-basic-usage.md#commit-your-composer-lock-file-to-version-control

@jonathangreen++

I agree.  Keep the lockfile.
For some reason https://github.com/Islandora-CLAW/drupal-project was blowing up with its lockfile committed.  But I can't precisely recall why we're not using them in the individual d8 modules, so it probably is a good idea to revisit the issue @ruebot

Blergh.  Blocked by https://github.com/Islandora-CLAW/CLAW/issues/585.

Restarting the build to see what happen now that @jonathangreen has set everything up to share a line_endings check.

Looks like I gotta rebase for this to start passing. I can get to it tonight. On my phone right now.

Looks like that previous fail was just a travis hiccup. Looks all good and working now.


WIP - experimentation with Java 11

(I think) PR 419 already does this as well as other related needed dependency changes.

Yes, I'm wondering if this now redundant. Running tests to confirm.

@jgallimore 12 is out now ;-)

Closing this one and I'll open a new one from the apache branch


Basic functionality needs JavaScript

Gitea version (or commit ref):
Git version:
Operating system:
Database (use [x]):

[x] PostgreSQL
[x] MySQL
[x] MSSQL
[x] SQLite

Can you reproduce the bug at https://try.gitea.io:

[x] Yes (https://try.gitea.io/ [logged in])
[ ] No
[ ] Not relevant

Log gist:

Description
Gitea works more or less well without JavaScript, but there are some features that are not accessible because the top menu and other parts of the GUI require JavaScript. Also (a minor thing) the message 'Please enable JavaScript in your browser!' is not internationalized.
See also https://github.com/gogits/gogs/issues/4778 and https://github.com/gogits/gogs/pull/4780

Please submit your PR also to gitea.
For i18n you need to add the phrase inside : /options/locale/locale_en-US.ini. The others files are generated from crowdin.

Duplicated here: https://github.com/go-gitea/gitea/issues/2527
Sorry for that, I didn't notice.

Closed by #2541 ?

@bkcsoft I think yes.


Cassandra native types tests

changes to Message for parsing cassandra types (move to separate PR?)
Tests for Cassandra native types (date, int, ascii, etc)
Started tests for Cassandra collections: only have list at the moment and am not testing selects yet as the ResultValue needs to have stuff added for parsing collections.

FYI: This is waiting on #406 and #433.


Pathname should be required for non-rails application

require 'activemerchant'
# => NameError: uninitialized constant ActiveMerchant::Billing::Pathname

cc @wvanbergen / https://github.com/Shopify/active_merchant/commit/0687ea2b4e7eda10ac621b92850ca1a2e3907c73

This looks good to me. @ntalbott ?

FYI: tests are passed since bundler automatically load pathname for us

Merged (by hand). Thanks for the contribution!


FlipViewExtensions implementation

GitHub Issue (If applicable): #360
PR Type
What kind of change does this PR introduce?

Feature

What is the current behavior?

Devs sad because they need to set a lot of things in order to link the FlipView and PipsPager control;
Devs sad because there's no easy way to navigate throw FlipView items using external Buttons

What is the new behavior?

Devs happy because now it's easier to link the FlipView and PipsPager control;
Devs happy because they can easily navigate throw FlipView items using external Buttons

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tested code with current supported SDKs
[ ] Docs have been added/updated which fit documentation template. (for bug fixes / features)
[ ] Unit Tests and/or UI Tests for the changes have been added (for bug fixes / features) (if applicable)
[ ] Wasm UI Tests are not showing unexpected any differences. Validate PR Screenshots Compare Test Run results.
[ ] Contains NO breaking changes
[ ] Updated the Release Notes
[ ] Associated with an issue (GitHub or internal)

Other information

Internal Issue (If applicable):

We should be including runtime/UI Tests for most changes moving forward.
Are we able to squeeze those into this PR or does this need to merge ASAP? Otherwise, we can pair on writing the tests after the release

Let's take the time to add a runtime test as well.

There are multiple comments that are marked as resolved, but aren't really addressed.


[#2883] refactor(ITUtils): move assertion functions from AbstractIT for clarity

What changes were proposed in this pull request?
Move the two utility-oriented functions, assertionsTableInfo and assertColumn, from AbstractIT.java to ITUtils.java, and make them public.
Update all references to these functions accordingly.
Why are the changes needed?
Currently, the AbstractIT class in the Gravitino project contains two functions, assertionsTableInfo and assertColumn, which are more utility-oriented rather than integral to the abstract test class itself.
Fix: #2883
Does this PR introduce any user-facing change?
N/A
How was this patch tested?
N/A

@yuqi1129 would you please help to review? Thanks.

Verified and it's okay with me.

Merging to main, thanks @laiyousin for your contribution.


A game?  Do you have any screenshots you can share??

I'm excited to see "figurace_game" as your repo's name!

I would love to see what you've made or are making.  If you have a moment to add a screenshot or two to your readme, I'm sure visitors would love to see as well!

Hi! We made this game as our final project in a course of Python at university :)
I'll add some screenshots to the readme, thanks for the tip.
PySimpleGUI was recommended by our teachers, great work!

PySimpleGUI was recommended by our teachers, great work!

Your teachers are clearly BRILLIANT!  Like geniuses!  LOL

Can I ask what grade you're in?  I'm thrilled you're using PySimpleGUI while learning to program!

Yes they are! We really enjoyed that course.
We are currently in 2nd grade. It has been very helpful for creating simple GUIs without much effort

We really enjoyed that course.

 the enthusiasm you have for programming.  That's so encouraging and motivating for me to see.  FUN.. it's the # 1 goal for PySimpleGUI, so sounds like maybe that goal was met.

It was indeed great fun. Thank you for taking your time to write these words!

MM'"""""`MM MMP"""""YMM MMP"""""""MM M""MMMMMMMM dP dP
M' .mmm. `M M' .mmm. `M M' .mmmm  MM M  MMMMMMMM 88 88
M  MMMMMMMM M  MMMMM  M M         `M M  MMMMMMMM 88 88
M  MMM   `M M  MMMMM  M M  MMMMM  MM M  MMMMMMMM dP dP
M. `MMM' .M M. `MMM' .M M  MMMMM  MM M  MMMMMMMM
MM.     .MM MMb     dMM M  MMMMM  MM M         M oo oo
MMMMMMMMMMM MMMMMMMMMMM MMMMMMMMMMMM MMMMMMMMMMM

Nice screenshots!

I like how you've made your buttons span the width of the window.  I don't recall seeing that done before the way you've done it doing down your window, with several buttons in a row.
One of the reasons I ask for screenshots and take a particular interest in conversing with students or beginners is that you guys/gals that are still learning are where new ideas come from.  You don't know the limits.  You haven't been taught the "right way" to do things.

 - Pablo Picasso

I love this quote by Picasso.  It captures how I want to write software... like a child... without limits... willing to try things that haven't been tried.
Thanks for the great conversation and thank you for teaching me a new technique to use in PySimpleGUI windows.  I hope it's OK if I, ahem, borrow your idea.  😉


Omit list suffix for computed fields

Omits list suffix for computed fields when the pgOmitListSuffix flag is set.
Similar to https://github.com/graphile-contrib/pg-simplify-inflector/pull/13
It's a breaking change as well so would be great to have it merged before 4.0.0.

Thanks!


I can't continue the airdrop claim

What version of Wallet are you using?
Ignition
What is happening?
I'm trying to make the airdrop claim and there's an error in my wallet. I made the report, but....i'm just still waiting.
steps to reproduce
Going into https://app.fuel.network/drop, press the claim button and...an error appears

How it should behave?
Should complet the claim
What os are you seeing the problem on?
Windows Desktop
What browser?
Chrome
What is your browser version?
What should i do?

Hi there,
We use GitHub issues as a place to track bugs and other development-related issues.
Please see the link below to our dedicated support line:
Help Center :  Click Here
Ticket ID: WB240
Note: Click on the live chat icon at the bottom corner of the page to start a conversation.

Hi @Mkillero , what wallet are you using when connecting to then claim?
I see it looks like your connection with the wallet has problem, you should be seeing this "connected" status

Hello @Mkillero
We apologize for the inconvenience you’re experiencing. For personalized assistance, please contact our support team directly.
Ticket ID:FUE678
Fuel support
To start a chat with a live support agent, click the live chat icon at the bottom corner of the page.

Hi @Mkillero , what wallet are you using when connecting to then claim?
I see it looks like your connection with the wallet has problem, you should be seeing this "connected" status

Luiz, thank you so much! I Just disconeccted my wallet, and connected it again, and works! thank you, mate!


Export not unzipped?

Did I miss anything? It's the ZIP file that is committed. I have to do the following to unzip. 🙏🏽
          # Find the zip file using the wildcard
          zip_file="$(ls *.zip | head -n1)"
          # Unzip the file
          unzip "$zip_file"
          # Get the top-level folder
          folder=$(unzip -Z1 "$zip_file" | awk -F/ '{print $1}' | sort | uniq)
          # Move the contents of the first-level folder to the current directory
          rsync "${folder%/}"/* .
          # Remove the first-level folder
          rm -r "${folder%/}"
          # Delete the zip file
          rm "$zip_file"

Have you altered the script in some way? Usually these two lines extract the .zip and remove it afterwards:
https://github.com/richartkeil/notion-guardian/blob/fd0e29902ee0eb72337779ec12e5ce3a8091e1cc/index.js#L103-L104C8
Also workspace.zip is in the .gitignore which prevents it from being committed.

No modifications. I have to do above, so I can unzip it. Thanks for checking.

I encountered the same issue
I believe the reason is that when calling notion exportURL, it returns a zip package. The exportFromNotion function compresses the zip package again.
const stream = response.data.pipe(createWriteStream(destination));

My solution is to use the onEntry property of extract to detect and unzip the zip package again. The code looks like this:
  await extract(workspaceZip, {
    dir: workspaceDir, onEntry: async (entry, zipfile) => {
      if (entry.fileName.toLowerCase().endsWith('.zip')) {
        const nestedZipFile = join(workspaceDir, entry.fileName);
        zipfile.openReadStream(entry, (err, readStream) => {
          if (err) throw err;
          const writeStream = createWriteStream(nestedZipFile);
          readStream.pipe(writeStream);
          writeStream.on('finish', () => {
            extract(nestedZipFile, { dir: workspaceDir })
          });
        });
      }
    }
  });

This problem should be fixed now. Thank you @heibaikn for the PR! I tried using your approach, however it did not work when I tested it with a large export (~1GB), so I fell back to another solution and used the opportunity to clean up the repo a bit.


DnsServerDsc: Fixed code coverage in the pipeline

Pull Request (PR) description

DnsServerDsc

Fixed code coverage in the pipeline (issue #246).

This Pull Request (PR) fixes the following issues

Fixes #246

Task list

[x] Added an entry to the change log under the Unreleased section of the
file CHANGELOG.md. Entry should say what was changed and how that
affects users (if applicable), and reference the issue being resolved
(if applicable).
[ ] Resource documentation added/updated in README.md.
[ ] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help.
[ ] Comment-based help added/updated.
[ ] Localization strings added/updated in all localization files as appropriate.
[ ] Examples appropriately added/updated.
[ ] Unit tests added/updated. See DSC Community Testing Guidelines.
[ ] Integration tests added/updated (where possible). See DSC Community Testing Guidelines.
[ ] New/changed code adheres to DSC Community Style Guidelines.

This change is

/AzurePipelines run

@Sudman1 now code coverage works for classes:
Codecov.io: https://app.codecov.io/gh/dsccommunity/DnsServerDsc/compare/253/tree/source
AzDO: https://dev.azure.com/dsccommunity/DnsServerDsc/_build/results?buildId=4325&view=codecoverage-tab

/AzurePipelines run


Avoid defining app credentials via BuildConfig

Issue
Currently, we're having a sample to define BASE_API_URL via BuildConfig. BuildConfig will not be obfuscated by proguard so this way is not safe to go, especially when we have more secrets like API_KEY or API_SECRET. We should provide them through a separate class which will be obfuscated in each flavor folder instead.
Solution
By having 2 environments as default: production and staging, we will need to define 2 new ApiEndpointUrl classes in each flavor.

Who Benefits?
Developers
I think this is essential because app credentials management is very important.
What's Next?

[ ] Remove BASE_API_URL definition via BuildConfig
[ ] Define 2 new ApiEndpointUrl classes in each flavor to provide API endpoint urls.

@Tuubz added reason to be essential ticket

@luongvo you might want to have a quick check again, simply read a field from BuildConfig, package an APK and then run jadx to reverse it. It seems like if we are not defining
-keep class com.example.BuildConfig { *; } in proguard file, it will be obfuscated like this:

Seems like it works nicely these days with proguard/R8. If so, we should continue to keep this practice.

or maybe it worked with the conversion from gradle.properties -> gradle.app -> BuildConfig -> app
https://medium.com/@abhishekdubey_70837/using-gradle-to-secure-app-keys-in-android-9d2796ca4007
the expected outcome is that the plain string must not be visible, either using the class way or this way 🆗

with the class obfuscation, only the var name is obfuscated FYI

@luongvo I have checked by decompiling the APK and the result is the same with @sleepylee . 🙏

Here is the apiEndpointUrl from the new approach

And this is from BuildConfig

The classes/ method is obfuscated only, not the value 😅

@manh-t it's public repository so we better not leaking client's detail.

I can confirm that both BuildConfig & ApiEndpointUrl  only obfuscate the function name and not the value itself 🙏

However, I do have a preference for ApiEndpointUrl (Ex: Secrets.kt):

Cleaner code separation
No need to Gradle sync after making an update

@luongvo Any thoughts? In case you agree, would you mind rewriting the description of this issue? 🚀

@Tuubz did you try the solution I provided above? It seems that I and @manh-t receive a new result here with BuildConfig: both the name + value are obfuscated ⭐

@sleepylee Oh, seems like I misunderstood then
Anyways, I tried the following for CoroutineTemplate:

Enable isMinifyEnabled & isShrinkResources to true
Move BASE_API_URL in to gradle.properties
Adapt build.gradle.kts so BASE_API_URL is used: buildConfigField("String", "BASE_API_URL", baseApiUrl)
Generate an APK
Run jadx on the APK
Open up the result in a text-editor
Search references for BASE_API_URL

Did I miss something here? 🙇

@Tuubz that sounds correct and the value URL is supposed to be obfuscated as well with this approach 🤔  Not sure what could be missing from your side, so could you please zip the codebase (remove your build/ part to minimize the size) and post it here for us to investigate?

@sleepylee @manh-t I tried this on CoroutineTemplate could you guys do the same? 🙏
If the value is still obfuscated on your side, then something is definitely missing in the code.
Or perhaps something is wrong with my jadx, when I run it says: finished with errors, count: 27 🙈

@sleepylee unfortunately, Toby and I got the same result 💸  Although I didn't keep BuildConfig in proguard, it doesn't obfuscate the BuildConfig's value as the image I provided above.
I'm using Worpt project for demonstration 🙏  Not sure if there is any extra config from your side, we'd love to know that 🙏

And this is from BuildConfig

Continue discussion here 👋

@luongvo Our plan is to go with the BuildConfig + properties approach instead, what do you think?
If you agree, I can create a new issue and reject this one

This issue is now invalid and should be closed


Use name in KongVault as prefix of vault in translation

Is there an existing issue for this?

[X] I have searched the existing issues

Problem Statement
From https://github.com/Kong/kubernetes-ingress-controller/issues/5395#issuecomment-1887813621.
Since it is hard to guarantee uniqueness of a field because of race conditions, and the prefix of vault in Kong has basically the same pattern as k8s object name. We could directly use name of KongVault object as the prefix of translated vaults in Kong configurations.
Proposed Solution

Move Prefix out of spec and add it to status of KongVault
Controller set status.prefix to the value of name when successfully accepted a KongVault and put it into cache
Translator set prefix of translated vault to name of KongVault

Additional information

Should we keep Prefix field in Spec of KongVault like Service does with ClusterIP field?

Acceptance Criteria

[ ] KongVault is translated to vault entity in Kong gateway with the prefix set to value of name
[ ] No vaults with dupe prefix generated after the change of CRD

Will not implement this since we implemented validating webhook + "tie breaker" in translator instead.


Make "spin" and "busy" keywords you can use to find progress indicator.

From someone trying Flutter for the second time:

found progress indicators in the gallery. A search for "spin" or "busy" on the dartdocs didn't help me.

Does gallery mean "flutter gallery" or "widget gallery aka catalog" ?

not clear

I used the gallery app.


Re-up create-turbo.

Anthony's To-Do's:

[x] Make sure all dependencies are up to date
[x] Find anything that may be broken in-editor
[x] Loosen ESLint configuration
[x] Make sure it deploys for CI
[x] Check with TypeScript team that we are following current best practices (post-ship, non-blocking)

Description
Making sure create-turbo is robust and modern.
Testing Instructions
Try it out with npx create-turbo@latest -e https://github.com/vercel/turbo/tree/update-create-turbo/examples/basic
Things to pay attention to:

[ ] Is tasking correct? Are all tasks successful out-of-the-box? Do I hit cache when I'm expecting?
[ ] If I add an unused variable in a workspace, do I see a warning in both my editor and when I run turbo lint?
[ ] Does auto-importing work across packages?
[ ] Anything else wonky in your editor?
[ ] Do I get proper hot-reloading in turbo dev?
[ ] Anything else that may come up!

Current setup is not showing auto-imports from the ui package.
Notable issue from TypeScript: https://github.com/microsoft/TypeScript/issues/53116#issuecomment-1730360536
It's possible I may be reading the guidance wrong. Will continue to investigate post-ship.


[nrf fromlist] west.yml: Update hal_nordic revision

The new revision fixes a problem with long build paths on Windows.
Signed-off-by: Rafał Kuźnia<EMAIL_ADDRESS>

please provide info of upstream PR in commit description, like here:
https://github.com/nrfconnect/sdk-zephyr/commit/7598e62f2e1a64191281f137ec553ed3017f1100


Adding new OH session

Adding the new Tuesday office hours session to the Getting Help page.

Preview: https://mit-orcd.github.io/orcd-docs-previews/PR/PR43/getting-help/#office-hours


Remove autoprefixer as it's already included

Remove autoprefixer as it's already included in postcss-preset-env:
https://github.com/csstools/postcss-plugins/tree/main/plugin-packs/postcss-preset-env#autoprefixer

What kind of changes does this PR include?

[x] Minor content fixes (broken links, typos, etc.)
[ ] New or updated content
[ ] Translated content
[ ] Changes to the docs site code
[ ] Something else!

Description
Remove autoprefixer as it's already included in postcss-preset-env:
https://github.com/csstools/postcss-plugins/tree/main/plugin-packs/postcss-preset-env#autoprefixer

LGTM. I'm good with the install instructions added!


python310Packages.aiohttp-jinja2: 1.5 -> 1.5.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.aiohttp-jinja2 is: Jinja2 support for aiohttp
meta.homepage for python310Packages.aiohttp-jinja2 is: https://github.com/aio-libs/aiohttp_jinja2
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.5.1 with grep in /nix/store/r5dz9q4995sa7419cmh5pvmfmcpxjsrx-python3.10-aiohttp-jinja2-1.5.1
found 1.5.1 in filename of file in /nix/store/r5dz9q4995sa7419cmh5pvmfmcpxjsrx-python3.10-aiohttp-jinja2-1.5.1

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

8 total rebuild path(s)

8 package rebuild(s)

First fifty rebuilds by attrpath
apache-airflow
appdaemon
python310Packages.aiohttp-jinja2
python310Packages.apache-airflow
python310Packages.connexion
python311Packages.aiohttp-jinja2
python311Packages.apache-airflow
python311Packages.connexion

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/r5dz9q4995sa7419cmh5pvmfmcpxjsrx-python3.10-aiohttp-jinja2-1.5.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/r5dz9q4995sa7419cmh5pvmfmcpxjsrx-python3.10-aiohttp-jinja2-1.5.1
ls -la /nix/store/r5dz9q4995sa7419cmh5pvmfmcpxjsrx-python3.10-aiohttp-jinja2-1.5.1/bin

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

2 packages failed to build:

apache-airflow (python310Packages.apache-airflow)
python311Packages.apache-airflow

5 packages built:

appdaemon
python310Packages.aiohttp-jinja2
python310Packages.connexion
python311Packages.aiohttp-jinja2
python311Packages.connexion

Maintainer pings
cc @peterhoeg for testing.

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

2 packages failed to build:

apache-airflow (python310Packages.apache-airflow)
python311Packages.apache-airflow

5 packages built:

appdaemon
python310Packages.aiohttp-jinja2
python310Packages.connexion
python311Packages.aiohttp-jinja2
python311Packages.connexion


:recycle: rework the README.md

Hello!
I wrote a few lines to update the readme from the nice steps you gave me 😄 . You can check the finished result in my fork
I'm open to review or suggestions! Don't feel obligated to merge it quickly.
As a side question: do you need any other issue I can help with?

One change: steps 2 jumps to 4 in the readme.

Whoops, fixed!


expression_caller sailfish failed for hg19

Hi, I am trying to run bcbio RNA-seq pipeline for six human samples. The yaml file is as below:
upload:
  dir: upload
details:
  - analysis: RNA-seq
    algorithm:
      fusion_caller: [oncofuse]
      variantcaller: [gatk-haplotype]
      #transcriptome_align: True
      aligner: star
      quality_format: standard
      #trim_reads: read_through
      #adapters: [truseq, polya]
      #tools_on: [gatk4]
      expression_caller: [cufflinks, stringtie, sailfish]
      transcript_assembler: [cufflinks, stringtie]
      #spikein_fasta: ../data/spikeins/spikeins.fa
    metadata:
      batch: TestBatch1
    description: Test1
    genome_build: hg19

However, error occurred when bcbio call sailfish to estimate the expression. The error information is as follows.

[2018-03-16T10:16Z] Warning: Using default Gaussian distribution due to insufficient paired-end reads in open ranges.  It is recommended that correct parameters (--frag-len-mean and --frag-len-std-dev) be provided.
[2018-03-16T10:16Z] > Map Properties:
[2018-03-16T10:16Z] >   Normalized Map Mass: 23325.37
[2018-03-16T10:16Z] >   Raw Map Mass: 23325.37
[2018-03-16T10:16Z] >   Number of Multi-Reads: 725 (with 1252 total hits)
[2018-03-16T10:16Z] >   Fragment Length Distribution: Truncated Gaussian (default)
[2018-03-16T10:16Z] >                 Default Mean: 200
[2018-03-16T10:16Z] >              Default Std Dev: 80
[2018-03-16T10:16Z] [18:16:54] Learning bias parameters.
[2018-03-16T10:20Z] Processed 33749 loci.
[2018-03-16T10:20Z] [18:20:03] Loading reference annotation.
[2018-03-16T10:20Z] [18:20:04] Re-estimating abundances with bias and multi-read correction.
[2018-03-16T11:13Z] Processed 33749 loci.
[2018-03-16T11:13Z] multiprocessing: run_stringtie_expression
[2018-03-16T11:13Z] multiprocessing: run_sailfish_index
[2018-03-16T11:13Z] Converting /app/smartngs/bcbio/genomes/Hsapiens/hg19/rnaseq/ref-transcripts.gtf to FASTA format.
[2018-03-16T11:14Z] Creating sailfish index for /app/datamart/RNAseq-example/seqc/work/inputs/transcriptome/hg19.fa with 31 bp kmers.
[2018-03-16T11:16Z] Version Info: Could not resolve upgrade information in the alotted time.
[2018-03-16T11:16Z] Check for upgrades manually at www.cs.cmu.edu/~ckingsf/sailfish.
[2018-03-16T11:16Z] writing log to /app/datamart/RNAseq-example/seqc/work/bcbiotx/tmp_vIEqC/hg19/logs/sailfish_index.log
[2018-03-16T11:16Z] RapMap Indexer
[2018-03-16T11:16Z] [Step 1 of 4] : counting k-mers
counted k-mers for 190000 transcriptsElapsed time: 8.16884s
[2018-03-16T11:16Z] Replaced 0 non-ATCG nucleotides
[2018-03-16T11:16Z] Clipped poly-A tails from 1480 transcripts
[2018-03-16T11:16Z] Building rank-select dictionary and saving to disk done
[2018-03-16T11:16Z] Elapsed time: 0.59684s
[2018-03-16T11:16Z] Writing sequence data to file . . . done
[2018-03-16T11:16Z] Elapsed time: 2.49467s
[2018-03-16T11:16Z] [info] Building 32-bit suffix array (length of generalized text is 286511308)
[2018-03-16T11:17Z] Building suffix array . . . success
[2018-03-16T11:17Z] saving to disk . . . done
[2018-03-16T11:17Z] Elapsed time: 19.5162s
[2018-03-16T11:17Z] done
[2018-03-16T11:17Z] Elapsed time: 46.8833s
processed 286000000 positions
[2018-03-16T11:19Z] khash had 118643244 keys
[2018-03-16T11:19Z] saving hash to disk . . . done
[2018-03-16T11:19Z] Elapsed time: 32.9192s
[2018-03-16T11:19Z] multiprocessing: run_sailfish
Traceback (most recent call last):
File "/app/smartngs/bcbio/anaconda/bin/bcbio_nextgen.py", line 241, in
main(**kwargs)
File "/app/smartngs/bcbio/anaconda/bin/bcbio_nextgen.py", line 46, in main
run_main(**kwargs)
File "/app/smartngs/bcbio/anaconda/lib/python2.7/site-packages/bcbio/pipeline/main.py", line 43, in run_main
fc_dir, run_info_yaml)
File "/app/smartngs/bcbio/anaconda/lib/python2.7/site-packages/bcbio/pipeline/main.py", line 87, in _run_toplevel
for xs in pipeline(config, run_info_yaml, parallel, dirs, samples):
File "/app/smartngs/bcbio/anaconda/lib/python2.7/site-packages/bcbio/pipeline/main.py", line 249, in rnaseqpipeline
samples = rnaseq.quantitate_expression_parallel(samples, run_parallel)
File "/app/smartngs/bcbio/anaconda/lib/python2.7/site-packages/bcbio/pipeline/rnaseq.py", line 134, in quantitate_expression_parallel
samples = run_parallel("run_sailfish", samples)
File "/app/smartngs/bcbio/anaconda/lib/python2.7/site-packages/bcbio/distributed/multi.py", line 28, in run_parallel
return run_multicore(fn, items, config, parallel=parallel)
File "/app/smartngs/bcbio/anaconda/lib/python2.7/site-packages/bcbio/distributed/multi.py", line 82, in run_multicore
items = [config_utils.add_cores_to_config(x, parallel["cores_per_job"]) for x in items]
File "/app/smartngs/bcbio/anaconda/lib/python2.7/site-packages/bcbio/pipeline/config_utils.py", line 305, in add_cores_to_config
return _update_config(args, _update_cores)
File "/app/smartngs/bcbio/anaconda/lib/python2.7/site-packages/bcbio/pipeline/config_utils.py", line 320, in _update_config
raise ValueError("Could not find configuration in args: %s" % str(args))
ValueError: Could not find configuration in args: {'files': ['..........

Can you help me??

Sorry about the issue and thanks for the report. Could you try updating to the latest development?
bcbio_nextgen.py upgrade -u development

I recently fixed this issue and think this will resolve your problem. Thanks much.

Hi @chapmanb ,
I tried to update the lastest development using
bcbio_nextgen.py upgrade -u development
However, error occurred as below:

Upgrading bcbio
Fetching package metadata ...........
Solving package specifications: .
Package plan for installation in environment /app/smartngs/bcbio/anaconda:
The following packages will be UPDATED:
conda:     4.3.29-py27_0 conda-forge --> 4.3.34-py27_0 https://mirrors.tuna.tsinghua.edu.cn/anaconda/cloud/conda-forge

The following packages will be SUPERSEDED by a higher-priority channel:
conda-env: 2.6.0-0       conda-forge --> 2.6.0-0       https://mirrors.tuna.tsinghua.edu.cn/anaconda/cloud/conda-forge

conda-env-2.6. 100% |########################################################################################################################| Time: 0:00:00  31.01 kB/s
conda-4.3.34-p 100% |########################################################################################################################| Time: 0:00:00   1.84 MB/s
Package plan for installation in environment /app/smartngs/bcbio/anaconda:
The following packages will be UPDATED:
bcbio-nextgen:          1.0.7a0-py27_7                bioconda                                                        --> 1.0.9a-py27_8            bioconda
ipyparallel:            4.1.0-py27_0                  https://mirrors.tuna.tsinghua.edu.cn/anaconda/pkgs/free         --> 6.1.1-py27_1             conda-forge
ipython-cluster-helper: 0.5.9-py27_0                  bioconda                                                        --> 0.6.1-py27_0             bioconda

The following packages will be SUPERSEDED by a higher-priority channel:
conda:                  4.3.34-py27_0                 https://mirrors.tuna.tsinghua.edu.cn/anaconda/cloud/conda-forge --> 4.3.34-py27_0            conda-forge
conda-env:              2.6.0-0                       https://mirrors.tuna.tsinghua.edu.cn/anaconda/cloud/conda-forge --> 2.6.0-0                  conda-forge

The following packages will be DOWNGRADED:
blas:                   1.1-openblas                  conda-forge                                                     --> 1.0-noblas               conda-forge
coincbc:                2.9.9-blas_openblas_1         conda-forge                                                     [blas_openblas] --> 2.9.8-1                  conda-forge
numpy:                  1.12.1-py27_blas_openblas_201 conda-forge                                                     [blas_openblas] --> 1.12.1-py27_nomkl_0      https://mirrors.tuna.tsinghua.edu.cn/anaconda/pkgs/free [nomkl]
openblas:               0.2.20-4                      conda-forge                                                     --> 0.2.19-2                 conda-forge
scikit-learn:           0.19.1-py27_blas_openblas_201 conda-forge                                                     [blas_openblas] --> 0.19.0-np112py27_nomkl_0 https://mirrors.tuna.tsinghua.edu.cn/anaconda/pkgs/free [nomkl]
scipy:                  1.0.0-py27_blas_openblas_201  conda-forge                                                     [blas_openblas] --> 0.19.1-np112py27_nomkl_0 https://mirrors.tuna.tsinghua.edu.cn/anaconda/pkgs/free [nomkl]

ERROR urllib3.connection:_match_hostname(360): Certificate did not match expected hostname: binstar-cio-packages-prod.s3.amazonaws.com. Certificate: {'subjectAltName': [('DNS', 'cloud.smartngs.com')], 'subject': ((('commonName', u'cloud.smartngs.com'),),)}


Highlight trailing whitespace

git-series should detect and highlight trailing whitespace the way git does.  (That includes respecting git's many different bits of configuration for that.)

Probably best to just reuse git for this. The command to check a single commit for whitespace issues is: git diff-tree --no-commit-id --root -c --check $commit. This will give a return code status and stderr contains the error output.

git-series has code to print diffs directly; that code needs to handle trailing whitespace directly.

I can work on this, if you can point me in the right direction!

@awesomeaniruddh First, extend DiffColors to get the configured color for whitespace errors.  Also read the git config item core.whitespace for what whitespace errors to highlight (see man git-config), and parse it into a set of booleans.  Finally, modify write_diff to detect those whitespace errors (focus on one at a time, starting with blank-at-eol), and highlight the appropriate parts of lines.

Hey, I actually have been more busy than I expected to be lately. I can no longer work on this issue. Sorry :(

@awesomeaniruddh No problem; thank you for poking at it, and thanks for the update so that someone else can work on it and not worry about duplicating effort.


[Item Duplication] Ring of Magnetism II + Hopper = Placeable Items Duping

Having Ring of Magnetisation II (from Extra Alchemy) equipped, while mining any placeable item that sits on a hopper, gives you that item in your inventory, while also sending one down the hopper line. I've duplicated quite a few Drying Basins by accident.
Seems to work with anything you can place on a hopper - confirmed with cauldron, log, soul sand, Ritual Table and block of Amethyst, as well as Beacon I cheated in for testing.

Does this happen with the Magnetism potion effect in general? Or is it specific to when using the ring?


@route('/view/file/int:file_id', methods=['GET']) generates error

If I remove 'int:' from the route string, it works as expected, but I want to specify parameter types as  the same way flask's own route() does.
Doesn't flask-classful support argument type such as int:file_id? Or does it provide any other way to specify parameter types?
Sungsoo Kim
Traceback (most recent call last):
  File "manage.py", line 62, in <module>
    manager.run()
  File "/Users/sungsoo/.pyenv/versions/3.5.2/envs/cms/lib/python3.5/site-packages/flask_script/__init__.py", line 412, in run
    result = self.handle(sys.argv[0], sys.argv[1:])
  File "/Users/sungsoo/.pyenv/versions/3.5.2/envs/cms/lib/python3.5/site-packages/flask_script/__init__.py", line 383, in handle
    res = handle(*args, **config)
  File "/Users/sungsoo/.pyenv/versions/3.5.2/envs/cms/lib/python3.5/site-packages/flask_script/commands.py", line 216, in __call__
    return self.run(*args, **kwargs)
  File "manage.py", line 30, in list_routes
    url = url_for(rule.endpoint, **options)
  File "/Users/sungsoo/.pyenv/versions/3.5.2/envs/cms/lib/python3.5/site-packages/flask/helpers.py", line 323, in url_for
    force_external=external)
  File "/Users/sungsoo/.pyenv/versions/3.5.2/envs/cms/lib/python3.5/site-packages/werkzeug/routing.py", line 1756, in build
    rv = self._partial_build(endpoint, values, method, append_unknown)
  File "/Users/sungsoo/.pyenv/versions/3.5.2/envs/cms/lib/python3.5/site-packages/werkzeug/routing.py", line 1671, in _partial_build
    append_unknown)
  File "/Users/sungsoo/.pyenv/versions/3.5.2/envs/cms/lib/python3.5/site-packages/werkzeug/routing.py", line 1679, in _partial_build
    rv = rule.build(values, append_unknown)
  File "/Users/sungsoo/.pyenv/versions/3.5.2/envs/cms/lib/python3.5/site-packages/werkzeug/routing.py", line 798, in build
    add(self._converters[data].to_url(values[data]))
  File "/Users/sungsoo/.pyenv/versions/3.5.2/envs/cms/lib/python3.5/site-packages/werkzeug/routing.py", line 1016, in to_url
    value = self.num_convert(value)
ValueError: invalid literal for int() with base 10: '[queue_id]'

Sorry, it may not be from flask-classful. I will check-up other things and report the result.
Thanks!

I have used a helper at http://flask.pocoo.org/snippets/117/ to list routes. It has a problem, not flask-classful.
Just ignore this issue!
Thanks!


Fix mishandling of SASL attribute parsing

When parsing SASL attribute strings, we were incorrectly allowing them to be split on = characters more than once per attribute. This broke for attribute values ending with =, most particularly Base64-like values, which appear often in - for example - the nonce strings sent by Supabase PostgreSQL severs.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (6ce96ab) 59.92% compared to head (339b9d1) 62.20%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #451      +/-   ##
==========================================
+ Coverage   59.92%   62.20%   +2.28%
==========================================
  Files         124      124
  Lines        9937     9936       -1
==========================================
+ Hits         5955     6181     +226
+ Misses       3982     3755     -227

Files
Coverage Δ

...IO/Utilities/SASLAuthentication+SCRAM-SHA256.swift
47.48% <100.00%> (+47.21%)
:arrow_up:

... and 5 files with indirect coverage changes


Extra 774 - Handle IAM credential report containing 'no_information' for a user's last console login date

A user who has never logged into the console, or not logged in since Oct 2014 will present as 'no_information' in the
'password_last_used' column of the credential report. Handle this scenario and output a failed message if it has been
more than MAX_DAYS days since the user was created, or an info message if it is less than MAX_DAYS
Fixes #501
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Example output - after creating a new console user but not logging in as them:
 7.74 [extra774] Check if user have unused console login
       PASS! User marcjay has used console login in the past 30 days
       INFO! User testuser has not used console login since they were created 0 days ago

Tested by temporarily setting MAX_DAYS to positive 100 (MAX_DAYS=100), to force a failure:
 7.74 [extra774] Check if user have unused console login
       FAIL! User marcjay has not used console login for more than 2 days
       FAIL! User testuser has never used console login since they were created 0 days ago

Credential report extract:
testuser,arn:aws:iam::123456789012:user/testuser,2020-04-11T18:45:05+00:00,true,no_information,2020-04-11T18:45:06+00:00,N/A,false,false,N/A,N/A,N/A,N/A,false,N/A,N/A,N/A,N/A,false,N/A,false,N/A2020-04-11T18:44:35+00:00

Thanks @marcjay


Startup script is removing variables by mistake

During Startup start.sh is removing variables in master.cf (e.g. ${recipient})
This leads to the following error:
relay=dovecot, delay=0.15, delays=0.12/0.01/0/0.02, dsn=5.3.0, status=bounced (command line usage error. Command output: /usr/lib/dovecot/deliver: option requires an argument -- 'd' Usage: dovecot-lda [-c <config file>] [-d <username>] [-p <path>]                    [-m <mailbox>] [-e] [-k] [-f <envelope sender>]                    [-a <original envelope recipient>]                    [-r <final envelope recipient>]  )

To add to this: the startup script also removes the $mydomain entry in the smtp_banner which makes the banner not RFC conform

My suggestion would be to limit the substitutions by providing the SHELL-FORMAT parameter to envsubst according this explanation on SO

At the moment I don't have time to investigate, but feel free to make a pull request.


Use Fog DNSimple API Auth

The Fog gem now accepts an API token for creating DNSimple records.
The API Token method is preferred so you can use 2FA auth as well as
not have your password log to console.
The Fog gem currently accepts the token as an additional param. If it
detects the password param it will fall back to username/password auth.
Relates to GH #27

Just realized we both implemented this. I just submitted another in #29 :/
I think mine ended up being a bit more robust, so let me know if it looks ok, and perhaps we could close this one?

Hi @pwelch and thank you for submitting this pull request. I've done a fair bit of cleanup in master so please go ahead and merge in current master to see where this falls. I am somewhat in favor personally of #29 due to the backward compatibility option, but open to see if you'd like to merge in master and get this a bit more upgraded with tests. :+1:

Closing this in favor of #33 thank you for your proposed contribution 👍


action receipt contract field is empty

https://iotexscan.io explorer action receipt contract field all is empty.
https://iotexscan.io/action/bf4eef1ee9d01cc70c09605edb29fb17a2da3293c8befa417bec405912f4902d

close because action is fail


Logical Operator Assignment Generates Invalid Javascript

Describe the bug
When a function contains an operator assignment (||=, etc) followed by a closure, SWC will generate invalid Javascript that triggers either an "Uncaught ReferenceError" (in strict mode) or, when not in strict mode, adds a global variable to window.
SWC creates a temporary variable when transpiling operator assignment. For some reason it pushes that variable's declaration into the closure, which means it's not in the outer function's scope.
Input code
function myFunc(options) {
  options.context ||= {}
  const closure = function() {}
}

myFunc({})

console.log({
  _$_options: window._$_options, // <= ES5
  _options: window._options // >= ES2015
})

Config
{
  "jsc": {
    "parser": {
      "syntax": "ecmascript"
    },
    "target": "es5"
  },
  "minify": false
}

Playground link
https://play.swc.rs/?version=1.3.43&code=H4sIAAAAAAAAA2WOwQrCMBBE7%2FsVQ%2FHQgLQq9CLWm%2F6AH9BDTKVQs9KkVGnz72ZpwIO33Zm3M9uOVvuOLZ6faxxzfsnmFGYCyhLZ6AycHzrts6gku9BsvXl7LEuNOUQjCs5D9%2BzGwaBGm3JzJX4gmjp756lYayKwDpRq56CIJIN7U%2FT8yKW%2B2TSp74h0%2FpO28t2pxuVWCfoHJkGos1CH3b6ioL4Ua1OT8AAAAA%3D%3D&config=H4sIAAAAAAAAA0WNSwqAMAxE956iZO2iuHDhbUJNpeKnNBEs4t2NLeguM28euRpjYGYHg7n01BAxMaUva8N5Ezy1AXIrskshChR4t9URTBNJWXBnO%2FvSwmANW%2FBZiceFqVb7eCz0P5Qcqap98Zr7ASqyi6aVAAAA
Expected behavior
The above input code should not set any properties on window, and should not raise errors in strict mode.
It should put the variable declaration outside the closure, like this:
function myFunc(options) {
    var _$_options;
    (_$_options = options).context || (_$_options.context = {});
    var closure = function() {};
}

Actual behavior
It puts the variable declaration inside the closure.
function myFunc(options) {
    (_$_options = options).context || (_$_options.context = {});
    var closure = function() {
        var _$_options;
    };
}

Version
1.13.43
Additional context
Target is es5 in example but the same problem exists for any target that doesn't natively support operator assignment (though the temp variable name is different.) Play with the target, module settings, and "use strict" in the playground to see how it behaves under different circumstances.
The same code in babel doesn't seem to use a temporary variable at all.

Introduced in #7067.
You could assign this issue to me. @kdy1

Thanks!


Create "projects" and "tools" subsites.

This should be designed nicely too.

This issue needs a bit more description. Also are these 2 sections dynamic enough that they would produce regular content?

So I guess this issue is related to https://github.com/opensourcedesign/resources/blob/master/README.md
Now still I don't understand the difference between a project an a tool :) and why they should be separate.

I think the idea is that tools are things we use as designers, whereas projects are open source projects that do good design.
Basically most of the stuff on resources under tools would be a nice "tools" page. We'd have some example images, maybe some tutorials for each tool, etc.
Projects would be a list of projects that we think are designer friendly but not necessarily aimed at designers - Mozilla, NextCloud, OpenFarm, etc. This might also include design tools!

So we have https://opensourcedesign.net/projects/ now which may not have existed when this was first proposed.
Projects could certainly do with some more care and attention. I remember a comment from 2019 FOSDEM from a person who visited our table and said 'What credentials does open source design have to say what Is good design or not' I think some rewording and potential way to 'pitch your OSS project' to be included could be good.
As for tools we have some listed in: https://opensourcedesign.net/resources/ under tools.
I think if it's okay @simonv3 we can close this issue and create new epics for improving both the 'Projects' page and the 'Resources' page?


Please consider fixing issues detected by repository checker

Notification from ioBroker Check and Service Bot
Dear adapter developer,
I'm the ioBroker Check and Service Bot. I'm an automated tool processing routine tasks for the ioBroker infrastructure. I have recently checked the repository for your adapter air-q for common errors and appropiate suggestions to keep this adapter up to date.
Please see the result of the check below.
ioBroker.air-q
   -

ERRORS:

[ ] :heavy_exclamation_mark: [E162] js-controller 4.0.0 listed as dependency but 4.0.24 is required as minimum, 5.0.19 is recommended. Please update dependency at io-package.json.

WARNINGS:

[ ] :eyes: [W034] @iobroker/adapter-core 3.1.4 specified. 3.1.6 is recommended. Please consider updating dependencies at package.json
[ ] :eyes: [W040] "keywords" within package.json should contain "ioBroker"
[ ] :eyes: [W181] "common.license" in io-package.json is deprecated. Please define object "common.licenseInformation"
[ ] :eyes: [W184] "common.main" is deprecated and ignored. Please remove from io-package.json. Use "main" at package.json instead.
[ ] :eyes: [W184] "common.title" is deprecated and replaced by "common.titleLang". Please remove from io-package.json.
[ ] :eyes: [W522] Please consider migrating to admin 5 UI (jsonConfig).

Please review issues reported and consider fixing them as soon as appropiate.
Errors reported by repository checker should be fixed as soon as possible. Some of them require a new release to be considered as fixed. Please note that errors reported by checker might be considered as blocking point for future updates at stable repository.
Warnings reported by repository checker should be reviewed. While some warnings can be considered as a suggestion and be ignored due to good reasons or a dedicated decision of the developer, most warnings should be fixed as soon as appropiate.
Feel free to contact me (@iobroker-bot) if you have any questions or feel that an issue is incorrectly flagged.
And THANKS A LOT for maintaining this adapter from me and all users.
Let's work together for the best user experience.
your
ioBroker Check and Service Bot
@mcm1957 for evidence

This issue has been replaced by new isse #84
This issue can be closed.
your
ioBroker Check and Service Bot


pie

Version
5.0.2
Steps to reproduce
正常引用
What is expected?
正常引用
What is actually happening?
electron引入报Cannot create property 'event' on string 'pieToggleSelect'

An error is reported when the electronic project is packaged: cannot create property 'event' on string 'pietoggleselect'; normal operation is OK, and an error will be reported after packaging

I have the same problem

I have the same problem.
Met his error when packaging a electron-vue project.
In development , it's OK.  in production mode, this error occurs.

An error is reported when the electronic project is packaged: cannot create property 'event' on string 'pietoggleselect'; normal operation is OK, and an error will be reported after packaging

兄弟，这个你是如何解决的？ 谢谢

降级到4 就可以

回复太速度了！  具体的版本号是多少呢？

我降级到4.9.0

哈哈，太感谢了！  666 ！

OK, I solved the problem.
By experience, this messages shows when node can NOT find some module.
The root cause in my case is the dependency and devDependency, node can not find this module.
previously,  I put the echarts to the devDependency segment, so it got error when built package for production.
so we move it to dependency segement, looks like:
// package.json
"dependencies": {
    "echarts": "^5.1.2",
    ...
}

then everything is OK .
also thanks to @xuzhonglin

Why do I still report an error when my data is originally in the dependency

Why do I still report an error when my data is originally in the dependency