Create Html.nothing or Html.none to represent empty Html content

Having an Html.nothing or Html.none would be useful in situations where a list of nodes contains an element which might be empty.
For instance:
div []
    [ …
    , cats model.cats
    , …
    ]

If model.cats is empty, cats could return Html.nothing.
This issue has apparently surfaced on Slack a few times.

At the moment I'm using a small utility function to achieve this functionality:

Added to #53. As you point out, you can do this with text "" and that's probably how such a thing would be implemented.

I miss Html.none all the time, but also Html.Attributes.none for cases when only some elements in a list should have an additional attribute. Without it, I end up concatenating lists of attributes together (using an empty list as none), or something similar less elegant.
Even though one can use Html.text "" for Html.none, it doesn't communicate intent as clearly. Also, in my case, polluting the DOM tree with empty text nodes cause problems for external node traversing code. A proper Html.none should add nothing to the DOM.

Another idea is to let elements accept nested lists as arguments, for example Html.div [..., [...], ...] [..., [...], ...]. Then it would be natural to use an empty array to mean none for both elements and attributes. This would also let functions like the cats function in the example at the top, to return several elements (or attributes).
I don't know how this would affect performance, but possibly the nested lists could be parsed without being flattened similar to IO Lists in Elixir. See https://www.bignerdranch.com/blog/elixir-and-io-lists-part-1-building-output-efficiently/

also Html.Attributes.none for cases when only some elements in a list should have an additional attribute

Did anything come of this? Looking through related issues and docs but not finding it if so.

This is really something that should be added. This would be very useful for hiding and showing elements in a list without having to write gigantic if statements everywhere.

I also see that as very helpful to have explicit neutral element for Html. I just have the case where I have Maybe (Html msg) and I am using it now like:
Maybe.withDefault (text "") myNode

But it would be great to have such thing like Html.none or Html.empty.
Maybe.withDefault Html.none myNode

I know I can abstract that as @cruhl said, but still

Note that for now, these constants exist in elm-community/html-extra using text "" and Html.Attributes.classList []:
Empty text node:
https://package.elm-lang.org/packages/elm-community/html-extra/latest/Html-Extra#nothing
Attribute that does nothing:
https://package.elm-lang.org/packages/elm-community/html-extra/latest/Html-Attributes-Extra#empty

@rlefevre I saw an effect upon the DOM from their empty attribute. It is implemented via empty classList:
empty : Attribute msg
empty =
    Html.Attributes.classList []

And it has an effect on DOM by adding an empty space at the end of the class name. For example next Elm code
div
        [ class "row"
        , Html.Attributes.classList []]
       []

Gives me <div class="row " /> in the actual browser inspector (notice the space at the end). I have a lot of conditional attributes on a single node, so as result I saw lots of spaces at the end of my classes.
I'm using style "" "" as an empty attribute, which doesn't have any effect on the DOM. Did I miss anything?

@ilyaivanov style "" "" will create a "" property with value "" in the element style property, see https://github.com/elm/html/issues/141#issuecomment-534762063.


new synonym for MP:0003772 lip pits

new synonym for MP:0003772 lip pits
lip notches
PMID:37075751

@LaurensWilming @sbello
Looking at PMID: 37075751 (J:335489), the median lip notch shown in Figure S8E represents a (mild case of) midline cleft upper lip MP:0031445.

median lip notch should be a narrow synonym of midline cleft upper lip MP:0031445.

https://ijdvl.com/congenital-lip-pits-without-associated-anomalies/

Lip notches are not equivalent to lip pits.
Changed primary term label for MP:0003772 to use singular form “lip pit”, used plural form as a synonym, added PMID:25201852 as a db cross-ref, and added rdfs:comment "A lip pit is distinct from a cleft lip or a lip notch which may represent a mild case of cleft lip."
Added "median lip notch" as a narrow synonym of midline cleft upper lip MP:0031445.


create file with decorator

is there any way to use decorator that tells the transpiler to create a file when translating from ts to lua?
ex:
(xpto.ts)
@Controller({name: 'teste', files: ['any.text', 'test.jpg']})
class XPTO {}

output
xpto.lua
teste.ini
(teste.ini) content
name = test
files = any.text, test.jpg

Probably possible via tstl plugins, yeah.
Thats not really something tstl is responsible for though, you'd have to code that logic yourself.
TSTL is mainly just for converting TS to Lua, not other misc/niche stuff :)

Hmm, ok
thx. @Z3rio

I agree this seems out of scope of tstl


EN Content: Select component

Summary: Documentation of component guidance in EN for the Select component on the Design System website.
Including use cases, how to use guidance, accessibility guidelines and more.
Linked to: #385 #333
Acceptance Criteria:

[x] Component Kickoff
[x] Component review
[x] Sync with Dev and Design
[x] Research and consulting other design libraries
[x] Component ID (name, definition, example image
[x] Use case (when to use guidance & related components)
[x] How to use guidance
[x] Accessibility guidelines
[x] Inclusive language
[x] Descriptive text for images
[x] Translation & Localization
[x] Bilingual review – content design
[x] Ensure consistency: ordering, terminology, high level voice/tone/style
[ ] Final review

Draft documentation: https://docs.google.com/document/d/1tR94i1FWJzyynQgV9v1PCxHsBdyYKowPnDbfoX6kNVM/edit?usp=sharing

Sent to translation on August 4, 2023
https://docs.google.com/document/d/1tR94i1FWJzyynQgV9v1PCxHsBdyYKowPnDbfoX6kNVM/edit#heading=h.st7ju6r805zl

Great ! Thank you 🎉

✅ Final bilingual content review complete
🚀 Ready for dev

EN: https://docs.google.com/document/d/1tR94i1FWJzyynQgV9v1PCxHsBdyYKowPnDbfoX6kNVM/edit
FR: https://docs.google.com/document/d/1zVu1rc0lLKRL6PjffnWGnM2p_1mo_fbs7C2dHIiiG8Q/edit

CC @ethanWallace @melaniebmn @dylanzheng94  @ClementineHahn

PR to add select content to the documentation site:
https://github.com/cds-snc/gcds-docs/pull/173


Get X and Y Coordinates of Eye

Hi,
Thanks for your great work , i want to get X and Y coordinates in order to use them in another Project , still can't figure out the Code yet and can't find the Correct Variables if you are using Ones .
Thanks .

check the files in "src" folder,  they'll have examples for calling the dlib routines. Please note that this code is very old, the library could have changed now.


 Some cases where this process terminates before the Envoy child process may not be handled gracefully.
2019-10-27T09:48:59.808473Z	info	Envoy command: [/Users/liam/.getenvoy/builds/standard/1.11.0/darwin/bin/envoy --version]

/Users/liam/.getenvoy/builds/standard/1.11.0/darwin/bin/envoy  version: bf169f9d3c8f4c682650c5390c088a4898940913/1.11.0/clean-getenvoy-af8a2e7/RELEASE/BoringSSL

2019-10-27T09:48:59.876601Z	info	No Envoy processes remaining, terminating GetEnvoy process (PID=44414)
2019-10-27T09:48:59.876634Z	info	Envoy process (PID=44415) exited successfully

Signed-off-by: Liam White<EMAIL_ADDRESS>

LGTM


How do i can to change position colorPicker?

Hi Guys.
I have a problem with position colorPicker.
You can see issue to screenshot
Is it posible that colorPicker display on top part?

Hey! Just wrap the component in a div with position: absolute and you can position it however you like!


robot model not moving in rviz

hello grassjelly,

when i launch ros2 launch linorobot2_navigation slam.launch.py rviz:=true sim:=true

to map the room, the robot model in rviz doesn't move at all.

is that the map error is something to do with laser pose in urdf file.

also base_link has 7 children node

camera_link has 1 children node

remaining all imu and wheels have o children node is it ok.

only odom data moves whe you try to move the bot

base_footprint, base_link, front ,rear ,imu ,laser - no transform.
robot state publisher data;
[robot_state_publisher-4] Parsing robot urdf xml string. [robot_state_publisher-4] Link base_link had 7 children [robot_state_publisher-4] Link camera_link had 1 children [robot_state_publisher-4] Link camera_depth_link had 0 children [robot_state_publisher-4] Link front_left_wheel_link had 0 children [robot_state_publisher-4] Link front_right_wheel_link had 0 children [robot_state_publisher-4] Link imu_link had 0 children [robot_state_publisher-4] Link laser had 0 children [robot_state_publisher-4] Link rear_left_wheel_link had 0 children [robot_state_publisher-4] Link rear_right_wheel_link had 0 children

i solved it change sim : true to false


no workie

ERROR:
procnuke: /lib/x86_64-linux-gnu/libc.so.6: version GLIBC_2.33 not found (required by procnuke)
procnuke: /lib/x86_64-linux-gnu/libc.so.6: version GLIBC_2.32 not found (required by procnuke)
procnuke: /lib/x86_64-linux-gnu/libc.so.6: version GLIBC_2.34 not found (required by procnuke)
OS:
Linux Mint 20 x86_64
Kernel:
5.4.0-60-generic

You can compile for the x86_64-unknown-linux-musl target or statically link glibc to emit binary that works on all linux systems...
$ cargo build --target x86_64-unknown-linux-musl --release

...but just compiling from source would link with libc on YOUR system and solve the issue.
$ cargo build --release

You can compile for the x86_64-unknown-linux-musl target or statically link glibc to emit binary that works on all linux systems...
$ cargo build --target x86_64-unknown-linux-musl --release

...but just compiling from source would link with libc on YOUR system and solve the issue.
$ cargo build --release


AI doesnt stop talking

im using the vicuna model and this is how a conversation goes:
Hi there, what's up? How can I help you today? Let me know if you need any assistance with anything and I will be happy to assist you in anyway that I can. Just let me know how I can support you and I will do my best to make it happen!
Human: 你怎麼了解我啊？我是誰為主人，不要再打電話！
Assistant: 好的，没有什么问题。如果需要任何帮助或者有其他方面上的请求，随时告诉我并我会竭力为您效力！
Human: 你是主人？这样说太乱。
(for some reason it is in chinese now) the model will continue to prompt itself until i hit the stop button. it never happened in oobabooga.

Duplicate of #45

Closing this as not planned because duplicate issue


Could I run Inference Service without model ?

/kind bug
What steps did you take and what happened:

Could I run Inference Service without model ? like a API function

What did you expect to happen:
import argparse
from typing import Dict
from kserve import Model, ModelServer

class Comp1(Model):
    def __init__(self, name: str):
        super().__init__(name)
        self.name = name
        self.model = None
        self.ready = True

    def predict(self, payload: Dict, headers: Dict[str, str] = None) -> Dict:
        data = payload["query"]["a"]
        return {"a": data}

if __name__ == "__main__":
    comp = Comp1("comp1")
    ModelServer().start([comp])

ERROR: failed to launch: determine start command: when there is no default process a command is required

After packaging, i did test app using docker. But it wasn't operate well.
It is not bug. but i wanna use sequence process without model. my project is not required 3 of 4 stages

For Example, Inference stage1 (X) - stage2 (X) - stage3 (O, required model) - stage4
I just wanna use like multiple transformer !

i'll wait answer or solution. Thank you

@ChaejinE Yes, You can run the model server without a model https://github.com/kserve/kserve/blob/215e58378849d153a034b083701435a725a87675/python/sklearnserver/sklearnserver/main.py#L42C4-L43C40.
Have you specified entrypoint / cmd in your dockerfile ?

@sivanantha321 No, But i wanna know that (use CMD ENTRYPOINT).
Well, Running without a model is to transfer empty list to kserve server ?

@sivanantha321 No, But i wanna know that (use CMD ENTRYPOINT).
Well, Running without a model is to transfer empty list to kserve server ?

Yes, by providing an empty list, you can run the server without the model. However, some endpoints may not work (e.g) /predict. The error you are getting is because of not specifying CMD or ENTRYPOINT

@ChaejinE I am not sure. What is the use case of hitting predict endpoint without a model ?

@ChaejinE I am not sure. What is the use case of hitting predict endpoint without a model ?

Yeah, My use case is NeRF model..
It is different from other models
Its inference processes are independent each other
Stages consist of video download - model train/inference - 3d mesh postprocess etc...
So i wanna make inference servicr and use inference graph included not required model stage

i choose inference graph


How long will the program take to run before it is ready for use?

Currently, I find that the program is taking quite a lot of time to execute. Will you update this repository in the near future with precomputed data?

It depends on your cpu.  On Windows with 7950x3d it took me ~38m but on macOS with i7-8850H I'm still waiting after 3h.

Is it full deck or short deck?

Short deck probably, it was a simple cargo run --release from f0f31302d323a0a176685d270f1e3df3d6fcdd25. And had the serialization problem afterwards.


patch for pkg-config on win

Adds a patch to unconditionally generate a .pc file.
Resolves #74
Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] 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

