Disk: Does specifying an existing partition letter without a size expand the partition to use the remaining free space on the drive

This is a question about the existing functionality.
Can we use this to resize a volume to use the remaining free space of a drive in addition to its existing space?
We would like to use this to resize partitions on VMs if the virtual disk has been resized. The wiki documentation says regarding Size: "Specifies the size of new volume. Leave empty to use the remaining free space."
However, reading the code for MSFT_Disk's handling of existing partitions, it looks like if size is not specified it might try to resize the partition to $null because $Size hasn't been set and $null is not equal to $assignedPartition.Size. At this point, behavior is up to if Resize-Partition errors when the -Size parameter is $null.

I assume that you don't want to specify the Size property, but want the Size to be "current size of entire disk" (not sure that equals to "remaining free space" above), and together with AllowDestructive should resize the partition?
Feels that the scenario you looking for is not tested among those "AllowDestructive" tests. Closest is this one; https://github.com/PowerShell/StorageDsc/blob/b31dbc236c6c7542f344a3eee5b8aa6b6055d0f4/Tests/Unit/MSFT_Disk.Tests.ps1#L1663-L1722
I think we should add a test for that scenario.
Hoping someone in the community can answer this better.
/cc @PlagueHO

Not the entire disk - if there's another partition on the disk we don't want to delete it. That's what I mean by "remaining free space", the unpartitioned space remaining on the disk. Other than that, yes I want it to resize the partition with AllowDestructive when no Size is specified to use up any space not in a partition.
I also didn't see any tests that cover this case and, like I said, I'm pretty sure the behavior is at the whim of how Resize-Partition handles Size being $null. I tested it this morning and got an error.

This sounds like a bug, labeling it as such, and help wanted so that anyone in the community can run with this.

It does indeed look like this is a bug and should be able to be fixed fairly easily by setting $Size to $supportedSize if Size is not set in the resource just before this line (https://github.com/PowerShell/StorageDsc/blob/2f8d3200aa799ea34ebae4c20707aefc27f6225b/Modules/StorageDsc/DSCResources/MSFT_Disk/MSFT_Disk.psm1#L499). I'll try and get a fix through for this tomorrow or the weekend.

This has required a bit more to it than what I first thought. Hopefully won't be much longer. Needed a lot more new tests to really validate that this isn't going to break anything.


Support for custom vcpkg triplets

I'm using some dependencies which use this crate to link C/C++ libraries on windows (For example leptonica and tesseract).
I am trying to enable cross-language LTO on a build targeting x86_64-pc-windows-msvc. This requires compiling C/C++ code using an LLVM compiler, which means not using MSVC's cl.exe compiler. I have gotten it to compile using clang-cl instead of MSVC's cl.exe and lld-link.exe instead of link.exe manually building the C/C++ code and instructing my dependencies via env vars to not use this crate and instead use the libs I generated, but I think ideally vcpkg can be made to work with it.
I found this issue https://github.com/microsoft/vcpkg/issues/2087 about clang-cl support in vcpkg, which was closed and vcpkg does not support a triplet for it, not as an official triplet, nor as a community maintained triplet shipped with vcpkg. However, vcpkg has support for "custom triplets" which are even less supported than community and you have to manually add to the vcpkg/triplets folder. And in that issue, someone shared their github repository with custom triplets which includes a x64-windows-llvm triplet that uses clang-cl. I'm opening this issue to check/discuss if somehow (Maybe through an environment variable) custom vcpkg triplet support would be in scope for this crate.
I am essentially thinking of something like:
REM I manually make a triplet file for vcpkg and add it to vcpkg/triplets before installing libs and then run this
set VCPKGRS_TRIPLET_OVERRIDE=x64-windows-llvm-thinlto
cargo build --release

vcpkg itself has an environment variable to inject overlay triplets: https://github.com/microsoft/vcpkg/blob/master/docs/users/config-environment.md#vcpkg_overlay_triplets.
Have you tried using that in combination with VCPKGRS_TRIPLET_OVERRIDE?

VCPKGRS_TRIPLET_OVERRIDE does not exist, it was hypothetical


Graphs: Implement Kruskal's Algorithm in C++

Assign this to me @akgmage I have done such question

@akgmage I would like to claim this issue If you would assign it to me

@akgmage I know about this and i would love to claim this issue if you would assign it to me.

Please assign this to me, Ive done these questions before

I have already solved this, assign this to me.
// Kruskal's Algorithm
// Kruskal's Algorithm : used to solve MINMUM SPANNING TREE  -->
// representing the graph as n nodes
// and n-1 edges such that from each node you can travel every other node
// with minimum total edge wieght
// Important concept used : Disjoint set
// which has Ultimate Parent and Union by Rank
// STEPS:
// sort all the edges according to wieght
// from all the sorted edges,
// if the new edge and spanning tree don 't have
// same Ultimate parent, add this edge
// else pass


add better error message when yaml file dne

Please confirm that this PR has done the following:

[ ] Tests Added
[ ] Documentation Added
[ ] Description in RELEASE_NOTES.md Added

per the title. don't think this is a big enough addition to warrant adding to release notes

Coverage increased (+0.02%) to 84.702% when pulling 9d5572bacc0b82bfc6c2dda063744d7ce515f972 on gidden:yamlerror into 381c4f6efccf1691feacff3f961b9b67645a422a on IAMconsortium:master.

Coverage increased (+0.02%) to 84.702% when pulling 9d5572bacc0b82bfc6c2dda063744d7ce515f972 on gidden:yamlerror into 381c4f6efccf1691feacff3f961b9b67645a422a on IAMconsortium:master.


style from css

Hi.
my css file.
.block_ {
border-bottom: 0;
border-top: 0;
color: blue;
display: block;
line-height: 1.65;
text-align: justify;
text-indent: 1em;
margin: 0;
padding: 0
}
my html file.
something.
How to get "p"'s style ?
thank you in advance,

Hi @octalflow
There's an example.
Layout in development stage.

thank you


Make radius & height in CapsuleShape2D independent

Addresses #25657, but for 2D.

Needs testing, especially the changes in shape_2d_sw.cpp, because I didn't know what I'm doing (seems to work though).
Here's some test scene:
CapsuleTest.zip

@KoBeWi Maybe the radius handle should be placed in the middle of the side, so that it isn't moved vertically when dragging it?

@KoBeWi Maybe the radius handle should be placed in the middle of the side, so that it isn't moved vertically when dragging it?

Rebased and applied.
I'm not planning to do 3D version.

Thanks!


Task: adaptive card slice

Overview
Removed redux content as

adaptive cards do not require network calls
adaptive cards is not a shared feature. It does not need to exist in the store

Closes #3190

What errors did you get?


grid - detailTemplate

The icons for switch on/ off the detail view should be a arrows (arrow right/ down...like accordion style).
Reason:
You have the same "plus / minus" icons by "grouping" and by the display of sub grids.
It's more usere friendly to show a difference.

Thanks for the recommendation. I will fix that in the new version.

Hello,
This is improved in the latest development build. You can find the build at https://github.com/atatanasov/gijgo/tree/master/dist
You can see an example about this at https://github.com/atatanasov/gijgo/blob/master/examples/grid/Grid.Grouping.grouping.groupBy.Bootstrap.3.html


added tab bar badge value

Referencing #7

This is a good attempt, but having only badges in init methods means you'll only be able to instantiate them once. I think [this] (https://github.com/antiguab/BATabBarController/pull/11) was more the goal


Report generator: CSV parsing and HTML frame

This is a WIP for how I propose to do report generation. It is missing some pieces, but fit for broad critique/input as I continue working on it.
The first step is complete here: putting CSVs for each trial into a single CSV for a given benchmark/fengine.
Some work which remains is to put all fengine CSVs into a single CSV for each benchmark/ folder. This could be the maximum trial from within each fengine CSV, or the median trial, or just all of them.
There also remains copying setOfCharts.html into the appropriate directories, as well as the creation of a main.html which would simply iframe all respective copies of setOfCharts.html. I believe main.html could be generated in golang, as it would be a series of very similar/formulaic lines of code.

I've just abstracted the innermost loop to it's own function, but the other 3 don't have very much going on inside - I'm thinking some helper functions to reuse 4-5 lines will be more appropriate

composeAllNamed(...) {
  for bmark in bmarks {
    path = join(master_path, bmark.Name())
    handleBmark(desired_report_fname, path)
  }
}

handleBmark(...) {
  ...
  for fengine in fengines {
    path = join(path, fengine.Name())
    handleFengine(desired_report_fname, path)
  }
}

handleFengine(...) {
  ...
  records = ...
  num_record_columns = ...
  for j, trial in trials {
    read_report_path = join(path, trial.Name(), desired_report_fname)
    handleTrial(read_report_path, records, j, num_record_columns)
  }
  this_fe_writer = ...
  this_fe.writer.WriteAll(records)
}

Seems fairly easy to abstract more without getting long parameter lists.

This worked pretty nicely, and in the end most arguments were only for the final step of handling trials

That should be all for Golang code, for now - appendFastestTrial could always be complemented with other functions, such as appendMedianTrial or appendAllTrials. So, hopefully this PR is done!


.img-responsive isn't centered when it's smaller than a parent

=)

Technically Bootstrap 3's docs don't make any statements about .img-responsive implying centering, and we explain how to accomplish centering for such images in the 2nd paragraph of http://getbootstrap.com/css/#images-responsive
Due to backward compatibility, I don't think we can change this in Bootstrap 3. It could be worth considering for Bootstrap 4 though.
@mdo Your thoughts?

Yes, I totally agree.  Now I understand.

Yup, we've left alignment to be up to folks using Bootstrap. I don't see us changing that up either in v4. The goal is to ensure images don't bust layouts—by constraining them to max-width: 100%—and not to prescribe a particular alignment.


Sharing experiences on teaching accelerometer data analysis - suggestions

Hi everyone,
The next webinar will be a discussion session on teaching + learning accelerometer data analysis. I'm preparing a few points to talk about, but it will be very much a discussion forum format. So I'd love to get suggestions on what the discussion could most usefully focus on. Some ideas so far:

Practicalities: what data to use? RStudio Cloud vs own laptops/installations etc?
The balance between 'hands-on' and theory
What level of understanding of what the software is doing is required by different kinds of users?
What are common barriers and how do you deal with them?

Would love to hear other ideas + also which of these you would like to focus on!!

Great list Rosemary. Here are a few additionally suggestions:

Are there any good open teaching materials (slides) for introduction lectures around physical behaviour assessment and accelerometry in particular?
What are good practical assignments. An easy but not very educational approach is to say to the students: "process this data with the software and try to understand the output". If there is more time a better approach could be to give students an example data set and ask them to process and clean the data and answer some basic research questions around data quality and interpretation. Analyse your own data could also be fun, but is less easy to control in terms of learning experience. Ideally you want them to learn about poor recordings and good recordings.

Thank you- these are great ideas and I'll make sure to incorporate them!

I've just uploaded the slides and notes - if you spot any errors/ places where what I typed didn't match what you said then let me know and I'll update them.

We talked in August about freely available teaching resources, and I just came across this (enormous) set of slides, which has some good + potentially relevant content and is under a licence which means you can use and adapt it.

Closing this issue now. It is cited in https://github.com/wadpac/oss-dev-webinar-series-pb-field, so people should be able to find this issue.


Show the blueprints in alphabetical order

Especially useful in case you numbered your blueprints:

Verified on version 8.1.0 rc1


Decompose autohealing from StatHealth

