set_chart method not defined error in 0.80pre to 0.83pre

In the new pre-releases forms package including 0.80pre to 0.83pre, doing a simple chart based on the GitHub example like
chartView.Chart=new BarChart() { Entries = entries };
would throw the set_chart method not defined error. the same code works well in 0.62. can't test it in 0.70/0.71 due to .net compatibility errors.

Hi!  Very strange, make me think that you have a mismatching dll. Make sure that you haven't inconsistencies between the Microcharts package references from your projects.


Entropy is gone

Hi,
C1 machine + Debian Jessie:
cat /proc/sys/kernel/random/entropy_avail
0

This is on na fresh C1 instance, however it was not always this way, here is munin chart of entropy history of another C1 instance:
http://i.imgur.com/bopzDbO.png

Related to scaleway/image-alpine/issues/51 and scaleway/image-ubuntu/issues/81

I met this issue on the gentoo image, C1 machine too, it is impossible to get a certificate with certbot due to this problem. Ask me anything, I’ll try to provide all the details.


Native fetch users on attachments

Summary
Native code for fetching the usernames on post attachments.
Ticket Link
Fix https://mattermost.atlassian.net/browse/MM-52937
Release Note
NONE

Cherry pick is scheduled.


origin of 'totalEmployees' variable in code example not clear

Description
Not sure whether this is an issue or not, but in the code example below I noticed that a 'totalEmployees' variable is used, which I cannot gather from context where it comes from / where it gets its value:
private async ValueTask<ItemsProviderResult<Employee>> LoadEmployees(
    ItemsProviderRequest request)
{
    var numEmployees = Math.Min(request.Count, **totalEmployees** - request.StartIndex);
    var employees = await EmployeesService.GetEmployeesAsync(request.StartIndex,
        numEmployees, request.CancellationToken);

    return new ItemsProviderResult<Employee>(employees, **totalEmployees**);
}

Page URL
https://learn.microsoft.com/en-us/aspnet/core/blazor/components/virtualization?view=aspnetcore-9.0
Content source URL
https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/blazor/components/virtualization.md
Document ID
26ed2425-63ec-e206-edcb-e51f17edb844
Article author
@guardrex
Related Issues

Hello @NikolaJacques ... It's just pulled via a method call elsewhere in the component, such as during initialization. I'll make a remark for that example. Leave this issue open, as it will close automatically when the PR merges later. Thanks for the issue!


Adding features to KenLM binding or changing API

Hello,
While I know that the ctclib-kenlm-sys crate is not a general purpose binding library, I'm currently in need of such a binding and your work is a good starting point.
I have a first question, related to KenLM's new method that requires a dict. I have used kenlm's query binary and did not have to use this to do queries. So I forked your repository and made some changes (making Model public, using base_score and then computing perplexity) and on the (tiny) samples that I have tried, I got the same results than the query binary.
So I'm wondering if I should try to add these changes (in a non breaking way, obviously) insode this lib or if you'd prefer me to fork the ctclib-kenlm-sys and make something more opinionated.
Let me know what you prefer, or if there's a way of using the KenLM struct in the way I intend to do, without changing code :)

I have pushed my changes on my local fork here, and I'll publish it on crates.io so that I can use it in my projects. If you are interested in me PRing my changes to your repo let me know!


Error: Unable to establish one or more of the specified browser connections. This can be caused by network issues or remote device failure.

TestCafe is unable to connect to the browser.
This started happening a week ago on some of our CI machines and seems like the issue is "spreading" to other CI machines as well.
Is there a known recent OS update that may start causing this?
I tried:
Chrome, Chromium, FireFox, Safari (headless and non-headless, when supported),
With and without concurrency,
With --no-sandbox and without.
With the testcafe executable and with the testcafe library (via our code).
With our test-suite and without (a single empty js test).
All fail for the same reason.
macOS 10.13.6
Node v8.11.1
TestCafe v0.22.0

When trying Safari I get this error:
Error: Was unable to open the browser "locally-installed:safari" due to error.
Error: Unable to run the browser. The file at /Users/vsts/agent/2.140.0/work/1/s/ux/portal/node_modules/testcafe-browser-tools/bin/mac/open.scpt does not exist or is not executable.

Hi @adrianaisemberg, could you please provide the error message which you get when starting Chrome and Firefox?

As in the title:
Unable to establish one or more of the specified browser connections. This can be caused by network issues or remote device failure

I guess you start the tests using Azure Pipelines or TFS, Looks like the vsts-agent version that you use (2.140.1) was released 5 days ago. Could you try to rollback to the some previous agent version?

@AndreyBelym I'm afraid I won't be able to do that.
What is the difference between running via the command-line versus the library?
Why does it work with one but fails with the other? (Only with Safari, otherwise they all fail)

@AndreyBelym I think I am running into this issue as well and I am seeing different results depending on the vsts-agent that gets picked up from the pool. Would the only way to select the agent be to have an agent in the pool with a lower target and then pick that agent? Or is there a way to define that on my vsts-ci.yaml file? I've been googling for a couple of hours and their documentation is not clear to me, I'd appreciate the help.
Thanks,

Hi @adrianaisemberg, @jossjacobo! Unfortunately I'm not familiar with TFS infrastructure, so I need some days to setup environment and debug TestCafe under a VSTS agent. Looks like that TestCafe has no permissions to spawn processes in VSTS environment, so it can't start any browser. I don't think that there should be no difference between running TestCafe from CLI wrapper or from your own script as a library, but I can't be sure without testing it. The error message for Safari is different, because TestCafe uses AppleScript to start Safari, and other browsers are started from their binaries directly.

@adrianaisemberg, @jossjacobo could you please clarify, do you use Microsoft-hosted agents or your own machines? If the latter, do you start your agents interactively or as a service?

Hello I am a Product Manager on the Azure Pipelines team.
@adrianaisemberg  and @jossjacobo We have typically seen this be an issue on mac when not running as a LaunchAgent or running interactively.  The agent update process simply replaces the binaries on disk and should not impact any configured permissions as far as launching processes is concerned.  However, if there is something we can do to help out please let me know.

We eventually switched to running on Linux.


Rework sysext release

Fix issues we have had with sysext releases. Previously upload failure would cause us to lose release assets, because the release gets deleted before upload.

@t-lo we need this before we push the next "latest" tag. Mathieu faced issues with previous releases.


Update service-fabric-reliable-actors-kvsactorstateprovider-configura…

…tion.md

Hi @sumukhs, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (sumukhs). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;


[Feature Request] Add back a Status that lets us distinguish between the two types of buffering — Play Buffering vs Paused Buffering

