Flash not working

In the example app, the flash works, but when integrating I encountered the following issue.
I was unable to use the flash, FlCameraMethodCall.instance.setFlashMode didn't turn it on and sometimes it crashed the app.
The camera is reinitiated (it turns black for a second) and I receive this log.
D/Camera2CameraImpl(12027): {Camera@a5053bb[id=0]} Releasing session in state CLOSING
D/Camera2CameraImpl(12027): {Camera@a5053bb[id=0]} Transitioning camera internal state: CLOSING --> REOPENING
D/CameraStateRegistry(12027): Recalculating open cameras:
D/CameraStateRegistry(12027): Camera                                       State
D/CameraStateRegistry(12027): -------------------------------------------------------------------
D/CameraStateRegistry(12027): Camera@c1ba6a2[id=1]                         UNKNOWN
D/CameraStateRegistry(12027): Camera@a5053bb[id=0]                         OPENING
D/CameraStateRegistry(12027): -------------------------------------------------------------------
D/CameraStateRegistry(12027): Open count: 1 (Max allowed: 1)
D/CameraStateMachine(12027): New public camera state CameraState{type=OPENING, error=null} from OPENING and null
D/CameraStateMachine(12027): Publishing new public camera state CameraState{type=OPENING, error=null}

Crash log:
F/libc    (19250): Fatal signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0x660 in tid 28867 (1.raster), pid 19250 (h.app.identifier)
*** *** *** *** *** *** *** *** *** *** *** *** *** *** *** ***
Build fingerprint: 'OnePlus/OnePlus8Pro_EEA/OnePlus8Pro:11/RP1A.201005.001/2106020006:user/release-keys'
Revision: '0'
ABI: 'arm64'
Timestamp: 2021-08-02 16:51:10+0200
pid: 19250, tid: 28867, name: 1.raster  >>> app.identifier <<<
uid: 10810
signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0x660
Cause: null pointer dereference
    x0<PHONE_NUMBER>000660  x1  00000077f0cb2370  x2<PHONE_NUMBER>000000  x3<PHONE_NUMBER>88c2ab
    x4  00000077f0cb2358  x5<PHONE_NUMBER>00004a  x6<PHONE_NUMBER>ff5140  x7  7f7f7f7f7f7f7f7f
    x8  0000007b5d51e470  x9  bced3c33b4707a25  x10<PHONE_NUMBER>430000  x11<PHONE_NUMBER>00001e
    x12<PHONE_NUMBER>906454  x13<PHONE_NUMBER>90649c  x14<PHONE_NUMBER>9064fc  x15<PHONE_NUMBER>000000
    x16 0000007b5b1e3c08  x17 0000007b5b087b18  x18 00000077f03ee000  x19<PHONE_NUMBER>000660
    x20<PHONE_NUMBER>000000  x21 b4000079d80b3920  x22 00000077f0cb4000  x23 b4000079d80b39d8
    x24<PHONE_NUMBER>3a67e0  x25 00000077f0cb4000  x26<PHONE_NUMBER>000155  x27<PHONE_NUMBER>000001
    x28<PHONE_NUMBER>000003  x29 00000077f0cb23e0
    lr  0000007b5b1de380  sp  00000077f0cb23e0  pc  0000007b5b087b20  pst<PHONE_NUMBER>001000

I can't see any useful information in the log ,  You can try using FlMLKitScanningMethodCall. instance. setFlashMode


How to use Dynamic value with :text like v-model variable ?

How to use Dynamic value with :text
like v-model variable ?

// text = 'Some text to be typed with vue-typer'

VueTyper does not support the v-model directive for two-way bindings. VueTyper is meant to be used as a 'label' or 'text view', rather than some interactive element such as a form input. That being said:

text can already be updated dynamically (but note that VueTyper will reset and start typing the new text from scratch)
The existing custom events may suit your needs
If you're looking for something like the standard  change event, then a PR would be welcome! (Just keep in mind how to deal with change events on different 'words', because text can be an array of 'words'.)


No Linux binaries of hlint v3.3.5 at release page

This
https://github.com/haskell-CI/haskell-ci/blob/master/src/HaskellCI/GitHub.hs#L353
fails.
I guess, your CI failed to produce the artefact?

Yes.
From release to release there is a wild discrepancy of packaging, some have linux some do not,` some have only sources.
This breaks the GitHub CI HLint action latest release install, so I set 3.3.1 in https://github.com/haskell/haskell-language-server/pull/2537.

Seems not just GitHub action, but haskell-ci also: https://github.com/xmonad/xmonad-contrib/runs/4616053963?check_suite_focus=true

I've just spun a new 3.3.6 which has a working release. Can someone confirm this fixes the haskell-ci issues etc

I've just spun a new 3.3.6 which has a working release. Can someone confirm this fixes the haskell-ci issues etc

Can confirm that at least xmonad's CI now works perfectly again!


Deprecated props are being used in the Storybook use-infinite-scroll hook's page.

Which package has an issue?
@yamada-ui/react
Package Version
1.4.8
Description
Deprecated props are being used in the Storybook use-infinite-scroll page.
Use fontSize instead of size.
Link to Reproduction
No response
Steps to Reproduce

Go to '...'
Click on '...'
Scroll down to '...'
See error

Browser
None
Operating System

[ ] macOS
[ ] Windows
[ ] Linux
[ ] iPadOS
[ ] iOS
[ ] Android

Do you know how to fix the issue?
None
Are you willing to participate in fixing this issue and create a pull request with the fix?
None
Additional Information
No response

Hi, I'd like to work on this! Just to clarify, the files that would be edited are stories/hooks/use-infinite-scroll.stories.tsx and docs/contents/hooks/use-infinite-scroll.ja.mdx?

Hi, thanks for the comment!
Yes, pls!
Yes, that file is correct.


can i use this for multi instance

suppose i have to sockets files
can i use this plugin for multi instance

We're using HAProxy in multi-process mode (so we can terminate SSL traffic across multiple cores), so each process has it's own set of stats, and it's own socket.
I think the simplest option would be a config option that took a 'name' option - that can default to 'haproxy', but overriding it would change the left most level of the metric name from (eg) haproxy.backend123.gauge.avg_response_time to $name.backend123.gauge.avg_response_time


moving master to the new metrics names in 1.7

@tzach please review

I need Scylla 1.7 RPM or Docker to test it

@amnonh I validated 1.6 and 1.5 dashboard work as is, so production scylla users will not be affected.
Once I can test scylla 1.7 I will merge.


Binaries in "example" directory?

How mature is the software currently? Is it still active and maintained? What's the reason to build the binaries in the "example" directory?

I perform only basic test on this package.  I am still willing to improve this repo, but currently have no device in hand, so any change request will be taken slowly.  I am not a expert on CMake utility, the output binary in "example" or "test" folder, maybe caused by add subproject directive, I guess.  I will look at it at my free time.

Use
set(EXECUTABLE_OUTPUT_PATH ${CMAKE_BINARY_DIR})
to make the binary output to build folder.


Copy gdbserver ourselves if gdbserver can't be found

This fixes debugging 64-bit native APKs. The previous solution wasn't
completely correct and required root on the device/emulator.
On Android, the files of an application are extracted to the system owned
directory /data/app/<appname>-<int>/lib
For 32-bit shared libraries, Android will create a symlink from the app owned
directory /data/data/<appname>/lib to maintain backwards compatibility with
previous API versions.
There is no such symlink for 64-bit libraries, since Android only supports
64-bit since API 21, at which point the app owned directory was no longer used
for libs.
@chuckries @gregg-miskelly @jacdavis

FYI the CI build doesn't seem to be checking unit tests anymore
http://dotnet-ci.cloudapp.net/job/microsoft_miengine_prtest/361/console

:+1:

:+1:


add all features as a virtual column

@AparnaKarve @imtayadeway this is a more generalized approach to https://github.com/ManageIQ/manageiq/pull/15600
But

I'm not sure how adding that many virtual columns to all models would impact performance
can't we change the REST API to filter on methods?
would dropping the ? be an backwards incompatiblity change?

can't we change the REST API to filter on methods?

That might be bad, because some methods have side effects

would dropping the ? be an backwards incompatiblity change?

I think we originally dropped the ?s to make these look more like bona fide columns, and also to avoid potential confusion when reading the query string in the URL (which starts with one).
Incidentally, there are probably some deletions that could be made since I think (since we already added a few of these individually IIRC)

@imtayadeway so you are saying, yes drop the ? ?
also, would it make sense to add all supports features a columns?
If you think so, we should ping the performance guys if this change is ok

@durandom yeah, I am 👍 to drop the ?, and with this in general if it doesn't adversely affect performance 😁
Grepping for existing virtual columns didn't turn up anything for me - perhaps @AparnaKarve will rememeber?

@kbrock could you have a look wether adding all those virtual_columns is ok?

@durandom I do like this generic approach that you have taken here.
Hopefully performance is not an issue here.
But if it is, then I think we should stick to the approach of cherry-picking only those support_* methods that we absolutely need to be queryable by the API.
I assume that this is a growing list and that there will be more and more support_* methods that would be added down the line, so that is a factor to be considered as well.

virtual columns are really meant for exposing things to reporting.  They are not really meant for exposing things to the API.  I am concerned this opens up a lot of things to reporting that users just don't care about.  This feels more like it should be an OPTIONS specific thing in the API.

@imtayadeway can we change the filter code to treat all filters that begin with supports_ as queries to the feature mixin?

virtual columns work best in the base class (or the root of the query) and ones that do not change in subclasses.
Having various subclasses override values in a column is a recipe for some serious issues.
But that may be more of a statement around the features api than virtual columns

@imtayadeway @durandom Considering all these above points, should we get #15600 merged, since it's a PR for a high priority BZ?

I think this comment from @Fryguy applies to #15600 as well, right?  I mean, in either case we're talking about adding virtual columns for API and not reporting, it seems.

@blomquisg Yes

@imtayadeway reg. https://github.com/ManageIQ/manageiq/pull/15614#issuecomment-316778634 - is there a central place where we could intercept filters that start with supports_ ?

@durandom it would be difficult given that it's backed by MiqExpression, which only deals with columns or virtual columns. This probably warrants a deeper discussion - it seems there are some growing pains to do with the idea of virtual columns as being for reporting, and the unfortunate fact that the API is leveraging them quite integrally while there is resistance to virtual columns growing to meet API needs. ISTM that if we can't do the latter there might be some serious work ahead.

This feels more like it should be an OPTIONS specific thing in the API.

@imtayadeway that sounds like not using MiqExpression and a query string, but some filter constraints passed in via headers? Not sure though.
So my naive approach would be to exclude supports_ from a MiqExpression and filter after all objects have been retrieved

@durandom we could go down the options route if it pertains to things about providers in general
If you want to have a go at modifying the filterer, this would be a great place to start. I have some reservations though about filtering on things that aren't attributes - for instance, who would they be discoverable?

@imtayadeway are you saying, that we should stay with exposing selected supports_ features via virtual attributes?
I'm not really experienced with how the API and options etc work.
Maybe we should discuss this face to face and also question the use case of filtering on supports_ features in general.

I am concerned that we are trying to expose data as if it were in the database, and in reality, it is stored in a class hierarchy. Seems these could be removed from the classes and put into the database somewhere? maybe as jsonb?
Also, do note that typically, we add things as virtual attributes and people tend to think that this is supported by sql. So they filter or sort on these columns. Since they are in ruby, we have to bring back all rows. It causes all sorts of hidden performance problems (the 30 minutes vs 5 seconds kind)


Batch matrix multiply leads to vulkan error on WGPU

I wasn't sure if batch matmul was supported, as this seems to be documented nowhere except in pytorch's documentation. It seems to work fine with small tensors but breaks down past a certain size:
type B = burn::backend::wgpu::JitBackend<WgpuRuntime<AutoGraphicsApi, f32, i32>>;
let a: Tensor<B, 4> = Tensor::random([500, 500, 4, 5], Distribution::Normal(-1.0, 1.0), &Default::default());
let b: Tensor<B, 4> = Tensor::random([500, 500, 5, 6], Distribution::Normal(-1.0, 1.0), &Default::default());
let out = a.matmul(b);
println!("{:?}", out);

I'd expect an output tensor of shape [500, 500, 4, 6], but instead I get the following error:
wgpu error: Validation Error

Caused by:
    In a ComputePass
      note: encoder = `<CommandBuffer-(1, 1, Vulkan)>`
    In a dispatch command, indirect:false
      note: compute pipeline = `<ComputePipeline-(4, 1, Vulkan)>`
    Each current dispatch group size dimension ([1, 1, 250000]) must be less or equal to 65535

So it seems there's a maximal dimension of 65535 for bmm. I would expect that this backend-specific limitation be abstracted away, i.e the backend should likely batch the bmm and recombine them automatically. Is there a current workaround for this?
I'm using burn 0.13.2 with vulkan version 1.3.283 on fedora 40.

For the time being, this seems to work, but it's much slower than I'd like:
/// Perform batch matrix multiplication by splitting first dimension
/// if larger than 65535 and recombining results
pub fn bmm<B: Backend>(a: Tensor<B, 3>, b: Tensor<B, 3>) -> Tensor<B, 3> {
    let batch_size = 65535;
    let [n1, i, j1] = a.dims();
    let [n2, j2, k] = b.dims();
    assert_eq!(n1, n2);
    assert_eq!(j1, j2);

    if n1 <= batch_size {
        return a.matmul(b);
    }

    let ranges: Vec<_> = (0..(batch_size as u32).div_ceil(n1 as u32)).map(|i| 0..(batch_size*(i+1) as usize).max(n1)).collect();
    let result_parts: Vec<_> = ranges.into_iter().map(|r| {
        let a_part = a.clone().slice([r.clone(), 0..i, 0..j1]);
        let b_part = b.clone().slice([r.clone(), 0..j2, 0..k]);
        let result_part = a_part.matmul(b_part);
        println!("{:?}", result_part);
        result_part
    }).collect();
    Tensor::cat(result_parts, 0)
}

It could be speed up by into_par_iter from rayon, but to my surprise, FloatTensorPrimitive is Send but not Sync...

Well, it's a limitation in the launching paradigm used to compute matrix multiplication. Maybe we could use another dimension to handle the batch part.


Calendar does not hide completely when choose a month which has 6 rows

Describe the bug
in the right bottom calendar, it will not hide completely if we choose a month which has 6 rows.
To Reproduce
Steps to reproduce the behavior:

Select a month which has 6 rows, for example, October 2021
Close the calendar
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Desktop:

OS: Windows
Browser :Chrome
Version :92.0.4515.107

That's weird coz I cannot reproduce the error. Still I will lower the bottom property of calender even more, just in case.


CLI defined in runtime.md should mirror that provided by the reference implementation (runc).

runtime.json specifies the expected command-line interface a runtime must provide. However, that minimal CLI does not match the (much more featureful) CLI provided by the reference implementation: runc.
Since (as of v1.11) runc is now used by docker (via containerd), it seems unlikely docker would want to remove functionality simply to align with runtime.md.
Note further that runc's CLI does not align with runtime.md either. The spec says runc should be invoked like this to start a container:
$runtime start <container-id> <path-to-bundle>
... and yet runc actually uses:
runc start [ -b bundle ] <container-id>
The spec states that a "stop" verb is required, but runc has none.
Please clarify what the full, required, runtime CLI should be as the current situation is confusing.
(Currently, I fear we're at the point that the implementation (runc) trumps the spec as defining expected behaviour given that to run under containerd requires a runtime to implement runc's vision of how it should be done, not what the spec [currently] says :-( ).

On Fri, May 13, 2016 at 10:10:07AM -0700, James Hunt wrote:

runtime.json specifies the expected command-line interface a runtime
must provide. However, that minimal CLI does not match the (much
more featureful) CLI provided by the reference implementation:
runc.

Those are hints, and are not a CLI specification 1.  I had been
working on a command-line API with @mikebrow and @duglin [2,3], but
slowed down when 4 was declined in favor of #225's
built-in-state-registry requirement.  I'm happy to start back up with
API-spec maintenance if there's renewed interest.
 Subject: Specifying the runtime's command-line interface
 Date: Tue, 18 Aug 2015 10:52:55 -0700
 Message-ID<EMAIL_ADDRESS>

#513 tagged as 1.0.0, so I'm tagging this issue 1.0.0 as well.

The decision in #513 is that the CLI is a requirement for testing, not for compliance.  You can see more information on the linked PR.


it worked for 1 day but Suddenly stucked in UI

[2021-04-17T10:33:45.411Z] ERROR: binance-api/37 on 384644bb0fb9: Execution failed. (version=0.0.65, server=cronjob, job=trailingTrade, uuid=9a3424e5-3b5a-4128-b35b-c6f5347e3b98, symbol=DENTUSDT, err.code=-1021)
Error: Timestamp for this request is outside of the recvWindow.
at /srv/node_modules/binance-api-node/dist/http-client.js:93:17
at runMicrotasks ()
at processTicksAndRejections (internal/process/task_queues.js:93:5)
at async execute (/srv/app/jobs/trailingTrade/step/get-account-info.js:23:23)
at async execute (/srv/app/jobs/trailingTrade.js:98:14)
at async CronJob. (/srv/app/server-cronjob.js:29:11)

my problem has been fixed by:
sudo yum install ntp
on centos7

Hi @allaf2
Good to hear that you fixed the issue. Please close the issue.

in ubuntu's (on wsl2) case, this worked, https://github.com/microsoft/WSL/issues/4149#issuecomment-521877012


update to godot 3.1

Thank you for updating this project!
I don't have time to check if it's working correctly, so I'm going to trust you it's ok.
Thanks!


Add examples for new singleQuitButton key

@natewalck please do the other two files as well. I may delete them soon, but for now want them to remain in sync


更新代码

🤔 This is a ...

[ ] New feature
[ ] Bug fix
[ ] Site / document update
[ ] Component style update
[ ] TypeScript definition update
[x] Refactoring
[ ] Code style optimization
[ ] Branch merge
[ ] Other (about what?)

👻 What's the background?

💡 Solution

📝 Changelog

English Changelog:
Chinese Changelog (optional):

☑ Self Check before Merge

[ ] Doc is updated/provided or not needed
[ ] Demo is updated/provided or not needed
[ ] TypeScript definition is updated/provided or not needed
[ ] Changelog is provided or not needed

update code


Fix 'warning CS0108: ... hides inherited member ...' compilation warnings

There are some compilation warnings due to hiding of inherited members. Will need to determine for each one whether the child class's member should be deleted or hidden using the new keyword.

C:\xRM-Portals-Community-Edition\Framework\Adxstudio.Xrm\Activity\PortalComment.cs(32,39,32,54): warning CS0108: 'PortalComment.FileAttachments' hides inherited member 'Activity.FileAttachments'. Use the new keyword if hiding was intended.

C:\xRM-Portals-Community-Edition\Framework\Adxstudio.Xrm\Marketing\DataAdapterDependencies.cs(30,20,30,30): warning CS0108: 'DataAdapterDependencies.PortalName' hides inherited member 'DataAdapterDependencies.PortalName'. Use the new keyword if hiding was intended.

C:\xRM-Portals-Community-Edition\Framework\Adxstudio.Xrm\Web\Providers\ContentMapEntityUrlProvider.cs(442,34,442,53): warning CS0108: 'ContentMapEntityUrlProvider.JoinApplicationPath(string, string)' hides inherited member 'AdxEntityUrlProvider.JoinApplicationPath(string, string)'. Use the new keyword if hiding was intended.

Please provide some analysis on each one in this issue prior to making changes.

FileAttachments is declared exactly the same on the base class (same visibility etc.); however the property on the base class isn't used anywhere in the solution - so I would either delete property from the base class or (safer option?)  hide on child class using new.

Portal Name has a different declaration on base and child classes:
protected string PortalName { get; private set; }

protected string PortalName { get; set; }

This for me is enough to say use new to hide base class to protect the private set; on the child.

Whilst the implementations of JoinApplicationPath look similar with different layout, there is a subtle difference. Child class implementation checks for empty string:
if (string.IsNullOrWhiteSpace(basePath)
|| basePath.Contains("?")
|| basePath.Contains(":")
|| basePath.Contains("//")
|| basePath.Contains("&")
|| basePath.Contains("%3f")
|| basePath.Contains("%2f%2f")
|| basePath.Contains("%26"))

vs.
if (basePath.Contains("?")
    || basePath.Contains(":")
    || basePath.Contains("//")
    || basePath.Contains("&")
    || basePath.Contains("%3f")
    || basePath.Contains("%2f%2f")
    || basePath.Contains("%26"))

This difference to me suggests we go with using new to hide the base class (seems like less risk).

Thanks for the analysis, we'll review the code as well and will get back to you to confirm what changes are appropriate before implementing them.

Regarding PortalComment.FileAttachments, this will have to be hidden with the new keyword to:

Satisfy the IPortalComment interface (a compilation error occurs if deleting PortalComment.FileAttachments)
Avoid breaking changes to external code that may be already be using Activity.FileAttachments and expects it to be an ICollection.

Regarding Adxstudio.Xrm.Marketing.DataAdapterDependencies.PortalName, I don't see any harm in simply deleting this property, given:

It's already being assigned a value in the constructor
It's protected, so access through inheritance is the only way of accessing this property
Another child class (Adxstudio.Xrm.Forums.DataAdapterDependencies) also inherits from Adxstudio.Xrm.Cms.DataAdapterDependencies but doesn't hide the property from the base class.

Please go ahead with this change.

Regarding Adxstudio.Xrm.Web.Providers.ContentMapEntityUrlProvider.JoinApplicationPath, agreed on hiding with the new keyword, with the additional change of also removing static since the method is private and only called within other non-static methods of that class.
Please go ahead with this change.

Happy for all these through one PR or want them on individually?

One PR is good, thanks!


Replace deprecated class#newInstance

Codecov Report

Merging #796 (9a2d203) into main (8edbed3) will decrease coverage by 0.06%.
The diff coverage is 80.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##               main     #796      +/-   ##
============================================
- Coverage     92.52%   92.47%   -0.06%
+ Complexity     2653     2652       -1
============================================
  Files           286      286
  Lines          5274     5275       +1
  Branches        553      553
============================================
- Hits           4880     4878       -2
- Misses          252      254       +2
- Partials        142      143       +1

Impacted Files
Coverage Δ

...tafaker/transformations/JavaObjectTransformer.java
39.39% <80.00%> (+1.89%)
:arrow_up:

... and 3 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Core

Song Name
Core
Artist Name
Zef
Source
Direct file download
Youtube Link
No response
Direct File Link
https://www.mediafire.com/file/tjc3yyl5hb467k0/Zef+-+Core.mp3/file
Song ID
4
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 56028719
Downloaded: No

Yes, I accept.
On Fri, Oct 25, 2024, 7:55 AM auto-nong[bot] @.***>
wrote:

Thanks for contributing!
Someone will soon review your request.
Here is the JSON result for the reviewer:
{
"name": "Core",
"artist": "Zef",
"source": "host",
"startOffset": 0,
"url": "https://www.mediafire.com/file/tjc3yyl5hb467k0/Zef+-+Core.mp3/file",
"songs": [
4
]
}
To accept the request, a reviewer needs to write "accept".
—
Reply to this email directly, view it on GitHub
https://github.com/FlafyDev/auto-nong-indexes/issues/5391#issuecomment-2436534678,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AR6ZRAJ4YGB56L2P7QQWRZLZ5GCGVAVCNFSM6AAAAABQSERQPOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZWGUZTINRXHA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Official release?

I've noticed a number of buffer overflow fixes/ UB fixes going in in the past few months.  Is there any chance of an official release containing these going out?

Yes, almost ready for next release. Only thing missing (but in the works now) is the JavaScript Lerc decoder update. If you don't need that, you can use current master.
Don't worry about the bug fixes you saw. Most if not all of them were the result of a very special testing: Bombarding the Lerc decoder with corrupted binary blobs, and at the same time have the checksum test disabled in the decoder.  You can ask Even  @rouault for more details if interested.

@tmaurer3 I am already running a build off the main branch due to the android bug from https://github.com/Esri/lerc/commit/a7b7462a124b854f04e6fa4e08ebb239d6ab77d4   I just would really prefer to be on a numbered release for internal maintainability/documentation purposes.  Was hoping enough things had occured to warrant one.

https://github.com/Esri/lerc/releases/tag/v2.1


include_field_ids flag in schema_to_pyarrow

Add flag to include/exclude Parquet field IDs to pyarrow schema
Fixes: https://github.com/apache/iceberg-python/issues/788

Thanks for the review @HonahX ! Could I ask for your help in getting this PR merged?


Add multiple filter select

Adds multi filter select abilities and and overlay/popup menu in mobile

not sure why the linter is failing here. it doesn't fail locally and it seems to be mad about code that's been removed... that or it's complaining about the filter array method?


docs: Interactive Vanilla JS example

At this moment Vanilla JS documentation lacks complete example that user can simply "copy-paste" or fork and run locally within seconds. This is especially important considering that Lexical can't be bootstrapped with just a few lines of code, while playground example we have for React is quite overwhelming (especially for less experienced engineers).
Additionally this fixes #5671
Here the proposal is to utilize StackBlitz that allow to run dev environment locally in user browser. For example react-router also uses it.
If this diff goes through - I'll do same minimalistic example for React.
Additionally I didn't create any tests for it intentionally for 2 reasons:

I don't want to spend extra time on tests before getting endorsement from core team on the approach
Let's discuss if we even need them as since dependencies versions here are locked - there is not much things that can go wrong. And tests wouldn't cover StackBlitz integration (most probable cause of issues). And if we want to have them - let's discuss if they can live alongside Lexical E2E tests.

P.S.: At this moment StackBlitz pointed to forked repo intentionally to allow functionality preview, this is to be fixed before merge

Looks good! These tests seem to be failing locally on main, so I'll fix that separately.

@acywatson I'm on it ;) It's unrelated to this PR though


Dangling file in /var/lib/cni/networks/podman after hard restart

Is this a BUG REPORT or FEATURE REQUEST? (leave only one on its own line)
/kind bug
Description
My server restarts hard occasionally (as in I have a brownout because my fuse triggers). When this happens I have the same dangling files reported here and here. Podman has no possibility to clean up.
Steps to reproduce the issue:

Hard restart server.

Describe the results you received:
Dangling files still in /var/lib/cni/networks/podman.
Describe the results you expected:
Cleaned up filesystem, no dangling files in /var/lib/cni/networks/podman.
Additional information you deem important (e.g. issue happens only occasionally):
Output of podman version:
# podman version
Version:            1.6.4
RemoteAPI Version:  1
Go Version:         go1.12.12
OS/Arch:            linux/amd64

Output of podman info --debug:
# podman info --debug
debug:
  compiler: gc
  git commit: ""
  go version: go1.12.12
  podman version: 1.6.4
host:
  BuildahVersion: 1.12.0-dev
  CgroupVersion: v1
  Conmon:
    package: conmon-2.0.6-1.module_el8.1.0+272+3e64ee36.x86_64
    path: /usr/bin/conmon
    version: 'conmon version 2.0.6, commit: 7a4f0dd7b20a3d4bf9ef3e5cbfac05606b08eac0'
  Distribution:
    distribution: '"centos"'
    version: "8"
  MemFree:<PHONE_NUMBER>8
  MemTotal:<PHONE_NUMBER>6
  OCIRuntime:
    name: runc
    package: runc-1.0.0-64.rc9.module_el8.1.0+272+3e64ee36.x86_64
    path: /usr/bin/runc
    version: 'runc version spec: 1.0.1-dev'
  SwapFree:<PHONE_NUMBER>0
  SwapTotal:<PHONE_NUMBER>0
  arch: amd64
  cpus: 4
  eventlogger: journald
  hostname: carter
  kernel: 4.18.0-147.5.1.el8_1.x86_64
  os: linux
  rootless: false
  uptime: 1h 35m 6.12s (Approximately 0.04 days)
registries:
  blocked: null
  insecure: null
  search:
  - registry.access.redhat.com
  - registry.fedoraproject.org
  - registry.centos.org
  - docker.io
store:
  ConfigFile: /etc/containers/storage.conf
  ContainerStore:
    number: 2
  GraphDriverName: overlay
  GraphOptions: {}
  GraphRoot: /var/lib/containers/storage
  GraphStatus:
    Backing Filesystem: xfs
    Native Overlay Diff: "true"
    Supports d_type: "true"
    Using metacopy: "false"
  ImageStore:
    number: 3
  RunRoot: /var/run/containers/storage
  VolumePath: /var/lib/containers/storage/volumes

Package info (e.g. output of rpm -q podman or apt list podman):
# rpm -q podman
podman-1.6.4-2.module_el8.1.0+272+3e64ee36.x86_64

Additional environment details (AWS, VirtualBox, physical, etc.):
physical server

I'll take this one.

@Schrottfresse Any chance you can test #5434 and see if this resolves it?

@mheon I will try to do it the next days.

I have the same issue on the centos 8, podman version 1.9.1.
Do I need to apply any setting to activate the #5434  ?
Thank you
Output of podman version :
# podman version
Version:            1.9.1
RemoteAPI Version:  1
Go Version:         go1.12.12
OS/Arch:            linux/amd64

Output of podman info --debug:
# podman info --debug
debug:
  compiler: gc
  gitCommit: ""
  goVersion: go1.12.12
  podmanVersion: 1.9.1
host:
  arch: amd64
  buildahVersion: 1.14.8
  cgroupVersion: v1
  conmon:
    package: conmon-2.0.6-1.module_el8.1.0+298+41f9343a.x86_64
    path: /usr/bin/conmon
    version: 'conmon version 2.0.6, commit: 2721f230f94894671f141762bd0d1af2fb263239'
  cpus: 8
  distribution:
    distribution: '"centos"'
    version: "8"
  eventLogger: file
  hostname: dmzAppServer
  idMappings:
    gidmap: null
    uidmap: null
  kernel: 4.18.0-147.8.1.el8_1.x86_64
  memFree:<PHONE_NUMBER>
  memTotal:<PHONE_NUMBER>
  ociRuntime:
    name: runc
    package: runc-1.0.0-64.rc9.module_el8.1.0+298+41f9343a.x86_64
    path: /usr/bin/runc
    version: 'runc version spec: 1.0.1-dev'
  os: linux
  rootless: false
  slirp4netns:
    executable: ""
    package: ""
    version: ""
  swapFree:<PHONE_NUMBER>
  swapTotal:<PHONE_NUMBER>
  uptime: 39m 44.73s
registries:
  localhost:5000:
    Blocked: false
    Insecure: true
    Location: localhost:5000
    MirrorByDigestOnly: false
    Mirrors: []
    Prefix: localhost:5000
  search:
  - registry.access.redhat.com
  - registry.fedoraproject.org
  - registry.centos.org
  - docker.io
  - localhost:5000
store:
  configFile: /etc/containers/storage.conf
  containerStore:
    number: 5
    paused: 0
    running: 5
    stopped: 0
  graphDriverName: overlay
  graphOptions: {}
  graphRoot: /var/lib/containers/storage
  graphStatus:
    Backing Filesystem: xfs
    Native Overlay Diff: "true"
    Supports d_type: "true"
    Using metacopy: "false"
  imageStore:
    number: 10
  runRoot: /var/run/containers/storage
  volumePath: /var/lib/containers/storage/volumes

For more detail, there are 4 running containers with dedicated IP settings (<IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>) in my centos 8.
After reboots (I use reboot command), there are usually 1 or 2 IP exists at /var/lib/cni/networks/podman

I have the same issue on the centos 8, podman version 1.9.1.
Do I need to apply any setting to activate the #5434 ?
Thank you
Output of podman version :
# podman version
Version:            1.9.1
RemoteAPI Version:  1
Go Version:         go1.12.12
OS/Arch:            linux/amd64

Output of podman info --debug:
# podman info --debug
debug:
  compiler: gc
  gitCommit: ""
  goVersion: go1.12.12
  podmanVersion: 1.9.1
host:
  arch: amd64
  buildahVersion: 1.14.8
  cgroupVersion: v1
  conmon:
    package: conmon-2.0.6-1.module_el8.1.0+298+41f9343a.x86_64
    path: /usr/bin/conmon
    version: 'conmon version 2.0.6, commit: 2721f230f94894671f141762bd0d1af2fb263239'
  cpus: 8
  distribution:
    distribution: '"centos"'
    version: "8"
  eventLogger: file
  hostname: dmzAppServer
  idMappings:
    gidmap: null
    uidmap: null
  kernel: 4.18.0-147.8.1.el8_1.x86_64
  memFree:<PHONE_NUMBER>
  memTotal:<PHONE_NUMBER>
  ociRuntime:
    name: runc
    package: runc-1.0.0-64.rc9.module_el8.1.0+298+41f9343a.x86_64
    path: /usr/bin/runc
    version: 'runc version spec: 1.0.1-dev'
  os: linux
  rootless: false
  slirp4netns:
    executable: ""
    package: ""
    version: ""
  swapFree:<PHONE_NUMBER>
  swapTotal:<PHONE_NUMBER>
  uptime: 39m 44.73s
registries:
  localhost:5000:
    Blocked: false
    Insecure: true
    Location: localhost:5000
    MirrorByDigestOnly: false
    Mirrors: []
    Prefix: localhost:5000
  search:
  - registry.access.redhat.com
  - registry.fedoraproject.org
  - registry.centos.org
  - docker.io
  - localhost:5000
store:
  configFile: /etc/containers/storage.conf
  containerStore:
    number: 5
    paused: 0
    running: 5
    stopped: 0
  graphDriverName: overlay
  graphOptions: {}
  graphRoot: /var/lib/containers/storage
  graphStatus:
    Backing Filesystem: xfs
    Native Overlay Diff: "true"
    Supports d_type: "true"
    Using metacopy: "false"
  imageStore:
    number: 10
  runRoot: /var/run/containers/storage
  volumePath: /var/lib/containers/storage/volumes

Package info (e.g. output of rpm -q podman or apt list podman):
# rpm -q podman
podman-1.9.1-1.1.el8.x86_64

Additional environment details (AWS, VirtualBox, physical, etc.):
Vsphere VM

Can you provide the names of the files? Do they prevent you from starting the containers which use those IPs?


DOCK/UNDOCK example request

Hello,
I'm currently working on some own packages to work with the irobot create 3 robot.
I'm running into problems when performing a dock/undock action. If i'm executing a dock/undock activity, I cannot check the status of the action. More specific, it seems that the action never finishes succesfully.
Even with the CLI commands to dock and undock, I never reveive a "Goal finished with status: SUCCEEDED" message.
Is it possible to provide a python example how to process the status of the docking, e.g. to wait to move to the next scheduled goals?
Thanks in advance

Hi @BertVanAcker, the Create 3 dock and undock actions work as standard ROS 2 actions, they don't have anything special.
You can find C++ examples for dock and undock.
Later today I plan to look in details at your code snippet and provide suggestions in the discussion https://github.com/iRobotEducation/create3_docs/discussions/131
I will keep this ticket open as a request to provide more general Python examples.


Pause function

Could be possible to implement a pause button? example: using the "select" button during tape play, pressing it will pause tape and pressing it again will unpause tape.
this would be useful with tapes that have more programs/games in it

How common is this? A higher priority would be proper seek control for multipart games, which would probably provide you with what you needed for this feature.

I've added experimental support for this in the pause-func branch. Test and I'll think about merging it into master as a temporary fix until proper play control is added

This is now in master


Collection was modified; enumeration operation may not execute.

Duplicate

upon screen refresh - accompanied by many
Error raising received TypeError: Cannot read property 'scrollTop' of undefined
jquery.signalR.js:18

of https://github.com/davidfowl/JabbR/issues/311


Remove unnecessary string conversions

To get a feel for the problem, I basically searched for all magic number occurences equal to 4096 (there are quite a lot) and replaced them all by 1048576. Might be not all of them needed enlarging but then I have not grown with the code so hard for me to go analyse that.
I can then increase packet sizes without the client or the server crashing.
The fundamental problem remains though that speed still rapidly decreases with packet size. I suspect there is room for optimization by considering whether all those allocs really need to be within the function calls themselves. I.e. you could probably allocate a big buffer at struct instantiation time and then point to that one over and over again rather than alloc and free ?
For what it's worth, output of my test client:
=======================
packet size 1000 Bytes:
=========================
Sent and received 1000 packets in 0.4144 seconds
Packet rate 2.41 kilo packets/s
Bit rate 19.3  Mbps
=======================
packet size 2000 Bytes:
=========================
Sent and received 1000 packets in 0.4268 seconds
Packet rate 2.34 kilo packets/s
Bit rate 37.5  Mbps
=======================
packet size 4000 Bytes:
=========================
Sent and received 1000 packets in 0.4800 seconds
Packet rate 2.08 kilo packets/s
Bit rate 66.7  Mbps
=======================
packet size 8000 Bytes:
=========================
Sent and received 1000 packets in 0.5553 seconds
Packet rate 1.80 kilo packets/s
Bit rate 115.2  Mbps
=======================
packet size 16000 Bytes:
=========================
Sent and received 1000 packets in 0.7618 seconds
Packet rate 1.31 kilo packets/s
Bit rate 168.0  Mbps
=======================
packet size 32000 Bytes:
=========================
Sent and received 1000 packets in 1.1680 seconds
Packet rate 0.86 kilo packets/s
Bit rate 219.2  Mbps
=======================
packet size 64000 Bytes:
=========================
Sent and received 1000 packets in 1.7495 seconds
Packet rate 0.57 kilo packets/s
Bit rate 292.7  Mbps
=======================
packet size 128000 Bytes:
=========================
Sent and received 1000 packets in 1.7782 seconds
Packet rate 0.56 kilo packets/s
Bit rate 575.9  Mbps
=======================
packet size 256000 Bytes:
=========================
Traceback (most recent call last):
  File "/Users/alainrolle/miniconda3/envs/py312/lib/python3.12/site-packages/urllib3/response.py", line 737, in _error_catcher
    yield

@alainrollejr ah, good to know, thanks. I'm refactoring more today, expecting some major improvements both in reliability in performance after it's done. Will keep you posted

@alainrollejr made some really major refactoring this weekend on this branch. Below is a selection of a couple test runs I made. I don't see a steep increase anymore with the size of the packet, although as you can see overall the rate is slower now. This is with connection: close though, which is not how it should work by default. Setting tcp_keep_alive should improve the performance up to 2x-2.5x based on experience.
=======================
packet size 1280 Bytes:
=========================
Sent and received 1000 packets in 1.4183 seconds
Packet rate 0.71 kilo packets/s
Bit rate 7.2  Mbps
=======================

=======================
packet size 8500 Bytes:
=========================
Sent and received 1000 packets in 2.3081 seconds
Packet rate 0.43 kilo packets/s
Bit rate 29.5  Mbps

=======================
packet size 10000 Bytes:
=========================
Sent and received 1000 packets in 2.2904 seconds
Packet rate 0.44 kilo packets/s
Bit rate 34.9  Mbps

packet size 12800 Bytes:
=========================
Sent and received 1000 packets in 2.5152 seconds
Packet rate 0.40 kilo packets/s
Bit rate 40.7  Mbps
=======================

=======================
packet size 100000 Bytes:
=========================
Sent and received 1000 packets in 2.0973 seconds
Packet rate 0.48 kilo packets/s
Bit rate 381.4  Mbps

=======================
packet size 128000 Bytes:
=========================
Sent and received 1000 packets in 2.1425 seconds
Packet rate 0.47 kilo packets/s
Bit rate 477.9  Mbps

packet size 128000 Bytes:
=========================
Sent and received 1000 packets in 1.9405 seconds
Packet rate 0.52 kilo packets/s
Bit rate 527.7  Mbps

@saviorand It's cool to see you're using some of the other structs from Gojo! I'm just going through the code to see if I spot anything else that might help speed up the code.

new_buffer() wants ownership of buffer b. Since that's declared as an owned arg, and ownership is not being transferred with the ^, Mojo is creating a copy of b. b is no longer used in the function after creating the buffer, so I think you can safely transfer ownership.
https://github.com/saviorand/lightbug_http/blob/feature/no-string-conversions/lightbug_http/sys/server.mojo#L172
var buf = buffer.new_buffer(b) -> var buf = buffer.new_buffer(b^)


Error getting the Blob SAS access key?

This tutorial does not work - I have associated a storage account, and subsequently a blob continer, to my IoT Hub.  I have passed a viable connection string into the code, and yet it is telling me that it is unable to generate a Blob SAS access token?

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

ID: 9de4513d-0fc7-233a-bf03-b28e7270d6df
Version Independent ID: 05faa9c8-478b-2c2c-7728-6ad29b5fcc28
Content: Use the Azure portal to configure file upload
Content Source: articles/iot-hub/iot-hub-configure-file-upload.md
Service: iot-hub
GitHub Login: @dominicbetts
Microsoft Alias: dobett

@manchejs0 Thanks for the feedback. We are actively investigating but wanted to clarify the actual tutorial you are following. Are you trying to follow the Python File Upload tutorial: Upload files from your device to the cloud with IoT Hub? Or are you simply following the instructions to configure file upload functionality and running into SAS token issues? It would be desirable to see your code, if you are not following a tutorial. Thanks, ~Mike

It looks like you are using the Node.js version the tutorial? I am going to close this issue, and work with GitHub Issue #13957 going forward, as it references the document we want to work with. Thanks, Mike


Set union added

@blowdee  Please create a new branch for every pull request you make.  Create a new branch from that master and then add your changes.

@Gotham13121997 Messed up a bit. Now it's merged.

Yeah. By creating new branches you can work on several issues side by side. Now you cant work on another issue unless this one is merged . So take care of it from the next time.

Tested and merged to master

Thanks @blowdee . Please create a new branch if you want to contribute to new issues


Update tezos/sys build to download headers

Still a draft, right now requires the environment vaiable OCAML_WHERE_PATH=$(pwd)/tezos/sys/lib_tezos/artifacts/include to be set when running cargo commands so that the OCaml headers can be found, waiting for cargo 1.52 to be release which includes a way to set environment variables that get passed down to the build process:
https://github.com/rust-lang/cargo/pull/9175
@bkontur btw, I am thinking that it may be better to remove the libtezos download/copy logic from the build script and handle it on a separate standalone script, doing so will simplify things.

Still a draft, right now requires the environment vaiable OCAML_WHERE_PATH=$(pwd)/tezos/sys/lib_tezos/artifacts/include to be set when running cargo commands so that the OCaml headers can be found, waiting for cargo 1.52 to be release which includes a way to set environment variables that get passed down to the build process:
rust-lang/cargo#9175
@bkontur btw, I am thinking that it may be better to remove the libtezos download/copy logic from the build script and handle it on a separate standalone script, doing so will simplify things.

@tizoc
could you, please, write more about "separate standalone scipt" solution?
Do you mean it like manual step before running cargo?
We need to think about to have possibility to build/run it, just with one command cargo run / cargo test because of CI, if anybody wants to add it to theirs CI and so on

@bkontur I am still thinking about it, but the basic idea is to move the current build script to it's own cargo subcommand (cargo prepare-tezos or something like that, I have to investigate how that works -- I just want to avoid having to rewrite it in bash, I like how the Rust one works).
So it would be the same code, but will have to be called manually once before the actual build/test/run commands. Then the build script will just verify that the hashes match and fail if they don't.
There are two reasons for this:

It all happening as part of the build is kind of a PITA because sometimes it inadvertently overrides things (not an issue most of the time, but kind of annoying when you are working on changes to libtezos)
More importantly, this script now needs to download some files (the headers) that are needed by it's dependencies, so by the time the build script runs, it is already too late (sometimes it doesn't even run because the dependency fails to build because the headers are still not there).

Replaced by #551 for now, closing.


Add proxy and strict ssl options

Pulls in proxy and strict-ssl settings from npm config file.
Replaces simple-get with nugget since the former does not support setting strict-ssl.

Hi there ! Thanks for you work :) LGTM


Element not found when matching :text with Regexp and text content includes leading/trailing whitespace

Meta -
Watir Version: 7.0.0.beta1
Selenium Version: 4.0.0.beta2
Browser Version: Chrome 89.0.4389.90
Browser Driver Version: Chromedriver 89.0.4389.90
OS Version: Windows 10
Expected Behavior -
browser.link(text: /^link text?/) should match the element:

			link text

This works successfully in 6.19.1.
Actual Behavior -
It does not match.
Steps to reproduce -
Given the page:

			link text

The element is not matched:
browser.link(text: /^link text?/).exists?
2021-03-24 01:46:11 INFO Watir Converted {:text=>/^link text?/, :tag_name=>"a"} to {:text=>/^link text?/, :xpath=>".//*[local-name()='a'][contains(normalize-space(), 'link tex')]"}
2021-03-24 01:46:11 DEBUG Watir Iterating through 1 elements to locate {:text=>/^link text?/, :tag_name=>"a"}
2021-03-24 01:46:11 INFO Watir Executing Script on Browser: getTextContent
=> false

Other similar locators are able to find the element:
browser.link(text: 'link text').exists?     #=> true
browser.link(text: /link text/).exists?     #=> true
browser.link(text: /^link text/).exists?    #=> true
browser.link(text: /link text?/).exists?    #=> true
browser.link(text: /^link text?/).exists?   #=> false

I believe the problem is the handling of leading/trailing spaces. When :text is fully converted to :xpath, the text comparison uses normalise-space() which strips the leading/trailing spaces. In contrast, when :text cannot be converted to :xpath, the Regexp is compared to text content that includes the leading/trailing spaces.

This was the behavior we decided on: http://watir.com/watir-6-10/#locating-elements-by-text
Using :text locator with a Regexp now locates based on everything in the DOM, not just visible text.
It's "bad" that we didn't figure out how to throw a deprecation notice for this, but the answer is to switch to using :visible_text

@titusfortner , not sure I follow your response. I agree that :text should match all the text in the DOM. In this case, all of the text is visible, so :text and :visible_text should really produce the same thing.
The issue here is that :text using Regexp returns different things based on whether the Regexp could be completely converted to :xpath or not:
browser.link(text: /^link text/).exists?    #=> true (was completely converted to :xpath)
browser.link(text: /^link text?/).exists?   #=> false (could not be completed converted to :xpath)

Why should the act of adding the ? (ie the last "t" being optional) in the second example cause the element to no longer be found?

The content of the DOM is whatever is returned from execute_js(:getTextContent, element)
In this case it includes additional whitespace characters (\n link text\n ).
I'm ok stripping white space from "visible text" but I don't think we should do so from "text".

Also, I may not have had the exact same reproduction case as you, even though I copy/pasted what you had into a file.

merged fix


Fixed plugin installation commands.

This should put the correct value for the cordova plugin install command in all org.apache.cordova plugins.

You'll need to rebase and cherrypick commit 4b33a4f; the other commit is still in this PR.

Sorry about that. I didn't notice it.
I'll open the PR at night
Thank you for the feedback!
Enviado desde mi Windows Phone

From: George<EMAIL_ADDRESS>Sent: 8/3/2015 4:01 PM
To<EMAIL_ADDRESS>Cc: Luis Rogelio Hernández<EMAIL_ADDRESS>Subject: Re: [ng-cordova] Fixed plugin installation commands. (#905)
You'll need to rebase and cherrypick commit 4b33a4f; the other commit is still in this PR.

Reply to this email directly or view it on GitHub:
https://github.com/driftyco/ng-cordova/pull/905#issuecomment-127405336


eslint-loader should not fail build

I don't think there's any way around this other than removing the eslint-loader. failOnWarning and failOnError are already defaulted to false.
https://github.com/webpack-contrib/eslint-loader#failonwarning-default-false
https://github.com/webpack-contrib/eslint-loader#failonerror-default-false
The quiet option only applies to warnings, not errors.
https://github.com/webpack-contrib/eslint-loader#quiet-default-false
So turns out, this issue is more of a nuisance than a problem.

I wonder if the eslintrc file is not taking those eslint-loader options into consideration and are just overriding them.

https://github.com/webpack-contrib/eslint-loader/issues/257

module.exports = {
  loader: 'eslint-loader',
  exclude: /node_modules/,
  enforce: 'pre',
  test: /\.js?$/,
  options: {
    emitWarning: true, // <== tell webpack to emit warning
    baseConfig: {
      extends: 'eslint-config-availity'
    }
  }
};


Merge pull request #1 from arunbhardwaj/main

Merging with latest changes, including UI bug fix

Apologies, please ignore this


[DOCS] Update CI and coverage badges in README

Update CI/CD badge to correctly reflect the "Test and Publish" workflow. Ensure consistency between GitHub Actions workflow name and badge display.

Update CI/CD badge to use "Test and Publish" workflow name
Fix badge URLs to match current repository structure
Ensure proper linking to GitHub Actions workflows

Resolves: Incorrect CI/CD badge display in README
⚡ Have you read the Contributing Guidelines?
Fixes #
Description
Clearly and concisely describe the changes in this pull request.
Type of Change

[ ] Bug fix (non-breaking change that fixes an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that would break existing functionality)
[x] Documentation update
[ ] Performance improvement
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes)
[ ] CI/CD related changes
[ ] Other (please describe):

Checklist

[ ] I have followed the project's coding style guidelines
[ ] I have added tests that prove my fix/feature works
[ ] All existing tests pass locally
[ ] I have updated the documentation accordingly
[ ] I have added appropriate type hints
[ ] I have updated the CHANGELOG.md
[ ] My changes generate no new warnings or errors
[ ] I have checked my code with black, isort, and mypy

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:white_check_mark: All tests successful. No failed tests found.
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.


Bitsandbytes issue

I'm using a slightly modified notebook (like https://colab.research.google.com/drive/1mvwsIQWDs2EdZxZQF9pRGnnOvE86MVvR?usp=sharing) to finetune a qwen2 model, specifically, my installation instructions are:
#%%capture

!mamba install --force-reinstall aiohttp -y
!pip install -U "xformers<0.0.26" --index-url https://download.pytorch.org/whl/cu121
!pip install --upgrade "unsloth[kaggle-new] @ git+https://github.com/unslothai/unsloth.git"

# Temporary fix for https://github.com/huggingface/datasets/issues/6753
!pip3 install datasets==2.16.0 fsspec==2023.10.0 gcsfs==2023.10.0

!pip3 install -U wandb

When it comes to resuming my training, the trainer_stats = trainer.train(resume_from_checkpoint=True) cells runs into the following error:
==((====))==  Unsloth - 2x faster free finetuning | Num GPUs = 1
   \\   /|    Num examples = 4,233,923 | Num Epochs = 3
O^O/ \_/ \    Batch size per device = 1 | Gradient Accumulation steps = 2
\        /    Total batch size = 2 | Total steps = 6,350,883
 "-____-"     Number of trainable parameters = 40,370,176

---------------------------------------------------------------------------
RuntimeError                              Traceback (most recent call last)
Cell In[24], line 1
----> 1 trainer_stats = trainer.train(resume_from_checkpoint=True)

File <string>:140, in train(self, resume_from_checkpoint, trial, ignore_keys_for_eval, **kwargs)

File <string>:404, in _fast_inner_training_loop(self, batch_size, args, resume_from_checkpoint, trial, ignore_keys_for_eval)

File /opt/conda/lib/python3.10/site-packages/accelerate/accelerator.py:2387, in Accelerator.clip_grad_norm_(self, parameters, max_norm, norm_type)
   2385             if parameters == [p for p in model.parameters()]:
   2386                 return model.clip_grad_norm_(max_norm, norm_type)
-> 2387 self.unscale_gradients()
   2388 return torch.nn.utils.clip_grad_norm_(parameters, max_norm, norm_type=norm_type)

File /opt/conda/lib/python3.10/site-packages/accelerate/accelerator.py:2331, in Accelerator.unscale_gradients(self, optimizer)
   2329 while isinstance(opt, AcceleratedOptimizer):
   2330     opt = opt.optimizer
-> 2331 self.scaler.unscale_(opt)

File /opt/conda/lib/python3.10/site-packages/torch/cuda/amp/grad_scaler.py:325, in unscale_(self, optimizer)
      0 <Error retrieving source code with stack_data see ipython/ipython#13598>

RuntimeError: unscale_() has already been called on this optimizer since the last update().

I think that something's gone wrong compatibility-wise. I've tried using different versions of pytorch, accelerate, transformers and trl, but the issue persists.
Please advise

@StrangeTcy Did you set fp16 = True or bf16 = True in the trainer args?
PS if these are Kaggle install instructions - there are updated ones here: https://www.kaggle.com/danielhanchen/kaggle-llama-3-2-1b-3b-unsloth-notebook

@danielhanchen
1.
trainer = SFTTrainer(
    model = model,
    tokenizer = tokenizer,
    train_dataset = dataset,

        fp16 = not is_bfloat16_supported(),
        bf16 = is_bfloat16_supported(),

    ),
)

-- yes, I did

I'll try that & get back to you, thanks

@StrangeTcy Ok that looks like a bitsandbytes issue - will investigate

@StrangeTcy If you initially trained with bitsandbytes < 0.44 and then tried to resume training with 0.44+ this can happen. I would recommend trying again with bitsandbytes==0.43.3.
@danielhanchen I also saw this question come up on Discord. Will try to have that fixed in the next bitsandbytes patch release.

@matthewdouglas interesting; I'll try that & report back, thanks

Thanks @matthewdouglas ! :) Sorry on the issue @StrangeTcy


Eth key rotation - Cannot remove old ethereum key from Multisig (remove_signer fails to return successful status)

Test test_ethereum_key_rotation - failure can be see by running this test
After ethereum key rotation occurs and confirms via an event the test adds the new eth key to the MS contract and then removes the old key from the MS contract.
In both add and remove signer trx, the submitter/account is the ganache main eth address and private key.
Adding the key to the contract works ok and status is 1 and is_valid_signer function returns True. Event confirms.
Remove signer step fails to confirm ok on the contract.
-- GetERC20MultiSigSignerRemovedBundlesRequest used to return signature bundle and nonce
-- trx submitted to add enough eth to perform removal trx with gas
        transaction = self._multisig.functions.remove_signer(
            old_signer,
            int(nonce),
            bytes.fromhex(signatures)).buildTransaction({
            'gas': 1000000,
            'gasPrice': 0,
            'from': account.address,
            'nonce': w3.eth.getTransactionCount(account.address)
        })
        self._send_raw_transaction(account, transaction)

Trx fails
old_signer is original validator ethereum address
nonce and signatures is taken from API GetERC20MultiSigSignerRemovedBundlesRequest
Tried removing fist before adding signer but witnessed same failed status.

Hey @vega-paul thanks for the ticket. I think we need somebody from smart contracts who can help with this. (Since Capsule has nothing to really do with smart contracts). @emilbayes could probably help


Check scrolling fullscreen mode

It seems that in fullscreen mode, scrolling does not (always? in mobile contexts?) work if the content exceeds the available viewport height.
Try to reproduce and fix.

Could not reproduce this. Seems to work fine on both, different versions of Android/Chrome and iOS/Safari.


Fix the rpm summary in copr template

Signed-off-by: Lokesh Mandvekar<EMAIL_ADDRESS>@flouthoc @baude @mheon @Luap99 PTAL. Trivial fix to make sure rpm doesn't show weird info.

LGTM

/lgtm

/lgtm

/lgtm
/approve


feat(ui-feedback): fix ui

What does this PR do?

fix UI glitches

Description of Task to be completed?

hide side-nav button on larger screen-size

change language in toast error message

How should this be manually tested?

view the hosted app on a monitor over 992px width

Any background context you want to provide?

feedback from scoreCard

What are the relevant pivotal tracker stories?
-https://www.pivotaltracker.com/story/show/153235156
Screenshots (if appropriate)

none

Questions:

none

Coverage increased (+1.009%) to 91.928% when pulling 05f94a5c39ca3d12b83c4487c20b908c11ef86e5 on feat/fix-ui/153235156 into 71a698e179e4af8118f3fa1c4023ea6c1f87b44d on defense-feedback.


modified slides and new figures

All images were successfully added to the slides. Slide text was properly modified. Merging now.


[WIP] Add conformance tests for ReferencePolicy

What type of PR is this?
/kind feature
What this PR does / why we need it:
Adds conformance tests for ReferencePolicy. Given that ReferencePolicy is not among the resources graduating to v1beta1, we may need to add some ability fro implementations to opt-in to conformance tests for specific experimental or alpha features, similar to the functionality proposed in #482
The HTTPRouteMissingReferencePolicy test could likely be consolidated into the primary valid HTTPRouteReferencePolicy test if we wanted to validate a mutation like deleting a ReferencePolicy causing an HTTPRoute to detach - I just wanted to confirm if that was the expected behavior, and if we want to mutate Kubernetes objects like that inside these tests, before implementing.
Which issue(s) this PR fixes:

Refs #20
Does this PR introduce a user-facing change?:

Add conformance tests for ReferencePolicy

/ok-to-test

Pushed up some relatively significant logical changes in response to reviews and discussion, also extracted https://github.com/kubernetes-sigs/gateway-api/pull/1114 for adding RouteConditionReason if that would be easier to review separately. This should be ready for another review pass now, and hopefully reaching some consensus around minimum expectations for Routes with invalid BackendRefs.

Thanks @mikemorris!
/lgtm


Unable to prisma init on existing mysql db with no data

Bug Report
I am currently unable to successfully run prisma init on a clean/new mysql database.
Current behavior
Prisma init fails to connect to the database:

Reproduction
Setup a blank MySQL database
Run prisma init
Expected behavior?
prisma init completes successfully and generates config
Investigation
I could be wrong but when I had a dig around in EndpointDialog.ts it looks like it's creating an introspector regardless of whether your database has existing data or not:

I am also experiencing this same issue. Any progress on this issue?

It seems only Postgres is supported ..
This screenshot is of the files in prisma-db-introspection.

@davesimtech, @prolink007: Thanks for reporting the issue. In your case, it is not working because the docker container is not aware of localhost as a resolvable URL. You need to provide an accessible URI to it with a valid MySQL server. Think of it as a VM, and localhost makes the VM search internally while the MySQL server is running on the host machine. I hope this clears it for you. Thanks.
@jamesmbowler : Indeed, introspection for MySQL is still not supported and it is tracked here: https://github.com/prismagraphql/prisma/issues/2506
But this issue is not related to that. Hence, closing it.
Please feel free to re-open/reach out in case there are further questions. Thanks.


singleagent_traffic_light_grid

Pull request information

Status: ready to merge
Kind of changes: bug fix
Related PR or issue:  script wasn't running

Description
changed array dimensions to proper size

Pull Request Test Coverage Report for Build 5403

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
5 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.1%) to 89.656%

Files with Coverage Reduction
New Missed Lines
%

examples/exp_configs/rl/singleagent/singleagent_traffic_light_grid.py
5
90.2%

Totals

Change from base Build 5351:
0.1%

Covered Lines:
9222

Relevant Lines:
10286

💛  - Coveralls


Better postgres Connection string support

This PR fixes two tiny issues I have had with the connection string option:

it gives a little demo in the help output
it shows the default port of postgres in case the connection string doesn't support one

🎉


THCudaCheck FAIL file=/pytorch/aten/src/THC/THCGeneral.cpp line=47 error=3 : initialization error

When I prepared the dataset, and run ddp_local.sh, I got the following error:

What's your CUDA and pytorch version?
Also what's your GPU?
Not sure if this is helpful https://blog.csdn.net/qian99/article/details/86716505 (in Chinese), but it seems that if you are using RTX 2080 Ti then there might be an issue that is related to the code.

CUDA 10.1, GPU can be 2080ti or V100, but all of the two types cannot run the code. I've setup detectron2 using the same machine, and it works well.

this is the initialization error, not invalid argument.
Process Process-3:
Traceback (most recent call last):
File "/home/zhubenjin/libs/anaconda3/lib/python3.6/multiprocessing/process.py", line 258, in _bootstrap
self.run()
File "/home/zhubenjin/libs/anaconda3/lib/python3.6/multiprocessing/process.py", line 93, in run
self._target(*self._args, **self._kwargs)
File "/data/repos/PointContrast/pretrain/pointcontrast/lib/multiprocessing.py", line 33, in run
du.destroy_process_group()
File "/data/repos/PointContrast/pretrain/pointcontrast/lib/distributed.py", line 157, in destroy_process_group
torch.distributed.destroy_process_group()
File "/home/zhubenjin/libs/anaconda3/lib/python3.6/site-packages/torch/distributed/distributed_c10d.py", line 555, in destroy_process_group
raise RuntimeError("Invalid process group specified")
RuntimeError: Invalid process group specified
THCudaCheck FAIL file=/pytorch/aten/src/THC/THCGeneral.cpp line=47 error=3 : initialization error

@poodarchu same question here, did you solve it?
Ubuntu 18.04
Cuda 10.2
pytorch1.5.0
python3.7
MinkowskiEngine v0.4.3
Traceback (most recent call last):
File "ddp_train.py", line 62, in main
mpu.multi_proc_run(config.misc.num_gpus,
File "/home_nfs/aoran.xiao/projects/3dPC/self-supervised/PointContrast/pretrain/pointcontrast/lib/multiprocessing.py", line 56, in multi_proc_run
p.join()
File "/home_nfs/aoran.xiao/anaconda3/envs/pointcontrast/lib/python3.7/multiprocessing/process.py", line 140, in join
res = self._popen.wait(timeout)
File "/home_nfs/aoran.xiao/anaconda3/envs/pointcontrast/lib/python3.7/multiprocessing/popen_fork.py", line 48, in wait
return self.poll(os.WNOHANG if timeout == 0.0 else 0)
File "/home_nfs/aoran.xiao/anaconda3/envs/pointcontrast/lib/python3.7/multiprocessing/popen_fork.py", line 28, in poll
pid, sts = os.waitpid(self.pid, flag)
File "/home_nfs/aoran.xiao/projects/3dPC/self-supervised/PointContrast/pretrain/pointcontrast/lib/error_handler.py", line 59, in signal_handler
raise ChildException(self.error_queue.get())
lib.error_handler.ChildException: Traceback (most recent call last):
File "/home_nfs/aoran.xiao/projects/3dPC/self-supervised/PointContrast/pretrain/pointcontrast/lib/multiprocessing.py", line 22, in run
du.init_process_group(proc_rank, world_size)
File "/home_nfs/aoran.xiao/projects/3dPC/self-supervised/PointContrast/pretrain/pointcontrast/lib/distributed.py", line 146, in init_process_group
torch.cuda.set_device(proc_rank)
File "/home_nfs/aoran.xiao/anaconda3/envs/pointcontrast/lib/python3.7/site-packages/torch/cuda/init.py", line 245, in set_device
torch._C._cuda_setDevice(device)
File "/home_nfs/aoran.xiao/anaconda3/envs/pointcontrast/lib/python3.7/site-packages/torch/cuda/init.py", line 153, in _lazy_init
torch._C._cuda_init()
RuntimeError: cuda runtime error (3) : initialization error at /opt/conda/conda-bld/pytorch_1587428266983/work/aten/src/THC/THCGeneral.cpp:47

It only happens when i set CUDA_VISIBLE_DEVICES by using part of GPU cards. e.g. 2  of 8 cards.

It only happens when i set CUDA_VISIBLE_DEVICES by using part of GPU cards. e.g. 2 of 8 cards.

nope, I used all the GPUs on a machine.

This solved the issue for me:
if __name__ == '__main__':     torch.multiprocessing.set_start_method('spawn')     main()
Reference: https://github.com/rusty1s/pytorch_geometric/issues/131


Answer format flexibility

Allow the player to write the answer in different ways (caps, use of . and ,) and still count it as right


[P2P] ValidationContext.BanDurationSeconds doesn't work as expected

See work item 3059.
https://dev.azure.com/Stratisplatformuk/StratisBitcoinFullNode/_workitems/edit/3059

I triggered tests again if they fail again @quantumagi please merge master where tests are stable now.


Warn or error on known bad rules configuration

Similar to https://github.com/honeycombio/refinery/issues/559, but tracking separately...
Today, if you have a typo in rules config like Sample instead of Sampler, refinery will just sample all traces. That can lead to way more event volume than desired until it's fixed.
Ideally, if there's a known bad configuration like this, we could either warn or error on it.

I've done some investigation, and it seems like the right answer here is to leverage jsonschema, since YAML and TOML are freely convertible to and from JSON -- at least at the level we use in Refinery config.
I also think it's probably best done as a separate tool that someone can run separately, rather than embedding it into the refinery executable. (Although I'd hear arguments to the contrary!)
Writing the schema is not technically difficult, but is a bit tedious so is expected to take several days of work. It's now on the list of things to do soon-ish.

I maintain a schema in CUE that could be used for validation as well, if you'd like to see it. It's focused more on code generation than validation, as it's imbibed with the default values used by the Refinery application.


Multibyte string extension does not described in composer.json schema.

The most valuable methods in ISO3166 class rely on multibyte string comparisons:
https://github.com/thephpleague/iso3166/blob/74a08ffe08d4e0dd8ab0aac8c34ea5a641d57669/src/ISO3166.php#L140-L153
So I think the library should describe ext-mbstring as a required extension in the composer.json schema.

Fair enough.


Sharing scope being deprecated by Google

https://github.com/hermanho/MMM-GooglePhotos/blob/843876a2470cb436221ca12aedf794a446778db5/google_auth.json#L4

We’re making two changes to the Google Photos APIs that will take effect March 31, 2025:

Removal of scopes: The Google Photos Library API readonly, sharing, and search scopes will be removed, with photo picking and search uses cases now supported by a new Google Photos Picker API.

It is a breaking change 🙁.

It have to be select per photo instead of album.
Picker session is valid within 7 days. It is required to download all medias with the session Id to local machine. It is a big different compare with current logic that it can poll and refresh the new photos from the selected albums.

Related issue tracker on Google:
https://issuetracker.google.com/issues/368816420
https://issuetracker.google.com/issues/368779600


Make workspace private and personal for the user

Issue: #227
Implement private workspace, every user create a workspace now is only private and not shareable with other user.
This feature implement a new DynamoBD table handle policies on top of Workspace Dynamodb table.
This implementation could facilitate sharing workspaces between users with "is writable" option during share process like Google Drive
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@hghandri this PR is in Draft for a while, do you plan to make it ready for review or still working on it?

Is there any plans to merge this - private workspaces are important feature for enterprise level use of this solution.

Hi,
I plan to work on it with the new GraphQL integration, I'll be back with new implementation.

The current version I’m using does not include GraphQL. Could you confirm if this branch is suitable for testing for private workspace feature ?

@abduljaleel : Hi Abdul Jaleel:
you could get these modification and adapt this new version for your current version with API Gateway.


Fix CommonJS export

This small fix addresses the CommonJS error introduced in https://github.com/dankogai/js-combinatorics/commit/591b4e47685642dc001b203c8f53f40341e71938
Error [ERR_REQUIRE_ESM]: require() of ES Module

combinatorics.js is treated as an ES module file as it is a .js file whose nearest parent package.json contains "type": "module" which declares all .js files in that package scope as ES modules.
Instead rename combinatorics.js to end in .cjs, change the requiring code to use dynamic import() which is available in all CommonJS modules, or change "type": "module" to "type": "commonjs" in package.json to treat all .js files as CommonJS (using .mjs for all ES modules instead).

Can be tested with the following example:
package.json
{
  "dependencies": {
    "js-combinatorics": "^1.5.5"
  }
}

index.js
const combinatorics = require("js-combinatorics");

console.log(combinatorics.version);

node index.js

Thank you!


Improvement of class TwitchChatEvent

Currently the class TwitchChatEvent has a type and a command.
The type is always "TWITCHCHATEVENT".
In lunch-break-rl/player.py#L31 there is no check for the command being "PRIVMSG".
In lunch-break-rl/twitchchatmanager.py#L14 there is this check.
Maybe we create three subclasses that inherit from TwitchChatEvent, namely TwitchChatMessageEvent, TwitchChatJoinEvent and TwitchChatLeaveEvent. This way, we only need to make one check whether we are dealing with the correct event type, instead of first checking if its a TwitchChatEvent and then which type it is.
Because it seems that also checking for the right command value can be easily forgotten. Or am I mistaken? What do you think?
This should only be a quality of life update.

I agree that the current implementation is a bit clunky. Instead of subclassing, how about changing the Event structure a bit? My suggestion is to differentiate on the type:
    def __init__(self, channel=None, command=None, message=''):
        event_types = {'JOIN': 'TWITCHCHATJOIN', 'PART': 'TWITCHCHATLEAVE', 'PRIVMSG': 'TWITCHCHATMESSAGE'}
        self.type = 'TWITCHCHATUNKNOWN'

        if command in event_types:
            self.type = event_types[command]

        self.channel = channel
        self.command = command
        self.message = message

Okay.
Isn't the information redundant given by self.command and self.type?
I know, that we need the type argument because that is a requirement tdl gives us on a custom event class.
So I personally would check for the type, not the command in my own script.

Sort of. The command attribute is used by dumps() to serialize the message. The type attribute is to be used to handle the event.
Most Python event frameworks use the type attribute (both Pygame and tdl do) and I'm trying follow the established practices.
Perhaps we should rename self.command to self._command?

Yeah, maybe the rename could do it. For dumps() the command information is also stored in the dictionary keys.

I'm not sure what you mean about:

Yeah, maybe the rename could do it. For dumps() the command information is also stored in the dictionary keys.

if command in event_types:
            self.type = event_types[command]

That is a locally scoped variable. It is just to help select an appropriate type.

Ah no, I was mistaken in thinking that the dictionary key could be easily deduced from the value, which is not an intended feature of a dictionary. Obviously^^

Like:
command = {'TWITCHCHATMESSAGE': 'PRIVMSG', 'TWITCHCHATJOIN': 'JOIN', 'TWITCHCHATLEAVE': 'PART'}[self.type]

?

Yes, like so.

I withdraw from my previous statement.

Right now I'm inclined to take the simpler solution which is to rename the command attribute to imply it isn't meant to be used as part of the API.

Yes, that is a good suggestion.


Tweaking copper ore Crashes on world gen

Minecraft Version

1.18.2

Forge Version

forge-1.18.2-40.0.36

Mod Version

OreTweaker-1.18.2-3.1.0

Describe the bug

When editing the config to lower copper ore gen it crashes the game when creating a new world

How to reproduce

change "maxVeinSize": 10 and"spawnRate": 8.0,, create new world and it will crash

**Link to [crashlog]

https://pastebin.com/SaT8k62L

Can you send me a link to a pastbin of your copper json file?

https://pastebin.com/CmyXeXj2
it only happens when changing the values for dripstone caves.

I will look into it first thing after I get home from work.
But it looks like there could be an issue with TerraBlender.
OreTweaker does not really like mods that change to much things about world generation.

it crashes without terrablender also

Okey, good to know. Then its not Terrablenders fault.
I will do some testing once and get back to you.

Ok did some more testing, if you change values for both entries it will crash, if you only change one or the other it does not crash, here the json that causes the crash
https://pastebin.com/0EK9MSYP
i removed bop and terrablender and still get the crash, thank you for looking into this.

Found the issue @Cracks777.
Ore Tweaker uses the properties in the copper_ore.json 📃 to generate a unique name for the ore object being registered to the game.
The exact alogritem for creating a unique name I use is:
String registryName = String.format("%s_%s_%s_%s_%s_%s_feature",
                Objects.requireNonNull(ore.getRegistryName()).getPath(),
                fillerName,
                minY,
                maxY,
                spawnRate,
                maxVeinSize
        );

But since both your entries have the exact same values for these probs the second entry ends up with a duplicated name of the first one. I am releasing a simple fix to check if a name is available, if not it will auto append a number behind it to not collide with the name of the first entry. This was a design flaw when I designed this system.
Thanks for finding it.
Will prob have to backport this to 1.16, 1,17, and 1.18. Working on a fix now. Until then you can change one of these probs to make it magically work again 🥳🥳

Created a test build if you want to validate that it works for you now.
https://github.com/EwyBoy/OreTweaker/releases/tag/v1.18.2-Fix-Test
If its stable I will release it on CurseForge tomorrow for 1.16, 1.17 and 1.18.1/2

New code should make it impossible to create duplicates.
private static String makeRegistryNameUnique(String name) {
        if (registryNames.contains(name)) {
            int i = 1;
            while (registryNames.contains(name + "_" +  i)) {
                i++;
            }
            return name + "_" +  i;
        }
        return name;
    }

    private static final List<String> registryNames = new ArrayList<>();

    private static String createUniqueRegistryName(Block block, String filler, int minY, int maxY, float spawnRate, int maxVeinSize, float discardChanceOnAirExposure, String distribution, boolean isDeepSlate, String featureType) {

        String registryName;

        if (isDeepSlate) {
            registryName = String.format("deepslate_%s_%s_%s_%s_%s_%s_%s_%s_%s_feature",
                    Objects.requireNonNull(block.getRegistryName()).getPath(),
                    filler,
                    minY,
                    maxY,
                    spawnRate,
                    maxVeinSize,
                    discardChanceOnAirExposure,
                    distribution,
                    featureType
            );
        } else {
            registryName = String.format("ore_%s_%s_%s_%s_%s_%s_%s_%s_%s_feature",
                    Objects.requireNonNull(block.getRegistryName()).getPath(),
                    filler,
                    minY,
                    maxY,
                    spawnRate,
                    maxVeinSize,
                    discardChanceOnAirExposure,
                    distribution,
                    featureType
            );
        }

        registryName = makeRegistryNameUnique(registryName);
        ModLogger.debug("Registry Name: " + registryName);
        registryNames.add(registryName);

        return registryName;
    }

yes, It worked just fine. thank you for the fast response.

Fixed for 1.16, 1.18.1 and 1.18.2
https://www.curseforge.com/minecraft/mc-mods/ore-tweaker/files


Formatting for formatting.

I just started using redbird, and I know this is parkinson's law of triviality type stuff, but I like redbird and I'd like others to come to your github page and see the headings of the readme formatted correctly.

Thanks!


Write a benchmark for protobuf::convert_output

Implement a benchmark in the benchmarks project for protobuf::convert_output. Use a reasonably sized object without strings and only using primitives such as i32, bool etc.

/bounty 50$

/attempt

Cancelling attempt


Initial value not displaying with TimePicker widget

Hi again,
I have had some difficulties using the TimePicker for update views (UpdateView), as I can't get it to display the initial value of the object to be updated, see the pictures below. As the pictures show, I have had no issues with DateTimePicker and its initial value (no issues with the DatePicker either).
Picture with TimePicker:

Picture without TimePicker:

My widget looks as follows:
class TaskUpdateForm(forms.ModelForm):
    class Meta:
        ...
        widgets = {
            "start_time": TimePicker(
                options={"format": "HH:mm:ss"},
                attrs={
                    "append": "far fa-clock",
                    "input_toggle": False,
                    "icon_toggle": True,
                },
            ),
        }

Can you tell, based on this information, whether this issue is on my or your end, and if so suggest what can be done to solve it?
I hope you can help.
Best regards,
Niels

@knasti I doubt this is the issue (which I'll look into), but for the missing icon, change far fa-clock to fa fa-clock.

@knasti Unfortunately, after doing some digging, this seems to be a problem with the underlying Tempus Dominus JavaScript library. For now, I've updated the README to show how you can set the default time in the TimePicker:
'defaultDate': '1970-01-01T14:56:00'
But I haven't been able to figure out a way to get it to populate initial. I'll keep digging, but for now, I hope this helps.

I am also experiencing same issue.
With a model form, at update (instanced form), the field value is not populated with data from database except if initialized with the datetime picker.
Any progress to solve this critical issue ?

I've tried setting that defaultDate and the correct time is shown when I toggle the control but there's no way to accept that defaultDate as the selected time, you have to change the selected time to get it to show up in the input control.
Any way around that or setting it by javascript for now? I've tried setting the value of the input box manually on page load but the time only flashes and then it quickly goes blank again.

The following approach worked for me. I had to customize the "script" portion of the generated html form, just replacing the "date" value with a django filter tag showing a date part (you could put any date), and a time part (the time from your database), separated by a "T" letter, which will render to this: "2019-07-08T22:34:50".
Hope it helps.

            {{ form.hora_cita.errors }}
            {{ form.hora_cita.label_tag }}

                <input type="text" name="{{ form.hora_cita.html_name }}"
                required id="{{ form.hora_cita.id_for_label }}" class="form-control datetimepicker-input"
                data-toggle="datetimepicker" data-target="#{{ form.hora_cita.id_for_label }}">

```

Rather than putting in work-arounds for the TimePicker only, when initial works for all other fields, I'm going to wait and hope this gets fixed in the underlying library.

Hi,
I have the same issue.
I store a TimeField in the database and I try to display it with the TimePicker.
Everything works fine for the creation but the update does not work.
Fields are not pre-filled with the stored value and I get a moment warning:
value provided is not in a recognized RFC2822 or ISO format
Indeed, the generated script contains a wrong formatted date:
function deferDateTimePicker_id_start_time() {
    if (window.jQuery && $.fn.datetimepicker) {
        $('#id_start_time').datetimepicker({
                                       "format": "HH:mm",
                                       "icons": {"time": "fa fa-clock-o"},
                                       "useCurrent": true,
                                       "stepping": 5,
                                       "date": "T11:30:00"
                                      });
    } else {
        setTimeout(function() { deferDateTimePicker() }, 50);
    }
}

Do you have any idea how to fix it?
If you want, I can put a small test example on my github to reproduce.

I had to get this 'working' so did the following:

In my Update View, get the submitted Time from the request data or load it from DB.
Pass this value as a context variable.
Use JS to push the value into the input field.

View
def updateRequest(request):
    template_name = 'update/request.html'
    context = {}
    if request.method == 'POST':
        form = updateRequestForm(request.POST)

        if form.is_valid():
            pass  # does nothing, just trigger the validation

        context['start_time_initial'] = form['start_time'].value()

    context['form'] = form
    return render(request, template_name, context)

HTML
document.getElementById("id_start_time").value = '{{start_time_initial}}';
My form field, if this matters
    start_time = forms.TimeField(
        required = True,
        widget=TimePicker(
            options={
            'useCurrent': True,
            'format':'HH:mm:ss',
            'defaultDate': '1970-01-01T14:56:00',
            },
            attrs={
                 'icon_toggle': True,
                 'autocomplete':"disabled",
                'append': 'fa fa-clock',

            },
        ),)

Will keep an eye out for a proper fix.


in custom-element components not correct handling type

Version
3.2.4
Reproduction link
https://codepen.io/pawel-golubew/pen/OJmeLvV
Steps to reproduce

Define component

const ShowPhoneNumber = Vue.defineCustomElement({
    props: {
        phone: String
    },
    template: `<div>Phone number is: {{phone}}</div>`,
});

customElements.define('show-phone-number', ShowPhoneNumber);

Use component

<show-phone-number phone="8-909-952-45-93"></show-phone-number>

Show in console

[Vue warn]: Invalid prop: type check failed for prop "phone". Expected String with value "8", got Number with value 8.
  at <App phone=8 > warn$1 @ vue@next:8290

What is expected?
Show number 8-909-952-45-93 as string
What is actually happening?
Show only first char string

There are some bugs here:
https://github.com/vuejs/vue-next/blob/872b3f7ec5e1a41c60018bb1f64d841c309b8939/packages/runtime-dom/src/apiCustomElement.ts#L248-L250
Will convert attribute to number.

Any update on this?


Functional Tests: Move Specification tests to source

EF has following test projects which are being published to nuget.org

Microsoft.EntityFrameworkCore.Relational.FunctionalTests
Microsoft.EntityFrameworkCore.FunctionalTests
Microsoft.EntityFrameworkCore.Relational.Design.FunctionalTests

These tests depend on dotnet-test-xunit. These tests should be split into Facts which rely on xunit (which will be published to nuget.org) and runnable component (which does not get published).

@smitpatel How will this impact my use of these test libraries? Most of my test suite inherit from these libraries...

@ErikEJ I believe impact is mostly going to be a rename of the package.
cc @roji who I believe is using this as well.


Update game post

@nataliele can you help me merge this one?

@nataliele  Thanks!

@eah13  I have finally made it, except I have problem of global variables. Can you have a look at my post? Thanks a lot!


How to run DReyeVR on three connected moniter display.

The idea is to not use a VR headset for now, and I would like to run the simulator on three connected mounted monitors. The front monitor would display the windshield field of view, and the side monitors would display the view of the side windows.

I assume three viewports must be rendered? If yes, would it give better performance compared to VR?

Hmm. The whole point of DReyeVR is to use Carla in VR, so it is not really designed at all with the intention of being run on three displays.
This could be possible by looking further into UE4 nDisplay technology, but we have no knowledge of whether or not it would be successful with Carla/DReyeVR.
Also the performance would probably be similarly bad since the main performance killer is the independent rendering needed by each viewpoint (in VR you need 2: one for each eye. In nDisplay you need n, one for each display). But theoretically it could be easier to stack GPU's together and have each GPU render one display when using nDisplay, so you could maybe maybe get better parallelism that way (eg. using three RTX 3090's for three displays rather than one 3090 for VR).
As an aside, I would definitely recommend checking our @brifsttar's great blog where he discusses nDIsplay, VR, Carla, and more related to driving simulation.

I realize that DReyeVR is not designed for multiple monitors display and that I will have to invest a significant(?) amount of time to implement this. However, we are really pushing for the completion of the study and submission to a conference. So, we are planning on testing multiple approaches at the same time, i.e., VR headset (with top-notch PC specs) and multiple monitors displays as a backup approach, and see what works out well. If VR works well, we plan to use that.
Thank you for such a detailed answer, and for mentioning further resources.

Some labs are using Carla on multiple monitors, like LIST in Luxembourg (video) or IBV in Spain (article). You can try to get in touch with them.
Depending on the field of view covered by the 3 monitors, you can fake it by configuring the main window to be extra wide. However, this obviously is not a valid projection and has a high performance cost, but I think it's what IBV is (was) doing. A few years ago on Carla's Discord (link), someone from LIST mentioned that they were prototyping their Arena with this:

My first attempt was to use the Carla python API to get 3 120deg Cameras side-by-side in the pygame view  - which worked in theory - but was way to slow.

But as @GustavoSilvera mentioned, nDisplay is the correct answer to this problem, as it's exactly designed for such use cases. But I have no clue on how much work is needed to get that running in DReyeVR/Carla, I just know how to make it work in vanilla UE.

Thank you so much for the insights @brifsttar and @GustavoSilvera. Based on your arguments, I think I will go with implementing ndisplay for DReyeVR.

Hello @FBanani, hope you are doing well. We figured that implementing Unreal Engine's ndisplay framework was not viable due to lack of computational and time resources. Therefore, we changed the settings (see here) to launch as windowed full screen and adjusted some parameters such as FOV. The final result was pretty good for our need. I have attached a video of our setup (disregard the moving green dot, it just our eye-tracking implementation).
https://github.com/HARPLab/DReyeVR/assets/76883453/ff3b5ab3-bc14-46b3-8f36-85d496c435ad
Below is my camera parameters (in config file):
[CameraParams]
FieldOfView=120.0       # horizontal field of view (only in stereo camera => NOT VR)
ScreenPercentage=100   # 100% is native resolution, increase for supersampling, decrease for subsampling
# all the intensities range from [0 (off) to 1 (maximum)]
MotionBlurIntensity=0  # how much motion blur in the camera
VignetteIntensity=0    # how intense the vignetting is (darkened corners)
BloomIntensity=0       # how intense the bloom is
SceneFringeIntensity=0 # how intense the SceneFringe is
LensFlareIntensity=0   # how intense the lens flares are
GrainIntensity=0       # how intense the grain is

Thank you so much. I will try it.
On Wed, May 1, 2024 at 8:41 PM Shiv Patel @.***> wrote:

[Caution: Email from External Sender. Do not click or open links or
attachments unless you know this sender.]
Hello @FBanani https://github.com/FBanani, hope you are doing well. We
figured that implementing Unreal Engine's ndisplay framework was not viable
due to lack of computational and time resources. Therefore, we changed the
settings (see here
https://forums.unrealengine.com/t/packaged-game-starts-fullscreen/399257)
to launch as windowed full screen and adjusted some parameters such as FOV.
The final result was pretty good for our need. I have attached a video of
our setup (disregard the moving green dot, it just our eye-tracking
implementation).
https://github.com/HARPLab/DReyeVR/assets/76883453/ff3b5ab3-bc14-46b3-8f36-85d496c435ad
Below is my camera parameters (in config file):
[CameraParams]
FieldOfView=120.0       # horizontal field of view (only in stereo camera => NOT VR)
ScreenPercentage=100   # 100% is native resolution, increase for supersampling, decrease for subsampling
all the intensities range from [0 (off) to 1 (maximum)]
MotionBlurIntensity=0  # how much motion blur in the camera
VignetteIntensity=0    # how intense the vignetting is (darkened corners)
BloomIntensity=0       # how intense the bloom is
SceneFringeIntensity=0 # how intense the SceneFringe is
LensFlareIntensity=0   # how intense the lens flares are
GrainIntensity=0       # how intense the grain is
—
Reply to this email directly, view it on GitHub
https://github.com/HARPLab/DReyeVR/issues/43#issuecomment-2089344536,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXQ7V2MQX2UTF5TUJXDDW3LZAGDUJAVCNFSM53TZ6SGKU5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TEMBYHEZTINBVGM3A
.
You are receiving this because you were mentioned.Message ID:
@.***>


Missing html Tag in the about.astro

There is no html Tag in the about.astro but in the blog.astro and index.astro is the tag.
This is somehow inconsistent.

Hey there, we just updated to Astro v3 and this issue should have been taken care of!
Thanks for opening an issue!