have the builders communicate to builder directly

builder outputs the build status on a git push, so it would be nice if deis/dockerbuilder and deis/slugbuilder could communicate what they're doing back to builder, so the end-user (who did the git push) can see what's going on. currently they have no insight into what's going on while the build is in progress.
We'll likely need to finish part or all of #29 before this can be possible.

this can probably be solved simply with running kubectl logs -f instead of having builder pods talk back to builder. it's currently done in the current builder


Walkthrough help UI degrades in older Android releases

Adapt It Mobile Version: 0.5.5 (alpha 6)
Operating System: Android
OS Version: Android 4.4, 4.3 (don't have older releases to try this on)
Severity: 3
Description: Testing out the hopscotch "walkthrough" help; it looks like the Android browser is less and less standards compliant the further back you go. This behavior must have started when we removed the crosswalk browser plugin. :-/
Steps to Reproduce:

Start AIM, create or copy a project, import a file
Open up the adaptation page (search or adapt from the last chapter)
Click on the help [?] button in the upper right of the screen

--> Behavior should be to display callout help "bubbles" over the different areas of the UI, with previous and next buttons to walk you through the controls. Instead, you see the following:
Android 4.3: Callouts are hidden behind everything -- part of the callout is visible through the toolbar, but you cannot see the text, nor can you see the next / previous buttons to iterate through the help screens.
Android 4.4: The callout shows in front, but the Next button cannot be clicked on the screen (the adaptation area behind it takes the click events).
I'm not sure what the behavior is on other versions of Android.

Retested this on Android 5.0.1 -- the help walkthrough is working properly. We are having similar issues in Android < 5.0 in the newer filter dialog for AIM 1.0 beta 6.

Marking this issue as closed/obsolete in AIM 1.3 --
Android versions 5.0 and lower are not supported in Adapt It Mobile 1.3 (see issue #406 / upgrading cordova-android to support the latest Android versions).


Google Pay Not Displaying on iPhone in React Native Embedded WebView

I’ve embedded https://www.mystoredemo.io/#/cart in my React Native app. When running it on an iPhone 15, I noticed that Google Pay does not appear within the app, although it does show up when accessed directly through Safari or Chrome. Could you help explain why this difference occurs?

In Safari Chrome:

React Native Embedded Webview

Originally posted by @vanlethi-dev in https://github.com/Adyen/adyen-web/issues/2929#issuecomment-2467183995

Hey @vanlethi-dev ,
It does not work out of the box using Webviews. You can read more about it here and here. Unfortunately there is nothing we can do from our side for this use-case.
Cheers!

Hi @ribeiroguilherme, I’ve reviewed the documentation above. It only mentions support for Android, not iOS. From what I understand, Google Pay isn’t supported within webviews on iOS apps. Is that correct?

Correct. As of today the isReadyToPay() API returns false for iOS WebViews. We are looking into ways to enable Google Pay for iOS WebViews in 2025.


there is memory leak at line number 76

consider breaking the reference to this object during dispose().

@jemisgoti Sorry for late response. I didn't noticed there was issue. I will try to investigate it when I have time.

consider breaking the reference to this object during dispose().

I have updated the code. Please check it if you have time.
Thank you.


Unexpected a result message from kubectl

I got an unexpected message by Kubectl when I upgrading Workflow.
This is same as the doc
~ $ kubectl --namespace=deis get service deis-router \
>     --output=go-template='{{ index .metadata.annotations "helm-keep" | println }}'
true

But at the next step, I got the unexpected message like this.
~ $ kubectl --namespace=deis annotate namespace deis helm-keep=true
error: --overwrite is false but found the following declared annotation(s): 'helm-keep' already has a value (true)

After adding --overwrite, it became similar to the document.
~ $ kubectl --overwrite --namespace=deis annotate namespace deis helm-keep=true
namespace "deis" annotated

Is it caused by my environment, or a typo in the document?
My environment is...
~ $ kubectl version
Client Version: version.Info{Major:"1", Minor:"2", GitVersion:"v1.2.4", GitCommit:"3eed1e3be6848b877ff80a93da3785d9034d0a4f", GitTreeState:"clean"}
Server Version: version.Info{Major:"1", Minor:"2", GitVersion:"v1.2.4+coreos.cni.1", GitCommit:"89e3af04ef436f049e2e2d169e57ad1712cac375", GitTreeState:"clean"}

I'll send the pull request if it is caused by a typo.

you got the error because you namespace already has the annotation helm-keep=true and hence don't need to do it again.

Is this issue caused just by my misreading? Or is it better to add any notices to the document?

you can add a notice saying to add annotate by checking first to see if its not present.

The docs prefix it by

To add a missing annotation, use kubectl annotate

So I think this is clear enough as-is.


Should some EventTargets also be Signals?

Some DOM event targets (such as MediaQueryList, returned by matchMedia as well as AbortSignal) have a single property and a single event that emits whenever that property's value changes. While signals could be offered as a second property, it might also be nice to have some way for a class to inherit both EventTarget and Signal.
const mql = window.matchMedia('(max-width: 900px)');
const isSmallScreen = new Signal.Computed(() => mql.get());

I'm also really happy to see Signal.subtle.watched/Signal.subtle.unwatched which makes it pretty easy to wrap these APIs in userland :) It's missing from a lot of framework Signal libraries.
function matchMediaSignal(query) {
  const mql = window.matchMedia(query);
  const abortController = new AbortController();

  const signal = new Signal.State(mql.matches, {
    [Signal.subtle.watched]() {
      this.set(mql.matches);
      mql.addEventListener(() => this.set(mql.matches), { signal: abortController.signal });
    }
    [Signal.subtle.unwatched]() {
      abortController.abort();
    }
  });
}

similiar issue: https://github.com/proposal-signals/proposal-signals/issues/88
Thank you for adding a use case.

This is an excellent question and corresponds much more closely to how I would like to see signals integrate with the web platform than #88. @wycats was telling me about an idea for ResizeObserver to integrate with signals in a similar way. Let’s develop more examples of how this integration should look—it is nice to see that the signal versions of web APIs can be built as a library on top, as demonstrated above.

For what it's worth, I think the critical question is not whether the EventEmitter emits an event with a single value when the value changes, but rather whether it's safe to ignore intermediate values until you're ready to use them in another computation.
(In my opinion, if a lossless stream of intermediate values are important, Observables are a much more effective abstraction).
That said, I think it is quite common for Signal-like modelling to make sense for situations that currently use EventEmitter and I think it's a wide open design space for exploration.
Broadly speaking, I think the *Observer DOM APIs are the strongest initial candidates, because those APIs are already intended to be used in discrete ways (and intentionally batch up changes until a particular phase in the rendering pipeline).


memberships: remove custom wasp Accept header

Memberships API is now official.
https://developer.github.com/changes/2014-09-16-finalizing-the-organization-and-team-membership-apis/
Fixes #192
/cc @willnorris

rebased onto master and merged as af9f30e4c34dbd5c5870e1bf8ce471e0f1e6e127


Colorful component works incorrectly on mobiles

I try to drag pointer of on Colorful component on mobile device, but nothing happens.  You may try it on your live-demo site from your phone  https://uiwjs.github.io/react-color/#/colorful

@kilimandzharov Upgrade v1.3.1


Run CompiledFunction in VM

Basing on https://github.com/d5/tengo/pull/330 I created PoC of running compiled functions. Please tell me what you think about this.

I'm not sure what use case this solves. You should be able to do the same by just having your script do the computation, set a global for input and then get the value from it after the computation. It provides for a cleaner separation between in-vm and out-of-vm computation.

Two cases for now:

Basic one - user/client provide script with predefined function names as handlers for specific actions in system.
More complex - user provide single script for multiple actions and can specify which function should be called for predefined and custom events in system.

Reusability of code make script-per-action bad solution. Having a "router"-function inside every script sounds bad too.
uGo  has this func (vm *VM) RunCompiledFunction(f *CompiledFunction, globals Object, args ...Object,) (Object, error), but it's 2.8k vs 73 stars on GH, so Tengo looks safer for longer future. We don't wonna run from fork/patched version too :)

Why do you think a router sounds bad? I don’t necessarily mean a router per script, but you could have a single entrypoint with a router, and load your scripts from modules and call the function there, similar similar to the case of how http is generally handled in php/js based “frameworks”.
That said, I do see your point as well. Your implementation itself looks fine, but I’m concerned about potential issues that might be introduced if these CompiledFunctions were called concurrently.
@d5 thoughts?

I could certainly also use this. I wanted to provide a Tengo API with callbacks but was unable to do so.

The router approach is introducing another in-memory structure access and function callback. Other than just performance - I don't see any bigger disadvantages, because I can provide my own module loader on Go side to side-load "client" scripts.
It just make it more complex and slightly slower.

Do I need to write integration tests or some manual tests with highly concurrent code will be enough?

I think it’d be better to have tests written in rather than manual testing.

Thanks for the changes. I won't be able to look into this for a couple of weeks but will get back to you as soon as I can.

Oops sorry, this is still on my list of TODOs. Just haven’t had the time to get to it yet.

Just wanted to voice a current use case for this although it might be an outlier.
Currently, I am working on a project that, for simplicity sake, basically runs Tengo scripts but adds custom modules that can be imported. I created a module that implements some of the functionality of the cobra library to add command/flag functionality to the scripts. For cobra, you can set what code runs when a command is passed as an argument by setting the command struct's Run field which is a func type. In order to implement this functionality in a Tengo module, I used this PR's code to be able to run a function that is passed in from the Tengo script. It has been working like a charm, with no issues that I have seen so far.

Yes, unfortunately this won't work because essentially I need the VM to be re-entrant (not mult-threaded).  The implementation completely wipes out the stack of the virtual machine, which is no good if it's being recursively called.
My use case very basic in data processing.  I want to write some user functions like C# LINQ-SQL's to work on collections of data:   select(),selectMany(), join(),groupBy(). All these functions take closures/lambdas as arguments.  In a scripting language, less imperative and more declaritive.  The functions return things that are so-called "queryable".  You could have something like:
Assume to arrays containing   myCollection []A  and type B   myOtherCollection []B
x  := From( myCollection ).Join( myOtherCollection, func( a , b ) true  { return b.AId == a.AId }, func( a , b ){
return { "a": a, "b": b } ).Sort( func( o1, o2 ) true { return o1.a.name < o2.a.name } )
Here we join two collections on matching field, map into a map containing both objects, then sort by the "a" object's name.

I wasn't here for a while, but if this issue still persists and no other implementation exist - I can fix the requested changes.
@kcsampson are you able to write example code that breaks with this changes so I can work on real example? To be honest I used those changes for a while, rewrote most code to parametrized Go functions and currently almost no traffic goes through tengo. Those were simple cases for access control.


--net-alias doesn't work for attachable overlay networks in swarm mode

Description
--net-alias doesn't do anything when running containers on attachable overlay networks in swarm mode
Steps to reproduce the issue:

docker network create -d overlay --attachable testnet
docker run --rm -it --net-alias aaaaaa --net testnet --name test alpine sh
docker inspect test

Describe the results you received:
The test container doesn't have a network alias.
Describe the results you expected:
The test container has a network alias.
Additional information you deem important (e.g. issue happens only occasionally):
Output of docker version:
Client:
 Version:      1.12.1-cs1-rc2
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   24231a4
 Built:        Mon Sep 19 00:55:12 2016
 OS/Arch:      linux/amd64

Server:
 Version:      1.13.0-dev
 API version:  1.25
 Go version:   go1.7.1
 Git commit:   94cd7ed
 Built:        Mon Oct  3 20:41:39 2016
 OS/Arch:      linux/amd64

Output of docker info:
Containers: 397
 Running: 13
 Paused: 0
 Stopped: 384
Images: 1551
Server Version: 1.13.0-dev
Storage Driver: overlay2
 Backing Filesystem: extfs
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge host null overlay
Swarm: active
 NodeID: cgqojbcnankxn0b87dp9og7sa
 Is Manager: true
 ClusterID: btcdet130qmlxzygia17v6uyf
 Managers: 1
 Nodes: 1
 Orchestration:
  Task History Retention Limit: 5
 Raft:
  Snapshot Interval: 10000
  Heartbeat Tick: 1
  Election Tick: 3
 Dispatcher:
  Heartbeat Period: 5 seconds
 CA Configuration:
  Expiry Duration: 3 months
  External CAs:
    cfssl: https://<IP_ADDRESS>:12381/api/v1/cfssl/sign
 Node Address: <IP_ADDRESS>
Runtimes: runc
Default Runtime: runc
Security Options: apparmor seccomp
Kernel Version: 4.7.3
Operating System: NixOS 17.03.git.4ed9343 (Gorilla)
OSType: linux
Architecture: x86_64
CPUs: 4
Total Memory: 15.53 GiB
Name: thisisfine
ID: 4XB6:RAOL:2UT7:W3TT:JBSR:ILHC:LREA:HUVU:5U7V:TI7E:WUT2:WQTS
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): true
 File Descriptors: 180
 Goroutines: 453
 System Time: 2016-10-13T17:58:42.58816-07:00
 EventsListeners: 2
Username: viktorstanchev
Registry: https://index.docker.io/v1/
WARNING: No cpu cfs quota support
WARNING: No cpu cfs period support
Insecure Registries:
 <IP_ADDRESS>
 <IP_ADDRESS>/8

Additional environment details (AWS, VirtualBox, physical, etc.):
physical

/cc @mrjana

I could reproduce the issue as well. Will get it fixed. thanks @vikstrous


iron-icon-demo only shows icon when run from demo/web/iron_icon/index.html

but not for example from demo/web/index.html (select iron-icon-demo)
Similar for google-chart demo which can only load the data files when launched from demo/web/google_chart/index.html

dup of #104


no color adaption  + btn sizes + headline color fixes

Download and Person card get no color adaption in any environment + correct btn sizes + headline color fixes
@apepper when merged we need to adapt some btn sizes in many pages in tenant

@apepper when merged we need to adapt some btn sizes in many pages in tenant

Can you please already create a working copy on the preview deployment and invite me to the working copy with the needed changes? That would be great.

@apepper https://edit-beta.scrivito.com/d0a154d76edf2a7bd991fc658e700a1d~http://localhost:8080/tickets?_scrivito_workspace_id=g4c828b32331d02c&_scrivito_display_mode=view

With the preview deployment: https://edit.scrivito.com/d0a154d76edf2a7bd991fc658e700a1d~https://agessler-color-sizes-fixes.scrivito-portal-app.pages.dev/tickets?_scrivito_workspace_id=g4c828b32331d02c&_scrivito_display_mode=view


Docstring: pd.to_datetime (issue #9107)

Following the issue "to_datetime returns NaT for old dates with coerce=True".
Do I understand correctly that online documentation is build automatically from docstrings? I.e. are there any other files to change?
Also it would be good to make a link to this section of "Caveats and Gotchas", but I'm not sure if the docstring is a good place to do this.

The online documentation is indeed built automatically from docstrings, but it doesn't exclude code in the examples:
http://pandas.pydata.org/pandas-docs/stable/generated/pandas.to_datetime.html#pandas.to_datetime
So it would nice to add the output to these examples (as well as the missing output for the previous examples in this docstring).

@shoyer - OK, corrected

Thanks!


Update testing.rst

Actually there is no "Reusing applications" topic on Usage chapter.

Thank you @yourwebmaker.


Add back in checking fix type

Added some code for checking the fix type before transmitting height or lat/lon. I want to switch the message we poll to the NAV-PVT message to make this look better and work better, but I'll include that in another PR.

Merged in logging changes from main. I will merge this branch to main and just remove the sending of fix data over the message queue from main for now. The branch will remain otherwise untouched in case we add it to the spec later and want to send it.


Istio admission webhook fails START_TIME formatting

Bug Description
Hi,
when trying to reformat a certificate expiration date I use format sting to create a new date format.
seen here:
https://www.envoyproxy.io/docs/envoy/latest/configuration/observability/access_log/usage#config-access-log-format-start-time
and using the sting formats seen here:
https://en.cppreference.com/w/cpp/io/manip/put_time
here is an example of my configuration:
http:

match:

uri:
prefix: /
route:
destination:
host: my-svc
port:
number: 443
headers:
request:
add:
X-Forwarded-For: '%DOWNSTREAM_REMOTE_ADDRESS%'
X-SSL-Client-Cert-Exp: '%DOWNSTREAM_PEER_CERT_V_END(%b %d %H:%M:%S %Y %Z)%'
X-SSL-Client-Cert-Issuer: '%DOWNSTREAM_PEER_ISSUER%'
X-SSL-Client-Cert-Subject: '%DOWNSTREAM_PEER_SUBJECT%'

But when saving the configuration Istio admission webhook fails the configuration with the following error:
could not be patched: admission webhook "validation.istio.io" denied the request: configuration is invalid: single % not allowed.  Escape by doubling to %% or encase Envoy variable name in pair of %
function code can be found here:
https://github.com/istio/istio/blob/master/pkg/config/validation/validation.go
detailed explanation can be found here:
https://www.envoyproxy.io/docs/envoy/latest/configuration/http/http_conn_man/headers.html#custom-request-response-headers
from what I can tell is that this validation is required for space within a URL phrase but should not be enforced on a time format as seen above
Thank you for taking a look at this
Version
istioctl
client version: 1.14.1
control plane version: 1.13.2
data plane version: 1.13.2 (7 proxies)

kubectl
Client Version: v1.24.1
Kustomize Version: v4.5.4
Server Version: v1.23.5

helm
v3.10.3+g835b733

Additional Information
No response

Just to clarify, are you 100% sure it is valid in envoy? And escaping via %% is not needed?

100% sure, once I've disabled the removed the validation hook it works as expected @howardjohn .

how can you get that from http headers?

how can you use that on http headers?
that format should be used for accessLogFormat on https://istio.io/latest/docs/reference/config/istio.mesh.v1alpha1/?

@zirain just like that X-TM-SSL-Client-Cert-Exp: '%DOWNSTREAM_PEER_CERT_V_END(%b %d %H:%M:%S %Y %Z)%'
(this is the format I need).
and it works as expected.

yeah, I got what you wanted.

the following lines need improved
https://github.com/istio/istio/blob/066f4b429663f2cd7afce744b2a8bdde7d1103a9/pkg/config/validation/validation.go#L381


Fix experimental settings

Changing settings in the experimentals section had no effect, this should fix that.
Something quite confusing (and probably at the root of the initial bug) is that live-common isEnvDefault is just a duplicate of getEnvDefault 😕
Type
Bug Fix
Parts of the app affected
Experimentals settings

The issue comes from a regression in ledger-live-common and should be fixed in LedgerHQ/ledger-live-common#504 instead


Wrong control flow analyse.

🔎 Search Terms
"control flow analyse"
🕗 Version & Regression Information

This is a crash
This changed between versions ______ and _______
This changed in commit or PR _______
This is the behavior in every version I tried, and I reviewed the FAQ for entries about _________
I was unable to test this on prior versions because _______
TS 5.3.3

⏯ Playground Link
https://www.typescriptlang.org/play?ts=5.3.3#code/C4TwDgpgBAYgTgQwOYFsIDthQLxQN4BQUUokAXFAOQBuAlgCYQD2lANEVAB4XoCuKAIwhwoAHyi90jAGa10EegQC+Y-B1IQKlYBE7A263cAAyEacB78hcZasLENWsLQDGwXnAiUO3KH0HCqpIycgrKBAQuTOgAzljSFPDIaJg4+CoIMVAI6CAR0pJutNFQKAhyABQAlBTUTAxqxLTSUBXSAHQaONi4NAzMlFAAZENQHZzduMFmofRVjcRQnu5w6BwqHM2tHV09vc5uHl7Do+OTElIz8nMLxMsea8RKmy1tneDQAIQ9VDp6lPN7IsorEsBNcOMOMQAPTQqAAC2E0GAiJIHygTBaExi8KYvAANvQMeh8SAoEI-FZhKxybwsLR9Fl-NYgpdZNc-EwAO7tdbKIA
💻 Code
type Fragment = {
  type: 'video',
  x: number | undefined
} | {
  type: 'text',
  textLeft: number
} | {
  type: 'picture'
  x: number | undefined
}

const f: Fragment = {} as any

function main(): void {
  if (f.type === 'video' && f.x === undefined) {
    return
  }
  if (f.type === 'picture' && f.x === undefined) {
    return
  }
  if (f.type !== 'text') {
    const x = f.x // line 22 is here
    // here the type of x should only be number, but it's number | undefined now.
  }
}

🙁 Actual behavior
In line 22, the type of x is number | undefined.
🙂 Expected behavior
It should only be number because it is already be narrowed before and not be reassigned again. I know that i can assign the f.x to a const variable fx and then narrow the variable fx as a workaround.
Additional information about the issue
No response

TypeScript doesn't perform this kind of analysis.

Obligatory link to https://github.com/microsoft/TypeScript/issues/52822#issuecomment-1434989002


CORE-13161 Implement interop grammar demo

Changes

Add skeleton token cordapp.

Info

This will be moved out of the corda-runtime-os repo at a later date.

Temporarily it also contains commits from runtime/os/5.1 branch.


update file structure

needs to apply to dragon6 modern stats api as well.
change the api source dir to src and the tests directory to tests and the resources can then house the nuget package.

Do you mean the project folders here? so 'DragonFruit.Six.API' and 'DragonFruit.Six.API.Tests'? If so couldn't this be moved into #209 / #210 ?

Do you mean the project folders here? so 'DragonFruit.Six.API' and 'DragonFruit.Six.API.Tests'? If so couldn't this be moved into #209 / #210 ?

well yeah, but it's a little late (i already went and renamed them all manually)

well yeah, but it's a little late (i already went and renamed them all manually)

Are we good to close this issue then or would you like to do it in the future?

Are we good to close this issue then or would you like to do it in the future?

would be nice for the future - so i suppose we should keep this open. I'll probably do this one anyways though

would be nice for the future - so i suppose we should keep this open. I'll probably do this one anyways though

Do you plan on doing this for the first 2021 update or later on?

Do you plan on doing this for the first 2021 update or later on?

probably target it for later on

probably target it for later on


Scene Text Transcription Implementation

This is an implementation of vitrivr reads. It allows Cineast to transcribe text from natural scenes using a two-step process. The text is detected using EAST and then the bounding box is cropped,  warped and processed to better suit the text recognizer.

Hey, thanks for opening that pull request. We generally don't merge anything directly into master but always work on the dev branch. I changed the base of the PR accordingly. This has caused a conflict in two of the files you changed, please have a look at them.

Hi, Sorry for the delay. I have made the requested changes by adding comments, removing the localhost files, and using logger.

@wolf-hash are you still working on this PR?

Closing for inactivity, can be reopened


Make App Responsive to work on Mobile Phones.

Currently app is clipped on mobile and toolbar is not great. I want a navbar holding all buttons and canvas to be aligned properly on mobile devices!

Can I work on this?

Can I work on this?

Sure, keep me updated.

I am not sure, I am finished yet, but it looks quite responsive. Can I have your feedback on the design? Should I host the branch I am working on, so it is easier for you to review the changes?

@Kyaw-Min-lwin Share the screenshot here. I will follow up👍

Ok @yashchaudhari008

Would like your feedback.

I am expecting something like this.
Also you can hide that instructions  "q and a to increase/decrease pen size" as they are mean only for PC users.

Ok, thanks this makes it quite clear.

Hey, how is this? @yashchaudhari008

Hey, how is this? @yashchaudhari008

Check https://www.w3schools.com/howto/howto_js_topnav_responsive.asp , can you make all buttons appear in dropdown when user click the button?

@yashchaudhari008  Should I make it so that all of the buttons, color selection box, and the pen size only show up when the user clicks a dropdown, or is it only for the buttons.

Only for buttons

Also make sure that footer is visible and canvas is properly scaled on mobile displays.

Also make sure that footer is visible and canvas is properly scaled on mobile displays.

The user needs to scroll down a little for them to see the footer.

@yashchaudhari008 is there anyone working on this other than me. Should I be worried about the other issues(others r working on) will it affect me. Sry this is my first time contributing to open source.

No need to worry.
Few Changes Required:-

Options button should hide on desktop.
Hovering over submenu should change background color (like buttons).
Window should fully fit on mobile screen(without any scrollbars).

I have hidden the options button desktop. I also added the hover(U need to either resize the browser or the inspect bar,  using the toggle inspect toolbar doesn't show you the hover even on the color select thing.) I will need help with the third one. I can't use css because the browser is automatically applying the height n width to the canvas. This is probably due to the js.

Okay, if first two changes are done you can send a PR so i can investigate further.


Find druggable site amino acid name and number

How can we retrieve the amino acids comprising the active site? Is there a method to obtain information on the amino acids in the binding site?
Thank you

Thank you very much for the question and your interest in using GrASP.
If you're using the production mode for the method, the probabilities can be retrieved from the output PDB files. They're output in the test metrics directory for the corresponding model (e.g., ./test_metrics/production/colors/train_full/trained_model_s_train_full_ag_multi_1680643832.8660116/cv_0/epoch_49/).


Storybook config environment variables are sent to the client

Storybook config variables also have the STORYBOOK_ prefix so they'll be sent to the client. Sources:

dev-server
static-build

Currently this is not so bad.
But we need to find a another prefix for above commands.

Fixed with #481


error when converting millisecond timestamps?

Hello,
I am using more and more this amazing package for data analysis!
I loaded a dataframe with spark_read_csv and the data looks like
# Source:   table<df> [?? x 32]
# Database: spark_connection
 NAME       Date
<chr>       <chr>
1  AA 2011-03-01T00:00:04.226Z

I want to convert the Date string to a timestamp, and change its timezone.
I am able to do use the Hive date functions (https://cwiki.apache.org/confluence/display/Hive/LanguageManual+UDF#LanguageManualUDF-DateFunctions) by typing
mutate( mytime = unix_timestamp(Date,  "yyyy-MM-dd'T'hh:mm:ss.SSS'Z'") ) that gives me back<PHONE_NUMBER>
However, I dont get how I can change the timezone or even get a Timestamp type back.
All of these solution below fail.
df %>%
  mutate(myminute = hour(mytime))

Error: org.apache.spark.sql.AnalysisException: cannot resolve 'hour(rxsbltsgvb.mytime)' due to data type mismatch: argument 1 requires timestamp type, however, 'rxsbltsgvb.mytime' is of bigint type.; line 1 pos 296;
df %>%
  mutate(myminute = from_utc_timestamp(mytime, 'UTC'))

df %>%
  mutate(myminute = as.POSIXct(mytime))

This is puzzling. Indeed,  from_utc_timestamp should accept primitive types such as BigInt
Any ideas? Being able to process timestamps in Spark is extremely important to me.
Many thanks!

@kevinushey @kevinykuo sorry to bring you in, but I believe this is an important issue for everyone involved with big datasets and timestamp variables. Could you please have a quick look an tell me if something is wrong here? Why can't I call these date functions?
Thanks!!!

Although I don't quite know the underlying reason why this isn't working as-is, it looks like you can explicitly convert arguments to timestamps and then construct using that, e.g.
library(dplyr)
library(sparklyr)
sc <- spark_connect(master = "local", version = "2.1.0")

df <- data.frame(dtime = as.numeric(Sys.time()))
tbl <- copy_to(sc, df, overwrite = TRUE)
tbl %>%
  mutate(time = from_utc_timestamp(timestamp(dtime), "UTC"))

Is that sufficient for you here?

switched regexp_extract with regexp_replace. Still does not work...

It looks like backslashes need to be double-escaped (not yet sure why), e.g.
tbl %>%
  mutate(time = from_utc_timestamp(timestamp(dtime), "UTC")) %>%
  mutate(chartime = as.character(time)) %>%
  mutate(year = regexp_extract(chartime, '(\\\\d+)'))

@kevinushey: i see you are using the timestamp function.. where is the doc for that? it doesn't seem to be listed in the Hive UDF docs and it also seems the only way to convert a string in the  "YYYY-MM-DD HH:MM:SS.fffffffff" format to an actual TimestampType

@alexanderchernyakovgithub see https://cwiki.apache.org/confluence/display/Hive/LanguageManual+UDF

@kevinykuo already saw this page: timestamp isn't a function listed on there unless i'm being terribly dense

@alexanderchernyakovgithub yeah doesn't seem like that function is documented, my guess is that it's SQL for casting to timestamp. BTW what's the string that's giving you trouble? Have you tried using the format string "yyyy-MM-dd HH:MM:SS"?

@kevinykuo  yes, i think you are right.. the way to convert back to strings seems to be string() which is also undocumented. i think what must be happening internally is that they are casting. thanks for taking a look. i wonder where @kevinushey knew to use the function from

Good question! I can't remember where I found it; but I do recall seeing it in some of Spark's own code, e.g. https://github.com/apache/spark/blob/7f2e62ee6b9d1f32772a18d626fb9fd907aa7733/sql/core/src/test/resources/sql-tests/inputs/inline-table.sql#L50-L51


Move apollo to deveps

Rather than use npx to execute apollo, I've added it to the dev
dependencies. Why? Uh, something about caching? Preference? Standalone-ness?
Version predictability? Something like that.

Add apollo to deveDependencies
Update generate script to use local apollo

@edd We had to end up using rest only for this so we can just remove all the apollo related things


WARNING issue.

Hello, I frequently get the error message below. Suppressing by python error handler doesnt help. Why do you think this faults happening? They interrupt outputs on the terminal.
WARNING: .\zbar\decoder\pdf417.c:89: : Assertion "g[0] >= 0 && g[1] >= 0 && g[2] >= 0" failed.
dir=0 sig=2768 k=6 g0=ffffffff g1=fc8 g2=134 buf[0000]=

Thanks a lot, that solved my problem.


WebApp - QR code of the receiving address is lost

Describe the bug
QR code of the receiving address is lost
To reproduce the bug
Steps :

Go to the Your address screen
Leave the QR screen on for a while
Observe the displayed screen

Actual

Expect : Show QR code

@dungnguyen-art Please check on both Extension and WebApp

I installed the QrCode library for testing, but I can't reproduce the case where the QR code shows as an empty blank. It might be an error with the library.
@haiyenvu23 @Sokol142196


Save -> Save... ; Save -> Review and save

Make it clear that clicking on the Save button doesn't actually save the changes immediately and that the user will be able to review them, as stated in ### Saving Edits

I prefer "Upload..." and "Review your changes and upload them to" ... and avoiding save as that may mean local/private Save.

Since we just changed this recently in #3777, I'd prefer not to keep changing the save language unless we notice an uptick in users that find it confusing.  (People have complained before when we change too many things too quickly, because they need to update translations and training materials).
Also, I agree conceptually with ellipses on the save button (as this usually signals another step will follow) but I'm worried about making the Save caption any longer because of issues raised in #3803, so I want to leave it how it is for now.
One frustrating thing about Transifex is that translators can't easily preview how their translated strings will look in iD, so I could imagine a situation where somebody adds the ellipses and then that makes the button caption look weird, but we don't find out about it until next release.
Thanks for the suggestions though!  Keep them coming!

Couldn't at least the tooltip be changed? Maybe to "Review your changes and upload them to" like suggested by manfredbrandl.
That shouldn't require a documentation change and would improve consistency (since the next button is "Upload", not "Save" anymore).

Couldn't at least the tooltip be changed? Maybe to "Review your changes and upload them to" like suggested by manfredbrandl.

Ok, yes, we can change the tooltip.


Bugfix: admin users ordered by full name

#170

This is difficult to test without a better understanding of how administrate works under the hood.
With normal controller actions, typically an instance variable is assigned to store the contents of what's being returned/rendered - in this case, administrate is using a slightly different approach and there doesn't seem to be a variable available to check.
It's also doing some special handling with the view contents (eg. response.body is returning an empty string).
I agree that tests should be written, but it's not obvious to me how to do so.
One thing of note - there are some existing controller specs for admin pages (ie. reports_controller_spec) that are incorrect (sadly, I wrote that one). The controller specs need to be sure to sign in an admin user before testing behavior or the test will get redirected before the controller code ever runs.

We did add specs, but they don't test the changes that were made in this PR, they are basically just smoke tests. 😄

Yeah, really the only change was ordered. We could assert position of users in the response.body, but I'm not that concerned about that. It's most important that the code is executed since it depends on internal code from administrate.

Yea, that's the weird thing... respond.body was '' when we were attempting to add the specs. Now that I think about it, maybe that was indicative of something else.

Controller specs don't render views by default. I generally prefer request specs, partially for this reason, but again, I was mainly concerned that the code was executed. :smile:

Oh interesting haha. The main app I work on does render controller views - must be configured as such. Good to know!


⚠ Bofrost.de has degraded performance

In 8deef4a, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1542 ms

Resolved: Bofrost.de performance has improved in b6692b2 after 34 minutes.


No Output on Spyder iPython Console

Spyder version information

Spyder version: 5.1.1 None
Python version: 3.7.9 64-bit
Qt version: 5.12.10
PyQt5 version: 5.12.3
Operating System: Windows 10

a. I am not getting an output, see what I am getting:
runfile('C:/Users/Akan/Exercises/untitled0.py', wdir='C:/Users/Akan/Exercises')
b. What should I do to correct this?

Hey @Imoh-mfon, you need to evaluate your function if you want to see any output in the console.
In other words, you need to add to your code
hello()


Emphasize the age of the Nodejitsu blog post and point to GitHub prominently

The Nodejistu blog post http://blog.nodejitsu.com/keep-a-nodejs-server-up-with-forever has been widely used on Stack Overflow and elsewhere to point to forver (instead of this GitHub page).
The problem is the blog post is really old and doesn't mention handy options like restart. Also, the GitHub links point to the indexzero repo.
It would be great to either update the blog post, or at least mention prominently at the top that the reference material is GitHub, and that the post was written some 2-3 years ago.

This did get updated.


📎 Rename biome_js_unicode_table to biome_unicode_table

Description
Since we use the crate in json and css crates it's better to make it more generic.
https://github.com/biomejs/biome/pull/1288#discussion_r1434848662

I'd like to work on this issue.


In-game console not printing properly

When in game, using any /luckperms command return '[L' per line for however many lines ought to be printed.  When I run the command in console, the command runs like normal.  I'm running on FTB Beyond 1.3.0 server with spongeforge 5.2.0 and minecraft 1.10.2

Which LP version?

filename: LuckPerms-Sponge-3.0.71.jar
it looks like the most recent version is .74, will update and and check to see if the problem persists.

updated and same problem persists.
list of sponge plugins:

SpongeForge v2254-5.2.0-BETA-2262
Nucleus v 0.27.0
Nucleus Mixins v 0.25.2
WorldEdit forge v mc1.10.2-6.1.4-dist
FAWE v (unknown)

I've not changed the modpack in any other way and am updating sponge to see if that helps

Updating sponge had no effect on the issue.

Running:

Forge 1.10.2-<IP_ADDRESS>4
SpongeForge 1.10.2-2254-5.2.0-BETA-2262
LP 3.0.74

So, by guess is it's an issue with your client or another plugin/mod which is messing with packets.

Ok, will do a check to see if i can figure it out and let you know what I find out. (time to start removing plugings)

Found it out.  It's a mod in the pack called ChatFlow that was reading [LP] as a regex expression and deleting everything that came after i guess, I'm looking into changing the config to make it work.  (This will be interesting because I've never used regex).
Am closing the issue regardless, as it's not a bug.


[BUG] qualification_summary.csv is absent when an app is processed but does not fit qualification criteria

Describe the bug
Take an app which is all RDDs , no dataframes. Such an app will be processed and SQL duration shows up as zero in the verbose output, which is expected.
However the qual tool output directory does not contain a qualification_summary.csv file for this run eventhough it was not skipped/failed or not applicable. It is easy to lose track of such apps when a large # of apps are being qualified.
Steps/Code to reproduce bug
Described above
Expected behavior
A successful run should generate the standard set of files we look at for final results, qualification_summary.csv being one of them.
Environment details (please complete the following information)
onprem HDFS

I think we can close this for now. Writing the core output files to local files has been fixed in https://github.com/NVIDIA/spark-rapids-tools/pull/1308


Distance calculator

This function will calculate the distance of obstacles (in cms) by taking the sensor data as input.

Hello! Can I work on this issue? I have a small doubt regarding the issue mentioned here.  In the code, there is only one function for both reading the sensor data and calculating the distance.  But there are 2 separate issues allotted. Shall I store sensor data and calculate distance in the same function double calcDistance

Hello, Can you please assign this task to me so I can contribute to this project? Thank you

Hi @deeppss, it would be better to have 2 different functions. As it is possible that someone can use IR sensor to get sensor data, that's why we have 2 separate functions.
I have assigned you the issue.
All the best.

Hey guys, @deeppss and @iamanshulgit, you both can work on this issue and raise a PR.


[multistage] Projection Not Pushed Down for Agg Queries with No Grouping Sets

Repro:
SELECT
  COUNT(*) -- the issue happens even if you have COUNT(A.playerID)
FROM
  baseballStats_OFFLINE AS A
  JOIN baseballStats_OFFLINE AS B ON A.playerID = B.playerID
WHERE
  A.hits > 10 AND B.hits < 5

The issue happens even if you use a sub-query:
SELECT
  COUNT(*)
FROM
  (
    SELECT
      A.playerID
    FROM
      baseballStats_OFFLINE AS A
      JOIN baseballStats_OFFLINE AS B ON A.playerID = B.playerID
    WHERE
      A.hits > 10
      AND B.hits < 5
  )

For each of the queries above, they will end up reading all the columns in the table-scan stage. The reason is that there's no projection node created by Calcite.
I am testing out a few approaches for a fix in this PR: https://github.com/apache/pinot/pull/10122
Btw, this is what GPT recommends:
cc: @walterddr

sounds like a good idea. :-P

we can leverage calcite's AggregateExtractProjectRule and split the aggregate with aggregate + project.
this only needed for count(*), i realized that sum and others are properly putting the project in place

did a bit of research on this topic:
the problem only occurs on COUNT(*) or COUNT(1), COUNT(col) this is b/c calcite will optimize out the argument since all columns in pinot are marked non-nullable.

several attempts have been tried on this one but the most promising so far is to replace COUNT with SUM(1) thus bypass calcite's optimization. with this project pushdown are being executed properly.
@ankitsultana also reported that the pushdown doesn't apply to semi JOINs with NOT-IN clause properly

SELECT
 SUM(1)
FROM
 baseballStats_OFFLINE
WHERE
 hits > 10 AND
 playerID NOT IN (SELECT playerID FROM baseballStats_OFFLINE WHERE hits < 5)

^ this seems to be related to the fact that NOT IN produces a in-equality JOIN condition that cannot be optimized into a proper JOIN and thus cannot apply proper pushdown rules.
which we will address separately.

@walterddr : Can we consider using RelFieldTrimmer?
I raised this: https://github.com/apache/pinot/pull/10156
It seems to work and I am testing it out on our clusters. The PR is not final as I am also reading more about this feature. There are some caveats called out in the code also regarding this.

is this issue fixed by #10187 ?

Yes I think we can close this now.


Port Bottle web server to Python 3

This could be challenging!
https://github.com/bottlepy/bottle
We want to convert the Bottle server and the WSGI server adapter to python 3 compatible. Bottle itself is OK but there may be more complications.

Preliminary testing has shown that the bottle server will work as-is.


'v3_lora_sd15_adapter.ckpt' contains no temporal keys; it is not a valid motion LoRA!

I need help please.I can't solve this problem

It is an SD Lora, not a motion Lora. Load it like a normal Lora.


Remove incompatible python version, add newer

Coverage decreased (-0.007%) to 64.067% when pulling 9802ead48be2a671d1c9b840acd0cc75873d70c3 on AlanCoding:fix_old_python into 81492750c7332e494d96ceb85736a2b75bc155d4 on ansible:master.


Features/quickstart

@cosmos-explorer  can you review my PR and make your comment? If nothing to correct, can you approve my PR?


Automotive needs temperature (CC work, should be copiable from Hospitality demo)

Basically you need to copy the elements from Hospitality AdjustTemperature Command into the Automotive AdjustTemperature command. This then needs to be tested.


Quit prompt

Since on QWERTY keyboards 'Q' and 'W' are next to each other and most of text editors have this feature, I thought it would have been nice to add a prompt to ask the user if he wants to save changes (or abort) on quit command.
I've implemented a variable unsaved_changes which becomes 1 after making any change. If the edit command ('e') is performed, the variable is set to 1, only if the line content has actually changed.
The variable is reset to 0 after performing save command ('w')
On quit the user is prompted, in case the variable has value 1, to save changes before exit or not. The user can also cancel the quit action.
I hope this change looks useful to you.

Awesome. was planning on adding this at some point so thanks for getting it done lol


Suggestion for WITH

Hello Brook,
Going through the pattern matching module, it seems like WITH could have its own separate lesson.
Something to consider, since it introduces an entire new way to do things in Elixir.

Great minds! I've already split this content.


Empty flash when record has errors on delete

Hi,
I'm trying display the record errors on index after destroy it, but it didn't work. The object has errors (from restrict_with_error), but after respond_with @record, my app redirects to index (because I'm using the CollectionResponder) and nothing happens. I had put a binding.pry on view to inspect the flash and it is empty.
Does anyone knows why is it happening? I had do many tests and I can't find the problem with my code.

If it render the view instead of redirect to index I can get the errors, like the create action. I tried remove the CollectionResponder, but the controller redirects to show instead of index only.

@thiagogsr I think that the FlashResponder is using the flash_now behavior as the action is failing. Can you try to pass flash_now: false on your respond_with call to see if it works as expected?

Yes, I tried it but the behavior was same

@thiagogsr damn =/ Can you write up a sample app that reproduces the error?

Sure, I will prepare it and send here

Done! https://github.com/thiagogsr/responders-restrict

@thiagogsr I checked your app and responders is working correctly. What's missing is that the scaffolded views don't add the :alert flash message to the markup - failure scenarios end up using the alert key instead of notice. Maybe your app has the same issue of not showing the alert flashes?
Here is the change I made to your sample app to display the flash message:
diff --git a/app/views/categories/show.html.erb b/app/views/categories/show.html.erb
index 3ece752..8169651 100644
--- a/app/views/categories/show.html.erb
+++ b/app/views/categories/show.html.erb
@@ -1,4 +1,5 @@

<%= notice %>

+
<%= alert %>

   Title:

Yes, it works.  The problem for me is: I have an API.
# Api
Category.find(n).destroy # false; Returns a json with the error;

There is a middleware adding the error to client model (and it works for create, update). The model has errors on my controller.
# Client
def destroy
  @category = Category.find(n)
  @category.destroy # calls the api. @category.errors is present
  respond_with @category
end

I tried comment the flash.actions.destroy.alert to see if it works, but didn't work. The app doesn't render the view like create or update, it always redirects to a view (show or index - if CollectionResponder is enabled), therefore the @category.errors is empty. Need I populate the alert manually for this case?

therefore the @category.errors is empty. Need I populate the alert manually for this case?

Probably, if your Category#destroy method doesn't work similar to how restrict_with_error works on Active Record by marking the record with errors. I'm closing this one as it doesn't seems to be an issue with responders, but if you manage to isolate the real issue from your app into a fresh repo I can give it a look to see if there's anything else you can do besides handling the flash.alert manually.


Update to Spring Boot 3.3.0

Update to Spring Boot 3.3.0

fine to make one for spring-boot only, but I tend to bump all deps before a patch release. If you are up to it, can you audit? Otherwise, I'll do post merge

Sure, will do!

@codefromthecrypt we should be all set, thanks for looking!

thanks a lot for the help, I'll cut a patch now!

actually I will bump alpine first


Enabling web support fails - git: fatal: 'origin/beta' is not a commit...

As the title says I am attempting to enable web support for Flutter (as per https://flutter.dev/docs/get-started/web) however the following error occurs at step 1:
➜  flutter git:(stable) flutter channel beta
Switching to flutter channel 'beta'...
git: fatal: 'origin/beta' is not a commit and a branch 'beta' cannot be created from it
Switching channels failed with error code 128.

I confirmed the origin is set correctly:
➜  flutter git:(stable) git remote get-url origin
https://github.com/flutter/flutter.git

then tried the following (as per https://github.com/flutter/flutter/issues/53231):
➜  flutter git:(stable) flutter channel master
Switching to flutter channel 'master'...
git: fatal: 'origin/master' is not a commit and a branch 'master' cannot be created from it
Switching channels failed with error code 128.
➜  flutter git:(stable) flutter upgrade --force
Flutter is already up to date on channel stable
Flutter 1.20.2 • channel stable • https://github.com/flutter/flutter.git
Framework • revision bbfbf1770c (13 days ago) • 2020-08-13 08:33:09 -0700
Engine • revision 9d5b21729f
Tools • Dart 2.9.1

Output of flutter doctor -v:
➜  flutter git:(stable) flutter doctor -v
[✓] Flutter (Channel stable, 1.20.2, on Mac OS X 10.15.6 19G2021, locale en-AT)
    • Flutter version 1.20.2 at /Users/samscholefield/_Flutter/flutter
    • Framework revision bbfbf1770c (13 days ago), 2020-08-13 08:33:09 -0700
    • Engine revision 9d5b21729f
    • Dart version 2.9.1

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/samscholefield/Library/Android/sdk
    • Platform android-29, build-tools 29.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.0)
    • Xcode at /Users/samscholefield/_X-code/Xcode-beta.app/Contents/Developer
    • Xcode 12.0, Build version 12A8189h
    • CocoaPods version 1.9.3

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 48.1.2
    • Dart plugin version 193.7547
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.48.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.13.2

[!] Connected device
    ! No devices available

! Doctor found issues in 1 category.

If flutter channel beta isnt working then try reinstalling flutter from flutter.dev

I’m unable to reproduce the issue to pinpoint the cause, the origin is correctly present. The issue needs further investigation.

Hello! Sorry to waste your time but I will close this issue as I have now resolved it.
Initially I used the download zip available here: https://flutter.dev/docs/get-started/install/macos but I now realise this really does only contain the stable channel, reinstalling using  git clone https://github.com/flutter/flutter.git fixed the issue for me.
Again, sorry for the time you've spent considering this issue report! Might be worth updating the docs to state something along the lines of 'if you plan to enable web development please clone using the following link' or something similar so that slightly less bright people, such as myself, don't go for the easy option!
Many thanks,
Sam


All .js files open with jsx grammar

All my .js files open with jsx grammar by default.
Not sure if this is by design but it breaks things for me and I think we should leave .js as they are unless you detect react components inside the source.

Hi! It's originally by design but I think with the new detection functionality we can remove .js from the extensions list.

Awesome, thanks.

atom-react isn't being used for my .js files ...  Instead its using the builtin babel functionality.  It seems like atom-react's functionality is better(?) ...
Is there an up-to-date best-practice on which extension to use for folks using react, webpack, and babel?


GraphQL response options only visible when a response is shown

GraphQL response options (copy response and download response JSON) is only visible when a response is available.

Hey @AndrewBastin,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: d3f01440-a6d6-11ea-9311-0306711df575

Hey @AndrewBastin just want to mention that graphql dependency has a v15.0.0 update which I think is not stable. And also, the whole new monorepo Graphiql would be our best bet. Graphiql does have some issues with validation language services - I bumped the version but it seems graphql v15. 0.0 support isn't stable.
Just fyi.

@liyasthomas thanks for the heads up. Can you elaborate on the relevant issues ?
We are using graphql dependency solely for language services, which should be powering any GraphQL based tool.
If it is a problem, we can push down the version.

The issue is with graplql language validation import. I'm checking what went wrong. I'll update once it's resolved.


More logs to show test progress

At the moment it's hard to tell which hunters were run. It would be good if there were a log for each hunter that's triggered to indicate what it's attempting, and whether it succeeded or failed.

fixed with #50


didn't work for me for gmail, outlook

I can only get the calendar names when the plugin runs and no events appear.
also, the icon is quite similar to journals calendar plugin. it will be good to have something that is easily distinguishable.

Can you make your you actually ahve tasks for the day? And also that the events aren't recurring.
On 25 Feb 2022, 5:20 AM +0400, ashishbhateja @.***>, wrote:

I can only get the calendar names when the plugin runs and no events appear.
also, the icon is quite similar to journals calendar plugin. it will be good to have something that is easily distinguishable.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Thank you for the revert.  And also that the events aren't recurring.
On 25 Feb 2022, 5:20 AM +0400, ashishbhateja @.***>, wrote:

I can only get the calendar names when the plugin runs and no events appear.
also, the icon is quite similar to journals calendar plugin. it will be good to have something that is easily distinguishable.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.Message ID: @.***>

Can you take a screenshot of your console? Using command option I
On 25 Feb 2022, 7:28 AM +0400, ashishbhateja @.***>, wrote:

Thank you for the revert.  And also that the events aren't recurring.
On 25 Feb 2022, 5:20 AM +0400, ashishbhateja @.***>, wrote:

I can only get the calendar names when the plugin runs and no events appear.
also, the icon is quite similar to journals calendar plugin. it will be good to have something that is easily distinguishable.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.Message ID: @.>
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.>

Could you click the top right corner of this dev tools window and click the console button there. Then try fetching tasks rn monitor for what's printed. This will really help me debug the issue

Could you try the latest release?

Regarding the new icon, do you have any suggestions from this website?
https://tablericons.com

Thank you! It works well, and it is amazing to see how fast it is. I hope you can find a way to include the instance of a recurring task as well.
The calendar icons appear similar to each other. How about using box-multiple-1 or fold-down?


Hide "History location" as default

as default view could we show only the devices that are in the list at the right (Dashboard)?
And in the filter we can have the option:

"Show historical location data"
(As currently it can be a bit messy on the map)

Location history is tracked here: https://github.com/NordicPlayground/thingy-rocks-dashboard-aws-js/issues/34 ... please elaborate there a bit more how you would like that feature to work.

as default view could we show only the devices that are in the list at the right (Dashboard)?

By default all devices are shown. I guess you mean that if "show only favorited devices" is enabled, the map should also only show these locations?


User Roles and Permissions

Description
Define a set of roles for users that can restrict the dataset manipulation. Some examples could be:

viewer: only can explore the records. Explore the dataset in a read-only mode
annotator: a role for annotator users. Can create labeling rules, change record annotations
reviewer: for advanced annotation flows. This role can resolve conflicts in the annotation agreement flow ( MUST BE DEFINED)
admin: can change settings, annotate, delete.

We can probably close this issue, as we have most of the roles specified here. Do you want to keep it open @dvsrepo ?


Replace consoleLogger and log solutions with debug

The main purpose of this pull request is to change the logging mechanism from scratch.
DEBUG=* npm start will log all of the components to the terminal with different colors.
Example:

PS: Have to bump version in order to accept this PR.

+1 on this!
you rock man!

Would like to see this instead of the logging built in, but wouldn't it make more sense to have the debug tokens be prepended with something unique to botkit? botkit:core:bot:debug?
Just so when using the lib with other debugable libs you have the ability to switch. Express for example is prepended with express:*, socket.io, nodemon, etc., all do the same.
... or is this PR abandoned in favour of something else that came afterward.

You're right @derrickpelletier 👍
This is an abandoned PR because none of the other contributors seemed interested on the topic of changing something that's been used in a very long time.

I'd love to re-address this later. The logging is very inconsistent and not great!


Automate verification of prod deployment sha1 (replace the API-based version check with a more reliable taskdef query)

This applies to both portal and explorer.

Hey team! Please add your planning poker estimate with Zenhub @tihuan @ebezzi

Note: sometimes our API endpoint to serve the sha returns nothing due to configuration changes in happy variables. Might be worth doing the task def check as a fallback.

I believe you'd want to do something like this:
    - name: Verify expected SHA1
      run: |
          if [ "${DEPLOYMENT_STAGE}" == "stage" ]; then
            API_URL=https://api.cellxgene.staging.single-cell.czi.technology/dp/v1/deployed_version
          elif [ "${DEPLOYMENT_STAGE}" == "prod" ]; then
            API_URL=https://api.cellxgene.cziscience.com/dp/v1/deployed_version
          else
            API_URL=https://api.cellxgene.${DEPLOYMENT_STAGE}.single-cell.czi.technology/dp/v1/deployed_version
          fi
        API_RESPONSE=$(curl -s $API_URL)
        API_SHA=$(echo "$API_RESPONSE" | jq -r '.["Data Portal"]')
        if [ "$API_SHA" == "${{ secrets.SHA1 }}" ]; then
          # pass the check
        else
          # fail
        fi

A few notes:

secrets.SHA1 should be updated to be the actual GHA variable we use for the docker tag SHA1. I just guessed that, no clue if that is actually what works
Figure out how to raise a failure in the check? Not sure what the typical expected behavior is here
Determine how to handle the case of the API endpoint returning nothing
This hasn't actually been tested =p


simplify_if

Coverage remained the same at 95.041% when pulling 0b496ea254fcee63acecbb8ac9971f3792d47161 on XenoAmess:simplify_if into 3d4ed4a8ac63db1e51601ffc31fed44dccbb276c on apache:master.


Shell scripts for bottling

Opening this as a “to do” item. Four shell scripts that I use to help in mass bottling:

nextbottles takes the unbottled formulas in order most-downloaded first, and filters to keep only those that are ready for bottling (all dependencies are already bottled)
allbottles takes the unbottled  formulas in order most-downloaded first, and for each tells you whether it can be bottled, or not
whatsmissing takes one formula as argument, and tell you whether it can be bottled or if (and what) unbottled dependencies should be bottled first. The search is not recursive.
botstats gives us the number of bottles formulas, and the total number of bottles. Good for motivation!

What is not handled:

head-only dependencies
linux-only dependencies

mostmissing tells you, for each unbottled formula, how many other formulas depend on it (not recursively). If you sort the output, it will show you the largest “blockers” in the dependency graph. It's actually a good way to prioritise at the beginning of the mass-bottling effort.

Review period ended.

I've pushed brew unbottled here which should address all these use cases.

Awesome @MikeMcQuaid, thank you so much!

@fxcoudert, why did you close it?

Ouch, huge misclick. Sorry! And I can't reopen or undelete the branch :(

You don't see it here https://github.com/fxcoudert/brew/branches ?

@maxim-belkin no it's a double misclick, somehow by reflex when presented with the "delete branch" button I removed it as well.
(If you pulled Mike's commit locally, git reflog would've save you too).

hub pr checkout didn't work (it fails to get the branch, which doesn't exist anymore)
So new PR: https://github.com/Homebrew/brew/pull/9300

🤷 gh pr checkout 9162 worked for me :)

Opened https://github.com/Homebrew/brew/pull/9308 with fixes and will merge today.


Exchange 2016 - Archiving issue

Hello,
When MBx and Archive Mbx are on different servers same dag, All archive mails are moved to the dumpster of the archives Mbx. I need MFCMapi to find them.
I can see the Prt_Retention flag on the MBx - Policy is working fine when both MBx and Archive MBx are on same server (different DB also)
Any hint where where i can look into?
Thx.

@Systech2017,
The issues section of this repository is intended for product documentation issues only.
Windows 10 Support
The issues section of this repository is intended for defects in the documentation itself.
Please consider opening a product support ticket by the following link below for your problem so that it gets addressed quickly.
Thank you.


Rename extension, update assets & documentation

Hi all,
I hope today finds you well! This issue is being opened to ask if you would be open to renaming your extension on GitHub to reference Camunda Platform 7 rather than Camunda BPM after its release on April 12th, 2022. (For example: camunda-platform-7-graphql) This change will also likely impact any screenshots you have in the README that reference Camunda BPM or previous versions of Camunda Platform, and this will likely need to also be updated throughout the repo's documentation as a whole. You can also make use of Shields.io badges to indicate which version of Camunda your extension is compatible with.
While it is not a requirement that you rename your extension to align with Camunda Platform 7, we would greatly appreciate it if you could make these changes, and merge them anytime after April 12th, 2022.
if you have any questions or require any assistance, please do not hesitate to reach out to @camunda-community-hub/devrel! :)

@jangalinski @saig0 is this something you are interested in updating?

I renamed the repo and added the C7 badge. Nothing more I like to do as I abandoned this project (for good reasons, see note in README) ... I suppose if no maintainer shows up in the next weeks/months you should archive this.
Is was a great project, it has lots of stars ... but it uses outdated and unmaintained libraries to function ... I can't keep up with this (and use spring boot eventing myself ...)

Thanks a lot for renaming - I will go ahead and archive it. It had its time :-)


migrate the Kubernetes plugin to mui 5

Hey, I just made a Pull Request!
Migrate the Kubernetes plugin to mui 5.
The  reason for using import Grid from '@mui/material/Unstable_Grid2' is to reduce tech debt in the future, in mui 6 there is the following warning:

The Grid component has been deprecated. Please use Grid v2 instead.

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

@mclarke47 @Rugvip I can't add reviewers so I'm tagging you in a comment as I know you are reviewing the K8s plugin PRs :)

Hi @sebalaini, given this https://github.com/backstage/backstage/issues/27726, I'm wondering if we should hold off on this for now.

hey @awanlin sure :) I didn't know about that RFC, though I wonder if that RFC should be mentioned in the documentation https://backstage.io/docs/tutorials/migrate-to-mui5 so that people can either migrate or wait to see what would be the decision :)

hey @awanlin sure :) I didn't know about that RFC, though I wonder if that RFC should be mentioned in the documentation https://backstage.io/docs/tutorials/migrate-to-mui5 so that people can either migrate or wait to see what would be the decision :)

