Embedded.OnEmpty.USE_NULL field are created when it's aggregate object is loaded

Code
@Table("service_request")
class ServiceRequest {

    @Embedded(onEmpty = Embedded.OnEmpty.USE_NULL, prefix = "name_")
    private PersonalName name;

    // other fields
    ....

}

public record PersonalName(
    @NonNull String firstname, @NonNull String lastname) {

    public PersonalName{
          if (firstname.isBlank()) {
                throw new IllegalArgumentException("firstname can't be blank");
            }

            if (lastname.isBlank()) {
                throw new IllegalArgumentException("lastname can't be blank");
            }
    }

}

Problem
In my database, the corresponding column of the field name: name_firstname and name_lastname are null. When load the aggregate, the framework is trying to create an object of PersonalName with null value, because I got NPE in the constructor.
I don't understand why the object is created when all the fields are null.
And I think the object should not be created, this will cause object with illegal state are created.

The framework is Spring Data JDBC

Version
Spring Data JDBC under spring-boot-starter-parent 2.7
DB driver is postgresql 42.5.1
<parent>
        <groupId>org.springframework.boot</groupId>
        <artifactId>spring-boot-starter-parent</artifactId>
        <version>2.7.0</version>
        <relativePath/> <!-- lookup parent from repository -->
</parent>

...

<dependencies>
        <dependency>
            <groupId>org.springframework.boot</groupId>
            <artifactId>spring-boot-starter-data-jdbc</artifactId>
        </dependency>
        <dependency>
            <groupId>org.postgresql</groupId>
            <artifactId>postgresql</artifactId>
            <version>42.5.1</version>
            <scope>runtime</scope>
        </dependency>
</dependencies>

Thanks, I was able to reproduce this.
A work around would be to not throw an exception in the constructor.
That said, I do consider this a bug.

Sorry, I have to revert that statement.
My original reproducer had a bug that I mistook for reproducing the problem.
The proper attempt at reproducing the issue fails to do so: https://github.com/schauder/issue-JDBC-1506-null-embedded
Could you please modify the reproducer to demonstrate the buggy behaviour?

OK, no problem, I will work on it

I can't reproduce it either both in demo and the original project, I think it's because the state of the DB isn't correct. Thanks for your help and sorry for the trouble.

Thanks for confirming this.


Using extract (2021)

Hi Biafra,
Super excited that the new cell extraction method is released and supported by CIAtah!
I was able to load EXTRACT into the external_programs folder, but under Cell Extraction Methods in the GUI, the only "EXTRACT" is still the 2017 version. Is this the correct method?
Omid

Hey Omid, hope all is well. Yep, that is the correct EXTRACT method. I have that updated locally but need to push and merge the updated branch sometime today/tomorrow. Let me know if you encounter an issues while running EXTRACT.
cheers!
-Biafra

@omidm7 This is updated in the most recent version.

Hi Biafra. I've been successfully running EXTRACT on my laptop, but on the lab computer, I keep getting the attached error.
post_extract_error.pdf
Could this be an issue of using Matlab2017b (desktop) vs 2020b (laptop)?
Omid

hey Omid, looks like the graphconncomp function is causing the error, does your 2017b installation have the Bioinformatics Toolbox installed?
Can you send the output of ciapkg.io.dependencyCheck on the MATLAB command line (no semicolon)?

Thanks, the 2017 version was missing Bioinformatics and a few other toolboxes. Updating everything now, should be good.
Thanks again!
O

Hi Biafra,
Any idea what could be causing this strange partitioning problem? I experienced it with two separate data sets - for some reason the 4 quadrants are not merging smoothly. I've uploaded the configuration/user config files if it helps. Appreciate your help with this!
Best,
Omid

2021_04_08_p001_m01_NULL000_extractAnalysis.mat.zip

hey @omidm7, let me take a look at this and get back with any thoughts on what parameters could be tweaked.

hey @omidm7, let me take a look at this and get back with any thoughts on what parameters could be tweaked.


The 23.05.11 version still can not work on Giraffe.

Please try <IP_ADDRESS>
https://github.com/Grigory-Rylov/android-layout-inspector/releases/download/v<IP_ADDRESS>/android-layout-inspector-plugin-<IP_ADDRESS>.zip


Updating missing metadata only

I've imported hundreds of bookmarks from delicious and I'd like to import pictures and cache those pages (ideally in batch-mode). Clicking the update button in the web interface does that, but it also updates the title of my bookmark. Is there a way to only fetch missing metadata?

Hi @muesli, I've added this feature int f4911d. Could you check it out ?

Seems to work fine!


2.0.0-beta.0

⚠ Please verify that this question has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
How can I set up direct v2. In v1
"Show Clickable Link
If checked, anyone who has access to this status page can access the monitor URL."
Even though I enabled this option, I can't see where the url is.
📝 Error Message(s) or Log
No response
🐻 Uptime-Kuma Version
1.23.15
💻 Operating System and Arch
Ubuntu 24.04 Lts
🌐 Browser
Google Chrome
🖥 Deployment Environment

Runtime:
Database:
Filesystem used to store the database on:
number of monitors:

Hi @ismailkucukonder ! On the status page, the name of your monitor will be clickable. The URL is not directly displayed.


[Book] Verifying Stone proof on Starknet

Add a documentation page describing how to create and verify Stone proof on Starknet. You will be using Integrity verifier available at https://github.com/HerodotusDev/integrity
In addition to verification instructions you’ll also need to provide a guide on how to deploy integrity contracts.
Path to your markdown file: docs/pages/advanced/starknet.md
References:

https://github.com/HerodotusDev/integrity?tab=readme-ov-file#starknet-proof-verification

Hi @dmirgaleev ,Please let me work on this when od hack starts.

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
Hi, I'm Erick part of Dojo Coding, and I'll be working on issue #35 as part of ODHack8.0.
While I don't have direct experience in writing documentation, I have a strong background in blockchain development, particularly with Starknet and smart contract verification. This aligns well with the task, and I’m confident that I can create a clear and comprehensive guide.
How I plan on tackling this issue
Here’s how I will approach the task:
Review the Integrity Verifier and relevant references to fully understand how to create and verify Stone proofs on Starknet.
Write a step-by-step guide that explains how to deploy integrity contracts and verify proofs on Starknet using the provided verifier.
Ensure clarity and accessibility, making the documentation easy to follow for both new and advanced users.
Place the markdown file in the specified path: docs/pages/advanced/starknet.md.
Self-review for accuracy and completeness before submitting the PR for review.

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I have some experience in cryptography. and cairo, and this can be leverged in learning about the stone prover and verifying the proof on starknet
How I plan on tackling this issue
prepare the call data for proof and use one of the verifier contracts on sepolia to verify the proof

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
hello i am a frontend developer and a technical writer
please can i work on this issue :)
i would love to be a contributor on this repo
How I plan on tackling this issue
please kindly assign me and i'll get straight to work

Hello @dmirgaleev! Thank you so much for assigning me the issue, where can I contact you for questions?

Hello @evgongora! Please, don't hesitate to join our Telegram group https://t.me/baking_bad_odhack to ask any questions and share progress! And happy hacking!

Friendly heads up that you have until Sunday (October 6) to push your final PR @evgongora


Game crashing - memory dump tool

An unknown bug is causing the game to crash at random times.
No consistent behaviour eg. while shooting/while running/while idle has been observed.
Look into memory analysis tool to get a dump of what is happening at crash times

This issue does not appear on Linux OS
Seems to stem from a problem with the DLLs required to run LOVE2D on Windows

Closing as replacing the lua51.dll file from LOVE 11.4 with the 11.3 version seems to have fixed the issue. Will reopen if necessary


docs: updating links to sources

The link I removed is no longer valid: http://support.saucelabs.com/entries/25614798-How-can-we-set-up-an-open-source-account-that-runs-tests-on-people-s-pull-requests- . I couldn't recover this thread or find any relevant information on http://support.saucelabs.com. So I updated readme with links to travis docs that seems to be the most relevant.

@cvrebert any concerns about that ?


WorldFactory: allow creator to pass in salt

follow up to https://github.com/latticexyz/mud/pull/2210

could we allow the sender to pass the salt themselves? so final salt is msg.sender + their salt
reason is: we might want some world address for the same world on multiple chains, and it might be the case that worldCount is different on different chains

not sure i have time right now, but it's pretty simple: instead of worldCounts[msg.sender]++ being part of the final salt calculation, it's a value passed by the caller as a function arg.
reason is: worldCount is already a form of value controlled by the sender, but it's annoying to "set it" because the only way to change it is to deploy worlds through that contract.
CREATE2 already reverts if a contract currently exists at that address, so I think we should let people pass whatever salt they want; both preventing them from being frontran by including the msg.sender in the salt and letting them pass whatever extra value they want (enabling the same world address on multiple chains)

Should we always require a salt in deployWorld? or is it useful to have a code path for an incrementing count that computes its own salt?

It might be useful to add a function parameter of bytes 'salt' to the WorldFactory.deployWorld function to help facilitate consistent deployment of world addresses across different networks.
Something like this might be useful here:
function deployWorld(bytes memory userSalt) public returns (address worldAddress) {
   // Deploy a new World and increase the WorldCount
   bytes memory bytecode = type(World).creationCode;
   uint256 salt = uint256(keccak256(abi.encode(userSalt, msg.sender, worldCounts[msg.sender]++)));
   worldAddress = Create2.deploy(bytecode, salt);
   IBaseWorld world = IBaseWorld(worldAddress);

   // Initialize the World and transfer ownership to the caller
   world.initialize(coreModule);
   world.transferOwnership(ROOT_NAMESPACE_ID, msg.sender);

   emit WorldDeployed(worldAddress);
 }

if there is a contract already deployed at that address, create2 would revert.

lmk if I should open a PR w this

lmk if I should open a PR w this

After some internal chat, we're thinking of nixing the notion of a world counter and just take in a bytes salt with each deploy call. Feel free to PR it!


Endpoint Protection

@jittata commented on Mon Jun 18 2018
This document titled Azure Security Center and Azure Virtual Machine on Linux is where it makes sense to use Security Center.
・ Endpoint protection recommendations
Although it is described, when confirmed with Microsoft support unit
We received an answer that endpoint protection is only supported by Windows and not supported by Linux.
From our public information, you can confirm that "Endpoint Protection" is valid only for Windows VM.
Integrate security solutions with Azure Security Center
https://docs.microsoft.com/en-us/azure/security-center/security-center-partner-integration#integrated-azure-security-solutions
* Please see “Integrated Azure Security Solutions”.

