Question: Why is Exchange of type *int in ListTickersParams?

Looking at ListTickersParams, it is defined as:
// Specify the primary exchange of the asset in the ISO code format. Find more information about the ISO codes at
// the ISO org website. Defaults to empty string which queries all exchanges.
Exchange *int `query:"exchange"`

Notice it is of type *int, I am wondering why that is?
From the documentation comment above the field:

...Defaults to empty string which queries all exchanges.

Also, when I use Polygon's website, I enter a string for the exchange parameter, for example if I wanted to filter for XNAS
https://api.polygon.io/v3/reference/tickers?exchange=XNAS&apiKey=mykey

This gives me results like so:
{
    "results": [
        {
            "ticker": "AACG",
            "name": "ATA Creativity Global American Depositary Shares",
            "market": "stocks",
            "locale": "us",
            "primary_exchange": "XNAS",
            "type": "ADRC",
            "active": true,
            "currency_name": "usd",
            "cik": "0001420529",
            "composite_figi": "BBG000V2S3P6",
            "share_class_figi": "BBG001T125S9",
            "last_updated_utc": "2022-11-23T00:00:00Z"
        },
        ...
    ]
}

shouldn't this be of type *string in ListTickersParams?

Hey @ClaytonNorthey92, thanks for submitting this! This does seem to be a bug in our client. I'll look into it a bit more and make the fix when I get a chance!

hey @morningvera thank you for the response, I had been working off of a commit in a fork of mine, I just opened a PR for it.  I hope that helps.


URI completion

Adds function uri_complete to uri.c.
Relates to:

https://github.com/baresip/baresip/pull/2135

Thank you for the review. I will close this PR as a result of the disccussion in baresip/baresip#2135 and your comments.


Complier Problems with Chrome and Edge

Complier loading, display and selection not working for current versions of Chrome and Edge
Note that ad blocking is disabled and these browsers support the compilers perfectly for remix.ethereum.org.
Loading the latest local compiler version and showing the list of compilers on Qmix broke between Chrome version 63 and version 64 (64.0.3282.140 on 2018-02-06). Enabling ads and popups makes no difference on versions 64 and later.  You know them, it’s when a page opens a pop-up to some destination AND navigates the page. Usually one of them is an ad or something that you didn’t want.
Starting in Chrome 64, these type of navigations will be blocked, and Chrome will show some native UI to the user - allowing them to follow the redirect if they want.

Full list of Chrome v64 changes at https://chromium.googlesource.com/chromium/src/+log/63.0.3239.84..64.0.3282.119
Some possibly related fixes for Remix are:
Move compiler to remix solidity #1027 https://github.com/ethereum/remix-ide/pull/1027
Fix loading compiler #1169 https://github.com/ethereum/remix-ide/pull/1169
Merge pull request #1169 from ethereum/fixCompilerLoading https://github.com/ethereum/remix-ide/commit/3fb2c1c78c76bf86ba0f4914e762bb2786e22097
Update package.json https://github.com/ethereum/remix-ide/commit/3778c622079e9b0d35ed26f03ca319e1ca071061

No, endless loop of "Loading Compiler". Are you on a PC with Win 10? I have tried across multiple PCs with Win 10 and none work with any version of Chrome > 63.


add robot benchmark

Requires

New release of MTK
https://github.com/JuliaComputing/Multibody.jl/pull/61
https://github.com/SciML/ModelingToolkitStandardLibrary.jl/pull/296
New release of MTKstdlib

First run

second run

Julia timings during second run is split up like this

Dymola results:
Run Number | Translation time | Total time
1, 5.075, 5.267
2, 3.912, 4.112
3, 4.024, 4.255

Multibody isn't in the registry?

No, no release has been made yet


MM-43327 : Remove unused states from state.entities.general redux store

Summary

Ticket Link
https://mattermost.atlassian.net/browse/MM-43327
Related Pull Requests
N/A
Screenshots

Release Note

NONE

E2E tests not automatically triggered, because PR has no approval yet. Please ask a developer to review and then try again to attach the QA label.

/update-branch

Error trying to update the PR.
Please do it manually.

/update-branch

/e2e-test

Successfully triggered e2e testing!
https://git.internal.mattermost.com/qa/cypress-ui-automation/-/pipelines/212681


Name inputs and remove unused inputs from testcase

When not given input_names, inputs are named by torch.onnx. The naming rule is not controllable and seems inconsistent, so we should name inputs by ourself.
With our naming, we can detect unused inputs and remove them from testcases.

/test

Seems few tests is failing: https://ci.preferred.jp/pytorch-pfn-extras/64749/#L383

/test

/test

/test


Suggestion: quicker GitHub Action

I was inspired by golangci-lint through their ease of use. I noticed the readme mentions running kubeconform using Docker in using GH_TOKEN, but with an action such as https://github.com/hermanbanken/kubeconform-action it is even easier & less fault tolerant to install kubeconform (no GH_TOKEN needed).
Feel free to integrate https://github.com/hermanbanken/kubeconform-action.

Alternative is mentioning a completely generic action such as https://github.com/jaxxstorm/action-install-gh-release.

Hi @hermanbanken ! Thank you for making a JS action variant for Github actions! Could you expand on what makes it easier to use? I remember looking at these JS actions and wondering what the benefit was, it seemed like it requires you to write a fair bit of js wiring code... is it faster? easier to use? The GH_TOKEN you don't need to generate, it is automatically provided by Github actions, do you see this as problematic? Thanks!

You don't need to send a GH_TOKEN to any action this way. So it is completely unauthenticated process, leaving you with no risk the action performs any API requests with said token.
It is also ridiculously fast. 0.1s-0.3s by my measures (see GH Action tab of my action).
I'm not sure how long the Docker login & image download takes, but it seems more cumbersome than simply having the tool available in the runner. You might want to combine the outputs or in other ways do some bash scripting, which is less then trivial with a Docker action.

I had a caching bug in my composite action, but the other one I mentioned is doing that correctly in a generic fashion.

