posty cursor

#317
so, this is a prototype and i'm not sure how much i like it.

reduced hitboxes for characters on main street
made the cursor be posty
bs follows her when party is active

some problems:

i don't know if this is actually a beneficial user experience, even if it is cute
i couldn't get the cursor to ever not be posty
there's a "duplicating" effect that happens whenever you hold the mouse button

i have a couple ideas how to fix this. for instance, next im going to try covering the entire displayable with a cursor effect and inverting the cursor behavior i wrote. i think by doing a double negative like this it might be an improvement, but the biggest hurdle is whether it'll let me tell it to use the OS's default cursor during the mouse define call

the main problem is that i cant seem to find any way for ren'py to use the operating system's mouse under most circumstances but switch to a custom mouse under other conditions. it's like you need to be using a special mouse at all times and you can change the mouse's appearance but that's all.

so i "solved" the issue by adding a picture that closely resembles windows's cursor. needless to say, this is horrible and not remotely optimal.

only looks good if you are a windows user AND you use the default cursor
it's stupid

some solutions to this, maybe?

the game commits to a custom cursor, which might not be the greatest idea
give up???

could we shrink posty to about cursor size?

I found the solution and have it in the branch.

I've given up. Unless we modify the Posty cursors to fit the Windows Cursor in there. It's a losing battle with our actual cursor.

your experimentation is helpful regardless. sorry it didnt work out

@pixequil About the "can't reset cursor" thing - that's probably an upstream (engine) bug. Maybe you should file an issue in the official Ren'Py repo?

The cursor looks better now that it's been sized down. I like that there are multiple different cursors with posty and posty & brand soda. I don't think the cursor switching to the default windows cursor during dialogue scenes looks too bad. It looks like it works.
I'm fine with committing it with the main branch if it can be done with resolving conflicts and if you're cool with it


After upgrading to version 4, my datatable no longer renders

Expected Behavior
The table should render as it did with version 3.7.8
I have the necessary packages installed and also @mui/styles
Also, I do not find anyplace in documentation that explains what are breaking changes that would need to be done after upgrading.
Current Behavior
getStylesCreator.js:24 MUI: The `styles` argument provided is invalid.
You are providing a function without a theme in the context.

You are providing a function without a theme in the context.
One of the parent elements needs to use a ThemeProvider.
create @ getStylesCreator.js:24
attach @ makeStyles.js:95
(anonymous) @ makeStyles.js:233
useSynchronousEffect @ makeStyles.js:187
useStyles @ makeStyles.js:225
WithStyles @ withStyles.js:56
renderWithHooks @ react-dom.development.js:14985
updateForwardRef @ react-dom.development.js:17044
beginWork @ react-dom.development.js:19098
callCallback @ react-dom.development.js:3945
invokeGuardedCallbackDev @ react-dom.development.js:3994
invokeGuardedCallback @ react-dom.development.js:4056
beginWork$1 @ react-dom.development.js:23964
performUnitOfWork @ react-dom.development.js:22776
workLoopSync @ react-dom.development.js:22707
renderRootSync @ react-dom.development.js:22670
performSyncWorkOnRoot @ react-dom.development.js:22293
scheduleUpdateOnFiber @ react-dom.development.js:21881
updateContainer @ react-dom.development.js:25482
(anonymous) @ react-dom.development.js:26021
unbatchedUpdates @ react-dom.development.js:22431
legacyRenderSubtreeIntoContainer @ react-dom.development.js:26020
render @ react-dom.development.js:26103
./src/index.tsx @ index.tsx:7
__webpack_require__ @ bootstrap:19
(anonymous) @ startup:4
__webpack_require__.O @ chunk loaded:23
(anonymous) @ startup:5
(anonymous) @ startup:5
MUIDataTable.js:61 Uncaught TypeError: Cannot read properties of undefined (reading 'down')
    at defaultTableStyles (MUIDataTable.js:61)
    at Object.create (getStylesCreator.js:19)
    at attach (makeStyles.js:95)
    at makeStyles.js:233
    at useSynchronousEffect (makeStyles.js:187)
    at useStyles (makeStyles.js:225)
    at WithStyles (withStyles.js:56)
    at renderWithHooks (react-dom.development.js:14985)
    at updateForwardRef (react-dom.development.js:17044)
    at beginWork (react-dom.development.js:19098)

Steps to Reproduce (for bugs)
I unfortunately cannot provide as it is a private repo, here is some code
import React from 'react';
import MUIDataTable, { MUIDataTableColumn } from 'mui-datatables';
import HtmlReactParser from 'html-react-parser';

const makeColumns = (cols: { name: string; label: string; setCellProps: (...args: any) => any }[]): MUIDataTableColumn[] => {
  const columns: MUIDataTableColumn[] = [];
  for (let i = 0; i < cols.length; i += 1) {
    const c = cols[i]; // eslint-disable-line security/detect-object-injection
    columns.push({
      name: c.name,
      label: c.label,
      options: {
        filter: false,
        sort: false,
        setCellProps: (cellValue, rowIndex) => c.setCellProps(cellValue, rowIndex),
        setCellHeaderProps: (value) => ({
          style: { color: 'inherit', fontWeight: 'bold', fontSize: '10pt', fontStyle: 'italic', width: c.name === 'numMethods' ? '150px' : 'auto' },
        }),
        customBodyRender: (value) => {
          if (c.name === 'numMethods') value = `${value}`;
          return (

              {HtmlReactParser(value as string)}

          );
        },
      },
    });
  }
  return columns;
};

export const runRowClick = (row, rowClicks, setSelectedRow) => {
  rowClicks.forEach((r, index) => {
    if (row[0].props.children === r.role.name) {
      r.onClick();
      setSelectedRow({ role: r.role, index });
    }
  });
};

export const checkIfSelectedRow = (row, selectedRow, pageRole) => {
  if (selectedRow.role && selectedRow.role.provenance === pageRole && row[0].props.children === selectedRow.role.name)
    return { style: { backgroundColor: 'rgb(0, 125, 188)', fontWeigt: 'bold', color: 'white' } };
};

export const setFontIfRowWasClicked = (value, rowIndex, selectedRow, pageRole) => {
  if (
    selectedRow.role &&
    pageRole === selectedRow.role.provenance &&
    selectedRow.index === rowIndex &&
    (selectedRow.role.name === value.props.children || `${selectedRow.role.allowedMethods.length}` === value.props.children)
  ) {
    return { style: { color: 'white', fontWeight: 'bold' } };
  }
};

export const RolesTable = ({ rolesTableData, rowClicks, pageRole }): JSX.Element => {
  const [selectedRow, setSelectedRow] = React.useState({});
  return (

      <MUIDataTable
        options={{
          filterType: 'dropdown',
          pagination: false,
          responsive: 'standard',
          filter: false,
          download: false,
          search: false,
          print: false,
          viewColumns: false,
          selectableRows: 'none',
          fixedHeader: true,
          setRowProps: (row) => checkIfSelectedRow(row, selectedRow, pageRole),
          onRowClick: (row) => runRowClick(row, rowClicks, setSelectedRow),
        }}
        columns={makeColumns([
          { name: 'name', label: 'Name', setCellProps: (name, rowIndex) => setFontIfRowWasClicked(name, rowIndex, selectedRow, pageRole) },
          {
            name: 'numMethods',
            label: 'Number of Methods',
            setCellProps: (value, rowIndex) => setFontIfRowWasClicked(value, rowIndex, selectedRow, pageRole),
          },
        ])}
        data={rolesTableData}
        title=""
      />

  );
};

Your Environment
React 17, node 14

I have an open source project and again it fails to render, here is the example, https://github.com/WebJamApps/JaMmusic/blob/717_mui-datatable-upgrade/src/components/TourTable/index.tsx
Also, this could be related to https://github.com/gregnb/mui-datatables/issues/1810

I have the same issue!

You need to add a default theme to your app
const theme = createTheme();

<ThemeProvider theme={theme}>
    <App />
</ThemeProvider>

@thefat32 this is not in mui version 5 and from what I found the one they have is not the same for version 5 it is not compatibly with React.strictmode and has been depricated, please let me know when MUI datatables will be fully ready for mui version 5, thank you

I think i'm just going to migrate to using https://mui.com/components/data-grid/
i'll go ahead and close this issue now

@JoshuaVSherman your issue was about not being able to run your project after upgrading to v4. The code I shared fixes your problem and is compatible with mui 5. Yes @mui/styles is deprecated. Yes it is not compatible with React's StrictMode, you probably will get warnings when using it but it will not affect production builds at all. It is very common to have this kind of tradeoffs in community projects that depends on the free time the manantainers and collaborators can give to it.

I was able to fix this by installing<EMAIL_ADDRESS>I am using mui v5.0.3.


ISPN-5883 Node can apply new topology after sending status response

Backport of https://github.com/infinispan/infinispan/pull/4321 for 8.2.x
Change LocalTopologyManagerImpl to synchronize on runningCaches more.

Pushed to 8.2.x, thanks


New CSV imports use website to match existing records

The crux of this is that, since an organization has n possible websites, a new record may use an existing org's alias site. In Burden, for instance, many orgs have websites such as example.com and example.co.uk, or some other alias. In a case where the alias exists in the CSV but not in 🐻, a duplicate org would be created.

Being addressed as part of #131.
Also worth noting there are no constraints on the model against multiple websites, we just prevent upon import.

Closing this because it's being addressed in #131


Some question about inference.py on ZeroScrolls

1.What device did you use to inference on ZeroScrolls dataset.
2.What is the input prompt length do you use when inferencing which is not mentioned in your paper ?

I also have some concerns about these questions


Terms of Service is hidden on some phones

I have tested here and the scrolling is working... user can scroll up to see it.

#175


Rename methods

