Support mapping types in OpenAPI schema

It is useful to return some grouped data
@dataclass
class User:
    id: str
    name: str

Example for Dict[str, list[User]]
{
  "type": "object",
  "patternProperties": {
    "^[a-z0-9!\\\"#$%&'()*+,.\\/:;<=>?@\\[\\] ^_`{|}~-]+$": {
      "type": "array",
      "items": {
        "type": "object",
        "properties": {
          "id": { "type": "string" },
          "name": { "type": "string" }
        },
        "required": ["id", "name"]
      }
    }
  }
}

I've pinned black to 23.12.1 because in the 24 release it wants to make a lot of changes. I guess it should be a separate patch.

Codecov Report
Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (bec8e14) 96.37% compared to head (7d6659b) 96.28%.

Files
Patch %
Lines

blacksheep/server/openapi/v3.py
84.21%
3 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #476      +/-   ##
==========================================
- Coverage   96.37%   96.28%   -0.10%
==========================================
  Files          69       69
  Lines        6487     6506      +19
==========================================
+ Hits         6252     6264      +12
- Misses        235      242       +7

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

I don't understand what is the meaning of this fragment:
                    description: Success response
                    content:
                        application/json:
                            schema:
                                type: object
                                properties:
                                    ^[a-z0-9!\"#$%&'()*+,.\/:;<=>?@\[\] ^_`{|}~-]+$:
                                        type: object
                                        properties:
                                            ^[0-9]+$:
                                                $ref: '#/components/schemas/Cat'
                                        nullable: false
                                nullable: false

The documentation here for OpenAPI specification 3.0 describes structures that look much simpler than this one:
https://swagger.io/docs/specification/data-models/dictionaries/

@RobertoPrevato Oh, I forgot about additionalProperties, thanks for the example. The idea was to also add type validation(via regex) for keys in dicts. The regex looks ugly, but I couldn't find another way to validate a type for keys.
I'll prepare changes with additionalProperties. I think it will look much better

it requires https://github.com/Neoteroi/essentials-openapi/pull/43


Load balancing across multiple kibana nodes fail

Hi,
We have 2 kibana nodes and we have installed logtrail on both nodes and we are using apache reverse proxy to balance the requests between the 2 nodes.
However once we start up both nodes in the same time we receive the attached error

there is no error reported in kibana logs and when we checked back with elastic support, they came back pointing toward the logtrail plugin as causing this issue, i have tried to use disable the plugin on both nodes and try again and it did resolve the issue.
Is there any possibility to resolve this issue from your side, i have very limited experience in this area so i cant locate the issue on my side.
Please let me know what kind of info you need to help you trace down the issue.
ELK stack version 5.5.1
Reagrds
Mohamed Farag

@mohamed-mfarag I have not tested logtrail plugin in this set up. Did you get any info from Elastic support regarding the issue cause by logtrail plugin? Can you also attach the errors if any, from the Chrome developer console?

@sivasamyk  Sadly Elastic didnt offer much as logtrail is not an officially supported plugin, Chrome does not show any errors, that was the strange thing and why i was not able to locate the error in the beginning.

Can you share the apache reverse proxy configuration for Kibana? I will try to reproduce the issue locally.

Please find it below :

<Proxy "balancer://balancer-kibana">
BalancerMember "http://kibana-app01:5601"
BalancerMember "http://kibana-app02:5601"

ProxyPass /kibana balancer://balancer-kibana
ProxyPassReverse /kibana balancer://balancer-kibana
ProxyPass /bundles balancer://balancer-kibana/bundles
ProxyPassReverse /bundles balancer://balancer-kibana/bundles
ProxyPass /logtrail balancer://balancer-kibana/logtrail
ProxyPassReverse /logtrail balancer://balancer-kibana/logtrail
ProxyPass /status balancer://balancer-kibana/status
ProxyPassReverse /status balancer://balancer-kibana/status
ProxyPass /api balancer://balancer-kibana/api
ProxyPassReverse /api balancer://balancer-kibana/api
ProxyPass /api/status balancer://balancer-kibana/api/status
ProxyPassReverse /api/status balancer://balancer-kibana/api/status
ProxyPass /plugins balancer://balancer-kibana/plugins
ProxyPassReverse /plugins balancer://balancer-kibana/plugins
ProxyPass /app balancer://balancer-kibana/app
ProxyPassReverse /app balancer://balancer-kibana/app
ProxyPass /elasticsearch balancer://balancer-kibana/elasticsearch
ProxyPassReverse /elasticsearch balancer://balancer-kibana/elasticsearch
ProxyPass /login balancer://balancer-kibana/login
ProxyPassReverse /login balancer://balancer-kibana/login
ProxyPass /logout balancer://balancer-kibana/logout
ProxyPassReverse /logout balancer://balancer-kibana/logout
ProxyPass /app balancer://balancer-kibana/app
ProxyPassReverse /app balancer://balancer-kibana/app
ProxyPass /es_admin balancer://balancer-kibana/es_admin
ProxyPassReverse / balancer://balancer-kibana/es_admin
ProxyPass /ui balancer://balancer-kibana/ui
ProxyPassReverse /ui balancer://balancer-kibana/ui
ProxyPass / balancer://balancer-kibana/app/kibana
ProxyPassReverse / balancer://balancer-kibana/app/kibana
ProxyPreserveHost on

@mohamed-mfarag I tried with attached conf (000-default.conf) and it works. Kibana version is 5.5.2
apache2 -v
Server version: Apache/2.4.18 (Ubuntu)
Server built:   2017-07-27T14:34:01

reverse-proxy.txt

Do you have xpack also enabled ?

From: Sivasamy Kaliappan<EMAIL_ADDRESS>Sent: Monday, September 18, 2017 10:01:25 AM
To: sivasamyk/logtrail
Cc: mohamed-mfarag; Mention
Subject: Re: [sivasamyk/logtrail] Load balancing across multiple kibana nodes fail (#180)
@mohamed-mfaraghttps://github.com/mohamed-mfarag I tried with attached conf (000-default.conf) and it works. Kibana version is 5.5.2
apache2 -v
Server version: Apache/2.4.18 (Ubuntu)
Server built:   2017-07-27T14:34:01
reverse-proxy.txthttps://github.com/sivasamyk/logtrail/files/1309998/reverse-proxy.txt
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/sivasamyk/logtrail/issues/180#issuecomment-330151783, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AOK3TSYOpNzeABd_VEQCOcJ_vw9uL_EKks5sjiNVgaJpZM4PY4MS.

No. Mine is plain Kibana installation with logtrail. I will try with x-pack installed.

Oops. Looks like xpack is licensed. I do not have license to run x-pack.

Sorry for the late reply, i was not able to update for the last 2 weeks, however i know there is a trail license for x-pack, would you be able to test using this one ?

@mohamed-mfarag There is a solution suggested in this link https://discuss.elastic.co/t/kibana-uncaught-typeerror/82107 for same Kibana error.

This issue is for only single node as far as i can see from the post and if its related to the user shell it should happen on both node but the issue we have happen only if we use both nodes and balance them using Apache - but if we stop one every thing work fine, have you managed to test with X-pack , please let me know if i can help ?

Hi @johnseekins
Thanks for your suggestion, that did solve the issue !


Preserve URL fragment during SAML handshake.

Hi, we are using Open Distro (1.6.0) with the security plugin activated, and integration with Auth0 SSO.
After logging in via SAML for the first time, we were brought to the default application within Kibana and we aren't preserving the original URL that the user tried to visit before being prompted to login.
Current SAML handshake flow:
-Unauthenticated user navigates to https://kibana.com/app/kibana#/dashboard/demo_dashboard
-User authenticates at IdP and is redirected back to Kibana
-User lands on kibana home page https://kibana.com/app/kibana loosing the deeplink (fragment after # )
We came to notice that Elastic has solved this issuee in https://github.com/elastic/kibana/pull/44513 , which was merged to several 7.x releases. We also understand that ODES 1.6.0 comes with elasticsearch 7.6.1, and was released 6 months later of the above mentioned PR.
Moreover, we are currently using the same IdP with many other services and the redirection works fine.
Thank you very much!

This would also be a big help for us.

The issue has been resolve with PR https://github.com/opensearch-project/security-dashboards-plugin/pull/1039


switched to terser-webpack-plugin from uglifyjs-webpack-plugin

I've been incorporating some changes from the dev branch into a project I built with the previous release, and I ran into this issue at build time: https://github.com/webpack-contrib/uglifyjs-webpack-plugin/issues/359
Turns out uglifyjs-webpack-plugin is no longer using uglify-es on the backend as it's no longer actively maintained, and they recommend switching to terser. There may be a good reason to stay on Uglify, I honestly haven't dug in too deeply, but this configuration is working on my project and it might save people from some small headaches in the future.

Coverage remained the same at 100.0% when pulling 3d72b0684a2585e60b7fe078b9f528304159c827 on robertaird:ReplaceUglifyJS into c875e09d7eedbf114f01e69823d9b24bfc375b04 on react-boilerplate:dev.

Same! Looks good to me! Thanks for noticing and opening a PR for this @robertaird!
Please just fix the package.json conflict with the correct versions:
"terser-webpack-plugin": "1.1.0",
"url-loader": "1.1.2",
"webpack": "4.21.0",

and update package-lock.json accordingly and then we'll merge it!

@julienben Should be all set!


[geoip] refactor maxmind provider lookup functions

Commit Message: refactor maxmind provider lookup functions
Additional Description:
Extracted common maxmind db lookup functions into lookupInMaxmindDb
Introduced isLookupEnabledForDbType and populateGeoLookupResultForDbType to handle db-specific logics
Risk Level: low
Testing: TODO
Docs Changes:
Release Notes:
Platform Specific Features:

seems PR is closed, possible to re-open or in this case sending a new PR is preferred?

@hwyuan reopened


Build failed on EC2 g2 instances

For bugs or installation issues, please provide the following information.
The more information you provide, the more likely people will be able to help you.
Environment info
Operating System: Ubuntu 14.04
Compiler: default
Package used (Python/R/Scala/Julia): default
MXNet version: latest master branch
Or if installed from source:
MXNet commit hash (git rev-parse HEAD):
If you are using python package, please provide
Python version and distribution:
If you are using R package, please provide
R sessionInfo():
Error Message:
/usr/local/cuda/bin/nvcc -c -o build/src/operator/tensor/matrix_op_gpu.o -std=c++11 -Xcompiler -D_FORCE_INLINES -g -O3 -ccbin g++   -Xcompiler "-DMSHADOW_FORCE_STREAM -Wall -O3 -I/home/ubuntu/mxnet/mshadow/ -I/home/ubuntu/mxnet/dmlc-core/include -fPIC -I/home/ubuntu/mxnet/nnvm/include -Iinclude -funroll-loops -Wno-unused-variable -Wno-unused-parameter -Wno-unknown-pragmas -Wno-unused-local-typedefs -msse3 -I/usr/local/cuda/include -DMSHADOW_USE_CBLAS=1 -DMSHADOW_USE_MKL=0 -DMSHADOW_RABIT_PS=0 -DMSHADOW_DIST_PS=0 -DMSDHADOW_USE_PASCAL=0 -DMXNET_USE_PROFILER=1 -DMXNET_USE_OPENCV=1 -I/usr/include/opencv   -fopenmp -DMSHADOW_USE_CUDNN=1  -DMXNET_USE_DIST_KVSTORE -I/home/ubuntu/mxnet/ps-lite/include -I/home/ubuntu/mxnet/deps/include -I/home/ubuntu/mxnet/cub -DMXNET_USE_NVRTC=0" src/operator/tensor/matrix_op.cu
src/operator/tensor/././indexing_op-inl.cuh(84): error: identifier "__shfl_xor" is undefined
Was working okay. Stopped compiling when updated to the latest master branch

@ap-hynninen

try cp make/config.mk config.mk then edit the flags again.

Looks like the "-gencode arch..." arguments given in CUDA_ARCH are missing.

I think I may be experiencing a related problem? Part way through compiling, I get several variations of the the error:
...\mxnet\src\operator\tensor./indexing_op-inl.cuh(84): error : identifier "__shfl_xor" is undefined
and the build fails to generate the libmxnet.dll
However the gencode arch arguments are not missing (as shown below) and trying a few variations did not help, which was the only possibly related solution that I found on stack exchange.
CUDA_ARCH := -gencode arch=compute_30,code=sm_30
-gencode arch=compute_35,code=sm_35
-gencode arch=compute_50,code=sm_50
-gencode arch=compute_50,code=compute_50
Done Using:
latest master branch
VS 2013
Windows 10
CUDA 8
CuDNN 8
OpenBLAS
OpenCV

windows build uses cmake, not config.mk. You need to change cmake options

Yes, please pull latest for MShadow, which has a revised Cuda.cmake which omits flavors below 30.

Around the middle of the mshadow/cmake/Cuda.cmake, there's some code (shown below). You can also explicitly set the architectures to build by defining CUDA_ARCH_ALL for cmake.  On my machine, reducing the number of architectures dramatically shortens nvcc compile times.
if(CUDA_ARCH_ALL)
set(mshadow_known_gpu_archs "${CUDA_ARCH_ALL}")
else()
if(${CUDA_VERSION} GREATER 7.5)
set(mshadow_known_gpu_archs "30 35 50 52 60 61")
else()
set(mshadow_known_gpu_archs "30 35 50 52")
endif()
endif()

Ooooh, I think I understand! Just to clarify, the requirement for using GPU is now different from http://mxnet.io/get_started/setup.html#requirements-for-using-gpus and Compute Capability now must be greater than 3 instead?

@piiswrong  I thought the requirement was >= 3.0 all along. Are we supposed to support 2.0? 2.0 support is possible with some code changes.

Just curious, what sort of hardware still requires 2.0?
On Sun, Feb 5, 2017 at 8:01 AM Antti-Pekka Hynninen <
<EMAIL_ADDRESS>wrote:

@piiswrong https://github.com/piiswrong I thought the requirement was

= 3.0 all along. Are we supposed to support 2.0? 2.0 support is possible
with some code changes.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dmlc/mxnet/issues/4833#issuecomment-277528846, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AKts_e5bJ30iTSrTu1lSyA0sWwKC61Slks5rZfJqgaJpZM4LyGE_
.

Fermi boards, see this thread:
https://github.com/dmlc/mxnet/issues/4886#issuecomment-277516062

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/45316

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/48968

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/49209

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/56365

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/57582

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/81410

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/81446

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/81478

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/81550

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/81567

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/81666

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/104260

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/106143

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/106183


Initial Fidesops Merge

Closes #995
Dealing with two repos
There are two pieces here, the first is the initial merge, and the second is the ongoing merging of those remote repositories into this one to continue keeping compatibilty.
Commands used to execute the merge:

git remote add fidesops https://github.com/ethyca/fidesops.git
git fetch fidesops --tags
git merge --allow-unrelated-histories fidesops/main

Commands used to keep parity with remote repos:

git pull --all (you should see it pull from both origin as well as fidesops
git merge origin/main & git merge fidesops/main

Once the final merge is deemed complete, we'll need to run git remote remove fidesops

Code Changes

[x] do the git merge and clean up any conflicts
[x] copy/pasta the frontend_test workflow
[x] fix/consolidate docker builds
[x] merge the alembic migration histories
[x] consolidate nox commands
[x] consolidate pr_checks
[x] add the fides worker command to run a worker

Steps to Confirm

[x] all workflows pass except for the fidesops tests

Pre-Merge Checklist

[ ] All CI Pipelines Succeeded
Documentation Updated:

[ ] documentation complete, or draft/outline provided (tag docs-team to complete/review on this branch)
[ ] documentation issue created (tag docs-team to complete issue separately)

[ ] Issue Requirements are Met
[ ] Relevant Follow-Up Issues Created
[ ] Update CHANGELOG.md

Description Of Changes
The goal of this PR is two-fold:

Prove that the merger can be done in an elegant way
Merge the code (not necessarily the products) and get all tests passing

Once those two goals are completed, there will be a slew of follow-up tasks to properly unify the products, with generally one issue per update
Things to do after this issue:

Docs Merger
Frontend Merger
Server Merger (a single main.py)
Config Merger (combine the logic for loading in the config values)
Re-implement optional dependencies

working on getting the conflicts fixed (29), going well so far, will push up when finished

it looks like using docker-compose up --wait wasn't working with the compose version in the workflow runners so I converted all of them to docker compose up --wait in the hopes that the version would support it

got tests passing, now going to do the heavy lift of merging the servers....

ok with the current state of the tests, going to go ahead and merge and then pop open all of the follow-up issues


double-beep: Blacklist sunwin.vn

double-beep requests the blacklist of the website sunwin\.vn. See the MS search here and the Stack Exchange search here.

sunwin\.vn has been seen in 4 true positives, 0 false positives, and 0 NAAs.

Approved by Makyen in Charcoal HQ


make GraphScope runnable on WSL2

What do these changes do?
to make GraphScope runnable on WSL2

use kind to launch local cluster on WSL2.

Related issue number
related issue #9
Fixes

Nice job, merged 👍


Support Elasticsearch 5.x command-line flags in service scripts

Pull request acceptance prerequisites:

[x] Signed the CLA (if not already signed)
[x] Rebased/up-to-date with base branch
[ ] Tests pass
[x] Updated CHANGELOG.md with patch notes (if necessary)
[x] Updated CONTRIBUTORS (if attribution is requested)

This PR should resolve problems around unsupported flags (like -Des versus -E) when Elasticsearch 5.x is installed. This probably isn't the cleanest way of solving the problem, but it's probably the most reliable given we're relying on Puppet's native APIs to fetch the installed package version at file write time.

Related to #623

I tested your PR, and I run into the problem that log4j2.properties file is not being created under the instance name.
/etc/elasticsearch/log4j2.properties   (exists)
/etc/elasticsearch/es-01/log4j2.properties (doesn't exist)
Copying the file manually with the correct ownership gave me a running instance

Hi @carroarmato0, yep, that is the next hurdle to take care of. I've been PR-ing the pieces that 5.x needs individually, and I think the log4j2.properties file is the last part, so I'll probably write out a 5.x acceptance test along with the new logging file requirements to verify 5.x works as expected.
This should be last piece of the plumbing needed to get us to 5.x, though!


Extend detached container end-to-end tests with compatibility testing

Closed because of repo going public, new PR at #3574


[LOG] improve the logging for OpenAI maximum context length

I tried to run the chatgpt evaluator, but your OpenAPI requestor seems to go into infinite retry loops:

WARNING:root:OpenAIError: This model's maximum context length is 4097 tokens. However, your messages resulted in 4452 tokens. Please reduce the length of the messages..
WARNING:root:Hit request rate limit; retrying...
(repeats forever)

And similar loops happen on trying gpt4 if you do not have access to the model or use an invalid api key. These seem to be caused by assuming any other error is a rate-limiting message.
I tried to fix the token limit loop by:

if "Please reduce your prompt" in str(e) or "This model's maximum context length" in str(e):

but it can still crash due to this line, probably since the sum of the generated samples is too long to fit.
if kwargs["max_tokens"] == 0:
   raise e

Thanks for flagging this; we'll look into this ChatGPT issue. For GPT-4 or Claude, if you set the API key and have access to the models, the error wouldn't occur. We'll work on adding better logging for failure to set the key or restricted access in that case.

the logging is now improved! Thanks


added two options: excludeNames and excludeTypes

Hi,
First of all, thanks for this module. I'm treating some complex objects, which i need ignore some properties, either by its type or its name. I solved it by creating two options:
excludeNames: Object properties with one of the excluded names will be ignored from json result.
excludeTypes: Object properties with one of the excluded types will be ignored from json result.

This is too specific to your need. I'd be open to a filter option that accepts a function that it expected to return a boolean of whether to keep the element. This should solve your need and a lot more.

I totally agree! made suggested changes.

Can you add it to the docs too?

Shouldn't the check be at the top level so you're able to filter anything?

where the check is, the filter function expect val and el params, and at the top level, only val exists. I'm not sure if i got it.

Why are you using .call?

no special reason to use .call, it can just opts.filter(val, el).

val should probably be val[el]

el provides a way to check the property name, what is not possible passing val[el] param. For example:
my object:
{
   prop1: 1,
   prop2: 2,
   prop3: {
      prop4: 4
   }
}

my filter:
filter: function (val, el) {
	return el !== 'prop1';
}

passing val[el] instead of val and el, it wouldn't be possible to filter by 'prop1', but only second-level elements.


docs(limesurvey): fix mysql keys in example

Thanks! Looks good from my side.
LGTM.

Lint failed due to Bitnami Dependency. We can fix that later.


Create API

Detailed Description
Create a simple API that could take in site level details and make a forecast
Context

wrap around this function - from quartz_solar_forecast.forecast import run_forecast
we can think about deploy this later
if we have an api, then lots of tech stacks, like websites could get a forecast

Possible Implementation

create fastapi that wraps the function run_forecast

Hey @peterdudfield , the run_forecast function uses make_pv_data in data.py, which in turn optionally uses the functions from enphase.py-where some functions require CLI inputs given by the user. Should it be like calling an API with a cURL ?

Perhaps we could focus on doing this without data from enphase first. I think a FastApi  could be a great way to do this

Hey @peterdudfield , the run_forecast function uses make_pv_data in data.py, which in turn optionally uses the functions from enphase.py-where some functions require CLI inputs given by the user. Should it be like calling an API with a cURL-which only Linux users will be able to use ? Otherwise we'd have to make a .ps1 (powershell) file for windows-specific users to use the api.

What is enphase.py

Perhaps we could focus on doing this without data from enphase first. I think a FastApi  could be a great way to do this

I already created the API, though I'm curious should it exist within this repos folder or can be on its own folder and GitHub repo?

Hey @peterdudfield , the run_forecast function uses make_pv_data in data.py, which in turn optionally uses the functions from enphase.py-where some functions require CLI inputs given by the user. Should it be like calling an API with a cURL-which only Linux users will be able to use ? Otherwise we'd have to make a .ps1 (powershell) file for windows-specific users to use the api.

What is enphase.py

enphase is a solar inverter, we trying to get this working so people can connect to their own solar panels and give a solar forecast

Perhaps we could focus on doing this without data from enphase first. I think a FastApi  could be a great way to do this

I already created the API, though I'm curious should it exist within this repos folder or can be on its own folder and GitHub repo?

I think what #146 has done with the API is the way forward. If #146 gets merged, can you build of that API?

Hey @peterdudfield , which folder should the api file be in? any suggestions? or should i go ahead with making a new folder called "api"/"run_forecast_api.py"

Perhaps we could focus on doing this without data from enphase first. I think a FastApi  could be a great way to do this

I already created the API, though I'm curious should it exist within this repos folder or can be on its own folder and GitHub repo?

I think what #146 has done with the API is the way forward. If #146 gets merged, can you build of that API?

Noted, guess I'm too late, looking forward to working on other issues.

Perhaps we could focus on doing this without data from enphase first. I think a FastApi  could be a great way to do this

I already created the API, though I'm curious should it exist within this repos folder or can be on its own folder and GitHub repo?

I think what #146 has done with the API is the way forward. If #146 gets merged, can you build of that API?

No need @aryanbhosale is already working on it. I think I'll just put my efforts on issue https://github.com/openclimatefix/Open-Source-Quartz-Solar-Forecast/issues/93

Perhaps we could focus on doing this without data from enphase first. I think a FastApi  could be a great way to do this

I already created the API, though I'm curious should it exist within this repos folder or can be on its own folder and GitHub repo?

I think what #146 has done with the API is the way forward. If #146 gets merged, can you build of that API?

No need @aryanbhosale is already working on it. I think I'll just put my efforts on issue #93

I'm not working on it since #146 has already put in some work into it, you could try it out!

Perhaps we could focus on doing this without data from enphase first. I think a FastApi  could be a great way to do this

I already created the API, though I'm curious should it exist within this repos folder or can be on its own folder and GitHub repo?

I think what #146 has done with the API is the way forward. If #146 gets merged, can you build of that API?

No need @aryanbhosale is already working on it. I think I'll just put my efforts on issue #93

I'm not working on it since #146 has already put in some work into it, you could try it out!

Yeah, I will look into it, since she's already created the API all that's left is creating the schema and the Open API docs for interactivity.
Though I'm also looking at issue https://github.com/openclimatefix/Open-Source-Quartz-Solar-Forecast/issues/93 since it's given me the most headache as I use Ubuntu 24.04 which uses unfortunately uses Python 3.12.

I think we should close this now, right?

I think we should close this now, right?

Yes I think this has been resolved


Fall back to $HOME/.minder.yaml when reading configuration

Summary
This allows us to have a predictable configuration file when reading the
client minder configuration. The idea is that we could set up
$HOME/.minder.yaml and that would serve as an entrypoint.
In the future, we could modify this if we want to dynamically change
the current working project.
Change Type
Mark the type of change your PR introduces:

[ ] Bug fix (resolves an issue without affecting existing features)
[x] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[ ] Refactoring or test improvements (no bug fixes or new functionality)

Testing
Review Checklist:

[x] Reviewed my own code for quality and clarity.
[ ] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[x] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

coverage: 53.392% (+0.002%) from 53.39%
when pulling 7a4a9a5bd37a8f16a73333e2e4a20181df1b7706 on config-homedir
into 31e0ca35bc25ec734381824f5fab64dcaf028539 on main.

coverage: 53.397% (+0.007%) from 53.39%
when pulling f27d63cc909ba1e438d23481df15f59cbc56deea on config-homedir
into 31e0ca35bc25ec734381824f5fab64dcaf028539 on main.

coverage: 53.313% (+0.02%) from 53.296%
when pulling eefe6a7c972a2e7b30deb00d1205fa389531285f on config-homedir
into 67208c5bb92e5357df26f970f55530d104fa3e40 on main.


Support ask questions to PDF files

It would be great if it can be somehow integrated with the following plugin:

https://github.com/scambier/obsidian-text-extractor

This way, one can ask questions about PDF files and images.
Maybe the following plugin is also related to this request:

https://github.com/liammagee/obsidian-topic-linking

My main focus currently is trying to provide sources for the answer (which note has the answer). And also better UI/UX + error handling etc. Although it would definitely be good to have this in the future.
For now, you can export the text into a new note using the text extractor plugin and use that instead, although I get that isn't as convenient.

Just to add some information. The following plugin now supports handling PDF files:

https://github.com/yzh503/obsidian-aicommander-plugin


Not working with aframe-ui-modal-component

Thanks for making this component, it saved me lots of time to work with the mouse click event.
I am trying to use mouse-cursor component with aframe-ui-modal-component, however, the mouse click event doesn't work when the entities are wrapped inside the ui-modal, any hints about this problem?
(Other entities are working as intended with mouse-cursor when they are not wrapped inside the ui-modal.)

Hi @iKoala sorry for the late response.
I haven't supported ui-model-component yet.
will do when I have time...
Thank you!


Remove Miri equality workaround function

When testing with Miri, hash equality is checked using a workaround function that attempts to mimic constant-time functionality. This is done only because the constant_time_eq crate is not compatible with Miri.
The workaround function does not use any optimization barriers, and is likely insufficient for constant-time operations. This is fine, since for Miri testing no constant-time functionality is particularly required. However, it seems slightly risky to keep it around in case of unintentional misuse or confusion.
Fortunately @oconnor663 has an open PR that would remove the Miri limitation altogether. In the meantime, this PR removes the workaround function in favor of standard variable-time PartialEq functionality with a more direct and scary comment.
As noted by @oconnor663, this doesn't fully mitigate the risk of misuse, but I think it's a reasonable update until the Miri limitation goes away.

LGTM, though it looks like it needs a tweak to pass CI:
error[E0277]: can't compare `[u8; 32]` with `&[u8; 32]`
   --> src/lib.rs:339:23
    |
339 |         return self.0 == other;
    |                       ^^ no implementation for `[u8; 32] == &[u8; 32]`
    |
    = help: the trait `std::cmp::PartialEq<&[u8; 32]>` is not implemented for `[u8; 32]`
help: consider dereferencing here
    |
339 |         return self.0 == *other;
    |                          +

Good catch! I'm surprised that my IDE didn't complain about this. Will update.

Should be good to go now.

@oconnor663: This is now updated to account for the new constant_time_eq version that includes your fix.

Thanks!


what is the starbit used for?

First of all, thans for your great cron job.
I read dayMatches code in spec.go but did not figure out why  you check starBit before return result.
Why use || operation when both starBit of dom and dow is 0. I think we always need use && to get result.
Could youhelp to explain some details of this?
Thanks in advance :)
// dayMatches returns true if the schedule's day-of-week and day-of-month
// restrictions are satisfied by the given time.
func dayMatches(s *SpecSchedule, t time.Time) bool {
	var (
		domMatch bool = 1<<uint(t.Day())&s.Dom > 0
		dowMatch bool = 1<<uint(t.Weekday())&s.Dow > 0
	)
	if s.Dom&starBit > 0 || s.Dow&starBit > 0 {
		return domMatch && dowMatch
	}
	return domMatch || dowMatch
}

It seems starBit means ""
https://github.com/robfig/cron/issues/70#issuecomment-246892837

https://pubs.opengroup.org/onlinepubs/9699919799/
https://pubs.opengroup.org/onlinepubs/007904975/utilities/crontab.html


partners: Add chroma partner package

Description: Adds chroma to the partners package. Tests & code mirror those in the community package.
Dependencies: None
Twitter handle: @akiradev0x

failing CI and would love to get ^ addressed to merge!

working on it


__user_exception at boot, RTC variables not persisted

I'm developing for an "Adafruit ESP32 Feather v2", and tried the esp-template with its default settings.
When I run cargo run, the output looks like this:
Finished dev [unoptimized + debuginfo] target(s) in 0.06s
     Running `espflash flash --monitor target/xtensa-esp32-none-elf/debug/esp-test`
[2024-01-21T12:33:32Z INFO ] Serial port: '/dev/ttyACM0'
[2024-01-21T12:33:32Z INFO ] Connecting...
[2024-01-21T12:33:32Z INFO ] Using flash stub
Chip type:         esp32 (revision v3.0)
Crystal frequency: 40MHz
Flash size:        8MB
Features:          WiFi, BT, Dual Core, 240MHz, Embedded Flash, Embedded PSRAM, VRef calibration in efuse, Coding Scheme None
MAC address:       ac:0b:fb:6b:43:d4
App/part. size:    184,016/8,323,072 bytes, 2.21%
[00:00:01] [========================================]      17/17      0x1000     [00:00:00] [========================================]       1/1       0x8000     [00:00:06] [========================================]      71/71      0x10000    [2024-01-21T12:33:41Z INFO ] Flashing has completed!
Commands:
    CTRL+R    Reset chip
    CTRL+C    Exit

ets Jul 29 2019 12:21:46

rst:0x1 (POWERON_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)
configsip: 271414342, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:2
load:0x3fff0030,len:7104
0x3fff0030 - _ESP_HAL_DEVICE_PERIPHERALS
    at ??:??
load:0x40078000,len:15576
0x40078000 - _ESP_HAL_DEVICE_PERIPHERALS
    at ??:??
load:0x40080400,len:4
0x40080400 - _DoubleExceptionVector
    at ??:??
ho 8 tail 4 room 4
load:0x40080404,len:3876
0x40080404 - _DoubleExceptionVector
    at ??:??
entry 0x4008064c
0x4008064c - __user_exception
    at /home/blinry/.cargo/registry/src/index.crates.io-6f17d22bba15001f/esp-backtrace-0.10.0/src/lib.rs:113
I (32) boot: ESP-IDF v5.1-beta1-378-gea5e0ff298-dirt 2nd stage bootloader
I (32) boot: compile time Jun  7 2023 07:48:23
I (34) boot: Multicore bootloader
I (38) boot: chip revision: v3.0
I (42) boot.esp32: SPI Speed      : 40MHz
I (47) boot.esp32: SPI Mode       : DIO
I (51) boot.esp32: SPI Flash Size : 8MB
I (56) boot: Enabling RNG early entropy source...
I (61) boot: Partition Table:
I (65) boot: ## Label            Usage          Type ST Offset   Length
I (72) boot:  0 nvs              WiFi data        01 02 00009000 00006000
I (79) boot:  1 phy_init         RF data          01 01 0000f000 00001000
I (87) boot:  2 factory          factory app      00 00 00010000 007f0000
I (94) boot: End of partition table
I (99) esp_image: segment 0: paddr=00010020 vaddr=3f400020 size=06b64h ( 27492) map
I (117) esp_image: segment 1: paddr=00016b8c vaddr=3ffb0000 size=00004h (     4) load
I (117) esp_image: segment 2: paddr=00016b98 vaddr=40080000 size=01860h (  6240) load
I (127) esp_image: segment 3: paddr=00018400 vaddr=00000000 size=07c18h ( 31768)
I (144) esp_image: segment 4: paddr=00020020 vaddr=400d0020 size=1ce84h (118404) map
I (187) boot: Loaded app from partition at offset 0x10000
I (187) boot: Disabling RNG early entropy source...
Hello world!
Loop...

The program seems to work fine, but is the __user_exception an error, or something to be expected? (When I go into deep sleep, it happens for every wakeup.)
What could I do to help you debug this further? Also let me know if I should file this issue in another project. Thanks! :)

To add context, I'm trying to find out why my program doesn't keep variables in the RTC RAM during deep sleep. I'm declaring them like this:
#[link_section = ".rtc.data"]
static mut COUNTER: u32 = 0;

But changes to it are not visible after a deep sleep. I figured the above "harder-than-expected-reboot" could be related?

entry 0x4008064c
0x4008064c - __user_exception

esp-flash tried to resolve hex values which look like an address and in this case, it resolved the entry point logged by the bootloader as the symbol __user_exception. Not really something to worry about
As for the RTC data: We have a macro ram which should place data into RTC RAM, e.g.  https://github.com/esp-rs/esp-hal/blob/c4486800548aaacec26883a6f4491ad6860388ab/esp32-hal/examples/ram.rs#L23-L30

Thanks, those macros seem really helpful!
Persisting these variables through a deep sleep doesn't seem to work for me, though. I made a minimal project to reproduce this: https://github.com/blinry/esp32-deep-sleep-rtc-test
It contains all three variants:
#[ram(rtc_fast)]
static mut RTC_FAST_DATA: u32 = 42;

#[ram(rtc_fast, uninitialized)]
static mut RTC_FAST_NOINIT: u32 = 42;

#[ram(rtc_fast, zeroed)]
static mut RTC_FAST_BSS: u32 = 42;

When I run it (on an Adafruit ESP32 Feather v2), it shows these values on the first run:
Value of RTC_FAST_DATA: 42
Value of RTC_FAST_NOINIT:<PHONE_NUMBER>
Value of RTC_FAST_BSS: 0

Which is what I'd expect. The code then increments all three variables.
When the deep sleep starts, I get the the above output about __user_exception and _ESP_HAL_DEVICE_PERIPHERALS; the chip seems to reboot. Then, I get outputs like these:
Value of RTC_FAST_DATA:<PHONE_NUMBER>
Value of RTC_FAST_NOINIT:<PHONE_NUMBER>
Value of RTC_FAST_BSS: 0

So none of the values is persisted. Do you have an idea why?

sleep_deep will power down RTC memory during sleep.
You can do something like this
    let mut cfg = RtcSleepConfig::deep();
    cfg.set_rtc_fastmem_pd_en(false);
    cfg.set_rtc_slowmem_pd_en(false);

    let timer = TimerWakeupSource::new(Duration::from_secs(3));
    rtc.sleep(&cfg, &[&timer], &mut delay);

    loop {}

instead to prevent RTC memory from getting powered down

Whoa, that's the solution! Thank you! I updated my minimal project.
Maybe something like this should be an additional example in the examples directory? I wasn't able to figure this out on my own.

@blinry Would you mind PRing an example? Totally okay if not, but like you said it would be helpful for others :).

I think this is solved now. @blinry it would still be really nice to have an example in esp-hal itself, if you find some free time :).


