Issues with resources when compiling

Hello,
Firstly great work so far on the plugin, I'm happy testing it out on my projects and give some feedback.
That said, there is one bug that I think I can workaround with gradle must merits better handling. So I'm performing an AOT compilation and when evaluating a namespace I read a resource file, ex:
(def ^:private schema
  (-> "graphql-schema.edn"
      io/resource
      slurp
      edn/read-string
      (lau/attach-resolvers resolvers)
      (lau/attach-scalar-transformers scalar-xformers)
      las/compile))

In my build.gradle I'm basing it off a lein build:
sourceSets {
    dev {
        clojure.srcDirs = ['dev']
        resources.srcDirs = ['dev-resources']
    }
    main {
        clojure.srcDirs = ['src']
        resources.srcDirs = ['resources']
    }
    test {
        clojure.srcDirs = ['test']
    }
}

Well it seems that since gradle separates classes and resources it can't seem to locate the file, maybe a classpath issue. I can create a test project with exactly this behaviour, when building I get the following error:
nestrada@OLP00002145:~/repos/services/punchcard$ gradle clean build
Starting a Gradle Daemon, 1 stopped Daemon could not be reused, use --status for details

> Task :compileClojure
Creating fresh Clojure runtime.
SLF4J: Failed to load class "org.slf4j.impl.StaticMDCBinder".
SLF4J: Defaulting to no-operation MDCAdapter implementation.
SLF4J: See http://www.slf4j.org/codes.html#no_static_mdc_binder for further details.
ERROR:  clojure.lang.Compiler.CompilerException   java.lang.IllegalArgumentException: Cannot open <nil> as a Reader., compiling:(graphql.clj:74:7)

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':compileClojure'.
> A failure occurred while executing gradle_clojure.plugin.internal.ClojureWorker
   > java.lang.IllegalArgumentException: Cannot open <nil> as a Reader., compiling:(graphql.clj:74:7)

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

* Get more help at https://help.gradle.org

BUILD FAILED in 33s
3 actionable tasks: 3 executed

If you have any suggestions and what I can do to mitigate this issue (and reading from resources statically in a namespace is done quite often so I would prefer not changing it) I would much appreciate it.
Otherwise I'll be happy to help in any way I can, I really prefer this to boot as I do a lot of mixed clojure/groovy these days.

I believe you could get this to work by doing the following:
clojureCompile {
  source sourceSets.main.resources
}

Looks like we have the ClojureCompile tasks configured to just have the Clojure source on the classpath (and your compile dependencies). This appears to be the way Gradle does this for Java, as well.
As for whether the default behavior should change, I'd want some of the other @gradle-clojure/developers to chime in. Not familiar enough with this use case to know how common it is. My first inclination would be to leave it as-is to align with other Gradle plugins.

Unfortunately this seems to create an odd behavior with shimdandy where all resources are evaluated as clojure files, ex it seems to think that my logback.xml file is a namespace of sort. For example, after setting this option, I'm getting the following errors
LF4J: Failed to load class "org.slf4j.impl.StaticMDCBinder".
SLF4J: Defaulting to no-operation MDCAdapter implementation.
SLF4J: See http://www.slf4j.org/codes.html#no_static_mdc_binder for further details.
ERROR:  java.io.FileNotFoundException   Could not locate logback__init.class or logback.clj on classpath.

:compileClojure (Thread[Task worker for ':',5,main]) completed. Took 28.641 secs.

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':compileClojure'.
> A failure occurred while executing gradle_clojure.plugin.internal.ClojureWorker
   > Could not locate logback__init.class or logback.clj on classpath.

* Try:
Run with --debug option to get more log output.

* Exception is:
org.gradle.api.tasks.TaskExecutionException: Execution failed for task ':compileClojure'.
        at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.executeActions(ExecuteActionsTaskExecuter.java:100)
        at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.execute(ExecuteActionsTaskExecuter.java:70)
        at org.gradle.api.internal.tasks.execution.SkipUpToDateTaskExecuter.execute(SkipUpToDateTaskExecuter.java:63)
        at org.gradle.api.internal.tasks.execution.ResolveTaskOutputCachingStateExecuter.execute(ResolveTaskOutputCachingStateExecuter.java:54)
        at org.gradle.api.internal.tasks.execution.ValidatingTaskExecuter.execute(ValidatingTaskExecuter.java:58)
        at org.gradle.api.internal.tasks.execution.SkipEmptySourceFilesTaskExecuter.execute(SkipEmptySourceFilesTaskExecuter.java:88)
        at org.gradle.api.internal.tasks.execution.ResolveTaskArtifactStateTaskExecuter.execute(ResolveTaskArtifactStateTaskExecuter.java:52)
        at org.gradle.api.internal.tasks.execution.SkipTaskWithNoActionsExecuter.execute(SkipTaskWithNoActionsExecuter.java:52)
        at org.gradle.api.internal.tasks.execution.SkipOnlyIfTaskExecuter.execute(SkipOnlyIfTaskExecuter.java:54)
        at org.gradle.api.internal.tasks.execution.ExecuteAtMostOnceTaskExecuter.execute(ExecuteAtMostOnceTaskExecuter.java:43)
        at org.gradle.api.internal.tasks.execution.CatchExceptionTaskExecuter.execute(CatchExceptionTaskExecuter.java:34)
        at org.gradle.execution.taskgraph.DefaultTaskGraphExecuter$EventFiringTaskWorker$1.run(DefaultTaskGraphExecuter.java:248)
        at org.gradle.internal.progress.DefaultBuildOperationExecutor$RunnableBuildOperationWorker.execute(DefaultBuildOperationExecutor.java:336)
        at org.gradle.internal.progress.DefaultBuildOperationExecutor$RunnableBuildOperationWorker.execute(DefaultBuildOperationExecutor.java:328)
        at org.gradle.internal.progress.DefaultBuildOperationExecutor.execute(DefaultBuildOperationExecutor.java:197)
        at org.gradle.internal.progress.DefaultBuildOperationExecutor.run(DefaultBuildOperationExecutor.java:107)
        at org.gradle.execution.taskgraph.DefaultTaskGraphExecuter$EventFiringTaskWorker.execute(DefaultTaskGraphExecuter.java:241)
        at org.gradle.execution.taskgraph.DefaultTaskGraphExecuter$EventFiringTaskWorker.execute(DefaultTaskGraphExecuter.java:230)
        at org.gradle.execution.taskgraph.DefaultTaskPlanExecutor$TaskExecutorWorker.processTask(DefaultTaskPlanExecutor.java:124)
        at org.gradle.execution.taskgraph.DefaultTaskPlanExecutor$TaskExecutorWorker.access$200(DefaultTaskPlanExecutor.java:80)
        at org.gradle.execution.taskgraph.DefaultTaskPlanExecutor$TaskExecutorWorker$1.execute(DefaultTaskPlanExecutor.java:105)
        at org.gradle.execution.taskgraph.DefaultTaskPlanExecutor$TaskExecutorWorker$1.execute(DefaultTaskPlanExecutor.java:99)
        at org.gradle.execution.taskgraph.DefaultTaskExecutionPlan.execute(DefaultTaskExecutionPlan.java:625)
        at org.gradle.execution.taskgraph.DefaultTaskExecutionPlan.executeWithTask(DefaultTaskExecutionPlan.java:580)
        at org.gradle.execution.taskgraph.DefaultTaskPlanExecutor$TaskExecutorWorker.run(DefaultTaskPlanExecutor.java:99)
        at org.gradle.internal.concurrent.ExecutorPolicy$CatchAndRecordFailures.onExecute(ExecutorPolicy.java:63)
        at org.gradle.internal.concurrent.ManagedExecutorImpl$1.run(ManagedExecutorImpl.java:46)
        at org.gradle.internal.concurrent.ThreadFactoryImpl$ManagedThreadRunnable.run(ThreadFactoryImpl.java:55)
Caused by: org.gradle.workers.internal.DefaultWorkerExecutor$WorkExecutionException: A failure occurred while executing gradle_clojure.plugin.internal.ClojureWorker
        at org.gradle.workers.internal.DefaultWorkerExecutor$2.waitForCompletion(DefaultWorkerExecutor.java:156)
        at org.gradle.internal.work.DefaultAsyncWorkTracker.waitForItemsAndGatherFailures(DefaultAsyncWorkTracker.java:99)
        at org.gradle.internal.work.DefaultAsyncWorkTracker.access$000(DefaultAsyncWorkTracker.java:34)
        at org.gradle.internal.work.DefaultAsyncWorkTracker$2.run(DefaultAsyncWorkTracker.java:83)
        at org.gradle.internal.work.DefaultWorkerLeaseService.withoutLocks(DefaultWorkerLeaseService.java:196)
        at org.gradle.internal.work.DefaultWorkerLeaseService.withoutProjectLock(DefaultWorkerLeaseService.java:145)
        at org.gradle.internal.work.DefaultAsyncWorkTracker.waitForCompletion(DefaultAsyncWorkTracker.java:79)
        at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter$1.run(ExecuteActionsTaskExecuter.java:130)
        at org.gradle.internal.progress.DefaultBuildOperationExecutor$RunnableBuildOperationWorker.execute(DefaultBuildOperationExecutor.java:336)
        at org.gradle.internal.progress.DefaultBuildOperationExecutor$RunnableBuildOperationWorker.execute(DefaultBuildOperationExecutor.java:328)
        at org.gradle.internal.progress.DefaultBuildOperationExecutor.execute(DefaultBuildOperationExecutor.java:197)
        at org.gradle.internal.progress.DefaultBuildOperationExecutor.run(DefaultBuildOperationExecutor.java:107)
        at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.executeAction(ExecuteActionsTaskExecuter.java:111)
        at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.executeActions(ExecuteActionsTaskExecuter.java:92)
        ... 27 more
