API End point changes - Data Product operations - Register a data product (POST)

User story
As a data producer
I want to register a data product (including metadata)
So that it becomes available in the data catalog
And I can start to publish my data
Note: Registering the data product end point already exists. The scope of the ticket would be to update the api end point based on the new design, and add in missing checks and responses.
API Design
https://dsdmoj.atlassian.net/wiki/spaces/DataPlatform/pages/4511991882/Initial+APIs#Data-Product-operations
POST data-product/register
accepts application/json, representing the metadata for the data product.
Validation

Schema must be valid according to https://github.com/ministryofjustice/modernisation-platform-environments/blob/main/terraform/environments/data-platform/data-product-metadata-json-schema/v1.0.0/moj_data_product_metadata_spec.json
Data product name must contain only letters, numbers, and underscores (TBC)
Data product name length max limit TBC
Data product with this name doesn't already exist

Success response
There is no response body so we should return status code 201 on success.
Error responses

use the standardised response functions for responses in the base image
Return 400 response on invalid schema
Return 400 response on invalid data_product_name
Return 409 if the data product already exists

Lambda behaviour
This already exists as create_metadata.
Double check the functionalities:

Validate schema
Extract name from schema
Validate name
Check if data product already exists
Store the metadata in the correct path for v1: DataProductConfig(name).metadata_path()
Create a folder structure for the data product (e.g. raw_data, curated_data paths)

Definition of done

[x] Checked/updated the new lambda
[x] Added swagger docs
[x] Deployed the new endpoint for API gateway
[ ] Added to the smoke test (if possible)
[ ] We are able to call the endpoint to create a new data product

Deploying new endpoint PR

PR for updating the lambda

Validation testing in code appears to be outside the scope of this ticket, and handled by the DataProductMetadata.validate function (tests here)
These specific examples will be tested through manual API testing.


github.com/prometheus/common/log not exist!!!

Hi, I am trying to install, and get this error:
package github.com/prometheus/common/log: cannot find package "github.com/prometheus/common/log" in any of:
/usr/lib/go-1.10/src/github.com/prometheus/common/log (from $GOROOT)
/root/go/src/github.com/prometheus/common/log (from $GOPATH)
I check the url "github.com/prometheus/common/log" and it is not exist anymore on github.com/prometheus.
could you help me please!.

Sadly, this project seems to be abandoned.
The prometheus/common/log module has been deprecated and removed in version 0.27.0 of the Prometheus client library, see https://github.com/prometheus/common/pull/306
Futhermore it uses Go 1.5 which is antique now ;)
If you are using PowerDNS version 4.3.0 (recursor) /4.4.0 (authorative server) or newer, this exporter is obsolete because PowerDNS authorative server and recursor and Dnsdist expose Prometheus metrics natively.

Software
Documentation
Minimum version

DNSDist
https://dnsdist.org/guides/webserver.html

PowerDNS authorative server
https://doc.powerdns.com/authoritative/http-api/index.html#metrics-endpoint
4.4.0

PowerDNS recursor
https://doc.powerdns.com/recursor/http-api/prometheus.html
4.3.0

Note: I've created a updated fork at https://github.com/Bobobo-bo-Bo-bobo/prometheus-powerdns-exporter but I think it's obsolete too, because PowerDNS now has Prometheus metrics out of the box.

@Bobobo-bo-Bo-bobo Hello Bobobo
I got error your folked exporter like this,
INFO[0000] Starting Server: :9120
ERRO[0003] Error scraping PowerDNS: json: invalid use of ,string struct tag, trying to unmarshal unquoted value into float64


Deserialization failures in Blazor WASM

If you believe you have an issue that affects the security of the platform please do NOT create an issue and instead email your issue details to<EMAIL_ADDRESS>Your report may be eligible for our bug bounty but ONLY if it is reported through email.
Describe the bug
Using System.Text.Json to deserialize an object works on SSB and fails on CSB with identical JSON input data.
To Reproduce
Steps to reproduce the behavior:

Using this version of ASP.NET Core 3.0.0
Create a Blazor application using the server template
Replace Index.razor with the attached file (within attached zip)
Add the file Index.razor.cs (also in attached zip)
Using NuGet, add Newtonsoft.Json to the project
Run & observer proper deserialization by both Newsoft & System.Text.Json (As indicated by an OK status on the Index page)
Now create a Blazor application using the WASM template with an Asp.NetCore host
Add the same two files in the pages directory
Edit the namespace in the .razor.cs to match your chosen name/directory
Run the application

Failure #1: WASM: System.MissingMethodException: Default constructor not found for type Deserialize_CSB.Client.Pages.ServiceResult`1[[Deserialize_CSB.Client.Pages.Process_DD[], Deserialize-CSB.Client, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null]]

Change the constructor for ServiceResult to public
Rerun the application

Failure #2: WASM: System.Text.Json.JsonException: The JSON value could not be converted to Deserialize_CSB.Client.Pages.ServiceResult`1+eStatus[Deserialize_CSB.Client.Pages.Process_DD[]]. Path: $.pStatus | LineNumber: 0 | BytePositionInLine: 12.
Index.zip
Expected behavior
I expect CSB and SSB to both deserialize correctly.
Screenshots
n/a
Additional context
C:\Users\ms>dotnet --info
.NET Core SDK (reflecting any global.json):
Version:   3.0.100
Commit:    04339c3a26
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.18362
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\3.0.100\
Host (useful for support):
Version: 3.0.0
Commit:  7d57652f33
.NET Core SDKs installed:
2.1.802 [C:\Program Files\dotnet\sdk]
2.2.402 [C:\Program Files\dotnet\sdk]
3.0.100 [C:\Program Files\dotnet\sdk]
.NET Core runtimes installed:
Microsoft.AspNetCore.All 2.1.13 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.All 2.2.7 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.1.13 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 2.2.7 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 3.0.0 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 2.1.13 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.2.7 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 3.0.0 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 3.0.0 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
To install additional .NET Core runtimes or SDKs:
https://aka.ms/dotnet-download

 private ServiceResult() { }

and
private Process_DD() { }

If you make the default constructor private the deserializer can't access it to create a new instance of your object to be able to set the properties. Have you tried making it public?

Sorry my "Step 11" should have said make both constructors public. It still fails. And it doesn't explain why they need to be public when the same code runs SSB and succeeds.

This issue was moved to mono/mono#17735


missing features for prediction

If some features are missing (from the optimal ones), when featurizing novel compounds (for predicting stage), predict() will not work. This happens for instance when some elements are not present in this prediction set, and corresponding features are thus not generated, but could have been selected during training.

Tagging @gbrunin as partially involved in this.


Fix Bootstrap responsive breakpoints

Typical projector resolution makes this look bad. Filing issue on mobile, might be a dupe.

👍  (I couldn't find a duplicate issue, thanks for filing)


Replace llvm.{smax,smin} intrinsics with builtins

LLVM passes are generating these intrinsics from certain nested loop
patterns. Do this replacement early instead of in SPIRVProducerPass so
that -hack-scf can take effect if necessary.

Why not make the fixup pass handle the llvm intrinsics instead?

Done. Thought I was keeping things simple by handling the intrinsics in just a single place, but I was wrong.


How to merge lora which is of safetensors type(adapter_model.safetensor) for LLaVA-v1.6

Command:
python scripts/merge_lora_weights.py --model-path /home/rohith/Documents/mistral-llava/  --model-base /media/rohith/CC2A79902A7977F2/models/llava-v1.6-mistral-7b/  --save-model-path /home/rohith/Desktop/mistral-llava-merge
Log:
Loading checkpoint shards: 100%|█████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 4/4 [00:03<00:00,  1.30it/s]
Traceback (most recent call last):
File "/home/rohith/LLaVA-1.6-ft/scripts/merge_lora_weights.py", line 22, in
merge_lora(args)
File "/home/rohith/LLaVA-1.6-ft/scripts/merge_lora_weights.py", line 8, in merge_lora
tokenizer, model, image_processor, context_len = load_pretrained_model(args.model_path, args.model_base, model_name, device_map='cpu')
File "/home/rohith/LLaVA-1.6-ft/llava/model/builder.py", line 112, in load_pretrained_model
mm_projector_weights = torch.load(os.path.join(model_path, 'mm_projector.bin'), map_location='cpu')
File "/home/rohith/miniconda3/envs/llava/lib/python3.10/site-packages/torch/serialization.py", line 986, in load
with _open_file_like(f, 'rb') as opened_file:
File "/home/rohith/miniconda3/envs/llava/lib/python3.10/site-packages/torch/serialization.py", line 435, in _open_file_like
return _open_file(name_or_buffer, mode)
File "/home/rohith/miniconda3/envs/llava/lib/python3.10/site-packages/torch/serialization.py", line 416, in init
super().init(open(name, mode))
FileNotFoundError: [Errno 2] No such file or directory: '/home/rohith/Documents/mistral-llava/mm_projector.bin'

had to include lora in model name
thats all fixed

Hi Rohith,
I am trying to use the med-vqa dataset you uploaded on hugging face but need help on that. Please let me know how I can find the entire image dataset for the same.
Regards,
Debarati

Hi Rohith,
I am trying to use the med-vqa dataset you uploaded on hugging face but need help on that. Please let me know how I can find the entire image dataset for the same.
Regards, Debarati

https://huggingface.co/datasets/rbojja/medical-vqa/tree/main/vqa-rad/images


Change namespaces to lowercase to match other packages/plugins

I'm hesitant to accepting this.  Lowercase namespaces is a Yii/Craft thing which this package isn't intended to be exclusively used with.

Would require a major release. Consider for Craft 4.


Update xml-rs dependency to 0.7.

This also updates the Cargo.lock file to the newer format.
All tests pass with the updated dependencies.

Everything looks good, thanks for your contribution!


teleport: init at 2.4.0

Motivation for this change
Teleport was did not have a pre-existing package for Nix.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Closing in favor of #33853.


Only allow mgmt-release from public repo

Allow release from only public repo.

/check-enforcer override


feat(Algebra/Group/Commute): port file

mathlib3 SHA:<PHONE_NUMBER>0d5b5d1e2d1f4f37e5420357bdc636
porting notes: mostly smooth

This completely rewrites the existing ad hoc port
there were several dubious translation warnings. I think none of them are issues, but I have left comments and also added ₓ to those declarations. If you want to see the translation errors see 46d095eabc7afb8d06e21f3b9c61c9a5eea1df5f
I fixed a naming error in Algebra/Group/Semiconj
simpNF told me that it could prove some lemmas, so I removed the simp attribute and left porting notes.

bors merge

bors merge


File variables can look empty when they're not

Describe the bug
File variables can look empty when they are not.
To Reproduce
Steps to reproduce the behavior using Alitair Mac app version 2.1.1 (<IP_ADDRESS>99):

In a tab, prepare a mutation that has an Upload scalar type argument.
Open the "VARIABLES" tray.
Create a new file variable name and choose a file using the "Choose File" button. The filename displays next to the button.
Close the "VARIABLES" tray.
Open the "VARIABLES" tray. "No file chosen" now displays instead of the filename.
Send the request. Note that the mutation succeeds, and that the previously selected file is uploaded despite appearing to have been unset.

Expected behavior
Closing and re-opening the "VARIABLES" tray should not reset the display of the file input value.

Thanks for reporting this! I'd look into it as soon as I can.


Fix potential race conditions in mongo-based user lock implementation

Current implementation of user lock is not atomic and thus prone to race conditions.
Proposed implementation fixes the issue by using an atomic findAndModify (with upsert).

Thanks. included in 2.0.1-SNAPSHOT :)


