Can't exit RemotePlayWhatever from within Steam Deck Game Mode

As of the latest update, it's now possible to add the AppImage of RemotePlayWhatever as a non-Steam game, and then launch from Game Mode to create invite/links.
Unfortunately, once a link has been created in the app, it is impossible to shutdown RemotePlayWhatever, and attempting to do so can cause the system to freeze, requiring restart.

Looks like it's just the way steam client in gampadui mode handles non-steam games at the moment. I was able to reproduce similar behavior launching multiple non-steam processes through clients gamepadui library.

The Steam UI "Force Quit" should send a signal to the process to shut it down. Does RPW have signal handlers for linux signals?

Well here's the neat thing, the whole point of RPW is to avoid hooking and use provided client api instead.
I was able to reproduce same behavior with two non-steam games launched simultaneously without RPWs "help", one tracked non-steam process gets stuck no mater what, you can verify with ps that process is indeed closed, but still displayed in running apps list on deck side.

Ah I see what your saying now. I was able to reproduce it also. I think ill put some more focus on getting this working through decky. That way one can avoid having to start a second non steam process.

While working on the RPW Decky plugin I noticed that sending guest invites freezes the UI when trying to close the "hooked" game but the same is not true for inviting friends directly via id. I suggest using the friends ID way of inviting people until this is resolved.

@joamjoamjoam amazing that you are doing this! Just curious if canceling the guest link and then attempting to shutdown helps at all. Personally I love the guest links as I have friends who join with tablets or phones who don't have Steam accounts.  Doesn't seem like we can cancel guest links via RPW CLI at the moment, but curious if canceling via the QAM UI and then shutting down does anything.

I did try canceling via the ui and had the same freezing issue. Ill double check tommorow and update  my comment. For friend links via the ui you have to cancel the session first too otherwose it will freeze so its possible the guest links work too.
Another issue we were seeing was when a guest link was active trying to access your friends would lock up the deck too. The steam client really didnt like the guest link for some reason.


Backend publish index table sorting not reacting

Affected Symphony version(s) : 2.7.7
PHP version(s) : 7
OS(es) : Ubuntu / Chrome, Firefox
Note
I think this is an older issue, so it should be present also on older Symphony versions like 2.6 and maybe even older / PHP 5.x .
Issue
When toggling sort order of a publish index table by clicking on a column's name, e.g. basic text field column type (asc desc toggling or changing column etc.), it takes two or three clicks to activate the sort, the first clicks doing like nothing.

Expected
Sort order should be activated on first click.

I get this too.

Would you have op_cache enabled by any chance ?

Would you have op_cache enabled by any chance?

Yes. Yay, good old op_cache again?

Would you have op_cache enabled by any chance ?

Yes.  Is that good or bad for Symphony, or should it not matter?

In case it helps, sorting columns in the Authors page works for me every time and the browser address bar changes with parameters, e.g. from /symphony/system/authors/ to /symphony/system/authors/?sort=last_seen&order=asc.  The Publish pages do not add parameters to the address bar when eventually sorted on any given column, however I can see that the underlying link does have parameters, e.g.

	Title

Let me explain OPcache a little bit. Generally, it's a very good thing, because it keeps the compiled version of PHP files in a cache — so it not only saves time reading the file next time, but compiling the file as well. The only thing needed for this to work is to check the original file's modification time every time the compiled version is needed. And that is a very, very "cheap" process compared to reading and compiling it. So PHP runs faster.
Unfortunately, somebody thought that it should even be better than that. A file's modification time shouldn't be checked on each and every request, but there should be a few seconds between the mtime checks at least. The opcache.revalidate_freq was implemented, with a default value of 2 (seconds).
Now this means that even if a PHP file changes, it may take up to 2 seconds for this to take effect (depending on when it was requested for the last time). For high-traffic (!) systems this may be a good thing and further improve performance, because it saves a lot of modification time checks in the filesystem.
But it comes at a price: When the PHP app writes PHP files itself (like Symphony writing to config.php, for example), these changes may not be reflected immediately (unless the app would drop the OPcache). This can be very confusing for the user.
For Symphony (unless you have really high traffic) I suggest to set opcache.revalidate_freq to 0. (If you are on shared hosting, you may ask your provider how to do it.) This will not mean that the cache does not work, it just means that the modification time is checked for each file request. (In my eyes this would have been a much better default value, because it causes less headache. The "super pro" users will touch those settings anyway.)
Regarding your issue: Without having tested anything (so from the top of my head), I am rather sure that the effect is caused by OPcache.

Hmm, I wonder why the sorting of Publish pages is being stored in config.php.  Does that mean if Author Alice sorts a Publish page on another column, Author Bob would see that Publish page sorted on that column the next time he loads the page?  If so, would be better all round if sorting preferences were stored in browser cookies?

Does that mean if Author Alice sorts a Publish page on another column, Author Bob would see that Publish page sorted on that column the next time he loads the page?

Yes.

If so, would it be better all round if sorting preferences were stored in browser cookies?

I'd prefer to see it stored in the user's session, or in the database, or — best IMHO — not stored at all. (Pagination could still retain the sorting by passing a request parameter between pages, as long as you don't leave the index page). But "permanent sorting" has always irritated me. Anyway, changing the behaviour is out of scope for Symphony 2.7. — no features, fixes only!
Maybe @nitriques finds the time to tell us how this is supposed to work in Symphony 3 or later. (I admit that I have not watched/followed the development closely.)

I'd prefer to see it stored in the user's session, or in the database, or — best IMHO — not stored at all.

I'd not thought of that. I agree.

@michael-e Thanks for the explanation, very nice!
We will remove it from the config that's for sure! But we would need to be able to configure a default sort, and then use a author-based one.

So, is this not being treated as a bug, but as a PHP configuration issue?

For Symphony (unless you have really high traffic) I suggest to set opcache.revalidate_freq to 0.

@michael-e I did this and the behaviour has been mitigated as you suggested. Also, maybe the Order Entries extension will now work for me.
I wondered however if opcache.blacklist_filename could be used instead to simply isolate config.php but whatever I tried with that did not work.

Well, that might work as well, I never tried. But I don't think that you will see any measurable advantage (compared to revalidating).

Thanks @michael-e for the excellent explanation before. I became a friend with opcache after that, I mean it. Besides that, @cylkee , I recently just tried to turn off opcache within my latest websites updates, and those sites performs with no noticeable setback.
I asume, it further depends on traffic volume etc., but I wanted to say, that I find it as a perfect option also to just turn the opcache off, as a trait if you will for having an easygoing backend.


tabAutocomplete doesn't work correct since 0.9.224 / 0.8.56

Before submitting your bug report

[X] I believe this is a bug. I'll try to join the Continue Discord for questions
[X] I'm not able to find an open issue that reports the same bug
[X] I've seen the troubleshooting guide on the Continue Docs

Relevant environment info
- OS: MacOS 15.0.1
- Continue version: 0.9.224+
- IDE version: VSCode
- Model: deepseeker-code-v2
- config.json:

  "tabAutocompleteModel": {
    "title": "DeepSeeker Coder v2 16b",
    "provider": "ollama",
    "model": "deepseek-coder-v2:16b-lite-instruct-q6_K",
    "apiBase": "https://OPENWEBUI/ollama",
    "apiKey": "open-webui-token",
    "contextLength": 8096,
    "completionOptions": {
      "temperature": 0.1,
      "topP": 0.95,
      "maxTokens": 600
    }
  },

Description
I have a setup of autocompletitions with open-webui involved. Open-webui serves as a proxy to Ollama. It was working previously fine, but with latest versions of continue.dev plugin instead of autocomplete i get explanations of what my code does.
I checked previous versions and seems like problem arrive starting from 0.9.224 pre-release and 0.8.56 versions. Previous versions works as expected.
I believe it is a bug. But maybe you changed something in configs i need change too?
Thanks.
To reproduce
No response
Log output
No response

@vills my guess is that this is caused by using the "ollama" provider for a non-Ollama API. The thing we did specifically lately was to let Ollama handle FIM formatting for us, but when going through OpenWebUI my guess is that the FIM endpoint doesn't exist or is routed to chat.
For my own understanding (you're not the only one to have this setup, and I haven't looked into it yet): what is the benefit of routing through OpenWebUI instead of directly to Ollama? If I had to point you to a solution, it would definitely be to just go direct, but I figure there's a reason you can't.

Thanks for sharing your idea. I'll check it in a few days.
I use ollama behind open-webui as it provides me with (obviously) UI and authorization/authentication (i use remote server with GPU that i share with my friends). Plus it can route to different payed platforms. It's kind of very simple to setup and get this features.

I am also experiencing the same issue, since 0.8.56 the tab autocompletions show a description of the code before the cursor rather than a code suggestion, where prior versions work as expected. I am connecting directly to an ollama server running locally.

Also hitting this issue. The big benefit of Open WebUI in this mix is that it handles authentication with integration with SSO providers and has API keys per user. In our setup that enable all developers in the company to use Open WebUI as the publicly available LLM proxy, reachable from their laptops, while Ollama itself is not reachable from the Internet.

I can confirm accessing remote Ollama instance works well. Looks like open-webui's proxy do something with with original requests. I'll try to dig deeper..

@vills you may want to take a look at the CompletionProvider code, specifically the provideInlineCompletionItems function. streamCompletionWithFilters might also be of interest.

I made some tests and responses from direct Ollama connection and Open WebUI proxied are differ - https://github.com/open-webui/open-webui/discussions/6720#discussioncomment-11335817.
I think Open WebUI do not proxies "suffix" parameter to Ollama and that's the issue with autocomplete.

@sestinj one more question. I see in codebase you detect "use FIM or not" by searching for ".Suffix" parameter in Ollama's model template. Is there a manual way to revert this behavior while bug isn't fixed in Open WebUI?

FYI, fixed in open-webui v0.4.3 release.

Great, thank you for taking care of this @vills!
Does this mean this issue can be closed now?


 Liveness and readiness are using /opt/liveness.sh, but readiness should be using /opt/readiness.sh.
Also, it looks like we only support liveness and readiness right now, we should add startup also.
here's an example of what I believe PD's lifecycle probes should look like:
  probes:
    liveness:
      command: /opt/liveness.sh
      periodSeconds: 10
      timeoutSeconds: 5
      successThreshold: 1
      failureThreshold: 3
    readiness:
      command: /opt/readiness.sh
      initialDelaySeconds: 120
      periodSeconds: 10
      timeoutSeconds: 5
      successThreshold: 2
      failureThreshold: 2
    startupProbe:
      command: /opt/liveness.sh
      failureThreshold: 90
      periodSeconds: 10

Adding startupProblem as well as re-organizing how the probes are defined, allowing the deployer to use standard k8s probe definitions out of the box.

Moving the probes section under global.container
Changing names: (liveness --> livenessProbe, readiness --> readinessProbe)
Adding startupProbe

The new default looks like:
    ############################################################
    # Probes
    #
    # Probes have a number of fields that you can use to more precisely control the
    # behavior of liveness and readiness checks.
    #
    # https://kubernetes.io/docs/tasks/configure-pod-container/configure-liveness-readiness-startup-probes/
    ############################################################
    probes:
      livenessProbe:
        exec:
          command:
            - /opt/liveness.sh
        initialDelaySeconds: 30
        periodSeconds: 30
        timeoutSeconds: 5
        successThreshold: 1
        failureThreshold: 4
      readinessProbe:
        exec:
          command:
            - /opt/readiness.sh
        initialDelaySeconds: 30
        periodSeconds: 5
        timeoutSeconds: 5
        successThreshold: 1
        failureThreshold: 4
      startupProbe:
        exec:
          command:
            - /opt/liveness.sh
        periodSeconds: 10
        failureThreshold: 90

Note: this is a breaking change if anyone has overriding probes in their own values file.  The fix is simply move their definition of their probes to live under global.container or the (productName).container, as well as adding "Probe" to the definition.
Note: Eventually, it is recommended to create an /opt/startup.sh (or some similar name) in the images, so that can be the standard script called instead of /opt/liveness.sh.  Initially, it can be similar to the standard /opt/readiness.sh, which simply calls liveness.sh.


Callback not working with new standard WKWebView, fallback to UIWebView required

https://github.com/chariotsolutions/phonegap-nfc/blob/fdba79f489f195c0ff8d8b4e98b2ecb8cb97e8c4/src/ios/NfcPlugin.m#L118
There is code missing (e.g. 'evaluateJavaScript') to make it work with the new standard WKWebView.
Currently the only solution is to use UIWebView as a fallback by putting
<preference name="CordovaWebViewEngine" value="CDVUIWebViewEngine" />
into the apps config.xml.

Xcode output after scanning a tag:
2017-10-30 09:44:42.260887+0100 MyApp[3369:2035346] -[WKWebView stringByEvaluatingJavaScriptFromString:]: unrecognized selector sent to instance 0x102031a00 2017-10-30 09:44:42.261806+0100 MyApp[3369:2035552] didInvalidateWithError Single tag read (null) 2017-10-30 09:44:42.265144+0100 MyApp[3369:2035346] *** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '-[WKWebView stringByEvaluatingJavaScriptFromString:]: unrecognized selector sent to instance 0x102031a00' *** First throw call stack: (0x186a9bd38 0x185fb0528 0x186aa91f8 0x190263cc4 0x186aa16e4 0x1869870dc 0x100c756b8 0x10108149c 0x10108145c 0x101086050 0x186a43f20 0x186a41afc 0x1869622d8 0x1887f3f84 0x18ff0f880 0x100c74204 0x18648656c) libc++abi.dylib: terminating with uncaught exception of type NSException

https://github.com/chariotsolutions/phonegap-nfc/pull/288/files


Cluster: Remove cluster node removal code and update messaging

Pull Request (PR) description
This is just one option to resolve #257 .
Remove code that removes a cluster node that is in the Down state from the cluster.
This Pull Request (PR) fixes the following issues

Fixes #257

Task list

[x] Added an entry to the change log under the Unreleased section of the file CHANGELOG.md.
Entry should say what was changed and how that affects users (if applicable), and
reference the issue being resolved (if applicable).
[ ] Resource documentation added/updated in README.md.
[ ] Resource parameter descriptions added/updated in README.md, schema.mof and comment-based
help.
[ ] Comment-based help added/updated.
[x] Localization strings added/updated in all localization files as appropriate.
[ ] Examples appropriately added/updated.
[ ] Unit tests added/updated. See DSC Community Testing Guidelines.
[ ] Integration tests added/updated (where possible). See DSC Community Testing Guidelines.
[x] New/changed code adheres to DSC Community Style Guidelines.

This change is

Closing this as I think this will be resolved in PR #272 instead. @nickgw reopen if it was wrong of me to close this.


Add koonwen to github-organizations

Thank you for your interest in OCaml-CI. Please follow the instructions from https://ocaml.ci.dev/getting-started to get ocaml-ci installed into your GitHub user. Please let us know if you have any problems

Thanks!


Host header include %3a80

Hi all,
My context:

kubernetes 1.2.2 (gke)
latest docker image
ingress like that

nothing too exotic :)
But when I was doing a curl, it always return a 301 (very frustrating).
After some debug, I find that træfik was using a bad Host header like that
Host:<IP_ADDRESS>%3a80
User-Agent:curl/7.26.0
[...]