Caused by: java.io.FileNotFoundException: Could not locate logback__init.class or logback.clj on classpath.
        at clojure.lang.RT.load(RT.java:463)
        at clojure.lang.RT.load(RT.java:426)
        at clojure.core$load$fn__6548.invoke(core.clj:6046)
        at clojure.core$load.invokeStatic(core.clj:6045)
        at clojure.core$load.doInvoke(core.clj:6029)
        at clojure.lang.RestFn.invoke(RestFn.java:408)
        at clojure.core$load_one.invokeStatic(core.clj:5848)
        at clojure.core$compile$fn__6553.invoke(core.clj:6056)
        at clojure.core$compile.invokeStatic(core.clj:6056)
        at clojure.core$compile.invoke(core.clj:6048)
        at gradle_clojure.tools.clojure_compiler$compiler$fn__163.invoke(clojure_compiler.clj:32)
        at gradle_clojure.tools.clojure_compiler$compiler.invokeStatic(clojure_compiler.clj:25)
        at gradle_clojure.tools.clojure_compiler$compiler.invoke(clojure_compiler.clj:23)
        at clojure.lang.AFn.applyToHelper(AFn.java:165)
        at clojure.lang.AFn.applyTo(AFn.java:144)
        at clojure.lang.Var.applyTo(Var.java:702)
        at clojure.core$apply.invokeStatic(core.clj:657)
        at clojure.core$apply.invoke(core.clj:652)
        at clojure.lang.Var.invoke(Var.java:385)
        at org.projectodd.shimdandy.impl.ClojureRuntimeShimImpl.invoke(ClojureRuntimeShimImpl.java:124)
        at org.projectodd.shimdandy.impl.ClojureRuntimeShimImpl.invoke(ClojureRuntimeShimImpl.java:117)
        at gradle_clojure.plugin.internal.ClojureWorker.run(ClojureWorker.java:60)
        at org.gradle.workers.internal.WorkerServer.execute(WorkerServer.java:31)
        at org.gradle.workers.internal.WorkerDaemonServer.execute(WorkerDaemonServer.java:38)
        at org.gradle.workers.internal.WorkerDaemonServer.execute(WorkerDaemonServer.java:25)
        at org.gradle.process.internal.worker.request.WorkerAction.run(WorkerAction.java:98)
        at org.gradle.internal.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:35)
        at org.gradle.internal.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:24)
        at org.gradle.internal.remote.internal.hub.MessageHubBackedObjectConnection$DispatchWrapper.dispatch(MessageHubBackedObjectConnection.java:146)
        at org.gradle.internal.remote.internal.hub.MessageHubBackedObjectConnection$DispatchWrapper.dispatch(MessageHubBackedObjectConnection.java:128)
        at org.gradle.internal.remote.internal.hub.MessageHub$Handler.run(MessageHub.java:404)
        ... 3 more

I removed the logback.xml just for testing, and my resources directory contains a datomic-schema.edn file, and this is what I get now:
> Task :compileClojure
Creating fresh Clojure runtime.
ERROR:  java.io.FileNotFoundException   Could not locate datomic_schema__init.class or datomic_schema.clj on classpath. Please check that namespaces with dashes use underscores in the Clojure file name.

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':compileClojure'.
> A failure occurred while executing gradle_clojure.plugin.internal.ClojureWorker
   > Could not locate datomic_schema__init.class or datomic_schema.clj on classpath. Please check that namespaces with dashes use underscores in the Clojure file name.

Maybe by fiddling with the main source set classpath rather than setting the resources as source is what is needed however I'm open to suggestions.

Ah, right our compile task isn't doing additional checks for which files are in the source, it just assumes they're all Clojure and infers namespaces for every file.
Maybe this would work:
clojureCompile {
  classpath += sourceSets.main.resources
}

I can't remember for certain off the top of my head, but I did think that the resource paths were expected to be added to the classpath.  I don't know if Gradle does this by default (we've not been fortunate enough to use Gradle is our work place just yet--we're eyeing it for polyglot stuff), but I'm pretty sure Leiningen does.  The Gradle docs don't really say what is done in the Java tasks. :-(

Gradle (for Java projects) does not put resources on the classpath of Java compilation. It probably isn't useful for Java, like it could be for Clojure.
With Gradle, resources always have a "process" task where (as opposed to compile for source code) you can do some filtering or substitution on the files. If we did put them on the classpath (by default) for Clojure we may want to use the ones coming out of that process task instead of the original source dir.

Similar to #40, we might be able to follow the approach Groovy uses to put Java on the classpath to do the same with resources.

Gradle (for Java projects) does not put resources on the classpath of Java compilation. It probably isn't useful for Java, like it could be for Clojure.

Yeah, I was going to say I don't think they do for compilation, but I believe they do for runtime-related stuff.  But your right, since compilation and execution are pretty much the same thing in Clojure, we might need to for Clojure.

The fix is part of 0.3.0. Please reopen if this didn't fix the issue.


Implementation of sub-group barrier operation

[x] Have you provided a meaningful PR description?

This PR contains implementation of sub_group_barrier function that synchronizes across all work-items in a sub-group. It currently lacks a test case to verify the functionality of the barrier call. The tests need to be implemented once sub-group feature is added.

[ ] Have you added a test, reproducer or referred to an issue with a reproducer?
[ ] Have you tested your changes locally for CPU and GPU devices?
[ ] Have you made sure that new changes do not introduce compiler warnings?
[x] If this PR is a work in progress, are you filing the PR as a draft?

After https://github.com/IntelPython/numba-dpex/pull/1310 is merged we have to follow sycl syntax and make it work using group_barrier(nd_item.get_sub_group(), MemoryScope.DEVICE)

Now that the group_barrier function accepts a reference to a group as argument, we no longer need a separate sub_group_barrier.
A barrier across a sub-group can be achieved by calling group_barrier and passing a reference to a sub-group which will extend group.


Fix 3rd party component logging

https://keboola.atlassian.net/browse/PS-2945
The searches rely on having critical message level - such as ('"component":"kds-team.') ('"priority":"ERR' OR 'priority":"EMERGENCY' OR 'priority":"CRITICAL') -kbc-devel- -'level":400' -'level":401' e.g. https://my.papertrailapp.com/groups/2307723/
This should log the application error as critical so that we don't have to update all searches.

to byl omyl


Introducing the data frame

This PR introduces the data frame to convey flip-flop-app payloads. A means to version the protocol, determine the source of the frame (client/server), the server address and port and Message Authentication Code (MAC) to authenticate the data are all provided.

As a general comment I think you are right to have the error detection, encryption, framing and addressing at this level.   These are all transport issues independent of the application protocol.

  No doubt you have considered this, but a greater range of station addresses and port numbers might be useful.

We could have a prose description of this level of the protocol.  How mush is implied beyond the wire format?  I think the media access rules could stand at this level.

Why confine the encryption implementation to the tests?  If the aim is to exclude (most) implementation from this library then fair enough.  In that case we could have another library with the encryption implementation.

 No doubt you have considered this, but a greater range of station addresses and port numbers might be useful.

I might be. I'm keeping RS485 in mind given the max number of servers on the bus being around 32. We can always restructure the protocol though by bumping its version if necessary.

We could have a prose description of this level of the protocol. How much is implied beyond the wire format? I think the media access rules could stand at this level.

We do have the primary README and the examples to back it up. Happy to add more but might need your help there.

Why confine the encryption implementation to the tests? If the aim is to exclude (most) implementation from this library then fair enough. In that case we could have another library with the encryption implementation.

Encryption is an external concern because there are different ways of going about it. For example, we can use the libraries we have here in the tests. However, we can also use the hardware accelerated CCM peripheral of the nRF52840.
Further, I'm probably going to avoid encryption in our actual implementation on the nRF52840 in the first instance. But I needed to do it here to explore the implications.


Live translator nitinkumar30

New project added where you can translate live from one language to another. Check it live here.

Hi,
your project is great, but please follow the proper directory convention mentioned in the readme file of the project.


update to workflow template

closes #5

Good idea!
One note: this should include documentation in CONTRIBUTING.md for easier onboarding of devs.
In the long run, it would be nice to set up a PR template for some of this


Add link to standards page on README

Mention data standard pages in the readme

@inodb can you connect with @aditigopalan to run through how the Data portal standards pages are currently setup and automated? We have a ticket open in #354

@inodb should we set up a meeting to talk about this?


Update dependencies to fix #4

Got typescript running, added tests to expose the broken behavior, updated dependencies, and the new tests pass, so it looks like that's all that was needed!

Oh, looks like type definitions are lacking now, so it won't compile to js. I'll fix that, add a postinstall script, and update the PR.

There we go. Including into a js project using it for tests works now, yay! That should do it.

Ah, frig, the first test that was there doesn't actually ensure that htmlLooksLike throws, just checks that if it throws, the message is right... so I accidentally just made it pass all the time and it looked fine. Working on this again...

I'll close this until I get something actually working.


Use correct workspace for release pipelinerun link

Fixes
https://issues.redhat.com/browse/RHTAPBUGS-473

Description
Determine the correct workspace for release pipelinerun and create a valid link in release details page.

Type of change

[ ] Feature
[x] Bugfix
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

Screen shots / Gifs for design review
https://github.com/openshift/hac-dev/assets/20013884/a90a0e17-1497-4bdd-b498-f93f8c5d80e6

How to test or reproduce?

Set up release objects.
Click on pipelinerun link in release details page.

Browser conformance:

[ ] Chrome
[x] Firefox
[ ] Safari
[ ] Edge

Verified, the release pipelinerun link now takes me to correct workspace.
/approved
/lgtm

/approve


protein stablizing activity

I don't understand why this term was made obsolete.
The comment says that this represents a process so
people should use 'protein stabilization'. What
criteria are being used to distinguish between these
two? Also, does this mean that ascribing function,
process and component for all genes is not going to
be possible?
Reported by: syrhee
Original Ticket: "geneontology/ontology-requests/3165":https://sourceforge.net/p/geneontology/ontology-requests/3165

thanks www.movied.org


PURCHASE-1107 Add SellerInfo component

Feature
https://artsyproduct.atlassian.net/browse/PURCHASE-1107
Show Partner's name (seller's name) in the artwork page.
Change
Added new SellerInfo component which fetches partner's data from an artwork and shows the name.
Selfie

Fails

:no_entry_sign:
No CHANGELOG added.

:no_entry_sign:
Missing Test Files:

src/lib/Scenes/Artwork/Components/__tests__/SellerInfo-tests.tsx

If these files are supposed to not exist, please update your PR body to include "#skip_new_tests".

  Generated by :no_entry_sign: dangerJS


Read credentials from  section in settings.xml

Fix https://github.com/wouterd/docker-maven-plugin/issues/92

Any news on this PR?
Does it seem correct for you or do you want me to adapt it?
I'd like to use an official release of this plugin instead my forked version...

I'll look at this ASAP, was at a conference last week and catching up this week. :)
Sorry for the delay.

I merged the PR, it will be released together with 5.0.0, thanksfor the work on this feature.


[New Port Request] < EventBus >

Library name
EventBus
Library description
A lightweight and very fast event bus / event framework for C++17
Source repository URL
https://github.com/gelldur/EventBus
Project homepage (if different from the source repository)
No response
Anything else that is useful to know when adding (such as optional features the library may have that should be included)
No response

