System.ArtifactsDirectory not set when Build.SyncSources=false

Have you try trouble shooting doc? Yes
Agent version and platform
Version of your agent? 2.127.0
OS of the machine running the agent? Windows Server 2016
VSTS type and version? on-prem TFS 2017 Update 3
What's not working?
System.ArtifactsDirectory is not set when Build.SyncSources is set to 'false'.
A build may want to capture/edit/publish artifacts without getting any source.  Consider a build who's job is to run & publish test metrics & data - it doesn't need to get any source because it uses the output of the CI build, however it still needs a place to put all of the data it is going to publish.  Artifacts is the natural place for this because it is automatically cleaned.  I've checked and it seems the folder 'a' still exists just the variable isn't being set.  I've tried both git & TFVC source repos and the behavior is the same.

Agent and Worker's diag log
Logs are located at the _diag under agent root folder, agent log prefix with Agent_, worker log prefix with Worker_. all sensitive information should already be masked out, please double check before paste here.

@abbottbd you shouldn't use Build.SyncSource=false anymore since it's not added for your scenarios.
you can set Dont' sync sources through definition editor UI which should has the correct behavior you want.

i am going to close this issue, let us know if you need more help.

I'm sorry but I that check box is only an option when using TFS GIT.  We are using TFVC source control and the check box is not displayed for TFVC, so therefore I must use the variable.

What's your scenario? What did you run in your build?

Our scenario is that we have a build that is designed to run all of our 15k unit tests with code coverage, We do not do this during our normal CI (code coverage) because it doubles the CI build time.  This build doesn't need to get any sources, it simply picks up the output of the CI build (all the compiled code) and then executes the unit tests.  We would like to 'not' have the build download any source control (because it isn't needed) however when we do that the artifacts directory is not set and so build tasks that are designed to use that variable fail.

@abbottbd try set agent.source.skip= true in your definition see whether that works.

Hi,
I have the same problem as described above. The hint to agent.source.skip=true helped a lot. Thank you.
The only thing is still missing the variable with the name of the workspace. In the logs of a non-skipped Get Sources I can read, it is a somehow complex procedure to find out this value:
Querying workspace information.
tf vc workspaces /format:xml /collection:https://tfs.healthcare.siemens.com:8090/tfs/IKM.TPC.Projects/ /loginType:OAuth /login:.,******** /noprompt
<Workspaces>
  <Workspace computer=...
  ...
  </Workspace>
</Workspaces>
Attempting to find a workspace: 'ws_1_430'
Candidate workspace: 'ws_1_430'
Definition mapping[0]: cloak 'False', recursive 'True', server path '...', local path 'C:\w\agent\_work\1\s'
Found mapping[0]: cloak 'False', recursive 'True', server path '...', local path 'C:\w\agent\_work\1\s'
Matching workspace found.

Is there an API or similar solution I can use instead of re-implementing that task?

@klapantius what's your scenario? you want to disable sync source but still have the TFVC workspace? the scenarios we target is either the agent pull source code for you, or you take over the entire get source process.

hey Ting,
we have a ticket at vsts-tasks in progress to get the possibility to download the sources to the workspace either with current timestamp (as it is now) or with the timestamp of the last check-in of each file (see the filteime option to the tf workspace /new command - not documented on the web).
Until it gets realized we are looking for the "cheapest" workaround. That is behind my question above. Actually I have a one liner to find out the workspace name if I know the folder, but that is a little bit dirty:
PowerShell

$wsname = tf workfold $wsroot | ForEach-Object { if ($_ -match '(?<branch>\$/.+): ' + [Regex]::Escape($wsRoot)) { return $Matches["branch"] } }

Now I would like to know if you can suggest me a better one. If not, then I think I can live with this solution.

I'm going to close this one out. The original question has been answered and the follow-up has an associated issue in the vsts-tasks repo.

@TingluoHuang hi, what if I wanna use yaml and don't use UI at all. How can I set Dont' sync sources to true with yaml only?

checkout: none


Fails to compile against recent boost libraries

Hi,
I wanted to try out lib-tftp-server on Ubuntu 17.10 which comes with more recent boost libraries (boost v1.62) and (of course) it fails to compile:
In file included from /home/malu/test/git/lib-tftp-server/src/tftp/server.hpp:27:0,
                 from /home/malu/test/git/lib-tftp-server/src/main.cpp:4:
/home/malu/test/git/lib-tftp-server/src/tftp/detail/tftp_packet_grammar.hpp:152:5: error: reference to ‘rule’ is ambiguous
     rule<Iterator, possible_request()> request;

This type of error can be fixed by explicitly defining the namespace via "qi::rule". However, after that the code runs in to some nasty boost assertions related to qi/karama stuff, which seems less trivial to fix:
/home/malu/test/git/lib-tftp-server/src/tftp/server.hpp:80:63:   required from here
/usr/include/boost/spirit/home/qi/nonterminal/rule.hpp:177:13: error: static assertion failed: error_invalid_expression
             BOOST_SPIRIT_ASSERT_MATCH(qi::domain, Expr);

Would be nice with a small update to make it compile against more recent boost libs. Thanks.

As of commit 62740b26926fdf96bdd8082df151181d9c7f7d65 from @dan-42 this is fixed.


added config parser with handling of yaml and json files, loading of …

…dicts, and handling of a default config

Closes #17

closes #1

Concerning the init.py blocking VScode from running pytest, I found this issue which claims its a configuration problem in VScode. I still need to test it but seems like a more solid solution.

The link you mentioned concerns unittest, which indeed are packages. As far as I understand, pytest tests are not packages, and therefore don't need the init.py. But I am not so sure myself. Fact is it works without the file, and does not with the file, without any additional config settings, as far as I know.

Maybe this helps:
https://docs.pytest.org/en/7.1.x/explanation/goodpractices.html#test-package-name
?
I am not sure I understand how it is supposed to be.

The problem with the test discovery with the init.py file came because I had the mpes package with some unittest tests in the same workspace. Now it seems to work.


PSDEVOPS-3772: Use prefork pool (default) so workers respond to probes even when busy

@RedHatProductSecurity/corgi-devs In order to add liveness probes, which will detect hung Celery workers and restart them, the worker must be able to respond to Celery management commands like "celery -A config inspect ping".
Using the solo pool means that there is only one Celery process / thread running inside the container, so whenever it's busy executing a task, the worker will not respond to any management commands / probes. This means the worker will appear hung and get restarted by OpenShift constantly (whenever it's executing a task) if we add a liveness probe.
We should use the prefork pool instead, which is almost the same as solo except that a parent process waits to respond to any management commands, as well as a child process which actually runs the tasks. I also fixed the Celery worker hostnames to match SDEngine to make configuring the probe easier for the different worker types. The type of worker (slow vs. fast) is still present in the hostname, just not in the prefix.

Nice, lgtm!


New buttons

#83

@frypan01 This needs a lot of fix for the website in mobiles in desktop mode.

Sir I am working on it though fixed the button but I am unable to understand the deformation of navbar in mobile view...If you can help .. . If you go through inspect and mover over to nav button and open ..you can see that it is there but may be within few layers that we can't see .


Consider removing TS linting and bundling?

Historically, we felt quite a bit of pain due to our complicated build process for the VS Code parts. Just today, an npm update broke vscode plugin, for example. However, the amount on typed script code we have is laughable, 3400 lines. It seems that, at this scale, npm ecosystem creates more problem than it solves.
My proposal is to completely remove mandatory formatting (tsfmt), linting (eslint) and bundling (rollup) out of the equation, and implement the whole extension as a single main.ts file.
WDYT?

Is it possible to not bundle, but still keep multiple files?

I think things like formatting are very good to have, and we certainly want to typecheck the file, right?
Probably we'll rather just move out the extension at some point?

an npm update broke vscode plugin

How did it break? Can we prevent such breakage without throwing out the whole bathwater?

Looks like an issue with a more recent vsce (the Code extensions packaging tool), which we can't avoid.

Is it possible to not bundle, but still keep multiple files?

It should be possible, but, to be honest, I see "keep the whole extension in a single file" as a benefit in itself. At less than 5k lines, ability to see everything by just folding bodies of functions is useful.

I think things like formatting are very good to have,

I am not entirely sure, given the quality of implementation issues and the amount of code in question. I think I personally am more frustrated by the front-end lints.

How did it break?

So I noticed yet another security vuln banner on the repository. I run npm update locally and submitted and r+ the PR. When I installed the new extension version, it just didn't start, at which point I just reveted the commit, without investigating further.

I don't see any changes in vsce 1.89.0, but it stops bundling the extension source code into the vsix, so Code can't load it any more.

Ah, I tracked this down to https://github.com/microsoft/vscode-vsce/issues/576. Our .vscodeignore makes vsce exclude main.js, but I'm still not sure how to fix it.

Having considered this, I don't think it makes sense at the moment. I think at some point we'd want to move the VS Code extension to another repo, and augment it with some real integration tests, and, at that point, the tradeoffs will be very different.


proposal: math: add Float, Integer, Complex, Number interfaces

In exp package there was constraints package that contained some interfaces, among which Ordered was added to Go 1.21.
I suggest adding those other number-related interfaces from the packages to the math package. They can be helpful and useful, and I think that math package is the right place for those interfaces (because they group numeric types).
Code that should be added to the math package:
type Complex interface { 	~complex64 | ~complex128 }
type Float interface { 	~float32 | ~float64 }
type Integer interface { 	~int | ~int8 | ~int16 | ~int32 | ~int64 | 	~uint | ~uint8 | ~uint16 | ~uint32 | ~uint64 | ~uintptr }
type Number interface { 	Complex | Float | Integer }

This has come up before, though without a definite answer. See #52427, which was closed with the suggestion to open a proposal specifically for this purpose. If I remember right, one of the main hesitations of doing this would be that packages that didn't need any other functionality from the math package would have to depend on it just to get access to the constraints. Probably not a huge problem, though, since the linker can probably just discard the unused rest of the package.

If the issue was importing whole package because of these interfaces, then what about creating math/constraints package?
And regarding Signed and Unsigned interfaces... that also can be useful. I just thought that these other interfaces were more important. But we don't lose anything by adding those two types that someone can use.

There's also #60274 that is for adding some helpers to math but notes that it could go in a package under math and if so that package would be a good place for the constraints.

@djordje200179

If the issue was importing whole package because of these interfaces, then what about creating math/constraints package?

As mentioned in #52547, if #57644 happens, then "constraints" is a bad package name regardless of path.

@djordje200179 Can you please give some (ideally real-world) examples of how and where this would be used?

@djordje200179 Can you please give some (ideally real-world) examples of how and where this would be used?

