Allows passing of Env instance through flow_params

Pull request information

Status: in development
Kind of changes: new feature
Related PR or issue: resolves issue #718

Description
Allows for the passing of custom Environments as class instances rather than strings. Its current implementation does not break the old functionality, instead allowing for either the class instance or a string to be passed in flow_params['env_name'].
This comes with a caveat: the class instance has to be defined in a separate file (from the script which calls make_create_env) and imported in your main script.

Current implementation allows for training a model, but fails when replaying the trained model via visualizing_rllib.py. Will investigate for now.

Trained models can be visualized if we store the entire module path name (like flow.envs.MergePOEnv) in flow_params['env_name']. So in its current form, we can pass Env objects at train time, but this Env will still be string encoded in flow_params, so that the environment can be reloaded without importing the Env object.
The caveat is still that the Env object must be importable when visualizer_rllib.py is called. So either we run visualization in a folder that contains the Env file, or the Env file be installed as a python module (independently of flow).

To make thing consistent, I propose that flow_params should store env_name and network as classes, while these classes get serialized through FlowParamsEncoder as "module.ClassName" strings. Decoding of the flow_params json would then reload these strings as classes.

Pull Request Test Coverage Report for Build 4491

44 of 44 (100.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 91.001%

Totals

Change from base Build 4484:
0.03%

Covered Lines:
9172

Relevant Lines:
10079

💛  - Coveralls


[MM-58586] Calls: Add startingCall parameter to calls widget startup params

Summary

To support https://github.com/mattermost/mattermost-plugin-calls/pull/767 we needed to send whether the widget was being used to start a call or join a call.
using startingCall allows us to default to Joining... as the message for older versions of desktop.

Ticket Link

https://mattermost.atlassian.net/browse/MM-58586

Release Note
NONE

@cpoile Ping me before you merge this and I can update and publish the NPM package.

FYI, waiting for a review of https://github.com/mattermost/mattermost-plugin-calls/pull/767 -- if that seems good, then we can close this as not needed. (Apologies for the time waste Devin.)

Yep, didn't need this. Thanks everyone, closing.


Update guide with Helm v3

Helm v3 removed the Tiller component. It would be nice to have the guide updated to use Helm v3 instead of the previous version based on Tiller.

The guide is now using Helm v3, Istio and all the other components in git are deployed with Flux Helm Operator (Helm v3 only).

Cool. Tnx.


fastq-extractor fails with barcode whitelist

Hi TRUST4 team, I encountered this problem when I tried to run TRUST4 on 10x single cell data.
The program works perfectly without specifying barcodeWhitelist. However, when I offer the barcode list, the program fails.
[Mon May 16 16:40:23 2022] TRUST4 begins.
[Mon May 16 16:40:23 2022] SYSTEM CALL: /home/me/miniconda3/bin/fastq-extractor -t 16 -f hg38_bcrtcr.fa -o TRUST_G6_S5_L004_R2_001_toassemble  --barcodeWhitelist 737K-august-2016.txt --barcodeStart 0 --barcodeEnd 15 -u $PATH/XXXXX_R2_001.fastq.gz --barcode $PATH/XXXXX_R1_001.fastq.gz
[Mon May 16 16:40:27 2022] Start to extract candidate reads from read files.
system /home/me/miniconda3/bin/fastq-extractor -t 16 -f hg38_bcrtcr.fa -o TRUST_G6_S5_L004_R2_001_toassemble  --barcodeWhitelist 737K-august-2016.txt --barcodeStart 0 --barcodeEnd 15 -u $PATH/XXXXX_R2_001.fastq.gz --barcode $PATH/XXXXX_R1_001.fastq.gz failed: 139 at /home/me/miniconda3/bin/run-trust4 line 48.

I ran the fastq-extractor command called by TRUST4, and Segmentation fault occured.
[Mon May 16 16:53:02 2022] Start to extract candidate reads from read files.
/var/spool/slurmd/job121813/slurm_script: line 127: 13568 Segmentation fault      (core dumped) /home/me/miniconda3/bin/fastq-extractor -t 16 -f hg38_bcrtcr.fa -o TRUST_XXXXX_R2_001_toassemble --barcodeWhitelist 737K-august-2016.txt --barcodeStart 0 --barcodeEnd 15 -u $PATH/XXXXX_R2_001.fastq.gz --barcode $PATH/XXXXX_S5_L004_R1_001.fastq.gz

The problem still exists after I allocate more memory for the task.
Is there any solution to this problem?
Also, I have some data in fastq.bz2 format. TRUST4 is able to run after converting the data to gz compression (but this is very time consuming). TRUST4 also fails with the original bz2 file input at the fastq-extractor step. Read file and barcode file have different number of reads. was reported by the fastq-extractor. this should not be caused by the data itself since the same data worked well after being converted to gz compression.
Is bz2 compression supported by TRUST4?

Hello, the fastq-extractor's "mysterious" failure with whitelist is fixed in a recent release. Which TRUST4 version are you using?
TRUST4 does not support bz2 format directly.  Is your fastq.bz2 for each file or contains mate-pair fastqs? If you have bz2 files for each mate, you may try to decompress to a stream, you may try command line like: "run-trust4 -1 <(bzcat read1.bz2) -2 <(bzcat read2.bz2) -f ..." .  You might lose 1000 reads since TRUST4 will use them to collect some statistics, and the streamed input can not be rewind back. But unless very unlucky, these 1000 reads should be from other irrelevant genes.  If you are concerned about those, you may try:
"run-trust4 -1 <(bzcat read1.bz2 | head -4000) -2 <(bzcat read2.bz2 | head -4000) -1 <(bzcat read1.bz2) -2 <(bzcat read2.bz2) -f ..." I think this would work.

I forgot your data is single-cell. For data with barcodes, you also need to duplicate the first 2000,000 barcode sequences, where TRUST4 needs to estimate barcode distribution for error correction.

Hello, the fastq-extractor's "mysterious" failure with whitelist is fixed in a recent release. Which TRUST4 version are you using?
TRUST4 does not support bz2 format directly. Is your fastq.bz2 for each file or contains mate-pair fastqs? If you have bz2 files for each mate, you may try to decompress to a stream, you may try command line like: "run-trust4 -1 <(bzcat read1.bz2) -2 <(bzcat read2.bz2) -f ..." . You might lose 1000 reads since TRUST4 will use them to collect some statistics, and the streamed input can not be rewind back. But unless very unlucky, these 1000 reads should be from other irrelevant genes. If you are concerned about those, you may try: "run-trust4 -1 <(bzcat read1.bz2 | head -4000) -2 <(bzcat read2.bz2 | head -4000) -1 <(bzcat read1.bz2) -2 <(bzcat read2.bz2) -f ..." I think this would work.

Thanks for replying do fast. Currently I'm using v1.0.5 and I met some issues in updating to 1.0.7 on the cluster. I'm contacting the admin to update it.
For the bz2 input, the steamed input really helps. Thanks a lot!

I forgot your data is single-cell. For data with barcodes, you also need to duplicate the first 2000,000 barcode sequences, where TRUST4 needs to estimate barcode distribution for error correction.

Do you mean the first 2000,000 reads in the file for --barcode parameter or the first 2000,000 barcode sequences in the file for --barcodeWhitelist? The 737K barcode whitelist seems to have less than 2000,000 barcodes.

The first 2M reads in the --barcode file.

Thanks!


Add Italian regional data

There is an official verified Github repo by the "Presidenza del Consiglio dei Ministri - Dipartimento della Protezione Civile": https://github.com/pcm-dpc/COVID-19
...and this should be the csv file for the italien regions:
https://raw.githubusercontent.com/pcm-dpc/COVID-19/master/dati-regioni/dpc-covid19-ita-regioni.csv
... and the explanations and translations for field names: https://www.sciencedirect.com/science/article/pii/S2352340920304200

and out of curiosity what would be the best way to handle regional data of different countries? at the moment everything is wrapped in the johns hopkins csv file. but what if you get the regional data source from e.g. italy, brazil, germany and lets say austria. each dataset from official sources. never the less the format might differ from country to country so the integration in covidtrends might be become cumbersome and bespoke for each country integrated more or less? :/

unfortunately for the current status at regional level, there is nothing that the gov webiste
https://github.com/MCilento93/ColoreRegioni


Deploy Service Fabric task doesn't work with secret variables

We are using Service Fabric Deploy task and managing application secrets inside Azure Key Vault. Service Fabric Deploy options to Override Application Parameters doesn't work with secret variables that are loaded previously from Azure Key Vault task. Secrets variable works fine with other tasks that allow us to pass task variables are arguments. e.g. Azure Resource Manager Task.
Environment
VSTS
Agent
Hosted VS 2017

@wahabakhtar can you share some debug logs?

@wahabakhtar pinging.

@ds-ms That was primarily the concern, because we don't want to share our keys as well as secrets. I was to recreate a separate environment to reproduce the error and logs but seemed like a lot of work.


Sidebar for dashboard mobile implementation with Ant Design

Description
This PR focuses on sidebar minimization to allow the Dashboard to be more mobile friendly.
Co-authored by @SheebaHashmi
Video Link
Loom Video
Trello Link
Dashboard sidebar minimized in mobile view
Type of change

[✔] Bug fix (non-breaking change which fixes an issue)

Checklist:

[✔] My code follows the style guidelines of this project
[✔] I have performed a self-review of my own code
[✔] I have removed unnecessary comments/console logs from my code
[✔] My changes generate no new warnings
[✔] I have checked my code and corrected any misspellings
[✔] No duplicate code left within changed files
[✔] Size of pull request kept to a minimum
[✔] Pull request description clearly describes changes made & motivations for said changes

really good job fixing the side bar minimization issues. this can make it much more usable specially on small screens. would you able to upload the video again? i couldn't find the Url. overall, great job.
This was a draft when you first reviewed, but now the Loom is up :)


Real-time 3D reconstruction using T265

Hi,
We can compute stereo depth from T265's two cameras like this.
Is it possible to reconstruct the 3D room with textured meshes using a T265 device + tablet PC in real-time?  It doesn't need to be high accuracy for me and  I want to display T265's pose too similar to this video.
Thanks.

Intel did something similar to that Project Tango video in a 2015 demo with a prototype RealSense Smartphone Development Kit (a cancelled Android smartphone with a RealSense ZR300 stereo camera and Project Tango).
https://www.youtube.com/watch?v=plwYJqz04GE
https://www.youtube.com/watch?v=Vpa8vKsapFk
Project Tango eventually got cancelled by Google and replaced by ARCore.
https://developers.google.com/ar/
Since you have computed depth from the T265, and the T265 has an IMU component like the RealSense D435i depth camera model, I am wondering if you might be able to make use of the guide below for generating SLAM with a D435i and ROS and apply it to your T265.
https://github.com/IntelRealSense/realsense-ros/wiki/SLAM-with-D435i?language=en_US

Thanks. I will try it.


U/wmwv/plot recarray using valid pixels