@ilongshan This library uses an older version of catch2 and since overrides are not active in the script mode, I am unsure, how is that handled here?

@motazmuhammad Not sure how to deal with it. Maybe pending the port util the project updates to the catch2 v3.

Is catch2 needed for something else than tests?
vcpkg port shall not run tests. So any catch2 dependency can be disabled.

@dg0yt , just for tests thank you, removing it does solve the problem,
another question in that area , it seems that the .lib files are not generated in the dynamic build, which causes this error
LINK : fatal error LNK1104: cannot open file 'lib\EventBusd.lib'
the dll files are existent however, this is a problem with dynamic linking only as if I set the linkage to static only the build works fine, the only way I can think of to solve this problem is via adding a patch to cmakelists file to force it to produce lib files, have you faced that before? or do you know how it should be solved? or should I make this library static only library?

If there is a DLL (for runtime) without lib (for build time), then the packages lacks proper exports for Windows. Then do not build a DLL. Start the portfile with:
if(VCPKG_TARGET_IS_WINDOWS)
    vcpkg_check_linkage(ONLY_STATIC_LIBRARY)
endif()

@dg0yt thank you, I will use that then


Custom IPFS Gateway

We are currently looking into using Trystero for our decentralised gaming client.
Is it possible to use our custom IPFS gateway instead of a public one to guarantee uptime?

Yes, in the config object you pass to joinRoom() (using the IPFS strategy) you can specify a swarmAddresses property which expects an array of addresses (for reference).
Check out @libp2p/webrtc-star-signalling-server
 for more details on running a signaling server.

Feel free to reopen if you have any questions.


Can't use multiple labels file in single rules_oci rule

I want to add two kind of labels:

labels for image lifecycle management - on this labels change image must be rebuild;
labels for image stamping (from which Git revision I can build same image) - this lables change everytime and should be excluded from cache key.

For this I want to create two labels file and pass it by custom or filegroup rule:

labels.txt - with labels for image lifecycle management;
labels_volatile.txt - with stamping labels.

After that I can add labels_volatile.txt to omitted_inputs srubbing rule and cache will ignore every labels_volatile.txt file change.
But currently I can pass only single file by labels attribute to rules_oci.

I don't quite understand the feature request here. How's label_list instead of label will help you with this?
Also this is possible today by using two oci_image targets, first one adding labels.txt and the second one adding volatile.txt.
IMO this is really a custom feature that the most will not benefit from.

I understand that most people are not interested in the problem of stamping on the farm.
But in this case, support is added by a trivial non-breaking change: https://github.com/bazel-contrib/rules_oci/pull/609

I agree that looks simple in terms of added lines, however it makes the public api a little more harder to understand.  This is possible to today by combining two oci_image with little-to-no-cost at build time.


Client Side Caching with Otter

Otter is a high performance lockless cache which uses proactive ttl and s3-fifo eviction policies.
Lockless is an important property that redis server-assisted client-side caching needs because the invalidation messages from redis should be applied as fast as possible. Applying invalidation messages should be the first priority if possible.
But in the current implementation, the pipelining goroutine handing invalidation messages competes with other reader goroutines to acquire the LRU lock. This will delay the invalidations and further block the pipeline. Using otter can solve this.

@rueian provide an option to use https://github.com/phuslu/lru?
otter is slow on set but fast on get, phuslu lru is more balanced and uses lesser mem

Hi @ouvaa, rueidis is read intensive but writes should be prioritized.

Hi @sshankar, sorry for my late reply.
Unfortunately, my progress on this is currently paused. I haven't finished adding singleflight load
mechanism to otter. That is a key feature that otter currently doesn't have. If we build the mechanism outside of otter, we will need to pay high overhead for it.
What makes you interested in otter?


integrate all contributor cli

instead of adding the contributor manually, we can utilise all contributor cli
https://www.npmjs.com/package/all-contributors-cli

Coverage remained the same at 100.0% when pulling 52a254bfeaf59ab67e49202fa7eb906347fab32e on all-contributor into 5357daf451657acf35bb5226a8f4d2cbacdcdf46 on master.


Bump version to 1.1

Update readme as well, to point to the NN guide in coremltools

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


Use an ordered list for homepage

@terror Been wanting to do this for a while. Makes the markup clearer, looks a little better, and is probably better for screen readers.


[BEAM-498] Add static factory method for DoFnInfo

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[x] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[x] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[x] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[x] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

This provides a clearer migration path away from deprecated code paths than constructor overloads which can only be selected by upcasting, etc.
R: @lukecwik

LGTM, you can merge once you fix the nit


pombase slim

mitochondrial gene expression was recently swapped for mitochondrial translation in the pombase slim.
I can't remember if you are importing or if we have a hard coded temporary list?

For now the slim list is in the config file in Subversion:
pombe-embl/website/pombase_v2_config.json
Look for the "go_slim_terms" section.
If you make changes, let me know so I can re-release.

I just updated the slim....

I just updated the slim....

I've re-released and mitochondrial gene expression is there now.

yep thanks!


TMX sprites issue

after this changes 48629f9cb13ebe47d90cfda8be2f1fdefc3fbf38
in codes like
auto spirte = map->getTileAt(Vec(1, 1));
sprite->setAnchorPoint(Vec2(0, 0));
is changing the sprite position when should change only anchor point position for sprite got from tmx map

@aismann could you take a look?

auto spirte = map->getTileAt(Vec(1, 1));
sprite->setAnchorPoint(Vec2(0, 0));

Sorry This code makes no sense.
Why you change the AnchorPoint?

just try this with axmol pre commit, and check if it gives the same result in master, to me seems changing the position of sprite, but setAnchorPoint should to change only position of anchor point

just try this with axmol pre commit, and check if it gives the same result in master, to me seems changing the position of sprite, but setAnchorPoint should to change only position of anchor point

Thats not correct:
If you change the anchorPoint you change also the position: look:

 // add "HelloWorld" splash screen"
 auto sprite = Sprite::create("HelloWorld.png"sv);
 auto sprite1 = Sprite::create("HelloWorld.png"sv);
 if (sprite == nullptr)
 {
     problemLoading("'HelloWorld.png'");
 }
 else
 {
     // position the sprite on the center of the screen
     sprite->setPosition(Vec2(visibleSize.width / 2 + origin.x, visibleSize.height / 2 + origin.y));
     sprite->setAnchorPoint(Vec2(1, 1));
     this->addChild(sprite, 0);

     sprite1->setPosition(Vec2(visibleSize.width / 2 + origin.x, visibleSize.height / 2 + origin.y));
 //    sprite1->setAnchorPoint(Vec2(1, 1));
     this->addChild(sprite1, 0);

     auto drawNode = DrawNode::create();
     drawNode->setPosition(Vec2(0, 0));
     addChild(drawNode);

     drawNode->drawRect(safeArea.origin + Vec2(1, 1), safeArea.origin + safeArea.size, Color4F::BLUE);
 }

Yes nothing with anchor point maybe but for some reason master is behaving different from old version, I am gonna to try to send you a code and data leave it open until that

Yes nothing with anchor point maybe but for some reason master is behaving different from old version, I am gonna to try to send you a code and data leave it open until that

The changes made in PR #1285 definitely do change the behavior, which was the intention of that PR, and not a side-effect of the PR.
PR #1285 and the related issue #1283 were open since late July, and the issue closed and PR merged only in the last week.  It was hoped that developers using the tile map code would give their input regarding those changes prior to this occurring, but unfortunately no-one did so.
Now, regardless of that, knowing that the behavior has changed, what we need to know is:
1 - Can you adjust your game code to cater for the updated behavior?
2 - If there is no trivial way to adjust your game code to cater for this new behavior, and you consider this to be a bug, then please explain your reasoning, or show examples of your usage so we can better understand the issue.

@Joilnen
Last but not least: You can use the Axmol 1.x path for your project
(Axmol 2.x has some more API changes which be different to 1.x)

@Joilnen
I think you can close this issue or have you one more question?

Ok What I could figure out of this was I had made some corrections, because this showed me slightly out of the position in old code, so when I update the axmol my scene showed me this out position again, I guess your commit fixed that, I am closing this one time it seems working better, I don't need to make those correction in code anymore, thanks

Ok What I could figure out of this was I had made some corrections, because this showed me slightly out of the position in old code, so when I update the axmol my scene showed me this out position again, I guess your commit fixed that, I am closing this one time it seems working better, I don't need to make those correction in code anymore, thanks

Thats was the reason for this PR. Thanks for feedback.


support: Add notion of primary admin.

