Update security scann

Description
Remove govulncheck as it no longer works with the reports generated.

For contributor use:

[ ] Targeted PR against master branch
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work
[ ] Code follows the standards mentioned here
[ ] Updated relevant documentation
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Added appropriate labels

Codecov Report

Merging #937 (77c8542) into master (d20db65) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 77c8542 differs from pull request most recent head c25e907. Consider uploading reports for the commit c25e907 to get more accurate results

@@          Coverage Diff           @@
##           master    #937   +/-   ##
======================================
  Coverage    2.50%   2.50%
======================================
  Files           5       5
  Lines         439     439
======================================
  Hits           11      11
  Misses        427     427
  Partials        1       1

Flag
Coverage Δ

unittests
2.50% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Error cant run command on nebula web console

Everytime i try run any nebula cmd, it yield this error:
2020/09/29 15:14:12 read tcp <IP_ADDRESS>:34482-><IP_ADDRESS>:3699: i/o timeout
2020/09/29 15:14:12.114 [D] [server.go:2802] | <IP_ADDRESS>| 200 |30.026451443s| match| POST /api/db/exec r:/api/db/exec
2020/09/29 15:14:12.451 [D] [server.go:2802] | <IP_ADDRESS>| 200 |27.894018774s| match| POST /api/db/exec r:/api/db/exec
2020/09/29 15:14:12 ErrorCode: E_SESSION_INVALID, ErrorMsg:

It appears on nebula-docer-compose_client_1 container. Could me get any help here. Thanks.

I managed to solve this problem, just stay in nebula-docker-compose v1.

You can restart your nebula server and connect it to try again, it seems to be the error about the nebula server.

Ok, thanks you, i will close this issue.

I'm seeing the same problem.


Update element links in readme

Summary
Updates the five element links in the readme to point to their respective repos.
Details and comments
Currently, the five element links in the readme direct to simply qiskit.org (same behavior for each link). This PR corrects this by having each link target an element-specific page. While pointing to github repos feels most natural for readers of this readme, feel free to let me know if you'd prefer a different location (e.g. https://qiskit.org/documentation/the_elements.html#terra).

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

Hmm... not sure why this still has a build queued after a couple days

Travis just failed to report back, everything ran as expected: https://travis-ci.com/github/Qiskit/qiskit/builds/179685334

Travis just failed to report back, everything ran as expected: https://travis-ci.com/github/Qiskit/qiskit/builds/179685334

Awesome, thanks!


HAWKULAR-300 Fix the availability refresh button.

Fix the availability refresh button(it was using  an old method name) and Make the salutation filter just upper case the first character because some names may want to preserve other case in the name.

Merging despite bogus C* errors on Travis.


utility methods

See https://github.com/samuelcolvin/pydantic/pull/475#discussion_r276064521, @NoAnyLove's comment:
I think the solution is to add pydantic.tools:

schema(dataclass_or_model)
to_json(dataclass_or_model)
parse_obj(dataclass_or_model, obj)
parse_file(dataclass_or_model, file)
etc.

That would provide a simple way of generating a schema that's unified for dataclasses and models without corrupting (making less vanilla) pydantic dataclasses.

Cool, this idea sounds great.