1. Many math calculation could be made generic, and not only for float64 (like previously done with functions in math package). Many of them couldn't accept Number as parameter, but they could accept Float or Integer, and not enforce float64/int.
2. Numpy-like and scipy-like matrixes and calculations for any type (generic matrix with add/scalar product/dot product, transpose, negations).
3. Data sources for charts, graphs, histograms..., and other statistics-related stuff.
4. If proposal for iterators becomes reality, then we could have function that creates a range within limits and with given increment
`for i := range Increment(2, 10, 2) { ... }`

Those things were first ones that came to my mind, but there are probably a lot more things that could be done.

Our experiences with x/exp/constraints was that approximately nobody used most of the constraints.  So I suggest that we focus first on APIs that might want to use these constraints.  Then if we those APIs prove popular, we can consider adding the constraints.  After all, the constraints might prove to be useful but they are never necessary.  We can go from uses to the constraints, rather than from the constraints to uses.

The current math package is pretty much all float64. Only a handful of functions in it even make sense on integers. On a quick scan, the only ones I see are Abs, Dim, Pow, and Pow10. It might make more sense to create some new package that focuses on whatever generic integer math needs to be done, and putting constraints there.

In my opinion, it would be okay to make Abs, Pow and Pow10 generic functions. We would avoid unnecessary casts between integer and float types.
Also, other functions wouldn't need to "force" float64 types. Someone that uses float32 type for his purposes after calling math functions needs to downcast returned result from float64 to float32.
Same thing with cmplx package, that "forces" use of complex128.

In my opinion, it would be okay to make Abs, Pow and Pow10 generic functions. We would avoid unnecessary casts between integer and float types.

While I agree with you and think it would be worth the cost, changing the signature of exported functions would break the backward-compatibility promise.

I would like to express my support for generic float in math, because i often work with float32 and it makes the math package un-practicable for my use case.
Also, curios how would it break the backward compatibility promise?
As i understood it, it would expand on the current implementation without taking away, and type inference from arguments should take care of the rest.

I would like to express my support for generic float in math, because i often work with float32 and it makes the math package un-practicable for my use case.

@BieHDC Thank you for your support!

As i understood it, it would expand on the current implementation without taking away, and type inference from arguments should take care of the rest.

I have also thought about same thing, but @mohamedattahri said that changing the signature breaks code. So, I'm not sure now what is the truth.

I have also thought about same thing, but @mohamedattahri said that changing the signature breaks code. So, I'm not sure now what is the truth.

// If this generic, this becomes an error because it
// would be a usage of an uninstantiated generic
// function.
f := math.Sin

@DeedleFake Thank you for clarification! I have forgot about function type and having variables.


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 1780

Questions ?
Please refer to the Backport tool documentation


Upload encryption output on youtube->donwload->test decryption

need to upload the codecs output video,download the videos and then test who is passing the decryption test

the codec tests was a successful,
upload the video munaly the video to youtube and download the video using Jdownloader software
all 8 codec passed the test-
vp09 ,mp4v ,avc1 ,avc3 ,hev1 ,mp4v, davc, xvid


build sim on CI

bors r+

bors try
(there was a bug with .yml sytnax)

bors try

Finally sorted all build errors out!
bors r=sentientwaffle


GOARCH is '386' and not 'i386'

Apparently runtime.GOARCH value is put into image metadata when building images (it's "386" on 32-bit x86 systems):
https://github.com/appc/spec/blob/master/ace/build_aci#L25
However, during validation "i386" is used (with an extra i):
https://github.com/appc/spec/blob/master/schema/types/labels.go#L24

Sigh, this is an unfortunate side effect of https://github.com/appc/spec/commit/41fc66c7d807725c546553039930986cd2777e70 and is probably going to bite us with other combinations too. Any bright ideas here, aside from aliasing a bunch of combinations?  /cc @mpasternacki

Well, I actually built 32-bit appc on a 32-bit box. So reproduction isn't directly related to the commit. I think operation on 386 is not supported now, but it might be supported in the future.

Sort of - check out some historical discussion here. We are not strictly using Go's concepts for these values (at the very least since the spec should be language-agnostic)
I am not really sure what to do here aside from removing the whitelist entirely.
@mpasternacki any bright ideas?

The removal of the whitelist looks good. The architecture is commonly called i686, as many post-386 instructions are widely used these days. And the technically correct name (which nobody uses) is IA-32.
However, appc still needs a way to recognize incompatible images, so standard names of popular architectures should be in the standard document and recognized by everybody, I think.


Update BlogHubPostModel.php

Unset blog_meta_temp if empty

Hello,
thank you very much for your contribution. I know it took waaaaay to long time, for which I can only apologize.
~ Sam.


add gosec linter, various small fixes

A few small fixes, added the gosec linter since it caught a few issues. I ignored all the math/rand usages, but I'm happy to switch them to crypto/rand if you'd like to not have the nolints.

Codecov Report
Base: 54.97% // Head: 54.81% // Decreases project coverage by -0.17% :warning:

Coverage data is based on head (2b32b30) compared to base (eefb568).
Patch coverage: 50.00% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #739      +/-   ##
==========================================
- Coverage   54.97%   54.81%   -0.17%
==========================================
  Files          52       52
  Lines        9824     9823       -1
==========================================
- Hits         5401     5384      -17
- Misses       3630     3642      +12
- Partials      793      797       +4

Impacted Files
Coverage Δ

node/full.go
61.80% <ø> (-0.17%)
:arrow_down:

types/serialization.go
75.10% <0.00%> (-0.32%)
:arrow_down:

block/manager.go
64.00% <100.00%> (-0.31%)
:arrow_down:

da/mock/mock.go
73.75% <100.00%> (ø)

types/validation.go
37.20% <0.00%> (-13.96%)
:arrow_down:

rpc/json/ws.go
53.96% <0.00%> (-7.94%)
:arrow_down:

state/executor.go
64.73% <0.00%> (-0.73%)
:arrow_down:

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


fix: add doc usage update

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

GitHub release
crate.io packages (1 packages published)

Your semantic-release bot :package::rocket:


Mirror traffic of service interface to a dest interface

Fix https://github.com/kube-vip/kube-vip/issues/589
Introduce a new env variable mirror_dest_interface, the user could set it to an second interface other than the service interface or default interface. Then all traffic of service that goes to the default interface or service interface will also be mirrored to the dest interface
Idea from https://gist.github.com/mcastelino/7d85f4164ffdaf48242f9281bb1d0f9b

Looks good, at the moment this is only implemented to "routing table mode". Will it be added for ARP and BGP?

Yeah thanks for reminding, I moved it to servicesWatcher so it will be triggered on all modes if enabled.
 I need to fix the logic to clean up the remaining qdisc before starting
I0227 01:45:29.725243       1 leaderelection.go:250] attempting to acquire leader lease kube-system/plndr-cp-lock...
I0227 01:45:29.757027       1 leaderelection.go:260] successfully acquired lease kube-system/plndr-svcs-lock
time="2024-02-27T01:45:29Z" level=info msg="mirroring traffic from interface eth0 to interface eth1"
time="2024-02-27T01:45:29Z" level=debug msg="interface eth0 has index 2\n"
time="2024-02-27T01:45:29Z" level=debug msg="interface eth1 has index 12\n"
time="2024-02-27T01:45:29Z" level=debug msg="step 1: tc qdisc add dev eth0 ingress\n"
time="2024-02-27T01:45:29Z" level=error msg="failed to add qdisc for index 2 : file exists"
I0227 01:45:29.760585       1 leaderelection.go:260] successfully acquired lease kube-system/plndr-cp-lock
time="2024-02-27T01:45:29Z" level=info msg="Node [wl-antrea-controlplane-2864w-vr78p] is assuming leadership of the cluster"
time="2024-02-27T01:45:29Z" level=info msg="Gratuitous Arp broadcast will repeat every 3 seconds for [<IP_ADDRESS>]"

Also the service controller can't start either.
Hi @thebsdbox do you think it make more sense to return fatal here/here/here so kube-vip could crash and restart so that it can inform the user that service watcher is not running. Instead of letting it running silently without letting customer know that the watcher is not actually running

Good question, it either logs errors and services stay in pending or the pod itself goes into crashLoopBack etc..
I think it might be best moving it to fatal.

Other than the few nits, I'm loving the new feature!

Is this ready to merge?

Yes it's ready now. I don't see the merge button though

I'll check if it's testable inside kind node by adding a different interface and create another pr for e2e if that's doable
Looks like it's doable https://gist.github.com/aojea/00bca6390f5f67c0a30db6acacf3ea91

Yes it's ready now. I don't see the merge button though

Big feature, it’s going to be merged into a v0.8.0

Going to merge this and quickly fix the linting afterwards. !


The example directory seems useless?

Currently we have

symbiflow-examples/examples/xc7

The extra examples directory doesn't seem to add much value?
It would probably be better to have the following

symbiflow-examples/xc7

It also means that a person who goes to this repo quickly sees which architectures there are examples for.

@pgielda / @kgugala What do you think?

Not sure. The thing is that xc7, eos-s3 (and potentially other names in the future) will pile up that is one thing, second the idea with "examples" is to make clear that this is where the examples are (and not e.g. some arch data needed for the scripts, etc).
In any case I do not have any strong opinions about it in any case so we can as well remove it.


Does the proxy not work?

Before submitting an issue, make sure you have:

[ x] Updated to the latest version
[x ] Read the README and docs
[ x] Searched the bugtracker for similar issues including closed ones