No worries, I mean the RFC came after you had submitted the PR, just wanted you to be aware. Your suggestion about a link in the migration guide makes a lot of sense, can you run with that?

hey @awanlin sorry for the delay, do you want me to add a link to the RFC in the doc and open a PR?

@LegendSebastianoL see https://github.com/backstage/backstage/pull/27849

@LegendSebastianoL see #27849

Noted :)


TRAINING savings on minimal LOSS for better LORA

Hi, Genious Kijai!
Why not to save lora every new minimal value? Or several last minimals?
If its not possible with every step, can minimal loss be taken every epoch? And saved minimal?

Thanx for your answer!

Hey, I've actually thought of this before, I just don't know yet what would be a good rule to do that. Technically it's simple to implement saving based on the loss values.

It was a simple task for a first sight. I making research hwo to do it better. But for SURE it's better to chage your vay to draw everaged LOSS. This is how TensorFlow make it:
def smooth2(scalars, weight):
    """
    EMA implementation according to
    https://github.com/tensorflow/tensorboard/blob/34877f15153e1a2087316b9952c931807a122aa7/tensorboard/components/vz_line_chart2/line-chart.ts#L699
    """
    last = 0
    smoothed = []
    num_acc = 0
    for next_val in scalars:
        last = last * weight + (1 - weight) * next_val
        num_acc += 1
        # de-bias
        debias_weight = 1
        if weight != 1:
            debias_weight = 1 - math.pow(weight, num_acc)
        smoothed_val = last / debias_weight
        smoothed.append(smoothed_val)

    return smoothed