StatHealth is doing too much.  The autohealing capability should be extracted into its own related component (StatAutoHeal?).

Done by 4935e46.


Python version pins need to be combined with Requires-Python

As briefly discussed at https://github.com/astral-sh/uv/pull/4989#discussion_r1684805329
When reading pins from .python-version files in workspaces, the Requires-Python information may need to be layered to resolve the correct Python interpreter e.g. with

3.10 and 3.12 on the PATH
A project that requires >=3.11
And a pin of cpython

We will currently resolve to 3.10 since ignore the project Python requirement entirely when there is a pin.
Instead, we should combine the PythonRequest::Implementation and the Requires-Python into a PythonRequest::ImplementationVersion(..., VersionRequest::Range(...)).

This could get pretty complicated. We may want to think about the problem space a bit before implementing the suggestion.


docs(theming): added tag theming docs

Closes #1093
📝 Description

Added docs for theming tags with examples.

⛳ Current behavior (updates)

No docs present

🚀 New behavior

Adds docs for theming tags

💣 Is this a breaking change (Yes/No): No
📝 Additional Information

@nikolovlazar I made the requested changes. But I had a question regarding the Anatomy section. I saw that the Input component had an image attached for the Anatomy section. So, do I also need to attach an image, and if yes, will this image be provided by Chakra-UI, or do I need to create the image myself?

@pybash1 I'll create that image for you. I might be pretty busy in the next week, so make sure the content of the PR is okay so I can just merge it and add the image myself.

@nikolovlazar sure!

@nikolovlazar the content looks fine to me, you can merge it if you want.


Options for using Duo's PAM module

This pull adds some attributes that allow you to use Duo's PAM module. With PAM, you can also allow Duo to provide the authentication for sudo and control whether you want the first factor to be the user's UNIX password or a SSH public/private key.
The attributes default to no so no alterations will happen to PAM unless specified. The ssh configuration managed through the sshd cookbook does put sshd_config under Chef management as defined by attributes however regardless.

This is great, thanks @mattlqx!

Thanks, I have one more that builds on this that gives the option to use the duosecurity repo and fixes some PAM requirements. I'll get that posted in a minute.


Polyfill

Is there potential for a flexbox polyfill for IE8 and IE9 (and IE10)?

I'm wondering this as well.
#52 says that it works on "every browser out there", but that sounds kind of insane.
@vjeux - can you confirm that this will work with IE8 and IE9?

This won't style the page for you, but it'd totally be possible to write a polyfill that's based on this library.

@RangerMauve - thanks, I think I may have misunderstood the goals of this project.
Just to be clear, is it safe to say that this project is meant to provide flexbox layouts to native environments (iOS, Android), and not to serve as a flexbox polyfill for browsers?

It's not bound to native at all, it just doesn't change the style of DOM nodes for you.
What it does do is compute the different sizing of elements given some CSS rules that use flexbox. If one wants to then use that sizing to render native stuff they can. If they wanted to, they could also use it to update the styles of DOM nodes in the browser.
But out of the box it just does calculations


feat: set mask on token

Mask the temporary token in the logs

Note. When installing dependencies, the package lock wanted to be updated as well. Recommend after merge, updating dependencies in a secondary PR. Or, doing that first.

Oh lol I just missed this one! This should be covered by #14 now (as well as one other call site where we export tokens)


Layers to controller or constructor LiveMap

Hello, I was looking at the liveMap package and I wanted to see if you can add an OverlayImage and CircleMarker layer in the LiveMap or LiveMapController builder. Because I could put the circle around the marker and could add images.

You can use the builder function from Flutter map markers to build your own markers and include any widget: example: https://github.com/synw/livemap/blob/master/example/lib/markers.dart#L34
For images check out this repo: https://github.com/synw/map_markers : it's a collection of markers and it has a photo marker if it can be useful to you. This repo is not documented yet.

Hello, sorry for the delay. And that the OverlayImage that is of the flutter_map the return and of type OverlayImage, and in the build of the marker it waits for a Widget, and in case the image by the example sent I can not use, therefore the image I use would be via overlayImage, because I pass the coordinates note , south, east and west to position the image on the map.

Reopening as this is not resolved. I don't know how overlay images work, I'll check that


signPSBT

Good afternoon. I'm trying to solve a problem with PSBT signature. I saw your project, it is wonderful. But I can't figure out how the code works. It seemed to me that _next/static/chunks are outdated... Maybe you have a PSBT signature function?

Hello mihasi . From this chunks I use signig function which is fixed and cant be outdated . But let me research my solution again as I was working 10 months  ago on it :d . Also tell me exactly ar which point you are geting problem .

In the file NFTIndex3.html there is this line -----> let signRawtxBitcoiadnjs: signSetup, signRawtx: signingPurchase = await Promise.all([window.import_chunk.e(6106), window.import_chunk.e(6548)]).then(window.import_chunk.bind(window.import_chunk, 19897));
I understand that it should load .js which are specified in HEAD.
This import  is probably outdated?
P.S. I'm currently trying to make a function that will sign PSBT. For you it is called signSetup(seed,{psbt:setup_psbt}).

The problem arose here window.import_chunk.e(6106). Since the import  does not work


chore: import repository https://github.com/q1323829945/test3.git

this commit will trigger a pipeline to generate the CI/CD configuration which will create a second commit on this Pull Request before it auto merges

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign q1323829945
You can assign the PR to them by writing /assign @q1323829945 in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


LLM leader board setting for mmlu.

I have pre download the MMLU dataset hails/mmlu_no_train, and I want to reproduce the result on MMLU dataset in LLM leader board (old-version). In the leader board, MMLU setting is 5-shot, hendrycksTest-abstract_algebra,hendrycksTest-anatomy,hendrycksTest-astronomy,hendrycksTest-business_ethics,hendrycksTest-clinical_knowledge...
However, in lm_eval/tasks/mmlu folder, there are several variants are supported. I want to know what parameters I need to set for my tasks. Thanks.
lm_eval version=0.4.3
Name: lm_eval
Version: 0.4.3
Summary: A framework for evaluating language models
Home-page:
Author:
Author-email: EleutherAI<EMAIL_ADDRESS>License: MIT
Location: /home/pclv100s-1/.local/lib/python3.9/site-packages
Editable project location: /home/pclv100s-1/Desktop/dsj/lm-evaluation-harness
Requires: accelerate, datasets, dill, evaluate, evaluate, jsonlines, more-itertools, numexpr, peft, pybind11, pytablewriter, rouge-score, sacrebleu, scikit-learn, sqlitedict, torch, tqdm-multiprocess, transformers, word2number, zstandard
Required-by:

leaderboard_mmlu_pro (available in main, but not v0.4.3 or any PyPI version yet) can be used!


How do I get access to the $fm variable?

I updated to FactoryMuffin 3 because I wanted to update to Faker 1.6 because I wanted to use FactoryMuffinFaker.
With FactoryMuffin 2, I would use League\FactoryMuffin\Facade as FactoryMuffin and then FactoryMuffin::loadFactories(__DIR__ . '/factories'); in my setupBeforeClass method of my base TestCase. I would then use FactoryMuffin statically throughout my test suite. Apparently the facade is gone with version 3.0 and everything must be called from an instance variable, correct?
Anyway, I can't figure out how to get a common instance of FactoryMuffin to persist in my test suite.
I tried changing the FactoryMuffin::loadFactories call into an instance and saving it in my TestCase object, but the setupBeforeClass method is static, so I can't call $this from within.
I tried moving the loadFactories call into the setUp method, but my tests all spit out "The model definition 'App\Object' is undefined".
The README says that the $fm variable will be "made available" upon calling loadFactories. Then a couple of paragraphs later it's talking about calling $fm->loadFactories. What? Where do I call loadFactories?
I looked at the code for the loadFactories method and I have no clue what the $fm declaration is supposed to do. Either this is a paradigm I've never seen before or the $fm declaration does nothing.
private function loadDirectory($path)
{
    $directory = new RecursiveDirectoryIterator($path);
    $iterator = new RecursiveIteratorIterator($directory);
    $files = new RegexIterator($iterator, '/^[^\.](?:(?!\/\.).)+?\.php$/i');

    // what the hell is this supposed to do?
    $fm = $this;

    foreach ($files as $file) {
        require $file->getPathName();
    }
}

The upgrade guide from 2.1 to 3.0 doesn't even mention that the facade is gone, much less that all static calls must now be instance calls. In short, everything is broken and I'm confused. The docs provide contradictory information, and the PHP League website is still on version 2.1.

The idea in 3.x is you create the instance, and then keep track of where you want it by yourself.

// what the hell is this supposed to do?

$fm = $this;

That sets the $fm fariable to the factory muffin instance so that whne you require each file, that variable is already set, so you can reference it.

Okay now I see what you're doing there. The local $fm variable is available in any loaded factories, because they are being required in that method after you declare $fm, whereas the $fm variable that is used throughout the test suite is the instance variable that it's up to me to create. I mean, technically they are the same object, but the scoping was confusing me. Thanks.


Fix segfault

UndefinedBehaviorSanitizer:DEADLYSIGNAL
==11071==ERROR: UndefinedBehaviorSanitizer: SEGV on unknown address 0x7f94ecbfb240 (pc 0x7fff633066f2 bp 0x7ffeea954ae0 sp 0x7ffeea954ae0 T7799094)
==11071==The signal is caused by a READ memory access.
    #0 0x7fff633066f1 in _platform_strlen (libsystem_platform.dylib:x86_64+0x16f1)
    #1 0x1052d2c02 in janet_cstring (janet:x86_64+0x100028c02)
    #2 0x10532c1b3 in populate_oids (core.so:x86_64+0x11b3)
    #3 0x10532c74f in cfun_exec (core.so:x86_64+0x174f)
    #4 0x1052dcea3 in run_vm (janet:x86_64+0x100032ea3)
    #5 0x1052ddbdd in janet_continue (janet:x86_64+0x100033bdd)
    #6 0x1052dd14e in run_vm (janet:x86_64+0x10003314e)
    #7 0x1052ddbdd in janet_continue (janet:x86_64+0x100033bdd)
    #8 0x1052cf981 in janet_dobytes (janet:x86_64+0x100025981)
    #9 0x1052df7f4 in main (janet:x86_64+0x1000357f4)
    #10 0x7fff6311e3d4 in start (libdyld.dylib:x86_64+0x163d4)

==11071==Register values:
rax = 0x415ad6da00000000  rbx = 0x415ad6da00000000  rcx = 0x00007f94ecbfb240  rdx = 0x00007f94ecbfb240
rdi = 0x00007f94ecbfb240  rsi = 0x00007f94ec400000  rbp = 0x00007ffeea954ae0  rsp = 0x00007ffeea954ae0
 r8 = 0x0000000000000006   r9 = 0x0000000000000000  r10 = 0x0000000000000007  r11 = 0x0000000000000007
r12 = 0x00007f94ec634ec0  r13 = 0x0000000000000011  r14 = 0x00007f94ecbfb240  r15 = 0x00007f94ec502d01
UndefinedBehaviorSanitizer can not provide additional info.
==11071==ABORTING

This might be a result of not marking stuff properly for gc. See fced73a for a possible solution. We might also need to either use refcounts or store references to active Results on the corresponding Connection, then when the Connection gets gc'd close the active results first.

To re-state in @bakpakin's words: an open requests counter on the connection could do that. The request object finalizer would decrement the counter on the connection, and if it was 0 and the connection was closed, only then actually clean up the connection. The connection finalizer would similarly decrement the counter, and only Pq free or whatever the connection if the counter was 0.