Python v 3.7.1
I updated the instabot version from 52 to 58. I pass the values from the proxy to the script (the script did not change, it remains the same):
bot.login(username='login', password='pass', proxy='http://host@ip:pass', use_cookie=False)
As a result, the script went online on my IP and not through a proxy ... With all the sad consequences((( Please check if anyone else has this problem?

Try:
bot = Bot(proxy='ip:port')

Try:
bot = Bot(proxy='ip:port')

thank

same issue


Backport "HBASE-23324 Deprecate clients that connect to Zookeeper (#5745)" to branch-3

Our objective is to remove ZooKeeper from our public interface, remaking it as an internal concern. Connecting to a cluster via ZooKeeper quorum will be considered deprecated starting in 2.6. Our default connection mechanism will switch to via RPC in 3.0 And finally we intend to remove the ZooKeeper connection mechanism from client-facing APIs in 4.0.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 39s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-3 Compile Tests _

+1 :green_heart:
mvninstall
2m 45s
branch-3 passed

+1 :green_heart:
compile
0m 19s
branch-3 passed

+1 :green_heart:
shadedjars
5m  7s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 15s
branch-3 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 26s
the patch passed

+1 :green_heart:
compile
0m 20s
the patch passed

+1 :green_heart:
javac
0m 20s
the patch passed

+1 :green_heart:
shadedjars
5m  7s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 15s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m 18s
hbase-client in the patch passed.

19m 37s

Subsystem
Report/Notes

Docker
ClientAPI=1.44 ServerAPI=1.44 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5757/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5757

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 826f2b4b66a4 5.4.0-169-generic #187-Ubuntu SMP Thu Nov 23 14:52:28 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-3 / b374e52893

Default Java
Temurin-1.8.0_352-b08

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5757/1/testReport/

Max. process+thread count
288 (vs. ulimit of 30000)

modules
C: hbase-client U: hbase-client

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5757/1/console

versions
git=2.34.1 maven=3.8.6

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 45s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-3 Compile Tests _

+1 :green_heart:
mvninstall
4m 10s
branch-3 passed

+1 :green_heart:
compile
0m 30s
branch-3 passed

+1 :green_heart:
shadedjars
7m 33s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 25s
branch-3 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 15s
the patch passed

+1 :green_heart:
compile
0m 29s
the patch passed

+1 :green_heart:
javac
0m 29s
the patch passed

+1 :green_heart:
shadedjars
7m 43s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 25s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
2m 39s
hbase-client in the patch passed.

30m 11s

Subsystem
Report/Notes

Docker
ClientAPI=1.44 ServerAPI=1.44 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5757/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5757

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 2e05b6506840 5.4.0-172-generic #190-Ubuntu SMP Fri Feb 2 23:24:22 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-3 / b374e52893

Default Java
Eclipse Adoptium-11.0.17+8

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5757/1/testReport/

Max. process+thread count
317 (vs. ulimit of 30000)

modules
C: hbase-client U: hbase-client

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5757/1/console

versions
git=2.34.1 maven=3.8.6

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 44s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ branch-3 Compile Tests _

+1 :green_heart:
mvninstall
4m 39s
branch-3 passed

+1 :green_heart:
compile
1m  0s
branch-3 passed

+1 :green_heart:
checkstyle
0m 26s
branch-3 passed

+1 :green_heart:
spotless
1m  3s
branch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
1m  6s
branch-3 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 16s
the patch passed

+1 :green_heart:
compile
0m 58s
the patch passed

-0 :warning:
javac
0m 58s
hbase-client generated 1 new + 34 unchanged - 0 fixed = 35 total (was 34)

+1 :green_heart:
checkstyle
0m 26s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
7m 44s
Patch does not cause any errors with Hadoop 3.3.6.

+1 :green_heart:
spotless
1m  4s
patch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
1m 19s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 15s
The patch does not generate ASF License warnings.

34m 53s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5757/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5757

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile

uname
Linux ce0289549599 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-3 / b374e52893

Default Java
Eclipse Adoptium-11.0.17+8

javac
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5757/1/artifact/yetus-general-check/output/diff-compile-javac-hbase-client.txt

Max. process+thread count
75 (vs. ulimit of 30000)

modules
C: hbase-client U: hbase-client

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5757/1/console

versions
git=2.34.1 maven=3.8.6 spotbugs=4.7.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Dynamic URIs

Create URIObject and replace URI class in JWebServer methods HashMap
Not sure as to what this is supposed to look like or wheter it's actually needed or not

Will be implemented.

Implemented.


Use api-compiler-0.0.8

api-compiler is needed to support the api-common-protos (from branch input-contract) that are needed to be installed in gapic-generator in order to parse annotated protos, using annotations.proto

Codecov Report

Merging #2254 into master will decrease coverage by 0.03%.
The diff coverage is 30.3%.

@@             Coverage Diff             @@
##             master   #2254      +/-   ##
===========================================
- Coverage     86.74%   86.7%   -0.04%
  Complexity     5134    5134
===========================================
  Files           452     452
  Lines         20378   20390      +12
  Branches       2189    2189
===========================================
+ Hits          17676   17680       +4
- Misses         1928    1936       +8
  Partials        774     774

Impacted Files
Coverage Δ
Complexity Δ

...google/api/codegen/gapic/LegacyGapicGenerator.java
84.44% <0%> (ø)
8 <0> (ø)
:arrow_down:

...om/google/api/codegen/gapic/GapicGeneratorApp.java
27.58% <0%> (-1.78%)
7 <0> (ø)

...a/com/google/api/codegen/gapic/GapicGenerator.java
86.2% <0%> (ø)
5 <0> (ø)
:arrow_down:

...a/com/google/api/codegen/config/ProtoApiModel.java
89.74% <0%> (ø)
17 <0> (ø)
:arrow_down:

...api/codegen/configgen/GapicConfigGeneratorApp.java
72.22% <0%> (ø)
4 <0> (ø)
:arrow_down:

...le/api/codegen/packagegen/PackageGeneratorApp.java
62.79% <0%> (ø)
3 <0> (ø)
:arrow_down:

...api/codegen/config/ResourceNameMessageConfigs.java
90% <100%> (ø)
21 <0> (ø)
:arrow_down:

.../java/com/google/api/codegen/advising/Adviser.java
80% <100%> (+1.05%)
5 <0> (ø)
:arrow_down:

...i/codegen/configgen/mergers/ProtoConfigMerger.java
66.66% <100%> (ø)
4 <1> (ø)
:arrow_down:

.../google/api/codegen/config/GapicProductConfig.java
73.75% <87.5%> (+0.02%)
38 <0> (ø)
: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 96180d7...764a6df. Read the comment docs.

PTAL

cc @jskeet wasn't this relevant to you for some reason?

Yup, could make it much quicker to run on Windows :) Will try...


Updating Readme for better testing

I believe this is something that might be useful for contributors if they miss lines in their testing and want to know which lines they missed :)

Thanks, this generally makes sense. Any particular reason why you are not using the pytest command mentioned below this?

Not a particular reason as such but testing gives me an excuse to take a break for like 3-4 minutes. Earlier I used to use tox and sometimes I used to miss statements in the test case I wrote and had to change the directory and use the pytest command. I wanted all this to be a single process, so I had to explore a little to find how to do that with tox. So that's the reason I thought this might be beneficial for someone out there. :see_no_evil:

Any thoughts?

Looks good to me. I use both tox and pytest, depending on the circumstance. For full tests I prefer tox. I'd tweak the grammar ever so slightly. Something like this:

To run complete tests with a full coverage report, you can use the following command

@cortesi Thanks, I have changed the grammar :)


Duplicate commerce chatbots

2 Commerce chatbots appear in Browse all Architecture . Both Chatbot for hotel reservations and Commerce Chatbot refer to ../../includes/cards/commerce-chatbot.md.

Thanks for reporting - this issue is under review.  This is a Microsoft Internal DevOps Bug ID AB#329248


p5.SoundFile page links gives 404 error

Actual Behaviour
getBlob and http link on the p5.SoundFile page gives 404 error

Expected Behaviour
Links should go to the desired page

Steps to reproduce it

Go to p5.SoundFile reference page.
Move to last second last row in Methods table.
Click the getBlob link in the method description.
It gives 404 error.

Screenshots of the issue

Would you like to work on the issue?
YES

You can go ahead with a fix. Thanks.

@limzykenneth  But what should be the correct url

The links both have an extra "docs/" in them. Just remove those and they should work.

Thanks! I will fix this asap

Reference for p5.SoundFile is over here.

Thanks!

I have created a PR 455 under the p5.js-sound repository which fixes this issue.

@Rahulm2310 i reviewed your PR and it was correct, it is now merged, thank you!

@montoyamoraga @limzykenneth Thanks!


Upload functions in batch

Add a function update page:

The entrance is called Upload in batch beside the Add button in function list page
User can control the mapping between file extension and existing environment by a table
User can click on Select functions from local to open a file selector to choose multiple files into the pending list
User can drag and drop multiple files into the pending list
The name, env of the functions in the pending list is editable, default name is file name and default env is selected from the file ext to env mapping
There is an switch of 'Add/Update' to let user choose from to create all the functions as new functions or just want to batch update the existed functions
When Upload is clicked, check all the field and then continue, if not, do nothing but show user the error
Create/Update the functions one by one, if error occurs during a specifc function, add the error to specific row in the pending list and continue to try the next one
Show pending/success/processing/failed state of item function in the pending list
User can remove one row of the pending function
User click on cancel to return to the function list page

Close by 7893c2d


Batman/search messages doc type and reason

Description

References

#0000

Codecov Report

Merging #273 (b086b5d) into main (ebff71e) will decrease coverage by 66.58%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##             main     #273       +/-   ##
===========================================
- Coverage   81.49%   14.91%   -66.58%
===========================================
  Files         342      338        -4
  Lines        6166     6328      +162
  Branches      622        0      -622
===========================================
- Hits         5025      944     -4081
- Misses        922     5364     +4442
+ Partials      219       20      -199

Impacted Files
Coverage Δ

...MarketActors/MarketActorAuthenticatorMiddleware.cs
0.00% <0.00%> (ø)

...Messages/RequestChangeOfSupplierMessageReceiver.cs
0.00% <0.00%> (ø)

source/Api/OutgoingMessages/PeekRequestListener.cs
0.00% <0.00%> (ø)

...ource/Api/SearchMessages/SearchMessagesListener.cs
0.00% <0.00%> (ø)

source/Application/Actors/CreateActorHandler.cs
42.85% <0.00%> (-42.86%)
:arrow_down:

.../OutgoingMessages/Dequeue/DequeueRequestHandler.cs
33.33% <0.00%> (-44.45%)
:arrow_down:

...cation/OutgoingMessages/Peek/PeekRequestHandler.cs
30.76% <0.00%> (-69.24%)
:arrow_down:

...plication/SearchMessages/GetMessageQueryHandler.cs
23.07% <0.00%> (-76.93%)
:arrow_down:

...rce/Application/SearchMessages/GetMessagesQuery.cs
0.00% <0.00%> (-100.00%)
:arrow_down:

source/Application/SearchMessages/MessageInfo.cs
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 48 more

... and 230 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Fix high CPU usage when running runserver_plus in Docker

Description
Installing the watchdog package enables the runserver_plus auto-reload feature to use file system events instead of consrtant file polling.
This is a fix for issue #3523.
Rationale
See issue #3523 for full description of the problem. As a summary, it fixes a high-cpu usage problem for the local development server.
Fix #3523

Thanks for the contribution! 🎉


feat: Extract method

What's changed?

Checklist

[x]  I hereby agree to the terms of the HertzBeat CLA
[x]  I have read the Contributing Guide
[x]  I have written the necessary doc or comment.
[x]  I have added the necessary unit tests and all cases have passed.

