UI - vertical BAR

Hi
can we get rid of vertical bar which is visible in mobile / small screen version of book stack?
Istead of that there could be a small top button that toggles left menu

Hi @akurzawa,
Could you could explain why you'd want to remove the vertical bar? That would help evaluate other potential solutions.

I can see the benefit in this, it would actually give a little more space for the content and provide a cleaner view for mobile

As of v0.26 this vertical bar shown in the above image no longer exists. Will therefore close this.


Added some documentation for sendmail #355

Added some documentation for sendmail https://github.com/go-gitea/gitea/pull/355

LGTM

LGTM


how to add music?

it doesn't upload music

I am closing this issue because there is too little information on how to reproduce the problem. Maybe it's connected with issue #4


Not visible on home app on ios and precondition

I cant seem to add it on home app and also i cant get the service for pre conditioning the car i have a range rover velar gasoline.

You'll have to give me a bit more information as to what you mean by cant seem to add it and cant get the service etc.
What are you trying to add and do you get an error?
What happens when you call the service. Again, any error?

I cant precondition the car and i dont see the option to do.

Sorry, I don't have much time these days and no longer have a JLR car to properly help you.  Maybe try posting on the forum and see if the users can help you.
https://community.home-assistant.io/t/jaguar-landrover-incontrol-integration/194586


Trying to export the onnx model reports an error

Here are the specific error messages：
Traceback (most recent call last):
File "tools/deployment/pytorch2onnx.py", line 325, in
args.cfg_options)
File "/home/anaconda3/envs/openmmlab/lib/python3.7/site-packages/mmdet/core/export/pytorch2onnx.py", line 89, in build_model_from_cfg
model = build_detector(cfg.model, test_cfg=cfg.get('test_cfg'))
File "/home、anaconda3/envs/openmmlab/lib/python3.7/site-packages/mmdet/models/builder.py", line 59, in build_detector
cfg, default_args=dict(train_cfg=train_cfg, test_cfg=test_cfg))
File "/home/anaconda3/envs/openmmlab/lib/python3.7/site-packages/mmcv/utils/registry.py", line 212, in build
return self.build_func(*args, **kwargs, registry=self)
File "/home/anaconda3/envs/openmmlab/lib/python3.7/site-packages/mmcv/cnn/builder.py", line 27, in build_model_from_cfg
return build_from_cfg(cfg, registry, default_args)
File "/home/anaconda3/envs/openmmlab/lib/python3.7/site-packages/mmcv/utils/registry.py", line 45, in build_from_cfg
f'{obj_type} is not in the {registry.name} registry')
KeyError: 'FSCE is not in the models registry'
Here are the specific error messages

Did you succeed in exporting the onnx model？ I also encountered the same error as you.

I didn't solve this either

I didn't solve this either


🙏 Extensions: Application Customizer - Additional Placeholders Requested

Category

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

Today we get PageHead & PageFooter... would like to have BEFORE/AFTER DIV's for QuickLaunch & PlaceHolderMain

If you are planning to share a new feature request (enhancement / suggestion), please use SP Dev UserVoice at http://aka.ms/sp-dev-uservoice.

OK... cross-referenced:
https://sharepoint.uservoice.com/forums/329220-sharepoint-dev-platform/suggestions/19536022-extensions-application-customizer-additional-pl

Thx AC for the input.
We'll absolutely introduce additional placeholders for the framework and this is a good input for getting the needed placeholders tracked. We would also love to have some details around the actual scenarios where you'd use the requested capabilities. That might be obvious in some cases, but actual business scenarios and use cases help to prioritize the requests. Thx.

Cross posting my response from UV.
Hey AC - While I know what you mean here, can you refine this to target the modern experiences? For example, PlaceholderMain is an asp.net masterpage div that doesn't exist in the newer code. Would you think of Pre/Post Placeholdermain to be ContentHeader and ContentFooter (as opposed to PageHeader and PageFooter?
Additionally, how would you expect scrolling to work in this scenario? For example - what should happen with a listview with infinite scroll vs. a page with 500k of text?

@patmill Will xpost from UV too :)
Specifically looking for any place these extensions are supported... if that's just modern experiences, then just those (although I hope these will also apply to classic pages, just as client-side web parts do). Understand the names are different... I was referring to the intent of where things would go... a picture might help. I see the RED is what you provide today... the GREEN is what I'm asking for:

As for infinite scroll... I'd expect it to behave like the components that wrap them. For instance, on the QuickLaunch, it doesn't scroll so I'd expect what I put right after it to be pinned to the bottom border of the QL. Same is true with the main body of the page.
@VesaJuvonen - Biz case: context help, messages to the user based on the context of what they are doing or reminders they haven't done something. I could do this by doing DOM manipulation from the header / footer, and that's what folks will do as these are common places we want to add / remove stuff... but that's now how we should be doing it... so don't give them a reason to do it and add it for them. Make sense?

I agree with AC. While @VesaJuvonen has mentioned that keeping our customization's within the confines of these placeholders, which I assume is the contractual safe place for custom content, ensures that it will not get destroyed or broken with future updates to the DOM, if there are not additional Placeholders to be populated, users will simply begin manipulating the DOM.
While this isn't how they should add customization's, as @andrewconnell as mentioned, if there are no other places, peeps will still do it.
The Pre/Post locations could also provide the ability to wrap that content with a custom CSS classname for ensuring specificity within a certain brand/design.....just one possible other thought.

Adding some additional placeholder requests here, as discussed in #656

Site metadata: custom site meta data (state, contacts, project duration...)
Search bar: custom search bar
Left navigation: having a global navigation, buttons to create documents from templates, showing company's applications...

To updated everyone on this topic: the scenario is well understood and we are working to be sure we tackle that properly. There's still an open conversation to understand if Placeholders is the right technology to use to enable such scenarios or it is something else.

I'm confused @lucaband ... why close the issue if the conversation is still open even if internally?

Andrew, thanks,
Reason why I closed was because I though I answered the initial question.
But I'm perfectly fine to keep it open, assign it to me, and reply back here once we have some concrete news.

This might be interessting for you as well: https://sharepoint.uservoice.com/forums/329220-sharepoint-dev-platform/suggestions/31752808-extensions-property-pane-sidebar-placeholder-r
I would like to see a sidebar placeholder on the right side.

I would like to achieve the below-highlighted stuff in the modern experience but it is not possible due to the limited number of placeholders. Please introduce more placeholders as requested.

@lucaband  Are there any updates on a solution to this issue?

as this is a new feature request and not specifically an issue, we'll close this, for now, to help on tracking actual issues in the issue list (trying to drive new feature requests to UserVoice). We will have some announcements coming up around this area during this spring - SPC18 and later.

I dont see any additional placeholders. The following only shows me Top and Bottom. Is there anything I have to do to get the other placeholders?
console.log(
"Available placeholders: ",
this.context.placeholderProvider.placeholderNames
.map(name => PlaceholderName[name])
.join(", ")
);

Sorry for the lack of clarity.  Feature requests are handled via user-voice.  There are currently no additional placeholders.

I dont think there are any new placeholders till today. I can only fine Top and Bottom. So people who want to customize have to manipulate DOM directly. @VesaJuvonen Please throw us a light on this.

Top & bottom are the only two.
Microsoft has said in recent public community calls they are working on some stuff related to extensions. With a big conference (Ignite) coming up in November, I'd suspect we may learn more there, but not likely much before that.

Any update on above request, i am looking for solution.

Afraid not @malleshatla
With new 1.10 version, we have an option to modify Search experiences (https://docs.microsoft.com/en-us/sharepoint/dev/spfx/building-search-extensions), but no new PlaceHolders yet

Locking the issue as it's already closed and no reason to keep on having chat on closed issues as those comments can easily get fully ignored.


typos

On http://elemental-ui.com/--
Tooklit
includig
devolpment

Thanks for that @winkler1 - too busy looking at the pixels we missed the words :)


Javascript Heap out of memory when compiling Semantic

I have been using Semantic UI with React for 6 months and haven't had any major issues.
However, now I tried to add some overrides and I am getting messages like the following one
Compiling...

<--- Last few GCs --->

  878830 ms: Mark-sweep 1265.8 (1407.9) -> 1265.7 (1412.9) MB, 971.5 / 0.0 ms [allocation failure] [GC in old space requested].
  879774 ms: Mark-sweep 1265.7 (1412.9) -> 1265.7 (1422.9) MB, 944.0 / 0.0 ms [allocation failure] [GC in old space requested].
  880738 ms: Mark-sweep 1265.7 (1422.9) -> 1268.8 (1406.9) MB, 964.2 / 0.0 ms [last resort gc].
  881691 ms: Mark-sweep 1268.8 (1406.9) -> 1271.9 (1406.9) MB, 952.7 / 0.0 ms [last resort gc].

<--- JS stacktrace --->

==== JS stack trace =========================================

    2: arguments adaptor frame: 3->1