Added screenreader text on close button

Added screen reader text on close button.
Connected to this issue: #706 & #665
Please check if you also get visual error on AlertRibbon close button on colored backgrounds in React:
https://storybook.seb.io/latest/react/?path=/docs/components-alertribbon--docs
Locally it looks like this line is not triggered anymore:
green\libs\chlorophyll\scss\components\alert-ribbon_mixins.scss
line 139 - background-color: var(--color);
Also green\libs\react\src\lib\card\card.stories.tsx file shows up as red/error?

Closing this now. Half I have merged and other half I created issues for.


RSDK-4713 RSDK-5626 MovementSensor

What changed

implement MovementSensor component (mostly copying from Zack's branch)
incidental: fix issue in handler map where only 1st model implementing an API is registered
add new wrappers to reduce the effort of adding a component:

ResourceRegistration2, one-liner to implement a resource registration -- this is in movementsensor.cpp for now but could be dropped into registry.hpp if useful in the future
stub_wrapper removes ~10 lines in each grpc client method
server_wrapper removes ~15 lines in each grpc server method

Also, @stuqdog, I know there are some plans afoot to eliminate the static initialization. There have also been some thoughts about making a cleaner separation between the client facing types and the server (e.g. module supporting) types. The current registration model requires that one part of the code (the registration type) know of both the Server and Client types, because there must be one ResourceRegistration subtype that can produce both types. That might be another thing to consider refactoring if/when the auto-registration is addressed.

I only really looked at the resource registration changes, so my LGTM is limited to that. Overall, I think resource registration needs to be refactored and I expect that the final result will look more like a template member on Registry which does the work that ResourceRegistration2 does now.

Potentially an unhelpful too-many-cooks situation if I review here, and my review is mostly qs 😆. I'll eventually follow up offline about those qs instead.


kernel: thread: Simplify k_thread_foreach conditional inclusion

Simplify k_thread_foreach API conditional inclusion by putting
the whole logic under CONFIG_THREAD_MONITOR config option.
Signed-off-by: Ramakrishna Pallala<EMAIL_ADDRESS>

Codecov Report

Merging #8981 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #8981      +/-   ##
==========================================
+ Coverage   52.25%   52.25%   +<.01%
==========================================
  Files         196      196
  Lines       24802    24801       -1
  Branches     5155     5155
==========================================
  Hits        12961    12961
+ Misses       9762     9761       -1
  Partials     2079     2079

Impacted Files
Coverage Δ

kernel/thread.c
81.57% <100%> (+0.53%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5d1b57f...0fe8907. Read the comment docs.


List of HLS protocol version support

I'm looking for limitations for support of different HLS protcols by the HLS player. Any advice would be greatly appreciated. Thanks.

nevermind. found it -> https://github.com/mangui/flashls


BLD: bump build number to pick new lapack95.

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender


tf.keras.backend.constant abortion

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: N/A
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below):2.1.0
Python version:3.7.6
Bazel version (if compiling from source):N/A
GCC/Compiler version (if compiling from source):N/A
CUDA/cuDNN version:N/A
GPU model and memory:N/A

Describe the current behavior
tf.keras.backend.constant abortion
Describe the expected behavior
expect an exception message if the input is not expected, instead of crash.
Standalone code to reproduce the issue
import tensorflow as tf
import numpy as np
tf.keras.backend.constant(value=np.ones((0,1,1)), shape=[36,23,53,24,117,82,47,124,112,69,53,0])

Output:
2021-01-26 16:30:57.093291: F tensorflow/core/framework/tensor_shape.cc:405] Check failed: 0 <= new_num_elements (0 vs. -1)
Aborted (core dumped)

I have tried in colab with TF version 2.1, nightly version(2.5.0-dev20210126) and was able to reproduce the issue. Please, find the gist here. Thanks!

I have tried in colab with TF version 2.1, nightly version(2.5.0-dev20210126) and was able to reproduce the issue. Please, find the gist here. Thanks!

I think the issue will be fixed by PR #46717.

I think the issue will be fixed by PR #46717.


Terminal not loading ~/.zprofile and not setting LC_ALL

VSCode Version: Code 1.9.1 (f9d0c687ff2ea7aabd85fb9a43129117c0ecf519, 2017-02-08T23:31:51.320Z)
OS Version: Darwin x64 15.6.0
Extensions:

Extension
Author
Version

concrete
vscode
0.1.10

vscode-eslint
dbaeumer
1.2.6

tslint
eg2
0.8.1

Theme-TomorrowKit
ms-vscode
0.1.3

debugger-for-chrome
msjsdiag
2.5.4

vetur
octref
0.3.8

vscode-icons
robertohuertasm
7.3.0

language-stylus
sysoev
1.6.0

vim
vscodevim
0.5.3

I'm using zsh as login shell, but when I launch terminal in VSCode, ~/.zprofile is not loaded.
VSCode:

Not running echo HI, not setting LC_ALL.

iTerm2:

@Tyriar

@octref hmm, I'm not sure why .zprofile wouldn't get sourced if you're running it as a login shell (["-l"] in shellArgs).
As for LANG being set, this should be fixed in Insiders (and stable when it's released today), a change was made to fall back to sensible language variants if only the language is provided by Electron.

LANG seems fixed now for 1.10. Previous I was using diff-so-fancy and it was complaining no default lang set.
Sorry seems I had a misunderstanding of zprofile. It's sourced when logging in and all the env variables are carried over, and shells spawned later will inherit all the env var. That's why I should only set env var like GOPATH there.
However iTerm2 does load zprofile each time I open a new terminal whereas VSCode doesn't.

@octref Normally ~/.zprofile will run on a login shell (ie. zsh -l), if it doesn't exist ~/.profile will. You can also set the file to source using shellArgs which might be helpful, or you could source it from your ~/.zshrc:
. ~/.zprofile

I'll close this off as it's the shell's responsibility to do this, let me know if you find more info though :smiley:


Question about other trained models

Are there any places to get trained models with other style images? I'm trying to make a few myself but they take a long long time!

NVM, I just found the repo for them.
thanks!

Hi @atomoton can you point me to the repo? I'm trying to obtain models for different styles, and it would save  me a lot of time if there are already out there. Thanks!

@rushilanirudh  https://github.com/gafr/chainer-fast-neuralstyle-models

@atomoton awesome, thanks for the link! Don't have any now, but will update if I do come across any :+1:


Extension Development In SharePoint 2019 Server Meet Problems

Category

[x] Question
[ ] Typo
[ ] Additional article idea

Question

I've generated a basic appliction customizer with the guide of offical document (https://docs.microsoft.com/en-us/sharepoint/dev/spfx/extensions/get-started/build-a-hello-world-extension) ,but the code cannot work in my enviroment, no matter live develop or add it to sharepoint apps.
when I'm in live develop ,the javascript function can work .like console/alert ,but use microsoft library function ,they all fails and bring errors.
Uncaught Error: No callbackHandler found.
at Function.e._sendMessage (sp-pages-assembly.js?uniqueId=2g4QD:119)
at Function.e.sendPageLoaded (sp-pages-assembly.js?uniqueId=2g4QD:119)
at t._onPostMessageLoading (sp-pages-assembly.js?uniqueId=2g4QD:119)
at t._onPostMessage (sp-pages-assembly.js?uniqueId=2g4QD:119))
when I upload it to sharepoint ,the page will also bring error and the javascripts will not appear(they can appear in live develop)
I tried to change SPFx/Node/@microsoft/generator-sharepoint version but they were all failed
is it really useful the extension function in sharepoint server 2019 ,the webpart I tried is useful but the extension is not
the project I developing will deploy in a enviroment which has no web , will it affect the function too?
sharepoint server 2019 was created by azure version

Environment details (development & target environment)

Your Developer Environment: [ Windows 10 ]
Target Environment: [ SharePoint Server 2019 ]
Framework: [ Node.js v10 ]
Browser(s): [ Chrome v84 | IE v11 | Edge]
Tooling: [ VS Code | SPFx v1.4.1 |<EMAIL_ADDRESS>]
Additional details: The more context you can provide, the easier it is (and therefore quicker) to help.

Sorry, but it's unclear what the issue is or what the specific problem is you're having. Can you share code / provide a repro as without that, there's not much to go off of ...