formatting of number in coffeescript or ruby?

Hi,
I have a situation where I have created a new widget that displays large numbers(e.g. 1234.67) as 1.2K. This does the rounding in Ruby. My problem is however that I have also put, into my new widget coffeescript, the following code:
     @accessor 'isTooGood', ->
     @get('current') > @get('last')

    @accessor 'isTooLow', ->
    @get('current') < @get('last')

    @accessor 'isTheSame', ->
    @get('current') -  @get('last') == 0.0

where the background changes colour if the current value differs from the last value. However this background changing colour now works intermittently. When I remove the rounding code in Ruby, it works fine.  Is there a way to do the rounding of 1234.67 to 1.2K in Coffeescript ?

Coffeescript is Javascript compliant, so you can use the Math.round() function there, though if you want to keep a decimal place you may have to do something silly like Math.round(n*10)/10.


Remove Auth + Add Saved Gradients Page

Problem
Everytime a user visits the website, there's new content present, new gradients and everything, now the problem is suppose a user likes a gradient and wants to save it for future, there is no feature yet that can save the user's chosen gradients that he/she might like. Now if you have seen colorhunt, they have this feature where they can save gradients directly on the local storage, instead of requiring to log in.
Solution
The best way to solve this issue is to simply save all the user saved gradients on their browser's local storage. I thought about it and came to conclusion that since authorization of the user is not beneficial for both the user and the site, why not just save all the user data on the browser local storage itself. It just solves the burden for the user to log in when they visit the website. Also the face that the user has to authorize just to save some gradients, kinda sounds absurd given the fact that authorizing is not as fast and simple as just clicking on the save button and finding the all the gradients on the saved gradients page everytime they visit.
Alternatives
Now of course saving the gradients in the local storage of the browser has it's own pros and cons, yes saving gradients would be faster but with authorization the user can access their gradients from any device they want to. Although for a website that is developed just to give the user some inspirations for gradients and also offer the CSS code won't be done on multiple devices I believe. Most of the time, if the user is visiting the website to save gradients and access them later then he/she would perhaps want to access them from the same device considering they code and design on that device. But adding auth wouldn't be a good UX perhaps since the user requires to login for something as simple as just saving the gradients, it just makes the session longer and in case the user does not have a google or a facebook account, then for them logging in may not be possible at all. Now about the upvote feature, I am still thinking about it, haven't came to a conclusion yet, but that's a problem for later and I'll create a separate issue to track that once we've tackled this issue.
Concept