And apache didn't like it (and he is right for one time...).
It seems that there is a problem on how is performed Host on backend :disappointed:

Very simple debug:
$ curl -IL http://<IP_ADDRESS>
HTTP/1.1 200 OK
Date: Wed, 27 Apr 2016 16:30:47 GMT
Server: Apache/2.4.10 (Debian) PHP/5.6.20
X-Powered-By: PHP/5.6.20
Link: <http://foo.bar.com/wp-json/>; rel="https://api.w.org/"
Content-Type: text/html; charset=UTF-8

$ curl -IL http://<IP_ADDRESS> -H"host:foo.bar.com"
HTTP/1.1 200 OK
Date: Wed, 27 Apr 2016 16:31:10 GMT
Server: Apache/2.4.10 (Debian) PHP/5.6.20
X-Powered-By: PHP/5.6.20
Link: <http://foo.bar.com/wp-json/>; rel="https://api.w.org/"
Content-Type: text/html; charset=UTF-8

$ curl -IL http://<IP_ADDRESS> -H"host:foo.bar.com:80"
HTTP/1.1 301 Moved Permanently
Date: Wed, 27 Apr 2016 16:31:17 GMT
Server: Apache/2.4.10 (Debian) PHP/5.6.20
X-Powered-By: PHP/5.6.20
Location: http://foo.bar.com/
Content-Type: text/html; charset=UTF-8

seems to be related to https://github.com/golang/go/issues/4172

Whoah, there as URL encoding issue somewhere...
I can't reproduce it.
Could you give us DEBUG traefik entire logs?