hi👍 suggest use the existed class common/src/main/java/org/dromara/hertzbeat/common/constants/CommonConstants.java to replace of common/src/main/java/org/dromara/hertzbeat/common/constants/Constants.java


chore(lint): introduced linting

introduced linting enforced by CI
fixed linting issues
added: lock obviously missing, preallocated slices

(followed the same set of rules as defined for kubescape/kubescape)

fixes #77

Signed-off-by: Frédéric BIDON<EMAIL_ADDRESS>

@dwertent  I am rebasing but CI no longer passes since you have merged #78 (fix is #80 ). Would you mind merging #80  first so we get back to a regular CI "green" status? Cheers

@matthyx @dwertent  rebased

@dwertent since we have the global CI workflow, do we still need that PR?

@matthyx  the change to the github workflow might be unnecessary. However, there are still some code changes...


Error when using setHours and setMinutes

I have this:
<DatePicker           selected={this.state.startDate}           onChange={this.handleChange}           showTimeSelect           minDate={new Date()}           minTime={setHours(setMinutes(new Date(), 0), 7)}           maxTime={setHours(setMinutes(new Date(), 0), 17)}           filterDate={this.isWeekday}           timeFormat="HH:mm"           timeIntervals={30}           dateFormat="d MMMM, yyyy h:mm aa"           timeCaption="Hora"           locale="es"         />
But  setHours and setMinutes throw an error. I am missing something?
Also, with filterDate I want to allow only to select a weekday. Does anyone have the function for isWeekday?

Hi, you need date-fns module for that, take a look at this example.

@no1lov3sme The URL which you shared it's throwing 404.

@no1lov3sme The URL which you shared it's throwing 404.

import setSeconds from "date-fns/setSeconds";
import setMinutes from "date-fns/setMinutes";

Source


Need YAML support to access and check the struct fields constraints

Summary:
Need YAML support to access and check the struct fileds constrainsts
Effected tests:
TC-GC-2.1
Tesplan screenshot:

Spec screenshot:

Effected tests
TC-BRBINFO-3.1
TC-CGEN-2.1


[auth]: Add openapi specification

Adding openapi specification for the auth api even though we don't really have any way of consuming it. So mostly in preparation for later.
First time writing one of these so let me know if I'm missing something.
The content of the yaml can be pasted into http://editor.swagger.io/ if you want to look at it.
Fixes #795

Closing the Auth milestone! 🎉


For comprehension calls blocking get on Futures.

For() works with iterables, and Future's iterator method returns an iterator of the future's eventual value. (return isEmpty() ? Iterator.empty() : Iterator.of(get());)
Is this the expected behaviour?