and this is without weight (previous tends to it with weight ->1)
loss_cumsum=np.cumsum(loss)/np.arange(1, len(loss)+1)
cumsum - green
smooth2 - darkblue
cyan - right way (without shift) or yours loss_conv = np.convolve(loss1, np.ones(window_size) / window_size, mode='full')

Yeah I'd like to improve the loss graphs, problem just had been I have no previous training experience so I don't really know this stuff, multiple graphs in one seems like a good idea indeed.

I'm not a python coder, but I have an idea in my mind how to make that. The node "Display Anything" from rgthree, outputs numbers from the "Loss_List" in the "Visualize Loss" node you have. I actually use those numbers, display them in Excel and sort to get the less loss save. But it reports it on every single step from every single loop. Which is not a bad thing...
In my mind there are a few ways to tackle this.

Every time it saves a loop, it saves the last number in the "Visualize Loss" list (this would be the one it just did) and appends that number to end of the name in that file.  --  The reasons would be we can visually see which one of the saves has the lowest number.
You could kind of automate it by having it make a save every single step it makes, but only keeps the lowest value it comes across. You could add a "keep the lowest 5" so it will always over-right the highest numbered file in the save folder. But remember the lowest number it comes across - when Visualize Loss spits out a lower number than that it remembers it saves that file over the highest number in the save folder.
So in the save folder it looks like this
0.42054999.safetensors
0.42091628.safetensors
0.40922244.safetensors
0.40924419.safetensors
0.40749725.safetensors

