Feature Request : Modify DropdownMenu to extend TextFormField to add validation

Use case
Currently the built-in Dropdown Menu widget is extending TextFeild class which doesn't support validation . I would i nice to have validation since i use this widget a lot and its good to have
Proposal
final Widget textField = TextFormField(
key: _anchorKey,
mouseCursor: effectiveMouseCursor,
canRequestFocus: canRequestFocus(),
enableInteractiveSelection: canRequestFocus(),
textAlignVertical: TextAlignVertical.center,
validator: widget.validator,
style: effectiveTextStyle,
controller: _textEditingController,
        i have modified the widget to extend TextFormField which supports validation

Have you tried https://api.flutter.dev/flutter/material/DropdownButtonFormField-class.html?

Closing this as a duplicate of https://github.com/flutter/flutter/issues/152131

yes i have tried it . it doesnt have the type to filter or search
functionality . so when you have a long list of suggestions its makes sense
to use the autocomplete widget where you can type to filter out the
suggestions. So my request is still a valid one
On Thu, 19 Sept 2024 at 11:46, Aawaz Gyawali @.***>
wrote:

Have you tried
https://api.flutter.dev/flutter/material/DropdownButtonFormField-class.html
?
—
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/155390#issuecomment-2360081976,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BDEPUV7ZGHYKZICYO44OFKTZXJT3FAVCNFSM6AAAAABOPDVNMWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRQGA4DCOJXGY
.
You are receiving this because you authored the thread.Message ID:
@.***>


Arduino Uno R3

I am working on an Arduino Uno R3 which i had bought from  http://www.w11stop.com/ard-uino  . I wanted to have the ability of controlling the Arduino wirelessly, like stopping the Arduino without having to pull the power-plug every time. Is it possible?

The forum...
http://forum.arduino.cc/index.php?topic=471126.0
...is the appropriate place for your question.


Torch Size Mismatch

Hey guys!
I had a quick issue while loading Magma from the checkpoint, and I was wondering if anyone encountered or knows how to solve the problem.
RuntimeError: Error(s) in loading state_dict for Magma:         size mismatch for lm.lm_head.weight: copying a param with shape torch.Size([50400, 4096]) from checkpoint, the shape in current model is torch.Size([50258, 4096]).
It seems like the size of the checkpoint model differs from the size of the model it is expecting from the rest of the code.
Thank you so much--this model looks super cool and I'm excited to use it!

Hi,
this is likely because you use the wrong transformers version. See https://github.com/Aleph-Alpha/magma/issues/27
Best,
Constantin


Ear 944 mes v4 changes

What is the context of this PR?
Fix an issue with the links implemented within MES.
How to review
Describe the steps required to test the changes (include screenshots if appropriate).
Checklist

[ ] New static content marked up for translation
[ ] Newly defined schema content included in eq-translations repo

Broken


ADD: Gas limit option

added gas limit option to validator and execution clients

@PatrickRL I'd start with the stuff @gbayasgalan mentioned. :)


what is the difference to quibble?

see: https://github.com/testdouble/quibble
It seems that the functionality is the same, isn't it?

@TimDaub esmock uses separated mock definitions for each module and I don't think quibble does that. One process can run simultaneous tests importing the same module but with different mock definitions. Esmock locates the original module from the filesystem and maps mocked values into the originally exported ones and quibble's author thinks that is a bad practice.
There could be other differences but I'm not familiar with quibble since researching it several months ago.

With quibble, I ran into issues where in ava test - if I used quibble to mock a module in one test of a file, but then used that module in another test too - it'd now allow me to run those tests in parallel. Instead the program would start throwing errors as the mocking isn't isolated to one test but needs to be reset with quibble.reset.
Am I understanding your comment correctly that esmock fixes this problem?

@TimDaub yes. And esmock will not have that problem. I believed quibble would have problems in that situation but haven't tested it myself. Thanks for sharing :) I'm glad to know about it.

my earlier comment isn't clear and maybe this example is better
const mockedInst1 = await esmock( '../src/test.js', { '../src/util.js': { isHappy: () => true } } );
const mockedInst2 = await esmock( '../src/test.js', { '../src/util.js': { isHappy: () => false } } );

mockedInst1 and mockedInst2 are separated instances with separately imported dependencies, even though their origin file is same.


minimization - Support additional frameworks

E.g., pytorch, Keras...

Consider postponing this until we know what we are doing with model wrappers (this may be redundant).


Splash screen in the WinUI 3 version

The WinUI 3 version is still pretty slow to startup. Consider having a splash screen instead of a blank background.

Winui 3 doesn't support splash screen and start up performance is being improved to hopefully not need one.

Winui 3 doesn't support splash screen and start up performance is being improved to hopefully not need one.

You can create your own yourself.

I don't think there would be much of a point since start up time would be the same anyway. Also when #9837 is implemented a splash screen wouldn't be needed. I'll leave it to @yair2 to say

We don't have plans to implement a splash screen, we're focused on improving performance and stability. Trimming and AOT support will help with startup speed as well.


Respect :selected_option_class when in :single mode

Per the docs, the :selected_option_class attr is only available when in :tags mode. However, I believe being able to style the current selection uniquely is valuable even in :single mode.
This is in fact how chrome's built-in select works (notice the check next to female, indicating that is the current selection):

If you're open to this change, I'd be happy to PR it. Just don't want work to go to waste.

Hi @AHBruns and thanks for pointing this out. I agree with your observation, and I'd welcome a PR for this change!
Just be aware that I'll be traveling for the next couple of weeks with little time to spend for coding. So it might take a while before I can review your PR, but I'll do it as soon as I find some time!
Thanks

I'm up against a tight deadline at work this week, but I will have time next week. Planning to PR this then assuming it's fairly straightforward.

This is no longer the case, the styles for the elements in the drop down work irrespective of the mode. The docs need to be updated


Add make commands to generate Thrift classes for crossdock

lgtm

Coverage remained the same at 73.569% when pulling 444f868aac774165eb169e4a18fd88b0743ab173 on gen-crossdock-thrift into aff9e746a9bd13d5551b54d8dd8248691d0e9d2f on master.


[LUNE] - Base setup and global components

Hi @camelburrito
As requested we are splitting up the previous PR and starting off with the base setup and global navigation.

Any suggestion on how to confirm I'm ok with my commits contributing to this PR? (following the googlebot message)

@WillDonohoe - do you have a url to test this, also looks like you might have merge conflicts.

@camelburrito A link to staging for this PR can be found here: https://gweb-amp-template-lune-review.firebaseapp.com/templates/lune/lune.amp

Any suggestion on how to confirm I'm ok with my commits contributing to this PR? (following the googlebot message)

@erwinmombay  - do you know of any?

Hey @camelburrito, could you please take a look at the latest changes? We'd really need this PR to be merged before the weekend, and we could address any further feedback (if you have any) in the next hour or so before our working day ends. Thanks!

I give you my consent to merge this PR containing my commits

I give consent for @ciampo and my commits to be merged

This was not possible for us to merge it, can you resolve merge conflicts and also send me a new pr with just one author committing

We're happy for this PR to be closed in favour of #515

We merged it on a different PR , so closing this


Fixed checking if handler is registered

For https://github.com/StarcounterApps/Website/issues/108.
Also removed double registering of the "/WebsiteProvider_AcceptanceHelperOne/ResetData" URI - strange thing that in 2.3.1 it did not appear.

Closing based on this comment https://github.com/StarcounterApps/Website/issues/108#issuecomment-307590002

@bouguila could you please if you can verify if you know how to reproduce the problem described in #108 and then check if this PR fixes it?

LGTM! the app starts without any problem on this PR

@un-tone pls merge


Error when submit form with asyncValidate and asyncChangeFields

Are you submitting a bug report or a feature request?

Bug report
What is the current behavior?

When submit form with asyncValidate and asyncChangeValues  got error because of that line:
https://github.com/erikras/redux-form/blob/master/src/handleSubmit.js#L95
asyncValidate called with no arguments, so here
https://github.com/erikras/redux-form/blob/master/src/createReduxForm.js#L684-L689
after webpack build conditions are inlined:
var fieldNeedsValidation =
  !submitting &&
  trigger === 'blur'
    ? !asyncBlurFields || ~asyncBlurFields.indexOf(name.replace(/\[[0-9]+\]/g, '[]'))
    : !asyncChangeFields || ~asyncChangeFields.indexOf(name.replace(/\[[0-9]+\]/g, '[]'));

trigger is not blur (its undefined, like all other args), asyncChangeFields is array
What is the expected behavior?
Do not throw error?
Sandbox Link

Example: JSFiddle
What's your environment?

React 16.2.0
Redux-Form 7.2.0
Ubuntu 17.10 Google Chrome 62.0.3202.94
Other information

Error stacktrace
Uncaught TypeError: Cannot read property 'replace' of undefined
    at Form._this.asyncValidate (VM731 redux-form.js:7774)
    at handleSubmit (VM731 redux-form.js:9630)
    at Form._this.submit (VM731 redux-form.js:7816)

UPD: My bad, code is ok, looks like it's babel/webpack issue?
source:
            const fieldNeedsValidation =
              !submitting &&
              (formHasAsyncFields ||
                (trigger === 'blur'
                  ? fieldNeedsValidationForBlur
                  : fieldNeedsValidationForChange))

after translation no parentheses (in JSFiddle too):
var fieldNeedsValidation = !submitting && trigger === 'blur' ? !asyncBlurFields || ~asyncBlurFields.indexOf(name.replace(/\[[0-9]+\]/g, '[]')) : !asyncChangeFields || ~asyncChangeFields.indexOf(name.replace(/\[[0-9]+\]/g, '[]'));

I'm having the same issue and see the same error stacktrace.
Uncaught TypeError: Cannot read property 'replace' of undefined
    at Form._this.asyncValidate (VM731 redux-form.js:7774)
    at handleSubmit (VM731 redux-form.js:9630)
    at Form._this.submit (VM731 redux-form.js:7816)

I had the same issue when using 7.2.0, had to revert my project to 7.1.2 since the issue was introduced in 7.2.0. This was introduced with #3531.


Build Docker images for linux/arm64 as well

While Macbooks with M1 can run linux/amd64 images, it is unbearably
slow. Building and publishing a linux/arm64 image provides a much
better experience.

Looks like a custom builder might help https://github.com/docker/buildx/issues/495#issuecomment-761562905

@vieux thanks, I'm trying with a suggestion from that thread

Unfortunately, just like using docker images for amd64 is too slow, running docker arm64 on a amd64 server (GitHub action) is just as slow, and the CI hits the hard limit of 6 hours per job.
I think for now I’ll just see if I can push the ARM image manually into the registry. For CI, we will have to either wait for GitHub to provide native ARM runners, or to cross-compile the ARM->MIPS compiler from amd64, which is a complex modification of the build script. Either way, this PR won’t fly for now.


tickets/DM-42598: get PyPi releases going, and replace pip with UV

Also enable trusted publisher and try to get a PyPi release.

This (and let's fact it, this wasn't much of a PR--basically just new content to test the release process) has been running for me successfully this weekend, so now I am going to bump the version number, merge it, and see if we get a release posted to ghcr.io.

Since we have uv we can almost certainly drop pip-tools, but let's leave that for a bit.


i10 User can edit the environment variable PATH

#10

Codecov Report

Merging #11 into master will increase coverage by 20.32%.
The diff coverage is 83.6%.

@@             Coverage Diff             @@
##           master      #11       +/-   ##
===========================================
+ Coverage   64.28%   84.61%   +20.32%
===========================================
  Files           3        3
  Lines          14       65       +51
===========================================
+ Hits            9       55       +46
- Misses          5       10        +5

Impacted Files
Coverage Δ

...rc/PathEditor.UnitTests/MainWindowViewModelTest.cs
100% <100%> (ø)

src/PathEditor.Core/MainWindowViewModel.cs
75.75% <72.41%> (+75.75%)
:arrow_up:

src/PathEditor.Core/RelayCommand.cs
80% <80%> (ø)

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 2ba460d...3e56ef0. Read the comment docs.


build: pull ceph base image from quay.io

pull the ceph image from quay.io instead of docker hub.
From ceph doc, the images are available in both quay and dockerhub https://docs.ceph.com/en/latest/install/
containers/#official-releases but the latest images are not updated in the docker hub.
closes #2796
depends on:  https://github.com/ceph/ceph-csi/pull/2797
Signed-off-by: Madhu Rajanna<EMAIL_ADDRESS>

@mergifyio backport release-v3.5


feat(quick-spin-docs): Marketing blurb back - and some other fixes and edits

Resolves Jira: [https://armory.atlassian.net/browse/SPLAT-359]

The title of this Pull Request needs to be changed to match the format:
<type>(<scope>): <subject>

See more details here: https://www.spinnaker.io/community/contributing/submitting/#commit-message-conventions
Some common examples are:
feat(armory.io): add docs.armory.io
fix(parser): grammar and spelling
style(loadingPage): made it super pretty!
refactor(helpMessages): changed order to make more sense

The title of this Pull Request needs to be changed to match the format:
<type>(<scope>): <subject>

See more details here: https://www.spinnaker.io/community/contributing/submitting/#commit-message-conventions
Some common examples are:
feat(armory.io): add docs.armory.io
fix(parser): grammar and spelling
style(loadingPage): made it super pretty!
refactor(helpMessages): changed order to make more sense

The title of this Pull Request needs to be changed to match the format:
<type>(<scope>): <subject>

See more details here: https://www.spinnaker.io/community/contributing/submitting/#commit-message-conventions
Some common examples are:
feat(armory.io): add docs.armory.io
fix(parser): grammar and spelling
style(loadingPage): made it super pretty!
refactor(helpMessages): changed order to make more sense

merging up changes from meeting and morning effort.


LayerNorm/GatedRMS inconsistency

Hi!
looking through pipeline it seems there are some inconsistencies with normalisation
# ReLA
input to GRMSNorm
# att code
output: Linear(inner_dim, dim) + GRMSNorm
# next in FF module
input to LayerNorm

here we have problem with double norm since we have last layer GRMSNorm in att and first layer LayerNorm in FF.
looking at the paper it seems that in ReLA GRMSNorm is applied to result of mult(attn, v) before output projection not after projection like in this code.
I also confused about usage of LayerNorm in FF should it be GRMSNorm instead? not clear from the paper as well

Aha I see, yup i remember sandwich norm paper :)
another difference I noticed: you use projection based gating (with Linear layer) in GRMSNorm, while original paper is using simple per element multiplication here: return normed_x *(x*gate).sigmoid() where gate = nn.Parameter(torch.tensor(dim))

@inspirit ohh apologies, yea, i didn't build that correctly https://github.com/lucidrains/rela-transformer/commit/b58b121e59463b5c73fa5dea9297c841b1a3a362
let me know if that works! i've seen the relu based attention in another recent paper https://github.com/lucidrains/FLASH-pytorch , so maybe there's something to it!

@inspirit how did it go? :) any interesting experimental results?