#15 Tracked.


CherryPick to 4.15: Ensure EndDialog action within a ForEachElement will end the dialog

Ensure EndDialog action within a ForEachElement will end the dialog

EndDialog from a nested ForEach

Add clarifying comments to ForEachElement

Fixes #6161

Pull Request Test Coverage Report for Build 289713

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.02%) to 78.679%

Files with Coverage Reduction
New Missed Lines
%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Actions/ActionScope.cs
2
96.43%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Actions/ForEachElement.cs
2
87.18%

Totals

Change from base Build 289319:
0.02%

Covered Lines:
24865

Relevant Lines:
31603

💛  - Coveralls


Integrating libemotly in mainline.

This change brings the first integration of libemotly with mainline. As
a way to keep things clear, these are the changes:

libemotly is added (EmotlyService.swift)
Two dependencies have been added to the tree: Alamofire and SwiftyJSON
Carthage is being used to manage the dependencies
CI (Travis) has been updated accordingly

In order to keep up, contributors have to:

Install Homebrew
Install Carthage from Homebrew
Bootstrap the build from the root (where Cartfile is), this means
running the command carthage --bootstrap
Open the project normally.

LGTM. Further changes in the architecture such as models and constants in different files can be done in a second time.


Finish multi-account support

This issue can be closed once we can reliably use the SDK to run a wallet containing multiple independent UFVKs (that may not be from the same seed, or for which we might not even know the seed).