Yes, we will add specialized For overloads for our monadic types (see #1629 (and #2171)).


scipy.cluster.hierarchy.dendrogram: cluster_sort='descending'

My issue is about 'descending' not working for options distance_sort and count_sort in scipy.cluster.hierarchy.dendrogram.
Below is the source code of scipy/hierarchy.py line 3537. The first if statement will catch 'descending' in the "or count_sort"
and therefore will never go into the "elif count_sort == 'descending'" statement.
Source Code scipy/hierarchy.py line 3537:

    if count_sort == 'ascending' or count_sort:
        # If a has a count greater than b, it and its descendents should
        # be drawn to the right. Otherwise, to the left.
        if na > nb:
            # The cluster index to draw to the left (ua) will be ab
            # and the one to draw to the right (ub) will be aa
            ua = ab
            ub = aa
        else:
            ua = aa
            ub = ab
    elif count_sort == 'descending':
        # If a has a count less than or equal to b, it and its
        # descendents should be drawn to the left. Otherwise, to
        # the right.
        if na > nb:
            ua = aa
            ub = ab
        else:
            ua = ab
            ub = aa
    elif distance_sort == 'ascending' or distance_sort:
        # If a has a distance greater than b, it and its descendents should
        # be drawn to the right. Otherwise, to the left.
        if da > db:
            ua = ab
            ub = aa
        else:
            ua = aa
            ub = ab
    elif distance_sort == 'descending':
        # If a has a distance less than or equal to b, it and its
        # descendents should be drawn to the left. Otherwise, to
        # the right.
        if da > db:
            ua = aa
            ub = ab
        else:
            ua = ab
            ub = aa
    else:
        ua = aa
        ub = ab

Scipy/Numpy/Python version information:

1.5.4 1.18.5 sys.version_info(major=3, minor=8, micro=0, releaselevel='final', serial=0)

@pierrechen1998, thanks for reporting the problem.  This was fixed in https://github.com/scipy/scipy/pull/12911, which will show up in SciPy 1.6.


Angelic Halo Rule Element Support

Allies in your halo's emanation who are healed by a heal spell gain a status bonus to Hit Points regained equal to double the heal spell's level.

This likely will require implementing a way to retrieve the spells level when calculating the value of the healing-received FlatModifier below:
{
  "key": "FlatModifier",
  "predicate": [
    "item:slug:heal"
  ],
  "selector": "healing-received",
  "value": "2 * @spell.level",
  "type": "status"
}

MAKE A PR< MR VAUX


Create an API only app

In this milestone I :

Created an API that lets its user fetch random greetings

Created greetings table
Created greeting model
Added a route for the API
Added controller for the API

@zdnahom, the app works. Kindly submit and the next reviewer will approve. The problem was the port in the API CORS was different from the URL used in the react app.

Okay @elisha2kyakpo1 thanks


Table of Contents part 6.md

Would this work?
Pull Request
Feel free to delete parts of this pull request template that do not apply to your pull request, this is just used as a guideline
Description
[Provide a concise description of the changes introduced by this pull request.]
Related Issue(s)
[Link any related GitHub issues or pull requests here.]
Changes Made
[List the specific changes made in this pull request.]

[Change 1]
[Change 2]
...

Screenshots (if applicable)
[Add screenshots or gifs to showcase the changes, if relevant.]
Checklist

[ ] I have reviewed the Contributing Guidelines for this repository.
[ ] I have followed the coding style and conventions of this project.
[ ] I have added/updated relevant documentation (if necessary).
[ ] The code is tested and all tests pass.
[ ] I have updated the version (if applicable).
[ ] All commit messages are meaningful and follow the established convention.

Additional Notes
[Include any additional information or context that may be relevant to this pull request.]

By submitting this pull request, I confirm that my contribution is made under the terms of the LICENSE for this project.

Yes, that is exactly what I was looking for


Save As Gif Should Default to Project Name.gif

When I save my project and then go to save a gif, I end up having to retype the filename again.  IMHO, it should use the project name as the default.  Thoughts?
How feasible would this be in the current code base?  I'm open to submitting a PR for this.

Do you mean that if you open a project file named "Example.stg", the suggested filename of the gif (and other output types) should be "Example.xxx"?
Also, do you think that it should use the same output folder?
This could be implemented, but it needs to be optional.

No, but that is another valid scenario where it should probably apply.  My use case is: record -> save prj -> save gif.  After I save the prj, I have to re-type the filename for gif.
Yes, same output folder.
Copy that.
I happy to assist with this request 😄

Oh, for that behavior there's a similar ticket open.
#125

thanks Nicke


refactor: Remove oio::Write::copy_from

Implement https://github.com/apache/incubator-opendal/pull/3017

dropbox is flaky now, please ignore for now.


EDC - Maintenance

Description
As a member of the EDC developer community,
I want to have a clean code base that follows the latest development guidelines
Therefore come code hygiene measures should be taken up, to clean up the code base, like:

[x] Extension hygiene: Getting rid of unsupported/not officially sanctioned extensions
[x] Extension refactoring: Rewrite/redesign legacy extensions that aren't up to Tx-EDC standards and coding guide lines
[x] Settings hygiene: All Tractus-X EDC properties should start with tx.edc. to be able to clearly distinguish them. We should go through all extensions and fix that. --> https://github.com/eclipse-tractusx/tractusx-edc/issues/1302

Initial creator: @paullatzelsperger
@paullatzelsperger: Feel free to extend the current description, we thought, that from a planning point of view, we could summarize these three points to one planning item and create concrete workitems during the development iteration.
Impact
None expected
Additional information

[ ] I'm willing to contribute to this feature

@lgblaumeiser that's fine for the time being. Ultimately all three of these items should become issues in the tractusx-edc repo

was presented in the open planning ⇾ contributors and committer are involved.

Hello @paullatzelsperger, @lgblaumeiser
Since the feature is a 24.08 feature and the development phase is coming to an end, we need a status on the feature.

[ ] Currently you are assigned (Responsible) → Is this correct? If not, please assign the correct contact person
[ ] Please check whether the status (backlog, work in progress ...) is set correctly
[ ] Please check whether there is an assignment to a committee or expert group (supported-by) to be found on the board
[ ] Please comment on the current status of the feature
[ ] Are all SubTasks (issues from other repositories that deal with the feature) linked? → The easiest way is to mention the feature here in the issue (via the ID) so we can see which teams/repositories are involved.

If you need any clarification, please get in touch, thank you very much.
Stephan

@stephanbcbauer this is a long-running issue, that is never really "done". I cannot tell you all maintenance tasks that'll ever be in advance, because we typically refactor, cleanup and maintain when we come across problems.
TL;DR: we could close this issue, and create dedicated ones whenever we do a refactoring, WDYT?

2024-06-11: Onoging activity, will be closed close to release with a documentation of issues closed during the development cycle.

2024-06-11:

[x]  Currently you are assigned (Responsible) → Is this correct? If not, please assign the correct contact person
[x]  Please check whether the status (backlog, work in progress ...) is set correctly
[x]  Please check whether there is an assignment to a committee or expert group (supported-by) to be found on the board
[x]  Please comment on the current status of the feature:
Ongoing activity, will be closed close to release with a documentation of issues closed during the development cycle.
[ ]  Are all SubTasks (issues from other repositories that deal with the feature) linked? → The easiest way is to mention the feature here in the issue (via the ID) so we can see which teams/repositories are involved.
@paullatzelsperger : Please add all activity issues and PRs connected to this activity to the item.

Will be closed with the release of the Tractus-X connector

Hello @lgblaumeiser
In a few weeks, we want to publish Tractus-X Release 24.08. For this, we need a clear status about the features (which will be included in the release and which will be cancelled or postponed to the next release).
Currently, the feature is still in status Work in progress. Please update the status by 09.07.24 eob.
If the feature is not ready for release 24.08, or it is possibly not planned at all, please set the status to Not in release. Please remember that in this case, any dependent components must be informed.
Thank you and best regards.

Hi @paullatzelsperger @wolf4ood @ndr-brt
can you add links to PR which you would rate under this category, so that we can close the issue, thanks!

adding few more:

https://github.com/eclipse-tractusx/tractusx-edc/pull/1401
https://github.com/eclipse-tractusx/tractusx-edc/pull/1414
https://github.com/eclipse-tractusx/tractusx-edc/pull/1396

https://github.com/eclipse-tractusx/tractusx-edc/pull/1399
https://github.com/eclipse-tractusx/tractusx-edc/pull/1402
https://github.com/eclipse-tractusx/tractusx-edc/pull/1309
https://github.com/eclipse-tractusx/tractusx-edc/pull/1340


Markdown to HTML XSS

If I do [some text](javascript:alert('xss')) in the editor, this will get rendered when i click on the link.
The format above is in markdown

Therefore, this is quite vulnerable to attacks: https://github.com/showdownjs/showdown/wiki/Markdown's-XSS-Vulnerability-(and-how-to-mitigate-it)
Is there any way we can solve this?

Hola

_**_``_**_


feat(c): Add growing queue functionality

Proposed Changes

Added ability to grow queue

TODO

Resolve memory free with 0 buff_size
Resolve automatically grow on enqueue if needed

/AzurePipelines run

@SanjoDeundiak thank you for sending this PR ❤
Our commit message convention requires that the messages our in lower case, could you amend the commit and change Add to add

@mrinalwadhwa thank you for such a nice feedback! I've fixed the commit message.

@SanjoDeundiak could you resolve conflict with the latest changes in develop. Thank you.

@mrinalwadhwa check it please

@SanjoDeundiak looks good, I'll confirm that it builds and then merge shortly. 👍


[BUG] Dashboards not working

🐛Bug Report
Describe the bug

?Cards missing etc.

To Reproduce

Expected behavior

Screenshots

Desktop (please complete the following information):

OS:
Browser
Version

Smartphone (please complete the following information):

Device:
OS:
Browser
Version

Additional context

Fresh Start on Dashboard


feat: Use ranges instead of a vector for input witness

Description
Problem*
Resolves #861
Summary*
Use ranges for representing witness inputs, instead of using a vector.
Additional Context
Documentation*
Check one:

[X] No documentation needed.
[ ] Documentation included in this PR.
[ ] [Exceptional Case] Documentation to be submitted in a separate PR.

PR Checklist*

[X] I have tested the changes locally.
[X] I have formatted the changes with Prettier and/or cargo fmt on default settings.

noir_js tests need to be updated


Unable to gclient sync clang successfully on Windows

When I try to run gclient sync on my Windows device, I get the following error:
________ running 'cipd ensure -log-level error -root C:\Users\gspencer\code\engine -ensure-file C:\Users\gspencer\AppData\Local\Temp\tmpwsfsjbep.ensure' in '.'
[P119436 13:44:02.363 client.go:1718 E] Failed to install fuchsia/third_party/clang/windows-amd64:6u9Xk_Dr7e_2dXmuP5zwTi2yfZ7ChIcJ3aVTfR-154wC - symlink clang.exe C:\Users\gspencer\code\engine\.cipd\pkgs\0\-dkLlv5Kjza9\bin\wy1tHfIub-rr:
A required privilege is not held by the client. (and 14 other errors)
Error: failed to update packages, see the log.
Error: Command 'cipd ensure -log-level error -root C:\\Users\\gspencer\\code\\engine -ensure-file C:\\Users\\gspencer\\AppData\\Local\\Temp\\tmpwsfsjbep.ensure' returned non-zero exit status 1

I've tried re-running cipd auth-login, blowing away my entire src dir and re-running gclient sync, with no change in results.
When I try and build, I get an error that ninja can't find clang (which is not surprising, given the above error).
This is currently stopping me from doing Windows development on the engine.

Also tried replacing depot_tools package, no effect.

Maybe related https://github.com/flutter/engine/pull/29736 /cc @bdero

When I sync to ed6652f79e7cb3822c9952628d489ee439fb419f (the change right before  flutter/engine#29736), I am able to sync again.
When I sync to 45ee58a41d6bdb1c9143089f2f632964f1e08b6f (the ref for flutter/engine#29736) , it fails with the the above error.
@bdero While I'm totally on board with wanting a working clangd, could we revert this change until we figure out the permissions issues?

@gspencergoog A roller was deployed recently for this. Can you try syncing to the version before the most recent windows clang SDK roll?

I tried syncing to 0e7213cbc4a29fb6ae8ccd28ae63e21e5dde8c7f, which is the change right before #29988, and I still get the error.

Hmm, I can't seem to reproduce this issue... Perhaps running the build once with elevated privileges would work? (For example, with admin powershell)
Either way, since this isn't happening on CI or my local build, would you mind helping me figure out why this is happening for you post-revert?

Sure, happy to help out.

@gspencergoog I was just able to reproduce this by wiping my deps clean with git clean -xdf and then running gclient sync -D!!
Would you mind working around it while I try to figure out how to fix it? You can get around this by running gclient sync as Administrator. Subsequent runs don't seem to require it (probably because the SDK isn't getting updated).
It's a somewhat nontrivial rollback since there are a bunch of changes in the buildroot and it has a roller now.

Sure, since there's a workaround now, we don't need to revert.


Loading bar interceptor throws exception when a transformResponse throws exception.

v0.8.0
Regarding the discussion at #50. I ran into this because my transformResponse was throwing an error (due the response being some error output and not the expected output). I had a very case specific transform because it was part of a $resource.
When an exception occurs in transformResponse the response seems to get rejected with only the error message (not the http response). Therefore rejection.config is always null/undefined in this scenario. Because of how angular loading bar works I was forced to manually check the status of the response and wrap potential breaking code in an if block to stop the exception occurring.
So when a transformResponse throws an exception, angular loading bar throws with the error in #50. I don't think that is a ideal. I see you need the config to do some stuff but perhaps it should only throw if rejection is falsey and if config is also falsey just fail silently?
  'responseError': function(rejection) {
          if (!rejection || !rejection.config) {
            $log.error('Broken interceptor detected: Config object not supplied in rejection:\n https://github.com/chieffancypants/angular-loading-bar/pull/50');
            return $q.reject(rejection);
          }

I've exactly the same problem and agree!
How did you workaround this?
Here is my code:
.factory('appendTransform', function ($http) {
      var appendTransform = function(transform) {
         var defaults = $http.defaults.transformResponse;

         // We can't guarantee that the default transformation is an array
         defaults = angular.isArray(defaults) ? defaults : [defaults];

         // Append the new transformation to the defaults
         return defaults.concat(transform);
      };
      return appendTransform;
   })

and
$http({method: 'GET', url: 'http://myserverurl', data: myData, transformResponse: appendTransform(fetchFromServerParser.getData);
Thank you in advance!

Is there any way to push append a transform response only if the response code is 200, in angular? Instead of always appending it

the same problem

the same problem

the same

+1

+1

v0.9.0 too.
@faceleg @darlanmendonca

+1

This project is dead!


You have been blocked from accessing Midjourney. This happens on repeated or serious ToS violations

You have been blocked from accessing Midjourney. This happens on repeated or serious ToS violations.
My server has 3 accounts that triggered this

I think this library is not handling verify human correctly, my 3 accounts also got banned

Have you managed to fix the ban?

Updated the library and saw this

how  can i  fix it ?
i need my account

as same

I lost 10 accounts(( Has anyone found a solution?

How do I unblock my account? I didn't do anything against the rules and I didn't use any forbidden words.. Is there an email address for Midjorney that can be emailed to them?

How do I unblock my account? I didn't do anything against the rules and I didn't use any forbidden words.. Is there an email address for Midjorney that can be emailed to them?

Same happened to me, I tried the billing support email but they refuse to discuss bans, and since I can't enter the discord server, I think there is nothing we can do.

Same thing happened to me. Last access in discord in November, they banned me in January. No explanations.

Got banned too, would love to know the exact reason. Think either as thought I was a bot (use it like a fidget spinner for OCD ADHD so often 1000s of images a day) or some somewhat risque images got flagged in the rate picture thing (should have paid more for private). Either way it would be nice to know and be able to appeal, though considering how bad the censorship is getting it might not be worth it.

I have also encountered such a problem, how to solve it

Open a new discord account, join MJ discord and ask support about your 'friend's' account ban.
I imagine you could make a new account and use anonymous credit card but don't know if MJ accepts those cards.
Losing a huge number of MJ images with no means to appeal has left a bitter taste. Moved on to SD. Takes longer to get good results but no hypersensitive censor or being forced to pay extra for privacy.

Using the new --cref will Get you Temporarily banned Then permanently banned when you seek support on Discord. As you can see from the photo that was generated and the prompt that was used there was no violation. When I try to seek support I was silenced.

I think Midjourney banned my IP address! WTF! I created new account and it cant even get into midjourney server!

got same problem: https://github.com/erictik/midjourney-api/issues/267

as same...

I think Discord and MidJourney are not user-friendly at all. Accounts get disabled frequently without any explanation. It's impossible to contact the support team to resolve the ban, which is really frustrating. I just purchased MidJourney yesterday, and today I can't use it anymore. I'm very annoyed and disappointed


convert_unifold_to_alphafold.py ?

Is there a convert_unifold_to_alphafold.py script? 😄

This is a good question.  As described in our preprint (Sec 2.2), we make several modifications to the model architecture, and thus, our model cannot convert back to the original Alphafold JAX framework.
If you want to run uni-fold in the Colab, we are very happy to help!

Hi guolinke, will it run in uni-fold-jax?

I think it can't be either. Uni-Fold-Jax uses the same model architecture as AlphaFold.

Thanks for offer to help running Uni-Fold on Colab!
My first attempt at installing Uni-Core failed :(
https://colab.research.google.com/drive/171kK3kCsAyPf3vXi4PcVrN91Af_tQjfk

Hi @sokrypton
I saw the CUDA version is 11.2

| NVIDIA-SMI 460.32.03    Driver Version: 460.32.03    CUDA Version: 11.2

while pytorch is torch.__version__  = 1.12.0+cu113, which is cuda 11.3.
Therefore, as the compilation needs the same CUDA version of the host machine and PyTorch, it failed.
So you can use an image with CUDA 11.3, or use our precompiled docker image:
docker pull dptechnology/unifold:pytorch1.11.0-cuda11.3-latest

As for the model parameters conversion, ping @BaozCWJ for help.

we just add a colab version of Uni-Fold, https://colab.research.google.com/github/dptech-corp/Uni-Fold/blob/main/notebooks/unifold.ipynb

I run the code in Running Uni-Fold( I change some path)as this:
bash run_unifold.sh /home/input_fasta/1.fasta /home/output  /path/to/database/directory  2020-05-01 multimer_ft    /home/Uni-Fold/all_model/multimer.unifold.pt
model name is : multimer_ft     model parameters file is:multimer.unifold.pt(in section Downloading the pre-trained model parameters)
but I meet this bug:

could u help me?

I run the code in Running Uni-Fold( I change some path)as this: bash run_unifold.sh /home/input_fasta/1.fasta /home/output /path/to/database/directory 2020-05-01 multimer_ft /home/Uni-Fold/all_model/multimer.unifold.pt model name is : multimer_ft model parameters file is:multimer.unifold.pt(in section Downloading the pre-trained model parameters)
but I meet this bug:  could u help me?

It looks weird.  Maybe the parameters doesn't download completely.  Could you please  try to  run the following code:
import torch x = torch.load("./multimer.unifold.pt")['ema']["params"]  for k in x.keys():     if "outer_product_mean.layer_norm_out.weight" in k and '0' in k:         print(k)

Hi @rabbit-0001 is this issue solved?

@ZiyaoLi does that mean convert_unifold_to_alphafold.py was created somewhere? :D

@ZiyaoLi does that mean convert_unifold_to_alphafold.py was created somewhere? :D

well this actually means that we do not plan to work on this feature, because of Guolin's comment https://github.com/dptech-corp/Uni-Fold/issues/14#issuecomment-1207314159 , and the fact that training with JAX is not a public demand... I close this issue as @rabbit-0001 seems to lose track of his problem.


Implement manual cell count mode

Added a toggle to the power menu to switch between manual and automatic cell count mode, as well as a slider to set the cell count.
In manual mode, the value from the slider is used.
In automatic, the existing method is used to determine the cell count automatically and the slider value is ignored.
Also moved the value init of the cell voltage slider and warning type button group from load_ini_setting() in main.c to the end of page_power_create() in page_power.c and replaced a few hardcoded values with defines.

just gave it a quick test, appears we need to remove main.c line 192.
that function is called after the ui init and thus the auto-detected value gets overwritten.
lets also up the max cell count to 6, with li-fe and li-ion batteries 6s is a safe configuration.

Was originally going with s_count because if you have two cells in parallel the cell count would be 2 but the battery would be 1S, cell_count is definitely more intuitive though 😄
Also upped max cell count to 6 and removed line 192 from main.c

oh and page_power.c line 210 needs to be adjust to 6; we have 2 new elements on that page now.

Good call on the max element count, didn't even realize I had to adjust that

awesome! LGTM now, maybe @evilC can give it a try as well.

I'll try and get around to it tomorrow

Seems to work, but I have two observations:

For the cell count slider, rotating the dial clockwise decreases the cell count, and anti-clockwise increases. Should this not be the other way around?
I am not sure if this is related to your changes, but the warning does not kick in when the voltage reaches the specified level, it kicks in when it falls below it.
ie I have warning set to 2.8v per cell, and cell count set to 4. I would expect the warning to kick in when the voltage reaches 11.2v, but it does not kick in until it reaches 11.1v

Seems to work, but I have two observations:
1. For the cell count slider, rotating the dial clockwise decreases the cell count, and anti-clockwise increases. Should this not be the other way around?

2. I am not sure if this is related to your changes, but the warning does not kick in when the voltage reaches the specified level, it kicks in when it falls below it.
   ie I have warning set to 2.8v per cell, and cell count set to 4. I would expect the warning to kick in when the voltage reaches 11.2v, but it does not kick in until it reaches 11.1v

Regarding 1, I implemented it so that the increase direction is the same as for the cell voltage, if the directions are different between the cell voltage and cell count slider there's a bug somewhere, if not, maybe it's worth thinking about a feature that lets the user set the direction they have to rotate in in order to increase/decrease values.
Regarding 2, that's not something I changed, as far as I can tell there's two places in the code where the cell voltage is checked and in both places the voltage needs to be lower than the set voltage.
Once in ui_statusbar.c: https://github.com/hd-zero/hdzero-goggle/blob/main/src/ui/ui_statusbar.c#L162
And once inosd.c: https://github.com/hd-zero/hdzero-goggle/blob/main/src/core/osd.c#L217

Regarding 1, I implemented it so that the increase direction is the same as for the cell voltage, if the directions are different between the cell voltage and cell count slider there's a bug somewhere, if not, maybe it's worth thinking about a feature that lets the user set the direction they have to rotate in in order to increase/decrease values.

You are indeed correct. I had not noticed it before, and now I have, I cannot un-notice it. I am guessing this is a standard slider control that is just re-used throughout the UI and it's how that control has been coded. Raised #124 as I think this should be changed

Regarding 2, that's not something I changed, as far as I can tell there's two places in the code where the cell voltage is checked and in both places the voltage needs to be lower than the set voltage. Once in ui_statusbar.c: https://github.com/hd-zero/hdzero-goggle/blob/main/src/ui/ui_statusbar.c#L162 And once in osd.c: https://github.com/hd-zero/hdzero-goggle/blob/main/src/core/osd.c#L217

Raised #125

Oh, and one more thing.
I see "Auto" mode has been preserved - and I am guessing that your code has not changed the fact that the warning code is fundamentally broken in Auto mode? (ie it mis-interprets a 4S 18650 pack at 3v per cell as a 3S pack, and does not warn you of low voltage)
So I am guessing I should raise another issue to request that when in auto mode, the Cell Voltage option should be replaced by a Pack Voltage option.
ie
In Manual mode, you set the cell count and the minimum voltage per cell
In Auto mode, you set the minimum voltage of the supply - it does not try to calculate the cell voltage

Oh, and one more thing.
I see "Auto" mode has been preserved - and I am guessing that your code has not changed the fact that the warning code is fundamentally broken in Auto mode? (ie it mis-interprets a 4S 18650 pack at 3v per cell as a 3S pack, and does not warn you of low voltage)
So I am guessing I should raise another issue to request that when in auto mode, the Cell Voltage option should be replaced by a Supply Voltage option.
ie
In Manual mode, you set the cell count and the minimum voltage per cell (eg 2.8v)
In Auto mode, you set the minimum voltage of the supply (eg 12v) - it does not try to calculate the cell voltage

Nope, didn't change anything about the way auto mode works, I just added the option to set the cell count manually.
Might be worth a discussion whether to keep calling it auto/manual mode at that point, or have something like "per cell" and "total".

@evilC can we please stick to the matters a given PR actually modifies.
i gather there are not further notes on the things that actually did change here.


PIP shows conflicting dependencies

$ pip install argostranslate

Collecting argostranslate
  Using cached argostranslate-1.0.5-py3-none-any.whl (13 kB)
  Using cached argostranslate-1.0.3-py3-none-any.whl (12 kB)
  Using cached argostranslate-1.0-py3-none-any.whl (12 kB)
ERROR: Cannot install argostranslate==1.0, argostranslate==1.0.3 and argostranslate==1.0.5 because these package versions have conflicting dependencies.

The conflict is caused by:
    argostranslate 1.0.5 depends on ctranslate2==1.14.0
    argostranslate 1.0.3 depends on ctranslate2==1.14.0
    argostranslate 1.0 depends on ctranslate2

To fix this you could try to:
1. loosen the range of package versions you've specified
2. remove package versions to allow pip attempt to solve the dependency conflict

ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/user_guide/#fixing-conflicting-dependencies

Configuration/environment:
--------------------------------------------------------------------------------
  Date: Wed Dec 23 15:59:00 2020 CET

                OS : Darwin
            CPU(s) : 12
           Machine : x86_64
      Architecture : 64bit
       Environment : Python

  Python 3.7.7 (default, Mar 23 2020, 17:31:31)  [Clang 4.0.1
  (tags/RELEASE_401/final)]

             numpy : 1.19.4
           IPython : 7.19.0
            scooby : 0.5.6
--------------------------------------------------------------------------------

pip 20.3.3

In this commit I froze pip versions to prevent dependency issues. It seems what is happening is that there is a conflict between argostranslate==1.0 which needs a generic version of ctranslate2 vs later versions which require a specific version. Are you trying to install multiple versions of argostranslate? Maybe try:
pip install --upgrade argostranslate

I just tried a full install from pip and had no issues:
$ virtualenv env
$ source env/bin/activate
$ pip3 install --no-cache-dir argostranslate


Buggy in safari.

Safari version 9.1.2
On safari, clicking on the input field works but clicking on the numbers does not. The weird behavior is sometime clicking on the numbers the calendar popups for a second and then goes away.

I have Safari 10.1 and it seems to be working fine for me. Is this behavior special for that specific version?

Closing due to inactivity.


Create New (non-classic) Dynatrace dashboards using environment tokens

Is your feature request related to a problem? Please describe.
I work in a large organisation with many environments, using OAuth tokens that have access to all environments will be a non-starter and very impractical to creating new (non-classic) dashboards in environments, in that users won't be able to self serve their creation with.
See:
https://github.com/dynatrace-oss/terraform-provider-dynatrace/blob/main/dynatrace/api/documents/document/service.go#L55
Describe the solution you'd like
The ability to create dashboards using terraform with environment access tokens similar to dashboard classic:
https://registry.terraform.io/providers/dynatrace-oss/dynatrace/latest/docs/resources/json_dashboard
Describe alternatives you've considered
Using dashboard classic terraform and manually upgrading the dashboards
Additional context
We would like to know what is on the roadmap so we can plan accordingly.
Many thanks

Hello @ukho-cfreeman, apologies for the late response here.
Your request is not entirely new. Quite a lot of customers and also Dynatrace internal users have stumbled across the fact, that the subject of an OAuth client currently needs to come with unrealistically high permissions.
All that originates from the fact, that the first REST API of Dynatrace, that required OAuth was the Account Management API.
There already exists an internal Dynatrace R&D ticket with the topic of lifting requirements here. I just cannot tell you an ETA for that.
When it comes for authentication via environment token for that REST API there is nothing scheduled yet, I'm afraid. We would gladly update the Terraform Provider in case that changes anytime in the future.
You may want to file a support ticket on the Dynatrace portal, if this is an urgent issue for you. Our Support Team is able to fast track topics that are getting reported via Zendesk ticket over and over again. We from the Terraform team unfortunately only have limited influence here.
Just drop us a message here, in case we haven't clarified your request in full. I will gladly re-open the ticket.


The website can't be opened anymore

https://logoly.pro/

where is your location?

Sorry, I notice here a bug, i will fix it today.

i revert some commit, and online again, try it @helloxkk


Fix #9010 - Swiss Public Transportation shows departure time in the past

Description:
http://transport.opendata.ch/ changed their API and now also returns the previous departure.
Related issue (if applicable): fixes #9010

Hi @michaelhertig,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!


update: sync release branch for 0.11.3 patch

What type of PR is this?
/kind cleanup
Any specific area of the project related to this PR?
/area build
Does this PR require a change in the driver versions?
What this PR does / why we need it:
For https://github.com/falcosecurity/falco/issues/2647, this syncs the 0.11.x release branch with the commits of the following  PRs:

https://github.com/falcosecurity/libs/pull/1148
https://github.com/falcosecurity/libs/pull/1149
https://github.com/falcosecurity/libs/pull/1160
https://github.com/falcosecurity/libs/pull/1168
https://github.com/falcosecurity/libs/pull/1173

Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
NONE

LGTM label has been added.  Git tree hash: 8c714c5e6032a6b6f53bfbb3e1e81fca5d69a016

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP, jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP,jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Error on opening Login Modal

For Newsletter there is no error but for login modal its getting error

Hi @razzul,
Fixed in e50d51a19d766d0850469f377c58c4d630fcdce9


Make generate_make_env module output install_yamls Makefile default values

Since edpm_prepare role needs to be aware of some default values we need to get into Ansible the already parsed Makefile default values.
Overridden values by the 'cifmw_install_yamls_vars' are passed along already overridden.
This PR has:

[ ] Appropriate testing (molecule, python unit tests)
[ ] Appropriate documentation (README in the role, main README is up-to-date)

Waiting for the e2e job to finish to double-check generated content, but it seems to be OK at first look


Amok crashes when opening chrome's developer tools

hi there, every time I open the chrome's developer tools when running Amok it blows and I get this error:
2015-12-08 22:54 node[33590] (FSEvents.framework) FSEventStreamFlushSync(): failed assertion '(SInt64)last_id > 0LL'
Amok was launched by this command: amok --browser chrome --hot index.js
If I keep the devtools closed it works just fine.
My env is:
OS: OSX El Capitan version 10.11.1 (15B42)
Chrome: Version 47.0.2526.80 (64-bit)
Node: Version 4.2.1
Thanks for your nice work.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/28962929-amok-crashes-when-opening-chrome-s-developer-tools?utm_campaign=plugin&utm_content=tracker%2F12689110&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F12689110&utm_medium=issues&utm_source=github).

2015-12-08 22:54 node[33590] (FSEvents.framework) FSEventStreamFlushSync(): failed assertion '(SInt64)last_id > 0LL'

That's from chrome, no stack traces from node dumped? (should complain about a null socket unless its a new case)

it always start with this stack:
λ ~/code/amok-test/ amok --browser chrome --hot index.js
2015-12-08 23:34:43.347 Google Chrome[37669:11341621] NSWindow warning: adding an unknown subview: <FullSizeContentView: 0x7f9e5373e9d0>. Break on NSLog to debug.
2015-12-08 23:34:43.348 Google Chrome[37669:11341621] Call stack:
(
	0   AppKit                              0x00007fff9c854807 -[NSThemeFrame addSubview:] + 107
	1   AppKit                              0x00007fff9c85454c -[NSView addSubview:positioned:relativeTo:] + 211
	2   Google Chrome Framework             0x0000000107affa43 ChromeMain + 51652483
	3   Google Chrome Framework             0x0000000107afe89c ChromeMain + 51647964
	4   Google Chrome Framework             0x0000000107b6b5d1 ChromeMain + 52093713
	5   Google Chrome Framework             0x0000000107abc77c ChromeMain + 51377340
	6   Google Chrome Framework             0x0000000107ac6d22 ChromeMain + 51419746
	7   Google Chrome Framework             0x0000000107bd6b4e ChromeMain + 52533390
	8   Google Chrome Framework             0x0000000107c0fc4a ChromeMain + 52767114
	9   Google Chrome Framework             0x0000000107c0f055 ChromeMain + 52764053
	10  Google Chrome Framework             0x0000000107c0ea82 ChromeMain + 52762562
	11  Google Chrome Framework             0x0000000107c0e1bd ChromeMain + 52760317
	12  Google Chrome Framework             0x0000000107c0dd0d ChromeMain + 52759117
	13  Google Chrome Framework             0x0000000107c0bd8f ChromeMain + 52751055
	14  Google Chrome Framework             0x0000000107c0bb13 ChromeMain + 52750419
	15  Google Chrome Framework             0x0000000107c0afb1 ChromeMain + 52747505
	16  Google Chrome Framework             0x0000000104a5d77f ChromeMain + 656575
	17  Google Chrome Framework             0x0000000104a5c69a ChromeMain + 652250
	18  Google Chrome Framework             0x0000000107e5bd4a ChromeMain + 55175818
	19  Google Chrome Framework             0x000000010810abf7 ChromeMain + 57989431
	20  Google Chrome Framework             0x0000000107e5a401 ChromeMain + 55169345
	21  Google Chrome Framework             0x0000000107e5e20d ChromeMain + 55185229
	22  Google Chrome Framework             0x0000000107e57f4f ChromeMain + 55159951
	23  Google Chrome Framework             0x0000000104f5045c ChromeMain + 5845404
	24  Google Chrome Framework             0x0000000104f4fa96 ChromeMain + 5842902
	25  Google Chrome Framework             0x00000001049bd302 ChromeMain + 66
	26  Google Chrome                       0x00000001049aef39 main + 9
	27  Google Chrome                       0x00000001049aef24 Google Chrome + 3876
)

it works just fine until I open the devtools, then boom:
2015-12-08 23:34 node[37668] (FSEvents.framework) FSEventStreamFlushSync(): failed assertion '(SInt64)last_id > 0LL'

After that the app continues to work on the browser, of course, but changes in the source doesn't get patched anymore and no other error arise.

UPDATE: the issue only happens when installing Amok from npm (1.1.3).
Now I've installed it from master and works fine, but I had to change the event listener from patch to scriptChange in order to trigger the UI update. I've also noticed it opens a separate tab with the dev tools, is it by design?

So for amok to disconnect is here expected, TL;DR:
Navigate to http://localhost:9222 and click your target there instead of clicking inspect on the page.

The error is just a bit different than what i would expect. It should crash but it should crash with JavaScript error and a stack trace not C++ assertion failures. Anyhow I digress.
The reason amok disconnects is because chrome forces it to. Debugging sessions cannot be shared with multiple clients. Amok is a client and chrome is a client. Chrome of course takes priority and just kills any current connection when you open embedded devtools via "inspect element".
To workaround this, amok ships with a websocket multiplexer that lets both devtools and amok (and firefox devtools if you prefer those) connect via a single connection to the target browser.
Poorly documented, I know. There's this blog post<EMAIL_ADDRESS>

Good to know then, I'll take your advice as it works for now, thanks a lot.
Just for curiosity, what's in your roadmap? I find Amok's approach much better then using eval...

In no particular order, the things my post-it notes say.

Node.js support.
It's in https://github.com/caspervonb/amok/pull/164. However that branch is like a hodgepot of hackery and needs to be cherry picked / re-done.

With Node.js support comes a rewrite of the executable script, tests need to be re-written to provide proper coverage.

With Node.js support the test suite for Amok.js is obsolete.

With Node.js support the documentation is obsolete, it's already somewhat out of sync.

The supporting modules need some TLC as-well, e.g I need to push releases on more or less each one, test coverage is subpar (e.g am currently polishing http://node-browser_process.js.org)

The protocol layer node-rdbg has zero coverage, and has known minor issues (e.g crashing instead of throwing an error when socket is null is likely coming from ws in there)

Decide what to do about the "plugin" API.

This is a bigger one, right now a "plugin" is essentially just initialisation "middleware", a function that is run before the "Runner/Harness" attempts to make a connection to its target client. It's just the bare minimum the CLI needs to function.
Since the next release will be a major release, I want to make sure to break everything now and not now and the again the next release.

Transforms!

This is a big one, A while back I wrote a plugin for browserify called amokify. I wouldn't expect it to work now but its a proof of concept that we can do non-destructive re-evaluation as an additional layer to what hotpatching offers (read this for more detail, demo, etc<EMAIL_ADDRESS>Now I targeted ES5, and v8 is essentially compatible with ES6 now. So had to scrap the implementation and need to R&D again.

Be better than browsersync

Because we have a debugger connection, and can record input events, screencasts, etc. We can do way better than browsersync in terms of its mutlicasting. This has to be after everything else, but it's ahead of us.

Mobile

Waay down the line, don't even have a test device anymore.

So to summarise, there's a bunch of tedious maintaince work to do. The two major features that are inbound would be Node.js support and re-evaluation transforms.

Hi Casper, thanks for the insights, the roadmap seems very promising. I'm quite interested in Amok, I think it has lots of potential, what I like the most is the fact it is using Chrome's debugging protocol instead of userland eval for hot patching the sources.


uv_pipe_bind always fails with EACCESS on Windows 8.1

It looks like the lpName parameter for CreateNamedPipe() specifically requires that the pipe name be in the format "\\.\pipe\pipename" (see https://msdn.microsoft.com/en-us/library/windows/desktop/aa365150(v=vs.85).aspx).  However, it doesn't appear as though this format is being detected or enforced in pipe.c.  This seems to create a "Catch-22."
When attempting to use "r = uv_pipe_bind(&server, "echo.sock"), it always throws a EACCESS error, and when attempting to use "r = uv_pipe_bind(&server, "\\.\pipe\pipename"), it compiles, but also throws an error when run.

What do you expect libuv to do? I don't really understand the report. You can specify the pipe name you want, if it's bogus an error will be produced, as you can see. Same as if you try to bind to /path/that/does/not/exist on Unix.
Pipes on Windows are represented by those names, we cannot make them consistent.

What this means is, as a developer, I have to do something like this:
#ifdef _WIN32
string pathname = "\\.\pipe\pipename"
#else
string pathname = "pipename"
...which sort of defeats the purpose of trying to build a cross-platform library.  Since you already are using separate implementations for Win/Unix, it Would Be Nice (tm) if on the Windows-side of the house, you checked to see if the pathname being sent in was compliant, and if not either (a) add the requisite "\\.\pipe\" prefix, or (b) throw a UV_EINVAL error (same error thrown when the pathname is null - which is also invalid).
Currently, if the 2nd option is attempted (specifying the compliant \\.\pipe\pipename), the winapi throws an unspecified error.  To get it working, you actually have to use this instead: "\\\\.\\pipe\\pipename" due to backslash resolution rules.

...which sort of defeats the purpose of trying to build a cross-platform library.

Usually I would agree, but not on this one. Unix domain sockets and Windows named pipes are only equivalent up to a given extent. On Windows they live in some limbo, not the actual filesystem.
Now, I wouldn't object a patch which checks if the given name starts with the required prefix and fails with UV_EINVAL otherwise.

Closing, there is nothing actionable here. @Tulrath feel free to send a PR with the extra check I mentioned here.


KR Translation

Korean translation added

So sorry for the long delay on this! Thanks so much for your contribution!  You rock, @nmsohn!

Whoops! Performing a little fix in https://github.com/github/opensourcefriday/pull/560


beartype error: unhashable type: 'list'

I installed the code and its dependencies. When I do from muse_maskgit_pytorch import VQGanVAE, MaskGit, MaskGitTransformer, it fails in muse_maskgit_pytorch when importing beartype:
/home/forrest/anaconda3/envs/mar2023_maskgit/lib/python3.9/site-packages/torch/onnx/_internal/_beartype.py:30: UserWarning: unhashable type: 'list'
  warnings.warn(f"{e}")
Traceback (most recent call last):
  File "/scratch/code/muse-maskgit-pytorch/./fni_main.py", line 3, in <module>
    from muse_maskgit_pytorch import VQGanVAE, MaskGit, MaskGitTransformer
  File "/scratch/code/muse-maskgit-pytorch/muse_maskgit_pytorch/__init__.py", line 2, in <module>
    from muse_maskgit_pytorch.muse_maskgit_pytorch import Transformer, MaskGit, Muse, MaskGitTransformer, TokenCritic
  File "/scratch/code/muse-maskgit-pytorch/muse_maskgit_pytorch/muse_maskgit_pytorch.py", line 16, in <module>
    from beartype import beartype
  File "/home/forrest/anaconda3/envs/mar2023_maskgit/lib/python3.9/site-packages/beartype/__init__.py", line 57, in <module>
    from beartype._decor.decormain import (
  File "/home/forrest/anaconda3/envs/mar2023_maskgit/lib/python3.9/site-packages/beartype/_decor/decormain.py", line 23, in <module>
    from beartype._conf.confcls import (
  File "/home/forrest/anaconda3/envs/mar2023_maskgit/lib/python3.9/site-packages/beartype/_conf/confcls.py", line 23, in <module>
    from beartype._cave._cavemap import NoneTypeOr
  File "/home/forrest/anaconda3/envs/mar2023_maskgit/lib/python3.9/site-packages/beartype/_cave/_cavemap.py", line 33, in <module>
    from beartype._util.hint.nonpep.utilnonpeptest import (
  File "/home/forrest/anaconda3/envs/mar2023_maskgit/lib/python3.9/site-packages/beartype/_util/hint/nonpep/utilnonpeptest.py", line 21, in <module>
    from beartype._util.cache.utilcachecall import callable_cached
  File "/home/forrest/anaconda3/envs/mar2023_maskgit/lib/python3.9/site-packages/beartype/_util/cache/utilcachecall.py", line 32, in <module>
    from beartype._util.func.arg.utilfuncargtest import (
  File "/home/forrest/anaconda3/envs/mar2023_maskgit/lib/python3.9/site-packages/beartype/_util/func/arg/utilfuncargtest.py", line 17, in <module>
    from beartype._util.func.utilfunccodeobj import get_func_codeobj
  File "/home/forrest/anaconda3/envs/mar2023_maskgit/lib/python3.9/site-packages/beartype/_util/func/utilfunccodeobj.py", line 21, in <module>
    from beartype._data.datatyping import (
  File "/home/forrest/anaconda3/envs/mar2023_maskgit/lib/python3.9/site-packages/beartype/_data/datatyping.py", line 129, in <module>
    BeartypeReturn = Union[BeartypeableT, BeartypeConfedDecorator]
  File "/home/forrest/anaconda3/envs/mar2023_maskgit/lib/python3.9/typing.py", line 243, in inner
    return func(*args, **kwds)
  File "/home/forrest/anaconda3/envs/mar2023_maskgit/lib/python3.9/typing.py", line 316, in __getitem__
    return self._getitem(self, parameters)
  File "/home/forrest/anaconda3/envs/mar2023_maskgit/lib/python3.9/typing.py", line 421, in Union
    parameters = _remove_dups_flatten(parameters)
  File "/home/forrest/anaconda3/envs/mar2023_maskgit/lib/python3.9/typing.py", line 215, in _remove_dups_flatten
    all_params = set(params)
TypeError: unhashable type: 'list'

I googled "beartype unhashable type import error," but I haven't yet been able to find anything about this error. Does anyone else hit this problem?

I'm using Python 3.9. Is that an ok version to use?
I am using beartype 0.12.0, which is what was automatically installed when I installed the muse-maskgit-pytorch code.

@forresti are you using python 3.9.1 by any chance? https://github.com/beartype/beartype/issues/99

...heh. The classic TypeError: unhashable type: 'list' exception, huh? Yeah. This isn't actually @beartype's fault. This is CPython's fault. Specifically, this is a bug in older implementations of the standard typing module under CPython < 3.9.2.
As @lucidrains suggests, you're almost certainly running an ancient version of CPython 3.9 (e.g., 3.9.0, 3.9.1). You desperately want to upgrade CPython. Of course, you desperately want to upgrade CPython anyway. Obsolete versions of anything are security risks that increase attack surface. Doubly so if the security risk in question is CPython itself. I grimace. :grimacing:


fix: Fixed computed and deprecated attributes

Codecov Report
Attention: Patch coverage is 69.69697% with 20 lines in your changes missing coverage. Please review.

Project coverage is 74.56%. Comparing base (1dbeb1b) to head (6860f71).

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #468      +/-   ##
==========================================
- Coverage   75.40%   74.56%   -0.84%
==========================================
  Files          92       92
  Lines       16544    16551       +7
==========================================
- Hits        12475    12342     -133
- Misses       3228     3374     +146
+ Partials      841      835       -6

Files
Coverage Δ

zpa/data_source_zpa_policy_type.go
82.14% <ø> (+0.26%)
:arrow_up:

...resource_zpa_application_segment_browser_access.go
60.95% <100.00%> (-1.98%)
:arrow_down:

zpa/resource_zpa_application_segment_pra.go
58.63% <85.71%> (-1.70%)
:arrow_down:

zpa/utils.go
62.84% <0.00%> (-8.76%)
:arrow_down:

... and 12 files with indirect coverage changes


Cleanup header namespace

Remove using from header files.
@JensWehner I'll see if I can get this done at least partly.

[ ] Fix tools headers
[ ] Fix csg files


Menus Update Part XXVII

This is a continuation of  https://github.com/Spuds/Elkarte/tree/createmenusubs and https://github.com/elkarte/Elkarte/pull/3050 in an ongoing saga to tame this monster.
Will need some continued refinement as we find the bugs and other rough edges and check things like the hook calling, compatibility and even using the new methods directly in all places.   But right now it seems to be working and I'm tired of looking at it :P
Also as always there were a few non related bugs I uncovered while working on this, so those are in the PR.
Lastly there were some more bugs in the testing code that this PR also fixes (and why the effected file count is so high), I think that using the phpunit convert to to exception was/is quietly hiding bugs (and not running the assertions either) and I think I got all of those, but you know the old test bed can be a major source of considerable pain in arse.

Codecov Report

:exclamation: No coverage uploaded for pull request base (development@6b41faf). Click here to learn what that means.
The diff coverage is 90.2%.

@@              Coverage Diff               @@
##             development    #3370   +/-   ##
==============================================
  Coverage               ?   20.89%
  Complexity             ?    15824
==============================================
  Files                  ?      382
  Lines                  ?    80288
  Branches               ?        0
==============================================
  Hits                   ?    16777
  Misses                 ?    63511
  Partials               ?        0

Impacted Files
Coverage Δ
Complexity Δ

sources/ElkArte/AdminController/Maintenance.php
0% <ø> (ø)
134 <0> (?)

sources/ElkArte/Modules/Drafts/PersonalMessage.php
0% <0%> (ø)
51 <0> (?)

sources/ElkArte/Controller/Profile.php
0% <0%> (ø)
84 <69> (?)

sources/ElkArte/Util.php
71.68% <0%> (ø)
84 <0> (?)

sources/ElkArte/Controller/PersonalMessage.php
23.54% <100%> (ø)
475 <0> (?)

sources/ElkArte/Menu/MenuItem.php
100% <100%> (ø)
16 <16> (?)

sources/subs/Menu.subs.php
15.02% <100%> (ø)
0 <0> (?)

sources/ElkArte/Menu/MenuSubsection.php
100% <100%> (ø)
8 <8> (?)

sources/ElkArte/Menu/MenuSection.php
100% <100%> (ø)
6 <6> (?)

sources/ElkArte/Controller/ModerationCenter.php
46.08% <100%> (ø)
207 <0> (?)

... and 4 more

Continue to review full report at Codecov.

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

https://github.com/elkarte/Elkarte/pull/3379 pulled the bug fixes and test updates out of this, which I knew would break this PR but had to move on.  This PR has commits dating back 3+ years so is just to difficult to maintain any-longer, needs to be redone if the updates to that menu subs beast is wanted.


pannellum locally in iOS

I have installed pannellum (.htm, .css and .js) in the local iOS DocumentsDirectory and able to run the pannellum, saying this, it means the UIWebView shows perfectly the pannellum.
But when trying to load any image (they are also in the DocumentsDirectory) I always get The file file:///var/....jpg could not be accessed.
It is not a question of the JPG, because in Web-Browser on Mac i can see it perfectly.

This sounds like an iOS, not an issue with Pannellum, possibly a security restriction. Using Safari's remote web debugging might point you in the right direction.
Since Apple forces one to use a Mac for iOS development, I can't help you debug this.

Hi Matthew,
thanks for your message. I have done Safari remote and got the following, see attached screenshot. As you can see, the image is there, but the return code is 0, see the screen on iPhone. What means cod = 0?
Saludos
Dietmar Schwarz Webers
<EMAIL_ADDRESS>Desarrollando y Capacitando para iOS
www.dswdev.com

El 25/08/2017, a las 19:48, Matthew Petroff<EMAIL_ADDRESS>escribió:
This sounds like an iOS, not an issue with Pannellum, possibly a security restriction https://stackoverflow.com/questions/33097019/angularjs-on-wkwebview-any-solution-for-handling-file-on-ios-9. Using Safari's remote web debugging might point you in the right direction.
Since Apple forces one to use a Mac for iOS development, I can't help you debug this.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/mpetroff/pannellum/issues/454#issuecomment-325065756, or mute the thread https://github.com/notifications/unsubscribe-auth/AGn9yROBOYosU12ZuRAznfRhP6gMadT4ks5sb2tUgaJpZM4PCBRh.

You need to use GitHub's web interface to add images; your screenshot didn't make it.