What is the need and use case of this feature?
Prior to v4, the State.Buffering state always meant it's buffering while trying to play. My application relied on this behavior.
In the v4 upgrade notes, it says State.Buffering Updated. Now emitted no matter whether playback is paused or not.
I would like a way to tell this very important distinction. In certain core cases (e.g. when I'm recording "play intervals" to my local database), this distinction is not incidental, it's a very imporant distinction. Unless I'm missing an API, there's currently no way for me to tell the difference between these two different types of buffering, other than maintaining a full history of the state updates myself and doing a very complex and brittle logic
Describe the ideal solution
Bring back the distinction between "Play Buffering" versus "Pause Buffering"
Describe alternatives you've considered
Trying to calculate this myself by looking at the full history of state changes, but with multiple tracks this quickly gets very complex and brittle
How I can Help
Happy to help however I can!

For now my application is going to assume that State.Buffering always means this "play buffering" case since that's far more common, and I'll need to add more strict idempotency guarantees (which adds a decent amount of complexity for me) in a number of call-sites. So the impact here on my application is notable. I'm guessing it's this way for others.
Was there a discussion about why this change was made?

Fwiw, I also only want to know about play buffering.  I'm already tracking state to know if the app was was previously playing so for me it's not too bad (I think!)

Thank you @lovegaoshi I did not find this in my searching of the docs or github, but I was using the wrong keywords.
This PR is the solution here: https://github.com/doublesymmetry/react-native-track-player/pull/2040
Closing


Centralize storage management / Avoid fragmented storage

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[ ] Other... Please describe:

What is the current behavior?
Alternative addition/deletion of comment led to a fragmented storage, i.e.the CSV file was containing empty lines.
What is the new behavior?
The storage management (add/update/delete) has been slightly refactored the be handle in one single place.
Empty lines are filtered be being saved.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

@michelcaradec thanks for the great contributions!


Serveur en panne (FormatException: Unexpected character (at character 1)

What

Serveur en panne (FormatException: Unexpected character (at character 1)
)

Additional case to be added in fetched_product.dart's method silentFetchAndRefresh, checking if the exception contains/starts with FormatException: Unexpected character (at character 1) and <!DOCTYPE html>.


HTTP HEAD does not return a response to the client

Step 1 - Set up a back end service that accepts http head as the http method for the resource
Step 2 - Write a similar code as given below
import ballerina.net.http;

@http:BasePath {value:"/http"}
service contentBasedRouting {
    string connection = "http://localhost:5301";

    @http:POST {}
    @http:GET {}
    @http:Path {value:"/callHead"}
    resource httpHeadResource (message m) {
        string resourcePath = "/abc";
        string headerValue;
        message response;
        message request = m;
        http:ClientConnector httpCheck = create http:ClientConnector(connection);
        response = http:ClientConnector.head(httpCheck, resourcePath, request);
        reply response;
    }

}

Step 3. Invoke the above service
Observations

When the communication is analyzed using tcp mon, below was seen that indicates, outgoing and incoming from and to ballerina

Incoming message from client to ballerina
POST /http/callHead HTTP/1.1
Host: localhost:5302
Connection: keep-alive
Content-Length: 122
Postman-Token: 81fe162a-0a87-173b-6cc4-9334f5faa477
Cache-Control: no-cache
Origin: chrome-extension://fhbjgbiflinjbdggehcddcbncdddomop
User-Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/59.0.3071.86 Safari/537.36
Content-Type: application/json
Accept: */*
Accept-Encoding: gzip, deflate, br
Accept-Language: en-US,en;q=0.8

{
   "Order":{
       "orderId":"ee1a9ec2-c8a5-4afe-8585-74df591f9990",
       "additions":"Chocolate Chip Cookies"
   }
}

Outgoing from ballerina
HEAD /abc HTTP/1.1
Connection: keep-alive
Content-Length: 122
Postman-Token: 0c5da918-1b64-c2bb-f384-b8a420967f5b
Cache-Control: no-cache
Origin: chrome-extension://fhbjgbiflinjbdggehcddcbncdddomop
Content-Type: application/json
Accept: */*
Accept-Encoding: gzip, deflate, br
Accept-Language: en-US,en;q=0.8
HOST: localhost:5301
User-Agent: ballerina/0.88-SNAPSHOT

{
   "Order":{
       "orderId":"ee1a9ec2-c8a5-4afe-8585-74df591f9990",
       "additions":"Chocolate Chip Cookies"
   }
}

Incoming to Ballerina
HTTP/1.1 200 OK
Content-Type: application/json
Content-Encoding: gzip
Content-Length: 145
Server: Jetty(6.1.26)

Outgoing from ballerina to client
None

This issues solved after adding HTTP HEAD annotations to server connector

As per the above comment. I am closing this issue.

I am re-opening the issue as we haven't tested with Jetty as in the given sample.

This issue is well fixed with the carbon transport refactor https://github.com/wso2/carbon-transports/pull/345

As the transport refactor is merged this issue is fixed in 0.93 release.


Sharded.getAllShardInfo() returns 160*shards info list not returns the original shards list

Sharded.getAllShardInfo() has a bug that returns 160*shards info list, each shard info has 160 same copies. This method should return the original shards list.
Please see the following code from Sharded.java:
  private TreeMap<Long, S> nodes;

  private void initialize(List<S> shards) {
    nodes = new TreeMap<Long, S>();

    for (int i = 0; i != shards.size(); ++i) {
      final S shardInfo = shards.get(i);
      if (shardInfo.getName() == null) for (int n = 0; n < 160 * shardInfo.getWeight(); n++) {
        nodes.put(this.algo.hash("SHARD-" + i + "-NODE-" + n), shardInfo);
      }
      else for (int n = 0; n < 160 * shardInfo.getWeight(); n++) {
        nodes.put(this.algo.hash(shardInfo.getName() + "*" + shardInfo.getWeight() + n), shardInfo);
      }
      resources.put(shardInfo, shardInfo.createResource());
    }
  }

  public Collection<S> getAllShardInfo() {
    return Collections.unmodifiableCollection(nodes.values());
  }

The fixed code:
  private final Map<S, R> resources = new LinkedHashMap<S, R>();

  public Set<S> getAllShardInfo() {
	return Collections.unmodifiableSet(resources.keySet());
  }

@EdwardLee03
Thanks for finding!
Btw, seems like we don't need to change getAllShardInfo()'s return. How do you think about?

I writes a Jedis-x framework, that is an extension of Jedis. It was primarily built to detect and remove the failed node automatically, also add the failed node that back to normal automatically.
Now, I use the return value this method to judge whether the PooledObject<ShardedJedis> is valid, and find this issue. Current, I use Map<JedisShardInfo, String> to filter the same JedisShardInfo copy. Though the function is OK, but the code is difficult to read. Please see the following code of CustomShardedJedisFactory:
    public boolean validateObject(PooledObject<ShardedJedis> pooledShardedJedis) {
        ShardedJedis shardedJedis = pooledShardedJedis.getObject();
        // FIXME "Sharded.getAllShardInfo() returns 160*shards info list not returns the original shards list"
        // https://github.com/xetorthio/jedis/issues/837
        Collection<JedisShardInfo> allClusterShardInfos = shardedJedis.getAllShardInfo(); // 返回的集群节点数量被放大了160倍，详见ShardedJedisTest.getAllShardInfo()测试用例
        // 过滤所有重复的Shard信息
        Map<JedisShardInfo, String> clusterShardInfoMap = new HashMap<JedisShardInfo, String>();
        for (JedisShardInfo clusterShardInfo : allClusterShardInfos) {
            if (!clusterShardInfoMap.containsKey(clusterShardInfo)) {
                clusterShardInfoMap.put(clusterShardInfo, "1");
            }
        }
        Set<JedisShardInfo> checkedShards = clusterShardInfoMap.keySet();
        logger.debug("Active Shard list for current validated sharded Jedis: {}", checkedShards);

        Set<JedisShardInfo> activeShards = serverStateCheckTimerTask.getAllActiveJedisShards();
        if (checkedShards.size() != activeShards.size()) { // 节点数不一样
            logger.debug("Find a pooled sharded Jedis is updated");

            shards = new ArrayList<JedisShardInfo>(activeShards);
            logger.debug("Active Shard list after updated: {}", shards);
            return false;
        } else { // 尽管节点数相同，但可能真实的节点列表是不同的(如，一台节点恢复正常了，正好另一台节点出现了异常)
            for (JedisShardInfo checkedShard : checkedShards) {
                if (!activeShards.contains(checkedShard)) {
                    logger.debug("Find a pooled sharded Jedis is updated");

                    shards = new ArrayList<JedisShardInfo>(activeShards);
                    logger.debug("Active Shard list after updated: {}", shards);
                    return false;
                }
            }
        }

        return true;
    }

@HeartSaVioR
The UT is pass.
Great job! :+1:  Thanks for your advice!

@HeartSaVioR
I don't know in which scenarios using this method? And the return value of this method has some ambiguity.

@EdwardLee03 Actually I don't know, too.
@xetorthio @marcosnils Do you know about that method, especially method's intention?


DAT-16210: refactor update-one-changeset command

This branch is a combination of:

DAT-16240
DAT-15639
and I believe it contains no changed files for DAT-16210. I expect that this PR will be closed without any changed files once the above branches are merged.

Impact

[ ] Bug fix (non-breaking change which fixes expected existing functionality)
[ ] Enhancement/New feature (adds functionality without impacting existing logic)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

Things to be aware of

Things to worry about

Additional Context

This one shares a lot of commits with DAT-16240 branch.

Yup. In the description of the PR I said I think this PR will actually end up going away once all of the branches it has merged into it are merged.


Bootloop after enabling device emulation, reset_mhpc doesn't work

I've also tried enabling Magisk Core-Only mode, which does fix the problem, making me think this is a bug with the module.

Device simulation has a really high chance if causing all kinds of issues. I believe I've written that in both the terminal ui and the documentation...
If the reset function doesn't work I would need to see the module logs to know what's wrong (it's explained in the docs), but you can also do things manually. Just delete /data/adb/modules/MagiskHidePropsConf/system.prop and /data/adb/mhpc/propsconf_late and everything should be reset.


Heroku Deployment Crashed

I'm submitting a ...

[X] bug report
[ ] feature request

This happened when I asked for a query in google.

@AnshulMalik @gabru-md ..

Unable to reproduce for now. Closing this until either of the following happens:

More info and proper steps to reproduce if this is an edge-case
The bug resurfaces randomly

It was resolved in https://github.com/fossasia/query-server/pull/214


Add support for encoding to and decoding from byte arrays

This is a solution for https://github.com/mathiasbynens/utf8.js/issues/7

Coverage decreased (-2.4%) to 90.184% when pulling ea9b9ec6c6925fda1cada546cd429c8f67e38126 on samal84:master into 5566334e1aa5347ba652c38dc186df08b47d8fb9 on mathiasbynens:master.

Coverage increased (+0.7%) to 93.252% when pulling 4e5fe054d4fb0c4a9743903fc8d3143534eb1899 on samal84:master into 5566334e1aa5347ba652c38dc186df08b47d8fb9 on mathiasbynens:master.

It seems the build is failing because the build server's js environment doesn't support the Uint8Array type. Should I add a commit with feature detection for Uint8Array?

+1

I've fixed up this branch so it builds again. The implementation is a bit cleaner too.

Time flies and TextEncoder/TextDecoder are now widely supported in js.


KFAnimatedImage pause/stop/resume

Hello!
Could we pause/stop and then resume animation in KFAnimatedImage? Thanks.

While we’re at it, I’d love to get an option for scrubbing as well. I noticed that the currentFrameIndex is read only.


acro-renal-ocular syndrome and acrorenoocular syndrome

MONDO_0011812 has an exact synonym of 'acrorenocular syndrome'
and is the parent of MONDO_0019863, named acro-renal-ocular syndrome
If these should be retained as distinct entities, then perhaps the exact synonym of 'acrorenocular syndrome' should be moved from MONDO_0011812 to MONDO_0019863. I am assuming the hyphens do not matter
Or should there be a merge?
See also: https://rarediseases.info.nih.gov/diseases/9182/index  --not distinct entities
https://www.omim.org/entry/607323  --not distinct entities

these seem equivalent to me too.
note to self, add synonyms:
Okihiro syndrome GARD:0009182, OMIM:607323


Step 11: Deploy Microsoft Defender ATP for macOSにて、マニフェストファイルは必要ありません 他

Step 11: Deploy Microsoft Defender ATP for macOS

Navigate to Advanced Computer Searches.

実際には、コンピュータを選択し、歯車アイコンを選択することで、次のステップを行うことができます

Select Open. Set the Display Name to Microsoft Defender Advanced Threat Protection and Microsoft Defender Antivirus.

Manifest File: Select Upload Manifest File.

マニフェストファイルについて作成する記述がありません
実際にマニフェストファイルなしで構成してもMDATPは動作します

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

ID: f791dc07-b2c6-3003-a829-cd1ec03a2918
Version Independent ID: 2c53adbc-600f-60b2-e8b2-b907335f4862
Content: Set up the Microsoft Defender ATP for macOS policies in Jamf Pro - Windows security
Content Source: windows/security/threat-protection/microsoft-defender-atp/mac-jamfpro-policies.md
Product: w10
Technology: windows
GitHub Login: @Dansimp
Microsoft Alias: dansimp

Machine Translation from Japanese, along with the original document text in English:

No manifest file is required

Step 11: Deploy Microsoft Defender ATP for macOS

iv. Navigate to Advanced Computer Searches.

In fact, you can do the next step by selecting your computer and selecting the gear icon

viii. Select Open. Set the Display Name to Microsoft Defender Advanced Threat Protection and Microsoft Defender Antivirus.

Manifest File: Select Upload Manifest File.

There is no description to create about the manifest file
MDATP works even if you actually configure it without a manifest file

I assume there is potential for looking into the correctness of the documentation here.
Disclaimer: I am not affiliated with Microsoft or the Microsoft Docs teams here on GitHub. I don't know the topic well enough to give any feedback on the topic.
I can only recommend to be aware of the fact that this repository is mainly for documentation feedback (which I assume this is), so hopefully someone with technical knowledge of Microsoft Defender ATP for macOS will reply here with questions and suggestions for how to move forward with this issue.

I'm providing feedback based on the results of a successful operation so that people using this document are not confused.
If my feedback is different from the purpose of this Github repository, then please close this ticket.
I will keep these bad knowns to share internally.

Fair enough, thank you for providing a clarification.
Are there any obvious mistakes in the Japanese translation, so we can point out any errors to the translation team?
I don't know, I do not know how to read kanji, so it will be helpful if you can point out any errors in the machine translated document.


Add key

Add key of the lines in the national plot and gsp plot

black: PV live updated
black: PV live initial estimate
solid yellow: OCF final Forecast
yellow dashed: OCF Forecast

where
at the bottom right

@braddf ill put this at done, so it easier to see what coding tasks are left. Obviously it still needs to be deployed to production/


Unable to run Kindlebox

Hi,
I am trying to run Kindlebox but am facing two issues:

When the shared folder on Dropbox is opened, it says there was some problem, but no details are shared.

On following the steps defined at: https://getbookdrop.com/#/home/devices/1
I am unable to see the button "Activate Kindlebox". I have shared a snapshot of my Amazon account on the following link:

https://cloud.githubusercontent.com/assets/25391011/22380971/69333a84-e4e1-11e6-9a6d-8119a7f6b111.png
Note: I do not use the paid Amazon services. Could this be the reason of my limited user experience?
Question: Is it possible to sync between my Kindle Android App and Kindle device using Kindlebox?
Thanks,
Zeshan

Sorry for the delayed response! Perhaps try the activation process again? For the second issue, the "Activate Kindlebox" button is actually a bookmarklet. On the Bookdrop website, you should click and drag the button to your bookmarks bar. Then, click the bookmarklet once you're on the Amazon site.


X-LHM025A-E03: switching between BIN and ASCII protocols

Sorry for asking here but official Zaber support provides zero information. My customer is using X-LHM025A motor in the different version of software. Older, written on C++, is using binary protocol; more modern software, written on C#, is using ASCII protocol.
And we're using Zaber Console app all the time to switch between protocols which is quite inconvenient.
I'm looking for simple code snippets, ideally for C++, to check and switch to BIN protocol if needed, and similar snippet for C#, to check and switch to ASCII protocol if needed.
I'm pretty positive, Zaber Tech software engineers are already have these parts of code done. So, could you please post 'em here?
Thanks a lot!

Hello @sensboston,
Here is an example in C# code of how to switch between the protocols:
using System.Threading;
using Ascii = Zaber.Motion.Ascii;
using Binary = Zaber.Motion.Binary;

const int ASCII_BAUDRATE = 115200;
const int BINARY_BAUDRATE = 9600;

while (true)
{
    Console.WriteLine("Switching to Binary");
    using (var connection = Ascii.Connection.OpenSerialPort("COM3"))
    {
        connection.GenericCommand($"tools setcomm {BINARY_BAUDRATE} 1");
    }

    Thread.Sleep(5000);

    Console.WriteLine("Switching to ASCII");
    using (var connection = Binary.Connection.OpenSerialPort("COM3"))
    {
        connection.GenericCommand(0, Binary.CommandCode.ConvertToASCII, ASCII_BAUDRATE);
    }

    Thread.Sleep(5000);
}

The command to switch to Binary is documented here.
The command to switch to ASCII is here.
Please note that the connection must be closed and a new connection open instead with the new protocol.
Please let us know if you have any further questions.

Perfect, thank you very much! On C# everything working as expected.
BTW, for C++ solution, I've added following code "at the top" of class initialization (to switch to the binary mode unconditionally):
            try {
                auto asciiConn = zaber::motion::ascii::Connection::openSerialPort(com_port_);
                asciiConn.genericCommand("tools setcomm 9600 1");
                std::this_thread::sleep_for(std::chrono::milliseconds(500));
                asciiConn.close();
            }
            catch (const zaber::motion::MotionLibException& e) {}

            // Original class code begin here
            conn_ = zaber::motion::binary::Connection::openSerialPort(com_port_);

Unfortunately added snippet not working as expected (if motor is in ASCII mode). This C++ solution is hard to debug (I should attach to the running process) so I can't debug :(
Could you please tell me, what's wrong in my code? Thanks a lot!

Hey @sensboston
I don't see anything wrong with your code. For sanity, I ran the following code, and everything worked as expected.
       while (true) {
        {
            std::cout << "Switching to Binary" << std::endl;
            auto connection = ascii::Connection::openSerialPort("COM3");
            connection.genericCommandNoResponse("tools setcomm 9600 1");
        }

        std::this_thread::sleep_for(std::chrono::milliseconds(500));

        {
            std::cout << "Switching to ASCII" << std::endl;
            auto connection = binary::Connection::openSerialPort("COM3");
            connection.genericCommand(0, binary::CommandCode::CONVERT_TO_ASCII, 115200);
        }

        std::this_thread::sleep_for(std::chrono::milliseconds(500));
    }

What is not working on the code? Does the device remain in ASCII mode? Do you get any exceptions in the catch block?

@martinzak-zaber, me either :( As I mentioned before, this C++ solution is very hard to debug; I can't simple set a breakpoint and trace ZMotor initialization code, should attach to process first but it's hard to catch a moment. It's not my code, I'll add some logging later to see what's happened. Yes, motor remains in the ASCII mode, and looks like later original code initialization raised an exception and failed :(
            conn_ = zaber::motion::binary::Connection::openSerialPort(com_port_);
            const auto device_list{ conn_.detectDevices() };


I can only guess that I'm doing something pretty dumb?

It seems your mkdocs.yml is something wrong. How about checking if the build passes on your laptop?
https://www.mkdocs.org/user-guide/configuration/#docs_dir

Feel free to reopen this issue if you still have a issue with this actions.


📝 — Document the LANGUAGE env var.

LANGUAGE env var wasn’t in the README, and you need it to use added dictionaries.

Thank you so much for this hint!
Time to get that merged?


v0.4 - Planned Features

If you have an opinion should look like ng-table and what features it should enter, please write it in the comments

I see some drag and drop requests here.  Here is an example of row drag/drop.  This depends on jquery-ui and ui-sortable.  http://plnkr.co/edit/A8H9t8?p=preview

+1 to horizontal scroll with fixed columns at the ends

https://github.com/esvit/ng-table/issues/189

1


feat(components): add Clipboard

Changes
bring in https://flowbite.com/docs/components/clipboard/ to flowbite-react

Summary by CodeRabbit

New Features

Introduced a new Clipboard component for copying text, data, or code snippets effortlessly.
Added variations of the Clipboard component with different icons and text configurations.
Included examples and usage instructions for the Clipboard component in the documentation.

Documentation

Added comprehensive documentation for the new Clipboard component, including usage scenarios and examples.

Style

Updated styles to support the new Clipboard component and its variations.

Hey @SutuSebastian --- can you please help me if I am doing any mistake? in storybook the 3 components I've created for Clipboard is working fine, but in docs it's not working correctly, can you help me debug? I am not able to identify why 2 Components:

With Icon
with Icon + Text

this 2 are not showing Copy Icon & Text..


Bump version of @types/node

I noticed it's still on 10.17 in places when I was trying to register a listener for an event added in 12.17 and intellisense wasn't picking it up.
Only updating those that were older than 12.17.  Happy to update others too though for whatever reason.
I wonder if we should try to keep this version and the version in .nvmrc in sync?  @tylerbutler , @curtisman - thoughts?

Old typing probably doesn't matter that much in general, as they are usually forward compatible.
But if we are updating, moving everything forward consistently would be good.

Old typing probably doesn't matter that much in general

True, but since I stumbled on a missing API I figured I'd do something about it.
Except YIKES more build errors.  Putting on hold for today at least...

PS - there's no @types/node at 12.18.3 (the version in .nvmrc) so I couldn't set it to match exactly.  @tylerbutler any objection to me bumping that too so they match, if I press on here?

PS - there's no @types/node at 12.18.3 (the version in .nvmrc) so I couldn't set it to match exactly. @tylerbutler Tyler Butler FTE any objection to me bumping that too so they match, if I press on here?

That's fine with me, but I would prefer bumping forward rather than back. I.e. 12.21.0, the latest v12 release.

Except YIKES more build errors.

Yeah, you'll probably hit a bunch of "value can be null or undefined" errors because the typings have improved.

In general,  typing package and node version isn't always one to one and I don't know if they actually following the same version scheme. To be safe,  we should bump to the closest older version so we don't accidentally depends on newer features

@markfields FYI @DLehenbauer took care of this in PR #5796.


Add cost per KWh to charges dashboard.

Added a new column to show how economic the charges at the location has been.
Not all sessions are equal i.e some charge by KWh, some by time. Additionally there could be a flat session charge.

Hi @adriankumpf
Wanted to check if I can get an estimate of when this could be addressed?
Thanks,
Jyothrilinga.

Superseded by #2693. Thank you anyway!


Add ability to merge extra (alpine/input/trigger/img/header) attributes instead of overwriting them

I noticed that when I used the ->extraAttributes() function twice on a component (for example indirectly inside a macro or function + directly using it when defining the component), that the old values would be overwritten. I didn't really expected this and I looked for a way to allow merging those attributes.
This PR introduces a second optional parameter that allows the developer to state that new attributes should be merged with the existing attributes. The merge is done using the ComponentAttributeBag, so that we mirror the default Laravel behaviour when merging attributes (e.g. merge classes, but overwrite type attribute).
I set this $merge = false parameter to false by default, since it would theoretically be a breaking change. Personally I'd like the parameter to be true by default, but that might be an idea for V3. :)
The only caveat is when using merging with existing attributes that are determined by a closure. I know directly evaluated those closures to merge the contents. There might also be a way around it, by constructing a custom Closure that evaluates the already existing attributes later, but that would probably quite increase the complexity of the code.
Thanks!

This would evaluate the extraAttributes value every time you'd pass a new value. Not sure if that's desirable performance-wise.

Performance-wise I don't think it's a huge issue, since the number of persons that will be using multiple closures in here will be probably very low. In addition, the default behaviour is still not to merge. So I think it's better to support a feature than not to support it just because there might be some very rare situations.
In addition, if people have closures that take a long time, that's probably more of an issue on their side.

I don't think this would work properly with closures, since they cannot be evaluated within setters.
This should be refactored so that we can store multiple sets of attributes (an array of arrays/closures) inside extraAttributes(). Then array_merge them all inside the getter, at once.

Thanks Dan, good solution. Will update the implementations to make use of that solution.

This one is ready for review @danharrin.
I chose not to use array_merge(), but use a temporary ComponentAttributeBag. This, because the ComponentAttributeBag has special behaviour on the merge() method, for attributes like class etc.

I also removed the Closure type from the protected properties. This theoretically could be a breaking change, because in theory people can extend the form components and have the protected array $attributes still overriden like protected array|Closure $attributes. I can also add keep the Closure type and mark that as ready for deletion in V3.

Thanks!


(UI) Craft Equipment Inplace conflict with skrwoor creative mode

Hello, as the title says, when in Craft Building UI with skrwoor creative mode we can build anything without any resources, but looks like (UI) Craft Equipment Inplace is forbid it so when we click the building on the UI it doesn't show any ghost building.

I don't understand. Craft Equipment Inplace has nothing to do with buildings.
Please provide the detailed reproduction steps, saves and the Player.log from %USERPROFILE%\AppData\LocalLow\MijuGames\Planet Crafter after you tried reproducing the issue.

Oops sorry! Just found out https://www.nexusmods.com/planetcrafter/mods/2/ was the problem, looks like it's not updated and also cut half of 60 FPS


Remove Data.Id and Type.Id and identity.ToId

@djspiewak I'm pushing a couple more comments to get rid of OldPrecise and Data.Binary. Hold off on merge until those are in so that we just have a single breaking release that encompasses all of this.

@djspiewak Data.Binary is a bigger can of worms, so I've pushed what I intend to push for this PR.

@djspiewak ready.


Switches in esp_matter_endpoint.h missing groups cluster

namespace dimmer_switch {
typedef struct config {
cluster::identify::config_t identify;
cluster::binding::config_t binding;
} config_t;
uint32_t get_device_type_id();
endpoint_t *create(node_t *node, config_t config, uint8_t flags);
} / dimmer_switch */
namespace color_dimmer_switch {
typedef struct config {
cluster::identify::config_t identify;
cluster::binding::config_t binding;
} config_t;
Should be...
It is also missing scenes, but I am not sure if that is needed.
Groups is required because a switch can be bound to a group of lights.
namespace dimmer_switch {
typedef struct config {
cluster::identify::config_t identify;
cluster::groups::config_t groups;
cluster::binding::config_t binding;
} config_t;
uint32_t get_device_type_id();
endpoint_t *create(node_t *node, config_t config, uint8_t flags);
} / dimmer_switch */
namespace color_dimmer_switch {
typedef struct config {
cluster::identify::config_t identify;
cluster::groups::config_t groups;
cluster::binding::config_t binding;
} config_t;

The groups cluster is optional. In the default <devce_type>::create() APIs, only the mandatory clusters are added. The endpoint_t handle can then be used to add any optional clusters, for example groups::create() in this case.
(This is also true for any clusters and its features; only the mandatory attributes and commands are added.)

Actually, the optional groups cluster is being added currently: https://github.com/espressif/esp-matter/blob/main/components/esp_matter/esp_matter_endpoint.cpp#L216. Since client clusters don't have any attributes, the config for groups this is not needed in the switch config.
But since this is optional, I think it should be removed from here.
(From this, it seems like there should be some documentation in the header file too, mentioning which clusters are actually being added.)

The goal is to get this use case working....
https://github.com/project-chip/connectedhomeip/issues/21243
To do that the Switches need to be group clients and the lights group servers.

You can enable any of the optional clusters/attributes/commands as per your usecase in the application code. The provision to handle them would be made in the SDK but they won't be added to the default cluster APIs.

Nice to have a paragraph in esp-matter docs explaining that you don't differentiate between client and server clusters. In your model the full client and server API is always available and then the linker decides whether to include the methods or not.  This will need to be explained to people coming off from SiLabs where since they will be searching for client/server modes.

Add the bit about client/server to the docs?

Yes, will see if we can add the documentation at some appropriate location. Closing the original issue as it is addressed.


k8s-image: support read/write from deployed cfs pod

add cfsctl into the docker image
add instructions to read/write

/cc @yunxing @yutongp


Fix gamification PNG Error on NGINX

FIX https://github.com/PrestaShop/PrestaShop/issues/17038

issue with PS<IP_ADDRESS> & PS<IP_ADDRESS> & the gamification module v2.3.2.

please add <IP_ADDRESS>

Please test that now everything is fine with Apache and Nginx.

Random CI error?

Hi and thanks for your contribution.
We can't accept your pull request because if something is working on Apache, it should work on Nginx.
More, if you use a / you can't add your shop in a subdirectory, the / will refer to the root URL.

Hi @davidglezz,
In the BO => Shop Parameters => Merchant Expertise page, all the images are not displayed anymore. (checked with Nginx server and apache server).
With PR:

Without PR:

Thanks to check it.

Hello @davidglezz
Thanks for this contribution, but since we found an error with this pull request we can't validate it.
Feel free to re-open this pull request when you have time to correct the error.

Hello!!
I request that the PR be reopened.
I have verified that it works. (today with <IP_ADDRESS>)
After:

My arguments in favor of the PR

Do not use relative path
_MODULE_DIR_ are generated by ps with user config

If not subdir:  /modules/gamification...
if subdir: /subdir/modules/gamif... because _MODULE_DIR_ = '/subdir/modules/'

_MODULE_DIR_ are used in a lot of modules,
If it does not work in this module, it should work in the other modules that use it.

@davidglezz I agree with the usage for _MODULE_DIR which is more relevant

Hi. you have forgotten to change the label to "waiting for QA" or the one that corresponds.

Hi @davidglezz,
I tried with 1767 on Nginx server => ok
https://www.awesomescreenshot.com/video/799046?key=6b1dffb6d5884596fe45b19340db7374
But, I tried with the PR on 177x branch => NOK

I deleted the cache manually, I cleared the cache from the Performance page and I used an incognito browser => Same issue.
Thanks to check and feedback.

@khouloudbelguith This should not have different behaviors with different versions of Prestashop.
Just use a constant that exists since 1.5, 1.4 or earlier.
Did you check the generated url? as I show in a screenshot https://github.com/PrestaShop/gamification/pull/72#issuecomment-660522885
did you try to access the url of the image?

Hello @davidglezz,
TEST OK on 177x
https://drive.google.com/file/d/1vuZv2xEjOMkve6uQ_KKd6ih6T8wo_xU9/view
Thank you

Thanks @davidglezz !


Add icons for twitter, facebook and stackoverflow in footer

Previously:

This pull request:

I could not add for Gitter because the css specified for Gitter in _sass/_icons.scss is incompatible with the other icons.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

I looked around for getting the Gitter icon to work. However the files at fonts/fontawesome-webfont.* are outdated.
I didn't want to update that since I don't know the licensing agreement this project has with Font Awesome. The pro package has significantly more icons available but that costs money.


Backspace icon

Icon Request

Icon name: backspace

Use case: For on-screen keyboards like wordle games:

Screenshots of similar icons:

There already is a delete icon! :)

@BinaryKitten good idea!

Great, thanks :)

@BinaryKitten good idea!

thanks for the kudos - not sure why i got it though.
good idea @hornta  :+1:


Add hardware-accelerated codecs for DEFLATE and LZ4

Description
Adds hardware-accelerated DEFLATE and LZ4 compression codecs for stored fields. The hardware in focus here is Intel (R) QAT, which is an integrated, built-in accelerator on the latest 4th and 5th Gen Intel Xeon processors. The implementation relies on the Qat-Java library.
The PR adds two additional valid values for index.codec:  qat_deflate and qat_lz4. It also introduces a new setting, index.codec.qatmode, that specifies the mode of execution for QAT.
Two values are supported for index.codec.qatmode: hardware and auto. A hardware execution mode uses only the QAT hardware, while an auto execution mode may switch to software if hardware resources are not available.
Closes
https://github.com/opensearch-project/custom-codecs/issues/130
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@mulugetam thanks for raising the PR. Could you please share some performance numbers for these modes?

@mulugetam thanks for raising the PR. Could you please share some performance numbers for these modes?

Here are some performance numbers for indexing using stack overflow workload

| qat_deflate relative to deflate | qat_lz4 relative to default
-- | -- | --
Total time | -11.4% | -2.5%
Mean Indexing throughput | 23.7% | 3.4%
Store Size | 2.5% | -12.5%

Thank you @asonje. @sarthakaggarwal97 we will also share the performance numbers for search when they're ready.

thanks @mulugetam @asonje for initial numbers. Out of curiosity, if the underlying algorithm is still same (in this case lz4, zlib), how are we seeing differences in store size?

Out of curiosity, if the underlying algorithm is still same (in this case lz4, zlib), how are we seeing differences in store size?

This is expected. As you know the store size will vary from run to run but it is still a good approximation of compression ratio.  qat_deflate  and deflate will have different compression ratios  because even though the algorithm is the same, their implementations are different. The resulting compressed bytes will not be identical in both cases even though they are compatible.

@mulugetam it looks pretty cool, could you please share what arch/oses it is available? (the arch part is somewhat clear, but not arch + os combinations, windows / linux / intel macs, ...)

@mulugetam it looks pretty cool, could you please share what arch/oses it is available? (the arch part is somewhat clear, but not arch + os combinations, windows / linux / intel macs, ...)

The QAT built-in accelerator is available on 4th and 5th gen Intel (R) Xeon Processors. This version requires amd64/Linux. For all other systems, the auto mode can be used to do a software-only compression.

@mulugetam when you have chance, could you please resolve the conflicts? thank you

@mulugetam when you have chance, could you please resolve the conflicts? thank you

Looks like it's asking me to insert new lines because existing code is not spotless formatted.

LGTM to me, I have nothing else, thank you @mulugetam ! Coudl you please create a documentation issue for 2.14 to document this new codec, in particular have these topics covered:

highlight CPU arch and OSes
restoring the snapshots on CPU archs where hardware acceleration is not available

Thank you.
[1] https://github.com/opensearch-project/documentation-website

@mulugetam have you tried running a long running tests with both QAT codecs? It would be good to have long running tests to foresee any corruption or potential memory leaks. In these tests, we can keep the indexing / updates going on, with optional search requests (which accesses stored fields). We can have a single data node configuration, and an index with multiple shards (so that we don't hit 2B document limits)

@asonje

As you know the store size will vary from run to run but it is still a good approximation of compression ratio.
Did we do a force merge to one segment before we captured the store size. If not, we can do it to know the approximate impact of QAT on store size.

Would you please help with the decompression perf numbers as well with top_hits aggregation query. This query is infamous for pointing out regressions over stored fields

Did we do a force merge to one segment before we captured the store size. If not, we can do it to know the approximate impact of QAT on store size.

Not initially. I have re-run the tests with force merge and max_num_segments=1 and updated the table

@mulugetam have you tried running a long running tests with both QAT codecs?

@sarthakaggarwal97 We have run OpenSearch-Benchmark multiple times, with both qat_deflate and qat_lz4, on a four-data-node setup, and have seen no issues.  How long is the duration you are suggesting we run?

@mulugetam have you tried running a long running tests with both QAT codecs? It would be good to have long running tests to foresee any corruption or potential memory leaks. In these tests, we can keep the indexing / updates going on, with optional search requests (which also accesses stored fields). We can have a single data node configuration, and an index with multiple shards (so that we don't hit 2B document limits early on)

@sarthakaggarwal97 Can we contribute the recommended search query for such testing to https://github.com/opensearch-project/opensearch-benchmark-workloads/

@mulugetam During the memory leak for Zstd, we observed a significant change in system memory (not JVM since most of the execution is native). Maybe we can do a nightly run to see if we have any such affects? I am open to your opinion on this!

@sarthakaggarwal97 @reta Thank you for bringing the issue of memory leaks to our attention! We have been running a single-node OpenSearch process for hours while monitoring the system memory utilization. We do not see a significant difference in system memory utilization between best_compression vs qat_deflate (same for lucene_default and qat_lz4). The below screenshot, for example, shows the %memused when multiple cycles of stackoverflow dataset are submitted for indexing.

Also, and more importantly, we have been testing the qat-java library using valgrind and other static code analysis tools and have not seen memory leaks.

@reta are we waiting for additional approvals?

@reta are we waiting for additional approvals?

Yes, would be good to have @sarthakaggarwal97 signoff , thank you @mulugetam !

@mulerm @sarthakaggarwal97 @reta
I've a few suggestions on this PR, and future changes in new custom codecs:

Before enabling these as generally available, can we introduce a feature flag which can allow us to mark new codecs as experimental? Each new codec can implement an isExperimental method, and there can be a boolean feature flag opensearch.experimental.feature.experimental_codecs.enabled which users can choose to enable if they would like to try this out?

Based on the PR my understanding is that the storage format does not change. I think in future, codec like lz4_qat should be inferred, and continue to work with other implementation of LZ4 in the example case. We should add tests to ensure that the storage remains compatible always if we're adding changes which impact only compute (e.g. use lz4_qat for compression, and lz4 for decompression)

@mgodwan thanks for raising these points. I agree, we should have a experimental feature flag to let the new codecs bake for some time before making them available by default.
@mulugetam other than that, one minor comment to check upon range of the compression levels.

Before enabling these as generally available, can we introduce a feature flag which can allow us to mark new codecs as experimental? Each new codec can implement an isExperimental method, and there can be a boolean feature flag opensearch.experimental.feature.experimental_codecs.enabled which users can choose to enable if they would like to try this out?

@mgodwan one of the reason why we have a custom-codecs plugin is to roll out experimental / stable codecs on opt-in fashion. The feature flags are specifically targeting OpenSearch core, because it is not really feasible to alter the flows in any other way, in contrast plugin could be uninstalled completely to phase out functionality. I think we should not introduce any feature flags for plugins.

in contrast plugin could be uninstalled completely to phase out functionality

Uninstalling plugin means not being able to read segment data written using the codec introduced.
Given the codec leads to different way to store the data, we need to ensure backwards compatibility is maintained in any new future release on how we deal with persisted format/metadata. While this is an opt-in through plugin, not having this as experimental forces us to do any changes in a backward compatible fashion where as keeping it experimental for say a couple of release cycles will allow us to see how it is being adopted and allow to make necessary changes without worrying about breaking existing production data for users (as experimental features add that warning by design).
Hence, I was proposing to add the feature flag as suggested (The feature flag itself can lie in core if we follow the interface to allow for an implementation of codec to be an experimental through an interface).

Hence, I was proposing to add the feature flag as suggested (The feature flag itself can lie in core if we follow the interface to allow for an implementation of codec to be an experimental through an interface).

Any plugin has to be  backwards compatibility  irrespectively of feature flags, if we are not sure that the plugin is stable enough - we should document it as experimental one, so people will use it knowing things could break. Feature flags are not applicable to plugins, however to your point, we could provide fine grained control over enabling / disabling a particular codec, this could be done using plugin settings.

I'm inclined towards having a forcing function in code rather than relying on the documentation to mark as experimental. If this can be achieved using plugin settings, that should be fine as well. At the end, feature flags themselves are modeled as settings only :)

I'm inclined towards having a forcing function in code rather than relying on the documentation to mark as experimental.

We need both I believe, the settings could be introduced as a separate change (to cover existing ZSTD as well), please create an issue so someone could pick it up, thank you.

2. Based on the PR my understanding is that the storage format does not change. I think in future, codec like lz4_qat should be inferred, and continue to work with other implementation of LZ4 in the example case. We should add tests to ensure that the storage remains compatible always if we're adding changes which impact only compute (e.g. use lz4_qat for compression, and lz4 for decompression)

Thanks. I believe some of us discussed similar ideas on Slack. An issue entry was also created: https://github.com/opensearch-project/custom-codecs/issues/130.

@reta do we have any pending issues that need to be resolved?

@reta do we have any pending issues that need to be resolved?

@mulugetam we still need a signoff https://github.com/opensearch-project/custom-codecs/pull/122#issuecomment-2103374633

We should add tests to ensure that the storage remains compatible always if we're adding changes which impact only compute (e.g. use lz4_qat for compression, and lz4 for decompression)

@mulerm I think we can add this test to ensure the only change would be in the compute.

@mulerm I think we can add this test to ensure the only change would be in the compute. What do you think @reta?

@sarthakaggarwal97 I am not sure we could pull it off, fo  lz4_qat we need a real hardware support, right? I don't think it is available in GA.

@reta @sarthakaggarwal97
Adding the suggested test in the current implementation is not sufficient and probably will not work, as the fieldsWriter and fieldsReader of Lucene  store BEST_COMPRESSION and BEST_SPEED as the value of the MODE_KEY attribute.
My thinking was that we should treat qat_deflate (vs. BEST_COMPRESSION) and qat_lz4 (vs. BEST_SPEED) as separate codecs until such a time when BEST_COMPRESSION and BEST_SPEED would just use the accelerator "transparently", if it is present.

My thinking was that we should treat qat_deflate (vs. BEST_COMPRESSION) and qat_lz4 (vs. BEST_SPEED) as separate codecs

Certainly +1 to that

I am not sure we could pull it off, for lz4_qat we need a real hardware support, right? I don't think it is available in GA.

If this is the case, I would highly advocate for this to be behind a plugin setting. https://github.com/opensearch-project/custom-codecs/issues/148

If this is the case, I would highly advocate for this to be behind a plugin setting. #148

But it is separate codec already, which users have to opt-in to use (not a default one)? So users have to pick a code AND set a setting to use it?

Given the precedent we have had with Zstd, I think its okay to keep the new QAT codecs as experimental for now.
Currently, there is not a clean way to mark the codecs as experimental, and I have opened an issue https://github.com/opensearch-project/OpenSearch/issues/13723.
With that, it would be nice if we can come up with a plan as well to make new codecs, here QAT, generally available in the future. Let me think back on it. One of the list I created earlier for Zstd was this for reference: https://github.com/opensearch-project/OpenSearch/issues/9502

Given the precedent we have had with Zstd, I think its okay to keep the new QAT codecs as experimental for now.

@sarthakaggarwal97 we have https://github.com/opensearch-project/custom-codecs/issues/148 to address the problem for every custom codec.

@sarthakaggarwal97 we have #148 to address the problem for every custom codec.

@reta @sarthakaggarwal97 Are we on hold now until #148 is implemented? I think we should not be, as it is a separate codec that users would have to opt in to use.

@reta I think we would need to introduce the experimental settings in OpenSearch, since we do the validation of index codecs in EngineConfig
Ideally would want to stop the creation of index only if the codecs is not experimental. I feel we would need a two changes. A new method in CodecSettings can tell us if the codec is experimental or not, and a feature flag setting will tell us whether we should make the experimental codecs available or not.

Thank you @mulugetam for this change.

@mulugetam would you please help with the backport as well? Thank you

@reta I think we would need to introduce the experimental settings in OpenSearch, since we do the validation of index codecs in EngineConfig

@sarthakaggarwal97 it would be great but we validation logic won't help us here I think: the codecs are registered by Apache Lucene SPI (the validation logic you are referring to only helps with ensuring the codec settings validness).

Ideally would want to stop the creation of index only if the codecs is not experimental. I feel we would need a two changes. A new method in CodecSettings can tell us if the codec is experimental or not, and a feature flag setting will tell us whether we should make the experimental codecs available or not.

That's one of the problems: we could extend CodecSettings  (this is ours) but none of the Codec implementations are required to implement it. Ideally, Apache Lucene SPI should have that feature, or alternatively, we could disable some codecs through settings.

the codecs are registered by Apache Lucene SPI (the validation logic you are referring to only helps with ensuring the codec settings validness).

I think a check like this will definitely help to disable its usage in the write path. For write path, NamedSPI interface is not used.

@mulugetam would you please help with the backport as well? Thank you

@reta @sarthakaggarwal97 https://github.com/opensearch-project/custom-codecs/pull/150


[BUG] npm init cloudflare fails

Is there an existing issue for this?

[X] I have searched the existing issues

What version of workers-rs are you using?
main
Describe the bug
when I do:
npm init cloudflare project_name worker-rust

I get an error:
Need to install the following packages:
<EMAIL_ADDRESS>Ok to proceed? (y) y

Error: EXDEV: cross-device link not permitted, rename '/tmp/2122340072f1d-91d055235fe5d/worker-rust' -> '/home/xxx/project_name'

npm ERR! code 1

Steps To Reproduce
linux 6.0.11-arch1-1
npm 9.2.0
run npm init cloudflare project_name worker-rust

Any workaround for this?

I had the same issue. What I did to get past this, was a manual copy of the template. It was already downloaded by running npm init.
An example using the above error output:
mv /tmp/2122340072f1d-91d055235fe5d/worker-rust -> /home/xxx/project_name

Another thing that is happening when running this command not exactly related to this issue but in the similar domain
npm init cloudflare project_name worker-rust
is that it just creates js default template instead of Rust one.
Anyone experiencing this as well ?

@frontmesh Yes, I'm also experiencing this...


unknown receiver 'GPlayerPlugin'

When I try to run on IOS I get this error
Xcode build done.                                           12.3s
Failed to build iOS app
Error output from Xcode build:
↳
    2019-11-17 14:17:55.694 xcodebuild[6536:8733356]  DVTAssertions: Warning in
    /Library/Caches/com.apple.xbs/Sources/IDEFrameworks/IDEFrameworks-14490.122/IDEFoundation/Provisioning/IDEProfileQualification.m:509
    Details:  Xcode could not determine type of profile "/Users/abozanona/Library/MobileDevice/Provisioning Profiles/ce304866-1719-43d0-871b-a8b45caeb42c.mobileprovision" (could be {(
    )})
    Object:   <IDEProfileQualification: 0x7fc5a0208580>
    Method:   -_profileTypeQualifier
    Thread:   <NSThread: 0x7fc598d1e500>{number = 2, name = (null)}
    Please file a bug at https://bugreport.apple.com with this warning message and any useful information you can provide.
    2019-11-17 14:17:55.701 xcodebuild[6536:8733356]  DVTAssertions: Warning in
    /Library/Caches/com.apple.xbs/Sources/IDEFrameworks/IDEFrameworks-14490.122/IDEFoundation/Provisioning/IDEProfileQualification.m:509
    Details:  Xcode could not determine type of profile "/Users/abozanona/Library/MobileDevice/Provisioning Profiles/ce304866-1719-43d0-871b-a8b45caeb42c.mobileprovision" (could be {(
    )})
    Object:   <IDEProfileQualification: 0x7fc59eef0690>
    Method:   -_profileTypeQualifier
    Thread:   <NSThread: 0x7fc598d1e500>{number = 2, name = (null)}
    Please file a bug at https://bugreport.apple.com with this warning message and any useful information you can provide.
    ** BUILD FAILED **

Xcode's output:
↳
    /Users/abozanona/Desktop/land4moviesflutterapplication/ios/Runner/GeneratedPluginRegistrant.m:34:4: error: unknown receiver 'GPlayerPlugin'; did you mean 'GplayerPlugin'?
      [GPlayerPlugin registerWithRegistrar:[registry registrarForPlugin:@"GPlayerPlugin"]];
       ^~~~~~~~~~~~~
       GplayerPlugin
    In file included from /Users/abozanona/Desktop/land4moviesflutterapplication/ios/Runner/GeneratedPluginRegistrant.m:8:
    /Users/abozanona/Desktop/land4moviesflutterapplication/ios/Pods/Headers/Public/gplayer/GPlayerPlugin.h:3:12: note: 'GplayerPlugin' declared here
    @interface GplayerPlugin : NSObject<FlutterPlugin>
               ^
    1 error generated.
    note: Using new build systemnote: Planning buildnote: Constructing build description

Could not build the application for the simulator.
Error launching application on iPhone X.

the description of the plugin says no iOS :( i wish it did work, the VLC alternative does work on iOS but has a drawback of not allowing multiple video players on screen at once. I hope this will have the ability to do so

@moeiscool @tcking Thanks for pointing that out. What if I want to use GPlayer on android and VLC player on ios. In this case I need to install both dependencies, use GPlayer on android & use VLC on IOS.
But in this case, even though I'm not calling GPlayer from IOS, it's raising an exception.
It'd be great if adding GPlayer as dependency on IOS won't raise any error.


How to record a screen or desktop using this tool?

Also interested in this.
Wanted to clarify if this question was left unanswered because the feature is not currently available (on Windows, it is done using ffmpeg -f gdigrab -i desktop), or because nobody bothered to answer?
@GeraldEugenio have you found any FFmpeg C# wrapper that supports screen recording?

https://github.com/cmxl/FFmpeg.NET/issues/7
does this answer your question?

Yep, but it turns out ffmpeg is very inefficient because gdigrab is an old gen tech. These days the desktop is in DWM DXGI buffers, and there's an efficient API in modern Windows SDK with less options but actually much more practical: https://github.com/MicrosoftDocs/SimpleRecorder/


fix(instrumenter): skip as expressions

Don't instrument TS as expressions as this can result in invalid code.
Fixes #2465

Thanks! I assume this will be in beta 6, so I can remove my temporary workarounds?


chore(l10n): update translations from Crowdin

i'm not sure why the base translation file was modified here, closing this PR so that crowdin can try again


Extract iteratee functions statically

Iteratee functions passed to parallel should be extracted and registered as static functions.

Implemented in babel-plugin-parallel-es


UX not built to spec

For issues related to the UX not built to spec

Closing this epic and using label "Type: UX Quality" moving forward instead.


Switch to latest ruby-graphql and better error handling

Problem
When using Graphlient from Exchange, where we use latest graphql-ruby, we noticed Graphlient raises error when trying to generate human readable string from error results in some cases.
Cause
problems was not available in error response so https://github.com/ashkan18/graphlient/blob/d583542beebb27192e369f800cd59b11735e68ba/lib/graphlient/errors/graphql_error.rb#L34 would fail when calling .map on nil.
Solution
We now check if problems exist we use the old approach, otherwise we try to generate string using path on root level of error and otherwise we fallback to just returning message.
Other Changes
As part of this, we updated are DummySchema to latest graphql-ruby where there are few new changes.
http://graphql-ruby.org/schema/class_based_api
As part of this upgrade now graphql fields are all converted to camel case, so fee_in_cents in all queries is now feeInCents while internally we can still access it with fee_in_cents.

If the graphql-ruby gem introduced a breaking change but we want graphlient to be compatible with multiple versions of it, there should be a build matrix for running tests against multiple versions of graphql-ruby to ensure compatibility.

@dblock yes will add test specially for the case that lead to this PR :)
@yuki24 this was not really a breaking change, they still support the old style but in the new style (and later versions) they are doing things that are more aligned with GraphQL best practices. Also Graphlient is not really designed to just work with graphql Ruby, it's a client for working with any GraphQL based API. We only use graphql ruby to create a dummy schema for the test so we may be good here. This PR mainly adds another generic way of dealing with GraphQL errors.

👍


Add Steep

Hi @mre
Could you count this as a part of the Hacktoberfest by adding hacktoberfest-accepted label to it?
Ref: https://hacktoberfest.digitalocean.com/hacktoberfest-update

Thanks for the PR @aliariff. Much appreciated. 😊

Could you count this as a part of the Hacktoberfest by adding hacktoberfest-accepted label to it?
Sure thing. Added the label. Let me know if I did something wrong. I actually was of the opinion that adding the hacktoberfest label to the project would be enough. 🤔
From your linked article:

PRs count if:
Submitted during the month of October AND
Submitted in a public repo AND (
  The PR is labelled as hacktoberfest-accepted by a maintainer OR
  Submitted in a repo with the hacktoberfest topic AND (
    The PR is merged OR
    The PR has been approved
  )
)

So I think merging the PR would have been enough. 😉
Anyway, better safe than sorry. Thanks for the PR. 👍

@mre Ah yeah you are right, I just saw that this repo is using the hacktoberfest topic. Thank you!


How to fetch records based on array of ID's with graphql API

Which Category is your question related to?
API
What AWS Services are you utilizing?
Cognito, AWS AppSync, graphql API
Provide additional details e.g. code snippets
I have array of todoID how do I fetch only records matching these ID's.
Currently I fetch everything and filter.
const allTodos = await API.graphql(graphqlOperation(listTodos));

Hi @apoorvmote, there are a few ways to accomplish this. The simplest is to pass in a filter expression to the list operation like this:
query FilteredLIst {
  listTodos(filter: {
    or: [
      {
        id: {
          eq: "id1"
        }
      },
      {
        id: {
          eq: "id2"
        }
      }
    ]
  }) {
    items {
      id
      title
    }
  }
}

This will save you a bit of bandwidth over the wire, but it will still cost you an entire scan of the ddb which can be quite expensive as the number of entries grows.
The recommended approach to perform this query efficiently (ie without a scan) is to have a GSI that correlates all of the todos you want to fetch. For example, say you had a "Project" entity that had multiple "Todo"s in it. Instead of storing all of the Todo IDs as a list in the Project, you could store the project ID as a field in the Todo and make that field the primary key of a GSI:
type Project @model {
  id: ID!
  name: string!
  tasks: [Todo!]! @connection(keyName: "byProject", fields: ["id"])
}

type Todo @model @key(name: "byProject", fields: ["projectId"]) {
  id: ID!
  title: string!
  description: string
  projectId: ID!
}

This way, when resolving the list of tasks in the project, a query (not a scan) is performed on the Todo table against the "byProject" GSI. You can read more about the @key and @connection directives here

Hi @edwardfoyle. What would you suggest if I wanted to get all of the Todos for multiple projects? So instead of the query being:
type Query {
    byProject(projectId: ID!): ModelTodoConnection
}

type ModelTodoConnection {
  items: [Todo]
  nextToken: String
}

it was:
type Query {
    byProjects(projectIds: [ID]!): ModelTodoConnection
}

Is there a better way to do this than just having a lambda resolver that loops through the byProject query?

@jamesalester If the client is specifying projectIDs on the fly, you would have to either have the client issue multiple byProject() queries or have a lambda resolver as you mentioned. If the projects that you are querying are known in advance though (such as "all projects in an organization" then you could include another field in the todo table such as "organizationID" and have a GSI on that field so you could query Todos "byOrganization"

Thanks @edwardfoyle. I really appreciate the answer!

Hi,
Do you know if there is a trick with Amplify to filter like this :
{filter: element hasOneOf : ["102265475","102265476" ]}
It's like a reversed "contains" that I'm trying to do.
Because for now the only solution I found was to dynamically draw the json filter, or call a query for every element of the array.


Clicking on link icon when selected text is already a link does not show the current URL

Hi,
Once a selected text has been converted into a link using the link icon, there is no way to know what the link URL was and modify it.
When I select text that is already a link and I click on the link icon, the prompt that asks what link to use has a blank content. I would expect it to be pre-filled with the current link content.
Would it be possible and not too difficult to implement this?
Thanks in advance

OK thanks @algorys for your tips, in the meantime I was trying to do it, here is my first implementation: (inspired from https://github.com/anuradhaspidy/widgeditor/issues/21)
`{
name: 'link',
title: 'Add a link (on selected text)',
result: function() {
var selection = document.getSelection();
if (selection == "")
{
alert("Please first select the text you want to link.");
return;
}
var range = selection.getRangeAt(0);
    var foundLink = false;
    var defaultURL = "https://";
    var emptyLinkPromptText = "Write here the link address (starting with http:// or https://) for selected text";
    var existingLinkPromptText = "Write here the link address (starting with http:// or https://) to replace on selected text";
    var promptText = emptyLinkPromptText;

    var linkNode = null;
    var rangeNodes = Array.from(range.cloneContents().childNodes);
    var sz = rangeNodes.length;
    for (var i = 0; i < sz; ++i){
      if (rangeNodes[i].nodeName.toLowerCase() == "a"){
        foundLink = true;
        linkNode = rangeNodes[i];
      }
    }
    if (!linkNode){
      var parentNode = range.commonAncestorContainer;
      var currentNode = parentNode;
      while (!foundLink && currentNode && currentNode.nodeName){
        if (currentNode.nodeName.toLowerCase() == "a"){
          foundLink = true;
          linkNode = currentNode;
        }
        else {
          currentNode = currentNode.parentNode;
        }
      }
    }

    if (foundLink){
      defaultURL = linkNode.href;
      promptText = existingLinkPromptText;
    }
    var url = window.prompt(promptText, defaultURL);
    if (url){
      pell.exec('createLink', url);
    }
  }
}`

It depends on what you want to do next, I stayed on something very simple because the goal of this editor is to be as light as possible.


Add support for Badge widget in the NavigationRail  widget

When https://github.com/flutter/flutter/issues/92257 is fixed by https://github.com/flutter/flutter/pull/99853.
Proposal
Add support for the Badge widget in the NavigationRail  widget

New Badge would have a child. No specific support is required for NavigationRail `. Closing.


Update package.json details, rename module to ghost-admin

no issue

updates package.json details to better reflect the separation from the Ghost package
update ember config and all import statements to reflect the new ghost-admin module name in package.json

Rebased and green 😄


Funding?

Hi I'm curious. Is anyone funding the current development?
How big is the team?

@dpriskorn My research group has a decent budget. The core developers are currently @joka921 and myself, as you can see from the commit messages. Beyond that there are several students (which come and go, but several of them also stay with the projects for a number of years) working on important subproblems and doing great work.
As already mentioned in https://github.com/ad-freiburg/qlever/issues/622: "We have a great team of people here, but it would be even greater to get help from the community. Where by help I mean very concrete help in the form of well-versed C++ programmers who would love to work on a project like this (I am sure that there are quite a few of those out there, we just need to connect to them)."


#898 cancel button in camera select of Scanning with barcode pausing the camera

solved issue #898

@shivanshsoni already fixed and merged #883

Hi @shivanshsoni, the issue is already solved I'm afraid


[BUG] Some basic gremlin queries are slow.

Describe the bug
Some gremlin queries are very slow, which are supposed to be very fast.

[x] g.V().limit(1)  # Sometimes 1 min, sometimes < 1s.
[ ] g.V().count() # timeout after 10 min, 900 million vertices
[ ] g.E().count() # cost 56s, 30 million edges

limit seems has been fixed by some recent commits;
count may need to refactor the GlobalGraphQuery implementation in store.

count may need:

maintain a meta info of number of vertices, number of edges in store, rather than count() each time;
fuse source and count in plan in ir-core;
support query for source+count in ir-runtime

Fixed by a flurry of PRs. Waiting to be confirmed by real life workloads.


Update Copyright in order to maintain consistency in OpenSearch clients

What is the issue?
The copyright message is in this repo is inconsistent with python's copyright message. A file NOTICE.txt also needs to be added.
I attempted to do this myself, however I did not have permissions to push to a protected branch that I created. Due to my attempts to make these changes myself, the branches have been populated with guiangumpac-update-copyright-patch and update-copyright. To maintain a clean set of branches, these two branches should be removed.

Done as part of #62


Fix fdroid build

It runs successfully in the GA CI. I reckon it should work in the fdroid pipeline too.
I still went for the script approach because it seemed fairly straight-forward.
merged f-droid companion: https://gitlab.com/fdroid/fdroiddata/-/merge_requests/10190

The f-droid companion has been merged. The pipeline run successfully there, with the encointer-wallet beta branch containing these changes.


python3Packages.angrop: remove postPatch

Motivation for this change
Changes are no longer needed and were stale.
Things done

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

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

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

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

[x] Fits CONTRIBUTING.md.

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

2 packages built:

python38Packages.angrop
python39Packages.angrop

Result of nixpkgs-review pr 125767 at d05bacf1 run on aarch64-linux 1

2 packages failed to build:

python38Packages.angrop
python39Packages.angrop

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.


Enforcing thread safe implementation of RandomNumberGenerator for V3Hasher

Addresses https://github.com/NuGet/Engineering/issues/2265.
The RandomNumberGenerator.GetBytes is not marked as thread safe. The default implementation of that abstract class is thread safe.
This change enforces it.

Did you ever consider just creating a new RandomNumberGenerator every time that method is called?
Why do we need to reuse the same instance across threads?
Is there a high cost to setting up the object?
If possible, I would prefer we just keep things simple and avoid the issue of thread-safety entirely.

@scottbommarito, I didn't. This code is borrowed from some other project and was security reviewed, I didn't want to make any functional changes to it.


Calling TitleFormatter On Week Change

Hi,
I currently have a custom title formatter and I am using the WEEKS view for the calendar. The custom title formatter I am using lists the current week (as well as the month), however the title formatter's format function is only called when the month changes, not when the week changes.
Is there anyway to have it be called everytime the week changes?

Hello? I am still having this issue, and it is preventing me releasing a new feature in my app.
If I need to, I am willing to make a PR, I'll just need some pointers on the location where the interface is called.


[ovnController] Handle nicMappings with dashes

Currently it do not work if nicMappings contain
dashes as port name is wrongly detected, like
for nicMappings: {"dpdk-data0": "ospbr"}, bridge
name get's "br-dpdk-data0" while port name as
"data0" instead of "dpdk-data0", this patch fixes
it.
It got missed while fixing other issue[1].
Also fixed some syntax errors and updated kuttl tests to include this case.
[1] https://github.com/openstack-k8s-operators/ovn-operator/pull/248
Related-Issue: OSPRH-7130
Related-Issue: OSPRH-3124

/hold until after beta freeze


Improve client side error handling

With this PR #1157 better error handling can be done in the web and command line clients.
The client api calls should be rechecked for proper error handling as @whisperity already mentioned, like in this case:
CodeChecker cmd runs --url whisperity
Internal server error on localhost:8001
Traceback (most recent call last):
  File "[...]/CodeChecker.py", line 101, in main
    args.func(args)
  File "[...]/libcodechecker/cmd/cmd_line_client.py", line 119, in handle_list_runs
    for run in runs:
TypeError: 'NoneType' object is not iterable

Note! The client properly handles this. (It says "product cannot be used because it does not exist" and then quits.) I purposefully commented this handling out, to see the error returning from the server.
Most parts of the client is written preemptive: we check if the product/run exists, before actually querying from said product/run.


database error

database error:
facing start the database error first while running the loginpageframe

for this you need to set use ssl to false in the jdbc connection stiring


run  time

It's been running all night with no results
def fetch_GPT_response(instruction, system_prompt, chat_model_id, chat_deployment_id, temperature=0):
print('Calling OpenAI...')
print("1.4\n")
response = openai.ChatCompletion.create(
temperature=temperature,
deployment_id=chat_deployment_id,
model=chat_model_id,
messages=[
{"role": "system", "content": system_prompt},
{"role": "user", "content": instruction}
]
)
print("1.5\n")
if 'choices' in response
and isinstance(response['choices'], list)
and len(response) >= 0
and 'message' in response['choices'][0]
and 'content' in response['choices'][0]['message']:
return response['choices'][0]['message']['content']
else:
return 'Unexpected response'
after print("1.4\n"), the operation is stuck. No result

Hi @yangyangyang-github
This is a typical openai API call.
If the function is going in an endless loop, can you please double check your openai credentials that was provided in the config.yaml file?
Also, we have already put guardrails in the api call function to stop making calls after some sufficient time using retry module. Please refer here. Hence, if you are using the same functionality, it should not go to an endless loop.
Let me know how things turn out for you.

Hi @yangyangyang-github This is a typical openai API call. If the function is going in an endless loop, can you please double check your openai credentials that was provided in the config.yaml file? Also, we have already put guardrails in the api call function to stop making calls after some sufficient time using retry module. Please refer here. Hence, if you are using the same functionality, it should not go to an endless loop. Let me know how things turn out for you.

I have added openai.api_key parameters to the file. What exactly is the openai credentials that was provided in the config.yaml file? May I have a look, please?

You should have a file named '.gpt_config.env' and store it in your $HOME path. Content of the file should be in the following format:
API_KEY=
API_VERSION=
RESOURCE_ENDPOINT=

Isn't it convenient for you to release this file?
------------------ 原始邮件 ------------------
发件人:                                                                                "BaranziniLab/KG_RAG"                                                                                @.>;
发送时间: 2024年2月28日(星期三) 中午1:38
@.>;
<EMAIL_ADDRESS>主题: Re: [BaranziniLab/KG_RAG] run  time (Issue #18)
You should have a file named '.gpt_config.env' and store it in your $HOME path. Content of the file should be in the following format:
API_KEY=
API_VERSION=
RESOURCE_ENDPOINT=
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

The file contains API credentials, which, like any other sensitive information, should ideally not be shared publicly. Hope you get it :)
Feel free to reach out if you need further assistance!

For Llama coda, when run to llm = llama_model(MODEL_NAME, BRANCH_NAME, CACHE_DIR, stream=True, method=METHOD), model = AutoModelForCausalLM.from_pretrained(model_name, device_map='auto', torch_dtype=torch.float16, revision=branch_name, cache_dir=cache_dir), the program simply exits.
------------------ 原始邮件 ------------------
发件人:                                                                                "BaranziniLab/KG_RAG"                                                                                @.>;
发送时间: 2024年2月28日(星期三) 中午1:54
@.>;
<EMAIL_ADDRESS>主题: Re: [BaranziniLab/KG_RAG] run  time (Issue #18)
The file contains API credentials, which, like any other sensitive information, should ideally not be shared publicly. Hope you get it :)
Feel free to reach out if you need further assistance!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Did you check if this is a memory issue? We are not using quantized versions of llama here, hence it could take a good chunk of memory. If you see here, you can see the size of the tensors for llama-13b and compare it with the memory of the machine that you are using.
I tried using llama-13b on p3.8x.large AWS instance which has following specs:
4 Tesla V100 GPU
64 GB GPU memory
32 vCPU
244 GB RAM


import createReactNativeComponentClass link has been moved

Upgrading to the latest in React, React-Native and other packages, it shown that react-native-svg broke because the createReactNativeComponentClass had been moved.
I managed to fix it on my own version by changing the links from
import createReactNativeComponentClass from 'react/lib/createReactNativeComponentClass';
to
import createReactNativeComponentClass from 'react-native/Libraries/Renderer/src/renderers/native/createReactNativeComponentClass';

Dupe of https://github.com/react-native-community/react-native-svg/issues/188

Apologies.


Add leap year code in Perl

Closes #17

Hey @ablazejuk. Perl really sat around for a long time before somebody picked it up. I appreciate you taking it on.


Sheet ID not found

Where is the sheet ID? I haven't been able to find it. Can you take a screenshot and show it to me? Thank you 🙏

https://stackoverflow.com/a/49042689 please check this once


IJ debugger

So we will know what is needed in Trace

Merge this early so I don't have to rebase it from time to from.
bors merge


incomplete output when using babel-plugin-transform-flow-strip-types

Our codebase uses flow type annotations which forced me to use jsdoc-babel despite babylon being now integrated into JSDoc. Unfortunately when using the transform-flow-strip-types-plugin there is no output generated for methods of classes with type annotations. This was touched upon before in this issue, which brought me on the right track while debugging.
As a workaround for now I pre-transform the code with flow-remove-types. This gives me the desired output.
Input code
// @flow
export default class SomeClass extends Parent {
  type = SOME_CLASS;

  _foo: Map<string, Object> = new Map();
  _bar = null;
  _baz = null;

  constructor(object: SomeObject, options: SomeOptions) {
    super(object, options);
  }

  /**
   * Applies the filter function
   */
  applyFilter() {
    [...]
  }

  /**
   * Attach the data
   */
  attach() {
    [...]
  }

  /**
   * Remove the data
   */
  detach() {
    [...]
  }
}

JSDoc configuration
{
  "plugins": ["node_modules/jsdoc-babel"],
  "babel": {
    "plugins": [
      "transform-class-properties",
      "transform-flow-strip-types",
      "transform-object-rest-spread",
      ["transform-react-jsx", {"pragma": "h"}]
    ],
    "babelrc": false
  }
}

JSDoc debug output
{ JSDOC_ERROR: DEBUG: JSDoc 3.5.5 (Thu, 14 Sep 2017 02:51:54 GMT)
    at Explain.verifyOutput (/Users/danielbechler/Documents/Projects/meteogroup-weathermap/packages/tools/node_modules/jsdoc-api/lib/jsdoc-command.js:113:19)
    at ChildProcess.handle.on.code (/Users/danielbechler/Documents/Projects/meteogroup-weathermap/packages/tools/node_modules/jsdoc-api/lib/explain.js:48:38)
    at emitTwo (events.js:125:13)
    at ChildProcess.emit (events.js:213:7)
    at maybeClose (internal/child_process.js:921:16)
    at Socket.stream.socket.on (internal/child_process.js:348:11)
    at emitOne (events.js:115:13)
    at Socket.emit (events.js:210:7)
    at Pipe._handle.close [as _onclose] (net.js:549:12) name: 'JSDOC_ERROR' }
(node:79423) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): Error: JSDOC_ERROR: DEBUG: JSDoc 3.5.5 (Thu, 14 Sep 2017 02:51:54 GMT)
(node:79423) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
Done in 7.35s.

Expected behavior
I expect output to be generated for the methods attach, detach and applyFilter.
Current behavior
There is no output generated for these methods at all.
Your environment

Software
Version

JSDoc
^3.5.3

Node.js
8.2.1

npm
5.5.1

Operating system
MacOS Sierra 10.12.6

Could be similar to the issue of @babel/transform-plugin-typescript. I don't know, just guessing and bringing this up based on that it works when you pre-strip the types, which presumably uses a different parser than Babel(?). The AST method someNode.remove() used by Babel and in those plugins in some cases removes comments preceding the node together with the node, so they are just gone for the next step in the pipe. For reference: https://github.com/babel/babel/issues/9916 -- Just in case this is a similar issue.


[02.echo-with-counter] Unify structure of 'readme.md'

Proposed Changes
Unify the structure of readme.md across the differente languages in sample 02.echo-with-counter.
Testing
The behaviour of the code was not modified.

No longer needed


block_stream might drop up to n_jobs - 1 complete frames

The offending line is here:
https://github.com/BirdVox/birdvoxpaint/blob/a1ecf858aed7d9539725ac033816e78c8d34d9fb/birdvoxpaint/util.py#L80
This line is meant to drop blocks that don't have enough samples. But since we're working with multiprocessing and are loading block_size * n_jobs, if we have block_size * (n_jobs - 1) + 3, it would drop that entire block instead of just the +3.
Another issue with that line is that it assumes a mono signal, so that's not good either.
A solution could be:
single_block_length = block_length // n_blocks
y_blocks = (
    y[:(y.shape[0] // single_block_length) * single_block_length]
    for y in y_blocks)

This just makes sure that y.shape[0] is divisible by block_length / n_jobs

yes, but now what if y.shape[0] < single_block_length ?

Right, good point. How bout this?
single_block_length = block_length // n_blocks
y_blocks = (
    y[:(y.shape[0] // single_block_length) * single_block_length]
    for y in y_blocks)

# +
y_blocks = (y for y in y_blocks if y.shape[0])

Not sure how many nested generators is too many lol 🤷🏼♀


Add a per-notifier ShouldNotify()

This way we are able to edit notification behavior per notifier.
This would be usefull to let some notifiers send notifications,
even when the state doesn't change, or with custom condition.

:+1:


Added Roman Numeral Converter

closes #34

Hey @hinedy , please follow the contributing guidelines. It is recommended to only work on an issue which is assigned to you.

Kindly create an issue with proper title and description and include this PR there. And then I'll review and merge this PR 👍

Thanks for the heads up! @SandeepKrSuman, I created an issue linked to this PR, Kindly review.


[Feature][File] Support config null format for text file read

Purpose of this pull request
Support config null format for text file read
Does this PR introduce any user-facing change?
add config key
How was this patch tested?
added
Check list

[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist
Add ci label in label-scope-conf
Add e2e testcase in seatunnel-e2e
Update connector plugin_config

[ ] Update the release-note.

@hailin0 I have a question, shouldn't this specify a certain field? Is it specifying a certain data now?

@hailin0 I have a question, shouldn't this specify a certain field? Is it specifying a certain data now?

This is a common setting and has nothing to do with columns

@hailin0 I have a question, shouldn't this specify a certain field? Is it specifying a certain data now?

This is a common setting and has nothing to do with columns

Got it


Add ship statistics

looks ok, consider collecting them into one stats object in big ship object instead of the individual ones

Can't. They are different across LBs.


The AI didn't fix what it said that fixed!

Describe the bug
I has been happining very often.
I ask the AI to fix a few issues, and the AI understand my request and it returns a message the issue is fixed, but it hasn't been fixed and it took a good amount of my tokens.
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-d2h55q
Steps to reproduce
1 - ask
2 - press enter
3 - verify
4 - not fixed
Expected behavior
To fix the issues that AI said that was fixed
Screen Recording / Screenshot
No response
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 131
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 72978578,
  "usedJSHeapSize": 67398182,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = yohanbrc
Chat ID = 8427544f3b2a

Additional context
No response

Yes seen this a lot of times.  The AI doesn't do what it says it did.  To be fair, we see this in foundation model LLMs directly.

Hi all,
Yes, at times, the AI is unable to resolve the problem and manual intervention is required. The team is hard at work continuing to add hard coded logic as able to fix common errors. Appreciate the feedback!


Remove Old C# Sample per @jongio

Based on conversation with @jongio, removing an old /api/csharp sample

@rajeshkamal5050 - I was talking with Isaac today and I don't think this is being used anymore, since we started with c#/mongo and then moved to c#/cosmos-sql instead.  If not used, I suggest we delete it to minimize maintenance costs.

I verified the folder is not used by any of the todo projects. It is totally dead code in there :)

/azp run azure-dev - template - tests

Repoman Generation Results
Repoman pushed changes to remotes for the following projects:
No changes detected.

@weikanglim  @ellismg , there's no templates to sync or test. The code is death/abandoned code.

/check-enforcer override


rancherd and new RancherChart

https://github.com/rancher/rancher/issues/28131

This PR rewrite rancher charts and pin rke2 to the latest version.
The following changes are done through rancher charts:

Remove the dependency of cert-manager. This removes cert-manager issuer types as the certs have to be provided on disk/secret in the future.
Remove ingress. Rancher is exposed on hostPort by default.
Remove different certs option. Now the certs have to be provided on disk/secrets.

If provided on disk, it has to be pre-mount on disk at the following location:
certificate file: /etc/rancher/ssl/cert.pem
private key file: /etc/rancher/ssl/key.pem
cacerts(if self-signed): /etc/rancher/ssl/cacerts

if provided by secrets, a secret name tls-certs on cattle-system namespace must pre-exist and contain the following data:

kubectl create secret generic tls-certs --from-file=key.pem=/path/to/key.pem --from-file=cert.pem=/path/to/cert.pem

Rancher deployment is now a daemonset instead of deployment.

Other rancher chart options remain unchanged. In order to configure custom value.yaml, HelmChartCRD must be leveraged.
For example:
apiVersion: helm.cattle.io/v1
kind: HelmChartConfig
metadata:
  name: rancher
  namespace: kube-system
spec:
  valuesContent: |
     # custome value.yaml...

This has to be pre-written into /var/lib/rancher/rke2/server/manifest before rancherd is launched

Per darren's request, the following changes have been made:

Certs can only be provided through hostpath. No secret hookup.
This is an internal rancherd charts that only lives in /cmd/rancher/charts.

The certs workflow works in the same way with single docker install https://rancher.com/docs/rancher/v2.x/en/installation/other-installation-methods/single-node-docker/, which allows user to provide certs under /etc/rancher/ssl on the host and we use hostpath to mount these into container. That will be the only way to provide certs.


[Feature Request] Extend the solution to add consideration of _spf.google.com

The solution supports enumeration of the IP addresses used by GCP and summarized here but it does not include peripheral Google services including e.g. GCS, Drive and (indirectly) summarized here as Google's Sender Policy Framework for Gmail.
Recommend: Extend the solution to support the _spf.google.com addresses

either (by default) enumerate the union of these netblocks
or by surfacing a selector for the user to decide cloud, spf, cloud & spf

done


Added smarter functions for routing with custom handlers

Added functions to let custom handlers mind the SsrMode of each route

I think the CI issue here is just that this is set up as a PR to main but it's assuming the effect-cleanups branch which is 0.5 — I think you probably want to make a PR to that branch instead.

I think the CI issue here is just that this is set up as a PR to main but it's assuming the effect-cleanups branch which is 0.5 — I think you probably want to make a PR to that branch instead.

 I didn't merge with that because it had hundreds of changes, now I realize that's cause my search/replace edited all the book and docs.


Slack: revoke token when authorization fails

This issue refers to this code comment. The installation process requires the installer to be an admin, because PolicyKit requires it. We're only able to check if the installing user is an admin after they perform the install, though. As you can see in the code, we raise an error if they are not an admin. At that point we should also uninstall the app and/or revoke the token.
https://api.slack.com/methods/apps.uninstall
https://api.slack.com/methods/auth.revoke

I updated the description. This relates to some broader issue across the OAuth-enabled plugins, though, which is we don't provide any mechanism for uninstalling the app OR for logging individual users out (only relevant for Slack Plugin currently which supports user login). We give the driver a way to log users in, but we don't give them a way to log users out (though they could hit the auth.revoke endpoint through the generic Slack method).

Related https://github.com/amyxzhang/policykit/issues/509