Removing https://github.com/Electric-Coin-Company/zashi-android/pull/410 as a prerequisite to this for the upcoming release, as discussed internally.

Closing as done as part of #1661


fix(checkbox, radio): not using theme border color

Fixes the checkbox and radio button not using the border color from the theme. The cause of the issue was that both components used the shorthand border, which ended up overriding the theme's border-color.

LGTM

LGTM


Update rpm distributions paragraph

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

Hello @daftaupe,
thank you for submitting this addition :+1:
Cheers,
Digitalcraftsman


Admin Access to dibbs-aws repository

What type of help do you need?

[ ] Question
[ ] New Repo
[ ] Delete Repo
[ ] User Membership (please make sure new members are familiar with the CDC open practices and set up their profile with name and org info to help people collaborate with them)
[ X] Other

Please describe how you'd like us to help.
Engineer will need admin access to the dibbs-aws repository in order to create Github secrets and variables for development work.

@marycrawford Hi Mary, Rin (@rin-skylight) or Emma Stephenson (@emmastephenson) are both admins to dibbs-aws  and can grant whomever they need access.
Of course, I can grant access too - but it's probably less back and forth if they're part of your team.

Closing due to inactivity for a month.


Add basic Swift concurrent version

Same code inside as in the normal version, but parallelized with Dispatch and uses TreeDictionary from https://github.com/apple/swift-collections.
When I have a free minute, I'll look into the actual algorithm.

Thanks, I will have to re-enable bounds checking as removing it is against the rules. Good news is that it doesn't affect the score on my machine


Error in Rebuild.sh

[2018.07.26-03.22.07:109][  0]LogCompile: Error: GetTransform: Override of UFUNCTION in parent class (Actor) cannot have a UFUNCTION() declaration above it; it will use the same parameters as the original declaration.
Linux Version:18.04
UE4 Version: 4.20.
How to fix this error, please?

Hi @zec37,
We don't support UE4.20 yet (see #625), the master branch is still using 4.18.


Running cells requires loopback adapter for localhost