Lets say it does a step and produce a 0.49100515 - It doesn't save it. But the next step it produces a 0.4010055, It saves that that one and deleted 0.42091628.safetensors.
This way it doesn't really matter how many step, it's more about how much time you want to let it run. Since it's only going keep 5 files you can just have it loop forever, until the user cancels the job.
But yeah I'm not a coder, I think the quickest dirties way is to simple include that the last number from the Loss_list when saving a state into the file name. That way the users can just visually see which one of the save sates has the lowest number of loss.


NonlinearMinimizerBase and derived classes: static properties overwritten by instances

The class NonlinearMinimizerBase has various static properties that are initialized/overwritten from instance constructors and methods. This prevents concurrent usage of any of the derived algorithms in parallel and is a very confusing API. E.g. the following does NOT work, the two calls to FindMinimum interfere with each other, and initialMu is also overwritten.
var lm1 = new LevenbergMarquardtMinimizer(initialMu: 0.01);
var lm2 = new LevenbergMarquardtMinimizer(initialMu: 0.02);
var model = ObjectiveFunction.NonlinearModel(..., x, y);

var scales1 = new[] {.1, 1, 10, 10};
var scales2 = new[] {.1, 1, 10, 10, 10, 10};
var start1 = new[] {1.0, 0.0, 0.0, 0.0};
var start2 = new[] {1.0, 0.0, 0.0, 0.0, 0.0, 0.0};