Ok, this is my code(https://github.com/tntboneman/testSPFXExtension), please verify it on sharepoint server 2019, thanks.

I don't have an SP2019 environment so tested in SPO... works fine for me.
Do you see any errors in the browser console? By chance, are there certificate errors listed?

Yes, there are quite a lot errors in browser console.

after a series of tests, I found only spfx 1.4.1 can use in my enviroment(sharepoint server 2019 on premise), any higher or lower version has no effect.

the typescript version should higher than 3.6.4, typescript dependencies in package.lock.json should all be the same version(remove node_modules, manual modify package.lock.json, reinstall node_modules).

if use @pnp/sp, 1.3.X version seems all right but some function like copy file will fail, the 2.0.4 version may be a better choice.

after these changes, I can see my extension items show in page(button and dialog), even still with some errors I think it's a correct way

@tntboneman said:

after a series of tests, I found only spfx 1.4.1 can use in my enviroment(sharepoint server 2019 on premise), any higher or lower version has no effect.

That's correct... SPFx v1.4.1 is what's included on SP2019... that version is pinned so you must use that version.
@tntboneman said:

manual modify package.lock.json

You should never do this... if anything, delete node_modules & the lock file, then run npm install and let NPM manage the lock file creation. Don't ever hand-edit this.
As for the connection errors, did you trust the dev cert as part of your local development workstation setup as the docs state?

@andrewconnell said:

As for the connection errors, did you trust the dev cert as part of your local development workstation setup as the docs state?

Yes, I ran the command 'gulp trust-dev-cert'.
Why I manual modify package.lock.json because I use pnpjs(https://pnp.github.io/pnpjs/SPFx-on-premises/).
I can run most logic (open dialog/choose templdate page) but get trouble in copy page now.
pnpjs throw a error:
Uncaught (in promise) TypeError: Cannot read property 'zoneEmphasis' of undefined
    at Proxy._ClientsidePage.mergeColumnToTree (localhost:4321/dist/command-set-list-command-set.js:10618)
    at Proxy._ClientsidePage.setControls (localhost:4321/dist/command-set-list-command-set.js:10498)
    at Proxy._ClientsidePage.fromJSON (localhost:4321/dist/command-set-list-command-set.js:10088)
    at Proxy.<anonymous> (localhost:4321/dist/command-set-list-command-set.js:10106)
    at step (localhost:4321/dist/command-set-list-command-set.js:813)
    at Object.next (localhost:4321/dist/command-set-list-command-set.js:794)
    at fulfilled (localhost:4321/dist/command-set-list-command-set.js:784)

perhaps this is no matter with @microsoft/sp

OK... now you're talking about a specific challenge you're facing with PnPJS. For that, you should direct questions to their repo: https://github.com/pnp/pnpjs

Thank you andrewconnell.


Outdated/unavailable contacts

From README.md
You can contact me via [matrix](https://matrix.to/#/@siranweb:matrix.org) or [telegram](https://t.me/siranweb)

Matrix: No answer.
Telegram: User does not exist.

Yeah, I'll remove it. GH issues tab is enough


Math problems under ubuntu 12.04 lts

Addind transform="scale(1,1)" to path crash SVGO.
Example:
<svg
 width="486.037"
 height="1000"
 viewBox="0 0 486.03677 1000"
 xmlns="http://www.w3.org/2000/svg">
<path d="M124.074 1000v-469.229h-124.074v-168.945h124.074v-144.301c0 -113.393 73.291 -217.525 242.169 -217.525 68.376 0 118.937 6.555 118.937 6.555l-3.984 157.766s-51.564 -.502 -107.833 -.502c-60.9 0 -70.657 28.065 -70.657 74.646v123.361h183.331l-7.977 168.945h-175.354v469.229h-178.632"
  transform="scale(1,1)"/>
</svg>

Looks like macosx not affected.

Hmm, couldn't reproduce with the current svgo version.

Let's drop is. I used SVGO for path transforms apply, and switched to svgpath pkg long time ago. Don't know status of this issue.


Jc/refactorquery

(Note that this PR merges into 1.0 branch, which is WIP and need not be fully functional yet.)
Checking in this PR now, even though more testing needs to be done (and I know there is one failing test). These issues will be noted in pivotal. However, this was a really far reaching refactor stemming from adding strong mode, so this is a reasonable stopping point to break up the subsequent work into more manageable tasks.
This PR also points at a temporary holding place for the postgresql driver. This will be replaced in a subsequent PR when the new postgresql driver is ready.
There are a lot of big changes.

Updated to use strong mode.
Removed ModelQuery, significantly improved interface for handling relationships and joins.
Altered the behavior of transient properties.
Refactored some code related to building data models.
Renamed a lot of properties to make them easier to understand.
Started building significant guide documentation in docs/

Can we turn the travis integration back on for this branch? I see 4 commits went to CI

added travis integration for PRs into 1.0

🆗  with me


[layer] bug fix for array initializer

Bug fix for array initializer storing the indices of the requested
weight so that accessing weights not requested always causes error.
resolves #1773
Signed-off-by: Parichay Kapoor<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #1774. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnstreamer.mooo.com/.


Issue with spacing

Since the <\span> does not have a width it gets mushed together like this:

But there is lots of space...

Any ideas for a fix?

Add size to the print statements. That worked on mine.

@atiaisaac how exactly? @Skillzore did you solve this?

return "%d%% | length=10" % (color, cpu)


Clojure: refresh breaks if there's a compilation error during refresh

Conjure version: 998603d
If there is a compilation error during refresh, the REPL is left in an inconsistent state and the only way to fix it (that I've found) is to restart it.
Steps to reproduce:

Create a new project: lein new contest
Start a REPL: lein repl

OpenJDK 64-Bit Server VM warning: Options -Xverify:none and -noverify were deprecated in JDK 13 and will likely be removed in a future release.
nREPL server started on port 64585 on host <IP_ADDRESS> - nrepl://<IP_ADDRESS>:64585
REPL-y 0.4.4, nREPL 0.8.3
Clojure 1.10.1
OpenJDK 64-Bit Server VM 14.0.2+12-46
    Docs: (doc function-name-here)
          (find-doc "part-of-name-here")
  Source: (source function-name-here)
 Javadoc: (javadoc java-object-or-class-here)
    Exit: Control+D or (exit) or (quit)
 Results: Stored in vars *1, *2, *3, an exception in *e

contest.core=>

Reload all the namespaces with Conjure

; Refreshing changed namespaces
contest.core
contest.core-test
; Refresh complete

Insert a syntactic error in core.clj (add (foo) at the end) and call :ConjureCljRefresh

; Refreshing changed namespaces
contest.core
contest.core-test
; Error while reloading contest.core
; (err) Syntax error (ArityException) compiling at (contest/core.clj:8:1).
; (err) Wrong number of args (0) passed to: contest.core/foo

At this point, refresh is broken: any combination of refresh, refresh-all or clear will simply not do anything. I supposed this is related to https://github.com/clojure/tools.namespace#warnings-and-potential-problems, specifically the "REPL namespace" issue.
Looking at nREPL debug output, it seems like it's not getting any namespaces to refresh:
; Refreshing changed namespaces
; debug: send
{:id "7c7ced09-10ba-4c11-a10d-a66bd146c05b"
 :op "refresh"
 :session "f12941e6-7483-428c-8a94-5c711ec37956"}
; debug: receive
{:id "7c7ced09-10ba-4c11-a10d-a66bd146c05b"
 :reloading {}
 :session "f12941e6-7483-428c-8a94-5c711ec37956"}
; debug: receive
{:id "7c7ced09-10ba-4c11-a10d-a66bd146c05b"
 :session "f12941e6-7483-428c-8a94-5c711ec37956"
 :status ["ok"]}
; Refresh complete
; debug: receive
{:id "7c7ced09-10ba-4c11-a10d-a66bd146c05b"
 :session "f12941e6-7483-428c-8a94-5c711ec37956"
 :status ["done"]}
; debug: receive
{:changed-namespaces {}
 :id "7c7ced09-10ba-4c11-a10d-a66bd146c05b"
 :repl-type "clj"
 :session "f12941e6-7483-428c-8a94-5c711ec37956"
 :status ["state"]}

Thanks for making Conjure and let me know if there's anything missing from this bug report!

This seems to have been resolved somewhere along the way, closing!


Add cleaning up data tailor if error

Pull Request
Description
Adds a step if there is any error in the data tailor customization download, it will cleanup all data tailor customizations that are done or failed.
Fixes #
How Has This Been Tested?
Unit tests

[ ] Yes

Checklist:

[ ] My code follows OCF's coding style guidelines
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked my code and corrected any misspellings

Note: The plotting tests are failing because they aren't setup to work without the RSS imagery, which is not available till December 19th

Codecov Report

Merging #136 (558a3a2) into main (0ad6a39) will decrease coverage by 1.74%.
The diff coverage is 73.68%.

:exclamation: Current head 558a3a2 differs from pull request most recent head 67b19b6. Consider uploading reports for the commit 67b19b6 to get more accurate results

@@            Coverage Diff             @@
##             main     #136      +/-   ##
==========================================
- Coverage   66.69%   64.95%   -1.75%
==========================================
  Files          10       10
  Lines        1039     1050      +11
==========================================
- Hits          693      682      -11
- Misses        346      368      +22

Impacted Files
Coverage Δ

satip/__init__.py
100.00% <ø> (ø)

satip/eumetsat.py
75.60% <72.22%> (+0.48%)
:arrow_up:

satip/app.py
90.90% <100.00%> (ø)

satip/download.py
56.09% <0.00%> (-11.59%)
:arrow_down:

satip/utils.py
71.14% <0.00%> (-0.29%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Front: primary with methods to call secondary

I'm starting this boring part :trollface:

Closing this one, to create separated tickets with good first contribution label


I'm sorry to bother you

I am a college student from China. As you know,
I can’t go out, because of so 2019 novel coronavirus (2019-nCoV),
And then I searched the project on GitHub and found your artwork.
I really like it. I watched your forum(www.studiopieters.nl), but I ’m not Clearly for circuit structure，
can you tell me? or give a PCB file if possible, it would be greatly appreciated. Thanks again.

Here is the scheme:
https://raw.githubusercontent.com/AchimPieters/ESP8266-HomeKit-Light-Sensor/master/images/Light Sensor.png

Thank you very much！


docs: update rome init explanation with current info

rome.json is now rome.rjson.

We should actually take down the current site because it's old and inaccurate. The latest website is in the PR #507 which will be merged on our first release.

Closing this. Old website site is already down and #507 and #616 were merged.


SVD - D1 v1.0 manual

Hi @duskmoon314
Google recently recommended this project to me when I was looking for the SVD allwinner D1.
I tried to do the same in relation to the manual v1.0 D1-H but had a issue during the extraction.
Do you have a script to extract the manual or is there another way?

I have taken a look at the issue and that is great work! To be honest, the svd in this project is just information copied from the pdf and reshaped via the regex expression. 😢
It seems that you have managed to generate a complete svd of D1-H. Do you have an interest in contributing to this project?

I have taken a look at the issue and that is great work! To be honest, the svd in this project is just information copied from the pdf and reshaped via the regex expression. 😢

Thanks.

It seems that you have managed to generate a complete svd of D1-H. Do you have an interest in contributing to this project?

Well, I suggest using the files attached to this issue. Then I'll try to generate D1s SVD and give you the copy.

https://github.com/ZigEmbeddedGroup/regz/issues/17#issuecomment-1115330029

Hi @duskmoon314
I uploaded all the SVDs in the zig issue mentioned above and noticed that there are already updates in d1-pac.
If it's already resolved, close this issue, otherwise I'll do a comparison on the SVDs and send PR.


pythonPackages.colorlog: 4.8.0 -> 5.0.1

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 119344 at f4092dd6 run on aarch64-linux 1

2 packages marked as broken and skipped:

python38Packages.optuna
python39Packages.optuna

3 packages failed to build:

apache-airflow
cot (python38Packages.cot)
python39Packages.cot

1 package skipped due to time constraints:

tts

18 packages built successfully:

errbot
python38Packages.abodepy
python38Packages.clldutils
python38Packages.colorlog
python38Packages.dask-gateway-server
python38Packages.phonemizer
python38Packages.pyenvisalink
python38Packages.segments
python38Packages.skybellpy
python39Packages.abodepy
python39Packages.clldutils
python39Packages.colorlog
python39Packages.dask-gateway-server
python39Packages.phonemizer
python39Packages.pyenvisalink
python39Packages.segments
python39Packages.skybellpy
rapid-photo-downloader

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement.

apache-airflow and cot are broken due to unrelated dependencies (croniter and pyvmomi->six, respectively)

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 119344 run on x86_64-linux 1

2 packages marked as broken and skipped:

python38Packages.optuna
python39Packages.optuna

3 packages failed to build and already failed to build on hydra master:

apache-airflow: plain log  |  pretty log
cot: log was empty
python39Packages.cot: log was empty

1 package failed to build and are new build failure:

(python38Packages.cot): log was empty

19 packages built:

errbot
esphome
python38Packages.abodepy
python38Packages.clldutils
python38Packages.colorlog
python38Packages.dask-gateway-server
python38Packages.phonemizer
python38Packages.pyenvisalink
python38Packages.segments
python38Packages.skybellpy
python39Packages.abodepy
python39Packages.clldutils
python39Packages.colorlog
python39Packages.dask-gateway-server
python39Packages.phonemizer
python39Packages.pyenvisalink
python39Packages.segments
python39Packages.skybellpy
rapid-photo-downloader


No team is found in the second option when trying to send a new message

The user app is correctly installed both for the users and for the team I am using for testing, yet the issue still occurs.
I have already checked the status of all three functions, verified the IDs and secrets, as well as all configurations.
In Application Insights, I see the following errors:

Could you help find and resolve the issue? Thank you.

We got exactly the same issues as Nei, as you can see below:

@neidantaslopez  @peddivyshnavi
There is this Graph API update that could be causing the issue.
https://techcommunity.microsoft.com/t5/microsoft-entra-blog/important-update-azure-ad-graph-api-retirement/ba-p/4090534

Hi @neidantaslopez
Thanks for reaching out to us.
Could you please confirm us with the following.
For making teams visible we need to add User app  for teams which should appear in dropdown.
could you please go through deployment guide whether Set-up Authentication and Add Permissions to your Microsoft Graph Azure AD app are added correctly or not

I’m experiencing the exact same scenario. Nothing has changed in our environment. The last time CC communications were sent out to end users was on August 9th, and there were no issues then. Something must have changed since.

Hi @neidantaslopez Thanks for reaching out to us. Could you please confirm us with the following. For making teams visible we need to add User app for teams which should appear in dropdown. could you please go through deployment guide whether Set-up Authentication and Add Permissions to your Microsoft Graph Azure AD app are added correctly or not

User app is already added to the team I want to appear in dropdown:

Set-up Authentication:

Add Permissions to your Microsoft Graph Azure AD app:

Hi @neidantaslopez
For making teams visible we need add that particular team to user app
Please follow below steps for making team visible:

Install User app.
Create a Teams channel with the people or Group you want to trigger user app installation to.
Navigate to installed user app from apps -> manage apps in Teams.
Click on the app and Select an option of Add to a Team.
Select created / desired Teams channel from Dropdown.
Click on set up / Submit button.

Hi @neidantaslopez For making teams visible we need add that particular team to user app Please follow below steps for making team visible:

Install User app.
Create a Teams channel with the people or Group you want to trigger user app installation to.
Navigate to installed user app from apps -> manage apps in Teams.
Click on the app and Select an option of Add to a Team.
Select created / desired Teams channel from Dropdown.
Click on set up / Submit button.

I have already followed all the steps. The user app is added to the team we want to appear in the dropdown, but the team still does not show up (which is why I created this issue).
Is there anything else I should check?

Hi @neidantaslopez
Could you please perform the below procedure.

Please uninstall the user app completely. Wait for sometime
Again re-install/ upload the user app at the tenant level
Add the user app to the specific team.
And try checking the option in the dropdown.

Hi @neidantaslopez Could you please perform the below procedure.

Please uninstall the user app completely. Wait for sometime
Again re-install/ upload the user app at the tenant level
Add the user app to the specific team.
And try checking the option in the dropdown.

We have already done that...
Do we have any other thing to check?

Hi @neidantaslopez
Can you also please confirm validating this
The error indicating that the issue might be due to policies. However please reverify if it meets the Auth Issue Fix requirements

Hi @neidantaslopez Can you also please confirm validating this The error indicating that the issue might be due to policies. However please reverify if it meets the Auth Issue Fix requirements

I don't think that that could be the problem because in the page you sent me it said:
"...The users in your organization might see an error while accessing the message authoring experience in the Company Communicator app in Teams, with the error message “Sorry, an error occurred while trying to access the service”."
We don't see and error while accessing the message authoring experience. The problem is that we are not able to pick Teams in the dropdown menu even though we have user app installed for them.

Hi @neidantaslopez
Could you please share screenshot of userapp
open teams-->Apps-->manage apps-->userapp

Hi Nei Dantas Lopez
we can start discussing about this ticket internally

From internal discussion , after sync all the three functions then reinstall user app issue got resolved marking ticket as closed.


Fixed a number of bugs related to workspace planner.

This PR addresses several major bugs unmasked by the workspace planner.

Fixed a bug in cloning an environment into itself
(needed for recursive @PlanningMethods)
Fixed a bug in incorrect formatting of RetimeTrajectory error messages.
Fixed numerous small issues in the workspace planner:

Returning a 1-waypoint trajectory when started in-contact with an object.
Fixed max_distance calculation error from missing numpy.copy()
Simplified some of the workspace planning logic.

I think we should print a warning if you try to clone an environment into itself. It seems very easy to run into aliasing issues like this. It also makes it easy to accidentally introduce a major bug in an otherwise innocuous typo:
with Clone(env, into=env): # whoops, I meant into=env2
    # do stuff on env2, which is now in the incorrect state

We could either add an optional argument to disable the warning (e.g. warn_on_same=False) or simply require the caller to not call Clone in this case. Thoughts?
In any case, the rest of the pull request LGTM, so I am merging it. Thanks for the bug fixes. :smile:


Compute TOT for FORESCE

We currently have CAT-equivalent data for FORESCE (a value for each PRMS segment). To get a value corresponding to the total upstream area (TOT):

Match the hru_segment column to the subseg_seg column in p1_prms_reach_attr
For each subseg_seg, get all upstream subseg_seg values by recursively checking the from column in p1_prms_reach_attr. That results in a vector of all subseg_seg that are upstream of the current subseg_seg.
Compute the TOT value for the subseg_seg by taking the area-weighted sum of the CAT values in each upstream segment.

Looking for some help thinking through this problem:
My steps to address issue above are the following:
There are two relevant target dfs for this process:

p1_prms_reach_attr which conveniently maps the immediate upstream source segment(s) from_segs and the downstream segment to_seg of each prms segment (subseg_id) . (Note that there are sometimes multiple from_segs - addressed in step 1 below.)

p1_prms_reach_attr %>% dplyr::select(subseg_id,subseg_seg,from_segs,to_seg)
# A tibble: 456 × 4
   subseg_id subseg_seg from_segs to_seg
   <chr>          <dbl> <chr>      <dbl>
 1 1_1                1 NA             3
 2 2_1                2 NA             8
 3 3_2                3 8              4
 4 4_1                4 3              5
 5 5_1                5 4;9           11
 6 6_1                6 NA            13
 [...]

p2_FORESCE_LC_per_catchment_reclass which has the proportion of LC Class (standardized w/ NLCD classes) per PRMS id (PRMS_segid)and hru_segment (which matches subseg_seg above

p2_FORESCE_LC_per_catchment_reclass[[1]]
# A tibble: 418 × 14
   PRMS_segid hru_segment total_PRMS_area prop_lcClass_3 prop_lcClass_2 prop_lcClass_1
   <chr>            <int>           <dbl>          <dbl>          <dbl>          <dbl>
 1 1_1                  1          15774.         0.184         0.00809       0.00206
 2 2_1                  2          30180.         0.0685        0.00823       0.000582
 3 3_1                  3          38735.         0.177         0.0139        0.00148
 4 3_2                  3          38735.         0.177         0.0139        0.00148

[...]

The processing steps are the following:

separate p1_prms_reach_attr$from_segs to have a unique row for each from_segs value - done
e.g.

> ## separate fro_segs into new rows (no more ; sep from segs)
> p1_prms_reach_attr %>% dplyr::select(subseg_id,subseg_seg,from_segs,to_seg) %>%
+     tidyr::separate_rows(from_segs,sep=";")
# A tibble: 561 × 4
   subseg_id subseg_seg from_segs to_seg
   <chr>          <dbl> <chr>      <dbl>
 1 1_1                1 NA             3
 2 2_1                2 NA             8
 3 3_2                3 8              4
 4 4_1                4 3              5
 5 5_1                5 4             11
 6 5_1                5 9             11
 7 6_1                6 NA            13
 8 7_1                7 NA            12
 9 8_2                8 2              3

Create a new col  p1_prms_reach_attr$all_from_segs populated with a vector of all upstream segments (from_segs) of each segment (^ comment above describes recursive function to do this). - stuck
Run separate_rows() again on the new col to have a new row for each value in all_from_segs - to do
Merge new vrsn of p1_prms_reach_attr with p2_FORESCE_LC_per_catchment_reclass by c(subseg_id == PRMS_segid) - to do
Run a group_by on merged df by PRMS_segid and summarize across all prop_lcClass values, calculating the area-weighted sum. This will give the proportion of each LC class for all upstream catchments of each PRMS segment  - to do

In ** step 2** +, how do I extract all correct from_segs associated with each $subseg_id when some segments have multiple from_segs (ex: 5_1). Please see gist and comment on code there.
See gist
Current outcome of recursive function in image below. Note the error in $all_from_segs for duplicate $subseg_id (e.g. when reach_attr_table$subseg_id == 5_1 & reach_attr_table$from_segs == 9, its reach_attr_table$all_from_segs vector starts with 4).

separate p1_prms_reach_attr$from_segs to have a unique row for each from_segs value

Instead of making a new row for each from_segs value, would it help to change the format of the from_segs column to be a vector of the segment values?

Thanks for looking into this, @msleckman! During our meeting today I mentioned that another approach to this problem might be to use the crosswalk table to take advantage of functions within nhdplusTools that identify upstream tributaries. I haven't tested it, but I think something like this would work, for example:
# create another 'target' for the comid_segs;
# what comid's intersect PRMS segments?
drb_comids_seg <- p2_prms_nhdv2_xwalk %>%
  select(PRMS_segid, comid_seg) %>%
  tidyr::separate_rows(comid_seg,sep=";") %>%
  rename(comid = comid_seg)

# Define function to return PRMS segs upstream of a given PRMS seg_id
get_upstream_prms_segs <- function(prms_line,nhd_lines,downstream_comids,segment_comids){

  # Filter downstream_comids to only include the PRMS subsegid of interest
  comid_down <- downstream_comids %>%
    filter(PRMS_segid %in% prms_line$subsegid)

  # Use nhdplusTools to find all COMIDs upstream of comid_down
  comids_UT <- nhdplusTools::get_UT(network = p1_nhdv2reaches_sf,
                                    comid = comid_down$comid,
                                    # set distance (km) at some arbitrary high value
                                    distance = 2000)

  # From the COMIDs that intersect PRMS segments, retain those COMIDs
  # that are upstream of comid_down
  PRMS_segs_upstream <- segment_comids %>%
    filter(comid %in% comids_UT) %>%
    pull(PRMS_segid)

 return(PRMS_segs_upstream)

}

# function returns upstream segs for one PRMS segment (x); lapply/map
# over get_upstream_prms_segs for each seg within PRMS network
x <- p1_reaches_sf[1,]
upstream_segs <- get_upstream_prms_segs(prms_line = x,
                              nhd_lines = p1_nhdv2reaches_sf,
                              downstream_comids = p2_drb_comids_down,
                              segment_comids = drb_comids_seg)

Will have to look at this Lauren as I am more deep into the recursive approach we've worked on earlier.
@jds485  realizing we'll need to add the current reach_attr_table$subseg_seg to the list in reach_attr_table$subseg_seg otherwise we are missing the area of the CAT.

realizing we'll need to add the current reach_attr_table$subseg_seg to the list in reach_attr_table$subseg_seg otherwise we are missing the area of the CAT

Yes, I figured that would happen after gathering all of the upstream HRUs with the gist function. If you think it's simpler to also return the current HRU, I can suggest a way to do that within the function

Commit https://github.com/USGS-R/drb-inland-salinity-ml/pull/97/commits/b233aef84a8a2ce0ae8dc93147e404d90bad998d adds this to the pipeline.
One stand-alone function was added recursive_fun() and the rest of the calculation was done in the same pipe in 2_process.R. We can later put this into a function but prioritizing other the remains blocks to resolve this issue.
Remaining problems:

Not all upstream are being captured at the moment. Issue noted in gist.
hru_area is used to aggregate metrics to PRMS scale. 2_process.R L:155 ( Given @lekoenig 's review of the NHD metadata, this needs to be updated (although this may automatically change when we use the updated catchments gpckg for the land cover processing.


ASTDefinitionBuilder description Trying to get property 'value' of non-object

I recently updated from php 7.0 to 7.3 and "webonyx/graphql-php": "~0.10.0", to "^0.13.8"
because the break, continue 2 issue.
ASTDefinitionBuilder description
Documentation says description should be a string and it is! so what am I missing here?
I'm getting the error:
Trying to get property 'value' of non-object
ASTDefinitionBuilder line 102
Currently causing problems:
        if ($node->description) {
            return $node->description->value;
        }

This modification works:
     return $node->description;

This was a change in SDL format. See this notice in UPGRADE.md: https://github.com/webonyx/graphql-php/blob/master/UPGRADE.md#breaking-descriptions-in-comments-are-not-used-as-descriptions-by-default-anymore
Old:
# Description
type Dog {
  ...
}

New:
"Description"
type Dog {
  ...
}

"""
Long Description
"""
type Dog {
  ...
}


Better prompting

Right now, there are still some quotes around the command sometimes. Try to mitigate the problem by finding better prompts.

Merged with feat/better_prompt


Change Project Delete to use red color text instead of blue 'X' png

Is your feature request related to a problem or area of OpenRefine? Please describe.
Users are not seeing how to delete a Project in OpenRefine.
User tweeting about this:
https://twitter.com/ruthbrarian/status/1243243600223391745

Took me forever to see the little Xs to delete them too

We also use currently use differing styles for Actions to perform on a Project:

Img
Text

Describe the solution you'd like
Make the style consistent.  Change the style to use red text instead:
Before change:

After change:
Delete

Describe alternatives you've considered
Do nothing and confuse users more.
Additional context
Before change:

After change:

I think it would be good to try to keep some consistency with the rest of the style of the UI (although this project list has already been butchered widely in that regard). A red "Delete" is quite aggressive.
We could consider other options, perhaps a trash can icon?

We need to not label things as "good first issue" when they are not fully spec'd out. It's discouraging to new contributors to continually have the goal posts moved.
I like the idea of the trash can icon in one of the colors from our current palette.

Sure, then perhaps the solid or outline one from https://material.io/resources/icons/?search=delete&style=baseline

The pitch on the lines currently is pretty tight, so there isn't a ton of room to work with. We might be able to get away with an 18 px version of "Delete forever" by increasing line spacing a pixel or two.

The current color seems fine to me (or at least as good as any since we already have like 4 different colors per project line).


Doesnt work with 7.4

This version needs PHP8. It fails in earlier versions due to the union types declarations in PluginManager.

I'm facing the same issue

The bug has been resolved in 3.4.0


SocketTimeoutException while Downloading in 4G  Cellular network. Works fine on Wifi

OkDownload Version
v1.0.2-SNAPSHOT
Problem Describe
I occur some problem, I also search on issues and wiki, I even checked the source code, but it can't help, so my real problem is:...
Log
There is the whole log from LogCat when I occur the problem(I know we can use this tool to let log more readable):

While downloading a file via enqueue api we see there is an issue with Cellular network downloading. Even though having good speed of 4g, it is giving Download failed with Socket Timeout exception

Reopening as was closed by mistake

I have the same problem with 4G Cellular, there's no problem with wifi.
Did you find anyway to resolve the problem?


Updating README with wiki links

@basukinjal do you want to remove the autotune mentions from the wiki as well?


Relay 13 (Rust compiler) Support

We tried to update to relay 13 with it's new compiler, however it appears that relay-compiler-webpack-plugin doesn't yet support Relay 13.
13:05:54 webpack.1 | > webpack-dev-server
13:05:54 webpack.1 | > ./bin/webpack-dev-server
13:05:54 webpack.1 |
13:05:56 webpack.1 | node:internal/modules/cjs/loader:936
13:05:56 webpack.1 |   throw err;
13:05:56 webpack.1 |   ^
13:05:56 webpack.1 |
13:05:56 webpack.1 | Error: Cannot find module 'relay-compiler/lib/language/javascript/RelayLanguagePluginJavaScript'
13:05:56 webpack.1 | Require stack:
13:05:56 webpack.1 | - /Users/rowan/www/runn/runn/node_modules/relay-compiler-webpack-plugin/dist/index.js

Is the plan for this plugin to support the new rust compiler in relay 13?

Hi @ro-savage . For my part, I don't use relay any more and won't have the time or motivation to do the change. I can merge a PR enabling support for it if someone opens one though.

@danielholmes - thanks for the quick response. Totally understandable and thanks for the awesome work putting this together in the first place.
Our team is trying to get Relay 13 working with webpack now - including trying a fork of this plugin.
If we end up going with a forked version, we'll submit a PR back to this repo.


Inconsistent promise API for exec.start()

I propose that the resolve method of the promise returned by start.exec() should be called with a stream so that it is consistent with the stream argument when the function is used with a callback, stream.exec(callback).
The callback in the exec.start(opts, callback) function is defined as your typical node callback (or function(err, stream) as per example in the Stream goodness chapter). When used as a callback, it is indeed a node stream. Typically, you expect the same object to be passed as an argument if you work with promises instead, e.g. one would expect this to be true:
const promise = exec.start(opts)
    .then(stream => {
         // do something with the stream
    });

However, it turns out that the stream object is in fact an Exec object where the stream has been added as an output property. Thus, this snippet reflects the current implementation:
const promise = exec.start(opts)
    .then(exec => {
         const stream = exec.output;         // <- why not pass the stream directly?
         // do something with the stream
    });

Ref: see the exec.start() implementation
dockerode version: 2.5.8

This was identified in: https://github.com/apocas/dockerode/issues/486
Will release a new major this weekend improving this.


Find and add any missing elements/components in demo

https://github.com/mubaidr/bulma-fluent/commit/e9b4d9a8b39b402d9110a0d83407c26717481689


ConfyUI-Extension installation outdated

Trying to install some of the newer nodes causes venv to be destroyed.
See also https://github.com/WASasquatch/was-node-suite-comfyui/issues/457#issuecomment-2282901221
I had a coomment there:

Ah you are using the extension. That is a pretty old extension which hasn't had an update in 7mo.

It is old and pretty much no longer maintained.
That being said, there is an option in the settings to select separated venvs for A1111 and ComfyUI to work together nicely.

#117

Ah, maybe there isn't one? I don't remember

https://github.com/ModelSurge/sd-webui-comfyui/blob/10e95a42e27f7e7c2b15284491d1e09f2ea9a971/lib_comfyui/comfyui_process.py#L51
If you install ComfyUI separately with its own venv, and reference the install location in the settings, it should work, assuming the extension still works at all


Change global background color

I'm currently working on my first presentation with impress (really like it btw, HTML is best). I was inspired by this presentation. Which introduced a rather hacky way of changing the global (documenty.body) background-color. I needed this functionality so I wrote a little plugin for it.
It looks for a data-background attribute on the slide it is changing to and applies a class of that name to the document.body.
Is this something that could be needed by others?
If so I would love to create my first pull-request :D

Hi
I think the recommended or typical way to change background color is to set it in your CSS. Example: https://github.com/impress/impress.js/blob/master/css/impress-demo.css#L88
Note that this should already be quite powerful. For example, if you want to change the color for a particular slide:
body.impress-on-step-1 { background: green; }

There shouldn't be a need to use Javascript for styling, plugin or otherwise. (Unless we're talking about really advanced effects and they do something else than just react to stepping to the next slide.)
That's a really nice presentation btw!

Ok I didn't know there were classes applied to the body based on the current slide. Still one would have to set the background for every single slide, so I prefer my method. But yeah, it's not neccessary.

For sure we could improve the documentation. There's a lot of good stuff like that in impress.js that I only learned once I worked on the code itself.
Also what would be helpful for this and other use cases would be if you could group steps into sections. For your use case it would mean you wouldn't need to write css for each individual slide, but could instead do something like body.impress-on-section-1. (Someone wanted the table-of-contents select menu to use sections too. So it would be a generally useful concept.)


Populating an entity with eager enabled causes 'Maximum call stack size exceeded' exporting with toJSON()

Describe the bug
When finding entities with populate: true or populate: ["entity"] causes a 'Maximum call stack size exceeded' error.
Bonus: how do you initialize @OneToOne relations from the object if it isn't loaded during the query, it isn't like a Collection?
//Entity1

@OneToOne(() => Entity2, e => e.joinprop, { eager: true })
    ent2: Entity2;

//Entity3
@ManyToOne(() => Entity1)
    ent1: Entity1;

// service

async list(
    ent1: Entity1,
    limit: number,
    offset: number
): Promise<[Entity3[], number]> {
    const connection = await Database.getConnection();
    return connection.em
        .getRepository(Entity3)
        .findAndCount({ ent1 }, { limit, offset, populate: ["ent1"] }); //or populate:true
}

Stack trace
Unhandled Promise rejection: Maximum call stack size exceeded ; Zone: <root> ; Task: Promise.then ; Value: RangeError: Maximum call stack size exceeded
    at Array Iterator.next (<anonymous>)
    at Function.processEntity (C:\Users\XX\Repositories\XXProj\node_modules\mikro-orm\dist\entity\EntityTransformer.js:51:56)
    at Function.processProperty (C:\Users\XX\Repositories\XXProj\node_modules\mikro-orm\dist\entity\EntityTransformer.js:43:38)
    at C:\Users\XX\Repositories\XXProj\node_modules\mikro-orm\dist\entity\EntityTransformer.js:21:51
    at Array.map (<anonymous>)
    at Function.toObject (C:\Users\XX\Repositories\XXProj\node_modules\mikro-orm\dist\entity\EntityTransformer.js:21:14)
    at Entity1.prototype.toJSON (C:\Users\XX\Repositories\XXProj\node_modules\mikro-orm\dist\entity\EntityHelper.js:28:62)
    at Function.processEntity (C:\Users\XX\Repositories\XXProj\node_modules\mikro-orm\dist\entity\EntityTransformer.js:52:26)
    at Function.processProperty (C:\Users\XX\Repositories\XXProj\node_modules\mikro-orm\dist\entity\EntityTransformer.js:43:38)
    at C:\Users\XX\Repositories\XXProj\node_modules\mikro-orm\dist\entity\EntityTransformer.js:21:51
    at Array.map (<anonymous>)
    at Function.toObject (C:\Users\XX\Repositories\XXProj\node_modules\mikro-orm\dist\entity\EntityTransformer.js:21:14)
    at Entity2.prototype.toJSON (C:\Users\XX\Repositories\XXProj\node_modules\mikro-orm\dist\entity\EntityHelper.js:28:62)
    at Function.processEntity (C:\Users\XX\Repositories\XXProj\node_modules\mikro-orm\dist\entity\EntityTransformer.js:52:26)
    at Function.processProperty (C:\Users\XX\Repositories\XXProj\node_modules\mikro-orm\dist\entity\EntityTransformer.js:43:38)
    at C:\Users\XX\Repositories\XXProj\node_modules\mikro-orm\dist\entity\EntityTransformer.js:21:51 RangeError: Maximum call stack size exceeded
    at Array Iterator.next (<anonymous>)
    at Function.processEntity (C:\Users\XX\Repositories\XXProj\node_modules\mikro-orm\dist\entity\EntityTransformer.js:51:56)
    at Function.processProperty (C:\Users\XX\Repositories\XXProj\node_modules\mikro-orm\dist\entity\EntityTransformer.js:43:38)
    at C:\Users\XX\Repositories\XXProj\node_modules\mikro-orm\dist\entity\EntityTransformer.js:21:51
    at Array.map (<anonymous>)
    at Function.toObject (C:\Users\XX\Repositories\XXProj\node_modules\mikro-orm\dist\entity\EntityTransformer.js:21:14)
    at Entity1.prototype.toJSON (C:\Users\XX\Repositories\XXProj\node_modules\mikro-orm\dist\entity\EntityHelper.js:28:62)
    at Function.processEntity (C:\Users\XX\Repositories\XXProj\node_modules\mikro-orm\dist\entity\EntityTransformer.js:52:26)
    at Function.processProperty (C:\Users\XX\Repositories\XXProj\node_modules\mikro-orm\dist\entity\EntityTransformer.js:43:38)
    at C:\Users\XX\Repositories\XXProj\node_modules\mikro-orm\dist\entity\EntityTransformer.js:21:51
    at Array.map (<anonymous>)
    at Function.toObject (C:\Users\XX\Repositories\XXProj\node_modules\mikro-orm\dist\entity\EntityTransformer.js:21:14)
    at Entity2.prototype.toJSON (C:\Users\XX\Repositories\XXProj\node_modules\mikro-orm\dist\entity\EntityHelper.js:28:62)
    at Function.processEntity (C:\Users\XX\Repositories\XXProj\node_modules\mikro-orm\dist\entity\EntityTransformer.js:52:26)
    at Function.processProperty (C:\Users\XX\Repositories\XXProj\node_modules\mikro-orm\dist\entity\EntityTransformer.js:43:38)
    at C:\Users\XX\Repositories\XXProj\node_modules\mikro-orm\dist\entity\EntityTransformer.js:21:51

To Reproduce
Steps to reproduce the behavior:

Have an entity that has eager enabled
Find with populate enabled
Crash.

Expected behavior
Return an object with all entities initialized.
Versions

Dependency
Version

node
12.9.1

typescript
3.5.3

mikro-orm
3.0.0-alpha.27

your-driver
mysql2

This is caused by serializing the entity, so as a workaround you could handle the recursion check manually in custom toJSON() implementation.

Yes. I see now, has to do with the object exporting; changed the title. Will have a proper look at this tomorrow, thank you.

Not sure I understand - are you asking how to do something or how something works in MikroORM internally? :] There are no collections in 1:1 relations, just one (owning side) pointing to the inverse via unique foreign key.

I was wondering if Mikro-ORM offered such feature that you can initialize OneToOne as it just returns a property with the number 1 if not pop. But... I based it on the JSON output instead of the objects in Node.. I have not asked anything :)

I was wondering if Mikro-ORM offered such feature that you can initialize OneToOne as it just returns a property with the number 1 if not pop. But... I based it on the JSON output instead of the objects in Node.. I have not asked anything :)

Maybe you mean this?
const e1 = em.findOne(Ent1, 1, { populate: true });
wrap(e1.foo).populated(false); // foo is 1:1 property that got populated and would cause the recursion error

That way you marked the reference as 'not populated by user', meaning it will be serialized as if it were not populated at all.

Fixed in 3.0.0-alpha.28.


Context is not set on serializers

There is a context keyword argument, but values passed there are not set on the object.
See: https://github.com/clarkduvall/serpy/blob/master/serpy/serializer.py#L87

The context argument was added for compatibility with Django REST Framework (#7), but it isn't actually used anywhere, so it's just ignored. If you need to use it I recommend making a custom Serializer subclass that sets context on the object.

That's what I'm doing now:
class ContextSerializer(serpy.Serializer):
    def __init__(self, *args, **kwargs):
        super(ContextSerializer, self).__init__(*args, **kwargs)
        if 'context' in kwargs:
            self.context = kwargs['context']

If it's a design decision that the context argument is going to be ignored, perhaps a note in the documentation to this effect would be possible?


Start implementation of Data Access Layer for Global State

Start implementation of EE side of data access layer with placeholders for a block storage access. Depends on #3157.

@mpapierski could you check if this is done?


Chrono Legionnaire

Port chrono legionnaire from TG
Take the modsuit and TEM from this, and put it over a RCD that can deconstruct players.
This is mostly for admemes and removing players with bad IC names, similar to the terminator ghost role.

I made chronolegioneer for another server, but they conceptually different from TG version (because i didn't found any video with them :|).
It was made for admemes, but need some rework and code cleanup - https://github.com/MetalSage/space-stories-archive/pull/108


[bug] Wrong refile path when capture on Windows

Thank you for this awesome plugin. This problem is a Windows specific problem I guess and I don't know if this problem is related to nvim-orgmode/orgmode plugin. I'v tested  nvim-orgmode/orgmode 's capture, and it works well.
When config org-roam directory as absolute path, the capture mode seems to generate wrong refile path.
-- orgmode
require('orgmode').setup({
  org_agenda_files = {"D:\\doc\\org\\*"},
  org_default_notes_file = "D:\\doc\\org\\refile.org",
})
-- org-roam
require("org-roam").setup({
  directory = "D:\\doc\\org\\roam",
})

When press <leader>nc enter capture mode set node name and wq, it will prompt Refile destination D:/doc/org/roam\D:documents/org/roam\20240430102233-roam_test.org dose not exist, Create now?. This file path seems like join the directory twice and when second joint, missing a slash after D:.
When config directory as relative path
require("org-roam").setup({
  directory = "roam",
})

When :pwd echos D:\some_code_base.
The refile path is D:\some_code_base\roam\roam\20240430102233-roam_test.org, it joint path twice too.

Hm, your separator is both / and \. I also don't know why it's appending roam twice. The Windows run of tests didn't have any problems with joining. The org roam directory definitely needs to be absolute, either something like /path/to/roam or ~/roam. But not sure why there are two path separators in the first example, or why roam appears twice in the second.

The second path (presumably joining two absolute paths together) is D:documents/.... So it's missing the \ following the drive.

Yes. It's missing some slash or backslash. So i wander there may some path join problem.

The Windows run of tests didn't have any problems with joining.

If you could not reproduce this problem, I can help to debug, but need a little help with where the path join happened. A brief call route is enough.

@KortanZ try pulling the latest commit. I booted up an old Windows vm and was able to reproduce, but with the latest commit it seems to go away.

@chipsenkbeil Now it works like a charm. Thank you!
Btw, I've also tested relative path(which is illegal as you told), the twice-join problem still, but i think we can safely ignore that if there is no other side effect.


REST call as alternative to SMTP?

Hi,
Any plans adding a (configurable) REST call as an alternative to the SMTP call?
There are quite a few services that have a REST API that could be used as destination.
Thank you.

Hey, thanks for asking. I currently do not have the time to implement it. But you can send me a pull request with the implementation and I merge it ASAP.


Disease clouds - buff?

So I had been thinking about your next incoming change, the healers.
How about increasing the time a disease cloud stays?

Disease clouds in Skirmish (by throwing cows) are kinda weak in general. They just stay for a short amount of time, have a fairly small radius and it's quite easy to move your troops around.
Also, throwing cows on towers and walls doesn't work at all, which makes many attacks of an opponent like the Wolf useless. When I play against him, he always tries to throw Cows on the towers and walls, instead at the people behind it.

I doubt that it will be easy to make disease clouds reach on to walls, or if that would actually be a desirable thing.
But we can agree that they need to be buffed.
Factors that can be buffed:
-Time the cloud poison stays in effect
-Radius / Range of the cloud disease effect
-Damage tick (this one is fine the way it is now)

Yeah, I'm in agreement that disease clouds need to linger much much longer. Maybe even grow stronger the longer they stay before eventually dying out, to at least vaguely simulate a disease outbreak. It'd make apothecaries much more useful to combat it.
Right now cow corpses just elicit a 'welp better move my swordsmen five steps to the right' sort of response. I think there needs to be a real sense of terror when the enemy starts flinging corpses at you. An actual risk of some sort of lingering outbreak that could potentially kill everyone in your castle if you're not prepared.

Why not have the people that walk through it get a poison effect that lasts for a certain time unless the person is healed damaging them over time? that way it would be a more consistent damage output and be a lot more effective when throwing the cows at crowded places. one could argue that the poison would have a chance to spread to nearby peasants and units as well 5% chance or something like that simulating an outbreak. (would have to be tested heavily through)

mhm sad times
I could already imagine a peasant/unit walk around with a small poison cloud following him around until dead/times up/healed with the cloud infecting other peasants (with a low chance) when crossing that units/peasants path

mhm sad times

Indeed!
There are no other real DoT sources ingame yet, right? Except fire of course, which does DoT while standing on it... hmm...

I could already imagine a peasant/unit walk around with a small poison cloud following him around until dead/times up/healed with the cloud infecting other peasants (with a low chance) when crossing that units/peasants path

Considering fire creates sparks that might "infect" (ignite) other nearby tiles (and buildings on those tiles)... I think "poison sparks" would probably be less complex to implement than moving DoT effects per unit.

DoT sources are as you said disease and fire now to realize my idea "all that needs to be done" would be that as soon as a person enters a cloud  one of the 4 (maybe more in the future) clouds would start following him around damaging him over time and if the person gets close to other people have a chance to give them a copy of the cloud that is following him around (and so on), until he is dead the timer on the cloud ran out or he is healed.

Right now cow corpses just elicit a 'welp better move my swordsmen five steps to the right' sort of response. I think there needs to be a real sense of terror when the enemy starts flinging corpses at you. An actual risk of some sort of lingering outbreak that could potentially kill everyone in your castle if you're not prepared.

This! Maybe it would be possible to implement the pestilence mechanic from the campaigns in Skirmish. The opponent throws a cow behind the walls and disease will start to spread. During the event in campaign mode, clouds start popping up everywhere in the castle. Perhaps it's possible to use the mechanic from that event in Skirmish, but without the drop in popularity,

Or... (just thinking out loud here, might not be practicable or hard to balance) ...skip the disease spreading if too complex to implement, but then DO include a popularity drop instead.  Then just annoy the hell outta him and cripple his economy with unhappy peasants demanding for increase food rations or lower taxes!
\o/

Could give fresh peasants a 1 min immunity/damage reduction by default to prevent the disease to completely flatten the economy from one cow.
Since the % of the illness spreading would be rather low you would, have to bombard the castle with cows to have it actually go really out of control. (needs to be tested ofc to actually know it would work or not regardless so I imagine it's not gonna happen any time soon if at all)

The proposed change sounds more realistic, but it would make it easier for a human lord against the AI. To prevent this, I would like it to be made (if possible) that the AI will actively try and move their units away from a disease cloud. For example, if I am attacking an Arabian AI with horse archers below their keep, if I throw cows below the keep and a gas cloud appears, the horse archers will try and move away.


Country note view

Description
Add functionality to view country notes of a given country.
Currently, the command is country nview c/COUNTRY_CODE. I will change this to country note view in a future issue because this conflicts with a current command country note and changing it for this PR will be very troublesome.
How it is implemented is based on the WidgetViewOption in CommandResult, if it is CLIENT, then show the client view. Else if it is COUNTRY_NOTE then show country notes view.
Country notes view is basically a list view consisting of Country Note cards.
Also updated internal storage of country notes in CountryNotesManager to use an ObservableList so that changes will be populated in real time on the GUI.
Fixes #129
Testing
Mainly wrote tests for files that are not in UI.
Remarks
Reason for not using WidgetViewBox is that it is too customized for client view, it would be quite hard to modify it to show a list of country notes.

This looks amazing!!!


[enhancement]: Will this plugin be updated to support the new Homebridge v2 changes

Describe the solution you'd like
I did a quick check to see if there would be issues with this plugin on a test version of HB v2 and there are errors thrown from the system.  While I do not have any issues with the changes for streaming of my cameras, I wanted to be proactive in the event there were issues with functions that I do not use.
Please find a snippet of errors below:
[7/19/2024, 7:45:15 AM] Loaded 3 cached accessories from cachedAccessories.0E6A37BC4BA6.
HAP-NodeJS WARNING: The accessory 'STATION_Homebase 3' has an invalid 'Name' characteristic ('STATION_Homebase 3'). Please use only alphanumeric, space, and apostrophe characters. Ensure it starts and ends with an alphabetic or numeric character, and avoid emojis. This may prevent the accessory from being added in the Home App or cause unresponsiveness.
HAP-NodeJS WARNING: The accessory 'STATION_Homebase 3_Siren' has an invalid 'Name' characteristic ('STATION_Homebase 3_Siren'). Please use only alphanumeric, space, and apostrophe characters. Ensure it starts and ends with an alphabetic or numeric character, and avoid emojis. This may prevent the accessory from being added in the Home App or cause unresponsiveness.
HAP-NodeJS WARNING: The accessory 'DEVICE_Backyard Camera' has an invalid 'Name' characteristic ('DEVICE_Backyard Camera'). Please use only alphanumeric, space, and apostrophe characters. Ensure it starts and ends with an alphabetic or numeric character, and avoid emojis. This may prevent the accessory from being added in the Home App or cause unresponsiveness.
HAP-NodeJS WARNING: The accessory 'DEVICE_Backyard Camera_Enabled' has an invalid 'Name' characteristic ('DEVICE_Backyard Camera_Enabled'). Please use only alphanumeric, space, and apostrophe characters. Ensure it starts and ends with an alphabetic or numeric character, and avoid emojis. This may prevent the accessory from being added in the Home App or cause unresponsiveness.
HAP-NodeJS WARNING: The accessory 'DEVICE_Backyard Camera_Motion' has an invalid 'Name' characteristic ('DEVICE_Backyard Camera_Motion'). Please use only alphanumeric, space, and apostrophe characters. Ensure it starts and ends with an alphabetic or numeric character, and avoid emojis. This may prevent the accessory from being added in the Home App or cause unresponsiveness.
HAP-NodeJS WARNING: The accessory 'DEVICE_Backyard Camera_Light' has an invalid 'Name' characteristic ('DEVICE_Backyard Camera_Light'). Please use only alphanumeric, space, and apostrophe characters. Ensure it starts and ends with an alphabetic or numeric character, and avoid emojis. This may prevent the accessory from being added in the Home App or cause unresponsiveness.
HAP-NodeJS WARNING: The accessory 'DEVICE_SmartDrop' has an invalid 'Name' characteristic ('DEVICE_SmartDrop'). Please use only alphanumeric, space, and apostrophe characters. Ensure it starts and ends with an alphabetic or numeric character, and avoid emojis. This may prevent the accessory from being added in the Home App or cause unresponsiveness.
HAP-NodeJS WARNING: The accessory 'DEVICE_SmartDrop_Motion' has an invalid 'Name' characteristic ('DEVICE_SmartDrop_Motion'). Please use only alphanumeric, space, and apostrophe characters. Ensure it starts and ends with an alphabetic or numeric character, and avoid emojis. This may prevent the accessory from being added in the Home App or cause unresponsiveness.

NOTICE TO USERS AND PLUGIN DEVELOPERS
> Homebridge 2.0 is on the way and brings some breaking changes to existing plugins.
> Please visit the following link to learn more about the changes and how to prepare:
> https://github.com/homebridge/homebridge/wiki/Updating-To-Homebridge-v2.0

I will need to change the way we name entity... this will remove all existing device since UUID is based of the name. Such a shame that underscore aren't compatible :( in addition I will need to convert all name to the new match criterias name with accent FYI will be an issue....

I was afraid of that.  I didn’t see if there is a requirement to be off of the old version by a specific date.  With such a big change, I can’t imagine that plugins would need to be updated right away.  fingers crossed

On Jul 19, 2024, at 8:24 AM, lenoxys @.***> wrote:
I will need to change the way we name entity... this will remove all existing device since UUID is based of the name. Such a shame that underscore aren't compatible :( in addition I will need to convert all name to the new match criterias name with accent FYI will be an issue....
—
Reply to this email directly, view it on GitHub https://github.com/homebridge-eufy-security/plugin/issues/592#issuecomment-2239301121, or unsubscribe https://github.com/notifications/unsubscribe-auth/A7TVUSTITEJD4BVJC2N46PTZNEOTBAVCNFSM6AAAAABLEUWCY2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMZZGMYDCMJSGE.
You are receiving this because you authored the thread.


Renderer should call component.superRender()

I thought that anytime you did a renderer, you should call the superclass renderer. Will double check, test, and either close or fix and add pr.

Starting to put 2 and 2 together. I'm guessing this was omitted because there is actually no markup in this component. So all the rendering is done in the renderer.


Have more than one cta button in header overlay

[X] This is a question about using the theme.
[x] This is a feature request.
[ ] I believe this to be a bug with the theme.

[ ] I have updated all gems with bundle update.
[ ] I have tested locally with bundle exec jekyll build.

Environment informations

Minimal Mistakes version: 4.9.0
Jekyll version: 3.6.2
GitHub Pages hosted: no
Operating system: Linux Debian

Hi! In my "old" web site I used multiple buttons in the index.md; it's straight forward in the body of the markdown. However, now I'm migrating, and I think I want to use the header overlay image; so my idea is to use the CTA option for buttons.
The question is: is there a way to have more than one cta_label / cta_url? If not, what would be the best approach to implement such layout? I attach a mock up of how I imagine the result.

+1 for this extension


Support Intellicode in VS Code. Probably using dataclass_transforms

Currently the HoloViz system is difficult to use in VS Code with as you don't get any context help. For example for Panel the only thing you get is

I.e. you don't get any docstring. You don't get any information on the arguments the __init__ accepts.
@philippjfr points out in https://github.com/holoviz/panel/issues/3085 that ..., we explicitly enabled signatures on Parameterized class constructors which should be the accepted way to declare the accepted kwargs.
@brettcannon points out in https://github.com/holoviz/panel/issues/3085 that For attrs at least, they implemented https://github.com/microsoft/pyright/blob/main/specs/dataclass_transforms.md which I believe is in the process of becoming a proposed PEP. It's documented at https://www.attrs.org/en/stable/extending.html?highlight=pyright#pyright .
Suggestion

Review the above specifications and potentially suggest improvements
Determine whether dataclass_transforms is a solution for Param. And if it is implement.

The alternative would be to reach out to pyright and ask why they don't use the signatures of pep362 and/ or if they could.

Thanks a lot @MarcSkovMadsen for opening this issue. As a VS Code user too I think we share the same feeling! This is something I'd be happy to work when I find the time.

According to https://github.com/microsoft/pyright/issues/2855 we would probably have to add the constructors manually all over the place or align the api of Param with the api of dataclasses.

There's also more details over in the Pyright issue in regards to using .pyi files if you want the type information to live outside of the code itself.

Just for reference I tried changing to the Jedi language server. But the result was the same

we would probably have to add the constructors manually

Sounds to me like we can construct this automatically by introspection?

I have the code to create stub .pyi files for all Parameterized classes in Panel.
https://github.com/holoviz/panel/pull/3132
I just need the functionality to add the relevant imports to the top of the file. And then test and probably spend lots of time on fixing surprices :-)
Note the challenge for VS code is that is works via static type analysis as opposed to Jupyter that works via dynamic analysis. VS Code use pyright. Pyright does not instantiate classes to inspect them. It uses static analysis only.


[FEAT]: Blog - Creating a Blog Post

Description

A registered user can create a blog post

The Blog Schema

Acceptance Criteria
Create Blog [POST] /api/blogs

A valid token is supplied to the Authorization header

If no Authorization header is supplied or an invalid token is supplied, the error should return a status code of 401

Request Header
        {             "Authorization": "Bearer eyJhbGciOiJIUzI1NiIsInR..."         }

A valid request body with correct validation is supplied:

The title field cannot be empty
The content field cannot be empty
The error should return a status code of 400 if the required fields are missing

Request Body
       {
           "title": "Blog Title",      required
           "content": "Blog Content",     required
           "tags": ["tag1", "tag2", ...],
           "imageUrl": "Image URL"
       }

Purpose

The purpose of the endpoint is for a registered user to be able to create a blog post.

Successful Response
Status Code 201
   {
       "status": "Success",
       "message": String,
       "data": {
             "blogId": String,
             "title": String,
             "content": String,
             "imageUrl": String,
             "tags": [String],
             "author": String,
             "createdAt": Date
       }
   }

Error Responses
Status Code 400
   {
       "status": "Bad request",
       "message": String
   }

Status Code 401
   {
       "status": "Unauthorized",
       "message": String
   }

Status Code 500
   {
       "status": "Internal server error",
       "message": String
   }

Testing

Unit tests

Test for valid authorization token in the header
Test for invalid authorization token
Test for authorization token expiry time
Test that the invalid requests body, e.g. test for each required field, returns the appropriate status codes and data
Test that the valid request body returns the valid status code and data

blog posts are by super amdin

Kindly check for the proposed changes.

@Timmy-id Will a blog post have a single text and image?
Your database schema doesn't show how multiple paragraphs and image inserts can be achieved.
Work on that

According to the Figma design, it only requires a single image to store its URL in the database.
The blog content will be stored as text @NdubuisiJr


Swift3

Fixed swift 3 conversion, added tableview height option
Awesome work!

Nice!. I'll be working all this month to make it more dynamic and I'll remove the Ionicons dependency, it seems they won't release a Swift 3 version :s

https://github.com/keitaoouchi/IoniconsKit
Here is a Swift 3 library.
If you could add this to a cocoaPod that would be awesome

Of course!. Consider it done in this week

On Oct 11, 2016, at 1:21 PM, hslightnin<EMAIL_ADDRESS>wrote:
https://github.com/keitaoouchi/IoniconsKit https://github.com/keitaoouchi/IoniconsKit
Here is a Swift 3 library.
If you could add this to a cocoaPod that would be awesome
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub https://github.com/igongora/UIDropDown/pull/2#issuecomment-253001103, or mute the thread https://github.com/notifications/unsubscribe-auth/AEHQz_DybcIyRxBAPLo7Z_YYzct1zN5_ks5qy9O4gaJpZM4KT20v.


Don't dealloc strings if the TensorBuffer is shared (#357)

There's no more straightforward way to get the information? It doesn't sound like TF_TensorMaybeMove() is always going to return null when the content is being shared.

There's no more straightforward way to get the information? It doesn't sound like TF_TensorMaybeMove() is always going to return null when the content is being shared.

If there is, I haven't found it.
BTW, what I really wonder is: Shouldn't string deallocation be done automatically on the native Tensorflow level when the last tensor that uses the TensorBuffer is released?

No, that doesn't happen. We need to allocate the strings manually, and deallocate them manually or we end up with memory leaks, see issue #251.

Oops, it looks like the modified code is not formatted correctly, @brychcy  can you please run mvn spotless:apply locally before pushing a newer version of this PR?

Oops, it looks like the modified code is not formatted correctly, @brychcy can you please run mvn spotless:apply locally before pushing a newer version of this PR?

Done.
At first it didn't do anything for me because origin/master pointed to my version and
origin/master
is used.
As the whole file was reformatted, I have done it in a separate commit.


e2e Tests for LNBits and BlueWallet

re-adding commented out tests

Does it make sense to put the first Protect your wallet steps into a helper function? this looks repetitive.

Does it make sense to put the first Protect your wallet steps into a helper function? this looks repetitive.

Good point. Added them into the same file for now:

commonCreateWalletUserCreate
commonCreateWalletSuccessCheck


Use current branch to figure out PR

Implements #194
Octo review start and Octo review resume can now be called from anywhere (not from a PR buffer) and Octo will try to figure out the PR details from the current branch

Hello, Im trying to use this feature but if I run Octo review start or Octo review resume, nvim prompts the error: No Octo buffer found.
There is a way to open a new review on current branch without executing Octo <PR url> or Octo pr list or Octo pr edit <PR number>?

No, Im sorry, that has not been implemented yet. You need to first open the PR buffer


Design Document for OpenDevin

I am interested in contributing to OpenDevin and want to understand the project better, particularly how to debug and navigate its systems.
Do you have a design document or any detailed documentation explaining how this agent system works? Specifically, I would like to know more about:

How the agent plans and executes commands
The architecture and flow of the system
Any debugging tips or best practices

Your guidance would be greatly appreciated.

Maybe the docs files in project can help you

We have a rough architecture diagram here, but it probably needs updating: https://opendevin.github.io/OpenDevin/modules/usage/architecture

@ramsey-coding does this newly added overview answer your question?
https://github.com/OpenDevin/OpenDevin/blob/main/opendevin/README.md


HOSTEDCP-632: hypershift-operator/controllers/hostedcluster: isUpgradeable minor-bump scoping

Godocs for Upgradeable:

Upgradeable indicates whether the component (operator and all configured operands) is safe to upgrade based on the current cluster state. When Upgradeable is False, the cluster-version operator will prevent the cluster from performing impacted updates unless forced. When set on ClusterVersion, the message will explain which updates (minor or patch) are impacted. When set on ClusterOperator, False will block minor OpenShift updates. The message field should contain a human readable description of what the administrator should do to allow the cluster or component to successfully update. The cluster-version operator will allow updates when this condition is not False, including when it is missing, True, or Unknown.

So we specifically doc it as only about 4.y -> 4.(y+1) minor updates when seen on ClusterOperator.  But we leave it unclear on ClusterVersion because when you set some ClusterVersion overrides, it can break patch updates, so QE asked us to also block patch updates in that case.
With this patch, I'm using availableUpdates and conditionalUpdates to look up a version associated with the proposed target release pullspec.  That's a bit less reliable than the current cluster-version operator behavior, which is extracting the proposed target version from the proposed release image itself (e.g. see here).  But it's probably sufficient for now, with the odds that the OpenShift Update Service serves bad data low.  And we can refine further in the future if we want.

superseded by https://github.com/openshift/hypershift/pull/2381
/close


Unable to add Support for CMM

hi I was making a mod that allows multiple menus. I successfully overriden yours on startup but, you call during post gui clear the buttons which is making it super difficult as your mod is really stuborn. I will allow people still to configure their cmm menu I just want them to have scrolling through multiple menus.
What I am currently able to do:
Override CMM completly and have nothing but, default menu array show up or simply have the ability to browse through your menu and then have the browsing buttons disappear
Things you could do:
set your event priorities to only be high or low not highest or lowest buttons shouldn't be appearing that isn't yours since you don't extend GuiMainMenu. If I was to code It I would keep only one of those events preferably high since low can get overriden. You also don't extend GuiMainScreen which is what mods unless specifically adding support are looking for so I found the button clearing completely unnecessary.
let your hash map be static in config class since only one instance of your mod is ever going to run had to go through alot of reflection just to add your menu back once I overriden yours
Don't create instanceof gui fake menu ever keep one gui this is unnecessary keep your custom configed one
Why:
because multiple menus are cool and I will be sure to add and I will be sure to add a config to disable menu browsing if people don't want it on

Well i don't remember why i changed it so my menus don't extend GuiMainMenu anymore but i'm guessing there was a reason for it^^.

You also don't extend GuiMainScreen which is what mods unless specifically adding support are looking for so I found the button clearing completely unnecessary.

That's what GuiFakeMain is for, mods are adding their buttons into that so i can "collect" them in the LOWEST init gui event handler.
In my Event Handler there is a field called CMMEventHandler.actualGui, if you add the switching buttons there they shouldn't get removed.
Also just to understand what you are making, does your mod allow you to switch between only CMM menus or also other menus that you made?

Hi thank you for responding:
My mod allows for switching between different menus and saves the current index so yours can and will have to be overridden:
What I would like you to do is:
Set your priority on your events to only be high and low:
Allow your menu to get overriden more easily and to render all buttons in your list.
I don't want to be a depeandancy on yours so could you please not remove any buttons past a certain point as a work around for mine? It allows for menus like this. It's also configurable what menus will and won't show up but, the orignal user has to code a whitelist if a menu has to be removed.
Config: https://pastebin.com/zeHshvic
Screenshot:

I know one main issue: You should set your gui fake menu to only extend GuiScreen and at the low or lowest override if it's equal to your class and only if. Take a look at my event class as an example. It should have the same overrides capability and you can lower your event to high. Because mods are going to check gui main menu as well unless trying to specifically override your main menu.
What you do now:
set the gui that already extends guiscreen
on lowest check if instnaceof gui main menu

merging issue with latest


Callout namespaces/pods peers do not include host-net pods

As discussed in sig-network-policy-api meeting 10th October
/assign @astoycos @Dyanngg

/lgtm


[stable/spinnaker] ensure spinnaker works behind nsx-t ingress

VMware NSX-T ingress requires a /* pattern for the path, this allows the user
to specify the type of ingress controller via annotations to ensure
correct behavior.
Signed-off-by: Christopher Banck<EMAIL_ADDRESS>Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/ok-to-test

/lgtm