Grammar tweaks

Subject-verb agreement
Fix some commas
Replace python with Python (proper noun)

I was again reading through, and I had my editor up, so... :-)
It is my theory that you are not native English speakers!
Amazingly well done. Would that I could write so well in any of my other languages. :-o :-/

Two more things I could clear up for you when I was there.

Is it the KV language, the Kv language, the KV language, the Kv language or the kv language? :-D Pick one and I'll fix 'em all up on that page.

Do you declare a widget in .kv or define one?  You use both, and "declare" more often, but I think it's a definition: https://stackoverflow.com/questions/1410563/what-is-the-difference-between-a-definition-and-a-declaration .  Let me know and I'll fix those up too.

I force pushed another change - the word "syntaxes" is very rare in English, and it's only when you're comparing two big things, like "different languages have different syntaxes".

And that leads me to a question: why not use Python's syntax here?  (You could actually add this feature and maintain backwards compatibility).
Right now:
#:import name x.y.z
#:import isdir os.path.isdir
#:import np numpy
#:set name value

is equivalent to
from x.y import z as name
from os.path import isdir
import numpy as np
name = value

So why not have your syntax be like:
#: from x.y import z as name
#: from os.path import isdir
#: import numpy as np
#: name = value

?
Much more intuitive.  You could even keep backwards compatibility and accept the old form too. I'm not talking about allowing general Python statements - just these two, imports and simple assignments.
Anyway, that's for a different pull request... :-)

I force pushed another change - the word "syntaxes" is very rare in English, and it's only when you're comparing two big things, like "different languages have different syntaxes".
And that leads me to a question: why not use Python's syntax here? (You could actually add this feature and maintain backwards compatibility).
Right now:
#:import name x.y.z
#:import isdir os.path.isdir
#:import np numpy
#:set name value

is equivalent to
from x.y import z as name
from os.path import isdir
import numpy as np
name = value

So why not have your syntax be like:
#: from x.y import z as name
#: from os.path import isdir
#: import numpy as np
#: name = value

?
Much more intuitive. You could even keep backwards compatibility and accept the old form too. I'm not talking about allowing general Python statements - just these two, imports and simple assignments.
Anyway, that's for a different pull request... :-)

Just for this part, it was easier when we started the language to do name, symbol = line.split(" ", 1) than trying to parse different form of Python. I just didn't thinked about it right away.

Hey, it's always easy for someone who didn't do the work to come in from
outside and say, "Why not do it this way?"  :-)
On Wed, Mar 20, 2019 at 10:57 AM Mathieu Virbel<EMAIL_ADDRESS>wrote:

I force pushed another change - the word "syntaxes" is very rare in
English, and it's only when you're comparing two big things, like
"different languages have different syntaxes".
And that leads me to a question: why not use Python's syntax here? (You
could actually add this feature and maintain backwards compatibility).
Right now:
#:import name x.y.z
#:import isdir os.path.isdir
#:import np numpy
#:set name value
is equivalent to
from x.y import z as name
from os.path import isdir
import numpy as np
name = value
So why not have your syntax be like:
#: from x.y import z as name
#: from os.path import isdir
#: import numpy as np
#: name = value
?
Much more intuitive. You could even keep backwards compatibility and
accept the old form too. I'm not talking about allowing general Python
statements - just these two, imports and simple assignments.
Anyway, that's for a different pull request... :-)
Just for this part, it was easier when we started the language to do name,
symbol = line.split(" ", 1) than trying to parse different form of
Python. I just didn't thinked about it right away.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kivy/kivy/pull/6214#issuecomment-474761924, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAPdsv7C9pH24iNS-tE770wu1tYK_FMRks5vYgYagaJpZM4b-y3W
.

--
/t
PGP Key<EMAIL_ADDRESS>https://tom.ritchford.com https://tom.ritchford.com
https://tom.swirly.com https://tom.swirly.com

Grrr, sorry!  I proofread it multiple times before!  (This time, I searched
for all URLs and occuc
I'm amended and force pushed that change, let me know if there are
objections to that in future (I can push fixup changes which I eventually
squash, for example).
On Wed, Mar 20, 2019 at 1:09 PM vchslv13<EMAIL_ADDRESS>wrote:

@vchslv13 commented on this pull request.
In doc/sources/guide/lang.rst
https://github.com/kivy/kivy/pull/6214#discussion_r267305966:

@@ -160,7 +161,7 @@ To have your display updated when your data change, you can now have just:
.. note::
Widget names should start with upper case letters while property names
should start with lower case ones. Following the `PEP8 Naming Conventions

https://www.python.org/dev/peps/pep-0008/#naming-conventions`_

https://www.python.irg/dev/peps/pep-0008/#naming-conventions`_

This change was added by mistake I presume?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kivy/kivy/pull/6214#pullrequestreview-216678924, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAPdshgP-cDL9kksIgQljcUK9pNQpx1Pks5vYiT8gaJpZM4b-y3W
.

--
/t
PGP Key<EMAIL_ADDRESS>https://tom.ritchford.com https://tom.ritchford.com
https://tom.swirly.com https://tom.swirly.com

Grrr, sorry! I proofread it multiple times before!
Well, it happens :) I'm glad I could help. Came across that line by accident.


[stdlib] Implement a padded p2p txn for testing.

Motivation
As requested by @andll, we want to test out the overall performance when having a larger transaction.
Yes
Test Plan
Added a test case to make sure this padding behaves the same as a normal p2p transaction.

@bors-libra r+

:pushpin: Commit b90614b has been approved by andll

:hourglass: Testing commit b90614b6b6f9b701d2bdf1bd8cfd9fa1dfb36ebe with merge 4ba1641934ccbfbc7742d1e6f4c8eb81c83f3a8a...

:bulb: This pull request was already approved, no need to approve it again.

This pull request is currently being tested. If there's no response from the continuous integration service, you may use retry to trigger a build again.

:pushpin: Commit b90614b has been approved by andll

:hourglass: Testing commit b90614b6b6f9b701d2bdf1bd8cfd9fa1dfb36ebe with merge 4f0f844df0797726f8c7f050c0ba022c7a9d1d5e...

:broken_heart: Test failed - checks-circle_commit_workflow

@bors-libra r=andll

:pushpin: Commit 2b0ccc3 has been approved by andll

:hourglass: Testing commit 2b0ccc3220704b6f75f5e12eeab6354b27abb0fd with merge f1598936a1b58447874b4c281c8602cb9fc0ea9e...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: andll
Pushing f1598936a1b58447874b4c281c8602cb9fc0ea9e to master...


Fix menu 'display open' issue on mobile

button won't toggle
might need to switch over to same menu system on Vagabondians?

had to change html to <details open="open">


SQL Queries

Hi
Is it possible to use the Big Query connector with SQL? I can't find how to use it, or even if it is possible?
Thanks

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

ID: 974455fc-88ff-6a48-4dd0-036419eaa849
Version Independent ID: 2700a36d-177a-7596-2d91-4f0fc9f9d695
Content: Connect to a Google BigQuery database in Power BI Desktop - Power BI
Content Source: powerbi-docs/desktop-connect-bigquery.md
Service: powerbi
Sub-service: powerbi-desktop
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Thanks for your comment, @CristianAven . This looks like a support request or product question rather than documentation feedback. You might find a solution in the Power BI Community, at https://community.powerbi.com. If not, you can get support by going to https://support.powerbi.com.


Properly support multiple input tensors

Summary of Changes

Support multiple input tensors
Add async inference to the API

Closes #55
Motivation
Some models may require multiple input tensors and so we can support those use cases. This change also brings the server in alignment with how the inference API is used in KServe.
Implementation
Originally, multiple input tensors in a request were used as pseudo batching from the user: each tensor corresponded to a separate inference request input to a model assuming that it only had one input. This approach is conflicting with the true purpose of multiple input tensors which is to support models that actually require multiple input tensors. Since all the current tests use models with one input tensor, they should have one input tensor.
The modelInfer API is blocking so to allow for batching in the new regime, I've added an asynchronous counterpart to KServe's modelInfer API that returns a future in C++. This may also work in Python with Pybind11 but the approach taken for Python examples for now is to use the multiprocessing library to make inferences in parallel in the examples.

Build Passed!


saving unsaved tabs in tab groups

when you have a tab that is open with working information/notes(ie xml responses you are trying to parse, notes from a client call describing an issue, a list of tasks to work on that feature ...etc) but you do not actually want to save the tab as a file in the working directory you must close the tab out and can not restore the tab because the notes were not saved. An easy solution with out work to tabstronuat is to have some notes directory where you save that you can restore the tab. However i would hate to have a directory full of notes just always building because when i am done working on that particular feature that i have grouped i would have to go to that directory manually and delete the notes file.

A possible solution is a close/minimize tabs feature that closes files and saves them in a tmp somewhere.
I use this extenstion for context switching as i am working on 10-15 different issues and features at any given time.


Fixed to set mPosition to ViewHolder

This line is not working, because the mPosition field declared in ViewHolder, not ViewHolder's sub classes.
Not working
mPositionField = holder.getClass().getDeclaredField("mPosition");

Working
mPositionField = ViewHolder.class.getDeclaredField("mPosition");

However, reflection is not a good solution, I think. Therefore,  I use the class ViewHolderDelegate  to access the mPosition field that is  declared as a package scope variable.

 viewHolder.mPosition is not public

Thx！！


launch export is shadowing Base function

https://github.com/JuliaGPU/CUDArt.jl/blob/40af40e6e4d8aba2aacd44f57545784390a1ed7d/src/CUDArt.jl#L17

Fixed.


Change options according to type of use.

[x] I have read through the manual page (man fzf)
[ x I have the latest version of fzf
[ x I have searched through the existing issues

Info

OS

[ ] Linux
[x] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[ ] bash
[x] zsh
[ ] fish

Problem / Steps to reproduce
I have my FZF_DEFAULT_OPTS set for something, especially I'm using --preview there - and it's great. Sometimes, however, I would like not to have preview - for example when using fzf for searching history (auto binded to ctrl-r). Is it possible to change options just for this 'method' of running fzf?

There is a warning in https://github.com/junegunn/fzf#preview-window that you shouldn't put --preview option in your FZF_DEFAULT_OPTS.

for example when using fzf for searching history

You can set FZF_CTRL_R_OPTS. Please read through the README page before opening an issue.


Data Collection Criteria

The eutils service will get you a lot of relevant information in XML or json just by submitting a list of pmids. GC

Yes, Kent is going to play with it when he returns, but I’ll take a look too.  I’m in the air right now – flying to DC, but more soon.
From: George Chacko <notifications<EMAIL_ADDRESS>Reply-To: rchampieux/guidelines_citation_patterns <reply<EMAIL_ADDRESS>Date: Tuesday, April 19, 2016 at 4:52 AM
To: rchampieux/guidelines_citation_patterns <guidelines_citation_patterns<EMAIL_ADDRESS>Subject: [rchampieux/guidelines_citation_patterns] Data Collection Criteria (#1)
The eutilshttp://www.ncbi.nlm.nih.gov/books/NBK25500/ service will get you a lot of relevant information in XML or json just by submitting a list of pmids. GC
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHubhttps://github.com/rchampieux/guidelines_citation_patterns/issues/1


Add internal builtin topics as DdsTopic(s)

PR https://github.com/eProsima/DDS-Record-Replay/pull/87 introduced a regression in which internal builtin topics were added as DistributedTopic instead of DdsTopic. As a consequence, these were not well processed when an allowlist was being provided (casting to DdsTopic failed).

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

Comparison is base (77ebb97) 20.03% compared to head (bd6d0f7) 19.64%.

Files
Patch %
Lines

ddsrecorder/src/cpp/tool/DdsRecorder.cpp
0.00%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #97      +/-   ##
==========================================
- Coverage   20.03%   19.64%   -0.40%
==========================================
  Files          29       29
  Lines        2700     2698       -2
  Branches     1412     1412
==========================================
- Hits          541      530      -11
- Misses       1704     1721      +17
+ Partials      455      447       -8

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


Integrate JSchema component within Workflow page

This integration enables the usage of JSchema component as mean to handle arguments edits with schema restrictions.

This solves:

#190
#195
Failed validation of the args data received from the server must only raise a warning, and not an error
Always validate data upon "Save", and raise an actual error if this fails (with logs visible to the user, not only in the browser console)
"Discard updates" should bring back the original data (i.e. the ones received from the server, or from the sandbox prompt)

It's ready to be merged.


Refactor Ava tests to Node test runner

react component library to wrap breadboard ui components
https://github.com/breadboard-ai/breadboard/pull/587

breadboard

https://github.com/breadboard-ai/breadboard/pull/642

agent-kit

agent-kit

breadboard-cli

breadboard-cli

breadboard-extension

breadboard-extension

breadboard-python

breadboard-python

breadboard-server

breadboard-server

breadboard-ui

breadboard-ui

breadboard-web

breadboard-web

breadbuddy

breadbuddy

cloud-function

cloud-function

coffee-bot-board

coffee-bot-board

core-kit

core-kit

create-breadboard

create-breadboard

create-breadboard-kit

create-breadboard-kit

discovery-types

discovery-types

gemini-kit

gemini-kit

graph-integrity

graph-integrity

graph-playground

graph-playground

graph-runner

graph-runner

hello-world

hello-world

json-kit

json-kit

llm-starter

llm-starter

node-nursery

node-nursery

node-nursery-web

node-nursery-web

node-proxy-server

node-proxy-server

palm-kit

palm-kit

pinecone-kit

pinecone-kit

schema

schema

template-kit

template-kit

website

website

Refactor breadboard-cli tests from Ava to Node


Multi-frame Buffer Issue

Hi,
Device: [S32K116]
OS: [none]
Version [v4.1.5]
In respect to multi-frame CANopen SDO write requests, the buffer pointer (COObjWrBufCont) always only contains the last frame (7 bytes), yet the buffer should contain 127 x 7 bytes. As such it is only possible to access the most recent 7 bytes transmitted. Could you confirm this issue please, and advise if it a planned to be fixed?
Many thanks,
Stu

Hmm, I don't understand your issue. Assuming you configure the following object entry in your dictionary:
// Assuming we want to provide the content of the memory area in Data within an object entry:
uint8_t Data[100]

There is no basic data type sufficient (INTEGER8, INTEGER32, etc..) so we use the data type "DOMAIN":
// Declare CANopen domain for the data
CO_OBJ_DOM DomainData = {
  0,              /* member for read position */
  sizeof(Data),   /* size of domain memory */
  &Data           /* start address of domain memory */
};

The object entry definition, which allows the data to be read by the CANopen master (or other nodes) is:
  :
  { CO_KEY(0x2345, 0, CO_DOMAIN|CO_OBJ____R_), CO_TDOMAIN, (uintptr_t)&DomainData },
  :


Create Style Guide

Description:
Define the branding, typography, colors, and UI component styles for the project. Document this in a style-guide.md file in the repository.
Acceptance Criteria:

A style-guide.md file exists with defined colors, fonts, and reusable UI components.

style-guide.md file was added. See commit for more information.

Need to add design wireframe/mockups since I suck at designing. Tool to be used in mockup is Visily.


Backport to Godot 3.5

Create a separate version of the plugin that supports Godot 3.5.2.

Fixed by #7.


My mom doesn't like me

pls help

PARM WHAT ARE YOU DOING HERE

What are you doing here?

someone sent me a link to this claiming it didnt work

Sure


[Fix/bn] Fix batchnormalize layer

Fix batchnormalizationLayer that epsilon is included toward saving
moving variacne that could lead to long term inaccuracy
Self evaluation:

Build test: [X]Passed [ ]Failed [ ]Skipped
Run test: [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Jihoon Lee<EMAIL_ADDRESS>

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


[Summer of code] Support async learner in controller mode

What is the purpose of the change
tracking issue: https://github.com/apache/rocketmq/issues/4330
Brief changelog
Add new broker role -- Async leanrer in controller mode
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Codecov Report

Merging #4367 (46ddc3c) into 5.0.0-beta-dledger-controller (f5c5f5c) will increase coverage by 0.05%.
The diff coverage is 88.57%.

@@                         Coverage Diff                         @@
##             5.0.0-beta-dledger-controller    #4367      +/-   ##
===================================================================
+ Coverage                            43.47%   43.53%   +0.05%
- Complexity                            6330     6344      +14
===================================================================
  Files                                  835      835
  Lines                                59359    59369      +10
  Branches                              8092     8094       +2
===================================================================
+ Hits                                 25807    25845      +38
+ Misses                               30210    30185      -25
+ Partials                              3342     3339       -3

Impacted Files
Coverage Δ

...mq/store/ha/autoswitch/AutoSwitchHAConnection.java
73.29% <80.00%> (-0.46%)
:arrow_down:

...ache/rocketmq/store/config/MessageStoreConfig.java
56.27% <100.00%> (+0.34%)
:arrow_up:

...cketmq/store/ha/autoswitch/AutoSwitchHAClient.java
75.65% <100.00%> (+0.09%)
:arrow_up:

...etmq/namesrv/routeinfo/BatchUnRegisterService.java
94.73% <0.00%> (-5.27%)
:arrow_down:

...ache/rocketmq/common/stats/MomentStatsItemSet.java
39.13% <0.00%> (-4.35%)
:arrow_down:

.../org/apache/rocketmq/store/ha/DefaultHAClient.java
58.46% <0.00%> (-0.52%)
:arrow_down:

.../apache/rocketmq/logging/inner/LoggingBuilder.java
64.71% <0.00%> (+0.31%)
:arrow_up:

...he/rocketmq/client/impl/consumer/ProcessQueue.java
61.00% <0.00%> (+0.45%)
:arrow_up:

...rocketmq/broker/processor/PopMessageProcessor.java
39.57% <0.00%> (+0.53%)
:arrow_up:

... and 8 more

Continue to review full report at Codecov.

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


oem: ibm: Assemble the image in a forked process

The assembling of the tarball image file is a long running process,
causing pldm to not reply to any other requests until it's done, causing
errors such as when the hypervisor tries to ping the bmc to determine if
it's running.
Move the assembling of the image to a forked process so that it runs in
the background. Move the assembling function to the code update class so
that it has access to the code update sensor to set it to error if an
failure occurs, since no parent process will wait for it.
Use double fork to avoid zombie tasks, this allows the grandchild to be
reparented to init.
Tested: Verified the hypervisor did not reset the BMC during an inband
update because it didn't receive a reply to its ping request.
Change-Id: I47d5fde1a1311e94d5b03f7fcf4c0ab15ba054e3
Signed-off-by: Adriana Kobylak<EMAIL_ADDRESS>

Gerrit: https://gerrit.openbmc-project.xyz/c/openbmc/pldm/+/47248

@sampmisr
Per @anoo1 comment:
merging without review, @mzipse  was interested in a driver so we can test the functionality while the upstream reviews are in progress


slash error

i have this
Error occurred while trying to read the file Url : https:/raw.githubusercontent.com/flyimg/graphic-assets/master/logo/raster/flyimg-logo-rgb.png
but I test /upload/w_300/https://raw.githubusercontent.com/flyimg/graphic-assets/master/logo/raster/flyimg-logo-rgb.png
thanks

Hi @fyher
Thank you for reporting the issue.
Could you please specifiy more details about it?

I don't know , I have the docker image, it's work on server A , i install it on server B and now I have this bug.  I send https://raw.githubusercontent.com/flyimg/graphic-assets/master/logo/raster/flyimg-logo-rgb.png and the code receive just http::/ , is delete one slash
i have open the code and is nos good, I add str_replace("http:/","http://"...) and its work :(

you can see it here http://flyimg.nolohi.fr and http://flyimg.nolohi.fr/upload/w_300/https://raw.githubusercontent.com/flyimg/graphic-assets/master/logo/raster/flyimg-logo-rgb.png

how is the configuration for the server B ?
I mean in http://flyimg.nolohi.fr your run Flyimg behind a reverse proxy? can you past here the configuration?

is the same config for the server A ,
the docker config
flyimg:
image: flyimg/flyimg-build
restart: always
ports:
- 8089:80
networks:
noholi:
aliases:
- flyimagenolohi.com

I mean the reverse proxy behind your domain http://flyimg.nolohi.fr
since access directly the running container: http://flyimg.nolohi.fr:8089/upload/w_300/https://raw.githubusercontent.com/flyimg/graphic-assets/master/logo/raster/flyimg-logo-rgb.png
is working perfectly.
So it should be something in the reverse proxy that redirects the traffic to the container on port 8089

the conf
<VirtualHost *:80>
ServerName flyimg.nolohi.fr
CustomLog /var/log/apache2/secure_access.log combined
<Proxy *>
Allow from <IP_ADDRESS>

ProxyPreserveHost On
ProxyPass / http://<IP_ADDRESS>:8089/
ProxyPassReverse / http://<IP_ADDRESS>:8089/
ProxyRequests Off

Sorry for my late response, honestly it looks an issue from the apache configuration which remove the double slash, you can play with configuration and try to find out where it happens.
but isn't from the Flyimg code source.

Ran into the same problem on nginx. Fixed by adding merge_slashes off; to nginx config.

Hi @fyher
I'm wondering if you still use Flyimg in your company/organisation ?
If so, it would be great if you could add your entry here in the Adoperts file
https://github.com/flyimg/flyimg/blob/main/ADOPTERS.md
many thanks!


runehook indexing services stops after catching up to the latest block.

Describe the bug
runehook indexing services stops after catching up to the latest block. Need to restart the process to start the indexing again.
To Reproduce
Steps to reproduce the behavior:
1, start process:
$ runehook service start --config-path Runehook.toml
...
Dec 24 21:02:07.618 INFO Edict DOG•GO•TO•THE•MOON<PHONE_NUMBER> tx: 0xf8216655b88a409d9927eb72a164ae55623d0e8de9202261a5e3ca52fd5e7424 (3627) @876206
Dec 24 21:02:07.633 INFO Flushing DB cache...
Dec 24 21:02:07.633 DEBG Flushing 5 supply changes
Dec 24 21:02:07.810 DEBG Flushing 22 ledger entries
Dec 24 21:02:07.812 DEBG Flushing 9 balance increases
Dec 24 21:02:08.060 DEBG Flushing 8 balance deductions
Dec 24 21:02:08.264 INFO Block 876206 indexed in 15.12s
Dec 24 21:02:08.286 INFO 1 blocks scanned, 1 actions triggered
Dec 24 21:02:08.291 INFO Caught up to bitcoind chain tip at 876206
Dec 24 21:02:08.293 INFO Listening for new blocks via Chainhook SDK
Dec 24 21:02:08.297 INFO Connecting to postgres at localhost:5432
2, After the process caught up to 876206 the indexing stopped. And I see the current block height reached 876,214 already.
I need to kill the process and start the process again to catch up to the newest block.
Expected behavior
I expect the runehook service can listen to the newest block and keep the indexing process going.

Hey! The problem is that after picking up the last block we close the loop by returning. I have changed it so that we always have to look after 60 seconds to see if there is a new block available.
My fix:
https://github.com/GregorWedlich/runehook/commit/852c9d67280dd5dae85dcd88b2b769b0682f383f

Thank you for the fix. I am trying it out.


fprintd: 1.94.2 -> 1.94.3, python312Packages.pypamtest 1.1.3 -> 1.1.5, fix build issues

Description of changes
Update and fix of python312Packages.pypamtest to fix build issue in fprintd (which coulnd't find the python bindings)
Also updates fprintd and pypamtest
Release notes fprintd
Changelog pypamtest
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] 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/)
24.11 Release Notes (or backporting 23.11 and 24.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.

Add a :+1: reaction to pull requests you find important.

Alternative fix: https://gitlab.archlinux.org/archlinux/packaging/packages/pam_wrapper/-/blob/main/pam_wrapper-1.1.5-site-packages.patch

Alternative fix: https://gitlab.archlinux.org/archlinux/packaging/packages/pam_wrapper/-/blob/main/pam_wrapper-1.1.5-site-packages.patch

Cool, didn't find that patch when I was looking for a solution 👍
I'm fine either way. In the end both provide the path for fprintd and fix the build issue on current master

I'm fine either way. In the end both provide the path for fprintd and fix the build issue on current master

Actually I prefer yours.

@ofborg build fprintd

So when can I build my system again and how can I get this to bypass? I removed all of everything relating to fprintd from my config but it still wont build because of this. Can I force it to build from this commit even if the hydra will take another month to fix itself?

So when can I build my system again and how can I get this to bypass? I removed all of everything relating to fprintd from my config but it still wont build because of this. Can I force it to build from this commit even if the hydra will take another month to fix itself?

Can you paste your build/error log?
How do you build your system? configuration.nix or flake.nix?
Do you use unstable?
You could use an overlay with an override of the fix, or (if you use flakes) add an input of the commit and use that as an overlay.
Also, I don't believe it will take that long to reach the unstable channel

@gador I am just going to keep my flake.lock from when the system worked and not update till I know the hydra fixed itself. Unrelated, but how is it that major breaking build things get past the hydra anyways? Doesn't it have to build the system and have it work BEFORE it yells out "known good" to all the users?

Well the major update to python3.12 happens on master (and later on unstable). This was communicated by @mweinelt (hexa) on discourse
The discussion about breaking changes was already stirred up a couple of days ago.
TL/DR: We need to update and upstream usually lags behind, so we need to fix stuff along the way.
This is unstable after all.
These kind of major changes take a long time to build on hydra and it is clear we need to fix a lot, anyway.

Doesn't it have to build the system and have it work BEFORE it yells out "known good" to all the users?

Only on stable (IMO)

I am just going to keep my flake.lock from when the system worked

I do the same. FWIW I keep a fork of nixpkgs and cherry-pick the fixes diretly to it ;-)


feat: Agenda URL #hash scrolls to 'now' or specific day

Fixes #7245
Allows scrolling to either #now or #agenda-day-YYYY-MM-DD on page load.
Adds a developer-only utility to modify server agenda dates to be the same as local browser time, so that the 'Now' button is available.

This is looking good on desktop, but on screens the size of an iphone 11, its not acting as intended. With the dev code uncommented, looking in this view, the now link doesn't change what's in the url bar, and pasting http://localhost:8000/meeting/agenda/#now into a blank tab doesn't cause the viewport to move to the red line.

@rjsparks the functionality should work now. I still need to fix the tests.

@rjsparks fixed. ready for review


Implement cryptographic validation of all payload and metadata in a zarf package

Implement hashing/signing for all elements of the package to ensure authenticity and integrity of the artifacts.

Migrated from https://repo1.dso.mil/platform-one/big-bang/apps/product-tools/zarf/-/issues/36

@Racer159 Are you actively working on this?  If so could you please move it to doing now on the project board?  I may get rid of thing if we don't fine value in it long term, but I like the visual nature of it.

https://github.com/defenseunicorns/zarf/issues/23 May be related / be a subtask of this.

Dropped myself as an assignee, this is still a backlog task, I just added myself to not forget about it at the time


private key management question

Hi,
We're evaluating MPC wallet solutions and third web is one of the candidates. I've been playing with the react sdk and it seems to do everything the doc suggests. I did however look into how the MPC part works and got confused.
As I understand it MPC works by reconstructing the private key from the user shard and another shard kept in KMS so signing should happen locally on user's device, right? In my example (by following the react tutorial) the signing seems to happen in third web's backend? Specifically here.
The client only sends the auth token saved in local storage, what roles does the user shard play in this flow, or is the user shard even used? Or perhaps I'm missing something obvious here?

Hey @haowang1013, the signing occurs in an iframe embedded on the page. This is so the developer cannot access the user's private key within the app itself.

Hey @haowang1013, the signing occurs in an iframe embedded on the page. This is so the developer cannot access the user's private key within the app itself.

Hi, not sure if I follow, the code I quoted above shows that the signing doesn't happen on the client side, no?
  const response = await clientFetch(
    `${getThirdwebBaseUrl("inAppWallet")}/api/v1/enclave-wallet/sign-message`,
    {
      method: "POST",
      headers: {
        "Content-Type": "application/json",
        "x-thirdweb-client-id": client.clientId,
        Authorization: `Bearer embedded-wallet-token:${authToken}`,
      },
      body: stringify({
        messagePayload: {
          message,
          isRaw,
        },
      }),
    },
  );

  if (!response.ok) {
    throw new Error("Failed to sign message");
  }

  const signedMessage = (await response.json()) as {
    r: string;
    s: string;
    v: number;
    signature: string;
    hash: string;
  };
  return signedMessage;


"Unauthorized" message returned

I have a function App which uses Graph API.
My problem is that when my Logic App is triggered I get an "Unauthorized" message.   My request body has the following format { "Id": Id }   ( "Id" is from the Dynamic List}.
I tried following the instructions listed here: https://stackoverflow.com/questions/53629900/unauthorized-error-when-trying-to-call-an-azurefunction-from-logicapp
It mentions that in order for a Logic App to call a Function App you need to provide the Function Apps "code" to the "Header" (a selection in the Logic App).   This solution hasn't worked either.
What else should I be looking at to try and get this to work?

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

ID: 087366a1-80f9-fd3a-c736-d1275f0c4aca
Version Independent ID: c12d4e3b-aa27-aa3e-1ec2-82116599c01d
Content: Add and call Azure functions from Azure Logic Apps
Content Source: articles/logic-apps/logic-apps-azure-functions.md
Service: logic-apps
GitHub Login: @ecfan
Microsoft Alias: estfan

Hi @GermanEZI Thank you for your feedback! We will review and update as appropriate.

@GermanEZI When you say your function uses Graph API , does that means your function is secured with App service authentication (Authentication / Authorization) as shown below ?

If yes, follow the below steps:

Enable the  System Assigned Identity in your function App.

You have to use the standard HTTP connector and wire the URL FunctionApp (including the key in the URL or header if your function needs them). For authentication you can set the method to "Managed Identity", which I had seen and used but failed. What I didn't see was the normally hidden "Audience" parameter in to which you need to place the App Id (from Azure AD) of the function you are calling.

@GermanEZI Checking in to see if the above post helped answering your question.

@KetanChawda-MSFT
Yes, I am using App Service authentication as you've described.  I'm using a technique called "Easy Auth" that is described in this article. Using Microsoft Graph in an Azure Function
The method used in this article, you're walked through editing the bindings such that a new parameter ("graphToken") is added to the function signature of your function app.  This parameter will contain the access token used to make the call to MS Graph

Using the "HTTP" connector in your example is giving me an error.  It's not clear to me how I should fill in the parameters
The URL from the portal looks as follows
https://myapp.azurewebsites.net/api/HttpTrigger1?code=xxxxxxxxxxx/xxxxxxxxxx/xxxxxxxxxxxxxxxxxxxxxxxxxxxxxx==
When I copy this URL to a browser and add the "Id" of a calendar event, the JSON result is returned.  This confirms that my code is working correctly:
https://myapp.azurewebsites.net/api/HttpTrigger1?code=xxxxxxxxxxx/xxxxxxxxxx/xxxxxxxxxxxxxxxxxxxxxxxxxxxxxx==&Id=AAMkADU4MzkxN2=
SOURCE CODE:
`#r "Newtonsoft.Json"
using System.Net;
using Microsoft.AspNetCore.Mvc;
using Microsoft.Extensions.Primitives;
using Newtonsoft.Json;
using System.Net.Http;
using System.Net.Http.Headers;
using System.Text;
public static async Task Run(HttpRequest req, string graphToken, ILogger log)
{
log.LogInformation("C# HTTP trigger function processed a request.");
string graphHead = "https://graph.microsoft.com/v1.0/me/events('";
string graphTail = "')?$expand=singleValueExtendedProperties($filter=id%20eq%20'Integer%20{00020329-0000-0000-C000-000000000046}%20Name%20TaskID')";
string Id = req.Query["Id"];
log.LogInformation(Id);

string requestBody = await new StreamReader(req.Body).ReadToEndAsync();
dynamic data = JsonConvert.DeserializeObject(requestBody);
Id = Id ?? data?.Id;

HttpClient client = new HttpClient();
client.DefaultRequestHeaders.Authorization = new AuthenticationHeaderValue("Bearer", graphToken);

var result = await client.GetStringAsync( graphHead + Id + graphTail);

return (ActionResult)new ObjectResult(result);

}
`

@GermanEZI can you confirm which parameter is not clear to you from my last post?
If it is audience parameter in the below screenshot , you can get this from the Function App --> Authentication / Authorization --> Select AD App --> Copy the Client ID and paste it in the Audience field.

After unsuccessfully trying other Id's, I used the ClientID for the Audience parameter.
This looked promising.  It took some time before it returned an "Internal Server Error"  (500)
I noticed that the content-length was 0 even though thee is a value in my "Body" parameter

@cgillum Can you please share your thoughts here?

@GermanEZI This seems most likely the restriction from the Graph Extension. I would recommend you to create a issue on the Microsoft Graph extension for Azure Functions. Once you post your issue on repo, it will have visibility across the product owners which is a better suited audience for such types of issues.
Since there is no doc update required at the moment, we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Remember last session request and add a shortcut to recreate

Users wish to save their last session tool + version selection. To do this, a "Remember this selection" checkbox should be introduced. When requesting a persistent session the next time, the last selection could be prefilled with the last selected tool. Alternatively, a shortcut button would be good.
We don't need to save the preferences server-side, saving in local storage will be sufficient.

Resolved by https://github.com/DSD-DBS/capella-collab-manager/pull/1428


Add minzoom override

In the case of a happytiff, no need to include semi-empty tiles for zoom levels below the happytiff's preferred zoom level (aka "happy zoom"). Add a minzoom override option to specify happyzoom.
mapbox-tile-copy will set the final minzoom to the value passed in via the options object and ignore the minzoom passed in via mapnik-omnivore

[x] Add a --minzoom option within bin/mapbox-tile-copy cli
[x] Add some validation of the minzoom value (numeric)
[x] Add tests

cc @camillacaros @rclark

Merged https://github.com/mapbox/mapbox-tile-copy/pull/37. Closing


Add Content-Type to PATCH request

fixes #20

Unless I am mistaken, this also adresses the concerns by @evert raised in https://github.com/tus/tus-resumable-upload-protocol/issues/14#issuecomment-66503134

@butonic Correct, the question from #14 has been answered.


ffuf: 1.2.1 -> 1.3.0

Motivation for this change
Update to latest upstream release 1.3.0
Change log: https://github.com/ffuf/ffuf/releases/tag/v1.3.0
Things done

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

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

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

Result of nixpkgs-review pr 119797 run on x86_64-linux 1

1 package built:

ffuf


Headless test using puppeteer

Headless test using puppeteer
Created a basic node server which serves an index.html file. First-Input-delay.js file added to its head. Puppeteer loads this file & triggers mousedown event.
Todos

[x] Puppeteer & node server configured to test user generated events
[x] Travis configured to trigger the test for every change
[x] Handle click & touchstart
[x] Handle keydown & mousedown

@philipwalton Would like to get a first round of feedback on this. So, i can get this PR ready by next week.
Need some help to configure 3 point of todos.

@philipwalton Could you please review my PR?

@philipwalton Final review pending on this PR. Please pick this when you are available.


Fix MQTT Light with RGB and Brightness

When an MQTT light is given an RGB and Brightness topic, the RGB
is scaled by the brightness as well as the brightness being set
This causes 255,0,0 at 50% brightness to be sent as 127,0,0 at 50%
brightness, which ends up as 63,0,0 after the RGB bulb has applied
its brightness scaling.
Related Issue: #13725
Checklist:

[X] The code change is tested and works locally.
[X] Local tests pass with tox. Your PR cannot be merged unless tests pass

I've now extended the PR to do the same thing for mqtt-json and mqtt-template

Thanks for the contribution! 🌟

@thinkl33t my h801 controller with tasmota doesn't count brightness by itself - how can I use brightness topic as well as rgb-wheel together in HASS? changing color currently resets brigthness to 100%

@armills is there any way to use mqtt light component with end software like tasmota to have rgb value calculated with brightness value before being sent?
Tasmota waits for already calculated values and any light color change makes whole strip go back to 100% even if I want to keep it around 30%

Hi @andriej,
Sorry to hear you're having problems, but merged PRs aren't an appropriate place for this discussion. Please feel free to ask these questions in our forum or chat server


SCIM role provisioning: Should you use the role id or use role value instead?

Hello,
I am trying to integrate a new application using SCIM roles.
To do this, I configured my roles attribute, using roles(AppRoleAssignmentsComplex([appRoleAssignments]).
Everything works as documented, but I am having a problem with an inconsistency.
Create contains these attributes
 {
           "primary": false,
           "type": "WindowsAzureActiveDirectoryRole",
           "displayName": "Admin",
           "value": "Admin"
 }

But PATCH creates these attributes:

{"id":"06b07648-ecfe-589f-9d2f-6325724a46ee","value":"Admin","displayName":"Admin"}

meaning, no Id is provided with create request.
Does this mean, we should not rely on the id, but on the value?
Could you please clarify this in the documentation?
Thank you in advance.
Best regards,
Danny

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

ID: 83b07a9e-5e16-32ed-265e-32a32998ce37
Version Independent ID: 09008259-4b53-2925-9679-ecad59cefe41
Content: Tutorial - Customize Microsoft Entra attribute mappings in Application Provisioning - Microsoft Entra ID
Content Source: docs/identity/app-provisioning/customize-application-attributes.md
Service: active-directory
Sub-service: app-provisioning
GitHub Login: @kenwith
Microsoft Alias: kenwith

@dbollaer
Thanks for your feedback! We will investigate and update as appropriate.

Hi @dbollaer
In the case of SCIM roles, the id attribute is not always required, and it is possible to rely on the value attribute instead. The value attribute contains the name or identifier of the role, and it can be used to uniquely identify the role in some cases.
However, it is important to note that relying solely on the value attribute may not be sufficient in all cases, especially if there are multiple roles with the same name or identifier. In these cases, it may be necessary to use the id attribute to uniquely identify the role.
If there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.

Updated the "Provisioning a role to a SCIM app" section to include a note and things to consider:
NOTE: When provisioning roles from enterprise applications, the SCIM standard defines enterprise user role attributes differently. For more information, see https://review.learn.microsoft.com/en-us/entra/identity/app-provisioning/use-scim-to-provision-users-and-groups#design-your-user-and-group-schema.

When integrating applications using SCIM roles, the id attribute is not always required, as for certain cases you can use the value attribute instead. For example, if the value attribute contains the name or identifier for the role, you can use it to provision the role. However, relying solely on the value attribute may not always be sufficient; for example, if there are multiple roles with the same name or identifier. In certain cases, it may be necessary to use the id attribute to properly provision the role.

#please-close


Add mssql driver on docs

But this feature will be released in v1.1

LGTM

Maybe we need to start versioning our docs 😛

But for user how to which feature is 1.0 or 1.1? We need to define it not only in issues

I asked for this at the beginning. There are two ways.

only document latest version
simply tag the docs and build docker containers for these who need older versions

I like 2

LGTM

So before a merge we need to tweak the drone config, tag 1.0.0 and than the new features can get documented.

Vote 2. Sound like make sense.

One contanier is need maybe, use different sub path, root path -> docs master, others are branches or tags(which is better? I prefer branches).
https://docs.gitea.io/
https://docs.gitea.io/1.0
https://docs.gitea.io/1.1

Using branches is better than Tag for maintenance. We can refer to laravel docs repo. https://github.com/laravel/docs

And how do you want to generate the version switch if every release got a separate branch?

List branches of this repo? :)

On a static page...

We could generated it before hugo and after git clone.
git clone
git branch > a.txt
hugo build

Now there are more places that mention the database support :)

L-G-T-M


[OAuth]: add more claims to introspection response

RFC7662 Token Introspectionhttps://datatracker.ietf.org/doc/html/rfc7662#section-2.2 specifies that only claim active is required to be returned. Additionally we also return scope, client_id, username and all requested claims according to userinfo response (incl. sub, iss)
We should also add token_type, exp, iat, nbf,  aud, jti
This would also allow libraries to cache the response more efficiently.
Requires changes to the OIDC Library.
Acceptance criteria

[ ] the claims above are returned on the response
[ ] documentation

fyi @buehler

this to some degree relates to #3997


Focal Loss

Hello， @JunMa11
I want to know whether focal loss is suitable for 3D image segmentation or not，and for segmentation with only two classifications of background and foreground，whether the network can be single-channel output and use focal loss.
Hope your help, thank you!

Hi @ruiruian

whether focal loss is suitable for 3D image segmentation or not，

Yes, It can be used in 3D segmentation tasks.
Now, it is designed for two-class and multi-class segmentation.

Thank you @JunMa11
I have a question, for two-class segmentation, whether the network can be single-channel output.

Hi @ruiruian ,
Sorry for my late reply.
Not yet. You should use to two-channel output with the current code.


Retrieve a thread by the main link instead of the last link

The plugin is just awesome
But it would be great if we just copy the link of the thread instead of the link of the last tweet in the thread.
Otherwise great work

Hi @skillhacker-code! I agree, it would be much more convenient. Unfortunately, this is a limitation of the current Twitter v2 API. Searching by thread/conversation ID is only available on the recent search endpoint (only has tweets from the last 7 days) or the academic search endpoint (which is expensive and limited to university researchers).
The current workaround of using the last tweet and working back up the thread will work for any tweet even if it's not recent and allows anyone with a free bearer token to capture threads.
Hope that helps explain the reasoning behind the behavior!

Thank you so much for answering and developing this awesome plugin . i have been searching for a way to save twitter thread for a long time and tried many solutions like https://threadreaderapp.com/ and it was cumbersome but yours plugin is awesome .


Feat/save wallet details

Description:
Updated version of #1453
Motivation and Context:

Fix test failures due to invalid try/catch format
Process preload.js with webpack and simplify packaging

How Has This Been Tested?
Manually (test dev, start, and package)
Types of changes:
Refactor
Checklist:

[ ] My code follows the code style of this project.
[ ] I have reviewed and updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes where needed.
[ ] All new and existing tests passed.
[ ] My commits have been squashed into a concise set of changes.

@korhaliv I think I have this working now. Can you review the last 2 commits in this PR and if they look good to you, you could cherry-pick them into your branch and existing PR.

Coverage increased (+0.08%) to 20.678% when pulling 208a7111289f54cc6790de5b5bd6fecc03d5e450 on mrfelton:feat/save-wallet-details into c8b6e84c2313d8a3032c49ea3d9859d034d20378 on LN-Zap:next.

Closing in favour of #1482


Disable Heading 1 style in certain TinyMCE fields

Short description
This PR disables the possibility of using the Heading 1 style in TinyMCE fields for text and descriptions, throwing a validation error if the <h1> tags are (manually) inserted into the field's content.
Proposed changes
First, the PR removes the possibility of selecting "Heading 1" as a style in certain TinyMCE fields by manually overriding which styles are visible in the TinyMCE styles list. The affected tinyMCE instances are in the following templates:

cms/templates/events/event_form.html
cms/templates/pages/page_form.html
cms/templates/pages/page_sbs.html
cms/templates/pois/poi_form.html

Second, the PR adds a specific clean_<field> method to the Django Forms being used in the aforementioned templates, that checks for the presence of <h1> tags in their fields and, if found, raises a validation error with an appropriate message.
The forms changed are:

EventTranslationForm (field: description)
PageTranslationForm (field: text)
POITranslationForm (field: description)

These three forms are used in the templates above. This way, we prevent the use of "Heading 1" styles both in front-end and in back-end.
Misc notes
I am not sure about the validation error message I used, which is: "Use of Heading 1 style not allowed."
Any suggestions would be accepted and changed into the Pull Request.
Also, ich spreche nicht Deutsch :-P so I wasn't able to add a properly translated string to the DE locale files. That would be a pending thing to do. If you can provide an equivalent message in Deutsch, I can add it to the proper locale file.
Resolved issues
Fixes: #516

Hey @timoludwig! Thank you so much for your comments! I'll be doing the proposed changes during the day and I'll let you know.

Hello @timoludwig! Sorry for the delay, I've pushed a couple of commits for both adding the translated string to the deutsch locale and explicitly using styles_format, I think this would solve the issue. Let me know if there's anything else I could do!


Fix ZiHeng's Reposense

Pull Request Test Coverage Report for Build 1114

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 82.965%

Totals

Change from base Build 1037:
0.0%

Covered Lines:
2781

Relevant Lines:
3352

💛  - Coveralls


ECB Menu Role Changes

Problem-
No Problem  amendments required according to new Roles / big companies structures
Current Menu have Ledger Pages with respective Pages
Contact Role can open Contact Ledger Page
Company Role can Open Company Ledger, Contract
same way for others
This was design in favor of Less Roles and same page functionality
Contact , Contract, Ledger was coming on same page before this. then new pages / view was created with same roles.
@vinpahilwani / @sureshsirvi  wants to structure Menu according to Big companies having more Staffs, like Separate staff for handling different kind of Work
ex. Client On-Boarding staff needs special role , and Menu need to be designed according to this,
Client / Company Service sellers can sell more services to Clients through Company Page. and Menu need to design according to this.
as well Company Relationship Managers can manage only their respective companies. (this requires major Role check changes)
same goes for Project.
@vinpahilwani , @sureshsirvi  will write requirement , and why this is important in their words correctly, above is just brief
Changes Required -
Menu Changes
Contact Ledger shift to Account Section
Company Ledger to Account Section
Project Deviation to Timesheet
and Roles requirement - Contact Ledger roles , not clear about details
Company Ledger
Project Deviation
Initiated by @vinpahilwani
Verified by @mitulpanch

I agree with structure requirements.
But developing time is high for Role changes, i consider current thing more clearer, and users dont want to learn at starting level when they are in 1st month of using Software,
If anything that removes learning software then this will be good thing to do.


Accounts, Book Details, Bookmarks, Likes, and Sorting

Implemented New Features

Created Accounts

User Registration

View
HTML
Form

User Login

View
HTML
Form

User Logout

View

Bookmarks on the Profile

Book Details Page

Likes - Books

View
Model
URL
Bookmarks - Books

View
Model
URL

Google Books API integration

Book Coverage Image
Book Summary Descriptions

Buttons back to recommendations and profile

Added robots.txt file
Added Testing

Recommendations Page with the Likes for Authenticated Users

Book Details Page with Bookmarks and Likes for Authenticated Users

Profile with Bookmarks for Authenticated Users

Login Page for Unauthenticated Users


Pillow deprecation warnings in torchvision/transforms/_functional_pil.py

🐛 Describe the bug
Attempting to use transforms for the builtin model weights results in deprecation warnings from Pillow:
$ python3 -mvenv venv
$ ./venv/bin/pip install -U pip
[...]
$ ./venv/bin/pip install torch torchvision torchaudio --index-url https://download.pytorch.org/whl/cpu
[...]
$ ./venv/bin/python -X dev -c 'import torchvision.models as m; m.ResNet50_Weights.DEFAULT.transforms()'
/tmp/venv/lib/python3.10/site-packages/torchvision/transforms/_functional_pil.py:242: DeprecationWarning: BILINEAR is deprecated and will be removed in Pillow 10 (2023-07-01). Use Resampling.BILINEAR instead.
  interpolation: int = Image.BILINEAR,
/tmp/venv/lib/python3.10/site-packages/torchvision/transforms/_functional_pil.py:286: DeprecationWarning: NEAREST is deprecated and will be removed in Pillow 10 (2023-07-01). Use Resampling.NEAREST or Dither.NONE instead.
  interpolation: int = Image.NEAREST,
/tmp/venv/lib/python3.10/site-packages/torchvision/transforms/_functional_pil.py:302: DeprecationWarning: NEAREST is deprecated and will be removed in Pillow 10 (2023-07-01). Use Resampling.NEAREST or Dither.NONE instead.
  interpolation: int = Image.NEAREST,
/tmp/venv/lib/python3.10/site-packages/torchvision/transforms/_functional_pil.py:319: DeprecationWarning: BICUBIC is deprecated and will be removed in Pillow 10 (2023-07-01). Use Resampling.BICUBIC instead.
  interpolation: int = Image.BICUBIC,

Versions
PyTorch version: 2.0.1+cpu
Is debug build: False
CUDA used to build PyTorch: None
ROCM used to build PyTorch: N/A

OS: Ubuntu 22.04.2 LTS (x86_64)
GCC version: (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0
Clang version: Could not collect
CMake version: Could not collect
Libc version: glibc-2.35

Python version: 3.10.12 (main, Jun 11 2023, 05:26:28) [GCC 11.4.0] (64-bit runtime)
Python platform: Linux-<IP_ADDRESS>-microsoft-standard-WSL2-x86_64-with-glibc2.35
Is CUDA available: False
CUDA runtime version: No CUDA
CUDA_MODULE_LOADING set to: N/A
GPU models and configuration: No CUDA
Nvidia driver version: No CUDA
cuDNN version: No CUDA
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True

CPU:
Architecture:                    x86_64
CPU op-mode(s):                  32-bit, 64-bit
Address sizes:                   48 bits physical, 48 bits virtual
Byte Order:                      Little Endian
CPU(s):                          12
On-line CPU(s) list:             0-11
Vendor ID:                       AuthenticAMD
Model name:                      AMD Ryzen 5 PRO 5650U with Radeon Graphics
CPU family:                      25
Model:                           80
Thread(s) per core:              2
Core(s) per socket:              6
Socket(s):                       1
Stepping:                        0
BogoMIPS:                        4591.21
Flags:                           fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht syscall nx mmxext fxsr_opt pdpe1gb rdtscp lm constant_tsc rep_good nopl tsc_reliable nonstop_tsc cpuid extd_apicid pni pclmulqdq ssse3 fma cx16 sse4_1 sse4_2 movbe popcnt aes xsave avx f16c rdrand hypervisor lahf_lm cmp_legacy svm cr8_legacy abm sse4a misalignsse 3dnowprefetch osvw topoext perfctr_core ssbd ibrs ibpb stibp vmmcall fsgsbase bmi1 avx2 smep bmi2 erms invpcid rdseed adx smap clflushopt clwb sha_ni xsaveopt xsavec xgetbv1 xsaves clzero xsaveerptr arat npt nrip_save tsc_scale vmcb_clean flushbyasid decodeassists pausefilter pfthreshold v_vmsave_vmload umip vaes vpclmulqdq rdpid fsrm
Virtualization:                  AMD-V
Hypervisor vendor:               Microsoft
Virtualization type:             full
L1d cache:                       192 KiB (6 instances)
L1i cache:                       192 KiB (6 instances)
L2 cache:                        3 MiB (6 instances)
L3 cache:                        16 MiB (1 instance)
Vulnerability Itlb multihit:     Not affected
Vulnerability L1tf:              Not affected
Vulnerability Mds:               Not affected
Vulnerability Meltdown:          Not affected
Vulnerability Mmio stale data:   Not affected
Vulnerability Retbleed:          Not affected
Vulnerability Spec store bypass: Mitigation; Speculative Store Bypass disabled via prctl and seccomp
Vulnerability Spectre v1:        Mitigation; usercopy/swapgs barriers and __user pointer sanitization
Vulnerability Spectre v2:        Mitigation; Retpolines, IBPB conditional, IBRS_FW, STIBP conditional, RSB filling, PBRSB-eIBRS Not affected
Vulnerability Srbds:             Not affected
Vulnerability Tsx async abort:   Not affected

Versions of relevant libraries:
[pip3] numpy==1.24.1
[pip3] torch==2.0.1+cpu
[pip3] torchaudio==2.0.2+cpu
[pip3] torchvision==0.15.2+cpu
[conda] Could not collect

They deprecated those in PIL 9 but ended up putting them back in PIL 10 (https://pillow.readthedocs.io/en/stable/releasenotes/9.4.0.html#restored-image-constants).
We had code to handle / hide the deprecation, but decided to remove it. Just upgrade PIL to 10 and those warning should disappear.


Bug in test 'should add and remove pointer events to element'

I tried running npm test on my laptop which has touch screen. I am checked to the commit 14246a8044ea6f819bec8197501e2790f4d1bbdc on dev branch. I am receiving the following error:
1) PIXI PIXI.interaction.InteractionManager add/remove events should add and remove pointer events to element:
     AssertionError: expected addEventListener to have been called exactly thrice, but it was called 7 times
    addEventListener(pointerdown, function bound onPointerDown() {}, true) at InteractionManager.addEvents (\pixi.js\dist\pixi.js:9:1374515)
    addEventListener(pointerleave, function bound onPointerOut() {}, true) at InteractionManager.addEvents (\pixi.js\dist\pixi.js:9:1374640)
    addEventListener(pointerover, function bound onPointerOver() {}, true) at InteractionManager.addEvents (\pixi.js\dist\pixi.js:9:1374765)
    addEventListener(touchstart, function bound onPointerDown() {}, true) at InteractionManager.addEvents (\pixi.js\dist\pixi.js:9:1375844)
    addEventListener(touchcancel, function bound onPointerCancel() {}, true) at InteractionManager.addEvents (\pixi.js\dist\pixi.js:9:1375968)
    addEventListener(touchend, function bound onPointerUp() {}, true) at InteractionManager.addEvents (\pixi.js\dist\pixi.js:9:1376095)
    addEventListener(touchmove, function bound onPointerMove() {}, true) at InteractionManager.addEvents (\pixi.js\dist\pixi.js:9:1376215)

Reproducing the issue
There are two ways:
Firstly, checkout to the mentioned commit on dev branch, then either,

run npm test on a device which has touch screen
OR
if your device does not have touch screen, go to file src/interaction/InteractionManager.js and change line 155

this.supportsTouchEvents = 'ontouchstart' in window; -> this.supportsTouchEvents = true;

then run npm test

Hypothesis for why the error is encountered
The failing test is in test/interaction/InteractionManager.js specifically on line 295
expect(element.addEventListener).to.have.been.calledThrice;
I think the test does not take into account the case when touch screen is used. Namely, when
this.supportsTouchEvents evaluates to true, four additional touch events are added to interactionDOMElement, as can be seen in file dist/pixi.js on line 34679:
        if (this.supportsTouchEvents) {
            this.interactionDOMElement.addEventListener('touchstart', this.onPointerDown, true);
            this.interactionDOMElement.addEventListener('touchcancel', this.onPointerCancel, true);
            this.interactionDOMElement.addEventListener('touchend', this.onPointerUp, true);
            this.interactionDOMElement.addEventListener('touchmove', this.onPointerMove, true);
        }

This explains why addEventListener is called 7 times (as the error log states) in stead of 3 times (which is currently expected by the error clause).

Closing as PR was submitted for this. Thanks!


macOS 10.12.6 install odb error

Hello, can't install odb, what could be the problem?
pool-192:~ admin$ brew install odb
==> Installing odb from max13/odb
==> Installing dependencies for max13/odb/odb: libcutl, libodb
==> Installing max13/odb/odb dependency: libcutl
Error: Calling fails_with :llvm is disabled!
There is no replacement.
/usr/local/Homebrew/Library/Taps/max13/homebrew-odb/libcutl.rb:21:in `<class:Libcutl>'
Please report this to the max13/odb tap!
Or, even better, submit a PR to fix it!

Hi @Iaroslawz, let me look into it

I've fixed many issues with my last commits ad2bb39:

Homebrew removed some keywords
ODB changed their repo link and shasums

You should untap and tap again my tap. It should work.

well the first error is solved, now error with make install
==> Installing max13/odb/odb
==> Downloading http://www.codesynthesis.com/download/odb/2.4/odb-2.4.0.tar.gz
Already downloaded: /Users/admin/Library/Caches/Homebrew/odb-2.4.0.tar.gz
==> ./configure --prefix=/usr/local/Cellar/odb/2.4.0 --libexecdir=/usr/local/Cellar/odb/2.4.0/lib --with-options-fi
==> make install
Last 15 lines from /Users/admin/Library/Logs/Homebrew/odb/02.make:
        .section __TEXT,__textcoal_nt,coalesced,pure_instructions
                 ^      ~~~~~~~~~~~~~
<stdin>:1417:11: warning: section "__const_coal" is deprecated
        .section __TEXT,__const_coal,coalesced
                 ^      ~~~~~~~~~~~~
<stdin>:1417:11: note: change section name to "__const"
        .section __TEXT,__const_coal,coalesced
                 ^      ~~~~~~~~~~~~
<stdin>:1423:11: warning: section "__const_coal" is deprecated
        .section __DATA,__const_coal,coalesced
                 ^      ~~~~~~~~~~~~
<stdin>:1423:11: note: change section name to "__const"
        .section __DATA,__const_coal,coalesced
                 ^      ~~~~~~~~~~~~
make: *** [install-recursive] Error 1

Curious... I was able to install it normally after the fixes.
Which version of macOS do you have? Can you also show me the other homebrew packages installed on your system (brew list)? And can you print the full log of installation please?

I have the same problem:
==> Installing odb from max13/odb ==> Downloading http://www.codesynthesis.com/download/odb/2.4/odb-2.4.0.tar.gz Already downloaded: /Users/stavrouv/Library/Caches/Homebrew/odb-2.4.0.tar.gz ==> ./configure --prefix=/usr/local/Cellar/odb/2.4.0 --libexecdir=/usr/local/Cellar/odb/2.4.0/lib --with-options-file=/usr/local/Cellar/odb/2.4.0/etc/odb/default.options CX ==> make install Last 15 lines from /Users/stavrouv/Library/Logs/Homebrew/odb/02.make:         .section __DATA,__const_coal,coalesced                  ^      ~~~~~~~~~~~~ <stdin>:27443:11: warning: section "__const_coal" is deprecated         .section __TEXT,__const_coal,coalesced                  ^      ~~~~~~~~~~~~ <stdin>:27443:11: note: change section name to "__const"         .section __TEXT,__const_coal,coalesced                  ^      ~~~~~~~~~~~~ <stdin>:27449:11: warning: section "__const_coal" is deprecated         .section __DATA,__const_coal,coalesced                  ^      ~~~~~~~~~~~~ <stdin>:27449:11: note: change section name to "__const"         .section __DATA,__const_coal,coalesced                  ^      ~~~~~~~~~~~~ make: *** [install-recursive] Error 1
Output brew list:
autoconf boost cmake confuse cpprestsdk freetype<EMAIL_ADDRESS>gdbm gettext git-flow glib go gradle icu4c imagemagick jpeg libcutl libffi libidn2 libodb libpng libtiff libtool libunistring node openssl<EMAIL_ADDRESS>pcre pkg-config postgresql python python3 -> python rbenv readline redis ruby-build sbt scala sqlite unrar wget xz

@Iaroslawz @BasilakisS Guys... I have no clue at all...
Looking around online says that it should be warnings and not fatal errors.
Plus, I'm on 10.13, so I can't downgrade and do a test.
Please try to remove all dependencies of odb (libcutl, libodb, libodb-*<EMAIL_ADDRESS>etc...), untap and retap if you haven't done it yet, and try brew install -v odb. Then paste the complete log (or a gist) and I'll try to look into it.
Do you both have Xcode installed and components installed as well?

brew install gmp
fixed my compiler problems!

@BasilakisS So it means gmp isn't installed with odb. Thank you for the hint

@Max13 I think so,
because I was getting following error messages:
In file included from diagnostics.cxx:5:0: ../odb/gcc.hxx:15:17: fatal error: gmp.h: No such file or directory  #include <gmp.h> ... In file included from cxx-lexer.cxx:5:0: ../odb/gcc.hxx:15:17: fatal error: gmp.h: No such file or directory  #include <gmp.h> ... ...


[Bug]: Long delay before new messages unpause the agent

Is there an existing issue for the same bug?

[X] I have checked the existing issues.

Describe the bug and reproduction steps
This is a flaky issue--no consistent way to repro that I can tell. I will try and spot a log pattern at least
Sometimes, when the agent is "waiting for user input", when I send a new message, nothing happens. If I refresh the page, the message is there (meaning it made it to the backend, so websocket is connected) and then the agent starts running.
EDIT: pretty sure it's actually a delay! the state changes only when a new event comes back from the agent. probably some async locking going on
OpenHands Installation
Docker command in README
OpenHands Version
main
Operating System
None
Logs, Errors, Screenshots, and Additional Context
No response

This sounds kind of like something weird I noticed on your branch. So this was on main. 🤔


Make it possible to build polls sending private messages to bot

Close #22

Add to the Features list: It's possible to build polls sending private messages to the bot. It helps to make a kind of preview of the future polls.


Add notification customization info

Solution for https://github.com/firebase/firebase-js-sdk/issues/4412 and https://github.com/firebase/quickstart-js/issues/495

@googlebot I signed it!

@Qendolin thanks! Happy to add this.


Fix ActiveScreen in custom navigation

The way as it was before was good, but when there are another Navigations, such as TabNavigation, inside this TabNavigation. The entire nested navigation won't get rendered.
I was sitting here for hours in the source code, but this finally made the move.

Thank you!  I had the same issue and haven't thought of ActiveScreen being the issue.

OMG thank you @JPeer264.  I too was stuck on this issue for hours!


feat/abs: add tensor abs operator

Added abs tensor operator
Pull Request type

Please check the type of change your PR introduces:

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?
No abs tensor operator

Issue Number: N/A
What is the new behavior?
You can now do tensor.abs()

Does this introduce a breaking change?

[ ] Yes
[x] No

Other information

SUMMARY.md

Got it. Updated!

LGTM!!

Good job guys


Advice

This is a great plugin, it helps me a lot.
If there is a full map, this is perfect for me.
About the full map, there is a plugin called "Map of content" which I think is an idea of how to display the full map. For the full map I have some ideas, as shown below
The first image just for illustration, just to show the structure of my files.
The second image, to indicate the effect I want.
The third image, to avoid misunderstanding, shows the exact folder structure.
My ideas are just for your reference, you just have to do what you think is right.

I have two more hierarchies in mind to be added to ExcaliBrain. A tag based hierarchy:
#reading #reading/book #reading/book/fiction #reading/blog etc… you get the idea. This is a tree that can be used for purpose of formatting the nodes as well as for navigation.
The other tree is the tree of folders (I would rather reference them as namespaces). I think this second is closes to what you are looking for.

Pefect double


September 30th Dropbox Update Actions Required - Will this break the gem?

September 30th dropbox is making changes to their API related to:
ACTION REQUIRED: ID Formats
ACTION REQUIRED: tokens and permissions
I see this gem hasn't gotten an update in a while, are these changes going to break this gem?
Or has everyone migrated over to Jesus's Dropbox API gem by now? https://github.com/Jesus/dropbox_api

I was able to override this method to get rid of the token format check. For our purposes (simple uploading/downloading/moving), this seems to work until we can switch to jesus/dropbox_api.
module Dropbox
  class Client
    def initialize(access_token)
      @access_token = access_token
    end
  end
end


Bump pre-commit dependencies

Bump pre-commit dependencies by running pre-commit autoupdate

These are now out of sync with requirements_lint?

I have not found a way to make them in sync with requirements_lint, they constantly get out of sync, once dependabot will bump them they will probably be in sync for a short time.

Looking at requirements_lint with this PR they will be in sync (since all are bumped to latest), I will check if we can bump dependencies in pre-commit automatically or maybe use a different solution. (Now we use them in tox, on another repo I just run pre-commit in the CI step)

I will check if we can bump dependencies in pre-commit automatically or maybe use a different solution

Alternatively, one could run the tool in pre-commit locally (instead of remote repos).

I went for an a GitHub workflow: https://github.com/home-assistant-libs/aiowebostv/pull/126


Fix quantity types

@TLCFEM

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.009%) to 81.146%

Totals

Change from base Build<PHONE_NUMBER>:
-0.009%

Covered Lines:
3426

Relevant Lines:
4222

💛  - Coveralls


recover primary core on startup

Replaces #817
In progress: needs testing and metrics

I believe we have merged the completed PR in the sui repo.


[improve][broker] PIP-192 Use msg property to mark compacted msgs from strategic compaction

Master Issue: https://github.com/apache/pulsar/issues/16691
Motivation
There is a possible edge case where bundle ownership could be in an invalid state in the new load balancer.
Let's say strategic compaction happened in the middle of the bundle transfer.
Owned->Transfer_Assigned // compaction happened.
After the compaction, when a tableview joins and builds its cache from the compacted topic, it will first see a transition, null -> Transfer_Assigned. However, because this is invalid, the tableview will skip this msg, causing an inconsistent view.
To avoid this issue,  this transition(null -> Transfer_Assigned) has been changed to a valid transition in the state diagram, but I realized that this could bring another wrong state when transfer and split occur concurrently.
Racing condition
leader : Owned -> Transfer_Assigned -> Released -> Owned
Broker-2 : Owned -> Splitting -> null(parent-bundle)
Wrong state: a parent bundle is owned by another broker after the split.
Owned -> Splitting -> null -> Transfer_Assigned -> Released -> Owned
Ideally, we need to mark if messages are compacted or not. Then, TableView can know (null -> compacted_messages) are valid transitions.
null -> Transfer_Assigned(compacted) // valid transition.
null -> Transfer_Assigned(non-compacted) // invalid transition.
Modifications
This PR added a system property to compacted msgs from strategic compaction.

Verifying this change

[x] Make sure that the change passes the CI checks.

This change added tests and can be verified as follows:

*Updaated unit tests.

Does this pull request potentially affect one of the following parts:
If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

We will have separate PRs to update the Doc later.
Matching PR in forked repository
PR in forked repository: https://github.com/heesung-sn/pulsar/pull/31

@heesung-sn I am concerned about "However, because this is invalid, the tableview will skip this msg, causing an inconsistent view."
If we skip the message when consuming the data, the consumer will only receive it again if the consumer disconnects. The read position of the cursor will move to the end of the topic, after the topic compaction task done, the compacted data will not dispatch to the consumer again. Is it will be a problem in this case?

Good point.
we should change the condition and accept all compacted messages are valid transitions (even if the prev msg is not null).

@heesung-sn I am concerned about "However, because this is invalid, the tableview will skip this msg, causing an inconsistent view."
If we skip the message when consuming the data, the consumer will only receive it again if the consumer disconnects. The read position of the cursor will move to the end of the topic, after the topic compaction task done, the compacted data will not dispatch to the consumer again. Is it will be a problem in this case?

Good point.
we should change the condition and accept all compacted messages are valid transitions (even if the prev msg is not null).

I misunderstood what you said. Please ignore my above comment.
I will close this PR and propose another solution without msg properties because, as you commented, topic cursors can reset to compaction horizon, which won't have message properties.

Raised a PR for an alternative solution.
https://github.com/apache/pulsar/pull/19546


Shuffling of datasets

Does fast-bert shuffle the train, val and eval datasets?

it shuffles the training dataset as it uses RandomSampler for data sampler.


Debian: no hstr distro for Wheezy and Jessie

The ppa for debian wheezy can't be found (and there isn't one for debian jessie either :confused:). Here are the sources I'm using:
deb http://www.clfh.de/debian wheezy main
deb-src http://www.clfh.de/debian wheezy main

Nothing for Stretch either

I created new PPA for Debian: http://www.mindforger.com/debian/

Any chance for armhf builds in that ppa too?

Hrm. Looks like that article you've linked pretty much covers the process. If you're after a GUI, have you seen Virtual Machine Manager?


Use parseEther for bridge amount

closes #82

Current dependencies on/for this PR:

main

PR #90   👈

This stack of pull requests is managed by Graphite.


Add a stats category for static badges

Right now they're showing up in "other," though I expect they make up
most of that category.
https://github.com/badges/shields/issues/966#issuecomment-469851361

Warnings

:warning:
This PR modified service code for static-badge but not its test code. That's okay so long as it's refactoring existing code.

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @paulmelnikow!

  Generated by :no_entry_sign: dangerJS against b25b8d8c1f6811d82d1acf1f1a93576d3d085a4f


sample z from VAE_GAN

Hi,
I have a question confused about the latent variable z
According to the code,

        # Encode
        enc = self.encoder(self.x_src)
        z, self.encoder_loss = self.latent_encoder.encode(enc, batch_size)
        loss += self.encoder_loss
        # Decode
        x_tilde = self.decoder(z)

It means that this z is first sampled according to VAE from:
z = z_mu + ex.exp(0.5 * z_logvar) * z_eps
and put into the decoder network to generate the new image and then input the discriminator.
But, z is sampled again according to:
            z_samples = self.latent_encoder.samples(batch_size)
            if self.discriminate_ae_recon:
                z = ex.Concatenate(axis=0)(z, z_samples)
            else:
                z = z_samples

and then input the decoder again:
        if gen_size == 0:
            raise ValueError('GAN does not receive any generated samples.')
        x = self.decoder_neggrad(z)

It seems that these two z input the same decoder network. But I think the first z from VAE is s.t.

The second z is just sampled from

How can these two types of z as input into the same decoder network?
And which are the generated images sampled by latent variable z?
This is just my opinion.
Please feel free if I make mistakes.
Thank you.

@karlTUM , z is the latent code of the image and will be used for reconstruction, while z_sample is a random sample from the prior and will be used to generate fake image for training discriminator and generator (shared with decoder in this model)

@tyl5566bryan  yeah, if z is the latent code of input image, z should be constant vector in my opinion. But z is also a random vector which is sampled by
z = z_mu + ex.exp(0.5 * z_logvar) * z_eps
how to understand that? Because in the original paper of VAE , the generated images are sampled in this way.

@karlTUM , that's exactly how this repo leverages the power of both VAE and GAN together.

@tyl5566bryan The confused point is that by minimizing the reconstruction loss, the decoder can reconstruct the input images better and better from latent code z:
z = z_mu + ex.exp(0.5 * z_logvar) * z_eps
But z and z_samples are from different distributions, can they share the same decoder network?

@karlTUM , both the prior for GAN and VAE are normal distribution, which z_sample follows here. At the same time you have the KLD divergence loss to enforce that z_mu and z_logvar (where z is sampled from) will not diverge too much from the prior distribution.
So sharing the same decoder is not a bad idea.

@tyl5566bryan  thank you very much. I think I understand better.


Inserting inconsistent data types on field of OBJECT(DYNAMIC) in table with PK leads to other fields not being visible

CrateDB version
5.1.4 , 5.2.6 , and nightly-5.4.0-20230419
CrateDB setup information
single-node
Problem description
An incorrect INSERT, which is accepted, results on data no longer being accessible.
The presence of a PK in the table seems necessary to reproduce the issue but I could not find out why.
Steps to Reproduce

CREATE TABLE test (id int PRIMARY KEY,obj OBJECT(DYNAMIC));

INSERT INTO test (id,obj)
VALUES  (0,{"this_is_a_text_field"='good',"someotherfield"='some other data'}),
	(1,{"thisfieldshouldbeanarray"=[] /* first record to be inserted has it empty */
		,"someotherfield"='some other data'});
--> INSERT OK, 2 rows affected

SELECT * FROM test WHERE id=1;
/*
+----+-----------------------------------------------------------------------+
| id | obj                                                                   |
+----+-----------------------------------------------------------------------+
|  1 | {"someotherfield": "some other data", "thisfieldshouldbeanarray": []} |
+----+-----------------------------------------------------------------------+
*/

SHOW CREATE TABLE test;
--> thisfieldshouldbeanarray is not listed

SELECT id,obj['someotherfield'] FROM test ORDER BY id;
/*
+----+-----------------------+
| id | obj['someotherfield'] |
+----+-----------------------+
|  0 | some other data       |
|  1 | some other data       |
+----+-----------------------+
*/

INSERT INTO test (id,obj)
VALUES ( 2
	,{"thisfieldshouldbeanarray"={}
	/* we make a mistake here and enter an empty object instead of an empty array */
	,"someotherfield"='abcdefg'});
--> INSERT OK, 1 row affected

SHOW CREATE TABLE test;
/*
|       "thisfieldshouldbeanarray" OBJECT(DYNAMIC)    |
*/

SELECT * FROM test WHERE id=1;
	SQLParseException[Cannot cast object element `thisfieldshouldbeanarray` with value `[]` to type `object`]

SELECT _raw from test where id=1;
{"id":1,"obj":{"thisfieldshouldbeanarray":[],"someotherfield":"some other data"}}

select * from test order by id;
/*
	+----+-----------------------------------------------------------------------+
	| id | obj                                                                   |
	+----+-----------------------------------------------------------------------+
	|  0 | {"someotherfield": "some other data", "this_is_a_text_field": "good"} |
	|  1 | {"thisfieldshouldbeanarray": {}}                                      |
	|  2 | {"someotherfield": "abcdefg", "thisfieldshouldbeanarray": {}}         |
	+----+-----------------------------------------------------------------------+

	***"some other data" is not visible for the record with id 1***
		but it is visible if we run SELECT id,obj['someotherfield'] FROM test ORDER BY id;

*/

Actual Result
|  1 | {"thisfieldshouldbeanarray": {}}                                      |

Expected Result
|  1 | {"someotherfield": "some other data", "thisfieldshouldbeanarray": []} |

Thanks for reporting @hlcianfagna. I am looking into the first part of the issue, inserting an empty array dynamically.
cr> create table t(a int) with (column_policy='dynamic');
CREATE OK, 1 row affected  (1.686 sec)
cr> insert into t(b) values ([]);
INSERT OK, 1 row affected  (0.044 sec)
cr> show create table t;
+-----------------------------------------------------+
| SHOW CREATE TABLE doc.t                             |
+-----------------------------------------------------+
| CREATE TABLE IF NOT EXISTS "doc"."t" (              |
|    "a" INTEGER                                      |
| )                                                   |
| CLUSTERED INTO 4 SHARDS                             |

I think the issue maybe due to the unknown base type of the array.

cr> select pg_typeof(null);
+-------------+
| 'undefined' |
+-------------+
| undefined   |
+-------------+
SELECT 1 row in set (0.001 sec)
cr> select pg_typeof([null]);
+-------------------+
| 'undefined_array' |
+-------------------+
| undefined_array   |
+-------------------+
SELECT 1 row in set (0.001 sec)
cr> select pg_typeof([]);
+-------------------+
| 'undefined_array' |
+-------------------+
| undefined_array   |
+-------------------+
SELECT 1 row in set (0.001 sec)

All above should not be valid types to be used in dynamic column creations.

Thank you @jeeminso , like you say, #14013 would address the first part of the issue, but there is another part as we do not know how many people out there may have already got records in this situation with the empty arrays, so the fact that when we query the specific records we get the cast error but that when querying multiple records other fields disappear remains a problem.

merged a workaround, closing. Thank you


Declaration of League\OAuth2\Client\Provider\MerchantOS::buildQueryString() incorrect

When I go to run a query to pull data like getItems() this is the result that I am getting back.
[ErrorException]
  Declaration of League\OAuth2\Client\Provider\MerchantOS::buildQueryString() should be compatible with League\OAuth2\Client\Provider\AbstractPr
  ovider::buildQueryString(array $params)

Please try the last version (v2.0.15).
Let me know if you have any more questions.


Element with id /subscriptions/xxxxxx is already registered

can't browse resources under VSCode extension (already reported for logic apps, but the same problem across)

This has likely been fixed by by logic apps already and we've made changes to Resource Groups so that logic apps resources should not be registered twice.


Deploying Infrastructure via Azure DevOps

Describe the bug or the issue that you are facing
Error when following the deploy with terraform guide, outer loop step 8
https://github.com/Azure/mlops-v2/blob/main/documentation/deployguides/deployguide_ado.md
When running the pipeline to create the ML infra, I get the error:
Repository mlops-templates references endpoint github-connection which does not exist or is not authorized for use
I have seen a similar issue (now closed) which indicated to install the Terraform for ADO extension. I have this installed.

Steps/Code to Reproduce
As described earlier
Expected Output
Pipeline runs successfully, provisioning the required AML resources
Versions
As described earlier
Which platform are you using for deploying your infrastrucutre?
Azure DevOps (ADO)
If you mentioned Others, please mention which platformm are you using?
No response
What are you using for deploying your infrastrucutre?
Terraform
Are you using Azure ML CLI v2 or Azure ML Python SDK v2
Azure ML Python SDK v2
Describe the example that you are trying to run?
As described earlier

I thinnk the problem is that the step-by-step guide for MLOps V2 with ADO and Terraform misses the service connection creation to create github-connection, as documented in quickstart.md for mlopsv2 repo:
3.3.5 Select "Github", select "Next", select "Personal Access Token" and paste your Github SSO Token in the Personal Access token field, in the "Service connection name" field, enter "github-connection", grant pipeline security access, then select "Save".
  Repeat this step, and for the "Service connection name" enter YOUR GITHUB ORGANIZATION NAME or YOUR GITHUB NAME.  The comment you made on Issue #72 was related to the terraform installer that wasn't installed.
The github-connection is not required as your repositories are in ADO. In case you still want to store the repository in GitHub and run the pipelines in Azure DevOps (ADO), then you can setup a GitHub connection. Here are the steps that you can follow this.

Go to your Azure DevOps project and navigate to the Project Settings page.
Select Service connections under the "Pipelines" section.
Click on the New service connection button and select "GitHub" (since the type of repository in your code is GitHub).
You will be prompted to enter the connection details, including the GitHub repository URL, a personal access token with the necessary permissions, and a connection name.
After entering the connection details, click on the Verify and Save button to create the connection.
Once the connection is created, you can refer to it by the name you gave it in your code, in this case github-connection.

You can now use this endpoint in your pipelines to connect to the specified GitHub repository. That's it! Once you have set up the endpoint, you can reference it in your Azure DevOps pipeline configurations, like the code you provided earlier.

@setuc Thanks for the detailed response, which is in fact what I did to get it working and move forward. However, as you say, I'm using ADO for repo management and running the pipelines, so the question would be, "why is gitgub-connection in the yml config for the ADO-only path, and how should an ADO-pure yml look for the resources>repositories part of the yml?"

I also encountered this issue when performing an ADO only deployment.  The problem is with https://github.com/Azure/mlops-project-template/blob/main/infrastructure/terraform/devops-pipelines/tf-ado-deploy-infra.yml, but also other files too, for example:
https://github.com/Azure/mlops-project-template/blob/main/classical/aml-cli-v2/mlops/devops-pipelines/deploy-batch-endpoint-pipeline.yml
https://github.com/Azure/mlops-project-template/blob/main/classical/aml-cli-v2/mlops/devops-pipelines/deploy-model-training-pipeline.yml
https://github.com/Azure/mlops-project-template/blob/main/classical/aml-cli-v2/mlops/devops-pipelines/deploy-online-endpoint-pipeline.yml
which all contain the hardcoded endpoint and type keys for a github connection that should not exist in an ADO only setup:

resources:
  repositories:
    - repository: mlops-templates  # Template Repo
      name: Azure/mlops-templates # need to change org name from "Azure" to your own org
      endpoint: github-connection # need to set up and hardcode
      type: github
      ref: main

This can be fixed in at least a couple of ways such as inline syntax similar to this reference or maybe a better approach from a resource consistency point of view by the following:

resources:
  repositories:
    - repository: mlops-templates
      name: mlops-templates
      type: git
      ref: main

I have only been testing on an ADO basis so I don't know whether the endpoint: github-connection is required for other scenarios (I presume so).  I think if it is required, then the YAML files should be either patched based on the deployment scenario, i.e. ADO or GitHub or multiple copies of the file maintained (probably less desirable).

@tonyskidmore thanks for the follow-up and info, appreciated. Frustrating to see little movement on all these open issues


Update main.py

Fix spelling error

Sorry for the delay. I'll pull this tonight when I have the time. Thanks for fixing the spelling error!


Request time out when using Google Fonts in footer

Expected Behavior
Google Fonts usable in footer. It works well in the main html.
Current Behavior
Request time out logged form gotenberg
Steps to Reproduce

Write an html footer
Include google Fonts in the head tag

<link rel="stylesheet" type="text/css" href="https://fonts.googleapis.com/css?family=Titillium+Web:300,400,600&display=swap">`

Generate a complete doc from Html (main html with footer)

Logs (LOG_LEVEL="DEBUG")
{"time":"2019-10-02T16:03:59.181326789Z","id":"","remote_ip":"...","host":"pdf-generator:3000","method":"POST","uri":"/convert/html","user_agent":"GuzzleHttp/6.3.3 curl/7.61.1 PHP/7.2.19","status":408,"error":"code=408, message=Request Timeout","latency":10004439015,"latency_human":"10.004439015s","bytes_in":18447,"bytes_out":30}
Your Environment

Version used: 5
Operating System and version: Ubuntu / docker-compose

@admin should i take a look at this ?

@pdoreau could you try with version 6?

@gulien same kind of message with v6 , "context has timed out" :
{"level":"error","msg":"cdp.Page: PrintToPDF: context deadline exceeded","op":"xhttp.htmlHandler: xhttp.convertSync: xcontext.MustHandleError: printer.chromePrinter.Print","time":"2019-10-02T17:42:12Z","trace":"..."}
{"bytes_in":"17318","bytes_out":"36","host":"pdf-generator:3000","latency":"10009876198","latency_human":"10.009876198s","level":"error","method":"POST","msg":"request failed","op":"context.Context.LogRequestResult","path":"/convert/html","referer":"","remote_ip":"...","status":504,"time":"2019-10-02T17:42:12Z","trace":"...","uri":"/convert/html","user_agent":"GuzzleHttp/6.3.3 curl/7.61.1 PHP/7.2.19"}

@pdoreau it seems that there are some limitations for footer / header: https://github.com/GoogleChrome/puppeteer/issues/2916#issuecomment-406366027.
Note: Gotenberg  and puppeteer uses the same Google Chrome API.

I’ll update the documentation to highlight those limitations 👍
For now a workaround would be to install the fonts directly in the Docker image.

@gulien can i update that info in doc?


Pigpen Cipher python Implementation

1> Implement a Pigpen Cipher encryption and decryption method and wrap it inside a class PigpenCipher
2> save it to pigpen_cipher.py
3> put the file inside decrypto/src folder
4> submit a PR.
Sample implementation : https://github.com/pyGuru123/Decrypto/blob/main/decrypto/src/binary_cipher.py
strictly follow above implementation and PEP8 Guidelines

uses images so not implementing


Add unittest.mock backport

Fixes:
Summary/Motivation:

To allow usage of unittest.mock in packages that import pyutilib.th as unittest

Changes proposed in this PR:

Legal Acknowledgement
By contributing to this software project, I agree to the following terms and conditions for my contribution:

I agree my contributions are submitted under the BSD license.
I represent I am authorized to make the contributions and grant the license. If my employer has rights to intellectual property that includes these contributions, I represent that I have received permission to make contributions and grant the required license on behalf of that employer.

It looks like the hard dependence on the mock library might be causing problems (I rarely use mock, and in our other discussion missed that mock was a separate package before Python 3.3).  I am not sure how others will feel about adding another dependency.  At a minimum, we should promulgate the mock from unittest in Python3:
if six.PY3:
    from unittest import mock
else:
    import mock

I wonder if it also makes sense to make the dependency optional:
if six.PY3:
    from unittest import mock
    mock_available = True
else:
    try:
        import mock
        mock_available = True
    except ImportError:
        mock_available = False

Thoughts?

Updated accordingly though I figured that using Python3 doesn't necessarily imply that mock exists in unittest since it wasn't around in 3.0

@ruaridhw: correct, but PyUtilib explicitly doesn't support 3.0-3.3, so we can be lazy.

Coverage decreased (-0.01%) to 61.919% when pulling ceada22944805c270f389d83f49f3e10aee67b8d on flexciton:feat/mock into 6f02a998a57f1e843dcde3409fad58cd15005c43 on PyUtilib:master.


[Discussion] Adding assay with existing name registers this assay additionally to a new study

Describe the bug
When trying to describe the error in #210 I accidentally created another assay with the same assay name but did not provide a study name. While arc commander does recognize that an assay folder with that name exists it still partially executes the command and registers the existing assay to a new study.
To Reproduce

arc init
arc i create --identifier Test_investigation
arc s add -identifier Test_study
arc a add -s Test_study -a Test_assay

arc a add -a Test_assay
Start Assay Init
ERROR: Assay folder with identifier Test_assay already exists.
Start Assay Register
Study with the identifier Test_assay does not exist yet, creating it now.
Done processing command.
 arc a list
Start processing parameterless command.
Start Assay List
Study: Test_study
--Assay: Test_assay
Study: Test_assay
--Assay: Test_assay
Done processing command.
Expected behavior
I would expect the command to be aborted and I am wondering, whether we want this behavior.
OS and framework information (please complete the following information):

OS: Windows 10
arcCommander v0.5.0-win.x64

Additional context
Unless this is in conflict with general isa specifications, I do think it is good though to be able to register one assay to multiple studies (e.g. you collect samples from several studies and perform one assay with all of them).

Hey, thanks for your input. Was just about to fix this issue when I also asked myself this question:

I would expect the command to be aborted and I am wondering, whether we want this behavior.

Of course the whole create study when no identifier is given thing is a bit of a convenience feature, that one wouldn't necessarily expect. But following the two scenarios

assay add with no study identifier -> create assay and study from assay identifier
assay add with study identifier but assay already existing -> register assay to study (create new if non-existing)

...
assay add with no study identifier but assay already existing -> register assay to study (create new if non-existing from assay identifier)  kind of makes sense

So not sure whether I should actually abide from the as-is logic

Alright, so closing this for now. Feel free to reopen if necessary.