HADOOP-17288. Use shaded guava from thirdparty. (branch-3.3)

https://issues.apache.org/jira/browse/HADOOP-17288

The test results are at
https://ci-hadoop.apache.org/blue/organizations/jenkins/hadoop-multibranch/detail/PR-2505/1/tests
Not sure, why there is no comment from jenkins here

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
30m  7s
Docker mode activated.

-1 :x:
patch
0m 11s
https://github.com/apache/hadoop/pull/2505 does not apply to branch-3.3. Rebase required? Wrong Branch? See https://cwiki.apache.org/confluence/display/HADOOP/How+To+Contribute for help.

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2505/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2505

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2505/3/console

versions
git=2.7.4

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

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

_ Prechecks _

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

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

-1 :x:
pathlen
0m  0s
The patch appears to contain 5 files with names longer than 240

+1 :green_heart:
test4tests
0m  1s
The patch appears to include 413 new or modified test files.

_ branch-3.3 Compile Tests _

+0 :ok:
mvndep
4m  1s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
28m 14s
branch-3.3 passed

+1 :green_heart:
compile
15m 40s
branch-3.3 passed

+1 :green_heart:
checkstyle
27m 21s
branch-3.3 passed

+1 :green_heart:
mvnsite
22m  7s
branch-3.3 passed

+1 :green_heart:
shadedclient
15m  0s
branch has no errors when building and testing our client artifacts.

-1 :x:
javadoc
5m 27s
root in branch-3.3 failed.

+0 :ok:
spotbugs
0m 46s
Used deprecated FindBugs config; considering switching to SpotBugs.

+0 :ok:
findbugs
0m 26s
branch/hadoop-project no findbugs output file (findbugsXml.xml)

-1 :x:
findbugs
29m 46s
root in branch-3.3 has 3 extant findbugs warnings.

+0 :ok:
findbugs
0m 32s
branch/hadoop-client-modules/hadoop-client-minicluster no findbugs output file (findbugsXml.xml)

-1 :x:
findbugs
0m 44s
hadoop-cloud-storage-project/hadoop-cos in branch-3.3 has 1 extant findbugs warnings.

+0 :ok:
findbugs
0m 36s
branch/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-tests no findbugs output file (findbugsXml.xml)

-1 :x:
findbugs
0m 49s
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-timelineservice-hbase-tests in branch-3.3 has 2 extant findbugs warnings.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 38s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
59m 41s
the patch passed

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

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

-0 :warning:
checkstyle
27m 20s
root: The patch generated 191 new + 25239 unchanged - 13 fixed = 25430 total (was 25252)

+1 :green_heart:
mvnsite
20m 32s
the patch passed

+1 :green_heart:
shellcheck
0m  0s
There were no new shellcheck issues.

+1 :green_heart:
shelldocs
0m 22s
There were no new shelldocs issues.

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

+1 :green_heart:
xml
0m 49s
The patch has no ill-formed XML file.

+1 :green_heart:
shadedclient
15m 51s
patch has no errors when building and testing our client artifacts.

-1 :x:
javadoc
5m 28s
root in the patch failed.

+0 :ok:
findbugs
0m 25s
hadoop-project has no data from findbugs

+0 :ok:
findbugs
0m 31s
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-tests has no data from findbugs

+0 :ok:
findbugs
0m 29s
hadoop-client-modules/hadoop-client-minicluster has no data from findbugs

_ Other Tests _

-1 :x:
unit
555m 38s
root in the patch passed.

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

1029m 57s

Reason
Tests

Failed junit tests
hadoop.yarn.applications.distributedshell.TestDistributedShell

hadoop.yarn.server.resourcemanager.scheduler.fair.TestFairSchedulerPreemption

hadoop.yarn.server.resourcemanager.TestRMHATimelineCollectors

hadoop.tools.dynamometer.TestDynamometerInfra

hadoop.hdfs.server.namenode.ha.TestHAAppend

hadoop.hdfs.server.datanode.TestBPOfferService

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2505/4/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2505

Optional Tests
dupname asflicense shellcheck shelldocs compile javac javadoc mvninstall mvnsite unit shadedclient xml findbugs checkstyle

uname
Linux 948f20ecfece 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
branch-3.3 / 4628647

Default Java
Private Build-1.8.0_275-8u275-b01-0ubuntu1~16.04-b01

pathlen
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2505/4/artifact/out/pathlen.txt

javadoc
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2505/4/artifact/out/branch-javadoc-root.txt

findbugs
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2505/4/artifact/out/branch-findbugs-root-warnings.html

findbugs
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2505/4/artifact/out/branch-findbugs-hadoop-cloud-storage-project_hadoop-cos-warnings.html

findbugs
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2505/4/artifact/out/branch-findbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-timelineservice-hbase-tests-warnings.html

checkstyle
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2505/4/artifact/out/diff-checkstyle-root.txt

javadoc
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2505/4/artifact/out/patch-javadoc-root.txt

unit
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2505/4/artifact/out/patch-unit-root.txt

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2505/4/testReport/

Max. process+thread count
4379 (vs. ulimit of 5500)

modules
C: hadoop-project hadoop-maven-plugins hadoop-common-project/hadoop-auth hadoop-common-project/hadoop-common hadoop-common-project/hadoop-nfs hadoop-common-project/hadoop-kms hadoop-common-project/hadoop-registry hadoop-hdfs-project/hadoop-hdfs-client hadoop-hdfs-project/hadoop-hdfs hadoop-hdfs-project/hadoop-hdfs-httpfs hadoop-hdfs-project/hadoop-hdfs-nfs hadoop-hdfs-project/hadoop-hdfs-rbf hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-web-proxy hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-applicationhistoryservice hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-timelineservice hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-tests hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-sharedcachemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-timeline-pluginstorage hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-timelineservice-hbase/hadoop-yarn-server-timelineservice-hbase-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-timelineservice-hbase/hadoop-yarn-server-timelineservice-hbase-client hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-timelineservice-hbase/hadoop-yarn-server-timelineservice-hbase-server/hadoop-yarn-server-timelineservice-hbase-server-1 hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-timelineservice-hbase-tests hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-router hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-timelineservice-documentstore hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-applications-distributedshell hadoop-mapreduce-project/hadoop-mapreduce-client hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-common hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-shuffle hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-hs hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-jobclient hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-services/hadoop-yarn-services-core hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-services/hadoop-yarn-services-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-csi hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-nativetask hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-uploader hadoop-mapreduce-project/hadoop-mapreduce-examples hadoop-tools/hadoop-distcp hadoop-tools/hadoop-dynamometer/hadoop-dynamometer-workload hadoop-tools/hadoop-dynamometer/hadoop-dynamometer-infra hadoop-tools/hadoop-archives hadoop-tools/hadoop-archive-logs hadoop-tools/hadoop-gridmix hadoop-tools/hadoop-aws hadoop-tools/hadoop-kafka hadoop-tools/hadoop-azure hadoop-tools/hadoop-aliyun hadoop-tools/hadoop-sls hadoop-tools/hadoop-azure-datalake hadoop-tools/hadoop-fs2img hadoop-client-modules/hadoop-client-minicluster hadoop-cloud-storage-project/hadoop-cos . hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-timelineservice-hbase/hadoop-yarn-server-timelineservice-hbase-server/hadoop-yarn-server-timelineservice-hbase-server-2 U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2505/4/console

versions
git=2.7.4 maven=3.3.9 shellcheck=0.3.7 findbugs=3.1.0-RC1

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

This message was automatically generated.

@saintstack  Thanx for the review.
To be precise, I didn't change anything in the pom.xml's, I just removed the java code from trunk and applied the patch to branch-3.3 and luckily it had no conflicts, Then I just compiled and the java code got regenerated wrt branch-3.3,
There is a javadoc build failure, that is there in branch-3.3 without my patch as well for aws,
Test failures doesn't look related

Thanx @saintstack for the review, I have rebased and merged this to branch-3.3

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

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

-1 :x:
patch
0m 10s
https://github.com/apache/hadoop/pull/2505 does not apply to branch-3.3. Rebase required? Wrong Branch? See https://cwiki.apache.org/confluence/display/HADOOP/How+To+Contribute for help.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hadoop/pull/2505

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2505/5/console

versions
git=2.17.1

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

This message was automatically generated.


feat: fixes button become inactive after first click on GetResult

🛠 Fixes #171

👨💻 Changes proposed
changes were made to the animator.ts file

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers
event listener for the copy code btn had some issues and have been fixed

📷 Screenshots

@JC-Coder build for this PR failed, there's an error, and running npm run build on your local machine will tell you what went wrong

Closing it means I can't merge

i've fixed the issues and i'll create a fresh request . hopefully i pass all tests


Redirect after saving based on permissions

If no browser permission saving will redirect to the same page.
One common use case scenario is while a user without browse_users permission saves his profile.
Fixes #2625

Would love to see a test for this with someone who hasn't the permission to browse.

I'll try to add it in a few days.

@emptynick
I added the test inside UserProfileTest since there is already a test for saving with browse permission there and is also the place where the "no browse permission" problem is more likely to happen.


Question about hiding/showing columns

Question:  how hard it is it to develop an interaction of hiding and showing columns? Example: say the job list table only showed four columns by default but if you clicked a giant plus button it would show all the rest of the columns. NOTE: This is really just an informational question and not an interaction we should commit to at this time!

I did a bit of research. It looks like it may be possible but difficult, I'd suggest we just keep it as is with the horizontal scroll and then make sure users can download it and have a different browsing experience


Use is_extension_available throughout the code base

A number of tests have got their own code using CL_DEVICE_EXTENSIONS to check for the presence of extensions. All tests should be using is_extension_available and it should be modify to handle error by throwing exceptions.

I will work on this

Thanks! Given the scale of the change, this is probably best tackled with several PRs.


Docker-compose.yml and Bridge.sol updated security - Separate Environment Variables

# 💡Separate Environment Variables

Instead of defining environment variables directly in the docker-compose.yml file, consider using a separate .env file to store sensitive information like passwords. This enhances security and makes it easier to manage different configurations for different environments.

👉Guide:
Separate Environment Variables:

Create a .env file in the same directory as your docker-compose.yml file with the following content:

- [1] MYSQL_DATABASE=relayer
- [2] MYSQL_ROOT_PASSWORD=passw00d

Bridge.sol

Function Visibility:
Some functions like suspendMessages,  banAddress, and processMessage have external visibility, meaning they can be called from outside the contract. These functions perform critical operations such as modifying the state of the contract (banning addresses, suspending messages). This broadens the attack surface and could lead to unauthorized actions if not properly secured.

PR - value:

Main intend is to secure the MYSQL_DATABASE and MYSQL_ROOT_PASSWORD Key values.

@cryptoryda Please create two separate PRs, one for each package. We will not merge PRs that change more than one at the same time. Also make sure all the linting passes (semvar titles etc).


Add field ManegedBy to application profile

Sorry, we do not accept changes directly against this repository. Please see
CONTRIBUTING.md for information on where and how to contribute instead.