Fix #22
This PR changes the client interface, callers must update to accommodate for it.

https://github.com/golang/go/wiki/CodeReviewComments#initialisms also says:

This rule also applies to "ID" when it is short for "identifier", so write "appID" instead of "appId".

Updated docs for README.md changes.


task definition volume

what
volume argument not supported in task definition
why
I need to mount my container on task definition volume to support lookup on docker Labels
references
https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/ecs_task_definition#volume

/test all

This repo will only give you the container definition
e.g.
module "container_definition" {
  source  = "cloudposse/ecs-container-definition/aws"
  version = "0.58.0"

  container_name  = "name"
  container_image = "cloudposse/geodesic"
}

resource "aws_ecs_task_definition" "service" {
  container_definitions = module.container_definition.some_output

  volume {
    name      = "service-storage"
    host_path = "/ecs/service-storage"
  }
}

That you guy that make sense to me.


Clicking "Listen" on an inactive profile should do something to the player / sound

If you hit "Listen" on active profile, then you go to a profile that does not broadcast, or your own (not broadcasting) and you hit "Listen" the UI of the player somewhat updates to show the music that was playing and was paused on your profile but it keeps playing the previous account's audio.
This I think should be with sound:
https://github.com/fly-apps/live_beats/assets/33274/3cba38f2-2327-4d1c-a02f-af2a94799951

what should probably happen is that the player stops (you can see it's still ticking seconds) and no sound should be coming out until I start playing the song


Improvement idea

Hi there, I'm the founder of django-superapp.
I'm wondering, wouldn't it be useful to structure your project using django-superapp and then create independent app templates that can be integrated very quickly by developers into their own projects?
Looking forward to your feedback.

Thank you for the suggestion! I was not familiar with django-superapp until now.
While I appreciate the idea, I don't think we can accommodate this directly in the app. It does give me an idea, however:
Lets open a section in the readme.md where we mention (but not endorse) other Django templating projects. We can include django-superapp, cookiecutter, and saasPegasus (paid).

The main idea behind SuperApp is to quickly set up projects and provide the ability to push changes to each app template for each project (https://django-superapp.bringes.io/apps/push-template-changes).
Anyway, I'm not trying to promote this library, as it was built primarily to solve my own problem. If it is also helpful to others, that's good.


[wip / spike] chore(emotion): upgrade to emotion 10

This adds the needed changes to support emotion 10

Sorry, just noticed an issue. The HeaderBar and Sidebar text colors don't switch to dark colors when you change the background color in the theme to be a light color.

I fixed the issue I mentioned in my last comment 🎉

@Poltergeist - shall we close this?


Files normally encrypted in working tree

Add configuration flag to keep files normally encrypted in working tree

Probably, this flag should also affect custom diff process from #9


Full external VRPN server support

As part of path tree, we now have the fundamentals to easily expose a better way of accessing external VRPN servers - device nodes contain the device name and server. For OSVR-managed devices these are populated automatically. We just need to add an improved way of configuring these devices in the server so that they are added to the path tree correctly and supplied with a JSON device descriptor.

Completed.


New tests to compare different models on the same lens and source parameters

We have some of these, particularly the Test/SIE directory.  The template can be used for new tests when needs arise.


New University

Centro de Estudios Superiores en Educacion Especializada
Av, Tecnologico S/N Esq Pespuntadores
Leon Gto Mexico

Please kindly provide the full name of your school on the first line of the file in your request

@alfredo-cesee Also, please provide us with:

the school official website URL
the school street address, including city and country
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

@alfredo-cesee You might miss my previous comment. Please react.

http://www.cesee.edu.mx
Av. Tecnológico S/N esq Pespuntadores, Fracc. Ind. Julian de Obregon, Leon
Gto, C.P. 37290
http://www.cesee.edu.mx/oferta-educativa.html
El sáb, 10 ago 2024 a las 11:09, Philip Torchinsky (<
@.***>) escribió:

@alfredo-cesee https://github.com/alfredo-cesee Also, please provide us
with:

the school official website URL
the school street address, including city and country
an URL of a page on the official website where the school offers an
IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are
submitting as an official email domain for the students.

—
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/22469#issuecomment-2282212569,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BKNIRYTXBMJKQQYLMZ7RHI3ZQZCMJAVCNFSM6AAAAABMHTBN2OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBSGIYTENJWHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

@alfredo-cesee First stage of verification passed, official domain for Centro de Estudio Superiores de Educación Especializada is cesee.edu.mx as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@alfredo-cesee I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the aforementioned issue. Thank you for your understanding!


More than one connecting and other encrypt method

Server side got crush while I trying connect more than one device. While connect CHACHA20 on android server side got crush similarly.
Server - Aruba, Ubuntu 18.04 LTS, kernel 4.15, BBR
Client0 - Windows 10 with GO-shadowsocks2
Client1 - Android 5.0.1, LG h522y, shadowsocks-android-4.5.7

Did you open Dns forward option in ss-android? If so, close it and it will be ok. see more https://github.com/shadowsocks/go-shadowsocks2/issues/102#issuecomment-414942262


[Feature Request] Autocomplete add event

Existing Component
是
Component Name
Autocomplete
Description
Autocomplete 组件是否添加录入的内容改变触发的事件

Sorry, we have no plan to support this.


bazel support: build all java_library and proto targets [phab:D154]

Created by schroederc at 2015-04-01 17:00:48:
This change includes a custom genproto Bazel rule that handles proto
dependencies.  It will soon also handle C++/Go generation as well.

schroederc wrote at 2015-04-01 17:08:50:
Add genproto TODOs for C++/Go

zarko accepted this revision at 2015-04-01 23:23:25

schroederc closed via commit 99cda53a6b93ab9ab0776841651b955afddbc546 at 2015-04-02 16:19:42


When downloading model archive, it would be nice to see the download progress on console

When downloading model archive, it would be nice to see the download progress on console, similar to how PyPi does it on pip install

A couple challenge with proposed feature:

MMS is running in background now. print progress in log file doesn't have much value.
logging is per line, won't be able to print progress bar like pip install did.
Downloading model may not always know size from HTTP header, it would be hard to calculate percentage.


Update README.md

Thank you for contacting me and the offer.
Unfortunately, I do not speak Chinese and am not currently interested in such a job.


Remove deprecated authorities function

build fails.
also, is there an appropriate PR to bring polkadot up to date? (i presume this breaks the API).

I can prepare a polkadot pr, but as the function was just a stub. There should not be that much breakage.


[CI] Support Multi-platform Docker Images

Desired Behavior
Meshery users, particularly those running Docker Desktop can benefit from multi-platform images of Meshery Server.
Implementation
Refactor existing stable and edge build and release workflows to use docker-build-push actions. See https://github.com/meshery/meshery/blob/master/.github/workflows/build-and-release-dde.yml as an example.
Be aware of local binaries and their needs:

layer5/kubeopenapi-json

Acceptance Tests

platform: linux/amd64,linux/arm64 should both be supported.2.

Contributor Guides and Resources

🛠 Meshery Build & Release Strategy
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

Hi, @leecalcote I agree with this. We need multiple platform support here.

New [multi-platform.yml](https://github.com/meshery/meshery-linkerd/blob/master/.github/workflows/multi-platform.yml) workflow put into place. ARM images are now built and pushed.


Configure $export command to filter MedMorph survey data

Configure $export command to filter MedMorph survey data, so that correct data is exported to EDAV

@briansok71
Stale ticket. Please rewrite to reflect teh business problem
User-defined export from FHIR server (as distinct from the Event-based triggers using Azure Functions)
Options...

$export
Logic App
PRIME FHIR Export GUI

GUI proper:
Exporter code for JSON: https://github.com/CDCgov/prime-public-health-data-infrastructure/tree/c1c3683800248938f6e78f3618f5fe316906b6a8/src/FunctionApps/python/FhirServerExport
JSON to Table: https://github.com/CDCgov/phdi/blob/main/phdi/tabulation/tables.py


kong no data error while enabling the plugin

I was trying to run the example py-hello plugin. I have already installed kong-pdk inside the kong image. After enabling the plugin I am getting the following error and all the requests are getting stuck.
2021/08/18 09:30:18 [error] 23#0: *14840 [kong] mp_rpc.lua:308 [py_hello] no data, client: <IP_ADDRESS>, server: kong, request: "GET /my-app/ HTTP/1.1", host: "localhost:8000"

config:
plugins = bundled,py_hello
pluginserver_names = python
pluginserver_python_socket = /usr/local/kong/python_pluginserver.sock
pluginserver_python_start_cmd = /usr/local/bin/kong-python-pluginserver -d /usr/local/kong/python-plugins
pluginserver_python_query_cmd = /usr/local/bin/kong-python-pluginserver -d /usr/local/kong/python-plugins --dump-all-plugins

py-hello plugin is placed in /usr/local/kong/python-plugins directory
kong-python-pluginserver is available at path /usr/local/bin/kong-python-pluginserver

@abhisheksharma2805 Could you turn on debug logging in Kong for collecting more avaiable info?

@fffonion I tried running this with version 0.27, 0.26 and 0.24. Surprisingly the plugin is working for version 0.24. I have attached the log files for 0.27 and 0.24. I have also added some logs in the plugin which you can see in 0.24 log file at the end, which indicates the plugin is being initiated and called correctly.
kong_logs_pdk_0.24.txt
kong_logs_pdk_0.27.txt

Hi @abhisheksharma2805
Would you please:

check sock file name created in /usr/local/kong an compare with socket name set in your kong configuration file
increase verbose logging level
try to start your plugin manually directly from the shell (kong-python-pluginserver --socker-name XXXX --d XXX --dump-all-plugins  -vvv .... )
try to test your configuration with the default py-hello plugin

Kind regards,
Mateusz

@mpromny Thanks for the suggestions. I have tried all the steps but still no luck.
I am using latest kong (2.5.0) with pdk 0.27 and python 3.6.8.
Custom kong image:
FROM kong:2.5.0
USER root
RUN apk update && apk add libffi-dev gcc g++ make musl-dev openldap-dev libressl-dev openssl lapack-dev openblas-dev libxml2-dev libxslt-dev postgresql-dev
RUN wget https://www.python.org/ftp/python/3.6.8/Python-3.6.8.tgz
RUN tar -xvzf Python-3.6.8.tgz
WORKDIR Python-3.6.8
RUN ./configure
RUN make && make install
RUN ln -sf /usr/local/bin/python3.6 /usr/bin/python && ln -sf /usr/local/bin/pip3.6 /usr/bin/pip
RUN pip install --upgrade pip
# install external requirements - pip install -r requirements.txt
RUN pip install kong-pdk==0.27
COPY python-plugins/. /usr/local/kong/python-plugins
USER kong
WORKDIR /

Kong configuration
plugins = bundled,py-hello
pluginserver_names = py
pluginserver_py_socket=/usr/local/kong/python_pluginserver.sock
pluginserver_py_start_cmd=/usr/local/bin/kong-python-pluginserver -d /usr/local/kong/python-plugins
pluginserver_py_query_cmd=/usr/local/bin/kong-python-pluginserver -d /usr/local/kong/python-plugins --dump-all-plugins

This same image and configuration works if I use kong-pdk==0.25

@abhisheksharma2805
I understand that it works with 0.25. Therefore I asked you to do the following steps:

check sock file name created in /usr/local/kong an compare with socket name set in your kong configuration file
-- please start the service and past output from ls -al /usr/local/kong/python*
increase verbose logging level
-- Have you done that?
try to start your plugin manually directly from the shell (kong-python-pluginserver --socker-name XXXX --d XXX --dump-all-plugins -vvv .... )
-- were you able to start python server from the shell? Is it working?
try to test your configuration with the default py-hello plugin
--  Have you tried this approach?

@mpromny Yes I have tried all the steps:
I am only trying to run the py-hello plugin as of now

socket filename is same as written in the config file - python_pluginserver.sock
I have set log_level=debug
I tried starting the plugin manually from the shell inside the container:kong-python-pluginserver -v --socket-name  pp.sock -d /usr/local/kong/python-plugins --dump-all-plugins

This gave the following output but I cannot find the socket  file pp.sock and I cannot see the plugin in the enabled list :

  DEBG - [22:18:17] loaded plugin "py-hello" from /usr/local/kong/python-plugins/py-hello.py
  [{"Name": "py-hello", "Phases": ["access"], "Priority": 0, "Schema": {"name": "py-hello", "fields": [{"config": {"type":
  "record", "fields": [{"message": {"type": "string"}}]}}]}}]

I also tried running the plugin inside the embedded server with the following configuration but still getting the no data error:

plugins = bundled,py-hello
pluginserver_names = py-hello
pluginserver_py_hello_socket=/usr/local/kong/py-hello.sock
pluginserver_py_hello_start_cmd=/usr/local/kong/python-plugins/py-hello.py
pluginserver_py_hello_query_cmd=/usr/local/kong/python-plugins/py-hello.py -d

@abhisheksharma2805
Thank you, got it.

Please set log_level=debug add and add -vvv  to the configuration.
pluginserver_py_start_cmd= ...  -vvv
pluginserver_py_query_cmd= ... -d -vvv

Have you noticed anything in the log file?

Please verify whether py-server/py-hello works if you set:
def access(self, kong: kong):

instead of
def access(self, kong: kong.kong):

If I enable -v option like this:

pluginserver_py_start_cmd=/usr/local/bin/kong-python-pluginserver -v -d /usr/local/kong/python-plugins
pluginserver_py_query_cmd=/usr/local/bin/kong-python-pluginserver -v -d /usr/local/kong/python-plugins --dump-all-plugins

I get the following error for both pdk 0.25 and 0.27
kong_1            | 2021/08/23 15:27:47 [debug] 1#0: [kong] process.lua:62 search config for pluginserver named: py
kong_1            | 2021/08/23 15:27:47 [error] 1#0: init_by_lua error: ...al/share/lua/5.1/kong/runloop/plugin_servers/process.lua:159: Not a plugin info table:
kong_1            | /usr/local/bin/kong-python-pluginserver -v -d /usr/local/kong/python-plugins --dump-all-plugins
kong_1            | DEBG - [23:27:47] loaded plugin "py-hello" from /usr/local/kong/python-plugins/py-hello.py
kong_1            | [{"Name": "py-hello", "Phases": ["access"], "Priority": 0, "Schema": {"name": "py-hello", "fields": [{"config": {"type": "record", "fields": [{"message": {"type": "string"}}]}}]}}]
kong_1            | stack traceback:
kong_1            | 	[C]: in function 'error'
kong_1            | 	...al/share/lua/5.1/kong/runloop/plugin_servers/process.lua:159: in function 'ask_info'
kong_1            | 	...al/share/lua/5.1/kong/runloop/plugin_servers/process.lua:195: in function 'get_plugin_info'
kong_1            | 	...local/share/lua/5.1/kong/runloop/plugin_servers/init.lua:262: in function 'get_plugin'
kong_1            | 	...local/share/lua/5.1/kong/runloop/plugin_servers/init.lua:270: in function 'load_plugin'
kong_1            | 	/usr/local/share/lua/5.1/kong/db/dao/plugins.lua:136: in function 'load_plugin_handler'
kong_1            | 	/usr/local/share/lua/5.1/kong/db/dao/plugins.lua:221: in function 'load_plugin'
kong_1            | 	/usr/local/share/lua/5.1/kong/db/dao/plugins.lua:273: in function 'load_plugin_schemas'
kong_1            | 	/usr/local/share/lua/5.1/kong/init.lua:515: in function 'init'
kong_1            | 	init_by_lua:3: in main chunk
kong_1            | nginx: [error] init_by_lua error: ...al/share/lua/5.1/kong/runloop/plugin_servers/process.lua:159: Not a plugin info table:
kong_1            | /usr/local/bin/kong-python-pluginserver -v -d /usr/local/kong/python-plugins --dump-all-plugins
kong_1            | DEBG - [23:27:47] loaded plugin "py-hello" from /usr/local/kong/python-plugins/py-hello.py
kong_1            | [{"Name": "py-hello", "Phases": ["access"], "Priority": 0, "Schema": {"name": "py-hello", "fields": [{"config": {"type": "record", "fields": [{"message": {"type": "string"}}]}}]}}]

This works for both v0.25 and v0.27
def access(self, kong: kong):
but still getting the error in v0.27.

2021/08/23 15:25:58 [error] 25#0: *202 [kong] mp_rpc.lua:308 [py-hello] no data, client: <IP_ADDRESS>, server: kong, request: "GET /my-app HTTP/1.1", host: "localhost:8000"

In v0.27, error is being thrown even before the constructor is called

@abhisheksharma2805
1)Sorry, I made a typo:
Please try this configuration:
  plugins: bundled, py-hello
  pluginserver_names: py
  pluginserver_py_socket: /usr/local/kong/python_pluginserver.sock
  pluginserver_py_start_cmd: /usr/local/bin/kong-python-pluginserver -p /usr/local/kong/ -d /scripts/directory -vvv
  pluginserver_py_query_cmd: /usr/local/bin/kong-python-pluginserver -p /usr/local/kong/ --dump-all-plugins -d /scripts/directory

I cannot duplicate your error but I'm using different custom image. Kong 2.5 has been used as a base image but I've installed different packages and I have Python 3.8.

To test whether you are able to start py-server manually, please execute the following commands:

/usr/local/bin/kong-python-pluginserver -p /usr/local/kong/ -d /scripts/directory -vvv &
/usr/local/bin/kong-python-pluginserver -p /usr/local/kong/ --dump-all-plugins -d /scripts/directory


Correct link in Description section

Instead of a link, we see [/dynamics365/business-central/dev-itpro/compliance/apptest-countries-and-translations]

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

ID: 12419013-90c4-b413-eb21-47f2873a88aa
Version Independent ID: f92a516c-815f-0510-bf47-533e361c0e3e
Content: The country codes specified in the 'supportedCountries' property must be valid ISO 3166-1 alpha-2 codes - Business Central
Content Source: dev-itpro/developer/analyzers/appsourcecop-as0056-appsourcecopconfigurationsupportedcountriesmustbevalidcountrycode.md
Service: dynamics365-business-central
GitHub Login: @SusanneWindfeldPedersen
Microsoft Alias: SusanneWindfeldPedersen

Thanks for catching this bug. It will be fixed with the next doc refresh. Thanks.


Error parsing dependencies when encountering double quotes.

django-migration-checker version: 0.6.0
Python version: 3.9.0
Operating System: OSX Big Sur.

Description
The conflict checker gave a false positive result when the dependencies in the migration files are defined with double quotes.
What I Did
Ran the conflict checker with a migration file using double quotes to define dependencies.
Suggestions

Update the regex to accept double-quotes as well.
Improve the error message when detecting a double quote.

We might just need to update the regex
https://github.com/tonyo/django-migration-checker/blob/master/django_migration_checker/base.py#L20


Export-sqlLogin

Is this:
Just place "x" to indicate selection (e.g. "[x]")

[x ] feature
[ ] bug:

Details
Function Export-sqlLogin does not consider the principal specific permissions. This is considered on the Export-sqlUser. I think this is a required feature as it will help to mimic all the actual permissions the login has.

@mvillegascuellar Can you give an example of what you are referring to by:

principal specific permissions

Just so it is clear to the individual that works on this issue.

Sure, when the login has an associated user that  has specific principal permissions rather than role based permissions, those are not exported when using the export-sqlLogin. However if I export the user (export-sqlUser), it does export those specific principal permissions. I think the export-sqlLogin should contain the export-sqlUser functionality.

The Export-SqlLogin should not export anything specific to a database user beyond what you see in the login properties in SSMS. The functionality of the login and user functions were separated in this way by design.
I believe most folks combine the two commands to get the desired outcome you are wanting. However, we can certainly work on a wrapper function that would do that (Export-DbaLoginUser ?). As of now though I am not aware of this being considered (no current issues or PRs).


Fix exportKI2 bug

説明 / Description
KI2 出力の不具合を修正する。
「右」「打」などの修飾をするための Position に常に初期局面を渡してしまっていて正しい判定処理ができていなかった。
チェックリスト / Checklist

MUST

[ ] npm test passed
[ ] npm run lint was applied without warnings
[ ] changes of /docs/webapp not included (except release branch)
[ ] console.log not included (except script file)

RECOMMENDED (it depends on what you change)

[ ] unit test added/updated
[ ] i18n

Codecov Report
Patch coverage: 64.86% and project coverage change: +0.01% :tada:

Comparison is base (a54809a) 69.90% compared to head (c5ee1f8) 69.92%.
Report is 2 commits behind head on main.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #583      +/-   ##
==========================================
+ Coverage   69.90%   69.92%   +0.01%
==========================================
  Files          83       83
  Lines       17343    17352       +9
  Branches     2311     2312       +1
==========================================
+ Hits        12124    12133       +9
+ Misses       2932     2931       -1
- Partials     2287     2288       +1

Files Changed
Coverage Δ

src/common/i18n/locales/en.ts
99.14% <ø> (-0.01%)
:arrow_down:

src/common/i18n/locales/ja.ts
98.73% <ø> (-0.01%)
:arrow_down:

src/common/i18n/locales/zh_tw.ts
99.11% <ø> (-0.01%)
:arrow_down:

src/common/shogi/kakinoki.ts
73.04% <50.00%> (ø)

src/common/shogi/record.ts
70.29% <64.70%> (+0.40%)
:arrow_up:

src/common/settings/app.ts
75.90% <100.00%> (+0.28%)
:arrow_up:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Adding new posts does not update the main page.

I've the problem that my new posts won't be added to the list of posts. Only the footer on other pages got changed and the left one is pointing to the same post. See here: https://www.tekl.de/news/openthesaurus-deutsch-v2020-05-08

I had this problem before, but it disappeard and I don't know when. I tried to edit/unpublish the main pages without any effects.

That is only temporary due to the caching mecahnism. The cache refreshs every 10 Minutes. You can refresh the cache manually, just go the page and hit F12 (with windows), not super sure what to do on a mac to refresh a page without cache. Anyway, after 10 Minutes it updates automatically.

Ah, okay. Now it's visible. But at least the pager shouldn't be altered until the refresh, I think. And please add it to the documentation: https://typemill.net/writers/create-pages
Couldn't you add just a small refresh button to the UI? Maybe below of the list of posts (in the backend) -- well, if it only belongs to posts.

These mechanisms are not really supported in V2 anymore because it focuses on users that use the author interface.


Zooming does not work if scrolling is disabled in webview IOS.

Bug description:
Im my Webview i set height because in my screen i have some webview and some native elements. This is all i wrapped in scroll view.
The problem that even if i set webview height correctly, in ios devices there is a webview scroll. So when i tried to scroll my screen sometimes i scroll webview.
So i have to disabled webview scroll, but after that zooming works incorrectly. It always zoom to the top left conner.
Is there is a possibility to disable only vertical scroll in webview?
To Reproduce:

set height to webview
disable webview scroll
try to zoom webview

Expected behavior:
zoom works correctly

Screenshots/Videos:
webview zoom to the top left conner

Environment:

OS: Mac os
OS version: 12
react-native version: expo 33
react-native-webview version: ~5.8.1

This issue doesn't seem to be resolved. I'm still encountering this when I set scrollEnabled=false. The focal point of the zoom gets pinned to the top left instead of from where I pinch from.


Show item name on mouse hover

Cipsoft adds on div title item name, i.e.:

This would be a cool feature to have, specially for newcomers.

We used to have that in an earlier version of the project, but we removed since it increased the static file size by a lot (3MB~). In the future we might do a second try at that with a different aproach.


Consider removing the conversion of event names into methods.

@craiglittle There is quite few methods that do not convert easily to dynamic method calls, such as :fail, it seems safer and more programatic to allow just trigger as the API gateway for events firing?

Agreed, I think that makes a lot of sense.


Batching works in an unexpected way when using futureStream

0
down vote
favorite
I am trying to use cyclops-react to batch the elements from a queue, based on size, but also on time, so it doesn't block when there are no elements
Maybe the functionality is not what I expected or I am doing something wrong
The complete code (Groovy) is like this with the producer in another thread:
            Queue<String> queue = QueueFactories.<String>unboundedQueue().build();
    new Thread({
        while (true) {
            sleep(1000)
            queue.offer("New message " + System.currentTimeMillis());
        }
    }).start();

    StreamSource.futureStream(queue, new LazyReact(ThreadPools.queueCopyExecutor))
            .groupedBySizeAndTime(10,500,TimeUnit.MILLISECONDS)
            .forEach({i->println(i + " Batch Time: ${System.currentTimeMillis()}")})

The output is:
    [New message<PHONE_NUMBER>332,  Batch Time:<PHONE_NUMBER>356]
    [New message<PHONE_NUMBER>348, New message<PHONE_NUMBER>352,  Batch Time:<PHONE_NUMBER>356]
    [New message<PHONE_NUMBER>355, New message<PHONE_NUMBER>357,  Batch Time:<PHONE_NUMBER>362]
    [New message<PHONE_NUMBER>362, New message<PHONE_NUMBER>364,  Batch Time:<PHONE_NUMBER>365]

But I was expecting one element in each batch since the delay between elements offered is 1 second but the batching is every half a second
Also I tried with an asynchronous stream (Groovy code):
    Queue<String> queue = QueueFactories.<String>unboundedQueue().build();
    StreamSource.futureStream(queue, new LazyReact(ThreadPools.queueCopyExecutor))
            .async()
            .groupedBySizeAndTime(10, 500,TimeUnit.MILLISECONDS)
            .peek({i->println(i + "Batch Time: ${System.currentTimeMillis()}")}).run();

    while (true) {
        queue.offer("New message " + System.currentTimeMillis());
        sleep(1000)
    }

Again, it only batches every 2 seconds sometimes waiting for two elements per batch, even if the timeout in the batch is half second:
    [New message<PHONE_NUMBER>780, Batch Time:<PHONE_NUMBER>819]
    [New message<PHONE_NUMBER>811, New message<PHONE_NUMBER>812, Batch Time:<PHONE_NUMBER>815]
    [New message<PHONE_NUMBER>814, New message<PHONE_NUMBER>819, Batch Time:<PHONE_NUMBER>823]
    [New message<PHONE_NUMBER>823, New message<PHONE_NUMBER>824, Batch Time:<PHONE_NUMBER>828]
    [New message<PHONE_NUMBER>828, New message<PHONE_NUMBER>831, Batch Time:<PHONE_NUMBER>835]

I did a third experiment with a non future non lazy stream, and this time it worked.
Queue<String> queue = QueueFactories.<String>unboundedQueue().build();
   new Thread({
       while (true) {
           sleep(1000)
           queue.offer("New message " + System.currentTimeMillis());
       }
   }).start();

   queue.stream()
           .groupedBySizeAndTime(10,500,TimeUnit.MILLISECONDS)
           .forEach({i->println(i + " Batch Time " + System.currentTimeMillis())})

Result:
  [New message<PHONE_NUMBER>017, New message<PHONE_NUMBER>027,  Batch Time ,<PHONE_NUMBER>055]
   [New message<PHONE_NUMBER>029,  Batch Time ,<PHONE_NUMBER>029]
   [New message<PHONE_NUMBER>033,  Batch Time ,<PHONE_NUMBER>033]
   [New message<PHONE_NUMBER>037,  Batch Time ,<PHONE_NUMBER>037]

Why is the behaviour of the batching seems to be wrong when you use a future stream?

Many thanks for the awesome bug report!
Testing against v2 the FutureStream impl is pulling values from the initial Queue 2 at a time for some reason (this is certainly bug).

So it appears to be behaving functionally as expected (but is underperforming), so I still think there are improvements to be made.
The reason they get batched into 2s for FutureStreams (at least in v2 of cyclops-react I will also check v1), is down to the overhead of transfering data between 2 Queues via a Future. I think in this case that transfer is unneccessary.
What is happening is this.

You have a Queue you are pushing values onto.
StreamSource.futureStream also creates a Queue.
Futures are used to transfer data from your Queue onto the internal one created via StreamSource
This additional overhead / task scheduling factors is large enough that we wind up with two values in our group.

When we do the grouping we check if any upstream values are present, when the next upstream value arrives we check to see if we are within our alloted time and size bounds. If value arrives within 500ms of the previous one we will ask for another value from the Queue and wait until one arrives. The additional overhead doing the unnecessary transfer means we eat into the 1 sec delay between values and there is a good chance the first value in any batch arrives within the 500ms limits, causing us to request another - adding two values to our batch / group instead of one.
As a work around this also works
 queue.stream()
                .groupedBySizeAndTime(10,500,TimeUnit.MILLISECONDS)
                .futureStream(new LazyReact())
                .forEach(i-> {
                    System.out.println(i + " Batch Time:" + System.currentTimeMillis());
                });

The solution is to create FutureStream directly from the user supplied Queue in StreamSource.

The work around you suggested didn't work for me..the forEach at the end of the stream doesn't create an asynchronous task and it blocks
I had to use it this way
        queue.stream()
                            .groupedBySizeAndTime(batchSize, batchTimeoutMillis, TimeUnit.MILLISECONDS)
                            .futureStream(new LazyReact(ThreadPools.getSequential()))
                            .async()
                            .peek(this::executeBatch)
                            .run();

Nevetheless there is still a small issue. I am using now the User supplied queue as you suggested:
When  I add just one element to the queue immediately after I call run(), that element never gets batched.. Only after the asynchronous stream has finished starting its Executor thread, and I add more elements to the queue, then it gets batched
I had this issue because in my unit test I only had one element added and it never got batched
If I add a Sleep long enough after the queue.stream().groupedBySizeAndTime()........run(), and then I add an element to the queue.. then it gets batched

Which version are you using?

2.0.0 MI4

Thanks!
When an element is added immediately after Stream construction it is ingested into the group / batch immediately and the Stream asks for another one from the Queue (and waits until it gets it). It will then get batched with the next element added to the queue.
The Queue class contains a streamBatch method so it would be possible to use that method to build time/ size based grouping that would timeout and continue processing after X ms (or other time units). I'll put together some example code to do that tomorrow.

It probably makes sense to add this type of functionality as an easy to use operator on Queue.
I think this should do what you want, max batch size of 10 or 500ms with timeout rather than waiting for the next value.
Queue<String> queue = QueueFactories.<String>unboundedQueue().build();
        new Thread(()->{
            for(int i=0;i<10;i++){

                queue.offer("New message " + i);
                sleep(10000);
            }
            queue.close();
        }).start();

        long toRun = TimeUnit.MILLISECONDS.toNanos(500l);

        queue.streamBatch(new Subscription(), source->{

            return ()->{
                List<String> result = new ArrayList<>();

                   long start = System.nanoTime();

                       while (result.size() < 10 && (System.nanoTime() - start) < toRun) {
                           try {
                               String next = source.apply(1l, TimeUnit.MILLISECONDS);
                               if (next != null) {
                                   result.add(next);
                               }
                           }catch(Queue.QueueTimeoutException e){

                           }

                       }

                    if(result.size()>0){
                        System.out.println("Result " +  result);
                    }

                start=System.nanoTime();

                return result;
            };
        }).filter(l->l.size()>0)
          .futureStream(new LazyReact(ThreadPools.getSequential()))
                .async()
                .peek(System.out::println)
                .run();

Hi. Thanks
I appreciate that with that level of control I can achieve what I need. Thanks for the example. I was looking at the streamBatch method but didn't find any example.
But I still think that the other way is more elegant, and I would have thought that both, the sync queue.stream and the async queue.futureStream should behave equally in terms of the results and they should be interchangeable. Not sure if other people agree

You are right it's definitely a bug.
The part above that describes the solution are the internal changes required so the behaviour is the same (of course as users externally we can do something similar, but there is a fix required).  Apologies if that wasn't clear.

StreamSource.futureStream creates it's own Queue (rather than reusing the users) which results in needless inefficient data transfer via Futures between the User's Queue and StreamSources, that additional overhead results in the inconsistent behaviour between FutureStreams and ReactiveSeqs.

Thanks a lot for your help

You are welcome. I'll add some separate issues for convenience methods on Queue, which have knowledge of the Queue and can thus do an accurate time out there. Should allow for a more elegant solution.

Merged on master


Tests: Re-enable WPT job for GCC CI

The logs for this last time it was enabled show that the WebDriver fails to accept the connection from wpt.py when run in the CI configuration. It works locally with ASAN+UBSAN, so there must be a deeper problem here.

Can we close this issue?

This issue is overcome by events. The specific job was removed for clang as well.


EXSWHTEC-115 - Implement tests for hipLaunchCooperativeKernel APIs

Negative parameter tests for hipLaunchCooperativeKernel and
hipLaunchCooperativeKernelMultiDevice
Positive parameter tests for hipLaunchCooperativeKernel
Basic positive scenarios for hipLaunchCooperativeKernel and
hipLaunchCooperativeKernelMultiDevice
Negative scenario for hipLaunchCooperativeMultiDevice with multiple
kernels on the same device.

Depends-on: #3049

PR moved to: https://github.com/ROCm-Developer-Tools/hip-tests/pull/59


feat(lifecycle-operator): add option to exclude additional namespaces

Description
Adds the option an option to the lifecycle-operator chart to exclude namespaces  in the MutatingWebhookConfiguration. This is useful to prevent possible deadlocks with other mutating webhooks running in other namespace, which might also prevent pods from being started.
This change should not result in any unexpected changes for users when upgrading. It preserves the default values for the excluded namespaces and does not change the logic of allowedNamespaces. In my opinion it also adds the benefit to be more explicit about the namespaces ("cert-manager" etc.)  which are excluded by default right now.
Fixes #2512
How to test
The change is rather trivial as it mimics the already implemented features. I only tested it manually by rendering the Helm Chart:
echo 'deniedNamespaces: ["foo"]' > test_values.yaml
helm template lifecycle-operator/chart -f test_values.yaml | yq e ' select(.kind == "MutatingWebhookConfiguration")'

I can try to add automated tests, but maybe someone could point me to the relevant ones (kuttl, helm-test...).
Checklist

[x] My PR fulfills the Definition of Done of the corresponding issue and not more (or parts if the issue is separated
into multiple PRs)
[x] I used descriptive commit messages to help reviewers understand my thought process
[x] I signed off all my commits according to the Developer Certificate of Origin (DCO)
see Contribution Guide
[x] My PR title is formatted according to the semantic PR conventions described in
the Contribution Guide
[x] My code follows the style guidelines of this project (golangci-lint passes, YAMLLint passes)
[x] I regenerated the auto-generated docs for Helm and the CRD documentation (if applicable)
[x] I have performed a self-review of my code
[ ] I have made corresponding changes to the documentation (if needed)
[ ] My changes result in all-green PR checks (first-time contributors need to ask a maintainer to approve their test runs)
[ ] New and existing unit and integration tests pass locally with my changes

Summary
Adds the option an option to the lifecycle-operator chart to exclude namespaces  in the MutatingWebhookConfiguration.
Fixes #2512

Thanks for the contribution @knutgoetz !


[Pg] Set query params into query.values

Fix issue #1054
To be aligned with pg-instrumentation, query params should be set to query.values

@dankochetov

@issackr those params are provided on each query execution: https://github.com/drizzle-team/drizzle-orm/pull/1055/files#diff-2433a4e30fcf9ecd1c74374f1543c34099e5592471f7072f6e3ea67b32e91dc1R62

@AndriiSherman Thanks for the response, maybe I was not accurate, my use case is a request hook that expect to get (from instrumentation-pg):
export interface PgInstrumentationExecutionRequestHook {
    (span: api.Span, queryInfo: PgRequestHookInformation): void;
}
export interface PgRequestHookInformation {
    query: {
        text: string;
        name?: string;
        values?: unknown[];
    };
    connection: {
        database?: string;
        host?: string;
        port?: number;
        user?: string;
    };
}

And in drizzle, query.values is undefined
I am working with instrumentation-pg over many ORMs that based on pg
like native pg, sequelize, typeorm, knex.js and more and they are all aligned with this configuration.
Thanks again for the great work

@AndriiSherman can take a look?

Sounds good to me. @dankochetov thoughts?

@dankochetov Thanks for the response
You are right, the instrumentation gets the query  text with placeholders and takes the parameters from the query object.
I think this is enough for the instrumentation query hook because you can extract all the data and do any manipulation as long as you have both query text and params.
pg client anyway knows to execute queries with placeholders and get the parameters as list

@issackr I think you misunderstood me a bit. What I meant is the initial parameters array might contain Placeholder instances instead of actual parameter values, in case of a prepared query execution. You might read about them here - https://orm.drizzle.team/docs/performance#placeholder. Those placeholders in the parameters array are replaced only before the query execution, so the params array that's passed to the driver might not be the same as the initial one, that you are setting into the query object in this PR.

@dankochetov oh right I was not aware of this feature in drizzle
I will change the PR to set the parameter after the placeholders setup

@dankochetov
Done, let me know if anything else is required
Thanks!

Any word on getting this merged in?

@dankochetov this is still waiting for a resolution

@AndriiSherman


Reproducing DAAC results on Procgen Miner

Hi @rraileanu
thanks for your scientific contributions!
I'm currently trying to reproduce your Miner results. After installing the dependencies, I simply run python train.py --env_name miner --algo daac to launch the training. My first run ended up pretty bad, not even near your reported score.
Here is the log of that training run.
daac.log
To verify that this is not just an extreme outlier, I got two more training runs running, which are at 12.5 million steps right now, but behave similar so far.
Did I miss something or how can I reproduce the results of your paper? I added DAAC to my custom PPO implementation and was struggling to get good results either.

For Miner, you should use the flag --use_best_hps to obtain the results shown in the paper. There is a set of HPs that works best across all games (which is the default one in the codebase), however there are a few games which are more difficult to tune and require different values for some of the HPs in order to get the best results and reproduce the numbers reported in the paper.
I hope this fixes the issue but please let me know in case you are still not able to reproduce the results for Miner!

Thanks, I'll try the flag now.

daac_best_0.log
daac_best_1.log
These are two runs using the --use_best_hps flag. The results seem to have quite a high variance.

Yes, I remember this being an issue with my runs as well. This is why the results are reported over 10 seeds, which I recommend as the minimum. Check out this paper https://arxiv.org/pdf/2108.13264.pdf for more robust metrics which should also hold for fewer seeds.

Indeed!
Have you tried DAAC on other environments than Procgen? I'm wondering if there is another environment that shows DAAC's superior performance, but trains faster.

I've only tried it on DMC with distractors but I don't think it's a particularly good environment for DAAC and it was quite difficult to tune it on DMC. I think something more similar to DMLab, MiniHack or MiniGrid might be reasonable environments to try this on. In particular, MiniHack and MiniGrid are quite fast to run and you can easily create your own tasks to probe different abilities, but they don't use visual inputs. DMLab is the type of environment where I would expect DAAC to work well but it is slower to run.

Minigrid can be easily rendered using env.get_obs_render(obs["image"], tile_size=28)
It also looks like that I can reproduce your training results with my custom code base now.
daac_miner_1.log
daac_miner_0.log
Thanks for the chat.


Export-Excel saving two worksheets selected

When using Export-Excel to add a new worksheet to an existing xlsx file, it saves with the first two worksheets selected.

I've just tested this, and when adding a sheet it doesn't change the active / selected sheet Unless -Activate is specified when only the new sheet becomes active.
Could you give a bit more info - what version are you using and ideally some script which causes this.

Using ImportExcel v6.0.0, Excel 2010 (32-bit), Windows 7 Ultimate (64-bit),
PSVersion 5.0.10586.117
Step 1: Using Excel, create an excel spreadsheet on your desktop named
"Test.xlsx" (put anything you want in Sheet1), then close the spreadsheet.
Run this script:
'$ExcelFile = "$ENV:USERPROFILE\Desktop\Test.xlsx";
$a = @(1,2,3,4,5)
$a | Export-Excel $ExcelFile -WorksheetName MoreNewStuff -BoldTopRow
-FreezeTopRow -AutoSize -MoveToStart -Show;
The "MoreNewStuff" and "Sheet1" sheets are both selected.
On Thu, Sep 5, 2019 at 11:48 AM jhoneill<EMAIL_ADDRESS>wrote:

I've just tested this, and when adding a sheet it doesn't change the
active / selected sheet Unless -Activate is specified when only the new
sheet becomes active.
Could you give a bit more info - what version are you using and ideally
some script which causes this.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dfinke/ImportExcel/issues/673?email_source=notifications&email_token=ABURROTPGOGUFWL2OHHBGGDQIES47A5CNFSM4ITIQ6MKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD57UEWY#issuecomment-528433755,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABURROX3NN6U5XC6HTOWJHTQIES47ANCNFSM4ITIQ6MA
.

OK. I can repro it now.
It only happens with movetostart, and interestingly it doesn't happen if the file is created by export excel, and added to. Only if the file was saved by Excel.
I'll dig a bit deeper. Unfortunately I'm going to be out of circulation for a few days so if I don't spot what it is fairly quickly it could be the end of next week before I get back to you.

I suspect that it has to do with marking the sheet active. When Excel open,
it does it automatically on that last sheet being used (unless you
shift-select multiple sheets), and this does not happen automatically with
Export-Excel.
It also looks like using -Activate makes that new sheet be the only one
that is active. I can live with this. However, I found this when doing
testing, and was deleting the new sheet and ended up deleting two sheets.
Then the file was saved before noticing that it deleted two sheets. So I do
think that there being a fix for this, even though using -Activate does
prevent it.
On Thu, Sep 5, 2019 at 3:37 PM jhoneill<EMAIL_ADDRESS>wrote:

OK. I can repro it now.
It only happens with movetostart, and interestingly it doesn't happen if
the file is created by export excel, and added to. Only if the file was
saved by Excel.
I'll dig a bit deeper. Unfortunately I'm going to be out of circulation
for a few days so if I don't spot what it is fairly quickly it could be the
end of next week before I get back to you.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dfinke/ImportExcel/issues/673?email_source=notifications&email_token=ABURROUBCIECPG776ZCSWITQIFNV7A5CNFSM4ITIQ6MKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6APQQI#issuecomment-528545857,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABURROUAFFHNH6OBLHSGFB3QIFNV7ANCNFSM4ITIQ6MA
.

It seems to be a bug in EPPlus.
If there are 2 sheets in the workbook and add a third, you can end up with the two pre-existing worksheets active ... and I get the same thing if I use moveBefore(1) or moveToStart. The .viewTabSelected property reports correctly but Excel doesn't process it correctly.

I confirm. It works like wgshef said.

Well, it looks like I'll just use the -Activate switch to control this.
Thanks for looking into it.
On Fri, Sep 6, 2019 at 2:39 PM Matthew Gray<EMAIL_ADDRESS>wrote:

I confirm. It works like wgshef said.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dfinke/ImportExcel/issues/673?email_source=notifications&email_token=ABURROXDCSIXI2KHRC3J6G3QIKPU5A5CNFSM4ITIQ6MKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6DWRXY#issuecomment-528967903,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABURROTVCI35F56XAXYDRNTQIKPU5ANCNFSM4ITIQ6MA
.


Add AMEX support

Some new tags, that might be good to identify at some point
2023-12-26 17:21:25.481189 +02:00 DEBUG emvpt -  -9F67: Unknown tag
2023-12-26 17:21:25.481386 +02:00 DEBUG emvpt -  -data: [00, 00, 01] = ...
2023-12-26 17:21:25.481596 +02:00 DEBUG emvpt -  -9F77: Unknown tag
2023-12-26 17:21:25.481774 +02:00 DEBUG emvpt -  -data: [34, 34] = 44
2023-12-26 17:21:25.874569 +02:00 DEBUG emvpt -  -9F50: Unknown tag
2023-12-26 17:21:25.874777 +02:00 DEBUG emvpt -  -data: [09, 78] = .x


Installation confuse.

Hello all,
just a little bit confuse when reach step 3.

Deploy the Code in AWS Lambda
Go inside your local src directory, select all files and folders and then create a zip file, make sure the zip file does not contain the src directory itself, otherwise Lambda function will not work.

but inside my folder not contain folder node_modules.
even if see at https://github.com/joanaz/MirrorMirrorOnTheWallSkill/tree/master/src
also don't have a node_modules folder.
it's this folder from MagicMirror folder? or from home folder?
or i miss a step?
please help

Lambda Blank Blueprint ..... it is still exist or not? cannot find it.

I can't find blank function also.

@afroguys and @zhihuai1982 - Amazon updated their dev console so I think the steps are different. I got through the configuration just now but I am receiving errors in testing. If and when I figure it out I'll post some notes to get through it.

@afroguys - Regarding the missing node_modules folder - did you perform step 2 from within the src folder? Running npm install from within the src folder will grab the necessary dependencies found in the package.json file and create the node_modules folder.
As for the blank blueprint - AWS has changed the layout so you have to choose different options and in a different order. Choose Author from scratch after clicking new function under Lambda.
You will need to create the new Alexa Skill prior to adding the trigger in Lambda.
Note - there is something missing that someone else identified as being required for this to work. Prior to uploading the .zip file created in step 3 to Lambda, edit the /src/MirrorMirror.js file and add a new line under region: "us-east-1", (~line 16) and add host: "YOURID.iot.us-east-1.amazonaws.com", so the entire section looks like this:
app.setup = function() {   app.device = awsIot.device({     keyPath: __dirname + "/certs/MagicMirror.private.key",     certPath: __dirname + "/certs/MagicMirror.cert.pem",     caPath: __dirname + "/certs/root-CA.crt",     clientId: "MirrorMirror" + (new Date().getTime()),     region: "us-east-1", 	host: "YOURID.iot.us-east-1.amazonaws.com",   });
You need to get your IOT endpoint created at the beginning for that line - to do that, go to AWS > IoT > Settings and copy the value under Custom Endpoint.
Credit to torresfm
After making that addition, save the MirrorMirror.js file and then create your zip for upload.

I'm still stuck at the creating a cert. I get the error that there is no such file or directory, open '/home/pi/package.json'
Any ideas what I missed??

Are you referring to creating the certificate using the script downloaded
from Amazon?
On Wed, Mar 7, 2018, 2:19 AM jer78<EMAIL_ADDRESS>wrote:

I'm still stuck at the creating a cert. I get the error that there is no
such file or directory, open '/home/pi/package.json'
Any ideas what I missed??
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/joanaz/MirrorMirrorOnTheWallSkill/issues/21#issuecomment-371046678,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC__ZPXqTxBHv2gb9N_E_mu5vmKjiP6xks5tb4nogaJpZM4SYaB3
.

Yes, I download the zip from Amazon, unzip, set permissions, then run i and get this error. It doesn't seem to be in the zip from Amazon. Where do I get this package.json?

Looking at the script downloaded from Amazon, it doesn't have anything specific about a package.json on it, the only thing it requires is some node modules. Did you already configure MagicMirror (which installs Node?)
# install AWS Device SDK for NodeJS if not already installed
if [ ! -d ./node_modules ]; then
  printf "\nInstalling AWS SDK...\n"
  npm install aws-iot-device-sdk
fi

Yes Node is installed. I think I know what the problem was. I needed to use sudo, otherwise the it didn't install aws-iot-device-sdk. I'll give it another try this weekend.

Hello,
I seem to get the same problem as # @jer78.
I downloaded the connection kit zip file from Amazon, unzipped it under /home/pi/connect_device_package and ran the start.sh script and I get the following warning:

Can anyone please clarify what am I missing or doing wrong?
Regards,
Andrew

I have configured all the lambda instance and alexa developer console as per the instruction for raspberrypi 3 ,model B. But when i am testing from alexa console i am only getting "Hello, queen what i can do for you" response. And for rest of the commands i am getting "There is problem with the response".? Can anyone plz help me to figure this out.

iotMice,
Are you entering the other commands after the wake word? With Alexa, you
have to invoke the skill first by using the skill wake command. Once you
get the response of "hello my queen" you should be able to try something
like "show me pictures of apples"
On Wed, Oct 3, 2018, 9:19 AM iotMice<EMAIL_ADDRESS>wrote:

I have configured all the lambda instance and alexa developer console as
per the instruction for raspberrypi 3 ,model B. But when i am testing from
alexa console i am only getting "Hello, queen what i can do for you"
response. And for rest of the commands i am getting "There is problem with
the response".? Can anyone plz help me to figure this out.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/joanaz/MirrorMirrorOnTheWallSkill/issues/21#issuecomment-426634236,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC__ZMd6u-BDTOecIQFkqN1NfnDerb6rks5uhLlOgaJpZM4SYaB3
.

Thanx for the reply. But i have figured out the mistake. Basically,
connection from my raspberrypi to aws was not happening. After changing few
permissions from aws device policy it worked fine.
On Wed, Oct 3, 2018 at 7:02 PM Jon<EMAIL_ADDRESS>wrote:

iotMice,
Are you entering the other commands after the wake word? With Alexa, you
have to invoke the skill first by using the skill wake command. Once you
get the response of "hello my queen" you should be able to try something
like "show me pictures of apples"
On Wed, Oct 3, 2018, 9:19 AM iotMice<EMAIL_ADDRESS>wrote:

I have configured all the lambda instance and alexa developer console as
per the instruction for raspberrypi 3 ,model B. But when i am testing
from
alexa console i am only getting "Hello, queen what i can do for you"
response. And for rest of the commands i am getting "There is problem
with
the response".? Can anyone plz help me to figure this out.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<
https://github.com/joanaz/MirrorMirrorOnTheWallSkill/issues/21#issuecomment-426634236
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AC__ZMd6u-BDTOecIQFkqN1NfnDerb6rks5uhLlOgaJpZM4SYaB3
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/joanaz/MirrorMirrorOnTheWallSkill/issues/21#issuecomment-426638538,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AoYAokbjdECZyq2_jGt866Ogm0PDktQtks5uhLxzgaJpZM4SYaB3
.

Glad you got it figured out. I got mine working so many months ago and forgot all of the troubleshooting I did :)

@andrew16x your screenshot tells me it's not a problem, as long as you get a root-CA.crt file generated in that step, you can ignore other errors.


Matt slim down mobile

src/js/mercator.js
802 - I would prefer we wrap if/else bodies in curly braces

On Chrome mobile view, the pull tab to open the sidebar doesn't seem to appear:

Whereas on the iOS simulator, it does appear:

Mobile UI is fixed. 👍
Galaxy S5 / Chrome:

iPhone 8:


perf: add support for OpenTelemetry with Jaeger collector

This PR adds support for OpenTelemetry tracing. Trace data are sent to the Jaegar endpoint. The latter should be run from within test/bdd/fixtures/jaeger with docker-compose up. Jaeger UI will be available here (update /etc/hosts with <IP_ADDRESS> jaeger.example.com entry).
Most probably that tracing functionality will be removed before release but for now it should help with investigating performance issues.
Signed-off-by: Andriy Holovko<EMAIL_ADDRESS>

We should better understand jaeger vs Prometheus


Add 'Access-Control-Allow-Headers': '*' to headers of DevServer configuration

🐞 Bug report
Command (mark with an x)

[ ] new
[ ] build
[x] serve
[ ] test
[ ] e2e
[ ] generate
[ ] add
[ ] update
[ ] lint
[ ] xi18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Is this a regression?

 No
Description

I'm currently building a micro front-end solution. In this I am loading files from one dev server in an other dev server using XHR. This works, but when I am adding applicationInisighs.js, this wants to add the request-id header to the CORS calls. This results into a CORS Error.
🔬 Minimal Reproduction

🔥 Exception or Error

Access to XMLHttpRequest at 'http://localhost:9010/assets/i18n/en.json' from origin 'http://localhost:9009' has been blocked by CORS policy: Request header field request-id is not allowed by Access-Control-Allow-Headers in preflight response.

🌍 Your Environment

Package                            Version
------------------------------------------------------------
@angular-devkit/architect          0.901.0
@angular-devkit/build-angular      0.901.0
@angular-devkit/build-ng-packagr   0.901.0
@angular-devkit/build-optimizer    0.901.0
@angular-devkit/build-webpack      0.901.0
@angular-devkit/core               9.1.0
@angular-devkit/schematics         9.1.0
@angular/cdk                       9.1.3
@angular/material                  9.1.3
@ngtools/webpack                   9.1.0
@schematics/angular                9.1.0
@schematics/update                 0.901.0
ng-packagr                         9.1.0
rxjs                               6.5.4
typescript                         3.7.5
webpack                            4.42.0

Anything else relevant?

Adding 'Access-Control-Allow-Headers': '*' to the following line would fix the problem.
https://github.com/angular/angular-cli/blob/master/packages/angular_devkit/build_angular/src/dev-server/index.ts#L457

In version 11, a new headers option was added which can be used to add custom headers.
See: https://github.com/angular/angular-cli/pull/19015


Feature request: Add hot reload support for transpiled languages

At ts-defold, we run the TypescriptToLua transpiler to convert Typescript to Lua. Since we use .ts files, they don't trigger hot reload in Defold Buddy.
I was able to add support in a fairly naive way: add '.ts' as a file extension that triggers hot reload, and add a slight delay to give the transpiler time to finish before calling the hot reload. View commit. This isn't the most elegant solution, but it works for me.
I understand this may be outside the scope of this extension, but I thought I'd start the discussion.

I would be glad to help add support for it. I tried ts-defold a while ago and can't remember how it worked. If it compiles to .lua files then in theory VS Code should detect that .lua files changed so it's a bit strange that it doesn't work.

Fixed in v1.4.0.

Works great. Thank you!


"swap limit capabilities" fail task

An option resources in deploy section allow to fail task after several (5-10) http requests to container.
journalctl -f -u docker.service
Dec 12 17:26:05 box-12280 dockerd[986]: time="2017-12-12T17:26:05.590714847+03:00" level=warning msg="Your kernel does not support swap limit capabilities,or the cgroup is not mounted. Memory limited without swap."
Dec 12 17:26:21 box-12280 dockerd[986]: time="2017-12-12T17:26:21.912492289+03:00" level=warning msg="Your kernel does not support swap limit capabilities,or the cgroup is not mounted. Memory limited without swap."
Dec 12 17:26:33 box-12280 dockerd[986]: time="2017-12-12T17:26:33.854011566+03:00" level=error msg="fatal task error" error="task: non-zero exit (1)" module="node/agent/taskmanager" node.id=brl973mia7iiirzwis8zliqy9 service.id=ndre6ij1ybqthqum678pnhpv8 task.id=vii5t6u3ilmp5595pdabc0dzd
Dec 12 17:26:34 box-12280 dockerd[986]: time="2017-12-12T17:26:34.070025675+03:00" level=warning msg="Your kernel does not support swap limit capabilities,or the cgroup is not mounted. Memory limited without swap."

uname -a
Linux box-12280 4.4.0-103-generic #126-Ubuntu SMP Mon Dec 4 16:23:28 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux

docker version
Client:
 Version:      17.09.0-ce
 API version:  1.32
 Go version:   go1.8.3
 Git commit:   afdb6d4
 Built:        Tue Sep 26 22:42:18 2017
 OS/Arch:      linux/amd64

Server:
 Version:      17.09.0-ce
 API version:  1.32 (minimum version 1.12)
 Go version:   go1.8.3
 Git commit:   afdb6d4
 Built:        Tue Sep 26 22:40:56 2017
 OS/Arch:      linux/amd64
 Experimental: false

docker-compose.yml
networks:
  somenet:
    driver: overlay
    attachable: true

services:
  someservice:
    deploy:
      resources:
        limits:  #
          cpus: '0.25'
          memory: 50M
      placement:
        constraints:
          - node.hostname == box-12280
    image: some.registry/someservice
    ports:
      - 8000:8000
    networks:
      - somenet
    volumes:
      - stage-nm:/var/dir/node_modules
      - stage-bc:/var/dir/app/bower_components
version: '3.4'
volumes:
  stage-nm:
  stage-bc:

Dockerfile
FROM ubuntu:latest

RUN apt -y update --fix-missing
RUN apt install -y nano git curl make python g++ nodejs npm --no-install-recommends

RUN ln -s /usr/bin/nodejs /usr/bin/node

RUN npm install bower -g

ADD app /var/dir/app
ADD .bowerrc .bowerrc bower.json package.json /var/dir/

VOLUME ["/var/dir/node_modules", "/var/dir/app/bower_components"]

WORKDIR /var/dir

EXPOSE 8000

ENTRYPOINT ["npm"]
CMD ["start"]

@zymtx5g79k please open this issue on https://www.github.com/moby/moby. Sorry for the trouble.


Use groupified versioning in Templates

Recent versions of oc have started throwing warnings about using ungroupified versions (v1 instead of template.openshift.io/v1) when using new-app, they are warnings for now but they might become unsupported later.
Other usages of such versioning were not causing the warnings to I didn't change them (notably, inside the templates)

Sigh... this so pointless change, just because some decided that breaking changes a are good (yes, I agree likely one day former format will stop working)


Toolbar Refactoring

Toolbar Refactoring
We decided to refactor the toolbar to a more simple and generic component in order to increase flexibility and improve its utility in a wider range of use cases.
Toolbar Base

The toolbar is 36px tall to match the height of our dropdowns and buttons.
All items not this height (icons, text) should be centered vertically within the height.
There should be no spacers on the left and right edges of the toolbar if there are already padding spacers being used with whatever component the toolbar is placed within (e.g. cards).

Toolbar Items: Group
Items can be grouped within the toolbar. Items within a group have 16px spacing between them. Items that are made up of multiple parts (for example the filter dropdown and input group) are treated as one item in regards to the spacing.
Below are examples of: a group of icons, a group of buttons, a group of icons and buttons. Note that no matter what the item is, it always has 16px of spacing on both sides.

Toolbar Items: Multiple Groups
A toolbar can have more than one group of items. Between groups there should be 24px of spacing.
Below is a group of icons next to a group of buttons. Note the cyan 24px spacer between them.

Toolbar Items: Single, ungrouped icons are treated as a group.
A single item that doesn't belong to a larger group is treated as a group of its own. It uses a 24px spacer on either side.
Below is a single button next to a group of icons.

Example: Card with toolbar
Below is an example of a toolbar within a card. This card has a header and a body section, the toolbar should always be placed in the header. The toolbar has a single button alongside a group containing action icons. Note how the toolbar has no padding on the right side as the padding is taken care of by the card component padding.

FYI
@mcoker @rachael-phillips @kybaker @mcarrano

@mceledonia agree that this refactoring makes sense to create a more flexible toolbar as we previously discussed.
@rachael-phillips @mcoker do we have this on the roadmap as something to deliver for April release? Sounds to me like this would be a breaking change so it would be good to get in sooner rather than later. Does this refactoring also require rework for React?

@mcarrano We could introduce it as a new component, then deprecate the old one, to add the component as a step, then at a later step, remove the old one. This will also involve breaking changes in at least the page layout (masthead buttons/menus, though not sure how the new masthead design might impact this), and potentially the form layout for the action row of buttons, whenever we decide to implement those changes.

@mcoker and I discussed this and wanted to offer the following as a potential path forward.
We can keep the existing Toolbar component which was designed for a very specific use case to sit on top of a table or list view. This is useful for anyone creating a standardized data view. We can then take this "refactored toolbar" into a new component that can be used in a more generalized way - for layout out masthead utility items, actions, or filters within a card. It can also be useful if someone needs to roll their own toolbar that does not fit neatly into one of the templates defined in the current Toolbar component.
I think the only challenge is what to call the new thing. It's still technically a toolbar, but we want to differentiate it. We were thinking maybe "Action Bar," but are open to ideas.
Thoughts @mceledonia @rachael-phillips ?

My question would the new Action bar cause breaking changes?

@rachael-phillips adding the new component wouldn't cause a breaking change. And using it in the page header (masthead utility items) wouldn't necessarily be a breaking change if we add support for the toolbar to be used and don't remove the old styles that support the current layout. Then we can go back later in a breaking changes release and remove support for the current layout and force users to use the toolbar there instead. I imagine we would also eventually replace the current toolbar with this toolbar (or action bar) as well.

What do you think @mceledonia ?

Additional proposal for a vertical divider between item groups, mainly utilized when there are two or more groups of the same item types next to each other (e.g. two groups of icons next to each other as seen below). In other words this is to be used when just the spacing is not enough of a visual separator between groups.

@mcoker @kybaker

Re: the order of the buttons up on this toolbar. Everywhere we're doing "Primary, Secondary, Tertiary" (on the bottom of forms, in wizards, etc) and in this toolbar design its "Tertiary, Secondary, Primary".
It would be better to have the buttons in the same order all the time to reduce confusion & enhance consistency.
(Unrelated, I have opinions about the ordering of the buttons in general, but consistency is key!)

I think @katierik makes a good point. On toolbar actions we have always had the primary action as the first (leftmost) button. I think this was an unintentional change. @mceledonia can you update your mock to avoid any confusion?

I agree 100% wherever we land, we should be consistent. It sounds like we might need to revisit button placement and have a larger discussion around it. Maybe we can find some time this week to meet and discuss?

Hey all, thought I'd add my thoughts here.
I think that we're interpreting what is a component that uses display: flex; as a unique component with its own unique layout. It is correct that this design shows a toolbar, the approach to layout should be consistent with anything else that is a flexbox layout. Therefore, I think it's a good idea to create a toolbar component that uses the pf-l-flex layout.
Adding markup/grouping items to control margin seems counterintuitive and unnecessary to me and I've heard the same feedback from others (on my team). Like, there has to be more than 1 thing to make a group. Rather, a modifier could be added to an item or conditional css for that item could be added..

@mattnolting should this issue be closed?

@mcarrano No, it should be left open. The pf-l-flex layout has been updated and I'm in the process of applying it to toolbar. Summit derailed development, but picking it back up now.

@mattnolting what's the status on this issue and how is it going? :)

@rachael-phillips After meeting w/team members about approach last week, we came to the conclusion that instead of applying selectors .pf-l-flex and modifiers to the toolbar (and any other component that relies on our standardized flex layout) effectively creates a blueprint for, rather than a complete, autonomous component and would ship with a much larger module size. As a result, I'm in the process of declaration abstraction by converting modifiers to rely on sass mixins/placeholders. This will allow us to ship only the required amount of css to support a given component, rather than a component and a complete layout.
The bulk of the css work is done and am working out some details. Documentation has been started with examples. It's going well :).

Thanks so much @mattnolting for the update!

Adding link to Toolbar Component System design doc for future reference: https://docs.google.com/document/d/12oxvXul8BxxjotRWSpJefyuiqcSm_gOYscEOXZXaqaI/edit?usp=sharing

Toolbar examples (should be able to support these)
https://marvelapp.com/project/4205050/
https://marvelapp.com/project/4204560/
https://marvelapp.com/project/4148169/
https://marvelapp.com/project/4100287/


Z coordinates of children isn't taken into account

This issue occurs in 0.8.* and 0.9.*.
This is slightly different than https://github.com/bevyengine/bevy/issues/6838. I was recommended to use the z-axis as z-index instead of ZIndex. However, it doesn't work as expected since the child's z-index doesn't seem to be taken into account:
// The building
commands
    .spawn((
        Building::GeneralShop,
        crate::game::OutsideWorld,
        SpriteSheetBundle {
            texture_atlas: texture.clone(),
            sprite: TextureAtlasSprite {
                index: 0,
                custom_size: Some(Vec2 {
                    x: GENERAL_SHOP_WIDTH,
                    y: GENERAL_SHOP_HEIGHT,
                }),
                ..default()
            },
            // z to 2 here
            transform: Transform::from_xyz(x, y, 2.0),
            ..default()
        },
    ))
    .with_children(|children| {
        children.spawn((
            SpriteSheetBundle {
                texture_atlas: texture,
                sprite: TextureAtlasSprite {
                    index: 1,
                    custom_size: Some(Vec2 {
                        x: GENERAL_SHOP_WIDTH,
                        y: GENERAL_SHOP_HEIGHT,
                    }),
                    ..default()
                },
                // z to 0 here
                transform: Transform::from_xyz(0., 0., 0.),
                ..default()
            },
            ZIndex::Global(1),
        ));
    });

// the player
commands
    .spawn((
        Player {
            is_running: false,
            waiting_for_rerun: false,
            old_x: 0.,
            old_y: 0.,
        },
        SpriteSheetBundle {
            texture_atlas: texture_atlas_handle,
            sprite: TextureAtlasSprite {
                custom_size: Some(Vec2 {
                    x: PLAYER_WIDTH,
                    y: PLAYER_HEIGHT,
                }),
                ..default()
            },
            // z to 1 here
            transform: Transform::from_xyz(0.0, 210.0, 1.0),
            ..default()
        },
    ));

In here, I have split my building in two parts: the roof and the rest so that the character passes under the roof but stay on top for the door. Little screenshot:

As the low part of the building has a z-index of 0, it should be under the player which has a z-index of 1, however it's not the case.

The transform of the child is relative to the parent's transform.
To have the global Z position of the child at zero, the child's Z translation should be -2.0 in this case.

Oh I see. Thanks!


Connection reuse if already established for UNL.connect

Connection synchronization was added to avoid the problems with contracts being associated with the wrong con and this solves the problem of making multiple requests to the same node in a short period. However, since transfers can be queued over the same connection there should also be some simple code to avoid creating a new connection if one already exists and just queue the transfer.
This should be easy to add in theory but it will also need tests (for reuse + rapid succession) since it changes the characteristics quite a lot.
(This is especially important as UNL.connect may consume a large number of source ports unnecessarily if this optimisation isn't added so its definitely worth adding queuing back.)

Queued transfer tests are still failing. Issue is intermittent. Not sure what the cause is yet.


ADD: TON support

As discussed in Discord we should add TON support to our wallet.
Please talk with @enriquesouza what will be most feasible way to implement it, as he alreadz started to investigate.

not possible yet , need to wait until wallet core add TON support first

QA done.
Able to receive and send funds:
https://tonscan.org/tx/2Sx8ysCCV4nr/ceJIMKiMzM8AUkJ8jYF4cC9qbFCliY=


It is not clear what the "SIZE" of a service is

From an outsider point of view, it is not clear what "SIZE" means. It should be self-explicative like "Size of your entity" or similar. Moreover, there should be a link to somewhere where it is explained what the size is, what is used for and how you can determine the size of your entity.
Finally, all the available sizes should be displayed, not just one.

Fixed.


Hot fix audience

This should be applied to the CLI immediately to correct a discrepancy between the audience used for user and service credentials.
This code was tested manually using https://github.com/HumanCellAtlas/dcp-cli/blob/d59c02e87f1fae64d37037036e6765ae7ec1e793/test/integration/dss/test_dss_api.py#L282

Codecov Report

Merging #193 into master will not change coverage.
The diff coverage is 50%.

@@           Coverage Diff           @@
##           master     #193   +/-   ##
=======================================
  Coverage   83.52%   83.52%
=======================================
  Files          37       37
  Lines        1463     1463
=======================================
  Hits         1222     1222
  Misses        241      241

Impacted Files
Coverage Δ

hca/util/__init__.py
89.26% <50%> (ø)
:arrow_up:

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 d59c02e...f450144. Read the comment docs.