Task.WaitAll(
   Task.Run(() => lm1.FindMinimum(model, start1, null, null, scales1)),
   Task.Run(() => lm2.FindMinimum(model, start2, null, null, scales2))
);

Commit 19bd5f576da4c75d0c1a088efc29d9ca9cee4356 introduced these changes and I don't understand why this was designed as it is.

Good point. I started working with Math.Lab and have like to fit e.g. >1.500 curves as parallel as possible. Therefore the library has to be thread safe ...
But I can find only questions about thread safety, no answers (only for Fit.Line from 2016)?

They are not necessarily about thread safety, e.g., nested optimizations in total least squares regression (of course you can work around this by doing a save and load of the variables). What is the rationale behind using static variables?
Commit 19bd5f5 was a refactor. The API has been like this since the beginning in commit aca34e13.

These properties are no longer static, starting with upcoming v5. Thanks for bringing this up, and thanks @diluculo for the PR!

I ended up making a similar fix on our local copy a few weeks ago, is there an estimation on when v5 is coming?

No, but there are pre-release packages already available on NuGet.


National halving time for UK appears incompatible with regional ones?

https://epiforecasts.io/covid/posts/national/united-kingdom/ shows regional halving times all -29 or higher, but national halving time is shown as -72 (-460 – -39)
I haven't checked the calculations in the code, but that appears to suggest the two are calculated in different ways?