I will note that we still may add a methods module or similar, and AnalyzedType also provides a wrapper exposing similar functionality tied to a specific type. (And has the upshot of encouraging more performant implementations by providing an obvious way to cache the "type analysis", unlike v1's parse_obj_as, etc.)


[AMD] Build LLVM::AtomicRMWOp operations with vector operand

In the case of 16 bit floats operands for tt::AtomicRMWOp, construct only one LLVM::AtomicRMWOp but use vector of elements.
Such approach allows to generate packed intrinsics and process 2 elements at once.
Added a lit test for f16 vectorized case.

I faced with a problem related to axis analysis while was preparing a PR.
Currently it is impossible to introduce lit tests for cases of interest because not properly initialized axis analisys does not allow vector size to be more then 1.
Also test_core::test_tensor_atomic_rmw logic takes pytorch results as golden. But seems like pytorch utilizes fp32 arith, then converts to fp16. In some cases the diference is quite big (due to overflow, I guess).
Currently working on amdgcn-comparison-based test, but I'd like to get rid of it in the future to avoid target, implementation specifics.

Dear reviewers, please  notice getting rid of bf16 atomic restriction change in our FE https://github.com/triton-lang/triton/pull/4925/files#diff-a28ba3a22fe48841e9eeede9c8ad5585e8f80b93e72397991d56a0ff2a4bb376R1306

We need correctness tests too.

Also test_core::test_tensor_atomic_rmw logic takes pytorch results as golden.

Hmm, it's using numpy for random and reference results? Maybe you can configure numpy element types properly for it?

yes, but Accuracy is still low.
Max absolute difference: 0.007812
Max relative difference: 0.3333

this numbers are the same with numbers before vectorization PR.
I'll dig into it more today

Hi @joviliast, I have a small test https://github.com/scxiao/triton_tests/blob/main/test_atomicAdd_fp16.py for fp16, but it seems like the global_atomic_cmpswap v2, v[0:1], v[2:3], off sc0 is still in the isa code, and I did not see atomic_add instructions. Is that expected? Thanks

Hi @joviliast, I have a small test https://github.com/scxiao/triton_tests/blob/main/test_atomicAdd_fp16.py for fp16, but it seems like the global_atomic_cmpswap v2, v[0:1], v[2:3], off sc0 is still in the isa code, and I did not see atomic_add instructions. Is that expected? Thanks

Hi @scxiao ,
Yes, it is expected.
I plan to support vectorization for such tensors further.
Currently only following tensors are vectorized:
https://github.com/joviliast/triton_tests/blob/add-f16-vec/test_atomicAdd_fp16_vec.py

Hi @joviliast, I have a small test https://github.com/scxiao/triton_tests/blob/main/test_atomicAdd_fp16.py for fp16, but it seems like the global_atomic_cmpswap v2, v[0:1], v[2:3], off sc0 is still in the isa code, and I did not see atomic_add instructions. Is that expected? Thanks

Hi @scxiao ,
Yes, it is expected. I plan to support vectorization for such tensors further. Currently only following tensors are vectorized: https://github.com/joviliast/triton_tests/blob/add-f16-vec/test_atomicAdd_fp16_vec.py

Thanks. I took a look later, it seems like, in my test, the elemsPerThread is 1, so there is no data to be vectorized. If I change to have 2 elemsPerThreads, it is vectorized, and we can see 2x speedup.

May want to wait for #4981 to land first to pick up the runtime tests there.

A similar patch of mine (for H100) was rejected at https://github.com/triton-lang/triton/pull/2708
But since then, it has become clear that we could use BF16 accumulators with minimal precision loss if we were to only use them for merging the result of a Split-K across SMs.
Should we revive the other PR in this case?

Good question. I didn't know we had prior discussions regarding it before; thanks for pointing out! I'm not sure I can say confidently about whether having bf16 atomic is useful for certain use cases, but we do have atomic_pk_add_bf16 native support in mi300x. so backend should be able to generate that directly.
For now I'll land the fp vectorization improvements in the backend; we should discuss about the frontend enablement separately.


Rework+Refactor the navbar code

We should rework and refactor the navbar code in a much more sensible way and what's should be merged and exclude on different widths.
Hidding least popular buttons from first, hence we don't have any UX research I can only bet.

language
chain/protocol selector
create
history
stats
profile

With search bar should be flexible and fill up the rest space next to the buttons.
Originally posted by @yangwao in https://github.com/kodadot/nft-gallery/issues/2769#issuecomment-1088467364

I will rework it later.


[TEST] Update ApolloClientProvider.js

Description
Changes proposed in this pull request:

...
...
...

Related issue(s)

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

/retest

/test all

/test pull-compass-console-lint

/test all


Add airframe meta data for airframe class

@hamishwillee This adds a class field to make the auto-generation easier. I've also updated other parts of the files where appropriate.

@hamishwillee Change is in - you can generate your stuff now.


Custom Template

Hi, @ajsb85
I'm going to add the possibility of using a completely customized template of Ruby ERB so that anyone can adjust the resulting HTML to the website needs

Can I help with this?


Setup test/benchmark harness

Blocked on #239
Eventing - https://github.com/knative/eventing/tree/main/test/performance
Kafka - https://github.com/knative-sandbox/eventing-kafka/tree/main/test/performance

/assign @ikvmw

Addressed by https://github.com/knative-sandbox/eventing-rabbitmq/pull/445


dashboard/api: better error description

2024/10/26 00:08:09 alert: error: api request "https://syzkaller.appspot.com/text?json=1&tag=ReproC&x=11003952980000" failed: 500 (%!w())

We have a 500 error w/o explanation. Let's dump some body here to better understand root-cause.
It is hard to filter AppEngine errors. Let's use "ae: " prefix.
Let's return traceID in body to simplify the errors analysis.

PTAL.
It should simplify the problems analysis.


Copyright violation

The COPYING file has been omitted, this is a copyright violation.

Hi,
What is file ?
How to get right?
Thanks

https://github.com/bitcoin/bitcoin/blob/0.8/COPYING
Just copy it back into the top-level folder.

success,
https://github.com/unitcurrency/unitcurrency/blob/master/LICENSE

Good job.


How to enable autoplay for breakpoints 1024 and 768. Also disable autoplay for the other breakpoints.

Check that this is really a bug

[X] I confirm

Reproduction link
https://demo.ntspl.co.in/
Bug description
Autoplay not working for the breakpoints 1024 and 768.
Below is the code. I am using swiper version "swiper": "^8.1.0",
coachSliderConfig: SwiperOptions = {
slidesPerView: 3,
spaceBetween: 20,
navigation: false,
autoHeight: false,
allowTouchMove: false,
loop: true,
effect: 'slide',
breakpoints: {
'0': {
slidesPerView: 1,
},
'640': {
slidesPerView: 1,
},
'768': {
slidesPerView: 2,
autoplay: {
delay: 1000,
disableOnInteraction: false
}
},
'1024': {
slidesPerView: 2,
autoplay: {
delay: 1000,
disableOnInteraction: false
}
},
'1199': {
slidesPerView: 3,
},
'1365': {
slidesPerView: 3,
},
'1600': {
slidesPerView: 3,
},
'1920': {
slidesPerView: 3,
},
},
};
Expected Behavior
No response
Actual Behavior
No response
Swiper version
8.1.0
Platform/Target and Browser Versions
chrome
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

[ ] I'm willing to open a PR

Do you want to ask a question? Are you looking for support? Stack Overflow or Swiper Discussions is the best places for getting support

Please, don't use GitHub issues for questions


Add a Zapier Integration for the Managed Service

In order to make VectorFlow more broadly acceptable to people who are not very technical, we want to add a Zapier integration.

No one currently needs this. Closing the issue


add default depth of 1 to tag filters

With latest changes to Stash on the dev channel (hierarchical tags), the initial filtering in this script would fail as no depth was specified in the query. I just added 1 as a default to get the script working.
Changes should not be merged until these changes are released on the stable branch for Stash

@rg9400  can you draft the PR for now? There is an open PR at stash https://github.com/stashapp/stash/pull/1733 that should fix the backwards compatibility for now.
Also in the PR default depth used is 0 so that should probably be used instead of 1 ( if needed )

@bnkai if it will fix backwards compatibility, there is no point in adding this to the script as it is not actually being used functionally, just adding the same compatibility. I will close the PR.


Another question

Again, can I create a own project without forking with this project but only copying some part of your code?
I'm asking before it happens some problem with laws or rights. I'm working to implement Plex not OSMC and forking can be annoying.
Waiting for your answer. Thanks again!

You can do whatever you want with the code 👍


how to Use DIST,Unet to train conic datase

@simongraham @vqdang  ，hello，I want to train conic dataset using DIST, Unet methods and I found that these methods are only available in tensorflow implementation.  If I want to use the tensorflow version of the codes to train conic datasets using DIST and other methods, how should I modify it? Thanks.

Convert https://github.com/vqdang/hover_net/blob/195ed9b6cc67b12f908285492796fb5c6c15a000/src/loader/augs.py#L283 to an approriate form like https://github.com/vqdang/hover_net/blob/326f17e0121b4338fd05caa83cfbb16a6851175e/models/hovernet/targets.py#L17 and replace https://github.com/vqdang/hover_net/blob/326f17e0121b4338fd05caa83cfbb16a6851175e/models/hovernet/targets.py#L102 with the new function
Define your UNet strcucture and slot it in https://github.com/vqdang/hover_net/blob/326f17e0121b4338fd05caa83cfbb16a6851175e/models/hovernet/net_desc.py#L149 as a replacement.
Change https://github.com/vqdang/hover_net/blob/master/models/hovernet/run_desc.py to what you want to do with UNet DIST.

@vqdang Thank you very much for your reply, I will try it later. I want to extract 256256 patches on the CoNSeP dataset (win_size = [512,512], step_size = [256256]) and set the output of the model to 256*256 as well, just like on the conic branch. Do I need to change the code in the training code, and where do I need to change in the inference code? This will be useful for my research, thanks.


Test change to validate triggers are working.

Looking into why PR triggers weren't working. Now that the trigger on the v4 branch is what I think it should be then things should trigger OK. Hopefully.

/azp run "AutoRest Java - CI"


About the renewal

Hello, everyone!
First, I'm so sorry because of no updates.
I'm very busy now by implementing some open source projects such as RosettaStone and CubbyFlow.
However, I'm preparing cpp-developer-roadmap and renewal of game-developer-roadmap by year´s end.
So, please wait. 🙏

Oh!! I am very anticipating the rewel. What will be enhanced in the renewl?


neighbor_ayat_keyboard.py:23: Перенести классы в...

The puzzle 899-b665940d from #899 has to be resolved:
https://github.com/blablatdinov/tg-quranbot/blob/81c692f32a605f1949dfbd33ece060e37f9fa898/src/srv/ayats/neighbor_ayat_keyboard.py#L23-L23
The puzzle was created by Ilaletdinov Almaz on 20-Jun-24.
role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

The puzzle 899-b665940d has disappeared from the source code, that's why I closed this issue.


Empty chapter

Chapter 9 of the novel I was reading is empty

Closing this as there are no details. Create a new issue and follow the issue template.


How to handle entities with multiple mappings

For example  Begin date (P49)  maps to P580 and P571

There are some cases with multiple mappings but different property datatypes in such cases the if only one type matches chose the matching type.
Example:

 3D model (external) (P1034)
 3D model (Wikimadia Commons) (P1232)

With the type mismatch handling added with #12 and #13 multiple mappings are now automatically resolved if one of the mappings has the same type as the source property. If multiple target properties have the same type the first is selected


Fix context for unique

Context was updated in the past and the template for ValueBase was removed, breaking its functionality.
Adding the template back and a test about how to use it.
fixes: 1797

#1797


[DOC] Glob capture docs

Create documentation for the glob capture feature base on https://github.com/pachyderm/pachyderm/pull/4123

@adelelopez can you please provide a summary of the change and what needs to be described in the documentation?

The PR #4123 has been held back. A new feature request #5624 has been created to migrate the code from 4123 to the API. It was labeled needs docs and will be documented when the implementation is done.
Closing this.

The PR #4123 has been held back. A new feature request #5624 has been created to migrate the code from 4123 to the API. It was labeled needs docs and will be documented when the implementation is done.
Closing this.


How to release a multi-member workspace with inter-crate dependencies?

I'm trying to find some example of how to publish multi-member workspaces that have inter-crate dependencies with version = * path dependencies using the release plugin.  The workspace looks something like this (somewhat simplified)

Cargo.toml
memberA/Cargo.toml
memberB/Cargo.toml
memberC/Cargo.toml

memberB has a dependency to A,  like this
[dependencies]
crateA = { path = "../memberA" , version = "*", registry = "some registry" }
The workspace version is managed in the root Cargo.toml and is inherited by all members.
The question I have is how to handle the version = * string in path dependencies.  When attempting to do a dry run of a release I always encounter this error.
error: all dependencies must have a version specified when publishing
There are no dependency cycle here.   Is the only option here to invoke cargo release on each member crate in turn?

I believe that error is coming from cargo or crates.io, not from cargo-release.
If you put in the version of the package in that field, cargo-release will automatically update it as you go.  You can do cargo release patch --workspace and it will bump the workspace version, the dependency version requirements, etc and publish all packages with that single command.
e.g. a commit generated by cargo-release when releasing clap: https://github.com/clap-rs/clap/commit/5e4facf76fb38e04edba6cb1ab67d4f2b1ec8550

Thanks @epage.  That helps seeing the example in clap.
I have one followup question. Is there a way to ignore the license | license-file requirement in package manifest during cargo release?
These are internal crates and not being published to crates.io. I tried adding --no-verify but that didn't seem to be sufficient.

"Internal crates" as in not being published at all or being published to an alternative registry?

Crates being published to an alternative internal registry.

For the rules that are crates.io specific, I would be fine with them being limited to only crates.io

btw as the core of this issue is resolved, I'm going to go ahead and close it.  Feel free to create another issue (or straight to PR) for the metadata validation


Remove support for "outerfaces"

Joking in the title. There was no outerface feature, but the scanner would scan outerface as interface! LOL!
JavaScript truncates bitwise integer math to 32 bits and that caused our actual limit in the keyword scanning algorithm that used 5 bits per letter to be 6 characters instead of the 10 we thought we had in the "safe" integer range. The fix is to use equivalent, but non-bitwise arithmetic.
Perf testing on my machine shows this is still faster than not having the optimization at all by 4 or 5% when scanning petstore.cadl thousands of times, but a little slower than the (broken) bitwise version. This isn't likely to be noticeable in a full compile, but could be useful if we ever need to quickly scan for something and skip most of the tokens.

It might be too late to take this breaking change. I can't be the only one using famespace as the lit alternative to namespace.


Help me remember which number is skylight

Hi! The current numbers mode often confuses me. I don't use it often, so usually I have already forgotten that skylight from above is the bottom number, and the light on the block below is the top number. We should swap them, but there's still the edge case of 6/9. Could we maybe make the skylight light blue? I'll see if I can make a PR.

Thank you for the Feedback!
Would symbols before the text help you? I'm thinking of a sun before the sky light level and a block symbol for the block light level.
I wouldn't recolor parts of the overlay, as that might be confusing to some people.

Symbols would help me distinguish them, but they would obstruct by view through glass blocks for no good reason. For an overlay on transparent blocks, each pixel used must have a good reason. Thus my ideas of just swapping positions, and potentially using light blue as the sky light color.

I didn't think about the obstruction of the block beneath, good point!
I do plan on adding rotation support to number mode, but because it isn't straightforward to implement right now it takes some time. When that is implemented, the top number will always be the sky light level.
If you don't need the sky light level number, you can hide those in the Lighty config screen as of Lighty v3.0.0-beta.4, so maybe that's a workaround for now. (unless you also need to see the skylight numbers)
I'll see if I get around to working more on Lighty in the next month, but I cannot promise when I'll get around to implement it. With that said, improving number mode has top priority right now, so next release will (hopefully) contain a solution for your problem :)


Use command instead of which

command -v is posix, avoids another dependency.

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "make" git@github.com:mrueg/libnetwork.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

@mrueg Please sign your commit

Superseded by https://github.com/docker/libnetwork/commit/54b97aee7814c57789f666c60eb8521b9ee61e92


Getting this error Kindly Check How can we solve this error?

Getting this error Kindly Check How can we solve this error?

This happens when the python script cannot find the screenshot file to upload. I would check to make sure that you followed all the directions, especially the stuff related to setting up the GoFullPage extension. Also, make sure you change the path = "C:/Users/[YOUR USERNAME]/Downloads/screenshots/*" in the file to be your username (in your case, its Administrator). If none of this works, try running the script with administrator privileges, as Windows Server can have issues with that. Also, there is a known bug I haven't bothered to fix where large Chegg links cause the bot to close the tab before it downloads, so that's another possibility.

It works for 1 time then when next time it runs its like it doesn't take screenshot all permissions are given and also I have increased the sleeping time to 15 sec to avoid to close it before it downloads...
I think keypress might not be working if there is any alternative kindly update me thanks

This happens when the python script cannot find the screenshot file to upload. I would check to make sure that you followed all the directions, especially the stuff related to setting up the GoFullPage extension. Also, make sure you change the path = "C:/Users/[YOUR USERNAME]/Downloads/screenshots/*" in the file to be your username (in your case, its Administrator). If none of this works, try running the script with administrator privileges, as Windows Server can have issues with that. Also, there is a known bug I haven't bothered to fix where large Chegg links cause the bot to close the tab before it downloads, so that's another possibility.

can we talk private?

It works for 1 time then when next time it runs its like it doesn't take screenshot all permissions are given and also I have increased the sleeping time to 15 sec to avoid to close it before it downloads...
I think keypress might not be working if there is any alternative kindly update me thanks

Yeah if that's the issue then I'm not sure what to do about it, you could try using mouse position inputs to run the extension instead, but I can't be bothered to write that unfortunately.