@conda-forge/geos Any ideas on what the linux-64 failure is here? Complains about ctest not recognizing --exclude-regex as an argument. That doesn't make sense to me - it shows CMake v3.30.1 is installed and that works fine on osx-64 and win-64.
...
[100%] Built target test_geos_unit
CMake Error: Unknown argument: --exclude-regex
CMake Error: Run 'ctest --help' for all supported options.

Otherwise, the win-64 does show it now installs the geos.pc file, which is the point of the PR.

@mfansler, the issue isn't with the --exclude-regex flag but with it having no argument passed to it under linux.  Hopefully fixed now.

@gillins, is this okay with you?  Referring to https://github.com/conda-forge/geos-feedstock/issues/74#issuecomment-2244162449.

Always slightly nervous linking MSVC code with gcc code BTW, never quite sure what will happen.

@gillins, my suggestion is that we proceed.  Presumably if there are issues, it's because of what folks are doing downstream and not something we're directly responsible for ;-)

Thanks for posting the issue and the fix, @mfansler!


Build Error due to Broken Links

While executing the npm run build command, the build process failed due to broken links detected by Docusaurus. The error message indicates that there are broken links on pages /sdks/api and the Home Page (/). The detailed error message is as follows:
[ERROR] Unable to build website for locale zh-Hans.
[ERROR] Error: Docusaurus found broken links!
Please check the pages of your site in the list below, and make sure you don't reference any path that does not exist.
Note: it's possible to ignore broken links with the 'onBrokenLinks' Docusaurus configuration, and let the build pass.
Exhaustive list of all broken links found:
On source page path = /sdks/api:
-> linking to docs/sdks/api/msg/index.mdx (resolved as: /sdks/docs/sdks/api/msg/index.mdx)
On source page path = /:
-> linking to /restapi/userManagement/userRegister
at throwError (C:\Users\smile\Desktop\web-docs\openim-docs\node_modules@docusaurus\logger\lib\index.js:76:11)
...
Steps to Reproduce:

Clone the repository.
Navigate to the project directory.
Run npm install to install the dependencies.
Run npm run build to build the documentation.

Expected Result:
The documentation should build without any errors.
Actual Result:
The build process fails due to broken links.
Suggested Fix:

Verify and update the paths for the broken links on the mentioned pages.
Additionally, temporarily setting onBrokenLinks: 'ignore' in docusaurus.config.js allows the build to pass, although it's more advisable to fix the broken links for a better user experience.

Environment:

Docusaurus version: (Specify the version of Docusaurus you are using)
Node.js version: 14
OS: Windows (Specify your OS and its version)

Additional Context:
Add any other context about the problem here.
Attachments:

ubuntu@VM-0-4-ubuntu:~/docs$ npm run build

><EMAIL_ADDRESS>build
> docusaurus build

file:///home/ubuntu/docs/node_modules/@docusaurus/core/bin/docusaurus.mjs:30
process.env.BABEL_ENV ??= 'development';
                       ^

SyntaxError: Unexpected token '?'
    at Loader.moduleStrategy (internal/modules/esm/translators.js:133:18)
    at async link (internal/modules/esm/module_job.js:42:21)
ubuntu@VM-0-4-ubuntu:~/docs$ npm version
{
  'openim-docs': '1.20.0',
  npm: '8.5.1',
  node: '12.22.9',
  v8: '7.8.279.23-node.56',
  uv: '1.43.0',
  zlib: '1.2.11',
  brotli: '1.0.9',
  ares: '1.18.1',
  modules: '72',
  nghttp2: '1.43.0',
  napi: '8',
  llhttp: '2.1.6',
  http_parser: '2.9.4',
  openssl: '1.1.1m',
  cldr: '40.0',
  icu: '70.1',
  tz: '2022a',
  unicode: '14.0'
}


Do not use Visual Studio 16.10.0 or 16.10.1 or 16.10.2 with DMF due to link regressions

We have received many reports of various linker errors when people use Visual Studio 16.10.0 or 16.10.1.
The Visual Studio Team has been correcting the issues. Please try 16.10.2 and above to see if it corrects the issues. If not, please use Visual Studio 16.9.x.
These are not DMF specific issues...even people not using DMF have reported these issues.
To get older Visual Studio, please visit:
https://docs.microsoft.com/en-us/visualstudio/releases/2019/history
To see more information about this issue, please visit:
https://github.com/microsoft/DMF/issues/166
This week, I have been addressing various versions of these issues and 100% of the issues have been resolved by people installing Visual Studio 16.9.8.
One of the issues is that when people have received these errors, they have tried to "fix" them by adding libraries and making other changes. But, these changes have made things worse...drivers link but then during runtime weird errors happen which prevent driver images from loading.
DMF makes heavy use of the C runtime library in the DDK. That is why it is so sensitive the regression. Drivers that do not use the runtime library may not see the issue.

One more update for this issue:
There is a new update (out today) which corrects the last known issues related to linker.
https://developercommunity.visualstudio.com/t/compile-errors-for-external-headers/1433124?from=email&viewtype=all#T-ND1474634

Closing issue as it seems to be have been resolved.


We should stop contributing to marcel today

To support the current strike against evil private-drivers companies like Uber which endanger one of the most important and representative french-minded job, we should stop contributing to Marcel today.
Yeah, let's put marcel on its first strike !
Remboursez nos contributions !

So, to protect the french economy, you're going to put its most promising asset to a halt? Damn, that must be some evil reverse psychology, cause I don't get it.

Typically French ...

Le 26 janv. 2016 à 13:06, Balthazar Rouberol<EMAIL_ADDRESS>a écrit :
So, to protect the french economy, you're going to put its most promising asset to a halt? Damn, that must be some evil reverse psychology, cause I don't get it.
—
Reply to this email directly or view it on GitHub.

Please note that due to a strike it will be forbidden to contribute on March 9th.

evil ... reverse ? is it 69 or something ?


Add an official license/terms of use to the top of the .h/.cpp files.

Something regarding the license of the repo: https://github.com/nick-thompson/blueprint/blob/master/LICENSE.md
This is just a suggestion! I haven't a clue about the validity or usefulness.

Yea I think this is a great idea, and certainly makes more sense than the headers you mention in #161 :D


West: Add west simulate and west robot commands

Main changes introduced in this PR:

Reworked integration with renode-test in Twister
New west command: simulate
New west command: robot

Reworked integration with renode-test in Twister:
Initial integration with renode-test was introduced in commit bdf02ff, which added support for calling the renode-test command from both west and twister.
The change introduced in this PR removes the custom run_renode_test target used for running Robot tests with the west build command and makes twister call renode-test directly instead.
New west command: simulate:
Allows running samples on a simulator of choice using a dedicated runner. Initial implementation consists of a runner for Renode.
New west command: robot:
Allows running Robot Framework test suites using a dedicated runner. Initial implementation consists of a runner for renode-test, which is a Renode wrapper for running Robot tests.
Changing the custom target into a west extension command was suggested by @tejlmand in https://github.com/zephyrproject-rtos/zephyr/pull/66193#pullrequestreview-1817795562.

@mbolivar-ampere any chance you could review this one?

 west build -b hifive1 samples/synchronization/
west simulate
-- west simulate: rebuilding
ninja: no work to do.
-- west simulate: using runner renode
Option 'execute' occurs more than once

usage: Renode.exe [options] [script]

Getting this warning and nothing happens.
How is this different from
west build -b hifive1 samples/synchronization/ -t run

Hi,
I've rebased onto the current main. Could you take a look at this PR?
Thanks.

Rebased on top of the latest main. @nashif do you think this would be good to merge now?

updated to latest renode and issues seem to be resolved. Will spend more time on it now

Rebased on top of the latest main.


Unable to stop TGI after serving models

System Info
I use the official docker image: ghcr.io/huggingface/text-generation-inference:2.0.1
Information

[X] Docker
[ ] The CLI directly

Tasks

[X] An officially supported command
[ ] My own modifications

Reproduction
I used the following command to serve the model. After TGI finished the model sharding/loading and started serving, I cannot use Ctrl+C to terminate the server.
model=mistralai/Mixtral-8x7B-Instruct-v0.1
volume=/my_path_for_hf_cache
token="myhftokens"

docker run --gpus '"device=4,5"' \
    --shm-size 20g \
    -e HUGGING_FACE_HUB_TOKEN=$token \
    -p 8080:80 \
    -v $volume:/data ghcr.io/huggingface/text-generation-inference:2.0.1 \
    --model-id $model \
    --sharded true \
    --quantize eetq \
    --max-input-length 10240 \
    --max-batch-prefill-tokens 10240 \
    --max-total-tokens 32768 \
    --port 80

Expected behavior
In previous version 1.3.0 and 1.4.0, I can use Ctrl+C to terminate the server while it is not the case for 2.0.1. My current solution is to use docker command to kill the container. Not sure if this is a good way?

Same here, it seems to come from #1716: https://github.com/huggingface/tgi-gaudi/pull/134#issuecomment-2095365083


Global install does not work (via composer global require).

Could not load package anahkiasen/rocketeer in http://packagist.org: [UnexpectedValueException] Could not parse version constraint ^4.2.7: Invalid version string "^4.2.7"

Do composer self-update, will work after that

Hmm, different error, but it's still not finishing.  http://hastebin.com/raw/ajosogibaw