This also seems to be true of the Effective reproduction no (r) - the range estimate for the UK is 1.1 (range 1.1 - 1.2) where as HIGHEST of the sub-regions are estimated to have 1.0. It would be good to explain this discrepancy to avoid undermining confidence in the estimates.
This is particuarly relevant as it means the country as a whole is shown as 'Increasing' whereas none of the regions are.

Hi,
Thanks for opening this issue and sorry for the delayed response. The difference is due to the differences in the data sources we are using. In the latest version of the estimates these issues should hopefully be mitigated so closing this for now.


start-tc-server.sh does not accurately indicate server termination

The code in start-tc-server.sh relies on the shell wait command to coordinate exit of the script with server termination.  Unfortunately, a TERM signal to the start-tc-server.sh process causes the wait to terminate early (while properly propagating TERM to the background server).  (From the Bash documentation ... "When Bash is waiting for an asynchronous command via the wait builtin, the reception of a signal for which a trap has been set will cause the wait builtin to return immediately with an exit status greater than 128, immediately after which the trap is executed.")  So, although server process termination is initiated, it is not complete when the wait completes and, in addition, the wait exit code does not reflect the exit code of the server process.
The sample Bash code below can be used to demonstrate both the interrupted wait and one possible way of correcting the issue.  The sample solution depends on proper operation of kill -0 $PID -- while this is part of the POSIX standard *NIX, there may be some systems for which this does not operation properly (see "The Zero Signal" in http://www.linux.org/threads/kill-signals-and-commands-revised.8096/).    It may be possible to avoid the use of kill -0 by relying on retrying wait until a 127 ("not an active child") is returned.

The following is the test script output from a sample run with the TERM handler enabled in the server (background) process -- as it would be in our Java server case:
$ HANDLE_TERM=true ./tryWaitDriver
0 ./tryWaitDriver starting ./tryWait
0 ./tryWaitDriver pausing 5 seconds before sending TERM to ./tryWait PID=58584
0 ./tryWait starting ... PID=58584
0 ./tryWait Started 58586; rc=0
0 ./tryWait Entering wait 58586
0 ./tryWaitBackground started; PID=58586, PPID=58584
0 ./tryWaitBackground entering sleep 30
5 ./tryWaitDriver sending TERM to ./tryWait PID=58584
5 ./tryWaitDriver waiting for ./tryWait PID=58584 to complete
5 ./tryWait Processing TERM signal; sending TERM to 58586
5 ./tryWait Exited wait 58586; exitValue=143            <== wait interrupted by SIGTERM
5 ./tryWait Entering wait 58586                         <== re-driving wait after interruption (process not done)
30 ./tryWaitBackground received TERM
30 ./tryWaitBackground exiting with rc=7
30 ./tryWait Exited wait 58586; exitValue=7             <== wait completed by background process exit
30 ./tryWait Terminated server; rc=7
30 ./tryWait Exiting; rc=7                              <== Exiting with *real* exit code
30 ./tryWaitDriver wait complete; rc=7

Note the exitValue from the following TERM signal processing in tryWait -- 143.  This is 128 + SIGTERM (15).  The real background process exit code isn't observed for several more seconds and after a second wait from tryWait -- 7.

The following sample output is from the scripts with the TERM handler in the server (background) process disabled.  This is included to demonstrate that the wait exit code is the same when indicating a SIGTERM-interrupted wait or wait picking up the exit code of a process without a TERM handler.
$ HANDLE_TERM=false ./tryWaitDriver
0 ./tryWaitDriver starting ./tryWait
0 ./tryWaitDriver pausing 5 seconds before sending TERM to ./tryWait PID=58648
0 ./tryWait starting ... PID=58648
0 ./tryWait Started 58650; rc=0
0 ./tryWait Entering wait 58650
0 ./tryWaitBackground started; PID=58650, PPID=58648
0 ./tryWaitBackground entering sleep 30
5 ./tryWaitDriver sending TERM to ./tryWait PID=58648
5 ./tryWaitDriver waiting for ./tryWait PID=58648 to complete
5 ./tryWait Processing TERM signal; sending TERM to 58650
5 ./tryWait Exited wait 58650; exitValue=143            <== wait interrupted by SIGTERM
5 ./tryWait Entering wait 58650                         <== re-driving wait after interruption (process not done)
5 ./tryWait Exited wait 58650; exitValue=143            <== wait completed by background process exit
5 ./tryWait Terminated server; rc=143
5 ./tryWait Exiting; rc=143
5 ./tryWaitDriver wait complete; rc=143

####TryWaitDriver####
#!/bin/bash
# Tests signal handling in BASH.  This script is the driver sscript for the
# ./tryWait and ./tryWaitBackground scripts used to demonstrate the effect of
# SIGTERM on a bash wait command.
#
# Run as
#   ./tryWaitDriver
# or
#   HANDLE_TERM=true ./tryWaitDriver
# enables the TERM signal handler in ./tryWaitBackground.
# Run as
#   HANDLE_TERM=false ./tryWaitDriver
# omits the TERM single handler in ./tryWaitBackground.
#
# When run with the TERM handler in tryWaitBackground enabled, the script shows that wait
# can be terminated by SIGTERM and the wait exit code reflects interruption of the wait
# command and *not* termination of waited-upon (tryWaitBackground) process.  When the TERM
# handler in tryWaitBackground is disabled, the script shows that wait is again still
# terminated by SIGTERM with the wait exit code reflecting the wait interruption.  The code
# in tryWait differs from the code in start-tc-server.sh in that the tryWait version repeats
# the wait (in a loop) until a "kill -0" indicates the background (server) process no longer
# exists.  This causes the wait command to be re-driven and the *second* wait provides the
# background process exit code.
#

echo "$SECONDS $0 starting ./tryWait"
./tryWait $SECONDS &
PID=$!

echo "$SECONDS $0 pausing 5 seconds before sending TERM to ./tryWait PID=$PID"
sleep 5
echo "$SECONDS $0 sending TERM to ./tryWait PID=$PID"
kill -TERM $PID
echo "$SECONDS $0 waiting for ./tryWait PID=$PID to complete"
wait $PID
echo "$SECONDS $0 wait complete; rc=$?"

####TryWait####
#!/bin/bash

if [ $1 ]; then
  SECONDS=$1
fi

echo "$SECONDS $0 starting ... PID=$$"

PID=0
function cleanup_TERM {
  echo "$SECONDS $0 Processing TERM signal; sending TERM to $PID"
  kill -TERM $PID
}
trap cleanup_TERM SIGTERM

start=true
while $start; do
  ./tryWaitBackground $SECONDS &
  PID=$!
  rc=$?
  echo "$SECONDS $0 Started $PID; rc=$rc"

  # Loop on wait until the background process is no longer alive.
  while kill -0 $PID 2>/dev/null ; do
    echo "$SECONDS $0 Entering wait $PID"
    wait $PID
    exitValue=$?
    echo "$SECONDS $0 Exited wait $PID; exitValue=$exitValue"
  done

  start=false

  echo "$SECONDS $0 Terminated server; rc=$exitValue"
done

echo "$SECONDS $0 Exiting; rc=$exitValue"
exit $exitValue

####tryWaitBackground####
#!/bin/bash
delay=30

if [ $1 ]; then
  SECONDS=$1
fi

echo "$SECONDS $0 started; PID=$$, PPID=$PPID"

if [ "$HANDLE_TERM" == "" -o "$HANDLE_TERM" == "true" ]; then
  function handle_TERM {
    echo "$SECONDS $0 received TERM"
    echo "$SECONDS $0 exiting with rc=7"
    exit 7
  }
  trap handle_TERM SIGTERM
fi

echo "$SECONDS $0 entering sleep $delay"
sleep $delay
echo "$SECONDS $0 exiting sleep $delay; rc=$?"

echo "$SECONDS $0 exiting with rc=2"
exit 2

What would be preferable to all of this messing around with shell-specific (both version and lineage) behavior is to determine a better way for galvan to force kill the entire process tree as that was the original reason for the initial script modification.  It is what we do on Windows since we had no other option.
While that has its own sort of oddities (depending on the environment of how the shell script is invoked, that might be insufficient to force all sub-processes to terminate) but it will be consistent with previous versions and the behavior we would see (or at least how we approach it) on Windows.

Given that Windows already needs this (which is why we also need the ability to look up the inferior process PID), in galvan, the corresponding Unix approach would be to invoke kill, directly:  kill -- -<PID> (note the extra "-" when compared to kill PID).
This would then allow us to revert the shell script to its original shape.

Coming up with a portable way to do this, without some special native, is pretty tricky.  The shells on different platforms appear to implement kill differently and other tools, like ps, also differ.
I still prefer reverting the start-tc-server.sh script to a simpler form and moving the complexity into galvan, though, so I am continuing to dig into this.  I am looking into some JNA (ipc-eventbus already uses it, under galvan) to interact with killpg while invoking the script in a way which may allow us to treat it as its own group.  So far, this isn't working:  most attempts to start the sub-process in its own group (script, for example) does the right thing when testing in a shell but not when run under the VM.
My current thinking is that we could find a portable invocation of ps which would allow us to get PID, PPID, and command (most likely we could just invoke ps -o pid,ppid,command since that seems to avoid any extenions - in isolation, it works on both Linux and OS X).  Each galvan ServerProcess could then inject an additional, unique parameter to the end of the start-tc-server.sh command (since this appears to not cause any problems and is passed through to TCServerMain).  This is so that we could use it as a unique eye-catcher for parsing ps output for the invocation in order to identify a specific PID to signal.  If the additional argument is questionable, we could use the server name, but only if we could further ensure that it would be unique across concurrent tests on the same machine.
These are all ugly work-arounds but they remove all of the complexity from the start-tc-server.sh script, isolating testing framework concerns purely within galvan.

I had luck with approach mentioned in the last comment:  injecting an eye-catcher into the TCServerMain, scraping it out of ps output, and then calling kill, directly.  This uses a version of start-tc-server.sh which reverts all the changes to run in the background and relay the signals with traps so it reverts us back to 4.x expectations, which is much more reliable.  The only changes left in the script were related to some white-space handling support, added more recently.
I will do some more in-depth testing and post a PR for core and galvan, tomorrow morning.


Bug in notifications

There is a bug that affects notifications. When I put a like in a post, I have a notification such as "Rafa has commented the post of Rafa", being Rafa the author of the post. Specifically, it happens me with giacotoktan@cats-pm and<EMAIL_ADDRESS>The two members are in a hub with Hubzilla dev branch, while mine is in the stable branch. Could that explain the error?

cats.pm is not in DEV -> https://cats.pm/siteinfo .

Now it happens with more members. It also affects the comments: I comment a post and the notification says that the author of the post is the author of my comment ....


Field: Improve the UI of the markdown field

We could make this an option of the Rich Text editor, many WYSIWYGs support makrdown as an option. E.g: https://simplemde.com/
Related: #616

Solved in #1357


Automated cherry pick of #2913: scheduler: fix disk local backend not select matched storage

Cherry pick of #2913 on release/2.12.
#2913: scheduler: fix disk local backend not select matched storage

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.12@35826cd). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff               @@
##             release/2.12   #2916   +/-   ##
==============================================
  Coverage                ?   7.06%
