[BUG] Error in trace-autogen-groupchat.ipynb

Describe the bug
I ran the notebook, it included installing requirements (it had promptflow in the requirements but no version)
I ran these lines and got the error below:
from promptflow.tracing import start_trace
# start a trace session, and print a url for user to check trace
# traces will be collected into below collection name
start_trace(collection="autogen-groupchat")

ImportError                               Traceback (most recent call last)
File c:\Users\945764\OneDrive - parkercorp\Desktop\promptflow_trace_test.venv\lib\site-packages\promptflow_init_.py:17
[16](file:///C:/Users/945764/OneDrive%20-%20parkercorp/Desktop/promptflow_trace_test/.venv/lib/site-packages/promptflow/init.py:16) # control plane sdk functions
---> [17](file:///C:/Users/945764/OneDrive%20-%20parkercorp/Desktop/promptflow_trace_test/.venv/lib/site-packages/promptflow/init.py:17) from promptflow._sdk._load_functions import load_flow as _load_flow
[18](file:///C:/Users/945764/OneDrive%20-%20parkercorp/Desktop/promptflow_trace_test/.venv/lib/site-packages/promptflow/init.py:18) from promptflow._sdk._load_functions import load_run as _load_run
File c:\Users\945764\OneDrive - parkercorp\Desktop\promptflow_trace_test.venv\lib\site-packages\promptflow_sdk_load_functions.py:15
[14](file:///C:/Users/945764/OneDrive%20-%20parkercorp/Desktop/promptflow_trace_test/.venv/lib/site-packages/promptflow/_sdk/_load_functions.py:14) from ._errors import MultipleExperimentTemplateError, NoExperimentTemplateError
---> [15](file:///C:/Users/945764/OneDrive%20-%20parkercorp/Desktop/promptflow_trace_test/.venv/lib/site-packages/promptflow/_sdk/_load_functions.py:15) from .entities import Run
[16](file:///C:/Users/945764/OneDrive%20-%20parkercorp/Desktop/promptflow_trace_test/.venv/lib/site-packages/promptflow/_sdk/_load_functions.py:16) from .entities._connection import CustomConnection, _Connection
File c:\Users\945764\OneDrive - parkercorp\Desktop\promptflow_trace_test.venv\lib\site-packages\promptflow_sdk\entities_init_.py:8
[6](file:///C:/Users/945764/OneDrive%20-%20parkercorp/Desktop/promptflow_trace_test/.venv/lib/site-packages/promptflow/_sdk/entities/init.py:6) path = import("pkgutil").extend_path(path, name)  # type: ignore
----> [8](file:///C:/Users/945764/OneDrive%20-%20parkercorp/Desktop/promptflow_trace_test/.venv/lib/site-packages/promptflow/_sdk/entities/init.py:8) from ._connection import (
[9](file:///C:/Users/945764/OneDrive%20-%20parkercorp/Desktop/promptflow_trace_test/.venv/lib/site-packages/promptflow/_sdk/entities/init.py:9)     AzureContentSafetyConnection,
[10](file:///C:/Users/945764/OneDrive%20-%20parkercorp/Desktop/promptflow_trace_test/.venv/lib/site-packages/promptflow/_sdk/entities/init.py:10)     AzureOpenAIConnection,
[11](file:///C:/Users/945764/OneDrive%20-%20parkercorp/Desktop/promptflow_trace_test/.venv/lib/site-packages/promptflow/_sdk/entities/init.py:11)     CognitiveSearchConnection,
[12](file:///C:/Users/945764/OneDrive%20-%20parkercorp/Desktop/promptflow_trace_test/.venv/lib/site-packages/promptflow/_sdk/entities/init.py:12)     CustomConnection,
[13](file:///C:/Users/945764/OneDrive%20-%20parkercorp/Desktop/promptflow_trace_test/.venv/lib/site-packages/promptflow/_sdk/entities/init.py:13)     OpenAIConnection,
[14](file:///C:/Users/945764/OneDrive%20-%20parkercorp/Desktop/promptflow_trace_test/.venv/lib/site-packages/promptflow/_sdk/entities/init.py:14)     SerpConnection,
[15](file:///C:/Users/945764/OneDrive%20-%20parkercorp/Desktop/promptflow_trace_test/.venv/lib/site-packages/promptflow/_sdk/entities/init.py:15)     QdrantConnection,
[16](file:///C:/Users/945764/OneDrive%20-%20parkercorp/Desktop/promptflow_trace_test/.venv/lib/site-packages/promptflow/_sdk/entities/init.py:16)     WeaviateConnection,
[17](file:///C:/Users/945764/OneDrive%20-%20parkercorp/Desktop/promptflow_trace_test/.venv/lib/site-packages/promptflow/_sdk/entities/init.py:17)     FormRecognizerConnection,
[18](file:///C:/Users/945764/OneDrive%20-%20parkercorp/Desktop/promptflow_trace_test/.venv/lib/site-packages/promptflow/_sdk/entities/init.py:18)     CustomStrongTypeConnection,
...
[27](file:///C:/Users/945764/OneDrive%20-%20parkercorp/Desktop/promptflow_trace_test/.venv/lib/site-packages/promptflow/init.py:27)     ) from e
[30](file:///C:/Users/945764/OneDrive%20-%20parkercorp/Desktop/promptflow_trace_test/.venv/lib/site-packages/promptflow/init.py:30) # flake8: noqa
[31](file:///C:/Users/945764/OneDrive%20-%20parkercorp/Desktop/promptflow_trace_test/.venv/lib/site-packages/promptflow/init.py:31) from ._version import VERSION
Exception: Promptflow may not installed correctly. If you are upgrading from 'promptflow<1.8.0' to 'promptflow>=1.8.0', please run 'pip uninstall -y promptflow promptflow-core promptflow-devkit promptflow-azure', then 'pip install promptflow>=1.8.0'. Reach https://microsoft.github.io/promptflow/how-to-guides/faq.html#promptflow-1-8-0-upgrade-guide for more information.
How To Reproduce the bug
Steps to reproduce the behavior, how frequent can you experience the bug:

Download the notebook and hit "run all"

Running Information(please complete the following information):
Windows 11, Python 3.9, promptflow==1.11.0
promptflow-core==1.11.0
promptflow-devkit==1.11.0
promptflow-tracing==1.11.0

I was able to resolve this by changing my working directory to src\promptflow-tracing, and NOT installing promptflow via pip


module 'easyclimate' has no attribute 'calc_detrend_data'

I can't use the function under ecl, it always shows errors

AttributeError: module 'easyclimate' has no attribute 'calc_detrend_data'

This is my simple code：
import easyclimate as ecl
CRU_tmean_ano_trend1 = ecl.calc_detrend_data(CRU_tmean_ano, dim = 'time')

when I try others functions, it also shows same errors.

AttributeError: module 'easyclimate' has no attribute 'open_tutorial_dataset'

Hi! I don't know what your platform is? Both Windows and Linux should be able to directly reference this method. You can use ecl.stat.calc_detrend_data and ecl.tutorial.open_tutorial_dataset for temporary use.


When is the first release expected?

The title says it all.
Also, if it's a pre-release, why do we have a v10? Are we not following semantic versioning?

Hi @tariq1890, thanks for reaching out.
We will have a GA release candidate soon. Semantic versioning will be followed once the tool is no longer in preview.


Documentation contains persistence instructions

Resolves Issue #26.
README updated with:

Up to date link to Docker Hub
Newer links to Docker networking documentation
Long-form command line flags in CLI code snippets (with the side effect that the code snippet no longer has a scroll bar..good!)
Instructions for using the container with host directories mounted into the container as volumes to persist configuration and logs between runs.

@joshrivers awesome!!!. Merged.


SysAdm should offer display-related preferences

At least
Allow the user to:

add a screen
set resolutions
position screens in relation to each other
remove a screen.

Plus, ideally
Allow the user to manually prefer a driver. An example use case:

automation causes use of VESA; and
the user knows that SCFB will be more appropriate.

This issue was moved to trueos/trueos-core#79


Inconsistency of parsing the realtime metric value.

Summary
During the definition of real time metric in a workflow, the value of metric is not parsed correctly when defined
using valid go template syntax
What happened/what you expected to happen?
if value in real time metric is defined like this  {{ workflow.duration }}. The next error shown in the log
However it works correctly like this {{workflow.duration}}
I'd expect that it works correctly if either valid go template syntax is applied.
It's minor issue. The custom parsing of the value doesn't trim suffix and leading white spaces. See here.
If you think the  fix is good idea, i can submit a PR.
What executor are you using? PNS/Emissary
N/A
Diagnostics
Paste the smallest workflow that reproduces the bug. We must be able to run the workflow.
# For complete documentation please see:
#
#   docs/metrics.md
apiVersion: argoproj.io/v1alpha1
kind: Workflow
metadata:
  generateName: hello-world-
spec:
  entrypoint: steps
  metrics:
    prometheus:
      - name: duration_gauge
        labels:
          - key: name
            value: workflow
        help: "Duration gauge by name"
        gauge:
          realtime: true                  # This metric will be emitted in real time. For more info see: docs/metrics.md
          value: "{{ workflow.duration }}"  # Use {{workflow.duration}} in workflow-level and {{duration}} in template-level

What executor are you running? PNS/Emissary
# Logs from the workflow controller:
kubectl logs -n argo deploy/workflow-controller | grep ${workflow}

# If the workflow's pods have not been created, you can skip the rest of the diagnostics.

# The workflow's pods that are problematic:
kubectl get pod -o yaml -l workflows.argoproj.io/workflow=${workflow},workflow.argoproj.io/phase!=Succeeded

# Logs from in your workflow's wait container, something like:
kubectl logs -c wait -l workflows.argoproj.io/workflow=${workflow},workflow.argoproj.io/phase!=Succeeded

' workflow.duration ' is not available as a real time metric

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

I think this is because of space, We should trim the space in code.

exactly. Should I submit a PR?


Update docs to clarify new work

Alaska Airlines Pull Request
Summary:
This PR makes minor updates to the README and new-icon issue template to help clarify new work requests.
Type of change:
Please delete options that are not relevant.

[ ] New capability
[x] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Orion Design System Team

:tada: This PR is included in version 3.1.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


How to remove animation for selected item?

first, thank you for such a great library.
i don't to show selected item. how to remove animation from every where
I set 0 for circle radius but its now working. please help me.

what is your expected behavior? which animation do you want to remove?
it makes no sense to "not show the selected item" with this control because the circle shows always the current selected one.
do you want that the cirlce stays always on the first entry even when the user selects the 2nd or third one??


chore(main): release 1.10.0

:robot: I have created a release beep boop
1.10.0 (2024-07-09)
Features

导出collection (b0dc063)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/luoyi58624/luoyi_dart_base/releases/tag/v1.10.0 :sunflower:


[Docker] Use pytorch base images for tlcpack/package-cpu to fix manylinux2014 compliance

Should resolve #142.
Once merged, I will do a followup PR hopefully fixing #141 and #138 as well.

@driazati


Login issues: need to confirm, that I want to leave the page, before the login process continues

When I login to Cate, first thing I see is the below screenshot.

I need to confirm, that I want to leave the page, before the login process continues.
This is very strange and confusing behavior, because I do not want to leave the page - I am just logging in.

I have the same but only with Firefox. All other browsers behave as expected.


Updated to 0.4.172 and got internal Vike bug

Description
0.4.171 works perfectly. This happens on update to 0.4.172.
Using v1 design API with Vue.
Pasted from browser console below. No server-side errors logged.
createErrorWithCleanStackTrace.js?v=9594635f:4 Uncaught Error<EMAIL_ADDRESS>You stumbled upon a Vike bug. Go to https://github.com/vikejs/vike/issues/new and copy-paste this error. A maintainer will fix the bug (usually under 24 hours).
    at createErrorWithCleanStackTrace (createErrorWithCleanStackTrace.js?v=9594635f:4:17)
    at assert (assert.js?v=9594635f:53:27)
    at parseGlobResults (parseGlobResults.js?v=9594635f:9:5)
    at setPageFiles (setPageFiles.js?v=9594635f:10:58)
    at pageFiles.js?v=9594635f:4:1

Can provide more info as needed. Thx!

Thank you for opening the issue.
Click on parseGlobResults.js?v=9594635f:9:5 and let me know which assert() you see: it's the one that is failing.

It landed on the first assert.
function parseGlobResults(pageFilesExports) {
    assert(hasProp(pageFilesExports, 'isGeneratedFile'));

That's quite unexpected: isGeneratedFile doesn't exist anymore. Did you $ rm -rf node_modules/.vite/?

Um, no. Lol. That fixed it!!! This was probably noted somewhere that I neglected to read.
Sorry about that.
And BTW, the work that has been done on this framework lately has been fantastic. The detailed docs are appreciated. Thank you!

No worries & I'm glad you're enjoying Vike.


Fixed: The clickhouse time is automatically written, causing the modi…

Fixed: The clickhouse time is automatically written, causing the modification operation to fail

@xgd16 Merged and commit in https://github.com/gogf/gf/pull/2830


python3Packages.pep8-naming: 0.13.1 -> 0.13.2

Description of changes

changelog: https://github.com/PyCQA/pep8-naming/blob/0.13.2/CHANGELOG.rst#0132---2022-08-19
diff: https://github.com/PyCQA/pep8-naming/compare/0.13.1...0.13.2
Things done

Built on platform(s)

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

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

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

[x] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Dupe of #187724


[TMP] - Tests for ragged tensors with TensorFlow 2.10

This PR adds a couple of tests to demonstrate the errors we get with TensorFlow 2.10 and ragged tensors on GPU.
Merlin Models bug ticket https://github.com/NVIDIA-Merlin/dataloader/issues/74
Example of error
    def call(self, inputs):
        embeddings = []
        for feature_name, feature in inputs.items():
            ragged_tensor = self.to_ragged(feature)
            embedding_inputs = ragged_tensor.to_sparse()
>           feature_embedding = tf.nn.safe_embedding_lookup_sparse(
                self.embedding.embeddings, embedding_inputs
            )
E           tensorflow.python.framework.errors_impl.InvalidArgumentError: Exception encountered when calling layer "my_model_8" "                 f"(type MyModel).
E
E           {{function_node __wrapped__Slice_device_/job:localhost/replica:0/task:0/device:GPU:0}} Expected begin and size arguments to be 1-D tensors of size 1, but got shapes [1] and [5,10] instead. [Op:Slice]
E

tests/unit/loader/test_tf_dataloader.py:694: InvalidArgumentError

2022-11-08 19:37:12.699694: W tensorflow/core/kernels/data/generator_dataset_op.cc:108] Error occurred when finalizing GeneratorDataset iterator: FAILED_PRECONDITION: Python interpreter state is not initialized. The process may be terminated.
         [[{{node PyFunc}}]]

The other thing thing to note is that evaluating all batches prior to  loader = iter([batch for batch in loader]) running model.fit doesn't produce the error, which suggests that it might be something to do with thread safety of certain operations


IntelliJ IDEA Community, Ultimate and Educational Editions are missing from the manifest repository

Description of the new feature/enhancement

Proposed technical implementation details (optional)

@vyujitanaka IntelliJ depends on Java. Unless there is a bundled version available, these may not be viable to add until support for dependencies in the Windows Package Manager is implemented.

https://youtrack.jetbrains.com/issue/IDEA-242025

The reason is different. Unless something has changed, IntelliJ installer requires a silent config file that winget can't provide. This has already been discussed in https://github.com/microsoft/winget-pkgs/pull/179 and in other package managers https://github.com/just-install/registry/issues/115. I opened an issue upstream, but they don't seem to understand it: https://youtrack.jetbrains.com/issue/IDEA-226645

Jetbrains toolbox is available in the repo, maybe that will help as a workaround for now.

It won't. We have domain users. Installing JetBrains products on domain accounts will make them barely usable, will slow logon and will be a big waste of space on the domain server.

I added IntelliJ CE in https://github.com/microsoft/winget-pkgs/pull/5212.

It looks like they are all present now:
winget search intellij
Name                            Id                               Version  Match         Source
----------------------------------------------------------------------------------------------
RubyMine                        JetBrains.RubyMine               2020.3.3 Tag: intellij winget
CLion                           JetBrains.CLion                  2020.3.3 Tag: intellij winget
Android Studio                  Google.AndroidStudio             <IP_ADDRESS>  Tag: IntelliJ winget
IntelliJ IDEA Ultimate Edition  JetBrains.IntelliJIDEA.Ultimate  2020.3.3               winget
IntelliJ IDEA Education Edition JetBrains.IntelliJIDEA.Edu       2020.3.4               winget
IntelliJ IDEA Community Edition JetBrains.IntelliJIDEA.Community 2020.3.3               winget

Windows 10 Install path changed unexpectedly.  Program Files vs Program Files (x86)
On my last machine I installed IntelliJ IDEA 2019.2.2 (Ultimate Edition) using the exe downloaded from the site https://www.jetbrains.com/idea/download/#section=windows
On my current machine installed IntelliJ IDEA 2021.3.2 (Ultimate Edition) using winget
The exe installed it in Program Files
However winget installed it in Program Files (x86)
It appears to be a 64 bit application, so I think it is unexpected to find it in Program Files (x86)
Is this a bug, or intentional?

Is this a bug, or intentional?

It's a bug with how JetBrains' installers work.
All WinGet does is run ideaIU-2021.3.2.exe with the /S installer switch.

confirmed, and to add clarification on how much of the path to provide to --location, this command gives the expected results for the current version
winget install -e --id JetBrains.IntelliJIDEA.Ultimate --location "C:\Program Files\JetBrains\IntelliJ IDEA 2021.3.2"


[HUDI-3280] Cleaning up Hive-related hierarchies after refactoring

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contribute/how-to-contribute before opening a pull request.

What is the purpose of the pull request
This PR cleans up Hive-related hierarchies after refactoring
Brief change log

Merged HoodieRealtimeFileSplit w/ BaseWithLogFilesSplit
Fixed RealtimeBootstrapBaseFileSplit
Moved methods form HoodieInputFormatUtils closer to where they're used
Cleaned up unused methods

List of commits in this patch:
https://github.com/apache/hudi/pull/4743/files/7d12ce874c3bfb9861b00449b90ebde0125388b7..9501b4ca02751b4b1223650fd85945691016f36d
Verify this pull request
This pull request is already covered by existing tests, such as (please describe tests).
Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@hudi-bot run azure

This is current MT layout in Spark DS
+------------------+----+--------------------+-------------------+-------------------+
|               key|type|  filesystemMetadata|BloomFilterMetadata|ColumnStatsMetadata|
+------------------+----+--------------------+-------------------+-------------------+
|        2015/03/16|   2|{4b2d6e17-e6bf-4b...|               null|               null|
|        2015/03/17|   2|{5e2c7eb4-e58d-48...|               null|               null|
|        2016/03/15|   2|{1abe3051-0997-4d...|               null|               null|
|__all_partitions__|   1|{2016/03/15 -> {0...|               null|               null|
+------------------+----+--------------------+-------------------+-------------------+

It differs from what we have discussed in HUDI-1296, and as i've already mentioned in the description that would be addressed in a follow-up PR after current implementation of MOR Relations will be cleaned up (read de-duped)


Revisit tag usage for imported notes

Currently, for notes that are exported without inclusion proofs on a client $A$ and imported on another client $B$ (where the note is created on a future block compared to client $B$'s view of the blockchain), the second client faces a potential problem of never getting inclusion proof data from syncs based on whether the note tag are actually of interest to it. To solve this we currently add the tag of non-committed notes at the moment of creating a sync request (in an "ephemeral" manner, meaning they won't get added the time after getting sync data for the note). However, as discussed, this puts more effort on the node and can likely be avoided in a number of different ways (using other RPC calls, etc.).

One potential way to do this may be to mark the notes which are missing MMR part of the proof somehow and then, once the client syncs beyond the block in which the note has been created, make a request to GetBlockHeaderByNumber (with include_mmr_proof = true) to get the MMR inclusion proof for the relevant block (if needed).
So, let's say the client is currently at block 100, and we've imported the note with the inclusion path against block 150. Let's also say that after next sync the client syncs to block 200. At this point, the client would check if it has any notes which are missing MMR proofs, and then make a request to GetBlockHeaderByNumber(block_num=150, include_mmr_proof=true). The returned MMR proof (and the block header) would be used to make the note "consumable".

There's a bunch of variables for scenarios like this. Namely:

has_inclusion_proof: Whether the imported InputNoteRecord has inclusion proofs (meaning inclusion in the note tree of the chain's block).
client_has_mmr: Whether the client has MMR data for the block the note was created in.
synced_past_block: Whether the client has synced past the note's block at import time. Note we don't necessarily know this at import time (based on whether the InputNoteRecord has the note origin)

There is also the option of the user having set verify at import time, to verify the note's existence.
Not all scenarios are interesting, but here are the relevant ones to this issue:
(!has_inclusion_proof, !client_has_mmr)
If synced_past_block is false:  At import time, we have the option to build the inclusion proof if the caller set verify_existence to true (by calling GetNotesById and GetBlockHeaderByNum). We are currently adding the tag to the sync request, which gets us both the MMR data and the note's inclusion proof.
We want to avoid this, but it won't be possible without eventually calling both GetNotesById and GetBlockHeaderByNum. I think originally we discussed not always verifying the note's existence because it would incur some privacy loss (the node now knows which note the client is interested in) which is why we started adding the tag to requests.
If synced_past_block is true: Same as above, except if the user does not verify the note's existence at import time, the client is softlocked with the note being unusable (unless re-imported) because adding the tag to the sync request does not get the client anything it needs.
If the note has an inclusion proof, but the client does not have MMR data, the client behaves the same way except it could avoid calling GetNotesById (it doesn't avoid it currently, this call is only based on whether the user set verify to true at import time).
So one way to make this all work is to follow @bobbinth's flow, but also potentially getting the note's inclusion proof at sync time as well: After syncing, for any note that does not have an inclusion proof, call GetNotesById and saving it (we can call it only once for multiple notes). After this, check whether we have all the necessary MMR data and get nodes for any block we need (one call per block). The downside of including this GetNotesById call is that we incur into the privacy loss I mentioned before. This is unavoidable if we don't know whether the note was included or not, unless we stop supporting exporting note records that don't have inclusion proofs. The other downside is that if the client has bad notes (eg, notes that were never eventually included for any reason), it could be a constant overhead forever. For this last problem, the client could eventually mark the note as never included and stop considering it? Depending on the different flows the protocol might support it might be difficult to tell.

The other downside is that if the client has bad notes (eg, notes that were never eventually included for any reason), it could be a constant overhead forever. For this last problem, the client could eventually mark the note as never included and stop considering it? Depending on the different flows the protocol might support it might be difficult to tell.

It feels like we may need to have some more sophisticated strategy here. For example, something like "exponential backoff" - if the expected note hasn't appeared on chain, double the time until the next check. We can also set some upper limit: once the interval exceeds some upper bound, stop checking and mark the note as "stale" or something like that.

should this be closed considering #375 got merged?

Closing this as #375 was merged and follow-ups/changes are being discussed in #405


http://<IP_ADDRESS>:48109 - missing IPv6 support

I would be happy if the program listens to "any" ports not just on the IPv4-sockets.
I expect something like this:
:::48109
instead of
<IP_ADDRESS>:48109
Simply replacing the host='<IP_ADDRESS>' by host='::' may break other installations, because aiohttp has set the ipv6-only flag somewhere. So the flag should be set to dualstack or it is necessary to open two separate sockets. Since the dualstack socket is not supported by all operating system so the two socket option may be the best solution.

This seems to be a solution: https://stackoverflow.com/questions/45907833/python-asyncio-how-to-listen-ipv4-and-ipv6-by-using-one-socket. Can you please try it would work for you?

I found this site too. But I am not very familiar with python. For me it would also be ok, with IPv6-only. But in general I expect that a server application is running in dual stack mode, or it is easy configurable by command line switches/ configuration file / prominent place within the code.


doxygen installation

Hi
These makefile lines fail, and were commented out:
docs : $(docs_top_index)
$(build_doxygen_index)
docs_api_directory := $(product_name)api_docs_html$(product_version_name)
docs_api : $(docs_top_index) | $(build_dir.packages)
$(build_doxygen_index)
$(Q)tar --transform "s,^,$(docs_api_directory)/," -C $(build_dir.doc)/api/html -zcvf $(build_dir.packages)/$(docs_api_directory).tar.gz .
Please provide instructions how to install doxygen. Does it rqure a specific version?

Avi,
To install Doxygen, use the Doxygen related steps within the install guide sections for Linux or Windows. The minimum version of Doxygen required is define in Doxyfile and at the time of this post is version 1.8.5.

root@ip-172-31-0-174:/home/AviManor/aws-cdi-sdk# sudo apt-get install –y build-essential libncurses-dev autoconf automake libtool libnl-3-dev cmake git doxygen libcurl4-openssl-dev libssl-dev uuid-dev zlib1g-dev libpulse-dev
Reading package lists... Done
Building dependency tree
Reading state information... Done
Note, selecting 'libncurses5-dev' instead of 'libncurses-dev'
E: Unable to locate package –y
root@ip-172-31-0-174:/home/AviManor/aws-cdi-sdk#

Hi Mike
I failed to follow your instructions. Added the log as a comment in the case, but since the case is closed I don’t know if you can read it.
Regards
Avi Manor | Real time software engineer |TAG Video Systems | IL +972 73 777 0438 | M: +972 54 5680225 | @.*** @.***>  | www.tagvs.com http://www.tagvs.com/
http://www.tagvs.com/
From: Mike Henry @.>
Sent: Thursday, July 22, 2021 11:06 PM
To: aws/aws-cdi-sdk @.>
Cc: AviManor @.>; Author @.>
Subject: Re: [aws/aws-cdi-sdk] doxygen installation (#51)
Avi,
To install Doxygen, use the Doxygen related steps within the install guide sections for Linux https://github.com/aws/aws-cdi-sdk/blob/developer_preview/INSTALL_GUIDE_LINUX.md#install-package-dependencies  or Windows https://github.com/aws/aws-cdi-sdk/blob/developer_preview/INSTALL_GUIDE_WINDOWS.md#configure-the-ec2-instance . The minimum version of Doxygen required is define in Doxyfile https://github.com/aws/aws-cdi-sdk/blob/a0d0cbf499e3c96ce3510810f58ae96bb6350480/doc/Doxyfile#L1  and at the time of this post is version 1.8.5.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/aws/aws-cdi-sdk/issues/51#issuecomment-885197575 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AUHYYWDT264Z2EYT24P7VHDTZB22JANCNFSM5AR66SHA .  https://github.com/notifications/beacon/AUHYYWAHAJB2P5KJQ7DYKQLTZB22JA5CNFSM5AR66SHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOGTBQWBY.gif

Avi,
To install Doxygen on Ubuntu, use something like this:  sudo apt-get install doxygen. Looks like you had a bad hyphen character in your command line for "-y". See in your log: Unable to locate package –y. We will leave this ticket open until you have confirmed that Doxygen has been properly installed and working for you.

Hi Mike
I have copy/paste the installation instructions and replaced each hyphen by hand.
The installation was OK, and documentation was created under build/documentation.
Thank you for your help
Avi


HTTP CONNECT ForwardProxy / stats: envoy_http_downstream_rq_xx - response class "1" counter increase

Hi, i'm using envoy v1.21.0 with standard ForwardProxy CONNECT termination configs.
However, noticed  that the prometheus metric envoy_http_downstream_rq_xx - response class "1" counter increases slowly during the day.
In the access logs all the requests are with response_code 200.
Just wonder what will cause this and what it means? Is it a bug or something else...

BR,
Stoyan

cc @jmarantz

May be this is because the status code 100(Continue) or status code 101(Switching Protocols)?

i don't see how this is possible because the traffic is HTTP1.1 and HTTP1.0 CONNECT my config: routes:
    - match: {connect_matcher: {}}
      route:
        cluster: dynamic_forward_proxy_cluster
        upgrade_configs: [{upgrade_type: CONNECT, connect_config: {}}]

also if it was 100, or 101 i believe it also should be logged in the access log like this, but as i said all i see is 200 there.

If a downstream request contains expect: 100-continue, then envoy will respond a response (with status 100) first and waiting the request body.
This response (with status 100) will be counted by the stats but no access log.

Understood, thx wbpcode. Wonder if is it possible to log that expect header with expect_hdr: "%REQ(expect)%"... i have no prior knowledge what headers the downstream servers put in their CONNECT request..

Understood, thx wbpcode. Wonder if is it possible to log that expect header with expect_hdr: "%REQ(expect)%"... i have no prior knowledge what headers the downstream servers put in their CONNECT request..

nope. expect header will be removed immediately when the 100-continue response is sent.


Restore tests, fix things underway

Restore existing Selenium tests, fix several things underway - related to Organizations and projects.
Organization tests still not fully done - need to implement deletion of organizations first.

Fixes #179


Running an init bash script on pew workon ...

To be fair this isn't an 'issue', but wasn't sure where to ask this.
Is there a way to run a bash script every time I run pew workon ... I need to run nvm use to match the node version, and that seems the most logical place to run it.

Which operating system and shell are you using?

Mac

On Catalina 10.15.2, I was able to do it by pasting the desired commands in .bashrc in my user home (~/) folder.
The Terminal app in Catalina uses .bash_profile for startup commands, while pew uses .bashrc
Does that work for you?
There's also an init.bash file in the library folder where pew is installed, but I wasn't able to edit that to get the desired effect.


Update substitution-tags.md

Description of the change: Adding alert to clarify substitution tags should be strings.
Reason for the change: substitution tags must be passed strings
Link to original source:

Closes #

Hello @emilyruppe,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


Add Header for User ID

Right now, there are headers for User Name, User Email, User Roles, and User Subject, but no header for the User ID. Currently, I am authenticating via oauth, which is authed against LDAP, and logging the user into netbox by passing a header. I'd prefer to use the ID they use for LDAP at the ID to identify them in netbox, however that doesn't seem to be an option.

@lbreitk , got it. Please ping back in a couple of weeks. Hopefully, by then I will finish my refactoring and then we can add the things you need.
it may be that the you thought about the solution solved your particular issue, but may have been an issue for others.
Consider thinking in terms of caddyfile directives. For example, here are some directives that I thought about.
jwt {
  inject header X-HEADER from claim email
  inject header X-USER from claim metadata.user
}

Additionally, please think about testing. If you propose a change, please provide unit tests.
To sum up, when you are ready, please open a PR to implement the above injections and associated unit tests.
Please do not hesitate asking question! Any question is a gem :-)

@lbreitk , I am ready to engage on this one.
Think in terms of JWT token fields and provide the instructions you want to see.
jwt {
  inject header X-HEADER from claim email
  inject header X-USER from claim metadata.user
}

Currently, here is what is being passed: https://github.com/greenpau/caddy-auth-jwt#pass-token-claims-in-http-headers

@greenpau Thanks for the reply! Unfortunately, I'm a Systems Engineer - my developers skills may be a bit limited to be of much more help :P But I will try to think of a good comprehensive reply :)

Hey @greenpau and @lbreitk I can give it a try, if that's okay, as this is exactly what I need. I am not super experienced in Go, but sounds fun anyway. :D

FYI, starting working on this issue.

@lbreitk , @phss , this is now working. See docs.


Maester of Sunspear

Games crash when a player tries to trigger the maester's ability (after you lose a challenge in which the maester is participating choose an attachment and return it to its owners hand).

Thanks for reporting. This is already fixed and will go live with the next deployment.

The fix should be deployed now.


IR-181: Use only one replica on single node clusters

/retest

/assign @ricardomaraschini

/assign @wzheng1 @bmcelvee @sferich888
Please review this PR and tag it with xxx-approved if everything is fine.

/retest

/assign @xiuwang  Thanks!

Test this pr with single node, the image-registry default replicas is 1.
$ oc get node
NAME                                         STATUS   ROLES    AGE   VERSION
ip-10-0-139-18.us-east-2.compute.internal    Ready    master   58m   v1.20.0+5f82cdb
ip-10-0-148-187.us-east-2.compute.internal   Ready    worker   52m   v1.20.0+5f82cdb
ip-10-0-189-41.us-east-2.compute.internal    Ready    master   57m   v1.20.0+5f82cdb
ip-10-0-223-113.us-east-2.compute.internal   Ready    master   58m   v1.20.0+5f82cdb
$ oc get pods -n openshift-image-registry
NAME                                               READY   STATUS    RESTARTS   AGE
cluster-image-registry-operator-688447bb48-lqbfp   1/1     Running   0          26m
image-registry-dc65bf8df-p8z2m                     1/1     Running   0          33m
node-ca-24n85                                      1/1     Running   0          53m
node-ca-724qv                                      1/1     Running   0          53m
node-ca-p2cx7                                      1/1     Running   0          53m
node-ca-wgrdt                                      1/1     Running   0          53m

/lgtm

@xiuwang for features we use xxx-labels, like this: https://github.com/openshift/cluster-image-registry-operator/pull/640#issuecomment-773716734

/label qe-approved

/retest

/label docs-approved

/retest

/retest

/label px-approved
/unassign @sferich888


Update release-notes for 1.3.6

Signed-off-by: Kavitha Conjeevaram Mohan<EMAIL_ADDRESS>Description
Add jackson package version bump PR under bug fixes in release notes
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #1068 (fa39a84) into 1.x (d249f4b) will increase coverage by 21.69%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##                1.x    #1068       +/-   ##
=============================================
+ Coverage     54.87%   76.56%   +21.69%
  Complexity      283      283
=============================================
  Files           244       38      -206
  Lines          7747     2155     -5592
  Branches       1551      228     -1323
=============================================
- Hits           4251     1650     -2601
+ Misses         3326      364     -2962
+ Partials        170      141       -29

Flag
Coverage Δ

dashboards-observability
?

opensearch-observability
76.56% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...application_analytics/helpers/modal_containers.tsx

...izations/count_distribution/count_distribution.tsx

...ashboards-observability/common/constants/shared.ts

...ublic/components/notebooks/components/notebook.tsx

...onfig_editor/config_controls/config_thresholds.tsx

...ed_components/plotly_viz_editor/plotly_actions.tsx

...ts/notebooks/components/helpers/default_parser.tsx

...y/public/components/explorer/slices/field_slice.ts

...ts/explorer/visualizations/drag_drop/providers.tsx

...tics/components/traces/service_breakdown_panel.tsx

... and 196 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Fix some STA clock issues

@maliberty @donn before this patch we run final STA with ideal clocks, unless the SDC file has set_propagated_clock in it. That seems bad since it could hide hold violations in a design.

@shalan fyi

@antonblanchard I find this a bit confusing.  You seem to want to make propagated clocks on by default (using unset_propagated_clock in many places) but then you don't make it the default and require a -post_cts flag.  It seems like you would want a -pre_cts flag.  This change doesn't seem to cover eco.

Makes sense, I'll flip the polarity and use -pre_cts.
One thing that confused me is there are two completely different ways to call the same STA code:

Standalone:

run_sta -post_cts -log $::env(routing_logs)/parasitics_sta.log

Inline:

set ::env(RUN_STANDALONE) 0
source $::env(SCRIPTS_DIR)/openroad/sta.tcl

The standalone version is only used in two places. Synthesis, which makes sense, since we are in yosys and need to call out to openroad. It's also called for our final STA, and I'm not entirely sure why. It does allow the STA output to be placed into its own log file, but is that enough of a reason?
Calling it directly means the caller is responsible for correctly setting up the SDC parameters, and we wouldn't have this issue.

idk, @donn any comments?

Well, sometimes it's a step and sometimes (most of the time) it's run as part of another step. The final sta is treated as a standalone step independent of routing iirc.

I created PR #969 to flip the polarity


strange values in the log (incomplete logging?)

Hi, my package list is empty at one instance, but they are read out in the logged json.
Can't find any error/failure in the logs, but there are many apparently cropped lines, such as these (marked bold):

20240205_151625 [endPointByID] endpoint passed to endPointByID: packages/id/179
XYZ.jamfcloud.com
20240205_151625 [ApiAction.getPackages] looking up: DNGConverter_15_4.pkg
.com/JSSResource/packages/id/179
h/token)
20240205_151625 [JamfPro.getToken] Generate token for source server: https://XYZ.jamfcloud.com
20240205_151625 [JamfPro.getToken] Use existing token from Optional(https://XYZ.jamfcloud.com/api/v1/auth/token)
20240205_151626 [JamfPro.getToken] Generate token for source server: https://XYZ.jamfcloud.com
20240205_151626 [JamfPro.getToken] Use existing token from Optional(https://XYZ.jamfcloud.com/api/v1/auth/token)
20240205_151626 [JamfPro.getToken] Generate token for source server: https://XYZ.jamfcloud.com
20240205_151626 [endPointByID] fetching XML from: https://XYZ.jamfcloud.com/JSSResource/packages/id/178
h/token)
20240205_151626 [endPointByID] endpoint passed to endPointByID: packages/id/185
XYZ.jamfcloud.com

I see the behavior on a 11.1.1 sandbox, on another 11.2.0 it works normally, but also with the strange log outputs.
I wanted to open an issue about the empty package list, but I think the logging would have to be fixed first.
tested with jamfcpr 4.0.2
Two Jamf Pro Sandbox Cloud Instances on 11.1.1 & 11.2.0
with User+PW and API Client
best regards and thx for your great work !

v4.1.1 has been posted. Please re-open the issue if the behavior persists.


Applying doubleSum over doubleFirst aggregations in nested groupBy fails

The query
{       "queryType": "groupBy",       "dataSource": {         "type": "query", "query": {           "queryType": "groupBy",           "dataSource": "data",           "aggregations":[{ "type": "doubleFirst", "name": "value_num", "fieldName": "value_num" }],           "dimensions": ["object_id"],           "intervals": "2017-06-30T00:00:00/2020-10-02T16:46:40",           "granularity": "day"         }       },       "intervals": "2017-06-30T00:00:00/2020-10-02T16:46:40", "granularity": "day",       "aggregations":[{ "type": "doubleSum", "name": "value_num", "fieldName": "value_num"}],       "dimensions": []     }
fails with error:
"errorMessage" : "Unknown type[class io.druid.collections.SerializablePair] for field",
"errorClass" : "io.druid.java.util.common.parsers.ParseException",

Log with stack trace
log.txt

how can i use nested groupBy ?

FYI, I got a fix here https://github.com/druid-io/druid/pull/5699 though it is a bit symptomatic, there might be a better way to address it, since this incompatibility of values returned by first/last aggregators might be hitting other places too.

Hi all,
DoubleFirstAggregator returns SerializedPairs  which is the intermediate format to merging doubleFirst values from multiple historicals. So, to compute some aggregations like sum over doubleFirst, you first need to finalize the intermediate result to be the final format which contains only the double value. You can use expression postAggregator to do this. Here is an example.
{
  "queryType": "groupBy",
  "dataSource": {
    "type": "query",
    "query": {
      "queryType": "groupBy",
      "dataSource": "twitter",
      "aggregations": [
        {
          "type": "doubleFirst",
          "name": "tweets",
          "fieldName": "tweets"
        }
      ],
      "postAggregations": [
        { "type" : "expression", "name": "tweets_first", "expression" : "tweets" }
      ],
      "dimensions": [
        "lang"
      ],
      "intervals": "2018-07-27/2018-07-28",
      "granularity": "hour"
    }
  },
  "intervals": "2018-07-27/2018-07-28",
  "granularity": "hour",
  "aggregations": [
    {
      "type": "doubleSum",
      "name": "value_num",
      "fieldName": "tweets_first"
    }
  ],
  "dimensions": []
}

@jihoonson thanks for the tip, indeed this makes the query work fine. I'm wondering though, if this use-case shouldn't be possible to handle automatically. In fact Druid has the concept of "finalized" and not-finalized values from aggregators and it can apply it correctly to first/last aggregators, which implement that pattern, e.g.:
{
    "queryType": "groupBy",
    "dataSource": "pm_data",
    "aggregations":[{ "type": "doubleFirst", "name": "value_num", "fieldName": "value_num" }],
    "dimensions": ["counter_id"],
    "intervals": "2017-06-30T00:00:00/2017-07-02T16:46:40",
    "filters": {"type": "selector", "dimension": "version", "value": "43:48501"},
    "granularity": "day"
}

works fine returning just value num without using expression postaggregator. Shouldn't it also apply the same logic in nested query? I suppose outer query will always treat its input as finalized, so in essence the nested query should finalize its results.
Actually, I would like to know how the mechanizm of finalizing results works, since I also had similar problem using Java client classes (com.simplaex.clients.druid.DruidClient, which I know is not supported approach, but still might shed some light on the issue here) when executing above (non-nested) group_by query - the client was not able to parse the results correctly, so I suppose there is some trick / special postprocessing / flag forcing finalization of results.

I'm wondering though, if this use-case shouldn't be possible to handle automatically.

@kskalski good point. We can do unless there's some use cases to get the non-finalized values from the inner query. Probably someone wants to compute a sort of custom aggregator on the pairs of <timestamp, first/last value>.

so I suppose there is some trick / special postprocessing / flag forcing finalization of results.

Yes, there is a query context flag, called finalize, to get finalized results. FinalizeResultsQueryRunner is the query runner which automatically checks this flag and do some necessary work. In GroupByStrategyV2.mergeResults(), the broker sets finalize to false to get non-finalized results from historicals. Once it merges all results from historicals, it finalizes the merged results. Please check ClientQuerySegmentWalker.makeRunner() and FluentQueryRunnerBuilder.applyPostMergeDecoration().

@jihoonson is it safe to explicitly set "finalized": true in the inner query context as an alternative to using an expression postAggregator?
I assume the broker will reset finalized to false so that it gets the unfinalized results from the historicals.  I've tested in a single machine setup, but wanted to confirm that this is expected to work properly in a fully distributed environment with multiple historicals.
{
  "queryType": "groupBy",
  "dataSource": {
    "type": "query",
    "query": {
      "queryType": "groupBy",
      "dataSource": "twitter",
      "aggregations": [
        {
          "type": "doubleFirst",
          "name": "tweets",
          "fieldName": "tweets"
        }
      ],
      "context": {
        "finalize": true
      }
      "dimensions": [
        "lang"
      ],
      "intervals": "2018-07-27/2018-07-28",
      "granularity": "hour"
    }
  },
  "intervals": "2018-07-27/2018-07-28",
  "granularity": "hour",
  "aggregations": [
    {
      "type": "doubleSum",
      "name": "value_num",
      "fieldName": "tweets"
    }
  ],
  "dimensions": []
}


Add pages to top menu

Adding .md files, like the program.md is not enough to have the jekyll website style pick them up and include them in a top menu in the homepage. Finding out how to do would be great.

Closed by https://github.com/unitaryfund/openquantumhardware-workshop/commit/0f9a110f97d0974d493949f35faf0dbd894c516b


Allow customised manuscript HTML for each TEI catalogue

Move convert2HTML.xsl for generating HTML for manuscript pages out of the medieval-mss repository, into consolidated-tei-schema, and create a stub XSLT stylesheet in each of the TEI catalogue repositories, which then imports the central one.
There seems to be a few obviously Medieval-specific things already in the XSLT, which will need moving out and possibly a generic replacement created.
This will provide the option for customisation (but no necessity to do so.) The import href will have to be a URL, meaning processing and previewing in Oxygen won't be possible offline, or if github goes down, but then the schema is already accessed via a raw.githubusercontent.com URL.

This is a good idea in theory, but it assumes more similarity than difference between the data in the different catalogues, and the output they require. Do we need to assess these issues before proceeding? Otherwise it may end up introducing more complexity than it removes. @eifionjones will be working on the Fihrist version of convert2html, so perhaps he could report back on this?

From: andrew-morrison<EMAIL_ADDRESS>Sent: 22 November 2017 10:51
To: bodleian/consolidated-tei-schema
Cc: Subscribed
Subject: [bodleian/consolidated-tei-schema] Allow customised manuscript HTML for each TEI catalogue (#4)
Move convert2HTML.xsl for generating HTML for manuscript pages out of the medieval-mss repository, into consolidated-tei-schema, and create a stub XSLT stylesheet in each of the TEI catalogue repositories, which then imports the central one.
There seems to be a few obviously Medieval-specific things already in the XSLT, which will need moving out and possibly a generic replacement created.
This will provide the option for customisation (but no necessity to do so.) The import href will have to be a URL, meaning processing and previewing in Oxygen won't be possible offline, or if github goes down, but then the schema is already accessed via a raw.githubusercontent.com URL.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/bodleian/consolidated-tei-schema/issues/4, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ATThVCEgtci_t9CPw3cWlYs6AUGbhe5Iks5s4_yzgaJpZM4QnNKd.

I'm keen on making sure we have as little customised behaviour between the catalogues as possible, since it will be easier for us to fix issues and make improvements if we can do that centrally, rather than have 10 different XSLTs that we need to update with the same fix.
I think @andrew-morrison has a solution where we can import the 'central' XSLT, whose behaviour can be overridden by local rules as necessary. This seems to me to be a solution that provides the most amount of flexibility while still ensuring we can support improvements across all catalogues.

Here is a random example of a British Library manuscript, first how it appears currently on the Fihrist QA site, with fields determined by Blacklight...

...and below is a screenshot of the result of running the TEI through an XSLT stylesheet that just imports the Medieval convert2HTML.xsl and wraps whatever its templates output in html and body tags (plus the CSS from Medieval's live site)...

...obviously, the links don't work.
I don't really have a good idea what kind of customizations might be desirable. Say if Fihrist want 'History' first, followed by 'Contents', then @eifionjones could write a new template to match msDesc that imposes an order (because Medieval appear to just follow the ordering in the TEI), and put that in their convert2HTML.xsl, and it would override. Pretty simple, self-explanatory, and unlikely to cause issues going forward. But if they want the dimensions handled as on their current web site, in a HTML table and without the 'written' field, that would require overriding a couple of templates, and maybe some UX work on the CSS.

I think it would be worthwhile trying this with a larger Fihrist MSS as well, since that's when the difference between the Blacklight display and the TEI HTML display will become apparent.
Following the TEI order is a good default behaviour. Was ordering specifically mentioned as an issue with the Fihrist data?

I think we'd definitely want Fihrist to be as standard as possible here, including following TEI ordering. I'll take a look at the xslt with some of our records and see how they look. My impression is that people will just be happy to be able to see the full information!

I've added two new XSLT files to the fihrist-mss repository's processing folder:

convert2HTML.xsl: Does nothing except import Medieval's convert2HTML.xsl
previewManuscript.xsl: Import the local convert2HTML.xsl and wraps whatever it outputs in html and body tags, adding CSS, for people previewing as they edit the TEI in Oxygen.

@eifionjones: If you want to try this out, maybe add some customizations to this new Fihrist convert2HTML.xsl if you find something that does need it, and let us know how it goes. Then, if this looks like it'll work, I can set everyone up with the same, only importing from a common XSL rather than Medieval's copy of convert2HTML.xsl (and I'll rename the common one to something else so it's less confusing.)

Can we point to a copy of the XSLT in this repo, instead of the Medieval one?

Just to clarify, this is temporary. I'll point all, including Medieval and Fihrist, to a copy of the XSLT which I'll move to this repository once we're all happy this method of allowing (minimal) customization will work.

Sorry, I've realised that importing one stylesheet into another causes a problem when run by generate-html.sh, because the original convert2HTML.xsl goes looking for TEI files based on a path relative to itself (the XSLT stylesheet), which won't be correct when that's imported from a different repository (and via a URL.)
I think the only way around it is to modify generate-html.sh to specify the path to the TEI files as a parameter. I'll set that up tomorrow morning.

That would be fine -- the generate-html.sh script is a convenience that can be amended as needed.

It's taken me a while to remember the secret sauce to get around this problem (which is to prefix 'file://' to force the use of a local path) but it's now working.
The 'processing' folder in Fihrist now has a generate-html.sh. Running this will batch convert all TEI in the collections folder into HTML in a 'html' subfolder of 'processing' which it creates (maybe not ideal as this is within the local git repository, so will show up as thousands of untracked files along with any new TEI anyone might have created.)
Also perhaps a Windows-equivalent shell script is needed, if other people are going to run it, say when Huw is on holiday? They'll have the Saxon XSLT processor if they've got Oxygen, but it also uses a tool called "tidy" to convert the almost-HTML output by the XSL into full HTML - I'm not sure why that can't be done in the XSL - which most won't have or know how to install.

I have taken a copy of previewManuscript.xsl into the medieval-mss/processing folder since it will be extremely useful there as well especially as we bring colleges into TEI cataloguing

@holfordm: Are you happy for me to switch Medieval over to using a convert2HTML.xsl that imports all the actual code to generate the manuscript HTML view from a consolidated XSL stylesheet common for everyone, with the option to override if necessary?
I've reviewed that XSLT, not changing the functionality, but modifying it to make any such customization easier (basically splitting any templates that contained lots of logic into separate templates which can be overridden individually.) I noticed a few potential problems (e.g. inconsistency in outputting an "Extent:" label) which I could now fix, and whichever of the issues recorded here are still relevant, once the code is all in one place.

Yes, please go ahead with this

I did the switch over this morning. Medieval's convert2HTML.xsl now imports all the actual templates for transforming TEI to HTML from msdesc2html.xsl here just like Fihrist.
Any fixes for issues that seem likely to be applicable to all TEI catalogues (e.g. multiple "Record Sources" headings) will be fixed in msdesc2html.xsl from now on. That does mean issues raised in the medieval-mss repository (or the fihrist-mss one) will likely be fixed by commits to the consolidated-tei-schema repository. Please bear with me while I experiment with GitHub's ability to link across repositories.
The next step is to set up the Fihrist QA site to display HTML for manuscripts. I've created an issue over in that repository for that (bodleian/fihrist-mss#15) so I'm closing this one as I think I've done all I can to permit sharing and customizing the HTML view between catalogues, and if does turn out to be impractical it would be easy to split them.


CI/CD build issues with Erlang 27

This seems like it might be a problem with zipper, katana-code, rebar3_hank, rebar3_formatter, or rebar3_lint?

CI/CD log for example failure: https://github.com/ut-proj/undertheory/actions/runs/10674332554/job/29584638948


feat(tutorial): complete step 5 - RTP

Closes #
{{short description}}
Bluemix link: carbon-tutorial-rodney-wilson.mybluemix.net
Changelog
New

{{new thing}}

Changed

{{change thing}}

Removed

{{removed thing}}

Per instructions in the tutorial step 5, please include your mybluemix.net URL for your deployed app in your pull request description.

Thank you. The PR description is updated with the Bluemix URL

👍  Looks good!
Feel free to stop this running application on IBM Cloud to avoid the possibility of incurring any charges 💵
Please see the wrapping up step to apply for your badge!
https://www.carbondesignsystem.com/tutorial/wrapping-up


fix(versions): update Activiti/activiti-cloud-dependencies versions into develop

UpdateBot pushed maven dependency: org.activiti.cloud.rb:activiti-cloud-runtime-bundle-dependencies to: 7.1.66

UpdateBot commands:
updatebot push-version --kind maven org.activiti.cloud.rb:activiti-cloud-runtime-bundle-dependencies 7.1.66


There is no option for responsive.

Hello
 There is no option for make responsive carousel so can add that?

@desaiarchana :tada:  Done!


Improve code and add tests in pkg/updateengine package

Looking at the existing code, the following improvements could be implemented:

[x] Fix styling:

[x] Use compacted one-line ifs.
[x] Don't use new keyword.
[x] Don't set struct fields directly, create local variable then pass it to struct instead.

[x] Return updateengine client interface rather than client directly.
[x] Take D-Bus connection as a dependency or accept a constructor method using an interface, to allow easier testing.
[x] Use AddMatchSignal() rather than raw bus object call.
[x] Limit receive channel direction on ReceiveStatuses to allow only sending values.
[ ] Test string-formatting status.
[ ] Implement error checking for converting D-Bus response body to updateengine status, right now code will panic on bad status.

Some done in #108.


Is there a demo of Vue?

Is there a demo of Vue?

https://github.com/Particle-Network/particle-web-demo/tree/master/packages/vue-vite-starter


Bump path-to-regexp to 1.9.0

Description
Bump path-to-regexp to 1.9.0
Issues Resolved
Resolves CVE: https://github.com/advisories/GHSA-9wv6-86v2-598j
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

CI is failing due to deprecated checkout action usage:
 Error: This request has been automatically failed because it uses a deprecated version of `actions/upload-artifact: v1`. Learn more: https://github.blog/changelog/2024-02-13-deprecation-notice-v1-and-v2-of-the-artifact-actions/

Ran unit tests locally:
Test Suites: 14 passed, 14 total
Tests:       1 skipped, 93 passed, 94 total
Snapshots:   29 passed, 29 total
Time:        173.425 s
Ran all test suites.
✨  Done in 176.80s.


InvalidMutabilityException with code similar to RandomLettersGenerator

I’m fairly new to iOS and Kotlin Multiplatform. I have a little app with Android and iOS targets here:
https://bitbucket.org/herrbert74_szinti/funquiz/src/main/
I tried to add a use case with a suspended function to the shared code. It's fairly similar to the RandomLettersGenerator example:
https://bitbucket.org/herrbert74_szinti/funquiz/src/main/shared/src/commonMain/kotlin/com/babestudios/shared/domain/usecase/game/InitGameUseCase.kt
Now, I want to use it in a ViewModel on the iOS side, but whatever I try, the whole class is frozen (I think). It fails with an InvalidMutabilityException, when I try to mutate the GameState. It happens here:
https://bitbucket.org/herrbert74_szinti/funquiz/src/main/iosApp/FunQuiz/Features/Game/GameViewModel.swift
What can I do to avoid freezing the state? Can I prevent the freezing or can I copy the state somehow into a mutable version?

Looking at your code I think the issues is with this line (but I am not 100% sure):
https://bitbucket.org/herrbert74_szinti/funquiz/src/990a7a9cb1a7ca6d5e4ca0a42671eb080fd9b538/iosApp/FunQuiz/Features/Game/GameViewModel.swift#lines-33
Do you have a stacktrace for the InvalidMutabilityException, by any chance?

What can I do to avoid freezing the state? Can I prevent the freezing or can I copy the state somehow into a mutable version?

There is not much you can do about the freezing apart from not switching threads and using only local variables, but that isn't really possible in most applications 🙂.
The best thing you can do is to share immutable (frozen) objects between threads and copy them when needed.
E.g. to "modify" the questionRounds in your Game class you could do the following:
val newGame = game.copy(questionRounds = newQuestionRounds)

Thanks for the quick answer.
It was failing at line 31. That was the stack trace:
Uncaught Kotlin exception: kotlin.native.concurrent.InvalidMutabilityException: mutation attempt of frozen com.babestudios.shared.domain.model.QuestionRound@2bc5b98
    at 0   shared                              0x000000010db649b1 kfun:kotlin.Throwable#<init>(kotlin.String?){} + 97 (/opt/buildAgent/work/c5a36d4d82b914cf/kotlin/kotlin-native/runtime/src/main/kotlin/kotlin/Throwable.kt:24:56)
    at 1   shared                              0x000000010db5eb5d kfun:kotlin.Exception#<init>(kotlin.String?){} + 93 (/opt/buildAgent/work/c5a36d4d82b914cf/kotlin/kotlin-native/runtime/src/main/kotlin/kotlin/Exceptions.kt:23:58)
    at 2   shared                              0x000000010db5ed7d kfun:kotlin.RuntimeException#<init>(kotlin.String?){} + 93 (/opt/buildAgent/work/c5a36d4d82b914cf/kotlin/kotlin-native/runtime/src/main/kotlin/kotlin/Exceptions.kt:34:58)
    at 3   shared                              0x000000010db8e76d kfun:kotlin.native.concurrent.InvalidMutabilityException#<init>(kotlin.String){} + 93 (/opt/buildAgent/work/c5a36d4d82b914cf/kotlin/kotlin-native/runtime/src/main/kotlin/kotlin/native/concurrent/Freezing.kt:24:85)
    at 4   shared                              0x000000010db8fddf ThrowInvalidMutabilityException + 431 (/opt/buildAgent/work/c5a36d4d82b914cf/kotlin/kotlin-native/runtime/src/main/kotlin/kotlin/native/concurrent/Internal.kt:109:11)
    at 5   shared                              0x000000010dc8e7f0 MutationCheck + 128
    at 6   shared                              0x000000010db42d1f kfun:com.babestudios.shared.domain.model.QuestionRound#<set-playersAnswer>(kotlin.Int?){} + 95 (/Users/zsoltbertalan/git/funquiz/shared/src/commonMain/kotlin/com/babestudios/shared/domain/model/GameModel.kt:25:35)
    at 7   shared                              0x000000010db4b8ef objc2kotlin.60 + 191 (/<compiler-generated>:1:0)
    at 8   FunQuiz                             0x000000010d3bcaa6 $s7FunQuiz13GameViewModelC15onAnswerClicked8positionySi_tF + 886 (/Users/zsoltbertalan/git/funquiz/iosApp/FunQuiz/Features/Game/GameViewModel.swift:31:41)
    at 9   FunQuiz                             0x000000010d3c5cec $s7FunQuiz8GameViewV4bodyQrvgySicfU1_ + 92 (/Users/zsoltbertalan/git/funquiz/iosApp/FunQuiz/Features/Game/GameView.swift:21:52)
    at 10  FunQuiz                             0x000000010d3b7bd2 $s7FunQuiz13AnswerContentV4bodyQrvg7SwiftUI9TupleViewVyAE0I0PAEE7paddingyQrAE10EdgeInsetsVFQOyAiEE4fontyQrAE4FontVSgFQOyAiEE5frame8minWidth05idealQ003maxQ00P6Height0rT00sT09alignmentQr12CoreGraphics7CGFloatVSg_A0_A0_A0_A0_A0_AE9AlignmentVtFQOyAE4TextV_Qo__Qo__Qo__AE6HStackVyAGyAiEE12onTapGesture5count7performQrSi_yyctFQOyAA0C6ButtonV_Qo__A15_tGGA17_tGyXEfU_A16_yXEfU_yycfU_ + 66 (/Users/zsoltbertalan/git/funquiz/iosApp/FunQuiz/Features/Game/Views/AnswerContent.swift:27:21)
    at 11  SwiftUI                             0x00007fff5c347014 $sIeg_ytIegn_TR + 12
    at 12  SwiftUI                             0x00007fff5c8dc874 $s7SwiftUI14EndedCallbacks33_2AC25C39D2868D32DD07A6EAF40D8166LLV8dispatch5phase5stateyycSgAA12GesturePhaseOyxG_ytztFyycfU_TA + 41
    at 13  SwiftUI                             0x00007fff5c566b4b $sIeg_ytIegr_TR + 12
    at 14  SwiftUI                             0x00007fff5c8dc95d $sIeg_ytIegr_TRTA + 17
    at 15  SwiftUI                             0x00007fff5c8dd277 $sIeg_ytIegr_TRTA.12 + 9
    at 16  SwiftUI                             0x00007fff5c566b5f $sytIegr_Ieg_TR + 12
    at 17  SwiftUI                             0x00007fff5c566b4b $sIeg_ytIegr_TR + 12
    at 18  SwiftUI                             0x00007fff5c8dc95d $sIeg_ytIegr_TRTA + 17
    at 19  SwiftUI                             0x00007fff5c8dd282 $sIeg_ytIegr_TRTA.19 + 9
    at 20  SwiftUI                             0x00007fff5c5407cf $s7SwiftUI6UpdateO3endyyFZ + 484
    at 21  SwiftUI                             0x00007fff5c62f22a $s7SwiftUI19EventBindingManagerC4sendyySDyAA0C2IDVAA0C4Type_pGF + 277
    at 22  SwiftUI                             0x00007fff5cb3a99e $s7SwiftUI18EventBindingBridgeC4send_6sourceySDyAA0C2IDVAA0C4Type_pG_AA0cD6Source_ptFTf4nen_nAA22UIKitGestureRecognizerC_Tg5 + 1875
    at 23  SwiftUI                             0x00007fff5cb38ed8 $s7SwiftUI22UIKitGestureRecognizerC4send025_062C14327F4C9197D92807A7H6DF7F3BLL7touches5event5phaseyShySo7UITouchCG_So7UIEventCAA10EventPhaseOtF + 66
    at 24  SwiftUI                             0x00007fff5cb39675 $s7SwiftUI22UIKitGestureRecognizerC12touchesBegan_4withyShySo7UITouchCG_So7UIEventCtFToTm + 131
    at 25  SwiftUI                             0x00007fff5cb38f60 $s7SwiftUI22UIKitGestureRecognizerC12touchesEnded_4withyShySo7UITouchCG_So7UIEventCtFTo + 40
    at 26  UIKitCore                           0x00007fff24ad439d -[UIGestureRecognizer _componentsEnded:withEvent:] + 217
    at 27  UIKitCore                           0x00007fff25095964 -[UITouchesEvent _sendEventToGestureRecognizer:] + 662
    at 28  UIKitCore                           0x00007fff24ac84eb __47-[UIGestureEnvironment _updateForEvent:window:]_block_invoke + 70
    at 29  UIKitCore                           0x00007fff24ac8175 -[UIGestureEnvironment _updateForEvent:window:] + 521

I copied the game class, but it was failing with the QuestionRound class. When I copied it, the exception went away. I'm not sure if this is the best solution, but at this stage of learning, it's great, that it works.
Thanks for your help and for the library!


Uniformity feature: Support mutual information for cells of observed rows

Should be much faster than unobserved rows, because the cluster assignment in each model is assumed known.  The current MI code path I am aware of only computes MI of columns for unobserved (new) rows

Being a little unclear on the math, I'm not sure what the use case looks like. Probability of dependence estimates the probability that mutual information between columns is nonzero? Similarity measures the mutual information between observed rows? What does this effectively mean for individual observed cells?

The architecture is that we have an infinite exchangeable set of tuples of random variables {(A_r, B_r, C_r)}_r, and we approximate the posterior distribution given certain assignments A_0 = a_0, C_1 = c_1, &c.  Currently we can only approximate mutual information for two random variables A_i, B_i in the same row i for which no values have been assigned.  This issue is to allow approximating it for two random variables from a row that has been observed.
The architecture more specifically for Crosscat is that there are additional categorical variables {(L_r, M_r, N_r)}_r which we cannot observe, nor even whose number can we observe.  Each Crosscat state is a sample from the distribution on latent variable numbers (views) and assignments (categories).  Each model estimator evaluates a Monte Carlo integral (1) over samples of Crosscat states of some function of a single Crosscat state.
In this case, approximating mutual information of variables of an entirely unobserved row from a single Crosscat state means evaluating a Monte Carlo integral (2) over samples of category assignments of some mutual information estimator (itself a Monte Carlo integral (3) over samples of the posterior predictive distribution on the variables given the category assignments).
What @axch proposes is to do is to implement approximation of mutual information of variables of an observed row from a single Crosscat state, in which implementation the Monte Carlo integral (2) is replaced by a single evaluation of (3), given the fixed category assignments of that observed row in that Crosscat state, rather than a Monte Carlo integral over samples of category assignments of evaluations of (3).


rbw: 0.4.4 -> 0.4.6

Motivation for this change
Update rbw
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[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.

tested locally, binaries work. merging


OS X: add various process status tests defined by wait.h

Adds the various process test macros specified in the wait(2) manpage on OS X. WCOREDUMP is implemented BSD-wide as it seems to be the only macro whose definition is constant across the BSDs.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

Thanks! Just one small nit but otherwise lgtm

Good call. I've pushed a change to rename it to _WSTATUS to match the header, but the obvious other solution here is to just repeat the (status & 0x7f) mask in each of the macros if that is undesirable.

@bors: r+ 06abf3a
Sounds good to me!

:hourglass: Testing commit 06abf3a with merge 6ad1be7...

:sunny: Test successful - status-appveyor, travis


page layout: sheet right-to-left

hi
i need to export right-to-left sheet, that A1 column is in right side of page (not left side)
in microsoft excel we have this option in "Pagelayout" tab > "sheet options" > "Sheet-Right-to-Left"
i searched all readme and codes for "righttoleft" or sth like that, but nothing was founded
are we have this option in sheetjs ?
thank you.

i found it !
i changed source code.
its works correctly.
we should change this line
https://github.com/SheetJS/js-xlsx/blob/b3873ea61583549a0ce1790b724e2a129ee38eb5/bits/67_wsxml.js#L196
from
return writextag("sheetViews", writextag("sheetView", null, {workbookViewId:"0"}), {});
to
return writextag("sheetViews", writextag("sheetView", null, {workbookViewId:"0", rightToLeft: "1"}), {});
but, as you know, it always use Right-to-Left, you can add an option for it

The sheetViews block should be generated dynamically.  If you have patience, we could accept a PR with the following changes:
The spec <IP_ADDRESS> states that there can be multiple sheet views, so the best place to store the information would be as an array within the Workbook attributes, namely Workbook.Views:
var wb = {
    Workbook: {
        Views: [
            {RTL:true}
        ]
        ...
    }
    ...
    Sheets: {...},
    SheetNames: [ ... ]
}

This would correspond to the following code:
var sview = {workbookViewId:"0"};
if( (((wb||{}).Workbook||{}).Views||[])[0] ) sview["rightToLeft"] = wb.Workbook.Views[0].RTL ? "1" : "0";
return writextag("sheetViews", writextag("sheetView", null, sview), {});

And your code would use:
function set_right_to_left(wb/*:Workbook*/) {
    if(!wb.Workbook) wb.Workbook = {};
    if(!wb.Workbook.Views) wb.Workbook.Views = [];
    if(!wb.Workbook.Views[0]) wb.Workbook.Views[0] = {};
    wb.Workbook.Views[0].RTL = true;
}

thank u very much 👍

Hi huys, the issue doesnt seems to be resolved.
I tried to add wb.Workbook.Views[0].RTL = true; but with no results.
when I changed (in the depths of the dist) to
return writextag("sheetViews", writextag("sheetView", null, {workbookViewId:"0", rightToLeft: "1"}), {});
it worked

Hi huys, the issue doesnt seems to be resolved.
I tried to add wb.Workbook.Views[0].RTL = true; but with no results.
when I changed (in the depths of the dist) to
return writextag("sheetViews", writextag("sheetView", null, {workbookViewId:"0", rightToLeft: "1"}), {});
it worked

I have written the code that will do this after many tries, the code must be like so
var wb = { Workbook: { Views: [{ RTL: true }] }, Sheets: {}, SheetNames: [] }
var worksheet = XLSX.utils.aoa_to_sheet([insert_your_data_here]);

XLSX.utils.book_append_sheet(wb, worksheet, "Sheet 1");
XLSX.writeFile(wb, "excel_template.xlsx");

Is this capability of changing the main X/Y axis from top left to top right (LTR To RTL) still supported in the community version?

@mahmoudmohamedramadan
Do you know if it is mandatory to first work with aoa and use the 2nd row you shared that utilizes aoa_to_sheet(...) in order for the RTL customization to work properly?


Hub.Clients.User() and Hub.Clients.Group() compare string case-sensitivly

Hub.Clients.User() and Hub.Clients.Group() function search connection case sensitively, e.g.
Hub.Clients.Group("abc") does not find connections via Hub.Groups.AddAsync(Groups.AddAsync(Context.ConnectionId, "ABC")

Versions of Server-Side NuGet Packages: 1.0.0-preview1-final

That's by design, the group names aren't case sensitive. You should normalize your group names before adding and sending to the group.

We periodically close 'discussion' issues that have not been updated in a long period of time.
We apologize if this causes any inconvenience. We ask that if you are still encountering an issue, please log a new issue with updated information and we will investigate.


What is the safest place to put secret keys in the backend

Hi,
I have a SECRET KEY for STRIPE that i have to use in the backend. This secret key should not be exposed to anyone. Where do you think would be the safest place to put this and how to do it.
Regards

@SumayaG Yeah it definitely shouldn't be in your code. You could make it a part of your build service. Or use AWS SSM. Here is some more info on it https://hackernoon.com/you-should-use-ssm-parameter-store-over-lambda-env-variables-5197fc6ea45b

@jayair Thanks, I'll look into AWS SSM.


bounding box area of match

hai thank you for making this good library for matching image but i have a question.
can i get the bounding box area of the matching image ? i think if that feature is added we can use lightglue for another use case like template matching maybe.
thank you

Hey @RickyGunawan09, sorry for the late reply. Yes you can compute the bounding box. After matching the images, just compute the bounding box around the matched points in both images (taking min/max of all xy-coordinates from the matched points).
Something like this should work (second example from the demo notebook):
from matplotlib.patches import Rectangle
import matplotlib.pyplot as plt
image0 = load_image(images / "sacre_coeur1.jpg")
image1 = load_image(images / "sacre_coeur2.jpg")

feats0 = extractor.extract(image0.to(device))
feats1 = extractor.extract(image1.to(device))
matches01 = matcher({"image0": feats0, "image1": feats1})
feats0, feats1, matches01 = [
    rbd(x) for x in [feats0, feats1, matches01]
]  # remove batch dimension

kpts0, kpts1, matches = feats0["keypoints"], feats1["keypoints"], matches01["matches"]
m_kpts0, m_kpts1 = kpts0[matches[..., 0]], kpts1[matches[..., 1]]

viz2d.plot_images([image0, image1])
viz2d.plot_matches(m_kpts0, m_kpts1, color="lime", lw=0.2)
viz2d.add_text(0, f'Stop after {matches01["stop"]} layers')

tl = m_kpts1.min(dim=-2).values  # top-left corner
br = m_kpts1.max(dim=-2).values  # bottom-right corner
axes = plt.gcf().axes
axes[1].add_patch(Rectangle(tl, br[0] - tl[0], br[1]-tl[1], fill=False, color="lime"))

kpc0, kpc1 = viz2d.cm_prune(matches01["prune0"]), viz2d.cm_prune(matches01["prune1"])
viz2d.plot_images([image0, image1])
viz2d.plot_keypoints([kpts0, kpts1], colors=[kpc0, kpc1], ps=6)


[Frontend] Add support for jax.jit and jax.grad on top of qjit functions

Implements option (1) presented in issue #15 to enable JAX transformation support on qjitted functions.
[sc-34661]

I may suggest updating the quick-start guideline including this new support in Catalyst.

I may suggest updating the quick-start guideline including this new support in Catalyst.

Good idea!


Allow use of 'no_proxy' in the proxies argument

Add the ability to add 'no_proxy' and a value to the 'proxies'
dictionary argument.
https://github.com/kennethreitz/requests/issues/2817
Closes gh-2817

Alright, good start @JohnVillalovos! I've left some feedback in the diff that would be good to have addressed, thanks. =D

Closing due to inactivity (not sure if that's our fault or not, sorry!). Please re-submit if you're still interested in contributing this code :)

As far as I know the submitted patch is still valid, though I haven't dug into it.
I thought I had responded to all comments and latest version took them into account.

@JohnVillalovos that's true but we cannot reopen this since your repository seems to have been deleted.

Okay. Looks like I have some merge conflicts. I will create another pull request after I fix it.

Thanks @JohnVillalovos

Created new pull request: https://github.com/kennethreitz/requests/pull/3865


make transcript writes enabled off be "sticky"

Every time I login, I see markedly slow performance with the first browser I try to open.  So I have to wait for it to finish, go back to the transcript, and unset the make transcript writes enabled option.
I'd like this setting to be remembered or have some way to set it to default to off.

I've added two ways to do this.  In theory this could be a way to hold client & server post login preferences.
Client:

In the Jadeite working directory, add a file called Jadeite.prefs with contents transcriptEnabled false  That should turn off your transcripts by default. fyi - this is the only command understood in this file right now.
Server:
As part of the work for #132 I added a server post login hook. Commit a block as such then login:
UserGlobals at: #JadeitePreferences put: [RowanAnsweringService new turnOffTranscriptWrites]

In theory, we need to have both client & server post login execution available, so that's why I added both.
Added in Oscar 3.0.26

The Jadeite.prefs doesn't work. Verified working directory: is the syntax correct?

The UserGlobals key system does work as expected.

@LisaAlmarode  It appears that the method in JadeitePreferences was stripped out - a perpetual frustration with Dolphin. But I added a sender to the method and it seemed to stick around in the .exe. Fixed in 3.0.27.


node.data is NoneType in map_batch_norm

Hi guys
I have a Caffe model which was transfer learned from GoogLeNet. While trying to convert it the script picks up everything (complete list of layers and such) but after "Saving source..." starts, i get:
....
transformer.py", line 149, in map_batch_norm
scale_offset = len(node.data) == 4
TypeError: object of type 'NoneType' has no len()
Any ideas of what is going on?

same here!

Well worked around it with:
def map_batch_norm(self, node):
if not(node.data is None):
scale_offset = len(node.data) == 4
kwargs = {} if scale_offset else {'scale_offset': False}
return MaybeActivated(node, default=False)('batch_normalization', **kwargs)
else:
return MaybeActivated(node, default=False)('batch_normalization', {'scale_offset': False})
In Transformer.py
No idea if that is OK to be honest but at least the error goes away. WOndering why that node.data is None

@maeotaku  it seems still not work？Does anyone here solve the problem ?

Sadly, I never got it to work properly, so I just moved to Caffe because I cannot lose time on my research :/

Hi guys. I find that the tensorflow code output has to be done along with the data output. It seems that if the code output is done alone, the graph variable of the TensorFlowMapper class in transformer.py is not initialized. It works for me.


Use the output of --live_transcribe

Hi, and thanks for your project, it works flawlessly and I had met no issue at all.
I would just like to ask if there is a way to "trap" or echo somewhere the real time transcriptions obtained with the option --live_transcribe True
Thanks in advance for your support

Sounds like a job for tee.

Thanks I did not know this package. I will try to write a sctipt to integrate tee with --live_transcribe.

Hi, and thanks for your project, it works flawlessly and I had met no issue at all. I would just like to ask if there is a way to "trap" or echo somewhere the real time transcriptions obtained with the option --live_transcribe True Thanks in advance for your support

Looks like You are the happy one who succeed to have live-transcribe to work, can  You share your cli args and what language do You transcribe?

yes, of course. I transcribe Italian... Here is the link to the script I am
now using for live transcription. It is a bit slow in transcribing, so I
always use small model.
https://github.com/fatualux/scripts/blob/main/realtimeWhisper.sh
..:FZ:..

Sent from my Android device -

Il gio 19 ott 2023, 20:44 Michael @.***> ha scritto:

Hi, and thanks for your project, it works flawlessly and I had met no
issue at all. I would just like to ask if there is a way to "trap" or echo
somewhere the real time transcriptions obtained with the option --live_transcribe
True Thanks in advance for your support
Looks like You are the happy one who succeed to have live-transcribe to
work, can You share your cli args and what language do You transcribe?
—
Reply to this email directly, view it on GitHub
https://github.com/Softcatala/whisper-ctranslate2/issues/52#issuecomment-1771528271,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIPQJXGQCASMRWLG6MIUCVDYAFYI3AVCNFSM6AAAAAA2NRAHGGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZRGUZDQMRXGE
.
You are receiving this because you authored the thread.Message ID:
@.***>


fix: require min two cosigners, validate duplicated keys, fix typography

Checklist

[x] JIRA -LW-10994
[x] JIRA -LW-10995
[x] JIRA -LW-10997
[ ] Proper tests implemented
[ ] Screenshots added.

Proposed solution
Explain how does this PR solves the problem stated in JIRA ticket.
You can also enumerate different alternatives considered while approaching this task.
Testing
Describe here, how the new implementation can be tested.
Provide link or briefly describe User Acceptance Criteria/Tests that need to be met
Screenshots
Attach screenshots here if implementation involves some UI changes

Allure Report
allure-report-publisher generated test report!

smokeTests: ✅ test report for b27c1250

passed
failed
skipped
flaky
total
result

Total
32
0
0
0
32
✅


Document Work for Arm System

Might be good idea to do this in coordination with new controls members. Basically describe all the functionality the Arm needs to achieve to give basic outline for drafting Arm code. May want to do this with #182 first since they are already familiar with that.

A little google doc @Coreyboy1820 made here

To bring up the topic on the organization of the inputs from the server, do we need information from the sensor sent to each motor or one value and all the motors operate under this single value? I understand we need: 1. Speed of arm 2. Angle of motor (multiple or one) 3. Values for hand motors (since these should be different from arm motor values). Depending on how we want to tackle the implementation of the arm.... We can give the hand modes, values, or even both. For example, open and closed mode or values for each motor in the hand. Another implementation issue we need to talk about is the single value for the motors or multiple values for the motors, we can pass in one value to one motor (the lead motor of the hand) for example the middle finger or thumb and have the other 4 fingers follow in a similar fashion when closing. What do you guys think? We are in the drawing board stage i'm assuming.

This ties into https://github.com/SJSURoboticsTeam/urc-central-2021/issues/184.. @kammce mentioned we need to use the accelerometer for homing.. not sure how reliable the accelerometer data already is or needs to be but if the homing is inaccurate then the angles we want to set our motors/joints to on the robot arm could subsequently be inaccurate..
Maybe a few things we could do with the mpu6050.. it has gyroscope data and we can combine that with the accelerometer data.. I remember people used accelerometer data for their trial project but maybe not gyroscope data. I think the current mpu6050 driver does not output gyroscope data but I could totally be reading the code wrong - https://github.com/SJSU-Dev2/SJSU-Dev2/blob/master/library/devices/sensors/movement/accelerometer/mpu6050.hpp
We could also do something fancy like a low-pass filter 0 https://www.pieter-jan.com/node/11

idk people's familiarity with gyroscope but it basically gives angular velocity along 3 axes: https://learn.sparkfun.com/tutorials/gyroscope/all

It doesn't need to be very accurate actually. We just need it within a 60 degree band which is pretty massive which has to do with how the motor does position control.

Ah sounds good! Thank you for letting me know!

tbh I'd personally like to explore these purely out of interest but probably not the best use of my time to help finish objectives for the robotics mission. Will probably look into it later.


Add xml support

Added support for XML deserialization in serde_extractor.
Fixed compile errors in tests with disabled features
Fixed warnings (unused import) when non_static feature is disabled

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files
Coverage Δ

src/config.rs
52.08% <ø> (ø)

src/data_providers/http.rs
66.66% <100.00%> (ø)


Use clockwork maid at end of day 2

Hi there,
Absolutely loving ploop. Saves me a ton of time when I'm busy. The only manual command I'm doing is using a clockwork maid at the end of fullday. Any chance that's something worth incorporating into ploop? Otherwise, happy to just do it manually since it's easy.
Thanks,
gregmasta

@gszliu Here's the code for clockwork maid
if (!page.contains_text("Clockwork Maid") && !page.contains_text("Meat Butler") && !page.contains_text("Meat Maid")) {
        if (available_amount($item[Clockwork Maid]) > 0 || buy(1, $item[Clockwork Maid], (8 * get_property("valueOfAdventure").to_int()) - profitOffset) == 1) {
            use(1, $item[Clockwork Maid]);
            print("Installed Clockwork Maid", "green");
        } else if (available_amount($item[Meat Butler]) > 0 || buy(1, $item[Meat Butler], (4 * get_property("valueOfAdventure").to_int()) - profitOffset) == 1) {
            use(1, $item[Meat Butler]);
            print("Installed Meat Butler", "lime");
        } else if (available_amount($item[Meat Maid]) > 0 || buy(1, $item[Meat Maid], (4 * get_property("valueOfAdventure").to_int()) - profitOffset) == 1) {
            use(1, $item[Meat Maid]);
            print("Installed Meat Maid", "lime");
        } else {
            print("Clockwork Maid, Meat Butler, and Meat Maid both outside price range.", "red");
        }
    } else {
        print("We already have a Maid or Butler installed", "red");
    }
    ```
    Either it's mall weirdness (blocked from the 1/day 25k clockwork shop or mall is out of stock) or ur VOA is too low


Alpine.js headless components conflict

Laravel Form Components Version
7.1.6
Laravel Version
9.50.1
Alpine Version
3.10.5
Bug description
$disclosure is a new magic method which ships with the new Alpine beta and is used in the new headless alpine components.
Including the @fcScripts causes the error, while, @fcJavaScript works OK.
Does fcScripts also load AlpineJs?
Steps to reproduce
No response
Relevant log output
No response

@fcScripts does also load Alpine from a CDN. I'm working on v8 of this package right now, and I'm thinking of just getting rid of that directive all together.

v8 of the package should work fine with those components. Feel free to re-open this issue if you're still having compatibility issues.


Fix #2688: Handle comma-separated list tags when using backup command

What is the purpose of this change? What does it change?
Handle comma-separated list tags when using backup command
Was the change discussed in an issue or in the forum before?
closes #2688
Checklist

[x] I have read the Contribution Guidelines
[x] I have enabled maintainer edits for this PR
[x] I have added tests for all changes in this PR
[x] I have added documentation for the changes (in the manual)
[x] There's a new file in changelog/unreleased/ that describes the changes for our users (template here)
[x] I have run gofmt on the code in all commits
[x] All commit messages are formatted in the same style as the other commits in the repo
[x] I'm done, this Pull Request is ready for review

There's another command that operates with []string for tags: the tag command.

Command tag the the right type no ? https://github.com/restic/restic/blob/master/cmd/restic/cmd_tag.go#L40

SetTags / AddTags / RemoveTags use raw lists, which means that they behave exactly like backup did.

Thanks for the contribution! In general I agree what's changed here, we just need to be a bit more explicit in the changelog about it:
Strictly speaking, this is a backwards-incompatible change: before, it was possible (but not sensible) to create tags for snapshots which contain commas, in the same way that you did in the issue. I suspect there are quite a few people who run restic backup --tag foo,bar in their scripts, so with this change this will create snapshots with two tags ["foo", "bar"] instead of one (["foo,bar"]). While not a problem in itself, this will make forget consider the snapshots to be in different groups and will apply the retention policy to each group separately.
How about we add a section:

Attention: If you have (maybe accidentally) run restic backup --tag foo,bar, you have created snapshots with a single tag containing a comma ("foo,bar"). Restic will now create snapshots with two tags ("foo" and "bar"), which may have unexpected results when using restic forget.

Thoughts?

@fd0 The change in behavior of forget would then be that it now is able to forget such snapshots, while that wasn't possible before? Would that qualify as "Change" in the changelog?

Can I do something more ? COVID19 steal my job so 🤷🏻♂

@SkYNewZ Sorry for taking so long to further guide this PR, I've somehow lost track of it. Could you also change the --set, --add and --remove options of the tag command to also use restic.TagLists instead of []string? These are from what I've seen the last places where a tag containing commas was not split. Fixing the broken tags is still possible using tag --set ....
To simplify fixing multiple tags something like restic snapshots --json --quiet | jq '.[] | select(contains({tags: ["test"]})) | .id' would work to get a list of snapshot ids with a certain tag (“test” in the example query). Then use restic tag --set cats --set dogs snapshotID [...] to correct the tags.

before, it was possible (but not sensible) to create tags for snapshots which contain commas [...] Thoughts?

I think many more people will have inadvertently provided --tag=a,b than --tag='Grammatical sentence, or something, deliberately containing commas'; so I'd suggest it's worth re-evaluating existing comma-containing tags and splitting them, storing escaped commas going forwards if it's desirable to admit them still.
It's backwards-incompatible no matter what, but I think silently correcting those wrongs would be friendlier than breaking such users' inadvertently-working filtering, at the expense of breaking extremely few people's (if anyone's) deliberate commas.
That is, IMO, the break should be such that the warning is more like:
Attention: `backup --tag=` now takes comma-separated tags where it previously did not. If you have used this as a feature, for tags that (deliberately) contain commas, you must retag your snapshots to escape the comma before upgrading. For example: `restic snapshots --tag=foo,bar --json | jq -r '.[].id' | xargs restic tag --set='foo\\,bar'`.

@OJFord Not entirely sure if you're suggesting some automatic processing of "foo,bar" tags or not, but regardless, we're planning on making the backup and tag commands coherent with the rest of the commands, such that they do treat e.g. --tag foo,bar as two tags, foo and bar. We will write a changelog that clearly informs that this might affect current forget usage, and include info on how to rewrite the tags. Is this in line with what you meant?

@rawtaz My suggestion is that this change not only going forward for new tags, but also retroactively for existing a,b tags, operating under the assumption that they were always intended to be a & b, not a\,b.
This would of course break (without intervention) for people with deliberately-comma'd tags, but I expect they are far fewer.

Still not sure what you are suggesting, but we agree that a,b was in most cases likely indended as two separate tags rather than the user writing a sentence. I believe that our current goal is in line with what you're thinking.

@OJFord restic snapshots --tag=foo,bar already (that is without this PR) considers foo and bar to be separate tags. That is also the reason why my suggested command to fix the tags requires some jq magic. As one couldn't properly use tags with commas before, I don't think its worth the trouble and complexity to introduce escaping for commas.
Splitting the tag lists of existing snapshots on commas when loading them, would suddenly change grouping by tags in the forget command which seems to be quite problematic to me.

@SkYNewZ Could you rebase again to the current master? That should hopefully fix the temperamental windows tests...
Right now the behavior of the --set, --add and --remove options of the tag command don't match the changelog ;-) . Please change the following lines to also use restic.TagLists
https://github.com/restic/restic/blob/8fca3e7a41f2bc3623a3953cbb681ef3a1f3eb51/cmd/restic/cmd_tag.go#L41-L43

Why restic.TagLists and no restic.TagList ?


Potable water should now return value submitted not 1

What? and Why?

Q_code 110 should now be written to the pck as the value entered by the user, not 1

Codecov Report

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

@@           Coverage Diff           @@
##           master     #156   +/-   ##
=======================================
  Coverage   84.34%   84.34%
=======================================
  Files          19       19
  Lines         990      990
=======================================
  Hits          835      835
  Misses        155      155

Impacted Files
Coverage Δ

transform/transformers/mbs_transformer.py
73.58% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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