Security context: 0x2019df4cfb51 <JS Object>
    3: map [native array.js:~994] [pc=0x2c9b820f36e6] (this=0x2275878ab299 <JS Array[3]>,bq=0x2275878ab2b9 <JS Function (SharedFunctionInfo 0x749c3e092c9)>,br=0x2019df404381 <undefined>)
    4: arguments adaptor frame: 1->2
    5: new Color [/home/rory/dev/weezhome/weezhome-web/node_modules/less/lib/less/tree/color.js:~7] [pc=0x2c9b825914c1] (this=0x2275878a...

FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory
 1: node::Abort() [/home/rory/.nvm/versions/node/v6.9.2/bin/node]
 2: 0x109608c [/home/rory/.nvm/versions/node/v6.9.2/bin/node]
 3: v8::Utils::ReportApiFailure(char const*, char const*) [/home/rory/.nvm/versions/node/v6.9.2/bin/node]
 4: v8::internal::V8::FatalProcessOutOfMemory(char const*, bool) [/home/rory/.nvm/versions/node/v6.9.2/bin/node]
 5: v8::internal::Factory::NewFixedArray(int, v8::internal::PretenureFlag) [/home/rory/.nvm/versions/node/v6.9.2/bin/node]
 6: v8::internal::TypeFeedbackVector::New(v8::internal::Isolate*, v8::internal::Handle<v8::internal::TypeFeedbackMetadata>) [/home/rory/.nvm/versions/node/v6.9.2/bin/node]
 7: 0xb48240 [/home/rory/.nvm/versions/node/v6.9.2/bin/node]
 8: 0xb485a9 [/home/rory/.nvm/versions/node/v6.9.2/bin/node]
 9: 0xb4a9a0 [/home/rory/.nvm/versions/node/v6.9.2/bin/node]
10: 0xb52046 [/home/rory/.nvm/versions/node/v6.9.2/bin/node]
11: v8::internal::Compiler::Compile(v8::internal::Handle<v8::internal::JSFunction>, v8::internal::Compiler::ClearExceptionFlag) [/home/rory/.nvm/versions/node/v6.9.2/bin/node]
12: v8::internal::Runtime_CompileLazy(int, v8::internal::Object**, v8::internal::Isolate*) [/home/rory/.nvm/versions/node/v6.9.2/bin/node]
13: 0x2c9b7d4092a7
Aborted (core dumped)
error Command failed with exit code 134.

Maybe I am doing something wrong, But this is the override I have in mind
@import (multiple) '../../../theme.config';

@media only screen and (max-width: @largestMobileScreen) {
  .ui.stackable.grid > .row > .column.remove-side-padding {
    padding: 1rem 0 !important;
  }
}

I tried adding this in site/collections/grid.overrides which got me the error.
However, we have actually been doing our overrides in the themes folder instead of the site folder, so I thought it was related and instead tried putting that code snippet in themes/default/collections/grid.overrides to no avail
I tried increasing old-space-size but that just made the compilation last a hell of  along time, I end up aborting.
Am I doing something wrong ? Is this a known bug ? What is happening ?

Hi @Thebigbignooby, which version of Node are you running? How much RAM is on your system? Have you tried using --max-old-space-size?

Hey @Thebigbignooby , have you been able to solve your issue? Because I am experiencing the same things


[BUG] AsyncCurl TimerHandle leaks

Describe the bug
AsyncCurl holds TimerHandle of call_later task invoked by libcurl's timer function and it is not released after the delay function is called.
To Reproduce
import asyncio
from curl_cffi.requests import AsyncSession

async def main():
    async with AsyncSession() as sess:
        for _ in range(3):
            print('before', len(sess.acurl._timers))
            try:
                await sess.get('http://httpstat.us/200?sleep=5000', timeout=1)
            except:
                pass
            print('after', len(sess.acurl._timers))

if __name__ == '__main__':
    asyncio.run(main())

Output
before 0
after 7
before 7
after 10
before 10
after 14

Expected behavior
Saved TimerHandle should be released after the delay function is called.
Versions

OS: Ubuntu 20.04 x64
curl_cffi version 0.5.9 / 0.5.10
pip freeze dump

Additional context
It seems using a WeakSet instead of list for acurl._timers could solve this problem.

Thanks, this might be the same problem as #113, this is a perfect bug report.

It should've been fixed in 0.6.0b7, feel free to reopen if the leak is not resolved.

The leaks issue was still there,please help!

The leaks issue was still there,please help!

Can you elaborate on your observations? Thanks.


Implement --quiet option for cli example

Not sure if there's any interest in this, or if this is the best flag name - but this allows --quiet to be passed to the cli example target so that it can be composed with other programs in a rudimentary way without writing a custom wrapper around sqlparser-rs.
For example, you could pipe the parsed AST into jq or some other JSON-expecting program:
$ cargo run --release --features json_example --example cli "$filename" --quiet | jq 'map(keys)[][]' -rc
Drop
CreateTable
Insert
Drop

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
9702

Relevant Lines:
11341

💛  - Coveralls

Marking as draft to signify this PR has been reviewed. Please mark this PR as ready for review when it is ready again.

@blx @pawel-big-lebowski is closing this for now because it's almost a year without an update.
@blx, ping me if you work on this code specifically; if so, I will open the PR again :)


Add Native Fns to Access fl::dtype Enum + _astype

Begin implementing functionality to allow control of type underlying tensor data.
In testing, tensor class method asType (and similar variations):
  asType(dtype: dtype) {
    fl._astype.native(this.ptr, dtype)
  }

were failing to update the type accordingly in the following test:
const t = sm.tensor(new Float32Array([1, 2, 3]))
t.asType(sm.dtype.Float64)
console.log(t.dType === sm.dtype.Float64, t.dType, sm.dtype.Float64)

Outputs:
false, 1, 2
As such, I've temporarily removed the asType tensor class method pending feedback; if it requires more involved changes to the Tensor class, it might be better to submit as a separate PR as this one is getting a bit crowded.

After some debugging w the help of @bwasti, we were able to identify the issue causing the failing test above. Added dtype.test.ts to test basic implementation (passing w changes to code).


VeraCode issue on CrashlyticsKit/Helpers/CLSAllocate.c

[REQUIRED] Step 1: Describe your environment

Xcode version: 11.3
Firebase SDK version: 6.13.0
Firebase Component: Crashlytics(Auth, Core, Database, Firestore, Messaging, Storage, etc)
Component version: 3.14.0
Installation method: CocoaPods

[REQUIRED] Step 2: Describe the problem
Steps to reproduce:
VeraCode has found a low severity issues in CrashlyticsKit/Helpers/CLSAllocate.c, called "Unchecked error condition ". The issue states that return value from malloc() is not been checked before been used in subsequent call.
Relevant Code:

Users/crashlytics/buildAgent/work/7e5c596e59839c6e/apple/CrashlyticsKit/CrashlyticsKit/Helpers/CLSAllocate.c [Path 1: CLSAllocatorSafeAllocateFromRegion, line 162 (1 step)]

Users/crashlytics/buildAgent/work/7e5c596e59839c6e/apple/CrashlyticsKit/CrashlyticsKit/Helpers/CLSAllocate.c [Path 1: CLSAllocatorSafeAllocate, line 177 (1 step)]

Users/crashlytics/buildAgent/work/7e5c596e59839c6e/apple/CrashlyticsKit/CrashlyticsKit/Helpers/CLSAllocate.c  [Path 1: CLSAllocatorSafeAllocate, line 182 (1 step)]

This issue has been fixed in:
https://github.com/firebase/firebase-ios-sdk/pull/4669


feat: mechanism for retry of failed tests

Implement a robust mechanism to automatically retry the generation of test cases for failed scenarios. This feature aims to enhance the reliability and efficiency of the test generation process by identifying failures and re-attempting their generation with improved prompts or configurations.
TODO

[ ] Failure Detection
[ ] new AI model trained for retry test generation specifically
[ ] limit number of retries to 3

This Issue has been blocked and must need following things to be completed:

[ ] Completion of App Workflow (to raise a PR with the generated test files to the branch)
[ ] Requires a Workflow which basically runs the Generated tests


[NTR] mucosa-associated lymphoid tissue lymphocyte

Preferred term label:
mucosa-associated lymphoid tissue lymphocyte
Synonyms
lymphocyte of mucosa-associated lymphoid tissue
MALT lymphocyte
Definition (free text, please give PubMed ID)
Any lymphocyte that is part of any mucosa-associated lymphoid tissue
Parent term
lymphocyte

closing for now as it's not clear whether these are different from other lymphocytes


add hover effects on "obtain line" and "cert. of analysis"

Problem
What is the problem this work solves, including
closes #27
Solution
What I/we did to solve this problem

modified css to apply hover effects at the cell level for the last two columns

Note:

The color toggle for the three little vertical strokes within the Tube icon isn't working. Should we make adjustments to the css or the svg?

I noticed that our cert svg slightly differs from the one in the design, not sure if this is significant

Type of change
Please delete options that are not relevant.

New feature (non-breaking change which adds functionality)

Screenshots (optional):

The Cert. icon in the design:

@rugeli I made one commit where I changed the selector to use a class name instead of the column number. This is more future proof because if we added or removed a column from the table, the selector would have gotten messed up

@rugeli I made one commit where I changed the selector to use a class name instead of the column number. This is more future proof because if we added or removed a column from the table, the selector would have gotten messed up. I also made it so the cell didn't get a hover effect if there wasn't a link

These are great changes to prevent potential issues!


Unable to build project using vcpkg

I am trying to build the project using vcpkg (first time, so probably missing something), however I don't get past the avrocpp_s stage.
Any idea what I might be doing wrong? I tried also vckpg install in both the root as bcpkg_ports/avro-cpp directory.
ramon@ramon-ThinkPad-X1-Carbon-Gen-9~/Desktop/PROJECTS/duckdb_iceberg: VCPKG_TOOLCHAIN_PATH='/home/ramon/Desktop/PROJECTS/duckdb_iceberg/vcpkg/scripts/buildsystems/vcpkg.cmake' make
mkdir -p build/release && \
cmake  -DEXTENSION_STATIC_BUILD=1 -DDUCKDB_EXTENSION_CONFIGS='/home/ramon/Desktop/PROJECTS/duckdb_iceberg/extension_config.cmake'  -DOSX_BUILD_ARCH=  -DVCPKG_MANIFEST_DIR='/home/ramon/Desktop/PROJECTS/duckdb_iceberg/' -DVCPKG_BUILD=1 -DCMAKE_TOOLCHAIN_FILE='/home/ramon/Desktop/PROJECTS/duckdb_iceberg/vcpkg/scripts/buildsystems/vcpkg.cmake' -DDUCKDB_EXPLICIT_PLATFORM=''  -DCMAKE_BUILD_TYPE=Release -S ./duckdb/ -B build/release && \
cmake --build build/release --config Release
-- git hash 1f98600c2c, version v1.0.0, extension folder v1.0.0
-- Extensions will be deployed to: /home/ramon/Desktop/PROJECTS/duckdb_iceberg/build/release/repository
-- Load extension 'iceberg' from '/home/ramon/Desktop/PROJECTS/duckdb_iceberg' @ 3f6d753
-- Load extension 'httpfs' from '/home/ramon/Desktop/PROJECTS/duckdb_iceberg/duckdb/extensions' @ 1f98600c2c
-- Load extension 'parquet' from '/home/ramon/Desktop/PROJECTS/duckdb_iceberg/duckdb/extensions' @ 1f98600c2c
-- Load extension 'jemalloc' from '/home/ramon/Desktop/PROJECTS/duckdb_iceberg/duckdb/extensions' @ 1f98600c2c
CMake Error at /home/ramon/Desktop/PROJECTS/duckdb_iceberg/CMakeLists.txt:32 (find_library):
  Could not find AVROCPP_LIBRARY_DEBUG using the following names: avrocpp_s

-- Configuring incomplete, errors occurred!
See also "/home/ramon/Desktop/PROJECTS/duckdb_iceberg/build/release/CMakeFiles/CMakeOutput.log".
make: *** [extension-ci-tools/makefiles/duckdb_extension.Makefile:56: release] Error 1

Small update, when trying the debug build (make debug instead of make release) I don't face any issues, and am able to build the extension.

hmm did you perhaps run make release without the VCPKG_TOOLCHAIN_PATH first ? In that case a make clean and rerunning VCPKG_TOOLCHAIN_PATH=... make should work

hmm did you perhaps run make release without the VCPKG_TOOLCHAIN_PATH first ? In that case a make clean and rerunning VCPKG_TOOLCHAIN_PATH=... make should work

This was indeed the problem, thanks!


In Exapme (E) Using correlated subqueries, misleading content

In the page at example E you wrote:
-START-

-END-
Now you are not clear enough here, because it seems to indicate that the first code example is not a correlated subquery, as you say after that  'The following example uses IN in a correlated, or repeating, subquery', however in fact the first code example with the EXISTS in the WHERE clause IS definitely a correlated subquery, since it depends on the Outer queries current row ProductModelId.

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

ID: 5b8c2959-4a3c-bf2e-4504-c84cb30660b7
Version Independent ID: c1e9b6b5-d398-ebd1-b447-ede189bd6c15
Content: SELECT Examples (Transact-SQL) - SQL Server
Content Source: docs/t-sql/queries/select-examples-transact-sql.md
Product: sql
Technology: t-sql
GitHub Login: @VanMSFT
Microsoft Alias: vanto

@SDP190  Thank you for the feedback and insight.
The documentation has been reworded and is pending review.