Adds the ability to plot maps that are rec_arrays.  Plots just False, True using the valid_pixels from the rec_array map.
Sort of fixes #16 along the way.  Although the plot can't quite distinguish between "not covered" pixels and False pixels.  Everything in the nside_coverage healpix that are listed as valid have to be either False or True and so pixels that haven't been explicitly set, and thus should presumably be invalid, get treated as False in the plot.  I believe this is is inherent to having a bool healsparse map.
This PR probably needs updated test images because the ones that I generate are just a little off (see #29).

Oh dear.

I was going to ask you to clean up the commit history, as well as a couple other refinements ... I mean the meandering commit history is not the end of the world.

@wmwv I reverted this.  Can you make a new PR and I'll make my comments on that?  Thanks!


chore(deps): Switch to hdf5_metno fork.

This is a bit of a mess of ndarray versions. It'll be easier once Polars upgrades to ndarray v0.16.


[CI] MacOS should use ConanCenter Python package

What
MacOS builds currently locate the system Python instead of using the ConanCenter package. We should use the ConanCenter cpython package.
Why
Our build instructions are not accurate in the case of MacOS. There are also differences between the ConanCenter package and the local Python when it comes to linking, which affects all OSes.
Notes
This was discovered during #714 whilst adding a CI job to use the OpenAssetIO-Test-CMake integration test project.
Switching from the system Python to the Conan package is straightforward - -DPython_FIND_FRAMEWORK=NEVER. However, this revealed a deeper problem.
The ConanCenter package brings with it a bunch of dependencies. This dependency graph is lost when using CMake's built-in FindPython module (as opposed to Conan's generated Findcpython.cmake), which instead queries sysconfig from a Python interpreter to get linker flags. Any dependencies in non-standard paths therefore don't get found.
In particular, Python.h attempts to #include <crypt.h>.  This is available from libxcrypt, installed by Conan as a dependency of cpython, but of course not on any system include path.
The MacOS system Python doesn't require this header because it is compiled with HAVE_CRYPT_H=0, and so Python.h skips over that #include.
Ubuntu works because crypt.h is pre-installed in the system path /usr/include (which is not good - we should use the libxcrypt Conan package).

Are we sure that the Conan built python is definitely then compatible with runtime python once installed via pip? Im pretty sure there was, at one point, some differences in encoding between VFX platform builds and OS distributed builds. This may no longer be the case though.


springwolf-ui: publish not working without subscribe property

Thank you for awesome work with this great project!
One issue found related to producers ui. Those are not working. Details below.
https://github.com/stavshamir/springwolf/blob/master/springwolf-ui/src/app/shared/asyncapi.service.ts
line:95         const isSubscribe = subscribe !== null;
when "subscribe" is null in back-end, in generated api spec json it is empty.
subscribe !=null, but undefined, so failing further.

Thanks for the nice words and for opening this issue.
Would you mind attaching the asyncapi.json? You can get it in localhost:8080/asyncapi/docs. At the moment, only consumers (subscribe) are supported, so subscribe should never be null at the backend.


zh:quick start ibm cloud

Updated quick start of IBM Cloud,
Removed old directory,
Fixed the images' links to avoid dependency crossing different language directory.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: fleeto
If they are not already assigned, you can assign the PR to them by writing /assign @fleeto in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content_zh/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @roganw. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

最终按照既有规定把links改回去了。
期待规则能根据实际情况进行改进，否则以后仍然可能出现类似的依赖问题。


[Question] The currently recommended config approach

There are multiple ways to get things done. The docs explain both .editorconfig, .globalconfig and rulesets. These issues mention both as well as other ways. And for those using VS or VSCode there are differences.
I'm using linux, vscode and the c# plugin. I've been using .editorconfig for a long time and I always have trouble.
Please tell me, is .globalconfig the current recommended approach? I want to use whatever the team recommends.

.editorconfig/.globalconfig seems to be the "new" way of doing things, but all of the docs for this project seem to mention the older stylecop.json stuff.


Ability to customize Toolbar theme and style

The default theme and style are fetched from themeResId variable

See Picker.Builder class's methods


Price-feeder: clean up error messages

Summary
We should clean up any copied error messages in our price feeder to use price-feeder/oracle/types/errors.go.
Ref: https://github.com/umee-network/umee/pull/1339#discussion_r965712426

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

Since price feeder noise is going back and forth, let's already assign it and move this task forward. @adamewozniak feel free to reassign and add it to Linear.


Use GitHub Logo for Navbar Link

Overview
PR Checklist

[x] I read and included these actions below

This PR uses GitHub Logo for Navbar GitHub Link. It uses the same method as docusaurus.io.

Ah, interesting. This docusaurus failed to build on my end, so I was trying to figure out why. I figured there was a proprietary yarn prepare:docs command in your package.json. We should preferably add this to CONTRIBUTIONS.md or README.md.

Ready for review.


README: Drop defunct badge

The Gemnasium service is no longer

Coverage remained the same at 99.204% when pulling 82d9959eb9b9b5ede142e97d915abe0d37869b52 on olleolleolle:patch-1 into cdc2087cc0651a5830d78d62f6e82592ab3d1692 on ruby-grape:master.


Feat/portal charts

Related to NFINT-491

Need to fix Docker build:

5.641 The following packages have unmet dependencies:
5.738  libc6-dev : Breaks: binutils (< 2.38) but 2.37-7 is to be installed

Will update Dockerfile in separate PR

I installed mermaid and rendered the output from flowchart <- processing_flowchart(year = 1), attached. Is it possible that some raw data nodes are missing? Thinking specifically about the JHU biobank with three edges going into one raw data node.

Aha! this is the PR I was looking for.
I updated the data table right before your review/updated rendering of the output.
SynXX302 is the input raw data node for both synXX189 and synXX656 output folders (somatic and germline variants). However, SynXX641 should be deleted since it was a test of the workflows before final processing was done. @anngvu I am happy to delete the corresponding row in the internal tracking table, but want to confirm with you before I go ahead :)

Hey @jaybee84 Yep, please delete row if it is a test and shouldn't be in final display.

done