For a few things like reactivate realm and scrub realm it'd be good to have a canonical set of people authorized to do the action (that's a little more narrow than "all admins").
The first step is to add an is_primary_admin field to UserProfile, with a migration that sets the oldest admin in each org to the primary admin. (Doesn't matter if the user is active.)
During realm creation, the new user should also have the is_primary_admin field set.
(As followup we can display this make it editable in the frontend settings.)

@zulipbot claim

@rishig to set the oldest admin as a primary admin, on migration, you mean to add a script in the migration file, that will set this field for all users, right? And then use migrations.RunPython. If so, I think I'll add following script
def set_oldest_user_primary_admin(apps: StateApps, schema_editor: DatabaseSchemaEditor) -> None:
    Realm = Realm = apps.get_model('zerver', 'Realm')
    UserProfile = apps.get_model('zerver', 'Realm')
    for realm in Realm.objects.all():
        realm_admins = UserProfile.objects.filter(realm=realm, is_realm_admin=True)
        oldest_realm_admin = realm_admins.order_by('-date').first()
        oldest_realm_admin.is_primary_admin = True
        oldest_realm_admin.save(update_fields=["is_primary_admin"])

yeah, exactly. There are a few errors in that script in particular (e.g. the first two lines), but it has the right idea. Thanks!

Yeah! That two times Realm is typed by mistake, I'll correct it soon :)

@zulipbot claim

Hey @johnfdepop! There is a significant amount of work already done on this issue in #12298. Can you pick something else to work on? Or if you are interested, you can work on the following part for this issue, which is not covered in #12298.
We need to display primary_admin field in user roles in the settings section. We'll also want to add the following models.
(1) Warn the primary admin about the loss of his primary_admin status, if he decides to transfer the primary_admin status.
(2) Not allow other admins to change the role of primary_admin.
(3) Not allow primary admin to deactivate himself.
Sorry for the inconvenience!

Hi @ruchit2801, sure I'll have a look at the other open issues. Very new, so looking for something small and fresh. All the best.

@zulipbot abandon

@zulipbot claim

@zulipbot claim

@zulipbot claim

@zulipbot abandon

@zulipbot claim

@zulipbot  abandon

claim

@zulipbot claim

@zulipbot claim

@zulipbot abandon

@zulipbot claim

@zulipbot claim

@zulipbot abandon

@zulipbot claim

@zulipbot abandon

This has been implemented as the "realm owner" feature in  #15116, #15267 and #15305.


How to rename the Dataframe columns in CAPS

I need to rename some of my data-frame columns in CAPS.
i found that it's possible in spark using alias method, but in order to use it i need to import spark.implicit.
this import dosen't work in your project, what is it's equivalent ?

i found  another way to rename the columns :
new_df=old_df.toDF("_c20","geohash")

Another possibility would be to alter the mapping when importing the data frames as a graph like:
val mapping = NodeMapping.on("id2").withPropertyKey("geoHash" -> "_c22").withPropertyKey("a" -> "_c23")

@HADDADENICAR Could you please provide a code example of the import that doesn't work? Normally any import of a Spark package should work the same way it would on vanilla Spark.

@Mats-SX  in InputDataFrameExample when i add import spark.implicits._  i  get an error telling "Cannot resolve symbol spark"

@HADDADENICAR You have to add that line after introducing the spark variable. That is, after row 45.
https://stackoverflow.com/questions/10537277/what-does-this-import-exactly-mean-in-scala


move dartdevc off analyzer task model

See https://github.com/dart-lang/sdk/issues/33546
Also, Analyzer task model has incorrect results for some things (e.g. mixin type arguments, top-level inference) and users are hitting these.

@JekCharlsonYu - we're hoping to cherry-pick this fix for 2.1.  It fixes some issues @yjbanov hit on new mixins.

I'm testing this CL: https://dart-review.googlesource.com/c/sdk/+/82469. I may need to update a more status entries, but otherwise I think it's in good shape.

@jmesserly - how is this looking?


Image builder global permission

Explain problem to solve
gloabl permission is only available on hcl format and cluster-api has json format.
Describe the solution you would like
Options 1:
Use preprocessor with osc-cli ?
Additional context
None
Environment
- Kubernetes version: (use `kubectl version`):
- OS (e.g. from `/etc/os-release`):
- Kernel (e.g. `uname -a`):
- cluster-api-provider-outscale version:
- cluster-api version:
- Install tools:
- Kubernetes Distribution:
- Kubernetes Diestribution version:

Image builder global permission exist in both format hcl and json, we'll be available in the next release of Packer


Different settings when using UAC

When you start Picotorrent in UAC (WIndows 10, 1511), you get questions asked as if you are starting it for the first time.
I cannot repeat this since I do not have another Windows copy lying around

Do you mean the dialog which asks if you want to allow it in the firewall? Or the dialog asking if you are sure you want to run the application?

Also, if you are using a version of PicoTorrent from AppVeyor, these are not signed. Only the official binary releases on GitHub are signed.

The dialog which asks if you want to run it the application. I'm using the official 0.10 and I tried running it as admin to see if the crash with speed settings still occurs.

The crash that occured for the DL/UL limits has been fixed, but not released.
I'm unsure if the UAC prompt is a problem or not. It seems as it is just Windows working as intended.

Closing this since I don't think the problem is with PicoTorrent - or at least, there's nothing we can do about UAC prompts.


[Bug]Home page/ My profile page/ Valid credentials/ My Profile UI - not user friendly

To Reproduce
Steps to reproduce the behavior:

Go to dev.podkrepi.bg
Click on on "my profile" icon on the menu bar
Enter valid credentials (already registered  user)
There is new page opened with My profile

Expected behavior
To see user-friendly UI/UX that is easy to understand and use.
Actual result:
Jason format info with info from Keykloak
Desktop (please complete the following information):

OS:  Windows 7 home premium
-google chrome (32 bites)
Version - 97.0.4692.99

Smartphone (please complete the following information):

Device: Samsung a22 5g[]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

@Polina1985 I thinks this issue can be closed.

@dplamenov Thanks for letting us know, indeed it can be closed now


Making dialogs moveable?

Am really liking vex, cheers!
Has there been any thought or work put into making the dialogs moveable? It's a requirement for the site I work on and I'd appreciate any guidance. I've had a look at the CSS and at first glance it looks like it might require significant changes.... am considering trying jQuery draggable.

jQuery draggable should work, but depending on your browser requirements, you may be able to get away with the HTML draggable property and some JS. I whipped up a little demo in jsFiddle.
Source:
var $vexContent = vex.dialog.alert({
    closeOnOverlayClick: false,
    message: 'Drag me'
});

$vexContent.attr('draggable', true);

$vexContent[0].addEventListener('dragstart', function(event) {
    var style, boundingClientRect, data;
    style = window.getComputedStyle($vexContent[0]);
    boundingClientRect = $vexContent[0].getBoundingClientRect();
    data = (parseInt(style.getPropertyValue('left'),10) - event.clientX) + ',' + (parseInt(style.getPropertyValue('top'),10) - event.clientY);
    if (style.getPropertyValue('left') === 'auto') {
        data = (- event.clientX) + ',' + (- event.clientY);
    }
    event.dataTransfer.setData('text/plain', data);
}, false);

document.body.addEventListener('dragover', function(){
    event.preventDefault();
    return false;
}, false);

document.body.addEventListener('drop', function(event) {
    var offset = event.dataTransfer.getData('text/plain').split(',');
    $vexContent[0].style.left = (event.clientX + parseInt(offset[0], 10)) + 'px';
    $vexContent[0].style.top = (event.clientY + parseInt(offset[1], 10)) + 'px';
    event.preventDefault();
    return false;
}, false);

Just include the jquery user interface library (jquery-ui.js) in your project. Then make the Vex dialog draggable by calling the draggable method on the vex dialog variable (e.g myvexdiag.draggable).
Cheers.

@softpacks In the changes proposed in #160, vex.open would not be returning a jQuery object, but it's an own object. I presume this means the draggable() method would automatically be available.

Draggable isn't going to happen natively. If you can find a way to make it work with jQuery go for it.


transparency of foreground and background colors for hints

I just wanted to test a completely transparent hint background (so that only the labels are shown) but when specifying alpha values for hint_{fg,bg}color, it seems as if the value for the background color is applied to the foreground color as well, i.e. setting hint_bgcolor = #00000000 and hint_fgcolor = #ffffffff has the effect of drawing completely invisible hints. Is this intended behaviour?

This is a known bug in the X implementation. Opacity is achieved by setting _NET_WM_WINDOW_OPACITY which affects the entire window. Short of grabbing screen content and doing alpha compositing in warpd (expensive), I'm not aware of a better way to achieve this.

The rectangle is part of the window. warpd uses the X11 shape extension in conjunction with the cached output of Xft drawing routines in a buffer which gets copied when the window is displayed. In theory it should be possible to use the shape extension to just crop out the drawn fonts but I don't know if this is possible/performant from memory and it wouldn't be worth the time investment.


How to add parameters to the request header

use /api/bookmarks/check/
Sorry about that, checked the documentation, and environment variable configuration.
I want to add request headers when requesting partial url:
Accept-Language: zh-CN,zh;q=0.9,en-US;q=0.8,en;q=0.7
I'm wondering if any of you experienced gurus out there can help me out.
I'm installing via Docker-Compose, so I'd like to be able to do this via environment variables

This doesn't seem to be a linkding related problem. You should be able to find this information in the documentation of whatever request library / REST client that you want to use.

这似乎不是一个与链接相关的问题。您应该能够在您想要使用的任何请求库/REST 客户端的文档中找到此信息。

No, through the browser plugin, the page cannot be correctly recognized and always displays age verification, but accessing it through the browser is normal.
After comparison, if the Accept Language zh CN is missing from the request, this issue will occur.
In particular, using web snapshots, the snapshots of this site are the same age verification.
Calling headless browser in docker I hope to add header
website as javbus.com


Add Mollie provider

I created a provider for Mollie.

I'd like to see your provider follow PSR-2 style but I don't think we require it.
:+1:


support alternate dockerfiles

Docker 1.5 has the ability to specify the dockerfile to use in docker build. It'd be great if we could override the default file via an environment setting like "dokku config:set myapp DOCKERFILE=Dockerfile.docs" or add it to the .env file.

Sounds like a good idea to me :smile:

I was gonna add a PR for this. I have a quick convention question though. Is there a reason for the DOKKU_DOCKERFILE_PORT versus using simply DOCKERFILE_PORT? I think it'd be better to just use DOCKERFILE_PORT and check first if it's already set to avoid overwriting it.

We had some issue with internal env variables getting overwritten because they were to ambiguous. Hence, the convention to prepend DOKKU_.

@mmerickel bump :)

It looks to me like this is solved by the docker-options build-phase plugin.


sendtoaddress - add argument for sending satoshi

sendtoaddress currently sends half of its amount to the asset receiver.
Add an argument to specify the satoshi to send.

done in https://github.com/rgb-org/kaleidoscope/pull/24 (ready to merge, waiting for a revision)


I don't understand how to use query.groupBy

Hi,
I've used query.groupBy as per example on a simple get operation (list all entities of a kind grouped by property):
groupBy(['prop']);
With one arguments I have the complete list of kind without any grouping applied.
With two arguments I have this response:
{
  "code": 412,
  "metadata": {
    "_internal_repr": {}
  }
}

Can you please help me understand how am I supposed to use the groupBy API?

I think this is a matter of setting the correct indexes to support that query. Maybe a search on the Datastore docs as well as this repo's issues will give some ideas. I'm going to close as this isn't a bug with our library. StackOverflow would likely be a better place for this question. Feel free to let me know if you can't find any answers, and we will bug the Datastore team :)


feat: Insert Note Index command

This PR:

Adds a new command InsertNoteIndexCommand that will let users to insert a list of wikilinks to direct children of current note.
Adds basic tests for this command.
The core logic will be refactored to be exposed to the cli in the later phase to allow advanced use cases.
This is phase 1 of RFC 18 -  Add Note Indexes.

Discussion: https://github.com/dendronhq/dendron/discussions/1076

PR for docs: https://github.com/dendronhq/dendron-site/pull/162

Updates:

Handled multi-vault cases.
Added new config to enable markers.
Added tests for said updates.
Revised docs PR to mention these updates.


DotNetCoreCli Custom Command Fails With EF