it seems to be less stable compared to normal softmax attention, I fused it with preceiver for my experiments, sometimes it gives slightly better results sometimes not :) the reason might be due to a small model inner dimension (128) and more sparse attention due to ReLu use

@inspirit yea, i thought it would be too good to be true if relu attention worked :disappointed: it must have worked for FLASH because they confine their quadratic attention to local windows


fix: add retry logic to recreate existing trustpolicy secret

What type of PR is this?

Which issue does this PR fix:
What does this PR do / Why do we need it:
If an issue # is not available please add repro steps and logs showing the issue:
Testing done on this change:

Automation added to e2e:

Will this break upgrades or downgrades?
Does this PR introduce any user-facing change?:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Merged, as this issue is impacting test stability.


Int widget

Initial ScreenUtil very dirty in beta version, plese check #181 😉
And i litle fixed readme files

I actually thought about doing this before, thank you


How to add a Box back to a LinearLayout?

I'm doing dynamic manipulation of the view tree, where I need to remove a view and re-add it as a child of an intermediate node;
        let v1 = layout.remove_child(pleaf).unwrap();
        let mut sublayout = match method.dir() {
            winmethod::Left => LinearLayout::horizontal().child(win).child(v1),
            winmethod::Right => LinearLayout::horizontal().child(v1).child(win),
            winmethod::Above => LinearLayout::vertical().child(win).child(v1),
            winmethod::Below | _ => LinearLayout::vertical().child(v1).child(win),
        };
        layout.insert_child(pleaf, sublayout);

It's not working though.
error[E0277]: the trait bound `std::boxed::Box<dyn cursive::traits::View>: cursive::view::ViewWrapper` is not satisfied
   --> src/window_manager.rs:125:78
    |
125 |                             winmethod::Below | _ => LinearLayout::vertical().child(v1).child(win),
    |                                                                              ^^^^^ the trait `cursive::view::ViewWrapper` is not implemented for `std::boxed::Box<dyn cursive::traits::View>`
    |
    = note: required because of the requirements on the impl of `cursive::traits::View` for `std::boxed::Box<dyn cursive::traits::View>`

Is there any way that this is possible that I'm missing?

You can do 2 things:

If you know the type T of the view, you can use AnyView::downcast() to go from Box<dyn View> to Box<T> then dereference to get a T, and add this T to the child.
Wrap the Box<dyn View> in a ViewBox, and add that to the child.

The first option needs you to know at compile time the type of the view. The second option means that the child will re-box the given already-boxed view, so it causes a bit more indirection.
The second solution could be improved if parent views could have a special path for to add ViewBox without re-boxing them. It could be done with specialization, or with a dedicated method.

Actually it seems I already added IntoBoxedView for this purpose, it just wasn't used for LinearLayout. You should now be able to add the Box<dyn View> directly, and it won't be re-boxed.

That's perfect, it just works now!
Thank you!

If you know the type T of the view, you can use AnyView::downcast()

The idea was to keep the information of what view is used abstracted, if possible. I guess wrapping it one time in a ViewBox would have worked too because from then on the type is known :smile: But this is great.


Vertical touch scrolling for multiple images

We have multiple images in a vertical column on 1 page. The pinch zoom and pan works fine, but if the user tries to scroll the page down, it won't let them. They have to touch somewhere other than the image to scroll the page. Do you have any ideas as to what could allow this behavior? If you point me in the right direction, I can try to create a pull request for it. Thanks.

#6

Sorry for late merge, but now its updated!


Choose methodology for combining data from census tracts into census block groups

We need a methodology for combining data from census tracts into census block groups so that we can use the DOE LEAD score and other future data sets that are at the tract level.

Notes:

Recommended to not get into percentiles at the tract level, keep raw value from tract, assign that to every single CBG, then calculate percentiles at the CBG level.

Communication: on health data, need to be clear that this was originally at tract level, else people might be concerned about small samples revealing private information.


pandora.tv can't show video

Windows 10 RS5
GoodbyeDPI v0.1.5rc3
@ECHO OFF
PUSHD "%~dp0"
IF "%PROCESSOR_ARCHITECTURE%"=="x86" (
set _arch=x86
) ELSE (
set _arch=x86_64
)
echo This script should be run with administrator privileges.
echo Right click - run as administrator.
echo Press any key if you're running it as administrator.
pause
sc stop "GoodbyeDPI"
sc delete "GoodbyeDPI"
sc create "GoodbyeDPI" binPath= ""%CD%%_arch%\goodbyedpi.exe" -1" start= "auto"
sc description "GoodbyeDPI" "Passive Deep Packet Inspection blocker and Active DPI circumvention utility"
sc start "GoodbyeDPI"
POPD
pandora.tv can't show video
only cna show video when stop GoodbyeDPI service

If the video is loaded over HTTP, then their HTTP server is not fully compatible with HTTP/1.1 specification. If it's loaded over HTTPS, most probably their server or balancer can't handle fragmented packets.

Seems that the video plays over HTTP. The best is to write site administrators with incompatibility examples.


Error: "cannot find GOROOT directory"

I think there's a bug when running on subdirectories.
Running staticcheck ./... locally works. I added the action to my Yaml file as:
    steps:
      - uses: actions/checkout@v2
      - uses: actions/setup-go@v1
        with:
          go-version: 1.16
      - run: go test ./...
      - name: staticcheck
        uses<EMAIL_ADDRESS>        with:
          run: staticcheck
          flags: ./...
          token: ${{ secrets.GITHUB_TOKEN }}

This is the output of the action:

It looks like the image uses the go binary which is installed inside it, and not the one from the host system.

Yeah, you'll need to manually add GOROOT: '' to the env section of your staticcheck step to avoid overwriting the container's own path for this. (In case anyone else runs into this problem.)


Emphasize types in the documentation

Most of the docs look like
>>> @attr.s
... class UserList(object):
...     users = attr.ib()

but types are good and auto_attribs is good. I propose changing the docs to inspire people to use auto_attribs or @attr.dataclass in the common case, like dataclasses does: https://docs.python.org/3/library/dataclasses.html

I agree the docs will be a look sexier if we use auto_attribs by default everywhere and document what you have to do to support Python 2 in a separate section.  The question is one of timing.
We could argue that DO IT NOW is the way to go, because better is better and Python 2 is so yesterday.
Another option is to do this cleanup as part of import attrs.

This is something that has tormented me forever. In my experience how people read docs, the choice that we have here is:

Status quo: everyone thinks attrs doesn't support type annotations.
We use auto_attribs: everyone thinks attrs is Python 3.6+ only.

And 2020 doesn't even change the complications of this whole thing, because supporting legacy Python versions is probably gonna be a solid value proposition in the coming years.
We will definitely do it as part of import attrs tho.

And 2020 doesn't even change the complications of this whole thing, because supporting legacy Python versions is probably gonna be a solid value proposition in the coming years.

Well… I agree that Py2 support is an appealing feature of attrs, but I do think that we can use auto_attribs in the docs and say somewhere "up front" that if you need to support Py2 that you'll need to use a more verbose syntax, which is described in the special "how to support Py2" section.
I think we are underselling attrs by not using the new and very nice syntax that a lot of great work went into supporting, and that treating Py2 as a special use case is an increasingly reasonable tactic.

I agree with all of this. I guess the actual reason why I'm hesitant is that I find auto_attribs rather clunky. OTOH my intentioned<EMAIL_ADDRESS>doesn't fit the namespace anymore and import attrs is kinda blocked on my fear and procrastination.
I'm open to suggestions on how to break this knot. 😞

I'm all for adding @attr.auto with everything we currently know from https://github.com/python-attrs/attrs/issues/487. I'm not too worried about not matching the namespace. People sometimes ask "what does ib mean?", but I don't think anybody would think twice about attr.auto.
If you're worried about getting import attrs right the first time, maybe it'd be helpful to have a testing ground. Perhaps add an attr.experimental or attr.zoo namespace for trying out new functions. That way we don't have to worry about polluting the main module namespaces, but we do get to use new functions and learn from iteration.

I'd call that attr._zoo or whatever (that is, prefix with _) to make it clear that nothing there is supported long-term.

this has been fixed in #863


Document exported functions and consts in pkg/mount

Addresses #11594.

@LK4D4 Do you know why the Jenkins build is failing? It says:

Error response from daemon: no such id: docker-pr4738

I've rebased my PR on upstream/master, so I'm not sure what the problem is.

@jamiehannaford It's failing because your third commit in not signed. Could you please squash your commits into a single (signed) one? Thanks!

@icecrime @LK4D4 Ah, okay. I've squashed and signed them - so ready for review again :smile:

Thanks! LGTM :+1:

this is legit thanks!

LGTM


Easier debugging of SPARQL queries

Add a --num_rows argument to more easily specify that value on the command line without having to edit the code. If --full is specified, it overrides --num_rows and all rows are processed. The wiki_to_netflix module still contains a default value for num rows for when it is run directly.
Add a --missing_out_path argument, which will collect all the missing movies and put them in their own CSV. If this argument isn't specified, no missing movie CSV is written. This path is relative to the output directory, so if you specify --missing_out_path=my_missing.csv you will find your file at out/my_missing.csv.
Here is an example of how to use these new arguments:
$ pipenv run dev -m missing.old.csv -n 1000
[WARNING] Could not find movie id 1 ('Dinosaur Planet', '2003')
....
missing: 486 (48.60%)
found: 514 (51.40%)
total: 1000

$ pipenv run dev -m missing.new.csv -n 1000
[WARNING] Could not find movie id 1 ('Dinosaur Planet', '2003')
....
missing: 524 (52.40%)
found: 476 (47.60%)
total: 1000

$ wc -l out/missing.old.csv
     486 out/missing.old.csv
$ wc -l out/missing.new.csv
     524 out/missing.new.csv
$ diff out/missing.old.csv out/missing.new.csv
5d4
< 7,null,8 Man,1992,null,null
7d5
< 10,null,Fighter,2001,null,null
16a15
> 28,null,Lilo and Stitch,2002,null,null
27a27
> 44,null,Spitfire Grill,1996,null,null
31a32
> 54,null,We're Not Married,1952,null,null
38d38
...

Note that in this example, the QUERY variable in wiki_to_netflix was manually edited between run. A future enhancement might be to allow the user to specify a file or python module path that contains the query to use.

Just one question, how do we make use of the missing outputs csv?

Just one question, how do we make use of the missing outputs csv?

You can see it in my example above. You can use wc -l <file> to see how many misses are in each file. You can use diff to see which movies are in one file and not the other. You can also presumably import the CSVs into a spreadsheet and compare them that way.


Benchmarker of tensorflow/lite initialize tensorflow

Current implementation of tensorflow/lite/testing have initialization of TensorFlow core.
https://github.com/tensorflow/tensorflow/blob/069c503b4ac5bf4c89aeb38332685e427e0d3e03/tensorflow/lite/testing/init_tensorflow.cc#L23-L30
Why this is needed? Benchmarker always include binary of tensorflow. I think this is enough.
diff --git a/tensorflow/lite/testing/init_tensorflow.cc b/tensorflow/lite/testing/init_tensorflow.cc
index ed4d123744..d8d639f410 100644
--- a/tensorflow/lite/testing/init_tensorflow.cc
+++ b/tensorflow/lite/testing/init_tensorflow.cc
@@ -20,12 +20,5 @@ limitations under the License.
 #include "tensorflow/core/platform/init_main.h"

 namespace tflite {
-void InitTensorFlow() {
-  static const char* kFakeName = "fake program name";
-  int argc = 1;
-  char* fake_name_copy = strdup(kFakeName);
-  char** argv = &fake_name_copy;
-  ::tensorflow::port::InitMain(kFakeName, &argc, &argv);
-  free(fake_name_copy);
-}
+void InitTensorFlow() {}
 }  // namespace tflite

I believe this is needed at least in this test:
https://github.com/tensorflow/tensorflow/blob/5b24fba0857394dab67359963726b3bcce071575/tensorflow/lite/tools/benchmark/benchmark_plus_flex_main.cc#L16
Basically, in flex mode, TF Lite will fall back to use TF ops, and that's the reason we need to include tensorflow core dependencies. Note that not all TF Lite benchmark tests need import core TF.


add Links model with custom root type

Codecov Report

Merging #52 (b1a0f41) into master (59abeb9) will decrease coverage by 0.45%.
The diff coverage is 70.00%.

@@            Coverage Diff             @@
##           master      #52      +/-   ##
==========================================
- Coverage   99.04%   98.58%   -0.46%
==========================================
  Files          29       29
  Lines         629      637       +8
==========================================
+ Hits          623      628       +5
- Misses          6        9       +3

Flag
Coverage Δ

unittests
98.58% <70.00%> (-0.46%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

stac_pydantic/shared.py
97.39% <62.50%> (-2.61%)
:arrow_down:

stac_pydantic/item.py
95.65% <100.00%> (ø)

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 59abeb9...b1a0f41. Read the comment docs.


feature: add schedule plugin

Signed-off-by: yunfeiyangbuaa<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
every company or organization has different requirement.so only one schedule way can't meet the needing.I make the scheduleMgr loading schedule ways from config file
reference #1002  it is the first part of the issue
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗？)
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Which issue you fixed? Please link this PR to it.

Codecov Report

Merging #1000 into master will decrease coverage by 0.42%.
The diff coverage is 10.44%.

@@            Coverage Diff             @@
##           master    #1000      +/-   ##
==========================================
- Coverage   46.54%   46.12%   -0.43%
==========================================
  Files         116      118       +2
  Lines        6903     6962      +59
==========================================
- Hits         3213     3211       -2
- Misses       3434     3494      +60
- Partials      256      257       +1

Impacted Files
Coverage Δ

supernode/plugins/plugin_manager.go
91.42% <ø> (ø)
:arrow_up:

supernode/daemon/mgr/scheduler/manager.go
18.49% <0%> (-3.43%)
:arrow_down:

supernode/daemon/mgr/scheduler/schedule.go
0% <0%> (ø)

...ernode/daemon/mgr/scheduler/schedule_plugin_mgr.go
0% <0%> (ø)

supernode/store/store_mgr.go
43.75% <100%> (ø)
:arrow_up:

supernode/plugins/plugins.go
100% <100%> (ø)
:arrow_up:

supernode/server/server.go
42.18% <50%> (-0.44%)
:arrow_down:

dfget/core/uploader/peer_server.go
86.39% <0%> (ø)
:arrow_up:

... and 2 more

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 0d2bbbe...cb4fdac. Read the comment docs.

Which issue you fixed? Please link this PR to it.
Thank you for your comment !!!!
emmmmmm,no issue links to this pr, the cdn has a store plugin to meeting different needing, and I think the schedule is the same.if necessary, I will open a issue linked it.

@Starnop please review it.Thank you !!

Which issue you fixed? Please link this PR to it.

Thank you for your comment !!!!
emmmmmm,no issue links to this pr, the cdn has a store plugin to meeting different needing, and I think the schedule is the same.if necessary, I will open a issue linked it.

Maybe you need to explain your PR carefully. If there is detailed info about why we need to modify, I  think this PR can be reviewed efficiently.

Which issue you fixed? Please link this PR to it.

Thank you for your comment !!!!
emmmmmm,no issue links to this pr, the cdn has a store plugin to meeting different needing, and I think the schedule is the same.if necessary, I will open a issue linked it.

Maybe you need to explain your PR carefully. If there is detailed info about why we need to modify, I think this PR can be reviewed efficiently.

Ok，I will open a  PR yesterday

@Starnop please review this pr,Thanks !


first_name , last_name added

i hope the proposed changes are fine

Sorry but i can't accept this, beside that issue was assigned to someone else.


Individual Dog Access Level Views

Individual Dog Access Level Views
Issue Number(s): #134
What does this PR change and why?
Conditionally renders frontend depending on the user role/association with the dog. Also implements backend logic for returning correct forms and logs depending on user and dog.
Checklist

[ ] Requirements have been implemented
[ ] Acceptance criteria is met
[ ] Database schema docs have been updated or are not necessary
[ ] Relevant reviewers (EM) have been assigned to this PR

Critical Changes

Database change / migration to run
Environment config change
Breaking API change

Related PRs
PR Number(s):
How To Test
Switch through admin/manager/volunteer/instructor roles to see views change depending on association to dog

Design looks great!

merged in with #143


Implement network.responseStarted event

Spec: https://w3c.github.io/webdriver-bidi/#event-network-responseStarted

See https://github.com/w3c/webdriver-bidi/pull/204#issuecomment-1570324761.


If platform is unsupported, don't try to install empty array of packages

Cookbook version
master
Chef-client version
12.16.42
Platform Details
Windows server 2012 R2
Scenario:
If node['platform_family'] isn't one of the supported values, default['xml']['packages'] = [] and users get a misleading error:
NoMethodError: undefined method `downcase' for []:Chef::Node::ImmutableArray

Steps to Reproduce:
I've got a user who reproduced it by running xml::default on Windows-based platform: https://github.com/martinb3-cookbooks/xmledit-cookbook/issues/19
Expected result
Perhaps this cookbook should blow up with a better message or skip installing packages if the array is empty (maybe something like only_if { node['xml']['packages'].length == 0 }?).

I ran into this with ark as well. We'll have to be more defensive in the future.

Give 3.1.0 a try and let me know if that works better for you. I added a pile of specs to avoid this in the future.

Thanks @tas50!


WIP: Enh/api config

Description of Changes

[ ] Wrote at least one-line docstrings (for any new functions)
[ ] Added unit test(s) covering the changes (if testable)

[ ] Included a screenshot or animation (if affecting the UI, see Licecap)

Issue(s) Resolved

Fixes #
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct:

Hello @goanpeca! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file spyder/preferences/configdialog.py:

Line 165:80: E501 line too long (80 > 79 characters)
Line 936:80: E501 line too long (83 > 79 characters)


Fix CVE-2023-50164 in 2-3-x

Parameters related to file uploads, esp. <xxx>FileName, will be added only after existing parameters with identical names (case-insensitive) have been removed.
Since the 2-5-x class HttpParameters is not available yet, the fix is done very locally in class FileUploadInterceptor, i.e. other potential parameter pollutions are not covered (e.g. during the later execution of other interceptors that also update the parameters).
The fix has been tested using a vulnerable test application.

Apart from assessing whether the fix works, I guess the other important question is whether it is worthwhile to produce a version 2.3.38 for the EOL release branch 2-3-x. The severity of this CVE could be one motivation, however, other severe vulnerabilities would remain unaddressed...


[Task]: Migrate checked in Colab notebooks to utilize new notebook integration tests

What needs to happen?
#26690 adds a script to test notebooks. We should now make sure that existing notebooks in our repository utilize this by adding a .spec file and ensuring they work appropriately.
Issue Priority
Priority: 3 (nice-to-have improvement)
Issue Components

[X] Component: Python SDK
[ ] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[ ] Component: IO connector
[X] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

.take-issue


Adding a Preloader

Adding a preloader to your website to improve its performance and ensure a smooth user experience.
@Sulagna-Dutta-Roy,  Please assign me this issue.

@Sulagna-Dutta-Roy, Please assign me this issue, I'm Gssoc'24 Contributor.

In which website you want to add preloader? Also this is an extension repository.
Use the feature request for raising the Issues @1754riya

Closing this as no update since last few days


Poor 2D performance on Windows 10

Running this driver on Windows 10 gives poor 2D performance. The cursor stutters and working with the system generally feels very sluggish. If I am not mistaken, using the old drivers will not work with Windows10 anymore. So there are not real alternatives to this drive, or?

I totally agree on this. When I played a 1080p video on Windows 10, It was too slow to do other job. I tried to rebuild qxl driver from this repository. But it didn't work out either. I found spice-server's network traffic had gone really high(compare to Windows 7). Is it related to this issue?

qxl-dod is a Display Only Driver, which technically is a frame-buffer driver, without any notion about 3D/2D acceleration  (unlike XDDM-style qxl driver designed to work on WinXP..Win7 platforms).

Thanks for response. We keep researching this subject for 2 weeks. I want to clarify just one thing. Is there any way that you know to accelerate 3D/2D performance on Windows 10?

Unfortunately, there is no way to get accelerated 3D/2D performance with DOD-style driver, because it always  works with software-based 3D/2D rendering. The current driver needs to be converted/extended  to Full Graphics or Render Only driver to show more or less advanced performance.

I believe the mouse cursor lag is unrelated to the 2D performance as it is just as slow with the Microsoft Basic Display driver. This might be becuase it isn't using a hardware cursor but one that is rendered as part of the composited desktop (or software cursor if you want).

The sluggish graphics performance and lack of guest-auto-resizing are a real show stopper for Win10 guests for regular desktop usage. Guess I have to switch back to Windows7 as a guest then.

Very little point in improving qxldod when virtual GL is around the corner. Also for better 2D just use RDP. RDP beats even the qxl driver for Window 7 in responsiveness. A new driver will have to be written to take advantage of the virgl capability in QEMU.

I will keep an eye on the virtual GL progress. Thanks for the input!


Add support for defining subtitle language preference order

Previously defining preference order as described in
https://github.com/rg3/youtube-dl#format-selection was not possible
for the --sub-lang option. This commit adds support for this feature.
I've implemented this feature by adding a new helper method
_get_first_match(). I hope it can be reused at other locations as well
in order to simplify the often very complex code.

Should I close this? Is there really no interest in this PR?

This PR is useful. It's not examined yet because we're examining some more older PRs...


HADOOP-17643 WASB : Make metadata checks case insensitive

NOTICE
Please create an issue in ASF JIRA before opening a pull request,
and you need to set the title of the pull request which starts with
the corresponding JIRA issue number. (e.g. HADOOP-XXXXX. Fix a typo in YYY.)
For more details, please see https://cwiki.apache.org/confluence/display/HADOOP/How+To+Contribute

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
7m 39s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ branch-3.3 Compile Tests _

+1 :green_heart:
mvninstall
30m  8s

branch-3.3 passed

+1 :green_heart:
compile
0m 36s

branch-3.3 passed

+1 :green_heart:
checkstyle
0m 28s

branch-3.3 passed

+1 :green_heart:
mvnsite
0m 42s

branch-3.3 passed

+1 :green_heart:
javadoc
0m 33s

branch-3.3 passed

+1 :green_heart:
spotbugs
1m  4s

branch-3.3 passed

+1 :green_heart:
shadedclient
16m 26s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 32s

the patch passed

+1 :green_heart:
compile
0m 27s

the patch passed

+1 :green_heart:
javac
0m 27s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 19s

the patch passed

+1 :green_heart:
mvnsite
0m 31s

the patch passed

+1 :green_heart:
javadoc
0m 22s

the patch passed

+1 :green_heart:
spotbugs
1m  2s

the patch passed

+1 :green_heart:
shadedclient
15m 54s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
1m 55s

hadoop-azure in the patch passed.

+1 :green_heart:
asflicense
0m 35s

The patch does not generate ASF License warnings.

79m 45s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3103/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/3103

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux 22949b6e82cb 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
branch-3.3 / ba089bda4ff2afc07ce5676fbc5d13011e009f02

Default Java
Private Build-1.8.0_292-8u292-b10-0ubuntu1~18.04-b10

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3103/1/testReport/

Max. process+thread count
541 (vs. ulimit of 5500)

modules
C: hadoop-tools/hadoop-azure U: hadoop-tools/hadoop-azure

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3103/1/console

versions
git=2.17.1 maven=3.6.0 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 40s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ branch-3.3 Compile Tests _

+1 :green_heart:
mvninstall
31m 13s

branch-3.3 passed

+1 :green_heart:
compile
0m 37s

branch-3.3 passed

+1 :green_heart:
checkstyle
0m 28s

branch-3.3 passed

+1 :green_heart:
mvnsite
0m 42s

branch-3.3 passed

+1 :green_heart:
javadoc
0m 32s

branch-3.3 passed

+1 :green_heart:
spotbugs
1m  3s

branch-3.3 passed

+1 :green_heart:
shadedclient
23m 13s

branch has no errors when building and testing our client artifacts.

-0 :warning:
patch
23m 33s

Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 32s

the patch passed

+1 :green_heart:
compile
0m 28s

the patch passed

+1 :green_heart:
javac
0m 28s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 18s

the patch passed

+1 :green_heart:
mvnsite
0m 31s

the patch passed

+1 :green_heart:
javadoc
0m 21s

the patch passed

+1 :green_heart:
spotbugs
1m  2s

the patch passed

+1 :green_heart:
shadedclient
22m 53s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
1m 57s

hadoop-azure in the patch passed.

+1 :green_heart:
asflicense
0m 34s

The patch does not generate ASF License warnings.

87m 35s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3103/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/3103

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux 3824e6376b24 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
branch-3.3 / 14cb4b4229390dedccceb937b303137dc7bf994d

Default Java
Private Build-1.8.0_292-8u292-b10-0ubuntu1~18.04-b10

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3103/2/testReport/

Max. process+thread count
536 (vs. ulimit of 5500)

modules
C: hadoop-tools/hadoop-azure U: hadoop-tools/hadoop-azure

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3103/2/console

versions
git=2.17.1 maven=3.6.0 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

Build report looks good


Parse custom date commands.

Sometimes folks like to post checkins for a day in the past (or maybe they're in a timezone ahead of the bot). It might be nice if folks could send the bot a date cue to get a topic with that date.

I think many users won't want to go back more than a few days, so you could probably get away with just days of the week to start? If it's a day of the week that's currently happening somewhere, use that date (so if it's Tuesday in Australia, use that Tuesday's date), but if it's not yet happening somewhere (with maybe a day of wiggle room for folks who like to post for the next day?) use that day of the week in the past.


breaking change: lsmysql.CheckScript() now returns error

BREAKING CHANGE: lsmysql.CheckScript() now returns error instead of CheckResult.
This change was made so that CheckScript's return value could include examples.
Use errors.Is(err, lsmysql.ErrDataAndDDL) and errors.Is(err, lsmysql.ErrNonIdempotentDDL) to classify the error.

Codecov Report
Base: 81.26% // Head: 81.30% // Increases project coverage by +0.03% :tada:

Coverage data is based on head (02b9acd) compared to base (f64702a).
Patch coverage: 94.73% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #54      +/-   ##
==========================================
+ Coverage   81.26%   81.30%   +0.03%
==========================================
  Files          12       12
  Lines        1201     1198       -3
==========================================
- Hits          976      974       -2
+ Misses        158      157       -1
  Partials       67       67

Flag
Coverage Δ

go_tests
9.59% <0.00%> (+0.02%)
:arrow_up:

mysql_tests
49.41% <94.73%> (-0.05%)
:arrow_down:

pg_tests
54.24% <0.00%> (+0.15%)
:arrow_up:

singlestore_tests
47.16% <94.73%> (-0.05%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

lsmysql/check.go
84.61% <92.85%> (+3.13%)
:arrow_up:

lsmysql/mysql.go
78.54% <100.00%> (-0.17%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Set __moduleName from context.id argument

This resolves #6359 using the method suggested by @vladima as the replacement to the original PR at https://github.com/Microsoft/TypeScript/pull/6098.
This context object is described at https://github.com/ModuleLoader/es6-module-loader/wiki/System.register-Explained#how-it-works, and was considered more extensible for future spec changes than a string second argument directly.

Hi @guybedford, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

@guybedford, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;

:+1:, looks like we have two unit tests that need to be updated.
@vladima, can you take a look.

I think this variable has to be injected after the prologue directive "use strict"

Thanks, I've made these corrections.

@guybedford can you please fix two remaining failing tests so we can pull in this PR?

@vladima sure I've got them passing now.

thanks!

@vladima can you merge this in release-1.8 as well.

:+1: Thanks for all the help getting this in.


feat: add Swift modules and an iOS application

Type of change
Add Swift modules and an iOS application to the repository to demonstrate:

Use of rules_swift and rules_apple.
Import of an external Swift package (e.g., SwifterSwift) using rules_swift_package_manager.
Integration with Xcode using rules_xcodeproj.

Chores

Update Bazel version to 6.3.2.
Update Go dependencies to latest.
Update Gazelle declaration to include Swift plugin.

Test plan

New test cases added

Unit tests can be run using bazel test //.... The UI tests for the iOS application can be run using bazel test //ios/LoggingClient/UITests.

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

@alexeagle I just updated the bazelrc to output more info about the UITest failure in CI. Could you approve the workflow?

I turned off that setting for now, I didn't mean to block your PRs on that approval!

I guess that UI test failure did not occur again. 🤷♂


Update ImGuiStringEndAdapter.cs

Fix access Violation

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

Oops, forgot to update that when I changed NullTerminateAndGetString. Thanks for the fix!
(Merged to main as https://github.com/MochiLibraries/Mochi.DearImGui/commit/32d4d6243408f237e8678e766e8c351574c06b25)


Interested by this project but not with mixins

Hi,
I'm interested in using this project, as I planned to build something very similar myself (and did not even know a spec was ongoing for CSS for this problem).
I don't like some details of the implementation, particularly the usage of mixins and the impossibility to easily use this with inline styling or change the component layout entirely on breakpoint reached.
I would rather use an API that use HOC and looks like:
class MyComponent extends Component {
  render() {
    const className = cx("container",{small: this.props.small, big: this.props.big});
    return (

the box

    );
  }
}

module.exports = ReactContainerQuery.injectQueryResult(MyComponent,{
  small: {
    minWidth: 400,
    maxWidth: 599
  },
  big: {
    minWidth: 600,
  }
});

I like the idea of having full control of the mediaquery result in plain JS and not using mixins.

Hi @slorber,
Thanks for your feedback. I don't remember exactly what was the obstacle of implementing a HOC solution when I begin. But I tried HOC approach this morning and I think it should work. I will work on that in the next few days.
About API using props you still can achieve the same thing now. You need to implement a lifecycle hook and set stateMap to this.state. But yes, if I can implement HOC, you should be able to just use this.props.

Hi @slorber, I have added HOC API and updated the doc. Let me know how that works for you.

thanks will try to see how it fits in my project when i have time to work on this


[Docs] Adding multi-word filter name support documentation to filter command

Current State
New feature for viewing multi-word filter names has been added to filter view command.
Need for documentation on how to view filter names which have multiple words
Need an update here: https://docs.meshery.io/reference/mesheryctl/filter/view

Contributor Guides and Handbook

📚 Instructions for contributing to documentation

Meshery documentation site and source

🛠 Meshery Build & Release Strategy
🎨 Wireframes and designs for Meshery UI in Figma (open invite)
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

/assign

Thank you @Aviral0702


Refactoring Retry

Code Coverage Summary [using JaCoCo]

Coverage Type
Coverage

Coverage:
15.87%

Branches:
21.74%


Fix enter new player lag

Could be the icons

Confirmed it is the icons, if you remove the select, there's no lag. Looking at ways to fix it now.

RIP, maybe if we turn off that broken "filtering"?

RIP, maybe if we turn off that broken "filtering"?

Nah it's not that, I've done some research and it looks like the Semantic UI dropdown/select just doesn't cope well with large amounts of options. I've seen at least one person say that they've had success using just a regular  so I might take a look at that if I can't get this working.

If it's okay to just have the icon name in the select and then the preview displayed for the user, I've got a solution doing that using a regular <select>. I'll push my changes to my branch (QH-12)

Nice, yeah that's a potential workaround too

Turns out that by deploying to GitHub pages, the JS is made more efficient, so there's no lag when entering a new player name. Since most (if not all) people will be accessing the application via the deployment, we can close this.


Log relevant information in JSON format

Current logs have this format:
Response with status 304 in 5 ms

They are not as useful as they should for a machine. What do you thing about logging them as:
Response {"statusCode": 304, "duration": 5}

I think we just need to change:
 logger.info('Response with status %d in %d ms.', res.statusCode, duration);

to:
 logger.info('Response', {statusCode: res.statusCode, duration: duration});

and would work for both logops and no-logops users

It would be nice to have this feature but there are several problems:

If logger is console.log, then


listen ip4:icmp : socket: operation not permitted macOS Monterey

Hello!
My code:
        path := fmt.Sprintf(`%v`, IPAddr)
	pinger, err := ping.NewPinger(path)

	if err != nil {
		return err
	}

	pinger.SetPrivileged(true)
	pinger.Count = environments.StatsPingCount
	pinger.Timeout = environments.TimeOutStatsPing

	err = pinger.Run()

	log.Println(err)

	if err != nil {
		return err
	}

	stats := pinger.Statistics()

	if stats.PacketsRecv == 0 {
		return errors.New("ping error")
	}

	return nil

I have next error when run code: socket: operation not permitted.
OS: macOS Monterey

Have you tried running as root?

Yes, process is freezy

What do you mean by "freezy"?
Could you include complete sample code that reproduces the issue, rather than just a snippet? Thanks!

Hello!
func (i *Object) Ping() error {
ip, err := net.ResolveIPAddr("ip4", i.IPAddr)
if err != nil {
	return err
}

pinger, err := ping.NewPinger(ip.String())

if err != nil {
	return err
}

pinger.SetIPAddr(ip)
pinger.SetPrivileged(true)
pinger.Count = environments.StatsPingCount
pinger.PacketsSent = environments.StatsPingCount
pinger.Timeout = environments.TimeOutStatsPing
pinger.Interval = 300 * time.Millisecond

err = pinger.Run()

if err != nil {
	errStr := fmt.Sprintf(`Ping %v error: %v`, i.IPAddr, err.Error())
	return errors.New(errStr)
}

stats := pinger.Statistics()

if stats.PacketsRecv == 0 {
	errStr := fmt.Sprintf(`Ping %v error: PacketsRecv = 0`, i.IPAddr)
	return errors.New(errStr)
}

return nil

}

In OS windows I have next problem:
read ip4 <IP_ADDRESS>: wsarecvfrom: A message sent on a datagram socket was larger than the internal message buffer or some other network limit, or the buffer used to receive a datagram into was smaller than the datagram itself.
Can You help? Maybe I must change something on host?

The last bit is a dupe of #168.

On Mac OS you need to pass privileged as false: pinger.SetPrivileged(false)

Really? Is this a Monterey thing? I'm on Big Sur and don't experience this:
$ uname -v
Darwin Kernel Version 20.6.0: Tue Feb 22 21:10:41 PST 2022; root:xnu-7195.141.26~1/RELEASE_X86_64

$ sudo go run cmd/ping/ping.go --privileged www.google.com
PING www.google.com (<IP_ADDRESS>):
32 bytes from <IP_ADDRESS>: icmp_seq=0 time=7.062ms ttl=119
32 bytes from <IP_ADDRESS>: icmp_seq=1 time=7.007ms ttl=119
32 bytes from <IP_ADDRESS>: icmp_seq=2 time=8.108ms ttl=119
32 bytes from <IP_ADDRESS>: icmp_seq=3 time=6.64ms ttl=119
32 bytes from <IP_ADDRESS>: icmp_seq=4 time=7.199ms ttl=119
32 bytes from <IP_ADDRESS>: icmp_seq=5 time=6.979ms ttl=119
^C
--- www.google.com ping statistics ---
6 packets transmitted, 6 packets received, 0 duplicates, 0% packet loss
round-trip min/avg/max/stddev = 6.64ms/7.165834ms/8.108ms/453.908µs

$ go run cmd/ping/ping.go www.google.com
PING www.google.com (<IP_ADDRESS>):
32 bytes from <IP_ADDRESS>: icmp_seq=0 time=7.081ms ttl=119
32 bytes from <IP_ADDRESS>: icmp_seq=1 time=7.198ms ttl=119
32 bytes from <IP_ADDRESS>: icmp_seq=2 time=7.815ms ttl=119
32 bytes from <IP_ADDRESS>: icmp_seq=3 time=7.115ms ttl=119
32 bytes from <IP_ADDRESS>: icmp_seq=4 time=6.976ms ttl=119
^C
--- www.google.com ping statistics ---
5 packets transmitted, 5 packets received, 0 duplicates, 0% packet loss
round-trip min/avg/max/stddev = 6.976ms/7.237ms/7.815ms/297.639µs

This is on macOS Monterey 12.3.1:
go run cmd/ping/ping.go --privileged <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>):
Failed to ping target host: listen ip4:icmp : socket: operation not permitted

This is without the privileged flag
 go run cmd/ping/ping.go <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>):
2022/04/25 16:38:50 DEBUG: Setting the timeout to: 27h46m40s
2022/04/25 16:38:50 DEBUG: Setting the interval to: 1s
2022/04/25 16:38:50 DEBUG: Done with timeout
2022/04/25 16:38:50 DEBUG: Done with interval
32 bytes from <IP_ADDRESS>: icmp_seq=0 time=19.303ms ttl=248
32 bytes from <IP_ADDRESS>: icmp_seq=1 time=17.288ms ttl=248
32 bytes from <IP_ADDRESS>: icmp_seq=2 time=35.344ms ttl=248
32 bytes from <IP_ADDRESS>: icmp_seq=3 time=15.612ms ttl=248


Merging to develop, added new functionality to have list of API keys

@jjg-123, I'm closing this PR as part of some repo cleanup. If needed, we can reopen this in the future. Thanks.


Give more evidence to the "Reset database connection" menu.

To use the "Reset database connection" menu you really need to know where it is (on the "Tables" word on top of the tables list).
Make a button instead.

Added a menu displayed as a "3 lines" icon, on top of the "Tables" panel. It contains the submenu "Connection Reset", which replaces the previously available context menu.
Done the same with the context menu on top of the "Table" panel. Added the "3 lines" menu, which contains as submenus all the previously available context menus (copy SQL, close tabs, check/uncheck all).


Fix known leaks in both swift and android SDKs

This was easier for the Swift one than the Android one, although I'm more confident that the android one is completely good now -- In the swift, I'm not 100% certain that TypedValues are being cleaned up, since we should have seen some of the same crashes that I did in android when things were freeing values returned by value_at_index.
Anyway, the Android one also has several changes that were needed in order to make it less likely to cause issues in the future. I'd like to go further here and make the rawPointer on RustObject private, but I stopped before doing so.

In general, I have a couple fears about the FFI code that I'm going to take this comment to express:
Manual memory management in languages that don't help you out is fairly error prone, and getting it wrong can lead to double frees or use-after-frees, both of which lead to security vulnerabilities (and are the types of things that Rust was designed to avoid).
The changes I've made don't help here0 -- leaking memory isn't unsafe (look at std::mem::forget) but once you stop leaking the memory you open yourself up to all sorts of nastiness. Unfortunately, I don't think it's realistic to think we could ship while leaking memory very frequently.
My biggest fear is that one of our consumers (Fenix? Lockbox?) ends up with a security vulnerability due to some memory safety error in the FFI boundary. I think this would be extremely embarrassing for all involved, and is completely possible.
The only solution I can think of is for us to run with either something like msan/asan/valgrind (which IDK if you can use for e.g. Java on Android), or possibly a debug heap that tracks that you're calling all the correct destroy_* functions exactly once, ensures nothing that we didn't allocate ever is freed with them, and that we never access memory that has been freed (e.g. it unmaps the pages where items were located after they're freed).
Even this would only be a half solution though, since it wouldn't catch calling a function other than a destructor with the wrong type. This would still be an exploitable memory safety error. ExternResult1 makes this sort of thing sound very plausible to me, and the Android code's heavy use of Pointer even more so2.
0: Well, some of them do. Removing heap allocated ExternResults, should make things a bit better. Same goes for some of the changes made to the android FFI, which make an effort to prevent double frees or use after frees by making sure you discard RustObject's rawPointer after it's freed.
1: Presumably, doing something like #[repr(C)] struct ExternResult<T> { ok: *mut T, err: *mut c_char } is our best option, especially since we hand write our .h/JNA.java files. We'd then write FooResult, and BarResult for each ExternResult<Foo>/ExternResult<Bar> in the store.h.
2: I gather the way to get more type safety for some of this is to define PointerTypes? JNI also is a bit better here since we'd be implementing a lot more of the bindings in Rust, where we could be confident about not making mistakes.

The latest two commits are a very different approach with the following benefits:

The type returned by FFI functions are now explicit. This mitigates issues where the documentation claims something returns an ExternResult that holds one type, but it actually holds another type. This was especially error prone given that we would convert from Result<T, E> to ExternResult with into(), which means we did so without ever typing out the actual return type!
Java is now much more type safe, while still using JNA. This was actually not as hard as I had feared. If we use this technique it should go a long way towards preventing passing pointers to functions that take different types.
The Swift SDK is still not type safe, but it's no worse than before. I will admit the ergonomics did suffer here, but I think benefits 1 and 2 make up for it. I had hoped that I'd be able to have an opaque T in a Unsafe{,Mutable}Pointer<T>, but no such luck.

I still haven't responded to all review comments

I think we can do something with typing in Swift. Possibly involving better annotations in the C header to give the opaque pointers a named type value and then casting to an UnsafePointer of that type. I can take a look at that if you want to move on though. As you say, it's no worse than it was before.

...

Now, you can cast an OpaquePointer to an typed UnsafePointer in Swift. I don't know what the result of this would be here, but I can experiment...

The issue is there's no T here, since it was an incomplete type in C. I thought about doing struct Foo {}; in the header, which would let us use UnsafePointer<T>/UnsafeMutablePointer<T>, and we could have all the types return the real pointer types and get type safety. Then swift would be back to having more type safety than the java code, since it understands const/non-const.
The reason I didn't do this is because it's very much not the layout of these structs, and it would allow swift code to create values of these types without getting them through from rust. Doing so would always be a bug.
There might be a way to signal this appropriately like struct Foo { void *__internal_fake_data; } or something though, where if you do Foo(__internal_fake_data: UnsafeRawMutablePointer(bitPattern: 0)) (or something similar) in Swift you know you're doing something very dodgy. I'd be fine with this, but it would be a strange pattern.

So, some further thinking is that while the "type safe swift" approach I linked in the previous comment (e.g. https://github.com/thomcc/mentat/tree/type-safe-swift, but the only difference is the single patch https://github.com/thomcc/mentat/commit/85322bebd2256c905e795c075cca167140f8f5a0) has some drawbacks (If you create a RawBlah(_internal_fake_data: ...) from swift it will be a memory safety issue basically 100% of the time), this actually seems much easier to audit and prevent than ensuring everybody passes their OpaquePointers to the correct functions.
If we do this we'll have type safety from Swift and Java, with the caveat that it depends on getting the declarations in the store.h/JNA.java equivalents correct. This is a big caveat, since we had mistakes in ours, but it also seems a lot easier to address with tooling.


vue3 : localized-link not working with object as "to" prop

If I use localized-link with a string as to parameter, I get a translated link. But if pass an object, I get an untranslated output:
// works, localized to /fr/lire
<localized-link to="/read">
// not localized, output /read
<localized-link :to="{ name: 'read' }">
<router-link :to="{ name: 'read' }">

See https://next.router.vuejs.org/api/ for to parameter.

After furter investigation, the localized-link component works in my main App.vue component, but not in my sub-components.
Note that I'm using routes "components" (with a s) key and lazy loading, that might be an issue?

Thanks for pointing this out. The translation on the link itself isn't implemented when using name inside the to attribute. However, if I'm not mistaken, when actually pressing the link, you should get redirected to translated path, because internally there is a beforeEach navigation guard, which translates the new path.
Concerning your second comment, it would be helpful if you can provide some stripped down code sandbox (or similar) demo.
On a sidenote, if you use path, it should get translated correctly.
<localized-link :to="{ path: '/read' }">

Sorry, I mixed up things in my second comment, it's actually another bug (or I am misusing the lib?) with language-switcher. I wanted to make some tests before posting a proper bug.
I indeed get redirected using the untranslated route, but I would like to see the link translated on mouse over.

Ok, let's keep this issue reporting the improper link translation when name property is used in the to attribute. If you figure out another bug, please open another issue, ideally with some sandbox demo 🙂

Just wanted to point out that this bug is not specifically tied to vue3 (as the issue title prefix indicates).
It's also present in the vue2 version. I went ahead and created a sandbox trying to demo the issue: https://codesandbox.io/s/vue-2-localized-urls-link-to-named-prop-bug-hssg5?file=/src/components/Navigation.vue

So, any update on this?

Ok, let's keep this issue reporting the improper link translation when name property is used in the to attribute. If you figure out another bug, please open another issue, ideally with some sandbox demo slightly_smiling_face

Any update on this?

Hi guys, sorry I'm leaving you stranded, unfortunately this project isn't a priority for me. I will try to get to it around the end of the year 😉
@alisspers, you are right, it's for both Vue 2 and Vue 3. Thanks for the sample code. It may be just this that needs implementing, but I need to consider it thoroughly.


Extended version has no support for SCSS

I tried downloading the hugo_extended_0.43_macOS-64bit.tar.gz release, which I gather is the version with SCSS support, and get this error:
Building sites … ERROR 2018/07/10 07:15:09 error: failed to transform resource: failed to transform "scss/main.css" (text/css): this feature is not available in your current Hugo version
Total in 13 msError: Error building site: logged 1 error(s)

Also, I double-checked the version of the binary and it says it's v0.44-DEV (not v0.43):
Hugo Static Site Generator v0.44-DEV/extended darwin/amd64 BuildDate: 2018-07-09T18:31:48Z

The version number is me rebuilding last night to get the WIndows build OK. I will fix that. I just tested the MacOS binary on my computer, and that works fine.
I suspect this is something different, and to me this does not look like the SCSS step -- but PostCSS? Does it work if you remove that step? Because of the mime type. We should improve on the error message here, of course.

@jojomi please read my post.

@bep I can confirm it's working if I remove the postCSS step. Am I missing a dependency perhaps?

Am I missing a dependency perhaps?

 npm install -g postcss-cli

We obviously have to work on the docs here ...


Consider support $ref in jsonschema

Thanks for the great tool json-schema-form!
it is very helpful because it is headless and can be used in any framework.
In many json schemas generated by other tools (e.g. OpenAPI, Pydantic, ...), you may see the keyword $ref which is used to reference another schema. This is a very useful feature to avoid duplication and keep the schema DRY.
It would be great if json-schema-form can support this feature. I have tried to use it but it seems not working. I have also checked the documentation and the source code but I cannot find any information about this.
Most of cases, the $defs is defined in the same definition, so it should be easy to implement.
ex) #/$defs/Person will point to the definition of {"$defs": { "Person": {... } } } in the same file.
So, before parsing the schema, we can resolve the $ref and replace it with the actual definition.
I've wrote some simple pseudo code to demonstrate how it can be implemented. I hope this will be helpful.
function getRef(path: string, object: any): any {
  const keys = path.replace(/^#/, '').split('/');

  let current = object;
  for (const key of keys) {
    if (key in current) {
      current = current[key];
    } else {
      return null;
    }
  }

  return current;
}

function resolveRefs(object: any, rootObject: any = object): any {
  for (const key in object) {
    const value = object[key];
    if (typeof value === 'object' && value !== null) {
      if ('$ref' in value) {
        const resolved = getRef(value['$ref'], rootObject);
        if (resolved) {
          object[key] = resolved;
        } else {
          // throw new Error('Cannot resolve reference: ' + value['$ref']);
          console.error('Cannot resolve reference:', value['$ref']);
        }
      } else {
        // go recursive
        resolveRefs(value, rootObject);
      }
    }
  }
  return object;
}

Then, add option to the createHeadlessForm function to enable this feature.
const { fields, handleValidation } = createHeadlessForm(schema, {
  resolveRefs: true, // or the referable other schema
})

I hope this feature will be implemented in the future. Thanks!

Hi @comfuture!
From your example, it looks like you specifically want to use $ref to support $defs rather than Base URI support. It isn't something we implemented as we don't require it at Remote. However, this has been brought up before so it's on our radar. Unfortunately I don't have a timeline as to when we'd be able to able to implement this, but if it's something that you want/need soon PRs are always very much appreciated! If you want to take a stab at it we would help guide it through to release.

Hi @ollyd,
Thank you for reviewing the issue.
In my case, I often import json schemas as part of the openapi spec rather than writing them directly, so I encounter $ref quite frequently.
I thought it would be useful since $ref appears even in the simplest examples of the OpenAPI spec.
https://github.com/OAI/OpenAPI-Specification/blob/main/examples/v3.0/petstore.yaml#L108
I understand that if $ref is not yet used in the main cases for Remote, it might not be considered for immediate implementation in the near future.
The example code implementation that I've wrote was just a minimal sample, so I did not deeply consider what side effects it might bring when making a PR.
If necessary on my end, I will start by preprocessing the schema in a manner similar to the example to achieve the objective.
Later, when the code quality is sufficiently contributable, I will consider contributing through a PR.

@comfuture, thanks for you help, feel free to open a PR supporting local $refs ☺

Do we have plans to support this? I'm trying to move from react-jsonschema-form to any other solution that have better performance and the $ref and "type": "array" is required for my use case.

@igorbrasileiro, we plan to support local $refs (defs), but is not prioritized. PRs are welcome ☺


Suggestion: Changing Bank IDs for shared storage

This feature might be challenging based on you're implementing Banks, but is one of my most used features from Dank Storage.
Basically, in Dank Storage each dank was assigned a sequential ID (I know Bank Storage uses UUIDs instead).  If two danks were the same size, you could manually change the ID of one dank to match another to have them share an inventory.
By doing this, you could effectively use Banks as a sort of Entangled Chest in addition to its other functionality.

This is already 90% complete.

I saw a commit called "added bank links" but didn't want to get my hopes too high :)

Links Added in 0.3.0
https://modrinth.com/mod/bank-storage/version/0.3.0


[Bug]: New skills are not added to skills.py and LLM can not "see" them

Describe the bug
When I in Autogen Studio add a skill, i the skills.py file is not updated, and the LLM can not "see" the skill and will try to make a new function itself.
Steps to reproduce
create a skill, see if it is added to skills.py and if the LLM can use it.
Expected Behavior
I expect that when i create a new skill in the UI it will be added to the skills.py code and the LLM will be able to use it
Screenshots and logs

skills.py

##### Begin of find_papers_arxiv #####

import os
import re
import json
import hashlib

def search_arxiv(query, max_results=10):
    """
    Searches arXiv for the given query using the arXiv API, then returns the search results. This is a helper function. In most cases, callers will want to use 'find_relevant_papers( query, max_results )' instead.

    Args:
        query (str): The search query.
        max_results (int, optional): The maximum number of search results to return. Defaults to 10.

    Returns:
        jresults (list): A list of dictionaries. Each dictionary contains fields such as 'title', 'authors', 'summary', and 'pdf_url'

    Example:
        >>> results = search_arxiv("attention is all you need")
        >>> print(results)
    """

    import arxiv

    key = hashlib.md5(("search_arxiv(" + str(max_results) + ")" + query).encode("utf-8")).hexdigest()
    # Create the cache if it doesn't exist
    cache_dir = ".cache"
    if not os.path.isdir(cache_dir):
        os.mkdir(cache_dir)

    fname = os.path.join(cache_dir, key + ".cache")

    # Cache hit
    if os.path.isfile(fname):
        fh = open(fname, "r", encoding="utf-8")
        data = json.loads(fh.read())
        fh.close()
        return data

    # Normalize the query, removing operator keywords
    query = re.sub(r"[^\s\w]", " ", query.lower())
    query = re.sub(r"\s(and|or|not)\s", " ", " " + query + " ")
    query = re.sub(r"[^\s\w]", " ", query.lower())
    query = re.sub(r"\s+", " ", query).strip()

    search = arxiv.Search(query=query, max_results=max_results, sort_by=arxiv.SortCriterion.Relevance)

    jresults = list()
    for result in search.results():
        r = dict()
        r["entry_id"] = result.entry_id
        r["updated"] = str(result.updated)
        r["published"] = str(result.published)
        r["title"] = result.title
        r["authors"] = [str(a) for a in result.authors]
        r["summary"] = result.summary
        r["comment"] = result.comment
        r["journal_ref"] = result.journal_ref
        r["doi"] = result.doi
        r["primary_category"] = result.primary_category
        r["categories"] = result.categories
        r["links"] = [str(link) for link in result.links]
        r["pdf_url"] = result.pdf_url
        jresults.append(r)

    if len(jresults) > max_results:
        jresults = jresults[0:max_results]

    # Save to cache
    fh = open(fname, "w")
    fh.write(json.dumps(jresults))
    fh.close()
    return jresults

#### End of find_papers_arxiv ####

##### Begin of generate_images #####

from typing import List
import uuid
import requests  # to perform HTTP requests
from pathlib import Path

from openai import OpenAI

def generate_and_save_images(query: str, image_size: str = "1024x1024") -> List[str]:
    """
    Function to paint, draw or illustrate images based on the users query or request. Generates images from a given query using OpenAI's DALL-E model and saves them to disk.  Use the code below anytime there is a request to create an image.

    :param query: A natural language description of the image to be generated.
    :param image_size: The size of the image to be generated. (default is "1024x1024")
    :return: A list of filenames for the saved images.
    """

    client = OpenAI()  # Initialize the OpenAI client
    response = client.images.generate(model="dall-e-3", prompt=query, n=1, size=image_size)  # Generate images

    # List to store the file names of saved images
    saved_files = []

    # Check if the response is successful
    if response.data:
        for image_data in response.data:
            # Generate a random UUID as the file name
            file_name = str(uuid.uuid4()) + ".png"  # Assuming the image is a PNG
            file_path = Path(file_name)

            img_url = image_data.url
            img_response = requests.get(img_url)
            if img_response.status_code == 200:
                # Write the binary content to a file
                with open(file_path, "wb") as img_file:
                    img_file.write(img_response.content)
                    print(f"Image saved to {file_path}")
                    saved_files.append(str(file_path))
            else:
                print(f"Failed to download the image from {img_url}")
    else:
        print("No image data found in the response!")

    # Return the list of saved files
    return saved_files

# Example usage of the function:
# generate_and_save_images("A cute baby sea otter")

#### End of generate_images ####

Additional Information
Used pip install autogenstudio
running on windows 10
python version 3.11
AutoGen Studio  CLI version: 0.0.44a

check me to on a mac, scratch fill skills.py contains only one skill

Hi @loke-bager,
Thanks for noting this. Can you confirm the following:

Each time you add or update a skill, it does not get automatically added to your Workflow.

Update workflow

Build -> Workflow -> Agent Workflow -> Agents ... add skill to agent .. save.
At this point, the workflow now has the skills you have created.

Note that a workflow is a snapshot/copy of the skills/model you add. Each time you create a new skill/model the workflow needs to be updated.
Also, how are you inspecting skills.py? e.g. inspecting scratch folder etc?

I tried this and it solved the problem for me. Thank you very much for the feedback. I was not aware of this flow.

check me to on a mac, scratch fill skills.py contains only one skill, running from source

have you been able to solve it? I am using AutoGen Studio v0.0.54, adding new skills doesnt seem to add them to skills.py


Creating a root screen for all preference screen components.

Part of an ensemble:

https://github.com/AppFormer/uberfire/pull/716
https://github.com/kiegroup/kie-wb-common/pull/894

jenkins retest this


Common issues resource

want to make this to help students fix their own code

FAQ tab on website

forgot to reassign
mixed and matched base and tidyverse
forgot parentheses  (error unexpected "(")
other common error messages - object not found - you misstyped the object name or never assigned it

"could not find function 'x'"

Didn't load library containing said function
Misspelled function

This is done https://github.com/jhudsl/intro_to_r/pull/152


Add module for custom IAM roles

This change adds a single, custom IAM role as a proof of concept for managing custom IAM roles through Terraform. This requires that the Cloud Build service account includes a role binding for roles/iam.roleAdmin. I have manually imported this role in sandbox. And, the plan applies with no changes.
NOTE: IAM recreations have a higher stakes than some resources.
See the warning from: https://registry.terraform.io/providers/hashicorp/google/latest/docs/resources/google_project_iam_custom_role

Warning:
Note that custom roles in GCP have the concept of a soft-delete. There are two issues that may arise from this and how roles are propagated. 1) creating a role may involve undeleting and then updating a role with the same name, possibly causing confusing behavior between undelete and update. 2) A deleted role is permanently deleted after 7 days, but it can take up to 30 more days (i.e. between 7 and 37 days after deletion) before the role name is made available again. This means a deleted role that has been deleted for more than 7 days cannot be changed at all by Terraform, and new roles cannot share that name.

This change is

Related to:

https://github.com/m-lab/etl/issues/1092


Remove "optional" TypeScript support

The original config by antfu had TypeScript as a optional choice, but in our case (roblox-ts), TypeScript is mandatory. Refactor any code to remove these optional choices for typechecking.

Closing this as it's no longer necessarily the case. Although yes, this is designed for roblox-ts, I have also started to use it in a couple places (e.g. @isentinel/dict-roblox), where I may not necessarily have typecript requirements. TS should still be the defaults, but it can be autodeteted (like it mostly currently is), and therefore the optional requirements can make sense.


Xmake-io.Xmake version 2.5.5

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@ItzLevvie Why is this happening now?!

You'll have to update the version folder using GitHub Desktop.

I could have thought of that myself. Thanks!


Function fail throw an error

node_modules/.pnpm/banditypes@0.2.2/node_modules/banditypes/dist/cjs/index.cjs:19
const fail = () => "bad banditype"();
                                  ^
TypeError: "bad banditype" is not a function

why trying to execute a function call on a string ?
https://github.com/thoughtspile/banditypes/blob/0b992f7e9ea04894fde8c1215eb6e85e1fcaefea/src/index.ts#L24

It's quirky, but it saves 20 bytes, and the errors don't contain any useful detail anyways. I might reconsider this design decision later.

I'm not sure to understand here but is this not enough ?
export const fail = () => { throw new Error("bad banditype") };


Updated versions

Updated versions of charts and Memgraph.

The test failed because I needed to set up a new license for testing the DX stuff.


Fixing EOL replace. Fixes #54

Tested out the original issue from #51 and this fix doesn't re-open the it.

Fixes #54.

// @pkese @dwelle

Doesn't fix. Here's me redo-ing what the replace does:

It seems the /\r?\n/g replaces stuff that it shouldn't, so there's certainly bug in scanInRange or its replace function.

@dwelle, just to make sure i understood, the patch above didn't fix.
But reverting back to this does correct?
        if (config.end_of_line && config.end_of_line in lineEndings) {
            const preferredLineEnding = lineEndings[config.end_of_line];
            const buffer = editor.getBuffer();
            buffer.setPreferredLineEnding(preferredLineEnding);
            buffer.setText(buffer.getText().replace(/\r?\n/g, preferredLineEnding));
        }

@vmuriart correct

In your case, from which line ending are you going, and to which line ending are you going? The case i tested is /n to /r/n which seemed to be the general case that @pkese reported to be the issue (when line ending lengths are different).

fair warning, my experience with lineendings is very limited, so forgive me if it sounds somewhat ignorant on my side :smile:

tl;dr: This may be used as an example to write the correct patch
This patch doesn't work when going from CRLF to LF.
At this point this pr isn't the correct fix to address both issues #54 without reverting #51.
Below going to document my findings so far in case it gives anyone else a direction to work on.
The underlying issues with buffer.setText(buffer.getText().replace(/\r?\n/g, preferredLineEnding)); is that setText destroys the and recreates the buffer with the next text and in the process loses the position of the cursor.
I tried extracting the position of the cursor before setText is executed by using go-to-line as a reference, but always returns to the top row instead of the called line
		if (config.end_of_line && config.end_of_line in lineEndings) {
			const preferredLineEnding = lineEndings[config.end_of_line];
			const buffer = editor.getBuffer();
			const currentRow = editor.getCursorBufferPosition().row;
			buffer.setPreferredLineEnding(preferredLineEnding);
			buffer.setText(buffer.getText().replace(/\r?\n/g, preferredLineEnding));
			editor.setCursorBufferPosition(currentRow);
		}

The problem with  buffer.backwardsScanInRange(/\r?\n/g, ...); is that we are pretty much just shooting in the dark hoping that it works without really understanding what its meant to do. Contrary to my pr it seems that backwardsScanInRange was the correct function to use but it isn't doing what we think it should do.
The one that I thought would do it for us was going to be buffer.replace(/\r?\n/g, preferredLineEnding)); but even on a function that should be pretty standard on its application, the output wasn't what I had expected and still returned funky text. This may indicate that there is something wrong with the buffer and how it does replace.
While writing all this, I noticed the applyChange that seems to be able to normalize the lineendings as we want to.
As a side note, I think the issue wasn't introduced by Atom 1.1 release, but rather by the fix that was introduced for #51. I'll try working on this again later next week. @sindresorhus feel free to close this pr if you feel it appropriate to do so.

My case was from LF (\n) to CRLF (\r\n). I'd be happy to look into this, but am under time pressure, too, so not sure when I get the chance.

Closing in favor of #61.


Different images on each walkthrough page?

How would I implement a different picture on each walkthrough page. so that when they scroll right or left, they see a different picture?
When I edit the image in the onBoardingViewController, it just changes the picture in the middle.

Maybe if I could just change the entire background image of each page?
Also, I'm using Swift.
Thanks

The current implementation only supports a single background image. If you want to have multiple, you'll need to alter the underlying code, but it's not likely to be a feature in the near future.

Do you know how to go about implementing it? I'm trying a few things, but maybe if you could lead me in the right directon. Really need this feature!

You would want to add a full-screen image view to the back of each page, under the text/icons.

Hmm Could you help me a bit more with how this would be done. I tried adding a UIView through code, but yea not working. Still not exactly sure where it would go.

nevermind, was able to figure this out a different way. Thanks!

Cool, good to hear!


Fix ci build issue for ads1145s0x

Fixes a bug in the driver and fixes the CI build for this driver.
The CONFIG_ADC_ADS114S0X_GPIO driver is dependent on CONFIG_ADC being enabled and CONFIG_ADC_INIT_PRIORITY > CONFIG_SPI_INIT_PRIORITY
These issues were discovered while I was working on https://github.com/zephyrproject-rtos/zephyr/pull/63019

@keith-zephyr I rebased to resolve a merge conflict with main.


Randomized abilities

Calc is not configured with randomized abilities in mind for player pokemon. // False, I found the setting. I was silly
Pokemon base abilities can't be changed to any ability. // False, I found the setting. I was silly
Import from save button does not import randomized abilities. // Still true

will fix the randomized ability importation from the savefile.

Fixed, i also sync a bit some gamesettings with the calculator in the way


Waila Integration for Custom Fluids is a bit off

First, thanks for solving my Fire Water issues.
Second, now I see this in Waila. Solveable?
https://ibin.co/36VCa3xM6b0r.png
It calls firewater Lava, it has 7 in parenthesis, and says the rate is 0x. It is working though. The fluid under the barrel is a flowed block of fire water one away from the source block that's in the center of a 3x3 grid of crucibles, but they all say the same thing, even the one in the center.
I copied your pastebin of the soon to be new default heat registry JSON.

Well the Liquid (Lava) means that the cobblestone is turning into lava. I don't know what's going on with the rate though, I'll see if I can recreate that.

That's weird, I can't seem to reproduce this. You wouldn't happen to be running this on a server, would you?

Yeah, I realized about the Lava pretty soon after I posted that. And yes, I am running it on a server.

And I'm not seeing it now either. So...idk. I'll let you know if I find a way to reliably reproduce it. I feel like I updated my client side settings as the same time as the server side, but I'm using a different computer atm, so I can't be 100% sure.
Thanks for looking into it.

Alright so I'm assuming the problem is that your configs aren't the same on the client as they are on the server. That would cause Waila to show something different from what's actually happening.


Help with authentication

Hi, I've been trying to authenticate using your guide to no avail.
As per the below:
fauxapi-auth: <apikey>:<timestamp>:<nonce>:<hash>

For example:-
fauxapi-auth: PFFA4797d073:20161119Z144328:833a45d8:9c4f96ab042f5140386178618be1ae40adc68dd9fd6b158fb82c99f3aaa2bb55

I have the following code:
apisecret="longsecrethere"
timestamp=`date +%Y%m%dZ%H%M%S`
nonce="**what is this and how do i get it?**"
hash=`echo ${apisecret}${timestamp}${nonce} | shasum -a 256 | tr -d "[:space:]-"`

curl -X GET --silent --insecure --header "fauxapi-auth: PFFAwZz0dFMaX1nrlr095Ei2:${timestamp}:${nonce}:${hash}" "https://<IP_ADDRESS>/fauxapi/v1/?action=rule_get"

With the above I'm getting failed auth. Any help on how to use the mentioned  is appreciated

Hi - sorry to hear you are having a hassle with this, I think the source of your trouble here is that you are not specifying the timestamp in UTC  with the --utc flag in the date command line, I suspect I need to document this better.
I'd encourage you to enable debug mode by adding &__debug=true to the end of your URL, this way you get a lot more detail about why the auth request is failing.   It's likely you also want to pipe the curl return data through jq to pretty print the JSON response data so you can human-read it easier.
I note that you are asking a question about what a "nonce" value is.  A nonce value is simply nonsense random data that exists so that no two auth tokens will be the same.
However, you can save yourself a lot of time by just looking at the supplied BASH library fauxapi_lib.sh - the bash function you are looking for is below.
fauxapi_auth() {

    fauxapi_apikey=${1}
    fauxapi_apisecret=${2}

    fauxapi_timestamp=`date --utc +%Y%m%dZ%H%M%S`
    fauxapi_nonce=`head -c 40 /dev/urandom | md5sum | head -c 8`

    # NB:-
    #  auth = apikey:timestamp:nonce:HASH(apisecret:timestamp:nonce)

    fauxapi_hash=`echo -n ${fauxapi_apisecret}${fauxapi_timestamp}${fauxapi_nonce} | sha256sum | cut -d' ' -f1`
    fauxapi_auth=${fauxapi_apikey}:${fauxapi_timestamp}:${fauxapi_nonce}:${fauxapi_hash}

    echo ${fauxapi_auth}
}