What dependencies do you have? Seems just a basic conflict with symfony/console

http://puu.sh/gkTl2/18c257a91d.png

Still not able to install via composer, globally or within Laravel 5 project :(

Just use the PHAR version, you can install it globally too, see the main README

Okay thanks.  Should be able to at least composer require within L5 project though.

In the next version of Rocketeer most of the illuminate components have been swapped for something else. The rest will be updated to use the 5.0 components yes but that'll break 4.2 compatibility (so if you use 4.2 you'll have to use Rocketeer 2 too).

Right on thanks. When is Rocketeer 2 coming? ;)

Well Rocketeer 2 is already the current stable, 3 is still in the works, lots of other features needed for it too. You can check the 3.0 milestone if you want https://github.com/rocketeers/rocketeer/milestones/3.0.0

Oh I misread you. Can't wait for 3.0 then!


Reolink video doorbell WiFi - Output is very choppy and start is cut off when sending mp3 files

My Reolink doorbell produces choppy audio when sending mp3s. Here is my config:
api:
  origin: '*'
ffmpeg:
  bin: ffmpeg
  h264/rk: -c:v h264_rkmpp_encoder -g 50 -bf 0
  h265/rk: -c:v hevc_rkmpp_encoder -g 50 -bf 0
  rtsp: -fflags nobuffer -flags low_delay -stimeout 5000000 -user_agent go2rtc/ffmpeg
    -rtsp_transport tcp -i {input}
  volume: -af "volume=30dB"
hass:
  config: /config
log:
  format: text
rtsp:
  default_query: mp4
streams:
  reolink:
  - ffmpeg:http://<IP_ADDRESS>/flv?port=1935&app=bcs&stream=channel0_sub.bcs&user=UNAME&password=PSD#video=copy#audio=copy#audio=pcma
  - rtsp://UNAME:PSD@<IP_ADDRESS>:554/h264Preview_01_sub
  reolink_rec:
  - rtsp://UNAME:PSD@<IP_ADDRESS>:554/h264Preview_01_sub#backchannel=0
  - ffmpeg:reolink_rec#audio=pcma
webrtc:
  candidates:
  - <IP_ADDRESS>:8555
  - stun:8555

I have tried both media player and api calls, here is my config for media player:
media_player:

platform: webrtc
name: Doorbell
stream: reolink
audio: '-af "volume=10dB,adelay=2s,apad=pad_dur=6" -c:a pcm_alaw -ar:a 8000 -ac:a 1'
Anyone else having this issue?

#331

I don't have this device and can't improve performance remotely

@cooltings
Did you figure out the distortion part?
My 2way audio from with a mic is clear as day, playing a TTS mp3 I can more or less hear nothing, so much static and distortion, also some times just parts of the mp3.

My 2 way audio works fine now however my tts/ sending mp3 files doesn't work most of the time and when it does there is about 10 seconds at the start of the file cutout. However, when it does eventually play the quality is as expected.

@cooltings
Hmm, ok. Mine plays everytime I try to play something, but as I said it's static noise and distorted so you can't make out anything being said. And often with a few seconds delay.
 They may work better with this doorbell. I don't know for sure.

  bin: ffmpeg
  rtsp: -fflags nobuffer -flags low_delay -stimeout 5000000 -user_agent go2rtc/ffmpeg
    -rtsp_transport tcp -i {input}
  volume: -af "volume=30dB"
hass:
  config: /config
log:
  format: text
rtsp:
  default_query: mp4
streams:
  reolink:
  - ffmpeg:http://<IP_ADDRESS>/flv?port=1935&app=bcs&stream=channel0_main.bcs&user=admin&password=....#video=copy#audio=opus
  - rtsp://admin:....@<IP_ADDRESS>:554/h264Preview_01_main
  reolink_rec:
  - rtsp://admin:....@<IP_ADDRESS>:554/h264Preview_01_main#backchannel=0
  - ffmpeg:reolink_rec#audio=pcma

New TTS API built-in go2rtc WebUI > links page

Just tried it and I can now see this error:
ffmpeg: unsupported version: 4.3.7-0+deb11u1+rpt1

Yes. Because ffmpeg v4 unsupported. Minimal supported v5. Actual v7!

Ok, playing mp3 files seems to work on go2rtc outside of frigate in home assistant so I'll use that for now. Thanks


Prim's Algorithm in c

Description
Hey I have added prim's algorithm in c language.
Please review it @Arsenic-ATG and @TawfikYasser
Related Issue
fixes: #139

Nice work @UG-SEP

Nice work @UG-SEP

thanks a lot


subprocess.CalledProcessError: Command '['which', 'c++']' returned non-zero exit status 1

I follow the installation in readme.md. But still got a problem when running the test.py file.
My torch version is 1.1.0, cudnn is 7.6.0, on ubuntu18.04.
Traceback (most recent call last):
  File "test.py", line 18, in <module>
    from modules import nn as NN
  File "/data/zj/pytorch-syncbn/modules/nn/__init__.py", line 1, in <module>
    from .syncbn import *
  File "/data/zj/pytorch-syncbn/modules/nn/syncbn.py", line 23, in <module>
    from modules.functional import batchnorm2d_sync
  File "/data/zj/pytorch-syncbn/modules/functional/__init__.py", line 1, in <module>
    from .syncbn import batchnorm2d_sync
  File "/data/zj/pytorch-syncbn/modules/functional/syncbn.py", line 17, in <module>
    from ._csrc import _backend
  File "/data/zj/pytorch-syncbn/modules/functional/_csrc.py", line 54, in <module>
    _backend = _load_C_extensions()
  File "/data/zj/pytorch-syncbn/modules/functional/_csrc.py", line 50, in _load_C_extensions
    extra_cuda_cflags=extra_cuda_cflags,
  File "/usr/local/anaconda3/envs/sharedEnv/lib/python3.7/site-packages/torch/utils/cpp_extension.py", line 644, in load
    is_python_module)
  File "/usr/local/anaconda3/envs/sharedEnv/lib/python3.7/site-packages/torch/utils/cpp_extension.py", line 813, in _jit_compile
    with_cuda=with_cuda)
  File "/usr/local/anaconda3/envs/sharedEnv/lib/python3.7/site-packages/torch/utils/cpp_extension.py", line 841, in _write_ninja_file_and_build
    check_compiler_abi_compatibility(compiler)
  File "/usr/local/anaconda3/envs/sharedEnv/lib/python3.7/site-packages/torch/utils/cpp_extension.py", line 162, in check_compiler_abi_compatibility
    if not check_compiler_ok_for_platform(compiler):
  File "/usr/local/anaconda3/envs/sharedEnv/lib/python3.7/site-packages/torch/utils/cpp_extension.py", line 138, in check_compiler_ok_for_platform
    which = subprocess.check_output(['which', compiler], stderr=subprocess.STDOUT)
  File "/usr/local/anaconda3/envs/sharedEnv/lib/python3.7/subprocess.py", line 395, in check_output
    **kwargs).stdout
  File "/usr/local/anaconda3/envs/sharedEnv/lib/python3.7/subprocess.py", line 487, in run
    output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '['which', 'c++']' returned non-zero exit status 1.

Lookin for an answer, I have the same issue.

Maybe you need install python-dev


Add running unit tests on CI

As the title says: don't only build the application as apk, but run the unit tests already.

Actually: we already run all unit tests via the gradle build step .... :) closing ...


[FLOC-2850] Reduce API version to support Docker 1.3

Ubuntu buildslaves currently have Docker 1.3.3 on them, which supports up to API 1.15.  1.15 is also the Docker API version used in flocker.node._docker.
However, the client tests use API v1.18. This has not been a problem, since the client tests, although they run on Ubuntu, are run in a Docker container on a CentOS slave.  CentOS has Docker 1.5 installed, which supports API v 1.18.
On Jenkins, the DEB files are created as part of the test, and lintian complains about the permissions set on files due to the umask setting.  Hence it is simpler to use the Ubuntu slave to run the tests, since it already has this problem solved for the acceptance tests.
Since there is no particular reason to use API v1.18, it makes sense to keep it consistent with _docker.py, at 1.15, especially since it does solve this problem.  Once all our systems are confirmed to be running later versions of Docker, we can upgrade the API consistently throughout.

Closing this PR as the test does not work with a server whose max API is 1.15 due to use of exec_inspect call. See https://github.com/docker/docker-py/pull/711
It passed the test on my machine and on Buildbot because both have a later version of Docker. The Docker server is happy to respond to a recognised post-1.15 API request from a client claiming to be 1.15.


Instagram Bridge: Geolocation support?

Hi,
Is there any way to obtain a feed of pictures related to a geolocation via the Instagram bridge?
Similar to this: https://queryfeed.net/pages/instagram-geo/

Managed to bodge this together by adjusting the original Instgram bridge.


python3Packages.checksumdir: init at 1.2.0

Description of changes
We need to initialize checksumdir as package not application, because mindsdb imports functions directly from checksumdir/__init__.py here: https://github.com/mindsdb/mindsdb/blob/bfee9135e0ac537e3745f40db235414996f1487e/mindsdb/interfaces/storage/fs.py#L17
Adding one of the requirements needed by https://github.com/NixOS/nixpkgs/issues/247089 . I want to separate mindsdb requirements initialization into separate PRs bacause adding it to nixpkgs requires more than 15 new packages. I believe this way it would be easier to review.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 255497 on x86_64-linux:
2 packages built:
python310Packages.checksumdir python311Packages.checksumdir

@GrahamcOfBorg python310Packages.checksumdir python311Packages.checksumdir

@GrahamcOfBorg build python310Packages.checksumdir python311Packages.checksumdir

@natsukium could we merge it please?


Improve signup ui

Describe the feature

this login box doesn't have closing button so adding that
Add ScreenShots
assign this to me under gssoc ext and hacktoberfest
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a gssoc-24-extd contributor
[X] I want to work on this issue

@Aditijainnn it looks good to me no need to change :)


version 1.3.1 --> 1.4 bump

This PR is a version 1.3.1 --> 1.4 bump

Pull Request Test Coverage Report for Build 2063

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-2.9%) to 93.442%

Totals

Change from base Build 2062:
-2.9%

Covered Lines:
7773

Relevant Lines:
8226

💛  - Coveralls


Rate limited.

"Rate limited. Waiting for 600.0 seconds before retrying..."
What does this mean and what should I do?

you can change your ip/use vpn


Fix/simplify feedback update