Note
Issues in this repo are for tracking bugs, feature requests and questions for the tasks in this repo
For a list:
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks
If you have an issue or request for the Azure Pipelines service, use developer community instead:
https://developercommunity.visualstudio.com/spaces/21/index.html )
Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Bug
Enter Task Name: DotNetCoreCLIV2
list here (V# not needed):
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks
Environment

Server - Azure Devops Server

If using TFS on-premises, provide the version:  Dev17.M153.5

Agent - Hosted or Private: Private

If using private agent, provide the OS of the machine running the agent and the agent version:
OS Name:                   Microsoft Windows Server 2019 Standard
OS Version:                10.0.17763 N/A Build 17763
Agent Version:           2.153.1

Issue Description
[Include task name(s), screenshots and any other relevant details]
Quotes are appended using the custom command option and ef migrations will not be recognized as a command.
Here is testing showing cmd.exe not working with quotes then working without quotes.

Powershell doesn't care and runs either way

Here is the command that is outputted from the logs.
"C:\Program Files\dotnet\dotnet.exe" "ef migrations --project projectlocation script --idempotent -v"
Could not execute because the specified command or file was not found.
I can have this command run in a different spot but I was trying to keep my references for the dotnet executable similar and this task seemed like a good spot to make that happen. I'm fairly new so I could be missing something simple.
I can confirm that there is no quotes in the YAML file as well.
Task logs
[Enable debug logging and please provide the zip file containing all the logs for a speedy resolution]
Troubleshooting
Error logs
[Insert error from the logs here for a quick overview]

Have a look at this
https://github.com/microsoft/azure-pipelines-tasks/issues/12548

Please re-open it if you face any issues.


Post processor for tokens

I use moo and have this rule:
list -> %leftParen expressions %rightParen

I need a post processor action as soon as %leftParen is seen.
I tried putting a post processor directive between %leftParen and expressions, but that is not allowed.
Does this mean I have to introduce a Nearly rule like
leftParen -> %leftParen {% ... %}

?

You can also do leftParen -> (%leftParen {% … %}).
But I caution you about writing postprocessors that have side-effects — nearley assumes postprocessors are pure and can be run in any order, as many times as it wants, with any input.

But I caution you about writing postprocessors that have side-effects

@Hardmath123 so how am I supposed to create an AST?

Take a look at the examples, such as https://github.com/Hardmath123/nearley/blob/master/examples/csv.ne


Publish: publication review works incorrect if publication has prompts and conversations simultaneously

EPAM AI DIAL chat version
0.14.0
What happened?
publication review works incorrect if publication has prompts and conversations simultaneously if publish it API directly, not through DIAL Chat
Confidential information

[X] I confirm that do not share any confidential information

verified on staging successfully


Add "used by" section to README

PyMC3 have accepted a PR to use this to check their notebooks :tada: , so it might be good to mention this in the README

alibi too https://github.com/SeldonIO/alibi/blob/master/.pre-commit-config.yaml :tada:

pandas-profiling as well https://github.com/pandas-profiling/pandas-profiling/blob/develop/.pre-commit-config.yaml :tada:

nlp-profiler https://github.com/neomatrix369/nlp_profiler/blob/master/.pre-commit-config.yaml :tada:

sktime too https://github.com/alan-turing-institute/sktime :tada:

reminder: add pymc resources

add https://github.com/scikit-hep/mplhep

https://github.com/intake/intake-esm too

some others:
https://github.com/OpenSourceEconomics/estimagic
https://github.com/kratsg/drstorage
https://github.com/matthewfeickert/ServiceX-exploration
https://github.com/matthewfeickert/heputils
https://github.com/matthewfeickert/talk-Argonne-lunch-seminar
https://github.com/nootify/torch-letter
https://github.com/WM-SEMERU/mlproj_template/blob/main/.pre-commit-config.yaml
https://github.com/Eazhi/World-Happiness-Analysis
https://github.com/deepcharles/ruptures (big deal!)


Fixed crosshair length translation error

Microsoft PowerToys version
0.72.0
Utility with translation issue
Mouse Utilities
🌐 Language affected
Polish
❌ Actual phrase(s)
Napraw długość krzyżyków

✔ Expected phrase(s)
Stała długość krzyżyków
ℹ Why is the current translation wrong
The current translation means "repair the length of the crosshairs" which is an example of translation without context. The word "fixed" can be translated as both "not moving" and "repaired" and the wrong translation occured here.

https://dev.azure.com/GlobalCollaborationService/Global Collaboration Service Project/_workitems/edit/390852 /loc

The work associated with this issue has been released as part of the 0.76 sprint. Please update PowerToys to the latest. https://github.com/microsoft/PowerToys/releases


fix couple of '@return:' into '@return ' (jdk 11 complains)

Rather cosmetic change, but needed to fix the documentation build in openJDK11

thanks!


AutoSizer and offset* property

AutoSizer is used for size calculation offsetWidth and offsetHeight
https://github.com/bvaughn/react-virtualized/blob/master/source/AutoSizer/AutoSizer.js#L165
but offset* property round value. When we have width like 303.1px, it rounds to 302px.
In this example content override right border https://codesandbox.io/s/qlwo8q5py9 .
If use getBoundingClientRect it solves this problem

@arshtepe that would return the same values unless some transforms were applied mdn docs
It also seemed to display properly for me on chrome.


Various minor fixes for next big stream

[ ] https://github.com/dashtesting/trello-github-etl/issues/16
[ ] Cleanup fallbackOwner & owner ('nix leading @', loose matches)
[ ] Link trello card id / task id to github issue
[ ] Deal with occasional status 500 throw new Error("not ok"); graphql
[ ] Add test for Transform.trello* if @devPalacio doesn't get to it first
[ ] Put - or other token in Fallback Owner, Owner, etc to indicate (handled, but no data given)

@riongull In the case that we've processed the custom field, but didn't have any data to put in, would you prefer - or N/A or None? We could leave it as an empty space in Prod, but we need something to differentiate abject failure from "data wasn't available, go check the logs to see why".

"None" works

Since GitHub didn't give an automated message like above, @coolaj86, I moved some of the tasks from this issue to https://github.com/dashtesting/trello-github-etl/issues/34.


Will the operator delete CloudSQL proxy container when application containers in Job terminate

It is not clear to me what will be the behavior of this operator if it installed CloudSQL proxy container side car in a K8s Job workload whose application (my code) containers have exited. Will it appropriately stop the CloudSQL proxy container after other containers have exited, or will the proxy container continue running and thus the Job will never stop.

I think you're looking for this https://github.com/GoogleCloudPlatform/cloud-sql-proxy-operator/issues/361.
Right now you'll have to manually configure quitquitquit and call it yourself.
cc @hessjcg in case I've missed some nuance here.

Short answer: no. When a job pod  is configured with the AuthProxyWorkload configuration that exists at the time the pod is created. The pod's podSpec cannot be modified after the Pod is created. So that job's pod will keep the same configuration regardless of how the AuthProxyWorkload.
However, if that job pod exits and a new pod is created, the new pod will have the configuration at the time the new pod is created.


[Feature request]: Pausing Background Audio when on break

Version

[X] I'm using version 1.7.0

Existing requests

[X] I've checked Existing requests

What operating system are you using?
Windows
Operating System Version
Windows 10 19043.1081
Problem description
Music or radio will still play in the background when the break launches in full-screen mode, and one can't help but want to pause the postpone stretchly.
Possible solution
Pause audio through the main system on Windows
Alternative solutions
Given time to pause the audio in the background.
Will cause issues on strict mode
Additional information
No response
Code of Conduct

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

Tried searching for package that would allow that but I am not seeing anything: https://www.npmjs.com/search?q=pause audio
Not sure what player you're using, but  I would guess most of player nowadays have keyboard shortcut to pause music. Also (on macOS and Linux machines that I have), there is a OS-level keyboard shortcut to pause player

@hovancik that is correct, it is a keyboard shortcut level pausing problem for Windows too.
Others have claimed that there are solutions for it
microsoft.com/en-us/store/p/stop-music/9nblggh0ccfd
microsoft.com/en-us/store/p/stop-music/9nblggh086wn
microsoft.com/en-us/store/p/stop-music-plus/9nblggh0852t

Those apps are shown as for mobile devices:

There are some other apps that do similar, I guess: https://www.howtogeek.com/319516/how-to-automatically-pause-your-music-when-other-sounds-start-playing/
I would need some NodeJS way how to do it, but I am not seeing any packages that can do that, except pausing sounds that my app starts.

This could also be done with https://github.com/hovancik/stretchly/issues/712
Users write pause command and resume command and write it to designed tasks.
I do not think messing with all sounds is good solution.


Update Subgraph/Onchain to add Gyro pools. Added SwapQuery example.

Update SOR package to 4.0.1-beta.3 which has support for ComposableStable.
Updates Subgraph/Onchain to add Gyro pools and ComposableStable.
Added SwapQuery example.

Need to update SOR package when released and test before Ready for review.


exclude behavior change between 5.x and 6.x

  "config": {
    "nyc": {
      "exclude": [
        "**/__tests__/**/*.js"
      ],
      "include": [
        "src/**"
      ]
    }
  },

This config was consuming code in src, but not files like src/PageContainer/__tests__/component.js in 5.x.
With 6.1.1, those files in __tests__ are counted in the % which sucks.

I had to add two pattern
        "**/__tests__/**/*.js",
        "**/__tests__/*.js"

Which looks like a regression to me :)

FYI, "**/__tests__/**" also works (yeah).

@novemberborn any thoughts about this, perhaps an update in how we process globs?

It seems I was wrong on the entire thing and confused by the order of reported file.
It seems  config is not in "config.nyc" but just "nyc", is this correct?

@MoOx yes, this was the breaking change between 5.x and 6.x -- we switched to using a new built in yargs feature to handle loading config from package.json.
Cool to close this?

Yeah can close.
One detail for better ux: provide a warning if people miss the config definition change :)


TIcked build to 1.1.1, reformat, rerender

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Change in authentication sample code

[Enter feedback here]
Kindly change armresources.NewResourceClient(subscriptionId, cred, nil) to armresources.NewClient(subscriptionId, cred, nil)
as the sdk repository doesn't have this former mentioned function anymore.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 0512b7d6-3f16-303f-44dc-4e1595d5ca62
Version Independent ID: 43cd9aca-e5ee-31f7-7c34-497c54ff9487
Content: Azure authentication with the Azure SDK for Go
Content Source: articles/go/azure-sdk-authentication.md
Service: azure
GitHub Login: @Duffney
Microsoft Alias: jduffney

Hi @shaheerem,
I appreciate you taking the time to file the issue, I'll get the docs updated per your suggestion.
Thank you, :)

