Ease the process to add extra volumes for local data registration

Hi 👋
I find it cumbersome to attach an external volume to the backend worker and server. The current process requires to copy / paste the volume list in the default values and add yours to the list but it's error prone and make it harder to maintain (at each upgrade I have to be sure that the default list have not been altered). Also using this method you either have to use hostPath or a StorageClass, you can't define the pvc/pv using another provisioner that would require annotations on the pvc for example.
I suggest:

Adding a more flexible extraVolumes and extraVolumeMounts entry to ease the configuration. This is a fairly common pattern among chart maintainers that help operations: https://hackernoon.com/the-art-of-the-helm-chart-patterns-from-the-official-kubernetes-charts-8a7cafa86d12 (chapter "3 Empowering Our Fellow Chart Developers")
Hardcode volumes that are required. The list of volumes in the chart default values isn't really optional at the moment: if I override the volumes variable and ommit a volume by mistake the backend won't work. It appears that nobody should ever touch this value so it make sense to me to simplify the values by hardcoding those mandatory volumes.

I hope the feedback help making the chart a little bit easier to use.

Hello @ClementGautier,
Since the release 1.8.0 of the substra-backend chart it is not necessary anymore to copy the whole persistence.volumes object.
For example if in your values you define only something like this:
persistence:
  volumes:
    customVolume:
      size: "2Gi"
      readOnly:
        server: false
        worker: true

you will have a new volume mounted on your backend and worker pod under the /var/substra/medias/ path.
Result in the rendered yaml:
containers:
      - name: substra-backend
        volumeMounts:
          - name: data-customVolume
            mountPath: /var/substra/medias/customVolume
            readOnly: false

tied to the following pvc:
      - name: data-customVolume
        persistentVolumeClaim:
          claimName: RELEASE-NAME-substra-backend-customVolume

but all the required pvc's (compositealgos, computeplan, ...) will still be present alongside this new one.
For the point number two they are not hardcoded to be able to specify volume sizes independently which can be useful sometimes, but as stated above you do not need to redefine the wole list of volumes anymore. This it a benefit from the transition we made from a yaml list to a yaml object.

Great! That fixes the issue then :)


Android build failed - package com.google.android.gms.license

Hi.
I can't build project with this error.
Error: more than one library with package name 'com.google.android.gms.license'