Environment data

VS Code version: 1.29.1
Extension version (available under the Extensions sidebar): 2018.11.0
OS and version: windows 10
Python version (& distribution if applicable, e.g. Anaconda): miniconda python 3.7
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): base conda environment
Relevant/affected Python packages and their versions: jupyter notebooks

Expected behaviour
jupyter notebook is started
Actual behaviour
jupyter notebook didn't start and "Running cells requires Jupyter notebooks to be installed." notification appear even if jupyter notebook has been installed

Steps to reproduce:

write some code

click "run cell"

this notification appear

Logs
Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

Output from Console under the Developer Tools panel (toggle Developer Tools on under Help)

i am able to run jupyter notebook from anaconda prompt

Are you sure the python you have selected on the bottom left of vscode has jupyter installed? Your console log makes it look like it doesn't.
What happens if you start the 'Python: Start REPL' command and then type
import notebook
notebook.version_info

You should end up with something like this if Jupyter is installed:
Python 3.6.4 |Anaconda, Inc.| (default, Jan 16 2018, 10:22:32) [MSC v.1900 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import notebook
>>> notebook.version_info
(5, 7, 2)
>>>

You'll end up with something like this if it isn't:
Python 2.7.14 (v2.7.14:84471935ed, Sep 16 2017, 20:25:58) [MSC v.1500 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import notebook
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ImportError: No module named notebook
>>>

This might also be a conda activate problem which might have been fixed in our latest release (released today)

I have similar issue, but with pipenv. Looks like jupyter cannot be activated from venv
VS Code version: 2018.12.1

Are you sure the python you have selected on the bottom left of vscode has jupyter installed? Your console log makes it look like it doesn't.
What happens if you start the 'Python: Start REPL' command and then type
import notebook
notebook.version_info

You should end up with something like this if Jupyter is installed:
Python 3.6.4 |Anaconda, Inc.| (default, Jan 16 2018, 10:22:32) [MSC v.1900 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import notebook
>>> notebook.version_info
(5, 7, 2)
>>>

You'll end up with something like this if it isn't:
Python 2.7.14 (v2.7.14:84471935ed, Sep 16 2017, 20:25:58) [MSC v.1500 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import notebook
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ImportError: No module named notebook
>>>

yes, i am sure the python i have selected on the bottom left of vscode has jupyter installed.

This might also be a conda activate problem which might have been fixed in our latest release (released today)

i have just installed the latest release and then i had different issue.

here is the output of developer tools

@MisLink  you're problem is likely the virtual environment. The workaround for now is run vscode from a terminal window that has '.venv/bin/python' on the path. We still have problems activating non conda virtual environments.
@tempegarit you're problem has be stumped. The insiders build should output more information on your error. Can you try that one? Thanks.

@rchiodo sure, but how can i used it? it has .vsix extension. I don't know how to install it.

Oh sorry, should have included the vscode instructions:
https://code.visualstudio.com/docs/editor/extension-gallery#_install-from-a-vsix

Is Miniconda3  (gamedev) your currently selected python? It looks like our enumeration of the kernelspec is failing. Maybe your installation doesn't include that.

I should have been more explicit.
I think this code here (ours) is failing:
https://github.com/Microsoft/vscode-python/blob/2e5e9b5b9eb4363683072ec59cbed7e16e2b1d92/src/client/datascience/jupyter/jupyterExecution.ts#L679
Although the isKernelSpecSupported should be checking this case.

Your original error was 'Failed to connect', but this console output shows 'Jupyter notebook failed to launch', so actually I think this is an internal error in the jupyter code (which makes the callstack make more sense)
That would actually be this code here:
https://github.com/Microsoft/vscode-python/blob/2e5e9b5b9eb4363683072ec59cbed7e16e2b1d92/src/client/datascience/jupyter/jupyterSessionManager.ts#L28
I'll try wrapping both in a catch and sending that out. Should be an hour or so.

@tempegarit in about half an hour a new insiders build should be available with a potential fix. If you can try it out, that would be great.
Thanks.

@tempegarit in about half an hour a new insiders build should be available with a potential fix. If you can try it out, that would be great.
Thanks.

Where should i download it from? is it here?

Oh sorry, should have included the vscode instructions:
https://code.visualstudio.com/docs/editor/extension-gallery#_install-from-a-vsix

Hello,
I'm experiencing the same issue. Not quite sure where should I ask it since old answers in google reference to some Jupyter plugin, and now VS Code marketplace tells me Jupyter is obsolete and migrated into this plugin... So, how do I run my Python code in interactive way, like with "-i" option for original interpreter?

My VS Code is Version: 1.35.0 (system setup)
Commit: 553cfb2c2205db5f15f3ee8395bbd5cf066d357d
Date: 2019-06-04T01:17:12.481Z
Electron: 3.1.8
Chrome: 66.0.3359.181
Node.js: 10.2.0
V8: 6.6.346.32
OS: Windows_NT x64 6.1.7601
My Python plugin is
Name: Python
Id: ms-python.python
Description: Linting, Debugging (multi-threaded, remote), Intellisense, code formatting, refactoring, unit tests, snippets, and more.
Version: 2019.5.18875
Publisher: Microsoft
VS Marketplace Link: https://marketplace.visualstudio.com/items?itemName=ms-python.python

Does the python you're using have jupyter installed? It's required for the interactive window to work.
You can verify this by starting a terminal window and typing 'python -m jupyter notebook --version'
If that fails, you need to install jupyter. There's a number of ways to get jupyter. Anaconda is probably the easiest.
If that doesn't fail, then can you go to Help | Toggle Developer Tools, click on the Console tab, Right click and save the file, then upload it here? Something else might be going wrong.

Uh, no, I didn't have one. Sorry, I'm not experienced much with Python frameworks and first time hear about Jupyter. I thought it was some sort of VSC extension itself. Now everything is fine, thank you! I believe, you could add note somewhere for beginners like me so they run "pip install jupyter" before using your extension.

Thanks for replying. We should probably just ship jupyter with the extension.

Closing this issue as it appears to be rare (no loopback adapter installed) and there's a workaround.


Improving blueslip javascript debugging experience in development

Right now, blueslip (static/js/blueslip.js, our frontend error reporting library) kinda expects that you'll have the javascript console open to see issues.  We could do a lot better:

Displaying JS tracebacks on the server console when they happen via the report_error codepath.
Checking for blueslip errors in our Node/Casper tests more explicitly so the tests fail if they happen (we might do this already; I'm not sure).
It's not clear to me that blueslip.fatal does the right thing in debug mode (in terms of still nicely displaying the traceback).

We've done a better version of the first part of this issue by displaying them in the main message view area.  The second I opened as https://github.com/zulip/zulip/issues/8675, and the third I verified is doing the right thing.  So we can close this.


Fix Windows & MacOS GH CI testing

Description
Since these VMs don't need to reboot, run tests directly on the runner instead of using a nested VM.
This should fix the testing failures with the Windows and MacOS testing from #143.
Signed-off-by: Robert Detjens<EMAIL_ADDRESS>Check List

[ ] All tests pass. See TESTING.md for details.
[x] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable.

          1 Error

:no_entry_sign:
Please include a CHANGELOG entry.

  Generated by :no_entry_sign: Danger

Looks like the Windows CI test is failing due to https://github.blog/changelog/2020-10-01-github-actions-deprecating-set-env-and-add-path-commands.
Fix here: actionshub/chef-install#15

Looks like the Windows CI test is failing due to https://github.blog/changelog/2020-10-01-github-actions-deprecating-set-env-and-add-path-commands before even getting to the integration converge.
Fix here: actionshub/chef-install#15

Would you mind temporarily using your branch here so we can see the fix in action?

Actions fix merged.

I'm working on resolving this in #146


Hackage release

Would you be averse to releasing 'parsix' to Hackage? I think it would be pretty useful.

I intend to do so eventually, after I've tried out some new things for Sixten's LSP integration and when I feel the API has stabilised a bit. If you still want to use it in the meantime, e.g. Stack has support for git dependencies.

Hej Olle!  Right now we are using it with nix and while it works fine it would be nice to have a hackage release to point to :)
Cheers!

Hej!
Since there seems to be so much serious business depending on this library, I've now made a release: https://hackage.haskell.org/package/parsix
PS I hope it's not related to chains of blocks!


Coupling with Nernst-Planck-Poisson

As discussed, I try the coupling with Nernst-Planck-Poisson.
First thing to try is a pore with inlet and outlet

                wall
     +-----------------------+
in   |                       | out
     |                       |
     +-----------------------+
                wall

So I have some questions:

How to set "do nothing boundary conditions"  at inlet and outlet ?
In the C++ pdelib, we had Femsolver.AssignElectricFieldAndCharges(potential, charge_density). Is is possible to have this here as well ? Or is there a way to specify an element-wise body force (which I could calculate if necessary for the time being)?

For the "do nothing boundary conditions" you have to do nothing: if no operators for boundary conditions are specified all degrees of freedom on the boundary are kept free and if the Neumann data is zero, nothing needs to be done.
In case the grids match in the sense that the FVM solution gives values for the nodes, I would suggest to first construct an FEVector that can carry everything: velocity, pressure, electrostatic potential, concentrations. For this we need finite element spaces:
FES_u = FESpace{H1BR{2}}(grid)
FES_p = FESpace{L2P0{1}}(grid)
FES_V = FESpace{H1P1{1}}(grid)
FES_c = FESpace{H1P1{nspecies}}(grid)
sol = FEVector([FES_u, FES_p, FES_V, FES_c])
Then we can write a kernel for the Lorentz force, probably something like
lorentz_kernel(result, args, qpinfo)
params = qpinfo.params
q = sum(params .* view(args, 3:nspecies+2))
result .= q * args[1]
result .= q * args[2]
end
The operator would look like
LinearForm(lorentz_kernel, [id(1)], [grad(3), id(4)]; params = charge_numbers) #14
Therein, the numbers correspond to the number of the FESpaces above, so grad(3) gives the gradient of the electrostatic potential and id(4) the concentrations of the species, which enter the kernel in that order via the argument args above.
Via the params argument constant coefficients can be funneled into the kernel, e.g. the charge numbers of the species in this case.
This LinearForm is then added to the ProblemDescription of a classical Stokes problem for the unknowns u and p.

ah, and in each iteration the coefficients for V and c can be overwritten by
sol[V] .= new_coefficients_for_V
sol[c] .= new_coefficients_for_c

And the solve call for the Stokes problem (assumed to have the ProblemDescription PD) looks a bit different:
sol = solve(PD, [FES_u, FES_p]; init = sol)
such that only the u and p blocks are solved (but the others can be used for assembly). I hope this works, we can fiddle out the details on Monday, if you like.


Enable Travis-CI.org to catch compiler issues (C89, future gcc warnings etc)

Hello again,
thanks for all the recent patches and improvements.
RE: c89 support  https://github.com/camgunz/cmp/issues/36
How can we prevent this (and other compiler gotchas) in the future?  I don't know your workflow but I can guess you might be a busy guy.
I propose setting up https://travis-ci.org  for this project that that will compile camgunz/cmp in a few ways (-std=c89 -Wal -Wextra -Werr) and maybe with Clang and it's static analyzer and beep if warnings occur.
I can set this up as a fork+PR  but in the end you'll have to be comfortable with travis-ci.org having access to your repo.
Regards,
n

I think Travis-CI only has a read access to the repository, so I don't know why it would be uncomfortable?
As a global remark, I second the suggestion of setting up Travis for this project, it will avoid future breakages.

travis-ci.org use to also have access to your private repos due to github's course-grain access control.  Not sure if that has been fixed or not.

According to https://docs.travis-ci.com/user/github-oauth-scopes/ this is only the case for private repositories nowadays.

I'll goof around with it today

I added some code to do this in #40. Works on my travis account.

Fab, you're the best.  I'm playing around w/ skipping today, we'll see how this works then.

@antoinealb does it matter if it's language: cpp vs. language: c in .travis.yml?

Ehh I'll goof around with it a lot, don't worry about it.  I guess it's time CMP grew some real infrastructure.

Comparing the two (https://docs.travis-ci.com/user/languages/cpp/ and https://docs.travis-ci.com/user/languages/c/) it seems  that the C++ builder is a strict superset of the C builder.

wow you guys are fast!  I'll take a look once it stabilize else too many cooks in the kitchen ... once you get it working it's quite nice, but getting to that point can be a bit tricky since you can't really replicate locally.

oh ha, the "basic travis.yml" is really that.  I'll look later tonight unless you beat me to it.

Heh yeah.  I want the test suite and coverage working through Travis; that's the goal anyway.  I'm working on skipping now but I'll probably get through all this today.

i can likely cut-n-paste stuff I did with client9/libinjection which compiled c code a few different ways (and as C++), and ran some static analysis tests.  For coverage I have used https://coveralls.io in the past.

I started working on this and got it mostly working (in develop).  It ignores cmp.c, which is pretty counterproductive, but coverage for that is only around 60% (measured with lcov) anyway so not stellar.  I think I'll spend some time rounding out the tests.

This is done and merged into master.  Closing.


chore: added v0.19.x release notes

This change is

Yes, you're right. Of course. Updated. PTAL


Fix typo in AABB docs

A single letter

Thanks! And congrats for your first merged Godot contribution :tada:


logic problem about device having short connection time

I mention I have a few knowledge about node.js
recently, I've been tested connecting BLE based devices with node-red, using node-red-contrib-generic-ble module.
but, there is a problem.
for example, I make up commuincation between device and node-red.
Only, the device has short connection time to save battery(ex. about a 50ms).
In this test environment, Generic BLE for Write module requests UUID write function, the function won't run.
I guess Generic BLE for Write module scan and search or inquire the device.
so, it needs more time for search or inquire.
While that process running, the device will be disonnected for saving energy, then, write function should have originally done doesn't run.
this is my only guess.
I use wireshark for Nordic BLE about this assumption and upload wireshark log(Log No.236~No.234
thank you for reading.
wireshark_log.zip

In this test environment, Generic BLE for Write module requests UUID write function, the function won't run.
I guess Generic BLE for Write module scan and search or inquire the device.

Periodical peripheral connection/disconnection caused the delay by Read/Write operations.
As of 3.0.0, I stripped the above behavior so Read/Write operations will be performed right after connecting a peripheral.
Well, 50ms seems to be too short to perform operations (Read/Write).
But you can try the new version of the node.


ERROR:bootloader reports invalid operation

I am trying to upload the version of the PX4 flight controller on Intel-aero ready to fly drone.
I have followed those guides:
https://dev.px4.io/en/setup/building_px4.html
https://docs.px4.io/en/flight_controller/intel_aero.html
but anytime I try to run the command: make aerofc-v1_default upload, the sistem erases the old flight controller but while installing the new one (the PROGRAMING phase) it stops at 0% saying:
ERROR: bootloader reports INVALID OPERATION

@zehortigoza This something you could help with?

I will try to look at this but in the mean time you can copy the firmware to Aero RTF board by hand(I don't remember where the firmware is build now) and then use Aero script to flash it: aerofc-update.sh


feat: add support for Adjust Partner Parameters

Overview
We need to enable Partner Parameters in Adjust https://help.adjust.com/en/article/record-events-ios-sdk#add-partner-parameters
Reasons to add this:

FB CAPI Event deduplication
Start sending product catalog info to networks

As a convention, all those properties reported to segment with prefix: adjust_pp_ will be sent as partner parameter.
ie:
{
  "properties": {
    "adjust_pp_event_id": 12345,
  }
}

will result in a partner parameter called event_id sent to the Adjust.

As a convention, all those properties reported to segment with prefix: adjust_pp_ will be sent as partner parameter

will this defined per event?
to avoid sending these properties to every destination, did you consider using event context? https://segment.com/docs/connections/spec/common/#context

As a convention, all those properties reported to segment with prefix: adjust_pp_ will be sent as partner parameter

will this defined per event?
to avoid sending these properties to every destination, did you consider using event context? https://segment.com/docs/connections/spec/common/#context

It seems that context has very specific fields that should be used only for its intended purpose.

Not sure if we should add this to user traits. It's actually a prop that we do need to send as partner parameter to Adjust.

we use context for Braze integration specific, see https://github.com/taxfix/webapp/pull/1434/files
we don't use existing property but a custom integrations map

What will be the strategy to add/identify required partner parameters per event?

What will be the strategy to add/identify required partner parameters per event?

Yes! It's a per event stuff. event_id might mean different things depending the event itself.

we use context for Braze integration specific, see https://github.com/taxfix/webapp/pull/1434/files we don't use existing property but a custom integrations map

Cool! Thanks! I will check this out then!

What will be the strategy to add/identify required partner parameters per event?

Yes! It's a per event stuff. event_id might mean different things depending the event itself.

who will maintain this mapping? will it be product teams?

What will be the strategy to add/identify required partner parameters per event?

Yes! It's a per event stuff. event_id might mean different things depending the event itself.

who will maintain this mapping? will it be product teams?

There's no need to maintain. If we add the prefix to the attribute, whoever that needs to send a custom parameter to Adjust can just add the prefix and configure the mapping in adjust. The native layer will know then that those parameters need to be treated as partner parameter and will be therefore added as such when constructing the Adjust event at the native layer in the Segment <> Adjust custom Taxfix implementation. (this is the actual beauty of having this defined using a prefix instead of a fixed mapping) Need to assess though if this is too problematic or not for other destinations as the name of the field/property will contain the "adjust" name on it. But anyway, it's a field that we do need to send and needs to be distinguishable from the rest either via a fixed mapping or a prefix convention.

@pierricklete I've changed the impl so that it makes use of the integrations object instead of the properties with prefix as suggested. I still need to change the tracking signature.

I still need to change the tracking signature in the tracking lib.

I think we should instead leverage on tracking library approach/capability to decouple the destination specific from the original call through destination specific middleware (here an example with segment/braze destination specific)

@pierricklete with regards to the changes in this PR do you think they are OK?

@pierricklete with regards to the changes in this PR do you think they are OK?

yes looks good but before merging this will be best to create a beta app using this branch, you can create rc version of native-deps changing the podfile: https://github.com/taxfix/mobile-app-native-deps/blob/7dc4b4995263bd5179d39125fb423d67eb011f3f/ios/Podfile#L37
once everything is looking fine we can merge in custom branch

@pierricklete , I've done the E2E test:

Mobile app: https://github.com/taxfix/mobile-app/pull/11091/commits/f9c9e4477e667054087962a4cc11587fbe13d642#diff-34cd54f8ed8699ca4a8069abbdf769eb2236001328f05e030bc7632f02b02165R302
Native deps: https://github.com/taxfix/mobile-app-native-deps/pull/356/files#diff-281ded35b124f5160d0a57e47e521ee6f49f233933e091a0d33f3fd42a74abc8R40
IOS Plugin changes (this branch)

E2E Gitlab: https://gitlab.taxfix.tech/developers/mobile-app/-/pipelines/519958


[Suggestion] SC3 wallet password instead of encryption key for each account

Based on my understanding, SC3 (extension) is more like Keychain.
I think for this kind of browser wallet, having one wallet password is much more intuitive.
It supports multiple accounts and users want to use multiple account conveniently.
Having a separate password for each account isn't convenient.
Also, probably for that reason, users are asked to set that password after importing account/key, which is very counter-intuitive.

In addition, don't user the term "Encryption key" in the above, since it sounds like a key of PKI. So users may think it should be some private key (or steem master password for that account).

In addition, don't user the term "Encryption key" in the above, since it sounds like a key of PKI
I've changed the term "Encryption key" to "Wallet password".

The tradeoff of having a password for all account is that only one person can use the app, this is why we decided to have a password (or encryption key) for each account. Ideally it should be possible to have multiple account per password and multiple password too.


iPad full screen scanning issue

Hi,
So far i have the entire install working on IOS.
I had to edit the .xib to make the camera window full screen on the iPad. This works.
The problem i have now is that the camera is working full screen in portrait mode but the iPad is in landscape mode.
How can i make the camera work in landscape?
Thanks

Hi there,
Thanks for the bug report. Can you please clarify one thing for me. When you say "I had to edit the .xib..." do you mean that you updated it to the latest version on this repository? Or do you mean that you have added custom code?
We'll look at this as soon as possible,
Best,
Tom

Hi,
Thanks for the quick reply!
I took the latest from github.
I then looked at the other incident report with a simular problem.
Then in xcode, i edited: /resources/ScannerViewController.xib
Set size: iPad full screen
Orientation: Landscape
This works on the iPad as it views the camera full screen but the camera is still in portrait mode so the picture is side ways.
Ive attached a picture of the problem.
Cheers,
Mark

Excellent. I can reproduce the bug on my end (we just use an iPod Touch) so we should have a fix soon.
Thanks for reporting this bug,
Tom

I just pushed a fix for this bug. Either start a new project or replace the ScannerViewController.m file with the new version and let me know if that works for you.

Amazing! Worked like a charm.
Thank you


List Site Directory

There is no way i can list files inside my site. I needed to my site directory and delete a file or copy file. It will be nice to be able to list the files directory for each site

Hi @iamprincesly. Please send a feature request to forge support.


Named export 'useDropzone' not found

What happened?
I am receiving
SyntaxError: Named export 'useDropzone' not found. The requested module 'react-dropzone' is a CommonJS module, which may not support all module.exports as named exports.
--
CommonJS modules can always be imported via the default export, for example using:

import pkg from 'react-dropzone';
const { useDropzone } = pkg;

at ModuleJob._instantiate (node:internal/modules/esm/module_job:124:21)
at async ModuleJob.run (node:internal/modules/esm/module_job:190:5)

> Build error occurred
Error: Failed to collect page data for /settings
at /vercel/path0/node_modules/next/dist/build/utils.js:1156:15 {
type: 'Error'
}

a similar issue to https://github.com/pingdotgg/uploadthing/issues/42. I have made sure to include:
experimental: {
    esmExternals: false, // THIS IS THE FLAG THAT MATTERS
},

in my next.config.cjs which looks as follows:
import { withContentlayer } from "next-contentlayer"

await import("./env.mjs")

/** @type {import("next").NextConfig} */
const config = {
  reactStrictMode: true,
  images: {
    domains: ["d1kfeng928x8ln.cloudfront.net"],
  },
  i18n: {
    locales: ["en"],
    defaultLocale: "en",
  },
  experimental: {
    esmExternals: false,
  },
}

export default withContentlayer(config)

I have deleted and reinstalled/generated my yarn.lock, node_modules.
I am using<EMAIL_ADDRESS>&<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Following https://github.com/pingdotgg/uploadthing/issues/42#issuecomment-1544397914 does resolve my issue, but obviously isn't sustainable.
Is there any requirement for the next.config to not be a commonjs file? I am using Next.js Pages directory
The concept of this library is great, thanks in advance for any help here!
Version
<EMAIL_ADDRESS><EMAIL_ADDRESS>Code of Conduct

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

Reproduction repo plzzz

For anyone encountering this issue, after building, make sure you see
- warn You have enabled experimental feature (esmExternals) in next.config.js.

Otherwise the
  experimental: {
    esmExternals: false,
  },

flag isn't being picked up. I found the issue to be that my next config was next.config.cjs and for some reason, it was getting any of the flags passed there, so after converting it to off of commonjs.
const withContentlayer = require("next-contentlayer").withContentlayer

require("./env.js")

/** @type {import("next").NextConfig} */
const config = {
  reactStrictMode: true,
  images: {
    domains: ["d1kfeng928x8ln.cloudfront.net"],
  },
  i18n: {
    locales: ["en"],
    defaultLocale: "en",
  },
  experimental: {
    esmExternals: false,
  },
}

module.exports = withContentlayer(config)


Not able to connect to local sqlserver from the command utility

What version of Flyway are you using?
4.2.0
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin, SBT plugin, ANT tasks)
command-line
What database are you using (type & version)?
Sqlserver 2014
What operating system are you using?
Windows
What did you do?
(Please include the content causing the issue, any relevant configuration settings, and the command you ran)
I am trying to using flyway. I am not able to connect to the database
In the flyway config file
flyway.url= jdbc:jtds:sqlserver://localhost:1434/databaseName=xxxxxxxxx;integratedSecurity=true;
In the command prompt
I typed as flyway baseline and also tried with flyway info
Prompt for Database user: Pressed enter key ( Because In the url I gave "integratedSecurity=true", so flyway will connect with SQL server using  windows authentication)
Prompt for Database password : Pressed enter key
What did you expect to see?
Successful connection to the SQL server and baseline the database
What did you see instead?
SQL State : 28000
Error Code : 18456
Message : Login failed for user Mydomainname\MyLoginName

Same problem here!! Any suggestions?

Did you provide the native sql server libraries for integrated security to the cli?

Download and unpack this:
https://www.microsoft.com/en-us/download/details.aspx?id=54629
Create new folder in flyway directory e.g. c:\flyway-4.x.x\native
Copy sqljdbc_auth.dll appropriate for your JVM build into this folder
Copy the sqljdbc4.jar into c:\flyway-4.x.x\drivers folder (if necessary)
Change the flyway URL removing jdts:
flyway.url=jdbc:sqlserver://localhost:1433;database=xxxxxxxxx;integratedSecurity=true;
Change the flyway.cmd to set up the library path to the directory where you copied the DLL:
%JAVA_CMD%  -Djava.library.path="%INSTALLDIR%\native" ...
See here for more info:
https://docs.microsoft.com/en-us/sql/connect/jdbc/building-the-connection-url

@axelfontaine what is changing for this in 6.0.0?  It works as-is with that download.

I've followed the documentation through and it appears to be sufficient to make the commandline work. Can we close this now, or are there other issues?

@juliahayward Ideally Flyway should perform the necessary setup so it works out of the box.

It looks like the work-around provided by @junkmanuk may work. Can someone elaborate on how to change the flyway.cmd as he describes above ^^^?
I've tried setting integrated security to false and used a user name password in the plug-in config and I still get a login failure. I'm curious if it's the result of the same issue.

Can someone elaborate on how to change the flyway.cmd as he describes above ^^^?

Just edit it with a text editor, it's a batch file

Note that a workaround is documented https://flywaydb.org/documentation/database/sqlserver#windows-authentication

As on 6.2.4 flyway no longer prompts for username/password when adding integratedSecurity to the url. See https://github.com/flyway/flyway/issues/2679
Add the dll needed to support it will be done as part of a separate issue. See https://github.com/flyway/flyway/issues/2694


Function already exists error after initial lambda deploy

Hi,
My deployment completes successfully the first time, but errors out on subsequent deployments with this error:
botocore.exceptions.ClientError: An error occurred (ResourceConflictException) when calling the CreateFunction operation: Function already exist: es_get_channel_http_microservice

Haven't run into this issue myself but looking at the code it only creates a function if you give a new name in the config, maybe give another try after pulling in the latest changes and reinstall botocore/boto3; if still doesn't work, you can try to change the script default to updating the function at aws_lambda/aws_lambda.py#L86 and see if you are still getting the same exception.

+1 Seeing this same behavior.

Believe this is addressed with the latest merged changes.  The client.list_functions() call to AWS returns paged results.  We now query for the "complete list" before checking to see if the function exists or not.  You were likely to start running into this if you had over 50 Lambda functions defined.

Thanks sir! will close


[DONT MERGE]Pytest update debugging

Please add a meaningful description for your change here

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

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

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

Run XVR_JavaUsingPython_Dataflow PostCommit

Run XVR_JavaUsingPython_Dataflow PostCommit

Run XVR_JavaUsingPython_Dataflow PostCommit

Run XVR_JavaUsingPython_Dataflow PostCommit

Run XVR_JavaUsingPython_Dataflow PostCommit

Run XVR_JavaUsingPython_Dataflow PostCommit

Run XVR_JavaUsingPython_Dataflow PostCommit

Python 3.7 is able to pickle the function but Python 3.9 is not able to pickle it.


Simulations seem too noisy. Fixing this requires changing model initial conditions and parameters.

We could obviously smooth the simulation results ourselves, but I think it would be more appropriate to figure this out on the model/simulation side

@purvislab, is this still an issue?


Ingame Icon

Is it possible to display a blood drop icon instead of the heart?
Because I use the plugin to monitor my blood sugar in the game.
👍

hmmm I'm guessing you are using a compatible data source that sends blood sugar as the heart rate field.
I could implement a feature for changing the icon but it will take a while because currently, the entire counter is from a unity asset bundle

Yes, I'm using a compatible data source to send blood glucose as a heart rate field.
A function to change the icon would be perfect. Many thanks for the answer.

Sorry for not having anything for this long. There is a rewrite happening for easier future updates and will definitely add the support after the rewrite

That sounds very good. Thanks for the feedback.

[ ] Learn how to load image files at game runtime 😁
[ ] Make the in-game UI for icon selection

Implemented in #70
Here is a build, tested with 1.37.3
HRCounter-3.0.0-4-bs1.29.1-0ce2b8e.zip

Seems like I need to update to 1.37.3, my 1.29.1 says no. ^^
But I think it's my game that's flawed.

I remember I did backport it to 1.29.1.
Let me double check and make a new build

Ok, now we're talking ^^ It works! Thx man ^^


Implement Unit Test

Add unit test to verify frontend functionality specified in issue #16

Moved to https://github.com/ufosc/course-planner-frontend/issues/4


New PicoC updated with PicoPro software

My new PicoC was blocked from registering with PicoBrew on regular WiFi since I can't sign up for a new account. I connected it to the Picobrew server it was updated to, what looks like, PicoPro software.
The screen selector is using the full size of the display, most of which is hidden behind plastic on the PicoC. It seems like PicoC and PicoP actually use the same displays. Is there any way to force update the Pico to a different software version for PicoC?

They run the same hardware - the c just uses cheap plastic in several places and has crappy kegs

We have confirmed that the firmware is indeed different, but accessed the same way (details in facebook). For the mothership there is a lookup table in the DB to which specific device the UID is associated with. Not sure of a UID format that identifies a C vs a S/Pro so instead going to have to add a config update to support this... however since we can't expand the Alias block in the config.yml I'm thinking of just rebuilding our config.yml format to support other future extensions (whatever that might be).

Here are my thoughts:
Option A:
device_metadata:
  #ProductID:
    #type: C # options include PicoBrewC,PicoBrew,Zymatic,ZSeries,PicoStill where PicoBrew is for both S and Pro
    #alias: Nickname
    #firmware: 0.0.34 # specific version to override the default firmware before by device type

or Option B we just reuse the aliases and create a new PicoBrewC that we use to differentiate between S/Pro and C... which would require someone to alias the machine to get firmware.
aliases:
  Zymatic:
    #ProductID : Nickname
  ZSeries:
    #ProductID : Nickname
  PicoBrewC:
    #ProductID: Nickname
  PicoBrew:
    #ProductID : Nickname
  PicoFerm:
    #ProductID : Nickname
  PicoStill:
    #ProductID : Nickname

For option Option B this would require us to map a PicoBrewC and a PicoBrew to the same MachineType which would be fairly easy to do and fits with our existing structure well... no need to break things until there would actually be a user benefit (like my want to force firmware to a specific device, but we could do that a different non breaking way to see below since this isn't something most users would do anyways)
firmware:
  # ProductID: 0.0.116

@chiefwigms and @WTremml went ahead with the approach as documented above as Option A if anyone feels strongly against let me know.
https://github.com/chiefwigms/picobrew_pico/pull/130

This is resolved now @WTremml feel free to factory reset and try this again, or just take my word for it 😉


support Llama2 fp8c16 inference

PR types

New features
PR changes

Others
Description

support Llama2 fp8c16 inference

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


Functions

I am rounding up with the contents on Function but check these contents for addition and corrections.

@Oluwasetemi @Olubebe @Rolalove please review. Thanks @RidwanAdebosin

Sure @Rolalove
I will work on your recommendation.

Great work @RidwanAdebosin 👏🏿
But at the beginning of the note you used green background colors on the code it contracts the color of the code I can't really read it

Thank you, @Olubebe .
I am going to remove all the extra colors in the slides.

@RidwanAdebosin, the green color is good for learning but the text inside it in dark mode is what needs to be brighter. Have the same look and feel for all the sides does not encourage another challenge/new vibe of a new chapter. If you can find a good blend for the headers in darkmode, the green and white shade need to invert like making the green lightness reduce while the white can be more bright. the code background seems interesting in light mode but in darkmode it overshadows and greenness is everywhere.
Another thing is please add something on apply and bind since they are closely related to call. But you did a great work in the content.

@Oluwasetemi I am not done with the apply and bind content and also concerning the header color, I can work on the blend so that it can be better when the window is in dark mode, but other contents ain't having the background which is going to make the contents from my end looks odd.
So, it's either we designed all header with a background color or we don't design any at all.

@Oluwasetemi I am not done with the apply and bind content and also concerning the header color, I can work on the blend so that it can be better when the window is in dark mode, but other contents ain't having the background which is going to make the contents from my end looks odd.
So, it's either we designed all header with a background color or we don't design any at all.

It's okay, we can have a todo to implement a good customizable header or feedback component with tip/info/success/danger like in the class tutorial. Until then we can even it out to look the same way. Thanks

Noted.

Sequel to the reviews, I have fixed it and completed the content on func.call().
I have started writing on func.apply()

Sequel to the reviews, I have fixed it and completed the content on func.call(). I have started writing on func.apply()

thanks for the good work. I cannot wait to merge the PR. >>>> 👌

Sequel to the reviews, I have fixed it and completed the content on func.call(). I have started writing on func.apply()

thanks for the good work. I cannot wait to merge the PR. >>>> 👌

I am done with it, please check now.

@RidwanAdebosin there is a conflict that need to be resolved before we can merge

@RidwanAdebosin there is a conflict that need to be resolved before we can merge

I will check it.


Add parsing for point electronic DISS EBIC F. Schöppach et al.

[ ] First working version available sufficient to provide an example of an HDF5 file for the Innsbruck group and F. Schöppach and this example converts the internal xmpmeta from DISS onto NeXus, a few pieces of concepts are currently mapped over, using the new pint-powered mapping functor approach but still that mapping code remains much more testing and is thus currently considered unstable.


🦖 Documentation: Add Logger Page (Advance)

Is this repo able to RESOLVES #136?
The Robo.js Logger documentation has been contributed with Logger Separate  Documentation!

@Pkmmte
Any idea why not showing?

Thanks for this! As for why it's not showing on the sidebar, it's because you have to edit the sidebars.js file in docusaurus.

Thanks for this! As for why it's not showing on the sidebar, it's because you have to edit the sidebars.js file in docusaurus.

Thanks 👍 done
can you review it & merge if required :)


Internal Server Error 500 If kit.config.trailingSlash: "always"

In SvelteKit one can configure whether routes should or should not have a trailing slash and this has relevance when it comes to relative urls. However, Svemix will throw an error if  kit.config.trailingSlash: "always".
This config will work:
import adapter from '@sveltejs/adapter-auto';
import preprocess from 'svelte-preprocess';
import svemix from 'svemix/plugin';

/** @type {import('@sveltejs/kit').Config} */
const config = {
	preprocess: preprocess(),
	kit: {
		adapter: adapter(),
		target: '#svelte',
		vite:
		{
			plugins: [svemix()]
		}
	}
};

export default config;

But this will not:
import adapter from '@sveltejs/adapter-auto';
import preprocess from 'svelte-preprocess';
import svemix from 'svemix/plugin';

/** @type {import('@sveltejs/kit').Config} */
const config = {
	preprocess: preprocess(),
	kit: {
		adapter: adapter(),
		target: '#svelte',
		vite:
		{
			plugins: [svemix()]
		},
		trailingSlash: "always" // this line is the problem
	}
};

export default config;

To test this, remember to delete the $__svemix__ directories after changing the config.
The error that one gets is An unknown error occured, which originates from load.ts.
I do not know for sure why this happens, but I have two speculative theories:

There is some string.split("/") action going on somewhere.
The kit.config.trailingSlash option messes with the generated Svemix files. In other words, when kit.config.trailingSlash: "always" one should search for the appropriate endpoint from a different location. This has most likely something to do with the endpoint index.ts files.

Explanation number two seems more probable, but I do not know for sure.

Closed by #31


Change OSEM config variables to MORAGA

This is a fork of the great OSEM software, so we need to change the name to MORAGA in the codebase.
First step is to change the config variables:
docker-compose.yml.production-example
dotenv.example
Gemfile
Gemfile.next
INSTALL.md
app/helpers/users_helper.rb
app/helpers/application_helper.rb
app/models/admin_ability.rb
app/models/email_settings.rb
app/models/user.rb
app/models/ability.rb
app/views/conferences/new_install.html.haml
app/views/conferences/show.html.haml
app/views/proposals/new.html.haml
app/views/proposals/show.html.haml
app/views/conference_registrations/new.html.haml
app/views/conference_registrations/edit.html.haml
app/views/layouts/_user_menu.html.haml
app/views/layouts/application.html.haml
app/views/layouts/_navigation.html.haml
app/views/devise/sessions/_new_embedded.html.haml
app/views/payments/_payment.html.haml
config/piwik.yml
config/database.yml
config/routes.rb
config/environments/development.rb
config/environments/production.rb
config/deploy.rb
config/application.rb
config/initializers/sentry.rb
config/initializers/devise.rb
lib/tasks/migrate_config.rake
spec/controllers/conference_registration_controller_spec.rb
spec/spec_helper.rb
spec/helpers/application_helper_spec.rb
spec/routing/routing_spec.rb
spec/support/omniauth_macros.rb

I see that it has been changed for configuration variables. Should this issue be closed?
But I find other results by searching for OSEM in the code:

CODE_OF_CONDUCT.md
CONTRIBUTING.md
Docker images belongs to osem.
CSS files (app/assets/stylesheets/application.css)
etc

Could you update the issue with the changes @davidsf? I'll make all the changes.


Add "copy all" button for logs editbox

Please add "copy all" button for logs in editbox. Both out and err

This will be added with #317

I'll close this issue and reference it in #317.


Add check-api-changes script to CI

When API code changes, we need to regenerate CRD configs using make generate manifests.
This script checks that regenerated files are pushed in the same PR which changes the API
code so CRD configs are not merged in next unrelated PRs.

/cc @ybettan

/ok-to-test
/lgtm
/assign @ybettan

/approve


VS2019 Imgui/SFML with FetchContent failing to link properly

When compiling I get unresolved external symbol "struct ImGuiContext * GImGui" (?GImGui@@3PEAUImGuiContext@@EA). Works just fine on linux. This is on VS2019 with CMake.
CMakeLists.txt:
cmake_minimum_required(VERSION 3.13)
project(bq-r)
include(FetchContent)

# libs
set(SFML_VERSION "2.5.1")
FetchContent_Declare(
	sfml
	GIT_REPOSITORY "https://github.com/SFML/SFML.git"
	GIT_TAG "${SFML_VERSION}"
)
FetchContent_GetProperties(sfml)
if(NOT sfml_POPULATED)
	FetchContent_Populate(sfml)
	add_subdirectory(${sfml_SOURCE_DIR} ${sfml_BINARY_DIR})
endif()

FetchContent_Declare(
	imgui
	GIT_REPOSITORY https://github.com/ocornut/imgui
)

FetchContent_MakeAvailable(imgui)

FetchContent_Declare(
	imgui-sfml
	GIT_REPOSITORY https://github.com/eliasdaler/imgui-sfml
)

set(IMGUI_DIR ${imgui_SOURCE_DIR})
set(IMGUI_SFML_FIND_SFML OFF)
set(IMGUI_SFML_IMGUI_DEMO ON)
FetchContent_MakeAvailable(imgui-sfml)

# not libs
set(CMAKE_CXX_STANDARD 20)

file(GLOB_RECURSE sources "game/*.cpp")

set(includes "game/")
set(libs sfml-graphics sfml-window sfml-audio sfml-network sfml-system ImGui-SFML::ImGui-SFML)
set(flags)

add_executable(bq-r ${sources})

target_compile_options(bq-r PUBLIC ${flags})
target_include_directories(bq-r PUBLIC ${includes})
target_link_libraries(bq-r ${libs})

I've been up since midnight trying to figure this out lol. The linker error is referencing this file: https://github.com/sarahkittyy/blockquest-remake/blob/main/game/gui/editor_tile_button.cpp

Hello.
Check this out and see if this works for you: https://github.com/eliasdaler/imgui-sfml-fetchcontent/tree/master
If it does, see what your CMake build is doing differently and try to figure out what goes wrong.

I tried submodules, ExternalProject, FetchContent, all sorts of things, and what ended up working was just integrating manually, copying the imconfig-SFML.h over and including all the source files of both directly to the build

I don't believe that's a good long-term solution. Did you try the repo I've linked?
It should work, I think.


[Feature Request] Automatic Propagation of PromptFlow Connection Changes to Live Endpoints

We are experiencing an issue with the behavior of PromptFlow connections in our current environment. Specifically, when a PromptFlow connection is modified, these changes are not automatically propagated to existing live endpoints. The only way to reflect these changes is by re-deploying the endpoint entirely, which causes operational inefficiency and potential downtime.
We would like the system to automatically propagate any changes made to a PromptFlow connection to live endpoints without requiring a manual re-deployment. This enhancement would streamline the workflow, reduce downtime, and improve the overall responsiveness of the system.
The only alternative we have currently is to manually re-deploy the endpoint after every change to the PromptFlow connection. This approach, however, is not ideal as it introduces unnecessary downtime and operational complexity.

I fully support this issue. The current need to manually re-deploy endpoints after modifying a PromptFlow connection is indeed a significant operational burden. Automatic propagation of these changes would not only reduce downtime but also enhance the overall efficiency and reliability of the system. Implementing this enhancement would greatly benefit users by streamlining workflows and minimizing disruption to live services.