2016-04-27T19:17:06.734955487Z 2016/04/27 19:17:06 connection accepted 1
2016-04-27T19:17:06.735763593Z 2016/04/27 19:17:06 <->
2016-04-27T19:17:06.735768613Z 00000000  47 45 54 20 2f 20 48 54  54 50 2f 31 2e 31 0d 0a  |GET / HTTP/1.1..|
2016-04-27T19:17:06.735773192Z 00000010  48 6f 73 74 3a 20 31 30  2e 33 2e 31 38 35 2e 38  |Host: <IP_ADDRESS>|
2016-04-27T19:17:06.735777693Z 00000020  30 3a 38 38 30 30 0d 0a  55 73 65 72 2d 41 67 65  |0:8800..User-Age|
2016-04-27T19:17:06.735782088Z 00000030  6e 74 3a 20 4d 6f 7a 69  6c 6c 61 2f 35 2e 30 20  |nt: Mozilla/5.0 |
2016-04-27T19:17:06.735786516Z 00000040  28 4d 61 63 69 6e 74 6f  73 68 3b 20 49 6e 74 65  |(Macintosh; Inte|
2016-04-27T19:17:06.735791004Z 00000050  6c 20 4d 61 63 20 4f 53  20 58 20 31 30 5f 31 31  |l Mac OS X 10_11|
2016-04-27T19:17:06.735795556Z 00000060  29 20 41 70 70 6c 65 57  65 62 4b 69 74 2f 36 30  |) AppleWebKit/60|
2016-04-27T19:17:06.735800036Z 00000070  31 2e 31 2e 35 36 20 28  4b 48 54 4d 4c 2c 20 6c  |1.1.56 (KHTML, l|
2016-04-27T19:17:06.735804847Z 00000080  69 6b 65 20 47 65 63 6b  6f 29 20 56 65 72 73 69  |ike Gecko) Versi|
2016-04-27T19:17:06.735809408Z 00000090  6f 6e 2f 39 2e 30 20 53  61 66 61 72 69 2f 36 30  |on/9.0 Safari/60|
2016-04-27T19:17:06.735813710Z 000000a0  31 2e 31 2e 35 36 0d 0a  41 63 63 65 70 74 3a 20  |1.1.56..Accept: |
2016-04-27T19:17:06.735818100Z 000000b0  74 65 78 74 2f 68 74 6d  6c 2c 61 70 70 6c 69 63  |text/html,applic|
2016-04-27T19:17:06.735822520Z 000000c0  61 74 69 6f 6e 2f 78 68  74 6d 6c 2b 78 6d 6c 2c  |ation/xhtml+xml,|
2016-04-27T19:17:06.735827030Z 000000d0  61 70 70 6c 69 63 61 74  69 6f 6e 2f 78 6d 6c 3b  |application/xml;|
2016-04-27T19:17:06.735831465Z 000000e0  71 3d 30 2e 39 2c 2a 2f  2a 3b 71 3d 30 2e 38 0d  |q=0.9,*/*;q=0.8.|
2016-04-27T19:17:06.735835818Z 000000f0  0a 41 63 63 65 70 74 2d  45 6e 63 6f 64 69 6e 67  |.Accept-Encoding|
2016-04-27T19:17:06.735840269Z 00000100  3a 20 67 7a 69 70 2c 20  64 65 66 6c 61 74 65 0d  |: gzip, deflate.|
2016-04-27T19:17:06.735844705Z 00000110  0a 41 63 63 65 70 74 2d  4c 61 6e 67 75 61 67 65  |.Accept-Language|
2016-04-27T19:17:06.735849258Z 00000120  3a 20 66 72 2d 66 72 0d  0a 58 2d 46 6f 72 77 61  |: fr-fr..X-Forwa|
2016-04-27T19:17:06.735853640Z 00000130  72 64 65 64 2d 46 6f 72  3a 20 37 38 2e 32 31 32  |rded-For: 78.212|
2016-04-27T19:17:06.735858029Z 00000140  2e 31 36 36 2e 32 35 0d  0a 58 2d 46 6f 72 77 61  |.166.25..X-Forwa|
2016-04-27T19:17:06.735862378Z 00000150  72 64 65 64 2d 48 6f 73  74 3a 20 31 30 2e 33 2e  |rded-Host: 10.3.|
2016-04-27T19:17:06.735866751Z 00000160  31 38 35 2e 38 30 3a 38  38 30 30 0d 0a 58 2d 46  |185.80:8800..X-F|
2016-04-27T19:17:06.735871292Z 00000170  6f 72 77 61 72 64 65 64  2d 50 72 6f 74 6f 3a 20  |orwarded-Proto: |
2016-04-27T19:17:06.735875690Z 00000180  68 74 74 70 0d 0a 58 2d  46 6f 72 77 61 72 64 65  |http..X-Forwarde|
2016-04-27T19:17:06.735880697Z 00000190  64 2d 53 65 72 76 65 72  3a 20 74 72 61 65 66 69  |d-Server: traefi|
2016-04-27T19:17:06.735889176Z 000001a0  6b 2d 69 6e 67 72 65 73  73 2d 63 6f 6e 74 72 6f  |k-ingress-contro|
2016-04-27T19:17:06.735893852Z 000001b0  6c 6c 65 72 2d 37 33 6d  75 73 0d 0a 0d 0a        |ller-73mus....|
2016-04-27T19:17:06.759486575Z 2016/04/27 19:17:06 handle connection error, err=EOF

for me it's ok ( kubernetes 1.2.2 (  barrebone ) )

Me too, I suspect something is sending url encoded host header to traefik.

more information for debug:
{
  "kubernetes": {
    "backends": {
      "foo.bar.com": {
        "servers": {
          "56c6cca4-0c8b-11e6-9bcb-42010af000e8": {
            "url": "http://<IP_ADDRESS>:80",
            "weight": 1
          }
        },
        "loadBalancer": {
          "method": "wrr"
        }
      }
    },
    "frontends": {
      "foo.bar.com": {
        "entryPoints": [
          "http"
        ],
        "backend": "foo.bar.com",
        "routes": {
          "foo.bar.com": {
            "rule": "Host:foo.bar.com"
          }
        }
      }
    }
  }
}

{
  "pid": 1,
  "uptime": "41m11.964036873s",
  "uptime_sec": 2471.964036873,
  "time": "2016-04-28 10:12:46.366580851 +0000 UTC",
  "unixtime":<PHONE_NUMBER>,
  "status_code_count": {},
  "total_status_code_count": {
    "301": 28,
    "404": 3
  },
  "count": 0,
  "total_count": 31,
  "total_response_time": "1.41761211s",
  "total_response_time_sec": 1.41761211,
  "average_response_time": "45.729422ms",
  "average_response_time_sec": 0.045729422000000006
}

time="2016-04-28T10:06:22Z" level=debug msg="Received event from kubenetes map[type:ADDED object:map[kind:Ingress apiVersion:extensions/v1beta1 metadata:map[namespace:default selfLink:/apis/extensions/v1beta1/namespaces/default/ingresses/wordpress uid:dbc3e4ff-0d28-11e6-9bcb-42010af000e8 resourceVersion:199144 generation:1 creationTimestamp:2016-04-28T10:06:21Z name:wordpress] spec:map[rules:[map[host:foo.bar.com http:map[paths:[map[backend:map[serviceName:wpfrontend servicePort:80]]]]]]] status:map[loadBalancer:map[]]]]"
time="2016-04-28T10:06:22Z" level=debug msg="Configuration received from provider kubernetes: {\"backends\":{\"foo.bar.com\":{\"servers\":{\"56c6cca4-0c8b-11e6-9bcb-42010af000e8\":{\"url\":\"http://<IP_ADDRESS>:80\",\"weight\":1}}}},\"frontends\":{\"foo.bar.com\":{\"backend\":\"foo.bar.com\",\"routes\":{\"foo.bar.com\":{\"rule\":\"Host:foo.bar.com\"}}}}}"
time="2016-04-28T10:06:22Z" level=debug msg="Last kubernetes config received less than 2s, waiting..."
time="2016-04-28T10:06:24Z" level=debug msg="Waited for kubernetes config, OK"
time="2016-04-28T10:06:24Z" level=debug msg="Creating frontend foo.bar.com"
time="2016-04-28T10:06:24Z" level=debug msg="Wiring frontend foo.bar.com to entryPoint http"
time="2016-04-28T10:06:24Z" level=debug msg="Creating route foo.bar.com Host:foo.bar.com"
time="2016-04-28T10:06:24Z" level=debug msg="Creating backend foo.bar.com"
time="2016-04-28T10:06:24Z" level=debug msg="Creating load-balancer wrr"
time="2016-04-28T10:06:24Z" level=debug msg="Creating server 56c6cca4-0c8b-11e6-9bcb-42010af000e8 at http://<IP_ADDRESS>:80 with weight 1"
time="2016-04-28T10:06:24Z" level=info msg="Server configuration reloaded on :80"
time="2016-04-28T10:06:25Z" level=debug msg="Round trip: http://<IP_ADDRESS>:80, code: 301, duration: 70.065096ms"
time="2016-04-28T10:06:30Z" level=debug msg="Round trip: http://<IP_ADDRESS>:80, code: 301, duration: 47.242796ms"
time="2016-04-28T10:06:35Z" level=debug msg="Round trip: http://<IP_ADDRESS>:80, code: 301, duration: 39.294382ms"
time="2016-04-28T10:06:40Z" level=debug msg="Round trip: http://<IP_ADDRESS>:80, code: 301, duration: 45.900569ms"

@guilhem: traefik seems ok here, it seems that the strange header comes from your client. What client is making request to traefik?

@emilevauge a simple curl:  curl -i http://<IP_ADDRESS>:80/ -H"Host:foo.bar.com"
$ curl -v http://<IP_ADDRESS>:80/ -H"Host:foo.bar.com"
* About to connect() to <IP_ADDRESS> port 80 (#0)
*   Trying <IP_ADDRESS>...
* connected
* Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 80 (#0)
> GET / HTTP/1.1
> User-Agent: curl/7.26.0
> Accept: */*
> Host:foo.bar.com
>
* additional stuff not fine transfer.c:1037: 0 0
* HTTP 1.1 or later with persistent connection, pipelining supported
< HTTP/1.1 301 Moved Permanently
< Content-Length: 0
< Content-Type: text/html; charset=UTF-8
< Date: Thu, 28 Apr 2016 10:20:51 GMT
< Location: http://<IP_ADDRESS>/
< Server: Apache/2.4.10 (Debian) PHP/5.6.20
< X-Powered-By: PHP/5.6.20
<
* Connection #0 to host <IP_ADDRESS> left intact
* Closing connection #0

$ curl --version
curl 7.26.0 (x86_64-pc-linux-gnu) libcurl/7.26.0 OpenSSL/1.0.1e zlib/1.2.7 libidn/1.25 libssh2/1.4.2 librtmp/2.3
Protocols: dict file ftp ftps gopher http https imap imaps ldap pop3 pop3s rtmp rtsp scp sftp smtp smtps telnet tftp
Features: Debug GSS-Negotiate IDN IPv6 Largefile NTLM NTLM_WB SSL libz TLS-SRP