Note I'm a bit paranoid about GH Actions because they make it impossible to vet the source code your running except when pinning to a commit. Any malicious person owning an action could update an action used in millions of repos to run different code by recreating the tag.
Having all actions pinned like this, it is much more convenient to only focus on JS actions and not have the trouble of also pinning Docker versions.
(Checking checksums would also be on my wishlist, if it isn't supported already.)

Ah - yes that's fair, I understand. I wonder if the token is still absolutely necessary to pull the image tbh :thinking:

Why would you need a token to pull public images from github container registry? Try docker run ghcr.io/yannh/kubeconform locally, you'll see pulling works fine.

Sorry, I was basing that comment on old information. I looked it up, this changed on June 21st of 2021.
https://github.blog/2021-06-21-github-packages-container-registry-generally-available/


make loads of templates

[ ] - Use a layout
[ ] - Create a home.html view with a header, a login and a repos partial
[ ] - Create an issues.html view with a header partial and an issues partial
[ ] - Create a timer.html view with a header partial, a time partial and an options partial (TBC)

Move header in the layout as it repeats itself everywhere really.

Header still left as a partial but all templates completed.


Reconnection attempts

So far the integration works great.  But i do see a lot of reconnection attempts in the logs.
This error originated from a custom integration.

Logger: custom_components.envisalink_new.pyenvisalink.envisalink_base_client
Source: custom_components/envisalink_new/pyenvisalink/envisalink_base_client.py:144
Integration: Envisalink_new
First occurred: April 27, 2023 at 5:11:08 PM (142 occurrences)
Last logged: 2:16:31 PM

Reconnection attempt in 30s

followed by
This error originated from a custom integration.

Logger: custom_components.envisalink_new.pyenvisalink.envisalink_base_client
Source: custom_components/envisalink_new/pyenvisalink/envisalink_base_client.py:500
Integration: Envisalink_new
First occurred: April 27, 2023 at 5:11:08 PM (142 occurrences)
Last logged: 2:16:31 PM

Command '00' failed due to timeout waiting for response from EVL

The alarm becomes available soon after.  Any ideas what could be causing this?  thanks!

What version of the Envisalink firmware are you running?  I've seen similar issues with older firmware versions.

01.04.184A
What I haven’t been able to pin down is if an HA reboot heals this or arming/disarming the system does. I haven’t been able to reproduce it. But somehow the issue goes away for days until it comes back mysteriously. This may be TMI, but the alarm is scheduled to arm/disarm on a daily basis. And so far hasn’t failed to do either.

That firmware should be fine.
However, my experience is that the EVLs can be a bit finicky and I've seen lot of people (myself included) hit by similar issues.  It typically ends up getting tracked down to the EVL itself becoming non-responsive for periods of time.  What triggers the unresponsiveness has varied but usually involves the EVL receiving some form of traffic at an inopportune time.  Sometimes that traffic has come from this HA integration and other times it's come from elsewhere (I believe someone's Plex server was querying devices on the network which was causing it to trigger).  Needless to say it's frustrating to track down.
A few thoughts:

If it comes and goes, is there anything else that changes on your network in similar intervals that could be interfering?
If you could enable debug logging for the integration and capture this happening, I'd be happy to take a look and see if there's any hints there.  Debug logging will introduce a lot of logs though so be aware of that before you turn it on.
If it's always the '00' command failing, you could change your keepalive_interval to something much larger to see if that reduces the frequency.  Not a fix but maybe some relief at least.

To enable debug logging, add something like below to your configuration.yaml:
logger:
  default: info
  logs:
    custom_components.envisalink_new: debug

i'll try this and try to catch the log entries when it's in this reconnection loop.  thanks

see attached for some of the log trace.  only 2 instances of this yesterday.  i capture a full day's worth of log if needed.
log.txt

The log file contains many duplicated lines but only for the underlying pyenvisalink logs which is odd.  Are you seeing these duplicated lines in your raw log files on your HA host?
If this isn't just a logging issue, it's pretty odd as it looks like there are three connections to the EVL when there should only be one (which is also enforced by the EVL itself).  Do you use anything in-between your HA and the EVL such as AlarmServer?

The log file contains many duplicated lines but only for the underlying pyenvisalink logs which is odd. Are you seeing these duplicated lines in your raw log files on your HA host?

no.  only seem to be the pyenvisalink entries that are triplicated

If this isn't just a logging issue, it's pretty odd as it looks like there are three connections to the EVL when there should only be one (which is also enforced by the EVL itself). Do you use anything in-between your HA and the EVL such as AlarmServer?

... and no.  i'll try removing the integration altogether, restarting, restarting again to try to clear up stuff, re-installing the integration, restarting, setting up the integration, restarting... and monitor the logs to see if that clears up the "gunk"

A complete log from when you start HA to when the issue first occurs could be helpful to track this down.  If that logging is indicative of an issue with the integration, logging from startup may provide some clues as to what's going on.

Unfortunately, I don't see anything unusual in that latest log that could point to a problem.
All the other keep-alive commands (^00) sent get a response back almost immediately but periodically something seemingly delays the response long enough for the command to timeout.  That could be the EVL itself becoming slow/unresponsive or possibly a network delay (is your HA on WiFi?) or your HA host getting sluggish.
If you're able to do a network capture between the HA host and your EVL and capture the period of time when the timeout occurs, that could help narrow down the root cause.  But I know that's not something many people are setup to do.

both the EVL and HA (raspberry) are wired.  i think i'll leave it alone for now, or at least until i find the time to setup the packet capture.  i see very low incidence of this now (maybe the reinstall helped) and the EVL-related entities seem to continue to function ok.


Fix outdated URLs

Thanks for the PR. I appreciate the efforts you have been putting in!
LMK if you have the correct link on that one, and I'll merge.


Don't add primary key for default id

With ecto, if the primary key is id, we don't need this as id is default, so don't add @primary_key {:id, :integer, []}.

This has been merged in to develop


Asynchronous Encounters Guide

Guide on how to represent text/email based care in terms of Encounters and Communications

fixes #1436

Coverage: 93.681%. Remained the same when pulling 85bc1d2598383d93c313406d1d6468f576505648 on rahul-async-encounters into 14a2f3b3772cd103e58ed92a8b1d3c489f2cb386 on main.


Force re-sync all the data possible?

Hello @pravking,
You can re-sync content of the core data by setting sync token to nil and then performing a sync request.
    /// Resets the sync token and allow for further sync of the content again. Make sure you cleaned up all your
    /// models before resyncing.
    func resetSyncToken(moc: NSManagedObjectContext) {
        let fetchRequest = NSFetchRequest<NSFetchRequestResult>(entityName: String(describing: SyncSpace.self))

        do {
            guard let syncSpace = try moc.fetch(fetchRequest).first as? SyncSpace else { return }
            syncSpace.syncToken = nil

        } catch let error as NSError {
            print(error)
        }
    }

Thanks for the solution @tomkowz.
Had to use 'SyncInfo' instead of 'SyncSpace' given in the code snippet to get it working.

Hmm... Okay, I see the SyncInfo does not expose the syncToken. I think you could then remove the SyncInfo from the store and then re-sync. Removing it from the store will have the same effect like setting it to nil.
    private func deleteSyncInfo() {
        let fetchRequest: NSFetchRequest<NSFetchRequestResult> = NSFetchRequest(entityName: "SyncInfo")
        let deleteRequest = NSBatchDeleteRequest(fetchRequest: fetchRequest)

        do {
            try managedObjectContext.persistentStoreCoordinator?.execute(deleteRequest, with: managedObjectContext)
            try managedObjectContext.save()

        } catch let error as NSError {
            print(error)
        }
    }

Btw. I know ContentfulPersistence just for a bit now but in my demo projects for debugging purposes I am using a custom SyncSpace instead of SyncInfo because of the easy access to the token.
When you switch to use SyncSpace you'll not need to do a delete request because token will be nil anyway.
Let me know if the above work for you.

Thanks @tomkowz removing SyncInfo from store is also working.
Also, is it necessary to clear all the models before re-syncing?
When I tried clearing only the SyncInfo model and then calling sync also the re-syncing is working.

I don't think this is necessary. If that works for you without removing data then that's fine. That was my advice just to make sure. I think we can close this one then.


Time for a new pip package?

Hey @dfm -
#AstroHackWeek 2015 is coming up, and it would be good if the kids can corner plot their sampled PDFs with PJM-compliant filled contours etc (nicely nailed by your good self in #41 and #45). What do you think about updating the package so that pip install --upgrade traiangle_plot gives them this new power?
Thanks!

Done and done. pip install -U --no-deps triangle_plot should give you version 0.2.0.
P.S. always use --no-deps just in case. If you don't a world of horror can come crashing down on you. I'll never understand why that's not the default!!!

Thanks @dfm!
@abmantz - your 2D credible intervals can now be filled in! :-)


feat: Add GraphDBBlock to LongtermAgentMemory and add GraphDBMemory

Description
This pull request introduces a new feature to the LongtermAgentMemory by integrating a GraphDBBlock and adding a new GraphDBMemory. The changes aim to leverage knowledge graphs within AgentMemory, enhancing the capability to store and retrieve structured information through graph databases.
Motivation and Context
close #846
Types of changes
What types of changes does your code introduce? Put an x in all the boxes that apply:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds core functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation (update in the documentation)
[ ] Example (update in the folder of example)

Implemented Tasks

[x] Implemented GraphDBBlock in LongtermAgentMemory.
[x] Added GraphDBMemory to handle graph-based data.
[x] Created tests for the new GraphDBMemory functionality.
[x] Updated documentation to reflect the new feature.

Checklist
Go over all the following points, and put an x in all the boxes that apply.
If you are unsure about any of these, don't hesitate to ask. We are here to help!

[x] I have read the CONTRIBUTION guide. (required)
[x] My change requires a change to the documentation.
[x] I have updated the tests accordingly. (required for a bug fix or a new feature)
[ ] I have updated the documentation accordingly.

Thanks @mrsbeep for the PR. There may be still something to be redeign for this PR.
An example on how to use Neo4jGraph for GraphRAG here:
https://colab.research.google.com/drive/1meBf9w8KzZvQdQU2I1bCyOg9ehoGDK1u#scrollTo=l4VANToobUzu
A couple things to be discussed:

What is the default structure of the LongtermAgentMemory? Should we always have both VectorDBBlock  and GraphDBBlock or allow different combinations?
Should we allow user to load a pre-built knowledge graph?
Should we allow agent to automatically convert chat messages into knowledge graphs?
How to score, retrieve and decide whether to use the retrieved results?
How to combine to retrieved KGs with vector_db_retrieve and chat_history?

What is the default structure of the LongtermAgentMemory? Should we always have both VectorDBBlock and GraphDBBlock or allow different combinations?
LongtermAgentMemory contains chat_history_block, vector_db_block, graph_db_block. You don't always need to have both VectorDBBlock and GraphDBBlock. You can include them as needed.
Should we allow user to load a pre-built knowledge graph?
The current design allows this.
Should we allow agent to automatically convert chat messages into knowledge graphs?
It would be nice if this feature existed. Let me create some conversion functions today.
How to score, retrieve and decide whether to use the retrieved results?
Scoring: The scoring mechanism is already partially implemented in the ChatHistoryBlock where historical messages are scored based on their proximity to recent messages and a keep_rate. A similar approach could be used for the vector and graph database retrievals by scoring based on similarity or relevance.
Retrieval: Each block implements its own retrieval method. For VectorDBBlock, retrieval is based on vector similarity, while GraphDBBlock might use structured queries.
Decision to Use Retrieved Results: The decision to use retrieved results can be based on the scores assigned during retrieval. Implementing threshold-based filtering or a ranking system to prioritize high-scoring results can ensure that only the most relevant information is included in the context provided to the agent.
How to Combine Retrieved Knowledge Graphs with VectorDBRetrieve and Chat History?
Combining these elements involves merging the retrieved results from each block into a cohesive context. The retrieve method in LongtermAgentMemory currently combines these by appending vector_db_retrieve and graph_db_retrieve results between the first and subsequent chat history records.
This combination could be optimized by:
Prioritizing certain types of information based on relevance or score.
Ensuring consistency and coherence in the context by resolving any potential conflicts or redundancies.
Implementing a context creation logic that respects the token limits and maintains the logical flow of conversation, potentially using the BaseContextCreator as a guide for structuring the final output.


Fix caddy quic udp port

Changes

Expose port 443/udp for Caddy proxy (to enable QUIC/HTTP3)

Thank you for your contribution. This was added via this PR: https://github.com/coollabsio/coolify/pull/3853 so this PR is no longer needed.

#3853 was for Traefik, so I think this one is still needed.

Ups, sorry about that.

Thanks for the PR no I am merging it:)


fix(select): add top aligned example, docs and fix icon rotation

fixes https://github.com/patternfly/patternfly-next/issues/2163

PatternFly-Next preview: https://patternfly-next-pr-2164.surge.sh

duplicate of https://github.com/patternfly/patternfly-next/pull/2026/


Use the IO.FS.rename to move file instead of relying on presence of mv program

The Cache program after detecting that LeanTar is old downloads it and attempts to place it in the CACHEDIR directory. It then attempts to rename it using the mv command it assumes is present on the platform. On windows cmd and powershell this does not work since mv is actually a shell built in and not an executable. (This will however work on MinGW based shells).
@digama0 added the function IO.FS.rename in Lean Lean4#2345 which calls the standard C/C++ function rename. This PR uses this function to move leantar instead of the runCmd command. When this fix succeeds the output looks like the following (in powershell).
PS W:\LeanStuff\graveyard_mathlib4\LeanCacheFix> lake exe cache get
leantar is too old; downloading more recent version
Attempting to download 3614 file(s)
Downloaded: 3614 file(s) [attempted 3614/3614 = 100%] (100% success)
Decompressing 3614 file(s)
unpacked in 323 ms

bors r+


Investigate slow Travis builds

This is bad enough it's affecting my development workflow - https://travis-ci.org/studentinsights/studentinsights/builds/113123635 took 20 minutes to even start the build.
I'm not sure if there's anything we can do here to improve this, or if there are other free or cheap services that are better, but if someone knows about alternatives or can help investigate, it would be awesome.

I disabled building on pushes, only on pull requests and merges: https://travis-ci.org/studentinsights/studentinsights/settings  cc @alexsoble

Just taking notes - Travis is super speedy this morning, and builds are starting within seconds!  It's 7:30am EST.

Seems like this hasn't been an issue in a while, so I'm going to close it. Please, anyone, reopen if you experience this.


fix: added additional apikeymodal tests

Contributes to #954
also in regards to downloadFileName/Type not being included in the testing those props are being handled in APIKeyDownloader

@dcwarwick


In Declarative Mode. When adding Asset through local_policy.yaml through Specific Rule

In Declarative Mode. When adding Asset through local_policy.yaml through Specific Rule there is an issue
1st.  Because when you use fully managed portal and add an asset, the asset will be seen in Asset tab also in main dashboard
2nd.  Also when adding an asset in full managed portal one can add alot of features like name of asset, tags, all URLs that are based on that particular asset eg an asset can have multiple URLs to be accessed but this is not in declarative mode
In delcarative mode when you add an asset the name of the url becomes the name of the asset
3rd.  since the asset is added form declarative mode this means it can not be seen in asset tab in visibility mode in portal and this lead to when downloading the fully policy form the portal the policy will differ from that is inside the server policy.json
How do i add multiple urls in the asset in delcrative mode as it is in fully managed mode also names of assets

Hi,
caused by issue #129, I will update there.


Disable auth to /postgres, increase current_database oid. resolves #4285

Changelog entry
Changelog category

Not for changelog (changelog entry is not required)

Additional information
...


mommy: 1.2.3 -> 1.2.4; migrate to by-name