According to the contents of this document, it is misleading as if Linux supports endpoint protection from the customer's point of view.
Thank you.

ドキュメントの詳細
⚠ このセクションを編集しないでください。 docs.microsoft.com で必須です ➟ GitHub の問題のリンク。

ID: 8872d787-1114-3fb0-8fd8-71bd29dea646
Version Independent ID: 8c788217-4f35-1a29-7c89-d965d6268b1d
Content: Linux での Azure Security Center と Azure 仮想マシン
Content Source: articles/security-center/security-center-linux-virtual-machine.md
Service: security-center
GitHub Login: @terrylan
Microsoft Alias: yurid

@v-maudel commented on Mon Jun 18 2018
こんにちは、@jittata ,
フィードバックをお送りいただきありがとうございました。
コメントを確認中で、改めてご連絡させていただきます。
敬具
Microsoft DOCS International Team

Hi team,
I'm moving this from the ja-JP repo since the contributor reported some accuracy issues in the document.
Kind regards
Microsoft DOCS International Team

@saurabhsensharma - Please reassign to the author of this article. The article metadata shows the author to be memildin. Thank you.

@saurabhsensharma - Also, how does the author get a translation of the issue? I believe that will be needed before the author can respond.

The comment was machine translated

Hi @saurabhsensharma and @CeciAc,
The specific page on which this comment was added is now out-of-date and about to be removed.
On this page, you can find a table listing the platform support for all Azure Security Center features:
https://docs.microsoft.com/azure/security-center/security-center-services
Beneath the main table, there is a detailed list of the endpoint protection solutions supported by Security Center. In that list, are the products Security Center can discover on Linux. If an endpoint protection solution from that list is discovered, Security Center won't recommend installing one.
https://docs.microsoft.com/azure/security-center/security-center-services#supported-endpoint-protection-solutions-
Those tables have undergone quite a few updates since this ticket was opened. If there's still information needed, please reopen this ticket or be in touch again.
#please-close


Create an RFP for various implementations benchmarking

Grant Application
This application is:

[ ] speculative (use this by default).
[x] an RFP response.

This application is:

[x] fully public.
[ ] public with private finances.

Abstract
Compare how various implementations of on-chain logic perform:

runtime modules
ink! contracts
solidity (and solidity-compilied-to-WASM) contracts

Checklist

[x] The README has been read and understood.
[x] The abstract above is succinct and complete.

Looks good and it doesn’t hurt to work on performance benchmarks. But I have two concerns: Does it make sense to already start benchmarking these implementations (e.g. evm)? And it might potentially lead to people focusing only on benchmarks, when in reality there is a lot more to it.


error: Loading sysroot: Failed to invoke RegisterClient:... when running rpm-ostree status after initial installation

What happened?
Install from universalblue-38-x86_64-20230418.iso
After EFI error (another issue filed), installer reboots.
Skip tour.
Run a terminal.
Welcome to Fedora Linux 38.20230421.0 (Silverblue).

This terminal is running on the host system. You may want to try
out the Toolbox for a directly mutable environment that allows
package installation with DNF.

For more information, see the documentation.

[john@fedora ~]$ rpm-ostree status
error: Loading sysroot: Failed to invoke RegisterClient: GDBus.Error:org.freedesktop.DBus.Error.NameHasNoOwner: Could not activate remote peer: startup job failed.
[john@fedora ~]$

[john@fedora ~]$ systemctl status rpm-ostreed
× rpm-ostreed.service - rpm-ostree System Management Daemon
     Loaded: loaded (/usr/lib/systemd/system/rpm-ostreed.service; static)
    Drop-In: /usr/lib/systemd/system/service.d
             └─10-timeout-abort.conf
     Active: failed (Result: exit-code) since Fri 2023-04-21 21:18:40 EDT; 18s ago
       Docs: man:rpm-ostree(1)
    Process: 3066 ExecStart=rpm-ostree start-daemon (code=exited, status=217/USER)
   Main PID: 3066 (code=exited, status=217/USER)
        CPU: 0

Apr 21 21:18:40 fedora systemd[1]: Starting rpm-ostreed.service - rpm-ostree System Management Daemon...
Apr 21 21:18:40 fedora systemd[1]: rpm-ostreed.service: Main process exited, code=exited, status=217/USER
Apr 21 21:18:40 fedora systemd[1]: rpm-ostreed.service: Failed with result 'exit-code'.
Apr 21 21:18:40 fedora systemd[1]: Failed to start rpm-ostreed.service - rpm-ostree System Management Daemon.

Apr 21 21:07:45 fedora systemd[1]: Starting rpm-ostreed.service - rpm-ostree System Management Daemon...
Apr 21 21:07:45 fedora systemd[1]: rpm-ostreed.service: Main process exited, code=exited, status=217/USER
Apr 21 21:07:45 fedora systemd[1]: rpm-ostreed.service: Failed with result 'exit-code'.
Apr 21 21:07:45 fedora systemd[1]: Failed to start rpm-ostreed.service - rpm-ostree System Management Daemon.
Apr 21 21:18:40 fedora (m-ostree)[3066]: rpm-ostreed.service: Failed to update dynamic user credentials: Permi>
Apr 21 21:18:40 fedora (m-ostree)[3066]: rpm-ostreed.service: Failed at step USER spawning rpm-ostree: Permiss>
Apr 21 21:18:40 fedora systemd[1]: Starting rpm-ostreed.service - rpm-ostree System Management Daemon...
Apr 21 21:18:40 fedora systemd[1]: rpm-ostreed.service: Main process exited, code=exited, status=217/USER

Relevant log output
Please attach or paste the logs from the Anaconda Installer https://docs.fedoraproject.org/en-US/quick-docs/anaconda/anaconda_logging/

From josh in chat, looks like it's related to this: https://fedoraproject.org/wiki/Changes/Shorter_Shutdown_Timer
Will continue to investigate, thanks for reporting!

Hi there,
This issue was filed on old ISOs. We have a new ISO generation tool, check here for download instructions:
https://universal-blue.org/installation/
The new ISOs use a more reliable method of creation and should work much better, thanks!

The issue is back after Kinoite upgraded to 40.
 rpm-ostree update
error: Loading sysroot: Failed to invoke RegisterClient: GDBus.Error:org.freedesktop.DBus.Error.NameHasNoOwner: Could not activate remote peer 'org.projectatomic.rpmostree1': unit failed

 systemctl status rpm-ostreed
○ rpm-ostreed.service - rpm-ostree System Management Daemon
     Loaded: loaded (/usr/lib/systemd/system/rpm-ostreed.service; static)
    Drop-In: /usr/lib/systemd/system/service.d
             └─10-timeout-abort.conf
     Active: inactive (dead) since Wed 2024-04-24 02:42:41 PDT; 18h ago
   Duration: 1min 41.713s
  Condition: start condition unmet at Wed 2024-04-24 20:54:38 PDT; 1min 6s ago
             └─ ConditionPathExists=/ostree was not met
       Docs: man:rpm-ostree(1)
   Main PID: 10611 (code=exited, status=0/SUCCESS)
     Status: "Exiting due to idle"
        CPU: 17.211s

systemctl start rpm-ostreed has no effect


client helpers use client properties

changed the _client.turn_off() and _client.turn_on() helpers to use the c.hidden property instead of the tagging/untagging dance in accordance to a comment in #140
so far this seems to work or the swallowing module and i haven't found any particular new corner case
testing for scratchpads and the tabbed module need to be done.

Works surprisingly fine for me. Sadly one exception I found are scratchpads where animations break in some cases (some incomming animations I think). The module probably checks for whether the scrachpad is already on the tag, which is true for this PR but false for the current code

hmmmm how about creating 2 sets of helpers, one just for the scratchpads and the other for everything else?
or perhaps have another argument for the helper that is only passed on the scratchpads module?

soon™