This happens when the python script cannot find the screenshot file to upload. I would check to make sure that you followed all the directions, especially the stuff related to setting up the GoFullPage extension. Also, make sure you change the path = "C:/Users/[YOUR USERNAME]/Downloads/screenshots/*" in the file to be your username (in your case, its Administrator). If none of this works, try running the script with administrator privileges, as Windows Server can have issues with that. Also, there is a known bug I haven't bothered to fix where large Chegg links cause the bot to close the tab before it downloads, so that's another possibility.

can we talk private?

Sure if you want to

This happens when the python script cannot find the screenshot file to upload. I would check to make sure that you followed all the directions, especially the stuff related to setting up the GoFullPage extension. Also, make sure you change the path = "C:/Users/[YOUR USERNAME]/Downloads/screenshots/*" in the file to be your username (in your case, its Administrator). If none of this works, try running the script with administrator privileges, as Windows Server can have issues with that. Also, there is a known bug I haven't bothered to fix where large Chegg links cause the bot to close the tab before it downloads, so that's another possibility.

can we talk private?

Sure if you want to

email me your discord ID at<EMAIL_ADDRESS>


Update bazel-toolchains pin

This fixed Bazel CI failures (bazelbuild/bazel#6742) such as   https://buildkite.com/bazel/bazel-with-downstream-projects-bazel/builds/613#6483c40c-c77d-4f29-bcf8-a8dba45b1df0

Travis is red because it couldn't download stuff. I cannot imagine how this PR could break it, so i'll go ahead and submit. Please shout at me if I'm not following the process.

Since you invited this: Please don’t self merge a red PR. We never self merge in this repo. Someone else can merge if they agree. We almost never merge red CI (even when it is only a README change).

Yeah I was unsure about the policy, and I apologize.
So merging a green PR without review is acceptable, and merging red is not? That explains why the review requirement is not enforced by the github.
Thank you for bearing with me.

We rarely merge a PR without review. It should have a really good reason
and then it should be green, yes.
Don't know if we want to add the review requirement on github though.
Feels bureaucratic to me but maybe it's a good signal to other bazel people.
Thank you for the fix btw.
On Tue, Nov 27, 2018 at 6:42 PM Marcel Hlopko<EMAIL_ADDRESS>wrote:

Yeah I was unsure about the policy, and I apologize.
So merging a green PR without review is acceptable, and merging red is
not? That explains why the review requirement is not enforced by the github.
Thank you for bearing with me.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/rules_scala/pull/646#issuecomment-442129007,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABUIF-6FzLTbVK-gfdWVceVip8cJGAKNks5uzWuMgaJpZM4Y1hpx
.


Add analytics integration via SEGMENT. Requires JEKYLL_ENV=production|development -> Added plugin to support that

@jfromaniello :dancers:


https://audiomack.com/jordanryanmurray/song/title-set-up-by-judas

https://audiomack.com/jordanryanmurray/song/your-mother-loves-her-children [https://audiomack.com/jordanryanmurray/song/title-set-up-by-judas](https://audiomack.com/jordanryanmurray/song/title-set-up-by-judas) [https://audiomack.com/jordanryanmurray/song/title-set-up-by-judas](https://audiomack.com/jordanryanmurray/song/title-set-up-by-judas) Thank`` you 🙇♀ for wanting to create an issue in this repository. Before you do, please ensure you are filing the issue in the right place. Issues should only be opened on if the issue relates to code in this repository.

If you have found a security issue please submit it here
If you have questions about writing workflows or action files, then please visit the GitHub Community Forum's Actions Board
If you are having an issue or question about GitHub Actions then please contact customer support

If your issue is relevant to this repository, please delete this text and continue to create this issue. Thank you in advance.

@dewski @skw @chrispat @skw @DanRigby @Jordanboonemurray @Jordanboonemurray @

https://audiomack.com/jordanryanmurray/song/title-set-up-by-judas


Add MFEM_VERSION definition to _config.hpp and config.mk [mfem-version-dev]

In both places MFEM_VERSION is an integer, e.g. version "3.3" -> 30300.

Between releases, we can set the version to something like "3.3.99" <-> 30399.

One minor thing -- is it possible to add a function in make that extract the major, minor and patch numbers from the MFEM version.
I'd like make info to print 3.3.0 (30300) if possible... the missing initial 0 I think can confuse some folks.

@mlstowell and @aaroncblack do you have any objections to merge this?

I prefer using separate variables, like MFEM_VERSION_MAJOR, MFEM_VERSION_MINOR, and MFEM_VERSION_PATCH.  It's easier for an occasional contributor to quickly understand.

OK, I'll add definitions for those macros using MFEM_VERSION, e.g.
#define MFEM_VERSION_MAJOR ((MFEM_VERSION)/10000)


Swap chalk for turbocolor

Hello @fson, @jesseruder and maintainers! 👋
This is a PR for your consideration that swaps chalk for turbocolor.
It ought to give you a perf boost as turbocolor loads >20x faster and applies styles >18x faster than chalk for essentially the same API. Let me know if this looks good to you and if you'd like to accept my contribution.

# Load Time
chalk: 15.190ms
turbocolor: 0.777ms

# All Colors
chalk × 8,729 ops/sec
turbocolor × 158,383 ops/sec

# Chained Colors
chalk × 1,838 ops/sec
turbocolor × 39,830 ops/sec

# Nested Colors
chalk × 4,049 ops/sec
turbocolor × 59,833 ops/sec

You can find more about the benchmarks here. Cheers!

Thanks for the PR! The benchmark looks quite convincing. However, we'd have to ensure that all the formatting stays the same after this change. There are probably quite some corner cases that chalk needs to handle, e.g. with different terminals and Windows support. Do you have any ideas how to ensure that this doesn't break anything? Right now turbocolor only has 5 test cases while chalk has over 50.
Seeing that you're the author of turbocolor (nice work!), and your library seems to be a drop in replacement for chalk, have you considered running it though the test suite of chalk? That would give us more confidence that there are no corner cases that could differ between the libraries. Before merging this PR, it would also be nice to see a couple of screenshots comparing the output before and after the change, probably from Windows too.

@fson That's a reasonable request. Do you have a timeline for that? I could certainly look into testing it out on Windows, but I have to admit it will take me some time. 😅

I don't have a timeline for that, I just view having a good test coverage as prerequisite for switching, given that chalk has that and also receives more testing through being one of the most installed node libraries. There's no rush, we could look into this again when the testing situation for turbocolor improves in the future.

Thanks for contributing to exp! exp has been replaced by Expo CLI. We welcome contributions in the new Expo CLI repository.
Contributing to Expo CLI

👋 @fson Thanks for letting me know!


fix(tests): test component builds

Feature or Problem

After switching to Rust 1.79 we had a problem building test components used in the repository -- this PR fixes that by updating the build.rs script we were using, and refactoring it a bit.
Related Issues

Release Information

Consumer Impact

Testing

Unit Test(s)

Acceptance or Integration

Manual Verification

Hey @brooksmtownsend dismissing the review here so we can get these changes merged -- I think the only big thing was the possible implications on wash due to changed output of cargo but considering I'm running 1.79.0 (and many people probably are), I think it's probably fine -- if there's an issue we can definitely open a follow up


twitter usrnm add,some undesired files to exclude block and sitemap add

Added

README.md
CNAME
Makefile
Rakefile
package.json
learn-es2015.md
index.md
files to the exclude block
and added the twitter_username

created the sitemap.xml

@midhruvjaink the first line of your sitemap should be removed. Also might be nice to auto-indent it.

The sitemap has been updated and auto-indented now!

Thanks for your PR.
The sitemap should be dynamic in some way (for new blog posts, new plugins, presets, ...).
After a few research I found https://help.github.com/articles/sitemaps-for-github-pages. Would you mind removing your sitemap and including that?

hey @xtuc I have removed the static sitemap

Can we merge it?

hey @existentialism
all the build checks have passed
thanks for the edit!

@midhruvjaink thanks for the PR!

Thanks for the consideration


Release 1.14.0

Last 2.x release of PySAL

Coverage remained the same at 83.748% when pulling 5d3e44f08d7bb2eb30237fc7d8a751f9635862fc on sjsrey:release.1.14 into 0a84818a05c22d9a36e168b9156abeaf98e69a55 on pysal:dev.

Coverage remained the same at 83.748% when pulling a75cf8f97724e3469936e1f03684db495832ac25 on sjsrey:release.1.14 into 0a84818a05c22d9a36e168b9156abeaf98e69a55 on pysal:dev.


LICENSE.txt missing

Hey there,
just thought I'd let you know that you mention an Apache license being in LICENSE.txt, but there is no such file in your repository.
Cheers,
Felipe

Thanks for pointing this out!


Stop supporting Laravel 5

Dropping support for Laravel 5.* will clean up testing.

We might as well do this together with #80


Changing just the avatar of an Assistant does not permit saving

Saving doesn't ever seem to be disabled to me, how do I repro this ?


[BUGFIX][SW-18218] Fixed error using getId function with articleorder…

…number instead id

Description
This pull requests fixes the wrong data in meta array of order position in pdf document template.

Questions
Answers

BC breaks?
no

Tests pass?
yes

Related tickets?
https://issues.shopware.com/issues/SW-18218

How to test?
Add meta.ordernumber to PDF in document template and check if the ordernumber is the same as position

Looks duplicate oft https://github.com/shopware/shopware/pull/1019 :)

Yep, it's duplicate. Didn't found anything in issues.shopware.com with my used searchterms and created issue and fixed it afterwards


Update allowed formats for file uploads

This follows the change in 1.

What problem does the pull request solve?

Checklist

[x] I’ve used the pull request template
[ ] I’ve written unit tests for these changes
[ ] I’ve update the documentation (in DOCUMENATION.md and CHANGELOG.md)
[ ] I’ve bumped the version number (in src/Notify/Notify.csproj)

Closing for now, due to an issue with the new format: https://github.com/alphagov/document-download-api/pull/164


UUID pasted into the main text while only working in the Highlight mode

While only working in the Highlight mode and only dragging existing Statement tags into the create anchor modal, the full text gets distorted by putting anchors into the main text.
Happened to me already 3 times, so it is hardly just my error - seems to be a bug of the app.

Cannot replicate now, probably fixed by another commit


Required to provide scan packages

Description
It seems like that I have to provide at least one scan package for mongock to work. But in my case I don't want to use package scanning and instead just provide a list of migrations via addChangeLogClasses. Is there any way to disable that validation or am I doing something wrong?
PRIORITY
NORMAL
Version and environment
Mongock

Mongock version: 5.0.35
Modules involved: Standalone
How Mongock is used: Builder

Environment

Ktor 2.0
MacOS M1

Steps to Reproduce

Create a mongock instance         val mongoClient = MongoClients.create(createClientSettings(config));      return MongockStandalone.builder()          .setDriver(MongoSync4Driver.withDefaultLock(mongoClient, config.database))          .addMigrationClasses(config.migrations)          .addMigrationScanPackages(emptyList())          .buildRunner()
Run the code

Behaviour
Expected behavior: Not throwing any errors and only execute the provided list of migration classes
Actual behavior: Throws error: Scan package for changeLogs is not set: use appropriate setter
How often the bug happens: Everytime

Hello @fkrauthan ,
yeah, the changeUnits location is required. Initially it was set by the scan packager, but later it also accepts classes directly. It's probably a minor bug.
For the time being, until we fix it,  just pass and dummy String as package. It won't find anything there, but it should complain neither.


Support for items schema for arrays/hasmaps

What's this PR do?

add itemsSchema property for @OA\ArrayValue and @OA\HashmapValue
fixes #4

It's still need check if it's working also in validations - @AdamLutka ? :)