@anngvu the rendering of this image AFAIR had the nodes to be clickable. In the version that is surfaced in the portal (https://www.synapse.org/#!Synapse:syn26451327/wiki/620082) the image is static. Is that the behavior you are expecting?


Added mouseover text to command history buttons

The mouseover text displays the whole command of each previously used command on hover, making it easier to identify longer-sliced commands and avoid mixing them up when trying to execute them on the console page.

Thank you.


Cannot Find Module 'editorconfig'

Description
Getting an error on all attempts to beatify. Below is just a simple css example. I get the error
"Cannot find module 'editorconfig' " below is the stack trace. Not sure what I need to be doing? Just installed atom and this package today on a new PC. I've yet to run into this issue on my Mac and other PC with Atom and the same packages.
Error: Cannot find module 'editorconfig'
    at Module._resolveFilename (module.js:543:15)
    at Module._resolveFilename (C:\Users\dkmya\AppData\Local\atom\app-1.31.2\resources\electron.asar\common\reset-search-paths.js:35:12)
    at Function.get_Module._resolveFilename (C:\Users\dkmya\AppData\Local\atom\app-1.31.2\resources\app\src\module-cache.js:354:58)
    at Module.require (file:///C:/Users/dkmya/AppData/Local/atom/app-1.31.2/resources/app.asar/static/index.js:40:43)
    at require (C:\Users\dkmya\AppData\Local\atom\app-1.31.2\resources\app\src\native-compile-cache.js:66:33)
    at Beautifiers.module.exports.Beautifiers.getOptionsForPath (file:///C:/Users/dkmya/.atom/packages/atom-beautify/src/beautifiers/index.coffee:589:23)
    at file:///C:/Users/dkmya/.atom/packages/atom-beautify/src/beautify.coffee:166:29
    at Promise._execute (C:\Users\dkmya\.atom\packages\atom-beautify\node_modules\bluebird\js\release\debuggability.js:313:9)
    at Promise._resolveFromExecutor (C:\Users\dkmya\.atom\packages\atom-beautify\node_modules\bluebird\js\release\promise.js:483:18)
    at new Promise (C:\Users\dkmya\.atom\packages\atom-beautify\node_modules\bluebird\js\release\promise.js:79:10)
    at beautify (file:///C:/Users/dkmya/.atom/packages/atom-beautify/src/beautify.coffee:77:10)
    at beautifyOnSaveHandler (file:///C:/Users/dkmya/.atom/packages/atom-beautify/src/beautify.coffee:536:9)
    at file:///C:/Users/dkmya/.atom/packages/atom-beautify/src/beautify.coffee:544:7
    at Function.module.exports.Emitter.simpleDispatch (C:\Users\dkmya\AppData\Local\atom\app-1.31.2\resources\app\node_modules\event-kit\lib\emitter.js:27:20)
    at Emitter.<anonymous> (C:\Users\dkmya\AppData\Local\atom\app-1.31.2\resources\app\node_modules\event-kit\lib\emitter.js:169:50)
    at Emitter.module.exports.Emitter.emitAsync (C:\Users\dkmya\AppData\Local\atom\app-1.31.2\resources\app\node_modules\event-kit\lib\emitter.js:172:18)
    at C:\Users\dkmya\AppData\Local\atom\app-1.31.2\resources\app\node_modules\text-buffer\lib\text-buffer.js:1349:38
    at <anonymous>

Input Before Beautification
This is what the code looked like before:
body {
  background: red ;

}

Expected Output
The beautified code should have looked like this:
body {
  background: red;
}

Actual Output
The beautified code actually looked like this:
Did not beautify
Steps to Reproduce

Add code to Atom editor
Run command Atom Beautify: Beautify Editor
This beautified code does not look right!

Debug
Here is a link to the debug.md Gist:
Checklist
I have:

[x] Tried uninstalling and reinstalling Atom Beautify to ensure it installed properly
[x] Reloaded (or restarted) Atom to ensure it is not a caching issue
[x] Searched through existing Atom Beautify Issues at https://github.com/Glavin001/atom-beautify/issues
so I know this is not a duplicate issue
[x] Filled out the Input, Expected, and Actual sections above or have edited/removed them in a way that fully describes the issue.
[ ] Generated debugging information by executing Atom Beautify: Help Debug Editor command in Atom and added link for debug.md Gist to this issue

Uninstalled and reinstalled atom beautify today for the 6th time since 2 days ago via apm. All seems to be working now!


Option to disable colours on output

I'm using vue-i18n-extract to generate a report as part of a CircleCI build pipeline.
I'm using the following command to output to stdout and a file which gets stored as a "build artifact".
npx vue-i18n-extract report -v "./**/*.?(js|vue)" -l "./locales/*.?(js|json)" | tee /tmp/i18n-report

Unfortunately, the stored result contains colour codes and winds up looking like this
┌────┬────────────┬────────────────────────────────────────┬────────┬──────────────────────────────┐
│ #  │ Language   │ File                                   │ Line   │ Missing i18n Entry           │
├────┼────────────┼────────────────────────────────────────┼────────┼──────────────────────────────┤

It would be great if there was an option to disable colours in the output or if the --output flag supported plain text.

For anyone else who encouters this, my workaround is to use sed to strip the colour codes.
npx vue-i18n-extract report -v "./**/*.?(js|vue)" -l "./locales/*.?(js|json)" \
| sed -r "s/[[:cntrl:]]\[[0-9]{1,3}m//g" \
| tee /tmp/i18n-report

Will look into!

We definitely should make the colors optional then.
Thank @Soviut for bringing this up.

Hey @Soviut, I have made PR #70  that uses the native console.table function. I tested with the tee command on mac and it seems like it solves your issue. It should be merged in soon.


Deployment fails with 'Rate exceeded'

What is the issue?
When the newest version of SPA is deployed on a new account, this attempts to deploy about 190 resources. The deployment fails with message 'Rate exceeded'.
How to reproduce the problem?
Deploy SPA on a fresh AWS account

New code has been deployed in production and works as expected.

New code has been deployed to create a brand new deployment of SPA, without any issue


Can you add a 'next page' button and keyboard shortcut?

Maybe Ctrl+ArrowKey?

Right now, there is a button at the bottom right of the thumbnail area. It's shortcut (if you hover over it, it will show) is ALT + PageDown.
Is this what you need?

Thanks I appreciate that.
One issue is that while stable diffusion is running, there is constant
popups on Diffusion Toolkit saying "One image added"
But the popups are directly over the pagination buttons :)
Thanks!
On Thu, Jul 27, 2023 at 6:56 PM David Khristepher Santos <
@.***> wrote:

Right now, there is a button at the bottom right of the thumbnail area.
It's shortcut (if you hover over it, it will show) is ALT + PageDown.
Is this what you need?
—
Reply to this email directly, view it on GitHub
https://github.com/RupertAvery/DiffusionToolkit/issues/133#issuecomment-1654862748,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGRYVSSHQFPOCT4CQQJU3LXSML6JANCNFSM6AAAAAA22ZKFOM
.
You are receiving this because you authored the thread.Message ID:
@.***>


Disk Space overview on Dashboard

Would be great to have some form of disk space view for all drives on a system, viewable from dashboard.
Additionally a file manager as well but might be a bit more complicated.

I agree :)

@Morphy99 Added it now :)

Cool thanks! Can only see HDD in dashboard no info or bar? Works if I go into system info

Clear browser cache, make sure that you have enabled it as well

Yep clearing cache did it! Thanks :)


Revert "release 5.39.0"

Reverts acr-lfr/generate-it#143

will merge in with vbump to resolve conflicts


i cant find the location on how to switch to use tan log

@lni would like to remove anything to do with pebbledb, how do i first use tan log?
found info here without mentioning how to use tan log.
also, how to disable pebbledb for the log if i use tan log too.
that's all. trying out now. trying to get it compile. go mod vendor is not helping with error message here:
https://github.com/lni/dragonboat-example/issues/28

I've never used tan but it seems pretty straight forward after reading the code.
import (
	"github.com/lni/dragonboat/v4"
	"github.com/lni/dragonboat/v4/config"
	"github.com/lni/dragonboat/v4/plugin/tan"
)

func main() {
	exp := config.GetDefaultExpertConfig()
	exp.LogDBFactory = tan.Factory
	nh, err := dragonboat.NewNodeHost(config.NodeHostConfig{
		Expert: exp,
	})
}


version 0.9.0 and SystemJS

Hi,
I have problem with version 0.9.0.  I use SystemJS.
The error is:
(SystemJS) Unexpected token <
SyntaxError: Unexpected token <
at eval ()
...
"Unable to load transpiler to transpile..."
Did anyone have the same problem?
Thanks

@arjack I'm experiencing the same error when trying to load the lib from angular seed. Looks like version issues.


Support wildcards or regex in [MBeanPattern]{} syntax

When monitoring Apache Flume, for example, everytime I wrote:
[org.apache.flume.channel] { ... }
[org.apache.flume.sink] { ... }
[org.apache.flume.source] { ... }

It would be easier to write:
[org.apache.flume.*] { ... }

TODO: wildcard or regex (If we choose regex, we can implement metrics renaming using captures in the future. On the other hand, wildcard is much simpler.)

Implemented and will be released in 0.0.4


Publish pre-release to npm

I'm trying to use this library, but am running into various issues. For example, I get these module resolution issues:
@acme/nextjs:dev: Module not found: Can't resolve 'http' in '/Users/user/static/org/company/website/node_modules/.pnpm/@convertcom+js-sdk@1.2.0/node_modules/@convertcom/js-sdk/lib'

@acme/nextjs:dev: Module not found: Can't resolve 'https' in '/Users/user/static/org/company/website/node_modules/.pnpm/@convertcom+js-sdk@1.2.0/node_modules/@convertcom/js-sdk/lib'

@acme/nextjs:dev: Module not found: Can't resolve 'querystring' in '/Users/user/static/org/company/website/node_modules/.pnpm/@convertcom+js-sdk@1.2.0/node_modules/@convertcom/js-sdk/lib'

@acme/nextjs:dev: Module not found: Can't resolve 'querystring' in '/Users/user/static/org/company/website/node_modules/.pnpm/@convertcom+js-sdk@1.2.0/node_modules/@convertcom/js-sdk/lib'

As well as this issue:
@acme/nextjs:dev: ApiManager.releaseQueue() { error: 'Unable to perform network request' }

I have a feeling that all of these issues are fixed on the pre-release-v1.3 branch, and I'd like to try it out. Please publish it to npm under some sort of pre-release or canary tag.

@mattddean we will publish the release soon enough, are you interested in joining the alpha testing phase?

@abbaseya absolutely

@mattddean good news, we're going to release these major changes anyway. I'll keep you posted.

@abbaseya that's great news! Any idea of the timeline? We're looking to launch convert.com on our site next week.

@mattddean we're planning to publish these update by Tuesday.

@mattddean we've just release v2.0.0, feel free to use it now. One note though, kindly use yarn for now. We're working on a more universal installation in the near future.

@abbaseya awesome!


NSURLErrorDomain - 1202 (Certificate)

Good afternoon,
just saw this library today, I'm still and originally using Webview from "react-native", then I saw your repo, so I wanted to give it a try.
However, I'm stuck with this error when the Webview try to launch ⏬

It's weird, 'cos I don't get this error with the original Component imported from React-Native...
I tried to add an NSExceptionDomains field in my Info.plist, but didn't work...
Does anyone have any ideas about how to resolve this, or for example, ignoring certificates restriction when Webview's loading ?

There is no reason it would be any different from the original component 🤔

Sorry, I actually lost the access to my original account (@MayeuldP ), so here I am again (sorry for the delay) :
I tried with the very basic example :
return (
      <WebView
        source={{ uri: "https://facebook.github.io/react-native/" }}
      />
);

It's actually working well for Android, on Simulator and Real devices (both)
However on iOS, I get the same error with the screen that I added in the first post.
Error loading page, Domain : NSURLErrorDomain, Error Code: 1202,
The certificate for this server is invalid.
You might be connecting to a server that i pretending to be "facebook.github.io"
which could put your confidential information at risk.

This is the kind of errors that I get in the log of the simulator :
Mar 22 17:02:33  com.apple.CoreSimulator.SimDevice.235866FF-3DC6-4044-9A28-1A29BBDD3E97[39710] (com.apple.StreamingUnzipService): Unknown key for Boolean: EnableTransactions
Mar 22 17:02:33  com.apple.CoreSimulator.SimDevice.235866FF-3DC6-4044-9A28-1A29BBDD3E97[39710] (com.apple.StreamingUnzipService): Unknown key for Boolean: EnablePressuredExit
Mar 22 17:02:33  com.apple.CoreSimulator.SimDevice.235866FF-3DC6-4044-9A28-1A29BBDD3E97[39710] (com.apple.WebKit.Networking): Unknown key for Boolean: EnableTransactions
Mar 22 17:02:33  com.apple.CoreSimulator.SimDevice.235866FF-3DC6-4044-9A28-1A29BBDD3E97[39710] (com.apple.WebKit.Networking): Unknown key for Boolean: EnablePressuredExit
Mar 22 17:02:33  com.apple.CoreSimulator.SimDevice.235866FF-3DC6-4044-9A28-1A29BBDD3E97[39710] (com.apple.WebKit.Storage): Unknown key for Boolean: EnableTransactions
Mar 22 17:02:33  com.apple.CoreSimulator.SimDevice.235866FF-3DC6-4044-9A28-1A29BBDD3E97[39710] (com.apple.WebKit.Storage): Unknown key for Boolean: EnablePressuredExit
Mar 22 17:02:33  com.apple.CoreSimulator.SimDevice.235866FF-3DC6-4044-9A28-1A29BBDD3E97[39710] (com.apple.WebKit.WebContent): Unknown key for Boolean: EnableTransactions
Mar 22 17:02:33  com.apple.CoreSimulator.SimDevice.235866FF-3DC6-4044-9A28-1A29BBDD3E97[39710] (com.apple.WebKit.WebContent): Unknown key for Boolean: EnablePressuredExit
Mar 22 17:02:33  com.apple.CoreSimulator.SimDevice.235866FF-3DC6-4044-9A28-1A29BBDD3E97[39710] (com.apple.backboard.display.archive): Unknown key for Boolean: EnableTransactions
Mar 22 17:02:33  com.apple.CoreSimulator.SimDevice.235866FF-3DC6-4044-9A28-1A29BBDD3E97[39710] (com.apple.backboard.display.archive): Unknown key for Boolean: EnablePressuredExit
Mar 22 17:02:33  com.apple.CoreSimulator.SimDevice.235866FF-3DC6-4044-9A28-1A29BBDD3E97[39710] (com.apple.uifoundation-bundle-helper): Unknown key for Boolean: EnableTransactions
Mar 22 17:02:33  com.apple.CoreSimulator.SimDevice.235866FF-3DC6-4044-9A28-1A29BBDD3E97[39710] (com.apple.uifoundation-bundle-helper): Unknown key for Boolean: EnablePressuredExit

And here is the error that I get in the Chrome Debugger.

[x] I, of course, checked that the librairies were both correctly linked for iOS and Android
React-Native version : 0.55.3
Xcode Version : Xcode 10.1
Simulator : iOS 12.1

I have the strange feeling that this problem has something to deal with xCode ... 🧐🤔

in my experience, it only occurs on simulator
React Native Environment Info:
System:
OS: macOS 10.14.3
CPU: (12) x64 Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz
Memory: 704.05 MB / 16.00 GB
Shell: 3.0.0 - /usr/local/bin/fish
Binaries:
Node: 8.15.0 - ~/.nvm/versions/node/v8.15.0/bin/node
Yarn: 1.13.0 - /usr/local/bin/yarn
npm: 6.4.1 - ~/.nvm/versions/node/v8.15.0/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
SDKs:
iOS SDK:
Platforms: iOS 12.2, macOS 10.14, tvOS 12.2, watchOS 5.2
Android SDK:
API Levels: 21, 22, 23, 24, 25, 26, 27, 28
Build Tools: 23.0.1, 23.0.3, 24.0.1, 25.0.0, 25.0.2, 25.0.3, 26.0.1, 26.0.2, 26.0.3, 27.0.2, 27.0.3, 28.0.2, 28.0.3
System Images: android-28 | Google APIs Intel x86 Atom, android-28 | Google Play Intel x86 Atom
IDEs:
Android Studio: 3.3 AI-182.5<IP_ADDRESS>14842
Xcode: 10.2/10E125 - /usr/bin/xcodebuild
npmPackages:
react: 16.8.5 => 16.8.5
react-native: 0.59.2 => 0.59.2
npmGlobalPackages:
react-native-cli: 2.0.1
react-native-create-library: 3.1.2

This is working on my end. Are you sure the network you are connected to is safe? Please try with another network connection.

It appears that it didn't works on Simulator on another network connection, however it worked on a real device (iOS), weird and quite contraining but ok on the end... You can close the issue !

maybe you are using some weird VPN or something :)