Hey @shaheerem.
I've made the suggested changes and updated the code to use the NewClient function instead of the NewResourceClient function. I also noticed that the readmes on pkg.dev also mention the old function. I'm working to update those references as well. A PR has been submitted will be merged shortly. You should see the docs update within the next 24-48 hours.
Thank you for taking the time to to submit the issue. :)
Have a wonderful day!

#please-close


Fix threading issue from switch to PCL

The earlier switch to PCL and Eigen (#14) introduced a multithreading bug that could cause mismatched LFS values, this change fixes that bug.

cheers


out_forward causes instability when data cannot be transferred

We are using fluentd over WAN. Sometimes we have network outages, and fluentd's buffers fill up. The unsent data cannot be dropped due to SLAs. The same data is used for real-time system monitoring when possible.
We have been looking at using a combination of writing out_file every second+compressing and copying to destination+in_file from disk outside of fluentd, but improving out_forward and in_forward would be preferred.
Before doing that work, do you have any issues with adding compression and increasing the buffer length of in/out_forward?

do you have any issues with adding compression

Not yet support. See this PR: https://github.com/fluent/fluentd/pull/637

increasing the buffer length of in/out_forward?

See this article: http://docs.fluentd.org/articles/out_forward#buffered-output-parameters

The unsent data cannot be dropped due to SLAs.

And using <secondary> with small retry_limit prevents data loss at long outage situation.
See this section: http://docs.fluentd.org/articles/output-plugin-overview#secondary-output

The conversation of compression should be kept on PR.


Adding effects on Ironman Homepage picture

Can put some effects on Ironman image when it is hovered and some background effects also.

Please provide GSSOC Ext 24 and Hactoberfest label to it,I can improve this

hii @aslams2020    i believe i can do this task efficiently as i have a strong hold over css styling. could you please assign me this task under GSSoC24 extd and hactoberfest24.

@AyutheCoder Can you do something like - The image always floats smoothly ? along with the text on it's left side.
Both speed should be smooth and steady.

@aslams2020 ,Yes Sir,I can do this

Thank you Sir,now I can start working ?


allowlist text alignment

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[x] Bugfix
[ ] Feature
[ ] Refactoring

Context

[x] Closes #9644
[ ] Requires deployment <snek/rubick/worker>

Before submitting pull request, please make sure:

[ ] My contribution builds clean without any errors or warnings
[ ] I've merged recent default branch -- main and I've no conflicts
[ ] I've tried to respect high code quality standards
[ ] I've didn't break any original functionality

Optional

[ ] I've tested it at </ksm/collection>
[ ] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Did your issue had any of the "$" label on it?

[x] Fill up your DOT address: Payout

Community participation

[x] Are you at KodaDot Ecosystem Telegram?

Screenshot 📸

[x] My fix has changed something on UI; a screenshot is best to understand changes for others.

https://canary.kodadot.xyz/ahp/collection/98
weird, the text to truncate is not even showing up here

switch the grid mode and then you could see it

😍 Perfect, I’ve sent the payout
💵 $20 @ 10.37 USD/DOT ~ 1.929 $DOT
🧗 14dLrAvjDw7cFLaxM7VWR6YtHd7bGZY2QpzXDhAJJ4yv6oGw
🔗 0x1af5b8ce33e7990376405f5bcc1a6718ebb87908eee308127c14020cbf50f6c9
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


python-gevent: disable c-ares module and fix CVE-2021-22931

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[x] Ready to merge

Summary

What does the PR accomplish, why was it needed?
Fix CVE-2021-22931. Additional context in https://github.com/microsoft/azurelinux/pull/8269
Change Log

CVE-2021-22931

Does this affect the toolchain?

NO
Associated issues

#xxxx

Links to CVEs

https://nvd.nist.gov/vuln/detail/CVE-2021-22931

Test Methodology

Pipeline build id: https://dev.azure.com/mariner-org/mariner/_build/results?buildId=523598&view=results

Taking this change in the main branch first to get a full test run and ensure this change does not breaks any existing functionality. Then merging https://github.com/microsoft/azurelinux/pull/8269


check if /splits-name-from-* exists before processing it

Closes https://github.com/huggingface/datasets-server/issues/963
Adding validation on split names - config level job runners to throw an error if the response of the same info has already been computed.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Codecov Report
Patch coverage: 100.00% and project coverage change: +2.05 :tada:

Comparison is base (4788650) 89.58% compared to head (c5fd189) 91.63%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #995      +/-   ##
==========================================
+ Coverage   89.58%   91.63%   +2.05%
==========================================
  Files         147       53      -94
  Lines        7854     3778    -4076
==========================================
- Hits         7036     3462    -3574
+ Misses        818      316     -502

Flag
Coverage Δ

jobs_cache_refresh
?

jobs_mongodb_migration
?

libs_libcommon
?

services_admin
?

services_api
?

services_worker
91.63% <100.00%> (+0.08%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...tests/job_runners/test_parquet_and_dataset_info.py
100.00% <ø> (ø)

...ob_runners/config/split_names_from_dataset_info.py
94.73% <100.00%> (+0.98%)
:arrow_up:

...r/job_runners/config/split_names_from_streaming.py
96.15% <100.00%> (+0.80%)
:arrow_up:

...src/worker/job_runners/parquet_and_dataset_info.py
85.01% <100.00%> (ø)

...nners/config/test_split_names_from_dataset_info.py
100.00% <100.00%> (ø)

..._runners/config/test_split_names_from_streaming.py
100.00% <100.00%> (ø)

... and 94 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Support for generating cascades

See https://github.com/FlexModel/FlexModel/pull/14

@niels-nijens what is the status of this?

Could you rebase your branch?

Rebase done


Declarative yaml File from existing apiman setup

I would like to get a yaml file from my existing setup to for example duplicate an organisation. Is that possible in theory ?

Hi @marvinosswald, this is a feature we'd like to have (#21), but it isn't there at the moment.
What kind of elements are you looking to export? Are there plans etc. or just public APIs?

just public api's at first, but later on simply everything. i would like to generate a .yml file i can bootstrap an apiman from.


[$2000] Deep links are broken on iOS

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Install the New Expensify app
Sign in and close app
In a browser, follow a deep link like https://new.expensify.com/settings

Expected Result:
App should open to the settings page
Actual Result:
Safari or other default web browser opens the page
Workaround:
Open the app directly
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[x] iOS / native
[ ]  iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: v1.2.71-1
iOS Version: 16.3.1
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by: @shawnborton
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1676413158858859
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01d64bfcd13e8d5909
Upwork Job ID:<PHONE_NUMBER>399916544
Last Price Increase: 2023-02-27

Triggered auto assignment to @arielgreen (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

@marcaaron we had an old Issue on this which was closed. There was a bigger discussion. I'll try to link it in a bit

@marcaaron @arielgreen
https://github.com/Expensify/App/issues/13882#issuecomment-1370603008

Thanks @mvtglobally (great memory)!
I'll leave a comment over there. They seem like the same issue so if anything we will keep this one open and leave that one closed.

Job added to Upwork: https://www.upwork.com/jobs/~01d64bfcd13e8d5909

Current assignee @arielgreen is eligible for the External assigner, not assigning anyone new.

Triggered auto assignment to Contributor-plus team member for initial proposal review - @parasharrajat (External)

Triggered auto assignment to @NikkiWines (External), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

Is it still open for new ones?

@NikkiWines @parasharrajat This seems to be a feature request. Did it ever worked on iOS? As far as I can see, DeepLinkWrapper was never implemented for native platforms. It is just done for Desktop.

Proposal
Please re-state the problem that we are trying to solve in this issue.
Deep links do not work on iOS.
What is the root cause of that problem?
The root cause of this issue is the feature not implemented for iOS and Android as seen in this file src/components/DeeplinkWrapper/index.website.js. The links themselves new-expensify://route work fine on apps when opened in browsers. However, we're restricting the app to open the links on mobile here src/components/DeeplinkWrapper/index.website.js.
What changes do you think we should make in order to solve the problem?

We can simply remove the conditions that are restricting the code to run on mobile browsers in src/components/DeeplinkWrapper/index.website.js file. I'm working on the code changes and will post it here soon.
What alternative solutions did you explore? (Optional)
None

Proposal:

On the web application side:

Add static apple-app-site-association manifest file that reflects deeplinkRoutes.js content

On the iOS app project side:

Declare necessary entitlements

On the React Native (JavaScript) side:

Generalize the logic currently contained in ReportUtils.openReportFromDeepLink adding support for other types of deep links. Let's say, creating a new helper named LinkingUtils.openFromDeepLink.

Also, some minor cofiguration will need to be done on the App Store Connect side.

This may be due to https://new.expensify.com/.well-known/apple-app-site-association not setting the correct content-type header value, it should be application/json.

@parasharrajat The apple-app-site-association file is a JSON file and I think iOS expects the server to return the correct content type otherwise it will fail to consider checking for deep links (universal links). Not much of a proposal as I didn't go deep to understand the root cause since the incorrect content-type may be a blocker, so it's worth the fix anyway before proceeding with frontend proposals (if the issue presists).

IIRC this worked fine for iOS previously, @mvtglobally @kbecciv do you know if this is a regression?

Moving the convo to Slack here for broader visibility

Not overdue, in discussion

@marcochavezf rhere was an older issue about it. Doesn't seem like a new issue https://github.com/Expensify/App/issues/13882
But not sure if it's a regression due to another Pr

@NikkiWines, @parasharrajat, @arielgreen Whoops! This issue is 2 days overdue. Let's get this updated quick!

Commented in the thread

Chatted with @NikkiWines, I'm going to take this issue

@marcochavezf @parasharrajat @arielgreen this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

Current assignee @parasharrajat is eligible for the Internal assigner, not assigning anyone new.

Moved internal, deleting Upwork job

Looked for help in infra to set the content-type header.

Created an issue (for infra) to set the correct MIME type

Related PR https://github.com/Expensify/App/pull/15708

It was reverted because it was breaking the file on iOS. More details here.

Yeah, this is an issue with the deployment process for web. A fix for this was merged a few mins ago.

Reviewing label has been removed, please complete the "BugZero Checklist".

The solution for this issue has been :rocket: deployed to production :rocket: in version 1.2.81-1 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:

https://github.com/Expensify/App/pull/15743

If no regressions arise, payment will be issued on 2023-03-17. :confetti_ball:
After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.

[ ] External issue reporter
[ ] Contributor that fixed the issue
[ ] Contributor+ that helped on the issue and/or PR

As a reminder, here are the bonuses/penalties that should be applied for any External issue:

Merged PR within 3 business days of assignment - 50% bonus
Merged PR more than 9 business days after assignment - 50% penalty

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

[ ] [@parasharrajat / @marcochavezf] The PR that introduced the bug has been identified. Link to the PR:
[ ] [@parasharrajat / @marcochavezf] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:
[ ] [@parasharrajat / @marcochavezf] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:
[ ] [@arielgreen] Determine if we should create a regression test for the bug.
[ ] [@parasharrajat] If we decide to create a regression test for the bug, please propose the regression test steps to the appropriate location to ensure the same bug will not reach production again.
[ ] [@arielgreen] Review the proposed regression test steps and location.
[ ] [@arielgreen] Link the GH issue for creating/updating the regression test once above steps have been agreed upon:

Looks like something related to react-navigation may have been mentioned in this issue discussion.
As a reminder, please make sure that all proposals are not workarounds and that any and all attempt to fix the issue holistically have been made before proceeding with a solution. Proposals to change our DeprecatedCustomActions.js files should not be accepted.
Feel free to drop a note in #expensify-open-source with any questions.

Oops, the PR to change the content-type for the AASA file was merged, but the deep links are not opening the installed iOS app instead of the browser.

The universal link diagnostics and the AASA validator indicate that everything is fine:
https://user-images.githubusercontent.com/6829422/224510009-236894b0-88eb-46ec-9d02-2d0b891f1d57.MOV
But links to production and staging are still opening the broswer
e.g.
https://new.expensify.com/r/3504895439653267
https://staging.new.expensify.com/r/3504895439653267

It may be worth to check if this is related to cache, I think apple check those urls once a week but not really sure

@marcochavezf Can you please remove the "HOLD" from the title, this is not fixed yet.

It may be worth to check if this is related to cache, I think apple check those urls once a week but not really sure

Yeah, that was my first thought, but it has already passed a week since the apple-app-site-association file was updated. Probably we'd need to wait for a few days more.

@marcochavezf, @parasharrajat, @arielgreen Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

Update here. It seems that we'd need to wait until the other change is deployed to production and test again.

@marcochavezf is this still held?

The other files were already updated on production and staging, but the deep links are not still working.

Looking for more ideas of what could be the issue here.

I did some research on that and found some interesting things.
It was extremely difficult for me to get any one app to handle universal links. However I discovered that most domains uses different formats of AASA file than we do. Great example (and working one!) is YouTube:
{
    "applinks": {
        "apps": [],
        "details": [
            {
                "appID": "EQHXZ8M8AV.com.google.ios.youtube.dev",
                "paths": [
                    "NOT /yt/*",
						// Whole bunch of NOTs here,
                    "*"
                ]
            },
            // tonns of other appIDs
		]
	}
}

The differences to our file are:

instead of components they have paths which is array of strings
they don't have webcredentials key within applinks

There are other domains that follow similar format (pretty much every one I checked) but with webcredentials in it.
Interestingly youtube.com is the only one that Diagnostics in Settings finds correct (apart from us, but unlike us YouTube actually works).

After some more investigation I have one more thought to be tested.
In fact, iOS does not query our associated file directly. Instead, it uses apple-cdn by getting from url such like https://app-site-association.cdn-apple.com/a/v1/new.expensify.com
For some reason it does return 404 for our domain, while it works for pretty much every other I tested. My guess would be that there is something wrong with our response when serving associated-domains file. In fact, there are quite some headers and two cookies being returned. Can we try simplify that as much as possible and leave only relevant headers and no cookies. For reference, this is set of headers returned by branch for one of apps:
content-type: application/json; charset=utf-8
content-length: 1362
vary: Accept-Encoding
server: openresty
date: Tue, 28 Mar 2023 14:34:49 GMT
strict-transport-security: max-age=31536000; includeSubDomains
x-cache: Miss from cloudfront
via: 1.1 2e4b77c76f89825e36f12179cf1b33ea.cloudfront.net (CloudFront)
x-amz-cf-pop: WAW51-P1
x-amz-cf-id: 8lgANq-rSQNzbOs4GuRIniAxHrXVjNrQCw9dVy48ZLPKPkdzc6OM-g==

Thanks for the insights @mczernek! We're discussing with the infra team the headers issue.

After more investigation seems that our firewall is blocking Apple's crawler from pulling the AASA file from our servers. Infra already updated the rule. So I think we should only need to wait a few days to corroborate if the apple-cdn is updated.

It's working now!
https://user-images.githubusercontent.com/6829422/228837759-78cf462a-a40b-4d6b-90f4-05a4f8bee6c1.MP4

I reinstalled the app, restated the phone and it worked 🎉
Good job!

For future reference, if someone has a similar problem in the future, the problem was the firewall.
We were getting this error from the headers of the apple cdn Apple-Failure-Reason: SWCERR00101 Bad HTTP Response: 403 Forbidden:

And someone else that had the same error, posted in the apple developer forum that the problem was their firewall configuration: https://developer.apple.com/forums/thread/675324?answerId=668081022#668081022

I will close this one since it's working on staging (app from TestFlight) and production. Thank you all so much for your help!


NIFI-11285 Unable to build module created by maven archetype plugin

Summary
NIFI-11285
To test the plugin in case of services:
mvn archetype:generate -DarchetypeGroupId=org.apache.nifi -DarchetypeArtifactId=nifi-service-bundle-archetype -DarchetypeVersion=2.0.0-SNAPSHOT -DnifiVersion=2.0.0-SNAPSHOT -DartifactBaseName=nifi-mytest-services -DgroupId=org.apache.nifi -DartifactId=nifi-mytest-bundle -Dversion=2.0.0-SNAPSHOT -Dpackage=org.apache.nifi.mytest

In case of processors:
mvn archetype:generate -DarchetypeGroupId=org.apache.nifi -DarchetypeArtifactId=nifi-processor-bundle-archetype -DarchetypeVersion=2.0.0-SNAPSHOT -DnifiVersion=2.0.0-SNAPSHOT -DartifactBaseName=nifi-mytest-processors -DgroupId=org.apache.nifi -DartifactId=nifi-mytestproc-bundle -Dversion=2.0.0-SNAPSHOT -Dpackage=org.apache.nifi.mytest

Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[x] Apache NiFi Jira issue created

Pull Request Tracking

[x] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[x] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[x] Pull Request based on current revision of the main branch
[x] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[x] Build completed using mvn clean install -P contrib-check

[x] JDK 11
[ ] JDK 17

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

Reviewing...

+1 LGTM, verified the PR allows the generated bundle to build successfully. Thanks for the fix! Merging to main and support/1.x


added namshi/jose vulnerability

Hey guys, we just found a vulnerability in one of our libraries and would like it to be pushed here so that more users will realize they need to upgrade :)
cheers!
We will also publish a post soon with a longer explanations so I will change the link in another PR, for now its just a link to the commit that fixes the issue.

thanks man!


ls does not work with protocol prefix

Many commands seem to work with a protocol prefix as they call _strip_protocol internally however ls doesn't seem to work for ls. It seems consistent with s3fs and gcsfs that this is supported as well by adlfs.

I ran into the same issue. If I try to do
fs.listdir("az://my-container/path")

I get an empty list. If instead I do
fs.listdir("my-container/path")

I get all the blobs in path. Every other command I've tried works fine with the protocol in place so this took a bit of exploration to figure out.


Resource owner password credentials grant for Authorization Server add-on

See Jmix issue: https://github.com/jmix-framework/jmix/issues/2499

Implemented in #775


Create a new method on Config to get ConfigAccessor

Hello , I tried resolve the issue #93.
Hope this helps , Thanks
Signed-off-by: Breno Pessoa<EMAIL_ADDRESS>

Hi Breno! Please see my comment on the original ticket!
We first need to discuss whether we like the feature at all.
But thanks anyway for getting started with contributing!
I bet there are other things which might also need some help!

Hi Mark! I will seek for another issue which for I can try.
Thanks for the feedback.


Rule 3.3.4: if ufw is used, then the rule is overwritten

See https://github.com/ansible-lockdown/UBUNTU20-CIS/blob/a1f61e2245b787534ce6be7e454d8338704ad154/tasks/section_3/cis_3.3.x.yml#L104
Use the additional rule (maybe an additional when for ufw check is missing...)
- name: "AUTOMATED | 3.3.4 | PATCH | Ensure suspicious packets are logged, ufw part"
  sysctl:
    name: "{{ item }}"
    value: '1'
    sysctl_set: yes
    state: present
    reload: yes
    ignoreerrors: yes
    sysctl_file: /etc/ufw/sysctl.conf
  with_items:
    - net.ipv4.conf.all.log_martians
    - net.ipv4.conf.default.log_martians
  notify: sysctl flush ipv4 route table
  when:
    - ubtu20cis_rule_3_3_4
  tags:
    - level1-server
    - level1-workstation
    - automated
    - patch
    - rule_3.3.4
    - suspicious_packets
    - sysctl

hi @CFoltin
Thank you for raising this issue. Feedback like this helps us to improve the content we supply.
I have been working through the possibilities in this request.
I believe if you have set IPT_SYSCTL= /etc/sysctl.conf
This request would not be valid and not be required
If you are using UFW and not changed the sysctl import file, still want it logged you need to add that line to the /etc/ufw/sysctl.conf as you have mentioned.
Reading into this it could be quite a bespoke setting and not actually directly listed as part of a requirement for CIS unless the options above and ufw import file not changed and the settings in the ufw file override and turn it off.
I believe what would be required is.
either
If the file /etc/ufw/sysctl.conf exists and contains *log_martians=0 then change that as well even if not using ufw?
This would also count for any other controls not just 3.3.4.
or
Do we then change the /etc/default/ufw to be IPT_SYSCTL=/etc/sysctl.conf and add a comment?
This means settings that are added into that file then are not used we not addressed in CIS?
Any thoughts?
thanks
uk-bolly

hi @CFoltin
Thank you again for raising this issue. This has now been merged into the devel branch. Aiming to put this into main if all is well in the next couple of weeks.
many thanks again
uk-bolly

This was added to PR https://github.com/ansible-lockdown/UBUNTU20-CIS/pull/74 that addressed a lot of fixes.  This has been merged to main.  Thanks @CFoltin  for the find.


Migrate to Go Dep

Hey there! I'm going to be working on a list_of_map style addition here in a moment. Go Dep is generally accepted standard for dependency management. (Or at least it was in my other projects 😄 )
So I figured I would toss this in. LMK if you're not interested @Ashald

I'm not an expert in Go and I just failed to figure out how to use Go Dep properly. 😞
It' be great to move to it as I understand it seem to be "the future" but I'd highly appreciate if you could update readme to include few instructions for noobs like myself how to use it. I'm not asking for a complete and thoroughful guide but just few basic commands like that would replace govendor init and govendor fetch +m. 🙂
Thanks!
P.S.: As for list_of_map - frankly I'm not sure if that can be implemented right now (depending on what you have in mind) given restrictions in HCL (https://www.terraform.io/docs/extend/schemas/schema-types.html#typelist). On the other hand, to the best of my knowledge HCL 2 is coming out really soon (I'd expect its release to be announced on October 23 at HashiConf) and I think it should add support for schemas of arbitrary complexity.

I think you are totally right 😂 and I had to give up. 😄 But in the meantime here is some of the commands for godep
Dep is a tool for managing dependencies for Go projects

Usage: "dep [command]"

Commands:

  init     Set up a new Go project, or migrate an existing one
  status   Report the status of the project's dependencies
  ensure   Ensure a dependency is safely vendored in the project
  version  Show the dep version information
  check    Check if imports, Gopkg.toml, and Gopkg.lock are in sync

Examples:
  dep init                               set up a new project
  dep ensure                             install the project's dependencies
  dep ensure -update                     update the locked versions of all dependencies
  dep ensure -add github.com/pkg/errors  add a dependency to the project

Use "dep help [command]" for more information about a command.

So what I did was dep init and then dep ensure and that's it!

I guess what was confusing to me is that dep seems to take way more time and space and I was not sure I understand reasons for that and that I actually need what it's doing. It seems that it resolves all dependencies transitively and tries to cache them all?
On MBP2018 4.5 GHZ, govendor init && govendor fetch +m takes 1s while dep init took 1 minute.
I'm not against dep I just want to be sure it's really a proper tool in this case. 🙂

Yea, I see what you mean. Not sure exactly how the dependency resolution of go dep works. But seems like govendor is clearly more performant. And the more I mess with HCL it seems to be the standard for terraform plugins, so I think it's not worth it.


Allow passing a new login & password as environment variables

This would allow for setting up the SQL Server instance using a randomly generated password for the SA login, creating the new login using the login name and password provided and then disabling the SA login.

FYI - We released CTP 1.4 today.  This release of the mssql-server-linux image now includes the mssql-tools package (sqlcmd and bcp) in it.
Executing sqlcmd as part of the entrypoint.sh script can be used for this kind of scenario for now.  Since this is such a commonplace requirement we want to make it easier in the future, but sqlcmd will provide a reasonable option until then.


How to disable ssl cert check if auto-signed

Hi all,
how I can disable the ssl certificate chec if the certificate is auto-signed
I read something here [1] but I was enable to find docs about if in Config class.

https://github.com/fguillot/picoFeed/issues/59

error log:
exception 'PicoFeed\Client\InvalidCertificateException' with message 'Invalid SSL certificate' in /home/cmignanti/code/georss-merger/vendor/fguillot/picofeed/lib/PicoFeed/Client/Curl.php:406
Stack trace:
#0 /home/cmignanti/code/georss-merger/vendor/fguillot/picofeed/lib/PicoFeed/Client/Curl.php(285): PicoFeed\Client\Curl->handleError(60)
#1 /home/cmignanti/code/georss-merger/vendor/fguillot/picofeed/lib/PicoFeed/Client/Curl.php(303): PicoFeed\Client\Curl->executeContext()
#2 /home/cmignanti/code/georss-merger/vendor/fguillot/picofeed/lib/PicoFeed/Client/Client.php(213): PicoFeed\Client\Curl->doRequest()
#3 /home/cmignanti/code/georss-merger/vendor/fguillot/picofeed/lib/PicoFeed/Reader/Reader.php(75): PicoFeed\Client\Client->execute('https://owm1.un...')
#4 /home/cmignanti/code/georss-merger/index.php(31): PicoFeed\Reader\Reader->download('https://owm1.un...', '', '', 'user', 'pass;...')
#5 [internal function]: {closure}()
#6 /home/cmignanti/code/georss-merger/vendor/slim/slim/Slim/Route.php(468): call_user_func_array(Object(Closure), Array)
#7 /home/cmignanti/code/georss-merger/vendor/slim/slim/Slim/Slim.php(1357): Slim\Route->dispatch()
#8 /home/cmignanti/code/georss-merger/vendor/slim/slim/Slim/Middleware/Flash.php(85): Slim\Slim->call()
#9 /home/cmignanti/code/georss-merger/vendor/slim/slim/Slim/Middleware/MethodOverride.php(92): Slim\Middleware\Flash->call()
#10 /home/cmignanti/code/georss-merger/vendor/slim/slim/Slim/Middleware/PrettyExceptions.php(67): Slim\Middleware\MethodOverride->call()
#11 /home/cmignanti/code/georss-merger/vendor/slim/slim/Slim/Slim.php(1302): Slim\Middleware\PrettyExceptions->call()
#12 /home/cmignanti/code/georss-merger/index.php(139): Slim\Slim->run()

The config option is missing it seems.
As for your problem: if you are required to support a self signed cert, it's better to import it to the installed certs rather than turning off the checks for everything else and opening yourself up to MITM attacks.
Check the last answer on this page http://stackoverflow.com/questions/5109661/add-self-signed-certificate-to-ubuntu-for-use-with-curl

It's not possible at the moment and my personal point of view is, that it shouldn't be possible. And because I take your security serious, I do not even show you the place within the code to disable certificate verification.
If someone can read your unencrypted traffic, it's an easy game to do a man in the middle attack if you use SSL without certificate verification. Long story short, disabling certificate verification is exactly secure as doing http requests unencrypted.
startssl.com for example issues ssl certificates for free. Another possibility is to setup your own Certificate Authority and import this CA certificate to your trusted CA certificates.

If someone else need the quick and dirty fix
diff --git a/vendor/fguillot/picofeed/lib/PicoFeed/Client/Curl.php b/vendor/fguillot/picofeed/lib/PicoFeed/Client/Curl.php
index 278eeb4..89f9b8e 100644
--- a/vendor/fguillot/picofeed/lib/PicoFeed/Client/Curl.php
+++ b/vendor/fguillot/picofeed/lib/PicoFeed/Client/Curl.php
@@ -242,6 +242,8 @@ class Curl extends Client
         curl_setopt($ch, CURLOPT_ENCODING, '');
         curl_setopt($ch, CURLOPT_COOKIEJAR, 'php://memory');
         curl_setopt($ch, CURLOPT_COOKIEFILE, 'php://memory');
+        curl_setopt($ch, CURLOPT_SSL_VERIFYHOST, 0);
+        curl_setopt($ch, CURLOPT_SSL_VERIFYPEER, 0);

         // Disable SSLv3 by enforcing TLSv1.x for curl >= 7.34.0 and < 7.39.0.
         // Versions prior to 7.34 and at least when compiled against openssl

@claudyus It is not even a qick'n'dirty fix, it's a big security hole!
I'm not sure if you didn't understand what I've written or you simply do not care about security.
To make my point clear once more: If you disable the peer/host validation, you do not need any encryption. Using unencrypted HTTP in that case, is your quick'n'dirty fix.
@fguillot Would you please remove  to comment above! It's obviously that we need to protect the users from themselves

Maybe we should add a FAQ section that discusses this issue and provides a link or description how to add self signed certs.

@mkresin it is just an hack to resolve the my ssl problem when I fetch info from a server configured as https (by another team) and I should retrieve those information inside aipsec lan. I pretty sure that in this situation I can relax my checks.
Feel free to remove the comment if it is so "dangerous"

it is just an hack to resolve the my ssl problem

@claudyus You named it a quick and dirty fix. That could be misleading to other people with the same problem. They could suspect an fixable error in picoFeed rather than a - normally - easy fixable configuration problem with their PHP installation. That's my problem with your solution.
Am I right, that your verified that it's really a self signed certificate? Especially with php on windows you have to do some additional steps to successful verify even "officially" signed certificates.
Have you every tried to ask the other team for the CA Certificate that signed the remote servers certificate?
I'm still interested in fixing your setup rather than disabling security related settings.

$response = $client->get($url); // guzzle
$body = (string)$response->getBody();

$reader = new Reader();
$parser = $reader->getParser($feed->url, $body, 'utf-8');

$data = $parser->execute();

3rd party client workflow with disabled ssl cert check.

@fguillot we should probably lock the issue :)
@zelenin please don't use that, just download the cert and add it to your trusted certs or add a startssl/letsencrypt/wosign cert