we don't need this to add a label


Gctoolkit can not analyze the timeOfLastEvent properly

In preunified CMS log, the last event will be a ParNew Event. However, the mailbox in GCToolkitVert is CMS_TENURED_POOL_PARSER_OUTBOX which is defined in initMailBox.

The timeOfLastEvent in below Log is 2022-01-07T02:04:00.482+0800 + duration rather than 2022-01-07T02:03:53.462+0800 + duration.
2022-01-07T02:03:53.462+0800: 3118.557: [CMS-concurrent-reset: 0.011/0.011 secs] [Times: user=0.02 sys=0.01, real=0.01 secs]
2022-01-07T02:04:00.482+0800: 3125.577: [GC (Allocation Failure) 2022-01-07T02:04:00.482+0800: 3125.577: [ParNew: 909774K->57145K(943744K), 0.0535306 secs] 1498304K->670630K(4089472K), 0.0539072 secs] [Times: user=0.28 sys=0.08, real=0.05 secs]

The related code
private String initMailBox(Diary diary) {
        if ( diary.isG1GC()) return GCToolkitVertx.G1GC_PARSER_OUTBOX;
        if ( diary.isGenerational())
            return (diary.isCMS()) ? GCToolkitVertx.CMS_TENURED_POOL_PARSER_OUTBOX :GCToolkitVertx.GENERATIONAL_HEAP_PARSER_OUTBOX;
        if ( diary.isZGC()) return GCToolkitVertx.ZGC_PARSER_OUTBOX;
        if ( diary.isShenandoah()) return GCToolkitVertx.SHENANDOAH_PARSER_OUTBOX;
        return GCToolkitVertx.GENERATIONAL_HEAP_PARSER_OUTBOX;
}

and
@Override
    public void start() {
        try {
            vertx.eventBus().
                    consumer(mailBox, message -> {
                        try {
                            JVMEvent event = (JVMEvent) message.body();
                            if (event instanceof JVMTermination)
                                return;
                            DateTimeStamp now = event.getDateTimeStamp().add(event.getDuration());
                            if (now.after(timeOfLastEvent)) {
                                timeOfLastEvent = now;
                            }
                        } catch (Throwable t) {
                            LOGGER.throwing(this.getClass().getName(), "start", t);
                        }
                    });
        } catch (Throwable t) {
            LOGGER.throwing(this.getClass().getName(), "start", t);
        }
    }

I trust you will come up with a good solution to this problem.

This is a well known issue… the plan is to provided a unified view of time.

FWIW, the RuntimeAggregation in the samples directory does the right thing here.

RuntimeAggregation can't catch  EventSource.GENERATIONAL and EventSource.TENURED in the same time with the reason that
Verticle of Tenured and Verticle of Generational will construct RuntimeAggregation object respectively.

@ZhangShushu123 - That is true, which is why there is a need for a unified view of time. I created RuntimeAggregation because I had an Aggregation that needed runtime duration to do a calculation and there was no other way to get the runtime. Which is another reason why there is a need for a unified view of time.
One thought is to put runtime duration into the JVMEvent itself, which would probably involve the method you listed above.

The real issue is having a consistent start time. If your log starts from 0.000, this is easy. But if your log starts from 2345432.000 then this is also easy. But if you look starts at 800.000 it starts getting a wee bit trickier to know what value to use. Your choice will all of the analytics that depend on knowing the exact runtime duration.  IMO, the answer is to have JavaVirtualMachine keep a global view of the clock. It will keep the decisions consistent which turns out to be as important if not more important than being precise.

This issue is resolved in the latest PR. I'll push a release for it to Maven Central very shortly


[FEATURE REQUEST] Provide some kind of functionality to help with providing C bindings for a library

As far as I'm aware (I really need to experiment with it more) to create a library in gtk-rs that could be used from other languages (C, Vala, Python, C++, etc.), or from Rust but dynamically with a stable ABI, it would be necessary to:

Write unsafe extern "C" bindings around any methods that should be exposed.
Manually write a .h header, with necessary transfer, nullable, etc. annotations.
Run g-ir-scanner to generate metadata from the header.

This isn't terrible, but I wonder if it would be possible to offer something for this that's kind of the opposite of the gir crate, to help generate the C wrappers and header:

Some sort of proc macro to annotate functions, to generate a C wrapper while writing just safe Rust code using native Rust types.
Automatically generate C headers matching the binding, with the correct transfer and nullable annotations.
Extract rustdoc documentation, and include in header file too.

Ideally this would provide a way to dynamically link a library built in gtk-rs from an application build in gtk-rs without writing anything unsafe on either end. And the library could be used from other languages.

Oh, and I didn't even mention that it would need to expose a _get_type() function, a #define calling that, and anything else needed for binding a GObject.

The _get_type is already exposed through the StaticType trait. I would say, we need a new macro or some attribute for each macro that would generate the extra traits needed for providing bindings.

Yeah, StaticType already covers that, it would just need a #[no_mangle] extern "C" wrapper to be called from outside Rust, which I don't think there is currently. But that's simple enough.

So something like seems to work:
#[no_mangle]
extern "C" fn my_type_get_type() -> glib::ffi::GType {
    MyType::static_type().into_glib()
}

Plus a header file with the usual boilerplate. The hard part is if you want a lot of methods, and need C bindings for those. What could be handy is some kind of proc macros for defining C ABI functions wrapping Rust types. A flexible enough solution would also be helpful for defining trampoline functions for callbacks, and for methods when defining IsSubclassable.
This seems problematic with gtk3-rs/gtk4-rs due to the INITIALIZED logic, since the caller of the cdynlib won't be setting that even if it has called gtk_init.

This seems problematic with gtk3-rs/gtk4-rs due to the INITIALIZED logic, since the caller of the cdynlib won't be setting that even if it has called gtk_init.

https://gtk-rs.org/gtk3-rs/stable/latest/docs/gtk/fn.set_initialized.html

It's not clear where to call that without the library having it's own init function, which seems like it shouldn't be necessary for every C library providing widgets.
I ran into this just calling g-ir-scanner. I don't think calling static_type() on a GtkWidget subclass should actually require that GTK is initialized, but seems to be checking that somewhere. And trying to call set_initialized in the *_get_type() function errors with "GTK was not actually initialized". Calling gtk4::init() there works but seems like it's definitely the wrong solution.

I have a plan for doing this using the class macro: https://github.com/jf2048/gobject/issues/28


#185 Allow for caching of non-GET requests

I believe there's a use-case for allowing some non-GET requests to be cacheable.
Given restrictions on the length of HTTP requests, it is sometimes necessary to embed data into requests. Elasticsearch is an example that makes extensive use of GET requests using data, but this pattern is not possible with FastAPI because OpenAPI/Swagger does not support GET requests with embedded data.
As a result, if you want to use a FastAPI and have an endpoint that can accept large payloads (for example, because you want some sort of proxy in front of Elasticsearch, or a similar service) it is necessary to use a non-GET method, e.g. POST. However, fastapi-cache does not cache such queries.
This PR removes the restriction on non-GET requests so that it is up to the user of FastAPI to determine whether a method should be cached.

I do agree there is a use-case, but simply removing the test here won't be enough. I do think the default should be to only cache GET, HEAD and OPTIONS, as the other common HTTP verbs are expected to affect server state, but it should be possible to override the default.
I'm definitely coming back to this project, soon, to continue implementing the release plan in #145. The plan includes making what methods are cached configurable.


Rubocop Configuration - Phase 2

fixes #42

Performance/ChainArrayAllocation
After benchmarking, I decided to NOT enable that cop, because it forces a coding style that feels backwards and in my opinion less clear (multiple lines vs chained-functional-calls), while there actually is an even more memory-efficient alternative.
In short, the cop is forcing this code
a.map(&:to_i).map { _1 % 3 == 0 ? _1 : nil }.compact.take(100)

to be written this way instead
  a.map!(&:to_i)
  a.map! { _1 % 3 == 0 ? _1 : nil }
  a.compact!
  a.take(100)

in order to be more performant.
And more performant it is. Here is a benchmark I wrote that compares them (code at the end of this message). We can see that what the cop suggests allocates 20 240 times less then the pure functional way.
Comparison:
   performance style:       1054 allocated
    functional style:   21333767 allocated - 20240.77x more

However, this is because we are "holding it wrong" most of the times, and because the implementation in Ruby is kind of evil, probably for backwards compatibility reasons.
TL;DR: Use .lazy.
In other languages, those return "iterators" automatically, making the whole thing "lazy". There are no .map creating a whole new array, and .filter creating yet another one. Those return all iterators and each element flows to the end of the chain before things are used, either as iterator, or "re-assembled" in an array or set or hash or whatever is needed.
Now, if we simply start the chain with .lazy, we end up using even LESS memory – 3 times less in fact – while keeping a more readable and concise coding style.
Comparison:
lazy functional style:       1832 allocated
   performance style:       5694 allocated - 3.11x more
    functional style:   21335307 allocated - 11645.91x more

Full code of the benchmark (using GEM benchmark-memory):
# frozen_string_literal: true

source_a = 1_000_000.times.map(&:to_s).to_a

def functional_style(a)
  a.map(&:to_i).map { _1 % 3 == 0 ? _1 : nil }.compact
end

def functional_lazy_style(a)
  functional_style(a.lazy)
end

def performance_style(a)
  a.map!(&:to_i)
  a.map! { _1 % 3 == 0 ? _1 : nil }
  a.compact!
end

Benchmark.memory do |x|
  x.report("functional style") { functional_style(source_a) }
  x.report("performance style") { performance_style(source_a) }
  x.report("lazy functional style") { functional_lazy_style(source_a) }

  x.compare!
end


Parameters of Python function QueueService.delete_message() do not match the code

The documentation says:
delete_message(queue_name, message_id, pop_receipt, timeout=None)
but the function signature in the package (azure/storage/queue/_queue_client.py) is
delete_message(message, pop_receipt, **kwargs)
When I changed my code to pass the message ID as the first parameter, the code worked.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: ed1f1bda-5060-2297-70bc-bfb7d00e8cd3
Version Independent ID: bc4006ae-8f63-6fac-2d51-0c3f6dabd40e
Content: azure.storage.queue.queueservice.QueueService class
Content Source: previous/docs-ref-autogen/azure-storage-queue/azure.storage.queue.queueservice.QueueService.yml
GitHub Login: @VSC-Service-Account

and BTW, the first parameter in the documentation does not make sense - since this function is a method of QueueService, which is already bound to a specific queue, so specifying a queue_name is redundant.

Thank you for the feedback @Amir-Katz-RSA , we will investigate and get back to you asap.

Hi @Amir-Katz-RSA, thanks for reaching out. The documentation you linked and created the issue on if the legacy documentation for the older versions of the SDK (<=2.X.X) but it seems you are using a later version (>12.X.X) which has a different structure.
Here is a link to the current documentation for the QueueClient in azure/storage/queue/_queue_client.py. Additionally, this landing page for the Python Storage documentation can be helpful place to start to ensure you are looking at current documentation and not legacy. You can check the "Version" drop down on the left side to ensure you are using "Latest",