We've just updated to the community webview over that from react-native directly and started to get this issue.
It may only be on the simulator but it's a huge blocker as in one case we use a webview for some authentication in the 1st step of an app - and can't get around it without some hackish code inserting credentials to redux 😓

I ran into this too. On simulator it gave me that ^^ error, but when I ran it on a iOS device the url loaded fine.

Glad to see that I'm finally not the only one ;)

After having added the following to NSAppTransportSecurity in our target's Info.plist file:
<key>NSAllowsArbitraryLoads</key>
<true/>

And rebuilding the project, it seems to be working on the simulator again 🤞

I'm having similar problem on both the simulator and the device. Below is the error that I get:
Error loading page
Domain: NSURLErrorDomain
Error Code: -1100
Description: The requested URL was not found on this server.

Let me know if you need more info from me to debug this.

I'm having similar problem on both the simulator and the device. Below is the error that I get:
Error loading page
Domain: NSURLErrorDomain
Error Code: -1100
Description: The requested URL was not found on this server.

Let me know if you need more info from me to debug this.

So I have fixed my own issue following this guide. What I did wrong in the first place was creating the external folder and put in the web files physically via finder instead of adding them via Xcode.

If anyone else comes across this with error -1202, my problem was my antivirus was issuing invalid certificates to the simulator, open safari and surf to the website, if it has an issue, view certificate, if it says it's not trusted and you can see the certificate is by your antivirus, turn your antivirus off, hope this helps someone out there

the community version is not working because it set useWebKit to true by default.
if u set useWebKit to false on community version.
it will be working

I have the same issue when trying to use my localhost server while developping.

the community version is not working because it set useWebKit to true by default.
if u set useWebKit to false on community version.
it will be working

Did the trick ! Thank you Sir !

the community version is not working because it set useWebKit to true by default.
if u set useWebKit to false on community version.
it will be working

but if useWebKit set false will use UIWebView
UIWebView is deprecated and will be removed soon
what should I do...

Do you have any updates?

A solution that may work for some is to follow this documentation: https://github.com/react-native-community/react-native-webview/blob/master/docs/Custom-iOS.md#allowing-custom-cas-certifica-authorities-and-enabling-ssl-pinning. Some of that documentation is outdated so you'll have to play with it but it was able to help me find a working solution. More information can be found in this commit https://github.com/react-native-community/react-native-webview/commit/136fbd84915e20912d21370c254f5206b5661d67

Not working on simulator...try info.plist all the keys need to be added. giving sam error:he certificate for this server is invalid. You might be connecting to a server that is pretending to be “<IP_ADDRESS>” which could put your confidential information at risk. Also, tried using uploading and trusting same self signed certificate on simulator doesn't work. Also changes to localhost from '<IP_ADDRESS>'.
Doesn't work.

I have still experienced the same situation. Does anyone have the solution on this issue?

Having same issue, trying to use localhost and render the URL on the webview, it works if I do it directly in safari, but I get the exception using react native web view. Has anyone solved this problem running on simulator and localhost?

+1

What solved it for me was accessing it on Safari in the simulator and accepting the warning to see the page. Then it worked inside the WebView.

It took a lot of time for me to figure this out:
If we come across the certificate issue, while connecting the bank accounts (PFA is the attached screen shot).
It is because ‘' antivirus preventing iPhone simulator to browse any websites.I was able to fix this issue by the following steps:

uninstall or turn off Antivirus in Mac.
erase all content and settings of simulator.
restart mac.

I had the same problem on Android emulator. It worked by downgrading the version to 10.3.0

I had the same issue when VPN is on. Accepting the certificate on Safari doesn't help. I still got the issue with iOS simulator.

Having same troubles have documented here
react: 16.13.1 => 16.13.1
react-native: 0.63.2 => 0.63.2
react-native-webview => ^10.9.2

Hey everyone, Is there any update on the same. I have experienced this issue a couple of times.

Did anyone find any solution?

+1

+1

This may help some but not all...
If your WebView source is written like:
        <RFWebView
          source={{uri: "www.instagram.com" }}
        />

change it to include https://:
        <RFWebView
          source={{uri: "https://www.instagram.com" }}
        />

This fixed the problem for me at least...


Update README.md

there was a typo in header

Merci beaucoup frère
Le ven. 2 oct. 2020 à 09:03, aayushsaini1<EMAIL_ADDRESS>a
écrit :

there was a typo in header

You can view, comment on, or merge this pull request online at:
https://github.com/HouariZegai/FXMaterialDesign/pull/1
Commit Summary

Update README.md

File Changes

M
README.md
https://github.com/HouariZegai/FXMaterialDesign/pull/1/files#diff-04c6e90faac2675aa89e2176d2eec7d8
(2)
Patch Links:

https://github.com/HouariZegai/FXMaterialDesign/pull/1.patch

https://github.com/HouariZegai/FXMaterialDesign/pull/1.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/HouariZegai/FXMaterialDesign/pull/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALISVOUW4ECORXQ7L2YBBXLSIWCMJANCNFSM4SBMQ2XQ
.


Add more examples to README

We should link to companies that are using Fern in production

Tracking in linear as part of homepage revamp


Is it possible to support ArchLinux?

I would like to use ArchLinux and this action does not seem to support Arch as of now.
Could this be done? if someone could point me to some instructions I'm happy to present a PR. Just not sure if I'm the best person for it.

Hey @armanm I think that's going to be a question for @wojtekmach or @ericmj who are responsible for the builds setup-beam pulls from.
You might want to open an adjacent issue on https://github.com/hexpm/bob/issues

Hey @armanm I think that's going to be a question for @wojtekmach or @ericmj who are responsible for the builds setup-beam pulls from.
You might want to open an adjacent issue on https://github.com/hexpm/bob/issues

Thanks @starbelly
@ericmj's response in https://github.com/hexpm/bob/issues/83#issuecomment-814834635 pretty much answers my question.

Adding a new OS requires building thousands of images so it's something we want to hold off with until there are enough that need it.

I'm going to close this


Broken link

Link to https://azure.microsoft.com/en-us/resources/samples/active-directory-dotnet-deviceprofile/?v=17.23h is broken

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

ID: a0ab71a0-a2df-fa37-83fa-080dece952f6
Version Independent ID: f74b7d51-aede-ed93-437e-601654d8fa7e
Content: Get started with the REST APIs for Azure DevOps Services and Team Foundation Server - Azure DevOps Services REST API
Content Source: index.md
Product: devops
Technology: devops-ecosystem
GitHub Login: @elbatk
Microsoft Alias: elbatk

Thanks for reporting the issue we will fix the link.


Lost transitive dependencies during mavenCentral migration

While upgrading from 2.1.1 to 2.2.1 we noticed that some dependencies disappeared from gradlew :app:dependencies.
Specifically junit + test-butler-api, and I can confirm this by noticing that they went from compile to runtime here:
https://mvnrepository.com/artifact/com.linkedin.testbutler/test-butler-library/2.1.1

https://mvnrepository.com/artifact/com.linkedin.testbutler/test-butler-library/2.2.1

Was this an intentional change?

Note: they still exist on debugAndroidTestRuntimeClasspath - Runtime classpath of compilation 'googleDebugAndroidTest' (target  (androidJvm))., so I think most users are fine, unless they accessed code from -api which is essentially com.linkedin.android.testbutler.ButlerApi.Stub.Proxy. I think users will be fine, but maybe worth noting in changelog?

Good catch. It may not have been intentional, but I think it's correct. Really I think the bug is we didn't realize junit was leaking to the compile classpath before this change 😄
We were already declaring junit as an implementation dependency, but I think our old publishing setup was incorrect and declaring it as an api dependency in the published artifact files. Now it's being correctly published as implementation (aka runtime) which explains what you're seeing.

Done!

Cheers! Thanks for the quick turnaround!


As a user I want to have project-wide prompt that is static and is sent with every inference call as part of the bigger prompt

Often bolt changes files I previously prompted it to NEVER DO. Or bolt forgets my prompted demands regarding code structure or libraries that are IN and libraries that are OUT or any other reasoning we previously had.
It would make everyone;s life much easier, if you'd allow me to have a "project-prompt" setting, where I can put some truths in that will be sent with every call to the OpenAI LLM.
Describe the solution you'd like:
Every Project also allows me to set a prompt that will be sent along every llm call
Describe alternatives you've considered:
There is none, really. I could ask bolt to write me a chrome extension that does the job for me ;)

I think what you're describing is in .bolt/prompt ?
But there is no easy way to change that, and changes do not seem to persist if you unload / refresh your project.
Note you can read and write to it in a terminal using cat within a session.
or load it in stackblitz and see it in the gui there


Update css3-tabsize.json

Win10 + IE11 (slow ring) has white-space working, and even completing in the F12 developer tools style editor

This feature is for the tab-size property, not the white-space one. And tab-sizedoes not appear to be supported in the IE technical preview.

Yes, we’re not working on tab-size right now.


Creating ComponentContainer with preloaded Component with manifest

OpenUI5 version: 1.98.0
Browser/version (+device/version): Latest Chrome on Windows
I'm trying to programmatically load a Component (with manifest) into a ComponentContainer as described here: https://ui5.sap.com/#/topic/346599f0890d4dfaaa11c6b4ffa96312
	Component.load({
		name: "samples.components.sample",
	}).then(function(oComponent) {
		var oContainer = new ComponentContainer({
			component: oComponent
		});
		// ...
	});

However, I get the following error: Assertion failed: setAssociation(): sId must be a string, an instance of sap.ui.base.ManagedObject or null
I think the problem boils down to the component not being a component yet, but a function called "oClassProxy", which doesn't extend ManagedObject: https://github.com/SAP/openui5/blob/master/src/sap.ui.core/src/sap/ui/core/Component.js#L2865
Maybe the documentation just is a bit fuzzy here: If my component has a manifest, am I always supposed to use the "manifest first" mode?
When I just call the method, the component is loaded just fine:
	Component.load({
		name: "samples.components.sample",
	}).then(function(oComponent) {
		var oContainer = new ComponentContainer({
			component: oComponent()
		});
		// ...
	});

Thank you very much for your support in Improving the documentation!

Hello @LukasHeimann ,
Thank you for sharing this finding. I've created an internal incident<PHONE_NUMBER>. The status of the issue will be updated here in GitHub.
Regards, Diana

Hi @LukasHeimann,
thanks for finding this, the documentation is just wrong and should be adapted.
The Component.load function just preloads the Component class.
The ComponentContainer's component association however expects an instance of that class.
So the correct thing would be to use Component.create() and not Component.load():
Component.create({ // create(), not load()!
		name: "samples.components.sample",
}).then(function(oComponent) {
	var oContainer = new ComponentContainer({
		component: oComponent // oComponent is a valid instance now
	});
});

Concerning your question about "manifest-first": simply put yes.
If your Component brings a manifest you should create it in a manifest-first fashion.
Simply using Component.create({ name: ... }) takes care of this for you, no further manifest: true is needed.
You can then just pass the Component instance to the ComponentContainer as shown above.
In you last snippet you call oComponent, that technically works somehow, but it's a constructor that should be invoked with new (if at all).
I would advise to use the Componen.create factory or specify the Component name in the ComponentContainer directly, like so:
// PS: I did not test this, but it should work ;)
var oContainer = new ComponentContainer({
	name: "samples.components.sample",
	manifest: true,
	async: true
});