==============================================
  Files                   ?     477
  Lines                   ?   76355
  Branches                ?       0
==============================================
  Hits                    ?    5396
  Misses                  ?   70507
  Partials                ?     452

Flag
Coverage Δ

#aFlag
7.06% <ø> (?)

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 35826cd...3d6b080. Read the comment docs.


is there an attrib that changes the name of element into a chosen field?

For example this:

where elements 0 is replaced by "Acknowledge" as if I were to use a string instead of enum.
I tried this workaround but it won't fool Unity :D

Yes
[Serializable]
public enum ANIMSTATE
{
    None,
    State,
    AnotherState,
}

[Serializable]
public struct AudioStateClips
{
    public ANIMSTATE state;
}

[RichLabel(nameof(LabelAudioStateClips), true)]
public AudioStateClips[] audioClips;

public string LabelAudioStateClips(AudioStateClips clip, int index)
{
    return $"{clip.state} ({index})";
}

wow nice! thanks!


make private key optional to support SSH agent usage

So this is just #22, but now using locals, correctly formatted and with a short mention in the README.
It works for me with my yubikey, i'd be especially interested if it solves the problem @fraank is having in #25.

Thank you for your hard work, I believe that is a very important PR.


[a11y][iOS] Overlaid widgets are not accessible by voice controls on iOS