Thank you. I think the reason I got to the old pages is because the Google search sent me there...

All is good


Creating Access Control Function

Where do you define your access control function? When I attempt to access the session data inside of my app factory or current user from login manager, even when using with app.app_context() I get a working outside of request context error.
def my_access_control_function():
"""
:return: True if the user is allowed to access the filemanager, otherwise False
"""
# You can do whatever permission check you need here
return 'logged_in' in session and session['role'] == 'admin'

It doesn't matter where you define your function.  If you are getting that error you are probably either calling your access control function somewhere instead of just passing it into flaskfilemanager.init or you may have accidentally unindented a line so that it's accessing the session data after the function is declared.
Here is an example from easycms (https://github.com/stevelittlefish/easycms/blob/master/easycms/init.py):
def ffm_ac_fun():
    return ac.can_access_filemanager()

flaskfilemanager.init(app, url_prefix=settings.filemanager_url_prefix,
                      access_control_function=ffm_ac_fun)

Make sure when you pass in your access control function you don't call it (i.e. there are no brackets after ffm_ac_fun on the last line above.

I was importing the tools to detect the current user before the auth and db were initialized, so I just needed to fix my circular dependencies....  works great,and thank you so much!


Generate presets tables for KerasCV

Generate presets tables on both the models page and the method pages.
Update the models page template.
Update the KerasCV index page with the latest README.
Update the API docs for models to only included the migrated backbones and tasks.

DO NOT MERGE till the new release of KerasCV.

Does this PR also fix https://keras.io/api/keras_cv/? It's a bit hard to tell

Does this PR also fix https://keras.io/api/keras_cv/? It's a bit hard to tell

No, it does not. What should we put on that page? Something similar to the README?

Does this PR also fix https://keras.io/api/keras_cv/? It's a bit hard to tell

No, it does not. What should we put on that page? Something similar to the README?

We already have the README info as keras.io/keras_cv. I think you just want some header links under the "Models" section.


Initial FhY textmate Grammar syntax and VS Code configurations

Initial FhY textmate grammar (using json format) defining new language syntax
VS code language config (for basic functionality working with a new coding language in vscode)
vscode launch script for extension development server for debugging purposes
package json for basic package setup

Please note this is just an initial implementation. We will want to provide better functionality to match several positions (e.g. argument variable names of a function, and function name (using the entity.name.function attribute). We will also want to validate basic use against a standard coloring scheme (such as vscode dark+ mode), and token generation of sample fhy code.


CALCULATOR LOGIC

I have added calculator logic.
Added 2 files, calculate.js, and operate.js
Added logic in the Calculator.js to perform basic checks before calling calculator.js function

Hi @emmanuellmhango 🙋♂,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request

[ ]  You have submitted a project, that has the wrong Gitflow. Your Pull Request should be from the feature-calculator-logic into the dev branch.

Make sure that you used correct Gitflow.

Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear. Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.
Invalid Code Review Request does not count into the code reviews limit.

Dear @Bondok6
I have created a new pull request against the dev branch, please check out the new pull request through this link
https://github.com/emmanuellmhango/first-react-project/pull/4


Logging contains newlines (because it uses ByteBuffer.debugDescription which IMHO it shouldn't)

AHC logs bytes of the HTTP connection (which is in itself questionable #689). But even if we think that's okay I don't think it should just log ByteBuffer.debugDescription which contains newlines and is supposed to be human readable.
If AHC thinks it needs to log the bytes then it should do so without logging all the other crap that ByteBuffer.debugDescription contains (especially the newlines, don't think we usually want to have newlines in our log metadata?).
body(ByteBuffer { readerIndex: 0, writerIndex: 98, readableBytes: 98, capacity: 98, storageCapacity: 128, slice: _ByteBufferSlice { 4..<102 }, storage: 0x0000600002c26500 (128 bytes) }
readable bytes (max 1k): [ 20 20 20 7b 0a 20 20 20 20 20 20 22 73 74 61 74 75 73 22 3a 20 22 30 22 2c 0a 20 20 20 20 20 20 22 70 72 73 49 64 22 3a 20 22 31 22 2c 0a 20 20 20 20 20 20 22 77 65 20 77 61 6e 74 20 74 6f 20 68 61 76 65 20 73 6f 6d 65 20 6e 65 77 6c 69 6e 65 73 20 68 65 72 65 22 3a 20 22 22 0a 20 20 20 20 7d ])

Much easier once https://github.com/apple/swift-nio/issues/2447 is done (e.g. by getting https://github.com/apple/swift-nio/issues/2475 in)

Happy to clean this up as well once I'm done with https://github.com/apple/swift-nio/pull/2475.

@weissi, okay, here's the plan:

I'll temporarily switch to natikgadzhi:feature/byte-buffer-hexdump branch for SwiftNIO in Package.swift so I can use hexDump
I'll replace buffer.debugDescription with buffer.hexDump(format: .xxdCompatible(maxLength: 1_000)).
Once Adding ByteBuffer.hexDump in xxd and hexdump -C formats apple/swift-nio#2475 gets merged, I'll cleanup Package.swift and we can merge that in, right?

Works for me assuming you mark it as "Work in Progress" PR such that the PR won't accidentally get merged ;)

Okay! I looked around in the code, and it's not immediately obvious where the buffer gets logged. If you have a hint on where to look — now is the time. My next step is to put together a minimal project that uses AHC so I can debug it stumble at the code that outputs the request/response bytes. Once I find where it is, it should be straightforward to swap out what/how it gets logged.

I think it is coming from here:
https://github.com/swift-server/async-http-client/blob/e1c85a65d82a5fcd49e74e6afb80362d058ca066/Sources/AsyncHTTPClient/ConnectionPool/HTTP1/HTTP1ClientChannelHandler.swift#L117-L119

What I learned today is that the default struct and enum string representation will use the debugDescription of any properties/associated types if available. Therefore the debugDescription of ByteBuffer is picked up here as httpPart is an enum HTTPPart and not directly a ByteBuffer.
After that discovery I think we should rethink the debugDescription of ByteBuffer.

Yep! @dnadoba

I think it is coming from here:

That's what I found, too — assumed that httpPart is some struct / class that wraps or has a bytebuffer property in it, and then it gets serialized into a String with debugDescription() by default.

After that discovery I think we should rethink the debugDescription of ByteBuffer.

Since there is no code that handles specific logging output of a bytebuffer in AHC, and it just logs the whole thing — I agree, it would be nicer to just have ByteBuffer provide a better format for the logs.
@weissi, should we consider something like this in ByteBuffer?
func debugDescription(format: HexDumpFormat = .debug) -> String {
    return self.hexDump(format: format)
}

And then add additional format (let's say there will be .xxd, .hexdump, and .debug). Debug format will default to what it currently outputs — some basic metadata, and the hex dump, where we can replace _storage.dumpBytes with .hexDump(format: .plain).

What I learned today is that the default struct and enum string representation will use the debugDescription of any properties/associated types if available. Therefore the debugDescription of ByteBuffer is picked up here as httpPart is an enum HTTPPart and not directly a ByteBuffer.
After that discovery I think we should rethink the debugDescription of ByteBuffer.

We need to fix HTTPPart. ByteBuffer's debugDescription is useful to debug ByteBuffers (which is what it's meant to do).

debugDescription should just be a version of description that is suitable for structural display as described and changed in this PR: https://github.com/apple/swift-nio/pull/2495.
Afterwards we no longer have new lines in the output and we need to think about https://github.com/swift-server/async-http-client/issues/689 and decided if the full contents of a request/response should actually be included in the trace log or not. We can close this issue then and shift the discussion over to https://github.com/swift-server/async-http-client/issues/689.
@natikgadzhi sorry for kind of taking over this issue. I have discovered this issue in related work over in swift-certificates and as this is against the documented behaviour I though this might be controversial PR.

All good! Thank you for keeping the discussion in the open, so I can follow along. Good to close this, I'll keep an eye on the follow up issues.


Fixes for NUnit3 Final.

Version Property Removed, NUnit3 doesn't use the version in the Tools-Path.
Default ToolName and ToolPath fixed.
IncludeCategory and ExcludeCategory removed as this isn't supported anymore, use the Where Property instead.
Added new properties: Agents, DisposeRunners, Where

Fix for #47

thanks!


For updateChatFilters, in what case will main_chat_list_position not be 0?

As per title.
My assumption would be that if user is subscribed to Premium and moved All Chats to any other position other than the first, it will be 1, 2, 3, etc. Am I correct?

Yes.


Extendable entrypoint plugins

Another variation on #140 with a few of the ideas from the discussion there and #139.
Plugin routers are now nested under a parent Plugin class which now acts as a way to combine multiple related pieces of functionality together (say db management routes and a CLI). This allows new plugin functionality to be added in other plugins or Xpublish related libraries without requiring the parent Plugin class to define everything.
Plugins are loaded from the xpublish.plugin entrypoint group. Plugins can be manually configured via the plugins argument to xpublish.Rest. The specifics of plugin loading can be changed by overriding the .setup_plugins() method.
Some other xpublish.Rest functionality has been refactored out into separate methods to allow easier overriding for instance making a SingleDatasetRest class that will allow simplifying xpublish.Rest.
The ds.rest accessor has been move out into it's own file.

Replacing with #146 which uses Pluggy rather than defining our own plugin management


Winter Witch prestige doesn't give Hex progression to Winter Witch archetype[BUG]

Attach a Bug Report
https://github.com/WittleWolfie/CharacterOptionsPlus#reporting-problems
Describe the bug
Respecced my Lord of Nothing character into a Winter Witch 5/Winter Witch 6 character and for some reason I didn't get any hexes after the initial 2 so at level 11 the character only has 2 hexes.
Screenshots

Fixed in next release


Error when publishing a dataset locally

Currently I'm trying out the ocean.py driver to test the C2D feature, but I'm not able to execute any of the quickstart examples besides the first one.
When trying to execute the publish flow, I'm getting the following error:
AttributeError                            Traceback (most recent call last)
/path/publish_consume.ipynb Cell 3 in <cell line: 44>()
     25 # Publish asset with services on-chain.
     26 # The download (access service) is automatically created, but you can explore other options as well
     27
     28 # Although the execution of this cell generates an error, it creates the asset on chain
     29 asset = ocean.assets.create(
     30     metadata,
     31     publisher_wallet=wallet,
   (...)
     41     datatoken_bytess=[[b""]],
     42 )
---> 44 did = asset.did  # did contains the datatoken address
     45 print(f"did = '{did}'")

AttributeError: 'NoneType' object has no attribute 'did'

So I assume the call is not returning anything at all. I've read that there are some issues when trying to do some stuff locally, and maybe someone has guidelines on how to the C2D example using barge.
I'm running the following command with barge:
./start_ocean.sh --no-dashboard --with-c2d

Everything starts correctly, or at least it seems so, but when trying to run the following code:
# Create Ocean instance
from ocean_lib.example_config import ExampleConfig
from ocean_lib.ocean.ocean import Ocean
config = ExampleConfig.get_config()
ocean = Ocean(config)

# Create wallet
import os
from ocean_lib.web3_internal.wallet import Wallet
private_key = os.getenv('PRIVATE_KEY')
wallet = Wallet(ocean.web3, private_key, config.block_confirmations, config.transaction_timeout)
from ocean_lib.web3_internal.constants import ZERO_ADDRESS

# Publish an NFT token
# It is created correctly
data_nft = ocean.create_data_nft('NFTToken1', 'NFT1', wallet)
print(f"Created data NFT. Its address is {data_nft.address}")

# Specify metadata and services, using the Branin test dataset
date_created = "2021-12-28T10:55:11Z"

metadata = {
    "created": date_created,
    "updated": date_created,
    "description": "Branin dataset 2",
    "name": "Branin dataset",
    "type": "dataset",
    "author": "Trent",
    "license": "CC0: PublicDomain",
}

# ocean.py offers multiple file types, but a simple url file should be enough for this example
from ocean_lib.structures.file_objects import UrlFile
url_file = UrlFile(
    url="https://raw.githubusercontent.com/trentmc/branin/main/branin.arff"
)

# Publish asset with services on-chain.
# The download (access service) is automatically created, but you can explore other options as well

# Although the execution of this cell generates an error, it creates the asset on chain
asset = ocean.assets.create(
    metadata,
    publisher_wallet=wallet,
    files=[url_file],
    datatoken_templates=[1],
    datatoken_names=["Datatoken 1"],
    datatoken_symbols=["DT1"],
    datatoken_minters=[wallet.address],
    datatoken_fee_managers=[wallet.address],
    datatoken_publish_market_order_fee_addresses=[ZERO_ADDRESS],
    datatoken_publish_market_order_fee_tokens=[ocean.OCEAN_address],
    datatoken_publish_market_order_fee_amounts=[0],
    datatoken_bytess=[[b""]],
)

did = asset.did  # did contains the datatoken address
print(f"did = '{did}'")

with the following .env
# For local development with barge
# Mandatory environment variables
OCEAN_NETWORK=development
OCEAN_NETWORK_URL=http://<IP_ADDRESS>:8545/
# Use 5003 https://github.com/oceanprotocol/ocean.py/issues/89#issuecomment-1015554488
METADATA_CACHE_URI=http://<IP_ADDRESS>:5003
PROVIDER_URL=http://<IP_ADDRESS>:8030

# Replace PRIVATE_KEY if needed
PRIVATE_KEY=0xc594c6e5def4bab63ac29eed19a134c130388f74f019bc74b8f4389df2837a58
SECOND_PRIVATE_KEY=0xef4b441145c1d0f3b4bc6d61d29f5c6e502359481152f869247c7a4244d45209
# Required by Ocean.py
ADDRESS_FILE=./address.json

It stalls. Checking the aquarius logs, I see that it creates multiple connections and that it is trying to find a did that apparently is not registered. This problem continues for some time until it returns the error that I mention.
2022-09-02 11:43:59,450 - aquarius.app.es_instance - DEBUG - elasticsearch::read::did:op:0640a540d598548315b3c66b6464ca694907c4ef73fdb0507986dbee4bb40f52
2022-09-02 11:43:59,455 - aquarius.app.es_instance - INFO - Asset with id did:op:0640a540d598548315b3c66b6464ca694907c4ef73fdb0507986dbee4bb40f52 was not found in ES.
2022-09-02 11:43:59,665 - aquarius.app.es_instance - DEBUG - elasticsearch::read::did:op:0640a540d598548315b3c66b6464ca694907c4ef73fdb0507986dbee4bb40f52
2022-09-02 11:43:59,670 - aquarius.app.es_instance - INFO - Asset with id did:op:0640a540d598548315b3c66b6464ca694907c4ef73fdb0507986dbee4bb40f52 was not found in ES.
2022-09-02 11:43:59,705 - aquarius.events.events_monitor - ERROR - Error processing MetadataCreated event: HTTPConnectionPool(host='<IP_ADDRESS>', port=8030): Max retries exceeded with url: /api/services/decrypt (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x40171df8e0>: Failed to establish a new connection: [Errno 111] Connection refused'))
event=AttributeDict({'logIndex': 0, 'transactionIndex': 0, 'transactionHash': HexBytes('0xab3b0cede36d0a90b5f08e8f9368f76f74538c248dc2231c41d00c4d44349027'), 'blockHash': HexBytes('0xafb64fc7a17cff933cc691778a4b657b334d5c7f80c1118a370e9780231ee685'), 'blockNumber': 21, 'address': '0xb38b5A1A0B7c48099a8C53dD179726bf8A416538', 'data': '0x000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000e000000000000000000000000000000000000000000000000000000000000001200000000000000000000000000000000000000000000000000000000000000160ccd7f795ed1ec0fb216ef7d9cb762fdc2e71823e3509f88c57949127cfdde51a000000000000000000000000000000000000000000000000000000006311ec7600000000000000000000000000000000000000000000000000000000000000150000000000000000000000000000000000000000000000000000000000000015687474703a2f2f3132372e302e302e313a383033300000000000000000000000000000000000000000000000000000000000000000000000000000000000000103000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000451049fcc82c5abacf2c0200e8780080e82b094b57b02b72df2f19f742075efe2c00a5f357b9d85b4bae5fc52772165b9313c64422463f0d143f61ed63417013d7daf062368945a9df6e3e345f3c15764a6a0cc56ef17761cffd0e748f0fdeb9a7b47c612ced934b92673ffedcd5626c316bed25bae0078d10cebd0e602f3a6906cbc734e3f9486aba728d123daf669845c9ca9a850bc02599e3add04cbd49a4efed6cec0783230e004c0a6d2813259bafe9fb3f0cc36374840f29d7f3c81a54346f152b71323b2628ae36dcc72eda1cfaad1a7f171bfe24e57ac7fed6084c41badc624d6c58c1f5c9e8e2d009221c3407a50fff3aaba6797ce8831da2cc13232c30f70f4ad1188607a1429aa0a79df39427283d525df86ee7f4ce5d5f0488a0f839bc0105117bd1fc54434cfe233a29917f2f3f1a745a35acd0b5b160fe59158c43e83482ce26dbfa1596ac979d60ac4d9f8b6543ed71bf2a98ec90bdff17ba9777492bffaa0aedd4b79571748c5060ac50a10b85f46e33199f6b29157036b3d470e937e03b436f17e9eed9e19d7072787afc0b38ba9c8054009cbc26a3c1dae096a7e1ddcfe6e6ec1afbfc540e6694ddc9f68944f709301811ceed8be51ce7364289ecb996998b4e16ef550f8b03a6855fd0771195972f50c238a4622cdc9e1ebd421a498fab5a46450c39f7a7950d2ec55bb791ca1876dcf995d0113a5daf86d4e819b3a3b00de3f125b59d731136a90fa78b822d41a742830de2387e8cc147e0df295d4dca249d8d97e414904a3bb993f0a33fc67e575b5b116256e3efca6be3ea3ec1c2381c97c51908db239968ebdb319154eda43d1c14b262bd9738750a93b841b1459b02e957c8c8443f97e530acf6e1cda37460290b55e63b29d7838db207a1721aac3e3d49fda9dd947f61d800ccf3511fe567e7f892f1ebff892754de4e499dcd970f52a8c99381de550ffc214f67e271cfafd0eb4aa7b6b75defb51305f1d70b63b7676936b42119ad4952aa513f8bf0e90e05e6b52ccf02bd8b289fd7d4ac480be9705e5880c5fbefc2d4e9f4b0d5a69de91129b6281e5697c9c7d77c2256ec6b414a6f6050565a9dd7e8290e3889839542e88d7a216e0bbb9d51dbc0d058d9e11136d4a433ddbea10a09b9a647ff4dae7cec76a8a3ff527907bef4d334e2fe5f6bca5ae78551b9c5c6369c761ff186dba80d892b0419a8efe33690866a6fde7fbcffd650fdda01e75d41d8a282b3f724d036b2902a7c09612ce4ae5db740be901caec822fec561a94e68367bc1b52a0934b47afa6b2e0db129aa18da859442487301536ba60a30d6d81130798845f191481b568348350add417016eb663073821bb82e71a9c69f6e5a17f6e9c3d4a776fbb020a78cc7d271109d9cd33d741fe67a4383c55959d98e3af3af935f4db9b85b4c6595ab41f24e1513cc9699cd6ecfe76af157620a30cef11488a3785b8c7c0855b237766b883325b4a330a2e3534d029fc9590dd9274f1e532291b888f822c5d32045cae5128a0dbb22d89eef07d29f1e10f8f8bc20c10845857000000000000000000000000000000', 'topics': [HexBytes('0x5463569dcc320958360074a9ab27e809e8a6942c394fb151d139b5f7b4ecb1bd'), HexBytes('0x000000000000000000000000e2dd09d719da89e5a3d0f2549c7e24566e947260')], 'type': 'mined', 'removed': False})
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/dist-packages/urllib3/connection.py", line 174, in _new_conn
    conn = connection.create_connection(
  File "/usr/local/lib/python3.8/dist-packages/urllib3/util/connection.py", line 95, in create_connection
    raise err
  File "/usr/local/lib/python3.8/dist-packages/urllib3/util/connection.py", line 85, in create_connection
    sock.connect(sa)
ConnectionRefusedError: [Errno 111] Connection refused

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.8/dist-packages/urllib3/connectionpool.py", line 703, in urlopen
    httplib_response = self._make_request(
  File "/usr/local/lib/python3.8/dist-packages/urllib3/connectionpool.py", line 398, in _make_request
    conn.request(method, url, **httplib_request_kw)
  File "/usr/local/lib/python3.8/dist-packages/urllib3/connection.py", line 239, in request
    super(HTTPConnection, self).request(method, url, body=body, headers=headers)
  File "/usr/lib/python3.8/http/client.py", line 1234, in request
    self._send_request(method, url, body, headers, encode_chunked)
  File "/usr/lib/python3.8/http/client.py", line 1280, in _send_request
    self.endheaders(body, encode_chunked=encode_chunked)
  File "/usr/lib/python3.8/http/client.py", line 1229, in endheaders
    self._send_output(message_body, encode_chunked=encode_chunked)
  File "/usr/lib/python3.8/http/client.py", line 1008, in _send_output
    self.send(msg)
  File "/usr/lib/python3.8/http/client.py", line 948, in send
    self.connect()
  File "/usr/local/lib/python3.8/dist-packages/urllib3/connection.py", line 205, in connect
    conn = self._new_conn()
  File "/usr/local/lib/python3.8/dist-packages/urllib3/connection.py", line 186, in _new_conn
    raise NewConnectionError(
urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPConnection object at 0x40171df8e0>: Failed to establish a new connection: [Errno 111] Connection refused

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.8/dist-packages/requests/adapters.py", line 489, in send
    resp = conn.urlopen(
  File "/usr/local/lib/python3.8/dist-packages/urllib3/connectionpool.py", line 785, in urlopen
    retries = retries.increment(
  File "/usr/local/lib/python3.8/dist-packages/urllib3/util/retry.py", line 592, in increment
    raise MaxRetryError(_pool, url, error or ResponseError(cause))
urllib3.exceptions.MaxRetryError: HTTPConnectionPool(host='<IP_ADDRESS>', port=8030): Max retries exceeded with url: /api/services/decrypt (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x40171df8e0>: Failed to establish a new connection: [Errno 111] Connection refused'))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/aquarius/aquarius/events/events_monitor.py", line 228, in handle_regular_event_processor
    event_processor.process()
  File "/aquarius/aquarius/events/processors.py", line 212, in process
    asset = decrypt_ddo(
  File "/aquarius/aquarius/events/decryptor.py", line 33, in decrypt_ddo
    response = requests.post(provider_url + "/api/services/decrypt", json=payload)
  File "/usr/local/lib/python3.8/dist-packages/requests/api.py", line 115, in post
    return request("post", url, data=data, json=json, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/requests/api.py", line 59, in request
    return session.request(method=method, url=url, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/requests/sessions.py", line 587, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/local/lib/python3.8/dist-packages/requests/sessions.py", line 701, in send
    r = adapter.send(request, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/requests/adapters.py", line 565, in send
    raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPConnectionPool(host='<IP_ADDRESS>', port=8030): Max retries exceeded with url: /api/services/decrypt (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x40171df8e0>: Failed to establish a new connection: [Errno 111] Connection refused'))
2022-09-02 11:43:59,717 - aquarius.events.events_monitor - INFO - Searching for MetadataUpdated events on chain 8996 in blocks 20 to 21.
2022-09-02 11:43:59,742 - aquarius.events.events_monitor - DEBUG - Searched blocks 20 to 21 on chain 89960 MetadataUpdated events detected so far.
2022-09-02 11:43:59,742 - aquarius.events.events_monitor - INFO - Finished searching for MetadataUpdated events on chain 8996 in blocks 20 to 21. 0 MetadataUpdated events detected.
2022-09-02 11:43:59,742 - aquarius.events.events_monitor - INFO - Searching for MetadataState events on chain 8996 in blocks 20 to 21.
2022-09-02 11:43:59,757 - aquarius.events.events_monitor - DEBUG - Searched blocks 20 to 21 on chain 89960 MetadataState events detected so far.
2022-09-02 11:43:59,758 - aquarius.events.events_monitor - INFO - Finished searching for MetadataState events on chain 8996 in blocks 20 to 21. 0 MetadataState events detected.
2022-09-02 11:43:59,758 - aquarius.events.events_monitor - INFO - Searching for OrderStarted events on chain 8996 in blocks 20 to 21.
2022-09-02 11:43:59,834 - aquarius.events.events_monitor - DEBUG - Searched blocks 20 to 21 on chain 89960 OrderStarted events detected so far.
2022-09-02 11:43:59,834 - aquarius.events.events_monitor - INFO - Finished searching for OrderStarted events on chain 8996 in blocks 20 to 21. 0 OrderStarted events detected.
2022-09-02 11:43:59,835 - aquarius.events.events_monitor - INFO - Searching for TokenURIUpdate events on chain 8996 in blocks 20 to 21.
2022-09-02 11:43:59,850 - aquarius.events.events_monitor - DEBUG - Searched blocks 20 to 21 on chain 89960 TokenURIUpdate events detected so far.
2022-09-02 11:43:59,851 - aquarius.events.events_monitor - INFO - Finished searching for TokenURIUpdate events on chain 8996 in blocks 20 to 21. 0 TokenURIUpdate events detected.
2022-09-02 11:43:59 8f6a54c50af3 root[74] INFO Stable connection to ES.
2022-09-02 11:43:59,883 - aquarius.app.es_instance - DEBUG - elasticsearch::read::did:op:0640a540d598548315b3c66b6464ca694907c4ef73fdb0507986dbee4bb40f52
2022-09-02 11:43:59,887 - aquarius.app.es_instance - INFO - Asset with id did:op:0640a540d598548315b3c66b6464ca694907c4ef73fdb0507986dbee4bb40f52 was not found in ES.
2022-09-02 11:44:00,101 - aquarius.app.es_instance - DEBUG - elasticsearch::read::did:op:0640a540d598548315b3c66b6464ca694907c4ef73fdb0507986dbee4bb40f52
2022-09-02 11:44:00,106 - aquarius.app.es_instance - INFO - Asset with id did:op:0640a540d598548315b3c66b6464ca694907c4ef73fdb0507986dbee4bb40f52 was not found in ES.
... -> It repeats for a while until it returns the error

I'm using a Mac with an M2 chip but I also tried it using a VM with Linux but the same issue appears.
Desktop:

OS: MacOS using M2 chip/ Linux  - Ubuntu 18.04.6 LTS

Running the same code on Rinkeby works fine and I was able to try some quickstart examples, but to run the C2D example, I'm not able to get anything back from the ocean.assets.create method

Update: I'm still facing the same issue, but it seems like Aquarius is not reading correctly from the chain. I can see transaction events within the ganache container, but the Aquarius container doesn't have anything besides the chain id and the last block.

Close: For some reason, Aquarius won't connect to the provider, and in order to make it work, I had to add to my /etc/hosts the following line <IP_ADDRESS>    ocean_provider_1 and in my .env, I had to replace the PROVIDER_URL with http://ocean_provider_1:8030


JSON parse failure on login

I figure this might be the right place to report this.  I tried running the demo client and got this after allowing access to my password:
cygnus@pickle:~/src/mattermost-demo-client$ .cabal-sandbox/bin/mattermost-demo-client
mattermost-demo-client: JSONDecodeException {jsonDecodeExceptionMsg = "mmGetJSONBody: Error in $: key "last_picture_update" not present", jsonDecodeExceptionJSON = "{"id":"en4wx66nnty75j63irwqzz41cr","create_at":1461001853400,"update_at":1466632136323,"delete_at":0,"username":"cygnus","auth_data":"","auth_service":"ldap","email":"jtd@galois.com","email_verified":true,"nickname":"","first_name":"Jonathan","last_name":"Daugherty","roles":"","last_activity_at":1471926878355,"last_ping_at":1471926885728,"notify_props":{"all":"true","channel":"true","desktop":"none","desktop_sound":"false","email":"false","first_name":"false","mention_keys":"cygnus,@cygnus"},"last_password_update":1461001853400,"locale":"en"}"}

(Fixed in associated pull request.)


zlib 1.2.8 changed download link

hi,
I think they changed the 1.2.8 path, 1.2.9 works.
ERROR: zlib/1.2.8@lasote/stable: Error in source() method, line 27
download("http://zlib.net/zlib128.zip", zip_name)
Error 404 downloading file http://zlib.net/zlib128.zip
Is it possible to find out easily which package uses zlib 1.2.8? I checked all installed conan packages but couldn't find it. Atm I am overriding it with 1.2.9 in my main conan-file
Cheers
Phibedy

Hi @Phibedy
I updated the zlib recipe some days ago, try doing conan install with the -u option to update the zlib/1.2.8@lasote/stable recipe or remove it first from local cache with conan remove zlib/1.2.8@lasote/stable. Please tell me if it works!
About your second question, if you are in a project you can use the conan info command to know about the dependencies but currently we don't have an specific search to know all the packages depending on an specific one. Maybe doing some grep and searching for zlib/1.2.8@lasote/stable could help.

@lasote, ah I just used  conan install <p> --build=outdated, but that does not update the recipe, thank you :)

zlib is now at 1.2.10, so neither http://zlib.net/zlib128.zip nor http://zlib.net/zlib129.zip exist.

@cinderblocks
Conan zlib 1.2.8 and 1.2.9 recipes are not using http://zlib.net/zlibXXX.zip urls anymore.
I download the source code using:
zip_name = "zlib-%s.tar.gz" % self.version
download("http://downloads.sourceforge.net/project/libpng/zlib/%s/%s" % (self.version, zip_name), zip_name)

So if it still fails in you computer use the --build=outdated or the -u option to get the latest recipe.
If someone want collaborate with a Pull Request for 1.2.10 would be great!


@timestamp is not getting when we send logs using bulk in fluentd ?

Getting :
{
"_index" : "index",
"_type" : "fluentd",
"_id" : "id",
"_score" : 1.0,
"_source" : {
"message" : "{"log":"2018-09-13 05:24:56 -0400 [info]: following tail of /var/lib/json.log\n","stream":"stdout","time":"2018-09-13T09:24:56.332149103Z"}",
}
}
Need :
{
"_index" : "index",
"_type" : "fluentd",
"@timestamp" : "timestamp value",
"_id" : "id",
"_score" : 1.0,
"_source" : {
"message" : "{"log":"2018-09-13 05:24:56 -0400 [info]: following tail of /var/lib/json.log\n","stream":"stdout","time":"2018-09-13T09:24:56.332149103Z"}",
}
}

Please help me .

can some one help Asap ?

it is very critical .

Hi Shubham,
Please share your  td-agent.conf file.
I guess you are either not defining time_format and time_key in the conf.

Hi shubham in the source tag, where you are defining the source of your data add the following line-
time_key time_local
time_format %Y%m%d%h%m%s

This is also es plugin setting not for Fluentd core. Closed.


Update the public angle to be generated every time property graph is generated

The public angle must be generated everytime the property graph is generated.
Currently there is one set of up public angles.

This is done.


🛠Fix: Disable Submit Button for Empty Username

Description:
This PR addresses the issue where the submit button remains enabled despite an empty username. The fix disables the submit button when the username input field is empty, improving form validation and user experience.
Closes : #63
Screenshot
https://github.com/user-attachments/assets/4fa5098f-eda4-4292-88c3-847c94c665b0
Checklist:

[x] Tested and verified fix
[x] Updated documentation (if applicable)
[x] Followed coding standards and best practices

Invalid PR, issue not approved
You should wait till your issue is assigned to you


Issue with using default tag with pointer type

TL;DR: When using the mod:"default" tag with a field of type *string, if the field is nil, the library assigns it to a pointer referencing an empty string ("") instead of the defined default value.

Context:
Hi team, thank you for the great work on this library!
While integrating it into my project, I encountered an unexpected behavior and am seeking your advice.
Here’s a simplified example to illustrate the issue:
package main

import (
	"context"
	"fmt"
	"log"

	"github.com/go-playground/mold/v4/modifiers"
)

var conform = modifiers.New()

type User struct {
	Name *string `mod:"default=abc"`
}

func main() {
	user := User{}
	fmt.Println("Name:", user.Name) // Output: nil (as expected)

	err := conform.Struct(context.Background(), &user)
	if err != nil {
		log.Panic(err)
	}

	fmt.Println("Conformed Name:", *user.Name) // Output: "" (unexpected, expected "abc")
}

Problem:
The Name field in the User struct is of type *string and is configured with the mod:"default=abc" tag.
However, if the field is initially nil, the library assigns it to a pointer referencing an empty string ("") instead of a pointer referencing the expected default value ("abc").

Expected Behavior:
If the Name field is nil, it should be assigned to a pointer referencing the default value ("abc") as defined by the mod:"default=abc" tag

Investigation and proposal:
I traced the issue to the following section of code in the modifiers/multi.go file:
	case reflect.Ptr:
		fl.Field().Set(reflect.New(fl.Field().Type().Elem()))
	}

This code initializes a pointer field to point to an empty value (e.g., an empty string).
However, it does not account for the configured default value defined in the mod:"default=..." tag. As a result, the pointer field ends up pointing to an empty string instead of the specified default value.

To fix this, I suggest updating the logic to use the configured default value when initializing the pointer. Here’s a proposed modification:
case reflect.Ptr:
	value := fl.Param() // Retrieve the configured default value
	fl.Field().Set(reflect.ValueOf(&value)) // Set the field to point to the default value
}

Please let me know if this behavior is intentional or if there is a recommended way to achieve the expected behavior. Thanks in advance!

@hnimtadd given your thorough example it does appear to be a bug, but I’ll also have to check it’s been a while since I’ve touched/looked at this code 😅
It might be a while until I can check thought, I’m in the middle of moving and everything including my computers are still packed so may be a week or two before I can.

Yes by all means a PR would be helpful

@hnimtadd you need to fork the repository, make the changes push to your branch and the make upstream PR

This should be working in Release https://github.com/go-playground/mold/releases/tag/v4.5.1


Add a donate button to README.md

The donate button is a link to https://boost-lab.app/tensorflow/tensorflow, which provides potential donors the freedom to financially support the developers of tensorflow.
At Boost Lab, our mission is to give developers the ability to receive continuous income for their open source work with minimal impact to their workflow.
For a more detailed explanation of how our platform works, we would love it if you check out our web app.

Assigning to @wicke for more input.

While appreciating the spirit of the idea, I don't think it's appropriate to do something like this: the majority of our developers are employed by Google or some other corporation. Please see https://opensource.google.com/community/ for some of the ways in which Google supports open source and its developers.

🙏 @ewilderj thank you for your reply. And you are absolutely right. Just by looking at the repo statistics alone, I saw that around 90 - 95% of the contributions are made by Google employees (my estimations are probably off haha). Therefore, financial support is most likely not an issue for those developers.
However, the donate button could be a way for other parties to show appreciation & support. For example, one of the many companies that use tensorflow, or even a YouTuber that teaches a course on this technology. Those parties could want to support tensorflow's development in ways other than just engineering hours.
But, I can see where you are coming from. Because tensorflow is backed by Google, external financial support is not a requirement.
Anyways, we appreciate your feedback. And perhaps Boost Lab is only tailored towards projects that does not have the backing of a tech giant.
BTW, we tried out tensorflow.js and we love what Google is doing with the technology 👍.

Thanks @jing-c! Looks like Boost Lab is a new initiative, so if you want any further conversation on this feel free to email me, and I can go into more detail. ewj at google.


Move daemon and stub to /usr/libexec

I was spending some time investigating how update_engine works, and when first
starting out it took me a little bit to figure out that there were separate
update_engine_client vs update_engine. Having them both in /usr/sbin is
annoying for bash completion.
This is just a minor "papercut"; I'm mostly using this to test running
through the development/contribution process.
I am assuming nothing tries to find these binaries directly and only uses the
systemd service and/or DBus API; I didn't see any obvious hits in repo grep update_engine.
Signed-off-by: Colin Walters<EMAIL_ADDRESS>

LGTM, but I worry if this will break anyone who has systemd dropins for update-engine. If they have a dropin overwriting the ExecStart to add additional flags, this will break them. I'm not sure if there's actually users doing that, but it's something to consider.

Do you (or anyone) have a sense of whether there are flags that people might want to tweak?  update_engine --help doesn't initially look to me like there are any outside of a development scenario but I'm obviously new here :smile:

While I think that this papercut is technically ok, I think that at this point in CL lifetime it is not worth pursuing this with the risk of breaking existing customized setups. I'd be inclined to just close this PR.


Touching the button while scrolling issue / or scrolling is executing but the screen not actually scrolled

Hi guys, I have been using Detox for 2 years now and I have been staying at version 14 for a long time and they have been okay. But as we are thinking of testing on newer OSes and devices we think we might need to update the detox testing version as well. And so far the speed of testing has been great.
However, I ran into this situation constantly where the screen supposed to scroll but it is either not scroll or the scroll animation will also trigger the button pressing.
As you can see in the gif below the screen suppose to keep scrolling but it hung here and there and also the selection supposed to be "AFL" but after the whole animation it changed into "Martial Art". Which was not what we wanted to test and this affected the whole test case.

Thanks in advance.

Dup of #2255


User can provide a custom name for starting a Job

Add a --name option to the nerd job run command that can be used to create a job with a specific name.

It should provide a nice error when the name already exists
if the user doesn't provide a custom name it should generate one that is reasonable human readable

This has been merged as part of #182


[Gekidou MM-48281] Fix issue with single image result extending to full height of the image

Summary
This PR fixes an issue when the file search results are returned with only one image. There is possibly a bug in the calculateDimensions function, but I chose to fix it in the file_results so that I don't have the possibility of messing up some other components that use that function.

Before
After

Ticket Link
https://mattermost.atlassian.net/browse/MM-48281
Checklist

[ ] Added or updated unit tests (required for all new features)
[ ] Has UI changes
[ ] Includes text changes and localization file updates
[ ] Have tested against the 5 core themes to ensure consistency between them.

Device Information
This PR was tested on:
Screenshots

Release Note

LGTM. I think you could also have deleted the isSingleImage prop from file_resul.tsx file as this prop is defaulted to false.

Correct. I made the prop optional in the File component and removed the prop from FileResult

Successfully building:
https://circleci.com/gh/mattermost/mattermost-mobile/25926
https://circleci.com/gh/mattermost/mattermost-mobile/25925

Artifact links:
https://output.circle-artifacts.com/output/job/5330bac1-5815-4d0e-aeca-88a5c2b620a9/artifacts/0/tmp/artifacts/Mattermost_Beta.ipa
https://output.circle-artifacts.com/output/job/4084d94b-d6df-4593-b0ac-285bef2f4b80/artifacts/0/tmp/artifacts/Mattermost_Beta.apk


Bato.to chapters not showing in the app, despite being available in the website

Steps to reproduce
Search manhwas with season 2 in Bato.to for example Sex stop watch
Look at the chapter number then compare to the website
App is lacking the season 2 chapters

Expected behavior
Chapters should show based on whats in the website (like in kotatsu)

Actual behavior
No chapters showing
Crash logs
No response
TachiyomiSY version
1.10.5
Android version
12
Device
Freeyond m5
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version 1.10.5.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

Have you perhaps tried refreshing the page? If you have, and the chapters still do not show, this would seem to be an extension issue rather than an issue with Mihon (or this fork specifically, for that matter). If you would like to, please create an issue through the extension repository source (hint: Keiyoushi).

Closing because its (probably) a extension issue.


20.3.0 results reader

This PR contains methods to read (a few) results.
Fix #20
Fix #109
New Features

[ ] C# Model.ReadStr(strPath, resultTypes, killProcess, endSession, checkOpenFiles)
[ ] Grasshopper: Model.ReadStr is now using ResultType enums (as strings in grasshopper) to read results.
[ ] Grasshopper: Results.ResultType. A GH_ValueList component to create the ResultType enums (as strings).

This should not break any old grasshopper scripts. The old Model.ReadStr has been marked as obsolete.

Notes to self:
Create new branch 20.3.0, checked out from master. 20.3.0-Results-Reader should merge to that branch instead. Before merging with master equivalent nodes in Dynamo should be created.


Track module checksums across all tagged versions of pachyderm go library

What is the goal / desired outcome? We need a way to keep track of checksums across all tagged versions of the pachyderm go library, as any of our dependencies can break our checksum. This will prevent issues like this going forward.

I have done some investigation on what data is required to write such a tool.
I created some example projects so it's easy to reason about what's going on.  github.com/jrockway/evil-module-user is small go program that depends on github.com/jrockway/evil-module.  When evil-module-user was written<EMAIL_ADDRESS>was a good module.  Then "they" released a new version that made the module "evil", and force-pushed the v0.0.1 tag again.  Now when you attempt to go run main.go in the evil-module-user repository, you get:
$ GOPROXY=direct go run main.go
go: downloading github.com/jrockway/evil-module v0.0.1
verifying<EMAIL_ADDRESS>checksum mismatch
        downloaded: h1:8hZfcGWxgwLDlFobA01ibehmq113HI83t6mw9lLzEUI=
        go.sum:     h1:ddshoT5twMMhZ18FKG+6u1gJ74vRDGLItYxftMxkbBI=

SECURITY ERROR
This download does NOT match an earlier download recorded in go.sum.
The bits may have been replaced on the origin server, or an attacker may
have intercepted the download attempt.

For more information, see 'go help module-auth'.

That's what you'd expect.  When the author of evil-module-user first ran their program that depended on evil-module, the checksum of<EMAIL_ADDRESS>was h1:ddshoT5twMMhZ18FKG+6u1gJ74vRDGLItYxftMxkbBI=.  Now it's h1:8hZfcGWxgwLDlFobA01ibehmq113HI83t6mw9lLzEUI=.
  Behind the scenes, go appears to just git clone that repository and checks out the tag; it does no comparison against the go.sum file.  (In non-GOPATH mode, i.e. in a directory with go.mod, it does check, and the check fails.)
I wrote a small tool to contact the Go module sum database server and ask what Google thinks the module checksum is (in pachyderm/sumlint):
$ go run main.go
0.159s<EMAIL_ADDRESS>0.020s https://sum.golang.org/tile/8/2/000.p/22
0.022s https://sum.golang.org/tile/8/0/x005/791.p/221
0.024s https://sum.golang.org/tile/8/1/022.p/159
[github.com/jrockway/evil-module v0.0.1 h1:ddshoT5twMMhZ18FKG+6u1gJ74vRDGLItYxftMxkbBI=]

Note that that's the "good" version of the module.
Where this is all going is that linting the checksums is going to be more complicated than I thought.  My initial thought was to just make an API request to the Google checksum database, and see if any lines in go.sum didn't match.  That isn't quite what we want in that case.  That lint would detect the case where a developer has a cached copy of an "evil" module, and the rest of the world has the good one.
I think the more common case is this force push case, and to detect that, we have to clone the repositories ourselves and compute the checksum on the repository.
Complicating factors is how many places the "good" module is cached in.  By default "go mod update" or "go get" will use the Google Module Proxy, which caches the "good" version of the module.  You'll only get the "evil" version of the module if you set "GOPROXY=direct" (or use your own proxy, probably common inside big companies).
I have nothing against writing a tool to go build every version without a module cache.  But now that I've looked into this deeply, I think that even if an upstream tampers with their modules, it's likely that users will still get the version that we specify in our go.sum file.
Obviously, the next step is to look into the exact cases that Pachyderm customers have experienced, to see why the cache didn't save them.  I will do so!

Oh... actually... I tried making this module
module github.com/pachyderm/example-broken-go-mod-project

go 1.14

require github.com/pachyderm/pachyderm v1.10.0

And go mod download does indeed fail:
$ go mod download
<EMAIL_ADDRESS>verifying module: checksum mismatch
        downloaded: h1:CxSSPe5FaVzWWqltiX5sYF13v25mGIacQyD+fSuDXZQ=
        sum.golang.org: h1:kfWVPEoi8pXR+0sndL2vOKoOdFNxLtBMoK2oZvScfHo=

SECURITY ERROR
This download does NOT match the one reported by the checksum server.
The bits may have been replaced on the origin server, or an attacker may
have intercepted the download attempt.

For more information, see 'go help module-auth'.

I am now looking to why a bad checksum is cached for our code.

After some discussion yesterday, it appears that we intentionally re-tagged some releases.  Going forward, we should not do that, and this should not be a problem.

The mitigation I recommend for users that want to use exactly Pachyderm v1.10.0 is to tag the same commit with a different version number.  go "helpfully" converts a commit hash in go.mod to this tag, unfortunately, so that won't work as a workaround.


bumped to 3.0.0-rc11

3.0.0-rc11

VFS-2764 Fix directories having 0B size in GUI
VFS-2764 Fix size of files being zero right after upload
VFS-2765 Add some error resistance when obtaining unexpected data from openid providers
VFS-2765 Display first login info to users logging in via credentials
VFS-2733 Fix provider key/cert filename

3.0.0-rc11

VFS-2764 Fix directories having 0B size in GUI
VFS-2764 Fix size of files being zero right after upload
VFS-2765 Add some error resistance when obtaining unexpected data from openid providers
VFS-2765 Display first login info to users logging in via credentials
VFS-2733 Fix provider key/cert filename


Allow dynamic server name and port number

Setting server parameters in config.yaml doesn't seem to be 100% effetive:
E.g. by changing:
server:
  host: localhost
  port: 9000

which gives HTTP/1.1 200 OK
to:
server:
  host: localhost
  port: 9500

an re-launching ./start.sh, the new port seems to be taken into account:
[INFO ] 2021-02-18 18:24:46.051 [armeria-boss-http-*:9500] Server - Serving HTTP at /<IP_ADDRESS>:9500 - http://<IP_ADDRESS>:9500/

But the website is no more responding:
$ http localhost:9500

http: error: ConnectionError: ('Connection aborted.', ConnectionResetError(104, 'Connection reset by peer')) while doing GET request to URL: http://localhost:9500/

In addition, if we change the host name, it doesn't affect the server:
in config.yaml:
server:
  host: some.server.org
  port: 9500

gives this after restarting the server:
[INFO ] 2021-02-18 18:24:46.051 [armeria-boss-http-*:9500] Server - Serving HTTP at /<IP_ADDRESS>:9500 - http://<IP_ADDRESS>:9500/

and the site is still unreachable:
$ http some.server.org:9500

http: error: ConnectionError: ('Connection aborted.', ConnectionResetError(104, 'Connection reset by peer')) while doing GET request to URL: http://some.server.org:9500/

Therefore, one should allow both the host name and port to be set up dynamically, e.g. using env variables.
For the moment, they seem hard-coded: https://github.com/baremaps/baremaps/blob/f13815324b22f287e5761bb1376184022544e373/baremaps-config/src/main/java/com/baremaps/config/Server.java#L19-L21

The hard-coded values are default values used when the configuration file does not specify them.
If you want the domain name some.server.org to work locally, you should probably add the following line to your /etc/hosts file, to prevent it from being resolved with DNS. some.server.org exists and has an IP address.
<IP_ADDRESS> some.server.org

Have you tried to do something like
http <IP_ADDRESS>:9500

It sounds like I misunderstood that the port specified in the config.yaml is the one used by the app inside the docker container, it is not the one published on the host, which I think it actually was. You won't be actually able to notice this if you use the same port number when mapping them (e.g. using --publish 9000:9000) during the container startup.
So, I let it match the given docker port in the config.yaml file. And everything works fine locally, both when reaching localhost:HOST_PORT and some.server.org
But then I face the CORS issues when I reach the site from another machine, for which I'll open a new issue even so it's probably related in some way.


CharFXTransform missing "character" property for RichTextEffects in v4

Godot version
4.0.rc5
System information
Ubuntu 20.04 LTS (not relevant)
Issue description
In 4.0 class CharFXTransform:
https://docs.godotengine.org/en/latest/classes/class_charfxtransform.html
We're missing the property "character" that was in v3:
https://docs.godotengine.org/en/stable/classes/class_charfxtransform.html#class-charfxtransform-property-character
Text effects such as Matrix will not work without this data...
https://docs.godotengine.org/en/stable/tutorials/ui/bbcode_in_richtextlabel.html#custom-bbcode-tags-and-text-effects
Or is there is another way to access it? If so it doesn't seem to be documented.
Steps to reproduce
Not applicable
Minimal reproduction project
Documentation is sufficient

It looks like it changed to glyph_index

It looks like it changed to glyph_index

You're right, thanks. I'll close this issue

It's changed to font and glyph_index.  Glyph index is font specific and not necessarily correspond to the single character (e.g., multiple characters combined int the ligature will have single CharFXTransform and index of the ligature glyph).
Check latest version of documentation for the updated Matrix example - https://docs.godotengine.org/en/latest/tutorials/ui/bbcode_in_richtextlabel.html

The new Matrix bbcode doesn't work for me. The end result text is different than original.
Possibly a bug:
char_fx.glyph_index = get_text_server().font_get_glyph_index(char_fx.font, 1, char_fx.glyph_index, 0)
Shouldn't this "do nothing"? I get scrambled text here due to different characters being returned. Tried a number of fonts...

char_fx.glyph_index = get_text_server().font_get_glyph_index(char_fx.font, 1, char_fx.glyph_index, 0)

This should not work, font_get_glyph_index takes character code and returns glyph index (these are different things).

@bruvzg
Wouldn't that mean that var value = char_fx.glyph_index is a mistake in the matrix bbcode? Shouldn't it be getting a character instead of the glyph index? I'm not sure how to get the character from the glyph index...

If I change the last in matrix to char_fx.glyph_index = value instead of calling font_get_glyph_index then it works

Wouldn't that mean that var value = char_fx.glyph_index is a mistake

This is a mistake, I think code supposed to use time and character range to generate matrix effect, ignoring original glyph index.

I'm not sure how to get the character from the glyph index

In general, it's not possible to get it in character. Glyph index is content dependent and not always corresponds to the exact character.

Seems like there should be a font_get_glyph_index equivalent function, such as 'font_get_char(glyph_index)'?


Update search text

Changing search text as described in a recent addition to issue #83.

Closing in favor of #100 and #101.


Add section for how to uninstall

based on #3

@liyanwei93 can you help check if deleting the whole app can uninstall? If so, can you help update document? Thanks!


Lost in timeouts for libs/18.yaml

Even with nix flake check/prefetch my nix-shell does get GHC / nar.xz packages /stack/cabal elsewhere

@carnotweat Could you give a better description of what you're trying to do, where you're seeing an error, how someone could reproduce your error, etc?

That's my bad
I merely needed to add " ip raw.githu..." to extra hosts in configuration.nix
175th line in
https://github.com/carnotweat/remoteflake/commit/29583cef13b4a9e8fa2e3e126cfc6d759df41119#


Added night theming + some bug fixes + smoother animations

1. Here is new night mode feature

2. Animations will be a little bit smoother

Fixed problem with setInterval, it was replaced with requestanimationFrame, ehich is better for performance

dark theme, finally!


Make task history horizontally scrollable

This PR fixes #14517. This adds a scrollbar and gets rid of the truncation with ... in the task history when the window is made smaller or zoomed in so that the data is viewable without a tooltip.
before:

after:

Pull Request Test Coverage Report for Build 681486704

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.001%) to 43.272%

Totals

Change from base Build 681189491:
-0.001%

Covered Lines:
25464

Relevant Lines:
53684

💛  - Coveralls


Client Allocation Request for: IPFS-scholar

Client Allocation Questions
Core Information

Name: IPFS-scholar
Website / Social Media<EMAIL_ADDRESS>DataCap Requested: 100TiB
Addresses to be Notarized: f25zkoqxmmmvnm5fq7txkirtpfj7xmzqoo2p6763y

Hey @liyue634 - thanks for your request. At this time, the FF is only considering data cap requests <5 TB. This is to ensure effective diligence relative to the tooling available and time. If you would like to request <5 TB, then please file a new request via the app.


Adapting ExchangeRatesApi to the new API format

Split endpoints by enterprise option
Handle new response format
The issue is described here: https://github.com/florianv/exchanger/issues/135
For discussion:
https://github.com/Seagull-4auKa/exchanger/blob/0e93e602455feac0566a76afef2c58469128255e/src/Service/ExchangeRatesApi.php#L37-L39
I didn't rename const LATEST_URL and HISTORICAL_URL to ENTERPRISE_LATEST_URL and ENTERPRISE_HISTORICAL_URL respectively. I don't think they can be used outside of the class or tests, but still left them as they are.

For me, your PR works fine, thanks!

Merge with master
Fix ExchangeRatesApiTest.php method names to snake_case
Update doc/readme.md
Update README.md - update base currency info for ExchangeRatesApi

Thanks @Seagull-4auKa :)


[BUG] CIS scan fails on RKE1 hardened HA setup

Rancher Server Setup

Rancher version: v2.6.7-rc3
Installation option (Docker install/Helm Chart): Helm Chart

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc):

Information about the Cluster

Kubernetes version: v1.24.2-rancher1-1
Cluster Type (Local/Downstream): Local

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider):

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom) Admin

If custom, define the set of permissions:

Describe the bug

On RKE1 hardened HA setup, CIS 1.20 and 1.23 hardened profile scan fails.

To Reproduce

Create RKE1 k8s v1.24, hardened HA cluster(3-nodes all roles) using this hardening guide.
Deploy rancher v2.6.7-rc3.
Install CIS benchmark 2.0.5-rc6 on local cluster.
Run the scans using 1.20 and 1.23 hardened profile scans.

Result

Observe the below failures in the scan:

1.23 profile :

1.20 profile:

Expected Result

Scan should pass successfully.

Additional context

The scan is failing due to known issue as described in the hardening guide
The fleet-content ClusterRole seems to be causing the failure as described in the guide.
Moving the issue back to testing.

From the above comment seems like it's a known issue and it's documented. CIS 5.1.5 check expects default service accounts have no roles or cluster roles bound to it. Based on the same closing the issue.


Reconstruct BTF types more accurately in getKernelType

This fixes warnings for struct/pointer-based types (e.g. file) that the spec type doesn't align with the BPF-identified type.
This issue was previously identified (with a fix written) in https://github.com/cilium/tetragon/pull/850. However, that fix was never merged, and changes validation logic instead of just type identification.

Thanks @cpaplham ! Sorry seems it took me a really long time to get to this I missed it in the PR lists.


Make custom image selector available from Nearby

Summary:
In the current state of the app, custom image selector is only available from contributions fragment. We also upload images from Nearby. What about making the custom image selector available from Nearby too ( If that has no disadvantages).

@Ayan-10  can I take this issue?

I think @nicolas-raoul can help you with that.

@nicolas-raoul Can I please take this issue ? Thanks

Hi @Eric-peng111 Would you mind working on bugs first per https://github.com/commons-app/commons-app-documentation/blob/master/android/Volunteers-welcome!.md ? Thanks a lot! 🙂

@nicolas-raoul Thanks for replying
I'd like to take first issue
However, I already fixed about 90% of the issue as this:

is it possible to let me do this at first ?
Sorry I did not know this policy.

Hard question 😱
OK for this time exceptionnally, but please keep in mind that it might take a very long time before we review it. Thanks for your understanding! 🙂

Thanks for letting me do this issue
@Ayan-10 @nicolas-raoul  can u assign this issue to me at this point?

Hi @nicolas-raoul
I think I done this issue since I tested by debug, so I submitted  the first PR to this issue, pls have a check when u have time
No need to rush since you said gonna take time to review
Thanks

Hi @nicolas-raoul do we still want to do this ? I still see Custom Selector not available in Nearby , if yes, then can I work on this ?

Yes thanks! 🙂

@shashankiitbhu Maybe you can build upon Peng's pull request?

@shashankiitbhu Maybe you can build upon Eric's pull request?

There seems to be a lot of conflicts since the Project has changed significantly since then but I'll definitely take a look at it.

@nicolas-raoul Added the Enhancement and since the Enhancement was to be made in NearbyParentFragment which was still not migrated to view binding so I also migrated it to view binding in the same PR so that it's easier to test , Please Review


nom8 is EOL

As a heads up, nom8 was an experimental, short-lived fork for ideas I was trying out for nom v8.
Options

Keep on using it
Switch to nom v7
Wait for the official, upcoming nom v8 which includes some of the ideas from nom8
Switch to my long-term fork, winnow (announcement)

Hi, Thanks for the heads up! uair currently depends on nom8 instead of regular nom crate to avoid linking two versions on nom. Since using toml brings up nom8 as a dependency anyway, I decided to go with nom8 for the parsing. I'll switch to using winnow as soon as toml (indirectly) depends on winnow.

With a cargo update you should get the latest toml_edit which should upgrade you to winnow.

With a cargo update you should get the latest toml_edit which should upgrade you to winnow.
It does. I didn't know that cargo doesn't automatically update indirect dependencies on cargo build/run. Thanks.


Add sigstore CLI flags --staging and --identity-token

Summary

This is a followup of #319 to allow passing an identity token via the CLI. The staging flag was also added to allow easier testing.
Release Note

NONE
Documentation

NONE

Thank you


2.0.0 Global variables?

Looks like 2.0 breaks the functionality when you could set some vars in an included file and use them in other files.

I can confirm this.

@mbosecke is it a feature or a bug?
This issue basically a show stopper for us, we can't upgrade because that would require rethinking/redoing/testing lots of templates we already have.

There was a discussion in #121 that changed the default behavior of the "include" tag to isolate the included template; the variables of the included template are now within their own scope and do not affect the variables of the template that included it. So yes, this was by design.
What's your use case for wanting an included template to be able to affect variables within the current template?
I still think it's a good idea to have included templates isolated by default but I'm open to ideas on how a user may be able to explicitly opt-out of that isolation.

Well, the main user case is that we create a settings file where are some global setting vars are defined and used everywhere else.
But i guess we will have to redo our templates.


Introduce Share->Print functionality

Task/Issue URL: https://app.asana.com/0/inbox/856498667309057/1120715488709057/1115574928442566
Tech Design URL:
CC:
Description:
Added Print option under app's Share menu.
Steps to test this PR:

Open any website.
Go to Menu -> Share.
'Print' option should be available.
'Create PDF' option should be hidden.

Please validate that 'Print' preview matches the one from Safari for any given page.

Internal references:
Software Engineering Expectations
Technical Design Template

Thanks @brindy :)


fix(slide-toggle): apply typography styles to slide-toggle

The slide-toggle didn't apply any typography styles to the content element
Now it applies the configurable typography variables.

Before:

After:

Fixes #633.

LGTM; we might want to revisit when we look at a more comprehensive typography approach, but this works for now.


'Auto detecting the task system failed' can be more user friendly

Related to #4019
Steps to Reproduce:

have a broken gulpfile.js
generate task runner > choose gulp
Notice the error
This should be a warning, not an error
There should be an action to open the output service, task channel to make it easier for the user to locate the error

Agree. But instead of an action we should automatically open the output. Setting to March to investigate. Might move to April since this is not a regression.

Nice!


fix: global retry

Motivation/Description of the PR

Resolves #3663
Resolves #3607

Type of change

[x] :bug: Bug fix

Checklist:

[x] Tests have been added
[ ] Documentation has been added (Run npm run docs)
[ ] Lint checking (Run npm run lint)
[x] Local tests are passed (Run npm test)

Thank you!
@DavertMik - when can we expect a patch release?

@kobenguyent Is it support Gherkin?
cc @DavertMik
Cause what I receiving when trying to use it with Gherkin (Added Before to cover Background)
Error processing suite.before event:
TypeError: Cannot read properties of undefined (reading 'retryBefore')


[BUG] Speaking at event not displayed

Description
On the events manage page the info is shown...

But on the main events page it is not displayed...

Screenshots
No response
Do you want to work on this issue?
None
Additional information
No response

can I do this?

can i take this?

can you please assign me this issue

can i take this?

Yes, I will assign you now. Thanks for contributing!

@eddiejaoude I have assigned this issue to someone as it was unlocked, but I thought this was by design bc of the 1:many nature of the community events page?
I believe this was discussed somewhere else that it only makes sense to list speaking on personal pages due to this issue, but please advise if I am misremembering.

Oh good point Amanda 😱 yes we don't know which to display if there are multiple people attending the same event.
I will close this issue, sorry for the confusion


Flesh Out Alert Policies with Bucket IAM Monitoring as a Use Case

Description
An important security control in GCP is the ability to create metrics of interest, create alert policies based on those metrics, and configure some kind of notification channel(s) where alerts can be sent.
As a concrete use case to flesh this out, I propose we set up a policy to monitor changes to cloud storage bucket permissions. The security value of this is to get immediate visibility when any important IAM changes happen to cloud storage buckets.
The main deliverables for this issue are to have documentation artifacts. Specifically:

[x] Concise explanations of what the security control does and how we implement the security control (include lots of links to upstream documentation where appropriate).
[x] Diagram(s) (if appropriate).
[x] Terraform code to reproduce the provisioning of the components.

Recommended steps:

Create a "Counter" log-based metric (see google_logging_metric for Terraform resource for this).
Create an alert policy (see google_monitoring_alert_policy) and a google_monitoring_notification_channel to send the alert to. For testing purposes, we can try just sending alerts to an email address.

Closed in https://github.com/PHACDataHub/infra-core/pull/35


Capture Ansible requirements in requirements.txt

While trying to initiate a VM on an OpenStack instance, I encountered https://groups.google.com/forum/#!msg/ansible-project/3pj7jknSqd8/cr4Zt6L2BQAJ.
Rather than hardcoding the version in various places in the documentation, this PR migrates all the Python dependencies into a top-level requirements.txt as we do for any Python app. Dependencies can be pinned and upgraded via PR.
Travis is also updated to consume the requirements file.

How did you get that error? The latest shade works for me:
$ pip show shade
Name: shade
Version: 1.13.2

(ansible) sbesson@lifesci-56045:ansible ((b5b15ca...)) $ ansible-playbook -e vm_key_name=seb-dundee -e vm_name=www-dev os-uod-web.yml

PLAY [localhost] ***************************************************************

TASK [setup] *******************************************************************
ok: [localhost]

TASK [fail] ********************************************************************
skipping: [localhost]

TASK [fail] ********************************************************************
skipping: [localhost]

TASK [Web external access security group] **************************************
ok: [localhost]

TASK [Web external access security group rules] ********************************
ok: [localhost] => (item=22)
ok: [localhost] => (item=80)
ok: [localhost] => (item=443)

TASK [Web VM] ******************************************************************
fatal: [localhost]: FAILED! => {"changed": false, "extra_data": null, "failed": true, "msg": "Error in creating instance (Inner Exception: create() takes at least 4 arguments (9 given))"}

PLAY RECAP *********************************************************************
localhost                  : ok=3    changed=0    unreachable=0    failed=1
```

@sbesson to test via gh-228

Retested via 1.13.2 both while creating a devspace and a Web box, I could not reproduce https://github.com/openmicroscopy/infrastructure/pull/231#issuecomment-268252442. I suspect this has to do with my own configuration files that I had to fix post OS reinstallation. I'll push an extra commit to pin shade to 1.13.2

Note while testing the devspace deployment with Ansible 2.2.0, I faced two issues:

https://github.com/ansible/ansible-modules-core/issues/5504
https://github.com/ansible/ansible/issues/18357

Downgraded Ansible to <IP_ADDRESS> in the meantime, I propose to update https://github.com/openmicroscopy/infrastructure/pull/231 accordingly.

Pushed the requirements I used for testing gh-228

Merging with travis green. There's a broader story here to be looked into in the new year.


WIP: Add cppcheck includes when modified buffer or no compile_commands.json

This can probably be expanded to look at -I directives from compile_commands.json to make it more robust. See comments in the code.
I don't know how to modify a buffer from a Vader Execute block, so those tests are failing right now. You may have a better idea.

You can use :set modified to mark a buffer as modified.
Is there any danger in always setting -I with the directory name? Does it make the results less accurate? If so, your solution is a good one. Paired with the code I had for improving how compile_commands.json files are found, this should be good.

I'm not sure it'd make a difference. I think cppcheck just ignores -I when --project is given.
Doing a :set modified or set modified does not seem to work in the test...

Actually, after setting modified, do I need to call ALE again? I've tried
set modified
ALELint

and
set modified
call ale#Queue(0)

but neither seem to work. I'd appreciate the help, if you want that last test. I don't have time to pursue this much further.
Let me know if you figure it out, or whether I can just remove the last test. I'll then clean up the pull request.

I escaped the paths.
Can you clarify what you think should be a separate function, and where it should live?

You can create a function in autoload/ale/handlers/cppcheck.vim that both versions of the linter can use. Make sure to update the code here to take the recent changes I've made for using build/compile_commands.json into account.

Alright, refactored, and all but that one test (where I set modified) pass. I don't know enough about Vader to figure this one out, so I'll leave it up to you from here. Thanks!

@bartlibert Does this look okay to you?

Looks fine to me. Nice improvement.

I fixed the test and merged this via the commandline now.