Hi @Thodd,
thank you for the swift response, looks like Component.create does the trick! Shall I leave this issue open until the documentation is adjusted?
You PS works as well (and it is in fact what I tried first). However, in my use case, I wanted some sort of caching of the components. I found that when I create multiple ComponentContainers with the same component (sequentially, with some time in between, and when the first container is already destroyed again), requests were sent again over the network. I wanted to avoid this by caching the promises of the Component factory and using them to create the ComponentContainer -- where I stumbled on this issue.
Kind regards
Lukas

@LukasHeimann sure just keep it open, my colleagues @stopcoder already adapted the documentation, though that takes some time until it's visible. We'll update this issue once the documentation is in.

Hi @LukasHeimann,
we adapted the documentation regarding Component.load and Component.create. The adaptation is already available in openui5 nightly build https://openui5nightly.hana.ondemand.com/topic/346599f0890d4dfaaa11c6b4ffa96312 and will be released with the upcoming version 1.99 as well.
Best regards,
Jiawei


API: Fixdo_service response

Inconsistent response. Should be normalized to match other responses.

old name do_service
command attribute is always null

https://github.com/MarechJ/hll_rcon_tool/pull/743

Fixed by #743


Fix javadoc

This commit should fix the jitpack biuld of stetho-realm.
https://github.com/uPhyca/stetho-realm/issues/47
I tested it by running which fails on the current master but succeeds after this change
./gradlew build publish

This PR contains many changes not related to Javadoc.
Could you please update this PR to contain only Javadoc fix?

I've fixed a bit different way.
I'm closing this issue.
Thanks!


Updated app-map.md - comment annotation for python

Updated comment annotation for python code block

@norelina : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


make the --help command work

my use of parent parsers and stuff makes the help command basically completely useless, fix this :)

the help command kind of works:
$ dump backup --help

but just:
$ dump --help

doesn't work. I figured this out for Captain at some point, I just need to figure out what I'm doing different in Captain and add it here


On first use, show a simplified screen

When the app is used for the first time, the database is empty. The user currently sees empty Resume Reading and Recently Added sections. Plus a My Tags section with a label "there is no tags in books.".
In this case, display a large dotted rectangle  containing a + icon with a label "To import ebooks, drag&drop EPUB files here or use the plus button in the menu above.".

After he has imported books, and until he opens a first book, the  Resume Reading is empty. We have to hide it from the view of the user.

fixed in 16ce135


cache issue

histogram-panel seems to load sometimes but not all the time for some users...
sometimes it gives a "missing plugin error" in place of the histogram.
Im not sure of what the issue is... perhaps cache.

perhaps it's just on mac's browser safari

I try to replicate the issue by using Safari.
I reload dashboard which has histogram panel 10 times, but I can't replicate it.
I cant't imagine the reason, why it can't work, sorry...

I close this issue.
If you still have a issue, please make new issue.


Contact Details
<EMAIL_ADDRESS>Name of Identifier
PermID
Link to Explainer
https://permid.org
One-liner Description
PermID is a schema from the London Stock Exchange Group (LSEG). The ODI worked with LSEG to make this open. It includes Organisation IDs as well as financial instrument, quote and person record types, obviously we are only interested in the former.
Price of Registering an Identifier
unknown - not sure that is possible as I believe driven by needs of internal LSEG systems.
Is the Identifier Open for Use?
Yes
Issuing Organization
London Stock Exchange Group (LSEG)
Types of Data Included

[X] Legal Name
[X] Registered Address
[ ] Registration Authority ID
[X] Country of Registration
[X] Entity Status
[X] Other (please specify in the additional information field)

Additional Information
PermID records cover every organisation that LSEG systems and products have come into contact with. Some records are very well populated (business sector, company website, registered and domiciled physically addresses, primary financial instrument and exchange quotes), some records are more sparse. The search and matching APIs are pretty good. The search ranking is very good, typically putting companies higher in the hierarchy to the top of search results. The search also appears to navigate company name changes quite well. Many PermID records are mapped to an LEI.
We have invested heavily in matching the companies we track on human rights activities to PermIDs as a way of making our data interoperable, particularly for financial services companies.

Thanks so much for this submission @trubens71! Looking for some optional additional details--curious if you might have an estimate of number of identifiers within the PermID system, and whether this has global coverage, or tends to have organizations that are concentrated in one country/address of registration region?


[FEATURE] Support preserve user context in group share level

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Search before asking

[X] I have searched in the issues and found no similar issues.

Describe the feature
In group share level, the use case can be:

all user share the same session
every session are independent
every user share their own session

We have supported the first two use case by change config kyuubi.engine.single.spark.session. It would be good to support the third.
Motivation
No response
Describe the solution
No response
Additional context
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

+1


Enhancement: Add download restrictions for uploaded image files

What features would you like to see added?
Currently, image files uploaded to LibreChat can be downloaded by unauthenticated users through the /images endpoint.
I would like to request the following restrictions to be added:

Only authenticated users should be able to download image files.
Users should only be able to download image files that they have uploaded themselves.

I have already implemented and verified the restriction to allow only authenticated users to download images in my local environment.
More details
By imposing restrictions on downloading image files, I believe there will be the following benefits:

User privacy will be protected.
The spread of inappropriate images can be prevented.
Server load can be reduced.

Moreover, I consider this feature addition to be important for enhancing the security of LibreChat. Currently, even unauthenticated users can access image files, which poses a risk of misuse by malicious users.
By placing restrictions on downloading image files, such security risks can be mitigated.
Technically, I believe this would involve adding a process to verify the authentication status and the uploader of the requesting user when downloading image files.
Please consider this request. Thank you.
Which components are impacted by your request?
UI, Endpoints
Pictures
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Note: several users wit intranet solutions that don't use HTTPS reported issues with images loading after locking them down. for this reason, I'm making lcoal image security via URL optional via config file: https://github.com/danny-avila/LibreChat/pull/2415
See docs: https://docs.librechat.ai/install/configuration/custom_config.html#config-structure


Micropub: Add support for delete action

See: 3.5 Delete.

[x] delete
[x] undelete

Undoubtedly buggy and untested, but will file issues separately. As far as this feature is concerned, it’s been added.


Different GeoPosition values for OnTouch and OnClick

Email received:
In our pluggable map MapClickedListener there are 2 methods that are called when clicking on the map. The first is an onTouch listener that will respond to the first touch of the screen regardless of event type (singe, double, long, drag, etc). This means this method is always called when the map is clicked. In the case of long press onTouch is called followed by the MapUserInteractionEvent. I took data on a single click with the mouse pointer and then on the same location a long press. Here is an example of the results.
A single click from the on touch method
On Touch: LatLonPoint[lat=34.765178249375595,lon=-86.41371591023108]
On Touch x,y: Point(815, 189)
on Event Single Press: LatLonPoint[lat=34.76517061862333,lon=-86.41370651566413]
A long press
On Touch: LatLonPoint[lat=34.69682246526478,lon=-86.56669172935936]
MapClickedListener: On Touch x,y: Point(624, 287)
on Event Long Press: LatLonPoint[lat=34.69638638712407,lon=-86.56609375423172]
On Long Press x,y: Point(624, 287)
On Touch: LatLonPoint[lat=34.765178249375595,lon=-86.41371591023108]
On Touch x,y: Point(815, 189)
on Event Long Press: LatLonPoint[lat=34.76517061862333,lon=-86.41370651566413]
On Long Press x,y: Point(815, 189)
Distance between touch and long press: 1.2068360080193425
Included at the end is the distance in meters of how far apart these locations are. Zooming out give a larger error and zooming in averaged about .1 m in distance between touch and long press.
I also tried a few times starting around the equator and long pressing further away. Distance between the onTouch calculation and the long press varried any where from about 1200m to 3800m with no relation to how far from the equator I was. This was zoomed way out to see large changes in the data set. I did a small set zoomed in to 2500m and moving away from the equator. Those numbers ended up pretty close to each other.
Log info
50000m zoom
[SINGLE TOUCH TO THE MAP]
MapClickedListener: On Touch: LatLonPoint[lat=34.69682246526478,lon=-86.56669172935936]
MapClickedListener: On Touch x,y: Point(624, 287)
PickNavigateController: ACTION_UP evt NOT dragging. Picks 0.
MapClickedListener: ClickType: CLICKED
MapClickedListener: on Event Single Press: LatLonPoint[lat=34.69638638712407,lon=-86.56609375423172]
[LONG PRESS TO THE MAP]
MapClickedListener: On Touch: LatLonPoint[lat=34.69682246526478,lon=-86.56669172935936]
MapClickedListener: On Touch x,y: Point(624, 287)
PickNavigateController: Long Press 0
MapClickedListener: ClickType: LONG_PRESS
MapClickedListener: on Event Long Press: LatLonPoint[lat=34.69638638712407,lon=-86.56609375423172]
MapClickedListener: Distance between touch and long press: 73.07587954789241
MapClickedListener: On Long Press x,y: Point(624, 287)
10m zoom
[SINGLE TOUCH TO THE MAP]
MapClickedListener: On Touch: LatLonPoint[lat=34.68413190998026,lon=-86.6487055783966]
MapClickedListener: On Touch x,y: Point(554, 350)
PickNavigateController: ACTION_UP evt NOT dragging. Picks 0.
MapClickedListener: ClickType: CLICKED
MapClickedListener: on Event Single Press: LatLonPoint[lat=34.684131887719715,lon=-86.6487055025367]
[LONG PRESS TO THE MAP]
MapClickedListener: On Touch: LatLonPoint[lat=34.68413190998026,lon=-86.6487055783966]
MapClickedListener: On Touch x,y: Point(554, 350)
PickNavigateController: Long Press 0
MapClickedListener: ClickType: LONG_PRESS
MapClickedListener: on Event Long Press: LatLonPoint[lat=34.684131887719715,lon=-86.6487055025367]
MapClickedListener: Distance between touch and long press: 0.0073648950868234085
MapClickedListener: On Long Press x,y: Point(554, 350)
System.out: Got a long press at latitude: 34.684131887719715 Longitude: -86.6487055025367
1000m zoom
[SINGLE TOUCH TO THE MAP]
MapClickedListener: On Touch: LatLonPoint[lat=34.765178249375595,lon=-86.41371591023108]
MapClickedListener: On Touch x,y: Point(815, 189)
PickNavigateController: ACTION_UP evt NOT dragging. Picks 0.
MapClickedListener: ClickType: CLICKED
MapClickedListener: on Event Single Press: LatLonPoint[lat=34.76517061862333,lon=-86.41370651566413]
[LONG PRESS TO THE MAP]
MapClickedListener: On Touch: LatLonPoint[lat=34.765178249375595,lon=-86.41371591023108]
MapClickedListener: On Touch x,y: Point(815, 189)
PickNavigateController: Long Press 0
MapClickedListener: ClickType: LONG_PRESS
MapClickedListener: on Event Long Press: LatLonPoint[lat=34.76517061862333,lon=-86.41370651566413]
MapClickedListener: Distance between touch and long press: 1.2068360080193425
MapClickedListener: On Long Press x,y: Point(815, 189)
System.out: Got a long press at latitude: 34.76517061862333 Longitude: -86.41370651566413