Internal: b/297151696
Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

On iOS, enable voice controls (Settings  > Accessibility > Voice Control, enable Show Hints)
Run the code sample (code below).
Observe that there is only one hint (the numeral 2) over the overlaid widgets (see screenshot for comparison with Android behavior).

Expected results
(It is not known if this is intended behavior on iOS)
There should be some way to interact with overlaid widgets.
Actual results
There is no way to interact with the overlaid widget via voice commands on iOS.
Code sample
Code sample
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      theme: ThemeData(useMaterial3: false),
      title: 'Flutter Demo',
      home: MyHomePage(),
    );
  }
}

class MyHomePage extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('a11y overlay bug demo'),
      ),
      body: Stack(
        children: [
          ListView(
            shrinkWrap: true,
            padding: EdgeInsets.zero,
            children: <Widget>[
              Semantics(
                onTap: () {},
                child: Container(
                  height: 500,
                  color: Colors.amber[600],
                  child: const Center(child: Text('Entry A')),
                ),
              ),
              Container(
                height: 500,
                color: Colors.amber[500],
                child: const Center(child: Text('Entry B')),
              ),
              Container(
                height: 500,
                color: Colors.amber[100],
                child: const Center(child: Text('Entry C')),
              ),
            ],
          ),
          Semantics(
            // "Tap 1" voice command on Android triggers this onTap.
            // on iOS, without the padding line below, there is no way to tap this widget via voice controls.
            onTap: () => print('Toolbar container tapped.'),
            child: Container(
              height: 300,
              // Uncommenting this line will give the underlying widget enough space to be seen by the voice controls on iOS.
              // padding: const EdgeInsets.all(20.0),
              color: Colors.green,
              child: TextButton(
                // "Tap 2" voice command on Android triggers this onTap.
                onPressed: () => print('Toolbar container button tapped.'),
                child: const Text('Toolbar container below app bar'),
              ),
            ),
          ),
        ],
      ),
    );
  }
}

Screenshots or Video

Screenshots of Android and iOS behavior
Android:

iOS:

Logs
No response
Flutter Doctor output
Doctor output
[✓] Flutter (Channel google3, on macOS 13.5 22G74 darwin-x64, locale en)
    • Framework revision 579991cbb3 (7 days ago), 2023-08-17T00:00:00.000
    • Engine revision 5019d6da65
    • Dart version 7e4e5796ee

[✓] Android toolchain - develop for Android devices (Android SDK version Stable)
    • Android SDK at google3
    • Platform Stable, build-tools Stable
    • Java binary at: /Applications/Android Studio with
      Blaze.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      17.0.6+0-17.0.6b802.4-9586694)

[✓] GoogleMac (on macos)
    • AppleWWDRCA installed.
    • Google development certificate installed.

[!] Android Studio (version unknown)
    • Android Studio at /Applications/Android Studio with Blaze
      Beta.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to determine Android Studio version.
    • Java version OpenJDK Runtime Environment (build
      17.0.6+0-17.0.6b829.5-9884540)

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio with Blaze.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      17.0.6+0-17.0.6b802.4-9586694)

[✓] VS Code (version 1.81.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension can be installed from:
      🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E300c

[✓] Connected device (2 available)
    • Pixel 4 (mobile) • 99211FFAZ0054E • android-arm64 • Android 13 (API 33)
    • macOS (desktop)  • macos          • darwin-x64    • macOS 13.5 22G74
      darwin-x64

! Doctor found issues in 1 category.

Internal tracking bug: b/297151696.

Updating the priority to match the internal bug

I did some investigation here, and I'm having trouble understanding what exactly the bug is based on the description and the internal discussion. In particular, the screenshots here don't match the sample code, and in my tests, this comment:

// Uncommenting this line will give the underlying widget enough space to be seen by the voice controls on iOS.
// padding: const EdgeInsets.all(20.0),

isn't correct for the sample code here, in that I don't see any change when adding this padding. It doesn't seem like the generic semantic nodes are visible no matter what in this example, even when completely removing layers, so it's not clear to me that the repro case here is actually correct.
Can we get a clearer, consistent set of steps and outcomes?

(As a general note about the internal discussion: iOS accessibility generally operates on a hit-test basis, so if it's impossible to tap a widget manually, I would not expect it to be surfaced for voice control on iOS.)

(As a general note about the internal discussion: iOS accessibility generally operates on a hit-test basis, so if it's impossible to tap a widget manually, I would not expect it to be surfaced for voice control on iOS.)

This very well might be WAI then.
I'm not sure why you're not seeing differences in before/after commenting that line. Here is what I see on iOS without padding (first image) and with padding (second image):


Frisby API Tests fail with HTTP 500 error

API tests often run into 500 errors without any functional reason. Maybe the server dies during test execution. Can probably not be tracked back via Git history as Frisby tests were failing on Travis-CI for a long time already but did not break the build.

CC/ @J12934 @wurstbrot

If i remember correctly this was made to access the raw body instead of the body parsed by body parser.
This was required to implement the parameter pollution challenge.

The JSON.parse(req.body) is the culprit here as it runs into a SyntaxError: Unexpected token o in JSON at position 1 which indicates that we are trying to parse something that already is an Object.

Mhhh this is weird that this apparently only happens in the api tests. Could this possibly something specific to frisby?

Maybe it's the way frisby sends its requests compared to how Angular does it? If Angular sends them stringified and frisby does not but sends plain JSON objects, that would explain the error only occuring during API testing. Just guessing, though, needs some actual fact checking...


Support the SHUB_APIURL and SHUB_STORAGE environment variables

@elacuesta I am having a hard time figuring out why tests get a gzip-decoding issue on PyPy. I wonder if it is OK to merge stuff and release despite that or if we should look into it, e.g. maybe some dep upgrade is causing issues.

Oh, it’s vcrpy 6+ that broke that.


docs(norelease): update readme

updates readme with some examples. updates PR step to use catalystsquad action for validating commits.

:tada: This PR is included in version 1.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


FM-8228 - convert to litmus

https://pipelines.puppet.com/team-modules/builds/595766

this PR can be closed


FAB shortly visible after app start if visibility=gone

Given gollowing scenario:

1 Activity with SlidingUpPanel + FAB
SlidingPanel default state = HIDDEN
FAB visibility = gone

If you start the application you will see that the FAB is shortly visible (~ 0.5 seconds) and will then be gone.
Afterwards showing/hiding the FAB works perfectly.
I hope to have some time this week and try to look into it on my own.

@Siegbaert This is an issue with the AppCompat Support Design Library. See Issue #182093 of the Android Issue Tracker. The good news is, they seem to have a fix for it, the bad news is, they didn't release it yet. So you'll have to wait.
I'll be closing this issue. If the issue still persists after the official release with the fix, please reopen it. Thank you!

Oh well... this didnt come up in my google search.
I wrote 2 different activities using a FAB. One with your changes to SlidingUpPanel lib and one without using the library. This bug didn't appear in the other activity so I thought it must be the lib causing the problem.
Sorry for bodering you and thank you very much!


fix: added finance card consistency

Added consistency in finance card
Description
Related issue(s)
Fixes #2574

The results of this PR look good but if there are more items per finance card, the card would expand vertically accommodating those items, which is not the desired behavior.
IMO we should go ahead with #2581

@Min2who
Can we have the content starting form the top instead of the bottom

@sambhavgupta0705 lemme see if i can

closing this PR as it has been fixed by https://github.com/asyncapi/website/pull/2581


can not connect docker

err log:

dir files:

system:
macos 12.0.1
M1

I haven't tested the package with M1 Macs yet.
Can you manually talk to the Docker socket via e.g. curl?

That is strange. Unfortunately I cannot reproduce this issue on my machine.
Maybe you can try running this in a Linux container to see if this could be related to the macOS Docker installation.

I was able to run healthy using Portainer

First, sorry for the late response but I hadn't much time in the past weeks.
I could reproduce that issue now. Docker required a Host header for each request. I added that missing header to the package.
Please update the the docker-swift-client package to the latest version, that should fix this issue. Does this work for you?

yeah , thanks, it working now.
but there is have a warning abount port

The other warnings/errors are caused by Apple's Network.framework and can be ignored, as far I can tell.


Set up axe tests on CI

We should run axe tests on CI so we catch accessibility issues before running g3 presubmits

Looks like we still run aXe tests on our e2e tests.


chore: disable release ci checks

disable release ci checks. (cherry pick from branch release-3.2.1)

#1843


fix(types): remove non-void constraint from queryFn result

discussed in #3541
trying to enforce not-void + not-undefined on the queryFn return type is creating pockets of complexity / unexpected typing behaviour, as TS is trying to both enforce and infer too many things at the same time.
removing for now, and adding some additional tests to check for the pieces that were clashing with the non-void constraint

able to use fetch() in queryFn with Promise<any> return type
able to use wrapped queries with custom fetcher functions typed to return TQueryFnData

:tada: This PR is included in version 4.0.0-beta.20 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket: