Wrong output results when using execute_qasm

Bug report
What did you try to do?
When I execute circuits using execute_qasm, I get thrown an experiment optimization warning with the message that the experiment might take longer to execute. The obtained results are wrong. For example, when I execute the following code
from quantuminspire.credentials import get_authentication
from quantuminspire.api import QuantumInspireAPI

authentication = get_authentication()
QI_URL = r'https://api.quantum-inspire.com'

name = 'My-Test'
qi = QuantumInspireAPI(QI_URL, authentication, project_name=name)
backend = qi.get_backend_type_by_name('QX single-node simulator')

qasm = '''version 1.0

qubits 2

H q[0]
CNOT q[0], q[1]
H q[0]
'''

backend_type = qi.get_backend_type_by_name('QX single-node simulator')
result = qi.execute_qasm(qasm, backend_type=backend_type, number_of_shots=1)

print(result['histogram'])

The resulting print statement is
OrderedDict([('0', 1.0)])

Which is not the correct output for this simple circuit. When I look in My QI, I see the correct circuit with the wrong results. When I execute the circuit again from within MyQI everything seems to work just fine. When I execute the same code using Qiskit everything seems to work for me.
What did you expect to happen?
Output for the given example:
OrderedDict([('0', 0.25), ('1', 0.25), ('2', 0.25), ('3', 0.25),])

What actually happened?
Output for the given example:
OrderedDict([('0', 1.0)])

Environment
Please specify:

Python version: Python 3.7.9
qiskit==0.30.1

It was pointed out to me that adding the flag full_state_projection=True will solve my problems.
...

result = qi.execute_qasm(qasm, backend_type=backend_type, number_of_shots=1)
print(result['histogram'])
# OrderedDict([('0', 0.25), ('1', 0.25), ('2', 0.25), ('3', 0.25)])

Or with more shots
...

result = qi.execute_qasm(qasm, backend_type=backend_type, number_of_shots=1024)
print(result['histogram'])
# OrderedDict([('1', 0.24609375), ('0', 0.2470703125), ('2', 0.23828125), ('3', 0.2685546875)])

It is however still unclear to me what the default setting full_state_projection=False is supposed to return and why it always returns  OrderedDict([('0', 1.0)]).

@RobertWezemanTNO Hi Robert, I will update the example in the README. Thanks for the suggestion.


Dataset Issue

Nice implementation! I wonder how you selected your training and testing images. Thanks!

Training and testing images were selected at random from imagenet

Hi, how many images have you used for training ? Also it looks like you have run the training for 24 thousand epoch??
I tried the same network  (keras) with 50k images.However, i'am confused how long (and how much images) i should train, since it takes 90 minutes for 1 epoch...
Also even though validation loss goes to 0.0069,a green outline of secret image seems to be always visible in the container image(i tried upto 20 epochs with this setup) ...
Here is my network...Looks like i'am missing something !!!


cJSON exporting symbols by default

It seems that by default, since none of these are defined: CJSON_HIDE_SYMBOLS, CJSON_IMPORT_SYMBOLS or JSON_EXPORT_SYMBOLS, it defaults to CJSON_EXPORT_SYMBOLS (see cJSON.h), which means that the sdk will generate some __declspec(dllexport) even if being statically linked. This ends up creating a .lib and .exp alongside the .exe output of anything linking against the aws sdk static libraries.
It'd be useful if BUILD_SHARED_LIBS is OFF, the cmake for aws defined CJSON_HIDE_SYMBOLS for its static library, or make it depend on USE_IMPORT_EXPORT.
What platform/OS are you using?
Windows 10 x64
What compiler are you using? what version?
MSVC 2017
What's your CMake arguments?
cmake . -DBUILD_ONLY="dynamodb" -DENABLE_TESTING="OFF" -DBUILD_SHARED_LIBS="OFF" -DENABLE_UNITY_BUILD="ON" -DFORCE_SHARED_CRT="OFF" -G "Visual Studio 15 2017 Win64"

Thank you for your suggestions, we've added them in https://github.com/aws/aws-sdk-cpp/commit/abfe95d7125b39ab23cbc604f4c9728010e71a12.


Contact form

Comments popup
When the popup loads, the webapp retrieves data from:
The selected API and shows details about the selected item.
The Involvement API to show the item comments.
When the user clicks on the "Comment" button, the data is recorded in the Involvement API and the screen is updated.
Cheers !

Hi @ishimwepati
Your project is complete! There is nothing else to say other than... it's time to merge it :
Congratulations! 🎉
Highlights
Comment Popup counter✔
Comment Popup display✔


Store sessions in the database

[!IMPORTANT]
Care should be taken when deploying this change. To reduce the risk of disrupting live user sessions, it should be deployed out of hours.
Users will be signed out because their existing sessions will be discarded. However the app is currently low-traffic enough that an out of hours deployment seems like a reasonable mitigation.

Context
Currently we use the cookie session store. This is the default session store which comes out of the box with Rails. The cookie store mostly works fine and is a good default, but there is one major drawback: storage space is limited to ~4 KB.
From experience, it's surprisingly easy to use up that space. Features such as multi-step forms (a.k.a. wizards) will often depend on using the session as temporary storage – and since such forms accept user input, the amount of session storage required will consequently be unpredictable and vary wildly.
If you over-fill session cookies, the application will raise CookieOverflow errors. As abundantly tasty as these errors sound, they don't make for a good user experience. Users will randomly start seeing the "something went wrong" page, and the server will give 500 Internal Server Error responses.
Thankfully Rails provides other session stores which don't have a hard limit on storage.
Changes proposed in this pull request
I've set up the Active Record session storage adapter. This means user sessions will be stored in the database, and we'll be able to read & write to sessions without worrying about the ~4 KB storage limit.
As recommended in the documentation, I have also set up a scheduled job which will prune the database of old & expired sessions. However rather than using the db:sessions:trim Rake task, it ended up being simpler to re-implement the trim functionality as an Active Job so it can be scheduled via Good Job.
Guidance to review

This PR should not result in any user-facing changes.
Does everything work the same as it did before?

Link to Trello card
https://trello.com/c/AVwbE2ax/504-use-a-database-backed-session-store

Thanks for the quick reviews @Kizr & @Nitemaeric! ⭐

1 thing to note with the TrimExpiredSessionsJob is that as we schedule this hourly, the session expiration may last 2 hours and 59 minutes if a user signs in at just the right time.

That's technically true, although in practice it won't make a difference because the user authentication always expires after 2 hours regardless. That is also the case with the current cookie-backed sessions which could technically last for any length of time (it's up to the browser when they get deleted).
This cleanup job is primarily there just to keep the database from filling up with dead sessions. But the frequency of the schedule isn't dictated by the length of the session timeout itself.


fix tensorflow travis config

ref issue #1103

Cool, thanks. However, it looks like it doesn't solve the TensorFlow installation issue: https://travis-ci.org/fchollet/keras/jobs/93735270

Cool, thanks!

https://github.com/fchollet/keras/pull/1016#issuecomment-156842267
I closed my PR till python 3 version...

A few days ago we decided to release next Tuesday or so, and we will need Travis integration before then. Python 2.7-only it is.


scipy.optimize.minimize(method='COBYLA') not threadsafe

I tried to run multiple optimisations with the cobyla method at the same time. Due to the asyncio based  communication with the modelling/calculation framework, I decided to use a multithreaded approach.
Platform/Distribution

Python3.7.2/scipy-1.1.0  via conda on Debian testing
Python3.7.2/scipy-1.1.0 via apt on Debian testing

In order to showcase my problems, I slimmed down the code to this example:
import concurrent.futures
import time
import scipy.optimize

def objective1(x):
    print(x, "1")
    time.sleep(0.1)
    return x[0]**2

def objective2(x):
    print(x, "2")
    time.sleep(0.1)
    return (x[0]-1)**2

min_method = "COBYLA"

def minimizer1():
    return scipy.optimize.minimize(objective1,
                                   [0.0],
                                   method=min_method)

def minimizer2():
    return scipy.optimize.minimize(objective2,
                                   [0.0],
                                   method=min_method)

with concurrent.futures.ThreadPoolExecutor() as pool:
    tasks = []
    tasks.append(pool.submit(minimizer1))
    tasks.append(pool.submit(minimizer2))

    for t in tasks:
        print(t.result())

(Yes, the original code used args, but I wanted to simplify it even more).
This code's output reads:
[0.] 1
[0.] 2
[1.] 2
[1.] 2
[0.125] 2
[2.] 2
[-0.25] 2
[1.5] 2
[0.125] 2
[0.75] 2
[-0.0625] 2
[1.125] 2
[0.03125] 2
[0.9375] 2
[-0.015625] 2
[1.03125] 2
[0.0078125] 2
[0.984375] 2
[-0.00390625] 2
[1.0078125] 2
[0.00195312] 2
[0.99609375] 2
[-0.00097656] 2
[1.00195312] 2
[0.00048828] 2
[0.99902344] 2
[-0.00024414] 2
[1.00048828] 2
[0.0001] 2
[0.99975586] 2
[-0.0001] 2
[1.0001] 2
     fun: 1.00020001
   maxcv: 0.0
 message: 'Optimization terminated successfully.'
    nfev: 16
  status: 1
 success: True
       x: array([-0.0001])
Call-back cb_calcfc_in__cobyla__user__routines failed.
Segmentation fault

(Obviously) The segmentation fault is unexpected, but also looking at the sequence of values - features 32 lines and roughly alternating objective values from objective 1 and 2, but only objective 2 seems to be evaluated.
The stack trace is:
#0  0x00007f5e94d24dc0 in f2py_rout.cobyla_minimize () from /home/XXXX/lib/python3.7/site-packages/scipy/optimize/_cobyla.cpython-37m-x86_64-linux-gnu.so
#1  0x000055b6ce95146b in _PyObject_FastCallKeywords (callable=0x7f5e94d73670, stack=0x7f5e88001680, nargs=1, kwnames=0x7f5e94d74a08) at /tmp/build/80754af9/python_1546061345851/work/Objects/call.c:199
#2  0x000055b6ce9ac378 in call_function (kwnames=0x7f5e94d74a08, oparg=<optimized out>, pp_stack=<synthetic pointer>) at /tmp/build/80754af9/python_1546061345851/work/Python/ceval.c:4619
[....]

There are two ways to avoid the problems above:

use a ProcessPoolExecutor
use the minimizer SLSQP

Here the output for the ProcessPoolExecutor:
[0.] 1
[0.] 2
[1.] 1
[1.] 2
[2.] 2
[-1.] 1
[-0.5] 1
[1.5] 2
[0.25] 1
[0.75] 2
[1.125] 2
[-0.125] 1
[0.9375] 2
[0.0625] 1
[-0.03125] 1
[1.03125] 2
[0.015625] 1
[0.984375] 2
[1.0078125] 2
[-0.0078125] 1
[0.99609375] 2
[0.00390625] 1
[-0.00195312] 1
[1.00195312] 2
[0.00097656] 1
[0.99902344] 2
[-0.00048828] 1
[1.00048828] 2
[0.00024414] 1
[0.99975586] 2
[1.0001] 2
[-1.e-04] 1
[0.0001] 1
[0.9999] 2
     fun: 1e-08
   maxcv: 0.0
 message: 'Optimization terminated successfully.'
    nfev: 17
  status: 1
 success: True
       x: array([0.0001])
     fun: 9.999999999997797e-09
   maxcv: 0.0
 message: 'Optimization terminated successfully.'
    nfev: 17
  status: 1
 success: True
       x: array([0.9999])

This output looks like I expect it.
I had a quick look at the Fortran/Python interface - sadly, I don't know enough about f2py to pin point the problem in the repository/code.

same with scipy-1.2.0 (python 3.7 on debian testing - pip install)

Problem still exists in Python and scipy version 1.4.1.
Is there any workaround or plan for a fix?

The evaluation of only objective2 is as expected, due to the GIL.
It seems the segfault happens when there is a context switch between the threads.

I think this might be an f2py issue.

The pyf-file is autogenerated and there is no handwritten C in the wrapper. The GIL is not released so it is not an issue with Fortran not being reentrant. I am thinking it might be indicative of a bug in f2py. @pv

f2py belongs to NumPy so the only thing we can do for now is to force single-threaded access to cobyla.

There is upstream fix in https://github.com/numpy/numpy/pull/16519 but it requires that SciPy be built against NumPy version 1.19 or later.


Dynamic File Links (Dreamweaver-esque)

Feature Request
In Dreamweaver 5.5, there is a bar below and attached to each tab that lets one simply click on the name of a file reference to access said file, you can also filter the dynamic links by extension in Dreamweaver.

I think you are referring to a bread-crumb like navigation control in the editor. We have an issue for that, so closing as duplicate.


[SPARK-17477][SQL] SparkSQL cannot handle schema evolution from Int -…

What changes were proposed in this pull request?
Using SparkSession in Spark 2.0 to read a Hive table which is stored as parquet files and if there has been a schema evolution from int to long of a column, we will get java.lang.ClassCastException: org.apache.spark.sql.catalyst.expressions.MutableLong cannot be cast to org.apache.spark.sql.catalyst.expressions.MutableInt. To be specific, if there are some old parquet files using int for the column while some new parquet files use long and the Hive metastore uses Long as its type, the aforementioned exception will be thrown. Because Hive and Presto deem this kind of schema evolution is valid, this PR allows writing a int value when its table schema is long in hive metastore.
This is for non-vectorized parquet, will create a separate JIRA for vectorized parquet reader and come up with a fix later.
How was this patch tested?
Manual test to create parquet files with int type in the schema and create hive table using long as its type. Then perform spark.sql("select * from table") to query all data from this table.

Can one of the admins verify this patch?

Do you mind if I ask to fix the title to be complete withoit ... ?

-1 : As far as I know, we are picking up a single Parquet file to read Spark-side schema. In this case, it is ambiguous to decide which one is "new" and "old". So, sometimes it'd be failed to read long as int and sometime it'd succeed to read int as long.
I guess we need to enable merge schemas to support to infer schema from Parquet first but we are not supporting merging schemas with upcasting - SPARK-15516. So, IMHO, SPARK-15516 blocks this.
If we talk about the case of setting the schema explicitly in this case, then, it'd turn into the subset of SPARK-16544. In this case, I submitted a PR already https://github.com/apache/spark/pull/14215 but I decided to close for a better approach. If this looks good, I'd like to bring and re-open my old PR.

@HyukjinKwon Yup this PR is very similar to yours.
For merging parquet schema, it won't work. Think about this: the table contains two parquet files, one has int, one has long. The DataFrame schema uses long (mergeSchema will also result in this case). So when reading the parquet file with Int, we still run into this problem.

Yea. I meant if we want to read old/new Parquet files without user-given schema with enabling merging schemas, then, we'd face SPARK-15516 first. This is why I thought that JIRA blocks this case.


Fix unit and scale depending on calculations

Cumulative calculations: change unit on the table to MtCO2 and scale accordingly
http://localhost:3000/ghg-emissions?calculation=CUMULATIVE

Percentage calculations: change unit on the table to %

I added a scroll to the last year and changed the Table component to hooks

I'm wondering if I should check if rightTable exists too but it should work like this


create and load luts from common lut formats

The cube format is text based and is probably the simplest to implement. This would allow creating LUTs in Photoshop or Affinity photo and previewing the results without testing in game.
https://wwwimages2.adobe.com/content/dam/acom/en/products/speedgrade/cc/pdfs/cube-lut-specification-1.0.pdf
A layer stack would look like the following.

levels adjustment to simulate post processing
lut adjustment layer using loaded 16x16x16 cube file
base layer

Test and document support for

[ ] Affinity Photo
[ ] Gimp 2.1
[ ] Photoshop CS6 (later versions also support export)

Test and document support for

[ ] Affinity Photo
[ ] Gimp 2.1
[ ] Photoshop CS6 (later versions also support export)


Cannot start nuxt:  Cannot find module 'nuxt3'

Environment

Operating System: Darwin
Node Version:     v16.14.2
Nuxt Version:     3.0.0-rc.1
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      runtimeConfig, modules, intlify, css, head, vite
Runtime Modules<EMAIL_ADDRESS>Build Modules:    -

Reproduction

Describe the bug
Commands npx nuxi dev and npx nuxi build throws error. Error can be seen in logs
Additional context
CONFIG
import { defineNuxtConfig } from "nuxt";

export default defineNuxtConfig({
  runtimeConfig: {
    DATABASE_URL: process.env.DATABASE_URL,
    CORAGEO_URL: process.env.CORAGEO_URL,
    CORAGEO_USER: process.env.CORAGEO_USER,
    CORAGEO_PASS: process.env.CORAGEO_PASS,
    CORAGEO_VERBOSE: process.env.CORAGEO_VERBOSE,
    ORGID: process.env.ORGID,
    UPLOAD_DIR: process.env.UPLOAD_DIR,
    API_URL: process.env.API_URL,
    EMAIL_VERIFICATION_EXPIRATION_DAYS: process.env.EMAIL_VERIFICATION_EXPIRATION_DAYS,
    PLATFORM: process.env.PLATFORM,
    SESSION_LIFESPAN_MINUTES: process.env.SESSION_LIFESPAN_MINUTES,
    RESET_PASS_EXPIRATION_DAYS: process.env.RESET_PASS_EXPIRATION_DAYS,
    public: {
      MAXSIZEMB: process.env.MAXSIZEMB,
      PAYMENT_URL: process.env.PAYMENT_URL,
      PAYMENT_KEY: process.env.PAYMENT_KEY,
    },
  },
  modules: ["@intlify/nuxt3"],
  intlify: {
    localeDir: "locales",
    vueI18n: {
      globalInjection: true,
      legacy: false,
      locale: "sk",
    },
  },
  css<EMAIL_ADDRESS>  head: {
    meta: [{ name: "viewport", content: "width=device-width, initial-scale=1" }],
  },
  vite: {
    css: {
      preprocessorOptions: {
        scss: {
          additionalData: "@import '@id-sk/frontend/idsk/all.scss';",
          quietDeps: true,
        },
      },
    },
  },
});

PACKAGE.JSON
{
  "private": true,
  "scripts": {
    "dev": "nuxi dev",
    "build": "nuxi build",
    "preview": "nuxi preview",
    "start": "cross-env NUXT_HOST=<IP_ADDRESS> node .output/server/index.mjs",
    "format": "prettier --config .prettierrc . --write"
  },
  "devDependencies": {
    "@intlify/nuxt3": "^0.1.10",
    "cross-env": "^7.0.3",
    "nuxt": "^3.0.0-rc.1",
    "prettier": "^2.6.2",
    "prisma": "^3.11.1"
  },
  "dependencies": {
    "@id-sk/frontend": "^2.9.0",
    "@prisma/client": "^3.11.1",
    "@vue-leaflet/vue-leaflet": "^0.6.1",
    "axios": "^0.26.1",
    "base64-img": "^1.0.4",
    "dayjs": "^1.11.0",
    "elmolis-corageo": "git+ssh://git@gitlab.elmolis.sk:Urbi/IntParkCoraGeo.git#devel",
    "hashids": "^2.2.10",
    "leaflet": "^1.7.1",
    "mustache": "^4.2.0",
    "sass": "^1.49.11",
    "sweetalert2": "^11.4.8"
  }
}

Logs
ERROR  Cannot start nuxt:  Cannot find module 'nuxt3'                                                                                15:31:03
Require stack:
- /Users/enimo/Desktop/Projects/Elmolis/PortalPresov/index.js

  Require stack:
  - index.js
  at Function.Module._resolveFilename (node:internal/modules/cjs/loader:933:15)
  at Function.resolve (node:internal/modules/cjs/helpers:108:19)
  at Function._resolve [as resolve] (node_modules/jiti/dist/jiti.js:1:192841)
  at resolveModule (node_modules/@nuxt/kit/dist/index.mjs:68:29)
  at importModule (node_modules/@nuxt/kit/dist/index.mjs:90:24)
  at loadNuxt (node_modules/@nuxt/kit/dist/index.mjs:798:43)
  at async load (node_modules/nuxi/dist/chunks/dev.mjs:6733:23)
  at async Object.invoke (node_modules/nuxi/dist/chunks/dev.mjs:6777:5)
  at async _main (node_modules/nuxi/dist/cli.mjs:46:20)

Facing this issue, tried all the steps from all the comments nothings seems working
 ERROR  Cannot start nuxt:  Cannot find any nuxt version from /home/manas/Documents/personal/myblog         10:21:48

  at loadNuxt (node_modules/@nuxt/kit/dist/index.mjs:533:11)
  at processTicksAndRejections (node:internal/process/task_queues:96:5)
  at async load (node_modules/nuxi/dist/chunks/dev.mjs:6805:23)
  at async Object.invoke (node_modules/nuxi/dist/chunks/dev.mjs:6874:5)
  at async _main (node_modules/nuxi/dist/cli.mjs:49:20)

@antfu @danielroe

@imanaspaul Would you open a new issue with a reproduction, please? 🙏

Check this :
https://stackoverflow.com/questions/76242903/nuxt-3-aliases-how-can-aliases-be-used-inside-a-config-file


Runtime SQLiteException IO error on WebGL platform

When I build and run WebGL build, it reports disk IO exception as below:
"SQLiteException: disk I/O error
at SQLite.SQLite3.Prepare2 (System.IntPtr db, System.String query) [0x00000] in <00000000000000000000000000000000>:0 "
I just testing WebGL with the demo in the readme to show player info in texts. But it failes now.
The Unity version I'm using is 2022.1.20f1. And I have selected the 3 files in Plugins/sqlite-amalgamation for WebGL platform.
Please refer to the log as below.

Hey @tealcwu, thanks for the report.
So, yeah, WebGL doesn't work with "real files" yet, we need a custom VFS for this first.
I started playing with a VFS based on the Indexed DB (#24) at some point. It actually works, but the problem is that saving data is too slow, a simple write query takes ~20ms, which is almost the entire frame time when playing at 30FPS.
Now, that is the case when using the rollback journal, turning journaling off or using in-memory journals perform ok (5~10ms).
I also want to try using SQLite's own Wasm/JS VFSs at some point. OPFS-VFS will likely perform better than my idbvfs.

And I have selected the 3 files in Plugins/sqlite-amalgamation for WebGL platform.

You don't need to do this on Unity 2022, this problem arised on Unity 2020, likely on 2021.2 as well, but 2021.3+ should be fine.

@gilzoide Hey Gil, thanks for your effort!
So you need more testing on different VFS strategies, right?

Hey @tealcwu, I found a much better way to handle I/O in idbvfs, making it quite faster.
If you'd like to try the code in branch idbvfs, feel free to do so.
Please share any feedback if you try it: if it works, if it's slow or not, etc...

Sure, Thanks Gil!

@gilzoide Test with idvfs branch in the same project. The new exception is cannot find table as below.
WebGL.framework.js.gz:3 SQLiteException: no such table: Player
at SQLite.SQLite3.Prepare2 (System.IntPtr db, System.String query) [0x00000] in <00000000000000000000000000000000>:0
This project works fine on Windows platform.
I'm sing Unity 2022.1.20f1 and Chrome 129.0.6668.70 (64bit).

Hmm, weird. Without a closer look at your project it's hard to tell what's going on.
Maybe the data was not persisted between sessions. Maybe the database wasn't opened correctly, even though the open call didn't throw any exceptions. Maybe the app logic in WebGL just didn't run the create table, possibly because of an exception thrown before it could happen in the same flow. Anyway, lots of possibilities.
There's a simple REPL sample in this project that you can import from UPM, that's what I'm testing this package with, running SQL manually and seeing that data is persisted correctly between sessions.

UnityPlayground.zip
Attached the test project I'm using. The script is from your sample code in this repo.

@gilzoide Clean the project and reimport idbvfs, then it works now!

Nice \o/
Thanks for testing it out ^^
I'll keep this issue opened until we merge #24.


Change api_url to contain /api endpoint

When I use mix and rebar3 to connect with hex API they have inconsistent naming of api_url configuration setting. Default rebar3's api_url points to https://hex.pm while mix points to https://hex.pm/api
kuba.odias@mac:~/git/process_group$ mix hex.config api_url http://hex.pdmbuilds.proximetry.com/api
kuba.odias@mac:~/git/process_group$ rebar3 hex config api_url
api_url: http://hex.pdmbuilds.proximetry.com/api

Request with 'rebar3 hex'  fails with HTTP 404 Not Found
kuba.odias@mac:~/git/process_group$ DEBUG=1 rebar3 hex key list
===> Load global config file /Users/kuba.odias/.config/rebar3/rebar.config
===> Expanded command sequence to be run: []
===> Expanded command sequence to be run: [{hex,key}]
===> Uncaught error in rebar_core. Run with DEBUG=1 to see stacktrace or consult rebar3.crashdump
===> Uncaught error: {case_clause,{error,404}}
===> Stack trace to the error location:
[{rebar3_hex_key,do,1,
                 [{file,"/Users/kuba.odias/.cache/rebar3/plugins/rebar3_hex/src/rebar3_hex_key.erl"},
                  {line,43}]},
 {rebar_core,do,2,
             [{file,"/home/tristan/Devel/rebar3/_build/default/lib/rebar/src/rebar_core.erl"},
              {line,153}]},
 {rebar_prv_do,do_tasks,2,
               [{file,"/home/tristan/Devel/rebar3/_build/default/lib/rebar/src/rebar_prv_do.erl"},
                {line,68}]},
 {rebar_core,do,2,
             [{file,"/home/tristan/Devel/rebar3/_build/default/lib/rebar/src/rebar_core.erl"},
              {line,153}]},
 {rebar3,main,1,
         [{file,"/home/tristan/Devel/rebar3/_build/default/lib/rebar/src/rebar3.erl"},
          {line,66}]},
 {escript,run,2,[{file,"escript.erl"},{line,759}]},
 {escript,start,1,[{file,"escript.erl"},{line,277}]},
 {init,start_em,1,[]}]
===> When submitting a bug report, please include the output of `rebar3 report "your command"`

Changing api_url to URL without /api works ok
kuba.odias@mac:~/git/process_group$ mix hex.config api_url http://hex.pdmbuilds.proximetry.com
kuba.odias@mac:~/git/process_group$ rebar3 hex config api_url
api_url: http://hex.pdmbuilds.proximetry.com
kuba.odias@mac:~/git/process_group$ DEBUG=1 rebar3 hex key list
===> Load global config file /Users/kuba.odias/.config/rebar3/rebar.config
===> Expanded command sequence to be run: []
===> Expanded command sequence to be run: [{hex,key}]
api_key

However such setting of api_url is not correct for mix.

The proper fix would be for rebar3_hex to use a different config file than mix since there are no guarantees that either will follow the other's format.

I agree however current implementation works only because of the default settings.
Another question is: should 'api_url' point to {HEX_PM_URL} or to actual API endpoint {HEX_PM_URL}/api ?

Since api_url is set when using other endpoints than hex.pm we need to consider that they may have a different URL structure and should not assume not /api.

What other endpoints than /api are used by rebar3_hex? I couldn't find any in the code.
Currently there's an incompatibility in naming of these URLs. Why 'mix hex.config' uses  ~/.config/rebar3/rebar.config in the first place?
How is it possible to use 'mix' and 'rebar3 hex' in parallel currently when non-standard API URL-s are used?

Only /api is used when using the hex.pm API, but this option is for when not using hex.pm so we should be flexible.

It is the other way around, rebar3_hex is using ~/.hex/hex.config.

It is not currently. That's what we should fix.

Yea, we should probably use a different config. The thinking was that sharing one would mean a user would only have to login once to hex and it be good for both tools, as well as being able to define proxies and additional repos in only one place.
As for separate development and hex devs having no control over it, happy to let you have control of the plugin ;)
Or maybe with hex_erl at least that would be in control of hexpm team and we'd just depend on it? So I would take care of just the rebar3 plugin portion.
Only issue may be we have different plans for how multi-repo will work. But I'm not sure that would actually be an issue for this but instead be just differences in how rebar3 handles resolving from repos internally (separate from rebar3_hex) and everything for this plugin would be the same between mix/rebar3_hex.

(...) would mean a user would only have to login once to hex and it be good for both tools (...)

Or maybe with hex_erl at least that would be in control of hexpm

Yes, I think that's a good idea, I can look into that.

Only issue may be we have different plans for how multi-repo will work.

@tsloughter I haven't been following development closely, are the plans written anywhere?


Add a ping-pong benchmark to crossbeam-channel

Ping-pong is an important benchmark since it measure latency
characteristics of the transport as opposed to plain throughput
characteristics. In addition, it also is a decent test of the
empty-channel read cost

While I'm sure it's possible, it wasn't actually clear how to do a clean ping-pong benchmark with the futures channels so I haven't added one yet.
I also didn't update the benchmark png since I have a different machine, and don't necessarily want to push a totally new set of baseline numbers. The generated one is here: https://i.imgur.com/O9b9CLx.png

@taiki-e any reason this wasn't merged?

This is a more complete version of what I wrote for demonstrating #821. I think having it as an official benchmark to measure performance of is valuable.

Sorry for the late response. I've read the first half of the https://github.com/crossbeam-rs/crossbeam/pull/492#issuecomment-617969071 and assumed this was a WIP state. Otherwise, I think adding such a benchmark is fine.


Interaction with Header component

I was wondering if there are any examples of how to dynamically change the Header.  For example, some screens need a Search Component - or a Save button, etc.  Also, in your current example you have Search component - how does one access what the user has typed?
Is the recommended way to add Context above your Layout component and communicate that way between Header and my app screens?

You can totally do that. The implementation depends on your router but the concept is the same and it is not relate to layout.
ex.
const App = () => {
  const { location } = useLocation() // react-router in this case
  return (
    <Root config={config}>
      <Header>
        {location.pathname === '/' ? <SpecificHomepageHeader /> : <GeneralHeader />
      </Header>
      ...
    </Root>
  )
}

Please help comment in #699

Thanks Siriwat - I can use your suggestion for changing the Header based on the Route.  But can you suggest how the Route should communicate with the Header?  Enable/disable,getClickEvent on a button,  get the contents of the search box, etc.
Thanks,
Bob

If you need to communicate between component, you need a state that is higher than these components. The state can be at App level or you can use Context to share between component. Last choice is to use javascript like get elementById and do something with the DOM.
If you can share your requirement in detail, I might be able to explain it better. However, I don't think this involve mui-layout, so I will close for now.

Thanks Siriwat - I'm fine with that answer.  I just wanted to know if there was anything build it - or some solution I was missing.


Add BucketFS overview/architecture diagram to documentation

Background:
The BucketFS can be quite complex, for that reason, an overview might be helpful.

First draft


installation documentation

note that users may need to install with pip install --user
notes on $PATH, especially if installing for user
note that if tenscalc unavailable, python -m tenscalc may be

on macOS 10.15 (Catalina) vm:
python3 -m pip install -U pip setuptools wheel
python3 -m pip install tenscalc

Decided to officially support only Linux & Windows. Doc updates should include notes that the package will install under macOS & FreeBSD, but only Linux and Windows are supported.

Closing as an issue. Will update documentation upon push of v0.1.2 release.


(putty) should depend on putty.install not putty.portable

Expected Behavior
choco install putty should install via putty.install.
Current Behavior
choco install putty depends on putty.portable.
Context
The FAQ says:

Nearly 100% of the time, the package with no suffix [...] is going to ensure the *.install.

Previous discussions:

https://chocolatey.org/packages/putty#comment-3853905911
https://chocolatey.org/packages/putty#comment-2393754748
https://github.com/ferventcoder/chocolatey-packages/issues/100

To be fair, one commenter wants the portable as default however, imho, they should be forced to install putty.portable.

Perhaps, but now we can't change just dependency as people want putty on path, so to do this I believe putty.install will need to create shims.
I personally think this is irrelevant.

I think this is still relevant. Every time I setup a new system putty.install sticks out like a sore thumb in the line of packages I install all at once.

What does that even mean ?

It's the only one that I commonly install that still requires the .install postfix:
> choco install 7zip putty.install nodejs yarn nmap wget hwmonitor

Well, in that case, we have to comply. That really looks horrific :)

here, let me fix that for you
> choco install 7zip.install putty.install nodjs.install yarn nmap wget hwmonitor

There, now it won't feel alone :laughing:
All joking aside, a meta package should not suddenly change from a portable package to a installer one, as this would be a breaking change (most that have a default install of choco probably won't notice this though).
Now, why do I say it is a breaking change? It actually got nothing to do with the program being available on the system path.
To think about why this would be a breaking change, we need to think about what the difference between an .installer and a .portable package is.
A portable package makes a program available to the the user without actually installing the program (ie: it won't be adding it to the Add/Remove Programs, and in Program Files directory). Portable packages is also required to not need administrator privileges to be installed (this would require additional setup when installing choco though).
Now, a .install package usually is a wrapper around an officially provided installer, and usually requires administrator privileges as well as actually installs the program on the users computer.
(there are other differences as well, but this is the focus when thinking about this request)
Bottom line, there is only one thing that will make it a breaking change, and that is just the administrator change, but this is big enough breaking change that it should not happen.

Also, why do you have a problem with using portable version in which case you will remove intruder.

It actually got nothing to do with the program being available on the system path.

The fact that PATH changes is also a problem as some scripts certainly use full path to the putty executable.
All that being said, where is it stated that we need to maintain compatibility forever ?

The fact that PATH changes is also a problem as some scripts certainly use full path to the putty executable.

While that is usually true, from my understanding this is not a problem when it comes to these two packages.

All that being said, where is it stated that we need to maintain compatibility forever ?

It is not stated anywhere, but when it comes to changing a portable dependency to a install dependency, then yes we should maintain compatibility (well maybe not forever, but a very long time).
Usually when there have been added breaking changes in the past, there have been made in a way that would allow existing users to keep their existing preference without making a change. In this case, we can not do that.
But let me ask you this.
Is this the way that you want to break compatibility, will you be dealing with the influx of emails from users telling us we have broken their usual builds? (This happens almost every time a breaking change have been made to a popular package, even when an upgrade path as been specified).

@majkinetor You did ask what I meant and I'm glad to have helped you sleep! 🎉
Back on subject, I've been installing PuTTY with chocolatey for years and every time (until recently because I started remembering) I would install the wrong PuTTY package only to wonder why my shortcuts/shell extensions aren't there and then spend 5-15 min researching only to finally remember, oh right, PuTTY is special because reasons.

I fully understand the implications of making even slightly breaking incompatible changes. Internet Explorer, for instance, decided to not make any breaking changes and built on legacy mistakes for decades which, ultimately, killed the product. I have no real problem with chocolatey going the same route but would prefer to just fix chocolatey. (I admit this is a bit of a stretch of a comparison, but the point stands).
IMHO, if it is broken, fix it. Depending on how many people will be affected, spend plenty of time making sure the transition is smooth.
I think it would be prudent to try to poll PuTTY users and see what the majority of us want.
If you really don't want to go through that trouble, fine. At least implement #1293 then.


Modding

Yo i was wondering if i could like request you to be a dev for an upcoming mod? its in fabric.
you can email me here @MusiJVR
<EMAIL_ADDRESS>

Resolved.

@Roma4ka-RedBad You wanna join too?


Add light theme app icon

The current app icon has a white background, in light theme, it has low contrast with system UI surfaces (eg. Taskbar, Start). We should specify a app icon variation that has a darker background (the blue one for example).
Relevant doc:

https://docs.microsoft.com/en-us/windows/apps/design/style/iconography/app-icon-construction

Fixed with commit 1142a944dbda799f866e4542e21de65dc2e22378.
Screenshots:


Unexpected ||

Describe the bug
SELECT DUAL.DUMMY || DUAL.DUMMY FROM DUAL causes web formatter to raise error.
Expected behavior
Format per usual:
SELECT
  DUAL.DUMMY || DUAL.DUMMY
FROM
  DUAL

Actual behavior
An Unexpected Error Occurred
Parse error: Unexpected "|| DUAL.DUMMY FROM DUAL"
Please report this at Github issues page.
Stack Trace:
Error: Parse error: Unexpected "|| DUAL.DUMMY
FROM
DUAL"
at E.value (https://unpkg.com/sql-formatter@latest/dist/sql-formatter.min.js:1:53856)
at E.value (https://unpkg.com/sql-formatter@latest/dist/sql-formatter.min.js:1:57875)
at E.tokenize (https://unpkg.com/sql-formatter@latest/dist/sql-formatter.min.js:1:26761)
at E.value (https://unpkg.com/sql-formatter@latest/dist/sql-formatter.min.js:1:11946)
at T.e.feed (https://unpkg.com/sql-formatter@latest/dist/sql-formatter.min.js:1:4085)
at Object.parse (https://unpkg.com/sql-formatter@latest/dist/sql-formatter.min.js:1:27486)
at O.value (https://unpkg.com/sql-formatter@latest/dist/sql-formatter.min.js:1:46992)
at O.value (https://unpkg.com/sql-formatter@latest/dist/sql-formatter.min.js:1:46861)
at Module.He (https://unpkg.com/sql-formatter@latest/dist/sql-formatter.min.js:1:256897)
at HTMLTextAreaElement.format (https://sql-formatter-org.github.io/sql-formatter/index.js:48:31)
Usage

How are you calling / using the library?

Web

What SQL language(s) does this apply to?

All as far as I can tell

Which SQL Formatter version are you using?

Web

Looks like this only fails with language: 'sql'. When you select any other dialect from the Language dropdown, the formatter doesn't crash. Transact-SQL also doesn't support it, but it won't crash as it detects || as two | operators.
Interesting that this hasn't been reported earlier as the || operator is even part of Standard SQL. But somehow we've failed to include it to operators list for the default formatter.
Will fix.

Fixed in 10.5.1


virtualbox: place mount.vboxsf in bin/ instead of sbin/

The mount tools aren't looking in sbin/ any longer.

Hmm. That failure is some download problem. :/

cc @aszlig

Looking into it tomorrow.

Great, thanks! I didn't complete this as I was a bit out of my depth and also wanted to know whether I'm on the right track. I'll be happy to help out more in the future on this one.


Fixes missing git folder actions under Untracked Changes group in scm viewlet

This PR fixes #91871

Thanks!


Support for filtering by more than one label

Hi! First of all, thanks for releasing kubefwd. This is going to save us huge amounts of time.
Is it possible to filter by more than one label, similar to the multiple namespaces feature? I would like to port-forward only a few services in our staging namespace using the app label. Something like this:
$ sudo kubefwd services -n staging -l app=website -l app=api -l app=search

Would this be difficult to add? Would you accept a PR for this?

After digging through kubefwd's source code and reading the Labels and Selectors page in the Kubernetes docs, I found that my use-case already works:
sudo kubefwd services -n staging -l "app in (website, api, search), phase = staging"

Awesome! 😄


Rival teams don't update after you select your starter

Describe the bug
When you start a new nuzlocke, the page loads with the Rival's team pre-selected. When you select the starter you chose, the page does not update with the Rival's team as per your starter selection.
E.g. for Blaze Black 2 Redux/ Volt White 2 Redux, Hugh's team is already selected as his Oshawott team. When I select Oshawott as my starter, it doesn't update to the Snivy team list.
To Reproduce

Start a new nuzlocke or open an existing one
Set/change your starter pokemon
See error
When selecting the starter pokemon (both clicking with mouse and selecting with keyboard navigation), the Rival's team doesn't update to the correct team.

Expected behavior
Changing the starter should trigger updating your Rival's team list for each Rival fight. E.g. choosing Oshawott should change the Rival's team to the Snivy team list.
Screenshots

Desktop (please complete the following information):

OS: Win 11 Pro 23H2
Browser Chrome
Version 123.0.6312.58

Note: I set this up as a Challenge Mode BW 2 Redux nuzlocke, but not sure if that is pertinent here.

I noticed this too, changing your starter's type using these buttons has the desired effect though. It would be nice if both inputs were bound somehow


Include Production WSGI (Gunicorn) to replace Flask Default Server

To Address: https://github.com/sustainable-computing-io/kepler-model-server/issues/259
A side effect of using Gunicorn is that it will replace the address with default address and default port (8000). This can be resolved by binding (ex. binding model_server to <IP_ADDRESS>:8100). Binding with <IP_ADDRESS> is acceptable but according to Flask, it is better to also introduce nginx server to act as a reverse proxy. Will this be necessary to include?
Also, since model_server AND offline_trainer (and in future online_trainer) expects to use their own Flask Apps, supervisord is needed to run multiple CMD for model_server and offline_trainer in Dockerfile.
Currently, this changes here works for metal docker compose so long as  python3.8 -u src/server/model_server.py is replaced by gunicorn -b -<IP_ADDRESS>.8100 -src.server.model_server:app. However, there is more functionality that can be introduced depending on what is acceptable or not.

@KaiyiLiu1234 https://developers.redhat.com/articles/2023/08/17/how-deploy-flask-application-python-gunicorn#containerization is good reference to follow

Binding with <IP_ADDRESS> is acceptable but according to Flask, it is better to also introduce nginx server to act as a reverse proxy. Will this be necessary to include?

I am not sure if the recommendation applies to containers / k8s.

Gunicorn should not be run as root because it would cause your application code to run as root, which is not secure. However, this means it will not be possible to bind to port 80 or 443. Instead, a reverse proxy such as nginx or Apache httpd should be used in front of Gunicorn.

The part applies to containers  is don't run as root. However, binding to port 80/443 isn't needed for containers.

You can bind to all external IPs on a non-privileged port using the -b <IP_ADDRESS> option. Don’t do this when using a reverse proxy setup, otherwise it will be possible to bypass the proxy.

Again, this is fine for a containerized env.  I feel following the article linked should be good enough to fix our issue.

@sthaha @sunya-ch I incorporated the Gunicorn changes into a test dockerfile for now. How we interact with the image will change now in docker composes and kubernetes resources (for instance command should not be python model_server.py - it should instead use gunicorn or no command as the CMD for the dockerfile can handle launching the gunicorn servers for model server, offline trainer and online trainer). I can also include in the makefile commands to run the new gunicorn test dockerfile in this PR or in a future PR?

@KaiyiLiu1234 Can we close this PR?

@sunya-ch Let's do it. We can reincorporate this in the future.


How to reduced height of bar chart and change inrement of bar Y axis

I have used Chartist.Bar Bar chart. Now i am not able to change default Y axis increment, it increment by 0.25, i want it to increment by 2. Please check screenshot for more.

Also , i want to increase font size of label and want to put place between them so they not overwrite each other.

Hello @jatinbphp 👋
Chartist has recently been updated to version 1.0 with some new exciting changes like ESM and TypeScript support. The migration guide can be found right here.
Please let me know if the issue is still relevant to you so we can further investigate it!


Upgraded to ruby 2.2.1

https://www.ruby-lang.org/en/news/2015/03/03/ruby-2-2-1-released/

:cocktail: :shipit:

Weirdly no travis?


Database filter is broken

I updated to 5.2.4 and now the database filter is broken.
I create a new filter:

and this is the result:

I tried restarting and reseting the filter. Then it worked again. Not sure what caused this.

Duplicate of #4554
Thanks for the bug report anyway.


Cross compilation of gfortran failed for riscv64

Describe the bug
Cross compilation of gfortran package fails for riscv64.
Many packages like python39Packages.scikit-learn depends on gfortran. So I can not use these on riscv64
Steps To Reproduce
Steps to reproduce the behavior:

Cross compile gfortran
$> nix-build '<nixpkgs>' -A pkgsCross.riscv64.gfortran

Expected behavior
Should compile
Error Log Snippet
affinity.o target.o splay-tree.o libgomp-plugin.o oacc-parallel.o oacc-host.o oacc-init.o oacc-mem.o oacc-async.o oacc-plugin.o oacc-cuda.o priority_queue.o affinity-fmt.o teams.o allocator.o oacc-profiling.o oacc-target.o
libtool: link: riscv64-unknown-linux-gnu-ranlib .libs/libgomp.a
libtool: link: ( cd ".libs" && rm -f "libgomp.la" && ln -s "../libgomp.la" "libgomp.la" )
make[4]: Leaving directory '/build/build/riscv64-unknown-linux-gnu/libgomp'
make[3]: Leaving directory '/build/build/riscv64-unknown-linux-gnu/libgomp'
make[2]: Leaving directory '/build/build/riscv64-unknown-linux-gnu/libgomp'
make[1]: Leaving directory '/build/build'
make: *** [Makefile:1013: all] Error 2
error: builder for '/nix/store/cx61iilfhfsalrfsls23a7wa4bij3b8a-gfortran-riscv64-unknown-linux-gnu-12.2.0.drv' failed with exit code 2;
       last 10 log lines:
       > libtool: link: (cd ".libs" && rm -f "libgomp.so.1" && ln -s "libgomp.so.1.0.0" "libgomp.so.1")
       > libtool: link: (cd ".libs" && rm -f "libgomp.so" && ln -s "libgomp.so.1.0.0" "libgomp.so")
       > libtool: link: riscv64-unknown-linux-gnu-ar rc .libs/libgomp.a  alloc.o atomic.o barrier.o critical.o env.o error.o icv.o icv-device.o iter.o iter_ull.o loop.o loop_ull.o ordered.o parallel.o scope.o sections.o single.o task.o team.o work.o lock.o mutex.o proc.o sem.o bar.o ptrlock.o time.o fortran.o affinity.o target.o splay-tree.o libgomp-plugin.o oacc-parallel.o oacc-host.o oacc-init.o oacc-mem.o oacc-async.o oacc-plugin.o oacc-cuda.o priority_queue.o affinity-fmt.o teams.o allocator.o oacc-profiling.o oacc-target.o
       > libtool: link: riscv64-unknown-linux-gnu-ranlib .libs/libgomp.a
       > libtool: link: ( cd ".libs" && rm -f "libgomp.la" && ln -s "../libgomp.la" "libgomp.la" )
       > make[4]: Leaving directory '/build/build/riscv64-unknown-linux-gnu/libgomp'
       > make[3]: Leaving directory '/build/build/riscv64-unknown-linux-gnu/libgomp'
       > make[2]: Leaving directory '/build/build/riscv64-unknown-linux-gnu/libgomp'
       > make[1]: Leaving directory '/build/build'
       > make: *** [Makefile:1013: all] Error 2
       For full logs, run 'nix log /nix/store/cx61iilfhfsalrfsls23a7wa4bij3b8a-gfortran-riscv64-unknown-linux-gnu-12.2.0.drv'.
error: 1 dependencies of derivation '/nix/store/m7gfaq1rhdvml8wpjvd60hb6p3xrmlg2-gfortran-wrapper-12.2.0.drv' failed to build

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.103, NixOS, 23.05 (Stoat), 23.05pre466479.19cf008bb18`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.13.3`
 - channels(root): `"nixos"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

The code snippet here doesn't seem very useful. Have you got the full log somewhere?

log.txt
Attach is the full log.

Also, it would be nice to have /build/build/riscv64-unknown-linux-gnu/libgfortran/config.log.

One of these days I am going to submit a PR that modifies setup.sh to cat config.log into the build stderr on failed builds...

Many packages like python39Packages.scikit-learn depends on gfortran. So I can not use these on riscv64

Ah, scikit-learn is not only built with gfortran, but expects to be able to invoke gfortran at runtime.  So it depends on a cross-built native compiler, something that doesn't get used much in nixpkgs.
I found at least part of the problem, see upcoming PR.  I don't know if it fixes the whole problem or not.


{AKS} Vendor 2022-08-02-preview for aks-preview

This checklist is used to make sure that common guidelines for a pull request are followed.
{AKS} Vendor 2022-08-02-preview for aks-preview.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
The precondition is to put your code inside this repository and upgrade the version in the pull request but do not modify src/index.json.

Queued this pipeline to generate new recordings.

AKS


Switch wakelock plugin

The current wakelock plugin (flutter_screen) lacks maintenance and has had AndroidX compatibility issues for over half a year.
To fix this, I changed the plugin to wakelock, which is also more lightweight because it has just the wakelock and no brightness adjustments.
The formatting cupertino_controls.dart is caused by flutter format and Pub complained about it before.
The version constraint is correct like this and I will ensure that there are no compatibility changes until 0.2.0 as I am the author of wakelock.

Closes #141 (that one is already solved, but including it in this comment will automatically close it upon merging).

Related
https://github.com/clovisnicolas/flutter_screen/pull/15

Awesome, thanks for the update!


Support automatic domains via kctf.cloud

Now, --domain-name is required for type=gce and you can set it to "none" or a domain.
If you choose kctf.cloud as a domain, we create a subdomain for you.
This made the dns option obsolete, so removing it (assume always true).
TODO: kctf.cloud needs to be added to the public suffix list.
Fixes #213
Fixes #195

I decided not to delete any existing managed-zones and instead encode the domain name in the zone name to avoid conflicts.
This is for 2 reasons:

deleting a zone is not easy since it needs to be empty
deleting a zone will release it from cloud dns, so someone else might be able to take it over.

I decided not to delete any existing managed-zones and instead encode the domain name in the zone name to avoid conflicts.
This is for 2 reasons:

deleting a zone is not easy since it needs to be empty
deleting a zone will release it from cloud dns, so someone else might be able to take it over.

Alright, all changes made.
In addition, I'm now patching the SOA entry for the new managed zone to reduce the ttl there as well.

Alright, all changes made.
In addition, I'm now patching the SOA entry for the new managed zone to reduce the ttl there as well.

Nice!
Just noticed the propagation is apparently SLOd at 120 seconds.
https://cloud.google.com/dns/docs/monitoring#verifying_dns_propagation
Could you also add this annotation?
external-dns.alpha.kubernetes.io/ttl: "1m"

On the domain setting, so that you can mark this PR as fixing issue #202

Nice!
Just noticed the propagation is apparently SLOd at 120 seconds.
https://cloud.google.com/dns/docs/monitoring#verifying_dns_propagation
Could you also add this annotation?
external-dns.alpha.kubernetes.io/ttl: "1m"

On the domain setting, so that you can mark this PR as fixing issue #202

@sirdarckcat can you make that proposed change in a follow up?

@sirdarckcat can you make that proposed change in a follow up?


Android back button on root screen

Hello,
first of all thanks for the great package!
I have a problem on Android:
the Back button should leave the application if this is the only screen in the stack. The standard Flutter "Counter" app works as expected but qlevar_router intercepts the Back button and nothing happens.

Hello @vasil-comstream ,
Thanks, I am glad you liked it.
This is fixed now in V1.6.1.
Thanks for reporting the issue 👍

Thanks!
Officially: that was the quickest fix ever! :)
Great work!


0xc0290904 not listed

Hi, I am having this error code in windows Hello but I did not find how to solve it.
Any idea?
Thank you?
Windows hello worked fine until yesterday...

Detalles del documento
⚠ No edite esta sección. Se requiere para docs.microsoft.com ➟ Vinculación de problema de GitHub.

ID: 6b4f9133-887f-17a6-95fc-3712de8a3fa1
Version Independent ID: 09d34ce8-1590-8202-3e70-e6299dadbb65
Content: Errores de Windows Hello durante la creación de PIN (Windows 10)
Content Source: windows/security/identity-protection/hello-for-business/hello-errors-during-pin-creation.md
Product: w10
GitHub Login: @DaniHalfin
Microsoft Alias: daniha

@officedocsbot assign @e0i

@officedocsbot assign @e0i

Hello, @juanheraclio  Thank you for submitting feedback.
In our understanding the error code that you have noted might actually have surfaced from other services rather than Windows Hello itself.
Find below a quoted reply from our Content Developer:
DaniHalfin wrote:

For error codes unlisted here, in most cases those are transient error codes
being surfaced by other services in the Windows Hello dependency chain.

We are working on something to improve the user experience in that regard,
but that will not solve the issues you are experiencing.

If we find a reoccurring error codes that isn't covered here and should be
(meaning, in most cases, it is coming from Windows Hello and not other
components) we'll make sure to add it to this page.

Also note that the feedback section that you have used to submit this issue is intended for product documentation issues only.
Please consider consulting to Feedback Hub for any of your suggestions, questions or feature requests to help Microsoft improve your experience. Find the link to Feedback Hub below:
Microsoft Windows 10 Feedback Hub
We are closing this issue but feel free to re-open it if you feel that we might have missed something.
Have a nice day.

@officedocsbot close


Is this still maintained?

Is this still maintained?
I had created a issue and pr, but no reply

First, you made your PR just a few days ago.
Open Source projects are not always paid, and the maintainer might not have the time to review and approve the request promptly.
This project is very selective in the PRs that get approved - hence there are 60 of those open at the time of writing.
Multiple reasons can be right here.
I don't want to discourage you - but that's how it is in Open Source. I've seen many PRs of mine get rejected or stale. This can be an opportunity to check if you work on the right stuff, or the quality of the code etc. etc.
It's also fine to maintain a friendly fork or even completely forking the repository.
What ever you decide, keep in mind that this whole takes time.

@oz123 Thank You
You had recently starred my server repo
If you have time, please look at my

pr #1254
issue #1253

I'm kind of occupied at the moment and focus on actual bug fixes mostly.  I'd actually like to phase out server adapters that do not offer a clear and convincing benefit over just hooking everything up the normal WSGI way.
To answer your question: Bottle is maintained, but considered mostly feature-complete. Stability is more important than new features.


[WIP] Travis matrix.

Problem
Long Travis CI build time.
Solution
Parallelize build.
Checklist

[ ] Unit test all changes
[ ] Update README.md if applicable
[ ] Add [WIP] to the pull request title if it's work in progress
[ ] Squash commits that aren't meaningful changes
[ ] Run sbt scalariformFormat test:scalariformFormat to make sure that the source files are formatted

@getquill/maintainers

PR is ready. But I'm not sure about joining two sbt commands (test and publish) in one and publishing cross-build artifacts separately (replaced +publish with publish). I can not test it because publish does not run on PR branches.

@fwbrasil Could you review this? Master branch build exceeds time limit after merging #731 and fails.

LGTM, feel free to merge once the build passes


pkg/secu: fix dropped error

This fixes anerr variable clobbered by an := inside an if block in pkg/secu.

Thank you for your PR. However, decryptedPrivateKeyBytes is a local variable, and there's no need to move others outside. Also, err is handled internally and is not ineffective.

@710leo the following error is unhandled because of the use of := inside the block:
https://github.com/ccfos/nightingale/blob/main/pkg/secu/rsa.go#L24


Flakey tests cause builds to frequently fail

Just recently, I've noticed an increase in build failures due to flakey tests. On some occasions I have to rebuild on circle multiple times before the tests eventually pass.
The purpose of this issue is to log the most commonly failing tests so that they may be fixed.

-- FAIL: TestService_GetConfig (0.03s)
	service_test.go:1070: expected to get an response on errC

@desa I had this in circle this morning:
--- FAIL: TestServer_Alert_Aggregate (1.18s)
	server_test.go:8749: unexpected tcp request:
		exp
		[{ID:id-agg Message:Received 3 events in the last 100ms. Details:message
		message
		message Time:1970-01-01 00:00:00.002 +0000 UTC Duration:2ms Level:CRITICAL Data:{"series":[{"name":"alert","columns":["time","value"],"values":[["1970-01-01T00:00:00Z",3]]},{"name":"alert","columns":["time","value"],"values":[["1970-01-01T00:00:00.001Z",4]]},{"name":"alert","columns":["time","value"],"values":[["1970-01-01T00:00:00.002Z",2]]}]}}]
		got
		[{ID:id-agg Message:Received 1 events in the last 100ms. Details:message Time:1970-01-01 00:00:00 +0000 UTC Duration:0s Level:CRITICAL Data:{"series":[{"name":"alert","columns":["time","value"],"values":[["1970-01-01T00:00:00Z",3]]}]}}]
	server_test.go:8775: unexpected topic events for aggregate topic:
		got
		{Link:{Relation:self Href:/kapacitor/v1preview/alerts/topics/tcp/events} Topic:tcp Events:[{Link:{Relation:self Href:/kapacitor/v1preview/alerts/topics/tcp/events/id-agg} ID:id-agg State:{Message:Received 1 events in the last 100ms. Details:message Time:1970-01-01 00:00:00 +0000 UTC Duration:0 Level:CRITICAL}}]}
		exp
		{Link:{Relation:self Href:/kapacitor/v1preview/alerts/topics/tcp/events} Topic:tcp Events:[{Link:{Relation:self Href:/kapacitor/v1preview/alerts/topics/tcp/events/id-agg} ID:id-agg State:{Message:Received 3 events in the last 100ms. Details:message
		message
		message Time:1970-01-01 00:00:00.002 +0000 UTC Duration:2000000 Level:CRITICAL}}]}

@desa @goller Go ahead an create two separate issues for those failing tests and we can track them down.

closing in favor or https://github.com/influxdata/kapacitor/issues/1361 and https://github.com/influxdata/kapacitor/issues/1367


Clone worked, but have some issues

Hello and thanks for making this plugin. It could work really well for me if I can work out some kinks.
I just did a "vagrant up" with the below Vagrant file. I received some warnings, but the box did indeed clone and I could start it OK.
Most values in the Vagrantfile were ignored and the resulting VM had the same memory, HD size, and processesor as the qemu_template, which are much different than what I specified in the Vagrantfile (512 ram in Vagrantfile vs 4gb in the template, for example).
Also,  I wonder if I can specify the proxmox VLAN using this plugin? I have lots of VLANs on my Proxmox cluster/LAN.
$ vagrant up
Bringing machine 'box' up with 'proxmox' provider...
==> box: translation missing: en.vagrant_proxmox.cloning_vm
==> box: Done!
==> box: translation missing: en.vagrant_proxmox.adjust_forwarded_port_params
==> box: translation missing: en.vagrant_proxmox.configuring_vm
Unable to communicate with proxmox server:
translation missing: en.vagrant_proxmox.errors.vm_configure_error
Vagrant.configure('2') do |config|
config.vm.provider :proxmox do |proxmox|
    proxmox.endpoint = 'https://pxmx01.xxxxxxxx.com:8006/api2/json'
    proxmox.user_name = 'root@pam'
    proxmox.password = '************'
    proxmox.vm_id_range = 900..910
    proxmox.vm_type = :qemu
    proxmox.vm_name_prefix = 'vagrant_'
    proxmox.qemu_os = :l26
    proxmox.qemu_disk_size = '30G'
    proxmox.qemu_storage = 'local'
    proxmox.qemu_template = 'centos7-template'
    proxmox.vm_name_prefix = 'vagrant_test_'
    proxmox.qemu_cores = 1
    proxmox.qemu_sockets = 1
    proxmox.qemu_nic_model = 'virtio'
    proxmox.qemu_bridge = 'vmbr0'
    proxmox.vm_memory = 512
end

config.vm.define :box, primary: true do |box|
    box.vm.box = 'dummy'
    box.vm.network :public_network, ip: '<IP_ADDRESS>', macaddress: 'ff:aa:cc:dd:bb:ee'
end

end

Virtual Environment 4.3-9/f7c6f0cd


Integrate custom action to list projects

Parent: #15

[ ] domain configuration and rules/stories
[ ] building an action server
[ ] implementing display_projects action
[ ] refactoring persistence into separate assembly


eval maths

How would you want the maths to be evaluated? Would it be

!eval 1+1 returning 2
!eval 1+1=2 returning true
Maybe integrate Wolfram Alpha?

wolfram has very limited api request count, it would be >eval ....

Its a third party library so I don't know how you feel about it but check this out https://ncalc.codeplex.com/

Thats what applemac used, im just very unsure of security implications.

I didn't use that one for the current version, I used https://mathosparser.codeplex.com/
Which shouldn't do anything weird, but you never know

https://ncalc.codeplex.com/
This project lost me in Evaluant.Calculator/NCalcParser.cs ~line 175;
public override string GrammarFileName { get { return "C:\\Users\\sebros\\My Projects\\NCalc\\Grammar\\NCalc.g"; } }
No way you need to burn in a file path...
https://mathosparser.codeplex.com/
This seems OK. Well documented, good to read code, and tests.

You can read my code with it in one of the commits of #126, it's reverted tn though

That commit is not bad, I commented in it.

this is done by dragon


HocusPocusProvider V2 initiates 2 connections despite setting the connect flag to false

Description
The HocusPocusProvider initiates 2 connections to our YJS backend despite setting the connect flag (from v1) to true. It should set up 1 connection.
Steps to reproduce the bug
Steps to reproduce the behavior:

Go to the examples in the repository https://github.com/BitPhinix/slate-yjs .
Upgrade the hocuspocus version on both the client and the server.
Open the client in a browser. And open the Network tab.
You would see 2 connections established to the hocuspocus backend.

Expected behavior
A single connection should be formed between the client and the server.
Screenshot, video, or GIF

Environment?

operating system: MacBook Pro 2021 16 inch Apple M1 Max Ventura 13.3.1
browser: Brave
mobile/desktop: Desktop
Hocuspocus version: 2.0.6

Additional context
The problem seems to be in this line where the connect parameter is not passed to the configuration.:
https://github.com/ueberdosis/hocuspocus/blob/ed6441c1d82e614aea388dfb84617c691d18d524/packages/provider/src/HocuspocusProvider.ts#L220

@osamatanveer I've just added an example to the provider documentation. If you want to use multiplexing, you'll have to create the websocket separately from the provider (see https://tiptap.dev/hocuspocus/provider/examples#multiplexing).

Thanks @janthurau. We're not trying to use multiplexing. It is just that when we connect using editor.connect() we see 2 web socket connection opened in the network tab. We still are facing this issue.

The same issue for me, as a dirty workaround I disconnect the configured websocket provider in addition to the Hocuspocus provider:
        provider.configuration.websocketProvider.disconnect()
        provider.disconnect()

Looks dirty, but works fine to me.

The same issue for me, as a dirty workaround I disconnect the configured websocket provider in addition to the Hocuspocus provider:
        provider.configuration.websocketProvider.disconnect()
        provider.disconnect()

Looks dirty, but works fine to me.

It looks like works fine to me too , thx


Projects from 3.6 do not work with 3.7 executable

Everything works fine until a mission finishes loading, whereupon this error happens:

Hilariously, I can't resize the error dialog to see the exact line, and there are four separate Not init'ed errors in the source here, which I recommend we differentiate slightly.
I see a bunch of lines like this in the console:
CustomMaterial(WaterMat) bind sampler[$refractBuff] and is not present on ShaderData(WaterShader)
ShaderData(ParticlesShaderData): samplerNames[0] = "$" are diferent to sampler in shader: $diffuseMap : register(S0)
ShaderData(ParticlesShaderData) sampler $diffuseMap not defined
ShaderData(OffscreenParticleCompositeShaderData): samplerNames[0] = "$" are diferent to sampler in shader: $colorSource : register(S0)
ShaderData(OffscreenParticleCompositeShaderData) sampler $colorSource not defined

And the last entry is
CustomMaterial - AL_VectorLightMaterial(1628) - Failed to load texture #ssaoMask$ssaoMask
GFXD3D9Shader::_compileShader - Error compiling shader: E_FAIL: An undetermined error occurred (80004005)
   C:/code/Torque3D-ProjectManager/bin/win32/My Projects/361 Test/game/shaders/common/lighting/advanced/vectorLightP.hlsl(70,27): error X3004: undeclared identifier 'prepassUncondition'
Failed to initialize material 'AL_VectorLightMaterial'

It's up to the OpenGL peeps to decide if we want to fix this, though for what it's worth I think we really should put a priority on keeping compatibility with old projects.

@LuisAntonRebollo and @Azaezel it would be great to get your opinion on this.

The first chunk at least is due to a lack of entries in the relevant game\core\scripts\client\ shader mapping definitions. For particles, that would be https://github.com/GarageGames/Torque3D/blob/development/Templates/Full/game/core/scripts/client/shaders.cs#L36 corresponding to https://github.com/GarageGames/Torque3D/blob/master/Templates/Full/game/shaders/common/particlesP.hlsl#L50 in a case sensitive manner for instance. Not really any way to rule that out, given it's a flat out requirement for the GL side of things as written.
Would need to root around a bit on the second, fatal chunk for where that ends coming from.

I guess a good idea might be to see if we can identify a minimal subset of changes so we can guide people through the port.

@eightyeight, can you retest plz?
If you find some problem tellme how i can reproduce :P

Oh wow, everything appears to work fine. What did you do? Thanks! :grinning:


added rules to block icmp & udp from sf services subnet

ICMP (Ping) and UDP are blocked from other machines from the SF services subnet.

Just one comment before merging , do we need to alter anything in "test_network_isolation" in sfpc ?
@vinaybheri : please share inputs so that @jagadish-kb can make relevant changes to the test.


$authProvider.facebook url doesnt work

This doesn't access the route as I'd expect. Every other parameter is working. But this isn't accesing the URL as expected. It works on a default URL but not the one I've specified here.
$authProvider.facebook({
			  name: 'facebook',
			  clientId: '486378764892124',
			  url: '/test/public/api/auth/facebook',
			  authorizationEndpoint: 'https://www.facebook.com/v2.5/dialog/oauth',
			  redirectUri: window.location.origin + '/bargainit/public',
			  requiredUrlParams: ['display', 'scope'],
			  scope: ['email'],
			  scopeDelimiter: ',',
			  display: 'popup',
			  type: '2.0',
			  popupOptions: { width: 580, height: 400 }
			});

@suharsha When you say URL, do you mean url (server-side auth implementation)  or redirectUri (where Facebook redirects to) above?
Make sure you can access those endpoints outside of Satellizer first via cURL or Rest API console.
I am going to close this issue because it doesn't look like an issue with Satellizer itself, but rather with your application. If I am incorrect, please feel free to re-open this issue!
Best,
Sahat

@sahat thanks alot. It was indeed an issue with my implementation. I hadn't defined the required permissions in "scope". Now it all works.


Add storybook-chrome-screenshot to addon gallery

Issue: none
What I did
Add storybook-chrome-screenshot addon to addon gallery.
How to test
$ cd docs
$ npm run dev
$ open http://localhost:8000/addons/addon-gallery/

I undid the changes that the automatic linter scripts made to the title. Should be good to go.

Hi @danielduan , Thanks for reviewed.
And sorry. I unexpectedly changed the title ...

@tsuyoshiwada welcome to storybook 😉

Hi @kazupon, Thx :smiley: !


[camera] Initialize failing on samsung s20 ultra 5g and samsung note 10+

I'm instantiating the camera controller like
controller = CameraController(cameras[CameraLensDirection.back], ResolutionPreset.medium);
Steps to Reproduce
I haven't reproduce it myself but users are notifying camera does not work on app and I've found this crash on my firebase console

Get a high-end samsung device (s20
Try to initialize a camera with the camera api

Expected results:
Camera functioning
Actual results:
User's device crash. This is the log from the firebase console
Caused by java.lang.IllegalArgumentException: No capture session available for current capture session.
       at io.flutter.plugins.camera.CameraUtils.getBestAvailableCamcorderProfileForResolutionPreset(CameraUtils.java:106)
       at io.flutter.plugins.camera.Camera.<init>(Camera.java:125)
       at io.flutter.plugins.camera.MethodCallHandlerImpl.instantiateCamera(MethodCallHandlerImpl.java:175)
       at io.flutter.plugins.camera.MethodCallHandlerImpl.lambda$onMethodCall$0(MethodCallHandlerImpl.java:65)
       at io.flutter.plugins.camera.f.onResult(f.java:6)
       at io.flutter.plugins.camera.CameraPermissions.lambda$requestPermissions$0(CameraPermissions.java:37)
       at io.flutter.plugins.camera.e.onResult(e.java:4)
       at io.flutter.plugins.camera.CameraPermissions$CameraRequestPermissionsListener.onRequestPermissionsResult(CameraPermissions.java:80)
       at io.flutter.app.FlutterPluginRegistry.onRequestPermissionsResult(FlutterPluginRegistry.java:196)
       at io.flutter.app.FlutterActivityDelegate.onRequestPermissionsResult(FlutterActivityDelegate.java:122)
       at io.flutter.app.FlutterActivity.onRequestPermissionsResult(FlutterActivity.java:140)
       at android.app.Activity.dispatchRequestPermissionsResult(Activity.java:8476)
       at android.app.Activity.dispatchActivityResult(Activity.java:8298)
       at android.app.ActivityThread.deliverResults(ActivityThread.java:5248)
       at android.app.ActivityThread.handleSendResult(ActivityThread.java:5296)
       at android.app.servertransaction.ActivityResultItem.execute(ActivityResultItem.java:51)
       at android.app.servertransaction.TransactionExecutor.executeCallbacks(TransactionExecutor.java:135)
       at android.app.servertransaction.TransactionExecutor.execute(TransactionExecutor.java:95)
       at android.app.ActivityThread$H.handleMessage(ActivityThread.java:2229)
       at android.os.Handler.dispatchMessage(Handler.java:107)
       at android.os.Looper.loop(Looper.java:237)
       at android.app.ActivityThread.main(ActivityThread.java:8034)
       at java.lang.reflect.Method.invoke(Method.java)
       at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:493)
       at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1076)

Fatal Exception: java.lang.RuntimeException: Failure delivering result ResultInfo{who=@android:requestPermissions:, request=9796, result=-1, data=Intent { act=android.content.pm.action.REQUEST_PERMISSIONS (has extras) }} to activity {com.logisticaldataservices.upstart/com.logisticaldataservices.upstart.MainActivity}: java.lang.IllegalArgumentException: No capture session available for current capture session.
       at android.app.ActivityThread.deliverResults(ActivityThread.java:5255)
       at android.app.ActivityThread.handleSendResult(ActivityThread.java:5296)
       at android.app.servertransaction.ActivityResultItem.execute(ActivityResultItem.java:51)
       at android.app.servertransaction.TransactionExecutor.executeCallbacks(TransactionExecutor.java:135)
       at android.app.servertransaction.TransactionExecutor.execute(TransactionExecutor.java:95)
       at android.app.ActivityThread$H.handleMessage(ActivityThread.java:2229)
       at android.os.Handler.dispatchMessage(Handler.java:107)
       at android.os.Looper.loop(Looper.java:237)
       at android.app.ActivityThread.main(ActivityThread.java:8034)
       at java.lang.reflect.Method.invoke(Method.java)
       at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:493)
       at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1076)

Flutter doctor:
[✓] Flutter (Channel dev, 1.20.0-1.0.pre, on Mac OS X 10.15.4 19E287, locale
    en-CA)
    • Flutter version 1.20.0-1.0.pre at
      /Users/josegeorges/Desktop/development/flutter
    • Framework revision f73f498da1 (5 weeks ago), 2020-06-18 08:23:22 -0700
    • Engine revision c0d52b98d1
    • Dart version 2.9.0 (build 2.9.0-16.0.dev 2b917f5b6a)

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

[✓] Xcode - develop for iOS and macOS (Xcode 11.5)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.5, Build version 11E608c
    • CocoaPods version 1.9.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 3.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 31.2.1
    • Dart plugin version 181.5656
    • Java version OpenJDK Runtime Environment (build
      1.8.0_152-release-1136-b06)

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

[✓] Connected device (3 available)
    • macOS      • macOS      • darwin-x64     • Mac OS X 10.15.4 19E287
    • Web Server • web-server • web-javascript • Flutter Tools
    • Chrome     • chrome     • web-javascript • Google Chrome 84.0.4147.89

• No issues found!

May be related to https://github.com/flutter/flutter/issues/61544,
I'll label this, but I don't have such device to test

Even I am experiencing the same with some users for Samsung m30s device. The camera is not initializing. Did not get the crashalytics on this but the capture button is disabled because of the same condition.
Code below
try {       cameras = await availableCameras();       controller = new CameraController(cameras[cameraSide],            ResolutionPreset.max);       await controller.initialize();     }
to Setup a camera.
Button disabled because of not Initialization.
 onPressed: controller != null &&                         controller.value.isInitialized                      ? onTakePictureButtonPressed                     : null

My team was able to resolve the issue. It isn't an issue from the plugin.
We were instantiating the available cameras on a map like:
camerasList = await availableCameras();
for (CameraDescription cameraDescription in camerasList) {
  cameras[cameraDescription.lensDirection] = cameraDescription;
}

and it seemed like the cameras from a Samsung S20+ retrieved from the availableCameras() method were:
07-29 23:03:01.977 I/flutter (26780): xm(0, CameraLensDirection.back, 90)
07-29 23:03:01.977 I/flutter (26780): xm(1, CameraLensDirection.front, 270)
07-29 23:03:01.977 I/flutter (26780): xm(2, CameraLensDirection.back, 90)
07-29 23:03:01.977 I/flutter (26780): xm(3, CameraLensDirection.front, 270)
07-29 23:03:01.977 I/flutter (26780): xm(4, CameraLensDirection.back, 90)

So our map was overriding camera 0 with 2 and 2 with 4.
We changed the instantiation of the map to:
for (CameraDescription cameraDescription in camerasList) {
  if (!cameras.containsKey[cameraDescription.lensDirection])
    cameras[cameraDescription.lensDirection] = cameraDescription;
}

So we use camera 0, and that one worked!
I guess we're still in a limbo as to why camera on index 4 didn't work, but this is good enough for us and thought would be good to document for other developers.
Thanks!


refactor: audit & merge install_base() tasks into the appropriate install functions

Some of install_base is not needed in the container use cases.
This will also make the various one-liners more robust.

Needs to wait for 3.x


Bug: derive_var_dthcaus does not work as expected when DEATH stored in AE and DS

What happened?
User reported they had DEATH recorded both in AE and DS with same date. admiral was selecting DS, but according to documentation the AE source was passed in first so should have been selected.
I reviewed the unit tests for this function, and it is not well tested for this, no tests had death recorded in both AE and DS and AE death was earliest, or dates matched.
Session Information
No response
Reproducible Example
Test code as follows:
adsl <- tibble::tribble(
  ~STUDYID, ~USUBJID,
  "TEST01", "PAT01",
  "TEST01", "PAT02",
  "TEST01", "PAT03"
)

ae <- tibble::tribble(
  ~STUDYID, ~USUBJID, ~AESEQ, ~AEDECOD, ~AEOUT, ~AEDTHDTC,
  "TEST01", "PAT01", 12, "SUDDEN DEATH", "FATAL", "2021-04-04"
) %>%
  mutate(
    AEDTHDT = ymd(AEDTHDTC)
  )

ds <- tibble::tribble(
  ~STUDYID, ~USUBJID, ~DSSEQ, ~DSDECOD, ~DSTERM, ~DSSTDTC,
  "TEST01", "PAT01", 4, "DEATH", "DEATH DUE TO progression of disease", "2021-04-05",
  "TEST01", "PAT02", 1, "INFORMED CONSENT OBTAINED", "INFORMED CONSENT OBTAINED", "2021-04-02",
  "TEST01", "PAT02", 2, "RANDOMIZATION", "RANDOMIZATION", "2021-04-11",
  "TEST01", "PAT02", 3, "COMPLETED", "PROTOCOL COMPLETED", "2021-12-01",
  "TEST01", "PAT03", 1, "DEATH", "DEATH DUE TO progression of disease", "2021-04-07",
  "TEST01", "PAT03", 2, "RANDOMIZATION", "RANDOMIZATION", "2021-04-11",
  "TEST01", "PAT03", 3, "COMPLETED", "PROTOCOL COMPLETED", "2021-12-01"
)

# Derive `DTHCAUS` only - for on-study deaths only
src_ae <- dthcaus_source(
   dataset_name = "ae",
   filter = AEOUT == "FATAL",
   date = convert_dtc_to_dt(AEDTHDTC),
   mode = "first",
   dthcaus = AEDECOD
 )

src_ds <- dthcaus_source(
   dataset_name = "ds",
   filter = DSDECOD == "DEATH" & grepl("DEATH DUE TO", DSTERM),
   date = convert_dtc_to_dt(DSSTDTC),
   mode = "first",
   dthcaus = DSTERM
 )
chk1 <- adsl %>%
  derive_var_dthcaus(src_ae, src_ds, source_datasets = list(ae = ae, ds = ds))

For PAT01 the DS record is erroneously selected.
When reviewing function on line 187 -188 we have the following:
tmp_source_nr <- get_new_tmp_var(dataset)
tmp_date <- get_new_tmp_var(dataset)

This is inside the loop
for (ii in seq_along(sources)) {

When the lines are moved to just before the loop, the code works as expected:
tmp_source_nr <- get_new_tmp_var(dataset)
tmp_date <- get_new_tmp_var(dataset)
for (ii in seq_along(sources)) {

We don't want a new temp var created for each loop, as then only the last source object has anything populated for the last temporary vars. This is why if DEATH in both AE and DS, only the last source object passed in is used for a subject, regardless of dates.

Also, I think if info about death coming from both AE and DS, we should look at AE, regardless of dates.

agree @millerg23 AE always has more granular info of the cause of death than DS

I wonder if we could have more control over the options, I think always ordering within domain makes sense, but maybe we can have an option of selecting first based on domain (order of source list), and also have option to take earliest across domains, it should then cater for most scenarios?
With current implementation we order:
filter_extreme(
      order = exprs(!!tmp_date, !!tmp_source_nr),
      by_vars = subject_keys,
      mode = "first"
    )

The new option wouls be to swtich these to order by source list then dates:
filter_extreme(
      order = exprs( !!tmp_source_nr, !!tmp_date),
      by_vars = subject_keys,
      mode = "first"
    )

Anyone want to go bug hunting? @pharmaverse/admiral

I wonder if we could have more control over the options, I think always ordering within domain makes sense, but maybe we can have an option of selecting first based on domain (order of source list), and also have option to take earliest across domains, it should then cater for most scenarios?

I agree that we should implement this option but I would not implement it in derive_var_dthcaus() but in derive_vars_extreme_event() (https://github.com/pharmaverse/admiral/issues/2138). It should be implemented such that the behavior is consistent with derive_extreme_event() (https://github.com/pharmaverse/admiral/issues/2140).


Enforce strict HTML rules supported by Goldmark renderer in safe mode

Is your feature request related to a problem? Please describe.
Release v0.18.6 disabled unsafe HTML in the markdown processor by default. However, the related rule MD033 is not enforced by the markdown lint configuration yet.
Describe the solution you'd like
Enable rule MD033 for the markdown lint configuration and update the contribution guidelines accordingly.
Describe alternatives you've considered
Theme users may still opt to allow mixed content by changing their specific configuration. These adjustments include:

Setting unsafe in [markup.goldmark.renderer] to true
Setting MD033 to false in .markdownlint-cli2.jsonc

Additional context
See discussion topic #466.

It appears Hugo supports some HTML elements anyways. They are an extension to the commonmark specification. Hugo uses the library goldmark, which has built-in support for GitHub Flavored Markdown. The documentation does not include a formal definition of supported HTML elements and attributes, however, the logic seems implemented in the HTML renderer.
It is suggested to extract the supported HTML elements from the HTML renderer only. These elements can then be input for a detailed configuration of MD033. The HTML renderer also specifies which exact attributes are allowed per supported HTML element, however, this seems unnecessary detailed for the rule configuration of markdown-lint.

After some more testing, it appears Hugo does not allow any HTML in safe mode. The markdownlint configuration can be either on or off for MD033.
However, there is an issue with the example shortcode. The shortcode bypasses the markdown renderer and injects any input into the final rendered page.

After some more testing, it appears Hugo does not allow any HTML in safe mode. The markdownlint configuration can be either on or off for MD033.

That is good, but I would propose to still enable MD033. People would add HTML to markdown for a reason, so it is good that they are reminded by lint that it won't work.

That is good, but I would propose to still enable MD033. People would add HTML to markdown for a reason, so it is good that they are reminded by lint that it won't work.

Yes, I agree. I did find another security concern though. The .Inner property of a shortcode gives full access to the raw input. You can process that with .Inner | .Page.RenderString to use the (safe) Goldmark renderer. However, this does not work when using embedded shortcodes. The embedded shortcodes are processed first; the parent only has access to the processed output. You are forced to use .Inner directly in the parent shortcode. However, you can then inject any input into the parent.

.Page.RenderString is capable of processing shortcodes, since version 0.100. So, why would .Inner | .Page.RenderString in the parent not work to process that embedded shortcode and its output?

The children output HTML, which is then processed by the parent (in that order). You would call .Page.RenderString on HTML output, which will not work.
I created a demonstration and possible workaround in the Hugo community. But perhaps I'm overcomplicating things.

I see. Changing outer.html to .Inner | .Page.RenderString would just change all HTML tags into plain text if HTML is not supported.

Also, how would you know a shortcode is used as an outer, or as an inner for that matter. Unless you specifically write it to be used for that, would you not always use .Inner | .Page.RenderString and never just .Inner?


test: refactoring e2e & adding tests archivistactl

This commit improves the archivistactl end-to-end (e2e) tests moving from shell script to testify test suite.
The refactoring enables Archivista to have a better view of its test coverage. This improvement also includes tests for other assert levels, such as expected messages and data content, which will avoid issues such as regression.

Thank you for adding all this new testing! I really like the structure of this and it should be easy to continue adding new tests to this.

I would like to add codecov to the repo to avoid reducing the coverage while making changes or adding features.


Consolidate and modularize toil-vg's chunked call

Main goals:

Common code base for running with or without docker, on or off cloud
Make dockerized_chunked_call runnable on its own, so separate version (in vg) doesn't need to be maintained
Usable to debug issues that arise on large datasets, and for heavier vg CI down the road.

@cmarkello Putting this up now so you can see changes as I commit them.  Mostly a fairly superficial refactor of dockerized_chunked_call.py for now.

@glennhickey Could you rebase this against the latest version of master. I now have a more complete set of unit tests to run on it.

Your changes to the parameters will result in different vcf files that won't match the vcf's that they are compared with on S3. So those will need to be changed as well.

OK I'll pull out any default parameter changes and make a separate PR out
of them after.
On Fri, Nov 18, 2016 at 2:15 PM, Charles Markello<EMAIL_ADDRESS>wrote:

Your changes to the parameters will result in different vcf files that
won't match the vcf's that they are compared with on S3. So those will need
to be changed as well.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/BD2KGenomics/toil-vg/pull/34#issuecomment-261616595,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA2_7gMS2392wBzsAj0xb457TrH7LRYWks5q_fk8gaJpZM4Kv4Dl
.

Though the changes to the default params in dockered_chunked_call.py should
have no bearing on the unit tests...
Anyway, the point's still valid.  Makes sense to fix default parameters in
a separate PR, which can include fixing things like having them specified
in multiple places.
On Fri, Nov 18, 2016 at 2:34 PM, Glenn Hickey<EMAIL_ADDRESS>wrote:

OK I'll pull out any default parameter changes and make a separate PR out
of them after.
On Fri, Nov 18, 2016 at 2:15 PM, Charles Markello <
<EMAIL_ADDRESS>wrote:

Your changes to the parameters will result in different vcf files that
won't match the vcf's that they are compared with on S3. So those will need
to be changed as well.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/BD2KGenomics/toil-vg/pull/34#issuecomment-261616595,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA2_7gMS2392wBzsAj0xb457TrH7LRYWks5q_fk8gaJpZM4Kv4Dl
.

Jenkins, test this please

@glennhickey Added fix to unit test to resolve the variant id inconsistency issue. I simply just ignored that column in the comparison.