Calling containerToGeo in the  OnTouch method converts the touch location to a GeoPosition on the map.  If you notice that containerToGeo takes in int values, so the pixel location that was originally a double value is truncated.  This causes the discrepancy to the values you see with MapUserInteractionEvent.  With MapUserInteractionEvent we calculate the GeoPosition using the double value, which is much more precise. We also pass back the pixel coordinates as int values versus their original double type due to android.graphics.Point only accepts int values.  Therefore, if you reverse calculate the GeoPosition, precision will be lost. The GeoPosition being returned in the MapUserIneteractionEvent is recommended over reverse calculating the GeoPosition using containerToGeo.

12-20 10:04:54.389 3727-3727/mil.emp3.dev_test_sdk D/MainActivity: Map User Interactive Event: CLICKED X/Y: 302 / 806  Lat: 39.9395810699428 Lon: -105.20784608281856
12-20 10:04:54.390 3727-3727/mil.emp3.dev_test_sdk D/MainActivity: Map User container to geo: Lat/Lon: 39.93990399775787 / -105.20801032091879
12-20 10:04:54.390 3727-3727/mil.emp3.dev_test_sdk D/MainActivity: Map User container to geo Float: Lat/Lon: 39.9395810699428 / -105.20784608281856
The first print statement is the location that we calculate ourselves,
the next print statement is using the old method containerToGeo with integer pixel coordinates,
the next print statement is using the new method containerToGeo with float pixel coordinates.
Now the reverse calculation can be calculated accurately using the pixel float coordinate values.


Add option for output file extension

Hello, I'd like to propose that an option for changing the file extension be made available. I took the liberty of making a PR with my suggestion. Would love to know what you think.
This adds a config option to set whether the output file should have a ".ts" or ".d.ts" extension.
Thank you

So, first off, sorry for the initial PR comment. I meant to only merge to my fork while I was playing around, but accidentally did a PR with no subject or commit message, and then I kinda panicked and edited one in real quick.
What I wanted to propose was an option to generate declaration files rather than ts files. In my project, I'm already using types defined in a global namespace, so having the namespace generated would save a ton of work adding imports to each file. I initially tried just importing the types into a declaration file and exporting them under a namespace declaration, but after a lot of trial and error, my conclusion was that there is an issue with the module system which hinders re-exporting types which themselves were default exports from a file. Sadly, I couldn't find any documentation about this, but changing the default exports to named exports made things work.
Anyway, I tried to make my proposed changes flexible by adding a "namespace" option to the initializer, which if not set generates ts files as normal, but if given a string value, generates d.ts files instead. It would declare a global namespace containing all of the generated types, making them globally available without an import statement. This did require more changes than I anticipated, so I hope that I'm not proposing something which is over-fit to my specific needs.
I'd love to know if this is a feature you feel is worth including.
Thanks again for the great project!

Hi Avram!
I like the addition of namespace, I think for certain types of projects it's reasonable to have global types.
For large projects, it's probably better to import types manually, though in those cases it might be viable to trigger generate separately for "areas" of the app, and use a different namespace for each.

I'd rather avoid the indentation setting, would you remove it from this PR?
I'm ok switching the default to tabs, but would prefer a separate PR for that change (maybe any users watching the issue can vote).

Sure thing, I removed the indentation option. I had added it in part because I happen to enforce tabs for my frontend files and eslint made a bunch of squigglies when peeking at the generated files (which is very much a non-issue).
Also, a quick note about the snapshot tests. I had to change how they are generated because on my Linux build, the "::" character sequence wasn't allowed in a filename. So this also includes a change to the snapshot generation which converts those to "__". I didn't include the updated snapshots because I didn't want to muddy the PR with a bunch of changes, but can do so if you'd like.

Hello, checking in to see if there's something else I do to make this PR work for you. Let me know, happy to do some refactoring or add some tests.

Hi Avram! Life's been busy around here (moved to a new place).
I intend to merge and release this PR, but need to find some time to test it myself. Thanks!

Funny, I actually just moved as well, and then immediately went on holiday. I believe this last commit addresses your points above, avoiding breaking changes makes a lot of sense for a minor release.

Is there anything else I can do to get this ready? I've been using it in my project and haven't had any issues, I'm referencing the branch directly in my Gemfile. I'd really like to get this rolled in to the gem because CircleCI is trying to run the tests in this project during my build process and failing.

Hi! This comment is still not addressed, which I think makes this change backwards-incompatible as is.
We'd want no changes to "normal" snapshots, plus all the new ones for "namespaced" tests.

I've never used Github comments before, so maybe I resolved them incorrectly, but I believe my last commit addressed all of the concerns. Exports are changed back to default, typo is corrected. I reverted the tests to assert default exports as well, and they are passing.

If you check this import for example, it will fail under tsc, because there is no Model export in that file.

Exports are changed back to default

Correct, but the imports are still using named imports.

Thanks for your contribution Avram!

Released in<EMAIL_ADDRESS>🎉

Hey, thank you for letting me contribute to this and for your patience while I figured things out. I'm not a professional developer, this is more of a hobby for me, so it was really fun to move through the steps of proposing and contributing a new feature.


Unable to load class 'org.objectweb.asm.util.TraceClassVisitor'.

Minecraft 1.16.5
ArcLoom 1.2
Only Forge and Mixin
ArcLoom 1.1 There is no such problem
[I use machine translation]

Duplicate of #132.


Fix Wasm translator bug: end of toplevel frame is branched-to only for fallthrough returns.

This makes the value of state.reachable() inaccurate when observing at
the tail of functions (in the post-function hook) after an ordinary
return instruction.

I'll see what I can whip up -- there currently aren't any direct unit tests for the state presented to the FuncEnvironment, though this does fix a test failure over in bytecodealliance/lucet#612.

Added a few tests of reachability info, with some plumbing on the dummy environment.


Imported curve from Blender is not closed.

When I create a spline/curve in blender and import it into Godot 3.0 its no longer a closed spline but open at the last segment. I am not sure if this is a Bettercollada issue or Godot import issue.
Blender version: 2.79
BetterCollada: Latest commit 836241a (downloaded yesterday)
Godot version:
3.0 stable
OS/device including version:
Win10
Issue description:
Expected to see a closes curve, but the last "step" of the curve is open.
Steps to reproduce:
Create a closed curve in blender and export it using bettercollada.

@jesperkondrup Can you still reproduce this bug in Godot 3.2.1 or 3.2.2beta4?

@Calinou Yes I am still able to reproduce this using Blender 2.82, Better Collada Exporter and Godot 3.2.1 stable.


initialize adt c structs

I started editing codegen.hs to generate initialization of c structs.
I ran into an issue where I didn't have all the information I needed to determine whether an ADT was an integer or pointer. I think this information needs to be passed into gen expr after the call to genTypeDef (I think  genTypeDef will be called in genTop?), or c macros need to be defined at the same as the adt c struct definitions (genTypeDef should generate C macros). I think the latter would be less invasive but I'd like to hear your thoughts on this.

I made changes to

Identifiers.hs (add runtime adt/mm related c-code types and identifiers)
Typdef.hs (generate C-code for each adt and store type info in a lookup table TypeDefInfo)
Codegen.hs (thread TypeDefInfo through state monad, c-struct initialization and pattern matching)

The C code generated should match draft-v2 branch's c definitions, and uses the following example as reference:
// data MyBool = MyTrue int | MyFalse int int
// let a = MyFalse 42 18

enum MyBool{MyTrue = 0, MyFalse};

void main(){
	ssm_value_t tmp;
	// ssm_new takes (max size across all data constructors, tag)
	tmp.heap_ptr = (struct ssm_mm*) ssm_new(2, MyFalse);
	((struct ssm_object*)tmp.heap_ptr)->payload[0].packed_val = ssm_int_obj(42);
	((struct ssm_object*)tmp.heap_ptr)->payload[1].packed_val = ssm_int_obj(18);
	ssm_value_t a = tmp;
}

This seems like it has more casts than it needs.
void main(){
	ssm_value_t tmp;
	// ssm_new takes (max size across all data constructors, tag)
	tmp.heap_ptr = (struct ssm_mm*) ssm_new(2, MyFalse);
	((struct ssm_object*)tmp.heap_ptr)->payload[0].packed_val = ssm_int_obj(42);
	((struct ssm_object*)tmp.heap_ptr)->payload[1].packed_val = ssm_int_obj(18);
	ssm_value_t a = tmp;
}

shouldn't it just be tmp.heap_ptr->payload[0].packed_val = ssm_int_obj(42);?
Similarly, shouldn't ssm_new() always return an ssm_value_t ?

The draft-v2 branch contains some macros to get rid of pointer casts entirely. You should be able to just write:
void main() {
  struct ssm_object obj = ssm_new(/* max xize */ 2, /* tag */ MyFalse);
  obj->payload[0] = ssm_marshal(42);
  obj->payload[1] = ssm_marshal(18);
  ssm_value_t a = ssm_from_obj(tmp);
}

The types of some of these macros are, roughly:
ssm_marshal : int -> ssm_value_t
ssm_from_obj : struct ssm_object * -> ssm_value_t
ssm_to_obj : ssm_value_t -> struct ssm_object *

Similarly, shouldn't ssm_new() always return an ssm_value_t ?

I wrote ssm_new() to return a struct ssm_object * because the caller still needs to initialize the payloads, so some kind of cast by the caller is just unavoidable. I figured that having ssm_new() (and allocating functions) return a more specific type was more self-documenting, so I had it return struct ssm_object *. I tried to make the caller perform as many casts as possible because ultimately the caller is the one with the most type information.

I've updated codegen.hs / identifiers.hs so (hopefully) the following code will be generated with this example:
// data MyBool = MyTrue int | MyFalse int int
// let a = MyFalse 42 18
enum MyBool{MyTrue = 0, MyFalse};
void main() {
  struct ssm_object* tmp = ssm_new(/* max size */ 2, /* tag */ MyFalse);
  tmp->payload[0] = (ssm_value_t) { .packed_val = ((42) << 1 | 1) };
  tmp->payload[1] = (ssm_value_t) { .packed_val = ((18) << 1 | 1) };
  ssm_value_t a = (ssm_value_t) { .heap_ptr = &(tmp)->mm };
}

I tried to have codegen generate the macros itself as much as possible. The code generated should be equivalent to
void main() {
  struct ssm_object* tmp = ssm_new(/* max size */ 2, /* tag */ MyFalse);
  tmp->payload[0] = ssm_marshal(42);
  tmp->payload[1] = = ssm_marshal(18);
  ssm_value_t a = ssm_from_obj(tmp);
}

Currently working on writing a unit test with the MyBool example in IR. Once I have it in IR I will test my work.


Access Denied for user?

07:55:05 [INFO] [LuckPerms] Performing initial data load...
07:55:08 [INFO] 07:55:08.054 [luckperms-hikari connection adder] DEBUG me.lucko.luckperms.lib.hikari.pool.HikariPool - luckperms-hikari - Cannot acquire connection from data source java.sql.SQLException: Access denied for user 'jacobal1_admin'@'cpc77299-basf12-2-0 cust104.12-3.cable.virginm.net' (using password: YES)

Why is this not working?
hasteb.in https://hasteb.in/upavafip.cs

well i created my own on the site

Are you sure your MySQL server IP address (it's called host in the LP config) is correct?
Either way, this is a configuration error, not a bug.


Added Marshal

https://github.com/utahiosmac/Marshal

One more thing, can I also push a new version of setUp() ?
var largeData: Data!
    var smallData: Data!

    override func setUp() {
        super.setUp()

        let bundle = Bundle(for: type(of: self))
        if let path = bundle.url(forResource: "PerformanceTest", withExtension: "json"),
            let data = try? Data(contentsOf: path) {
            largeData = data
        }
        if let path = bundle.url(forResource: "PerformanceSmallTest", withExtension: "json"),
            let data = try? Data(contentsOf: path) {
            smallData = data
        }
    }

To avoid NSData -> Data casting inside each method.

@bre7 I agree, makes sense not to cast it everywhere. However, let's make that another PR as it's not related to adding Marshal :)

Thanks; waiting for Travis to build successfully then we'll merge.

I avoided using custom operators, but for the record (and your README), Marshal does provide one <|

@bre7 Thanks for the PR, merged it now 🌮 🎉

Awesome ! I submitted a new one 🤓


feat: solve the problem of number in Chinese text

This PR introduces several improvements to the text normalization and processing pipeline, ensuring a more robust conversion of mixed English and Chinese text to Pinyin with tone marks. The changes enhance text compatibility and address edge cases in the input data.

the simplest way is detect language -> zh -> use this function.
You'd better use this function in prepare dataset


Valid plan rejected for no clear reason

In the attached zip file there is a domain, instance and a plan that seems valid (INVAL says so), while VAL says that it is not. The only explanation I can think of is that the action defined in the domain has no preconditions.
bug3.zip
Cheers,
Mohammad

Thanks for these. I will take a look and get back to you.
Sent from Samsung tablet.
-------- Original message --------From: mabdula<EMAIL_ADDRESS>Date: 03/06/2018  17:27  (GMT+00:00) To: KCL-Planning/VAL<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [KCL-Planning/VAL] Valid plan rejected for no clear reason (#29)
In the attached zip file there is a domain, instance and a plan that seems valid (INVAL says so), while VAL says that it is not. The only explanation I can think of is that the action defined in the domain has no preconditions.
bug3.zip
Cheers,
Mohammad
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.
{"@context":"http://schema.org","@type":"EmailMessage","potentialAction":{"@type":"ViewAction","target":"https://github.com/KCL-Planning/VAL/issues/29","url":"https://github.com/KCL-Planning/VAL/issues/29","name":"View Issue"},"description":"View this Issue on GitHub","publisher":{"@type":"Organization","name":"GitHub","url":"https://github.com"}}
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/KCL-Planning/VAL","title":"KCL-Planning/VAL","subtitle":"GitHub repository","main_image_url":"https://assets-cdn.github.com/images/email/message_cards/header.png","avatar_image_url":"https://assets-cdn.github.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/KCL-Planning/VAL"}},"updates":{"snippets":[{"icon":"DESCRIPTION","message":"Valid plan rejected for no clear reason (#29)"}],"action":{"name":"View Issue","url":"https://github.com/KCL-Planning/VAL/issues/29"}}}
{
"@type": "MessageCard",
"@context": "http://schema.org/extensions",
"hideOriginalBody": "false",
"originator": "37567f93-e2a7-4e2a-ad37-a9160fc62647",
"title": "Valid plan rejected for no clear reason (#29)",
"sections": [
{
"text": "",
"activityTitle": "mabdula",
"activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png",
"activitySubtitle": "@mabdula",
"facts": [
{
"name": "Repository: ",
"value": "KCL-Planning/VAL"
},
{
"name": "Issue #: ",
"value": 29
}
]
}
],
"potentialAction": [
{
"name": "Add a comment",
"@type": "ActionCard",
"inputs": [
{
"isMultiLine": true,
"@type": "TextInput",
"id": "IssueComment",
"isRequired": false
}
],
"actions": [
{
"name": "Comment",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n"commandName": "IssueComment",\n"repositoryFullName": "KCL-Planning/VAL",\n"issueId": 29,\n"IssueComment": "{{IssueComment.value}}"\n}"
}
]
},
{
"name": "Close issue",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n"commandName": "IssueClose",\n"repositoryFullName": "KCL-Planning/VAL",\n"issueId": 29\n}"
},
{
"targets": [
{
"os": "default",
"uri": "https://github.com/KCL-Planning/VAL/issues/29"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n"commandName": "MuteNotification",\n"threadId": 341873591\n}"
}
],
"themeColor": "26292E"
}

This one is not very interesting, either - the precondition field is not optional so the parser fails in the action and then gets confused by the rest of the input.

From: mabdula<EMAIL_ADDRESS>Sent: 03 June 2018 17:27:09
To: KCL-Planning/VAL
Cc: Subscribed
Subject: [KCL-Planning/VAL] Valid plan rejected for no clear reason (#29)
In the attached zip file there is a domain, instance and a plan that seems valid (INVAL says so), while VAL says that it is not. The only explanation I can think of is that the action defined in the domain has no preconditions.
bug3.ziphttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FKCL-Planning%2FVAL%2Ffiles%2F2066133%2Fbug3.zip&data=01|01|derek.long%40kcl.ac.uk|6f41fad86308443dec4108d5c96edaef|8370cf1416f34c16b83c724071654356|0&sdata=TDz2%2BHxko2q0zTNrAWUA6jWILnfWA3ZVQIt4mCr5nks%3D&reserved=0
Cheers,
Mohammad
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FKCL-Planning%2FVAL%2Fissues%2F29&data=01|01|derek.long%40kcl.ac.uk|6f41fad86308443dec4108d5c96edaef|8370cf1416f34c16b83c724071654356|0&sdata=L8M4FJgzzD1URUVGumWtDqgBnB%2BwXt6IUk93ktmCXW0%3D&reserved=0, or mute the threadhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAHSnVBLN8iJxBfe0np9JyTv1lmSg3JXQks5t5A5dgaJpZM4UYJO3&data=01|01|derek.long%40kcl.ac.uk|6f41fad86308443dec4108d5c96edaef|8370cf1416f34c16b83c724071654356|0&sdata=AhoNn5NIKuaGEHhNf726M4ZGoaqGDWyMzst%2FRreQaak%3D&reserved=0.

Yes, quite right, VAL does not accept a missing precondition field. The rule in the parser is:
c_action_def :
OPEN_BRAC  ACTION NAME  c_args_head OPEN_BRAC c_typed_var_list CLOSE_BRAC
PRE c_pre_goal_descriptor
EFFECTS c_effect
CLOSE_BRAC
|   OPEN_BRAC ACTION error CLOSE_BRAC
;
Kovacs' syntax was his own effort to synthesise a grammar and postdates the construction of VAL by very many years. The PDDL syntax in the original PDDL 2.1 paper, extended to account for PDDL3.1 (constraints), PDDL2.2 (TILs, derived predicates) and TIFs (analogous to TILs, but assigning values to numeric fluents), PDDL+ (events, processes) are all handled. The PDDL2.1 paper is confusing in that it specifies fields for durative actions (conditions and effects) as required, but for actions as optional. So, this specification suggests that VAL should be permissive about preconditions. However, it is not.
I think that the PDDL2.1 BNF is about as close as you will get, other than scrutiny of the pddl.yacc file. The main issues of divergence will be in the corner cases (such as you identify).
Cheers
Derek

From: mabdula<EMAIL_ADDRESS>Sent: 06 June 2018 10:41:33
To: KCL-Planning/VAL
Cc: Long, Derek; Comment
Subject: Re: [KCL-Planning/VAL] Valid plan rejected for no clear reason (#29)
I see. Just to confirm my understanding, the syntax accepted by VAL does not conform to the BNF by Kovacs or the one in McDermot et. al's report on PDDL1.2, which both seem to suggest that the precondition field is optional. Is that correct? And if it is, is there a document summarising the syntax accepted by VAL?
Cheers
Mohammad
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FKCL-Planning%2FVAL%2Fissues%2F29%23issuecomment-395009901&data=01|01|derek.long%40kcl.ac.uk|2de23ef01c20436e0ea808d5cb91b21e|8370cf1416f34c16b83c724071654356|0&sdata=FMmlcNCDukYYZOoz%2FwTjFeXp6ol%2BoIgf9w6FK%2FRxlsA%3D&reserved=0, or mute the threadhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAHSnVAAHKU3PkqPk5G4Yf-PFPSur_XI6ks5t56PNgaJpZM4UYJO3&data=01|01|derek.long%40kcl.ac.uk|2de23ef01c20436e0ea808d5cb91b21e|8370cf1416f34c16b83c724071654356|0&sdata=Xb%2BlRSjeHWxw1vQgTTc0hsWu0JOYBwjXFChTN%2FddotE%3D&reserved=0.

I see, thanks a lot for your response.
Cheers
Mohammad

@mabdula @DerekLong101 This is fixed in #25

should be closed

As noted.


Added logic and unit tests for a simple version of Header forwarding …

…(from responses from api.xmidt to some TR1D1UM caller)

Codecov Report

Merging #10 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #10   +/-   ##
=======================================
  Coverage   66.43%   66.43%
=======================================
  Files           4        4
  Lines         283      283
=======================================
  Hits          188      188
  Misses         95       95

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 4edb4c3...91705c4. Read the comment docs.


Adding french and english translation

An other pull request was made for this issue. This pull request is deprecated.


Elm command should have a command or switch to print out the version (and nothing else)

It is useful to be able to extract the version of the installed compiler programmatically. Right now the way to do this is to run the elm command and then taking the first line of the output. Even then, the output contains extra text that may change between releases.
It would be better to have an explicit command, such as elm version or a switch (-v or --version) that output the version number and nothing else.

+1 for elm make --version. Others are more subjective. But it's low priority.

+1 for -v and --version both.

I don't understand why the version would be hidden behind elm make. After all, simply running elm right now gives you the version along with the usage guide. From my end user perspective, the elm command is Elm.

It's not, though. elm-make is actually it's own thing.

elm --version could print just the Elm Platform version, while the individual tools would print their own versions.

I've tripped on this too from time to time. Ultimately I'd like elm --version to have output the platform version as well as each component. It could be convenient over checking the version of each component one at a time (for the rare scenario where you would need to do such a thing.)

@mgold @Fresheyeball I agree with you both, here. I really think that this makes the most sense:
$ elm -v or $ elm --version for an Elm platform version & the version of all included components.
$ elm make -v or $ elm make --version for the version of the make component only.

I think there's an issue open for this on the elm-make repo (https://github.com/elm-lang/elm-make/issues) but I'm not sure.
In any case, that's where this should be. Also, the version should be uniform across all command line tools now. For the CLI's that have this flag, you can see this now:
$ elm-repl --version
0.16.0
$ elm-reactor --version
0.16.0

Oh, I see, it does not work on elm, elm-make, or elm-package at the moment. I'll open an appropriate issue for this.


Incorrect displayed day

Mods: Serene seasons, SeasonHUD and Serene seasons fix + journey map for display
The displayed day was incorrect even before the seasons fix mod.

I'll take a look when I get home later today. Which game version was this example on?

My thanks mate!

e5049fe
Fixed in 1.4.16

Working properly thank you in the name of the server!


New default cover art image

The top one is the current extension art fallback, and is no longer used on the Last.fm site
The middle and bottom ones are both used in the Last.fm redesign ( here & here )

Thoughts on whether/which way to proceed?

I like the note :-)

Bottom one looks better ;)

+1 for bottom cover art image. I think it looks better than the top and middle ones.

Relevant?

Added "note" icon.


Pep8 refactor

For the UoA-DSC-Experiements I had a look at it and did some basic re factoring for pep8/pylint compatibility. I tested it afterwards and everything still worked.

Awesome job, thanks heaps!


Template not copying files to root directory in GitHub Actions runners

Okay this seems to be a larger issue. On a GitHub Actions runner, copier seems to run successfully, however, top-level files copied over in the template (such as .flake8) don't actually get copied over. This is the source of flake8 behaving differently in Actions than locally.
Manual touch and mkdir calls work fine, it is something to do with copier failing to properly create top-level files, despite thinking that it did so. Files nested in at least one folder copy over just fine, however. A workaround could be to keep master versions in .tooling, and run CI/CD off of that, but it still doesn't solve the issue of Copier improperly copying top-level files.
It may be related to copier-org/copier#273 somehow. That Copier is careful when dealing with folders that it did not create, and this somehow leads to a file not being properly created at root? Will have to look further.

This is bypassed by simply unignoring most files, especially those necessary for runners. Someday it may be possible to re-ignore these files, but for now let's just commit them to every repo. The new file-nesting feature declutters the root directory whether or not these root tooling configs are committed anyways.


Molecules layer checkbox causes blank screen

Describe the bug
After unchecking the molecules on the spatial component, checking the molecules again causes the page to go blank.
To Reproduce
Steps to reproduce the behavior:

Un-check the molecules layer in the spatial component
Check the molecules layer in the spatial component

Expected behavior
All components should remain on the screen and the molecules should re-appear in the spatial view.
Environment:

Release or git hash: 9e9f154
Browser: Firefox Developer Edition
Browser version: 69.0b2

Possibly related to https://github.com/hubmapconsortium/vitessce/issues/211

In the console: Error: "deck.gl: assertion failed."
The assertion that's failing is assert(!this.internalState && !this.state);

deck: error while initializing ScatterplotLayer({id: 'scatter-plot'})
 Error: "deck.gl: assertion failed."
    assert assert.js:3
    _initState layer.js:934
    _initialize layer.js:650
    _initializeLayer layer-manager.js:452
    _updateSublayersRecursively layer-manager.js:383
    _updateLayers layer-manager.js:338
    setLayers layer-manager.js:193
    setProps layer-manager.js:154
    setProps deck.js:211
    _updateFromProps deckgl.js:162
    shouldComponentUpdate deckgl.js:50
    React 13
    unstable_runWithPriority scheduler.development.js:255
    React 3

When I check out 135dc7df9a8f6a70d / v0.0.15 there is now problem: I can toggle the molecules on http://localhost:3000/?dataset=linnarsson-2018 without any problems. Figure out which commit broke this...


Sending image in non asp.net app.

I have a class library where I need to send emails with images.
I get the following error when using @Html.EmbedImage.  I'm using the MultiPart example but in a class library.
Unable to compile template. The name 'Html' does not exist in the current context.
Is this possible?

I have also spent some hours trying to get around this issue.
Mine solution was as follows (pseudo C# code):
Attachment someImageAttachment = new Attachment(imagePath);
emailModel.SomeImageContentId = someImageAttachment.ContentId;
email.Attach(someImageAttachment);
in Email.cshtml:
<img<EMAIL_ADDRESS>alt="SomeImage" />
Works perfectly in every e-mail client I have tested and the image is not visible as an attachment, which makes this solution even more awesome ;)
Hope it helps you and saves you lots of time :)

@JerryDoubleU  Wow, that is a nice workaround!

@JerryDoubleU that worked fine. Awesome workaround! Thanks!

@JerryDoubleU  Thanks for posting!


How to do file upload with Sugar-Framework?

Please can you provide a guide on how to do file upload with sugar-framework.
Also, a sample with other common / everyday use-cases would be appreciated.
Thanks.

I was able to get this to work as expected with a few tweaks from your original gist.
In mix.exs:
defmodule SugarTest.Mixfile do
  use Mix.Project

  def project do
    [app: :sugar_test,
     version: "0.0.1",
     elixir: "~> 1.2",
     build_embedded: Mix.env == :prod,
     start_permanent: Mix.env == :prod,
     deps: deps]
  end

  def application do
    [applications: [:logger, :sugar],  # Note the inclusion of :sugar here
     mod: {SugarTest, []}]
  end

  defp deps do
    [{:sugar, "~> 0.4.10"}]
  end
end

In config/config.exs (pretty typical):
use Mix.Config

config :sugar_test, SugarTest.Repos.Main,
  adapter: Ecto.Adapters.Postgres,
  database: "sugar_test_main",
  username: "user",
  password: "pass",
  hostname: "localhost"

config :sugar, router: SugarTest.Router

config :sugar, SugarTest.Router,
  https_only: false,
  http: [port: 4000],
  https: false

In lib/sugar_test/router.ex:
defmodule SugarTest.Router do
  use Sugar.Router
  plug Sugar.Plugs.HotCodeReload

  if Sugar.Config.get(:sugar, :show_debugger, false) do
    use Plug.Debugger, otp_app: :sugar_test
  end

  plug Plug.Static, at: "/static", from: :sugar_test

  # Uncomment the following line for session store
  # plug Plug.Session, store: :ets, key: "sid", secure: true, table: :session

  # Note the use of Plug.Parsers, and the use of the actual parsing
  # module (Plug.Parsers.MULTIPART) instead of an atom (:multipart)
  plug Plug.Parsers, parsers: [ Plug.Parsers.MULTIPART ]

  # Define your routes here
  get "/", SugarTest.Controllers.Main, :index
  post "/upload", SugarTest.Controllers.Main, :upload  # Note the POST
end

In lib/sugar_test/controllers/main.ex:
defmodule SugarTest.Controllers.Main do
  use Sugar.Controller

  def index(conn, []) do
    conn |> render
  end

  def upload(conn, _) do
    require Logger
    Logger.debug(conn |> inspect)

    case conn.params["image"] do
      %Plug.Upload{filename: filename, path: path} ->
        {:ok, image} = File.read path
        {:ok, file} = File.open filename, [:write]
        IO.binwrite file, image
        File.close file

        conn |> render(filename: filename)  # Note use of render, not resp
      nil ->
        raw conn |> resp(400, "image was not attached!")
    end
  end
end

In lib/sugar_test/views/main/index.html.eex:
<!doctype html>

  <title>main/index - SugarTest</title>

  <form enctype="multipart/form-data" action="/upload" method="post">
    <input type="file" name="image">
    <input type="submit">
  </form>

In lib/sugar_test/views/main/upload.html.eex:
image[<%= @filename %>] was saved!

Needless to say, there's a lot that needs documenting.  Some of it is in the scope of Sugar (render v. resp, as well as some troubleshooting of MIME types and headers).  Most is in the scope of Plug (in particular, Plug.Parsers and Plug.Upload; some of the docs there aren't entirely accurate).
I'd like to spin up a "Tips and Tricks" section of the Sugar docs site for these sorts of things.  @slogsdon, if you've got any other suggestions for such a section (I'm already thinking of covering file uploads, JS/CSS asset management techniques, login/session management, and Phoenix/Sugar controller compatibility), I'm all ears.

Thanks.

No problem.  Closing for now (since this is now addressed).


Outdated variants?

Hi,
As per a comment on a previous issue, images should be getting updates on the first Saturday of each month. However, looking at the PR that introduced the dynamic github actions runner fetcher, it seems like at least ghcr.io/cirruslabs/macos-sonoma-xcode:15.1 has not been updated to reflect that change.
Is this schedule still applicable?
Many thanks 🙏

You guys are amazing! Thank you so much!

For the next version of macOS we'll probably switch to an omni variant with several latest Xcode versions. Which we'll update even more frequently. 👌

@fkorotkov is the same cron schedule maintained with the current release setup? First Saturday of each month?

Yes, waiting for 5am EST tomorrow myself to see how it goes and fix things if needed.

Good morning! Looks like https://github.com/cirruslabs/macos-image-templates/pkgs/container/macos-sonoma-xcode is still not updated with the latest changes.

Morning! My bad. The cron jos is scheduled at 7am EST every first Saturday of a month. Next invocation should be in 35 minutes and should appear here. Build will take an hour or so though.

Didn't work because of a silly typo. Changed from to 9am which will retry in 25 minutes

Indeed - the second run worked great! Thank you!


Enable deserializing IRedisCacheItem and IRedisCacheItem

Fixes #13

Thanks. I've added a test to cover this. If you know how to handle this for MessagePack, would you be kind enough to do the same? I'm not sure if its possible from the docs as the custom serializers dont seem to get applied with abstract types.

At the moment I don't have time I'm sorry.

When can you push to nuget?

Next version should go out tonight.


Github Pages

Go to repository Settings and set up GitHub Pages (from the main branch), update repository description with the proper website link. Follow the link to check if everything is fine - you should see your index.html content.

Here is a link


recover shitfuck UI

detyi;ofdsvep984ubtoireutreio WHY IS IT POSSIBLE TO TRASH MY ENTIRE REPO WITH ONE BUTTON

prayge


Make sure that signed messages represent specific-enough propositions

This issue keeps track of making sure that each signed message represents one statement.  Preventing:

two different methods take the same format by accident, and some messages can be interpreted in two ways
a contract keeps track of which signed message has already been used, but there can be multiple instances of that contract, so the same message can be used as many times as the number of contract instances.

Duplicate of #907.


⚠ Bofrost.lu has degraded performance

In 331b28e, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2287 ms

Resolved: Bofrost.lu performance has improved in e461e00 after 12 minutes.


refactor(horovod-test): change file structure

Description
1. Why the change?
Second step of https://github.com/intel-analytics/BigDL/pull/7672.
This PR refactors the testing file structure of pytest horovod-tf2 and horovod-torch
TODO: remove the PR-test and only keep the nightly-test?
2. Summary of the change

3. How to test?
The following orca unit test

[ ] Orca-Pytest-Horovod-Py37-Spark3 ( same as nightly)
[ ] Orca-Pytest-Ray-Py37-Spark3
[ ] Orca-Pytest-Ray-Py38-Spark3

merged


Move file hashing to a worker thread, add async method

Move file hashing to thread, add async method, fixes #16

@sindresorhus added async method

@sindresorhus This one is ready, as far as I can tell.

You need to add thread.js to files in package.json.

Can you document in the method that uses worker threads that they use worker threads and are async?

Also note https://github.com/sindresorhus/crypto-hash/commit/46f2f289de46cfeae7091e8322af62572ab960bd

@sindresorhus  Style fixed, added notes regarding worker_threads to relevant methods in ts and readme.
What do you think of moving worker_threads functionality into microlib in the future? We have partially mirroring implementation both in hasha and crypto-hash now(more functionality in hasha). I think have enough info to unify it.

What do you think of moving worker_threads functionality into microlib in the future? We have partially mirroring implementation both in hasha and crypto-hash now(more functionality in hasha). I think have enough info to unify it.

Yeah, that would be great. I have a note somewhere on my computer about asking you to do that, actually.

Looks good :)