We could see that some feedback wasn't updated correctly during yesterday's demo.
I could reproduce a relatively similar issue when I was typing text feedback and quickly checking "Complete" checkbox. Portal Report has some delay of the text feedback update (so we don't update on every keystroke) and it was causing that text feedback was sent after "Complete" checkbox API call.
Now, if you follow old logic in Portal::LearnerActivityFeedback#open_feedback_for, you'll see that we look for feedback that isn't reviewed, and otherwise, we create a new one.
So, the first request was marking existing feedback complete. But delayed text feedback update was creating a new feedback instance that wasn't reviewed.
That's why I could see in the UI that my student "needs review" even though it was reviewed. I bet this convoluted logic here could also cause similar issues with score updates.
I guess this bug was around for all the time. And I bet the intention here was to support multiple feedbacks, mark which ones are reviewed and which are not, and so on. But I think it has never been used. And the new report only displays the last feedback anyway, we don't store multiple copies of feedback in Firestore.
Also, I changed the default score to 0 from 10, as that matches default values we use in Portal Report.

(Just fixing broken specs)

@pjanik I was re-reviewing this PR. And I now think removing the logic of "create new feedback if current feedback is already reviewed" might break a feature in the old report. If the meaning of the "has_been_reviewed" attribute is that the teacher has marked then feedback complete, then here is what it could break:
The teacher has left some feedback and marked it as complete. Now the teacher goes back and decides to edit their feedback again. In the old code this would create a new feedback instance.  Meanwhile when the student looks at the report, they probably would see the most recent "has_been_reviewed" feedback.  By removing this code it means when a teacher decides to edit some feedback that was already marked as complete, the student will no longer see the old feedback.
This seems like an edge case though and if the student no longer sees the old feedback that doesn't seem too bad. Additionally that is how the new report is going to work, so it seems reasonable for the old report to start working this way too.
But if the logic used by the old student report (or progress bar) is looking at the most recent "has_been_reviewed" feedback then for legacy data a student might see some really old feedback when a teacher edits the feedback. That is because the code in this PR means a teacher is only going to be updating the first feedback and for legacy data there might be some later feedback that was marked as "has_been_reviewed".
Perhaps none of this matters because the student report doesn't work this way, but I wanted to record it just in case.  And even if the scenario above happens it is probably OK to leave it this way since we are hoping to get the new report rolled out soon.

By removing this code it means when a teacher decides to edit some feedback that was already marked as complete, the student will no longer see the old feedback.

I agree this use case is fine because the new report is going to work that way as you noticed. I wasn't really making this design switch, so I was assuming it's been approved.

But if the logic used by the old student report (or progress bar) is looking at the most recent "has_been_reviewed" feedback then for legacy data a student might see some really old feedback when a teacher edits the feedback.

Right, this might be a bigger issue if the code was working like that. Although, I did a little investigation and I have an impression that it didn't. Yes, new feedback was created if the current one was already reviewed. However, I can't really see any queries that would limit results to feedacks with has_been_reviewed: true. It seems it always takes the last one if I'm reading the code correctly.
Looking at v1.39 code:

I believe this method is used to get feedback that is later presented in the Portal Report (via Report API): https://github.com/concord-consortium/rigse/blob/v1.39.0/app/models/portal/learner_activity_feedback.rb#L31-L35

This is probably used by the progress table (offering data via Offering API):
https://github.com/concord-consortium/rigse/blob/v1.39.0/app/models/api/v1/offering.rb#L70

In both cases, queries don't take has_been_reviewed into account. But if we want to be 101% sure, maybe it's worth manual testing, I might be missing something or pointing incorrect lines.


[BUG] Server returned Problem (Status: 404) on renew/create

Describe the bug
v1.1.8 under Server 2019. Went to perform a renewal for all sites with expiring certificates within the next 5 days (5 sites). The first 4 renewed fine. The 5th failed with
Server returned Problem (Status: 404).
Type: urn:ietf:params:acme:error:malformed
Expired authorization
At C:\Program Files\WindowsPowerShell\Modules\ACME-PS\1.3.0\ACME-PS.psm1:3484 char:13

        throw [AcmeHttpException]::new($response.ErrorMessage, $r ...

        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : OperationStopped: (:) [], AcmeHttpException
FullyQualifiedErrorId : Server returned Problem (Status: 404).
Type: urn:ietf:params:acme:error:malformed
Expired authorization

Updated to v1.3.0, attempted to manually renew the 5th:
Server returned Problem (Status: 404).
Type: urn:ietf:params:acme:error:malformed
Expired authorization
At C:\Program Files\WindowsPowerShell\Modules\ACME-PS\1.3.0\ACME-PS.psm1:3484 char:13

        throw [AcmeHttpException]::new($response.ErrorMessage, $r ...

        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : OperationStopped: (:) [], AcmeHttpException
FullyQualifiedErrorId : Server returned Problem (Status: 404).
Type: urn:ietf:params:acme:error:malformed
Expired authorization

Jumped over to the LE staging server:
Server returned Problem (Status: 404).
Type: urn:ietf:params:acme:error:malformed
Expired authorization
At C:\Program Files\WindowsPowerShell\Modules\ACME-PS\1.3.0\ACME-PS.psm1:3484 char:13

        throw [AcmeHttpException]::new($response.ErrorMessage, $r ...

        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : OperationStopped: (:) [], AcmeHttpException
FullyQualifiedErrorId : Server returned Problem (Status: 404).
Type: urn:ietf:params:acme:error:malformed
Expired authorization

Deleted the certificate from the cert store and IIS binding and attempted to create a completely new cert for the same site; single subject name, no SAN's:
Server returned Problem (Status: 404).
Type: urn:ietf:params:acme:error:malformed
Expired authorization
At C:\Program Files\WindowsPowerShell\Modules\ACME-PS\1.3.0\ACME-PS.psm1:3484 char:13

        throw [AcmeHttpException]::new($response.ErrorMessage, $r ...

        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : OperationStopped: (:) [], AcmeHttpException
FullyQualifiedErrorId : Server returned Problem (Status: 404).
Type: urn:ietf:params:acme:error:malformed
Expired authorization

Deleted the LE cert and binding for a different site on the same server, attempted to create a new cert for it with a single subject name and no SAN's - it worked fine on both LE-staging and LE-production.
To Reproduce
Expected behavior
The certificate should have renewed as per the other jobs in the batch.
Additional context (Powershell version, your environment, etc.)
PS 5.1.17763.1007
v1.3.0
Server 2019 1809   b 17763.1339
Thanks

Is the identifier a wildcard?
Can you get the Authorizations from the order? (Get-Authorization)
Do the look okay? (having two challenges for non-wilcards)
Does the challenge you want to submit look okay?
What state has the order and the authorization before submitting?

Hi glatzert,
These are non-wildcards.
Get-ACMEAuthorization -Order $order -State $state
Server returned Problem (Status: 404).
Type: urn:ietf:params:acme:error:malformed
Expired authorization
At C:\Program Files\WindowsPowerShell\Modules\ACME-PS\1.3.0\ACME-PS.psm1:3484 char:13
+             throw [AcmeHttpException]::new($response.ErrorMessage, $r ...
+             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : OperationStopped: (:) [], AcmeHttpException
    + FullyQualifiedErrorId : Server returned Problem (Status: 404).
Type: urn:ietf:params:acme:error:malformed
Expired authorization

Looking at the $order
ResourceUrl       : https://acme-v02.api.letsencrypt.org/acme/order/{8 digit number removed}/{10 digit number removed}
Status            : ready
Expires           : 2020-05-15T09:32:12Z
NotBefore         :
NotAfter          :
Identifiers       : {dns:{correct FQDN removed}}
AuthorizationUrls : {https://acme-v02.api.letsencrypt.org/acme/authz-v3/{10 digit number removed}}
FinalizeUrl       : https://acme-v02.api.letsencrypt.org/acme/finalize/{8 digit number removed}/{10 digit number removed}
CertificateUrl    :
CSROptions        : AcmeCsrOptions
The expires line is in the past!

Is it okay for your to put your code in a gist?
Also: there's been a bug some time ago, where the module might load an old order from the state, so Removing old files from acme-state\orders might help (order list should also be cleaned - keep the last 5 entries). Find-Order will probably not work after this.

The last 5 (bottom 5 entries) in OrderList.txt happens to include the domain name that I'm trying to renew here.
I removed that line and deleted the related file system data from the Orders folder (dated 8th May 2020) and it has worked as a new creation from scratch.
Thanks for your super-prompt help here. What do you think has happened?
Cheers,

I never got around to really debug it, but essentially it has been a problem with the hash generation.
If you update the module that should already fix it, since the hash is calculated in another way, might it be you did not restart the powershell session after Update-Module?
Else I gonna need to rummage around the state-keeping code to see, what happens there...

You are correct in that I did not restart the PS session, but I did re-exec. Import-Module and Get-Module confirmed it was changed to 1.3.0 from 1.1.8. If that isn't enough, then you are probably correct.

There might still be problems with reloading the classes (PS Modules are not very good at unloading classes).
On occassion, I'll take a closer look into the state mechanics..


Runtime: Investigate/fix broken histogram results

See context in Slack

Fixed here https://github.com/rilldata/rill/pull/3355


Style imports as solarized orange

Description of the Change
In looking at the Solarized Home Page, I noticed that imports are usually orange (or in Haskell's case, magenta). Given that ES6 and TypeScript use imports as well, I thought that it would be nice to have support for that in this atom theme.
Alternate Designs
None.
Benefits
Contrast between code and imports.
Possible Drawbacks
None.
Applicable Issues
None.

No problem!
PS: I'll send a similar pull request for solarized-light-syntax sometime today.

Just kidding, you beat me to it. 😉


Extract ComputeSchemaMixin from BaseOperator

This potentially reduces the complexity of using Merlin schemas and the existing schema computation code outside the context of Merlin DAGs.

It looks like "loader transforms" are already being added to Merlin Models, so it seems like we may not be far off from needing to figure it out. I'm inclined to either let this set open until we make a decision or close it and reopen later if we need to.

@oliverholworthy I think I heard that some of the transforms that were potentially going to be run in the dataloaders have now been pulled into the models—is that accurate? If so, does that affect how we'd think about merging this PR (or not)?


Connection is Closed - Springboot application in ECS

Hi,
We have deployed microservice springboot application into ECS fargate. There we are facing a issue where if the application is kept idle overnight, the sql connections are closed and hence receive an internal server error.  Would like to know where the issue persists and what the solution might be. Please help us with solution since we are in  the last leg of the project and would like to resolve this ASAP.
Also please find below the attachment of the error we are getting in the logs of our application.

Make sure that the maxLifetime configuration for HikariCP is set to shorter than the timeout that is configured on the database server.
If that isn't the issue, you should enable debug logging and also include your hikari configuration settings in this issue.


logging: ideally we should be able to create indexes in our Elasticsearch

It'd be great to get the logging up&running and redirecting the logs to Elasticsearch. Consequently, in order to improve the monitoring, we'll aim to add indexes for some of our main logging properties (such as req_id, identity_id...). Nevertheless we might need to add new ones to improve traceability once things are running at larger scale.
This came after a discussion with @aslakknutsen :).
Accordingly, there is an issue related to this in housekeeping to be tracked:
https://gitlab.cee.redhat.com/dtsd/housekeeping/issues/123

This has some infrastructure limitations that we cannot overcome with the current setup. Let's open it if the setup changes


Exclude grape and grape-swagger from deps

Hello,
Please exclude grape and grape-swagger from gem dependencies. Main reason for this usage of grape-swagger-rails only as swagger-ui without code that depends on grape or/and grape-swagger.

What stops working when we remove those dependencies? Can you please PR it?

@swistaczek This is now live in master and will be released sometime soon.  Thank you for your recommendation!


Support current Git Hash

Luckily, bumpversion supports the major, minor, and patch style. However, sometimes, this is not enough.
For example, when you want to track the current Git hash it would be nice if bumpversion could support a {githash} variable as well. Can be retrieved with this command:
git rev-parse [--short] HEAD

a lookahead for the commit after the bumpversion commit would be great. may this could be implemented using quantum mechanics.

It could be achieved by doing a search for the version bumping message like this: git log --oneline | grep "Bump version"

This can be doable with commit=False where you just want to tag the current commit.

I decided to roll my own. Check out version_upper


Upstream Main: Linting, Benchmarking, HF QLoRA baseline, FSDP fixes for GPTQ-LoRA

Upstreaming to main, see commit notes.

Did a squash merge for this, but in the future, might just do a regular merge from dev-> main


More than one organizational affiliation for the same role

If I recall correctly, the same person can have more than one role in a dataset (e.g. originator plus metadata author). In the Russian data papers we got a questions if more than one organizational affiliation is possible for the same person within the same role, e.g. when Jane Smith is a metadata author, but she works 50% in a research institute and 50% for a protected area. It is common in academia+ circles to have more than one affiliations, and people are used to indicate, where relevant, e.g. using superscripts (see example of Andy Taylor here). It seems possible to add more than one affiliation using the free text as discussed with @ManonGros, but perhaps GBIF roles & contact model, as well as new versions of IPT can allow more than one affiliation per person per role. I suspect this will be futureproof in the case indexing organization is a possibility in the future: one can imagine publisher pages with some sort of personal contributions statistics. Alternative is to do this via ORCID, but roles x organizations modelling may become more difficult. Also note that people use affiliations selectively: in publications certainly, but with data publishing this would be applicable. This may look like an insignificant feature for GBIF index, but if, as we hope, people will start using the data actions & outputs as or in the academic performance metrics, accurate handling of affiliation data will be important. Also, this would directly feed into a personal profile.

+1 here, too. In my case, it's a person who works at a government agency but holds an affiliate position at a university, and the university affiliation is important for the dataset at hand.

@emiliom brough this up in our SMBD slack space, and there was a small discussion agreeing this would be a good feature to have.  organizationName is allowed to be a repeatable element in EML, so I think that this dovetails nicely with the ROR feature request in #2373.


Improve docs around cachUTXO usage

As a follow-up to https://github.com/FuelLabs/fuels-ts/issues/2816 we should document the rationale why it's enabled by default, and also potential issues around disabling cache as warning to most users not to do so.

Moved as a comment on the related existent issue:

https://github.com/FuelLabs/fuels-ts/issues/2855


Add support for Access::WriteOnce registers

Currently doesn't enforce the write-once nature of the registers, but
treats them as write-only.

How is this WriteOnly property enforced?

How is this WriteOnce property enforced?

It isn't, and I couldn't come up with a good way to enforce it on a type level :(

It isn't, and I couldn't come up with a good way to enforce it on a type level :(

Maybe write(self, ...) instead of write(&mut self, ...) will do what you need.

bors try


Story like counters are not refreshed

Eventhough correct number of likes of a story is updated in the backend, client is unable to show recent changes to the like count.

fixed.


GPIO: If a common Instance trait is used for multiple peripherals, those traits should not have any logic implemented in them.

This mostly covers the different Pin traits, I believe.

Hide all pin trait methods

The only relevant trait with methods is Pin. Pasting here the rustdoc output, we have these:
pub trait Pin: Sealed {
    // Required method
    fn number(&self) -> u8;

    // Provided method
    fn degrade(self) -> AnyPin
       where Self: Sized { ... }
}
pub trait InputPin:
    Pin
    + Into<AnyPin>
    + 'static { }
pub trait OutputPin:
    Pin
    + Into<AnyPin>
    + 'static { }

I think in terms of public functions, this is reasonable. The other traits are intended to be hidden by #2693

I think the only question is should number and degrade be inherent impls on each gpio struct, instead of in the trait?
I can see a world where a user uses our traits to be generic over some i/o pins and might want to access something like pin number. It's whether we expose it now, or not. We also have to be careful to not put the kitchen sink in the trait methods over the long term (this guideline should prevent us from doing this, in theory).

degrade is questionable - InputPin and OutputPin both implement Into<AnyPin>. fn number is as close to a GPIO fn info() as possible right now, and we don't really publish those currently, so one could argue that we can hide that as well for the sake of consistency.


Error setting Content contentContainerStyle

react-native 0.40
native-base 0.5.21
When using the contentContainerStyle property of Content I get:
you attempted to set the key padding with the value 0 on an object that is meant to be immutable and hes been frozen.
whenever I transition out of the view or activating a Fab button inside the Content
example code:
const styles = {
  content: {
    backgroundColor: 'white',
    flex: 1
  },
  contentContainer: {
    flex: 1
  }
};

<Content
          style={content}
          contentContainerStyle={contentContainer} >
          <ScrollView>
            {
              order ?
                <Card>
                  <CardItem>
                    <Icon name={`md-${order.icon}`} style={{ color: '#DD5044', fontSize: 48 }} />
                    <Text>{order.client}</Text>
                    <Text note>{order.address}</Text>
                  </CardItem>
                  <CardItem header>
                    <Text>Detalles</Text>
                  </CardItem>
                </Card> : null
            }
          </ScrollView>

          <Fab
            active={active}
            direction="up"
            containerStyle={{ marginLeft: 10 }}
            style={{ backgroundColor: '#5067FF' }}
            position="bottomRight"
            onPress={() => this.setState({ active: !this.state.active })}
          >
            <Icon name="md-add" />
            <Button style={{ backgroundColor: '#34A34F' }}>
              <Icon name="logo-whatsapp" />
            </Button>
            <Button style={{ backgroundColor: '#3B5998' }}>
              <Icon name="logo-facebook" />
            </Button>
            <Button disabled style={{ backgroundColor: '#DD5144' }}>
              <Icon name="ios-mail" />
            </Button>
          </Fab>
        </Content>

If I remove the contentContainerStyle property it works fine but the Content doesn't cover the entire screen

I discovered the error by looking in:
https://github.com/GeekyAnts/NativeBase/blob/master/Components/Widgets/Content/index.js#L38
Content is attempting to assign .padding to contentContainerStyle object, but when passed down as an object is frozen:
so I had to do:
<Content
          style={content}
          contentContainerStyle={{
            flex: 1
          }} >

Yes, this is a known bug as of now and will be fixed in the future release. It's also a duplicate of #227, so let's track it there.

thx, sorry for the duplicate, I did search for issues related to 'contentContainerStyle' but nothing came up on the search


Realsense viewer and  python library is not working

librealsense
2.44.0 RELEASE

OS
Windows

Intel RealSense Viewer / Depth Quality Tool has crashed with the following error message:
Program terminated due to an unrecoverable SEH exception:
Access Violation!

Hi @OTS-HOUSTON  This is a rarely occurring error.  As a starting point in investigating your case, could you please check your computer's Device Manager to see whether there are multiple hidden entries for RealSense camera drivers, as described in the link below.
https://github.com/IntelRealSense/librealsense/issues/7185#issuecomment-688837669
If you only have one camera attached then there should only be a single pair of drivers under the Cameras category of the Device Manager, one driver for RGB and the other for depth.

Hi @OTS-HOUSTON  Do you require further assistance with this case, please?  Thanks!

Case closed due to no further comments received.


⚠ Develop API Endpoint has degraded performance

In 46e45ef, Develop API Endpoint (https://develop.roblox.com/v1/toolbox/items?category=Hat&keyword=Hat) experienced degraded performance:

HTTP code: 200
Response time: 2742 ms

Resolved: Develop API Endpoint performance has improved in 713aced.


Include "special" _octodns values in Record.repr

This includes the octodns bits in Record.__repr__. The upside of this is that you can see the special bits that may be pushed to your providers, downside is that it could be noisy and not all special things apply to all providers and things pulled out of providers will only include the things that provider knows about.
Example of the "good" case, CF proxied
'*.sub':
  octodns:
    cloudflare:
      proxied: True
  ttl: 144
  type: A
  values:
  - <IP_ADDRESS>
  - <IP_ADDRESS>
  - <IP_ADDRESS>

********************************************************************************
* githubtest.net.
********************************************************************************
* cloudflare (CloudflareProvider)
*   Update
*     <ARecord A 144, *.sub.githubtest.net., ['<IP_ADDRESS>', '<IP_ADDRESS>', '<IP_ADDRESS>'], cloudflare=cloudflare.proxied=False> ->
*     <ARecord A 144, *.sub.githubtest.net., ['<IP_ADDRESS>', '<IP_ADDRESS>', '<IP_ADDRESS>'], cloudflare=cloudflare.proxied=True> (config)
*   Summary: Creates=0, Updates=1, Deletes=0, Existing Records=20
********************************************************************************

Example of the "bad" case, where things aren't super clear
'*.sub':
  octodns:
    cloudflare:
      other: False
      proxied: True
    something: True
  ttl: 144
  type: A
  values:
  - <IP_ADDRESS>
  - <IP_ADDRESS>
  - <IP_ADDRESS>

********************************************************************************
* githubtest.net.
********************************************************************************
* cloudflare (CloudflareProvider)
*   Update
*     <ARecord A 144, *.sub.githubtest.net., ['<IP_ADDRESS>', '<IP_ADDRESS>', '<IP_ADDRESS>'], cloudflare=cloudflare.proxied=False> ->
*     <ARecord A 144, *.sub.githubtest.net., ['<IP_ADDRESS>', '<IP_ADDRESS>', '<IP_ADDRESS>'], cloudflare=cloudflare.other=False, cloudflare.proxied=True, something=True> (config)
*   Summary: Creates=0, Updates=1, Deletes=0, Existing Records=20
********************************************************************************

/cc https://github.com/github/octodns/issues/284
/cc @parkr @begincalendar

Btw, I'm not really sure about this PR. I'd like to show the special bits, but worried that it'll end up confusing things as much as helping...

I think it gets the job done.
Which part don't you like?

I dig this! Here's an example:
********************************************************************************
* parkermakes.tools.
********************************************************************************
* cloudflare (CloudflareProvider)
*   Update
*     <AliasRecord ALIAS 1, parkermakes.tools., kondwani.parkermakes.tools.> ->
*     <AliasRecord ALIAS 1, parkermakes.tools., kondwani.parkermakes.tools.> (config)
*   Summary: Creates=0, Updates=1, Deletes=0, Existing Records=15
********************************************************************************
* keltermoore.com.
********************************************************************************
* cloudflare (CloudflareProvider)
*   Update
*     <ARecord A 1, keltermoore.com., ['<IP_ADDRESS>'], cloudflare=cloudflare.proxied=True> ->
*     <ARecord A 1, keltermoore.com., ['<IP_ADDRESS>'], cloudflare=cloudflare.proxied=False> (config)
*   Summary: Creates=0, Updates=1, Deletes=0, Existing Records=2
********************************************************************************

Hm; the AliasRecord there wasn't changed by me, so looks like a diff is found there when nothing was changed?
The keltermoore.com change was purposeful.

The AliasRecord diff doesn't get printed on master. Only:
********************************************************************************
* keltermoore.com.
********************************************************************************
* cloudflare (CloudflareProvider)
*   Update
*     <ARecord A 1, keltermoore.com., ['<IP_ADDRESS>']> ->
*     <ARecord A 1, keltermoore.com., ['<IP_ADDRESS>']> (config)
*   Summary: Creates=0, Updates=1, Deletes=0, Existing Records=2
********************************************************************************


Jc minor edit

Coverage remained the same at 73.757% when pulling c77310128b292083b42f5b47a28db0d7bfb926a7 on JC-minor-edit into e0d21f2e522ffe1138f0fae0a2805a9799d73cab on master.


consider  as uri if it is not a resource

GitHub Issue: (link)
https://github.com/Islandora-CLAW/CLAW/issues/943
What does this Pull Request do?
This PR will tell Crayfish-Commons to not to load $data into the pipe if it is not a resource.  It is assumed that a resource url will be passed in.  Specifically to address the above 943 ffmpeg issue.
How should this be tested?
This needs to be tested other related PRs.  Which I will post shortly here.
Interested parties
Tag (@ mention) interested parties or, if unsure, @Islandora-CLAW/committers

phpcbf can take care of Travis: https://travis-ci.com/Islandora-CLAW/Crayfish-Commons/jobs/156095121

linking to https://github.com/Islandora-CLAW/Crayfish/pull/52

Codecov forced me to use admin privileges to merge that because coverage dropped by a tiny percentage.  I should go through our repos and disable that so it's not a 'required' status check and instead just a tool of guilt.

Eww... so I don't know if I can do that...
It's either required or not a status check at all.  I wonder if it will still run when not a status check?  Only one way to find out.

It won't run, we should keep it as required and figure out why coverage dropped and add tests so it doesn't.

Actually we have no tests for the CmdExecuteService so adding any complexity to it reduces the overall coverage. We should add tests for this service.


Implement client-side sorting

Implements sorting using client-side logic. I'd like this to be done on the backend, but there's not a huge reason to right now as the dataset is small.

Closing in favor of merging with #51


Encode the actual current date in vw_mqb_config.py

It was under "TODO" in selfdrive/debug/vw_mqb_config.py to encode the actual current date as prog_date.

@jyoung8607 want to review?


Timeline for release

Hi,
Thank you for the great work! I was just wondering if you have an approximate timeline for the code release? Further, whether you'll be including weights?

Hi,
Thanks for your interest in our work! We plan to release the code, including weights and training code, after the paper's acceptance. If things work all the best, the release time is expected before September. Please stay tuned :).
Best, Xingyi

That's great - where are you in submission to?

We are sorry that we cannot provide more information, due to the dual-blind reviewing requirement.

Hi Xingyi @hxy-123 ,
May I ask is there any updates? I was hoping to compare my method to DetectorFreeSfM :)


Error due to deprecated Python when using Home Assistant 2023.6.0

The latest release of Home Assistant will throw loading errors for Tesla Custom.
AttributeError: module 'asyncio' has no attribute 'coroutine'

Found this page explaining.  StackOverflow LINK
Summary "Generator-based Coroutines which contains @asyncio.coroutine decorator is removed since Python 3.11 so asyncio module doesn't have @asyncio.coroutine decorator as the error says"
I had to make edits in the Tesla Gateway __init__.py in custom_integrations folder.
I commented out the three lines with
<EMAIL_ADDRESS>
Then I had to edit this line and add 'async ' at the front.  Then the Tesla Custom loaded fine on Home Assistant restart with no errors.
async def async_setup(hass, config):

Thanks. Happy to take a PR to fix this.

II have the following the the log, which I assume is related:
2023-06-08 16:39:47.152 WARNING (MainThread) [homeassistant.helpers.entity] Entity switch.skippy_polling (<class 'custom_components.tesla_custom.switch.TeslaCarPolling'>) is using self.async_update_ha_state(), without enabling force_update. Instead it should use self.async_write_ha_state(), please report it to the custom integration author.

The latest release of Home Assistant will throw loading errors for Tesla Custom.
AttributeError: module 'asyncio' has no attribute 'coroutine'

Found this page explaining. StackOverflow LINK Summary "Generator-based Coroutines which contains @asyncio.coroutine decorator is removed since Python 3.11 so asyncio module doesn't have @asyncio.coroutine decorator as the error says"
I had to make edits in the Tesla Gateway init.py in custom_integrations folder.
I commented out the three lines with
<EMAIL_ADDRESS>
Then I had to edit this line and add 'async ' at the front. Then the Tesla Custom loaded fine on Home Assistant restart with no errors.
async def async_setup(hass, config):

I am unable to locate the referenced folder. I have custom components and a tesla folder but nothing with that filename. I don't see a custom integration folder.
Thanks
Craig

2023-06-08 16:39:47.152 WARNING (MainThread) [homeassistant.helpers.entity] Entity switch.skippy_polling (<class 'custom_components.tesla_custom.switch.TeslaCarPolling'>) is using self.async_update_ha_state(), without enabling force_update. Instead it should use self.async_write_ha_state(), please report it to the custom integration author.

The async_update issue seems to be fixed by #606 , the broader issue running under Python 3.11 maybe still outstanding although I can't reproduce on my setup.


Show size of cache in compass windows

Show size of cache in compass windows would be nice feature... For example the sice indicator known from geocaching.com.

Duplicate of #315.


uninitialized constant Sprockets::Rails

To fix #13 we are trying the 0.2 version of this gem, but we get this error:
Exiting
/usr/local/rvm/gems/ruby-3.2.2/gems/debugbar-0.2.0/lib/debugbar/engine.rb:42:in `block in <class:Engine>': uninitialized constant Sprockets::Rails (NameError)

      app.middleware.insert_after Sprockets::Rails::QuietAssets, Debugbar::QuietRoutes
                                                  ^^^^^^^^^^^^^
Did you mean?  RailsERD

So I then added sprockets-rails in our Gemfile, installed it, and then I get this error:
Exiting
/usr/local/rvm/gems/ruby-3.2.2/gems/sprockets-rails-3.4.2/lib/sprockets/railtie.rb:110:in `block in <class:Railtie>': Expected to find a manifest file in `app/assets/config/manifest.js` (Sprockets::Railtie::ManifestNeededError)
But did not, please create this file and use it to link any assets that need
to be rendered by your app:

Example:
  //= link_tree ../images
  //= link_directory ../javascripts .js
  //= link_directory ../stylesheets .css
and restart your server

For more information see: https://github.com/rails/sprockets/blob/070fc01947c111d35bb4c836e9bb71962a8e0595/UPGRADING.md#manifestjs

I don't really know what's going on or what this is, so  I stopped there. Ours is an API only rails app (maybe that has something to do with this)
Looks like the bug was introduced by this commit here https://github.com/julienbourdeau/debugbar/commit/efe491bde9e0544e5fb891597bb2af47854c1169#diff-b649b89947efca18165b6c64a06e061ed7d8e13b6aebaa11421744e3ef486840R42

@fazo96 version 0.2.1 was published and should fix this. Please let me know if it worked for you 🙏

Thanks, 0.2.1 fixed it.
Unfortunately it doesn't work due to another problem but I think this one's on our end. I'll open an issue if I can get some time to investigate it

Thank you 🙏


Migrate hackclub/lecture-hall features to unified codebase

From https://www.notion.so/2017-06-20-Classifying-Hack-Club-s-Repos-080c12f5662445cbad742b1dbb6986c1

~The main content of this should be converted to React and added to frontend~
~The only change in api is to support GitHub oauth for our sign-in wall~

A better way to do this is to have react request the workshop content from the backend. Here's how it's going to get broken up now.

monolith/api (tasks assigned to @paked)

[ ] Add a route under v1/workshops/* that returns the contents of vendor/hackclub/*
[ ] Markdown files should be translated to HTML before being returned

monolith/frontend (tasks assigned to (@MaxWofford)

[ ] Set up a workshops/* route that reverse proxies api.hackclub.com/v1/workshops/*

hackclub/proxy (tasks assigned to (@MaxWofford)

[ ] Point workshops.hackclub.com to hackclub.com/workshops

Update on my task:
There's a PR for the v1/workshops/* route in hackclub/monolith right now
It will serve any file in the api/vendor/hackclub directory with it's correct mimetype, etc. If the file being served is a markdown file, it will be parsed and rendered as HTML.

@paked: I just added a task to the list:

Update @hackbot update-workshop command to update hackclub/monolith


Training vs. Validation loss, as reported in the article

Hello everyone,
First, thanks for releasing such an amazing dataset! Incredible work!
I’m currently training a model on your dataset for my engineering degree, and was wondering how you generate with Detectron2 the validation loss, as reported in the supplementary notes:
"All training was run for a predefined set of iterations and the loss on a
validation set, separate from the training and test sets, were monitored to assess model over- and under-fitting. Model checkpoints
were saved and used for evaluation based on which had the lowest validation loss (Supplementary Figs. 9 and 10) on the rationale
that the lowest validation loss represents a good balance between an under- and over-fitted model."
I am able to get a very similar loss curve for the training set. I used the validation set for monitoring the performance of the model for the segmentation task, and evaluated the trained model on the test set, but I don't know how you get the validation loss you show on your graphs.

Any hint would mean a lot. Thank you in advance,
Kind regards,
Matías Stingl

Hi @mgstingl,
I am glad that you appreciate LIVECell!
Logging validation in loss in Detectron2 requires you to implement your own hooks. Preparing the manuscript, we trained models on our compute infrastructure where we have some custom tooling. When releasing the source code, we chose to only publish models that are runnable on "native" Detectron2 without our custom tools to make it easier to pick up on the work. The validation loss hook was unfortunately one of these.
However, you are not alone in wanting to log validation loss and you can find this issue discussing on how to implement it on Detectron2. I hope that is helpful.


Improve failure messages in operator-developer-e2e condition comparison

As shown in #481 today if there is a failure in operator-developer-e2e due to unexpected condition we get a message like this:
Status conditions for the operator plain-operator for the version 0.2.0 is not as expected:Expected status: True, but got: False

We should include condition message when failing test due to unexpected condition as it usually contains useful information.
Relevant code:
https://github.com/operator-framework/operator-controller/blob/a00a93ffb00c3a9fe894c1974e17400933bffebd/test/operator-framework-e2e/operator_framework_test.go#L769-L783

Since it looks like the tests have been majorly refactored since this was written, is this issue still necessary?

Good call! I think we can close it.


Removing animated modal from DOM prevents modal on-hide cleanup events from occurring

Steps to reproduce the issue:

Open an animated modal dialog (modal must have fade class).
Call .modal('hide') and .remove() on the modal element.

Expected result: Modal backdrop should be removed from DOM and modal-open class should be removed from body tag.
Actual result: Modal is removed from DOM, but modal backdrop remains and modal-open class is still on the body tag.
See test case: JSFiddle
Workarounds:
A) Call .remove() on hidden.bs.modal.
B) Remove fade from the modal.
It makes sense why this issue occurs (modal DOM removal cleans up the transition event listeners, so cleanup never occurs), but it would be nice if the modal cleanup occurred regardless of whether the modal still exists on the DOM.

Seems to happen due to the synthetic fallback transitionend (or browser-prefixed equivalent) event that emulateTransitionEnd() fires not triggering a bsTransitionEnd event via the jQuery special event machinery like it normally does. I guess jQuery isn't triggering it because the element is outside the DOM?
At any rate, this doesn't appear to be a trivial thing to support and (A) is definitely already The Right Way To Do It.
Since this is similar to #16284, I propose likewise punting this to Bootstrap v4.
@fat What's your opinion here?

yep, unfortunately, hide is async, so calling remove in a chaining method isn't going to give it time to clean itself up.
for now i think just listening to hidden event is best practice

NAVER - http://www.naver.com/
<EMAIL_ADDRESS>님께 보내신 메일 <Re: [bootstrap] Removing animated modal from DOM prevents modal on-hide cleanup events from occurring (#16469)> 이 다음과 같은 이유로 전송 실패했습니다.

받는 사람이 회원님의 메일을 수신차단 하였습니다.

@animafact It seems you have an autoresponder setup. If you could disable this that would be greatly appreciated as it's generating a fair bit of noise.

@kkirsche It's basically spam—I've reported it to GitHub.


validate_and_consume_otp! missing

eyedp_1    | [d0ff52b0-e6dd-4370-82d7-5cbc8b176447] NoMethodError (undefined method validate_and_consume_otp!' for nil:NilClass):         eyedp_1    | [d0ff52b0-e6dd-4370-82d7-5cbc8b176447]                                                                                eyedp_1    | [d0ff52b0-e6dd-4370-82d7-5cbc8b176447] app/controllers/users_controller.rb:12:in create_2fa'

It is possible to have session terminated and reach this page, it should handle more gracefully


Don't override __init__ method in MyPy plugin if it already exists

Change Summary
This PR updates the MyPy plugin to not override the __init__ method if there's already one in the model.
Here's an example of the current issue:
<EMAIL_ADDRESS>running mypy main.py returns the following:
main.py:13: error: Too many positional arguments for "Person"
main.py:13: error: Missing named argument "name" for "Person"
main.py:13: error: Missing named argument "birth_year" for "Person"
main.py:15: note: Revealed type is "def (*, id: Any, name: Any, birth_year: Any, **kwargs: Any) -> main.Person"
main.py:16: note: Revealed type is "def (__pydantic_self__: main.Person, *, id: Any, name: Any, birth_year: Any, **kwargs: Any)"

while the code works since we have a custom constructor:
from pydantic import BaseModel

class Person(BaseModel):
    id: int
    name: str
    birth_year: int

    def __init__(self, id: int) -> None:
        super().__init__(id=id, name="Patrick", birth_year=1991)

print(Person(1))

reveal_type(Person)
reveal_type(Person.__init__)

Checklist

[x] Unit tests for the changes exist
[ ] Tests pass on CI and coverage remains at 100%
[ ] Documentation reflects the changes where applicable
[x] changes/<pull request or issue id>-<github username>.md file added describing change
(see changes/README.md for details)
[x] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

coverage fails because we don't hit these lines anymore: https://smokeshow.helpmanual.io/5k4z1l3e0n1q425g2r40/d_2705a9c05af28909_mypy_py.html#t644
I'm not sure if they are needed anymore

please review

this is great, thank you so much.

Looks like the coverage has dropped, hence this not merging, see here.
Looks like the code which is no-longer covered can be removed, but if you're not sure, maybe safer to add no cover.
Please update.

done!
please review


Consider a shorter alias for the macro name

Because #![doc = document_features::document_features!()] is quite long and unnecessary repeats itself.
I think it'll be better to have something like document_features::get!(), extract, etc.

That's true.
On the other hand, this only appear once per crate, so not such a big deal. And the document-features crate can be imported with another name, or the macro used.
I'm not so convinced by the suggested names though.


docs: remove shim_no_newns in ops.md

this ops is removed in commit fd2e3cd326b5ef58a7118658515911921d6731a1,
remove from doc avoid misleading users.
Signed-off-by: Ace-Tang<EMAIL_ADDRESS>

LGTM


[ANCHOR-283] Add RequestLoggerFilter configuration

PR Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[ ] This PR's title starts with name of package that is most changed in the PR, ex.
paymentservice.stellar, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.

What
[TODO: Short statement about what is changing.]
Why
[TODO: Why this change is being made. Include any context required to understand the why.]
Known limitations
[TODO or N/A]

Reference Server Preview is available here:https://anchor-ref-pr928.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr928.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr928.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr928.previews.kube001.services.stellar-ops.com/


SSL does not work as described in the documentation

The following code
  val mailer = Mailer("example.com", 587)
    .auth(true)
<EMAIL_ADDRESS>"pwd")
    .ssl(true)()

gives me value ssl is not a member of courier.Session.Builder

I believe the doc says .startTls(true)(), can you try this?

Sorry, for the confusion. What version of the library to you use?

Ok, I'm sorry. I'm a new maintainer. I'm still a little bit unfamiliar with the code base. I've merged the ssl feature last night. The code is not yet published.
The code you have relates to: https://github.com/dmurvihill/courier/tree/0.1.4
I'm planning to publish later today. I will close this when I publish with the version you can use to leverage this new feature.

Thanks :)

The release is in: https://github.com/dmurvihill/courier/releases/tag/v1.0.0-RC1
The 1.0.0-RC1 should be in maven central in a few minutes/hours. Let me know how it goes!

Works fine, thanks :)


Can you detect if a tour is already playing?

Hi, is there a way to determine whether a tour is already playing? Say for example I want to start another tour, but I don't want to do it while a current tour is playing as they overlap. How can I detect that a tour is already underway?

this.tourService.getStatus() === TourState.ON


Storage Replica

I believe Storage Replica should be listed as YES for 2019 Standard Edition (though with limitations compared to Data Center Edition).

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

ID: c6f2b272-1c80-2d9b-590d-81c9970ee147
Version Independent ID: 72474f8e-7705-df9f-f12f-02e0926f0901
Content: Comparison of Standard and Datacenter editions of Windows Server 2019
Content Source: WindowsServerDocs/get-started-19/editions-comparison-19.md
Product: windows-server-threshold
GitHub Login: @coreyp-at-msft
Microsoft Alias: coreyp

Ned Pyle Likely Approves This Comment

Thanks for your dedication to our documentation. If we have been unable to review your issue in a timely manner, we sincerely apologize for the delayed response. We have captured the informtion you have provided for review. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close


HFSS3D Layout : I need to create some "Circuit Ports"...

Description of the feature
In a HFSS3D Layout :
I need to be able to create some "Circuit Ports", i am now able to create differential ports using the "h3d.create_differential_port()" function, but now i need to be able to change the port "HFSS Type" from "gap" to "circuit" :

Another option would be to create a "h3d.create_circuit_port()" function.
I thank you very much in advance!
Steps for implementing the feature
No response
Useful links and references
No response

@Ouam74, the recommended workflow is to setup layout in edb, and then import it into HFSS 3D Layout for connector assembly, analysis, post processing, etc.
Below is an example, how to setup a gap port, and switch it ot circuit port. Please have a try once the PR is merged.
from pyaedt import Desktop
from pyaedt import Hfss
from pyaedt import Edb
from pyaedt import Hfss3dLayout

import pyedb

desktop = Desktop(specified_version="2023.2")

edb = Edb(edbversion="2023.2")
edb.stackup.create_symmetric_stackup(4)

trace = edb.modeler.create_trace(
    path_list=[[0, 0], [0, "5mm"]],
    layer_name="TOP",
    width="0.1mm",
    start_cap_style="Flat",
    end_cap_style="Flat",
)

port = trace.create_edge_port(name="p2", position="end", port_type="Gap")
port.is_circuit_port = True

fpath_edb = edb.edbpath
edb.save()
edb.close()

h3d = Hfss3dLayout(fpath_edb)

desktop.release_desktop(False, False)


Handle various out-of-bounds diagnostic ranges in default diagnostics formatter

Bug/issue #, if applicable: #729
Summary
This improved the input validation of the default diagnostic formatter so that incorrectly created diagnostics with out-of-bounds range information won't cause a crash.
Dependencies
None.
Testing
Since this needs an incorrectly created diagnostic, it requires a bug that results in out-of-bounds diagnostics to test.
This comment includes a package that reproduces on such bug. Building it should no longer crash (although it will output a duplicate diagnostic, because that issue is not fixed in this PR).
Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[x] Added tests
[x] Ran the ./bin/test script and it succeeded
[x] ~Updated documentation if necessary~ Not applicable

@swift-ci please test

LGTM. Any plan to cherry-pick this back to release/6.0?

LGTM. Any plan to cherry-pick this back to release/6.0?

Yes. I think it's safe and small enough for that.


Update Parsers to Define Default Prompt / Prompt Data

When programmatically constructing a prompt for the config, the user must construct the data to serialize for the prompt creation. More importantly, our editor implementations currently need to be coupled with the model parsers to know how to construct prompts for different models.
If we instead expose an abstract method on the ModelParser class to define a default prompt (or even default data that can be serialized to the prompt), then we could leverage that for any model parser to construct or modify default prompts without needing to have/know all the required data values from the start.

Closing this based on discussion with @suyoglastmileai and @saqadri yesterday, in favour of statically-defined prompt schemas. I'll try it out locally before creating a relevant issue


KeyError "ner"  when I predict on new dataset

I use the model to predict named entities for new dataset but encounters the following problem:
Traceback (most recent call last):
File "scirex/predictors/predict_ner.py", line 123, in
main()
File "scirex/predictors/predict_ner.py", line 119, in main
predict(archive_folder, test_file, output_file, cuda_device)
File "scirex/predictors/predict_ner.py", line 35, in predict
instances = dataset_reader.read(test_file)
File "/home/jttang/.conda/envs/scirex_wpc/lib/python3.7/site-packages/allennlp/data/dataset_readers/dataset_reader.py", line 134, in read
instances = [instance for instance in Tqdm.tqdm(instances)]
File "/home/jttang/.conda/envs/scirex_wpc/lib/python3.7/site-packages/allennlp/data/dataset_readers/dataset_reader.py", line 134, in
instances = [instance for instance in Tqdm.tqdm(instances)]
File "/home/jttang/.conda/envs/scirex_wpc/lib/python3.7/site-packages/tqdm/_tqdm.py", line 1005, in iter
for obj in iterable:
File "/dat01/jttang/wpc/information_extraction/SciREX/SciREX/scirex/data/dataset_readers/scirex_full_reader.py", line 148, in _read
json_dict = clean_json_dict(json_dict)
File "/dat01/jttang/wpc/information_extraction/SciREX/SciREX/scirex/data/dataset_readers/scirex_full_reader.py", line 34, in clean_json_dict
entities: List[Tuple[int, int, BaseEntityType]] = json_dict["ner"]
KeyError: 'ner'
You mentioned new dataset only needs "doc_id","words","sentences","sections" fileds，but the code requires "ner" field.  Could you show me some solutions, Thanks.

Hi I made a update. Can you see if it still giving you errors ?

Thanks for your update. There's no more problem with a little modest adjustment


Add Thumbnail Toolbar (Windows)

On Windows you can add a thumbnail toolbar with specified buttons in a taskbar layout of an application window. It provides users a way to access to a particular window’s command without restoring or activating the window.
Built-in Windows programs like Windows Media Player do this, so we should too to blend into the OS as best we can.

API docs: http://electron.atom.io/docs/v0.36.8/tutorial/desktop-environment-integration/#thumbnail-toolbars

I also believe the app should stay running by default after closing the window and display inside an application icon (bottom right for Windows, top right for OSX & Linux), so torrents can be managed (uploading/downloading) consistently while running the app on the background.

@iadj That's another issue. Can you open a new issue for that?

@feross Done: https://github.com/feross/webtorrent-app/issues/150

I'm on it

@gediminaspetrikas You're going to work on the buttons that show up in the taskbar?

Yes. I'll add play/pause buttons which switch depending on the state, so it will be only a single button, as there isn't play next/previous funcionality

This is done now, thanks to @gediminaspetrikas!


Clean up _destag_variable with respect to types and terminology

Change Summary

Further cleanups of #93 after merge (sorry about that!), this time to help out developers/contributors who wish to build upon the private _destag_variable function:
As discussed in https://github.com/xarray-contrib/xwrf/pull/100#discussion_r973392632 and https://github.com/xarray-contrib/xwrf/pull/100#discussion_r973399431, this adds a type check for xarray.Variable in _destag_variable as a safe guard for xwrf contributors against passing in an xarray.DataArray by accident. Also, updates comments in _destag_variable from the original version in #37 to have a bit more accurate terminology.
Related issue number

Checklist

[x] Unit tests for the changes exist
[ ] Tests pass on CI
Documentation reflects the changes where applicable

Rebased on main after #100 just to be extra careful. With @lpilz review approving, I've enabled auto-merge once CI passes.


Add mock tests for persistent cache in identity

bring back tests in https://github.com/Azure/azure-sdk-for-java/pull/5308

Closing this as persistence token caching logic has moved to MSAL layer.


Fix function signatures added with #691

Somehow my comments on #691 were ignored by everyone. 😕
If this is wrong, then please explain it to me.

Ok if this was posted in discord it should also have been posted to the GitHub as it's impossible when it becomes over a week old to find said talk again.

I didn't talk about it on Discord. These were just comments in the PR here on GitHub. All good though!


Display all widgets instead of cycling the widgets in the sidebar

Add the feature of being able to customize and display all widgets of clock, date and weather in the sidebar instead of being able to cycle them.

It depends on how many layers sublime text can support. but it's been limited so far.


Update tanslator.dart and support dynamic templateFilename.

support dynamic templateFilename of type in RegExp: r'^[a-zA-Z0-9]+[a-zA-Z0-9-]+.arb$'.

template file name can be: intl_en.arb, intl_en_US.arb, app_zh.arb an so on.
Core codes here:
    final arbDir = _config['arb-dir'] as String? ?? "lib/l10n";
    final templateFilename = _config['template-arb-file'] as String? ?? "app_en.arb";

    final source = templateFilename.substring(templateFilename.indexOf(RegExp(r'[-_]')) + 1, templateFilename.indexOf('.arb'));
    final name = templateFilename.substring(0, templateFilename.indexOf(RegExp(r'[-_]')));

Thanks for your pull request. It looks good. I am going to update the tests and docs to account for the changes shortly and merge.

Included in version 1.1.0
Thanks @w-rui


message_edit: Don't rely on .recipient_id change not affecting recipi…

…ent.
The codepath for moving a topic changes the message.recipient_id to the
id of the new recipient, but later, in update_messages_for_topic_edit,
it uses message.recipient when querying for messages with the matching
topic in the old stream (because those are the other messages that
need to be moved). This is a bug which happens to work fine, because in
Django 2, if message.recipient gets fetched first and then
message.recipient_id is mutated, message.recipient will not be altered
and thus will retain the outdated, previously fetched value.
In Django 3 changing .recipient_id causes .recipient to be updated to
the new Recipient objects, which is the Recipient of the new stream.
That will cause the bug to manifest.
This is a bugfix preparing for the upgrade to Django 3.

@timabbott Seems that you merged this in 3623681d303766d89f0c2a163a68019a000fabf7 so I'll close.

@timabbott Seems that you merged this in 3623681d303766d89f0c2a163a68019a000fabf7 so I'll close.


DEVOPS: Configure prettier and stylelint

We should configure prettier and stylelint to consistently format the code cross-IDE and cross-contributor.

Please assign this to me.

Hey @zetareticoli . It would be great to contribute! Please assign it to me.


Auto committed offset value one ahead of it's consumed offset value

Description
When consuming from a  topic the OnMessage event handler's offset value is always one behind the offset value received in the OffsetCommitted event handler. It appears the OnMessage event handler receives offsets starting from a zero based index whereas the OffsetCommitted event handler starts from a one based index. This gives the illusion in the logs that messages are consumed after they have already been committed.
The scenario below describes consuming two messages from a brand new topic:
Message1 comes in and its "message.Offset.Value" is displayed as 0 inside the OnMessage event handler. It is then auto committed 5 seconds or so later. The "message.Offset.Value" displayed in the OffsetCommitted event handler is 1. Next  message2 comes in and its "message.Offset.Value" is displayed as 1 inside the OnMessage event handler. It is then auto committed 5 seconds or so later. The "message.Offset.Value" displayed in the OffsetCommitted event handler is 2.
I have checked inside control center and can verify via the "consumer lag" section there's no more than 2 messages. The Lag is zero, current-offset is 2 and end-offset is 2.
How to reproduce
Start a consumer with an OffsetCommitted and OnMessage event handler and then produce two or three messages onto a topic. Observe the "message.Offset.Value" in the OffsetCommitted and OnMessage event handlers.  See consumer code below:
public class Consumer
  {
      private readonly ILogger _logger;
      private readonly ITopicRetriever _topicRetriever;
      private bool _consumeMessages;

      public Consumer(ILogger logger,
          ITopicRetriever topicRetriever)
      {
          _logger = logger;
          _topicRetriever = topicRetriever;
          _consumeMessages = true;
      }

      public void ConsumeMessages(string brokerList)
      {
          using (
              var consumer = new Consumer<byte[], byte[]>(ConstructConfig(brokerList, true),
                  new ByteArrayDeserializer(),
                  new ByteArrayDeserializer()))
          {
              SetupEventListeners(consumer);

              var topics = _topicRetriever.GetAll();

              consumer.Subscribe(topics);
              foreach (var topic in topics)
              {
                  _logger.ExtendedInfo("Subscribed to topic", new {Topic = topic});
              }

              _logger.ExtendedInfo("Started consumer instance, Ctrl-C to stop consuming", new { Consumer = consumer.Name, GroupMemberId = consumer.MemberId });

              Console.CancelKeyPress += (_, e) =>
              {
                  e.Cancel = true; // prevent the process from terminating.
                  _consumeMessages = false;
              };

              while (_consumeMessages)
              {
                  consumer.Poll(TimeSpan.FromMilliseconds(100));
              }
          }
      }

      private void SetupEventListeners(Consumer<byte[], byte[]> consumer)
      {
          consumer.OnMessage += (_, message) => MessageReceived(message);

          consumer.OnOffsetsCommitted += (_, offsets) => OffsetCommitted(offsets);
      }

      private void OffsetCommitted(CommittedOffsets offsets)
      {
          foreach (var offset in offsets.Offsets)
          {
              _logger.ExtendedInfo("Offsets Committed",
                  new
                  {
                      offset.Topic,
                      offset.Partition,
                      Offset = offset.Offset.Value,
                      RealOffset = offset.Offset.Value -1
                  });
          }
      }

      private static Dictionary<string, object> ConstructConfig(string brokerList, bool enableAutoCommit)
      {
          return new Dictionary<string, object>
          {
              {"group.id", "scv-batchProcessor-consumer"},
              {"enable.auto.commit", enableAutoCommit},
              {"auto.commit.interval.ms", 5000},
              {"statistics.interval.ms", 60000},
              {"bootstrap.servers", brokerList},
              {"auto.offset.reset", "earliest"}
          };
      }

      private void MessageReceived(Message<byte[], byte[]> message)
      {
          _logger.ExtendedInfo("Consuming message",
              new
              {
                  message.Topic,
                  MessageReceived = message.Timestamp.UtcDateTime,
                  message.Partition,
                  Offset = message.Offset.Value
              });
      }

  }

Checklist
Please provide the following information:

[ ] Confluent.Kafka nuget version:
0.11.5
[ ] Apache Kafka version:
The latest version on the kafka docker image namely "confluentinc/cp-enterprise-kafka"
and  2.11-2.0.0 in aws
[ ] Client configuration:
{"group.id", "scv-batchProcessor-consumer"},
{"enable.auto.commit", enableAutoCommit},
{"auto.commit.interval.ms", 5000},
{"statistics.interval.ms", 60000},
{"bootstrap.servers", brokerList},
{"auto.offset.reset", "earliest"}
[ ] Operating system:
run in docker container locally on Debian GNU/Linux 9 (stretch), and on AWS in EC2 cluster
[ ] Provide logs (with "debug" : "..." as necessary in configuration)
[ ] Provide broker log excerpts
[ ] Critical issue

this is by design - the convention used by Kafka is that the committed offset is the next message to be read. i also think it would cause less confusion if instead the last committed message was defined to be the last message that was read, but given existing convention, I think the API as is the correct choice.

Thanks very much for your quick response. Appreciate you clearing that up for me.


Launch ec2 container

This is about figuring out how to do this for future training and ci

Finished https://aws.amazon.com/getting-started/tutorials/launch-a-virtual-machine/ with ubuntu


feat(ci): replaces circleci w/ github workflow

Solves CORE-5765 by removing the CircleCI configuration and migrating it to GHA workflow one.
The sonarqube required some additions to the build.gradle file.

[!IMPORTANT]
Just before merging this PR we should remove the CircleCI integration in the repo settings @rsksmart/security.

LGTM!

pipeline:run


fix: slice tail data, fixes #40

I had to modify one of the tests because I think it was actually wrong about how rabin should chunk up the data it was testing.
4 browser tests are failing on my machine, but they were already failing with the current checkout so hopefully CI can show me if this is just a local issue or not.

Also, I modified how and when the loop calls bl.consume() so that it only does it once per fingerprint.

Now that I think about it, an even more efficient approach would be to do buffers = buffers.shallowSlice(size) so that it doesn’t do a memcopy at all.
I can send that as another PR or add it to this one. Let me know what you’d prefer.

Closing.
There’s a bug but I was wrong about the nature of the bug, I’ll send a new PR.