Yesterday I could build my project with any problem, but suddenly today morning I can't build the project.
The sources are same with yesterday`s. I did not changed a single line.
I couldn't find any clue of package com.google.android.gms.license form google search.
I know that multiple packages using  com.google.android.gms can cause dependency problem, so I configured build.gradle so that not conflict each other, and it worked till today morning.
But what iscom.google.android.gms.license ? No search results on google.
How can I fix this? Any Help?
configurations

Windows 10

package.json

"dependencies": {
  ...
  "react-native": "0.54.2",
  "react-native-admob": "^2.0.0-beta.3",
  ...
}

Errors

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':react-native-admob:processReleaseResources'.
> Error: more than one library with package name 'com.google.android.gms.license'

build.gradle

in gradle.properties googlePlayServicesVersion=11.8.0

dependencies {
  ...
    compile(project(':react-native-admob')) {
       exclude group: 'com.google.android.gms'
    }
    compile "com.google.android.gms:play-services-ads:$googlePlayServicesVersion"

    compile(project(':react-native-maps')) {
        exclude group: 'com.google.android.gms'
    }
    compile "com.google.android.gms:play-services-base:$googlePlayServicesVersion"
    compile "com.google.android.gms:play-services-maps:$googlePlayServicesVersion"

    compile(project(':react-native-onesignal')) {
       exclude group: 'com.google.android.gms'
    }
    compile "com.google.android.gms:play-services-gcm:$googlePlayServicesVersion"
    compile "com.google.android.gms:play-services-analytics:$googlePlayServicesVersion"
    compile "com.google.android.gms:play-services-location:$googlePlayServicesVersion"

    compile(project(':react-native-google-sign-in')) {
        exclude group: 'com.google.android.gms'
    }
    compile "com.google.android.gms:play-services-auth:$googlePlayServicesVersion"
  ...
}

This is happening because of the latest google play services 12.0.0. I also ran into this issue. So for now I just manually edited build.gradle (node_modules/react-native-admob/android/build.gradle) and changed compile 'com.google.android.gms:play-services-ads:+' to compile 'com.google.android.gms:play-services-ads:11.8.0'
play services 11.8.0 was the last known working version on my react native project. Once I downgraded to 11.8.0, few other dependencies that my project were using also threw the above error..I went ahead and downgraded them all to 11.8.0.
I know this is not the ideal solution but this unblocks me and I can proceed with my app development work.

@vijaykarthik
I added little enhancement to your solution, I could build my project without modifying build.gradle files under /node_modules/.
I updated /android/build.gradle (not /android/app/build.gradle) like below.
def googlePlayServicesVersion = '11.8.0'

allprojects {
    repositories {
				...
        configurations.all {
            resolutionStrategy {
                // react-native-admob
                force "com.google.android.gms:play-services-ads:$googlePlayServicesVersion"
                // react-native-maps
                force "com.google.android.gms:play-services-base:$googlePlayServicesVersion"
                force "com.google.android.gms:play-services-maps:$googlePlayServicesVersion"
                // react-native-onesignal
                force "com.google.android.gms:play-services-gcm:$googlePlayServicesVersion"
                force "com.google.android.gms:play-services-analytics:$googlePlayServicesVersion"
                force "com.google.android.gms:play-services-location:$googlePlayServicesVersion"
                // react-native-google-sign-in
                force "com.google.android.gms:play-services-auth:$googlePlayServicesVersion"
            }
        }
				...
    }
}

This worked for me.

Thank god for this community.

I bumped build fail issue:
> Task :react-native-admob:compileDebugJavaWithJavac FAILED
/Users/patw/Projects/xxx/node_modules/react-native-admob/android/src/main/java/com/sbugert/rnadmob/RNAdMobRewardedVideoAdModule.java:20: error: RNAdMobRewardedVideoAdModule is not abstract and does not override abstract method onRewardedVideoCompleted() in RewardedVideoAdListener
public class RNAdMobRewardedVideoAdModule extends ReactContextBaseJavaModule implements RewardedVideoAdListener {
       ^
1 error

And @nopedev's solution can solve this issue too! Thanks!

I've catched this error with the same text as @patw0929.
Thanks @nopedev for the fix.
In general, you want to force downgraded version of Google Play Services to all com.google.android.gms:play-services-whatever your dependencies use.
In my case react-native-device-info used latest com.google.android.gms:play-services-gcm.

thx

It is actually not a solution. Problem is that this library does not work with newer play-services versions.
For me, even after I set play services to 11.8.0, I still get the error:
error: RNAdMobRewardedVideoAdModule is not abstract and does not override abstract method onRewardedVideoCompleted() in RewardedVideoAdListener

It is actually not a solution.

Yeah, it's indeed just a workaround.

For me, even after I set play services to 11.8.0, I still get the error

Have you also forced all other Google Play Services dependencies, that are used by other packages, to 11.8.0?

@nopedev  We also had the same issue with the mstore app (mstore.io) but then i found out the issue.
Thanks your great comment.

Just point your package to the github repo instead of the version. #273 allows you to simply use the package without any of the modifications stated above.

@SwiftPolar how might one go about doing that? Can you provide an example please.

In package.json:
...
"react-native-admob": "github:sbugert/react-native-admob#master",
...

This is not the best way to do things but I suppose as a workaround its good enough.

Cool thanks

This is not the best way to do things but I suppose as a workaround its good enough.

It would be better to point to a specific commit in the master branch (by specifying the last commit's hash after the #) instead of pointing to master branch directly.

@angly-cat Yes that is a safer choice too.

@sametcilli You mean you still had this problem with the recent 2.0.0-beta.5 version?

I guess, I installed 3 days ago, it has an error, I installed v2.0.0-beta.2

@nopedev solution was working for me for the last few days but now it's crashing on app startup again. Has something changed again with the SDK?

@nopedev
solution worked.

i have resolved issue by updated package
npm i --save react-native-admob@next

Anyone still having this issue?

I still have this issue with nopedev's code in version 2.0.0-beta.5

@Override
public void onRewardedVideoCompleted() {
    sendEvent("rewardedVideoDidClose", null);
}


Map filters reset after a post location is clicked and closed

notes in the gif

clicking the "x" keeps the filters (first click)
clicking on the map resets the filters? it might be because it's filtering on a region..

expected behaviour

when a modal is open, clicking off it should be treated like a "x"
if the map is clicked while a filter is on, the filter should be an and not or
we should also probably add what it's filtered on after Search Result

We should just kill clicking on a region to filter then. Keeping track of which state we're in an whether clicking on a region to filter is okay is too much work.

good suggestions ^

I've removed filtering on region via clicking.


feat: Add support for DSSE Rekor type

This is in preparation for switching over the Rekor entry type in the slsa github generator to be the newer DSSE type. This adds support for searching for both intoto v001 and dsse v001 entries.

@laurentsimon Lemme know if there's anything I'm missing since I don't have full context into this codebase.


Parse error: expected 0x42 fmt if vorb missing

This error happens when trying to convert some .wem files (obviously cant share them because of copyright reasons). The error is consistend i.e. the same files give the same error.
I'm using a batch file to convert all of them:
for %%f in (*.wem) do "./ww2ogg.exe" %%f --pcb "./packed_codebooks_aoTuV_603.bin"
pause
for %%f in (*.ogg) do revorb.exe %%f
pause

same problem here.
why is this issue closed?

same problem here. why is this issue closed?

As the readme notes:

Parse error: expected 0x42 fmt if vorb missing suggests that the input is not Vorbis data at all, and so it is not supported by this program.

I suggest trying to decode the file with vgmstream, it supports more format, and it will tell you what codec is used.


Enabled SA1627 by default, moved text strings to resources.

Fixes #1692
Also moved the text strings to the resources.

Current coverage is 79.45%

Merging #1724 into master will increase coverage by +0.01% as of ada05ba

@@            master   #1724   diff @@
======================================
  Files          545     545
  Stmts        32645   32649     +4
  Branches      9030    9030
  Methods
======================================
+ Hit          25936   25941     +5
+ Partial       5245    5244     -1
  Missed        1464    1464

Review entire Coverage Diff as of ada05ba

Powered by Codecov. Updated on successful CI builds.

:+1:


replaces deprecated apiVersion with the latest one

What this PR does / why we need it:
Replaces the deprecated apiVersion: kudo.dev/v1alpha1 to the latest apiVersion: kuttl.dev/v1beta1.
The release 0.4 came with an update to the apiVersion for the kuttl tool. But this was not updated in the documentation. This PR fixes that.
Which issue(s) this PR fixes:

Fixes https://github.com/kudobuilder/kuttl/issues/234

thanks @rishinair11 this is really appreciated!


Fix params hashing

Description
Fixed hashing when dict passed as parameter for task via TOML config.
Motivation and Context
Fix #2538
Have you tested this? If so, how?

I ran test cases from #2538
I've included dict parameter into config example which loads in tests.

Wow, so many commits O.o

I've cherry-picked all commits into #2540


mDNS / Avahi / Bonjour / Zeroconf support

Hi,
I would like to access my workstation (linux.local) with ConnectBot using the mDNS name.
Currently ConnectBot is unable to resolve the hostname.
I assume the configuration on my workstation is correct, because if I use a zeroconf browser from the play store it is correctly discovered. Connecting with the ip address is also working.

Same problem here

Android needs to support mDNS natively for all DNS lookups in my opinion, but lacking that, it would be really nice to see it added to the app.

:+1:

I've read that Android API level 16 and up support Zeroconf/Avahi.
It would be nice having this feature (that I also need).

@jvalencia80 As far as I know, Android has supported it for a while. However, it does not by default use mDNS for lookups. The app developers themselves will have to manually let the app do an mDNS lookup, besides the normal DNS resolution.

Is this resolved? I see there is a commit referencing this feature request, but I still can't use MEANS.

s/MEANS/mDNS/

No, that's a wrong reference.
Someone forked the repo, created a pull request in the fork, which coincidentally had the same issue number as this issue in this repo.
Then later that pull request was merged from that fork to this repo, and github misinterpreted the reference.

Any recent development on this area? Would a PR be welcome?


docker share, docker access

I'm finishing up testing and I need to remember to change the default URL from staging to prod when I mark this ready for review.

I'd prefer to bury the docker stuffs into a "docker" folder and that should come at the end of the left nav

I've responded to feedback and this is ready for final review.


testing pydantic v1.10.0a2

TO NOT MERGE
This is pull request is just to test pydantic v1.10, see https://github.com/pydantic/pydantic/discussions/4359

Hi, please can you kick off tests so I can check if v1.10 will break anything.

pydantic v1.10 drops support for python3.6 which was EOL at the end of last year, so those failures are to be expected.

Great, this is passing except for 3.6 which is expected.

pydantic v1.10 drops support for python3.6 which was EOL at the end of last year, so those failures are to be expected.

We've looked at our download-stats and realised that people still use Python 3.6 a lot, so we re-added support for it.

:disappointed: sorry to hear that.
I'm afraid it's definitely not possible to re-add it for pydantic v1.10.

Not that I want to ask for it! Luckily, for now the pain is manageable.

I guess we can still support pydantic v1.10 conditioned on python being 3.7 or newer? Not sure whether that will require ad hoc fixes here and there and cause us headache, but we can try.

Or you can release a new version that support pydantic v1.10 and drops python 3.6 and tell users to use the old version if they're still (for some strange reason) using python 3.6.
But python 3.6 is EOL, no longer supported, toxic, run away.

But python 3.6 is EOL, no longer supported, toxic, run away.

You're knocking at an open door 😄


bug: Cairo 2.6.0 contracts fail to compile

Bug Report
On declaring contracts compiled with Cairo 2.6.0, Madara returns a 500 error
2024-04-30 12:44:13 Failed to convert BroadcastedTransaction to AccountTransaction: Failed to compile to Sierra
The same issue can be resolved when compiling with Cairo 2.5.0.
Steps to reproduce:
casm.contract_class.json
sierra.compiled_contract_class.json
The files linked above are ones generated with Cairo 2.6.0, attempting to declare them causes an error in Madara

Sorry, missed the mention 🙈
@tdelabro yes exactly, had the same issue, mentioned here https://github.com/keep-starknet-strange/madara/blob/main/crates/client/starknet-block-import/README.md#notes
IIRC one of the proposed solutions was to have multiple external compilers for the according version ranges.

repository archived in favor of https://github.com/madara-alliance/madara


Investigate returning __splicedPackages when importing nixpkgs

The NixOS module for Nixpkgs does this since https://github.com/NixOS/nixpkgs/pull/190358. Should all callers of Nixpkgs do so? If so, make Nixpkgs always return __splicedPackages.

I have a theory that we could do something like this to essentially fix all splicing problems:
diff --git a/pkgs/top-level/stage.nix b/pkgs/top-level/stage.nix
index 3886ae04e492..e95a303bdc1a 100644
--- a/pkgs/top-level/stage.nix
+++ b/pkgs/top-level/stage.nix
@@ -288,4 +288,4 @@ let

 in
   # Return the complete set of packages.
-  lib.fix toFix
+  lib.fix (x: toFix (splicePackages x))


direction for input instead for output

is there a way to set direction for input ?

i want the line like i draw instead
is it possible?

Don't think so, but you may want to create a newline for the text?
SOP Penerimaan Material\nSub Prosedur : Maetrial Retur

thanks for replying, sorry for late reply.
i mean is there array so that it won't overlap on other process box

Mabe try with align-text:
https://github.com/adrai/flowchart.js/issues/221#issuecomment-846919013


connectFactoryObservable: Object parameters will cause infinite loops

Hey, thanks for the project. You saved me a lot of headaches.
The following code will lead to an infinite loop when used in a component:
export type ArticleQuery = {
	searchTerm?: string,
	author?: string
}

const [useArticleQuery] = bind((query: ArticleQuery) => searchForArticles$(query))

This is likely, because the NestedMap only expects an array of arguments and doesn't deeply compare objects.
I would suggest taking a look at the system that react-query uses for key generation. They sort the entries of an object before converting it to json. (See the function  stableStringify)
Maybe it is also somehow possible to detect when a loop occurs and throw an error instead of potentially sending many requests to a server.

:wave: @SebKranz
Thanks a lot for your kind words. I'm very sorry that you found yourself in that situation. The "Factory Overload" is very tricky. We should improve the docs to explain it better. We should add examples that illustrate the different ways that it can be leveraged, and we should also explain why we have decided to compromise its ergonomics in order to avoid memory leaks.

I would suggest taking a look at the system that react-query uses for key generation. They sort the entries of an object before converting it to json. (See the function stableStringify)

This library is very different from react-query. IMO it's a lot more powerful and perf-sensitive than react-query because it allows consumers to have full-control over how their caches. Also, accessing a stream stored on a factory bind is a constant time (O(1)) operation, and we must keep it like that because this function is evaluated for every react update.
The bindings on @react-rxjs/core will never do the kind of "magic" that react-query does, because doing that would be detrimental for many of the use-cases for which this library was created.
However, we are aware that there are many use-cases for which the core API is not ergonomic. That's why have created the @react-rxjs/utils package. We will keep improving it to accommodate other common patterns.

Maybe it is also somehow possible to detect when a loop occurs and throw an error instead of potentially sending many requests to a server.

This is a really good suggestion. I think that it would be possible to do some of those checks on the development build and to show warnings when we detect a case that it's likely to fall for this pitfall. Thanks for the suggestion! We will act on it, for sure! Also, we will improve our eslint plugin to detect some of these cases.
It will take us a few days to update the docs and I would like to help you solve your particular problem before then. However, I'm afraid that I don't have enough context to help you. It's possible that you don't even need to use the "Factory Overload". Could you please give us more context on the problem that you are trying to solve? A codesandbox with a basic proof of concept would be ideal.
Thanks and looking forward to hearing back from you.

Hey, apollogies for the late replay.
My use case is that I have some url-parameters that would be pointless to store as global state since they're handled by my router.
I use these parameters to make a request to a server. (With a cache in between thats not unlike react-query)
Because the Endpoint has many optional parameters, and because these parameters are passed through multiple functions, I collect them in an object and created an interface for them.
But more generally, I think that objects are an elegant way of describing complex API's since JS only supports positional arguments.
On a side note: I believe that it's important to be able to easily switch between the reactive and imperative style. There are cases where the imperative approach is more intuitive or even required (eg. by a framework).
connectFactoryObservable is an important bridge to all kinds of other frameworks (router, useState, or even useSelector). Therefore, maybe it should be more flexible?
However, will first read the updated documentation, so that I can better understand the reasons behind your decision.

Hi @SebKranz ,
I'm very sorry for the late response.

On a side note: I believe that it's important to be able to easily switch between the reactive and imperative style. There are cases where the imperative approach is more intuitive or even required (eg. by a framework).

💯

connectFactoryObservable is an important bridge to all kinds of other frameworks (router, useState, or even useSelector). Therefore, maybe it should be more flexible, even at the cost of performance?

Yes, you are correct connectFactoryObservable is an important bridge for covering the impedance mismatch between the 2 different mental models. Although, I would rather leave it the way it currently is and add a util that provides the extra-functionality that you are looking for.

However, will first read the updated documentation, so that I can better understand the reasons behind your decision.

Yesterday we did a small update. It's still far from perfect, but it does explain the most important things. The next thing that I'm going to do is to add an example in the tutorial that explains in detail other use-cases and gotchas of the parametric bind. I'm very sorry that it's taking us so long to address this.

My use case is that I have some url-parameters that would be pointless to store as global state since they're handled by my router.

I use these parameters to make a request to a server. (With a cache in between thats not unlike react-query)

Because the endpoint has many optional parameters, and because these parameters are passed through multiple functions, I collect them in an object and create a type for them.

But more generally, I think that objects are an elegant way of describing complex API's since JS only supports positional arguments.

In a situation like the one that you are describing, I think that the best way to approach that problem (taking into account the "philosophy" of this library) would be to have a stream that listens to the url changes, then you can use that stream to create the query and the article, something more or less like this:
const [useArticle] = bind(url$.pipe(
  map(getQueryFromUrl),
  switchMap(searchForArticles),
))

If for some reason, you also needed to access the ArticleQuery from somewhere else, then you could do this:
const [useArticleQuery, articleQuery$] = bind(url$.pipe(
  map(getQueryFromUrl),
))

const [useArticle] = bind(articleQuery$.pipe(
  switchMap(searchForArticles)
))

Please let me know what your thoughts are or if you have suggestions for better (or more ergonomic) ways of approaching these problems. Although, please be mindful of the motivation of the library and the trade-offs that we have chosen. The trade-offs, we can change them, of course, as long as they don't compromise the main goals of the library.
Sorry again for getting back to you so late. I will try to be a lot more responsive in the future.

Hey, so I guess I won the contest for the most delayed reply in this thread ;)
I have no excuse other than: I forgot.
Your suggested solution works and seems reasonable.
When I realize that the amount of state-information that needs to be passed through the react-hierarchy is usually small, the limitation appears far less restrictive.
So far, I haven't come across other use-cases where I needed to pass a large amount of arguments to bind.


Confusing error reason when option has no "long" name

Rust Version

rustc 1.25.0 (84203cac6 2018-03-25)

Affected Version of clap

2.31.2

Steps to Reproduce the issue

define an arg like this: .arg(clap::Arg::with_name("--test"))
start a clap program with  ./app_name --test

Actual Behavior Summary
error: Found argument '--test' which wasn't expected, or isn't valid in this context

USAGE:
    wwrap [--test]

For more information try --help

As you see, the logging is kinda self-contradictory. "We don't accept --test because we only accept --test".
Expected Behavior Summary
runtime error at parse time, or any other solution giving correct error reasons

UPD: actually the error reason seems to be different. I didn't provide a "long" name for an option, which is why it didn't work. The "Actual Behavior" above seems to be misleading anyway, IMO, so I'm leaving the issue open.

It looks like maybe the intent was to use Arg::from_usage instead of with_name. I agree it's a slightly confusing message, but I can't actually think of a better way to word it. Perhaps a debug lint if an arg name starts with a single or double hyphen?
I'm going to close this for now as it's working as intended, and a lint against this would need a way to turn off the lint which is more complicated than I think this issue warrants.


#739 Missing instance of At for ListMap

Changes

add At instance for ListMap
fixes #739

LGTM, thanks Jens

Cool, I'll take a look.

Great. Thanks Jens


compiler warnings for 10.0.7

Warnings show when built on Debian/Unstable using GCC 13. Please see the attached build log for details.
No warning appear when built on Debian/Stable using GCC 12.
dhcpcd_10.0.7-1_i386.build.txt

Testing for closefrom ... no
I would say your glibc header is broken as this is the test case
#include <unistd.h>
int main(void) {
        return closefrom(3);
}

What does config.log have to say about that?

Attached.
config_log.txt

Can you test that please?

This seems to fix it.
dhcpcd_10.0.7-1_source.build.txt

I put out a new release with this and another compile fix in.


[Question] What is the different between dreambooth and fine-tune in sd-scripts

There is only Japanese version of fine-tuning, and usually people refer dreambooth to "fine tune" native stable diffusion/other diffusers.
So what is the main difference between the "dreambooth training" and "fine-tune training" in this repo?
When I make a decision, what are the criteria I should refer to?
From my rough understanding, for dreambooth, you can train some image for multiple times (with folder name modification).
And for fine-tune, is this for tuning the model with many images in the similar class?
Really confused about this.

Thanks, this is really helpful


Validate complexity of passwords

The OWASP sheet at https://www.owasp.org/index.php/Ruby_on_Rails_Cheatsheet#Authentication has a recommended validator for passwords in the user model as follows:
   validate :password_complexity
   def password_complexity
      if password.present? and not password.match(/\A(?=.*[a-z])(?=.*[A-Z])(?=.*\d).+\z/)
          errors.add :password, "must include at least one lowercase letter, one uppercase letter, and one digit"
      end
   end

Let's plug it in!
Marking as beginner friendly because it's a good way to try out model validations; not marking as deploy since we only really need it for release 2, and not Release 1: Lisa Edition.

@mccabe615 pointed out the following in #417 : "Should there be stronger requirements on password strength? Such as, minimum 10 characters, include at least one, special char, number, upper case and can't use the word password in any form."
He's definitely right that the regexp above is a bare minimum and that we should ratchet up the complexity, as well as not allowing any 'password1234' kinda stuff.

@colinxfleming I'll try to take a crack at a PR in the next couple of days to add a complexity validation for this.

Thanks, bud. Let me know if you run into any trouble spots or if there's
anything I can unstick for you.
On Sat, Jun 25, 2016 at 3:22 PM Mike McCabe<EMAIL_ADDRESS>wrote:

@colinxfleming https://github.com/colinxfleming I'll try to take a
crack at a PR in the next couple of days to add a complexity validation for
this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/colinxfleming/dcaf_case_management/issues/389#issuecomment-228566126,
or mute the thread
https://github.com/notifications/unsubscribe/ADsA9BYXrwez-za7QWst33y6T1raM_b0ks5qPX_pgaJpZM4I6PP8
.

@colinxfleming just checking, is #383 a duplicate?


add X after my name

@howard-ch-wang you have a conflict, ask your TA for help to fix it

@mathurinm The conflicts have been resolved, please review again, thank you!


TSN only uses 1 segment?

Hi, maybe I am missing something, but according to log like the below one, looks like when using TSN 2D model, only 1 test_segment, i.e. 1 frame is used? I thought it is supposed to average results from input frames, say 8 frames?
https://file.lzhu.me/projects/tsm/models/log/testlog_uniform_TSM_kinetics_RGB_resnet50_avg_segment5_e50.log
Thanks!

Hi, sorry for the confusion. The num_segments parameter is used only when you enable TSM or Non-local in this codebase. For TSN, the parameter is not considered. But during testing, we feed 8 frames into the TSN model.

Got it, thanks Ji!


Add append log flag to forever so you can stop/start the forever server

Without the append flag (-a), foreverjs would error out, stating that the log file for node-squarespace-server already existed. THis appends the log, allowing the server to start correctly.

Looks good. Tested and works. Thanks dude!


add support for redux-devtools-extension,

a very useful extension for Chrome, FireFox, and Electron

Codecov Report

Merging #92 into master will not impact coverage.

@@           Coverage Diff           @@
##           master      #92   +/-   ##
=======================================
  Coverage   77.23%   77.23%
=======================================
  Files         112      112
  Lines        2719     2719
  Branches      278      278
=======================================
  Hits         2100     2100
  Misses        413      413
  Partials      206      206

Impacted Files
Coverage Δ

src/store/configureStore.js
76.92% <100%> (ø)
:white_check_mark:

Continue to review full report at Codecov.

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

Awesome, thanks!


Issue/1318 update dependencies

Fixes #1318
Guava and httpclient are part of spring boot. Should we make another PR for spring boot changes?

Codecov Report

Merging #1327 (72e8430) into master (0a21a91) will increase coverage by 13.03%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master    #1327       +/-   ##
=============================================
+ Coverage     75.05%   88.09%   +13.03%
=============================================
  Files           167       32      -135
  Lines          2682      378     -2304
  Branches        492       55      -437
=============================================
- Hits           2013      333     -1680
+ Misses          410       32      -378
+ Partials        259       13      -246

Impacted Files
Coverage Δ

.../main/kotlin/org/zalando/zally/core/RuleDetails.kt

...in/org/zalando/zally/core/ContextRulesValidator.kt

...o/zally/ruleset/zalando/SnakeCaseInPropNameRule.kt

...ally/exception/InaccessibleResourceUrlException.kt

...rg/zalando/zally/apireview/ServerMessageService.kt

cli/zally/utils/formatters/pretty_colorizer.go

...ally/ruleset/zalando/LimitNumberOfResourcesRule.kt

cli/zally/utils/request_builder.go

...eset/zalando/QueryParameterCollectionFormatRule.kt

cli/zally/domain/rules.go

... and 125 more

Continue to review full report at Codecov.

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

In favor of #1343.


Verify node report page

Depends on agent.  See if we can integrate the logs/events/metrics/etc. calls from the PolyResource class, or use the appropriate SavedSearches with filters.

Assigning to @jxstanford re: comment above.
client ticket is #236.

closing due to agent redesign and ui redesign goals.


--penalty-weight for convert_pinhole_model

Hi... I have a feature request ("urghh" I hear you say)... but I hope this is the simplest of tweaks:
Would it be possible to add the option --penalty-weight (from cam2rpc) to convert_pinhole_model, when converting to an RPC distortion camera.
I find this option very useful in cam2rpc, but convert_pinhole_model seems too sensitive to high frequency variations at times.
Cheers, J

A small additional point...
Given the distortion varies more for a given number of pixels at the edges of an opticalbar camera, a uniform --sample-spacing might not capture well the variations at the edges of the image(?). Perhaps this explains why pinholeRPC doesn't well approximate opticalbar at the edges (as seen using mapprojected images).
Would it be possible to densify the sampling at the the left/right edges of the camera, using less points across the middle of the camera?
Cheers, J
(P.S. I know you guys won't have time to look at this, but I put the points out there for when you do, and also in the hope these are very simple things to adjust in the source code(?).)


Fix distribution of demo discipline incidents has_exact_time

Who is this PR for?
Devs working with the demo data.
What problem does this PR fix?
Demo data doesn't match production!
# production:

DisciplineIncident.count
=> 16884

DisciplineIncident.where(has_exact_time: true).count
=> 15450

DisciplineIncident.where(has_exact_time: false).count
=> 1434

Looks like the demo data generator has it backwards, instead of ~90% of discipline incidents not having an exact time, ~90% do have an exact time in the database.
What does this PR do?
Make the demo data generator match what we see in production, 91.6% of discipline incidents have an exact time.

oops, thanks for fixing up! 🌮 👍

spec failure is unrelated


Neither success or failure is triggered with authorizeWithCallbackURL

Hi
I have some trouble that authorizeWithCallbackURL never triggers neither success or failure. Instagram is correctly opened, user is authenticated, and it reopens the app correctly. But nothing is run from success or failure (in example below nothing is printed...)
What can be the reason for this behavior?
    oauthswift.authorizeWithCallbackURL(
        NSURL(string: "oauth-swift://oauth-callback/instagram")!,
        scope: "likes+comments", state:"INSTAGRAM",
        success: { credential, response, parameters -> Void in
            print("It works!")
        },
        failure: { error in
            print(error.localizedDescription)
        }
    )

OS targeted (with version): IOS 9.2
OAuth provider: Instagram
OAuthSwift version: 0.5.1

My mistake - missing the AppDelegate code for correct IOS version.


gdal packages added

testing on gdal packages

The GPU problem has not been fully resolved, will continue to work on it tomorrow.


It's possible to have a user online even if he isn't

I got into a situation when running the app locally, that even when I closed all my web browsers and restarted Meteor that a user was still listed as online.
The UserSessions collection also had a session in it.

How I caused it is sort of complicated, but it's 100% possible.

Hm... eventually, about 10 or so minutes after the restart, the UserSession document was removed. And the user's status switched to offline.

@elie222, all sessions are removed when a new server starts, if you are using instance-status all sessions related to one instance are removed if server is 60 seconds after lost connection or if server fire event 'exit' this sessions are removed immediately.
https://github.com/Konecty/meteor-multiple-instances-status/blob/master/konecty:multiple-instances-status.js#L7

@elie222 we had an issue fixed by https://github.com/Konecty/meteor-user-presence/pull/31 which prevented connections from being removed if they dropped right after calling meteor login on client side.


Failed Community node update

When navigating to the Community nodes page to check installed modules, I want to update this one from v0.1.0 to 0.2.4, I click update. This n8n gives me a green check mark in the bottom right with:
Package updated
n8n-nodes-splitinbatches-advanced updated to version 0.2.4
However, when I hit refresh on the page, I am still on the old version...
I've deactivated the workflow where I am using this node, but even then, I have this update loop bug... n8n is on the latest version

you would probably need to reboot the instance for the update to take affect.

to be clear, this has nothing to do with the node it self but how n8n updates them.


[SPARK-24706][SQL] ByteType and ShortType support pushdown to parquet

What changes were proposed in this pull request?
ByteType and ShortType support pushdown to parquet data source.
Benchmark result.
How was this patch tested?
unit tests

Test build #92503 has started for PR 21682 at commit e9d5625.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/603/
Test PASSed.

Merged build finished. Test PASSed.

Test build #92503 has finished for PR 21682 at commit e9d5625.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92503/
Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/612/
Test PASSed.

Test build #92519 has started for PR 21682 at commit e9d5625.

@HyukjinKwon btw, why we don't support pushdown for these types? Any historical reason?

cc @gengliangwang

@maropu We accidentally dropped the pushdown of these data types when we refactored the file formats. You can check the change history and find the PR.

@gatorsmile aha, thanks.

re: https://github.com/apache/spark/pull/21682#issuecomment-401676889 - I couldn't recall related things about it. Thanks @gatorsmile. So, is it a regression? shall we update the migration guide to note that the performance effect?

It is a regression that was introduced in Spark 1.2. Almost 4 years ago. https://issues.apache.org/jira/browse/SPARK-4453
Thus, I think no need to document it. The pushdown can be treated as new features. : )

Wow .. so it was 4 years ago .. okay.

+1
I agree with some of the minor refactoring suggestions, but overall this looks correct to me.

Test build #92589 has started for PR 21682 at commit b16a607.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/654/
Test PASSed.

Test build #92589 has finished for PR 21682 at commit b16a607.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92589/
Test PASSed.

ping @wangyum

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/797/
Test PASSed.

Test build #92781 has started for PR 21682 at commit 3e66530.

Test build #92781 has finished for PR 21682 at commit 3e66530.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

thanks, merging to master!


security and search improvements

store secrets in Mac OS keychain instead of plaintext file
improve search by allowing partial matches for space separated input in username and service name so you can search for partial service followed then by partial username
add build_deps.sh to install declared python dependencies into ./vendor directory
add local build.sh to package files and deps into alfredworkflow locally
update github action to use local build.sh

@chkpwd I know this may be a alot of changes and fine if you don't want to go in this direction. I just wanted to offer the contributions I added to get the workflow where I wanted.

This looks sick! I'll do some manual testing for Keychain. Will report back my findings and finally merge, assuming all looks good.
Thanks again for improving the project @duckworth


Example format_custom_text does not update eggs

Description
In the project file examples.py, in function format_custom_text, it seems that the code is changing the variable "eggs", with "format_custom_text.update_mapping(eggs=i*2)", but the value displayed in the progress bar never changes from the original value, "3".
Thank you for a very nice package!
FormatCustomText looked like just what I needed for my application, but the output wasn't showing changes to my variables.  After struggling for a few days to figure out what was wrong with my code, I went back and ran the example again and noticed it has the same problem.
I worked around this problem by basing my code on examples.py: user_variables instead.
Code
python3 ./examples.py format_custom_text
Versions

Python version:

$ python3
Python 3.5.2 (default, Apr 16 2020, 17:47:17)
[GCC 5.4.0 20160609] on linux
Type "help", "copyright", "credits" or "license" for more information.

import sys; print(sys.version)
3.5.2 (default, Apr 16 2020, 17:47:17)
[GCC 5.4.0 20160609]

Python distribution/environment:

I don't know, but I found this online:
CPython: this is the default, and what you should be using if you have no idea.
So I think I am using CPython.

Operating System:

Linux Mint 18.3 Sylvia, Cinnamon 64 bit
$ uname -a
Linux rayna 4.15.0-96-generic #97~16.04.1-Ubuntu SMP Wed Apr 1 03:03:31 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Package version:

3.51.0

It seems that a recent update broke this: https://github.com/WoLpH/python-progressbar/issues/221
It's not that it doesn't work at all but the deepcopy makes you lose the reference to the correct formatter...

Release 3.51.1 has a fix :)

I see that 3.51.1 does indeed fix it.  Thanks!


What is the right and 100% working configuration under Ubuntu 16.04LTS?

Hi,
anyone can tell me the right, TESTED and 100%  WORKING configuration under Ubuntu for deep3d?
This config is work???
Ubuntu 16.04LTS -64bit
Cuda 8
DNN 5.1
Nvidia 1080-8gb
mxnet from git
deep3d from git
Thank You So So much and Merry Christmas!!!
Best
Moty

Do you mean working values in your config.mk? I'm running on Ubuntu 16.04, Cuda 8.0, CUDNN 5.1, with an Nvidia GTX 1080.
Did you try following the deep3d readme? If so, what went wrong (error output would be great). It says:
Please open mxnet/config.mk and set USE_CUDA and USE_CUDNN to 1. Then, append EXTRA_OPERATORS=path/to/deep3d/operators to path/to/mxnet/config.mk and recompile MXNet.

To build MXNet with CUDA and CUDNN support, you also need to copy make/config.mk to the mxnet base directory, and set USE_CUDA_PATH to your CUDA installation path (like /usr/local/cuda), and make sure you've put the CUDNN header in /usr/local/cuda/include and libraries in /usr/local/cuda/lib64.

Hi, thanks I know the installation guide... but I read some issue what cuda and DNN combination has a great quality for deep3d.  And on other forum has report problem with 7.5- 5.0 combination quality and bug for gtx 1070 and 1080 and python compatibility problem under cuda 8, other user says the great combination is cuda7 and dnn4  under 14.04 so... now then Im a little bit confuse. So You say under 16.04 cuda8 and dnn5.1 work fine with deep 3d?

 Can You tell me how can I test the deep3d for example how can I convert to 3d the demo.jpg?


Fixed some minor spelling and formatting issues.

@badlogic

Can you consolidate these into one commit please.

@Tom-Ski - done!

Why is it now my commit?

You touched it first!
Spelling fixes look good, not sure why some versions changed.

@Tom-Ski - I think I squashed my commits on top of your commit. :stuck_out_tongue_closed_eyes:

@NathanSweet - the versions changed are the ones from Tom's commit: https://github.com/libgdx/libgdx/commit/da807c12ca6ebef76c53cd582a2212c03018063e

Is this PR good to go? Or do you want me to remake it?

I find it very weird that you can commit in someone else's name :dancers:


Move the yaml separator inside the condition

The chomp "{{-" removes newline also - so we end up with a resource yaml which looks like:
---kind: Service
apiVersion: v1
...

This causes errors in ArgoCD: "error unmarshaling JSON: while decoding JSON: Object 'Kind' is missing in ..."

[ ] Chart version not bumped (the versions are all bumped and released at the same time)
[ ] README.md updated with any new values or changes
[ ] Updated template tests in ${CHART}/tests/*.py
[ ] Updated integration tests in ${CHART}/examples/*/test/goss.yaml

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

ok to test


Add support for Swift Package Manager

Swift Package Manager is integrated into Xcode 11, and it would be nice for me to move to that from cocoapods. Are there plans to add support? I think a valid Package.swift is all that's needed

That should indeed be easy. Do you want to file a PR?

I actually looked into this, and it's harder than you would expect, since this project uses resources, and you have to specifically support that. Especially xib files. So I'm guess support isn't going to come soon.

Right, I did not think of that. I think SPM now supports resources as of Swift 5.3, though: https://useyourloaf.com/blog/add-resources-to-swift-packages/. Haven't looked into it myself, though.

Yes, but it requires you to change your bundle loading to use Bundle.main which will break cocoa pods

On Apr 15, 2021, at 3:06 AM, Daniel Alm @.***> wrote:

Right, I did not think of that. I think SPM now supports resources as of Swift 5.3, though: https://useyourloaf.com/blog/add-resources-to-swift-packages/. Haven't looked into it myself, though.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I guess we could make that a compile-time or runtime option, though. Or just use Bundle(forClass:).


[Feature Request] Warn when multiple route names are the same

Is your feature request related to a problem? Please describe your use case.
Just spent a good amount of time trying to figure why injections were not working on certain routes, after digging the code a bit more, it seemed like when the route names are the same, the injection will overwrite the value in the SignatureRegistry
Describe the solution you'd like
Something like this, showing which route names are already in the registry and maybe also mentioning which route have injections (although that last one would be for debugging purposes, maybe have a toggle?):
[2022-08-08 21:17:46 -0400] [80724] [DEBUG] Dispatching signal: server.init.after
[2022-08-08 21:23:19 -0400] [81202] [DEBUG] Injecting into route 'manager.cases.get_cases' {'user': (<class 'src.utils.User'>, <Constructor(func=create)>)}
[2022-08-08 21:23:19 -0400] [81202] [WARNING] Following route has already been injected: 'ioc_manager.templates.get_rule_filters'
[2022-08-08 21:23:19 -0400] [81202] [WARNING] Following route has already been injected: 'ioc_manager.cases.get_case_iocs'
[2022-08-08 21:23:19 -0400] [81202] [WARNING] Following route has already been injected: 'ioc_manager.cases.get_case_iocs'
[2022-08-08 21:17:46 -0400] [80724] [INFO] Starting worker [80724]

For the warning, I've just added a simple check here:
https://github.com/sanic-org/sanic-ext/blob/b65cd9f28c8d1e6ee0aad91685bfb84fc4925ac6/sanic_ext/extensions/injection/registry.py#L54-L59
With
    def register(
        self,
        route_name: str,
        injections: Dict[str, Tuple[Type, Optional[Callable[..., Any]]]],
    ) -> None:
        if route_name in self._registry:
            logger.warning(f"Following route has already been injected: '{route_name}'")
        self._registry[route_name] = injections

Not sure if this should be a part of sanic_ext or sanic

Another way would be to list all the routes and their route names, and then warning on the duplicate ones (I would personally love this since you sometimes don't know which routes/handlers actually get registered when you have a big codebase with nested blueprints) - unless that's already a feature

This is a known thing. We will probably start with a warning and eventually raise an error for duplicate named routes. It will probably be a part of sanic-routing in finalize.


Missing assets for release v0.140.0

Release v0.140.0 is missing the pre compiled binaries other releases have.
I would like to have them, to be able to download the binary instead of compiling it myself.

duplicate of https://github.com/roboll/helmfile/issues/1928


Custom naming policy

Jira no- CM-786

@TomBak5 - Please ensure that the PR only contains code for the article. This PR contains code from multiple folders and projects. The PR should contain only commits related to the article (i.e. only code in the
json-csharp/CustomNamingPolicy folder)
Also, please note from the author guidelines that the PR should contain the Jira in the title:

So this PR should be named something like: CM-786: Custom Naming Policy


function name casing

I have noticed two different casing styles for functions, both are recommended
Example:
function SomeController() { }
function someFactory() { }
In the .Net world, method names use the pascal casing, which means, the first one would be a recommended srtle.  Could you please tell me, which casing style is recommended for javascript and why do we have 2 different styles here.
Thanks
Sam

The first one is likely because controllers are instantiated, they behave just like classes, while services are not.

JavaScript is more like the Java world where only classes have PascalCasing as they are instantiated. All the rest uses camelCasing. I know it strange at first, I also came from .net before.

Thank you gustavohenke and dietergeerts for letting me know why the difference exists.

controllers are instantiated [...] while services are not

Just a little correction:
There is this notorious provider vs factory vs service ambiguity in Angular. The function passed to .factory() is not instantiated (it is "invoked"), whereas the functions passed to .provider()* and .service() are in fact instantiated. (All the above will result in a Service being registered.)
That's why you might see the following casing:
angular.module('...').
  provider('Foo', FooProviderFn).    // Uppercase, because `FooProviderFn` is instantiated
  provider('Bar', barProviderObj).   // Lowercase, because `barProviderObj` is an object
  service('Baz', BazServiceFn).      // Uppercase, because `BazServiceFn` is instantiated
  factory('Qux', quxFactoryFn);      // Lowercase, because `quxFactoryFn` is invoked

--
*: Note that the provider can also be an object with a $get() function property.

Wow. Thank you @gkalpak for teaching me something today. Didn't know I was able to not define $get() in a provider.

@gkalpak  +1 for the detailed explanation

Just to be clear, $get() must always exist; but the provider can be either an object or . constructor function (creating a provider instance with a $get() method):
// Provider as an object
.provider('Foo', {
  ...
  $get: function () {...}
});

// Provider as a constructor
.provider('Bar', function () {
  ...
  this.$get = function () {...};
});

i thnk we're set here now, right?

closing this for now. Thanks for the answers


The JSONForm contains an element whose type is unknown: "tabs"

Adding the exact text that works in your JSON Form playground (Display - Navigation tabs) to my own instance returns the error, "The JSONForm contains an element whose type is unknown: "tabs"" even though tabs should be known.  I also do not see the "tabs" shown in the playground in any of the documentation.  Is that example.   Is this functionality not supported or is the structure wrong in the playground?
settings.html.zip

Hi
Do you miss a dependency maybe ?

Closing as no activity and no answer for OP


TextField text overflow behaves wildly in FireFox

Description

Added a TextField usingTextField field = new TextField("test");
add(field);

used jetty:run
wrote "this is fine" times 3 in the text field

Expected outcome
Text field with "this is fine this is fine this is fine".
Actual outcome

Browsers Affected

[ ] Chrome (72: no)
[x] Firefox (tested on 65.0.1)
[ ] Safari
[ ] Edge (44.17763.1.0/18.17763: no)
[ ] IE 11
[ ] iOS Safari
[ ] Android Chrome

Present in both V13.0.0.beta1 and V13.0.0.beta2 - both version use text-field 2.3.2.

I have the same problem and i found how to resolve it with 2 ways:

Add placeholder for text field;
Add background for "value" with css in theme modules (dom-module) e.g. :

[part~="value"]:empty{
background: white !important;
}

Confirmed and still reproducible with Firefox 70 😱

The problem seems to be a rendering issue and goes away when I comment out mask-image usage:
https://github.com/vaadin/vaadin-text-field/blob/6d5f3742b5402887dc908a32a859d1c1d366d42f/theme/lumo/vaadin-text-field-styles.html#L44
https://github.com/vaadin/vaadin-text-field/blob/6d5f3742b5402887dc908a32a859d1c1d366d42f/theme/lumo/vaadin-text-field-styles.html#L63
@jouni do you have an idea how to workaround that?

Sigh, I was expecting that this is most probably caused by the mask-image (even before reading Serhii’s comment).
The mask-image seems to be causing more trouble than what it’s worth. I’m considering if we should just drop it.
Though, I thought it should be disabled when the field is focused? It doesn’t look like it is in the screen recording.
https://github.com/vaadin/vaadin-text-field/blob/6d5f3742b5402887dc908a32a859d1c1d366d42f/theme/lumo/vaadin-text-field-styles.html#L47-L52

Gotcha. Seems like this selector should be split in 2 parts, or else it does not apply 😮

Is this fix already released? I am experiencing the problem with Vaadin Flow 14.0.15 as well as 14.1.1.

This is released and should be included to 14.1.1 but I will double check.

This is released and should be included to 14.1.1 but I will double check.

I'm sorry. This is working in 14.1.2 for me.


Add -u git to docs when using docker exec with root installation

This fixes a minor issue in the documentation for SSH Container Passthrough for non-rootless installs. The non-rootless Dockerfile and docker-compose do not set USER/user so docker exec will run as root by default. While running as root, gitea commands will refuse to execute, breaking these approaches. For containers built with the rootless instructions, docker exec will run as git by default so this is not necessary in that case.
This issue was already discussed in #19065, but it does not appear this part of the issue was ever added to the documentation.

isn't that only valid for the rootles containers?

No. The docker exec command is run in the container as the same user that the build the container if no -u argument is provided. The rootless containers are built as git (not root) so they don't default to root, so they don't need this. The root containers are built as root (not git), so it docker exec defaults to root, so they need this.
With root image:
# id command output when starting the container
$ docker run --entrypoint id gitea/gitea:1.21.5
uid=0(root) gid=0(root) groups=0(root),1(bin),2(daemon),3(sys),4(adm),6(disk),10(wheel),11(floppy),20(dialout),26(tape),27(video)
# id command output when execing into an existing container
$ docker exec gitea id
uid=0(root) gid=0(root) groups=0(root),1(bin),2(daemon),3(sys),4(adm),6(disk),10(wheel),11(floppy),20(dialout),26(tape),27(video)

With rootless image:
# id command output when starting the container
$ docker run --entrypoint id gitea/gitea:1.21.5-rootless
uid=1000(git) gid=1000(git) groups=1000(git)
# id command output when execing into an existing container
$ docker exec gitea_rootless id
uid=1000(git) gid=1000(git) groups=1000(git)

The only reason the gitea server even starts with the root configuration is because it is run with su-exec git here. But docker exec does not use the entrypoint script, so it won't do that automatically like it does for docker run.

I was unable to create a backport for 1.22. @scribblemaniac, please send one manually. :tea:
go run ./contrib/backport 29314
...  // fix git conflicts if any
go run ./contrib/backport --continue

I was unable to create a backport for 1.22. @scribblemaniac, please send one manually. 🍵

I'm assuming this is a bug with the bot. It appears to have been backported to 1.22 just fine in #30259.

@scribblemaniac yup, all is well, nothing more needed for you to do. Thanks for the PR :)


IOS control center issue

I am using ionic to build a media player app. Once I added an Interstital banner, it will show up a YouTube link in IOS control center. Even I added remote control plugin , it alsway auto override my setting.
Is this issue related to admob plugin ? Please help, thanks.

I don't think it's caused by admob plugin, at least I never receive such report.
It's simple to verify this. just go to your admob portal, change your ad settings, only accept text and picture ad, exclude video. then try again, can you still see the youtube link?
Since your app is a media player app, is there any other reasons?


Bump senzing from 3.2.0 to 3.3.0

See https://github.com/Senzing/senzingapi-runtime/issues/31

Pull requests:

https://github.com/Senzing/docker-web-app-demo/pull/99


I can't modify the function to return anything other than collection...

Challenge Record Collection has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/58.0.3029.110 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

// Setup
var collection = {
    "2548": {
      "album": "Slippery When Wet",
      "artist": "Bon Jovi",
      "tracks": [
        "Let It Rock",
        "You Give Love a Bad Name"
      ]
    },
    "2468": {
      "album": "1999",
      "artist": "Prince",
      "tracks": [
        "1999",
        "Little Red Corvette"
      ]
    },
    "1245": {
      "artist": "Robert Palmer",
      "tracks": [ ]
    },
    "5439": {
      "album": "ABBA Gold"
    }
};
// Keep a copy of the collection for tests
var collectionCopy = JSON.parse(JSON.stringify(collection));

// Only change code below this line

function updateRecords(id, prop, value) {
  //if exists function --
  console.log("falsy");
  return false;
}

// Alter values below to test your code
updateRecords(5439, "artist", "ABBA");

Hi there,
Could you please describe what you are having trouble with?  The function updateRecords is returning false in your code snippet, and you will need to alter it to return the right type.  Could you post a little bit more on this?

Sorry, yes, I altered the function to try to have it return false ( or anything) but in the console log it always just returned the entire "collection" array. I eventually passed the exercise, but it wasn't like other exercises in that the output in the console didn't do what I told it to... other than returning the collection array. It was a different behavior than other exercises and was difficult because debugging my code was a bit harder because I couldn't output just a single thing to the console to make sure that line of code was working. Does this explain it better?.

Could you copy and past the code that you were expecting to pass but having trouble with?  Maybe there's something we can explain better here in the instructions.

Could someone tell me how to resolve this issue? I just wrote
return id;
and thats it in my updateRecords function and all it keeps returning is the collections object.

I didn't get any resolution other than making the function to modify the
collection and solving the challenge. It always returns the collection for
some reason, but it does show modifications to it.
On Thu, Jun 8, 2017 at 6:57 PM Vedant Kashyap<EMAIL_ADDRESS>wrote:

Could someone tell me how to resolve this issue? I just wrote
return id;
and thats it in my updateRecords function and all it keeps returning is
the collections object.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/freeCodeCamp/freeCodeCamp/issues/15153#issuecomment-307146222,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYFARrd-MDAMwcrbLk9zMAgmG3cwnyLfks5sCBnOgaJpZM4NsoLF
.

--
Will Waymel
+357 99 019520 (Cyprus)
+720 515 7105 (US)
LinkedIn http://www.linkedin.com/in/steven-william-waymel

Ummm according to me the function will always return collection...
The main aim of the challenge is to modify the copy of the Collection and then equate it back to the collections object...
You can solve the challenge that way and the changes will be reflected on the collections object too... It works for me.

Have the same issue.
Change 'return collection;' for 'return "I am a duck";' and see the 'problem'.
So far in all the FCC challenges, the "return" in the main function shows the return value in the console. I'm used to modify the return value in examples to debug, so I always play around with it.
This challenge is different however as it always returns the collection regardless of what you put in it. It's as if the return inside updateRecords is completely ignored for some reason.
@vedant15188 you are right about the goal of this challenge, however it is inconsistent with the way all the previous challenges showed return value in the console.
For the sake of consistency, I hope this will be looked into. Thanks!


Sleeping Mat - server functionality

Hi Lothrazar - Is it out of current spec for the sleeping mat to work on servers?
I didnt see anything to this effect in your mod description but I have been using mats as starting gear in SP, but once I put up a server instance neither myself nor the other player on the server can make it work (clicking with it in hand does absolutely nothing)
I'm running version Cyclic-1.11.2-1.5.7.jar.  If it helps I can get you FML logs or modlists etc. Just let me know.

A modlist could help but it might not matter.  I know for sure that back in 1.10 i played on a server and it worked alongside normal beds, and also it worked with morpheus.
I have not yet played multiplayer in 1.11.2 , what should happen is that 2 players sleeping in whatever thing (bed/mat) pass the full night, ill have to investigate.

I just tested this on my 1.11.2 server. forge <IP_ADDRESS>9.  I passed the night ok... but i was the only person on the server.
I know its been a few days but modlist would help
also when you say "does absolutely nothing" what does that mean. like literally nothing? and does it make a difference if 1 or more than one person online?

Hey...  thanks for the follow-up! When I right click in hand, literally nothing happens,
Check this out
You can find the mods I use on my pack here

Update (did you see my Discord messages?)
It wasn't you!  Apparently Mcjty did something with his worldgen in his "LostCities" mod that hosed sleeping bag items.  I vcaught this because he fixed this yesterday and posted an update and included a changelog item that mentioned this.
Good arguement foir detailed change notes!
As I said in Discord - sorry to wate any time spent here on your part.


How does the Config file work?

I have downloaded the latest dev. I when i try to start the bot i get a message:
usage: pokecli.py [-h] [-cf CONFIG] [-a AUTH_SERVICE] [-u USERNAME]
[-ws WEBSOCKET.SERVER_URL]
[-wss WEBSOCKET.START_EMBEDDED_SERVER]
[-wsr WEBSOCKET.REMOTE_CONTROL] [-p PASSWORD] [-l LOCATION]
[-lc LOCATION_CACHE] [--forts.spin FORTS.SPIN] [-w WALK]
[-k GMAPKEY] [-d DEBUG] [-t TEST] [-du DISTANCE_UNIT]
[-ec EVOLVE_CAPTURED] [-rt RECONNECTING_TIMEOUT]
[-hr HEALTH_RECORD] [-ac FORTS.AVOID_CIRCLES]
[-mcs FORTS.MAX_CIRCLE_SIZE]
[--catch_randomize_reticle_factor CATCH_RANDOMIZE_RETICLE_FACTOR]
[--catch_randomize_spin_factor CATCH_RANDOMIZE_SPIN_FACTOR]
[--map_object_cache_time MAP_OBJECT_CACHE_TIME]
pokecli.py: error:
"max_steps" was removed from the configuration options.
You can now change the behavior of the bot by modifying the "tasks" key.
Read https://github.com/PokemonGoF/PokemonGo-Bot/wiki/Configuration-files#configuring-tasks for more information.
This is my config file:
"tasks": [
  {
    "type": "HandleSoftBan"
  },
  {
    "type": "CollectLevelUpReward"
  },
  {
    "type": "IncubateEggs",
    "config": {
      "longer_eggs_first": true
    }
  },
  {
    "type": "TransferPokemon"
  },
  {
    "type": "EvolvePokemon",
    "config": {
        "evolve_all": "true",
        "first_evolve_by": "cp",
        "evolve_above_cp": 500,
        "evolve_above_iv": 0.8,
        "logic": "or",
        "evolve_speed": 20,
        "use_lucky_egg": true
    }
  },
  {
    "type": "RecycleItems",
    "config": {
      "item_filter": {
        "Pokeball":       { "keep" : 100 },
        "Potion":         { "keep" : 10 },
        "Super Potion":   { "keep" : 20 },
        "Hyper Potion":   { "keep" : 30 },
        "Revive":         { "keep" : 30 },
        "Razz Berry":     { "keep" : 100 }
      }
    }
  },
  {
    "type": "CatchVisiblePokemon"
  },
  {
    "type": "CatchLuredPokemon"
  },
  {
    "type": "SpinFort"
  },
  {
    "type": "FollowCluster",
    "config": {
        "radius": 50,
        "lured": true
    }
  }
],
"max_steps": 5,
"forts": {
  "avoid_circles": true,
  "max_circle_size": 50
},
"websocket_server": false,
"walk": 4.16,
"action_wait_min": 1,
"action_wait_max": 4,
"debug": false,
"test": false,
"health_record": true,
"location_cache": true,
"distance_unit": "km",
"reconnecting_timeout": 15,
"evolve_captured": "NONE",
"catch_randomize_reticle_factor": 1.0,
"catch_randomize_spin_factor": 1.0,
"catch": {
  "any": {"catch_above_cp": 0, "catch_above_iv": 0, "logic": "or"},
  "// Example of always catching Rattata:": {},
  "// Rattata": { "always_catch" : true }
},
"release": {
  "any": {"release_below_cp": 0, "release_below_iv": 0, "logic": "or"},
  "// Example of always releasing Rattata:": {},
  "// Rattata": {"always_release": true},
  "// Example of keeping 3 stronger (based on CP) Pidgey:": {},
  "// Pidgey": {"keep_best_cp": 3},
  "// Example of keeping 2 stronger (based on IV) Zubat:": {},
  "// Zubat": {"keep_best_iv": 2},
  "// Also, it is working with any": {},
  "// any": {"keep_best_iv": 3},
  "// Example of keeping the 2 strongest (based on CP) and 3 best (based on IV) Zubat:": {},
  "// Zubat": {"keep_best_cp": 2, "keep_best_iv": 3}
},
"vips" : {
     "Any pokemon put here directly force to use Berry & Best Ball to capture, to secure the capture rate!": {},
    "any": {"catch_above_cp": 1200, "catch_above_iv": 0.9, "logic": "or" },
    "Lapras": {},
    "Moltres": {},
    "Zapdos": {},
    "Articuno": {},

    "// S-Tier pokemons (if pokemon can be evolved into tier, list the representative)": {},
    "Mewtwo": {},
    "Dragonite": {},
    "Snorlax": {},
    "// Mew evolves to Mewtwo": {},
    "Mew": {},
    "Arcanine": {},
    "Vaporeon": {},
    "Gyarados": {},
    "Exeggutor": {},
    "Muk": {},
    "Weezing": {},
    "Flareon": {}

}

}

"max_steps": 5, <----------- Delete this line
"forts": {
"avoid_circles": true,
"max_circle_size": 50
},

Looks like you're using an old config setup. It would probably be easiest to edit the new config.json.example to reflect what you'd like and replace your current one.

Lol i didn't even saw that line. Its was a new config file it was called: config.json.cluster.example.


Issue with modulefile setting the MAALI_LOADED_PRGENV, COMPILER and COMPILER_VER variables

(Tested on chaos-int with modules/<IP_ADDRESS>): The modulefile created by maali/1.4.3 is setting the MAALI_LOADED_PRGENV, COMPILER and COMPILER_VER with set command which is not successfully setting the variables outside the scope of brace brackets. Using setenv instead to set the value of the variables does propagate it outside the if condition scope.

This needs to be moved to the maali project.


IE8 support

IE8 reached EOL: https://www.microsoft.com/en-US/WindowsForBusiness/End-of-IE-support - So we could close this issue?

The reason I opened this issue is because in my previous job (a huge e-commerce) we had to support IE8.
Closing it.


Angular 16 compatibility

Since Angular 16, View Engine libraries are incompatible, will you release a new version compatible with the Ivy engine?

@dfabregaswefox it will take a bit longer, since most of the libraries used in the project are outdated so I'm updating them.

Thank you for taking care of this

@dfabregaswefox version 6 with support of angular 16 has been released.


Python3.3

i think this change is good for python3 user :)

helper_functions.py
  -from urlparse import urljoin
  +from urllib.parse import urljoin

user.py
  -import urllib
  +import urllib.request, urllib.parse, urllib.error

Those changes breaks everything on Python 2.7. I couldn't merge this pull request

ok maybe work as a branch just for pepole how want use for python3.3

@garni-kh Agree. I created the python_3 branch. You could make a pull request into it. Then i'll change the README.md to let people know what to do if they want to use python 3 with this project! Thanks for contributing!)


Hierarchical Forecasting :  Middle-out approach

This approach is still missing. Generate new columns with MO prefix based on base forecasts for all hierarchy levels.

https://github.com/antoinecarme/pyaf/commit/0bedd00df33a749f9dd2c1aa8cbed889ea5548af

Committed middle-out approaches.
https://github.com/antoinecarme/pyaf/commit/03a72b442e3fc0a610911dceae175a6ad16dc474
Added options for controlling the combination methods.
Added hierarchies tests with various combination approaches (BU, TD, …
Added grouped time series tests with various combination approaches (……BU, TD, MO, OC)

The middle-out approach is now supported.
Closing this issue.


Update documentation for new adaptive timestepping arguments

Following #698 we need to update the documentation of the stepsize;
https://festim.readthedocs.io/en/latest/userguide/stepsize.html#stepsize
@KulaginVladimir do you want to be assigned to this?

Yep, I don't mind.
If #734 is planned to be merged soon, I can also add documentation on CustomSolver


Switch the overcloud to use the admin network

We are waiting on https://review.openstack.org/#/c/572370/ to land

We could also remove the IP allocations from network-allocations.yml

I've retagged the images with:
docker image ls | awk '{if($1 ~ /^<IP_ADDRESS>:5000/){test=$1;gsub("<IP_ADDRESS>:5000","<IP_ADDRESS>:5000",test);print $1 ":" $2, test ":" $2}}' | while read line ; do docker tag $line ; done

Some of the intermediate layers have the <none> tag, the above one liner created the equivalent with a latest tag (not sure if you can tag something with <none>). E.g:
<IP_ADDRESS>:5000/eod/centos-binary-base                            latest              03f9abe52515        2 months ago        403MB
<IP_ADDRESS>:5000/eod/centos-binary-base                          <none>              03f9abe52515        2 months ago        403MB

Not sure if this is going to be an issue.

The retagged images will need to be pushed to the registry so that they can be pulled down by the nodes. Normally we do this from the seed.

Just realized that I set the docker tags to have the IP of the controller. The seed doesn't yet have an IP allocated for the devops network. Can we manually add an ip allocation to network_allocations.yml?

Yes you can add the node's devops IP, <IP_ADDRESS>.

The docker systemd config is configured by kolla-ansible bootstrap-servers command, in the baremetal role. This is run as part of kayobe seed|overcloud host configure.

defunct project, closing.


The sizes attribute only if srcset is also present

When I test HTML markup by https://validator.w3.org I got this error:
Error: The sizes attribute may be specified only if the srcset attribute is also present.

More info here:
https://rocketvalidator.com/html-validation/the-sizes-attribute-may-be-specified-only-if-the-srcset-attribute-is-also-present
Changes:

attribute srcSet renamed to srcset
render attribute sizes only if srcset is also present

Hi @adam-riha, thanks for submitting a PR and sorry I missed it! Version 1.0.6 has both issue you are raising fixed.


Migration misses release notes

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

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

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

[ ] Yes (provide example URL)
[ ] No
[ ] Not relevant

Log gist:

Description
When migration a repo from Github or even from another Gitea instance (e.g. as mirror), while tags are coming along, release notes are lost (and probably attachments made to them as well – I did not test that as I have no attachments to any of my releases). Experienced this way with multiple repos:

Github to Codeberg (Migration)
Codeberg to my local Gitea (Mirror)
Github to my local Gitea (Mirror)

Note that Codeberg currently runs Gitea 1.9.0 as well. At least in this constellation (Gitea 1.9.0 → Gitea 1.9.0) I had expected everything to come along.
If this is the expected behavior, please count this as a feature request. If not, it's a bug report. Not the highest prio in either case, but not "cosmetical" either (so at least medium :wink:).

We support migrating releases from github only. You have to fill github personal token on username and then you will find release/issues/prs options to select.

I did that when migrating from Github to Codeberg – but still had to fill the release notes again as they were lost. To clarify: Tags came along and were shown in the "releases" tab. Just the notes (which in Gitea you create via the "New Release" button) were lost. Everything else came along fine.

Ah, wait, ignore that: They came along from Github to Codeberg (with the exception of "Drafts without tags", which I already filed as separate issue), just not from Codeberg to my local Gitea. Which would make this issue here a feature request to have this functionality also available when migrating between Gitea instances.
Apologies for my confusion!

As the stale bot chimed in: Shall I file this as a separate feature request then (to include all features of Github-to-Gitea migration also with Gitea-to-Gitea migration)?

@IzzySoft I think you can rename this issue's title or close this one and open a new one.

Thanks @lunny – closing this then in favor of #8380.


Changelog for bower 1.7.0

We'll also need to add "Update bower-config" and copy-paste changes from there

@sheerun Add the following?

Properly restore env variables if they are undefined at the beginning
Properly handle default setting for config.ca
Display proper error if .bowerrc is a directory instead of file

This were changes in previous version of bower-config. We need to release new bower-config and update bower to use it.

Right. Then it should be Allow the use of environment variables in .bowerrc. Can you confirm @sheerun There was no commit referring to the bump to 1.2.3

Yes

And other things we did in bower-config since 1.2.3 release

Closing as this was on the master

Hm?

Reopened the PR here


why not support 307 post REDIRECT

[ ] Feature Request. Start by telling us what problem you’re trying to solve. Often a solution
already exists! Don’t send pull requests to implement new features without first getting our
support. Sometimes we leave features out on purpose to keep the project small.

why not support 307 REDIRECT
case HTTP_TEMP_REDIRECT:
// "If the 307 or 308 status code is received in response to a request other than GET
// or HEAD, the user agent MUST NOT automatically redirect the request"
if (!method.equals("GET") && !method.equals("HEAD")) {
return null;
}
Doing so can do
case HTTP_TEMP_REDIRECT:
// Does the client allow redirects?
if (!client.followRedirects()) return null;
    String location = userResponse.header("Location");
    if (location == null) return null;
    HttpUrl url = userRequest.url().resolve(location);

    // Don't follow redirects to unsupported protocols.
    if (url == null) return null;

    // If configured, don't follow redirects between SSL and non-SSL.
    boolean sameScheme = url.scheme().equals(userRequest.url().scheme());
    if (!sameScheme && !client.followSslRedirects()) return null;

    // Redirects don't include a request body.
    Request.Builder requestBuilder = userRequest.newBuilder();
    if (HttpMethod.permitsRequestBody(method)) {
      if (HttpMethod.redirectsToGet(method)) {
        requestBuilder.method("GET", null);
      } else {
        requestBuilder.method(method, null);
      }
      requestBuilder.removeHeader("Transfer-Encoding");
      requestBuilder.removeHeader("Content-Length");
      requestBuilder.removeHeader("Content-Type");
    }

    // When redirecting across hosts, drop all authentication headers. This
    // is potentially annoying to the application layer since they have no
    // way to retain them.
    if (!sameConnection(url)) {
      requestBuilder.removeHeader("Authorization");
    }

    return requestBuilder.url(url).build();

The current behavior (and comment in the code) seems at odds with RFC 7231 section 6.4.7:

The 307 (Temporary Redirect) status code indicates that ... the user agent MUST NOT change the request method if it performs an automatic redirection to that URI. The user agent MAY use the Location field value for automatic redirection.

This is our current workaround (not great):
try (Response response = client.newCall(request).execute()) {
    if ((response.code() == 307) || (response.code() == 308)) {
        String location = response.header(LOCATION);
        if (location != null) {
            request = request.newBuilder().url(location).build();
            return execute(codec, client, request);
        }
    }

    // handle response ...
}

Where did you add this workaround @electrum? Thanks!

This is our current workaround (not great):
try (Response response = client.newCall(request).execute()) {
    if ((response.code() == 307) || (response.code() == 308)) {
        String location = response.header(LOCATION);
        if (location != null) {
            request = request.newBuilder().url(location).build();
            return execute(codec, client, request);
        }
    }

    return handleResponse(response);
}

Just wondering whats the essence of setFollowSslRedirects() and  setFollowRedirects() which are actually enabled by default but don't seem to work.


Issue when trying to run cadrl as policy for human agents

Hi,
I am trying to run experiments where the human agents are running cadrl as policy instead of Orca .
However, I keep running into issues where the variable self.env is None when trying to update the step of each agent with the trained cadrl policy.
Do you have any clue on how I can solve this issue?
Thank you !

@fb1812 have you figured it out? I don't have a clue since it's been a while since I touched the code. If you share the command you run, I can look into it over the weekend.


安装requirements的时候，一直提示cmake相关的错误

CMake Error: CMAKE_C_COMPILER not set, after EnableLanguage
CMake Error: CMAKE_CXX_COMPILER not set, after EnableLanguage
-- Configuring incomplete, errors occurred!
CMake is in the system path. Version:               cmake version 3.31.1

CMake Error: CMAKE_C_COMPILER not set, after EnableLanguage CMake Error: CMAKE_CXX_COMPILER not set, after EnableLanguage -- Configuring incomplete, errors occurred! CMake is in the system path. Version: cmake version 3.31.1

Try installing C++ Build Tools


feat: support for aarch64 linux

Description
Provides support to install Hurl >= 4.1.0 for ARM64 Linux.
Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas

@EstebanBorai You've done a tremendous job! Thank you!
Could you rebuild the distribution file and push it after the suggestions are accepted?

@EstebanBorai You've done a tremendous job! Thank you!
Could you rebuild the distribution file and push it after the suggestions are accepted?

Hi @tarampampam! Thank you so much! Sure thing, I just pushed build assets.


logging: Stop forcing pre-Python 2.5 legacy config behavior.

The disable_existing_loggers option to the logging.config module
turns on a rather complicated behavior of disabling some, but not all,
loggers that might have been already configured when the call to
logging.config.dictConfig or logging.config.fileConfig is made:

This behaviour is to disable any existing loggers unless they or
their ancestors are explicitly named in the logging configuration.
(https://docs.python.org/3/library/logging.config)

Turns out the only reason this is there is as a compatibility hack to
match the behavior of Python 2.4 and below.  See the thread where the
new behavior was introduced: https://bugs.python.org/issue3136
Just as the author of the new behavior explains in that thread from
2008, the legacy behavior forces all logging configuration to be
awkwardly centralized in one place.  That makes the code harder to
read, and it perennially causes confusion when a perfectly
normal-looking logging.getLogger call at the top level of one module
mysteriously has no effect, while that in another module works fine,
under the influence of the details of what gets imported when.
So, switch to the shiny new behavior of Python 2.5.
This may cause a few of the logfiles in ZULIP_PATHS to become active
that have been dormant for a long time.

This is great, merged, thanks @gnprice!  We should plan to do a checkup on chat.zulip.org after the next time we deploy there, to make sure what is showing up in the various log files makes sense.  I guess I'll just deploy to chat.zulip.org now to make that easy.

Thanks! Yep, a deploy now to check on what starts getting logged is just what I was thinking.


4.14.1 not on the ocaml-ci yet

This breaks RWO attempting to update to 4.14.1 as only 4.14.0 is available.
https://ci.ocamllabs.io/github/realworldocaml/book/commit/3d8469813cf0518d8d3cefc682d629c8040f5adb/variant/debian-11-4.14_opam-2.1

It's fixed - or at least, the 4.14.1 part of it is!

Thanks! Indeed, it's failing "healthily" now ;-)


backdrop not work properly on some tag

I use nebular v3 dashboard template and when i using ngx-tour for my header component, backdrop feature not work for its tag elements.
Icing on the cake (means worst than previous problem) is backdrop style contain all element except header component tags

i just changed the z-index value  and working properly


Make possible to use CMake --target install for library installation

This is already works for libre, but doesn't work for librem currently

Thanks!


The calculation of width parameter?

Hi, I am confuse about this implementation:
def call(self, x):

        C = tf.expand_dims(self.centers, -1)  # inserts a dimension of 1
        H = tf.transpose(C-tf.transpose(x))  # matrix of differences
        return tf.exp(-self.betas * tf.math.reduce_sum(H**2, axis=1))

I learned RBF neural network from this video
There is a width parameter:

and then

I haven't seen the use of width parameter in your code. So, I modefied the code like this
def call(self, x):
        C = K.expand_dims(self.centers)
        XC = K.transpose(K.transpose(x)-C)
        D = K.expand_dims(K.sqrt(K.mean(XC**2,axis=0)),0)
        H = XC/D
        return K.exp(-self.betas * K.sum(H**2, axis=1))

The full code with experiments is here.
My code overfit easily, but not with your code.  Am I wrong?
Thank you for solving my problem~~~ ^_^

Hi, I just use beta insted of width
ie. instead h = exp( - (x-c)^2 / r) , I use exp( - beta * (x-c)^2)
so it should be beta = 1/r

I see now that you modified the code, I will have a close look on that later.
Overfitting may be connected to too small width, with narraw gaussians you overfitt more easily than with wider gaussians.

Hi, I just use beta insted of width
ie. instead h = exp( - (x-c)^2 / r) , I use exp( - beta * (x-c)^2)
so it should be beta = 1/r

@PetraVidnerova  Thanks for your reply~~~ You mean that the width is learned directly!
But look into the theory, the width has its own equation. There are two ways as implemeted in this code using numpy.
So, I wonder that if it is possible to add these two ways in keras implementation or it is unnecessary to add them because of the learning skill by the neural network.

Both widths and centers are tuned during learning in my implementation, since here the whole network is trained using back propagation process (there are many ways of RBF learning).
In the simple example test.py, value 2.0 is provided, that means they are all initialized uniformly to this value.
But you can write you ouwn initializer and initilize the widths as you wish. Just instead beta=2.0 in the constructor, you should use
beta = MyBetaInitializer().
If you do not want them to change during learning, it has to be changed: instead of trainable=True use trainable = False, in build method of RBFLayer.

Both widths and centers are tuned during learning in my implementation, since here the whole network is trained using back propagation process (there are many ways of RBF learning).
In the simple example test.py, value 2.0 is provided, that means they are all initialized uniformly to this value.
But you can write you ouwn initializer and initilize the widths as you wish. Just instead beta=2.0 in the constructor, you should use
beta = MyBetaInitializer().
If you do not want them to change during learning, it has to be changed: instead of trainable=True use trainable = False, in build method of RBFLayer.

I reviewed the theory again. The width equation is derived from center parameter. But in reality,there should be a width coefficient df, like this

So the learning parameter in RBF layer maybe the center and width coefficient.
But the problem is that, in your code the forward inference is :

Refering to the theory, it should be:

In these two equations, they will have different derivative for center parameter:

This difference will lead to different update for center parameter during learning~~~
My opinion is that the parameter settings in your code is still the same as before,  but the inference function should be changed:

That is great comment! You are perfectly right.
Basically, there are several approaches to RBF network training.
The basic one is to 1. set centers, 2. set widths, 3. calculate output weights, one by one.
The training in my code uses backpropagation and tunes all network params simultaneously. Back prop needs random initialization, however center are widths are not initialized randomly but to meaningful values. I use random or k-means init for centers, and for widhts just uniform values.
You approach to widths suggest, that they depend on the values of centers. So the change of centers during learning should also influence the values of widths. If centers were constant, we can hide the term (sqr(1/n sum ...))  in beta, but since we are changing them, we should ad this to the computation of h2.

That is great comment! You are perfectly right.
Basically, there are several approaches to RBF network training.
The basic one is to 1. set centers, 2. set widths, 3. calculate output weights, one by one.
The training in my code uses backpropagation and tunes all network params simultaneously. Back prop needs random initialization, however center are widths are not initialized randomly but to meaningful values. I use random or k-means init for centers, and for widhts just uniform values.
You approach to widths suggest, that they depend on the values of centers. So the change of centers during learning should also influence the values of widths. If centers were constant, we can hide the term (sqr(1/n sum ...)) in beta, but since we are changing them, we should ad this to the computation of h2.

Yes，the K-means is only used to initial the center parameter but changed while training.
And then, if I specify the term (sqr(1/n sum ...)) explicitly, and modefied the code like that

I get strange result:

But your code is right:

I get smaller training loss than your implementation(1.5925e-04 vs 6.8916e-04) ,but get worse prediction results~~~~ T_T
It is really strange. Do I have a wrong modefication or misunderstand the theory?
The attached code is my experiment with your data.  I would appreciate it if you have time to help me with that? Thank you very much.
rbf_for_tf2.zip

Could you try smaller number of hidden units?

Could you try smaller number of hidden units?

Firstly, I thought it got overfit. So, I changed the hidder units for 10 to 5. But the loss was still small and the result was still bad.

Then I find it doesn't look like that the model gets overfit, because the test data is the same as train data in your code.
I just wonder why the modified code get smaller loss than yours, but get even worse predict result. In my mind, the smaller 'MSE' loss means the model fits the train data better, but the experiment seems not support it.

MSE is just square of difference from training data, so in this simple example it is really strange that you get smaller MSE.
It looks something is going wrong. I will have to take a closer look on that. Thanks for you effort.

And we should put some points apart to test set , since in this simple example only trainset is used and it is not correct approach.

In my first code, I made two simple dataset;
The training set is:
def test_data2(sample_number = 1000):
    all_data = np.random.rand(sample_number*2,2)
    data1 = all_data[all_data[...,0]>all_data[...,1]]
    data2 = all_data[all_data[...,0]<=all_data[...,1]]
    y1 = np.zeros((sample_number,1))#class0
    y2 = np.ones((sample_number,1))#class 1

    train_data = np.vstack((data1,data2))
    train_label = np.vstack((y1,y2))

    shuffle_idx = np.arange(sample_number*2)
    np.random.shuffle(shuffle_idx)

    train_data = train_data[shuffle_idx]
    train_label = train_label[shuffle_idx]
    return train_data,train_label

the test set is:
## create dataset
def test_data1(sample_number = 1000):
    mean0=[2,7]
    cov=np.mat([[1,0],[0,2]])
    data1=np.random.multivariate_normal(mean0,cov,sample_number)

    mean1=[8,3]
    cov=np.mat([[1,0],[0,2]])
    data2=np.random.multivariate_normal(mean1,cov,sample_number)

    y1 = np.zeros((sample_number,1)) # class 0
    y2 = np.ones((sample_number,1)) # class1

    train_data = np.vstack((data1,data2))
    train_label = np.vstack((y1,y2))

    shuffle_idx = np.arange(sample_number*2)
    np.random.shuffle(shuffle_idx)

    train_data = train_data[shuffle_idx]
    train_label = train_label[shuffle_idx]
    return train_data,train_label

The predicted result seems like no problem:

But when I changed the test data to a line:
x1 = np.linspace(-2,12,1000)
x2 = np.linspace(-2,12,1000)
test_x = np.vstack((x1,x2)).T

In this line dataset, your code get reasonable result:


⚠ ClamAV has degraded performance

In 5609cb8, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 78 ms

Resolved: ClamAV performance has improved in 35911c4 after 7 minutes.


Changing the Monomodal Walk

As PO of the OJP
I suggest a change in the request for a monomodal walking leg
to get a much faster TripResponse.
Dani Rubli from Mentz told me to change the monomodal TripRequest as following:

delete the IndiividualTransportOptions in the Origin
add the ojp:ItModesToCoverwalk</ojp:ItModesToCover> in the Params-Section

OJP SDK v1.0
2023-04-01T12:10:42.624Z

2023-04-01T12:10:42.624Z

7.906561
46.602738

46.602738,7.906561

2023-03-30T15:21:00.000Z
--> delete
--> delete                        walk
--> delete                        PT3000M
--> delete

7.907601
46.598295

46.598295,7.907601

true
true
true
true
--> add                    walk


Cli command chain without interaction

Feature Request
Command chains are not possible to run without interaction. Chains are only available with human interaction. If the input is not interactive, it never executes the chain.
This listener is stopping when the input isn't interactive:
vendor/laminas/laminas-cli/src/Listener/TerminateListener.php
public function __invoke(ConsoleTerminateEvent $event): void     {         if ($event->getExitCode() !== 0 || ! $event->getInput()->isInteractive()) { // Here it stops when its not interactive             return;         }

Q
A

New Feature
yes

RFC
no

BC Break
no

Summary
Execute command chains without interaction, maybe with a --force-chain-execution parameter to not change the past behaviour.

@michalbundyra
Can you please have a look at the problem?
Thanks in advance! 👍🏻

@sbo-develop @froschdesign
I was looking into it. Here is the original PR: https://github.com/laminas/laminas-cli/pull/19 with the chain feature.
I do not remember exactly why we did not allow non-interactive mode with chains, but I guess it was due to 'security'.
That's why we added prompts to check if you'd like to run the next command in the chain, and a warning if the command you are about to run is a 3rd part command.
I was checking and we can potentially drop the condition you have noted @sbo-develop, but I believe it would be a BC Break.
The other option we can do, we can provide a new configuration option (which will be default to false) something like allow_non_interactive_chains. But then I started thinking - ok, if we have this maybe it should not be just bool, but you should be able to decide if you want to allow 3rd-party commands as well or not (so: 0 - disabled, 1 - just own + laminas commands, 2 - everything).
The other feature might be to introduce one more configuration - to specify a whitelist of the namespaces we allow commands from.
Also, current behaviour in non-interactive mode is that we break after first command and do not execute any chained commands. If we change it to run automatically all other commands I believe we should have also an option to "skip chain". Let see the example - we have two commands:
1. first-command
2. second-command

the chain is defined that first-command => second-command
so now (when it is not possible to run chains in non-interactive mode) we get:
bin/laminas first-command --no-interaction

will run JUST first-command and it will stop.
If we allow chained runs in non-interactive then both commands will be executed.
So we would need to have an ability to 'revert the future' from the cli, for example:
bin/laminas first-command --no-chain --no-interaction

And it makes me to the point, that maybe allowing chain in non interactive mode should be a separate argument?
So if we want to execute all chained commands in non-interactive mode we would need to do:
bin/laminas first-command --with-chain --no-interaction

or even we can implement it as argument with some options (just local command, also 3rd part or whitelisted ...).
As the feature seems to be quite nice and easy, I believe we should take it all into consideration to make a right decision. And at the end document it well, so it is easy to use.  Now I am on the side that chain should not be called automatically on non-interactive mode, unless user opted-in in the config.
Hope it all makes any sense :)

Very well thought through answer. I‘m with you to have it with opt in and configurable ( former behavior must be kept the same ). An additional parameter with default execute all, except you choose otherwise.

I've forked it ( https://github.com/sbo-develop/laminas-cli ) and just disabled the stop for chains when in interactive mode, works for me. If i get some advice how to add a new parameter the right way you would like it, i could implement these changes and send a pull request.


Responsive field width

Hello Sahil,
Testing your PCF component with different optionsets made be realize it is responsive to values width, not form width.
Then when a selected value has smaller width than other values, then these other values are not fully displayed.
See below :

The selected value "Other Type" is smaller than the other values and we cannot read all values properly.
Is it possible to adapt the component to make it responsive to the form, like a normal optionset would do ?

Thanks for the help.
Sébastien.

I'm not sure why you're able to face this. I've updated the Solution file again, try to import it again. Solutions

Hello Sahil,
I tried with both solutions managed and unmanaged. I tried on another field, on another form. Still get the same display, where the field is reponsive to the value, not the form.
Thanks for the help.
Sébastien.