on the apache side, here is complete log (with log_forensic module):
+439:5721e497:57|GET / HTTP/1.1|Host:<IP_ADDRESS>%3a80|User-Agent:curl/7.26.0|Accept:*/*|X-Forwarded-For:<IP_ADDRESS>|X-Forwarded-Host:<IP_ADDRESS>%3a80|X-Forwarded-Proto:http|X-Forwarded-Server:gke-test-clients-default-pool-b2614ade-boeo|Accept-Encoding:gzip

@ekozan As I can see on your side there is same problem than me. Sended host is Host: <IP_ADDRESS>:8800 so with host:port.

sadly, I can't set passHostHeader with kubernetes

@guilhem I need to know your architecture:

what client: what makes request to traefik?
what servers behind traefik?

@emilevauge as seen with logs from @ekozan behavior seems to be a "normal" one (træfik act the same way on both cases).

OK, then can you close this issue?

I don't know. Do you want this behavior to happen :)
Clearly it crash apache2 "as it".

it's the normal behavior  https://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html#sec14.23

14.23 Host
The Host request-header field specifies the Internet host and port number of the resource being requested, as obtained from the original URI given by the user or referring resource (generally an HTTP URL,
as described in section 3.2.2). The Host field value MUST represent the naming authority of the origin server or gateway given by the original URL. This allows the origin server or gateway to differentiate between internally-ambiguous URLs, such as the root "/" URL of a server for multiple host names on a single IP address.
   Host = "Host" ":" host [ ":" port ] ; Section 3.2.2

A "host" without any trailing port information implies the default port for the service requested (e.g., "80" for an HTTP URL). For example, a request on the origin server for http://www.w3.org/pub/WWW/ would properly include:
   GET /pub/WWW/ HTTP/1.1
   Host: www.w3.org

A client MUST include a Host header field in all HTTP/1.1 request messages . If the requested URI does not include an Internet host name for the service being requested, then the Host header field MUST be given with an empty value. An HTTP/1.1 proxy MUST ensure that any request message it forwards does contain an appropriate Host header field that identifies the service being requested by the proxy. All Internet-based HTTP/1.1 servers MUST respond with a 400 (Bad Request) status code to any HTTP/1.1 request message which lacks a Host header field.
See sections 5.2 and <IP_ADDRESS> for other requirements relating to Host.

ok so this is not a problem with træfik on the Host header :relieved:
This issue can be closed. thank you
But as I can see it can be another one:
why in my example and one from @ekozan X-Forwarded-Host = Host?
in my case it should be:
Host:<IP_ADDRESS>:80
X-Forwarded-Host:foo.bar.com

instead of
Host:<IP_ADDRESS>:80
X-Forwarded-Host:<IP_ADDRESS>:80

But should be wrong :/

You can set PassHostHeader=true as seen here http://docs.traefik.io/basics/#frontends
There is an issue to make it by default #216

@emilevauge there is no way to set passHostHeader with kubernetes provider.
But having a way to change default value for any parameter is a great thing.
I checked with nginx on front of apache2 and it works fine.
I'm closing this issue and hope it will help people who may have same problem.


Adding an annotation in existing annotation

Hello. I'm trying drawing an annotation in existing annotation using polygon tool. But, it failed.
After dragging anywhere in annotation to start drawing,  if I click another point, existing annotation is selected..
Is there any way to draw polygon in existing annotation?

This works in the standard version. I'm assuming you are using the OpenSeadragon version? I.e. this is a duplicate of this: https://github.com/recogito/annotorious-openseadragon/issues/75
It's fixed already and will be available with the next release.

This works in the standard version. I'm assuming you are using the OpenSeadragon version? I.e. this is a duplicate of this: recogito/annotorious-openseadragon#75
It's fixed already and will be available with the next release.

Oh, sorry. I added selectAnnotation  event handler and handler calls selectAnnotation API depending on specific condition.
So, selection was canceled whenever selectAnnotation API called by clicking a point to draw polygon.

I'm not sure I understand your use case. You listening to the selectAnnotation event? And inside the handler you call anno.selectAnnitation? (To deselect the annotation again under certain conditions?)
Either way, the selectAnnotation event is not fired when you start creating a new annotation, except in the OpenSeadragon version because of the bug linked above. (Which is now fixed.)


[BEAM-10906] Add basic ToRows transform.

This will make it easier to create schema'd PCollections.
This includes improvements to type inference to be able to better
understand the Row type and apply getattr(...).

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @udim

Please add a JIRA to the description

Actually, thinking about this again, I think accepting positional arguments is desirable--this is analogous to a "select" statement and having to write (a="a", b="b", ...) is verbose. (On that note, would this be better named Select? @TheNeuralBit for your thoughts as well.)

I agree we'll want a way to do simple projections with positional args, something like Select('a', 'b', ...), maybe that should just be a separate transform that requires a schema'd input and only accepts positional args? Then we'd avoid the confusion of mixing positional and kwargs.
I think the one argument against that is users might have some structured type that they want to assign a schema to, and then their only option would be ToRow(a='a', b='b', ...). But I'd argue the right solution in that case is broader schema-inference support so they don't even need ToRow.

If we had a separate transform Select, what would be the delta between that and ToRow? (In particular, I think we'd still want to support expressions like Select('a', b=lambda x: x.foo()).

I was thinking Select would only support positional arguments so specifying a function wouldn't be possible. I guess then you would need another utility for that (could be Assign like the pandas operator?). That's a slippery slope though and it wouldn't be a great UX to have N different specialized transforms for effectively the same thing.
I'm +1 on this accepting both positional and kwargs and calling it Select.

Note I do think Assign is still a good idea as a convenient version of Select that keeps existing columns.

I renamed it to Select. (I also thing an add-new-field[s]-keeping-existing-ones operation would be useful too, though Assign doesn't seem to capture what that means. Future work.)

Yeah I guess AddColumns or similar would be much more intuitive. assign is pretty confusing in pandas.


Alias resolution with Style Dictionary

I've noticed your plugin adds the alias for a token as
"$value": "{base.primary.yellow}"

When you try to resolve aliases in style dictionary i'm getting a blank output file
Should the plugin output an alias path the same as the style dictionary example?
"$value": "{base.primary.yellow.value}"

Interesting. Thanks for feedback @cailborg
I'll check it out.
Which version of Style Dictionary do you use?

Just latest at the moment, so 3.8.0

Thank, and for which platform, CSS?
I'll check it in the morning 🙂

Yep CSS and SCSS

Should work fine, without value at the end. I've checked a test JSON file with Style Dictionary playground — here is a link
Looks like the reason in something else.

Also, I made Style Dictionary utils package. So, you can try to transform your tokens with it — https://github.com/tokens-bruecke/sd-utils
Here is a test repo with Github Actions — https://github.com/tokens-bruecke/tokens-deploy-with-actions

I made a custom build. In this build, the plugin will add .value at the end of the alias string.
Can you check if it works for you?
plugin-test-2.zip

Also, might be the reason in this
"$value": "{base.primary.yellow.value}"
that you're using DTCG format $value

Added value or $value (if it's the DTCG format) at the end of the alias paths.

@PavelLaptev wouldn't it be more appropriate for this change to live in the style dictionary utils? The alias example provided by the spec does not append value to the end of an alias. Cobalt's (a DTCG parser) example matches the spec. Style dictionary is the only place I've seen so far where value is appended.

@dev-nicolaos thank you for your feedback. I think you're right. I'll leave adding value at the end of an alias optional.

@dev-nicolaos updated. It's now optional


[Issue #3476] Fix last separator of properties string is missing when using batch send

What is the purpose of the change
This problem introduced since 4.9.1, may cause tag incorrect.
in https://github.com/apache/rocketmq/pull/2961 MessageDecoder.messageProperties2String remove last separator. MessageDecoder.string2messageProperties can parse the string with or without last separator, but CommitLog.MessageExtEncoder.encode(MessageExtBatch, PutMessageContext) force append some properties without separator.
Brief changelog
This problem need fix in broker side. since different client may send messages in different format.
Verifying this change
I will check compatibility of this commit tomorrow.
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage decreased (-0.02%) to 54.54% when pulling c282367a8b948b3275f6d27616520a6878f440cf on areyouok:fix_batch_tag into b35f3e4459e8cf32116bb15133aaa27d5ff63cfb on apache:develop.

I think there is no problem in this commit.


rustc warning: trait objects without an explicit dyn are deprecated

Following warning is shown while using<EMAIL_ADDRESS>is used as a dependency:
warning: trait objects without an explicit `dyn` are deprecated
   --> /home/trivikr/.cargo/registry/src/github.com-1ecc6299db9ec823/either-1.5.3/src/lib.rs:839:32
    |
839 |     fn cause(&self) -> Option<&Error> {
    |                                ^^^^^ help: use `dyn`: `dyn Error`
    |
    = note: #[warn(bare_trait_objects)] on by default

The code which needs to be updated:
https://github.com/bluss/either/blob/cd15a341118b5af765450dcb6380c6dee17ed0f2/src/lib.rs#L839-L841

It needs to be updated to allow this warning, until we change the min Rust version.

Since that lint is also newer than our minimum Rust, you may need to stack it:
#[allow(unknown_lints, bare_trait_objects)]


feat: Create skeleton component

Changes description 🧑💻

Add skeleton components
Context 🤔

#3
Checklist ✅

[x] I have reviewed the submitted code
[x] I have tested on a phone device/emulator
[x] I have tested on a tablet device/emulator
[x] I have tested on a kiosk device/emulator

Screenshots 📸
Phone

Tablet

Kiosk

Other info 👋

You can download the apk here :
sample-debug.apk :
https://app.bitrise.io/app/e9ae480151be4514/build/6916b172-d8f5-4e28-82c1-bf663fc0537b/artifact/6b9f906ea2e36f4b/p/83095b5d8aa3d7d34acdf4939d2b219f

You can download the apk here :
sample-debug.apk :
https://app.bitrise.io/app/e9ae480151be4514/build/45ce3f7d-c1ae-4dfb-b418-0576cf6e21fb/artifact/17a8707b24a67ef9/p/e3dc7e462b86ecf02b17e9c37e83cfa3

✅

✅

You can download the apk here :

You can download the apk here :
sample-debug.apk :
https://app.bitrise.io/app/e9ae480151be4514/build/b919ee01-2742-47cb-80bc-af1c3b3335a7/artifact/1d36710d79265dc1/p/b3e3736f5ac9b76409738b76bd3a9a42

You can download the apk here :

You can download the apk here :

You can download the apk here :

You can download the apk here :

You can download the apk here :

You have a lot of lint errors with your contribution:
> Task :skeletons:ktlintMainSourceSetCheck FAILED
/bitrise/src/skeletons/src/main/java/com/decathlon/vitamin/compose/skeletons/VerticalShimmer.kt:22:1 Needless blank line(s) (no-consecutive-blank-lines)
/bitrise/src/skeletons/src/main/java/com/decathlon/vitamin/compose/skeletons/VitaminSkeletonColors.kt:21:1 Unexpected blank line(s) before "}" (no-blank-line-before-rbrace)
/bitrise/src/skeletons/src/main/java/com/decathlon/vitamin/compose/skeletons/VitaminSkeletonColors.kt:22:1 Needless blank line(s) (no-consecutive-blank-lines)
/bitrise/src/skeletons/src/main/java/com/decathlon/vitamin/compose/skeletons/VitaminSkeletons.kt:8:1 Unused import (no-unused-imports)
/bitrise/src/skeletons/src/main/java/com/decathlon/vitamin/compose/skeletons/VitaminSkeletons.kt:10:1 Unused import (no-unused-imports)
/bitrise/src/skeletons/src/main/java/com/decathlon/vitamin/compose/skeletons/VitaminSkeletons.kt:11:1 Unused import (no-unused-imports)
/bitrise/src/skeletons/src/main/java/com/decathlon/vitamin/compose/skeletons/VitaminSkeletons.kt:14:1 Unused import (no-unused-imports)
/bitrise/src/skeletons/src/main/java/com/decathlon/vitamin/compose/skeletons/VitaminSkeletons.kt:37:1 Needless blank line(s) (no-consecutive-blank-lines)
/bitrise/src/skeletons/src/main/java/com/decathlon/vitamin/compose/skeletons/VitaminSkeletons.kt:50:1 Unexpected blank line(s) before "}" (no-blank-line-before-rbrace)
/bitrise/src/skeletons/src/main/java/com/decathlon/vitamin/compose/skeletons/VitaminSkeletons.kt:69:1 Needless blank line(s) (no-consecutive-blank-lines)

You can download the apk here :

You can download the apk here :
sample-debug.apk :
https://app.bitrise.io/app/e9ae480151be4514/build/ccfc38d2-5f43-44b4-b8dc-65fca27abb96/artifact/e0a37ef4d78ef3f3/p/ed41324ba7fce2fb480f4cd2a8ae92e2

You can download the apk here :
sample-debug.apk :
https://app.bitrise.io/app/e9ae480151be4514/build/04838b06-0069-40b4-9492-6004aa812092/artifact/d149eb9e2a604c29/p/be4018bda39d921ee17c248d1cb07ee3


tweaks to new V1 revision

Allow alternate Mythril API URL base
Add listAnalysis()
Fix example programs
Add simple test that I forgot to add the last time I looked.

Pull Request Test Coverage Report for Build 72

4 of 20 (20.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-12.2%) to 85.057%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

index.js
2
4
50.0%

lib/simpleAuthRequester.js
2
16
12.5%

Totals

Change from base Build 70:
-12.2%

Covered Lines:
110

Relevant Lines:
126

💛  - Coveralls

All of this will get reincarneted and fixed in a future PR.


ci: dependabot can't pass ci

There are some issues that not provides access to dependabot start and finish CI jobs correctly

    if: github.actor!= 'dependabot'   # ignore the pull request which comes from user dependabot.


chore(package): release v3.0.1

coverage: 97.686%. remained the same
when pulling a7dff9c6dc55f6b200550b3b1ec803bf8edd7c8d on release-3.0.1
into 61d1933e5607d55e213c6d16dc69ab5086eab708 on master.


添加腾讯翻译君接口

如题，付费接口添加腾讯翻译君
https://cloud.tencent.com/product/tmt

close with https://github.com/ZGGSONG/STranslate/releases/tag/<IP_ADDRESS>


tac: should fail when stdin is closed but doesn't

The uutils version of tac disagrees with GNU tac in the case where stdin is closed on the command line:
GNU prints an error message to stderr and returns exit status code 1:
$ tac <&-
tac: 'standard input': read error: Bad file descriptor
tac: -: Bad file descriptor

<&- means close stdin.
uutils prints nothing and returns exit status code 0:
$ ./target/debug/coreutils tac <&-

This is causing one of the GNU tests to fail.

Libstd opens stdin, stdout and stderr as /dev/null if they aren't open at program start to prevent std::io::{stdin,stdout,stderr} from pointing to unrelated file resources that are opened later. See https://github.com/rust-lang/rust/blob/17dfae79bbc3dabe1427073086acf7f7bd45148c/library/std/src/sys/unix/mod.rs#L54-L57

The stdlib could open /dev/null with different permissions in that case so that reads/writes still fail, like this: https://github.com/tavianator/bfs/blob/f4772fe4c13eb8b9b3c199d4e1703c242babcfd7/main.c#L88-L114

I came across this because one of the tests from gnu/tests/tail-2/follow-stdin.sh also uses <&- to provoke the bad fd error. (Actually <&- is also used in: gnu/tests/misc/{tac-2-nonseekable, sort-merge-fdlimit, sort-continue, tty}.sh and gnu/tests/cp/cp-mv-backup.sh).
I tried this, but it didn't work and I don't know why:
fn get_errno() -> i32 {
    Error::last_os_error().raw_os_error().unwrap()
}
pub fn stdin_is_bad_fd() -> bool {
    let fd = stdin().as_raw_fd();
    // FIXME: this is never `true`, even with `<&-`
    unsafe { libc::fcntl(fd, libc::F_GETFD) == -1 && get_errno() == libc::EBADF }
}

@jhscheer It's because of what @bjorn3 wrote in https://github.com/uutils/coreutils/issues/2873#issuecomment-1024940029

@jhscheer It's because of what @bjorn3 wrote in #2873 (comment)

Thanks.
There's a discussion about why stdlib is reopening the file descriptors here.
@tavianator did you open an issue with your suggestion at the rust-lang repo?
So I guess, unless stdlib implements a better workaround than the one currently used, it's not likely we will ever pass the GNU tests mentioned above.

Ah so in the linked PR, https://github.com/rust-lang/rust/pull/75295#issuecomment-673178574 suggests using pipes to accomplish something similar to my suggestion in https://github.com/uutils/coreutils/issues/2873#issuecomment-1033288915.  But RFC 1014 explicitly conflicts with that, as pointed out in https://github.com/rust-lang/rust/pull/75295#issuecomment-673900935.  So we can't do that unless we're okay with reverting that RFC I guess.
I didn't open an issue, but there's already one: https://github.com/rust-lang/rust/issues/88825


working with Copilot

The changes are related to the format of the Python file in VS Code.
I think they have nothing to do with Copilot.
There's a possibility that this change was proposed by Copilot.

I close this pull request.
This is because I can't find a reason to merge this settings.json file of VS Code.
In my opinion, it's preferable for this kind of configuration file to have flexibility for each developer.


Externalize geoid url variable

Hello!
Can we consider to externalize the hardcoded parameter 'ARIA_GEOIDS' ?
It will be helpfull in order to change the url without code modification (if the link does not respond for example). We could imaging to place a link to local geoid files for example.
import os
ARIA_GEOIDS = os.environ.get('ARIA_GEOIDS', 'https://aria-geoid.s3.us-west-2.amazonaws.com')
GEOID_PATHS_AGI = {
    'geoid_18': f'{DATA_PATH}/geoid_18.tif',
    'egm_08': f'{ARIA_GEOIDS}/egm08_25.tif',
    'egm_96': f'{DATA_PATH}/egm96_15.gtx',
}

I think this is a good idea - we could add a keyword argument to the main stitcher routine.


Script Comparability with Python2

I understand python 2 support is going to end soon but alot of us are still using it
python covid19.py
Traceback(most recent call last):
  File "covid19.py", line 50, in <module>
    today = datetime.today().replace(hour=23, minute=59, second=59, microsecond=59).replace(tzinfo=GMT).timestamp()
AttributeError: 'datetime.datetime' object has no attribute 'timestamp

not important but i thought i should add it

I dont mind using a different module to allow for cross compatibility, do you know of a module available in Python 2/3 that would allow me to do the same thing?


feat: check pikalytic & showdown season version

[x] Use Github actions to schedule check season

Add feature at https://github.com/louis70109/find-your-pokemon/blob/main/.github/workflows/check-season.yml


Post release automated changes for fabric releases

Post release automated changes for azure-arm-fabric

API change check
API changes are not detected in this pull request.


Ability to control the Locust process' exit code

PR for #587

Looks good! Could you add an example on how to set the return code based on rps or avg response time as well?

I've extended the example to also check for average response time as well as response time percentile.
I'm hesitant on adding an RPS check to the example because I think it's a slightly weird condition to use (since it's very dependent on both the number of simulated users as well the actual code of the User classes). It shouldn't be hard to figure out for anyone who still wants to do it, but I don't think it should be encouraged by an example in the official docs.

And maybe do a log entry in the example as well.

Good idea!

Ok to merge! I still think an RPS check is a good idea, especially for CI purposes. Yes, RPS depends on your test plan, but that is constant over CI runs. There are cases where RPS might drop even though response times do not rise (e.g. delays reading test data, load gen overload etc) which you would definitely want to catch.
But if you are -1 on that, I can just comment it in the ticket (which was about RPS, just like the slack question that made me reopen it :)

Lets worry about (possibly) adding rps thresholds to the documentation later. Merging.


Add option for terse logging.

When using pip-accel for e.g. Travis-CI, the log output is much to long:
2015-10-19 23:50:55 testing-worker-linux-docker-357adccc-3199-linux-7 pip.req.req_set[3303] INFO Requirement already satisfied (use --upgrade to upgrade): wheel in /home/travis/virtualenv/python2.7.9/lib/python2.7/site-packages (from -r tests/requirements.txt (line 9))
2015-10-19 23:50:55 testing-worker-linux-docker-357adccc-3199-linux-7 pip.download[3303] INFO File was already downloaded /home/travis/.cache/pip-accel/sources/check_manifest-0.25-py2.py3-none-any.whl

Neither timestamp nor hostname, not even the process-id is meaning full here. Instead aligning the length of the logger names would ease reading the logs.
Please add an option for setting the log format, or at least for disabling timestamp and hostname. Another option would be to add an option to use the standard logging module instead of colorlogs.
(When thinking about this, the later option would a) quickly be implemented, b) allow configuring the logformat later c) reuse the pip-accel config-file for defining the loggers (see https://docs.python.org/2/howto/logging.html#configuring-logging)

Hi Hartmut and thanks for the feedback!
Your request is reasonable and should be fairly simple to implement, so I'll see if I can add support this week. The current version of coloredlogs doesn't support configurable log formats and this has been the number one complaint since I released that package (see e.g. issue 6 and issue 7). Because of this I started working on adding support for user defined log formats last week — it's almost done. So I guess your timing is impeccable :-).

Hi Hartmut! I've released coloredlogs 3.0 which introduces support for user defined log levels.
If you change your .travis.yml to first install pip-accel, then upgrade to coloredlogs >= 3.0 (despite the huge changes most of coloredlogs has remained backwards compatible) and set the environment variable $COLOREDLOGS_LOG_FORMAT you should already be able to customize the log format. Is there any chance you can test this already? (to make sure the improvements/changes satisfy your requirements)
Regardless though, I'll leave this issue open until I've added a configuration option to pip-accel that allows to customize the log format from pip-accel configuration files.

Thanks. Does exactly what I need :-)

Hi again Hartmut! I've just released pip-accel 0.37 which adds support for thelog-format and log-verbosity  configuration options (the corresponding environment variables are also supported).  I believe the problem you reported is now fully resolved so I'm closing this issue. If you encounter any issues or disagree then feel free to reopen the issue. Thanks again for the feedback!

Thanks.


Accessibility issue on using webchat on mobile app

How to shift between adaptive cards on an Android phone using accessibility gestures. As per the latest 4.12.0 release, we can use up and down arrow key using keyboard but the same thing can't be achieved on using the bot in Android phone using gestures. Is there any alternative/solution for this ?

@tanuj01 are you saying Android gestures are not working on mobile (inside Adaptive Cards) when navigating the transcript?

@corinagum the Android gestures are working in case user wants to traverse from top to bottom of the webchat through each Adaptive card and it's adaptive buttons.
But my ask is, just like we use Up and Down arrow keys on keyboard to traverse directly between Adaptive cards and if user wants to interact with the card then only the user has to press Enter button otherwise using Up and down arrow keys, user can move to next message or Adaptive card.
So, can the same functionality be achieved in mobile through gestures. Currently I have tested all possible gestures but unable to achieve the above mentioned functionality on mobile through gestures.

I can only test on iOS as I don't have Android, but here are my notes: (@compulim can you verify this behavior on Android?)

Assuming the Adaptive Card is attached in the latest activity: when focused in the send box, swiping left multiple times will focus inside the transcript and automatically focus inside the last section of the Adaptive Card. Input items (if applicable) are interactable as normal.
'Press Enter to interact' text should not be spoken on mobile - I have updated #3758 with this extra item
When starting with focus at the top of the activity, mobile behavior is wonky where the activity repeats twice before the user can focus INSIDE of the Adaptive Card. Repro steps:

Send 'card calendar', and focus on this activity from the user (I used tap as opposed to swipe)
Swipe right multiple times:

Speak timestamp of user's activity
"Bot said, landmark:"
"Showing calendar" (activity)
"1 attachment"
Speak timestamp of bot activity
"Press enter to interact" <-- needs to be removed on mobile (#3758)
"Bot said"
"Showing calendar"
"Bot attached"
"Adaptive Card design session"

As seen in steps 2-4, then 7-9, double aria labels are being read twice when they shouldn't be.


Outlet from one nav.item to another

Hi,
in master-detail pattern it's normal to outlet from master's list section on the left to detail's content section on the right. But, would it be possible to use one nav.item as a master and another nav.item as the outlet detail, please? For example:
{{#bs-nav type="tabs" as |nav|}}
{{#nav.item}}{{#nav.link-to "orders"}}Orders{{/nav.link-to}}{{/nav.item}}
{{#nav.item}}{{#nav.link-to "order"}}Order{{/nav.link-to}}{{/nav.item}}
{{/bs-nav}}
Say, in the Orders tab I have a list of Orders. When selecting one order, I would like to swap to another tab - Order detail to display its order lines. Or, is it possible to make a tab selected programmatically, please?
Thanks. Karel

You can programmatically make nav items active, by overriding the default behavior of active. See https://www.ember-bootstrap.com/api/classes/Components.NavItem.html#property_active.
Hope that helps!?

Thank you for your response. I'll try to override it as suggested. If I succeed I'll post the code.
Karel


Fix/yup silently fail

LGTM! perhaps this could be a minor update?

yep, I'll open a PR for the v1 as well


Add Prettier formatter for JS files

Continues https://github.com/webpack/webpack.js.org/pull/2247/

lmk if that needs to be closed 😄

@evilebottnawi @anshumanv @EugeneHlushko Ping
@anshumanv I would like to include all your changes, let's do it after merging this one.

WIll update 👍

What is the blockage here ?

@montogeek Why prettier is blocked?
@chenxsan I think we need focus on this after webpack@5 upgrading

Sure, I can do it after the webpack v5 upgrade if @montogeek doesn't have time to continue it.

There is no blocker at this moment, I will restart it instead, can do it soon :)

@montogeek Hi, do you still have time doing this? Otherwise I can take care of it.

Closed via https://github.com/webpack/webpack.js.org/pull/4518


Show a warning when a user entered slug got changed

Short description

This applies to pages, events and pois.
Proposed changes

Show warning when the user provided slug automatically got changed
For pages, link to a version that uses the slug
for pois and events, link to the latest translation of the object and note that the slug might have been used by a previous version of this object

Side effects

/
Resolved issues

Fixes: #1521

Pull Request Review Guidelines

This is only my preferance and probably an over-kill but I have one thing:
The link in the warning opens the form for the event/poi which is occupying the user-input slug. If the user doesn't read the last part of warning message "or one of its previous versions" carefully, they probably try to free the occupied slug by changing the slug in the form, which but doesn't work.
To avoid this, I suggest to show a preview instead of form.

I agree that this can be confusing to users. Although the best case solution would probably be to make events&pois link to a version view, like pages do. Should I open a new low prio issue to create versions views for events and pois?

Should I open a new low prio issue to create versions views for events and pois?

Those already exist: #1154 & #1155 :wink:

Should I open a new low prio issue to create versions views for events and pois?

Those already exist: #1154 & #1155 😉

Cool 😃
Then the links in warning message can be adjusted in #1154 and 1155 together.

Merging should work now :smile:


Use the atom-text-editor tag instead of the editor class.

In stylesheets/supercollider.less: Use the atom-text-editor tag instead of the editor class.

new release is out.


'WP2StaticNetlify\Controller' not found in /bitnami/word....

When I try to activate wp2static-addon-netlify I'm getting this error in wordpress:

Fatal error: Uncaught Error: Class 'WP2StaticNetlify\Controller' not found in /bitnami/wordpress/wp-content/plugins/wp2static-addon-netlify-1.0.1/wp2static-addon-netlify.php:27 Stack trace: #0 /bitnami/wordpress/wp-content/plugins/wp2static-addon-netlify-1.0.1/wp2static-addon-netlify.php(41): run_wp2static_addon_netlify() #1 /opt/bitnami/wordpress/wp-admin/includes/plugin.php(2286): include_once('/bitnami/wordpr...') #2 /opt/bitnami/wordpress/wp-admin/plugins.php(191): plugin_sandbox_scrape() #3 {main} thrown in /bitnami/wordpress/wp-content/plugins/wp2static-addon-netlify-1.0.1/wp2static-addon-netlify.php on line 27

Hi @DashBarkHuss,
If using from the source code here, you'll need to run composer install, to make the required classes available.
Here's a pre-compiled zip installer, which can be easier:
http://wp2staticdlsnov2020.s3.amazonaws.com/wp2static-addon-netlify-1.0.1.zip
Please let me know if either of those helps resolve this

Thanks Leon for the quick reply. The link to the precompiled one isn't working. That was the same issue I had with this page which is why I eventually tried to get the Netlify add on from the github releases section.
All the other links on the thanks for donating page work, just not the Netlify one.

Thanks @DashBarkHuss, not sure what's going on with that particular link for you. I downloaded it again here fine, here it is attached for you:
wp2static-addon-netlify-1.0.1.zip
To debug why that other link isn't working, we could maybe try different browsers, or add a cachebusting string at the end, like: https://wp2staticdlsnov2020.s3.amazonaws.com/wp2static-addon-netlify-1.0.1.zip?cachebust=1234

Thanks the cache busted string worked!


Fixing small bug in mWater api pull

mWater decided to change the allowed forms of datetime which broke our parsing of the datetime columns (start_dt, end_dt and malfunction_end_dt).
Work:

Simple fix in clean_mwater_notes to correctly parse DT
When this was joined with old field notes, end_dt became numeric values. Adding end_dt  = as.POSIXct(NA, tz = "MST")) in clean_field_notes seemed to have resolved that and end_dt is a datetime object again

Review Req: Make sure the field notes seem to get pulled in correctly on your side and thoughts on whether we should move to using parse_datetime to give multiple formats in the event that this type of change occurs again

@juandlt-csu I added the datetime parsing if you want to test using that instead of the changes I made before

Neglected to merge to main before adding realistic range commits. @kathryn-willi check out the last couple commits for my recent changes.
Work:

created a scratch function grab_raw_data.R to quickly pull raw data. I needed to share data with a grad student and now there is a relatively easy way to query what you want from the flagged dataset and the pre 2023 dataset.
created add_realistic_flag.R and its associated yml. This is essentially a copy of add_spec_flag but using realistic ranges rather than sensor ranges. the yml could be trimmed down to only the parameters of interst (ph, sc, temp) but I left everything else from sensor_spec
small updates to figures for 2023, going to chat with MR about it next week to share with FC

Going to merge and start incorporating your new flag as well as a few others. Please stay tuned for another PR from me sometime next week! :)


Allow proxying to restart by not resetting the access mode on stop

This change partially fixes #2644.
On Chrome, it is now possible to reconnect to a proxy through the 'Try again' button.
No regressions introduced on Android.

This change is

Could you add a comment in to both definitions of proxyAccessMode_ explaining its behavior?  Is it ever reset to NONE?

Added the comments.
No, it's never reset to NONE. If we reset it on stop, then we get a single shot at reconnecting (the restart attempt calls stop if unsuccessful).

Review status: 0 of 2 files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

:+1:

FYI, I'm hoping to roll a release this afternoon and this seems like a very important PR to merge first :-)


Why "React"?

Hi,
Thanks for the lib, I'm using it and it's cool. But why is there "react" in the name? From what I see in the code there is nothing related to React in this library, it's plain vanilla JS.

Any suggestions for the better name?

Maybe Facebook Pixel Helper.
I guess the word "react" is driving most of the SEO tho so you can keep it in the readme.
But right now it's pretty confusing.


Release

Description
Please provide a brief description of the changes made in this pull request.
Related Issue
Closes #
Checklist

[x] I have tested the changes thoroughly.
[x] I have updated the documentation.
[x] I have added appropriate tests.
[x] I have checked for any linting errors.
[x] My changes generate no new warnings.
[ ] I have commented my code.

Codecov Report

Merging #88 (891fb81) into master (38277ca) will decrease coverage by 22.68%.
Report is 11 commits behind head on master.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@              Coverage Diff              @@
##             master      #88       +/-   ##
=============================================
- Coverage     72.91%   50.23%   -22.68%
+ Complexity      131      102       -29
=============================================
  Files            12       12
  Lines           635      635
  Branches         60       60
=============================================
- Hits            463      319      -144
- Misses          160      306      +146
+ Partials         12       10        -2

Flag
Coverage Δ

IntegrationTests
?

UnitTests
50.23% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 7 files with indirect coverage changes


✨ Support sveltekit

Description
It would be awesome if there was documentation about how to integrate with sveltekit

Why did you close it, have you found a way to use it with SvelteKit?

@Andres6936 https://docs.master.co/css/setup/svelte
In v1, you just need to import @master/css as a normal JS package and the JIT engine will be enabled immediately.
In v2, we'll include AOT setup guides for each framework.

@alpap @Andres6936
Now the AOT mode of 2.0 Beta already supports Svelte Kit, but it is not yet able to scan the class name of the entry file like app.html, this is not a big problem, but we're looking for best practices.
Check out the Svelte Kit + Master CSS example: https://github.com/master-co/css/tree/beta/examples/svelte-kit

Initialize MasterCSSVitePlugin: beta/examples/svelte-kit/src/routes/%2Blayout.svelte#L4

Wrong link for this one ref ;) : https://github.com/master-co/css/blob/beta/examples/svelte-kit/vite.config.ts#L8

After some refactoring and fixes, Master CSS v2 AOT is fairly stable in Svelte Kit. 🎉

Check out the latest example and guide: https://github.com/master-co/css/tree/beta/examples/svelte.


[sw/silicon_creator] Disable flash in pre-CRT exception handler

Disable accesses to flash before triggering the reset. Equivalent to #8171 but for the default pre-CRT exception handler.

@mundaym, I tried rerunning the verilator tests but irq_asm_functest times out (this test is skipped on CW310).

Closing since the changes in this PR were moved to #9657.


Is there any option to generate pdf for chart

What problem does this feature solve?
easy to  generate pdf
What does the proposed API look like?
getDataURL({ type: 'pdf' });

duplicate of #5181
Try this solution. It uses 3rd party library pdfmake.
📌 please close issue if problem solved.

thank you

@helgasoft thank you


[JS] can I get the audio file in Real-time Speech-to-Text?

i am looking for a way to get the audio file when i use
SpeechSDK.AudioConfig.fromDefaultMicrophoneInput()
,but all most answers said it should be
SpeechSDK.AudioConfig.fromStreamInput(pushStream)
so the fromStreamInput is only way in JS?
and in this official page.the Realtime Speech-to-Text use it too?

Speech Studio should let you record from the microphone and download the wav file.
If you're using the SDK directly you could enable audio logging (off by default) on the SpeechConfig object and then use this REST API to retrieve the audio:
https://{region}.dev.cognitive.microsoft.com/docs/services/speech-to-text-api-v3-0/operations/GetBaseModelLogs

Speech Studio should let you record from the microphone and download the wav file.
If you're using the SDK directly you could enable audio logging (off by default) on the SpeechConfig object and then use this REST API to retrieve the audio: https://{region}.dev.cognitive.microsoft.com/docs/services/speech-to-text-api-v3-0/operations/GetBaseModelLogs

Thank you for your reply,and i got the log.but it is a list,and in this doc
https://eastus.dev.cognitive.microsoft.com/docs/services/speech-to-text-api-v3-0/operations/GetEndpointLog
i should give the logId but i can't find a way to get it...
there is a resultId,but not logId

@Aethey See https://github.com/Azure-Samples/cognitive-services-speech-sdk/issues/1742.

@Aethey See #1742.

Thank you，
but this answer is about Get Custom Model Endpoint Logs,not Get Custom Model Endpoint Log,if i want to get a file not a list, i need a logId,but how to get the logId。

@Aethey In 1742 it was asked

Could you tell how can I get the specific audio's logid that I want .

but if this indeed does not match your question then @mahilleb-msft to the rescue.

@andreasjl @mahilleb-msft Please advise.

Since there is apparently not going to be any further advice on this question, my take is that based on https://github.com/Azure-Samples/cognitive-services-speech-sdk/issues/1742#issuecomment-1319835713 (highlight added)

you can enumerate custom models endpoints logs through Get Custom Model Endpoint Logs, check .values[].self on the result.

you need to select one value of .values[].self from GetEndpointLogs response and use that as the value of logId for GetEndpointLog. (If it's really like that, obviously the API could be better described.) Of course, if there are several audio files then you need to "know" which one you want - maybe based on name, kind and/or createdDateTime - or just download them all.

Closed as answered. Please create a new issue if more support is needed.


don't filter topics if its explicit disabled

During the investigation of #92 I found out, that even when the topic check is disabled via config (topic-check": {"enabled": False}). All plugins will be used to check for topic filtering.
The reason is that in the method topic_filtering the value None is passed for filter_plugins to the method map_plugin_coro
https://github.com/Yakifo/amqtt/blob/8961b8fff57007a5d9907b98bc555f0519974ce9/amqtt/broker.py#L728-L738
In the manager then, if the value is None all available plugins will be used.
https://github.com/Yakifo/amqtt/blob/8961b8fff57007a5d9907b98bc555f0519974ce9/amqtt/plugins/manager.py#L167-L169
My fix will immediately return True (topic_result) if in the topic-check-configuration explicit enabled was set to False.
If the key enabled is missing all plugins will be used as it was done before (backward compatibility).

As workaround until this PR is merged, you can add the following to your config:
"topic-check": {"enabled": True, "plugins": []}

This bit me today as well.

LGTM @FlorianLudwig

@FlorianLudwig Can this be merged or do I miss something?


Remove padding around CardActions

We use action buttons in the card footer component similarly to how the modal footer positions action buttons. The difference is that there is about 16px of left padding on the CardAction component which prevents the buttons from aligning flush to the left. By default, could we remove the left padding?

Hey @marshmalien, <CardActions> goes in <CardHead> and positions actions at the top/right of the card. You can see a demo of that here - https://www.patternfly.org/v4/documentation/react/components/card#with-image-and-actions
Since buttons in the footer of a card aren't a fundamental part of the card like they are actions associated with a modal, we don't have a layout to space actions in a card footer. If the actions in the card are associated with a form, you could use the form component and the <ActionGroup> component. Otherwise you could use the flex or split layouts (or custom CSS) to place a row of actions in the card.
Would one of those options work for you? We have a proposal for a generic action list component that would work in this use case that we could bump up in priority as well - https://github.com/patternfly/patternfly-next/issues/2021

The Master-Detail design layout gives us the option to place Details in a card, so I placed the actions in the card footer because that is where they're located in the modal 🤷♀ For this use case, a generic action list component would be perfect! Bumping up the priority would really help us out. Thank you @mcoker

@marshmalien were you able to come up with a solution for now? Curious if we can close this and you can track the progress of the action list component via https://github.com/patternfly/patternfly-next/issues/2021. We should be able to slate work for that following the major version bump we're going to work on for the next few milestones.

Yes, we can go ahead and close this issue. We will swap our current card actions implementation when the action list component becomes available. Thank you!


Cursor out of alignment after startup

Sometimes, the cursor can be out of alignment for a few pixels. It's resolved when switching apps and back again.

Hit the same issue in win7(virtual machine), my workaround is adjust resolution(then you can adjust back).

Hey, I seem to encounter this issue again, I can't resize the window until moving the cursor a few pixels lower than the window's boundary.

So apparently the VM is using the host’s cursor which is slightly misaligned.
A kind of workaround is to do a screenshot (cmd shift 5), then the host os's mouse overlays with the correct location, as shown below.


Error whenever n_jobs > 1

If I set n_jobs to be more than 1 I get an error:
Traceback (most recent call last):
  File "/home/robert/miniconda3/envs/uav/lib/python3.6/site-packages/joblib/externals/loky/process_executor.py", line 418, in _process_worker
    r = call_item()
  File "/home/robert/miniconda3/envs/uav/lib/python3.6/site-packages/joblib/externals/loky/process_executor.py", line 272, in __call__
    return self.fn(*self.args, **self.kwargs)
  File "/home/robert/miniconda3/envs/uav/lib/python3.6/site-packages/joblib/_parallel_backends.py", line 608, in __call__
    return self.func(*args, **kwargs)
  File "/home/robert/miniconda3/envs/uav/lib/python3.6/site-packages/joblib/parallel.py", line 256, in __call__
    for func, args, kwargs in self.items]
  File "/home/robert/miniconda3/envs/uav/lib/python3.6/site-packages/joblib/parallel.py", line 256, in <listcomp>
    for func, args, kwargs in self.items]
  File "/home/robert/.local/lib/python3.6/site-packages/zones/raster_stats.py", line 359, in worker
    image_array[null_idx] = no_data
IndexError: index 58 is out of bounds for axis 0 with size 1

Here is as an example (in ipython)
import zones
rast, vect = zones.datasets.load_01_single()
rast2 = "/tmp/raster_test.tif"
!gdalwarp -tr 1 1 {rast} {rast2}
zs = zones.RasterStats(rast2,
                       vect,
                       verbose=2,
                       n_jobs=2,
                       band=1)
gdf = zs.calculate('mean')

Note that this doesn't happen if you use the original test data, but I think that is because it is so small that no parallel stuff is happening. So I increased the size of the image by resampling.

There was no check for multi-dimensional arrays when indexing null values. The check existed in the single core process, but not in the joblib process.

Fixed in 1ebd72c4ca6ddf854a00cb716263636c1c0a20e5

Thanks, tested it on the small example, works well. I'll test it now on the larger example.


Time zone aware schedules

First of all, this looks like a really useful library that fits nicely into the current project i'm working on.
One of my needs is to have schedules on specific days in different time zones.  I can't see anywhere to configure the time zone for the current schedule, isn't this supported as of now? If not, i see you are using the Klock library after a quick poke into the codebase. Klock supports time zone offsets, isn't it possible to use this feature?

Hello, thanks for your feedback:)
Yes, as I know, it is available in Klock out of the box and I will try hard to include opportunity to set it up as soon as possible

Hello, @kfh , I have implemented work with time zoned date time #16 , but I am not pretty sure that it is exactly what you have meant in this issue:) anyway, you may take a look this PR and maybe give some suggestions

Hello, @kfh , I have implemented work with time zoned date time #16 , but I am not pretty sure that it is exactly what you have meant in this issue:) anyway, you may take a look this PR and maybe give some suggestions

Thanx for the effort! Maybe i wasn't too clear, but what i'm looking for is a way to run ie a schedule on every monday at 09:00 in a given timezone(ie "Europe/Berlin") so that we are independent of where the code is running(Cloud in a given region etc). I think it would make sens to offer a thin convenience layer on top of the Klock API so that one could just pass in timezone strings which then would be converted to offsets behind the scenes. I'm thinking someting along the lines of what they are doing in kotlinx-datetime.

The problem is that you will be able to use it only with DateTimeTz, because of simple DateTime have no any time zone variant and in fact may be used only as utc analog. I think, I can add an option in krontab string and builder, but it will work only with next(DateTimeTz) avariant of next method

The problem is that you will be able to use it only with DateTimeTz, because of simple DateTime have no any time zone variant and in fact may be used only as utc analog. I think, I can add an option in krontab string and builder, but it will work only with next(DateTimeTz) avariant of next method

There are of course cases where the timezone is insignificant, ie when a schedule is based on running every 5 minutes or every hour or so. I guess this could be solved in different ways internally, ie using _ DateTimeTz_ everywhere and just using the default when the time zone is not relevant, it probably defaults to UTC time anyways. The important bit is that one has the possibility to pass in the time zone when it matters and that the underlying code handles this case properly.

Hello:) i have included minutes offset into string parsing and you may test it if you will include artifacts from github packages of this repository

This is my answer for this comment

DateTime.now() will return object with current system locale date and time. So, you can pass this object to next method and get next time relatively to system locale
Now there is opportunity to use offset part. This part will works ONLY in next(DateTimeTz) method

If to be honest, currently I am not pretty sure how should we made including an offset in config due to the fact that it will not work with DateTime if you are not passing and offset parameter
Of course, you are welcome with some with PullRequest with you point of view for this issue

This is my answer for this comment

DateTime.now() will return object with current system locale date and time. So, you can pass this object to next method and get next time relatively to system locale
Now there is opportunity to use offset part. This part will works ONLY in next(DateTimeTz) method

If to be honest, currently I am not pretty sure how should we made including an offset in config due to the fact that it will not work with DateTime if you are not passing an offset parameter

My suggestion was that you use DateTimeTz everywhere to make the api cleaner. It would probably mean that you have to rewrite some code i guess. If the next() method signature is something like the following:
suspend fun next(relatively: DateTimeTz = DateTime.now().utc): DateTimeTz?
shouldn't that cover all cases? When the time zone is not relevant, ie when a schedule runs every minute, the time zone doesn't matter anyway, right. So it's not significant that it's utc based. And when you actually want to adjust the time zone that also works as you have done in the pull request.

Of course, you are welcome with some with PullRequest with your point of view for this issue

Noted! :)

I see your point:) Anyway, currently I am still thinking about how to implement it in the most comfortable way

This is my answer for this comment

DateTime.now() will return object with current system locale date and time. So, you can pass this object to next method and get next time relatively to system locale
Now there is opportunity to use offset part. This part will works ONLY in next(DateTimeTz) method

If to be honest, currently I am not pretty sure how should we made including an offset in config due to the fact that it will not work with DateTime if you are not passing an offset parameter

My suggestion was that you use DateTimeTz everywhere to make the api cleaner. It would probably mean that you have to rewrite some code i guess. If the next() method signature is something like the following:
suspend fun next(relatively: DateTimeTz = DateTime.now().utc): DateTimeTz?
shouldn't that cover all cases? When the time zone is not relevant, ie when a schedule runs every minute, the time zone doesn't matter anyway, right. So it's not significant that it's utc based. And when you actually want to adjust the time zone that also works as you have done in the pull request.

Of course, you are welcome with some with PullRequest with your point of view for this issue

Noted! :)

There are two options about solving of offset problem:

Simple. We are just adding timezoned DateTimeTz and pass offset via some parameter like offset: Minutes in methods like buildSchedule/createSimpleScheduler/etc.
Difficult (as solved for now). We are adding option to parsing and get opportunity to serialize/deserialize and save/load configuration for each string-oriented KronScheduler

If you see any other option about realization, I would be glad to hear it. Anyway, both ways in the options above were included. Currently I am thinking about default realization for method next(DateTimeTz), because of it is not very obvious what woulb be better to get as default way - we are assuming that method KronScheduler#next(DateTime) usually called in utc offset or local

@kfh hello, now you can checkout new version with time zones and weekdays :)


[Project Addition]: Facial Expression Detection

Deep Learning Simplified Repository (Proposing new issue)
:red_circle: Project Title : Facial Expression Detection
:red_circle: Aim : To Detect the Facial Expressions
:red_circle: Dataset :https://www.kaggle.com/datasets/fatihkgg/affectnet-yolo-format
:red_circle: Approach : Try to use 3-4 algorithms to implement the models and compare all the algorithms to find out the best fitted algorithm for the model by checking the accuracy scores. Also do not forget to do a exploratory data analysis before creating any model.

📍 Follow the Guidelines to Contribute in the Project :

You need to create a separate folder named as the Project Title.
Inside that folder, there will be four main components.

Images - To store the required images.
Dataset - To store the dataset or, information/source about the dataset.
Model - To store the machine learning model you've created using the dataset.
requirements.txt - This file will contain the required packages/libraries to run the project in other machines.

Inside the Model folder, the README.md file must be filled up properly, with proper visualizations and conclusions.

:red_circle::yellow_circle: Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.
"Issue Title" and "PR Title should be the same. Include issue number along with it.
Follow Contributing Guidelines & Code of Conduct before start Contributing.

:white_check_mark: To be Mentioned while taking the issue :

Full name :  Veera Venkata Karthik Barrenkala
GitHub Profile Link : https://github.com/Karthik110505
Email ID<EMAIL_ADDRESS>Approach for this Project : I will using some of the pretrained models like yolov8, yolov5, mediapipe and detect the facial expression and make a model to detect the expression from images. If possible from video and a webcam also.
What is your participant role? (Mention the Open Source program): GSSoC24

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

@abhisheks008  Please assign me this issue

Hi @Karthik110505 this type of project is already present here in this repo. Have a look, https://github.com/abhisheks008/DL-Simplified/tree/main/Facial Emotion Detection

@abhisheks008  Yes,bro. After seeing the project only I made this issue. In that project , he/she only took the data , perfromed some model training and implemnted some evaluation methods.
But in  my project i want to build a detection model which can make bounding boxes and detect the expression, for the images. I also want to make this work for videos and while using webcam, which didn't done in the already present project.
Now, it's your wish . Should I continue or look for some other issue?


JS Behat tests failing due to session or auth issue (selenium & chromedriver)

Hi, guys.
It looks like JS behat tests failing due to some session or auth issue:
$ vendor/bin/behat
@managing_products
...
  @ui @javascript
  Scenario: Adding a text attribute to product
    Given I want to create a new simple product
      Expected to be on "http://localhost:8080/admin/products/new/simple" but found "http://localhost:8080/admin/login" instead (Sylius\Behat\Page\UnexpectedPageException)

PS: I've followed all installation steps for clean copy of latest plugin skeleton. Chromedriver and selenium server also latest.
Also, how about adding it to travis and add badges? As I can see - config is ready for that.
Thanks

@pamil, I guess it's similar to what we've been talking about regarding this thread: https://forum.sylius.com/t/behat-javascript-scenarios-can-t-be-logged-as-admin/618

I'm also running it on OSX (if this matters).

I believe there can be something with resolving .env files for the plugin application... I've managed to make it work, but I was also getting this error at the beginning. The problem was, the server I had run was using different env than Behat tests. Could you please try to run set -a && source tests/Application/.env.test && set +a? It can solve the problem, but still, it's questionable should we require such a step or should it be resolved automatically.

Thank you a lot about idea, @Zales0123
I think the solution will be copying tests/Application/.env.test to tests/Application/.env rather than sourcing it. That is because server:run command gets .env or .env.dist be default regardless what was provided at --env= option (and I'm wondering why it designed like that).
Will check this soon.

Just checked.
Yeah. Executing cp tests/Application/.env.test.dist tests/Application/.env before vendor/bin/behat --tags="@javascript" make it works.

👍 could you please open a PR with installation instruction update? We will think about how to solve it in a better way, but for now it would be better than nothing :)

I've found a way how to fix that without any changes in workflow. We just should fix bin/console to take env from input parameter. See reference to 0d8c117 above.

It looks like an issue caused by @FriendsOfBehat SymfonyExtension v1, could you open a new issue if it still exists with v2?


Fix video aspect ratio

Many of the videos appear to be at the wrong aspect ratio, just enough to look weird.

[ ] Identify which videos are affected by this
[ ] Figure out what change is needed to correct the problem
[ ] Write a script to download the videos from the Internet Archive, correct them, and re-upload them (overwriting the existing video, not adding them as new videos)

It seems that legislative video is actually 720 by 543, not 720 by 480. headdesk

Wow, this resized video looks terrible.  It's 720 by 543, and just looks really bad:

Compare to the original (with improper width:height ratio):

I don't understand what's going on here. How can the "wrong" aspect ratio look so crisp, while the "right" aspect ratio looks like a 3rd-generation VHS tape?

I tried ripping the video in HandBrake, setting the anamorphic to "640x480Loose," and enabling decombing, and got much better results.

So the question is how to convert existing video—basically, all of last year's—without going back to the DVDs.

Argh—I got that right in HandBrake exactly once. I didn't note the settings, and now I can't recreate them. :-/

Aaand it turns out that the video in question (House, 2016-01-20) just looks awful on the original DVD. I can't believe I wasted so much time on this.


String length Bug

main(List<String> args) {
  List SL="".split('-');
  int len=SL.length;
  print("SL= $SL");
  print("Length = $len");
}

outputs:
SL= []
Length = 1
The length should be zero.

From the docs:

If this string is empty, the result is an empty list if pattern matches the empty string, and it is [""] if the pattern doesn't match.

var string = '';
string.split('');                       // []
string.split("a");                      // ['']

The length calculation seems fine.

Correct - the result of "".split("-") is the one-element list containing the empty string, so the output is correct.


Add UTF-8 charset to index.html

I think this is a reasonable default, and it might prevent surprises down the road for users, like I encountered when trying to bundle d3.js, which includes a variable named ε = 1e-6.

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

I think the encoding is mostly a product of the webserver rather than the page itself, but I'm still all for this. Thanks!

Ah yeah, I've just been using the raw file so that might be a contributing factor. Thanks!


How should we handle spammers?

Opening this issue to discuss our options regarding spammers, following a request from WWC in https://opencollective.freshdesk.com/a/tickets/3052, https://opencollective.freshdesk.com/a/tickets/3053 and https://opencollective.freshdesk.com/a/tickets/3054 to block someone submitting expenses to their collectives.

We have two paths that we need to be able to solve:

Users submit expenses that are clearly SPAM.
Example: https://opencollective.com/bubis-makaronas.

What should be our policy to ban these users? How should we proceed?
Possible option: introduce a limited mode for users, blocking all community actions (submit expenses, contact...etc) and showing a banner at the top Your account has been limited. Please contact support if you think this is a mistake.

Users submit expenses that it's unclear if they're SPAM or not. Definitely not enough to ban the users, but WWC are asking if they can block them. Example: https://opencollective.com/sarah-harder

We could re-use the system implemented in https://github.com/opencollective/opencollective-api/pull/2808, but it completely blocks the feature for the user (and not only for this collective).
Like on Github, add a feature to block users from a collective

1. Users submit expenses that are clearly SPAM.
I agree with the idea of limited mode for spammers. Whatever we do, we should show a banner to a user letting them know that we've taken action (whether it's limiting or fully blocking the account) just in case it's a false positive.
2. Users submit expenses that it's unclear if they're SPAM or not.
Community moderation is really important no matter what the reason, so I think having a Github-style feature to block users from a collective is a great idea. It can be used for spammers, harassers, etc.
Maybe we can give communities the option to set a minimum bar to clear before you are able to do certain things? i.e. you have to set a profile photo before being able to claim an expense. This would maybe make it a little harder for spammers to do this stuff?

Related: #2002 #2006

Another example of expenses as a SPAM vector: https://opencollective.com/lord-justine#section-transactions
We need an option to flag a user as "banned" to prevent them from doing any further action on the website.

Followed up with the limited account idea in https://github.com/opencollective/opencollective/issues/2695
We still need to define a procedure for banning users permanently.

Example of a slightly different flavor of spam, where they are hoping we'll just pay without looking too closely, as opposed to posting for linkspam or pageviews: https://opencollective.com/madrid-retreat-1ev/expenses/12316

@alanna I've seen a bunch of these lately (and removed some).

This discussion has now been turned into a project: https://github.com/opencollective/opencollective/issues/2741