As of 1.6.0, if two values are freed in the same GC cycle, they will be freed in the reverse order they are allocated (LIFO). This is not guaranteed by the API, but by the current implementation. It also means we don't need to mess with counters or the like right now - the finalizer for a connection will always run after the finalizer for a request if the connection is allocated first (which it must be, as it is used to create the request).

Cool, so I think the basic structure of core.c is good for now then. The above segfault obviously isn't related to the gc issue, and I haven't hit it in a while, so closing this for now.


Updated files to comply with style standards

Coverage increased (+0.005%) to 98.577% when pulling 0d7fe3949b1d11c747fc5d1b5336eb3071bcb2d6 on flake8_init into b12141d25e1df2ab85bc6154e6bc38aec2b485d6 on dev_0.0.1rc1.


Added jump search in python

Added jump search on Jupyter Notebook with comments wherever required.
Please add the hacktoberfest2021 tag.
Kindly review the notebook.

The PR is not getting reflected on hacktoberfest, is there any problem?

The PR is not getting reflected on hacktoberfest, is there any problem?

Can you please add the 'hacktoberfest2021' tag?


Add generic type to BatchDict

Codecov Report
Patch coverage is 100.00% of modified lines.

Files Changed
Coverage

src/redcat/batchdict.py
100.00%

:loudspeaker: Thoughts on this report? Let us know!.


Update c3 needle_in_a_haystack() exercise

Updated test to have Flux String signature instead of Flux Object. For the test to pass, the type of the Flux also needs to change, which is outside the scope of what the test teaches.

Thanks for contribution!


[WIP] Document OpenAPI v3 (beta) support

/retitle [WIP] Document OpenAPI v3 (beta) support
/sig api-machinery

/milestone 1.24

enhancement ref: https://github.com/kubernetes/enhancements/issues/2896

/sig api-machinery

/assign

@Jefftree Reminder that tomorrow (Tue April 5) is the deadline for 1.24 Docs Ready for Review. Please push your docs content for this enhancement and remove the work-in-progress label when ready for review.
We have a tight turnaround to getting docs reviewed and approved for the release, and we’re hoping to have all docs in for review by tomorrow. Thank you!

Closing in favor of #32784
(fine to reopen if that's not right)
/close


add ocplib-endian to jbuild rules; test on 4.06 as well

this fixes #30

now that (a) cstruct.3.2.0 is out (which does not depend on ocplib-endian) and (b) https://github.com/ocaml/opam-repository/pull/10774 is merged to restrict all released mirage-profile to cstruct {< "3.2.0"}, I can either have the newest cstruct (which I like to use) or mirage-logs (which depends on mirage-profile).
can we have a release of mirage-profile, @talex5?  I'm happy to do the release if you don't have the time to do it, just want to check whether there's anything outstanding I should wait for first!?

Sure, go ahead!


Dart 2.0.0-dev.69.0 has more VM runtime problems similar to #7

Upgrading to Dart VM version: 2.0.0-dev.69.0 (Tue Jul 17 14:57:16 2018 +0200) on "macos_x64" now causes failures when running unit tests under the VM. This seems similar to https://github.com/spebbe/dartz/issues/7. Using your ddc_compat branch and still run fine under DDC. Here are the load errors that now cause all unit tests to fail when run in the VM:
Unable to spawn isolate: file:///Users/richj/.pub-cache/git/dartz-c789fc294ebde28cfd5318b9954faa3927630e2f/lib/src/either.dart:70:26: Error: Declared type variables of 'EitherMonad::map4' doesn't match those on overridden method 'Applicative::map4'.
  @override Either<L, E> map4<A, A2 extends A, B, B2 extends B, C, C2 extends C, D, D2 extends D, E>(Either<L, A2> fa, Either<L, B2> fb, Either<L, C2> fc, Either<L, D2> fd, E fun(A a, B b, C c, D d)) =>
                         ^
file:///Users/richj/.pub-cache/git/dartz-c789fc294ebde28cfd5318b9954faa3927630e2f/lib/src/either.dart:67:26: Error: Declared type variables of 'EitherMonad::map3' doesn't match those on overridden method 'Applicative::map3'.
  @override Either<L, D> map3<A, A2 extends A, B, B2 extends B, C, C2 extends C, D>(Either<L, A2> fa, Either<L, B2> fb, Either<L, C2> fc, D fun(A a, B b, C c)) =>
                         ^
file:///Users/richj/.pub-cache/git/dartz-c789fc294ebde28cfd5318b9954faa3927630e2f/lib/src/either.dart:64:26: Error: Declared type variables of 'EitherMonad::map2' doesn't match those on overridden method 'Applicative::map2'.
  @override Either<L, C> map2<A, A2 extends A, B, B2 extends B, C>(Either<L, A2> fa, Either<L, B2> fb, C fun(A a, B b)) =>
                         ^
file:///Users/richj/.pub-cache/git/dartz-c789fc294ebde28cfd5318b9954faa3927630e2f/lib/src/either.dart:76:26: Error: Declared type variables of 'EitherMonad::map6' doesn't match those on overridden method 'Applicative::map6'.
  @override Either<L, G> map6<A, A2 extends A, B, B2 extends B, C, C2 extends C, D, D2 extends D, E, E2 extends E, F, F2 extends F, G>(Either<L, A2> fa, Either<L, B2> fb, Either<L, C2> fc, Either<L, D2> fd, Either<L, E2> fe, Either<L, F2> ff, G fun(A a, B b, C c, D d, E e, F f)) =>
                         ^
file:///Users/richj/.pub-cache/git/dartz-c789fc294ebde28cfd5318b9954faa3927630e2f/lib/src/either.dart:73:26: Error: Declared type variables of 'EitherMonad::map5' doesn't match those on overridden method 'Applicative::map5'.
  @override Either<L, F> map5<A, A2 extends A, B, B2 extends B, C, C2 extends C, D, D2 extends D, E, E2 extends E, F>(Either<L, A2> fa, Either<L, B2> fb, Either<L, C2> fc, Either<L, D2> fd, Either<L, E2> fe, F fun(A a, B b, C c, D d, E e)) =>
                         ^
file:///Users/richj/.pub-cache/git/dartz-c789fc294ebde28cfd5318b9954faa3927630e2f/lib/src/free.dart:70:24: Error: Declared type variables of 'FreeMonad::map4' doesn't match those on overridden method 'Applicative::map4'.
  @override Free<F, E> map4<A, A2 extends A, B, B2 extends B, C, C2 extends C, D, D2 extends D, E>(Free<F, A2> fa, Free<F, B2> fb, Free<F, C2> fc, Free<F, D2> fd, E fun(A a, B b, C c, D d)) =>
                       ^
file:///Users/richj/.pub-cache/git/dartz-c789fc294ebde28cfd5318b9954faa3927630e2f/lib/src/free.dart:67:24: Error: Declared type variables of 'FreeMonad::map3' doesn't match those on overridden method 'Applicative::map3'.
  @override Free<F, D> map3<A, A2 extends A, B, B2 extends B, C, C2 extends C, D>(Free<F, A2> fa, Free<F, B2> fb, Free<F, C2> fc, D fun(A a, B b, C c)) =>
                       ^
file:///Users/richj/.pub-cache/git/dartz-c789fc294ebde28cfd5318b9954faa3927630e2f/lib/src/free.dart:64:24: Error: Declared type variables of 'FreeMonad::map2' doesn't match those on overridden method 'Applicative::map2'.
  @override Free<F, C> map2<A, A2 extends A, B, B2 extends B, C>(Free<F, A2> fa, Free<F, B2> fb, C fun(A a, B b)) =>
                       ^
file:///Users/richj/.pub-cache/git/dartz-c789fc294ebde28cfd5318b9954faa3927630e2f/lib/src/free.dart:76:24: Error: Declared type variables of 'FreeMonad::map6' doesn't match those on overridden method 'Applicative::map6'.
  @override Free<F, G> map6<A, A2 extends A, B, B2 extends B, C, C2 extends C, D, D2 extends D, E, E2 extends E, FF, F2 extends FF, G>(Free<F, A2> fa, Free<F, B2> fb, Free<F, C2> fc, Free<F, D2> fd, Free<F, E2> fe, Free<F, F2> ff, G fun(A a, B b, C c, D d, E e, FF f)) =>
                       ^
file:///Users/richj/.pub-cache/git/dartz-c789fc294ebde28cfd5318b9954faa3927630e2f/lib/src/free.dart:73:25: Error: Declared type variables of 'FreeMonad::map5' doesn't match those on overridden method 'Applicative::map5'.
  @override Free<F, FF> map5<A, A2 extends A, B, B2 extends B, C, C2 extends C, D, D2 extends D, E, E2 extends E, FF>(Free<F, A2> fa, Free<F, B2> fb, Free<F, C2> fc, Free<F, D2> fd, Free<F, E2> fe, FF fun(A a, B b, C c, D d, E e)) =>
                        ^

Hi and thanks for reporting!
They are really tightening the noose on me now :-)
I've pushed a change on ddc_compat where all the previously overridden mapX/liftX operations are moved from the Applicative instance for a type to static methods on the type itself. For example, OptionMP.map2(some(1), some(2), (int a, int b) => a+b) is now Option.map2(some(1), some(2), (int a, int b) => a+b).
I've also started to add some type specialized traverse/sequence operations, such as IList#traverseOption here and there, which will eventually completely replace the generic traverse/sequence mechanism. These should work in a fully type safe way on both Dart 1 and 2.
Hope that keeps you going until the next Dart 2 dev release :-)

Thanks - our unit tests are running again in the VM. Also I was able to change some ugly code over to using traverseX operations. Too bad they need to be specialized - I guess it's progress...
Google is in clean up mode since Dart 2 stable is "happening very soon". You may have already seen this: Getting ready for Dart 2, and making your packages look great on the Pub site!.


Should colons be allowed in method names?

In working out a script that gathers completions, I asked a question on IRC:

Can a :: sigil appear in names of methods/subs?

Initially the response was "that doesn't even make sense" but was followed up by:
class Foo { method foo::bar () { say 665; }; }; Foo.new."foo::bar"(); #-> 665

I don't understand what the usefulness of this feature is.  I've done some skimming through S02#names and I can't find any mention of "you can use colons in method names but you'll need to access them via quotes" (though I could've missed it).
I confess, I don't quite have a firm grasp on the :: sigil and package lookup, but why would this be a useful feature?

The feature isn't really that ':' can be in a method name -- it is that ANYTHING can be in a method name. An illustrative example is using ' ' (a single space) as a method name.
class CrayCray {
    my $method = method (|args) { 42 };
    $?CLASS.^add_methmethod(' ', $method)
}
say CrayCray.new." "() # outputs "42"

You can see here that I wasn't able to declare this method using the normal syntax. So maybe another question is why we are able to use the ":" in the method name without having to go out of our way to make it look fancy. I guess... because it's not ambiguous because you can't put namespaces in normal method declarations.

I'm actually for allowing as much as possible in method names. Not so much for Perl 6 code as method names that have to be quoted all the time aren't that useful, but for interop with other languages which may have strange naming conventions.

@niner I'm all for allowing any kind of character in method names but I think it's a bit confusing to just let colons occur in method declarations without needing to add them via a special method (ala the space method @awwaiid posted).  At the very least, somewhere along the way, a warning/error should be issued as the :: syntax is meant to be a specific syntactic/semantic operator/sigil in a few different contexts (all of them revolving around names of things).