Repair filebrowser capability

The https://www.tiny.cloud/docs-4x/configure/file-image-upload/#file_browser_callback parameter was removed in TinyMCE 5 and replaced by https://www.tiny.cloud/docs/configure/file-image-upload/#file_picker_callback.
This breaks current django-tinymce filebrowser support.

Fixed by #316


Vocab and npz files format

Hello,
First of all, thanks so much for sharing the code. However, it is difficult to use the code as there are minimum information on how to prepare the data. I would appreciate it if you can share the format of vocab and npz files with us. As currently Im not able to run the code on a dataset similar to Yelp. I guess there may be something wrong with the format of my data.

Any help here?

I did not find any section for the preprocessing steps you taken. I appreciate it if you could share the format with us or at least share the preprocessing step you have used.

To make my question more clear, did you pass the document based on the position of the vocal in each document or the frequency, or frequency +normalization?

Hi,
Could you follow the example mentioned in the README about Wikitext-103 for data preparation? In particular, the script  examples/domains/wikitext103_wae.py has code to make vocab and npz files https://github.com/awslabs/w-lda/blob/master/examples/domains/wikitext103_wae.py#L177-L183

@fnan Thanks so much. I was trying to do similar thing but thanks for highlighting the script. I still have  not got it to work. By any chance do you have the script of preprocessing for the Yelp data? My data is similar to Yelp dataset I want to check which part I am doing it incorrectly.  I really appreciate it.

I think Im clear now where the problem was. Just wanted to make sure that I understood the code correctly. While preparing the wiki document you treat each line of each document as one of the element of the list (or each line is a document instead of the whole document itself) Am I correct? if so, did you do the same for other datasets?
Appreciate it if you can spend some time and clarify this for me. Thanks :).

Each element of the train_doc_list should be a whole document. In wikitext103, a document is broken into multiple lines in the input file. The token_list_per_doc function chunks the input lines into documents.


Problem using m/s in wind barbs

Tracker for matplotlib/matplotlib#4385

Closing since whether the matplotlib issue is a bug is unclear.

I am not sure if my question is related, but shouldn't skewt.plot_barbs() and station_plot.plot_barb() include conversion to knots before passing values to matplotlib.quiver.Barbs?

I don't know that we want to force folks into knots, but maybe a kwarg to specify a unit might be handy for users. As @dopplershift pointed out it would save converting u and v individually. Open an issue?

A kwarg can be a solution, yes. I think now the problem is that plot_barbs() function is not aware of units of wind speed, so if you pass winds in m/s, it will treat it as if they are in knots. But barb symbols are tied to units, so a plot with winds in m/s as arguments would look wrong.

For what it's worth, I've seen BM used in GEMPAK (see this page), and used it myself on occasion. It's good to know now about what WMO has published. But, would it still be worth having the option to do the non-standard m/s-based barbs?

@jthielen what do you mean by non-standard barbs? According to WMO (thanks, @akrherz),

Wind is represented by barbs and solid pennants in black, the full barbs representing 5 m s -1 or 10 knots, the half barbs representing 2.5 m s -1 or 5 knots and the solid pennant representing 25 m s -1 or 50 knots.
So it should matter what the input is. The barbs are like a different kind of units that you need to convert your data to.

@dennissergeev - I think what @jthielen is getting at is there are situations in which one might plot barbs in units of m/s. Not WMO compliant, but following the Python idea of not guessing in the face of ambiguity, I'm worried about changing units by default. It's also a API break we'd need to warn on. (I believe @dopplershift was worried about that when we chatted last night about this.)

@jrleeman I see your point. Then a kwarg option is the best. Or at least updating the docstring.

I think the kwarg option is a good addition. I don't think it's worth breaking backwards compatibility for this, and I don't think we should be forcing pre-conceived notions. In my experience, I'm less thinking of m/s than of just using mph. Either way, that should be up to the user, not us.


Support for internationali

Support for internationali

Plus one on this
If you can just adapt the plugin and provide the english.js localization we can move forward from that


Dataset creation error: Path must be unique

Hi, I am trying to create a dataset but I ran into this error.
kaggle datasets create -p molatTest2Stripped --dir-mode zip
It uploads the files with success prompts but then outputs
Dataset creation error: Path must be unique
What does this mean?
here's a sample dataset to produce this error.
https://drive.google.com/file/d/1zW-ncPAJSEICbui5tlSKMp_EpkfVwt2h/view?usp=sharing

Sorry for the very late answer, would it be possible for you to share the sample dataset again?

I was getting this error when the folder was like this (I used Collab) :
/content/Data
---->file1.csv
---->file2.csv
---->dataset-metadata.json
etc.......
and the Command was :
!kaggle datasets create -p /content/Data/
But when I compressed all the files and made them into one file(files.zip)
then the folder becomes like this :
/content/Data
---->files.zip
---->dataset-metadata.json
the files were uploaded successfully and the link of dataset appeared
This is my attempt, maybe it will help you


Move markers to rewrite-gradle

Will pass when rewrite build is complete from commit https://github.com/openrewrite/rewrite/commit/3bd8271be963c025ce0856072d0f09360e3184cd


Is fave-button's size fixed?

It seems that I cannot change the size of the fave-button. Is it possible to make the size the same as the imageView's size?

Thanks @muratyasarr, If you're using storyboard to place a button, the size of the fave-button is based on the size you specify for it, for manual creation is based on the frame you pass to it at creation. Yo can see that in the demo app, where bottom and center buttons have different dimensions.


Syntax Error