We should probably add self signed cert best practices to the docs btw


Create mur_sst_tutorial_local.ipynb

added a cloud tutorial for MUR SST

@cgentemann , I'm still waiting to be added back to the NASA Org... sigh!


OPS team Configure personal GitHub settings

Add a "code owners" file to the repo and include members of the team that will be monitoring.

@junminahn walked team through this last week.


fix recyclable drop

Fixes a panic I noticed while trying the helix integration. My guess is that:

Static storage transitively contains recyclable values.
The storage gets dropped at thread exit.
Some recyclable value Drop impl is called and attempts to access the thread local.
And that panics b/c the thread local Drop call is ongoing.

Did this happen at exit? I must have not run into it myself
Anyway this look good to me

Did this happen at exit? I must have not run into it myself

Yes, it always happened when exiting helix. FWIW I ran your helix branch together with the latest master from Steel (with some minor modifications to get it to compile).

I must just be behind then. I'll take a look this weekend

@jrvidal What were the other issues you had with helix + steel? I'm trying with latest master right now and running into some unexpected behavior with multi arity functions from the built ins

I just uploaded a couple branches:
https://github.com/jrvidal/helix/tree/mwp-steel-integration-hack
https://github.com/jrvidal/steel/tree/steel-for-helix
I might be able to add more details later.


9958: Stop preserving search query between missions

Uses URLSearchParams to build, modify, and stringify the url params without the search query.
Note: Normally I'd use the more conventional npm query-string lib for this but it seems like the legacy JS code wouldn't deal well with that. This should be a robust solution, as far as I know. Related SO question here as well.

Done. I haven't specifically tested on a real IE browser, think it's worth downloading a VM and setting that up or do you have any other way to test?

Browserstack baby

Hah. All sorts of things are broken in IE, including all React and anything that uses the legacy JS ELMO global. Navigation is also significantly impaired because several important link handlers don't trigger.
I think IE may be a lost cause (and clearly no tears have been shed by users, at least not loudly), I propose reverting the last commit and officially denoting IE as unsupported. Thoughts?