[Doc] add MULTIPLY function doc (#15457)

What type of PR is this：

[ ] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #15457
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto backported to target branch

[ ] 2.5
[ ] 2.4
[ ] 2.3
[ ] 2.2

cherry-picked


Todos

[x] Minimize docker production image size (use webpack for production)
[ ] koa-jwt
[x] babel-plugin-closure-elimination

I think you should add nginx to this project


not translated status . Awarenes type 12

Describe the bug
Today status shown that was not translated
Steps to reproduce
language: en-GB
category: Met
event: awareness_type=12,awareness_level=2.0
responseType: Monitor
urgency: Future
severity: Moderate
certainty: Likely
effective: 2023-01-11T12:00:00+00:00
onset: 2023-01-13T08:00:00+00:00
expires: 2023-01-13T22:59:59+00:00
senderName: Rijkswaterstaat
headline: Flooding - Warnings for Noord-Brabant - Nederland
description: Increased discharge of the river Meuse due to precipitation in the  Ardennes region in Belgium and in northern France.
instruction: More information: https://www.rijkswaterstaat.nl/water/waterdata-en-waterberichtgeving/waterbericht/index.aspx
web: http://meteoalarm.eu/en-GB/0/0/NL013-Noord-Brabant.html
contact<EMAIL_ADDRESS>awareness_type: 12; flooding
awareness_level: 2; yellow; Moderate
attribution: Information provided by MeteoAlarm
device_class: safety
friendly_name: meteoalarm
Screenshots

Card version
2.4.1
Integration type
Meteoalarm
Installation type
Home Assistant Community Store (HACS)
Card Configuration
binary_sensor:

platform: meteoalarm
country: "netherlands"
province: "Noord-Brabant"

Integration Configuration
binary_sensor:

platform: meteoalarm
country: "netherlands"
province: "Noord-Brabant"

awareness_type=12,awareness_level=2.0 is headline provided by your local meteo alerts agency. There is not much i can do with that. The only fix I can suggest is turning on override_headline


Lithos vs Vagga

Good day, @tailhook !
Tell me, please: how could Lithos be compared with Vagga?
Thanks!

Lithos is a production-grade supervisor for containers. Its task is to run containers and restart them if needed. That's basically it.
Vagga is a development tool. It supports building and versioning containers. As well much richer way to configure and run commands with their dependencies. But is not suited for long-running processes.
The rule of the thumb: lithos is used on servers and vagga is used on workstations and CI.

@tailhook , thanks! But that's I saw in descriptions by myself :smile:
But technically? Ok, another question: what are there common features? They both can run containers?
Explain: based on that description's ideas, I thought that second have to be an extension of first, or vice versa. But it's not...

Yes. They both run containers. They share some libraries. What's different:

Lithos runs as root by design (because this is the only way to support some features), vagga runs as user because this is the only way to run arbitrary downloaded project securely
Configuration file formats are very different: lithos expects config put into the container with all the environment and parameters set. While vagga reads config first and then starts to build container, and allows to run anything in container (limiting folders which you might use)

I.e. this contrast is enough to make them different products rather than superset of one another. Also lithos is quite small: 5k LoC, mostly parsing configuration and converting it into system calls (but also convenience tools like lithos_clean and lithos_ps). While vagga is bigger because it has to provide all the convenience wrappers around package managers and other build tools.
Does this sound more clear?

Yes, thanks!


LGPL

is better than GPL but still is not suitable for Apache projects or many other entities.
A more liberal license would be nice.

I'm willing to switch it to an Apache license as I have considered doing before.  I looked into contributing it as an Apache project altogether, although it falls short of the active community requirement desired for Apache projects.
Anyways, I did appreciate the requirement to contribute library changes back when I originally created the library.  It's good to have folks contribute back, unless there is a good reason not to do so.  I've granted individual permission to use it under a different license due to some of the restrictions in their environments, but I'll just change the license for the project over now.
Thanks for the feedback.  Hope you contribute if you have any good suggestions/modifications.

License change completed.  I'll up the minor rev if needed, however I'm considering everything 0.1.2+ to now fall under the Apache license.


 In the meantime you can build the container locally with
docker build -t computer-use . && docker run \
    -e ANTHROPIC_API_KEY=$ANTHROPIC_API_KEY \
    -v $HOME/.anthropic:/home/computeruse/.anthropic \
    -p 5900:5900 \
    -p 8501:8501 \
    -p 6080:6080 \
    -p 8080:8080 \
    -it computer-use

built nicely and executed without error, I think:

not sure what is necessary to make VNC connect properly, though.

is there a way to simply use the Bash Tool included in this update without the rest of the desktop usage features? It would be much easier to integrate into existing dev workflows that way IMO

Does this persist after refreshing the browser or restarting the container? Can you access http://localhost:6080/vnc.html or vnc://localhost:5900

restarted browser, removed/reran container.
$ cat r.sh
docker run \
  --platform linux/amd64 \
  -e ANTHROPIC_API_KEY=$ANTHROPIC_API_KEY \
  -v $HOME/microservices/anthropic-quickstarts/.anthropic:/home/computeruse/.anthropic \
  -p 5900:5900 \
  -p 8501:8501 \
  -p 6080:6080 \
  -p 8080:8080 \
  -it cud
$ ./r.sh
Xvfb started successfully on display :1
Xvfb PID: 9
starting tint2 on display :1 ...
starting mutter
starting vnc
PORT=5900
starting noVNC
noVNC started successfully
✨ Computer Use Demo is ready!
➡  Open http://localhost:8080 in your browser to begin

495a5c14f0a3   cud                                  "./entrypoint.sh"        7 seconds ago   Up 7 seconds   <IP_ADDRESS>:5900->5900/tcp, :::5900->5900/tcp, <IP_ADDRESS>:6080->6080/tcp, :::6080->6080/tcp, <IP_ADDRESS>:8080->8080/tcp, :::8080->8080/tcp, <IP_ADDRESS>:8501->8501/tcp, :::8501->8501/tcp   zen_nightingale

chrome shows a pending Connecting... when i directly access 6080 but is still stuck I believe:

no change in firefox behavior

note: i have seen other acquaintances get their container running, so I assume this must be a PEBCAK issue on my part somewhere


Custom web parts don't always appear in editor on page load

Category

[ ] Question
[ ] Typo
[X] Bug
[ ] Additional article idea

Expected or Desired Behavior
When opening and editing a page with a full-width column, all the custom web parts that support full bleed should appear in the list.
Observed Behavior
In some instances, only the first one or two web parts are showing in the list on the full-width column. Removing the column and then re-adding it will sometimes show all the web parts.
Steps to Reproduce
Create a new page and add a full width column, save and refresh the page. You may only see one or two custom full-width web parts, although this is somewhat intermittent.

As this is out-of-the-box functionality, we can't route this through the dev process. Please submit this through the SharePoint Online admin support channels from the tenant admin tool. We can only support dev topics through this issue list. Sorry for the inconvenience.


handler.go:63:30: undefined: transport.StreamFromContext

$ go get -u github.com/improbable-eng/grpc-web/go/grpcwebproxy
# github.com/mwitkow/grpc-proxy/proxy
go/src/github.com/mwitkow/grpc-proxy/proxy/handler.go:63:30: undefined: transport.StreamFromContext

It looks like grpc-go upstream removed the StreamFromContext method in the transport package with this commit: https://github.com/grpc/grpc-go/commit/57640c0e6fd5e562b3a53f6b72738759165c80e1

PR https://github.com/mwitkow/grpc-proxy/pull/23 should address this issue


Add l1 alias as single column variant of ls

As title says it would be nice to have a shorthand out-of-the-box for the ls -1 command.
Nothing fancy but makes piping through grep/other commands easier and quicker.
cc @mcornella

When ls is piped it always uses single-column form. See for example ls | cat.


Added Laplace smoothing parameter tuning

Tuned laplace smoothing parameter to the Naive Bayes classifier in module 7.
I experimented with 3 different values 0.01, 1 (default) and 2 and have noted that both the parameters 0.01 and 2 improve upon the default value of 1. However there is much more significant improvement when the parameter value is closer to 0. This suggests that not a lot of smoothing is required. However, would be happy to discuss any other inferences we can draw.

I got a chance to look at your work. It is a nice addition.
I had to manually copy n' paste your cells into the master branch to avoid a messy merge conflict.
Your work is now on the master branch.
I'm closing this PR without merging it.
Please keep contributing.

Thank you, I am glad to be able to contribute, will try to avoid messy merge conflicts the next time. Sure, would be happy to keep contributing!


Added a transition to the buttons Aasanas on Yoga Therapy Page

Issue #349
I've given the transition to the buttons on hover on Aasanas Section.

@MohitNamdev22 have you pulled the recent changes?
Seems like you haven't. The site has gone some changes after merging some PRs.
__
  Summary: :question: I have a question

@Susmita-Dey I had pulled the code yesterday before starting working on this issue

@MohitNamdev22 Your changes will override the PR changes that was merged recently. Kindly try to fetch and pull. I'm saying this because laughing therapy has no coming soon page now as in the original site which is not reflected in your PR.

Okay no issues, I'll pull it again today, then merge it 👍

@Susmita-Dey Pulled & merged the latest code to my repo

@MohitNamdev22 Can you pull the recent changes once more?
After you're done kindly tag me here and also on discord to merge your PR.
__
  Summary: :hammer_and_wrench: This code needs a fix

Okay @Susmita-Dey 👍

@Susmita-Dey I've pulled the latest code


MOVE errors when autoPipeline=true and all commands are custom

A lot of the calls that I perform to redis are to the JSON redis module, and so I need to use the custom redis.call() function. e.g. client.call('JSON.SET', 'some_key', '$', ...).
However, I noticed that I got the following error rather sporadically:
ReplyError: MOVED 6734 <IP_ADDRESS>:31002
    at new Promise (<anonymous>)
    at executeWithAutoPipelining(...)
    ...

I ended up figuring out why this happens:
It seems like when autoPipeline=true and ALL commands sent in the pipeline are custom commands (i.e. only commands sent via client.call(...), the commands are sent to a random node instead.
This is because when the pipelineSlot is calculated based on the keys, it has no keys to reference, as everything is considered as args. This results in a random pipeline slot being chosen. Which is often wrong, so it causes the pipeline to fail.
If there is a single native command though (for example redis.set()), then it is able to determine which node to send it to correctly, and everything works fine. Also, if i set autoPipeline=false, then I get zero move errors. These two observations were made as I was debugging the library code.
It seems like it would need a better way to determine which node to send it to. Either allowing to specify the number of keys with client.call() or a some sort of heuristic that inspects the first arg and assumes that is the key (like it is done elsewhere in the code).

I had faced this similar issue, and ended up putting all the JSON commands in autoPipeLineIgnoredCommands options.


Added another example for g:startify_custom_header

This example shows a random quote and cow, centered and using a wider speech balloon. Adapted to also work with the newer cowsay_apj project (https://github.com/cowsay-org/cowsay).
Also changed temporary g:script_variable into s:script_variable
Nice plugin, BTW!

Hi,
I appreciate the PR, but have to decline it.
Apart from the few things I reviewed, I don't think it's an improvement over example #2. It merely adds another invocation of system() using a cowsay fork. I'm super reluctant in adding more examples when they don't show off something new.
Thanks anyway!

Yes, sorry for the pull request, I agree. I started configuring NeoVim in a hypomanic state and began to enjoy lots of plugins like yours. Think less, do more. Got embarrassed later, but decided to just let it go.


Track using violations

There's no way we can visit every identifier. Instead, figure out which
constant violations hamper the minification.
Oh, and some test cases. Those should probably be cleaned up. :grin:

Also, the jQuery benchmark runs some ~40ms faster, react ~30ms.

This looks like a better approach, if we can resolve the edge cases in my comments (above). @kangax @boopathi thoughts?

@jridgewell I'll merge this once the multi-layer referencing is ready.

Nice!


Missing icons

Hi. Not all icons are present here. Please add these if possible
fa-list-alt
fa-snowflake-o
fa-area-chart
fa-commenting

Until my pull request is merged you can find the full 924 list at https://github.com/thenslerse/FontAwesome-iOS


Magic Item Inventory Management

User story
As a player I should be able to manage which magic items my characters are currently holding so that I can have a finer control over which rewards I take from completing adventures.

Narrative
As a player, I should be able to add, edit, and remove magic items from my character.

Acceptance criteria

#
Given
When
Then

1
I am a player
I navigate to my characters magic items list
I can see a list of the magic items that my character has

2
I am a player
I add a magic item to my character
The item is added and the list updates to reflect the changes

3
I am a player
I delete a magic item from my character
The item is deleted and the list updates to reflect the changes

4
I am a player
I edit a magic item from my character
The item is updated and the list updates to reflect the changes

Hey team! Please add your planning poker estimate with ZenHub @IanjhPhillips @luigibesani @willyyhuang


RequeueRejected parameter explained wrongly in README

Fixes gh-300

@chenghaoharvey Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@chenghaoharvey Thank you for signing the Contributor License Agreement!

Any response or what should i adjust for this PR?

Thanks for the contribution.


Investigate returning intervals from neural net

Read about implementing a NN to learn bayesian distributions (https://pyro.ai/examples/bayesian_regression.html) but this seems to require quite some work, so leaving it outside of the project scope.


fix #1526: multipart messages should consume their epilogue

Currently a multipart message epilogue is left unread which when nested
in another multipart message causes a ValueError to be raised because
the parent reader expects to find its multipart boundary not the
sub-message's epilogue.
fixes issue #1526
What do these changes do?
Allows reading multipart sub-messages that were created with aiohttp.multipart.MultipartWriter
Are there changes in behavior for the user?
ValueError for an incorrect boundary is not thrown anymore.
Related issue number
#1526
Checklist

[x] I think the code is well written
[x] Unit tests for the changes exist
[x] Documentation reflects the changes
[ ] Add yourself to CONTRIBUTORS.txt

The format is <Name> <Surname>.
Please keep alphabetical order, the file is sorted by names.

[ ] Add a new entry to CHANGES.rst

Choose any open position to avoid merge conflicts with other PRs.
Add a link to the issue you are fixing (if any) using #issue_number format at the end of changelog message. Use Pull Request number if there are no issues for PR or PR covers the issue only partially.

Current coverage is 98.85% (diff: 100%)

Merging #1532 into 1.2 will increase coverage by <.01%

@@                1.2      #1532   diff @@
==========================================
  Files            30         30
  Lines          6984       6985     +1
  Methods           0          0
  Messages          0          0
  Branches       1163       1163
==========================================
+ Hits           6904       6905     +1
  Misses           40         40
  Partials         40         40

Powered by Codecov. Last update 63d1ad5...2e2f06b

@kxepal ^^^

@terencehonles
The biggest problem of multipart examples is that you cannot be sure where newlines were added for formatting and where they are mandatory (and where they are CRLF, not just CR).
No, I don't have any good examples right-by-the-hand, need to search over and check the others implementations - how they do handle such cases.
Personally, I just feel bad about having additional blind readline because there you may eventually read not what you expect to read. Say, what if there will be no epilogue at all and the "blank" means nothing, not a "blank line" as you expect?
So I would prefer to fix the writter at the first place as there we won't loose or break the data that we do not own. And only if this doesn't helps, try to fix the reader with extra caution. So far it's possible to do without any notable breaks.
Will return to this issue a bit later after few more investigations.

@kxepal well I'm not advocating keeping it blind. I wanted to start the discussion, but I think we should probably be handling the case when there is an epilogue. So by just fixing the writer we are sweeping the problem under the rug. I'm fine with doing both, but I wasn't sure if the epilogue really should be omitted rather than blank (I'm still not convinced)

@terencehonles
Yea, I got your point. And you're absolute right. There are actually two fixes: let writer not emit an epilogue (no any good reason to have it) and let reader handle epilogue right (by ignoring as everybody does) (and may be preambula as well with the same effect if we're going to fix these holes). Each can solve your problem, but both of them is good to have.
I started from a writer side since it's simpler to fix. But this certainly doesn't makes epilogue problem go away, there just need a little bit more dirty work to do.

@terencehonles @kxepal what is status of this ticket?

@fafhrd91 I don't have enough previous multipart experience to know what should be allowed and how strict the code should be. I'm not sure if @kxepal had any time to look into the issue any further as we were spending time on #1530, but now that it is closed I'd like to spend more time on this PR.
Not that it really matters, but I've kept my work unblocked by using this branch for my personal development, but I will need a proper solution by the time I push my work to production.

as i understood #1530 was ready and it is merged now.
@kxepal i need your assistance

That rogue readline really annoys me, but I don't have a time to propose anything better right now and it seems that it really solves the issue and it seems like it doesn't breaks anything. So let's merge it and leave room open for further improvements. If it'll break something, we would have a way to look into.
+1

@terencehonles could you rebase your branch against master and then re-create PR against master

@fafhrd91 will do.
@kxepal I'll look at the code path this week and see if I can propose something nicer (I'm also not very much a fan of the blind readline with the limited documentation I've read, but I needed to propose something)

@terencehonles i merged pr, if you want to make more improvements let's move it to separate pr.
thanks!

@fafhrd91 Ok, sorry I got a flu for the past couple days and didn't have a chance to update this.
@kxepal I looked and the code and there are two things we could do. I decided to just implement something and you can check out #1581


Enhance tags field type

We wanted to use the tags field type to share types between different collections and to have autocomplete suggestions from both collections.
To facilitate this, I added the following to field-tags in this PR:

Loading autocomplete suggestions like select field type (for example to suggest tags used in other entries in same collection)

{
  "src": {
    "url": "collection=collection_1",
    "value": "tags"
  }
}

Loading multiple sources (to share tags between collections)

{
  "src": [{
    "url": "collection=collection_1",
    "value": "tags"
  }, {
    "url": "collection=collection_2",
    "value": "tags"
  }]
}

Setting minLength option to 0, ie. show autocompletions when focusing the input
Setting autocomplete delay from option (used to filter options directly when using static list or from sources)
Don't suggest tag already in use

This PR should be backwards compatible with specifying a URL in the autocomplete option if no sources are used.

Thanks!


Check if its useful to have nocurrent_pass and account_field_split

Problem:
We want to allow the user to change its email address. To do that, its necessary to fill in the current password but when using simplesamlphp_auth there's no such password.
We need a solution to allow users to change their email without having to fill in their current password.
Solution:
We can use nocurrent_pass module to set current password as an optional field.
Additionally, it would be nice to add account_field_split in order to remove the password field from "Account form display".
Implementation:
Add account_field_split and nocurrent_pass to Openfed and enable them by default.
Go to /admin/config/people/accounts/form-display and remove the "Password" and "Current password" from the enabled list.
There won't be an update hook for old projects so in order to add this to old projects it must be done manually.

Ruben, this is only needed for sites that will login using eID, which is likely to be a small minority of Openfed sites. So "enable them by default" seems like the wrong approach.
Also, what about "mixed" cases, where people can use either eid or username/password (or where some roles use username/password, others eid) - not something we need to solve right now but this approach doesn't fully handle those.

Indeed, enabling it by default is too much...


Bundler could not find compatible versions for gem "rails"

Hello,
I am new to Rails and Ruby in general, so I am probably doing something wrong.
I tried installing sprangular using the recommended method of sprangular install <app_name>
After install, I get the following notice:
The git source git://github.com/spree/spree_gateway.git is not yet checked out. Please run `bundle install` before trying to start your application
when doing bundle install i get the following warning:
Warning: this Gemfile contains multiple primary sources. Using `source` more than once without a block is a security risk, and may result in installing unexpected gems. To resolve this warning, use a block to indicate which gems should come from the secondary source. To upgrade this warning to an error, run `bundle config disable_multisource true`.
It starts to install and then
Resolving dependencies......
Bundler could not find compatible versions for gem "rails":
  In Gemfile:
    spree_core (= 2.4) ruby depends on
      rails (~> 4.1.8) ruby

    rails (= 4.2.3) ruby

From what i understand, the problem comes from  the fact that the Gemfile indicates this # Bundle edge Rails instead: gem 'rails', github: 'rails/rails'  gem 'rails', '4.2.3' but spree requires another version ?
ruby version: ruby 2.2.1p85
rails version: 4.2.3
Thanks !

You're gem file probably has the rails version specified explicitly
gem 'rails', '4.2.3'

Just remove the explicit version and then bundler will install the rails version required by spree
gem 'rails'


Allow client to specify Bing Maps API Key in  directive

Hi,
When starting up using angular-cesium, it can become confusing to the client (me) when an empty Cesium Widget shows up:

The issue was caused by using the default Bing Maps API key. No globe is rendered. There is no way to specify the Bing Maps API key in the <ac-map> directive, and there is no hint in the README instructions to tell us to change the key by doing Cesium.BingMapsApi.defaultKey = 'xxxxxxxxxxxx'; somewhere in our controller.
It would be nice to see this as an option for the <ac-map> directive, or at the very least to see a helpful hint in the README.

Maybe adding this as an attribute to the map directive?


How to get All Data when Paginator is set to true.

By default, When paginator is set to true, the response of get methods comes back as an object containing data and meta tags. The data is paginated by default.
Okay what if I want whole data only and no pagination info with it.
I think it should not override the response format unless we pass "?page=1" in url.
For eg. Let an api for Brands.

when I pass url /brands : It should return whole data as paginator=false returns.
when I pass url /brands?page=1 : only then data must be in the format as paginator=true returns

@amandeep33 I just published a beta of loopback-paginator. Please run npm i loopback-paginator@beta and verify if everything works fine. I am currently not able to test it myself, as I am pretty busy.

@mahnuh I tested this. Now response is fine in both cases. But in first case when no page parameter is passed, data is still paginated i.e, 10 records only.

@amandeep33 please try v2.0.0-beta.1

@mahnuh I am still getting same results i.e, 10 records only.

@amandeep33 I just tested it and it works fine. I published v2.0.0 to npm. Please make sure you have the correct version installed (npm i loopback-paginator@2.0.0).

@mahnuh It's working fine now. Thanks for such quick enhancement.


Huebot Call ends after 11 seconds

I have the following configuration:
ServieDnsName : huebotstk.cloudapp.net
ServiceCNAME : itcanoe.com
the certificate is from itcanoe.com
The bot makes the call but it doesnt do recognition and it closes

Could you try if the messages to the bot works?
What is the last entry of SAN in the certificate? Is it self issued or issued by a trusted CA? For speech to work, did you get a speech subscription ( a free trial here -> https://azure.microsoft.com/en-us/pricing/details/cognitive-services/) and use this key in the .cscfg.


New platforms

Hi, just a simple question.
Are you planning or is it possible to add Didi and Cabify to this repo?

Hi! Good question. Yes, there are people opening PRs for support for new platforms all the time, and I happily accept those.
I don't have time to build those integrations myself, but there is a guide on how to do it here: https://github.com/tschoffelen/react-native-map-link/blob/master/docs/add-app.md


[v1.1] Refine DENAS Distributed Training Settings in NLP BERT

Please help review this PR. Thx. @xuechendi


Settings detail right side panel positioned poorly

Opens at the height of the setting clicked. Should be always fixed at the top on the right, unaffected by scrolling.
It was like that on one version before the pop-up returned

The issue is that the panel often is bigger than the window, so you'd have two scrollbars right next to each other, which could be very confusing.
That's why I made them open on the option level, because I think it's much more intuitive, but I can bring back the option to make it fixed.


Vulnerability issue in debug package

Vulnerability is in ms package used by debug. Please refer to https://snyk.io/vuln/npm:ms:20151024.
debug package v2.6.7 has addressed this issue.

Bumped the version :point_up: thanks for reporting

@vkarpov15 hi, have you release a new version for this security issue? I see the version 2.3.0 is released in March.

Released 2.3.1 and mongoose 4.10.2 to address this :+1:


replace all API with mutiny reactive

This looks good! Moving the logic from Resource to Service class is a good practice.
I would suggest one change to the create(). I don't know if the controller.renderArchive is async (off the work to backend thread). If not, I'd suggest doing so. You may use the Quarkus EventBus to pass the work to backend thread.

Nope, it is not, I'll take a look on that later, thx

I don't think it's necessary to do this refactor! The original resource logic is simple enough, all are controlled in the controller. No need to extract, at least, by now. If you want to do something about Uni, just doing that is enough.

Wha.... but why don’t we follow the design patterns quarkus officially follows. Why do we put logics into API announcing codes anyway.

I'm not gonna make this patch anymore, do what you would like to, and good luck


fix: some adapters don't support visibility

Dear @dkarlovi
these changes go haywire with the unit tests.
Could it be you should provide a default mode instead of 0?
What happens with adapters where there should be an error when there is no visibility?
Thanks for your contribution!

@m2mtech for example, Flysystem's Azure Storage adapter just throws an exception.

@dkarlovi Sorry I did not express myself clearly enough.
If you have a look at LocalFlysystemAdapter, it throws an exception when there is a problem getting the visibility. How to you differentiate that there is an exception because of a problem from that the adapter throws the exception because it does not implement a functionality?
Personally I think that when Azure does not support visibility, everything on Azure - concerning flysystem - is public and the adapter return PUBLIC, instead of throwing an exception. But that is philosophy we can argue about...
So the question is should we ignore all errors concerning visibility, send a PR to the Azure adapter to return PUBLIC instead of the exception?

@m2mtech I think it's more complex than that. Azure will in some cases support visibility, but by default it will not. But in that case private visibility is actually a better choice here maybe?

Dear @dkarlovi
with the latest commit you can optionally allow to ignore visibility errors, e.g.:
FlysystemStreamWrapper::register('fly', $filesystem, [
    FlysystemStreamWrapper::IGNORE_VISIBILITY_ERROS => true,
]);

Could you please check if this solves your issue?


cherrypick add simdjson build framework

cherrypick add simdjson build framework (#4598) from https://github.com/facebookincubator/velox
Summary:
For issue https://github.com/facebookincubator/velox/issues/3658. The previous work https://github.com/facebookincubator/velox/pull/4053 shows simdjson can get great performance gain in json function. The design doc is at 4486 Add the simdjson library dependency and build system first.
Pull Request resolved: https://github.com/facebookincubator/velox/pull/4598
Reviewed By: Yuhta
Differential Revision: D45778530
Pulled By: kgpai
fbshipit-source-id: 815b8081ccb16e0f2a7c0c64e520de671e19543a

We are doing rebase recently and will include this upstream commit in the rebased code. It may require some days. Thanks!

We are doing rebase recently and will include this upstream commit in the rebased code. It may require some extra days. Thanks!

Got it.


super slow

I was running this example, and the program only used 1Mbps bandwidth and it took hours to finish import 12k out of 300k items(there is only 3 key value pair in each), and then got a timeout response, do i setup something wrong?

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

ID: 21b7030e-5d76-31d8-9e62-e2675aed96c9
Version Independent ID: f04c4507-f9d4-7c12-d147-bcf7a428cef9
Content: Bulk import data to Azure Cosmos DB SQL API account by using the .Net SDK
Content Source: articles/cosmos-db/tutorial-sql-api-dotnet-bulk-import.md
Service: cosmos-db
GitHub Login: @ealsur
Microsoft Alias: maquaran

@xmseraph Thanks for the feedback We are checking this.

@xmseraph The sample application tries to push a high volume of data to an account endpoint on a 50K RU/s container.
Due to the volume of data (300K items) the CPU consumption might be high in a local dev machine, if you are running the sample in a command prompt (for example with dotnet run) or in Visual Studio (it would open a Command Line), you can see that if there are failures on some items, it would show a log line with the response's status code.
If the responses show RequestTimeout/ServiceUnavailable or if your environment is experiencing high CPU, you need to adjust the load of data for what the environment can handle. See our Troubleshooting guide: https://docs.microsoft.com/en-us/azure/cosmos-db/troubleshoot-dot-net-sdk#request-timeouts
If you are customizing the data load (maybe using a smaller provisioned container) make also sure that the provisioned RU/s match the volume of data or the requests will get throttled.

@xmseraph Please let us know if you need any further help or we can close this issue by EOD.


LLMRouterChain uses deprecated predict_and_parse method

System Info
langchain v0.0.216, Python 3.11.3 on WSL2
Who can help?
@hwchase17
Information

[X] The official example notebooks/scripts
[ ] My own modified scripts

Related Components

[ ] LLMs/Chat Models
[ ] Embedding Models
[X] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[ ] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[X] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
Follow the first example at https://python.langchain.com/docs/modules/chains/foundational/router
Expected behavior
This line gets triggered:

The predict_and_parse method is deprecated, instead pass an output parser directly to LLMChain.

As suggested by the error, we can make the following code changes to pass the output parser directly to LLMChain by changing this line to this:
llm_chain = LLMChain(llm=llm, prompt=prompt, output_parser=prompt.output_parser)

And calling LLMChain.__call__ instead of LLMChain.predict_and_parse by changing these lines to this:
cast(
    Dict[str, Any],
    self.llm_chain(inputs, callbacks=callbacks),
)

Unfortunately, while this avoids the warning, it creates a new error:

ValueError: Missing some output keys: {'destination', 'next_inputs'}

because LLMChain currently assumes the existence of a single self.output_key and produces this as output:

{'text': {'destination': 'physics', 'next_inputs': {'input': 'What is black body radiation?'}}}

Even modifying that function to return the keys if the parsed output is a dict triggers the same error, but for the missing key of "text" instead. predict_and_parse avoids this fate by skipping output validation entirely.
It appears changes may have to be a bit more involved here if LLMRouterChain is to keep using LLMChain.

I am seeing the same issue on v0.0.221 , Python 3.10.6  Windows 10

I have the same issue on v0.0.229, Python v3.10.12

I have the same issue on v0.0.230, Python v3.10.6

same issue here

langchain v0.0.232
[/opt/homebrew/lib/python3.11/site-packages/langchain/chains/llm.py:275](https://file+.vscode-resource.vscode-cdn.net/opt/homebrew/lib/python3.11/site-packages/langchain/chains/llm.py:275): UserWarning: The predict_and_parse method is deprecated, instead pass an output parser directly to LLMChain.

langchain v0.0.235, Python 3.9.17 Windows 10

I am seeing the same issue on v0.0.257  Python 3.9.12  RedHat

same issue v0.0.270 python 3.11.3 windows

Same warning on v0.0.275 python 3.11.3 WSL on Windows 11

I also get the same error when using this option:
print(compressed_docs[0].page_content)
If I remove the page_content method, I do not get this error.  Also if I use this (CharacterTextSplitter), I do not see the error
text_splitter = CharacterTextSplitter.from_tiktoken_encoder(chunk_size=500)
Here is a sample function where this happens:
def demo(question):
    '''

    Follow the steps below to fill out this function:
    '''
    # PART ONE:

    loader = TextLoader('/map/to/document/data.txt', encoding='utf8')
    documents = loader.load()

    # PART TWO
    # Split the document into chunks (you choose how and what size)
    # text_splitter = CharacterTextSplitter.from_tiktoken_encoder(chunk_size=1000)

    text_splitter = RecursiveCharacterTextSplitter.from_tiktoken_encoder(
    chunk_size=1000, chunk_overlap=100, separators=[" ", ",", "\n"])
    docs = text_splitter.split_documents(documents)

    # PART THREE
    # EMBED THE Documents (now in chunks) to a persisted ChromaDB
    embedding_function = OpenAIEmbeddings()
    db = Chroma.from_documents(docs, embedding_function, persist_directory='./App')
    db.persist

    # PART FOUR
    # Use ChatOpenAI and ContextualCompressionRetriever to return the most
    # relevant part of the documents.
    llm = ChatOpenAI(temperature=0)
    compressor = LLMChainExtractor.from_llm(llm)
    compression_retreiver = ContextualCompressionRetriever(base_compressor=compressor,
                                                          base_retriever=db.as_retriever())
    compressed_docs=compression_retreiver.get_relevant_documents(question)

    print(compressed_docs[0].page_content)

y'all any fix to it? What are we supposed to do

Got the same warning while using load_qa_chain chain_type='map_rerank',

I posted a very similar issue #10462, using SelfQueryRetriever. I received some sort of solution from the chatbot. However, I don't have a clue how to implement it.

I posted a very similar issue #10462, using SelfQueryRetriever. I received some sort of solution from the chatbot. However, I don't have a clue how to implement it.

I solved it by extending SelfQueryRetriever and overwriting the _get_relevant_documents method. Below is an example (In this example, I overwrite _aget_relevant_documents, because I need async features in my case. You can do the same to _get_relevant_documents. ):
class AsyncSelfQueryRetriever(SelfQueryRetriever):
    async def _aget_relevant_documents(
        self, query: str, *, run_manager: AsyncCallbackManagerForRetrieverRun
    ) -> List[Document]:
        """Asynchronously get documents relevant to a query.
        Args:
            query: String to find relevant documents for
            run_manager: The callbacks handler to use
        Returns:
            List of relevant documents
        """
        inputs = self.llm_chain.prep_inputs({"query": query})

        structured_query = cast(
            StructuredQuery,
            # Instead of calling 'self.llm_chain.predict_and_parse' here,
            # I changed it to leveraging 'self.llm_chain.prompt.output_parser.parse'
            # and 'self.llm_chain.apredict'
            # ↓↓↓↓↓↓↓
            self.llm_chain.prompt.output_parser.parse(
                await self.llm_chain.apredict(
                    callbacks=run_manager.get_child(), **inputs
                )
            ),
        )
        if self.verbose:
            print(structured_query)
        new_query, new_kwargs = self.structured_query_translator.visit_structured_query(
            structured_query
        )
        if structured_query.limit is not None:
            new_kwargs["k"] = structured_query.limit

        if self.use_original_query:
            new_query = query

        search_kwargs = {**self.search_kwargs, **new_kwargs}
        docs = await self.vectorstore.asearch(
            new_query, self.search_type, **search_kwargs
        )
        return docs

I posted a very similar issue #10462, using SelfQueryRetriever. I received some sort of solution from the chatbot. However, I don't have a clue how to implement it.

I solved it by extending SelfQueryRetriever and overwriting the _get_relevant_documents method. Below is an example (In this example, I overwrite _aget_relevant_documents, because I need async features in my case. You can do the same to _get_relevant_documents. ):

Hi! Very appreciated with your sharing! Can you show the packages which are imported from?
Thanks a lot!

I posted a very similar issue #10462, using SelfQueryRetriever. I received some sort of solution from the chatbot. However, I don't have a clue how to implement it.

I solved it by extending SelfQueryRetriever and overwriting the _get_relevant_documents method. Below is an example (In this example, I overwrite _aget_relevant_documents, because I need async features in my case. You can do the same to _get_relevant_documents. ):

Hi! @ellisxu Very appreciated with your sharing! Can you show the packages which are imported from? Thanks a lot!

The package is from langchain.retrievers.self_query.base import SelfQueryRetriever and the langchain version I use is 0.0.279.

I am still running into this with SelfQueryRetriever using langchain==0.0.302....is there any resolution here?

This is unfortunate as this part of Lang Chain is used in the DeepLearningAI course


Remove GenericOperation

Don't need to go through the vm state machine.
/cc @laijs
Signed-off-by: Crazykev<EMAIL_ADDRESS>

@laijs This PR is ready now.


dirPagination: pagination not updating after filtering

The pagination controls won't update after put stuff in the search textbox, ie. if you search for Asad, the data did get filtered but the pages didn't get rearranged.
Expected result:
Pages rearranged
Actual result:
Pages were filtered individually
Demo:
http://plnkr.co/edit/aco9fM?p=preview
Thanks for your time!

See this FAQ answer

basically the paginate pipe needs to go after the filter.


fix(js): strings in FFI structs and JSON conversion in react native

Fixes XXXFromJson() functions that were losing the input string value due to the usage of vectors
Fixes createPresentation and verifyPresentation, that were losing strings that were passed to FFI structures, and also were not being freed after usage
Updates some methods like createRevocationStatusList and verifyPresentation according to last API changes

I thought @blu3beri had also dome some work on the revocation interval override. Is this the same or different?

Oh I haven't seen that (actually thought it was already merged, got confused with analogous PRs from askar and indy-vdr).
There are a few overlapping changes (not so many). What about merging this and updating the other PR?
BTW what's missing on the other one to get it ready for review?

BTW what's missing on the other one to get it ready for review?

Fully testing it in RN with AFJ :)

What about merging this and updating the other PR?

I'll update the PR


fix(config): use semantic-release

Codecov Report

Merging #2 (0f1dba6) into main (f98c864) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #2   +/-   ##
=======================================
  Coverage   95.53%   95.53%
=======================================
  Files           5        5
  Lines         179      179
  Branches       50       50
=======================================
  Hits          171      171
  Misses          8        8

Flag
Coverage Δ

unittests
95.53% <ø> (ø)

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

Continue to review full report at Codecov.

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


Issue/736 also enable batched partial compiles for termination states

Description
Core part of https://github.com/inmanta/lsm/issues/736
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[x] Attached issue to pull request
[x] Changelog entry
[x] Type annotations are present
[x] Code is clear and sufficiently documented
[x] No (preventable) type errors (check using make mypy or make mypy-diff)
[x] Sufficient test cases (reproduces the bug/tests the requested feature)
PR adding test in lsm module: https://github.com/inmanta/lsm/pull/752
- [ ] Correct, in line with design
[x] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )
- [ ] If this PR fixes a race condition in the test suite, also push the fix to the relevant stable branche(s) (see test-fixes for more info)

Processing this pull request

Merged into branches master in c3e75c5a35a9877f4e3e4691fd7890352148175f

Processing #7733.


Support VSCode's RPM/tarball installation as well

Current toolbox-vscode implementation is primarily focused on Flatpak installation of VSCode, but I believe that it could support RPM/tar.xz installation of VSCode as well. The main reason one may prefer this approach rather than the Flatpak one is the fact that VSCode updates from Microsoft repository usually come faster than from Flathub (at the time I'm writting, the Flathub latest version is 3 releases behind the official RPM/tar.xz one).
An eventual RPM support needs to take the following details into account (I will edit the list as needed):

[ ] Support VSCode config location at ${HOME}/.config/Code/User/globalStorage as well as ${HOME}/.var/app/com.visualstudio.code/config/Code/User/globalStorage
[ ] The podman-host.sh script needs to be adapted to spawn podman directly rather than flatpak-spawn --host podman, but settings the same environment variables.
[ ] The ~/.local/bin/code symlink may conflict with default code binary path, depending on how user's PATH variable is set.
[ ] Maybe provide a way to automatically download VSCode RPM from official repository and install it with dnf / rpm-ostree / whatever.
[ ] Maybe support VSCode's tar.xz automatic download and uncompress at /usr/local or /opt folder as well (for ones worried about polluting their Fedora Silverblue/Kinoite installations with layered RPMs). In this case, toolbox-vscode could also check for updates for the tar.xz case.

I think it's better to keep the Flatpak up-to-date then dodge around the issue. So on Silverblue/Kinoite systems - I'd like to keep recommending the Flatpak.
But if you already have a working VSCode setup on a non-rpm-ostree system, it would be very nice if you could just start using Toolbox and have the integration work properly. Also see: https://pagure.io/fedora-workstation/issue/283 - we're planning to add the RPM repository to Fedora as a 3rd party repository.
I've been holding off on doing anything in this area waiting for requested changes on #20
Some further notes.

I'd rather not prompt for tarball installation. If the user already has it installed via tarball and in the path, that should work.
Sorting out all the possibilities is hard. I think it's something like:

If the Flatpak is installed, use that
Otherwise, if 'code' is in the path ignoring $0, run that
If 'pkcon' is present, and pkcon -p resolve vscode  | grep -q '^Available' prompt to install via pkcon
Otherwise, prompt to install via Flatpak

podman-host.sh can check if it's in a flatpak by looking for /.flatpak-info
Finding the next 'code' in the path can be done with something like:

read -d '' -ra path_entries < <(which -a code)
next_code=
for (( i=0; i < ${#path_entries[@]}; i++ )) ; do
    if ! [ "${path_entries[$i]}" -ef "$0" ] ; then
        next_code=${path_entries[$i]}
        break
    fi
done

(For installation, I'd actually prefer to be able to ask GNOME Software to prompt to install vscode by the best option for the system, but 'gnome-software --install' doesn't come close to being usable for that. The argument to it is specific to the backend and it either installs without prompting or fails without useful feedback.)


Emergency bug fixes for show_export.html.erb and jquery.timeago.js

Some bugs that prevented show_export page from loading at all.
Bugs in the jquery.timeago.js localisation

I hot fixed the server. Please merge these in when you have a chance to sync everything up

I'm not completely happy with the jquery.timeago.js changes. Its a bit weird to use internationalization with the way they swap in the %d value. It sounds like we may go a different route though in the future (something other than jquery?) so we can leave it for now I guess since its working.


simple change

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:
Testing the merge bot.

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

Signed-off-by: Jason Kuster<EMAIL_ADDRESS>

@merge-bot merge

@beam-testing merge

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

@beam-testing merge

Error: Setup of temp directory failed, try again..

@beam-testing merge

Error: Clone failed. Please try again..

@beam-testing merge

Error: Setup of temp directory failed, try again..

@beam-testing merge

Error: Setup of temp directory failed, try again..

@beam-testing merge

Error: Clone failed. Please try again..

@beam-testing merge

Error: Setup of temp directory failed, try again..

@beam-testing merge

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

@beam-testing merge

Error: Setup of temp directory failed, try again..

@beam-testing merge

Error: Merge was not successful. Please try again..

@beam-testing merge

Error: Setup of temp directory failed, try again..

@beam-testing merge

Info: PR merge succeeded!.

@beam-testing merge

Error: Clone failed. Please try again..

Error: Clone failed. Please try again..

Info: PR merge succeeded!.

@beam-testing merge

Info: PR merge succeeded!.

Info: PR merge succeeded!.

@beam-testing merge

@apache-merge-bot merge

@apache-merge-bot merge


Use secp256k1 library

Also fixes #184 and #185.

I got the following error while building the library
./autogen.sh: 3: ./autogen.sh: autoreconf: not found

and had to install the following package to make it work:
apt-get install dh-autoreconf

Additionally, I get the following warnings:
Warning: File listed in .stack-work/downloaded/9e22127442813813d17b47539e0afef2fd70ef1f4a5b24d7c117b8e78b24cfb0.git/secp256k1.cabal file does not exist: src/libsecp256k1-config.h.in
Warning: File listed in .stack-work/downloaded/9e22127442813813d17b47539e0afef2fd70ef1f4a5b24d7c117b8e78b24cfb0.git/secp256k1.cabal file does not exist: src/stamp-h1

apt-get install autoconf


Default sqlite db name is too generic

The default of sqlite3.db is way too generic and can easily cause conflicts.
I'm also not sure about the location (~/.ndn), which I believe is semi-private business of the ndn-cxx library and shouldn't be used by random applications... At the very least the db file should be created in an application-specific subdir of .ndn

Fixed with https://github.com/JonnyKong/ndn-python-repo/commit/30aefcf6631d12ffed164ea2a0ab01ca1c455844. Moved all repo-related files to "~/.ndn/ndn-python-repo/".


Timeout of install and enable vscode-smoketest-check extension on web

PS C:\Users\merogge\repos\vscode> yarn smoketest --build C:\server\vscode-server-win32-x64-web --web --browser chromium
yarn run v1.22.10
$ cd test/smoke && yarn compile && node test/index.js --build C:\server\vscode-server-win32-x64-web --web --browser chromium
$ yarn --cwd ../automation compile && tsc
$ npm run copy-driver && npm run copy-driver-definition && tsc

><EMAIL_ADDRESS>copy-driver C:\Users\merogge\repos\vscode\test\automation
> cpx src/driver.js out/

><EMAIL_ADDRESS>copy-driver-definition C:\Users\merogge\repos\vscode\test\automation
> node tools/copy-driver-definition.js

Running web smoke tests against C:\server\vscode-server-win32-x64-web

*** Test data: C:\Users\merogge\AppData\Local\Temp\t89241XJIvyY4iV0Y
*** Preparing smoketest setup...
*** Cloning test project repository...
*** Running yarn...
[1/4] Resolving packages...
[2/4] Fetching packages...
[3/4] Linking dependencies...
[4/4] Building fresh packages...
*** Smoketest setup done!

  VSCode Smoke Tests (Web)
    VSCode Smoke Tests (Web)
Starting built server from 'C:\server\vscode-server-win32-x64-web\server.cmd'
Server stdout:

*
* Visual Studio Code Server
*
* Reminder: You may only use this software with Visual Studio family products,
* as described in the license https://aka.ms/vscode-remote/license
*

Extension host agent listening on 8000

Server stderr: Ignoring option browser: not supported for server.

Server stdout: Web UI available at http://localhost:8000/?tkn=24b4a29c-136a-46cf-9d3a-b8015abbd94f

Server stdout: [12:49:46] Extension host agent started.

Server stdout: [12:49:49] [::1][93e28eb9][ManagementConnection] New connection established.

Server stdout: [12:49:51] [::1][91fb8de6][ExtensionHostConnection] New connection established.

Server stdout: [12:49:51] [::1][91fb8de6][ExtensionHostConnection] <17440> Launched Extension Host Process.

      Search
        - has a tooltp with a keybinding
        √ searches for body & checks for correct result number
        √ searches only for *.js files & checks for correct result number
        - dismisses result & checks for correct result number
        √ replaces first search result with a replace term
      Quick Access
        √ quick access search produces correct result
        √ quick access respects fuzzy matching
      Notebooks
        - moves focus in and out of output
        - cell action execution
      Language Features
        √ verifies quick outline
        √ verifies problems view
        √ verifies settings
      Editor
        √ shows correct quick outline
      Statusbar
        √ verifies presence of all default status bar elements
        √ verifies that 'quick input' opens when clicking on status bar elements
        √ verifies that 'Problems View' appears when clicking on 'Problems' status element
        √ verifies that 'Tweet us feedback' pop-up appears when clicking on 'Feedback' icon
        √ checks if 'Go to Line' works if called from the status bar
        √ verifies if changing EOL is reflected in the status bar
      Extensions
Server stdout: [12:50:16] Installing extension: michelkaporin.vscode-smoketest-check

Server stdout: [12:50:17] Downloaded extension: michelkaporin.vscode-smoketest-check c:\Users\merogge\AppData\Local\Temp\t89241XJIvyY4iV0Y\d\data\CachedExtensionVSIXs\michelkaporin.vscode-smoketest-check-0.0.1

Server stdout: [12:50:17] Extracted extension to C:\Users\merogge\AppData\Local\Temp\t89241XJIvyY4iV0Y\extensions-dir\.0a8f15be-37ba-4836-9622-e0631d06944c: michelkaporin.vscode-smoketest-check

Server stdout: [12:50:17] Renamed to C:\Users\merogge\AppData\Local\Temp\t89241XJIvyY4iV0Y\extensions-dir\michelkaporin.vscode-smoketest-check-0.0.1

Server stdout: [12:50:17] Installation completed. michelkaporin.vscode-smoketest-check

Server stdout: [12:50:17] Extensions installed successfully: michelkaporin.vscode-smoketest-check

        1) install and enable vscode-smoketest-check extension
Server stdout: [12:51:15] [::1][93e28eb9][ManagementConnection] The client has disconnected gracefully, so the connection will be disposed.

  15 passing (2m)
  4 pending
  1 failing

  1) VSCode Smoke Tests (Web)
       VSCode Smoke Tests (Web)
         Extensions
           install and enable vscode-smoketest-check extension:
     Error: Timeout of 60000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (C:\Users\merogge\repos\vscode\test\smoke\out\main.js)
      at listOnTimeout (internal/timers.js:554:17)
      at processTimers (internal/timers.js:497:7)

error Command failed with exit code<PHONE_NUMBER>.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
PS C:\Users\merogge\repos\vscode>

Seems to be running fine now.


Add text panel

https://grafana.com/docs/grafana/latest/panels-visualizations/visualizations/text/

Resolved by #38


Typos

"Embedding Power BI content from a Government Community Cloud (GCC), can only be done with an Office 356 SKU. Other national cloud customers can use Office 356 or Azure SKUs."
Office 356 => Office 365

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

ID: 286e7693-df7f-d311-952b-9d5d846f57d3
Version Independent ID: a9728923-20a2-cafc-72a8-a26b4e1a5d72
Content: Embed content in your application for government and national cloud - Power BI
Content Source: powerbi-docs/developer/embed-sample-for-customers-national-clouds.md
Service: powerbi
Sub-service: powerbi-service
GitHub Login: @KesemSharabi
Microsoft Alias: kesharab

@harriyott fixed in master. Will go live in next publishing cycle later today.


Cannot Use HighPrecisionMoneyDraft with CartAddCustomLineItemAction

Expected: The CartAddCustomLineItemAction accepts a HighPrecisionMoneyDraft for the money field because the HTTP supports this.
Actual: When trying to assign a HighPrecisionMoneyDraft in the CartAddCustomLineItemAction, typescript generates an error.
Type 'HighPrecisionMoneyDraft' is not assignable to type 'Money'.
  Types of property 'centAmount' are incompatible.
    Type 'number | undefined' is not assignable to type 'number'.
      Type 'undefined' is not assignable to type 'number'.ts(2322)
cart.d.ts(1351, 14): The expected type comes from property 'money' which is declared here on type 'CartAddCustomLineItemAction'

The HTTP API accepts HighPrecisionMoneyDrafts, though.
Example request body for POST /carts/{cart-id}:
{
  "version" : 1,
  "actions" : [ {
    "action" : "addCustomLineItem",
    "name": {
        "de": "test-line-item"
    },
    "slug": "test-line-item",
    "money": {
        "type": "highPrecision",
        "preciseAmount": 12345,
        "fractionDigits": 3,
        "currencyCode": "EUR"
    },
    "taxCategory": {
        "typeId": "tax-category",
        "key": "no-tax"
    }
  } ]
}

Received cart response:
...
    "customLineItems": [
        {
            "totalPrice": {
                "type": "centPrecision",
                "currencyCode": "EUR",
                "centAmount": 1234,
                "fractionDigits": 2
            },
            "id": "7433e8f8-ecef-452c-b689-8cfade67d199",
            "name": {
                "de": "test-line-item"
            },
            "money": {
                "type": "highPrecision",
                "currencyCode": "EUR",
                "centAmount": 1234,
                "preciseAmount": 12345,
                "fractionDigits": 3
            },
            "slug": "test-line-item",
            "quantity": 1,
            "discountedPricePerQuantity": [],
            "taxCategory": {
                "typeId": "tax-category",
                "id": "d6aaae6c-52bc-4c4e-9e3c-8bd1fd8c6dca"
            },
            "state": [
                {
                    "quantity": 1,
                    "state": {
                        "typeId": "state",
                        "id": "5ce39fce-cc0c-4db2-ab03-553599fa4df2"
                    }
                }
            ],
            "perMethodTaxRate": [],
            "priceMode": "Standard"
        }
    ],
...

Also raised as SUPPORT-19383 in jira.commercetools.com

Uh. Nice catch and a tough one.
What happens here is that the only for HighPrecisionMoneyDraft the centAmount becomes optional as it will be calculated by the platform.
So we either make centAmount also required for the HighPrecisionMoneyDraft also it's not necessary on API level or centAmount will become optional for all derived types.
As a workaround for you would it be possible to provide a rounded centAmount too?

I released a new version. This comes with an union type for money which is added to all properties using Money

Thank you very much. Everything works as expected.


Forced function calling (mode "ANY") no longer working

Description of the bug:
I have been successfully using forced function calling with gemini-1.5-pro so far, but now I'm getting an INVALID ARGUMENT error response stating that the mode 'ANY' is not enabled for API version v1beta.
Actual vs expected behavior:
{
  "error": {
    "code": 400,
    "message": "Function calling mode `ANY` is not enabled for api version v1beta",
    "status": "INVALID_ARGUMENT"
  }
}

Any other information you'd like to share?
No response

This issue appears to have been resolved in the latest model update (August 30, 2024). The function calling mode 'ANY' has been re-enabled for API version v1beta, and it now works again with gemini-1.5-pro. Closing the issue. Thank you.


Update ibc go commit to v9.0.0-beta.1

Is this meant to be merged already or do we rather wait for the first RC? If not then let's convert to draft just in case.

Is this meant to be merged already or do we rather wait for the first RC? If not then let's convert to draft just in case.

Oh right sorry, I put it in draft


PHARYNX and WORM analysis types for fluorescent image processing

could you merge this but reject my local changes to MWTracker/processing/batchProcHelperFunc.py ?

I cannot accept the deletion of VID_SUBSAMPLE. I need that step. It is possible to define the steps to be done in the command line using processMultipleFiles.py and specifying them in --analysis_checkpoints. I plan to incorporate this into the GUI later on.

Yes, I realised that, hence I mentioned it in the comment for the pull request - sorry if that was not visible
Can you accept the rest and just reject those changes relating to VID_SUBSAMPLE for now?

On 6 Feb 2017, at 14:38, ver228<EMAIL_ADDRESS>wrote:
I cannot accept the deletion of VID_SUBSAMPLE. I need that step. It is possible to define the steps to be done in the command line using processMultipleFiles.py and specifying them in --analysis_checkpoints. I plan to incorporate this into the GUI later on.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/ver228/Multiworm_Tracking/pull/25#issuecomment-277700662, or mute the thread https://github.com/notifications/unsubscribe-auth/ADP-0GtWuUY3cxAk6ZGr_BwP2QGYZu0aks5rZzBegaJpZM4L4LzN.

Sorry, I just read the first msg. I cannot accept changes individually. You have to alter the local changes before merging.

I was familiar with how to skip the vid_subsample in the command line for tracking, is this now also possible for the compression step?

On 6 Feb 2017, at 14:40, ver228<EMAIL_ADDRESS>wrote:
Sorry, I just read the first msg. I cannot accept changes individually. You have to alter the local changes before merging.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/ver228/Multiworm_Tracking/pull/25#issuecomment-277701187, or mute the thread https://github.com/notifications/unsubscribe-auth/ADP-0DCU6ueL_OALxROklWk_yXXblHBIks5rZzDDgaJpZM4L4LzN.

I corrected my local changes so they should now be compatible


[Bug] : An unexpected error has occurred

[Bug] : An unexpected error has occurred
Route : https://www.hacksquad.dev/myteam
Time of occurrence : Oct-8-2022, Couple of hours before (not exactly noted)
Description :
The error was a random one. while clicking "mysquad" button on navbar, it takes to the above route. but a blank white page showing "An unexpected error has occured".
Right now it works fine. Just wanted to report this incident.

Hi @najeebkp
Thank you for reporting the issue.
The problem happened for a short period of time while deploying new set of changes to the website.
Most likely you won't see it ever again 🤞


should/should_not syntax

#define CEDAR_MATCHERS_ALLOW_SHOULD before importing SpecHelper to use should syntax:
3 should equal(3);
view.subviews should_not contain(buttonView);

I moved the should syntax code into its own file and turned it on by default, but otherwise I merged unchanged.


Member: Highlight the upcoming Ensemble

When many ensembles are scheduled in the future, it can be a bit confusing not to see the upcoming one (e.g., today's) at the top of the list.

No longer needed, since the list is split into Upcoming and Past, and the first one in the upcoming list is always the one that will happen soonest.


Some suggestions on DPM refactoring

I am thinking about DPM supporting pulsar, in order to make DPM support multiple message queues, I think it is necessary to Refactor the code of DPM.

@xieus @haboy52581 I have a few more questions about the structure of NetworkConfiguration:

What information should be included in NetworkConfiguration when only security groups are updated.
What information should be included in NetworkConfiguration when only router information are updated.

@xieus @haboy52581 I have a few more questions about the structure of NetworkConfiguration:

What information should be included in NetworkConfiguration when only security groups are updated.
What information should be included in NetworkConfiguration when only router information are updated.

We should have a new API in DPM that support SG and router (including update of routing table and rules). The method of the API is either PATCH or PUT, and the NetworkConfiguration object includes the port that requires information update, and a list of SGs or Routers to be updated.
DPM should convert the PM message to protobuf message, with

the operation type of port as INFO
the operation type of SGs or Routers as UPDATE

I am thinking about DPM supporting pulsar, in order to make DPM support multiple message queues, I think it is necessary to Refactor the code of DPM.

Thanks @chenpiaoping Sounds good to me. At first glance, the PR added a few items that have been long overdue. I will look further with more comments.

@chenpiaoping Please check https://jenkins.alkaidcloud.io/job/alcor-controller-pr-test/990/console to find some compile error for DPM.

@chenpiaoping I knew that you've checked in a portion of this PR to master in PR https://github.com/futurewei-cloud/alcor/pull/366. Do you have some remaining files to be checked in? If so, please rebase from master; otherwise we can close this PR. Let me know.

@chenpiaoping I knew that you've checked in a portion of this PR to master in PR #366. Do you have some remaining files to be checked in? If so, please rebase from master; otherwise we can close this PR. Let me know.

No, please close it.

Part of the PR has been merged to master in #366 . Close this PR as it becomes outdated.


Modal dialog for prompt to install ipykernel

For #5798

ook like the linked issue says it should be behind an experiment?

That was my initial plan, but I don't think we need to do that.
Based on @claudiaregio some users are not seeing this message (validated in a QP)

Codecov Report

Merging #5799 (3c31a2d) into main (fa32f62) will increase coverage by 2%.
The diff coverage is n/a.

:exclamation: Current head 3c31a2d differs from pull request most recent head 2ed5ff1. Consider uploading reports for the commit 2ed5ff1 to get more accurate results

@@          Coverage Diff           @@
##            main   #5799    +/-   ##
======================================
+ Coverage     69%     72%    +2%
======================================
  Files        403     403
  Lines      26934   26924    -10
  Branches    3927    3926     -1
======================================
+ Hits       18847   19402   +555
+ Misses      6544    5890   -654
- Partials    1543    1632    +89

Impacted Files
Coverage Δ

...science/jupyter/kernels/kernelDependencyService.ts
76% <ø> (-9%)
:arrow_down:

...t/datascience/jupyter/jupyterInvalidKernelError.ts
66% <0%> (-34%)
:arrow_down:

src/client/datascience/jupyter/jupyterSession.ts
75% <0%> (-5%)
:arrow_down:

src/client/datascience/baseJupyterSession.ts
69% <0%> (-3%)
:arrow_down:

...t/datascience/kernel-launcher/localKernelFinder.ts
81% <0%> (-1%)
:arrow_down:

...datascience/jupyter/liveshare/hostJupyterServer.ts
61% <0%> (-1%)
:arrow_down:

src/client/datascience/jupyter/kernels/helpers.ts
71% <0%> (-1%)
:arrow_down:

...ent/datascience/notebook/notebookEditorProvider.ts
82% <0%> (-1%)
:arrow_down:

src/client/datascience/jupyter/jupyterNotebook.ts
72% <0%> (+<1%)
:arrow_up:

... and 41 more

I would like to create an experiment just so we can track improvement from our other version (show how many people are taking an action on the exp vs. how many took it on the old one)


Fix all social media links in the landing page

The landing page is already up and running, but the social media links are redundant.

Reassign to me

Okay
On Thu, Jul 2, 2020 at 10:54 AM Adebayo seun<EMAIL_ADDRESS>wrote:

Reassign to me
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/codeclannigeria/codeclannigeria-frontend/issues/17#issuecomment-653144335,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANRT6JGU4S5FY6IY26PWTTLRZTCVBANCNFSM4NS4EEKQ
.


refactor(linter): NoRedeclare rule use run_on_symbol not run_once

eslint/no_redeclare rule iterates over all symbols. Use run_on_symbols for this, instead of run_once.

#5201  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @overlookmotel and the rest of your teammates on  Graphite

I had expected this to produce a small perf gain, as it avoids an extra round of iterating through all symbols, but it's actually showing as manginally slower rather than faster.
I don't know how much variance there is in linter benchmarks - possible the apparent perf drop is just noise in the benchmark? Or is it likely for real?

I merged main branch in and now benchmark says it's faster! I guess it is just noise in the benchmarks...


added return types for getter methods

Since the methods may not return the intended type, they're wrapped in an Optional. Although I think all methods should have explicit return types, that should be further analyzed.

According to the library documentation, Optional is simply a wrapper for a Union[X, None], which can be written as X | None. So both syntaxes should be valid and equivalent, and a matter of preference.

According to the library documentation, Optional is simply a wrapper for a Union[X, None], which can be written as X | None. So both syntaxes should be valid and equivalent, and a matter of preference.

Cool, Thanks for the clarification and your contribution
Merging this


Not compatible with XCode 7 and Swift 2

SocketIOClientSwift is out of date, should be upgraded to version 3 which includes swift 2 support.

Sorry, didn't see issue


Use java defaults when the user hasn't specified trusted certs

Purpose
$subject.

Codecov Report

Merging #19913 into ballerina-1.0.x will not change coverage.
The diff coverage is n/a.

@@               Coverage Diff                @@
##           ballerina-1.0.x   #19913   +/-   ##
================================================
  Coverage            20.51%   20.51%
================================================
  Files                   52       52
  Lines                 1394     1394
  Branches               216      216
================================================
  Hits                   286      286
  Misses                1095     1095
  Partials                13       13

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 6d497eb...89675a9. Read the comment docs.


Generate code coverage reports

This replaces the mochaTest grunt task with mocha_istanbul to track code coverage. Coverage reports are output to test/coverage and sent to coveralls.io. A coverage badge has been added to the README.

ohhhhh, this is exciting

@ascott1  What's a code coverage report?  I'm excited too, just want to know what about!  :grinning:

Test coverage?

@stephanieosan Bingo. It's called code coverage instead of test coverage because you're covering the code in tests.
It's good practice to write automated tests that verify your code works as expected (and that it doesn't break as updates are made down the road). As a codebase grows, it can be hard to keep track of which parts of the application are covered by tests and which are not.
 They generate reports that tell you "Hey! Function ABC and XYZ don't have any tests written for them." They'll also tell you the percentage of your code that is covered by tests. e.g. If you update the application and your code coverage drops from 100% to 85%, it means you failed to write proper tests for that new code.

I love this!


upgrading mypy to 0.790 fails

Continuing from #4567, trying to upgrade mypy to 0.790 fails with:
xarray/core/utils.py:466: error: Value of type variable "_LT" of "sorted" cannot be "K"

See also https://github.com/pydata/xarray/pull/4567#issuecomment-723546797 and https://github.com/pydata/xarray/pull/4567#issuecomment-724016427

For context, mypy 0.790 now requires __lt__ if something should be sorted. Thus the following fails (used to work):

from typing import TypeVar
T = TypeVar("T")

def sort(lst: List[T]) -> List[T]):
    return sorted(lst)

which returns the error message above (_LT stands for _LowerThan). The workaround is to define a TypeVar that is Sortable:

from typing import TypeVar, List, Protocol

class Sortable(Protocol):
    def __lt__(self, __other: Any) -> bool: ...

SortableT = TypeVar("SortableT", bound=Sortable)

def sort(lst: List[SortableT]) -> List[SortableT]):
    return sorted(lst)

What we want is a sortable dict key. However, we define dict keys as Hashable (e.g. dims) which is not sortable. I see two possibilities:

type all keys of a SortedKeysDict as SortableHashable (that would be a lot)
ignore this and add # type: ignore to core/utils.py#L466

But maybe there is a better way?
Here is how 1. could look like (simplified):

import collections
from typing import Any, Dict, Iterator, Mapping, Protocol, TypeVar

class HashableSortable(Protocol):
    def __lt__(self, __other: Any) -> bool:
        ...

    def __hash__(self) -> int:
        ...

HashableSortableK = TypeVar("HashableSortableK", bound=HashableSortable)
V = TypeVar("V")

# use UserDict for brevity
class SortedKeysDict(collections.UserDict[HashableSortableK, V]): # used to be [K, V]
    def __iter__(self) -> Iterator[HashableSortableK]:
        return iter(sorted(self.data))

class Dataset:

    _dims: Dict[HashableSortable, int] # used to be [Hashable, int]

    def __init__(self, dims) -> None:

        self._dims = dims

    @property
    def dims(self) -> Mapping[HashableSortable, int]:
        return SortedKeysDict(self._dims)

Thanks for the excellent research @mathause
I would vote for #2. I think in time we could move away from SortedKeysDict given that dicts are now ordered. (There is a recent issue on this but I couldn't immediately find it)

Jup, given the effort required for 1. I also vote for 2. I learned a lot about typing/ mypy, though...