Summary
I'm following the steps in your README to setup Stripe elements in my React app but it won't compile.
./node_modules/react-stripe-elements/src/components/Element.js
SyntaxError: <mypath>/node_modules/react-stripe-elements/src/components/Element.js: Unexpected token, expected "," (5:13)

  3 | import PropTypes from 'prop-types';
  4 | import isEqual from '../utils/isEqual';
 >5 | import {type ElementContext, elementContextTypes} from './Elements';
    |              ^
  6 |
  7 | type Props = {
  8 |   id?: string,

Obviously I'm missing some dependency or I'm on a wrong version of React. So it seems the guide in your README is not complete with all required steps. What am I missing here?

Can you share what your code looks like (specifically, where you are importing components from react-stripe-elements)? Also, does your build tool attempt to compile files within node_modules?
This looks like an issue with your build tool attempting to do so, and being unable to parse the type ElementContext import, which is for the Flow type system used internally by this project.

I've set my project up with create-react-app. I didn't change any of the advanced stuff around building. I've also not had this problem with any other library yet. But it sounds very likely that it's trying to compile the node_modules.
I will look into if that's what's going on and if so how to turn it off.

Ok! If you can create a repo with instructions on a simple reproduction, happy to help debug.

Ah I figured it out.  My IDE added an import
import injectStripe from "react-stripe-elements/src";
instead of
`import injectStripe from "react-stripe-elements";
Sorry for wasting your time there.


Basic website setup with short description of myself

Added a picture, a button and a short description of myself

Feel free to merge these changes to main.


⚠ Fetchers has degraded performance

In 07a41b4, Fetchers ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 253 ms

Resolved: Fetchers performance has improved in f7acd22.


In Default ErrorView render by yii\web\ErrorAction ，the Yii::$app->response->statusCode is not correct

What steps will reproduce the problem?
use  Composer to initialize  a clean Yii2 advanced app template
in frontend/views/site/error.php   add the following line
<?= Yii::$app->response->statusCode?>

then, navigate to a none-existing Url Location in Browser,such as http://yaa-frontend.dev/xxx
What is the expected result?
the above PHP code added to   frontend/views/site/error.php    should output 404
What do you get instead?
the  above PHP code added to   frontend/views/site/error.php    just output 200  in page content
this is not correct
Note: HTTP code in Browser is 404, this is correct.

Additional info

Q
A

Yii version
<IP_ADDRESS>

PHP version
PHP 7.1

Operating system
Ubuntu 16.04 LTS

Reproduced with latest code from master branch, it's indeed a bug.

Fixed. Preparing pull request.

@samdark  Thanks  for  your effort

@ihipop Related PR is not merged yet. We'll close it when it will be finished.


Add Indonesian language selection ability.

Description
Add Indonesian(Bahasa) language selection ability to the interface. I'll raise a PR for this.

this was actually closed in #863


Plugins are triggered and previous context is lost on error shown

Environment

Operating System: Linux
Node Version:     v20.10.0
Nuxt Version:     3.9.3
CLI Version:      3.10.0
Nitro Version:    2.8.1
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      devtools, modules
Runtime Modules<EMAIL_ADDRESS>Build Modules:    -

Reproduction
https://github.com/daniluk4000/nuxt-error-reset-context
Describe the bug
After showError is used, it almost feels like whole page is reloaded:

All previous pinia data is lost
All plugins are executed once again

Additional context
God please tell me this is not intended. This leads to duplicate requests and logs lose on our side.
Logs
Inside console

Triggered //Initial plugin trigger
bar //Initial pinia data has been set
Triggered //Second plugin trigger after showError. Pinia data will be lost then

Basically, any thrown error inside defineRenderHandler leads to this.

As I've trugged deep into the code, the problem is probably that we can't rerender our app if any error happenened.
The only thing that could delay some component from rendering is onServerPrefetch, but we can't wait inside of it forever since we can't know if remaining app has been rendered or not.
Solution could be is to listen for "@resolve" hook for that, but this is currently impossible due to https://github.com/vuejs/core/pull/8555 not merged.
Also solution could be to render app twice (don't use Nitro's error handler) and on second render pass all the payload and state to this second render, and that will also skip mounting any plugins twice there.

Yes, that is the case (and intentional). To render an error page, we render the app a second time, but just the error page and not app.vue.
Plugins might do more things than set payload/state, such as inject helper functions. So we do need to rerun them. You can add your own logic to avoid rerunning logic. You can check in your plugins if useError().value is set.

Yes, that is the case (and intentional). To render an error page, we render the app a second time, but just the error page and not app.vue.
Plugins might do more things than set payload/state, such as inject helper functions. So we do need to rerun them. You can add your own logic to avoid rerunning logic. You can check in your plugins if useError().value is set.

Yes I could, but sadly pinia context is lost as well :(

@danielroe do you think this worth trying to solve when @resolve will be supported on SSR? Basically, that's how it worked in Nuxt 2 (I know that there was totally different logic).
If at least state (including useState & pinia) would be passed to this reloaded context, it wouldn't be such a problem.


Use {} for if statement (even if block is one line)

Summary of the changes / Why this improves CrateDB
Checklist

[ ] User relevant changes are recorded in CHANGES.txt
[ ] Touched code is covered by tests
[ ] Documentation has been updated if necessary
[ ] CLA is signed

Can we enable a checkstyle rule for this?

Can we enable a checkstyle rule for this?

We can, but I'm not sure if we should as most of the idea generated equals methods will fail to pass the rule. Which might be something we can live with, but I'm not entirely sure if we should do it.

I think we should decide to not allow that by a checkrule before we change it for some parts of the code by dedicated commits. Otherwise it would be only partly implement which I don't see a point in doing. Personally I'm even fine having one-liner without it.

I think we should decide to not allow that by a checkrule before we change it for some parts of the code by dedicated commits. Otherwise it would be only partly implement which I don't see a point in doing. Personally I'm even fine having one-liner without it.

I think it can be a source of bugs and I don't see how fixing individual instances at a time can hurt (as fixing every occurrence implies deeper analysis due to generated code). Given the "severity and scale" of this change I'm also fine with dropping it

I think we should either enforce it or not bother about it. Otherwise we end up having style discussions on PRs which I think we should avoid.
If the intellij generated code is a problem we could probably check-in some adapted templates? We already have codeStyle stuff in the idea folder anyway.


can't connect to rds database - possibly incorrect database.json file

Not sure what I'm doing wrong here, but I keep getting an error
Error: getaddrinfo ENOTFOUND [rds.server.address]
I've been trying to look through your documentation and code to find how the database.json should be structured, and I've tried to follow what I've found.
My database.json file is (password and addresses have been changed)
{
	"dev": {
		"url": "postgres://USER:PASS@ENDPOINT_ADDRESS:PORT/DB_NAME",
		"host": "ENDPOINT_WITH:PORT",
		"user": "USER",
		"password": "PASSWORD",
		"database":"DB_NAME",
		"schema": "public",
		"driver": "pg"
	}
}

I've tried looking through the documentation of parse-database-url as well, but no documentation there is pointing to the issue either.
Going through the config.js file, I can see that the database.json seems to be parsed correctly, but it just refuses to get a connection.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/10257418-can-t-connect-to-rds-database-possibly-incorrect-database-json-file?utm_campaign=plugin&utm_content=tracker%2F73887&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F73887&utm_medium=issues&utm_source=github).

You should whether use url or host,user,password,database but not both.
See this for an example configuration:
http://db-migrate.readthedocs.org/en/v0.9.x/Getting Started/configuration/

thanks wzrdtales, you are correct that trying both url and host results in some... strange behaviour.
What I'm seeing is that if I use host, user, password, database, the connection object is correct, but the getaddrinfo error, which I've been unable to pin down.
If I use URL, the object that gets passed to pg.connect is my localhost details, so it appears url is not being picked up at all. If I use just a string, the string is parsed into the correct database object, but I still get the getaddrinfo error.
I'm using the same database configuration in a kraken.js app also connecting through pg, and that connects without issue. I've been trying to debug what's going wrong, or why it would be different with db-migrate, but it shouldn't be.
I've decided to fork db-migrate and see if I can get that working with kraken, so I can use the kraken config. That may point out the issue, in which case I'll post back here.
Thanks for your help.

Thank you for investigating on this, if you need any further help don't hesitate to ask.

@pedalpete To note: If you discover the root of your issue and a solution to it, I would appreciate if you would open a pull request to https://github.com/db-migrate/english-docs and document this.
Thank you

@pedalpete any progress on this?

I don't know if its related but I'm having similar issues.
$ ./node_modules/.bin/db-migrate --env staging --dry-run --verbose --config config/database.json up
[INFO] dry run
[INFO] Using staging settings: { driver: 'pg',
  host: 'localhost:5433',
  port: '5433',
  database: 'adatabase',
  user: 'auser',
  password: '******' }
[INFO] require: ./pg
[INFO] connecting
[INFO] connected
[SQL] select version() as version
select version() as version
[ERROR] Error: getaddrinfo ENOTFOUND
    at errnoException (dns.js:37:11)
    at Object.onanswer [as oncomplete] (dns.js:124:16)

I was debugging it some and noticed that index.connect in executeUp works
fine but some time after calling migrator.driver.createMigrationsTable the
issue arises
Setting a breakpoint in dns.js (sb('dns.js', 37)) showed that the host it
couldn't resolve was localhost:5433, which seems to suggest the issue is
related to specyfing the port.

host: 'localhost:5433',
  port: '5433',

Why do you specify the port twice?

@ecksun This is possibly your issue, the host is only the host. The way you defined your configuration applies to connection strings, but in this case you're using parameters.
Also take look at the parameter list of pg.
https://github.com/brianc/node-postgres/wiki/Client#new-clientobject-config--client
So just config like the following:

host: this is only the hostname/domain/ip
port: here you specify your port

The port field was added later when I started to try to find a workaround to the issue, before I only had specified the host field ("localhost:5433").
I never realized that the parameter list is just passed through to node-postgres, I don't know if it's because I missed that part of the documentation or if its simply missing but I do think it might be something we could improve upon.

I'm always open to improve our documentation though :)
Does it works now for you or do you still have issues?

Yes, it works when I specify the port separately.
I would suggest you quickly write about how the config parameters are used and
what they are in the
[configuration](https://github.com/db-migrate/node-db-migrate/#configuration
section).
And if it is so that the configs are only passed through, just mentioning that
should be more than good enough (doesnt have to be more than a sentence
really).
I would send a PR if I knew better how the configs were dealt with :(

Ok perfect, I guess I will rewrite this documentation part and separate simple examples and structurize the in depth configurations.


allow type parameters on values other than classes and functions

i can do this:
function getter<T extends { value: U}, U>(data: T): U {
    return data.value;
}

but i cannot do this:
interface Accessor<T extends { value: U; }, U> {
    get(data: T): U;
    set(data: T, value: U): T;
}

const accessor: Accessor = { // <-- error, requires type arguments that i don't have yet
    get: data => data.value,
    set: (data, value) => ({ ... data, value })
}

i wish i could do something like this:
const accessor<T extends { value: U; }, U>: Accessor<T, U> = {
    get: data => data.value,
    set: (data, value) => ({ ... data, value })
}

consider allowing type parameters on values other than functions and classes

then how would T and U be known when accessor is used? do you have to specify accessor<number, string> or you imagine some sort of inference taking place? if so how?

just in the exact same way as it is for functions:

yes, one way it to implicitly set them accessor<string, number>.get(xxx)
yes, another way is to let inference guess: takeAccessor(accessor)

just like with the functions where type arguments are required on each call, the arguments for values need to be provided or inferred at each reading
so yes it needs to happen on every use

in fact at the moment i use functions to do exactly that:
function alwaysAccesorOf<T, U>(): Accessor<T, U> {
   return {
      get: data => data.value,
      set: (data, value) => ({ ...data, value })
   };
}

alwaysAccessorOf<string, number>().get(xxx) // <-- i wish i could avoid a function call just
                                            // for the sake of providing type arguments
                                            // and simply do:

accessor<string, numbe>.get(xxx);

I run into this fairly often.
Related issues:
https://github.com/Microsoft/TypeScript/issues/19728
https://github.com/Microsoft/TypeScript/issues/17574

also related to https://github.com/Microsoft/TypeScript/issues/15877

Tracking at #17574


exclude tests/data from pre-commit

this is significantly impacting performance of pre-commit.ci and has set off the abuse detection -- figured I'd spare the repo instead :)

Oh, gosh, thanks! I though I had those excluded+


Consider side-effect protection

If anything in the pipeline touches e.g. filesystem, destructive changes can be inadvertently executed. I think it would be a nice feature to execute in a "protected" environment by default, although this won't be easy to achieve of course.
One can attempt to intercept syscalls using ptrace or other debugging interfaces in the subprocess, but this is incredibly messy. A cleaner approach might be to execute in a container with root fs mounted from outside and an overlayfs on top preventing any write access.
This is tricky and is guaranteed to have potential holes; we can never support 100% destruction-less environment while simultaneously exposing real OS. Maybe we can protect the FS but that's not subset of syscalls that would allow for screw-ups: signals, shared memory, network and many more would need to be addressed and i'm sure there'd be more things that could be missed.
Is this even worth considering given these complications? I immediately wanted to implement this since I often iterate on pipelines that manipulate file collections, but I realize now FS is just a part of it, and providing some protection may be worse than just being honest with user and letting them shoot themselves in the foot.

I get what you're saying, but I don't think this is a direction I want to take things -- my goal is to be as close to the normal shell experience as possible. In fact I think in my ideal world this would be a built-in feature of shells, rather than a separate utility. Basically I want this to be just the same as working out a long pipeline and piping it to less for testing, just with reduced friction.
Since you have to be aware of doing destructive operations while doing that kind of less experimentation, I think that it's reasonable to have the same expectation here.
The idea of doing that kind of experimentation in a "safe" container environment does have some appeal to me in some situations, I like that suggestion, but I don't think that needs to be a feature of pipeline -- it sounds like something separate.

Yes that makes sense, after some consideration I came to the same
conclusion. Isolation tools already exist and it’s up to the user to decide
to use them. I will experiment with it a bit and if there’s a cool but easy
way to integrate via, say an alias that spawns pipeline under docker or
some such, maybe you can put that in the wiki but that’s not something that
needs to be part of the pipeline tool itself.
On Fri, 10 Jan 2020 at 17:23, Brian Palmer<EMAIL_ADDRESS>wrote:

I get what you're saying, but I don't think this is a direction I want to
take things -- my goal is to be as close to the normal shell experience as
possible. In fact I think in my ideal world this would be a built-in
feature of shells, rather than a separate utility. Basically I want this to
be just the same as working out a long pipeline and piping it to less for
testing, just with reduced friction.
Since you have to be aware of doing destructive operations while doing
that kind of less experimentation, I think that it's reasonable to have
the same expectation here.
The idea of doing that kind of experimentation in a "safe" container
environment does have some appeal to me in some situations, I like that
suggestion, but I don't think that needs to be a feature of pipeline --
it sounds like something separate.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/codekitchen/pipeline/issues/6?email_source=notifications&email_token=AAQNHTSL34MTHZW7KP6ZTY3Q5CHH3A5CNFSM4KFA55D2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIUHYSI#issuecomment-573078601,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQNHTS57ADKOOQZV57FS43Q5CHH3ANCNFSM4KFA55DQ
.

--
The best the little guy can do is what
the little guy does right

I'm going to close this ticket but if you end up coming up with something that'd be cool, send me over a PR


eliminate redundant wires/copies

Chisel generated RTL has a lot copies, e.g.:
  assign RRTestCombinational_79_1_io_rready = _T_1512;
  assign RRTestCombinational_79_1_io_wvalid = _T_1515;
  assign RRTestCombinational_79_1_io_wdata = _T_604;
  assign _T_168_ready = _T_2351;
  assign _T_168_valid = _T_3492;
  assign _T_168_bits_read = _T_3484;
  assign _T_168_bits_index = _T_3486;
  assign _T_168_bits_data = io_in_0_hwdata;
  assign _T_168_bits_mask = _T_3488;
  assign _T_168_bits_extra = 1'h0;
  assign _T_202_ready = 1'h1;
  assign _T_202_valid = _T_2354;
  assign _T_202_bits_read = _T_238_bits_read;
  assign _T_202_bits_data = _T_3478;
  assign _T_202_bits_extra = _T_238_bits_extra;
  assign _T_238_ready = _T_2353;
  assign _T_238_valid = _T_2352;
  assign _T_238_bits_read = _T_168_bits_read;
  assign _T_238_bits_index = _T_168_bits_index;
  assign _T_238_bits_data = _T_168_bits_data;
  assign _T_238_bits_mask = _T_168_bits_mask;
  assign _T_238_bits_extra = _T_168_bits_extra;

This makes RTL larger (slower to compile, simulate with waveform, etc.), more difficult to debug RTL (e.g. trace signal drivers in DVE).
Also, code coverage metrics can be skewed (not sure how coverage tools deal with this).
It should be possible to replace module-internal copies, .e.g. replace _T_2352 with _T_238_valid in the code above and get rid of _T_2352.
_T_2352 can be put in comments if it is needed for whatever reason (e.g. debugging):
  wire _T_238_valid; // _T_2352

Another suggestion is to generate wire assignments to reduce generated code size, i.e.:
  wire AB = A & B;
  wire CD = C & D;
  wire O = AB | CD;
  wire F = ~O;

instead of:
  wire F;
  wire AB, CD, O;
  assign AB = A & B;
  assign CD = C & D;
  assign O = AB | CD;
  assign F = ~O;

Can you submit the chisel code that generates this?

^ It's not a bug, b/c the output is legal but it's really bad QOR. I'm pretty sure almost all sufficiently large Chisel code generates something like this (i.e. uses assign [nothing to do w/ large designs; would minimize # of lines if you just did the wire thing] + "complicated" optimizations seem to throw in random gen_1 = gen_2 blah blah). Makes visual inspection (for the purposes of debugging anything -- even if you say it's not the point to have readable Verilog) a PITA, and if it's true that it significantly affects tool run times/coverage, that's even worse! I imagine it's probably one of those annoying optimization things that people didn't really want to do, b/c it's just... really annoying and doesn't hurt correctness.

Yes, I agree with @shunshou - it is not a bug - it is an obvious optimization.
Take any large Chisel RTL, generate Verilog and you'll find plenty of examples of useless T wire copies.

In that regard, Chisel3 hasn't changed very much from Chisel2 ;)

Actually a firrtl issue, ucb-bar/firrtl#414


python312Packages.cryptography: 42.0.5 -> 42.0.8

Description of changes
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

https://cryptography.io/en/latest/changelog/#v42-0-8
Looks useless to me.

https://github.com/pyca/cryptography/compare/42.0.5...42.0.8
Oh well.

Having to explain over and over why we don't have an update is equally annoying. Just throw it low importance into staging and it will bubble through it.


Error when using gloo as DDP backend

Hello! Thank you for your great work on implementing VQ layer. When I use the VQ layer in DDP mode and use gloo as the backend as suggested in README, I got the following error:
terminate called after throwing an instance of 'gloo::EnforceNotMet'                                                                                what():  [enforce fail at ../third_party/gloo/gloo/transport/tcp/pair.cc:510] op.preamble.length <= op.nbytes. 8773632 vs 8386560
Do you have any ideas on how to solve this problem?
I also tried to use nccl as the backend, however the program only hangs forever...

@Saltychtao Hey! So I recently fixed a bug with distributed training (and yet another one this morning in regards to nccl)
What version are you on? Could you retry on the latest version, and if that doesn't work, send me a script for reproducing the error?

@lucidrains Hello, I have tried the newest version of the code, however it still hangs on when using nccl.  Besides, I am currently using your vector-quantize-pytorch library with an complex seq2seq framework fairseq, so it will take me some  time to isolate an working script that could reproduce the error. Anyway, I will send it to you as soon as possible.

I am using this library in fairseq, and it turns that the all_reduce function from torch.distributed  used in the library is not compatible with fairseq.  When using functions from fairseq.distributed_utils, everything works well!


Added separate commands for opening and closing the outline window

Just a small change to add separate commands for opening and closing the outline window.
This is helpful for working with plugins like panelmanager.vim.

Thanks for the PR !


Add support for solc 0.8.27

https://github.com/ethereum/solidity/releases/tag/v0.8.27
Cc: @Evalir, @mattsse

blocked by #8771


Need to monitor the compatibility issues and add the approved project to the compatible website

The certification requests are here . We need to monitor that and get the request added to the compatible page of microprofile.io once they are approved according to the MicroProfile TCK process.

Can we add this step to the current Github Issue template?

https://github.com/eclipse/microprofile-marketing/issues/690

The new wiki page has been added to the https://microprofile.io/compatible/
Thank you @rstjames for that step and Emily for enabling a draft review! :)


namespace sync conundrum

AEM 6.1
grabbit 7.1.1
I have two AEM 6.1 systems that unfortunately have a namespace conflict.  The same URI is associated with different prefixes.
Does it make any sense to add an option to make namespace sync optional in the configuration file?  Please be gentle - I'm not a developer :-)
Interesting note - I was expecting (hoping?) that package installation would fail due to the namespace conflict but it didn't.  Installing a content package that contains this type of namespace conflict results in the prefix changing to the one already defined on the receiving system.  Prior to using grabbit we were transferring content with packages.  It wasn't until we started using grabbit that we noticed a namespace conflict.

It's an interesting scenario. It's likely the URIs provide the same definition for the time being. This is probably the only scenario where an optional namespace sync would bare fruit - typically, if there was a mismatch with mismatched definitions, or a missing namespace - nodes/properties would fail to write (in, and outside of Grabbit).
I would say, I'd probably look to keep the namespace prefix URIs consistent so that it doesn't bite later on.

To clarify - in this scenario, the grabbit transfer DOES fail with error "OakName0001: Invalid namespace prefix".
In my opinion it would be much better if the package installation also failed, but it doesn't -- installing the same exact content via package succeeds.  The namespace conflict seems to be automagically resolved during package installation by changing the conflicting prefix to the prefix with the same URI that already exists.  Replication also behaves this same way.
Agree 100% about keeping namespace consistent -- unfortunately we are using grabbit to sync with a different service provider's AEM that we don't control.

Hi @himmelmrm - its weird that you are getting an exception when doing this. There is a bit in the code for this scenario to "ignore" those cases where "mapping for incoming prefix already exists".
https://github.com/TWCable/grabbit/blob/master/src/main/groovy/com/twcable/grabbit/client/batch/steps/namespace/NamespaceSyncTasklet.groovy#L88-L95
As you can see, Grabbit is explicitly checking for that case. Also, the NamespaceHelper that the code relies on is supposed to automatically do the same thing that the Package Manager does. I am wondering whether something has changed in Oak for AEM 6.1 that is may be throwing a different exception in this case.
If possible, could you provide some more information of the stacktrace you are getting? I am wondering whether the invalid namespace prefix error is due to a different issue.
Making the namespace sync optional is not too hard, I am wondering if its really necessary, or should we update the code to handle this scenario better.
cc @jbornemann

The namespace sync occurs at the start of the job:
28.11.2017 11:38:35.624 INFO [clientJobLauncherTaskExecutor-3] com.twcable.grabbit.client.batch.steps.namespace.NamespaceSyncTasklet Start writing namespaces.
28.11.2017 11:38:35.635 INFO [clientJobLauncherTaskExecutor-3] com.twcable.grabbit.client.batch.steps.namespace.NamespaceSyncTasklet Finished writing namespaces.
... when the sync job reaches the first node that has the conflicting namespace, the attached stacktrace is logged and the job terminates with a status of FAILED.
grabbit-stacktrace.txt


⚠ Bofrost.es has degraded performance

In b83b715, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1548 ms

Resolved: Bofrost.es performance has improved in af8272f after 6 minutes.


How to use hook to get the validation loss?

Recently, I have implemented a simple method that wants to get the validation loss. The code below
@HOOKS.register_module()
class MyHook(Hook):
    def __init__(self):
        pass

    def after_train_epoch(self, runner) -> None:
        model = runner.model
        model.eval()

        for i, data in enumerate(runner.val_dataloader):
            outputs = runner.model.train_step(data, runner.optim_wrapper)

Although, it can correctly load the validation data, I get this error message
Traceback (most recent call last):
  File "/home/miislab-server2/Heng/Heng_shared/AOI-Project/model_zoo/mmdetection/tools/train.py", line 122, in <module>
    main()
  File "/home/miislab-server2/Heng/Heng_shared/AOI-Project/model_zoo/mmdetection/tools/train.py", line 118, in main
    runner.train()
  File "/home/miislab-server2/Heng/envs/AOI/lib/python3.8/site-packages/mmengine/runner/runner.py", line 1745, in train
    self.call_hook('before_run')
  File "/home/miislab-server2/Heng/envs/AOI/lib/python3.8/site-packages/mmengine/runner/runner.py", line 1839, in call_hook
    getattr(hook, fn_name)(self, **kwargs)
  File "/home/miislab-server2/Heng/Heng_shared/AOI-Project/model_zoo/mmdetection/mmdet/engine/hooks/my_hook.py", line 20, in before_run
    outputs = runner.model.train_step(data, runner.optim_wrapper)
  File "/home/miislab-server2/Heng/envs/AOI/lib/python3.8/site-packages/mmengine/model/base_model/base_model.py", line 113, in train_step
    data = self.data_preprocessor(data, True)
  File "/home/miislab-server2/Heng/envs/AOI/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/miislab-server2/Heng/envs/AOI/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1527, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/miislab-server2/Heng/Heng_shared/AOI-Project/model_zoo/mmdetection/mmdet/models/data_preprocessors/data_preprocessor.py", line 140, in forward
    self.pad_gt_masks(data_samples)
  File "/home/miislab-server2/Heng/Heng_shared/AOI-Project/model_zoo/mmdetection/mmdet/models/data_preprocessors/data_preprocessor.py", line 191, in pad_gt_masks
    data_samples.gt_instances.masks = masks.pad(
  File "/home/miislab-server2/Heng/Heng_shared/AOI-Project/model_zoo/mmdetection/mmdet/structures/mask/structures.py", line 346, in pad
    return BitmapMasks(padded_masks, *out_shape)
  File "/home/miislab-server2/Heng/Heng_shared/AOI-Project/model_zoo/mmdetection/mmdet/structures/mask/structures.py", line 268, in __init__
    self.masks = np.stack(masks).reshape(-1, height, width)
ValueError: cannot reshape array of size 12582912 into shape (704,1024)

So, I printed the height and width, I found that it would change every time.
But when I try to run the below code it can work correctly
@HOOKS.register_module()
class MyHook(Hook):
    def __init__(self):
        pass

    def after_train_epoch(self, runner) -> None:
        model = runner.model
        model.eval()

        for i, data in enumerate(runner.train_dataloader):
            outputs = runner.model.train_step(data, runner.optim_wrapper)

Could someone help me with that, I will appreciate.

@g824718114 Thank you for providing the method, but may I ask for some modifications to the test_dataloader in the configuration file
Collate_fn=dict (type='yolov5_collate ')
Where is this defined? Because I haven't used YOLOv5, or is this a universal modification method?

I'm sorry that I didn't make it clear. This is not a universal modification. The collate_fn in test_dataloader need to be the same as train_dataloader.
---Original---
From: @.>
Date: Tue, Jun 11, 2024 09:42 AM
To: @.>;
Cc: "Guo<EMAIL_ADDRESS>Subject: Re: [open-mmlab/mmdetection] How to use hook to get the validationloss? (Issue #11331)
@g824718114 Thank you for providing the method, but may I ask for some modifications to the test_dataloader in the configuration file
Collate_fn=dict (type='yolov5_collate ')
Where is this defined? Because I haven't used YOLOv5, or is this a universal modification method?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>


trying to achieve Right to Left CarouselLayoutManager

Can anyone help to achieve like the following? I have tried and could not make it.
Please help. Thanks in advance.

Hi @thetnaingmizo
This is not a part of the lib. You can look how this library is done inside and change its sources to achieve this sort of effects. You will need to change: generateLayoutOrder or fillChildItem. The goal here is to change current library elevation.


docs: add more adopters of Dragonfly

Signed-off-by: Allen Sun<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
According to issue "Wanted: Who is using Dragonfly | 如果贵司正在使用Dragonfly请留言", we have found that more and more enterprises are using Dragonfly. So this PR tries to add them as adopters.

Enterprise
detailed link
Author

Huawei
https://github.com/dragonflyoss/Dragonfly/issues/219#issuecomment-483990997
@ClownBin

JFrog
https://github.com/dragonflyoss/Dragonfly/issues/219#issuecomment-453917023
@hpejustin

ZTE
https://github.com/dragonflyoss/Dragonfly/issues/219#issuecomment-467680315
@xq2248

sunus
https://github.com/dragonflyoss/Dragonfly/issues/219#issuecomment-475845644
@anjia0532

If you are also using Dragonfly, Please do not hesitate to tell us. And we are looking forward to adding you into the adopters list.
Ⅱ. Does this pull request fix one issue?

none
Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗？)
no need
Ⅳ. Describe how to verify it
none
Ⅴ. Special notes for reviews
none

LGTM.


Client error

Seems to be fixed with https://github.com/peterwilli/Endless-AWSW/commit/7557181067cf66ccc9a6f14c67c8f42b21bc2031


Made menus narrower to save screen real estate

did you try with the stdlib menus? Some names there get pretty long

Not sure about this, narrower = smaller click target, more difficult to navigate. We have literally nothing else in that vertical space, so all space under the menu is wasted. How about a dynamic CSS rule based on the display height?

did you try with the stdlib menus? Some names there get pretty long

Yes, it looks fine for me.

@CyberShadow: I think I was wrong, that doesn't reduce the click target, only the padding.

Not sure about this, narrower = smaller click target, more difficult to navigate. We have literally nothing else in that vertical space, so all space under the menu is wasted. How about a dynamic CSS rule based on the display height?

The library reference requires too much scrolling, so I though the narrow menus would be a good idea.

The library reference requires too much scrolling, so I though the narrow menus would be a good idea.

That's true. But why not just apply this to the library reference?

That's true. But why not just apply this to the library reference?

I think that would look weird because the menus would be inconsistent with the rest of the site.

There is already some inconsistency, the module list uses a monotype font.

Oh oh... s/narrower/shorter/ and I understood everything


Priority: align_multiline_comment should run before phpdoc_summary

The PHP version you are using ($ php -v):
PHP 7.1.18 (cli) (built: May 25 2018 19:18:59) ( NTS )
Copyright (c) 1997-2018 The PHP Group
Zend Engine v3.1.0, Copyright (c) 1998-2018 Zend Technologies
    with Zend OPcache v7.1.18, Copyright (c) 1999-2018, by Zend Technologies

PHP CS Fixer version you are using ($ php-cs-fixer -V):
PHP CS Fixer 2.12.1 Long Journey by Fabien Potencier and Dariusz Ruminski
The command you use to run PHP CS Fixer:
vendor/bin/php-cs-fixer fix -v --using-cache=no
The configuration file you are using, if any:
Basically the same as dmvdbrugge/dynamic-components, except explicit_string_variable, header_comment, no_alternative_syntax, and no_unset_on_property
If applicable, please provide minimum samples of PHP code (as plain text, not screenshots):

before running PHP CS Fixer (no changes):

/**
 * {
"csp-report": {
"document-uri": "http://example.com/signup.html",
"referrer": "http://evil.example.net/haxor.html",
"blocked-uri": "http://evil.example.net/injected.png",
"violated-directive": "img-src *.example.com",
"original-policy": "default-src 'self'; img-src 'self' *.example.com; report-uri /_/csp-reports",
}
}
 */
function doSomethingWithCspReports()
{
}

after first run:

/**
 * {.
 * "csp-report": {
 * "document-uri": "http://example.com/signup.html",
 * "referrer": "http://evil.example.net/haxor.html",
 * "blocked-uri": "http://evil.example.net/injected.png",
 * "violated-directive": "img-src *.example.com",
 * "original-policy": "default-src 'self'; img-src 'self' *.example.com; report-uri /_/csp-reports",
 * }
 * }
 */
function doSomethingWithCspReports()
{
}

after second run:

--- Original
+++ New
@@ @@
  * }
- * }
+ * }.
  */
 function doSomethingWithCspReports()
 {
 }

expected:
Immediately arrive at the last step

Meta: this is case 36 of #3844

This issue is superseded by #3879


Synchronize issues with Github

Is there a way to synchronize issues with Github?
I write a little php script that reads gogs database and calls Github issue API to create it. I think this feature should be a part of the issue creation in gogs.
Another way, user should querying Github issues and create/update its in Gogs

No there's no way to synchronize issue with Github at the moment. Only way to do it is manually like you're doing now.
For the Feature-Request part, I'd argue that it'd be more useful to implement webhooks for issues, since otherwise Gogs would have to implement a new 'issue-provider' everytime someone wants to use YetAnotherIssueTracker (I'm fairly certain that that exists btw :laughing:). And you'd be able to use you're current php-script with very little changes :wink:
Personally, I use BitBuckets integrated issue-tracker, Trello (not really a 1:1-sync...), Github issues and Asana. So already we're at 4 different providers and I could keep listing more :unamused:
Having issue-tracking exposed on a webhook would also allow for Slack-integration and similar services without having to write YetAnotherServiceIntegration.
All-in-All, Webhooks rules, and hardcoded integrations sucks :stuck_out_tongue_winking_eye:

If we can have bidirectional synchronization through webhook, i'll sign this deal :smile
@bkcsoft do you have any tutorial on how to set up issues webhook?
If i my understanding is correct, system A (github ie) triggers the webhook on event (issue creation /modification ie) and calls an url in system b (gogs ie). And what next? There is a way to create gogs issue at this call receiving?

Thanks your feedback!
What is the use case of sync issues from GitHub?

I want to use github as a public repo where users should create issues, synchronize these issues on gogs (which is used as our private repo with dev comments on issues).

@nioc bi-directional would require you to setup webhooks both ways, so github fires a webhook to update gogs, and gogs would fire a webhook to update github. Github has this feature, so you can today do unidirectional syncing from github to gogs (e.g. for using gogs as a backup-site), but Gogs doesn't have this type of webhook (only push-events and branch/tag-events are handled) so you can't do it bi-directional or have Gogs serve as a master...
@Unknwon Mainly for backups or for having a dummy-repo with issue-tracker on Github and have everything code-related on Gogs, but still be able to handle issues from within Gogs

If Gogs where to get on par with Github webhook-wise, one could write third-party services to synchronize everything between github and gogs, for redundancy if one of the sites goes down (network-issues, harddrive-failures, etc) (Generally a good idea :wink: )

@bkcsoft understood. If you can explain me what to set in the github webhook for create / update issues on gogs side, I'll be grateful :smiley:

In the repo-page goto Settings=>WebHooks=>Add Webhook then check Let me select individual events. and check Issues (nothing else). set the URL to wherever you have your php-script running (preferably have it check the secret...) and see these two links for payload-description for Issues
https://developer.github.com/v3/activity/events/types/#issuecommentevent
https://developer.github.com/v3/activity/events/types/#issuesevent

Unless someone actually wants an out-of-the-box integration with github (generally not what you want), this would be a duplicate of #2319 :smiley:

Thanks for the links. If i write such a script, i'll post it as a gist for sharing.
Agreed with the duplicate, i'll close mine.

Wouldn't two-way sync of issues be a mess ... :sweat:


spring integration issues fix

What and why?
How to test?
Trello
https://trello.com/b/EgV9fGPY/rasrm

/deploy sinhaa

/deploy sinhaa

/deploy sinhaa

/deploy manna

/deploy sinhaa

/deploy manna


_check_estimator_name no longer included in sklearn.utils.validation

Hi Isak,
"from sklearn.utils.validation import (
_check_estimator_name,
_check_y,
check_consistent_length,
warnings,
)"
This import is no longer working with the most recent version of scikit-learn.
The same is happening with '_fit_context' from 'sklearn.base'. Any suggestions? Thank you!

I think the first one is now in sklearn.utils.estimato_checks under the name of "check_estimator".
The second one must be now "check_X_y"

Hi!
what version are you using?

I'm using the latest one, wildboar 1.2

1.1 requires scikit-learn <1.4. The currently unreleased 1.2 requires scikit-learn >=1.3

Could you give me the version of scikit-learn. It should work for 1.3 and later

Ok I updated scikit-learn to 14.0, but now I'm facing this error:
"requests.exceptions.ReadTimeout: HTTPSConnectionPool(host='datasets.wildboar.dev', port=443): Read timed out. (read timeout=3)"

I recently reduced the timeout and "it was working fine on my machine"... I will need to increase this.
I think you can do:
from wildboar.datasets import refresh_repositories
refresh_repositories(timeout=30)

before your call to load_dataset (or which ever function you used)

OK, now it perfectly runs, thank you!

Could you tell me if you were using scikit learn 1.2 or 1.3 when you got the initial error? I want to ensure that it works with both and I think it should but I could be mistaken

When I read your answer I immediately updated, I'm not sure but I think it was the 1.2

Is there a way to check the version history of installed packages?

I don’t think so. I will test and see if I can reproduce with 1.3 otherwise I think you were using an older version.


Replace WebView with React Native Community WebView

Summary
WebView component is being moved to https://github.com/react-native-community/react-native-webview and this is a more up to date component in hope that it fixes the edge case reported in the ticket
Ticket Link
https://mattermost.atlassian.net/browse/MM-12544

Build Apps for PR label detected. Will generate the app if build succeeds (checks pass and no conflicts with base branch).

Build the App failed. See the logs for more information.

Build Apps for PR label detected. Will generate the app if build succeeds (checks pass and no conflicts with base branch).

This does appear to solve the problem.
I will complete some regression testing on SSO shortly.

@DHaussermann don't forget to approve the PR once you are satisfied

Regression tested login and logout with multiple accounts with different authentication methods.

email
LDAP
SAML (Okta and OneLogin)
Oauth (Google and GitLab)
Tested iOS but also brief look at Android

Added test steps to task.
@enahum after a couple dozen login tests - There was one single case where the SSO login pages no longer loaded and I had to restart the app. I then never came accross it again when trying to repo. So, it seems while it's still technically possible to hit the bug it is now very unlikely.
I would say we should move ahead with the code change we have now since it's obviously a huge improvement from before where the bug was simple to repo. It's likely the amount of effort needed to repo goes well beyond any real world scenario. If I manage to find repo steps again I can create a new task to be triaged.


Missing tia/analysis/model subfolder [Anaconda3/Python 3.6]

I installed using "pip install tia" from Anaconda3/Scripts (i.e. Anaconda's pip) and can see the package installed in Lib/site-packages/tia. However the subfolder tia/analysis/model is missing.
When I try to use "import tia.analysis.model as model", I get the following error:
import tia.analysis.model as model
Traceback (most recent call last):
File "C:\ProgramData\Anaconda3\lib\site-packages\IPython\core\interactiveshell.py", line 2862, in run_code
exec(code_obj, self.user_global_ns, self.user_ns)
File "", line 1, in
import tia.analysis.model as model
File "C:\Program Files\JetBrains\PyCharm Edu 4.0.2\helpers\pydev_pydev_bundle\pydev_import_hook.py", line 21, in do_import
module = self.system_import(name, *args, **kwargs)
File "C:\ProgramData\Anaconda3\lib\site-packages\tia\analysis_init.py", line 2, in
from tia.analysis.model import *

tia is not implemented for python 3

Thanks. Found this link in the issues #11 that seems to work well.
https://github.com/PaulMest/tia/


Add rebase with merge commit merge style (#3844)

This PR adds the Rebase and Merge (--no-ff) merge style as requested in #3844.

Known Bugs:

[x] As with the Rebase and Merge style, if the rebase can not be done cleanly, the merge will result in a 500 error page Tracked in #4078

Codecov Report

Merging #4052 into master will decrease coverage by 0.02%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #4052      +/-   ##
==========================================
- Coverage   19.97%   19.94%   -0.03%
==========================================
  Files         153      153
  Lines       30478    30507      +29
==========================================
- Hits         6089     6086       -3
- Misses      23477    23508      +31
- Partials      912      913       +1

Impacted Files
Coverage Δ

models/pull.go
16.75% <0%> (-0.42%)
:arrow_down:

routers/repo/pull.go
0% <0%> (ø)
:arrow_up:

routers/repo/setting.go
2.58% <0%> (-0.01%)
:arrow_down:

models/repo_unit.go
37.28% <0%> (-0.65%)
:arrow_down:

routers/repo/issue.go
0.91% <0%> (-0.01%)
:arrow_down:

modules/avatar/avatar.go
81.25% <0%> (-18.75%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2eabf18...9eb247b. Read the comment docs.

I have created a seperate issue to track the error during rebase #4052.
This PR is ready to be reviewed now.

@InExtremaRes Currently I am waiting for the review process. You too can leave suggestions/improvements to the code. ;)
Besides that you can try to convince some maintainers in the discord channel to review it. As the 1.5.0 Release has been cut off and is mostly done, there might be someone willing to do it.

@apricote I'd love to review/suggest/improve the code but I don't know Go. What is that discord channel you mentioned?

@apricote you may find a maintainer that will resolve conflicts for you, however my personal approach is to only touch someones PR when they've asked for help resolving a conflict. My preference to start reviews is that all conflicts are resolved (so that I can trigger drone to build against the most recent commit in master).
Like you said, the best way to get someones attention is to bring it up in the #develop channel in discord.

@InExtremaRes https://discord.gg/NsatcWJ is the discord server

This seems like a nice feature. It would have been nice to use it a few times that i remember. How is this going?

file conflicted.

@apricote I was testing this branch in our server and I can notice a couple of issues:

The message for the merge commit is not customizable.
The text head_repo is appended to the branch's name at merging.

Looking at the changes it seem like 2. is deliberated (example). Why is that the default behaviour? I think this does not align very well with the default of git, that always preserve the name of the branch.

Before @InExtremaRes 's question resolved, don't merge.

@InExtremaRes

This behaviour was copied from the Rebase style, which did not use/need a commit message. This should be fixed.
This behaviour was also copied from the Rebase style.

Rebase: https://github.com/go-gitea/gitea/blob/b387fc147af2078935f51ce35af23e4e3f6e3c7a/models/pull.go#L391
RebaseMerge: https://github.com/go-gitea/gitea/blob/b387fc147af2078935f51ce35af23e4e3f6e3c7a/models/pull.go#L416
I do not know why the branch is called like this.

maybe @lafriks could know that.

@apricote That's interesting. Well, when you rebase a branch (and merge --ff as in default rebase style) there is no commit merge that needs a message and then there's no evidence of that branch's existence anymore, in the commit logs at least. With this PR the message of the commit and by default the name of the branch are both kept in the history so that name could be helpful in some cases.
For the latest I just can guest. I think is for safety: if the rebase/pull/merge/something fails the branch can just be discarded without touching the original one and, as I said, in the Rebase and Squash styles the name of that branch is not seen anymore. But just my guess, and as @lafriks said maybe @lunny knows better and can explain.

There is another, small issue I didn't realize before. The author of the commit is set to the Gitea user instead of the user merging the PR, which is the behavior in the default merge strategy, and presumably the desire one.

@lafriks

That is ok that message is not customizable and it is correct that head_repo is prepended to branch name for local checkout, that is just not to conflict with exiting branch.

I think I understand the rationales behind, but I would prefer that even if the name of the branch is prepended the commit message would not, since it is clearer in the log what branch was merged. Most of the time the usefulness of this merge style, as opposed to classic rebase style, is that it's straightforward to see what commits are coming from specific branch and are (probably) related or part of the same feature.
@apricote Is it possible to keep the merge commit with the name of the original branch and the author as the user doing the merge? BTW, thanks for this feature, it is working pretty well until now.

Build failure seems to be unrelated:
package github.com/jteeuwen/go-bindata/...: github.com/jteeuwen/go-bindata/...: invalid import path: malformed import path "github.com/jteeuwen/go-bindata/...": double dot
Makefile:95: recipe for target 'generate' failed

@InExtremaRes I added the option to set a custom commit title and message. Also the author of the merge commit is now the user doing the merge.

The default merge commit message is now the same as with a usual merge:

Previous default  message: Merge branch 'head_repo_feature-1'
Current default message: Merge branch 'feature-1' of apricote/merge-test into master

Thank you @apricote. I'll be happy to keep testing and report back later. Since I already executed this when the migration version was 74, is there anything I need to do?

@InExtremaRes If you keep using this branch and eventually switch back to the master, you will skip the Migration v74, this migration adds a column (I think) and skipping it will result in a unexpected/invalid database schema.
The migration introduced in this PR is idempotent and can safely be re-run.
You will want to change the current database version to 73 (using standard sql): UPDATE version SET version = 73 WHERE id = 1;
On the next server start, this will retrigger all migrations > 73 and your database should be in the desired state.


Can I refresh csrf token after failed request?

It works perfect for the first time that request is sent.
useCsrfFetch("/api/signup", {
    method: "POST",

But it throws CSRF Token Mismatch error after sending the request second time (with the same csrf token). How to refresh csrf token for second request?

Can you please provide a reproduction link ?

If you redirect the user (via the server) back to the current page, this should help. Assuming, you are doing server-side rendering.

If your interface follows the same-origin policy with cookie mechanism, after the first successful request, the server regenerates the CSRF token and sets it in the response header with "Set-Cookie."
If your interface employs cross-origin policy, using the browser OPTIONS mechanism, the server generates the CSRF token and includes it in the response header with "Set-Cookie" when handling the OPTIONS request.