:: is a required feature in method names because of the way coercions work.
http://docs.perl6.org/language/functions#Coercion_Types
A single ':' is even more necessary because of method foo:sym in
grammar actions.
On Mon, Jan 18, 2016 at 5:44 AM Jake Russo<EMAIL_ADDRESS>wrote:

@niner https://github.com/niner I'm all for allowing any kind of
character in method names but I think it's a bit confusing to just let
colons occur in method declarations without needing to add them via a
special method (ala the space method @awwaiid https://github.com/awwaiid
posted). At the very least, somewhere along the way, a warning/error should
be issued as the :: syntax is meant to be a specific syntactic/semantic
operator/sigil in a few different contexts (all of them revolving around
names of things).
—
Reply to this email directly or view it on GitHub
https://github.com/perl6/specs/issues/106#issuecomment-172363415.

@LLFourn thanks for the linkage!  So in that situation they are allowed because they need to match how the type is written.  Could you provide a link or explanation wrt to grammar actions, I don't see anything like that in the grammar actions section.
It still feels like some discussion could be had regarding whether they should be declared the same or if maybe there could be a pertinent error message given if you tried to call one, e.g.,

"Cannot locate &bar, this method call contains :: perhaps you meant to enclose the call in quotes."

https://github.com/rakudo/rakudo/blob/nom/src/Perl6/Actions.nqp#L1570
I haven't tried but it probably works in vanilla Perl 6 too. (btw we truely need to work on the grammar docs).
If you mean that error messages could be improved when invoking methods with "::" that don't exist I can agree with you. Keep in mind that qualified method calls are a thing:
Lloyds-iMac:~ llfourn$ p '"foo".Mu::say()'
foo
Lloyds-iMac:~ llfourn$ p '"foo".Mu::sa()'
Cannot find method 'sa'
  in any find_method_qualified at gen/moar/m-Metamodel.nqp line 1101
  in block <unit> at -e line 1

Thanks everyone for clearing this up for me!


Are the scripts used to parse the PDF files available?

I'm curious about how the CSV files were generated and was hoping to look at the scripts that were used for parsing the PDF documents found in the repository.  Are these available somewhere?

For most, I convert the PDFs to excel or word and go from there. No specific script, since the data is just a quick copy-paste away.
I did write a small API in R to gather US data under CDC Report, depending on whether I scrape those from the web archives or directly from the CDC site.

Are you open to accepting pull requests that automate this process?  It might be easier to keep things up to date this way.

Feel free!


rootFsPartSize: Add option to specify ROOTFS_PARTSIZE

This is in particular helpful if you have a device like bpi-r3 which comes with 8gb-emmc.

Thanks!

Hey, thanks that was a quick merge - appreciate this :) Are you open for further expansion of the functionality?

Yes. Just don't break existing interfaces, I use this.


Test apiview project param

Test changes

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-template


Should Hash::const_hash be public

sha2561::Hash::const_hash is only used in our repo in the function Midstate::hash_tag.
It is assumed (by me) that users are only using all the Midstate stuff to create do do tagged hashing and for that they would likely use Midstate::hash_tag directly.
Said another way, Hash::const_hash looks like its a helper function for use in Midstate::hash_tag, is there any real use case for it or should we make it private.

Yes, there are real usecases for people having hashes that they can compute at compile time. Defining constants for things like the genesis hash, for example.
I think that, other than maybe the name, it's fine to stabilize it.

Aight, thanks.

I think that, other than maybe the name, it's fine to stabilize it.

Ideally, &mut references and platform intrinsics would be const and we wouldn't need it at all. Maybe in 20 years.


Reduce deprecated API usage

This reduces the amount of deprecated API calls. It doesn't resolve all deprecated warnings.

As quite some time passed and v3.0.0 is released now, I rebased the changes on the master branch.


Deactivate name search

This PR removes the filter functionality in the Semantic Hub to search for specific BAMM elements.

Can you please provide a quick description of the intention of these change for documentation reasons?

Thank you for updating the description and the PR.
There seems to be an issue with the GitHub action for gitleaks. But the actual generated report did not contain any findings.


add missing features and 14th and 15th editions

Closes https://github.com/CanadaHonk/test262.fyi/issues/46
I think i missed a few from the 14th edition so if i should move any lmk

Looks good, ty!


Replace label-has-for with label-has-associated-control (to match eslint-plugin-jsx-a11y)

eslint-plugin-jsx-a11y's label-has-for has recently been deprecated and replaced by label-has-associated-control. This plugin should probably follow suit. I find the default behavior of label-has-for to be confusing, since using both wrapping and a for attribute is redundant. (And for styling reasons, it is often not desirable to wrap <input>s with <label>s.)

but how about this line in the doc: https://github.com/vue-a11y/eslint-plugin-vuejs-accessibility/blame/master/docs/label-has-for.md#L8
is it really safe for all assistive technologies to use only one of them?

Do any assistive technologies actually still not support both methods?

I don't know, do you know how to check? :sweat_smile: is there a caniuse AT version? :sweat_smile: this would be nice

I found this: https://a11ysupport.io/tech/html/label_element (it seems a really cool site), and there are reports of the implicity relationship between the label and the control element that are not working correctly (mainly on voice control).

But they are old, and I don't know how to teste them to update the data :thinking:

I also did some research, and found this document from 2008, which states that "some assistive technologies do not correctly handle implicit labels (for example, <label>First name <input type="text" name="firstname"></label>)". The modern equivalent simply recommends the usage of explicit labels... never even mentioning implicit labels.
It seems that the only lack of support that ever existed was for implicit labels, and explicit labels (i.e. using for) have always been well-supported and/or recommended. With that in mind, it seems like we should update the rule to at least not complain about explicit <label>s. There seems to be no accessibility advantage to using <label> as a wrapper if you've already given it a for attribute. The rule could still complain about <label>s without fors, of course.

Nice resource, are you suggesting we change the default configuration of the rule? because currently we as user can change de "requireness" of nesting 🤔 (not a maintainer here)

Yeah, I'm suggesting we change the default configuration.

I wrote an article on accessible form elements recently (https://austingil.com/how-to-build-html-forms-right-accessibility/) and my research aligns with some of the comments stated above. Implicit labels are supported by most AT, but not all. As far as I could find, explicit labels are supported by all, or I didn't find anything stating the contrary.
In my opinion, enforcing explicit labels via ID should be the default requirement, but using a wrapping label should not be (although users can opt-in). The reason is that by setting a more strict default, more users will encounter errors (possibly leading to more issues here), and it restricts the markup options for users. In some cases, it's not possible to wrap an input with it's respective label because they are actually quite far apart. On the other hand, I'm not sure I see any benefit from enforcing labels wrap inputs as the default setting. I don't believe label+ID is any more accessible than just ID, but I reserve my right to be wrong ;)

explicit labels are supported by all

Hmm, I'd just like to point out that this link: https://a11ysupport.io/tech/html/label_element, suggests that explicit label do not work with voice access on android, that said neither does implicit label.

Yeah, good catch. I should have said that everything that supports implicit labels supports explicit labels, but the opposite is not true.

This is still not working in "eslint-plugin-vuejs-accessibility": "1.1.1". Any updates on the solution to fix this false positive issue?

This is still not working in "eslint-plugin-vuejs-accessibility": "1.1.1". Any updates on the solution to fix this false positive issue?

Well, @aitelia, you could very well open a PR to update the rule name and the default configuration to just check for explicitly labels instead of enforcing wrapping+id, but be aware that you as an user can fix this issue for yourself on your project just by updating the rule definition as stated on #119

@vhoyer yes, workaround did help. Hope this gets fixed sometime in future.


link insertion

when we are doing assignments that need glitch or github assignment its tiring to paste link again and again
it woud be good it same link remains pasted after we open next challenge , user still will be able to replace it with new glitch or github link if he/she wants

so you mean the link of a previous project should show up as default for the current one?

so you mean the link of a previous project should show up as default for the current one?

yes of the previous challenge

If each section used the same boilerplate throughout, the first lesson's solution link field would be empty with only the normal placeholder test , then local storage could be used to track/populate each proceeding lesson with the url used for the previous lesson  However, since some sub sections have multiple boilerplates, it might be confusing for users to auto-populate a previously used url for a lesson where they would need to enter a new url.  Ideally, when a new boilerplate is used, the lesson 's solution link would be empty again with only the normal placeholder text.

If each section used the same boilerplate throughout, the first lesson's solution link field would be empty with only the normal placeholder test , then local storage could be used to track/populate each proceeding lesson with the url used for the previous lesson However, since some sub sections have multiple boilerplates, it might be confusing for users to auto-populate a previously used url for a lesson where they would need to enter a new url. Ideally, when a new boilerplate is used, the lesson 's solution link would be empty again with only the normal placeholder text.

but it would be great where it same link can be used eg:- glitch link

What we might be able to do is just set a local storage that will contain something like this:
{
  "challenge-name/link": {
     saved-link: "https://link-to-the-project.com"
  }
  "challenge-name/link": {
     saved-link: "https://link-to-the-project.com"
  }
}

And then make the I've Completed This Challenge button trigger a function that will take the link, parse it into the json, and store it into the local storage.
Then have a function check if there is an input or if there is already a way to check if the challenge requires a link, and take the url the user is currently in, and then take the JSON and search for the specific challenge-name/link and then fill in the input with the saved url.
Just a suggestion 🙂

Thank you for reporting this issue.
This is a standard message notifying you that the issue has become stale, so we are going to close it.
If you think we're wrong in closing this issue, please request for it to be reopened. Thank you and happy coding.


MockSupportPlugin installCopier is not working

I want use Copier with Comparator in MockSupportPlugin.
Comparator is working with both of MockSupportPlugin(MockSupportPlugin::installComparator ), MockSupport(MockSupport::installComparator)
Copier is not working with MockSupportPlugin(MockSupportPlugin::installCopier) but is working with MockSupport(MockSupport::installCopier)
when I want to use withOutputParameterOfType( new function v3.8 ) it is not work even if installed Copier at main() function with MockSupportPlugin also it is not work both of mock() and mock_c()
but when if copier installed in test fucntion( mock().installCopier ) it is work.
-Not working-
In Main()
MockSupportPlugin mockPlugin;
TimeCopier copier1;
TimeComparator comparator1;
DryCtrlParamComparator comparator2;
UserSetInfoComparator comparator3;
mockPlugin.installCopier("T_Time", copier1); <- not working
mockPlugin.installComparator("T_Time", comparator1); <- working
mockPlugin.installComparator("T_DryCtrlParam", comparator2); <- working
mockPlugin.installComparator("T_UserSetInfo", comparator3); <- working
-Error message-
MockFailure: No way to copy type (T_Time). Please install a MockNamedValueCopier
=Working=
In TestFunc()
TimeCopier copier;
mock().installCopier("T_Time", copier);

I ran into the same problem.  I found that when MockNamedValueComparatorsAndCopiersRepository::installComparatorsAndCopiers iterates through the supplied repository, the new nodes created are always comparator nodes (the second parameter to the constructor is p->comparator_) and therefore the any copier nodes have their copier_ member set to NULL.
My solution was to create a new MockNamedValueComparatorAndCopierRepositoryNode constructor that takes 4 parameters, name, comparator, copier, and next and then use that constructor to make the copy in MockNamedValueComparatorsAndCopiersRepository::installComparatorsAndCopiers.  There might be a better solution, but it worked for me.


Nothing happens

I successfully connected it to my google calendar, but typing anything like "do this tomorrow" or a date doesn't do anything. no text is underlined or highlighted, there is no popup or option to sync those to the calendar.
I have made sure it is enabled.
Any idea what might be going on?

Hi! Thanks for opening this issue.
To keep the plugin lightweight only a subset of all the possible calendar events are recognized. In the near future users will be able to use custom patterns to make the experience more enjoyable. Can you share the unrecognized pattern?

Other than the exact words you used in the example, this plugin seems to pick up pretty much no pattern. No amount amof specifying dates or days of the week triggers it. It seems to be completely arbitrary.

Hi there, thank you for the very long (but not very useful) response. I'm once again asking you to share some of the not recognised patterns.
I developed this plugin for personal use, so I implemented patterns that I find familiar. For this reason, if you're kind enough to post some of the unrecognised sentences I can consider working on them too.

no thanks


we'd better make a issue label.

I think labels are a great tool for managing issues efficiently.

@ho1234c I agree with you. We can categorize it such as study, work, Q&A, and etc.

Yes, you can suggest labels.


Could I modify this to make it portrait mode?

Hi @anishathalye awesome template! I wanted to know if there's a possibility I could make this into portrait mode? Possibly A0 size?
Thanks!

You can set an arbitrary shape by editing this line: https://github.com/anishathalye/gemini/blob/master/poster.tex#L12
If you make it portrait, you might want to adjust the number of columns, column size, column spacing, etc. (see here: https://github.com/anishathalye/gemini/blob/master/poster.tex#L26-L29)

For anyone here in the future a cleaner way of achieving this would be to set:
\usepackage[orientation=portrait, size=a0, scale=1.5]{beamerposter}

in poster.tex.
This ensures you have the correct scaling for a0 (or whatever size you choose).
Of course, if you have an unconventional paper size then the suggestion above is the way to go.


Upgrade gradle to 8.3

Fix warning for deprecated "automatic loading of test framework implementation dependencies".


async/await

Eventually we want to move from generators to async/await in Kybra. This might be possible very soon once we have the more complete stdlib, but RustPython might give us some problems as the async story doesn't seem very good.

There was a few async function bug fix including https://github.com/RustPython/RustPython/pull/5276


Getting a snapshot should be nullable

Current dependencies on/for this PR:

main

PR #9   👈

This stack of pull requests is managed by Graphite.


fix order of MAPPO reshape in MPE & Hanabi

also remove done masking in MPE as this doesn't seem to do anything. Instigated by #85

hey alex you inadvertently pushed your user in mappo_homogenous_rnn_hanabi.yaml. Same for start_wandb_sweep.py I guess?

Yep am running the sweep on this branch, will clean it up before we merge
:)
On Fri, 3 May 2024 at 08:17, mttga @.***> wrote:

hey alex you inadvertently pushed your user in
mappo_homogenous_rnn_hanabi.yaml
https://github.com/FLAIROx/JaxMARL/pull/87/files#diff-b204beb1b655085327a876984fe802f5bb58c1331d262420e7b8f185e746f711.
Same for start_wandb_sweep.py I guess?
—
Reply to this email directly, view it on GitHub
https://github.com/FLAIROx/JaxMARL/pull/87#issuecomment-2092444800, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AP32SW3G5VCZCTAA2GJRIHDZAM2YBAVCNFSM6AAAAABG5IDD7OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJSGQ2DIOBQGA
.
You are receiving this because you authored the thread.Message ID:
@.***>

ah ok sweet :)


graph::SetDefaultDevice with shared library build

Hi,
I have problem using graph::SetDefaultDevice using the shared library build. I understand why the static build will not work for my example below (it does more than inference). But the tensorflow_cc shared library should work the same as one built with bazel. If I build the code from within tensorflow using bazel it works fine but if I use  libtensorflow_cc with cmake it doesn't, why ?
#include "tensorflow/cc/client/client_session.h"
#include "tensorflow/cc/ops/standard_ops.h"
#include "tensorflow/core/framework/tensor.h"
#include "tensorflow/core/graph/default_device.h"

int main() {
  using namespace tensorflow;
  using namespace tensorflow::ops;
  Scope root = Scope::NewRootScope();
  // Matrix A = [3 2; -1 0]
  auto A = Const(root, { {3.f, 2.f}, {-1.f, 0.f} });
  // Vector b = [3 5]
  auto b = Const(root, { {3.f, 5.f} });
  // v = Ab^T
  auto v = MatMul(root.WithOpName("v"), A, b, MatMul::TransposeB(true));

  GraphDef def;
  TF_CHECK_OK(root.ToGraphDef(&def));

  graph::SetDefaultDevice(false ? "/device:GPU:0" : "/cpu:0", &def);
  for (auto &node: *def.mutable_node()) {
        node.set_device("/cpu:0");
        std::cout << node.name() << " = '" << node.device() <<"'"<< std::endl;
  }
  std::cout << "=======================\n";

  std::vector<Tensor> outputs;
  ClientSession session(root);
  // Run and fetch v
  TF_CHECK_OK(session.Run({v}, &outputs));
  // Expect outputs[0] == [19; -3]
  LOG(INFO) << outputs[0].matrix<float>();
  return 0;
}

I get the following error when running the resulting program
2018-09-06 18:18:13.853316: I tensorflow/core/platform/cpu_feature_guard.cc:141] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 FMA
2018-09-06 18:18:13.856079: F /home/daniel/tensorflow_cc/example/example.cpp:27] Non-OK-status: session->Create(def) status: Not found: No attr named '/cpu:0' in NodeDef:
     [[Node: Const = Const[dtype=DT_FLOAT, value=Tensor<type: float shape: [2,2] values: [3 2][-1]...>, _device="/cpu:0"]()]]
     [[Node: Const = Const[dtype=DT_FLOAT, value=Tensor<type: float shape: [2,2] values: [3 2][-1]...>, _device="/cpu:0"]()]]
Aborted (core dumped)

Hi, does it work correctly for you, if you move
  GraphDef def;
  TF_CHECK_OK(root.ToGraphDef(&def));
  graph::SetDefaultDevice(false ? "/device:GPU:0" : "/cpu:0", &def);

right after
  Scope root = Scope::NewRootScope();

?

Thanks! yes, it does work that way.
How do I use this fix for a program that loads a pb file (not construct it) ?
And more importantly why compiling the original program (without your fix) from within tensorflow using bazel works but not when using a shared tensorflow library. I have also tried the same program from windows using tensorflow.dll files I got from here and got the same problem.

Sorry, I am not really sure. You can try to freeze your graph before trying to load it as it will drop the assigned device from the individual nodes. It could be related to the issues you mention. Let us know if you manage to make it work.

Hi, you can try the latest master that stands on TF 1.12 if you still have the issue.

@dshawul Were you able to resolve this?

@FloopCZ I am having this issue with the bazel build not being able to call SetDefaultDevice. Is there a solution for this that you know of?

@ttdd11  No, but I used a workaround of using bazel to compile the program directly with a BUILD file.
That works correctly with the above program, but if i try to use the shared build of tensorflow_cc it still doesn't work even with latest version of tensorflow.

@dshawul Can you explain a bit more? Instead of linking the tensorflow_cc to you app you are building your app with bazel and the source?

Yes, that is correct.
Since my app was simple, I took that easy solution as soon as i figured out how to construct a bazel build file for my app. Also, I didn't need to ship libtensorflow_cc.so file along with my app when building with bazel, which was a plus.
I have spent days looking for solutions before going the bazel way. For more details on why monolithic tensorflow builds have this problem, please refer to the links I gave above.

Thanks for the advice. My application is unfortunately somewhat complicated and we required debugging regularly in visual studio. However I may need to do this given tensorflow has had this issue for a while and we require rtx support which means cuda 10 and later versions of tensorflow.

@dshawul I think I am going to try what you are doing and just maintain two build. Can you provide some advice on using the BUILD file or copy some samples on how you got tensorflow source into the build file?

First I put my app somewhere inside the tensorflow source tree, e.g tensorflow/cc/myapp. This is a must to build with bazel. Then I write a a BUILD file and put in the myapp directory. Example build file:
load("//tensorflow:tensorflow.bzl", "tf_cc_shared_object")

tf_cc_shared_object(
    name = "myapp.dll",
    srcs = [
	"app.cpp",
	"app.h",
    ],
    deps = [
        "//tensorflow/cc:cc_ops",
        "//tensorflow/cc:client_session",
        "//tensorflow/core:tensorflow"
    ],
    defines = [ "TENSORFLOW" ]
)

My app is actually a shared object not an exe. Put in all your *.cpp and *.h in the "srcs". You can define predifined macros in defines -- if you want for example a -DTENORFLOW to be passed to the compiler.
Then execute:
bazel build --config=opt --config=monolithic //tensorflow/cc/myapp:myapp.dll
Then you should get myapp.dll that is a standalone binary (not libtensorflow dep) in
tensorflow/bazel-bin/tensorflow/cc/myapp
Hope that helps

@dshawul Okay well this could actually work. Just need to isolate all tensorflow to a few files and build them as a dll, then link to those to get the tensorflow working. Is this basically what you did?

Note that you can directly build an exe (your app) with bazel. My app was a dll from the beginning. It is not required that you build a shared library for the workaround. Good luck.

Okay sounds good. I'm just thinking for ease of debugging if I build a dll then I can link it and test other code pretty easily. Does this sound like it should work?

Yes that sounds good. It is also good to have all tensorflow code in a separate module.

@dshawul this has been very helpful. How did you configure tensorflow with different options (such as CUDA 10) before running your build file? Did you just run the configure.py first and then run the build as you described above?

@dshawul I've gotten the lib to build and sadly and unable to use it. Here is my code for the lib. It's very simple I just want to see if the SetDefaultGraph works on windows.
This is my header:
#ifndef INFERENCELIB_H
#define INFERENCELIB_H
#ifdef TF_API
#define TF_API __declspec(dllexport)
#else
#define TF_API __declspec(dllimport)
#endif
//Standard includes
#include
//tensorflow includes
#include "tensorflow/cc/ops/const_op.h"
#include "tensorflow/cc/ops/image_ops.h"
#include "tensorflow/cc/ops/standard_ops.h"
#include "tensorflow/core/framework/graph.pb.h"
#include "tensorflow/core/framework/tensor.h"
#include "tensorflow/core/graph/default_device.h"
#include "tensorflow/core/graph/graph_def_builder.h"
#include "tensorflow/core/lib/core/errors.h"
#include "tensorflow/core/lib/core/stringpiece.h"
#include "tensorflow/core/lib/core/threadpool.h"
#include "tensorflow/core/lib/io/path.h"
#include "tensorflow/core/lib/strings/stringprintf.h"
#include "tensorflow/core/platform/env.h"
#include "tensorflow/core/platform/init_main.h"
#include "tensorflow/core/platform/logging.h"
#include "tensorflow/core/platform/types.h"
#include "tensorflow/core/public/session.h"
#include "tensorflow/core/util/command_line_flags.h"
class TF_API InferenceLib
{
public:
InferenceLib(const char* strGraphPath, bool& bStatus, const int nGPU = -1);
~InferenceLib();
private:
std::unique_ptrtensorflow::Session	m_TFSession;
bool									loadGraphAndSession(const char* strPathGraph, int nGPU = -1);
};
#endif //INFERENCELIB_H
This is the cpp:
#include "InferenceLib.h"
using tensorflow::Flag;
using tensorflow::Tensor;
using tensorflow::Status;
using tensorflow::string;
using tensorflow::int32;
InferenceLib::InferenceLib(const char* strPathGraph,
bool& bStatus,
const int nGPU			/=-1/)
{
if (loadGraphAndSession(strPathGraph, nGPU))
{
bStatus = true;
}
}
bool InferenceLib::loadGraphAndSession(const char* strPathGraph, int nGPU /= -1/)
{
bool bReturn = true;
std::string path = strPathGraph;
tensorflow::GraphDef graph_def;
Status load_graph_status = ReadBinaryProto(tensorflow::Env::Default(), path, &graph_def);
if (!load_graph_status.ok())
{
bReturn = false;
}
tensorflow::SessionOptions sessOps;
if (-1 != nGPU)	//here asssigning the graph to a specific gpu, useful is building multiple instances of this
{
	std::string strGPU = std::to_string(nGPU);
	tensorflow::graph::SetDefaultDevice("/gpu:" + strGPU, &graph_def);
	sessOps.config.set_allow_soft_placement(true);
	sessOps.config.set_log_device_placement(true);
	sessOps.config.mutable_gpu_options()->set_allow_growth(true);					//this line avoids large preallocations (I don't care about this)																					//sessOps.config.mutable_gpu_options()->set_per_process_gpu_memory_fraction(50); //100 should be defauly
}

m_TFSession.reset(tensorflow::NewSession(sessOps));
Status session_create_status = (*this->m_TFSession).Create(graph_def);

if (!session_create_status.ok())
{
	bReturn = false;
}
return bReturn;

}
InferenceLib::~InferenceLib()
{
}
My build file is:
load("//tensorflow:tensorflow.bzl", "tf_cc_shared_object")
tf_cc_shared_object(
name = "InfLib2.dll",
srcs = [
"InferenceLib.cpp",
"InferenceLib.h",
],
deps = [
"//tensorflow/cc:cc_ops",
"//tensorflow/cc:client_session",
"//tensorflow/core:tensorflow"
],
defines = [ "TENSORFLOW",
"TF_API"]
)
I run this in Powershell:
bazel build --config=opt --config=monolithic //tensorflow/cc/InfLib2:InfLib2.dll
And the lib builds. But I get link errors when calling to any function and the .lib seems to be corrupted (at least depends says that its not a valid 32 or 64 bit lib).
Does this look correct? I've been on this for about a month now and don't seem to be any closer. Any help would be greatly appreciated. The symbols I need are actually in the dll so I'm thinking its how I built it.

I think you need to put more dependencies 'deps' for your case that will cover all your tensorflow includes. I had only three because i only included these:
#ifdef TENSORFLOW
#include "tensorflow/core/platform/env.h"
#include "tensorflow/core/public/session.h"
#include "tensorflow/core/graph/default_device.h"
#endif

You have many more includes than me. A good test to see if bazel solution works is to compile the example I provided in my original post by putting it in //tensorflow/cc/example/example.cc and using
the below build file
   load("//tensorflow:tensorflow.bzl", "tf_cc_binary")

   tf_cc_binary(
       name = "example",
       srcs = ["example.cc"],
       deps = [
           "//tensorflow/cc:cc_ops",
           "//tensorflow/cc:client_session",
           "//tensorflow/core:tensorflow",
      ],
   )

I don't think I need all those includes. I will try again without them. Here is the link error for reference:
__cdecl tensorflow::core::GetVarint32PtrFallback unresolved symbol

@dshawul I think this is working now besides that one symbol. It is located in source\tensorflow\core\lib\core. I look at the BUILD file for core and I should be able to add "//tensorflow/core:lib" to get those symbols but that did not work. Are you running inference from a loaded .pb with your app? Seems like the trouble line is when I use TensorShape which your example doesn't use.


PosixPath exception in docker deployment

EdgeDB Version: EdgeDB production and nightly build of docker container
EdgeDB CLI Version:  N/A
OS Version: Docker container from EdgeDb docker hub

Steps to Reproduce:

Deploy the container to remote machine using docker compose
Container reboots after hitting bootstrap exception

This happens with both the production (lastest stable) version as well as the nighly version.  I have not tried any other image.  The ouput from the docker logs is as follows.  I am not sure where to start with the troubleshooting of this. I am using the stable and nightly version directly from the hub (i.e. unmodified).
The error log ends with the last statement below and then the container restarts... starting from the bootstrap message.
================================================================
                          WARNING
                          -------

EDGEDB_SERVER_DEFAULT_AUTH_METHOD is set to 'Trust'.  This will
allow unauthenticated access to this EdgeDB instance for all who
have access to the database port! This might include other
containers or processes on the same host and, if port 5656
is bound to an accessible interface on the host, other machines
on the network.