Hi @msebolt, I hope you will see my comment although it was closed.
Firstable thanks for you taking time to work on that issue, I saw the changes you committed here.
However to my understanding, the first code example may still need some clarification, as shown in the image at the first code segment there are 2 examples of a query, the first using EXISTS  vs the second using IN. We should be more explicit by explaining what's the difference between them is, and that the first one is a correlated subquery while the second one is not. @msebolt am I right? can you see what bothers me?


The demo is about 5x faster than it should be

Describe the bug
The demo gif is about five times faster than would be needed for a human to figure out what is happening. This speed is fine now that I know how Navi works, but the speed was is crazy for when I was first trying to understand what Navi did. First having to look at every part of every frame and then work out what it's trying to show was not even close to impossible. It might also be ok if browsers had video controls on gifs, but they don't :(
To Reproduce
Steps to reproduce the behavior:

Be completely new to Navi
Go to https://github.com/denisidoro/navi#navi---
Try to follow the GIF

Expected behavior
The gif progresses at a pace that works for new users without having to load externally in mpv or some other tool
Screenshots

What do you think about this one?

Wow, awesome work! Thank you for that.
However, putting myself into the head space of a new user...
I first see you run demo 0 which is a bit confusing (however I get it, although I'm wondering if this would work: https://github.com/sloria/doitlive)
On the next screen I'm watching the part that's moving and completely ignoring the heading above assuming it's just part of the command prompt, which upon pausing I think this is the main part you wanted me to see. I think a lot of presentations make sure to type out descriptions they want people to see.
Upon pausing it become apparent how you wish me to view the video. However at the start it really seems to be diving in at the deep end and more focused on configuring navi than an intro for somebody completely new. It does seem to become more of an intro at 29 seconds, but perhaps you could start with that? I think the first part is fantastic for people who already know what Navi is and want to take their use to the next level.
The fact that it's now in a format that allows pausing is much better than before. I hope this doesn't come across as critical. I just figured you've probably spent a lot of time on Navi and so I wanted to be honest in the hope it helps your project become even more successful.

I've updated it once more. Please let me know what you think

That is sooo perfect :heart: :clap:

Thank you for the suggestions!

I've updated it once more. Please let me know what you think

Great demo !


Update BypassingModuleMocks.md

Summary

Attempts to correct small typo in documentation.
Test plan

No tests. Just text content.

Absolutely. Thanks for providing the link. https://github.com/facebook/jest/pull/8472

We usually scope changes to the docs (versioned included) in a single PR, but I just went ahead and merged as is


Grammatical errors in public API

My technical knowledge of WASM is limited, so my contributions are in turn limited to flagging grammatical errors in the public API.  I hope this is helpful! 🙂
As I was glancing over the API documentation, I noticed the ImportedMemories and ExportedMemories methods on the CompiledModule interface.  In English, the "countable" plural memories is used to talk about episodic memory (e.g. childhood memories), whereas the "uncountable" plural memory refers to a storage medium.  Thus, symbol names such as ImportedMemory and ExportedMemory seem more appropriate to me.  If we want to emphasize the fact that there are several discrete memory segments being imported/exported, I would suggest the names MemoryImports and MemoryExports instead.
Obviously, this is not a critical issue, but I thought I would bring it up anyway.  If this feedback is helpful, I would be happy to review the rest of the documentation and submit PRs to improve the docs.  Let me know!

Strange, the specs occasionally refer to memory segments as memories.    I'm still in favor of renaming, but again, this isn't a critical issue.

We had a conflict on name with api.Module where we ended up naming ExportedMemoryDefinitions because if we said ExportedMemories (for definitions which is what CompiledModule returns), it would have clashed conceptually with ExportedMemory which returns a different result (instance vs definition). We would have a similar naming issue with MemoryExports.
I think it is worth changing CompiledModule to match what api.Module returns ExportedMemories -> ExportedMemoryDefinitions even if it is longer, as it is both more consistent and avoids the grammar glitch noted here. The end user impact will be low as these are niche functions. To rename in general things to Export/Import prefix would be more breaking iotw.
@mathetake wdyt

Sg


[listennotes.com] Add extractor

IMPORTANT: PRs without the template will be CLOSED
Description of your pull request and other information

Add extractor for listennotes.com
Fixes #5262
Template

Before submitting a pull request make sure you have:

[x] At least skimmed through contributing guidelines including yt-dlp coding conventions
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8 and ran relevant tests

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Fix or improvement to an extractor (Make sure to add/update tests)
[x] New extractor (Piracy websites will not be accepted)
[ ] Core bug fix/improvement
[ ] New feature (It is strongly recommended to open an issue first)

Please avoid force-pushing

i suggest you to take extract json ld as well using self._search_json_ld, like dirkf said

i tried to extract it. Nothing interesting was found.

Please avoid force-pushing

OK. Do I need to squash many commits to one?

Do I need to squash many commits to one?

No. I will squash during merge. Feel free to make as many commits as you need

Why need extracting duration field? Where is it used?

Some people needs video metadata for various reason (eg: analytics, etc). I think it's better if we can get all metadata that we can get.
For information what field we can used, please take a look at extractor/common.py especially at InfoExtractor class

I added fallback, full description and more fields. Pls have a look and let me know if you think the fields I chose make sense


added placeholders for upcoming functions

Added api reference documentation for upcoming models:

DCGAN
UNET
CVAE

I know @itayniv is working on Sentiment documentation, so I've left that out for now.

This merge cause query breaks, logs here:

Placeholder needs to work with optional display function. I'll fix it when I got home tonight.


Generated connection resolvers not using custom keys specified by @key directive

When using as custom primary key with @key, the generated connection resolvers does not use the keys specified in @key directive, it defaults to "id"
E.g. the model below the BlogPosts connection between Blog and Post
type Company @model {
  id: ID!
  name: String
  blogs: [Blog] @connection(name: "CompanyBlogs")
}

type Blog @model @key(fields: ["company","id"]){
  company: ID!
  id: ID!
  Company: Company @connection(name: "CompanyBlogs", keyField: "company")
  name: String!
  posts: [Post] @connection(name: "BlogPosts")
}

type Post @model @key(fields: ["company","id"]) {
  company: ID!
  id: ID!
  Company: Company @connection(keyField: "company")
  title: String!
  blog: Blog @connection(name: "BlogPosts")
}

Looking at the  blog: Blog @connection(name: "BlogPosts") connection, the resolver should use the company, id composite key specified in type Blog @model @key(fields: ["company","id"]), instead the generated Post.blog.req.vtl resolver uses the old id as key
{
  "version": "2017-02-28",
  "operation": "GetItem",
  "key": {
      "id": $util.dynamodb.toDynamoDBJson($util.defaultIfNullOrBlank($ctx.source.postBlogId, "___xamznone____"))
  }
}

Expected behavior
The generated resolver should use the partition/sort keys as specified in the @key directive
{
  "version": "2017-02-28",
  "operation": "GetItem",
  "key": {
            "company": $util.dynamodb.toDynamoDBJson($util.defaultIfNullOrBlank($ctx.source.copmany, "___xamznone____")),
      "id": $util.dynamodb.toDynamoDBJson($util.defaultIfNullOrBlank($ctx.source.postBlogId, "___xamznone____"))
  }
}

amplify @ 1.7.0

#1562

Duplicate of #1562

Closing as this is a duplicate fo #1562


Allow namespace wildcards for revoke

This one's pretty much copy&paste from the respective grant functions and therefore should work out of the box, but since I didn't get a chance to test it yet feel free to complain...
Personally I think somoene should refactor the permission grant and revoke stuff a bit so that they rely on a helper method doing the most work. It's never a good sign when there's technically the same code on multiple places... (not to mention maintainability)
Edit: produces error, fix incoming...
Edit2: tested to be working now :D

Manually merged, thanks!


Incorrect example for syncing

I believe the example command should either have the 'Name' parameter or 'Path' but not both.
Sync-NavApp -ServerInstance YourDynamicsNAVServer -Name ExtensionName -Tenant TenantID

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

ID: bc95f9ad-6f07-d6af-0b08-9ec2742a1fd6
Version Independent ID: ad1cec21-3e17-820c-7963-f7f51d61e818
Content: How to: Publish and Install an Extension v2.0 - Business Central
Content Source: dev-itpro/developer/devenv-how-publish-and-install-an-extension-v2.md
Service: dynamics365-business-central
GitHub Login: @jswymer
Microsoft Alias: jswymer

Hello, thanks for the feedback! The article has been fixed and an update will be published soon.


Fix TorchTabularTextDataset.len method

Since the TorchTabularTextDataset accepts the labels as None, we have an issue in the __len__() method that uses the self.labels to return the len.
In this PR, we suggest replacing it with the length of self.encodings.

Thanks for all the effort Doug! I've merged both PRs in


Has stopped working after upgrade 2.0.4 → 2.0.5

Here’s a traceback.
/home/tanner/workspace/ROOTWELT/studio/project/env/bin/python3.4 /home/tanner/workspace/ROOTWELT/studio/project/studio/manage.py runserver
Traceback (most recent call last):
  File "/home/tanner/workspace/ROOTWELT/studio/project/studio/manage.py", line 8, in <module>
    execute_from_command_line(sys.argv)
  File "/home/tanner/workspace/ROOTWELT/studio/project/env/lib/python3.4/site-packages/django/core/management/__init__.py", line 338, in execute_from_command_line
    utility.execute()
  File "/home/tanner/workspace/ROOTWELT/studio/project/env/lib/python3.4/site-packages/django/core/management/__init__.py", line 312, in execute
    django.setup()
  File "/home/tanner/workspace/ROOTWELT/studio/project/env/lib/python3.4/site-packages/django/__init__.py", line 18, in setup
    apps.populate(settings.INSTALLED_APPS)
  File "/home/tanner/workspace/ROOTWELT/studio/project/env/lib/python3.4/site-packages/django/apps/registry.py", line 85, in populate
    app_config = AppConfig.create(entry)
  File "/home/tanner/workspace/ROOTWELT/studio/project/env/lib/python3.4/site-packages/django/apps/config.py", line 86, in create
    module = import_module(entry)
  File "/home/tanner/workspace/ROOTWELT/studio/project/env/lib/python3.4/importlib/__init__.py", line 109, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 2254, in _gcd_import
  File "<frozen importlib._bootstrap>", line 2237, in _find_and_load
  File "<frozen importlib._bootstrap>", line 2226, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 1200, in _load_unlocked
  File "<frozen importlib._bootstrap>", line 1129, in _exec
  File "<frozen importlib._bootstrap>", line 1471, in exec_module
  File "<frozen importlib._bootstrap>", line 321, in _call_with_frames_removed
  File "/home/tanner/workspace/ROOTWELT/studio/project/studio/mptt_comments/__init__.py", line 1, in <module>
    from mptt_comments.models import MPTTComment
  File "/home/tanner/workspace/ROOTWELT/studio/project/studio/mptt_comments/models.py", line 8, in <module>
    from tinymce.models import HTMLField
  File "/home/tanner/workspace/ROOTWELT/studio/project/env/lib/python3.4/site-packages/tinymce/models.py", line 6, in <module>
    from tinymce import widgets as tinymce_widgets
  File "/home/tanner/workspace/ROOTWELT/studio/project/env/lib/python3.4/site-packages/tinymce/widgets.py", line 10, in <module>
    import tinymce.settings
  File "/home/tanner/workspace/ROOTWELT/studio/project/env/lib/python3.4/site-packages/tinymce/settings.py", line 17, in <module>
    JS_ROOT = getattr(settings, 'TINYMCE_JS_ROOT', finders.find('tiny_mce', all=False))
  File "/home/tanner/workspace/ROOTWELT/studio/project/env/lib/python3.4/site-packages/django/contrib/staticfiles/finders.py", line 249, in find
    for finder in get_finders():
  File "/home/tanner/workspace/ROOTWELT/studio/project/env/lib/python3.4/site-packages/django/contrib/staticfiles/finders.py", line 264, in get_finders
    yield get_finder(finder_path)
  File "/home/tanner/workspace/ROOTWELT/studio/project/env/lib/python3.4/functools.py", line 448, in wrapper
    result = user_function(*args, **kwds)
  File "/home/tanner/workspace/ROOTWELT/studio/project/env/lib/python3.4/site-packages/django/contrib/staticfiles/finders.py", line 277, in get_finder
    return Finder()
  File "/home/tanner/workspace/ROOTWELT/studio/project/env/lib/python3.4/site-packages/django/contrib/staticfiles/finders.py", line 129, in __init__
    app_configs = apps.get_app_configs()
  File "/home/tanner/workspace/ROOTWELT/studio/project/env/lib/python3.4/site-packages/django/apps/registry.py", line 137, in get_app_configs
    self.check_apps_ready()
  File "/home/tanner/workspace/ROOTWELT/studio/project/env/lib/python3.4/site-packages/django/apps/registry.py", line 124, in check_apps_ready
    raise AppRegistryNotReady("Apps aren't loaded yet.")
django.core.exceptions.AppRegistryNotReady: Apps aren't loaded yet.

Process finished with exit code 1

Oh can you try to import tinymce after staticfiles?
If you are not using staticfile then we've got to make a fix.
Can you try changing https://github.com/aljosa/django-tinymce/compare/2.0.4...2.0.5#diff-1d5fa3b685a77f9452e28aa136cd8319R3 from line 3 to line 17?

Yes I use contrib.staticfiles. Here's what my settings.INSTALLED_APPS looks like:
INSTALLED_APPS = [
    'grappelli',
    'modeltranslation',
    'filebrowser',
    'django.contrib.admin',
    'django.contrib.auth',
    'django.contrib.contenttypes',
    'django.contrib.sessions',
    'django.contrib.messages',
    'django.contrib.staticfiles',
    'django_comments',
    'django.contrib.sites',
    'django.contrib.humanize',
    'rosetta',
    'social.apps.django_app.default',
    'account',
    'django_bleach',
    'tinymce',
    'sorl.thumbnail',
    'mce_filebrowser',
    'sitetree',
    'bootstrapform',
    'pinax_theme_bootstrap',
    'bootstrap_pagination',
    'mathfilters',
    'haystack',
    'mptt',
    'captcha',
    'rest_framework',
    'text_blocks',
    'navigation',
    'blog',
    'bootstrap_calendar',
    'mptt_comments',
]

Moving import statement from line 3 to line 17 didn't help (guess it shouldn't in my case).

I've got the same problem.

Same problem

Is there still an issue with that or are it reproduced on more recent version - 2.2.0.

I'm trying to use it for a project and I'm seeing this error with 2.2.0. It happens after I do
from tinymce.models import HTMLField
in models.py
Django==1.8.4 and python 3.4

This are the changes that occured between 2.0.4 and 2.0.5.
https://github.com/aljosa/django-tinymce/compare/2.0.4...2.0.5
The fact that we use finder at the app loading rathen than latter when the function is used can be the problem here.

@gandalfar  Can you somehow give a minimal reproducable code example so we can debug it?

@gandalfar I have provide some basic app for specified version. Can you check it or add what other package do you use?

Hi, thanks for being so proactive about this issue. I was swamped with organising conference, but I'm now looking through the 100+ packages in requirements.txt to see if I can reproduce this problem.

I can't reproduce this on a simple app. Will try again later on primary app, to see if I can find cause by removing things instead of adding them.

Ok, managed to somehow reproduce this. If I comment out 'haystack' from django-haystack==2.4.1 in my project, then it doesn't crash with django-tinymce==2.3.0
I have to do some more work, to get it to crash in your sample app.

Same problem here,  is there a hotfix for this?

@wyx2000 can you update to django-tinymce 2.3.0

Updated. Still same issue. I have to change the code to remove the line 3 and hardcode the root path
Sent from my Samsung device
-------- Original message --------
From: Rémy HUBSCHER<EMAIL_ADDRESS>Date: 2016-03-22  9:14 AM  (GMT-08:00)
To: aljosa/django-tinymce<EMAIL_ADDRESS>Cc: wyx2000<EMAIL_ADDRESS>Subject: Re: [django-tinymce] Has stopped working after upgrade 2.0.4 → 2.0.5 (#132)
@wyx2000 can you update to django-tinymce 2.3.0
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

I had the same problem. Moving my import Model into function fixed the issue!
http://stackoverflow.com/questions/35659282/django-post-migrate-signal-issue

Does anyone have a solution for this issue?


[BREAKING CHANGE] Introduce ImageStyle and DefaultImageStyle

THIS IS A BREAKING CHANGE, STUFF WILL BREAK!!
This moves all parameters from Image, except for the ImageProvider, into a new class: ImageStyle. ImageStyle, like TextStyle, can be merged with another ImageStyle and can be copied with new values.
 The fallback style also contains the default values that RawImage expects (repeat, currently) and a default value for gaplessPlayback (which is false). Also, the difference between new DefaultImageStyle and DefaultImageStyle.merge is highlighted, and also advises users of new DefaultImageStyle to make sure they inherit the values from the default ImageStyle.
I've also provided a unit test for DefaultImageStyle. This also allowed me to find a bug in RenderImage, where the colorBlendMode setter didn't actually update the value. There should probably be a lint for that >_<;
Also, all existing code using Image has been updated (Gallery, Complex Layout, Flutter Markdown).

Can you elaborate on the motivation here? What problem does this solve?

Say I had an album widget that displays album art in multiple places (in the AppBar when expanded and in ListTiles).
For the AppBar, you could use BoxFit.fitWidth to keep contents from being scaled (just clipped), and in the ListTiles you could use BoxFit.contain to ensure that the entire area gets filled. Without this, you'd have to pass around ImageProviders, but then it limits you from setting a color filter, changing alignment, etc. Also, passing around ImageProviders would limit your code to only showing images instead of taking any widget.
In an application I'm working on, I grab a widget from the model that would usually be an Image, but I would also like to pass Placeholders (from a dummy data model which has no actual Images) or do an animation on load later down the road.

@ds84182 I'm a PM on Flutter, and I'd love to learn more. If you're up for it, please send me an email: sethladd @ google. Thanks!

This sounds very similar to IconTheme, which we use to solve this problem for icons. Maybe we should extend what IconTheme does? IconTheme is used by IconImage to apply this to Images, as well as by Icon to apply it to text-based Icons. It does size, opacity, and colour, right now.

@ds84182 any opinions on the IconTheme thing?
@abarth any opinions?

I'm quite confused at how extending IconTheme would be an alternative to this, could you clarify?

Well IconTheme is basically your DefaultImageStyle, just for ImageIcon instead of Image.

I don't see a reason to extend what IconTheme does because images have more style properties than text does. IconTheme is currently limited to what both Image and Text have in common (size, opacity, color), while ImageStyle controls these (size, opacity, color) and more (fit, blending, repeat modes)... Perhaps I'm misinterpreting extend here...

Yeah, if you want to be able to control the repeating then that wouldn't fit IconTheme.
I don't feel doing this for Image widgets as a whole would be a great idea. Imagine if someone enables repeating at the root of their app for whatever reason. Now all their image icons and avatars would look weird. I think it's non-intuitive for images to have this kind of shared behaviour.
I think it would make more sense to have a new AlbumTheme widget and an Album widget, and have the Album widget pass the relevant properties on to the Image from the AlbumTheme.

(That approach would also let you have separate inheritance for icons, avatars, albums, and other categories of images, rather than having all of them share a single group of properties.)

I think we shouldn't do this, for the reasons given above. I'm going to close this PR. I apologise that your experience with Flutter is that we reject your input, and I hope you understand that that makes me sad too. :-( Your code quality was very high and I would love to see more contributions. I encourage you to discuss major changes first, to get people to buy in to the solutions before writing code.
Thanks for your contribution.


Sharing unseen-domain data

Thanks for sharing the test data which includes the seen-domain data. Is there any plan to release the unseen-domain part as well?

@sivasankalpp could be the better person to answer this question. If you do want to test on an unseen domain, you can do so by removing the data of the target domain from training data.

Hey Song, thanks for forwarding. By unseen data, I refer to the covid-related documents and conversations. Adding it will increase the completeness of the dataset.


adding reboot functionality

This is something I whipped up, thoughts? The basic idea is to be able to restart the tokio runtime if it for some reason goes down.
It introduces a few new pieces, mainly:

Reboot state, which just saves the args to `Engine::new
adds the reboot function, which updates the MistralRs sender
Modifies the MistralRs.sender to have a RwLock, so we can safely swap it out with a new sender if the engine goes down
Since we now have a RwLock on the sender, the get_sender() function can potentially fail, so now get_sender() returns a Result.

Before I keep going on this, wanted to get anyone's thoughts on it!
Maybe still to do

I have not worked on the PyO3 stuff, I'm not super familiar with it -- is anything needed there in terms of supporting a restart?
Make more formalized errors to allow not using the anyhow Errors in the main MistralRs struct

@EricLBuehler thanks for the thoughts!

My concern is that we will not know if the engine itself has crashed and therefore there is no more sender to reboot to. I think that if the sender was closed, we should check if the engine itself is alive somehow before restoring it.

Agreed - let me think more about this before my next commit here

Also, I assume that you left out the rebooting logic from interactive mode on purpose? I think that we should include this logic though in the mistralrs_pyo3::Runner methods send_completion_request and send_chat_completion_request.

Will add those when I've made another stab at the above

Okay this is my next stab, here's a quick overview:

On init, save engine parameters into the RebootState, so we can reboot a 'fresh' engine if needed (not purely fresh, uses the same pipeline).
On init, also save the engine thread handler into engine_handler, so we can check if the thread has completed
On get_sender, checks to see if the engine thread has completed. If it has, then the engine is dead.
Then we try to reboot the engine by setting up a new engine, and storing a new mspc channel sender + engine handler

This is a little more robust than the last approach; previously, maybe somehow the sender was dropped without the engine going down. In this new approach, the engine has definitely crashed if the thread has returned.
Also this now auto-reboots on get_sender(), so no extra logic is needed on the server side.
I don't believe there's a potential for deadlock with the RwLocks on the sender + engine_handler, just added that comment in reboot_engine for clarity. It could probably be slightly more efficient; i.e., if multiple senders try to reboot the engine at the same time, we might have a bit of holdup on the locks, but I don't think it should be too drastic since we also check if the engine is alive in reboot_engine before actually rebooting (which prevents multiple subsequent reboots).
Thoughts on this newest version? Still some to-dos with the error stuff, but I'll handle that if this looks like a solid approach.

Here's a version that adds some error types, reworks the logging (warns on a reboot needed), and resolves the merge conflict.
I was able to test that adding a panic into the engine.run() did crash the runtime and it was able to restart (before immediately crashing again because of the panic).
Not sure why the windows check failed, let me know if there's something I can do about that -- looks like a tokenizer didn't have permissions?

Ok, great!
The Windows check is flaky for some reason.

I don't think the clippy stuff is from this PR -- let me know if there's anything else to do for this one!


Eliminate Cargo.lock.msrv

We no longer need it since updating our MSRV to 1.56.1.
Issue #1809

bors r+


Modify config to have overrides that correspond to multiple file types

Currently the config is optimized for .scss files. As when this package was created we were only using these files for styling. We now have other ways of styling that we need to lint.
This pull request aims to refactor the config to adapt for all package styles.

.scss
modules.css
.pcss postcss
.tsx styled react components

To accomplish this, I utilized stylelint's overrides options for changing rules and syntax for other versions.
The config now starts with general rules that can be applied to all situations. These rules are meant to be turned on in all the config file types.
https://github.com/primer/stylelint-config/blob/91d79a769f5f71a1717eb05ad0b09bf91d045c27/index.js#L42-L112
After that I have a config for each type in the overrides:
SCSS
https://github.com/primer/stylelint-config/blob/91d79a769f5f71a1717eb05ad0b09bf91d045c27/index.js#L114-L128
Styled react components
https://github.com/primer/stylelint-config/blob/91d79a769f5f71a1717eb05ad0b09bf91d045c27/index.js#L129-L161
PostCSS
https://github.com/primer/stylelint-config/blob/91d79a769f5f71a1717eb05ad0b09bf91d045c27/index.js#L162-L182
CSS Modules
https://github.com/primer/stylelint-config/blob/91d79a769f5f71a1717eb05ad0b09bf91d045c27/index.js#L183-L188

Cool 😍 do you want me to test this against dotcom, or did you already give it a go?

I already tested in dotcom, going to test in PVC and PCSS also


Fix cartopy

Summary:
Related issue:
Suggested reviewer(s):
Reviewer checklist:

[ ] The headers of all files contain a reference to the repository license
[ ] 100% test coverage of new code - meaning:

[ ] The overall test coverage increased or remained the same as before
[ ] Every function is accompanied with a test suite
[ ] Tests are both positive (testing that the function work as intended with valid data) and negative (testing that the function behaves as expected with invalid data, e.g., that correct exceptions are thrown)
[ ] Functions with optional arguments have separate tests for all options

[ ] Examples are supported by doctests
[ ] All tests are passing
[ ] All names (e.g., files, classes, functions, variables) are explicit
[ ] Documentation (as docstrings) is complete and understandable

The checklist is based on the S-ENDA conventions and definition of done (see General Conventions). The above points are not necessarily relevant to all contributions. In that case, please add a short explanation to help the reviewer.

This PR fixes multiple issues.

Fixing issue #169 after removing cartopy as dependency from sol-indexer.
Fixes #162. Removed debug logging that was used for testing code, and not needed.
Fixing #4 from github.com/metno/solr-indexer

Ready for merge CI containers now working


feat: Handle out-of-band seeks

Resets the Kafka Consumer state to handle admin seeks pushed from the server.

SubscriberState was refactored out into SinglePartitionSubscriber. The main motivation was to give each subscriber state its own mutex and update its internal state (for auto-commits) atomically when pulling messages.

SingleSubscriptionConsumerImplTest was mostly left as-is to verify that this is a no-op.

Added SinglePartitionSubscriber::onSubscriberReset() to reset subscriber state and wait for commits when the RESET signal is received from the server.

Undelivered messages are discarded. Further auto-commits are prevented until post-seek messages are received.

I think the majority of your code is correct for the PR. The main thing I'm getting stuck on is what the seek behavior should be in the presence of client-initiated seeks. I think its reasonable to say that this is unsupported with a comment in the section here https://github.com/googleapis/java-pubsublite-kafka#about-pubsub-lite-kafka-shim and wherever the equivalent section is in the docs.

I think the majority of your code is correct for the PR. The main thing I'm getting stuck on is what the seek reset behavior should be in the presence of client-initiated seeks. I think its reasonable to say that this is unsupported with a comment in the section here https://github.com/googleapis/java-pubsublite-kafka#about-pubsub-lite-kafka-shim and wherever the equivalent section is in the docs.

I'll note in the Devsite docs for admin seek that mixing use of client-initiated and admin seeks is not recommended, as they would clobber each other.
Also, I amended this PR to disable handling admin seeks when the Consumer auto-commit setting is false, since this could also lead to races:

Client responds to reset signal. Server moves admin seek cursor to commit cursor table.

Subscribe stream breaks before sending any messages.
Subscribe stream will reconnect reading from the user commit cursor, not the admin seek cursor.


use another_brother in IOS simulator

Is it possible to use this library in the IOS simulator? It works on a physical device, but when i try to run it in the simulator, it gives the following error:
ld: building for iOS Simulator, but linking in dylib built for iOS, file 'PROJECT_DIRECTORY/ios/Pods/BRLMPrinterKit/BRLMPrinterKit/BRLMPrinterKit.framework/BRLMPrinterKit' for architecture arm64
    clang: error: linker command failed with exit code 1 (use -v to see invocation)

Hi @BentEngbers ,
Thanks for the question. Unfortunately using it in the Simulator is not  due to some missing binary, but I can't remember the specifics of why. Deploying to an actual device will work.
Thanks again,

Thanks for the help!


Added AMD support

Cleaned up the factory loader
Tweaked a bunch of tests where the date was not parsing correctly
Tuned the the .jshintrc file

The date failing test might actually be a problem in the date parser. I haven't looked into it, but changed the time to one that doesn't fail. It might be worth looking into.

@dnemoga Do you approve this commit?

Thanks for the enhancement!
Don't edit the dist files. If you do, other PR's will be unmergeable. Update the src files. Rebase your commits so there will be only one commit with changes to src.

@alexpusch updated

Please remove the dist/*  files changes from your commit. I would update them on next version release.

Sorry. fixed.


Refactor db

Types of changes
Thanks for sending a pull request! Please fill in the following content to let us know better about this change.
Please put an x in the box that applies

[x] Bugfix
[ ] New feature
[x] Refactoring
[ ] Breaking change (any change that would cause existing functionality to not work as expected)
[ ] Documentation Update
[ ] Other (please describe)

Description
Rewrite the templates and db related codes.
Rank query by frontend, the back button on rank page now works well even after multiple queries submit.
Steps to Test This Pull Request
Just test like past.
Expected behavior
No changes, but it brings more karma for me.

The score record steps and methods need rewrite in the future.


Use conda run ... printEnvVariables.py instead of activating conda env

We have had a number of issues with conda activation (failing to start jupyter, switch kernels, etc).
We should just use conda run to activate & dump the env variables.
We stopped using conda run as it didn't support streaming outout (which I believe has been resolve since then), however we don't need that to dump env variables.
This should fix a number of issues we are having with conda environments.
Fix would be required in core extension and should be simple enough to implement.

Closing this as we rely on Python extension to do this.


Problem with OSSRH_GPG_SECRET_KEY secret

GitHub repo: hyperledger/fabric-gateway
The Maven publishing workflow to publish Java artifacts (in this case to GitHub Packages) has recently started failing with the following error:
gpg: signing failed: No secret key

The workflow uses the OSSRH_GPG_SECRET_KEY organization secret as the GPG secret key. Until recently, this has been working fine. The workflow has not changed for a long time. I notice that this secret was last updated 3 weeks ago. Perhaps the updated key is not good, or the format has changed?

@bestbeforetoday the change was to add other repos. Let me dig in.

I need to recover these tokens - I'm working on it.

@bestbeforetoday I've updated them

I am still seeing the same error:
[INFO] --- maven-gpg-plugin:3.2.4:sign (sign-artifacts) @ fabric-gateway ---
[INFO] Signer 'gpg' is signing 5 files with key default
gpg: no default secret key: No secret key
gpg: signing failed: No secret key

As an intermin fix, I added my own credentials as repository-level overrides of these secrets, and publication works fine with them, so there does seem to be an issue with the org-level secrets.
Note that the OSSRH_GPG_SECRET_KEY should be an ASCII-armored private key. With GPG this is exported from the local keyring something like:
gpg --export-secret-key --armor KEY-ID > secret-key.asc

The contents should look something like:
-----BEGIN PGP PRIVATE KEY BLOCK-----

mDMEXzfPyBYJKwYBBAHaRw8BAQdA/pBUfiCdarWpVHzmjfg8mHEQPXGZpea0clIw
/U...etc...
-----END PGP PRIVATE KEY BLOCK-----

Strange. Identus is able to use it. I will look into it later today.

@bestbeforetoday is it working now? https://github.com/hyperledger/fabric-gateway/actions/runs/8685040064/job/23813717831

I added my own credentials to override the org-level secrets so publishing would work. I've removed my overrides now so we'll see when the next change goes in.

thanks. if it doesn't work I don't know what to do - the same credentials are working for Identus

Still fails here: https://github.com/hyperledger/fabric-gateway/actions/runs/10737559636/job/29779682539#step:4:178
I don't obviously see where Identus is actually using these secrets to do GPG signing of Maven artifacts. As mentioned above, this was all working for me for a long time. Nothing changed in my build pipelines but a change to the secrets broke the signing. My own credentials (in repository-level secrets) continue to work fine.

@ryjones The Java publishing to Maven Central for the fabric-protos repository is seeing the same failure here. The fabric-chaincode-java build is able to publish to Maven Central OK using the organization credentials. The deference between this build and the ones that are failing is that fabric-chaincode-java is using Gradle for publishing, with the secrets passed directly into the in-memory signing process using environment variables, whereas the failing builds are publishing using Maven with the signing done using GPG.
A few possibilities occur to me that might be worth checking:

The key stored in the secret is not a private key. I haven't tested this but the Gradle signing process might be happy with any key that can be used for signing, which might include keys other than secret keys. If using GPG, the key needs to be exported using gpg --export-secret-key --armor KEYID (to export the private key), not with gpg --export --armor KEYID (which exports the public key).
The secret contains more than one key. The build failure includes the message "no default secret key". If the GPG keyring contains multiple keys, you would need to specify the key ID on export, not just use gpg --export-secret-keys --armor.
Unsupported key type. This seems unlikely but for reference, the key that is working for me in place of the organization secret is an ed25519 key.
The key has expired. Apparently similar failures occur with GPG signing and the Maven signing plugin if the key has expired so it might be worth double-checking.

@bestbeforetoday I re-did the export as you directed and re-ran the two jobs. It looks like they both worked.

Fantastic, thank you!


Python: Azure credentials cleared when creating AzureCognitiveSearchMemoryStore instance

https://github.com/microsoft/semantic-kernel/blob/c4ef6ab227fc967ab12291cc862852e66d6d75ae/python/semantic_kernel/connectors/memory/azure_cognitive_search/utils.py#L56-L57
The two lines above are clearing the arguments passed to the function, which means that it's impossible to construct a AzureCognitiveSearchMemoryStore instance using anything other than an admin key. This rules out using Managed Identity.
My workaround is to pass admin_key=ManagedIdentityCredential().get_token("https://search.azure.com/.default").token. This is already similar to how I'm handling Azure OpenAI auth, so it's no big deal, it's just not clearly documented anywhere.

Correction: my workaround is to monkeypatch semantic_kernel.connectors.memory.azure_cognitive_search.azure_cognitive_search_memory_store.get_search_index_async_client to return my own client.
The MSI token doesn't work as an admin key because it needs to be passed with an Authorization header.

https://github.com/microsoft/semantic-kernel/pull/2740


PHOENIX 6604 Allow using indexes for wildcard topN queries on salted tables

See: https://issues.apache.org/jira/browse/PHOENIX-6604
Basically due to incorrect resolution of columns any index on a wildcard query * or count(*) will incorrectly be rejected for salted tables.

I cannot run tests locally - probably because I no longer JDK8. I think the test is correct ... let's get a test run it.
Since I'm no longer working in this area I have very limited time for this. :(

Hopefully fixed the simple test (as I said, I cannot run any test locally)

Sorry, I've been away for a while, What do I need to do to trigger the test run and link this PR to the Jira issue?


Wavelet Trees in C, C++, Python, JAVA

Is your feature request related to a problem? Please describe.
A wavelet tree is a compact data structure used for efficiently storing and querying information about a sequence of values or symbols. It is particularly useful for handling large datasets or sequences in a space-efficient manner.
The basic idea behind a wavelet tree is to recursively divide the sequence into smaller segments or intervals, representing different levels of resolution or granularity. At each level, the wavelet tree stores information about the frequencies or occurrences of symbols within that interval. This hierarchical structure allows for efficient searching, counting, and range queries on the original sequence.
Describe the solution you'd like
Time Complexity is O(log(max(A)) , where max(A) is the maximum element in the array A[].
Sample Input
Enter size: 5
Enter the elements: 1 2 3 4 5
Sample Output:
Answer: 2

@Kumar-laxmi please assign me this issue

Assigned!
@anand-harsh : C, C++, Python and Java


Mark WASM buildbots as stable

The wasm32-emscripten and wasm32-wasi build bots have been stable for a while. Mark them as stable.

Here we go! 🚀

@tiran: Recently, I got a few buildbot failures. I ignored them since you was tracking these WASM workers and there was work-in-progress. Since they are now marked as STABLE, I will pay more attention to them. If they are too many (random) failures, I suggest to move back affected workers to UNSTABLE until it's solved.
@tiran: I will keep in touch anyway, and I will not change the buildbot status (stable/unstable) without asking you ;-)
I'm mostly using<EMAIL_ADDRESS>to track worker failures.

@tiran @ethanhs @erlend-aasland and others: Congrats! Sorry my previous message is not very positive. In fact, I'm very happy of what's happening on Python 3.11/3.12 with WebAssembly! It's exciting!

No problem! The WASM buildbots have been stable over the last weeks. We'll get even better hardware soon. On two, three occasions a new commit broke the builds. One time an update to EMSDK 3.1.14 caused failures, too. 3.1.15 is fine again. I'll keep an eye on the buildbots.


[RT] fix(text2num): speed up and allow any radix

Rewrite implementation in a way that will not use exception throwing for empty input processing (valid according to BYOND).
Allow any radix (up to 36), same as BYOND allows

fix #1247

Did you profile again?

Did you profile again?

No

Did you profile again?

Checked text2num in profile again during working on further optimizations:

0.02% of CPU time compared to 5% before the fix. Moreover, the profile shows much fewer text2num calls cause they are so fast now that they fall between the profiler snapshots, and it does not see them. I even had to increase the number of snapshots per second for the function to appear in the profiler report.

That's a really crazy one, but I supported this too


Moving pin by tap and hold

Depends #9

Please support moving pin by tap and hold .
Your package help me a lot 👍

Okay,
I'll add Pin.IsDraggable property and Map.PinDragStart, PinDragEnd and PinDrag events.
Plz check this thread. 👀

@XnainA
I deployed v1.5.0-beta1 to nuget - https://www.nuget.org/packages/Xamarin.Forms.GoogleMaps/1.5.0-beta1 .
This contains as follows:

#10 Moving pin by tap and hold (Android / iOS only)
#45 Support Map.MapClicked and MapLongClicked event (Android / iOS only)

Also I added samples for these.
Please check BasicMapPage and CustomPinsPage in XFGoogleMapSample(dev branch) .
If you (and somebody) say "OK" then I'll deploy as stable version. 👌

v1.5.0 is now stable! 🎉


Fx default tags not cleared between files

Codecov Report

Merging #136 into master will decrease coverage by 0.70%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #136      +/-   ##
==========================================
- Coverage   77.65%   76.94%   -0.71%
==========================================
  Files          30       30
  Lines        1423     1436      +13
==========================================
  Hits         1105     1105
- Misses        318      331      +13

Impacted Files
Coverage Δ

robocop/checkers/tags.py
40.90% <0.00%> (-1.95%)
:arrow_down:

robocop/checkers/naming.py
51.78% <0.00%> (-6.00%)
:arrow_down:

robocop/run.py
47.33% <0.00%> (-0.64%)
:arrow_down:

utest/test_argument_validation.py
99.26% <0.00%> (-0.02%)
:arrow_down:

Continue to review full report at Codecov.

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


Canonical URLs

What is the canonical url for a work or a work version? These would be the urls that DOIs resolve to, or are otherwise the unique url for a given work or work version. All published works and their versions would have one.
Options

Use Blacklight's catalog controller, with the UUID:
/catalog/552b93f8-5997-4d5e-9954-a42b7e5bf7fa
Use a different pathname, and/or a different kind of identifier:
/resources/1234asdf

Architectural Considerations
With option 1, we'd be invested more into Blacklight's Solr-based representation of the resource. This has the advantage of already being there, but means we'd need to maintain the Solr representation along with the Postgres one, if we're intending to still continue using it for display purposes.
Option 2 would allow us to put the work representation in Postgres, and relegate Blacklight's activity to searching only.

Seth and Briana would like to go to Option 2 and UUIDs (12/9/2019)

TODO

revert fe666eb0d62603cb1098bce4b69938b7f1124186 and generate the actual public url


Issue with time travel with store dev tools

One more issue I've encountered -- when traveling back in time using the dev tools, the <select> element only updates the selected option once, despite the value changing correctly in the store if I output the control value to the template directly.

@MrWolfZ This is where the writeValue() call is getting filtered out:
https://github.com/MrWolfZ/ngrx-forms/blob/master/src/control/directive.ts#L115

@MrWolfZ this can also be reproduced using an <input> if you enter a character and delete it (repeat that a few times) and then try to time travel.

It's because the lastViewReportedValue is not updated when state is controlled via the store, which is what happens when time traveling with the dev tools.

@MrWolfZ What do you think to changing lastReportedViewValue to lastReportedValue and also updating it when the value is set by the state rather than user input? I think that by setting it after calling writeValue()  will solve this issue.

@nathanmarks excellent suggestion. That's exactly what I had in mind. I'll call it latestKnownViewValue and it should fix your issue. The problem is event worse than just time travel since if you change the view value to 'A', then change the state to 'B' and back to 'A' the second change will not be applied since it matches the last value reported by the UI. I'll fix this tomorrow (it's almost midnight here).

I fixed this issue in version 1.0.2 which I just released.


solved the minor bug in bitcoin screen

Problem / Issue No.

Clearly describe the Problem or provide reference to Issue Number
Bottom overflowed in Summer of Bitcoin Screen When the Keyboard was opening for the search #132

Describe Problem / Root Cause

Briefly tell the cause of the problem and provide relevant information
The Problem was that there was too much padding between the year column and the below data

Solution proposed

Explain your code changes and how it fixes the issue.
I have removed the padding according to the issue

Additional Information

Relevant information regarding the solution.

Screenshots

Original Screenshot (Problem/Issue)

Updated Screenshot (Fixes/Solution)

Please add gssoc lable before merging


Pull Req for multi VRF support

Right now it seems the BGP role does not support multiple VRF.
In addition to that, the VRF part is very limited.
With this PR I'm trying to address multiple VRF for the BGP configuration, and at the same time add more parameters.

Examples:
dellos_bgp:
  vrfs:
  - name: "AAA"
    ipv4_network:
      - state: present
        address: "<IP_ADDRESS>/24"
      - state: present
        address: "<IP_ADDRESS>/24"
    address_family:
    - type: ipv4
      state: present
      redistribute:
      - route_type: l2vpn
        state: present
    neighbor:
    - type: peergroup
      name: "xlink"
      state: present
      bfd: "true"
    - type: peergroup
      name: "test_deflt"
      password: Banana44
      state: present
      bfd: true
      subnet: "<IP_ADDRESS>/24"
      address_family:
      - type: ipv4
        state: present
        default_originate: true
    - type: ipv4
      description: "Crosslink"
      interface: vlan2601
      peergroup: "xlink"
      peergroup_type: ibgp
      admin: up
      state: present
    - type: ipv4
      description: Test
      admin: down
      state: present
      ip: <IP_ADDRESS>
      address_family:
      - type: ipv4
        state: present
        activate: true
        soft_reconf: true
        next_hop_self: true

@ssasso Thanks for the PR. Our team is currently testing the PR. Once qualified we will merge it.


Add acceptance tests for dns_firewall

This adds some basic acceptance tests to ensure the DNS Firewall resource works correctly.

cheers legend
TF_ACC=1 go test ./internal/services/dns_firewall/ -run "^TestAccCloudflare" -count 1 -v
=== RUN   TestAccCloudflareDNSFirewall_Basic
--- PASS: TestAccCloudflareDNSFirewall_Basic (4.32s)
=== RUN   TestAccCloudflareDNSFirewall_Update
--- PASS: TestAccCloudflareDNSFirewall_Update (11.79s)
PASS
ok  	github.com/cloudflare/terraform-provider-cloudflare/internal/services/dns_firewall	17.334s


Resolve issues #12 (gzipped S3) and #13 (readline)

This pull request includes the changes from https://github.com/piskvorky/smart_open/pull/75 (which I have now closed).

Looks like Python 3.x builds are failing. I'm new to Python 3.x, but will see if I can fix these issues during the week.
On a side note, I'm not sure why these problems have come up now as opposed to https://github.com/piskvorky/smart_open/issues/75 - the code is mostly the same. If anyone can shed light onto this, I'd appreciate it greatly.

Looks good. Just waiting for Travis builds to come back after we moved project to another org.
Adding a couple of lines in the CHANGELOG would be good too.

@tmylk What date and version should I use in the changelog? Are placeholders acceptable?

@mpenkov Thanks for updating the changelog! Once gzipstream tests run, then will merge in.

Thanks a lot for this great PR!

You're welcome. Thank you for such a useful module!
On Mon, Jun 27, 2016 at 23:35 Lev Konstantinovskiy<EMAIL_ADDRESS>wrote:

Thanks a lot for this great PR!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/RaRe-Technologies/smart_open/pull/76#issuecomment-228867188,
or mute the thread
https://github.com/notifications/unsubscribe/ABDOVL-hBEWiQfikcyNdFG5mE-9oAwjTks5qQDQhgaJpZM4Iz08B
.


Watcher on $attrs.itemsPerPage added multiple times means all fire

I am using bootstrap pagination twice on a page, once above the content and once below. For both of these, the $attrs.itemsPerPage watcher in pagination.js on line 17 is added. When I modify itemsPerPage both watches fire. This is undesirable as update logic on my page is tied to the changing of the page number which calculateTotalPages() can cause to occur. This in turn causes ng-change to fire twice.
The current code is this.
$scope.$parent.$watch($parse($attrs.itemsPerPage), function(value) {
        self.itemsPerPage = parseInt(value, 10);
        $scope.totalPages = self.calculateTotalPages();
});

Would this be a good solution?
$scope.$parent.$watch($parse($attrs.itemsPerPage), function(newValue, oldValue) {
        if (newValue !== oldValue)
        {
                self.itemsPerPage = parseInt(newValue, 10);
                $scope.totalPages = self.calculateTotalPages();
        }
});

edit: what I thought was happening isn't actually happening (its still being called twice) but ng-change is not being fired twice due to that, that was something weird going on in the code I'm refactoring.

So something that should be noted, the $watch callback will only fire if newValue !== oldValue is already true. Something else is likely going on with what you are observing.
Here is a Plunker for you to modify/tweak if it helps for you to find what is the problem.


fix package names for Cloud5

we have ruby2.1 there
this only broke when trying to install from git

:+1: for this fix, but I'm pretty sure this is not the only path where it needs to be fixed.

+1

Is the list of packages up-to-date, actually?
Also, I'm not sure we want to merge stuff in release/tex/master yet as we're not using the branches yet.

Currently it have to be done against master, and not only for the tex path.

-1, should be fixed in master instead of release/tex/master


WIP: KFP V2

Description
TODO:

[x] Add KFP Artifact repo
[ ] Update docs
[x] Update CI/CD pipelines for publishing workflow
[x] Remove Tensorflow pipelines
[x] Add container builds to CI/CD
[x] Test CI/CD
[x] Add tests for new component
[x] Serving container image
[x] test serving container image
[x] Copyright headers

How has this been tested?
Please explain how you have tested the new changes.
Checklist

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have successfully run the E2E tests, and have included the links to the pipeline runs below
[ ] 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
[ ] I have updated any relevant documentation to reflect my changes
[ ] I have assigned a reviewer and messaged them

Pipeline run links:

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun


Cannot read mat files saved from Python with 7.3 format

I saved a .mat file from Python using hdf5storage.savemat with the default format 7.3.
In my C++ application, I opened the file, read the structure, and checked the number of fields.
matioCpp::File input(file);
matioCpp::Struct outStruct = input.read("robot").asStruct();
std::vector<std::string> listVar = input.variableNames();
std::cout << "Num variables: " << listVar.size());

However, the output was 0. I verified the content in Matlab, and it was correct.
In summary, the file opening works fine, but the variable returned by the read method is empty https://github.com/ami-iit/matio-cpp/blob/a0daf0691d492b2ed50910ea984f97bc2f945b80/include/matioCpp/File.h#L163
I solved the problem by saving the .mat file with format 5 instead of 7.3.
cc @S-Dafarra

Thanks @isorrentino, I started opening a PR to print an error when a variable is not found in the file:  https://github.com/ami-iit/matio-cpp/pull/74
In any case, I suspect it is an issue of matio. Can you also comment with the actual error message?

Sure!
isorrentino@IITICUBLAP263:~/dev/robotology-superbuild/build/src/bipedal-locomotion-framework (master)$ ./bin/RobotDynamicsEstimationUnitTests
Randomness seeded to:<PHONE_NUMBER>
HDF5-DIAG: Error detected in HDF5 (1.10.7) thread 1:
  #000: ../../../src/H5L.c line 1168 in H5Literate(): link iteration failed
    major: Links
    minor: Iteration failed
  #001: ../../../src/H5L.c line 3350 in H5L__iterate(): link iteration failed
    major: Links
    minor: Iteration failed
  #002: ../../../src/H5Gint.c line 853 in H5G_iterate(): error iterating over links
    major: Symbol table
    minor: Iteration failed
  #003: ../../../src/H5Gobj.c line 692 in H5G__obj_iterate(): can't iterate over symbol table
    major: Symbol table
    minor: Iteration failed
  #004: ../../../src/H5Gstab.c line 553 in H5G__stab_iterate(): iteration operator failed
    major: Symbol table
    minor: Can't move to next iterator location
  #005: ../../../src/H5B.c line 1203 in H5B_iterate(): B-tree iteration failed
    major: B-Tree node
    minor: Iteration failed
  #006: ../../../src/H5B.c line 1160 in H5B__iterate_helper(): B-tree iteration failed
    major: B-Tree node
    minor: Iteration failed
  #007: ../../../src/H5Gnode.c line 1000 in H5G__node_iterate(): iteration operator failed
    major: Symbol table
    minor: Can't move to next iterator location
HDF5-DIAG: Error detected in HDF5 (1.10.7) thread 1:
  #000: ../../../src/H5L.c line 1168 in H5Literate(): link iteration failed
    major: Links
    minor: Iteration failed
  #001: ../../../src/H5L.c line 3350 in H5L__iterate(): link iteration failed
    major: Links
    minor: Iteration failed
  #002: ../../../src/H5Gint.c line 853 in H5G_iterate(): error iterating over links
    major: Symbol table
    minor: Iteration failed
  #003: ../../../src/H5Gobj.c line 692 in H5G__obj_iterate(): can't iterate over symbol table
    major: Symbol table
    minor: Iteration failed
  #004: ../../../src/H5Gstab.c line 553 in H5G__stab_iterate(): iteration operator failed
    major: Symbol table
    minor: Can't move to next iterator location
  #005: ../../../src/H5B.c line 1203 in H5B_iterate(): B-tree iteration failed
    major: B-Tree node
    minor: Iteration failed
  #006: ../../../src/H5B.c line 1160 in H5B__iterate_helper(): B-tree iteration failed
    major: B-Tree node
    minor: Iteration failed
  #007: ../../../src/H5Gnode.c line 1000 in H5G__node_iterate(): iteration operator failed
    major: Symbol table
    minor: Can't move to next iterator location
[2023-11-06 19:24:50.150] [thread: 56007] [blf] [info] Num variables --> 0
[2023-11-06 19:24:50.150] [thread: 56007] [blf] [info] Number of fields --> 0

Just to double check, can you open the file in MATLAB?

Just to double check, can you open the file in MATLAB?

I already did it, I wrote it in the first comment. It has the expected content.

Just to double check, can you open the file in MATLAB?

I already did it, I wrote it in the first comment. It has the expected content.

Ah sorry, I misread the first comment.

@isorrentino could you also link or upload the file that is not working?

It would also be interesting to see the output of matdump -g -f whos <filename>.mat .

In any case, I guess this is expected considering that it is a matio tool: https://github.com/tbeu/matio/blob/da3c5f61154ac000dcc3980fbe939cda3da2447b/tools/matdump.c

Yes, I wanted to quickly make sure matio-cpp was not involved in any way.

The same error happens also with hdf5 1.14 :
(libmatio) traversaro@IITICUBLAP257:~$ matdump -f whos four_joints_two_ft_two_imu_with_noise_bias_friction.mat
HDF5 error #000 in H5Literate2()
      file : H5L.c:1647
      major: Links
      minor: Iteration failed
HDF5 error #001 in H5L__iterate_api_common()
      file : H5L.c:1612
      major: Links
      minor: Iteration failed
HDF5 error #002 in H5VL_link_specific()
      file : H5VLcallback.c:5517
      major: Virtual Object Layer
      minor: Can't operate on object
HDF5 error #003 in H5VL__link_specific()
      file : H5VLcallback.c:5483
      major: Virtual Object Layer
      minor: Can't operate on object
HDF5 error #004 in H5VL__native_link_specific()
      file : H5VLnative_link.c:368
      major: Links
      minor: Iteration failed
HDF5 error #005 in H5L_iterate()
      file : H5Lint.c:2146
      major: Links
      minor: Iteration failed
HDF5 error #006 in H5G_iterate()
      file : H5Gint.c:869
      major: Symbol table
      minor: Iteration failed
HDF5 error #007 in H5G__obj_iterate()
      file : H5Gobj.c:649
      major: Symbol table
      minor: Iteration failed
HDF5 error #008 in H5G__stab_iterate()
      file : H5Gstab.c:507
      major: Symbol table
      minor: Can't move to next iterator location
HDF5 error #009 in H5B_iterate()
      file : H5B.c:1171
      major: B-Tree node
      minor: Iteration failed
HDF5 error #010 in H5B__iterate_helper()
      file : H5B.c:1133
      major: B-Tree node
      minor: Iteration failed
HDF5 error #011 in H5G__node_iterate()
      file : H5Gnode.c:941
      major: Symbol table
      minor: Can't move to next iterator location
(libmatio) traversaro@IITICUBLAP257:~$ mamba list
# packages in environment at /home/traversaro/miniforge3/envs/libmatio:
#
# Name                    Version                   Build  Channel
_libgcc_mutex             0.1                 conda_forge    conda-forge
_openmp_mutex             4.5                       2_gnu    conda-forge
c-ares                    1.21.0               hd590300_0    conda-forge
ca-certificates           2023.7.22            hbcca054_0    conda-forge
hdf5                      1.14.2          nompi_h4f84152_100    conda-forge
keyutils                  1.6.1                h166bdaf_0    conda-forge
krb5                      1.21.2               h659d440_0    conda-forge
libaec                    1.1.2                h59595ed_1    conda-forge
libcurl                   8.4.0                hca28451_0    conda-forge
libedit                   3.1.20191231         he28a2e2_2    conda-forge
libev                     4.33                 h516909a_1    conda-forge
libgcc-ng                 13.2.0               h807b86a_2    conda-forge
libgfortran-ng            13.2.0               h69a702a_2    conda-forge
libgfortran5              13.2.0               ha4646dd_2    conda-forge
libgomp                   13.2.0               h807b86a_2    conda-forge
libmatio                  1.5.24               h31675a7_0    conda-forge
libnghttp2                1.55.1               h47da74e_0    conda-forge
libssh2                   1.11.0               h0841786_0    conda-forge
libstdcxx-ng              13.2.0               h7e041cc_2    conda-forge
libzlib                   1.2.13               hd590300_5    conda-forge
ncurses                   6.4                  h59595ed_2    conda-forge
openssl                   3.1.4                hd590300_0    conda-forge
zlib                      1.2.13               hd590300_5    conda-forge
zstd                      1.5.5                hfc55251_0    conda-forge

Instead  h5dump four_joints_two_ft_two_imu_with_noise_bias_friction.mat works fine, so the problem perhaps is in libmatio ?

Opened issue upstream: https://github.com/tbeu/matio/issues/221

Hi @S-Dafarra, just to let you know, I saw that the issue here https://github.com/tbeu/matio/issues/221 has been closed.

Yes, the problem should be fixed in the release https://github.com/tbeu/matio/releases/tag/v1.5.25
Closing.


change read me

Motivation:
Explain the context, and why you're making that change.
To make others understand what is the problem you're trying to solve.
Modification:
Describe the idea and modifications you've done.
Result:
Fixes #.
If there is no issue then describe the changes introduced by this PR.

Codecov Report

Merging #216 into master will decrease coverage by 0.07%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #216      +/-   ##
============================================
- Coverage     71.36%   71.29%   -0.08%
+ Complexity      960      957       -3
============================================
  Files           341      341
  Lines         14213    14213
  Branches       2286     2286
============================================
- Hits          10143    10133      -10
- Misses         2879     2882       +3
- Partials       1191     1198       +7

Impacted Files
Coverage Δ
Complexity Δ

...ent/aft/impl/ServiceHorizontalMeasureStrategy.java
84.44% <0%> (-2.23%)
26% <0%> (-1%)

...n/java/com/alipay/sofa/rpc/common/SofaConfigs.java
83.92% <0%> (-1.79%)
0% <0%> (ø)

.../sofa/rpc/bootstrap/DefaultClientProxyInvoker.java
87.09% <0%> (-1.62%)
0% <0%> (ø)

.../sofa/rpc/registry/zk/ZookeeperRegistryHelper.java
75% <0%> (-1.43%)
0% <0%> (ø)

...ava/com/alipay/sofa/rpc/config/RegistryConfig.java
89.21% <0%> (-0.99%)
0% <0%> (ø)

...alipay/sofa/rpc/registry/zk/ZookeeperRegistry.java
59.37% <0%> (-0.79%)
0% <0%> (ø)

.../alipay/sofa/rpc/registry/local/LocalRegistry.java
57.22% <0%> (-0.61%)
25% <0%> (-1%)

...ay/sofa/rpc/client/AllConnectConnectionHolder.java
62.84% <0%> (-0.28%)
0% <0%> (ø)

.../main/java/com/alipay/sofa/rpc/tracer/Tracers.java
44.87% <0%> (+2.56%)
0% <0%> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


Samuel Wang Individual Review Ticket Week 2

Made Flashcard system
Implemented notes, trying to make discussion board with new table
discussion board crud progress

Reviewed By Lucas Ho
Score: 1.8/2
Overall, a very good job. Commits tangible shows how he understands databases and how to write code. Good usage of javascript and database usage to store information and create and update things inputted. Very good boj.


add parenthesis so that expression compiles

To prevent : value to is not a member of Boolean val result = 1.to(10) == 1 to 10

Changes here were incorporated eventually elsewhere -- closing


Empty cool and mcool files

Hi!
I am running the Hi-C Processing Pipeline on my data but I noticed some issues when generating the cool and mcool files at the matrix generation step.
While the .hic matrix looks perfectly fine by some downstream analysis, I have noticed that both .cool and .mcool files are empty. By empty, I mean that the contact matrices stored in these formats is full of zeros or NaN. However, the matrix stored in the .hic is how I would expect it. Both matrices are generated from the same pairs file.
Here I show the commands that I used to generate the .cool file:
singularity exec 4dn-hic_latest.sif cooler cload pairix -p ${THREADS} -s ${MAX_SPLIT} ${CHR_SIZE}:${BIN_SIZE} ${PAIRS} ${SAMPLE_DIR}/${SAMPLE}.cool
I think the problem occurs already when generating the .cool file in the very first step as it immediately results to be empty, I checked its emptiness by plotting the matrix as follows:
clr = cooler.Cooler('bin.cool')
mat = clr.matrix(balance=False).fetch('chr1')
plt.matshow(np.log10(mat), cmap='YlOrRd')
Also, the maximum value of the matrix is a 0.
This doesn't occur for the .hic matrix, whose plots look fine and whose matrix is not filled with 0s.
Thank you in advance for your help!

Hi Lara,
Hm, it's tough to say what is happening. Can you see any logs or output from the singularity command that could help identify the issue?
One particular issue that we've seen is that the chromsizes file needs to exist in the same directory.  You can see slightly more context in our cwl for that step, which I'm guessing you've already seen.

If you are having issues with the pairix loader, you can also try cload pairs which doesn't require the index file.

Hello, thank you for your replies!
I was able to find out what the issue was: I must have compromised the index file that is needed to compute the cool matrices but not the hic matrix.


ExternType fails static assert because the C++ type has a copy constructor

Hello!
I'm creating a wrapper for OpenImageIO, and I'm running into an issue where I can't pass the static check https://github.com/dtolnay/cxx/blob/743caa283ced87d021e67c65ced1860fe0a11ecc/gen/src/write.rs#L436 because of this copy constructor https://github.com/OpenImageIO/oiio/blob/665c634b12ea8bc206e7a5dd9475cffa974e8522/src/include/OpenImageIO/typedesc.h#L165.
I could be wrong, but it it looks like this should pass, so this may be a case where removing the static check may be okay, however, having this in the first place did stop me to make sure I wasn't being a complete idiot and brought me to where we are now.
So, I have two main questions:

Is this static assert correct, and I have to wrap this in a unique pointer (or some option C)?
If this static assert can go away, then are we able to do this in a way where we can say "I'm absolutely sure that this type is okay to use" and still keep the static checks in place? Or, would it be on me in my c++ wrapper side to make sure the static asserts are in place?

I think #359 is the proposal to provide just the sort of override you need.

Ack. That's what happens when you post before really looking at the issues. I'll close this thread.


Add verification of the range of integer values for TLVReader -- 2nd attempt

Problem
Currently, TVLReader's method Get does not validate sizes of requested integers. In case when the size is less than size of TVL field, incorrect value is returned to caller; no error condition is indicated.
Summary of Changes
Size validation is added, error conditions are handled properly.
Fixes #5152
[see https://github.com/CHIP-Specifications/connectedhomeip-spec/issues/2115 and https://github.com/project-chip/connectedhomeip/pull/5286 discussions]

@bitkis Any update on the PR feedback?


(next) - use the App and PageContext types provided by Next

Summary
This uses the built-in types provided by Next to type the context used in our getInitialProps.
Fixes: https://github.com/FormidableLabs/urql/issues/1213
Testable at: https://codesandbox.io/s/twilight-rgb-vxuwu?file=/pages/index.tsx
Set of changes

Type getInitialProps with AppContext and PageContext

Closing since this doesn't solve the issue, this will most likely relate to using the generic available on withUrql

Closing since this doesn't solve the issue, this will most likely relate to using the generic available on withUrql


sw-6615: Uses string values 'true' and 'false' for nullable booleans

Codecov Report
Patch coverage: 100.00% and project coverage change: +1.27 :tada:

Comparison is base (fda7e4d) 70.74% compared to head (94a9518) 72.01%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #356      +/-   ##
==========================================
+ Coverage   70.74%   72.01%   +1.27%
==========================================
  Files          23       25       +2
  Lines        1145     1147       +2
==========================================
+ Hits          810      826      +16
+ Misses        287      273      -14
  Partials       48       48

Impacted Files
Coverage Δ

internal/server/client_tasks.go
77.01% <100.00%> (-4.89%)
:arrow_down:

... and 4 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


GPIO Wakeup (TZ-1122)

Question
Good morning everyone,
I have a doubt regarding the GPIO wakeup mode combined with Zigbee light sleep using the command esp_zb_sleep_now();.
Is it possible to wake up the ESP32-H2 with GPIO (without using EXT1)?
I have done some tests, but I couldn't get it to work, while I have no issues with EXT1.
Reading the documentation, I found this:
https://docs.espressif.com/projects/esp-idf/en/stable/esp32h2/api-reference/system/sleep_modes.html#gpio-wakeup
My goal is to wake up the ESP32-H2 when one of N pins changes state and then invert the condition for wakeup.
For example:

pin12 and pin13 are configured to wake up on a low level.
pin12 goes low, pin13 remains high, so the ESP wakes up and inverts the wakeup condition only for pin12.

So the new configuration for the next wakeup will be:

pin12 wakes up with HIGH, pin13 wakes up with LOW, and so on...

Is this possible?
Thanks!
Additional context.
No response

Another problem I'm encountering is putting the device to sleep and waking it up on state changes, rather than on a specific level.
For example:
Wake up on a low pin state, go back to sleep, and wake up again when the state goes high.
Is this possible?

Hi @sbusceti
It sounds like a question that is not related to the zigbee stack. Could please raise a question to IDF project?
Personally, it seems to be feasible, you could find some examples for light sleep/wakeup here:https://github.com/espressif/esp-idf/tree/master/examples/system/light_sleep


fix(code): make bg more bright

To have higher contrast and ease readability for the code blocks, I suggest using the same background color as the default GitHub syntax highlighting.
Before

After

I like the change for big code blocks. 👍 Wondering if we should keep the darker one for inline code snippets though, the lighter background kinda disappears at very small scale.

That's a good idea!
And btw the only thing I don't like about the lighter color is that there is less blue in there. We could play around a bit more to find a light color with more blue as well.

Added an item to the Design board: https://www.notion.so/prismaio/Docs-Review-code-block-color-inline-and-block-9f468709d8794740b513718a248799d8

Following up on this here - I think this is reasonable. As far as the right color to use - it looks like the color of the actual page background is close to the lightness @timsuchanek used and is (obviously) in our palette. Thoughts @kuldar ?

Taking Design team's comments as approval and merging. :)


IMG Height

Max image height on the preview image now matches the value of
data-height if set. This also solves issue #16

I've just helpfully realised I didn't update the minified version.

Already included in the plugin.


Add Crystal lang

Remove duplicate Rust language entry
Add Crystal lang to the list of languages

Looks good! But please create an issue next time to add a new language 🙂.
Because last year this repository got into the spam category due to these types of pull requests

@all-contributors please add @ur5us for content


I wanted to add query string to the hub connection. Couldnt found a way out?

I wanted something equivalent to [$.connection.hub.qs] which we set up while using jQuery.
Please Help!!

Yes, I can implement this feature. Should have it done by the end of the week.

Hi , did you try doing it?
I tried , but on first loginof my application if successfully works , but on refresh of page i am not able to send back the query string again , though the connection is trying to reconnect.
async connection() {
if (!this.connected) {
this.connected = await this.hubService.connect(this.url).toPromise();//environment.signalRURL
this.hubService.connection.qs = { 'ClientId': this._auth.getUser().SystemUserId };
this.hubWrapper = this.hubService.register(this);
}
}

Sorry. Christmas time is busy for me. I will try to get to this some time this week.

Thanks for the response , but i figured out a way .
i modified your library with an extra parameter in initConnection() method.
Thanks!!


No corresponding source for the libffi build

There is no pointer to the source code for the specific version of libffi that is included in binary form in this repository, nor instructions to rebuild it. This means that if I find a problem with libffi, I can't debug it.

Hi @mwcampbell,
We haven't included the sources for libffi because we built it from two separate code bases. We built it once for arm64 with this revision https://github.com/atgreen/libffi/commit/536366349ced9d57c873d64e219ba30992a83c06 and we built it for armv7, i386 and x86_64 with this revision https://github.com/atgreen/libffi/commit/20562ac0427c3578250d04c6e34fb0127d4551cf (release 3.2.1). If we could automate the build of libffi we would have included it as a submodule like we did WebKit, but as it is right now we can't build armv7 and arm64 from the same revision.
The source isn't otherwise modified, so combining the builds of those two revisions in a fat binary will yield the same libffi library that's currently in the repo.
Unfortunately, debugging libffi right now means building the correct revision locally and replacing the production library in the repo with the debug build.

The issues that precluded libffi from building for armv7 and arm64 from the same revision have been resolved in our fork - https://github.com/nativescript/libffi.


[qwik-pod] Accessing a File is not showing anything

Trying to access something like blob/master/README.md/ is not populating the component with the content of the file.

This has been fixed here https://github.com/thisdot/starter.dev-github-showcases/issues/1085