Description of changes

patches have been upstreamed: https://github.com/FWDekker/mommy/releases/tag/v1.2.4
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.


HashLoadBalancer unit test

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #4383 (e40c9ad) into master (ff7e980) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #4383      +/-   ##
============================================
- Coverage     68.38%   68.36%   -0.02%
+ Complexity     7516     7510       -6
============================================
  Files          1020     1020
  Lines         28813    28813
  Branches       2568     2568
============================================
- Hits          19704    19699       -5
- Misses         7575     7579       +4
- Partials       1534     1535       +1

Impacted Files
Coverage Δ

...henyu/common/cache/MemorySafeWindowTinyLFUMap.java
73.33% <0.00%> (-8.34%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


getNewLineNo() and "No newline at end of file"

Hello! Git generated diff:
diff --git a/file.php b/file.php
index 272b2e4..7a9961d 100644
--- a/file.php
+++ b/file.php
@@ -1,3 +1,6 @@
 first line
 second line
-third line
\ No newline at end of file
+third line
+fourth line
+
+some other line

Current file content:
first line
second line
third line
fourth line

some other line

Display line numbers:
use ptlis\DiffParser\Parser;

$parser = new Parser();

$changeset = $parser->parseFile('C:/server/projects/dev/master-protect/git-line.diff', Parser::VCS_GIT);
$files = $changeset->getFiles();

foreach ($files as $file) {
    $hunks = $file->getHunks();
    foreach ($hunks as $hunk) {
        $lines = $hunk->getLines();

        foreach ($lines as $line) {
            echo 'getOriginalLineNo: ' . $line->getOriginalLineNo() . ' getNewLineNo: ' . $line->getNewLineNo() . ' ' . $line->getContent() . "\n";
        }
    }
}

Output:
getOriginalLineNo: 1 getNewLineNo: 1 first line
getOriginalLineNo: 2 getNewLineNo: 2 second line
getOriginalLineNo: 3 getNewLineNo: -1 third line
getOriginalLineNo: 4 getNewLineNo: 3  No newline at end of file
getOriginalLineNo: -1 getNewLineNo: 4 third line
getOriginalLineNo: -1 getNewLineNo: 5 fourth line
getOriginalLineNo: -1 getNewLineNo: 6
getOriginalLineNo: -1 getNewLineNo: 7 some other line

Why getNewLineNo() gets +1 for line number after "No newline at end of file"? I expected that "third line" is 3, "fourth line" is 4 and "some other line" is 6. Is this a mistake or do I understand something wrong?

Hi dmrsol, the simple answer is that this is an side-effect of the file format for diff files that git uses patches.
The complete answer is a little more complex:
Git includes this line as there is a long-running convention that files will always be terminated with a newline, and most (if not all) Linux terminal commands assume that this is the case. The rationale for this is to make working with files more convenient at the terminal. An example of this in action can be seen when you 'cat' a file that doesn't include a newline - your cursor position will be left inline with the end of the file & not aligned left on a fresh line.
As a diff is a 'line-by-line' algorithm it doesn't encode newlines explicitly - in the average case each line of a diff can be assumed to be terminated with a newline.
The consequence of the interaction between these conventions / rules is that diff needs to explicitly encode the absence of a newline in a 'phantom line' - a line that doens't exist in the file but must be tracked to allow proper merging from a patch file.
Finally, the reason that I chose to use "the final line number + 1" as the line number when represented in memory is that at the time of writing I couldn't think of any non-sucky way to represent this information. There were two obvious choices to me at the time:

Re-using the final line number (as you suggested) would lead to strangeness in tools that consumed these data-structures that rightfully assume that line numbers MUST be unique.
Creating a 'phantom line' meant that the generated data-structure was 'lying' about the structure of the diff'd file, but was in-line with the textual representation used in patch files.

Obviously I chose the latter at that time, but on reviewing that decision in light of this question I don't think that is the right way of doing things. I'm going to work up a change so that whether there is a newline or not at the end of a hunk is encoded in the hunk data-structure. This will allow me to eliminate the phantom  "No newline at end of file" line from the data-structure but correctly track that data when re-encoding back to the textual representation.

It would be great! Thanks for the detailed answer

I've just merged a branch that should fix this issue - you should be able to confirm this by pulling from the master branch.
I'm planning on making a 0.7.0 release fairly shortly so that a stable release containing these fixes can be used.

Everything is ok now. Thank you


Documented path to dmgcmd is incorrect

The path to the dmgcmd utility for self-hosted integration runtimes is incorrect. The current path created by installation as of this date is C:\Program Files\Microsoft Integration Runtime\4.0\Shared. But the doc should be updated to reflect the path varies by (some undocumented) release schedule.

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

ID: 32785d15-8202-6e44-f1d2-420ef0600f3f
Version Independent ID: 3af5fa49-7bd4-18ce-f8dc-aa4a2b8c6478
Content: Create a self-hosted integration runtime - Azure Data Factory
Content Source: articles/data-factory/create-self-hosted-integration-runtime.md
Service: data-factory
GitHub Login: @nabhishek
Microsoft Alias: abnarain

@yobyot Thanks for the feedback and bringing this to our notice. At this time we are reviewing the feedback and will update the document as appropriate.

@yobyot The PR for the fix has been created and it will be merged soon. Thanks again for pointing this out, I am closing the issue based on this update, please feel free to comment if you have any further queries.
Thank you.


yii\db\command execute() don't handle errors on multiple queries

What steps will reproduce the problem?
Try to import db schema (from mysqldump or PhpMyAdmin schema export) with some error on non-first query. In example, change "int" column type in some table to "intFOO" to cause mysql error.
Schema:
SET FOREIGN_KEY_CHECKS=0;
DROP TABLE IF EXISTS `auth_assignment`;
CREATE TABLE `auth_assignment` (
  `item_name` varchar(64) COLLATE utf8_unicode_ci NOT NULL,
  `user_id` intFOO(10) UNSIGNED NOT NULL,
  `created_at` int(11) DEFAULT NULL
) ENGINE=InnoDB DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci;

Example code (console action):
public function actionReset()
{
    $schemaFile = 'some/path/schema.sql';
    try {
        $command = Yii::$app->db->createCommand(file_get_contents($schemaFile));
        $command->execute();

        Console::output('Schema imported.');
        return self::EXIT_CODE_NORMAL;
    } catch (\Exception $e) {
        Console::error($e->getMessage());
    }
    return self::EXIT_CODE_ERROR;
}

What is the expected result?
Execution ends with proper exception.
What do you get instead?
"Schema imported." on stdout - no exception thrown, or error reported.
Additional info
Read this question and approved answer: How to check if db execute sql fails?
I believe that Yii should handle this case?

Q
A

Yii version
2.0.12

PHP version
7.1.x

Operating system
Kubuntu 17.04

Delimiters in MySQL
https://stackoverflow.com/a/10259528/3419535

Attempting to use DELIMITER with a client that doesn't support it will cause it to be sent to the server, which will report a syntax error. For example, using PHP and MySQLi:
$mysqli = new mysqli('localhost', 'user', 'pass', 'test');
$result = $mysqli->query('DELIMITER $$');
echo $mysqli->error;
Errors with:
You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'DELIMITER $$' at line 1

I had the same problem if I execute a procedure that calls more than one procedure. If there is an error, the error will not be raised in Yii. But if I execute just that one procedure, the error will be raised. So this seems to be a bug of Yii. Any suggestion how to detect the errors in a multiple procedure call?

Any suggestion how to detect the errors in a multiple procedure call?

Yes. See https://github.com/yiisoft/db/issues/79#issuecomment-330821059

I also link a solution in Additional info in this issue: https://stackoverflow.com/a/46280130/1829368
@thomaswruss read topic before ask.


cmd/dist: no such tool "dist"

What version of Go are you using (go version)?

$ go version
go version go1.21rc2 darwin/arm64

Does this issue reproduce with the latest release?
Yes.

I use go tool dist list to get the architectures available for cross-compilation (CI workflow).
What did you expect to see?
The list of architectures available for cross-compilation. This worked fine in Go 1.20.x.
What did you see instead?
❯ go tool dist list
go: no such tool "dist"

cc @rsc

Our current thinking is that shipping go tool dist is excessive, but breaking scripts that rely on go tool dist list is also unfortunate. Perhaps we can allow cmd/go to impersonate dist when the command is go tool dist list and the dist tool isn't present.

Mentioning that there is a workaround that can be used with go1.21rc2:
~ $ go version
go version go1.21rc2 darwin/arm64
~ $ go tool dist list
go: no such tool "dist"
~ $ GOROOT=$(go env GOROOT) go run cmd/dist list
aix/ppc64
android/386
android/amd64
android/arm
[...]

It is more verbose than just go tool dist list, and slightly slower since go builds the dist command on the fly. But that wouldn't be noticeable in the context of a CI job.


Social

Updated social links

Let's stick to the social part on this branch. We can add the ui test later probably directly to master.


Implement load_data

load_data()
Retrieve data from the GitHub API
This function makes an HTTPS request to the GitHub API and retrieves your selected data. The data is
stored in the Analysis object.
Parameters
None
Returns
None

Completed here: https://github.com/haahmad/DSI_assignment_module/pull/6


Thank you!


trajectory duration in hours and minutes

closes #283

From my understanding this still does not display minutes, because the duration to which the format string is applied comes from trajectory.durationDays which returns the difference in full days by truncating fractional days to zero.

Looks good now, thanks!


How to disable physics simulation?

Didn't find it in documentation and code. I use ngraph with fabric.

+1

I think that would be helpful possibility to allocate layout computation on selected communities of nodes.
I also load sub-graphs on demand, and actually much of the performance goes to adjust nodes that are not close to the areas a user is looking at.
Typically, the screen area.
I tried to pin nodes that are off the screen area and then free them as soon as they get into the visible are, but the result is not perfect: as soon as they get visible, the edges "rebound" cause all of the sudden forces are applied. And you see things moving around.
So I think that a good approach to test would be to apply forces proportionally to the distance of currently selected node: say, spring-force applied to n-th neighbors is set to 1/n-th, or with a custom function (I play with logaritmic adjustments now).
But don't have coding skill so advanced to modify the library (I am better at designing interaction and UX).
Or even simpler, it may be to turn on and off the layout engine.
Now, I found only layout.isNodePinned() as possible solution.
I did not found how to set "parameters" to "avoid" force-computation.
Alternatively you can use constant layout and set position programmatically for new layed out nodes - but please share if you find a good logic for expanded-on-demand graphs!!


[charts/redis-ha] Add additional labels

What this PR does / why we need it:
Consumers may have a requirement to have governance labels applied to all (or a subset of) resources. Currently the consumer has the option to specify some labels, but instead of playing whack-a-mole it may be easier to have a key for labels that will be applied to all resources.
Which issue this PR fixes

fixes #142

Special notes for your reviewer:
Let me know if a different key to additionalLabels is required.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

Also merged from upstream master and incremented chart version. Ready for review again.


Incomplete API doc snippet shown for publishMessage function from NATS client

Description:
When hovering over publishMessage function, I am seeing an incomplete API documentation for the function as below.

Publishes data to a given subject.

ballerina # nats:Error? result = natsClient->publishMessage(<@untainted>message); #

Parameters
nats:Message message : Message to be published

Returns
nats:Error?() or else a nats:Error if there is a problem when publishing the message

Steps to reproduce:
Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

Checked in Ballerina Swan Lake Beta 1
Closing since the issue seems to be resolved.
Hovering over publishMessage prompts the following:


After update to 0.1.71 elementum gone

After update to 0.1.71 elementum is gone on kodi 19 matrix. I can't install it from the repository. Only a fresh installation helps. Android tv box, android 9. This is happened to several of my friends as well.

Log required.

@elgatito I don't have a log anymore since I reinstalled everything.

i also have kodi 19 + android 9 but it have not updated to 0.1.71 by some reason intil today. in repo it was saying that 0.1.70 was the latest version.
on test PC i had to update manually.
maybe it is somehow connected to the fact that full zip was uploaded after OS/arch specific zips, i dunno.
but it works fine - nothing is "gone". actually i am not sure that is meant by "gone".

@antonsoroko It means that after auto update to 0.1.71, it disappeared from kodi. It's no longer in addons. No more elementum icon. When I try to install it from elementum repository nothing happens. In the end, I installed everything from scratch.

the same problem, updated to 71 and addon is gone. But after kodi restart it's came back. Enough to be a workaround )

Hallo, how to log??

@Gallup99 http://elementum.surge.sh/contribute/log/

@matke-84 is this still relevant?

вряд ли
у меня свежая версия
ср, 19 мая 2021 г. в 20:59, Anton Soroko @.***>:

@matke-84 https://github.com/matke-84 is this still relevant?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/elgatito/plugin.video.elementum/issues/720#issuecomment-844338919,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAENHOIGPSGZJHVKG6LQIOTTOP4BLANCNFSM4ZJBFY4A
.

--
Regards,
Valentin A. Poroxnenko


Trying to run gulp

E:\bingo-master\node_modules\gulp-jest\lib\index.js:29
_jestCli2.default.runCLI(options, [options.rootDir]).then(function (_ref) {
^
TypeError: Cannot read properties of undefined (reading 'runCLI')
at DestroyableTransform._transform (E:\bingo-master\node_modules\gulp-jest\lib\index.js:29:23)
at Transform._read (E:\bingo-master\node_modules\readable-stream\lib_stream_transform.js:184:10)
at Transform._write (E:\bingo-master\node_modules\readable-stream\lib_stream_transform.js:172:83)
at doWrite (E:\bingo-master\node_modules\readable-stream\lib_stream_writable.js:428:64)
at writeOrBuffer (E:\bingo-master\node_modules\readable-stream\lib_stream_writable.js:417:5)
at Writable.write (E:\bingo-master\node_modules\readable-stream\lib_stream_writable.js:334:11)
at DestroyableTransform.ondata (E:\bingo-master\node_modules\readable-stream\lib_stream_readable.js:619:20)
at DestroyableTransform.emit (node:events:513:28)
at DestroyableTransform.emit (node:domain:552:15)
at addChunk (E:\bingo-master\node_modules\readable-stream\lib_stream_readable.js:291:12)
Node.js v18.8.0

Try with older Node.js version.

how older?

happened same with node v16.20.2, do you remember what version you used?

Probably 14x....

@jbello94 Did it work?

@jbello94 Did it work?

Nop 😕

Which versions of Node and npm have you tried with?


ElvUI Dev officially support boss6+ (in development)

Compare

We just officially added support for boss6+ which doesn't require OnUpdate UpdateAll; this patch will let the boss6+ frames handle like *target units (which get updated at a set interval)
To prevent incompatibly we've blacklisted this when they are not using SUF in order to prevent FPS loss of firing UpdateAll at a specific rate (hitting every element).

an update, we released 12.72 which means, this change was put in place and this addon will automatically be disabled when the user is not using SUF as stated above.

That's great to hear that ElvUI has built-in support!  I'll remove ElvUI from the addon in the future, but for now I've updated the README.


(LCS Command): (Does not support IDX return type)

Describe the feature
Currently LCS command supports Integer and String return types. For the IDX return type it will return an array of objects instead. A more elegant solution would be to implement an IDX data type specific for this usage.
Use Case
An elegant api suggestion.
Proposed Solution
No response
Other Information
No response
Acknowledgements

[ ] I may be able to implement this feature request
[ ] This feature might incur a breaking change

Client version used
1.0
Environment details (OS name and version, etc.)
Windows 10

Completed by #1716 and #1660


Sniper is unaffected by soldier syringe?

Can someone clarify how exactly it works with Sniper? Does it increase reload window or smth? Is there any article/readme file to check how different items affect Sniper?
Also, thanks for this awesome mod! Character is very balanced and its playstyle is absolutely fantastic!

Currently, Syringes only increase the charge speed of Steady Aim. In the next update, the cooldown of SPOTTER: Feedback will be affected by attack speed as well.
Once the DLC releases, I plan to have Sniper's good/perfect reload window increase in size with attack speed, as well as adding the weak point mechanic in some form.
Glad to hear you're enjoying the mod!


Cannot download images on windows 10

Expected behavior
Should be able to download images from docker hum
Actual behavior
(HTTP code 500) server error - {"message":"Get https://registry-1.docker.io/v2/: dial tcp: lookup registry-1.docker.io on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:57254-><IP_ADDRESS>:53: i/o timeout"}
Information about the Issue
Steps to reproduce the behaviour

Install docker on windows 10 and try downloading an image
.

Please make sure that you're on a stable connection and use a solid DNS such as CloudFlare DNS <IP_ADDRESS>
https://blog.cloudflare.com/announcing-1111/


About us or Contact us Page

An About Us or Contact Us page is really necessary when it comes to a service-based website such as Kaam Do, and I am skilled at this task and will start working on this immediately. Kindly assign this task to me under GSSoC'24.

@ananyabansal16 Contactus Page is assigned to Someone ! Meanwhile you can Go Ahead with Aboutus Page !
Design good One


Using training or validation data for the MMN network updates?

Hi, thanks for the nice work and code!
According to the paper, the MMN network is updated based on the super-networks on the validation data (see https://papers.nips.cc/paper/2020/file/d072677d210ac4c03ba046120f0802ec-Paper.pdf close to the end of page 4).
On the other hand, in the code, the MMN network is updated in https://github.com/microsoft/Cream/blob/main/lib/core/train.py#L42 and uses training data.
Am I missing something?

Hi,
Thanks for your interest in our project!
Different slices of current batch are used in the meta network update process.
Training: https://github.com/microsoft/Cream/blob/e0de68affb0827b2a0108e7afa633dbc97b233f0/lib/models/MetaMatchingNetwork.py#L50
Validation: https://github.com/microsoft/Cream/blob/e0de68affb0827b2a0108e7afa633dbc97b233f0/lib/models/MetaMatchingNetwork.py#L86
Best,
Hao.

Hi,
thanks for the quick response!
I was uncertain whether an entirely different validation set is used, or whether training data can be used both for matching and MMN training, thanks for the clarification.
Best,
Kevin

Hi,
Thanks for your interest in our project!
'Validation' used in the update of MMN is not from ImageNet Val, instead images in 'Validation' are sampled from ImageNet Train. To summarize, in the process of updating MMN, we split ImageNet TrainSet into Train and Val, only for MMN. ImageNet Val images are only used in the process of calculating validation loss of supernet.
Best,
Hao.


Made changes in the font properties

Along with the fonts, I have also changed some of the properties of logo image.

try changing the font/color/ ... for category mentioned in projects as for food project its rarely visible

I have made the changes

issue #98

resolve conflict i'll merge this

Please give me time till 8 july. I will surely resolve all the conflicts._


Multiple calls to mount().update() required for testing async componentDidMount

Current behavior
My componentDidMount() does something like:
    this.setState({
       data: .....
    });

    try {
        const data = await fetchDataAsync();
        this.setState({
           data: ...
        });
    } catch (error) {
        this.setState({
            error: `Error fetching repos: ${error.message}`
        });
    }

When I try to test this component is properly setting the error state if  fetchDataAsync() fails, I need to call wrapper.update() 3 times to get my test to pass:
    it('sets error state when failed to fetch data', async () => {
        const error = {
            message: 'an error message'
        };

    apiMock.fetchDataAsyn = jest.fn().mockImplementation(() => Promise.reject(error));

    const wrapper = mount(<Component />);
    // this is required 3 times to get the state to update properly
    await wrapper.update();
    await wrapper.update();
    await wrapper.update();
    expect(wrapper.state().error).toMatch(error.message);
});

Expected behavior
Not to have to call wrapper.update() 3 times to have the state updated
Your environment
Mac OS X
react-scripts 1.1.1
API

[ ] shallow
[X ] mount
[ ] render

Version

library
version

Enzyme
3.3.0

React
16.2.0

Jest
22.0.4

Adapter

[X ] enzyme-adapter-react-16
[ ] enzyme-adapter-react-15
[ ] enzyme-adapter-react-15.4
[ ] enzyme-adapter-react-14
[ ] enzyme-adapter-react-13
[ ] enzyme-adapter-react-helper
[ ] others ( )

Just realised I was using update() totally the wrong away, my problem is the same as in #1581, I end up having to use a setTimeout on all tests which trigger some async operation and it feels really strange...
I thought update() could help me somehow, but it does not seem so and I do not understand what is its intended use case, but I guess that is another issue, this should be closed as a duplicate of #1581


Fix for registering entity via scaffolder catalog:register action results in warnings

resolves #17818
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Do not close. Still waiting for review.

I disagree with this fix.
I'd say that explicit manual registration of an individual file (either via Create on the Backstage page, or via scaffolder actions) is a very strong signal that we now claimed "ownership" of this entity and we absolutely do not want anybody else to be able to mess with it.
With your change, an unsuspecting admin user might register a very core piece of infrastructure or whatnot believing that their action is permanent, and then some malicious (or unwitting) actor manages to stampede over that registration by just throwing a yaml file anywhere that is subject to discovery, overwriting that same entity ref, without any warnings or notifications to anyone that it's been attempted.
It's almost to the point that I'm not sure what a good fix would even be. If you mix the world of registration and discovery, you are bound to get collisions. I'd argue that the registrations should always win - they are the primary mode of operation and the most intentful one.

I seem to have misunderstood the purpose of the catalog:register scaffolder action. Our intention was to have the catalog updated immediately after creating a new catalog file, instead of waiting for the next discovery. This would enhance usability and allow for better testability through integration tests.
What would be the right way to have an entity visible in the backstage immediately after running a scaffolder?

catalog:register adds a singular URL (into the new repo you just created) into the locations table, which in turn gets picked up instantly by an entity provider which creates a corresponding kind: Location entity out of it. Some time later, when that location gets processed, the URL it actually pointed to is read and turned into the final entities that were in that YAML file.
This is not perfectly instantaneous; it's a kind of asynchronous process behind the scenes.
This way of getting entities into the catalog is also completely separate from the "competing" process of instead using a discovery provider (they are two different provider implementations that don't overlap in any way with each other).
If I understand things correctly you are already using discovery and would like to continue using discovery, is that correct? If so, what you would need is some way of signalling to that provider that there's new data for it to discover, so that it runs sooner than its next scheduled interval. Some providers implement event listening such that they can react to webhooks of files getting created in the VCS. That's the ideal path forward, but the events system is still fairly young and not all providers implement support for it.

If I understand things correctly you are already using discovery and would like to continue using discovery, is that correct?

Yes we use the GitlabDiscoveryEntityProvider

If so, what you would need is some way of signalling to that provider that there's new data for it to discover, so that it runs sooner than its next scheduled interval. Some providers implement event listening such that they can react to webhooks of files getting created in the VCS. That's the ideal path forward, but the events system is still fairly young and not all providers implement support for it.

Ok this means that the VCS is required to support webhooks for the relevant events AND that every backstage instance is registered for these webhooks (making development much harder in terms of localhost)
What do you think about a mechanism, to trigger each EntityProvider via a new scaffolder action (e.g. catalog:promote). This will:

iterate through each known EntityProvider
ask this provider if it can handle the promoted URL
if so the provider starts discovery for the new URL

So we need a new interface, which can be implemented by the EntityProviders which support promotion of newly created catalog descriptors. Since each EntityProviders will know how to handle an URL and so can trigger the relevant discovery.

It'd be a new mechanism and one that'll take a fair bit of consideration.
First, messages need to reach the consumers somehow. Remember also that the consumers are distributed. We could invent an HTTP endpoint in the catalog service itself that then fans it out to all of the entity providers it knows of. But that is a fairly tight coupling and a narrow interface. I'd prefer if we could instead make it so that this relies on the event bus, just a new message.
Second, the semantics of the message sent needs to be settled. It may seem simple but there are different sets of parameters that make sense for different sets of providers. How do you notify the LDAP provider and the GitLab provider uniformly, as well as any complex provider out there that we don't even know what it looks like because it's closed source? It'll fairly likely turn out to be provider specific for better or worse.
Third, remember that these are distributed, asynchronous workers. They often run on horizontally scaled nodes across datacenters. If host A in the US receives the message and does a small delta update, while host B in Asia is doing the full scan and then issues a full update, B will destroy the result that A posted.
It's tricky. We highly value discussion like this and want to find great cohesive solutions for sure, so don't be too discouraged. But it may need to start in the form of some RFC and then notifying the catalog SIG about it and seeing where it goes.

@freben please can you add / notifying the catalog SIG to / about #18160

As pointed out by @freben, this PR does not solve the underlying problem. It would only deactivate the warning and thus cause even more problems due to entity conflicts. A potential solution to my problem is defined in #18160, so I close this PR now.


regarding PropTypes in reactjs Documentation

PropTypes are not valid
use propTypes
p should be small
showing error while using capital

I think in the previous versions of reactjs it is not an issue but now it logs errors

Could you point to specific section of a page that contains this typo?


make $*ERR unbuffered

When we switch from OUT to ERR for diagnostics, we took away the buffering, so now the ERR output shows up after the work is done where it's useless.

Will be available in release after 0.9.3


Add permalinks, doc links and original source code link URLs to all ta rules

@AndjaKocic , @holderfolyf - would it be feasible to expand this to all the other ta rules?

@AlejandroGuiradoGarcia - if of interest you could assist with this?


Nullable false not working

Prerequisites

[X] I have written a descriptive issue title
[X] I have searched existing issues to ensure the bug has not already been reported

Fastify version
4.9.2
Plugin version
4.1.0
Node.js version
18.5.0
Operating system
Windows
Operating system version (i.e. 20.04, 11.3, 10)
10
Description
I tried to not allow null in string but it's not working:
How can I not allow null, even string array like this: [null]
Steps to Reproduce
Code that I tested not working
S.string()
S.string().raw({nullable: true})
S.string().not(S.null())

Workaround:
S.ifThenElse(S.null(), S.not(S.null()), S.string())

Expected Behavior
No response

Could you add a Minimal, Reproducible Example?
Without it, we are unable to help you.
This module generates json schema, so that code generates:
{$schema: "http://json-schema.org/draft-07/schema#", type: "string", nullable: true}

https://runkit.com/embed/wdwudrwu14aa


Allow passing of extra keyword arguments to child classes

I'm currently trying to add https://github.com/rsinger86/drf-flex-fields serializers as children to my Polymorphic Serializers. However they rely on extra keyword arguments being passed in to the children on init. This is something Polymorphic can't really handle right now.
I would like to propose a new keyword argument called child_kwargs which is a set of kwargs that will be passed to the children along side the normal kwargs.
A PR is on its way for this.

PR made: https://github.com/apirobot/django-rest-polymorphic/pull/34

+1.
Being able to use drf-flex-fields with polymorphic is very much needed.

Useful mention: https://github.com/rsinger86/drf-flex-fields/issues/12#issuecomment-770628228


Move Mk2Expansion to Spacedock

Should pick up the 1.1 release there.

Failed due to dependency not available for KSP 1.1, merging as will not work in 1.0.5


Chore: Improve prettierrc to be more specific

You might want to extract the prettier config from this repo entirely, so that it can be shared across more repos (or all of SEB)? We've done it for some internal repos that all share the same config as a npm package.
See Prettier: Sharing configuration for more info. The only downside is that it doesn't seem to be any way to share the .prettierignore file, but maybe that is by design(?).


BBC appears to be down again (cont)

Further to Issue 104:
A similar, but slightly different failure.
After the 3 second "channel splash", just left with the spinning "loading" icon at program start.
There was a 403 on request for _http://vod-hds-uk-live.edgesuite.net/crossdomain.xml_.
Added vod-hds-uk-live.edgesuite.net to zones.override and all OK now.
I am not at all familiar with BIND. Is it possible to "wildcard" zone entries? eg " vs-hds-uk-live.* " ?

Cheers, I've added this and other domains now to the file.
Read: http://serverfault.com/questions/229085/wildcard-dns-with-bind


Initialize Allocation to eliminate compile error

Class Allocation is not initialized and I will get a compile error like this.
error: ‘worklist.absl::lts_20220623::InlinedVector<absl::lts_20220623::cord_internal::CordRep*, 2, std::allocator<absl::lts_20220623::cord_internal::CordRep*> >::storage_.absl::lts_20220623::inlined_vector_internal::Storage<absl::lts_20220623::cord_internal::CordRep*, 2, std::allocator<absl::lts_20220623::cord_internal::CordRep*> >::data_.absl::lts_20220623::inlined_vector_internal::Storage<absl::lts_20220623::cord_internal::CordRep*, 2, std::allocator<absl::lts_20220623::cord_internal::CordRep*> >::Data::allocated.absl::lts_20220623::inlined_vector_internal::Storage<absl::lts_20220623::cord_internal::CordRep*, 2, std::allocator<absl::lts_20220623::cord_internal::CordRep*> >::Allocated::allocated_capacity’ may be used uninitialized [-Werror=maybe-uninitialized]

We need the CLA.
I can't reproduce this problem. Can you provide reproduction instructions?

Exactly, the error is a warning. But I use -Werror, so it failed to compile.

Reproduction instructions need to include the compiler version and operating system. Please provide this information.
-Werror is unsupported. You should only compile your own code with -Werror. We have a CMake option ABSL_USE_SYSTEM_INCLUDES for this.
We should fix this though. Please let us know the exact version of the tools used.

Reproduction instructions need to include the compiler version and operating system. Please provide this information.
-Werror is unsupported. You should only compile your own code with -Werror. We have a CMake option ABSL_USE_SYSTEM_INCLUDES for this.
We should fix this though. Please let us know the exact version of the tools used.

cmake: 3.20.2
GCC: 11.2.1
Linux OS: CentOS Linux release 7.9.2009 (Core)
LInux Kernel: Linux ost 5.10.104-linuxkit #1 SMP Thu Mar 17 17:08:06 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

-Werror is unsupported. Maybe it is better to support Werror, isn't it?

Maybe it is better to support Werror, isn't it?

Supporting -Werror means we have to support whatever arbitrary set of warnings people decide to enable, even bad or buggy warnings. It's fine to compile your own code with -Werror if you want, but don't force -Werror on your dependencies. So no, we will never support -Werror.

Get it

Any more questions to merge this code?


Recognising wrong output and also "onSpeechPartialResults" is returning only 1 word in array

Scenario: I've two options a) female b) male
Everytime I say male it returns "Mail". Is there anyway where I can get all the similar results which sound like male/mail from this library?
@wenkesj @lucianomlima

In Android, partial results returns all the words of similar sound but not in the case of iOS

Sorry, I never used this lib on RN.

Hello @Nehal-29! Could you provide a reproducible demo, either another repo or in the example app so I can test it out?


Support rename operation for MaterializeView storage

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en
For changelog. Remove if this is non-significant change.
Category (leave one):

New Feature

Short description (up to few sentences):
Previously, a command RENAME TABLE materialized_view TO another_name returned an exception Not Implemented with a message Method rename is not supported by storage MaterializedView. This PR add a potential implementation.
#4295
Detailled description:
DROP DATABASE IF EXISTS test;
CREATE DATABASE test;

use test;

DROP TABLE IF EXISTS materialized_source_table;
DROP TABLE IF EXISTS materialized_view_table;

CREATE TABLE materialized_source_table (
    `id` UInt32,
    `date_partition` Date,
    `val` UInt32
) Engine = MergeTree()
PARTITION BY date_partition
ORDER BY date_partition;

INSERT INTO materialized_source_table VALUES(0, toDate('2018-01-01'), 1);

CREATE MATERIALIZED VIEW materialized_view_table
ENGINE = AggregatingMergeTree() PARTITION BY toYYYYMM(date_partition) ORDER BY (date_partition)
AS SELECT
    date_partition,
    minState(val) AS min_value_state
FROM materialized_source_table
GROUP BY date_partition;

INSERT INTO materialized_source_table VALUES(13, toDate('2019-04-01'), 20);

RENAME TABLE materialized_view_table TO materialized_view_table_migrate;

SHOW TABLES

┌─name───────────────────────────────────┐
│ .inner.materialized_view_table_migrate │
│ materialized_source_table              │
│ materialized_view_table_migrate        │
└────────────────────────────────────────┘

find . | grep materialized_view_table_migrate

./dbms/programs/server/data/test/%2Einner%2Ematerialized_view_table_migrate
./dbms/programs/server/data/test/%2Einner%2Ematerialized_view_table_migrate/format_version.txt
./dbms/programs/server/data/test/%2Einner%2Ematerialized_view_table_migrate/201904_1_1_0
./dbms/programs/server/data/test/%2Einner%2Ematerialized_view_table_migrate/201904_1_1_0/columns.txt
./dbms/programs/server/data/test/%2Einner%2Ematerialized_view_table_migrate/201904_1_1_0/count.txt
./dbms/programs/server/data/test/%2Einner%2Ematerialized_view_table_migrate/201904_1_1_0/date_partition.bin
./dbms/programs/server/data/test/%2Einner%2Ematerialized_view_table_migrate/201904_1_1_0/primary.idx
./dbms/programs/server/data/test/%2Einner%2Ematerialized_view_table_migrate/201904_1_1_0/date_partition.mrk
./dbms/programs/server/data/test/%2Einner%2Ematerialized_view_table_migrate/201904_1_1_0/min_value_state.mrk
./dbms/programs/server/data/test/%2Einner%2Ematerialized_view_table_migrate/201904_1_1_0/min_value_state.bin
./dbms/programs/server/data/test/%2Einner%2Ematerialized_view_table_migrate/201904_1_1_0/checksums.txt
./dbms/programs/server/data/test/%2Einner%2Ematerialized_view_table_migrate/201904_1_1_0/partition.dat
./dbms/programs/server/data/test/%2Einner%2Ematerialized_view_table_migrate/201904_1_1_0/minmax_date_partition.idx
./dbms/programs/server/data/test/%2Einner%2Ematerialized_view_table_migrate/detached
./dbms/programs/server/metadata/test/%2Einner%2Ematerialized_view_table_migrate.sql
./dbms/programs/server/metadata/test/materialized_view_table_migrate.sql

Mostly Ok.
A few concerns remain, you can address them in a "follow up" pull request.


ConvexPolygonShape2D can't be edited with mouse like CollisionPolygon2D

Godot version:
3.1.dev.f72f744
Issue description:
If you select ConvexPolygonShape2D a as collision shape, you can't draw its points with mouse, you can only manually enter their coordinates in the Inspector, which is tedious.
Compare to CollisionPolygon2D which allows to use mouse to create a polygon.
Steps to reproduce:

Create 2D scene
Add CollisionShape2D
Select ConvexPolygonShape2D as its shape
Click anywhere on the scene hoping to start drawing polygon
Nothing happens

It's still true on 3.2.1 stable for both ConcavePolygonShape2D and ConvexPolygonShape2D

And also on 3.2.2 beta1

Yeah, I recall myself stumbling upon exact issue, but I'm not sure whether this should be a feature, because for polygon-based shape editing Godot has CollisionPolygon2D node specifically, which also handles:

proper orientation of polygon being built (vertices could be set in either clockwise of counterclockwise order, which is important both for rendering and probably physics too, because inertia should be computed from polygon area, see #29867).
convex/concave polygons, where polygons are decomposed into convex or remain segment-based.

So, there are several options to address this issue:

Actually implement it.
Remove ConvexPolygonShape2D and ConcavePolygonShape2D from the resource selection popup.
A mixture of (1) and (2), read on.

For simplistic polygon shape editing (which do not relate to physics, perhaps gameplay needs), this does make sense. Yet there are some inconsistencies between ConvexPolygonShape2D and ConcavePolygonShape2D, namely the former is represented as points, while the latter is represented as discontinuous segments, and makes it somewhat difficult to edit. This might add some extra complexity (which is possible to resolve), yet #32979 aims to fix it anyway.
To be honest, I don't quite see the point of having different convex/concave shapes to begin with, I'd just implement a single PolygonShape2D, add perhaps adding an enum to specify and/or calculate the convexity of polygons, just like in CollisionPolygon2D node with the Build Mode property. -1 class for Godot, less bloat. 😛

I've also expressed my views in the proposal at https://github.com/godotengine/godot-proposals/issues/1126#issuecomment-651185404, and could likely work on fixing this already in Godot, just need a blessing from other core devs.

Figured out how to edit concave (as well as convex) shapes through a similar node: https://github.com/GoostGD/goost/pull/2#issuecomment-652667575. I think it would be nice to add this feature to Godot, just because this would promote ease of use for plugins/modules (currently I have to edit the shape through another class, from which the shape is then assigned to CollisionShape2D, or it has to be first saved to disk as a resource and then set for both classes to propagate changes, which is not intuitive).

Upon further research I figured there's CollisionShape2DEditor which handle most common types except polygon types:
https://github.com/godotengine/godot/blob/b65ebfc7e720a947d3dab16824aa9975d50c5771/editor/plugins/collision_shape_2d_editor_plugin.cpp#L148-L152
But I find implementing polygon editing directly within this class to be problematic because there's already AbstractPolygon2DEditor which has it's own editing logic for that...
So, might be just better and easier to create CollisionShape2DPolygonEditor and register the plugin alongside CollisionShape2DEditor, and provide some comments about that.

Some discussion from IRC #godotengine-devel channel:
Xrayez> reduz: does it makes sense to fix/implement #21394?
Xrayez> If so I could work on that. Currently it's only possible to do this via CollisionPolygon2D, but not CollisionShape2D.
reduz> Xrayez: i think there is no benefit in doing that to be honest, this is a shape that only makes sense to use via code
Xrayez> reduz: would it make sense to remove ConvexPolygonShape2D and ConcavePolygonShape2D from the resource popup in CollisionShape2D?
Xrayez> so that users don't get confused, and go straight to CollisionPolygon2D node for that.
reduz> No, I would just put a configuration warning if you put a convexpolygon in a collision shape, saying this type of shape can't be edited manually, and suggest using a CollisionPolygon node instead
reduz> I mean we could make an editor, but whats the point
Xrayez> Well that's fairly easy to add since there's a AbstractPolygonEditor.
Xrayez> That's mainly due to https://github.com/godotengine/godot-proposals/issues/1126
Xrayez> Which wouldn't be solved on the core, but would allow consistency between what the core provides and modules.
reduz> not so easy because you need to ensure the polygon is always convex
reduz> so again I would not bother
reduz> adding a warning to avoid confusion imo is best
Xrayez> Ok I'll go for that.

I came here after some googling because the warning is actually confusing. I'd vote for removing the option in the droplist. Unless there is a use-case for not removing it ?

@Mayt38 The option can't be removed from the dropdown as it displays all resources that inherit from the "expecte" type.


On add action button click open add action popup

Done. Please review.

add action will open popup to add link, other option will be disable in that case.

Done.


Update README.md

This commit conflicts with the README, can you update your pull request by resolving the conflicts?


Laravel 5 Support

Is there any plan to upgrade to Laravel 5 ?

Hi,
Yes, but no sooner than 2-4 months


enhance: useResource array parameters as readonly

Makes it easier to pass tuples defined with "as const" assertion as paramters.

Fixes # .
Motivation

export const compResources= [
  [SomeResource.listShape(), {}],
  [OtherResource.listShape(), {}]
] as const;

const Comp = () => {
  const [someData, otherData] = useResource(...compResources);
}

The arguments passed to useResource() will be readonly [S, P], which errors due to the useResource array arguments being defined without readonly modifier.
Solution

Add the readonly modifier to all array arguments of useResource: useResource(readonly [S1, P1], ...).
There's no reason not to add this modifier, as these arrays should not be mutated inside this function anyway.
Open questions

That was quick! Thanks 😄

May want to have the same for all the hooks

Released in 5.0.0-beta.0 (0 is after beta.4 - accidentally missed it)


Reland "Migrate benchmarks to package:web"

This attempts to reland https://github.com/flutter/flutter/pull/126848
This was reverted because it made some unexpected changes to our perf measurements. After landing https://github.com/flutter/flutter/pull/127900, we have much less noise in our benchmarks, so I'd like to reland this and see if there is still a significant measurement difference.

You have some import errors:
warning • Unused import: 'package:web/web.dart' • dev/benchmarks/macrobenchmarks/lib/src/web/bench_platform_view_infinite_scroll.dart:10:8 • unused_import
  error • Undefined class 'Element' • dev/benchmarks/macrobenchmarks/lib/src/web/bench_platform_view_infinite_scroll.dart:18:11 • undefined_class
  error • Undefined name 'html' • dev/benchmarks/macrobenchmarks/lib/src/web/bench_platform_view_infinite_scroll.dart:18:38 • undefined_identifier

https://ci.chromium.org/p/flutter/builders/try/Linux analyze/57506?

You have some import errors:
warning • Unused import: 'package:web/web.dart' • dev/benchmarks/macrobenchmarks/lib/src/web/bench_platform_view_infinite_scroll.dart:10:8 • unused_import
  error • Undefined class 'Element' • dev/benchmarks/macrobenchmarks/lib/src/web/bench_platform_view_infinite_scroll.dart:18:11 • undefined_class
  error • Undefined name 'html' • dev/benchmarks/macrobenchmarks/lib/src/web/bench_platform_view_infinite_scroll.dart:18:38 • undefined_identifier

https://ci.chromium.org/p/flutter/builders/try/Linux analyze/57506?

Thanks! Yeah I had an issue with my rebase, I think I got it resolved.

Unfortunately, the benchmark runs from the try bot still show the discrepancy on totalUiFrame: https://ci.chromium.org/ui/p/flutter/builders/try/Linux web_benchmarks_html/21744/overview
Looking a little closer to see if I can find the culprit.


header and footer settings are ignored

Hi,
I would make a own footer with picture, like the example from theme guide. But I see heade and footer settings are ignored by asciidoctor-pdf. I looked for the default theme in repository and see the definiton of footer brings page numbers to PDF. But I tested this with simple document then I get no page numbers.
The definition from default theme in repo:
# NOTE In addition to footer, header is also supported
footer:
  font_size: $base_font_size_small
  font_color: $base_font_color
  # NOTE if background_color is set, background and border will span width of page
  border_color: dddddd
  border_width: 0.25
  height: $base_line_height_length * 2.5
  padding: [$base_line_height_length / 2, 1, 0, 1]
  vertical_align: top
  #image_vertical_align: <alignment> or <number>
  # additional attributes for content:
  # * {page-count}
  # * {page-number}
  # * {document-title}
  # * {document-subtitle}
  # * {chapter-title}
  # * {section-title}
  # * {section-or-chapter-title}
  recto_content:
    #right: '{section-or-chapter-title} | {page-number}'
    #right: '{document-title} | {page-number}'
    right: '{page-number}'
    #center: '{page-number}'
  verso_content:
    #left: '{page-number} | {chapter-title}'
    left: '{page-number}'
    #center: '{page-number}'

The simple document:
= The Book's Title
:author: First Last
:email<EMAIL_ADDRESS>:toc:
:toc-placement: preamble
:doctype: book

This journey begins on a bleary...

== Cavern Glow

The river rages through the cavern, rattling its content...
The river rages through the cavern, rattling its content...

The river rages through the cavern, rattling its content...

The river rages through the cavern, rattling its content...

The river rages through the cavern, rattling its content...

The river rages through the cavern, rattling its content...

The river rages through the cavern, rattling its content...

v
The river rages through the cavern, rattling its content...

v
The river rages through the cavern, rattling its content...

The river rages through the cavern, rattling its content...

The river rages through the cavern, rattling its content...

The river rages through the cavern, rattling its content...

The river rages through the cavern, rattling its content...

v
The river rages through the cavern, rattling its content...

== Cavern Glow 2

The river rages through the cavern, rattling its content...

=== FF

v
The river rages through the cavern, rattling its content...

[source,bash,numbered]
----
ln -s /opt/redmine/.ssh/redmine_gitolite_admin_id_rsa /opt/redmine/redmine/plugins/redmine_git_hosting/ssh_keys/redmine_gitolite_admin_id_rsa
ln -s /opt/redmine/.ssh/redmine_gitolite_admin_id_rsa.pub /opt/redmine/redmine/plugins/redmine_git_hosting/ssh_keys/redmine_gitolite_admin_id_rsa.pub
ln -s /opt/redmine/.ssh/redmine_gitolite_admin_id_rsa /opt/redmine/.ssh/id_rsa
ln -s /opt/redmine/.ssh/redmine_gitolite_admin_id_rsa.pub /opt/redmine/.ssh/id_rsa.pub
----

v
The river rages through the cavern, rattling its content...

The river rages through the cavern, rattling its content...

The river rages through the cavern, rattling its content...

The river rages through the cavern, rattling its content...

The river rages through the cavern, rattling its content...

v
The river rages through the cavern, rattling its content...

Now I call:
asciidoctor-pdf myad.adoc -o toll.pdf

The resulting document has no page numbers in their footer.
The Version:
$ asciidoctor-pdf -v
Asciidoctor PDF 1.5.0.alpha.8 using Asciidoctor 1.5.2 [http://asciidoctor.org]
Runtime Environment (ruby 2.1.6p336 (2015-04-13 revision 50298) [x86_64-linux-gnu]) (lc:UTF-8 fs:UTF-8 in:- ex:UTF-8)

Is this an failure in asciidoctor-pdf or do I a mistake in using?
Best regards

I also was in the same situation. By adding ":pagenums:" to adoc file, I resolved this problem.

This helps thanks. Its activate header and footer

We are discussing making the pagenums attribute enabled by default, though I think perhaps we should also consider activating the running header/footer by default and allowing it to be disabled using the existing noheader and nofooter attributes.

It is possible to add a date or revision attribute to header or footer?

Absolutely. You can use any AsciiDoc attribute in the content. In fact, the content itself is parsed as an AsciiDoc paragraph.
To get the date, just use {docdate}. To use revision, just use {revnumber}. Of course, you can also define your own attribute in the AsciiDoc document and then reference it in the header/footer content.
Perhaps we should emphasize this point with an example in the theming-guide.adoc. Would you like to propose a change? (see https://github.com/asciidoctor/asciidoctor-pdf/blob/master/docs/theming-guide.adoc#implicit-attributes).

thanks. I think adding to guide is an good idea.

I've updated the theming guide with the proposed changes.


Multiple problems with ender dragon entity

I am currently running

SpongeVanilla version: 1.12.2-7.0.0-BETA-372

Java version: 8

Operating System: linux

Plugins/Mods: none

problem (see scc below)

Interact with block
Entity is spawned on the server side but client receives no packet
to see the dragon player have to re-login.

problem

player is kicked after dismounting (pressing shit) and cannot login back.

After dismounting:
[21:06:09 INFO]: NeumimTo lost connection: Invalid move player packet received
[21:06:09 INFO]: NeumimTo left the game
[21:06:09 WARN]: handleDisconnection() called twice

After reloging server kicks me with following log:
[20:58:20 INFO]: NeumimTo joined the game
[20:58:21 INFO]: NeumimTo lost connection: Invalid move player packet received
[20:58:21 INFO]: NeumimTo left the game

    @Listener
    public void start(InteractBlockEvent event, @Root Player player) {
        Entity dog = player.getWorld().createEntity(EntityTypes.ENDER_DRAGON, player.getLocation().getPosition());
        Sponge.getScheduler().createTaskBuilder().execute(() ->
                dog.addPassenger(player)
        ).submit(this);
    }

    @Listener
    public void blockEntityDamage(DamageEntityEvent event, @Root EnderDragon enderDragon) {
        event.setCancelled(true);
    }

    @Listener
    public void blockWorldDamage(ChangeBlockEvent event, @Root EnderDragon enderDragon) {
        event.setCancelled(true);
    }

Try to spawn the entity via world.spawnEntity(Entity e), from EntityUniverse, before adding it's passenger.

That helped with the first problem, but dismounting remains broken

I've traced it down to a "divide by zero" error in EntityDragon.

Fixed in 01f6126d4b8229ca019b4f81293320d768ca6dc3.


[NEW] Support collapse/uncollapse quotes, replies and long text attachments

Your Rocket.Chat.Android version: 2.0.0 beta 6
Quoted messages are cut off after certain count of characters which at times make it difficult to read messages which are coming via webhooks. For example, comments on issues from GitLab etc.

@rafaelks

@mmarif4u It's merged already. 👍

Hi @rafaelks, i am on rc1 now and i still see the same cut off messages. You can check the channel i shared with you via DM. Let me know if you need more info.

@mmarif4u the reply/quote is cut off or the message is cut off?

@mmarif4u I see. The issue @rafaelks is talking about and yours are different ones.
As I suspected before from your pictures it's coming from bot and treated as quotes as you said.
The quote being cut off was intended behavior to reduce the noise on the chat but as it seems we didn't count on webhooks messages.
Anyway we're going to address this soon probably by making collapsable/expandable quotes.

@leonardoaramaki yes, you are right. Its from bot.
I understand sometime it would be considered noise. But at times, like comments on issues(mostly we use RC to check while out, coming through bots) are helpful to read straight away instead of going to GitLab interface in this case.
I appreciate considering this to implement even if collapsible/expandable. :+1:

@mmarif4u Can you please confirm if this is still happening in 2.1.0?

Yes, its still there in 2.1.0.

Hi there,
the same cut off or even worst is happening with Trello webhook integrations.
Please, have a look at the images below for more info:
How it is shown on Android app

How it is shown on Rocket.Chat desktop app

Server Setup Information:
Rocket.Chat Server: 0.64.1
Operating System: Linux
Mobile device model and OS version:

Google Pixel 2 - Android 8.1.0

Best regards,

@leonardoaramaki any updates on this?.
Also links are now shown as attachments which can be seen above. This was introduced in 2.1.0 while i guess enhancing the file objects.

@mmarif4u it's on our queue of tasks, but we haven't started fixing this yet. I appreciate your patience anyhow. This issue is already on our roadmap.

Closed by https://github.com/RocketChat/Rocket.Chat.Android/pull/1695

Hi Guys,
This is still an issue :(
Any tips you can give on how to fix or if an option exists to disable?


Cannot manage to get past ngrok step

I get constantly the same error when adding my ngrok url to my AwwBot

Validation errors:
interactions_endpoint_url: The specified interactions endpoint URL could not be verified.

Used claude to guide me through all steps still - no cigar.
Wondering if anyone here has a tip for me on what could be causing it?

Have you set all the environment variables?

Ahh - you'll want to submit the Cloudflare Workers URL instead then, but just make sure you've published the worker properly too!!

Will give it another shot later - I must succeed!!!


MPI3D dataset Link is down

Hi,
I Really like the dataset!
Is it maybe possible to update the link for the mpi3d_toy, mpi3d_real, and mpi3d_realistic?  The old ones (google cloud links) can not be accessed anymore.  A copy of it is available online but it does not contain all ground truth labels. Thank you very much!
UserProjectAccountProblemThe project to be billed is associated with a closed billing account.The billing account for the owning project is disabled in state closed

Best regards,
Qin

Hi Qin
I found the same problem, but I guess the order of the images is simple as described in the README.

The datasets are provided in the form of numpy arrays. Once the data is loaded, you can use array.reshape([6,6,2,3,3,40,40,64,64,3]) to obtain an array where the first 7 dimensions corresponds to data generative factors as in the table above

So, we can probably retrieve the original label just by reshape. The code here does it. I'm not sure the mirrored dataset you posted keeps the original order though.


fix memory error in element integration

This PR fixes a bug found by @kswartz92 where some element geometries were trying to read floating point values from the {source/flux} outputs from the q-function, even when one of them was a serac::zero type.
Unfortunately, I already fixed this bug a year ago for the simplex elements https://github.com/LLNL/serac/pull/873 (https://github.com/LLNL/serac/pull/873/commits/03281a80e95e30a50ca43504177459026340f8c2) but neglected to apply the fix to the remaining element geometries. I'm sorry for any LIDO headaches caused by this.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (bb0a463) 88.78% compared to head (2a69e6f) 88.82%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1072      +/-   ##
===========================================
+ Coverage    88.78%   88.82%   +0.03%
===========================================
  Files          153      153
  Lines        12389    12385       -4
===========================================
+ Hits         11000    11001       +1
+ Misses        1389     1384       -5

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


listAll() problem

Hey, when I list the datas with listAll(), the response is :
package.name@421d9f28,
package.name@421da700,
package.name@421dad40
My database contain :public Bets(Context context, int betid, int dicef, int dicet, int dices, int betamount,  String worl)
How can I correct this ?
Thanks.

I believe you're using an outdated version of the library. If the master branch doesn't fix your issues, let me know.


ffmpeg_7: fix darwin build

Description of changes
FFmpeg 7 on x86_64-darwin currently fails to compile with AV Foundation enabled, so disabling for x86_64-darwin on ffmpeg 7 or newer.
Also, xeve and xevd fail to compile with non GCC compilers, so disabling them on non gnu cc.
A Test for ffmpef_7-full on Darwin both architectures is currently still failing.
ast 10 log lines:
       > HOSTCC tests/audiogen.o
       > HOSTCC tests/videogen.o
       > COPY   tests/data/filtergraphs/anequalizer
       > COPY   tests/data/filtergraphs/firequalizer
       > COPY   tests/data/filtergraphs/compand
       > GEN    tests/data/hls-list.m3u8
       > make: *** [tests/fate/filter-audio.mak:209: tests/data/hls-list.m3u8] Error 134
       > make: *** Waiting for unfinished jobs....
       > GEN    tests/data/hls-list-append.m3u8
       > make: *** [tests/fate/filter-audio.mak:219: tests/data/hls-list-append.m3u8] Error 134

Full build log https://hydra.nixos.org/build/263289438/nixlog/1

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

@JohnRTitor could you build on x86 via rosetta? That's what's being fixed here. aarch64-darwin was working already.

Result of nixpkgs-review pr 322724 run on aarch64-darwin 1

@DontEatOreo I assume you cross compiled it on x86-darwin?

Result of nixpkgs-review pr 322724 run on x86_64-darwin 1

1 package marked as broken and skipped:

handbrake

7 packages failed to build:

ffmpeg_7-full
ffmpeg_7-full.bin
ffmpeg_7-full.data
ffmpeg_7-full.dev
ffmpeg_7-full.doc
ffmpeg_7-full.lib
ffmpeg_7-full.man

14 packages built:

ffmpeg_7
ffmpeg_7-headless
ffmpeg_7-headless.bin
ffmpeg_7-headless.data
ffmpeg_7-headless.dev
ffmpeg_7-headless.doc
ffmpeg_7-headless.lib
ffmpeg_7-headless.man
ffmpeg_7.bin
ffmpeg_7.data
ffmpeg_7.dev
ffmpeg_7.doc
ffmpeg_7.lib
ffmpeg_7.man

Huh, full failed? Could you post the log?
@ofBorg build ffmpeg_7-full

Huh, full failed? Could you post the log?
@ofborg build ffmpeg_7-full

https://termbin.com/tk9f

The issue on ffmpeg_7-full on Darwin has something to do with ffmpeg not finding libquirc.

The paste link is dead.

If nobody figures this out before I finish dealing with FFmpeg 4, I’ll try to take a look at it then.

Found the issue it seems libquirc was not linked correctly it was found during compile time but wen trying to run the build ffmpeg binary it failed to find it.
For now i disabled as i do not know why it was linked incorrectly.

@ofborg build ffmpeg_7-full

Building ffmpeg_7-full on native x86_64-darwin.
It seems dodgy that quirc contains a .so on a platform that uses .dylib. I don’t know if that’s the cause or not, but it looks like @toonn, who I know was working on this recently, got that fixed upstream: https://github.com/dlbeer/quirc/pull/144.
Perhaps we could consider including that patch or bumping to that commit and see if it fixes the issue?

It does build!

Result of nixpkgs-review pr 322724 run on aarch64-darwin 1

1 package marked as broken and skipped:

handbrake

7 packages built:

ffmpeg_7-full
ffmpeg_7-full.bin
ffmpeg_7-full.data
ffmpeg_7-full.dev
ffmpeg_7-full.doc
ffmpeg_7-full.lib
ffmpeg_7-full.man

My branch fixing ffmpeg_7-full includes avfoundation, quirc and xev{d,e} patches, PR incoming later today.
Re .so on Darwin. It's really just an aesthetic matter TBH, Darwin doesn't actually care about the extension. Though you're right that it hints at no considerations having been made for Darwin.

The combination of #328420, #328422,  #328424 and #328428 should fix both ffmpeg_7 and ffmpeg_7-full on Darwin (and as a bonus potentially fix xev{d,e} support on aarch64-linux pending testing).
The changes are mostly simple but #328422 might hold things up a bit due to its complexity.
I don't mind fixing things up if this PR gets merged but it would be nice if #328424 is considered (would imply changes to this PR) so that we never need to disable AVFoundation support.

I personally prefer @toonn’s changes as they avoid regressing functionality and don’t seem too complex to me.

This PR was already partially incorrect because of @toonn's merged changes :( Hopefully we can get the xev{d,e} changes in soon and revert the unnecessary AVFoundation disabling.


Documenting oc-mirror vs oc mirror [183]

Description
Docs updating oc-mirror vs oc mirror
Fixes #183
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Docs only
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

/ok-to-test

/retest


Configurable Recurssion Depth

Right now it returns null whenever recurssion is detected. There should be a configurable amount of recurssion allowed.

added in d2a5717


Could if expressions be reverted to their previous form?

Recently we switched from:
var grade set to "Pass" if score > 30  else "Fail"
to
var grade set to if score > 30  then "Pass" else "Fail"
which reads less well, but was driven by 'left-recursion' issue in parsing because the rule was defined as
ifExpression: expression IF expression ELSE expression;
where ifExpression is itself an option for expression
However, could the left-recursion issue be avoided by defining it as:
ifExpression: term IF expression ELSE expression;
or even
ifExpression: term IF expression ELSE term;

The short answer is 'no' - making that change results in stack overflows, suggesting that even using term instead of expression for the first part does not avoid left-recursion.


Use pre-commit

Codecov Report

Merging #64 (1c59001) into main (01a5a2b) will decrease coverage by 0.15%.
The diff coverage is 84.84%.

:exclamation: Current head 1c59001 differs from pull request most recent head ac5a197. Consider uploading reports for the commit ac5a197 to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #64      +/-   ##
==========================================
- Coverage   67.77%   67.63%   -0.15%
==========================================
  Files          13       13
  Lines         450      448       -2
  Branches       64       64
==========================================
- Hits          305      303       -2
  Misses        132      132
  Partials       13       13

Files
Coverage Δ

zyte_api/__main__.py
50.58% <100.00%> (ø)

zyte_api/__version__.py
100.00% <100.00%> (ø)

zyte_api/_async.py
88.88% <100.00%> (ø)

zyte_api/aio/retry.py
57.69% <ø> (ø)

zyte_api/apikey.py
100.00% <100.00%> (ø)

zyte_api/constants.py
100.00% <100.00%> (ø)

zyte_api/errors.py
75.00% <100.00%> (ø)

zyte_api/stats.py
80.00% <100.00%> (ø)

zyte_api/utils.py
92.00% <100.00%> (ø)

zyte_api/aio/errors.py
81.25% <0.00%> (ø)

... and 2 more


Do you mind looking into Fiber adaptor for net/http interface to make this library work with fiber applications?

here's the link: https://docs.gofiber.io/api/middleware/adaptor/
`go
package main
import (
"fmt"
"os"
"github.com/danielAsaboro/controllers"
"github.com/gofiber/fiber/v2"
"github.com/gofiber/fiber/v2/middleware/adaptor"
"github.com/gofiber/fiber/v2/middleware/cors"
"github.com/zishang520/engine.io/types"
"github.com/zishang520/socket.io/socket"

)
type Message struct {
Name    string json:"name"
Message string json:"message"
}
func main() {
app := fiber.New()
var opts *socket.ServerOptions = socket.DefaultServerOptions()
var _ = &socket.ServerOptions{
// 	 cors: {
//     origin: "http://localhost:8100",
//     methods: ["GET", "POST"],
//     credentials: true
// },
// transports: ['websocket', 'polling'],
// allowEIO3: true
}
_ = &types.Cors{
	Origin:      "*",
	Methods:     []string{"GET", "POST"},
	Credentials: false,
}

// opts.SetCors(corsOptions)

app.Use(cors.New())
fmt.Print("printing addresses")
fmt.Println(opts)

io := socket.NewServer(nil, opts)
// io.of
// http.

app.Use("/socket.io/", adaptor.HTTPHandler(io.ServeHandler(opts)))
// http.Handle("/socket.io/", io.ServeHandler(nil))
// go http.ListenAndServe(":3000", nil)

io.On("connection", func(clients ...any) {
	client := clients[0].(*socket.Socket)
	client.On("event", func(datas ...any) {
	})
	client.On("disconnect", func(...any) {
	})
})
app.Get("/", controllers.HomeRouteHandler)

app.Listen(":" + os.Getenv("PORT"))

`
printing opts gave me nil all through...
&{{           0xc0001703f0 } {   }      }

This is part of the plan and will be a refactored branch.

that would be much appreciated @zishang520 as this would allow me serve both rest and websocket on the same port
is there any way to  help? would love to document as I can tell your schedule doesn't accommodate it

Thank you for your support. I will try to prioritize the implementation of this plan in the near future. After all, the plan has been on hold for some time. I am really sorry.

Sorry, due to technical issues, I cannot solve this problem: https://github.com/valyala/fasthttp/issues/965. Currently, this requirement will be updated after fasthttp has a next step plan.
Currently, the submitted engine.io-server-go-fasthttp project and socket.io-server-go-fasthttp project cannot be used normally

New progress: Currently it can only be implemented based on the Websocket transport interface, and the Polling transport interface will be abandoned.

Fasthttp has been updated not long ago (https://github.com/valyala/fasthttp/pull/1525/commits/9c12719b5fe70391ac9babc759478786ab96dff3), and it was found to work properly in the test case:
package main

import (
	"os"
	"os/signal"
	"syscall"
	"time"

	"github.com/gofiber/fiber/v2"
	"github.com/gofiber/fiber/v2/middleware/adaptor"
	"github.com/zishang520/engine.io/v2/log"
	"github.com/zishang520/engine.io/v2/types"
	"github.com/zishang520/socket.io/v2/socket"
)

func main() {
	log.DEBUG = true
	c := socket.DefaultServerOptions()
	c.SetServeClient(true)
	// c.SetConnectionStateRecovery(&socket.ConnectionStateRecovery{})
	// c.SetAllowEIO3(true)
	c.SetPingInterval(300 * time.Millisecond)
	c.SetPingTimeout(200 * time.Millisecond)
	c.SetMaxHttpBufferSize(1000000)
	c.SetConnectTimeout(1000 * time.Millisecond)
	c.SetCors(&types.Cors{
		Origin:      "*",
		Credentials: true,
	})
	socketio := socket.NewServer(nil, nil)
	socketio.On("connection", func(clients ...interface{}) {
		client := clients[0].(*socket.Socket)

		client.On("message", func(args ...interface{}) {
			client.Emit("message-back", args...)
		})
		client.Emit("auth", client.Handshake().Auth)

		client.On("message-with-ack", func(args ...interface{}) {
			ack := args[len(args)-1].(func([]any, error))
			ack(args[:len(args)-1], nil)
		})
	})

	socketio.Of("/custom", nil).On("connection", func(clients ...interface{}) {
		client := clients[0].(*socket.Socket)
		client.Emit("auth", client.Handshake().Auth)
	})

	app := fiber.New()

	// app.Put("/socket.io", adaptor.HTTPHandler(socketio.ServeHandler(c))) // test
	app.Get("/socket.io", adaptor.HTTPHandler(socketio.ServeHandler(c)))
	app.Post("/socket.io", adaptor.HTTPHandler(socketio.ServeHandler(c)))

	go app.Listen(":3000")

	exit := make(chan struct{})
	SignalC := make(chan os.Signal)

	signal.Notify(SignalC, syscall.SIGHUP, syscall.SIGINT, syscall.SIGTERM, syscall.SIGQUIT)
	go func() {
		for s := range SignalC {
			switch s {
			case syscall.SIGHUP, syscall.SIGINT, syscall.SIGTERM, syscall.SIGQUIT:
				close(exit)
				return
			}
		}
	}()

	<-exit
	socketio.Close(nil)
	os.Exit(0)
}


AWS Security Hub Module

Issue #, if available:
Description of changes: added AWS Security Hub Module
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

closing in favor of https://github.com/awslabs/cid-framework/pull/161