Use only for DEVELOPMENT and TESTING in a known environment
without sensitive data.  Otherwise, it is strongly recommended
to use password authentication via the EDGEDB_SERVER_PASSWORD
or EDGEDB_SERVER_PASSWORD_HASH environment variables.
================================================================
Bootstrapping EdgeDB instance on the local volume...
INFO 45 - 2024-11-26T21:07:46.101 edb.server: starting Gel server 6.0-dev.9019+041c0cb, built on 2024-11-26T01:35Z from revision 2a9ebc33a (2024-11-25T01:06Z) for aarch64-unknown-linux-gnu
INFO 45 - 2024-11-26T21:07:46.101 edb.server: instance name: '_unknown'
INFO 45 - 2024-11-26T21:07:46.390 edb.server: Using 16 max backend connections based on total memory.
---- Exception occurred: Command '[PosixPath('/usr/lib/aarch64-linux-gnu/edgedb-server-6-dev9019/lib/python3.12/site-packages/edb/../../../../bin/pg_config')]' returned non-zero exit status 1. ----
1. subprocess.CalledProcessError: Command '[PosixPath('/usr/lib/aarch64-linux-gnu/edgedb-server-6-dev9019/lib/python3.12/site-packages/edb/../../../../bin/pg_config')]' returned non-zero exit status 1.
---- Traceback ----
    /usr/lib/aarch64-linux-gnu/edgedb-server-6-dev9019/bin/edgedb-server.py, line 7, in <module>
        > sys.exit(main())
    /usr/lib/aarch64-linux-gnu/edgedb-server-6-dev9019/lib/python3.12/site-packages/click/core.py, line 1157, in __call__
        > return self.main(*args, **kwargs)
    /usr/lib/aarch64-linux-gnu/edgedb-server-6-dev9019/lib/python3.12/site-packages/click/core.py, line 1078, in main
        > rv = self.invoke(ctx)
    /usr/lib/aarch64-linux-gnu/edgedb-server-6-dev9019/lib/python3.12/site-packages/click/core.py, line 1666, in invoke
        > rv = super().invoke(ctx)
    /usr/lib/aarch64-linux-gnu/edgedb-server-6-dev9019/lib/python3.12/site-packages/click/core.py, line 1434, in invoke
        > return ctx.invoke(self.callback, **ctx.params)
    /usr/lib/aarch64-linux-gnu/edgedb-server-6-dev9019/lib/python3.12/site-packages/click/core.py, line 783, in invoke
        > return __callback(*args, **kwargs)
    /usr/lib/aarch64-linux-gnu/edgedb-server-6-dev9019/lib/python3.12/site-packages/click/decorators.py, line 33, in new_func
        > return f(get_current_context(), *args, **kwargs)
    /usr/lib/aarch64-linux-gnu/edgedb-server-6-dev9019/lib/python3.12/site-packages/edb/server/main.py, line 784, in main
        > server_main(**kwargs)
    /usr/lib/aarch64-linux-gnu/edgedb-server-6-dev9019/lib/python3.12/site-packages/edb/server/main.py, line 758, in server_main
        > asyncio.run(run_server(
    /usr/lib/aarch64-linux-gnu/edgedb-server-6-dev9019/lib/python3.12/asyncio/runners.py, line 194, in run
        > return runner.run(main)
    /usr/lib/aarch64-linux-gnu/edgedb-server-6-dev9019/lib/python3.12/asyncio/runners.py, line 118, in run
        > return self._loop.run_until_complete(task)
    uvloop/loop.pyx, line 1518, in uvloop.loop.Loop.run_until_complete

    /usr/lib/aarch64-linux-gnu/edgedb-server-6-dev9019/lib/python3.12/site-packages/edb/server/main.py, line 556, in run_server
        > cluster, args = await _get_local_pgcluster(
    /usr/lib/aarch64-linux-gnu/edgedb-server-6-dev9019/lib/python3.12/site-packages/edb/server/main.py, line 351, in _get_local_pgcluster
        > cluster = await pgcluster.get_local_pg_cluster(
    /usr/lib/aarch64-linux-gnu/edgedb-server-6-dev9019/lib/python3.12/site-packages/edb/server/pgcluster.py, line 978, in get_local_pg_cluster
        > instance_params = pgparams.get_default_runtime_params(
    /usr/lib/aarch64-linux-gnu/edgedb-server-6-dev9019/lib/python3.12/site-packages/edb/pgsql/params.py, line 140, in get_default_runtime_params
        > version = buildmeta.get_pg_version()
    /usr/lib/aarch64-linux-gnu/edgedb-server-6-dev9019/lib/python3.12/site-packages/edb/buildmeta.py, line 173, in get_pg_version
        > pg_config = subprocess.run(
    /usr/lib/aarch64-linux-gnu/edgedb-server-6-dev9019/lib/python3.12/subprocess.py, line 571, in run
        > raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '[PosixPath('/usr/lib/aarch64-linux-gnu/edgedb-server-6-dev9019/lib/python3.12/site-packages/edb/../../../../bin/pg_config')]' returned non-zero exit status 1.
ERROR: Could not complete instance bootstrap

Can you provide a bit more information about your environment please?  Machine type, OS version and Docker version specifically.

Hi @elprans.  I have used an older container (about a year or so ago) of EdgeDB on this OS and SBC, so I know it has worked  before.  This issue was encountered when setting up a new SBC (i.e. unprovisioned SBC) with the latest version of the container.  I don't have access to the board at the moment to confirm specifics but I can provide some details now and follow-up on Friday with confirmation.
I am running the container on an SBC from Toradex (Mallow carrier, Verdin iMX8M+ SOM module) running the default Torizon OS image (i.e. unmodified, provided by Toradex, TorizonCore with evaluation containers, 5.7.2+build.2.0).  This OS image comes with docker installed.
It's an ARM platform, thus the aarch64 version of EdgeDB.  I am deploying the lastest container (unmodified) from EdgeDB via a docker compose file that is setup as follows:
version: '3'
name: oxygen-stack
services:
 edgedb:
    image: edgedb/edgedb:nightly
    container_name: edgedb
    environment:
      - EDGEDB_SERVER_SECURITY=insecure_dev_mode
    volumes:
      - type: volume
        source: edgedb-data
        target: /var/lib/edgedb/data
      - type: volume
        source: dbschema
        target: /dbschema
    ports:
      - "5656:5656"
    restart: unless-stopped
volumes:
  dbschema:
  edgedb-data:

Can you add these to the environment and attach the full log here please:
environment:
  - EDGEDB_DOCKER_LOG_LEVEL=trace
  - EDGEDB_SERVER_LOG_LEVEL=d

Sorry for the delay.  The updated information... the board I am using is from Toradex and it is running TorizonCore 5.7.2+build.20 (dunfell) aarch64.  The processor is Verdin iMX8M Plus WB on the Mallow carrier board.  The kernel version is listed as 5.4.193-5.7.2+git.b60d3160fd04.
I added the environment vars you mentioned above and set the container to not restart.  This is the log file that it generated.
_edgedb_logs-2.txt

Update to this... I tried creating containers with edgedb images that were tagged as 2, 3, 4, and 5.  It looks like the latest is v6.  Anyway, v2, 3, and 4 work just fine.  The container is created, the server runs, and I can do an inital migration to create the tables in my dbschema file.  Version 4 and 5 do not work, and generate the same errors and restart behavior seen when using the "latest" tag.  I am not sure if this helps somewhat.


[Bug][UI Next][V1.0.0-Alpha]  The Create Time and Update Time fields returned an error

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
The Create Time and Update Time fields returned an error

What you expected to happen
Show the correct
How to reproduce
I created the workflow at 5pm and the back end returned to create and update at 9am
Anything else
No response
Version
dev
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I preliminarily judged that this problem may be caused by the time zone. The back-end time has been processed, but the front-end has not yet started. This is a data incompatibility point.


[xteve] PVC Permission denied

Because of USER kah in dockerfile pvc container can't create folders in /config
/config/backup/ permission denied (Folder could not be created.)


merge from arlac77/npm-package-template-esm-only

package.json

chore(package): add ^16.0.1 remove ^16.0.0 (devDependencies.semantic-release)

Codecov Report

Merging #642 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #642   +/-   ##
=======================================
  Coverage   93.99%   93.99%
=======================================
  Files           4        4
  Lines         283      283
  Branches       25       25
=======================================
  Hits          266      266
  Misses         17       17

Continue to review full report at Codecov.

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

:tada: This PR is included in version 8.1.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Helm values for subchart are not rendered properly

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
When using the official Helm chart for Cilium as a subchart, some values are not rendered properly.
Setup:

Create a new chart that has the official Cilium chart as a dependency:

apiVersion: v2
name: my-cilium
type: application
version: 0.0.0
dependencies:
  - name: cilium
    version: 1.12.6
    repository: https://helm.cilium.io

Provide some custom values for the subchart according to the Helm rules:

cilium:
  operator:
    replicas: 4
    image:
      repository: "my-repo.com/cilium-operator"

Template the parent chart:

helm template cilium . --show-only charts/cilium/templates/cilium-operator/deployment.yaml

Expected results:
Cilium operator deployment is rendered with parent chart values taken into account e.g.:
---
# Source: cilium/charts/cilium/templates/cilium-operator/deployment.yaml
apiVersion: apps/v1
kind: Deployment
metadata:
... OMITED ...
spec:
  replicas: 4
... OMITED ...
    spec:
      containers:
      - name: cilium-operator
        image: "my-repo.com/cilium-operator-generic:v1.12.6@sha256:eec4430d222cb2967d42d3b404d2606e66468de47ae85e0a3ca3f58f00a5e017"        imagePullPolicy: IfNotPresent
 ... OMITED ...

Actual results:
Default values are applied:
---
# Source: cilium/charts/cilium/templates/cilium-operator/deployment.yaml
apiVersion: apps/v1
kind: Deployment
metadata:
 ... OMITED ...
spec:
  replicas: 2
 ... OMITED ...
    spec:
      containers:
      - name: cilium-operator
        image: "quay.io/cilium/operator-generic:v1.12.6@sha256:eec4430d222cb2967d42d3b404d2606e66468de47ae85e0a3ca3f58f00a5e017"
        imagePullPolicy: IfNotPresent
 ... OMITED ...

Cilium Version
Cilium Helm chart version: 1.12.6
Cilium version: v1.12.6 (irrelevant for this issue)
Kernel Version
irrelevant for this issue
Kubernetes Version
Kuberentes version: v1.23.15
Helm version: v3.11.0
Sysdump
No response
Relevant log output
No response
Anything else?
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Please confirm if you have passed the custom value as part of template command correctly.
$ cat temp_values.yaml
cilium:
  operator:
    replicas: 4
    image:
      repository: "my-repo.com/cilium-operator"

$ helm template cilium . -f temp_values.yaml --show-only charts/cilium/templates/cilium-operator/deployment.yaml | grep image
        image: "my-repo.com/cilium-operator-generic:v1.12.6@sha256:eec4430d222cb2967d42d3b404d2606e66468de47ae85e0a3ca3f58f00a5e017"
        imagePullPolicy: IfNotPresent

Never mind. We have some custom configuration in our values and there was a YAML syntax problem. I've tried to run a few other tests and everything works as is should.
Sorry, for bothering you. Closing this issue.


On zoom panel the icon still selected even the user selected outside the icon

SEVERITY: Minor
MODULE: Bubble Chart
Platform:

Windows 7 / Firefox 45.0
Windows 7 / Chrome 49.0

RELEASE: v0.13.4-4
STEPS TO REPRODUCE:

Navigate to the URL: http://tools-stage.gapminderdev.org/tools/bubbles
Bubble Chart will be loaded.
Hover the mouse below the zoom in icon
Click and observe the icon gets selected

ACTUAL RESULTS:
The icon gets selected when user clicks under the zoom in or zoom out button
EXPECTED RESULTS:
The icon should only be selected if the mouse is over the icon and user clicked
SCREENSHOT:

it's a feature!
the click areas are a little bigger, so we don't require user to perform a precise action
it helps especially on touch devices if a touch is recognised a bit away from the target


Ops search results are no longer present

In the current version of napari-imagej, I'm not seeing any Ops search results, only Commands and Script templates:

The Commands results do have ops mixed in, because ImageJ Ops v1 Op plugins are Commands, but they won't work the same as when they are invoked via the OpListing mechanism.

@ctrueden what happens when you run, in the napari console:
from imagej import ij; ij.prefs().clearAll();

Does that fix the issue?
Just wondering if it is related to the PrefService at all.

fwiw I get different results:

@ctrueden had the idea that this may be tied to the checkboxes in fiji; if they become unchecked, categories of SearchResults might become unavailable in napari-imagej.
This seems like it might be the culprit. I was able to reproduce, taking the following steps:

Open Fiji
Search something, and then uncheck Ops
Open napari-imagej
Search something, observe no Ops results
Open the napari console, and type from imagej import ij; ij.prefs().clearAll()
Search something, and see that Ops appear again.


Fix profile name in default storage class example

Fix the profile name in the default storage class example.

@tksh164 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

Thanks for catching this issue @tksh164 !


Patch bug in HomeroomTable showing dates of notes

Who is this PR for?
K8 teachers
What problem does this PR fix?
Note dates don't work on HomeroomTable, a bug in https://github.com/studentinsights/studentinsights/pull/1931.
What does this PR do?
Fixes the bug.
Screenshot (if adding a client-side feature)
before

after

Checklists

[x] Author improved specs for code in need of better test coverage

selfie


Check for argument conversion raises TypeError: Subscripted generics cannot be used with class and instance checks (regression!)

With 4.0.1 I am seeing this error that does not occur on 4.0.0:
Unexpected error: TypeError: Subscripted generics cannot be used with class and instance checks
Traceback (most recent call last):
  File "c:\d\hpha-robot\.tox\py38\lib\site-packages\robot\utils\application.py", line 83, in _execute
    rc = self.main(arguments, **options)
  File "c:\d\hpha-robot\.tox\py38\lib\site-packages\robot\run.py", line 447, in main
    result = suite.run(settings)
  File "c:\d\hpha-robot\.tox\py38\lib\site-packages\robot\running\model.py", line 296, in run
    self.visit(runner)
  File "c:\d\hpha-robot\.tox\py38\lib\site-packages\robot\model\testsuite.py", line 226, in visit
    visitor.visit_suite(self)
  File "c:\d\hpha-robot\.tox\py38\lib\site-packages\robot\model\visitor.py", line 85, in visit_suite
    if self.start_suite(suite) is not False:
  File "c:\d\hpha-robot\.tox\py38\lib\site-packages\robot\running\suiterunner.py", line 72, in start_suite
    ns.handle_imports()
  File "c:\d\hpha-robot\.tox\py38\lib\site-packages\robot\running\namespace.py", line 55, in handle_imports
    self._handle_imports(self._imports)
  File "c:\d\hpha-robot\.tox\py38\lib\site-packages\robot\running\namespace.py", line 66, in _handle_imports
    self._import(item)
  File "c:\d\hpha-robot\.tox\py38\lib\site-packages\robot\running\namespace.py", line 74, in _import
    action(import_setting)
  File "c:\d\hpha-robot\.tox\py38\lib\site-packages\robot\running\namespace.py", line 127, in _import_library
    lib = IMPORTER.import_library(name, import_setting.args,
  File "c:\d\hpha-robot\.tox\py38\lib\site-packages\robot\running\importer.py", line 46, in import_library
    lib = TestLibrary(name, args, variables, create_handlers=False)
  File "c:\d\hpha-robot\.tox\py38\lib\site-packages\robot\running\testlibraries.py", line 53, in TestLibrary
    lib = libclass(libcode, name, args or [], source, logger, variables)
  File "c:\d\hpha-robot\.tox\py38\lib\site-packages\robot\running\testlibraries.py", line 89, in __init__
    = self.init.resolve_arguments(args, variables)
  File "c:\d\hpha-robot\.tox\py38\lib\site-packages\robot\running\handlers.py", line 93, in resolve_arguments
    return self.arguments.resolve(args, variables)
  File "c:\d\hpha-robot\.tox\py38\lib\site-packages\robot\running\arguments\argumentspec.py", line 87, in resolve
    positional, named = converter.convert(positional, named)
  File "c:\d\hpha-robot\.tox\py38\lib\site-packages\robot\running\arguments\argumentconverter.py", line 30, in convert
    return self._convert_positional(positional), self._convert_named(named)
  File "c:\d\hpha-robot\.tox\py38\lib\site-packages\robot\running\arguments\argumentconverter.py", line 34, in _convert_positional
    converted = [self._convert(name, value)
  File "c:\d\hpha-robot\.tox\py38\lib\site-packages\robot\running\arguments\argumentconverter.py", line 34, in <listcomp>
    converted = [self._convert(name, value)
  File "c:\d\hpha-robot\.tox\py38\lib\site-packages\robot\running\arguments\argumentconverter.py", line 58, in _convert
    return converter.convert(name, value)
  File "c:\d\hpha-robot\.tox\py38\lib\site-packages\robot\running\arguments\typeconverters.py", line 91, in convert
    if self._no_conversion_needed(value):
  File "c:\d\hpha-robot\.tox\py38\lib\site-packages\robot\running\arguments\typeconverters.py", line 461, in _no_conversion_needed
    return isinstance(value, self.types)
  File "C:\Python38\lib\typing.py", line 769, in __instancecheck__
    return self.__subclasscheck__(type(obj))
  File "C:\Python38\lib\typing.py", line 777, in __subclasscheck__
    raise TypeError("Subscripted generics cannot be used with

It appears that the cause of the error is that an argument is a Union but the components of this union are indexed types. The robot framework code does an isinstance check for these types, but that is not allowed. Before making an instance check we need to find the non-indexed base type.
I believe the type in question is:
typing.Union[typing.Callable[[hpha_di.api._internal.context.Injectable[typing.Any]], hpha_di.api._internal.provider.Provider[typing.Any]], typing.AbstractContextManager[typing.Callable[[hpha_di.api._internal.context.Injectable[typing.Any]], hpha_di.api._internal.provider.Provider[typing.Any]]]]

This regression is caused by #3897. Shouldn't be overly complicated to fix. With generics we should try to find the base type, as proposed, but we probably should also handle possible other errors gracefully as well.
This is a pretty severe issue so we probably need to release 4.0 2 sooner than originally planned.

This ought to now be fixed. I'll create RF 4.0.2 beta 1 as soon as other issues targeted for 4.0.2 are fixed which could be already tomorrow. I hope you @Carsten-Leue can test the fixes latest then.

Turned out that also TypedDict fails when used with isinstance (#3969). The fix for generics handled only them and actually made the situation with TypedDict even worse than it was earlier. This problem has now been fixed in more generic manner.


D435i Frames didn't arrived within 5 seconds

Required Info

Camera Model
D435i

Firmware Version
<IP_ADDRESS>

Operating System & Version
Ubuntu 20.04

Kernel Version (Linux Only)

Platform
laptop

SDK Version
2.54.1

Issue Description
I'm attempting to use the D435i on Ubuntu 20.04 by Realsense Viewer. When I attempt to connect to the camera I get the "Frame Didn't arrive within 5s" error.

Hi @andyliao89  I note that the drop-down menu for selecting a Visual Preset camera configuration option in the side-panel is missing.

Does the problem still occur if you reset the settings of the RealSense Viewer to their defaults using the instructions at the lnk below, please?
https://support.intelrealsense.com/hc/en-us/community/posts/360052013914/comments/1500000416522

Hi @andyliao89 I note that the drop-down menu for selecting a Visual Preset camera configuration option in the side-panel is missing.

Does the problem still occur if you reset the settings of the RealSense Viewer to their defaults using the instructions at the lnk below, please?
https://support.intelrealsense.com/hc/en-us/community/posts/360052013914/comments/1500000416522

I've tried it and it doesn't work.

Hi @andyliao89 I note that the drop-down menu for selecting a Visual Preset camera configuration option in the side-panel is missing.

Does the problem still occur if you reset the settings of the RealSense Viewer to their defaults using the instructions at the lnk below, please?
https://support.intelrealsense.com/hc/en-us/community/posts/360052013914/comments/1500000416522

After I clicked the advanced mode, the drop-down menu showed up.

Thank you. Are you using the official 1 meter USB cable supplied with the camera or a USB cable of your own choice?

I'm using the official ones.

Has RGB previously functioned correctly and only stopped working today, or has it never worked since the camera was purchased?

Is the RGB problem still present if the computer is shut down completely and then restarted with the power button?

Does unplugging the camera, waiting a couple of seconds and then plugging it back in correct it?

Is RGB still absent if the rs-capture example program is launched instead of the Viewer?

Has RGB previously functioned correctly and only stopped working today, or has it never worked since the camera was purchased?
Is the RGB problem still present if the computer is shut down completely and then restarted with the power button?
Does unplugging the camera, waiting a couple of seconds and then plugging it back in correct it?
Is RGB still absent if the rs-capture example program is launched instead of the Viewer?

1.It has never worked since the camera was purchased.
2.Yes.
3.No.
4.I get nothing if the rs-capture example program is launched.

I will discuss your RGB issue with my Intel RealSense colleagues.  Thanks very much for your patience.

How long ago was the camera purchased, please?  If it is still within the warranty period then you can contact the retailer that it was purchased from in order to perform an RMA return of the camera to the retailer.

I will discuss your RGB issue with my Intel RealSense colleagues. Thanks very much for your patience.

Thank you and look forward to your reply.
But now I have a new problem, is there any solution?

I replied above at https://github.com/IntelRealSense/librealsense/issues/12220#issuecomment-1734016033 to ask when your camera was purchased.  Thanks very much.

I replied above at #12220 (comment) to ask when your camera was purchased. Thanks very much.

I bought it in February this year.

There are sometimes warnings and errors at launch and the camera still works, but a Right MIPI Error like the one that you received can indicate that there is a more serious problem involving the hardware inside the camera.

Hi @andyliao89  Do you require further assistance with this case, please?  Thanks!

Thank you very much !


sdk(rust): Use same Apache 2.0 license as all other SDKs

Engine & CLI use the same Apache 2.0 license.
If you agree with this change @kjuulh, please:

edit the commit message & remove this paragraph
append your Signed-off-by
commit & force push

Otherwise, let's talk!

Will do so tomorrow xD

@gerhard done 🚀

Thank you @kjuulh 💪


Impact of cache on e2e latency

Suppose we cannot keep all of the states of stateful operators in memory. For hash aggregation or hash join, we would fetch states from S3 by the key of each row in the stream chunk.
In the current implementation, the latency of a stream chuck processed by one stateful operator would be determined by the largest latency of one of its rows. All the rows of a stream chunk processed by the operator is a necessary condition for the operator to emit output downstream.
Suppose for each row, the cache hit rate is P = 95%. With a stream chunk of N = 32, 64, 128 rows, the probability of all rows' states in cache is P^N = 19%, 3%, 0.1%. In spite of a very high 95% cache hit rate, the whole chunk is still very likely to be blocked by fetching states from S3 due to a small number of rows missing their corresponding states in the cache.
One immediate thought is to just use a smaller chunk size, i.e. 32. There are several implications:

Does a smaller chunk size hurts the effectiveness of vectorization? The amortization gets worse.

Compare to a chunk size of 64, a 64-size chunk is now split into two 32-size chunks. Although each 32-size chunk is less likely to be blocked by cache miss, the latency in expectation X added to all the chunks behind these two chunks is not clear. Suppose the latency of fetching states for a 64-size chunk and 32-size chunk is M and N respectively:
if the chunk size is 64, X = (1-3%)M = 0.97M
if the chunk size 32, X = (1-19%)^2 * 2N + 2 * (1-19%) * 19% * N = 1.62N
Does M = 2*N because 64 = 2*32? I don't know but I guess not very likely due to some constant overhead.

One clear advantage thought is that if the first 32-size chunk does not need to fetch states, it will be sent downstream and thus gets pipelined earlier. If there are multiple stateful operators in the job, we may consider that both 32-size chunks are fetching states at the same time but at different operators. Therefore, what is described in (2) would happen less frequently at some downstream operators because there will not be several stream chunks waiting in line to get processed by the same operator. But I am not able to quantify this advantage.

Notice what we really care about is the latency of each epoch barrier but not each individual stream chunk as we only allow the users to query when one epoch finishes. Allowing stream chunks to be processed out of order may alleviate the problem. Of course, there could be some constraints on whether two stream chunks can be reordered, e.g. within the same epoch, across two different epochs. This is because the epoch may represent some timestamp/transaction from a database source so that our system can inherit the isolation level/serializability from that source.

Just some early ideas.

Since this issue is mentioned in #2428, and a potential solution #2897 of reordering rows subsumes reordering chunks, so close this issue now.


what about cdn urls in precache? Latest info.

Library Affected:
workbox-sw, workbox-build, etc.
I've seen lots of discussions about adding CDN origin for assets by @jeffposnick . Reading them all will confuse me more. So if it's possible, I will ask what's the final situation about this case in a workbox way?
Q1) I am using manifestTransforms and add my functions in it so that I transform urls myself. I add CDN if I want to or not. Is this how you decided to finally solve this problem?
Q2) Really interesting thing is that using CDN for assets might lead to one problem. Even though folks manage CDN themselves, it's possible that on CDN , one of the file mightn't be present, but it will still be put in our manifest. Now, if the urls start with CDN origin, when the service worker install starts, it's gonna fail for that single url which will cause failure of SW installation.  I use precacheAndRoute, but as far as I can see, if there's an error related to even file from 100 files, SW doesn't get installed, right?  So this seems a little bit risky for CDN Urls. Is this something you have thought about too and found no other way than just developers deciding to follow that risk or not ?

I don't think that there's any difference between the approach here whether the URLs in your precache manifest are served from a CDN or server from your origin server.
The recommended deployment flow, whether you're deploying to a CDN or to an origin server, is to regenerate your service worker's precache manifest as part of your build process, and then push out that latest service worker as part of the same deployment that includes the corresponding updated assets.
One thing specific to a CDN is that you might need to purge your CDN's cache of unversioned URLs, like service-worker.js, if your CDN is also responsible for serving them.
Regarding your second question, the idea behind the precache manifest is that all of the URLs listed need to be available and updated to the corresponding versions in order for the new service worker's installation to succeed. You're correct that any missing URLs, even 1 out of 100, will cause the installation to fail and the new service worker will become redundant. Installation will then be re-attempted the next time the service worker update flow happens.
I don't see why failures should be treated differently when the URLs are loaded from a CDN, and if you would prefer that missing URLs don't cause installation failure, you should use runtime rather than precaching for those URLs.
(If there's something unreliable about your CDN deployment process that causes URLs not to be present following a deployment, that sounds like an issue that you should fix regardless of whether you're using a service worker or not.)


Display the port value

Before:
Hello from Cloud Run! The container started successfully and is listening for HTTP requests on $PORT

After:
Hello from Cloud Run! The container started successfully and is listening for HTTP requests on port 8080

Wrong default repo. See https://github.com/GoogleCloudPlatform/cloud-run-hello/pull/101 instead


Updated to 2.4.1 tag

Checklist before submitting

[x] Did you read the contributor guide?
[ ] Did you update any affected documentation?
[ ] Did you write any tests to validate this change?

Description
Refreshed to 2.4.1 tag
Review process to land

All tests and other checks must succeed.
At least one maintainer must review and approve.
If any  maintainer requests changes, they must be addressed.

@jayfurmanek - Merging this one too.


Don't import two things to the same Html alias

This is confusing. I've had people ask me and ask the internet (http://stackoverflow.com/questions/38495352/import-html-app-as-html). When someone is learning it's probably best to avoid the issue of aliasing functions verses namespaces, unless it's the subject of the lesson (which it isn't here)

Thanks, I have been trying to hunt down all the places where it is this way and change it to be this.
I am also changing things a bit in 0.18 to make this problem no longer come up, so the 0.18 branch of this repo is already updated to not have this potential for confusion in the first place.

sounds great @evancz!


Reactive datetime picker does not work with   reactive_forms v^12.0.0

I get the following error when I install dependencies for my project
Because reactive_date_time_picker >=0.2.0 depends on reactive_forms ^11.0.0 and builder_app depends on reactive_forms ^12.0.0, reactive_date_time_picker >=0.2.0 is forbidden. So, because builder_app depends on reactive_date_time_picker ^0.2.2, version solving failed.
Please update the dependency of the package so that it can be used with the latest reactive form.

Use dependancy overrides until update

use latest reactive_date_time_picker: ^0.3.0


Add German translation

Adds German translation
Adds option in Settings modal
Adds German language option to README

Fixes #305

It is recommended to add the following code in ``src/Resources/Locales/de_DE.axaml` line 2 to fall back to English if there is a missing translation.
  <ResourceDictionary.MergedDictionaries>
    <ResourceInclude Source="avares://SourceGit/Resources/Locales/en_US.axaml"/>
  </ResourceDictionary.MergedDictionaries>


Add String cast

Closes #56
This issue was blocking me from building for Android 12 so I went ahead and opened a PR so it is easily usable for those who need it. 🥂

Hey @emmett-deen, thanks for raising this PR. I ended up raising #59 to fix the issue and refactored the code a bit. So I'm going to close this PR.


Factoring out some common code in proto3 python client.

@dumbunny

@dumbunny PTAL, it also includes your changes.

LGTM