Frameworks

Lots of changes in this one:

The data format for hosting changed from pickle to compressed json (more secure and 3x smaller)
This requires that the add_protein_attributes method returns pure python objects. They are converted to tensors at a later stage.
That means we might just fit AF-SwissProt for hosting as well.
I removed all dependencies on pyg and torch, we're now framework independent (they are only imported if necessary)
pyg.download, pyg.extract_tar, torch.load and torch.save are replaced with custom functions in torch_pdb.utils.io. load and save handle pickle and json.gz files, download_url has a progress bar now ;)
Added testing for the downloads as well. The number of downloads is patched to a small number where possible. Takes ca. 2min for all datasets.
The whole graph conversion part from TorchPDBDataset moved to torch_pdb.representations.graph. I also implemented a point cloud representation. Easily extendable (e.g. voxel and surface), each Representation is passed the list of protein dictionaries and can do whatever it wants with it. They also export to different DL frameworks.
we now support pyg, torch, dgl, nx, and tf
these need proper testing though
the name argument is not necessary anymore (the files are named after the parameter choices)
The API changed a little bit. I separated raw data, representation, and frameworks, because I thought it makes sense. Looks like this now:

from torch_pdb.datasets import ECDataset
ds = ECDataset().to_graph(eps=12).pyg()
ds = ECDataset().to_graph(k=5).dgl()
ds = ECDataset().to_point().torch()
...

But I'm not 100% sure with this, let me know if you prefer another style, can be easily adapted.
fixes #27 fixes #23 fixes #26

maybe wait with the merge until I have finished the release pipeline, I'm finding smaller bugs here and there still


[Bug]  Tags and orther column display incorrectly when tag is empty

Search before asking

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

What happened
When tags of Application is empty, each column of data in the job row of the home page is displayed incorrectly.
StreamPark Version
1.2.4/dev
Java Version
1.8
Flink Version
1.13
Scala Version of Flink
2.11
Error Exception
TypeError:Cannot read properties of null(reading 'split')

Screenshots

Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

indeed a bug, thanks for you fix it.


Adding jsbeautifier to pyproject.toml to fix ModuleNotFoundError

PR Checklist
When Starlite is installed with starlite[full] and starlite run is used, the following exception is raised
ModuleNotFoundError: No module named 'jsbeautifier'

This PR adds jsbeautifier to the "full" section in pyproject.toml and also does poetry update

[ ] Have you followed the guidelines in CONTRIBUTING.rst?
[ ] Have you got 100% test coverage on new code?
[ ] Have you updated the prose documentation?
[ ] Have you updated the reference documentation?

@all-contributors add @Alc-Alc code


pyppeteer

Can someone add pyppeteer ?
https://github.com/miyakogi/pyppeteer
It's a port of puppeteer in python. Seems to be dead but still usable.

Wow this looks really cool! @stolati I'm happy to accept a PR for this, if you want to make one.

@stolati Will you be making a PR for this?


Refs #RHINENG-2640 - set reporter to "discovery" or "satellite"

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

Comparison is base (66e92f1) 78.79% compared to head (f488ca9) 78.79%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #71   +/-   ##
=======================================
  Coverage   78.79%   78.79%
=======================================
  Files          23       23
  Lines         613      613
=======================================
  Hits          483      483
  Misses        130      130

Flag
Coverage Δ

unittests
78.79% <100.00%> (ø)

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

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

With confirmation from this Jira comment, I suppose we can merge this PR to main branch now.
Also, just saw this new fix https://github.com/RedHatInsights/yuptoo/pull/72 , I guess we better merge after it.
Hi @patilsuraj767 , what do you think?


Failed to create CloudWatch entities due to missing log group

When first applying the terraform code, the CloudWatch metrics and alarm failed to be created because the log group does not yet exist. I had to manually trigger the created Lambda function so the log group got created and rerun the terraform code to finish creating all CloudWatch entities that depend on the log group existing.

Thanks for pointing this out. I was adding things iteratively and must have incorporated the CW metrics/alarm objects after I had already created and tested the Lambda function. I've updated lambda.tf and alarm.tf to explicitly create the log group and ensure it exists before the CW metric filter is added.


Impossibility To Make Chaptgpt Answer Me In My Obsidian (With Systemsculpt Ai).

Describe the question
Even though I use the correct API (i think), I have the latest version of obsidian and the plugin. I have an issue: I never get an answer from GPT in the conversation. Thanks for your answer.

I'm having the same issue. I'm using a localhost via LMStudio. Everything is properly configured and was working just fine until the most recent updates. Now there is no response when using the chat.

Interesting; I will look into this and do testing with new vaults/configurations to see what the problem is. Have either of you tested/tried it out on the latest 0.6.2 update to see if it's resolved?

i had the update, but chat wasn't working until i closed the chat and used the command palette to open a new chat. works now. thanks.

Interesting; I will look into this and do testing with new vaults/configurations to see what the problem is. Have either of you tested/tried it out on the latest 0.6.2 update to see if it's resolved?

It isn't. I guess I'm doing something wrong. With the new version I opened a new chat, ask something, it doesn't answer. Maybe it comes from my gpt API that isn't correct ? But from what I see it isn't the case either. Could you help me out maybe ?

Hi @BraksWord - I'm going to need more details to help you out. Which endpoint are you using, OpenAI / Grok / OpenRouter / Local? Which model? If you're using an API, do you have enough purchased credits with that API provider in order to use it? A screenshot or a few would really help out here. Has it worked in the past? Etc.

Hey, @SystemSculpt thx for helping me. There are as much information as I can give you.
I use Open AI, I get the API there (see picture below), (FYI, I'm not paying anything, and I think the problem might come from here ?).
It has never worked in the past by the way, and here is what it looks like (another picture below).
But, the fact that I can see a price, and that you asked me if I had purchase credits (and I didn't), make me think that I was the problem 😅 I thought I could Open AI inside Obsidian, like with chat GPT...😅

Thank you


Optimize the file management of GitHub Actions Windows builds (OSOE-156)

The Windows builds of our GitHub Actions workflows are always a lot slower than the Ubuntu ones for every operation, even checkout (at least 2-3x slower, all the way up to 40x sometimes). Let’s try to make this better by tackling file management-related Windows issues.

See this thread for a discussion about this with various tips. But this thread is now lost due to GitHub deleting the community forum, see: https://github.com/orgs/community/discussions/30387
Check if we can configure Windows Defender and if it makes GHA builds faster: Check if under Windows we can configure Windows Defender to exclude devenv.exe, MSBuild.exe, and dotnet.exe from the checked processes, as well as the folder of the checkout directory, and if that makes builds faster.
Related issue.
Windows with a better filesystem

To be checked
See the attached spreadsheet for measurements.

[x] Disabling Windows Defender by @0liver

This doesn't seem to have any noticeable effect, and especially not a positive one.

[x] "While there is no ext4 driver for Windows there is WinBtrfs for Btrfs, another major Linux file system. Supposedly it’s better than NTFS though worse than ext4. None of the other major Linux file systems (xfs, f2fs, reiserfs, reiser4) have Windows drivers." by @DAud-IcI

Tried WinBtrfs out on this commit, see measurements in the spreadsheet. If anything, it's slower by about 10%. Corresponding GHA commit.

[x] NTFS settings:

[x] "On NTFS playing around with fsutil might be helpful, though I assume the VM maintainers have already did this. If not, should open a PR on actions/virtual-environments and request it." by @DAud-IcI.

Needs reboot, so should rather be done in the image. Since rebooting a GitHub-hosted runner doesn’t work (see this run), none of these settings can be changed. There doesn't seem to be a way to run startup scripts or anything like that.
Shortnames are explicitly enabled in the GHA image.
See this script that I used to set NTFS settings.

[x] Similar config recommendations.

[x] "As for git performance, looking for a different command line client based on libgit2 might help. (actions/checkout@v3 just front-ends normal git) Or perhaps the GitHub CLI which uses a different re-implementation called go-git." by @DAud-IcI

The checkout is slower than under Ubuntu (though lately it seems to be "just" at most 50% slower, and usually not too much slower) it's still just 30s-1m out of a 16-30m build. Even if we optimize this a lot, it won't really matter. So, I don't think we should put any effort into reinventing the git wheel, which seems like quite a lot of work anyway.

[x] Virtual file system with better "lot of small files" support

Solid File System was perhaps something like this, though it doesn't seem to be an active project. It seems the successor is CBFS Storage. It can create virtual drives too. It's not open-source though, so we can't include it in our open-source actions (a license is $1500).
WSL 2 can mount ext4 drives, including a VHDX file.

This is available under Windows 11 Insider Build but can also be installed from the Microsoft Store too. You still need Windows 11, though (the indicated minimum requirement of "Windows 10 version 22000 or higher" is actually Windows 11).
Here's a blog post about how to create a VHDX file to begin with and how to mount it. Here's an answer about the same thing with CLI samples.
setup-wsl action. Note that this is only WSL 1 because the currently used Azure VMs don't support nested virtualization..
Comment about manual setup but the WSL enable shouldn't be necessary anymore. PS-based installation is here.
Server installation for WSL.
See this script with basics of WSL drive mounting. However, this can be nicer by using New-VHD and Mount-VHD, which need Hyper-V enabled (which is necessary for WSL 2 anyway).
WSL 2 is not possible to use under GitHub-hosted runners.

[x] Other performance increase options?

Nobody seems to care much about .NET build perf on GHA.
We've tried NuGet and NPM caching before without benefits.
BuildJet seems interesting, but the thing they don't offer is Windows builds.
No other ideas...

Measurements.xlsx
Conclusion
Summary of findings:

Disabling Windows Defender doesn't help.
Using a pluggable virtual file system (WinBtrfs) is slower.
Due to not being able to reboot, we can't change NTFS settings.
Git performance is not a big issue.
Perhaps ext4 would be better as a virtual file system, but it's impossible to use on GitHub-hosted runners.

Next steps:

Tuning NTFS settings can work with self-hosted runners, using our own image. Mounting an ext4 virtual drive with WLS 2 too, for that additionally we need to use an Azure VM type that allows virtualization. I've written scripts for these (or the basics of these) that may be used during packing our own images.
BuildJet sounds promising, and I'm testing it, but it's only for Ubuntu.

In a CI environment it should be feasible to disable Windows Defender altogether, no?
E.g. from an administrative PS shell (source):
Set-MpPreference -DisableRealtimeMonitoring $true

Here's a serverfault answer that does an additional step; might be helpful.

Perhaps not completely disable, I'd err on the safe side by using only the above-mentioned configuration.

Well, this is disappointing, also CC @DAud-IcI and @BenedekFarkas. See my findings and conclusions above. Do you have any other suggestions, perhaps? I'm out of ideas.
It seems that any Windows tuning option we have left necessitates using self-hosted runners.

Well, that's depressing. I don't have other ideas, sorry.
By the way the WSL angle isn't really clear to me. Isn't WSL just testing on Linux with extra steps? WSL1 is basically reverse Wine and WSL2 is a nice frontend for Hyper-V running a real Linux kernel. So it seems to me that going through that you wouldn't really test a Windows application and wouldn't detect any Windows-specific problems.

It is :(. I'd have used WSL only to mount an ext4 virtual drive as a normal Windows drive (which doesn't seem to be possible from under Windows otherwise, but which is a fully supported scenario with WSL 2) and have the working directory there. So the build and test execution would all happen under Windows, just using ext4 as storage. (Hoping that it provides a performance gain over NTFS with our usage patterns.) I ultimately did that with WinBtrfs too. The results I got there hint that perhaps ext4 wouldn't offer advantages either. (My suspicion is that this is due to the old hardware being used for runners.)

I don't have anything to add, unfortunately. Just that I remember that we had a problem with attached drives on Vultr too and build performance tanked due to bad I/O performance, that's we had to switch larger VMs just for the sake of the larger NVMe storage, not the CPU/RAM...

I put some more thought and research into this whole question, but nothing, really.

Nothing worked here, unfortunately. While I'm still testing BuildJet and larger GH-hosted runners, optimizing Windows builds didn't yield any results, so closing this.


[BUG] - title

Description
Error occurred when executing SDPromptReader:
invalid literal for int() with base 10: 'None'
File "C:\Users\rober\Downloads\AI Creations\ComfyUI\execution.py", line 153, in recursive_execute
output_data, output_ui = get_output_data(obj, input_data_all)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\rober\Downloads\AI Creations\ComfyUI\execution.py", line 83, in get_output_data
return_values = map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\rober\Downloads\AI Creations\ComfyUI\execution.py", line 76, in map_node_over_list
results.append(getattr(obj, func)(**slice_dict(input_data_all, i)))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\rober\Downloads\AI Creations\ComfyUI\custom_nodes\comfyui-prompt-reader-node\nodes.py", line 161, in load_image
seed = int(
^^^^
I use impact wildcard nodes for my prompt strings/wildcards. Can I assume this is incompatible with the prompt reader?
This workflow was attempting to load an image that I have genned with verifiable metadata using the batch loader connected to the prompt reader.
Reproduction steps
No response
Image file
No response

looks like this was due to the complexity of my workflow. Removing all impactwildcard nodes and just using a SINGLE clip text encode for positive and a single for negative, I was able to get the SDPromptreader to work successfully. So i'm thankful it works, but it seems like this is severely limiting.

Instead of using the Prompt Reader node to extract metadata from your complex workflow, you can use a Prompt Saver node to save images. This ensures metadata reading is not affected by the complexity of your workflow, and it also allows you to generate images that are compatible with other metadata reading tools.


help: Fix copy-paste bug in wildcard mention documentation.

Bug recently introduced in 8264b8cde08563a1fd1ca20a37f38614fa3c098e. (#27738)

Merged, thanks @alya!


Add company API

We made a fork https://github.com/hinthealth/freshdesk-ruby that we will make improvements over this gem


CNAMResponse model

Create a new type of Response called CNAMResponse which supports all of Voip Innovations API methods for it.

This is done.


[docs fix] update "Module Resolution Strategies"

update "Module Resolution Strategies" to reflect how TS actually works
Partially addresses https://github.com/microsoft/TypeScript/issues/39555#issuecomment-656994164
cc @DanielRosenwasser and @orta, the more i look at the source code for this behavior the more it feels like this cannot be what is intended given our strong preference for node style resolution instead of classic. then again, I don't really know what went into writing that code.
but it looks wrong at first glance, bc it appears to opt in all ES20XX targets into classic module resolution... which doesnt seem like what we want??

reviews accepted!

These changes are in the v2 site, and this repo is getting archived 👍


I have a merge request that fixes a bug

What's the bug?
If the Steam version of PVZ starts with popcapgame2 as the process name, the software will not execute as expected.
How to found the bug?
I have a computer running Windows 10. After updating it yesterday and opening PVZ, I tried to run this project but found that the process could not be recognized. Through program debugging and checking the information in Task Manager, I discovered that the launch process was popcapgame2, not popcapgame1.

But when I thought it was an intermittent update of PVZ, I used another computer running Windows 11. In Steam online mode, I ran PVZ, and by checking in Task Manager, I found that the launch process was still popcapgame1.

How to solve the bug?
Considering the existence of different process names, I added an array that includes the process names in the submission. I also handled the statements where appName needs to be checked for equality with constant values.
I will submit a merge request later.

Here are the pr


[Request] Non-pink slime recipe options

I'm not a fan of industrial slaughter in MC. I play an animal friendly game. I don't even like mob spawners. I'd like to see a way to create a pink slime substitute or have an alternative in recipes, even if it's only a config option.

Use Craftweaker to change the recipes. Making config options to change recipes are bit tedious, I have done it once, I don't want to repeat the experience.

As reasons go, that's pretty reasonable. But, it was worth the ask.
Thank you for the great work and for the quick and considered response.


Make env overridable?

Thanks for this project! I've been finding it very useful.
I recently ran into an issue when I wanted to add jupyterWith to an existing project's shell.nix, which already contained several dependencies external to the Python environment.
My first thought was to add something like
pkgs.python3.buildEnv.override { extraLibs = [ (jupyterlabWith { ... }) ]; }

to buildInputs in the existing mkShell call, but then I was missing additional deps (e.g. generateDirectory) and env vars defined in (jupyterlabWith { ... }).env.
I'm relatively new to Nix, so I may be missing a standard way to do this, but I was eventually able to solve the problem by making env overridable, and then replacing mkShell in my existing expression with (jupyterlabWith { ... }).env.override { ... }. Here is my proposed change:
https://github.com/mcwitt/jupyterWith/commit/bea6658d3686f5fb7cc57b0129adaf0902600f0a
Does this seem like a reasonable way to allow composition of jupyterWith with existing environments?

Ah, sorry for the noise I think I just figured it out. What I was actually looking for was overrideAttrs (not override) which already exists on all derivations (including env). So I have something like
(jupyterWith { ... }).env.overrideAttrs (oldAttrs: {
  buildInputs = [ ... ] ++ oldAttrs.buildInputs;
  shellHook = ''
     ...
  '' + oldAttrs.shellHook
})

which seems to do what I want.
Sorry for the noise. I wonder if it might be worth adding an example like this to the readme, since this might be a common use case. (Happy to add my example if helpful).


button added

Fixes Issue
Closes #1230

Changes proposed

Screenshots

Checklist
Please refer pr #1448 for the proposed changes

[x] You've linked this PR to the correct issue.
[x] You have checked that the code is working correctly.
[x] You ⭐ the repository!

Note to reviewers

Resolve conflicts ASAP and tag me

Honestly someone named lakhan ya something seems to be adding someting again and again, this was te second time I had to resolve the conflict manually

@Souravvmishra Please check now

Please see this snap @Souravvmishra , I can't find any = in data.js line 865

Good to go ?
@Souravvmishra

in data.js , i only added my github username

I can see around 20 lines added to the file data.js. Look into it at the files changed tab.

not added by me @adity2911 ,other pr's are getting merged so branch conflicts,I'll clear the present conflict,then you check again


feat(Dialog): don't override type attribute if set on mat-dialog-close

only default the type attribute on mat-dialog-close if it isn't set
#16909

Can you update the API gold? Just run
yarn bazel run //tools/public_api_guard:material/dialog.d.ts_api.accept

(we have golds for d.ts files to prevent public APIs from being changed inadvertently)

yarn bazel run //tools/public_api_guard:material/dialog.d.ts_api.accept

I ran it and it completed successfully but I didn't see anything change in the source?

The failing tests don't look to be related to the dialog.


Update build scripts for atom-shell name change to electron

Github has changed the name of atom-shell to electron as per
https://github.com/atom/electron/pull/1389

I've only tested this on OSX, not sure what's the process to get this tested on Windows or Linux? Do I need to ask someone else to try or is there some automated way of getting this done?
Cheers

Confirmed working on Linux after changes made in pull request #1899.

I just ran the build on Windows (Cygwin), supposedly successfully, including these changes and the changes in the related pull request by @miraleung.
But I can't run the build via light.sh or builds\lighttable-0.8.0-windows\LightTable.exe. When running light.sh I'm told (in a dialog box, produced by the Atom/Electron executable):

Error opening app

The app provided is not a valid atom-shell app, please read the docs on how to write one:
https://github.com/atom/atom-shell/tree/master/docs

When I run the builds...\LightTable.exe executable I get more details (also in a dialog box):

A JavaScript error occured in the browser process

Uncaught Exception:
Error: Cannot find module 'optimist'
at Function.Module._resolveFilename (module.js:328:15)
at Function.Module._load (module.js:270:25)
at Module.require (module.js:357:17)
at require (module.js:376:17)
at Object. (C:@GitHub\LightTable\builds\lighttable-0.8.0-windows\resources\app\core\main.js:8:16)
at Module._compile (module.js:452:26)
at Object.Module._extensions..js (module.js:470:10)
at Module.load (module.js:347:32)
at Function.Module._load (module.js:302:12)
at Object. (C:@GitHub\LightTable\builds\lighttable-0.8.0-windows\resources\atom.asar\browser\lib\init.js:112:10)

I ran npm install optimist and now I can run script/light.sh and get a (weird) window:

Hey @kenny-evitt, unfortunately I don't have a Windows machine to test this on, but I've done some research to hopefully help track it down.
The optimist dependency is vendored into the repo, so an npm install in deploy/core shouldn't be necessary. If you do a clean clone, you should have deploy/core/node_modules/optimist. Do you know if you had it on the first build?
This also got me wondering why it was vendored, maybe there were custom changes we made to it which might break on Windows, though judging by #1781 it's a vanilla dependency so there's nothing special there. If a clean clone doesn't work, then maybe merge PR #1781 and see if getting the dependency from a package.json fixes the build?
Also, after manually running npm install optimist with the weird window, did LT at least work?
Again, sorry I don't have access to a Windows machine!

Seems okay on OS X, Will confirm this is healthy in conjunction with #1899 on Linux tomorrow, and then try to figure out how to test it on Windows. Ideally it'll build on all 3 platforms before merging.
Thanks for the contribution.

@martinchooooooo I haven't gotten the build to work – at least not consistently. Note that I merged the changes in this PR as well as the ones in #1899.
Details
After running npm install optimist, neither LightTable.exe or light.sh worked, i.e. I saw the same window depicted in my last screenshot and no LT commands worked.
I started looking into how to debug Atom/Electron. I opened my regular Light Table executable to try to lookup a 'DevTools' command to try in the build executable. I noticed the following error in the console – in the regular Light Table:
thread[stderr]: Couldn't read dir<EMAIL_ADDRESS>
I checked the permissions for that directory and there was a "Special" ACL entry denying my account explicitly from accessing it. I checked further and all of the files and sub-directories also had the same deny entry. I wrote and ran a PowerShell script to remove all of those deny entries. I re-ran the build executable but there was no visible change in its behavior, i.e. the LT logo/splash image was still missing and LT didn't respond to any commands. I checked my regular Light Table executable but there were no entries in the console this time.
I kept looking online into how to debug Atom/Electron and I found code that I added to main.js to open the Chrome DevTools window:
 window.openDevTools({
    detach: true
  });

Here's what I saw logged to the DevTools console:
Failed to load resource: net::ERR_FILE_NOT_FOUND
<EMAIL_ADDRESS>Failed to load resource: net::ERR_FILE_NOT_FOUND
<EMAIL_ADDRESS>Failed to load resource: net::ERR_FILE_NOT_FOUND
<EMAIL_ADDRESS>Failed to load resource: net::ERR_FILE_NOT_FOUND
<EMAIL_ADDRESS>Failed to load resource: net::ERR_FILE_NOT_FOUND
bootstrap.js:16926 Uncaught ReferenceError: Cowboy is not defined
LightTable.html?id=1:36 TypeError: Cannot read property 'init' of undefinedmessage: "Cannot read property 'init' of undefined"stack: (...)get stack: function () { [native code] }arguments: nullcaller: nulllength: 0name: ""prototype: StackTraceGetter__proto__: function Empty() {}<function scope>set stack: function () { [native code] }__proto__: ErrorLightTable.html?id=1:36 script.onload
LightTable.html?id=1:37 TypeError: Cannot read property 'init' of undefined
    at HTMLScriptElement.script.onload (LightTable.html?id=1:34)

I found the missing files in sub-directories of the deploy directory. I decided to try rebuilding. I cleaned-up my local repo first via git checkout -- . and git clean -fd; the only changes then reported by Git (via git status) were the clojure and plugins directories. I re-ran build.sh. I ran light.sh this time, instead of the Atom/Electron executables directly, and this time it seemed to work, i.e. I saw the logo/splash image and I could run commands.
I tried opening a Clojure InstaREPL tab but I encountered errors. [I don't have access to all of my notes. I can add the details later.] I was able to open DevTools in LT with the command devtools. The errors referenced the compiled JS code; I found what seems to be a relevant GitHub issue about not being able to use source maps in builds on Windows.
I updated the version of the lein-cljsbuild plugin in project.clj, re-ran build.sh, and ran light.sh and everything seemed to finally work! The previous set of errors weren't being logged in the DevTools console.
However, I had noticed that the build command lein cljsbuild clean had failed, so I started looking into that. Apparently the lein-cljsbuild deprecated lein cljsbuild clean; lein clean can be used instead. I changed build.sh to use the latter, and the clean command now seemed to work, but the build itself now failed and it seemed to indicate that the problem was that the newer lein-cljsbuild plugin uses a newer and incompatible version of ClojureScript.

I had a problem building on another Windows 7 computer – see #1918. But I fixed it and LT seems to running fine.


Rename base.py to more descriptive names.

Fixes #24 (Finally!!)

That import order thing is weird.
j5/components/battery_sensor.py:7: in <module>
    from j5.components import Component, Interface
E   ImportError: cannot import name 'Component' from 'j5.components' (/root/project/j5/components/__init__.py)

Would making j5.components.battery_sensor etc. import these items from j5.components.component instead of from the top-level j5.components be cleaner?


tigera-operator not supporting deploy floatings

HI:
i want to use flaotingip feature, but i found tigera-operator no support fip, the doc recommand deloy via manifest,  i'm confused about this, thanks for any provide information
Expected Behavior

Current Behavior

Possible Solution

Steps to Reproduce (for bugs)

Context

Your Environment

Operating System and version:
Link to your project (optional):

We never added support to the operator because this feature is intended for OpenStack (which doesn't support the operator install anyway). It's disabled by default because it can cause route hijacking (and if it were ever added it would still need to be disabled out of the box) and gives too much power to anyone who can edit pods.
I'm curious what you're trying to do that a Kubernetes Service can't do?

thanks for reply, i'm running vms with k8s+kubevirt+calico , i'm considering how to set floatingip from vms in my env


support for "or" at OSM features tags filter

I'm submitting a feature request
Brief Description
OSM features tags filter is selecting objects with have all listed tags (AND)
Please at a support also OR to select feature with have at least one of the listed tags.
What is the motivation / use case for this feature?
watching different tags in using only one filter to do not have duplicate listings of one object when different tags was changing

+1 - we have a problem of users changing major road classes without community consultation and manually monitoring highway = motorway / trunk / primary / secondary and their *_link  would be tedious.


Fix errgroup context name conflicts

NOTE
Currently, the context returned from errgroup.WithContext overrides the existing ctx variable, and as a result, the ctx is canceled after calling eg.Wait(), and users receive an error. To fix that problem, I've renamed the context returned from errgroup.WithContext as egCtx and pass the name to child templates.

LGTM !


SimpleMenu not support for dark mode

@Gericop I changed
    <style name="Theme.MyTheme"<EMAIL_ADDRESS>    </style>

to
    <style name="Theme.MyTheme"<EMAIL_ADDRESS>    </style>

But the background color of "SimpleMenu" has not changed.

Will be fixed in the next release.

Fixed in v<IP_ADDRESS>.

Thanks:)


Allow for model discrimination without the creation of another BaseModel

Initial Checks

[X] I have searched Google & GitHub for similar requests and couldn't find anything
[X] I have read and followed the docs and still think this feature is missing

Description
Hello,
I want to select the right model based on user input. The user control the value of only one field, and the rest of the model is assumed to be supplied correctly. I read about discriminated unions but the slight issue is that I'm forced to create a class to be able to type one attribute of this class with the actual type union I want.
What I was imagining was a function available in pydantic that does that. Here is my implementation:
def discriminate_pydantic_unions(input: dict[str, Any], model_unions: type):
    """
    This function is purely a pydantic utilitarian function. Read
    https://github.com/pydantic/pydantic/issues/9512 for more context.

    Given a union of pydantic models, will output the one that matches
    with the input. It is based on the discriminated union principle
    https://docs.pydantic.dev/2.0/usage/types/unions/#nested-discriminated-unions
    """

    class Discri(BaseModel):
        results: model_unions

    return Discri(results=input).results

class G(BaseModel):
    one: str

class H(BaseModel):
    one: int

It yields:
discriminate_unions({"one": "h"}, G | H)
>>> G(one="h")
discriminate_unions({"one": 3}, G | H)
>>> H(one=3)

Here is a more elaborate example:
class LikelihoodA(BaseModel):
    likelihood: Literal["a"]
    regularization: Literal["None"]

class LikelihoodB(BaseModel):
    likelihood: Literal["b"]
    regularization: Literal["None"]

class RegularizationA(BaseModel):
    regularization: Literal["a"]

class RegularizationB(BaseModel):
    regularization: Literal["b"]

linear_regression_models = Annotated[
    LikelihoodA
    | LikelihoodB,
    Field(discriminator="likelihood"),
]
regression_models = Annotated[
    RegularizationA
    | RegularizationB
    | linear_regression_models,
    Field(discriminator="regularization"),
]

discriminate_pydantic_unions({"regularization": "a"}, regression_models)
>>> RegularizationA(regularization='a')
discriminate_pydantic_unions({"regularization": "b", "likelihood": "a"}, regression_models)
>>> RegularizationB(regularization='b')
discriminate_pydantic_unions({"regularization": "None", "likelihood": "a"}, regression_models)
>>> LikelihoodA(likelihood='a')

Affected Components

[ ] Compatibility between releases
[ ] Data validation/parsing
[ ] Data serialization - .model_dump() and .model_dump_json()
[ ] JSON Schema
[ ] Dataclasses
[ ] Model Config
[ ] Field Types - adding or changing a particular data type
[ ] Function validation decorator
[ ] Generic Models
[ ] Other Model behaviour - model_construct(), pickling, private attributes, ORM mode
[ ] Plugins and integration with other tools - mypy, FastAPI, python-devtools, Hypothesis, VS Code, PyCharm, etc.

should not this be mentioned in https://docs.pydantic.dev/2.0/usage/types/unions/#nested-discriminated-unions? I feel like pydantic can do a lot, but sometimes we just do not know it can

@MarcBresson,
Feel free to open a PR with a docs update, I'd be happy to review! We already cover TypeAdapters in the docs, but you're right, we don't touch on them in the discriminated unions section.


Advection-diffusion equations coupled to Euler equations

Hello,
the code I work on solves for Euler equations coupled to N advection-diffusion (AD) equations. I started from ex18 and added the advection term of the AD equations along with the numerical method for computing the flux at the cell interface.
Implementing the diffusion term is similar as it requires a numerical flux. MFEM provides functions to compute the discretized diffusion terms (DiffusionIntegrator and DGDiffusionIntegrator) as illustrated in ex14.cpp.
Is it possible to use the same classes (DiffusionIntegrator and DGDiffusionIntegrator) to compute the weak form of the diffusion terms and only add their contributions to the advection equations?
Thanks,
Marco

Hi Marco,
It should be possible to create a separate BilinearForm corresponding to the DG discretization of the diffusion term (as in ex14) and then use that in FE_Evolution::Mult, applying it to the relevant sub-component.
Does this make sense?

hi Veselin,
it does make sense and I will give it a try. If I have questions, I will
let you know.
Thanks,
Marco
On Wed, Mar 6, 2019 at 3:18 PM Veselin Dobrev<EMAIL_ADDRESS>wrote:

Hi Marco,
It should be possible to create a separate BilinearForm corresponding to
the DG discretization of the diffusion term (as in ex14) and then use that
in FE_Evolution::Mult, applying it to the relevant sub-component.
Does this make sense?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mfem/mfem/issues/778#issuecomment-470260406, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEfDChwfuWKCJeI5mziVqupAfsfxnJrYks5vUCKMgaJpZM4bc-TU
.

--
Marc-Olivier Delchini

I'm closing this issue now. Feel free to reopen if you have further questions.

Hi,
I would like to re-open this issue as I have a question related to this topic.
I was able to add a ParBilinearForm to FE_Evolution::Mult as suggested in the previous emails. I leveraged the built-in Diffusion operators and it works well with constant diffusion coefficient in serial. When I run the same code in parallel, I get an error message related to a size mismatch:
Assertion failed: (width == x.Size()) is false:

 --> Input vector size (3648) must match matrix width (3672)

 ... in function: virtual void mfem::SparseMatrix::AddMult(const mfem::Vector &, mfem::Vector &, const double) const

 ... in file: /Users/mxd/ORNL-Research/MFEM/mfem/linalg/sparsemat.cpp:555

My approach is the following: I currently solve Euler equations with one convection-diffusion equation. The diffusion operator D is only defined for the convection-diffusion equation: its size is smaller than the whole system size. In the function FE_Evolution::Mult(), I added the following lines:
 if (k > dim + 1)
    {
      mfem::Vector xk(x.GetData() + k * vfes.GetNDofs(), vfes.GetNDofs());

      mfem::Vector yval(vfes.GetNDofs());

      D.Mult(xk, yval);

      zk -= yval;

      zk += b;
    }

The linear form b stores the boundary conditions for the diffusion operator only. This approach works well in serial even with multiple convection-diffusion equations. In parallel it is not working and I am not sure why.
Would it be safer to define the diffusion operator for all equations (Euler equations and convection-diffusion equations) and set the diffusion coefficient to zero for the Euler equations?
As the convection-diffusion equations are coupled to Euler equations, the diffusion coefficient could be a function of local temperature for example, which implies to recompute the diffusion coefficient  at each new time step. I had a look at the heat equation example that has a non-linear diffusion coefficient but I am having issues with the DGDiffusionIntegrator in parallel. I am now questioning my current approach that uses the built-in diffusion operators.
Should I implement a DGDiffusionIntegrator that computes the diffusion coefficient on the fly (alike the Riemann solver for Euler equations), or is there a trick to make the diffusion coefficient function of the solution itself while using the built-in diffusion operator?
Thanks,
Marco

I have more information to share on this issue. I print the size of the diffusion operator (width and height) and other vectors (size()) of interests when running in parallel on two processors. The output I get is the following:
xk: 4096 0
yval: 4096 0
b: 4096 0
D: 4096 4096 0

The line of the code triggering the error message is D.Mult(xk, yval).
xk is the vector storing the solution vector for the kth equation. The product between the solution vector xk and the diffusion operator D is stored in yval. The LinearForm vector b is used to set the boundary conditions. The last index shown on each line is the processor index. As you can see all operator and vectors have the expected size. The error message I get is the following:
Assertion failed: (width == x.Size()) is false:
 --> Input vector size (4096) must match matrix width (4128)
 ... in function: virtual void mfem::SparseMatrix::AddMult(const mfem::Vector &, mfem::Vector &, const double) const
 ... in file: /Users/mxd/ORNL-Research/MFEM/mfem/linalg/sparsemat.cpp:555

I do not understand why the width of the matrix is found to be 4128 in the assertion. I get the exact same output on the processor index 1 with the same exact error message. In serial, the code runs fine.
The diffusion operator D is defined with the below code that is consistent (I think) with the examples using DG for diffusion operator:
  mfem::ParBilinearForm D(&fes);
  mfem::ConstantCoefficient one(1.0);
  const double sigma = 1.0;
  const double kappa = 0.1;
  D.AddDomainIntegrator(new mfem::DiffusionIntegrator(one));
  D.AddInteriorFaceIntegrator(new mfem::DGDiffusionIntegrator(one, sigma, kappa));
  D.AddBdrFaceIntegrator(new mfem::DGDiffusionIntegrator(one, sigma, kappa));
  D.Assemble();
  D.Finalize();

Any help would be appreciated.
Thanks,
Marco

Hi, @delcmo ,
I believe the trouble you're having is due to the distinction between "finite element objects" and "linear algebra objects".  "Finite element objects" like, ParBilinearForm, ParLinearForm, and ParGridFunction are sized according to fes.GetVSize().  This size is determined by the number of degrees of freedom to fully describe all of the local elements.  However, in parallel, some of those degrees of freedom are shared with other processors.  This leads to shared degrees of freedom being duplicated on neighboring processors.  "Linear algebra objects" on the other hand only contain the degrees of freedom needed to solve the global problem so they do not contain any duplicates and are smaller.
When you see the following lines in ex14p.cpp, a, b, and x are finite element objects and will have larger sizes whereas A, B, and X are linear algebra objects and will be smaller (when run in parallel).
   HypreParMatrix *A = a->ParallelAssemble();
   HypreParVector *B = b->ParallelAssemble();
   HypreParVector *X = x.ParallelProject();

When running in parallel both representations are useful so you may need to keep both of them handy.
Best wishes,
Mark

Mark,
thanks for your help. The use HypreParMatrix class solved my problem.
I am still struggling with:

the differences between 'Par' classes and their serial equivalent and
when/how to use them.
HypreParMatrix class: it is needed for solving the diffusion equation
with DG (ex14p) or DG advection (ex9p), but for example ex18p that solves
Euler equations with DG in parallel does not use HyprePar objects.

I looked at the examples and also the documentation but could not figure
out a clear explanation.
Thanks,
Marco
On Thu, Apr 25, 2019 at 12:29 AM Mark L. Stowell<EMAIL_ADDRESS>wrote:

Hi, @delcmo https://github.com/delcmo ,
I believe the trouble you're having is due to the distinction between
"finite element objects" and "linear algebra objects". "Finite element
objects" like, ParBilinearForm, ParLinearForm, and ParGridFunction are
sized according to fes.GetVSize(). This size is determined by the number
of degrees of freedom to fully describe all of the local elements. However,
in parallel, some of those degrees of freedom are shared with other
processors. This leads to shared degrees of freedom being duplicated on
neighboring processors. "Linear algebra objects" on the other hand only
contain the degrees of freedom needed to solve the global problem so they
do not contain any duplicates and are smaller.
When you see the following lines in ex14p.cpp, a, b, and x are finite
element objects and will have larger sizes whereas A, B, and X are linear
algebra objects and will be smaller (when run in parallel).
HypreParMatrix *A = a->ParallelAssemble();
HypreParVector *B = b->ParallelAssemble();
HypreParVector *X = x.ParallelProject();
When running in parallel both representations are useful so you may need
to keep both of them handy.
Best wishes,
Mark
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mfem/mfem/issues/778#issuecomment-486515534, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABD4GCXSJJTS6XVC6W36PJTPSEXRVANCNFSM4G3T4TKA
.

--
Marc-Olivier Delchini

Hi Marco,
If I understand correctly, you resolved this so I'm closing this issue.
Regarding your more general questions: the Par classes are used for parallel computations, where on each processor we have the serial object and the Par just provides the coupling information between them. You do need these Par classes to solve problems in parallel, in Example 18p that is ParNonlinearForm.
Cheers,
Tzanio


Wait for snapshot to be deleted in DeleteCSISnapshot function

Change Overview
Waits for VolumeSnapshot to be deleted before completing the execution of DeleteCSISnapshot function
Pull request type
Please check the type of change your PR introduces:

[ ] :construction: Work in Progress
[ ] :rainbow: Refactoring (no functional changes, no api changes)
[ ] :hamster: Trivial/Minor
[x] :bug: Bugfix
[ ] :sunflower: Feature
[ ] :world_map: Documentation
[ ] :robot: Test

Issues

#1202

Test Plan

[ ] :muscle: Manual
[ ] :zap: Unit test
[ ] :green_heart: E2E

LGTM

Awesome! Thank you! 😍


FYI - RarityLib 1.1.2 Breaks Ability to Choose cards in game

Reverting to 1.1.1 works.
Using these mods with up to date versions as of 7/3/22
BepInExPack_ROUNDS by BepInEx
BlockForcePatch by willuwontu
BPP by BPP_Team
BSC by BossSloth
CardBarPatch by BossSloth
CardChoiceSpawnUniqueCardPatch by Pykess
CardsPlus by willis81808
CardThemeLib by Root
Classes_Manager_Reborn by Root
CM by XAngelMoonX
CR by XAngelMoonX
DamageIndicators by willis81808
EvenSpreadPatch by willuwontu
ExtraMapObjects by BossSloth
GunProjectileSizePatch by Root
GunUnblockablePatch by Pykess
HDC by HatchetDaddy
ItemShops by willuwontu
KeysCards by Keys
LegRaycastersPatch by Pykess
LobbyImprovements by RoundsModdingCommunity
Map_Embiggener by Pykess
MapsExtended by olavim
MapsPlus by willis81808
MMHook by willis81808
ModdingUtils by Pykess
PCE by Pykess
Performance_Improvements by RoundsModding
Pick_N_Cards by Pykess
PlayerJumpPatch by Pykess
RarityLib by Root
RespawnPatch by willuwontu
Root_Cards by Root
RoundsWithFriends by olavim
SetRounds by Ascyst
TemporaryStatsPatch by Pykess
UnboundLib by willis81808
WillsWackyCards by willuwontu
WillsWackyManagers by willuwontu
WillsWackyMapObjects by willuwontu
ZeroGBulletPatch by TeamDK
ZOMC by Zom_23

yup, missed a line of code in the card spawn patch while i was updating it. bug should now be fixed with version 1.1.3


Use modular builders to build a simple C hello program

This PR changes the structure of the nickel-nix to be a starting point to write modular and composable builders, in order to create packages or devshell. We don't want to rely anymore on mkShell on the Nix side but use pure Nickel abstraction over the simple derivation function.
Content
This PR adds the most basic builder (a bit of sugar over derivation), and clean many parts of the Nickel library. The shell example has been removed, and a purely derivation-based example shell will be added in a subsequent PR. We also get rid of nix_string_hack and use nickel's very new symbolic strings to emulate Nix string contexts.
Next

should we get rid of the sources input type, now that we have symbolic strings and import_file? (If yes, let's do this in another PR)
add a basic shell builder from the devshell/nickel-devshell branch

@thufschmitt @djacu gentle reminder to review this PR :)


Use void to indicate that parsing will never return an Err

Make it more explicit, both in documentation and the type system, that parsing will never error.
(I would've used ! but it's unstable)

It seems the ! type is stabilized for this usecase. Do you want to update the PR?


ci: hand over 'dev' to gardenbuild for 'dev' version

How to categorize this PR?

/kind enhancement
/area os
/os garden-linux
What this PR does / why we need it:
With the changes introduced in PR #583 and PR #600, the pipeline needs to supply the version parameter to garden-build.sh in a different fashion and especially honor the dev version string in VERSION.
Release note:

ci: hand over 'dev' to gardenbuild for 'dev' version

/ready


chore: upgrade helmfile to 0.149.0

v0.149.0 is the latest version which avoids the backwards-incompatible logic in https://github.com/helmfile/helmfile/pull/592, but contains the fix in https://github.com/helmfile/helmfile/pull/499 which unblocks me from using export HELMFILE_USE_SELECTORS = true in my bootjob Makefile.

/assign @msvticket

/ok-to-test

/approve

thank you!


Exclude composer vendors and assets

It would be really great to exclude composer vendors and assets. That would mean that we can throw away auto-generated files and external dependencies and fetch them during the build.
E.g.

Add lib/vendor/ to .gitignore (Should I commit the dependencies in my vendor directory? TL;DR no.)
Add auto-generated files from the assets directory to .gitignore
Integrate Travis CI to build release zips with the generated assets and fetched external dependencies

The benefits should be obvious:

No more conflicts in pull requests because auto-generated files were changed
Smaller repository footprint

I would like to put that into a CR but that would need some effort so I'd like to know if it's worth the effort or if you would decline it anyways.

@aheinze waiting for feedback here.

No need for that right now. but might get relevant in the future...


REX-Ray Config StorageDrivers Backwards Compat

This patch should provide backwards compatibility with the old property rexray.storageDrivers. If this property is set, its first element will be assigned to the property libstorage.service if that property is undefined. This is regarding issue #471.

[Current coverage][cc-pull] is 44.32%

Merging [#481][cc-pull] into [master][cc-base-branch] will decrease coverage by 1.41%

@@             master       #481   diff @@
==========================================
  Files             2          2
  Lines           188        194     +6
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits             86         86
- Misses          102        108     +6
  Partials          0          0

[][cc-pull]

Powered by Codecov. Last updated by [ca8bc17...b096884][cc-compare]
[cc-base-branch]: https://codecov.io/gh/emccode/rexray/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/emccode/rexray/compare/ca8bc17b02c5d904804ea4584b184f394c2c923e...b096884d317a4e67668dfd0d885bb671e84eb5e1
[cc-pull]: https://codecov.io/gh/emccode/rexray/pull/481?src=pr

@clintonskitson Can you think of any other properties we should include? This seemed to most obvious. I'm not going to bother with the volumeDrivers or osDrivers properties since those were never set to anything other than the defaults in previous versions of RR.

@akutz I believe we have backward compat built in for other options. As long we are verbosely describing the general options that were set in the log to cover that scenario then I believe the backward compatibility based primarily on storage driver makes sense.


Hope support fiber

https://github.com/gofiber/fiber
⚡ Fiber is an Express inspired web framework written in Go with ☕

Please feel free to submit a custom template for this and I will add to the project.


New feature: EXISTS k1, k2..Kn

The changes that are included in the PR.
Added new command 'EXISTS' and Store function 'GetNoTouch' for getting key without updating last accessed time for that record

Design document, if any.

Information on any implementation choices that were made.
This is implemented as per the Redis EXISTS command.

Evidence of sufficient testing. You MUST indicate the tests done, either manually or automated.

Added integration tests in test folder.

Hi @gauravsarma1992, Updated the code, Please review.

@apoorvyadav1111 there are merge conflicts with your branch. Please rebase your branch and resolve the conflicts.

Hi @gauravsarma1992, resolved conflicts and rebased. Please review.

Hi @gauravsarma1992, thanks for your review. Please merge the branch at your convenience.

@JyotinderSingh can you review as well? I don't think I have write access anyway


Question : Custom parameter with CodeCoverage

Is it possible?
      # Run Unity Testrunner
      - name : Run Unity Testrunner
        uses: game-ci/unity-test-runner@v2
        id: tests
        with:
          projectPath: ${{ matrix.projectPath }}
          unityVersion: ${{ matrix.unityVersion }}
          testMode: ${{ matrix.testMode }}
          artifactsPath: ${{ matrix.testMode }}-artifacts
          customParameters: -debugCodeOptimization -enableCodeCoverage -coverageOptions enableCyclomaticComplexity;generateHtmlReport;generateBadgeReport -coverageResultsPath ${{ matrix.projectPath }}CodeCoverage/

That should work I believe. Have you tried it?
Just note that the path has to be relative to your CI context, which is what is mounted inside the container.

Thank you for answer.
I put the options below, but nothing seems to be created in the CodeCoverage folder.
I searched for it with the find command, but it doesn't seem to be there.
Is there any other way to determine that CodeCoverage has run normally?
customParameters: -debugCodeOptimization -enableCodeCoverage -coverageOptions enableCyclomaticComplexity;generateHtmlReport;generateBadgeReport;assemblyFilters:+* -coverageResultsPath ./CodeCoverage

A few options:

In the logs there may be a mention of using coverage
you could fork test-runner and edit run_tests.sh to issue a status command (git status) or recursive list command (ls -Ralph) to indicate which files are affected.

Thank you for answer.
The cause seems to be that the unity license file expired yesterday.
Thanks for your help.


ValueError: Input contains NaN, infinity or a value too large for dtype('float32').

I've generated embeddings (GloVe and word2vec) on a corpus I have and am attempting to compare their performance using evaluate_on_all.py. I'm able to run the script on the GloVe vectors but when running on the word2vec embeddings I get the following. I've checked the matrix and do not see any nans or infs. Has anyone else had this problem?

Traceback (most recent call last):
File "./evaluate_on_all.py", line 83, in
results = evaluate_on_all(w)
File "/home/ubuntu/miniconda3/envs/attextract/lib/python3.5/site-packages/web-0.0.1-py3.5.egg/web/evaluate.py", line 409, in evaluate_on_all
categorization_results[name] = evaluate_categorization(w, data.X, data.y)
File "/home/ubuntu/miniconda3/envs/attextract/lib/python3.5/site-packages/web-0.0.1-py3.5.egg/web/evaluate.py", line 98, in evaluate_categorization
linkage="ward").fit_predict(words[ids]))
File "/home/ubuntu/miniconda3/envs/attextract/lib/python3.5/site-packages/scikit_learn-0.18.1-py3.5-linux-x86_64.egg/sklearn/base.py", line 410, in fit_predict
self.fit(X)
File "/home/ubuntu/miniconda3/envs/attextract/lib/python3.5/site-packages/scikit_learn-0.18.1-py3.5-linux-x86_64.egg/sklearn/cluster/hierarchical.py", line 686, in fit
X = check_array(X, ensure_min_samples=2, estimator=self)
File "/home/ubuntu/miniconda3/envs/attextract/lib/python3.5/site-packages/scikit_learn-0.18.1-py3.5-linux-x86_64.egg/sklearn/utils/validation.py", line 407, in check_array
_assert_all_finite(array)
File "/home/ubuntu/miniconda3/envs/attextract/lib/python3.5/site-packages/scikit_learn-0.18.1-py3.5-linux-x86_64.egg/sklearn/utils/validation.py", line 58, in _assert_all_finite
" or a value too large for %r." % X.dtype)
ValueError: Input contains NaN, infinity or a value too large for dtype('float32').

Closing for now. Please let me know if this is an issue still though.


fix: a11y column sorting

Fixes #28
The field a11y is now part of the underlying data (set in MetadataContainer.js) so it can be sorted.

Merged in 1d483202c741697fde42775e3fd46e19f1d39ad2


Better support onChange on keyDown enter

Is your feature request related to a problem? Please describe.
I'm trying to have a modal display below a static date to allow editing it.
Something like that:

I would like the following behavior for the popover:

Submit (and hide popover) when pressing enter
Submit (and hide popover) when choosing a date using the date picker modal
Hide popover when clicking away (using MUI's Popover for that)

I've tried registering for the onKeyDown event of DatePicker, but it fires before onChange, which means that at this point I don't have the new date...
  const handleKeyDown = event => {
    console.log('keyDown', { event });
    if (event.defaultPrevented) {
      return;
    }

    event.stopPropagation();

    if (event.key === 'Escape') {
      hide();
    }

    if (event.key === 'Enter') {
      submit();
    }
  };

I've tried putting the submit logic inside onChange, but onChange fires when opening the date picker modal, which hides the popover and the containing DatePicker element...
Describe the solution you'd like
I'm not sure what the solution should look like. Perhaps do not fire onChange when clicking on the icon to open the date picker modal?

I had some more ideas, but they don't work:

Use a setTimeout to submit. This is very hacky and won't work because submit captures a reference to the current value (see extract below).
Find a way to register my handleKeyDown after the date picker's textfield... Haven't found a way to do that yet.
What would be nice would be for onChange to include the date and the event that triggered the onChange. This way I would be able to check if it's for a keypress. onChange(date, event).

import * as React from 'react';
import { Formik } from 'formik';
import { makeStyles } from '@material-ui/styles';
import Popover from '@material-ui/core/Popover';

// SIMPLIFIED

const EditInline: React.FC<Props> = (props) => {
  const [anchorEl, setAnchorEl] = React.useState(null);
  const [value, setValue] = React.useState(props.initialValue);
  const isEditing = Boolean(anchorEl);

  const onEdit = ev => {
    setAnchorEl(ev.currentTarget);
  };

  const hide = () => setAnchorEl(null);

  const submit = () => {
    if (value === props.initialValue) {
      console.log('not submitting, value did not change');
      hide();
      return;
    }

    const payload = { [props.name]: value };
    console.log('submitting', { payload });
    client.patch(props.url, payload).then(
      () => {
        hide();
      },
      () => {
        hide();
      }
    );
  };

  const handleKeyDown = event => {
    console.log('keyDown', { event });
    if (event.defaultPrevented) {
      return;
    }

    event.stopPropagation();

    if (event.key === 'Escape') {
      hide();
    }

    if (event.key === 'Enter') {
      submit();
    }
  };

  return (
    <>
      {props.children}
      <Popover
        open={isEditing}
        anchorEl={anchorEl}
        onClose={() => submit()}
      >

          <DatePicker
            name="value"
            value={value}
            onChange={value => {
              console.log('onChange', { value });
              setValue(value);
            }}
            autoFocus
            disableOpenOnEnter
            clearable
          />

      </Popover>
    </>
  );
};

export default EditInline;

It would be a fairly simple & backward compatible change actually.
In https://github.com/dmtrKovalenko/material-ui-pickers/blob/develop/lib/src/_shared/DateTextField.tsx
Pass the original event to commitUpdates(value, ev) and then forward it to onChange(newValue, ev) as well. Would only require changes in handleBlur and handleKeyPress.

What I ended doing (roughly):
import dateParse from 'date-fns/parse';
import dateFormat from 'date-fns/format';

...

    if (event.key === 'Enter') {
      if (kind === 'date') {
        newValue = dateFormat(
          dateParse(event.target.value, 'dd-MM-yyyy', new Date()),
          'yyyy-MM-dd'
        );
      }
      submit(newValue);
    }

I basically grab the TextField content, parse it, format it, and use it to submit to the backend. This is not a great solution because it basically re-implements the parsing and formatting part of DatePicker...

Closed because of #862

Hey @dmtrKovalenko what will be the plan here? #862 does not give a lot of details...


Fix issue when used with devise case_insensitive_keys and strip_whitespace_keys

Devise downcases and strips keys listed in case_insensitive_keys and strip_whitespace_keys, respectively.
https://github.com/plataformatec/devise/blob/master/lib/devise/models/authenticatable.rb#L63
These before_validations don't get triggered by devise_invitable because it saves without validations. This means, for example, that an email address like<EMAIL_ADDRESS>can be saved to the database. That user won't be able to sign in at all since devise expects the email field in the database to be downcased and stripped!

Thank you

just ran into this issue and was confusedly looking at the source until i realized it had already been fixed
thank you to you both


fix(timer): close timers before Neovim exits

When running debug builds of Neovim which defines EXITFREE, not closing the timers will cause exit to hang whilst Neovim manually frees them.

LGTM.


Ability to use Nested Child Query with parent child inner hits

Hi,
I'm currently using the HasChildQuery class to include additional documents within my result set. I need to include the inner_hits part of the query in order to get the right results but it looks like this ability isn't part of the HasChildQuery implementation when dealing with parent/child documents (https://www.elastic.co/guide/en/elasticsearch/reference/current/search-request-inner-hits.html#parent-child-inner-hits).
Do you know if this is planned to be included in later versions? I saw that inner_hits support was added for other queries but not for parent child ones.

Yes, it is in the plans. inner_hits is quite complex, that's why it's not there yet.


Touch pad is not sensitive enoug

Describe the bug
It takes many passes across the touch pad to move the cursor across the whole screen.
Images/videos/anything that would help
Please attach a log (it's in /tmp/hypr/hyprland.log) and, if crashing, a crashdump (coredumpctl, then coredumpctl info )

sensitivity...?

Im accustomed to being able to move from corner to corner of the screen with the touchpad with just one pass, but in Hyprland it takes 4 passes and in weston it only takes one pass.

yes, there's a sensitivity setting in the config...

ok, good to know.


Garbled characters in Japanese environment

Thank you for developing amazing tool!
If possible, I'm glad that you make this code support for double-byte character languages

.

The garbled characters were resolved by adding a line to encode in UTF-8 before "POST" the json of the original source below.
Is it possible to reflect this line in the source code?

Sorry I've missed this one, does the issue still persist with the enhancements that were made in the latest 1.3.6 release related to encoding?
I'm referring to this from the release notes:
Private function Invoke-IntuneGraphRequest has been updated with UTF-8 encoding content type support. From this version and on, when the script file is properly encoded with UTF-8 with BOM, special characters should now work as expected.


[WIP] Upgrade to clang-9 for scan build

Going a different direction here, going to add a proper --warn option to cmake

You are doing duplicate and conflicting work here currently. You might want to hold off on this until my travis/cmake PR has been merged. (Or at least rebase it on top of that for WIP work)

Thanks for the heads up, will take a look.

Will definitely wait for your changes to drop before doing my updates. Thanks!


System.DllNotFoundException: Unable to load DLL 'DlibDotNetNative': A dynamic link library (DLL) initialization routine failed. (Exception from HRESULT: 0x8007045A)

2020-08-26 18:11:43.163	Error	Error in Main Method - System.DllNotFoundException: Unable to load DLL 'DlibDotNetNative': A dynamic link library (DLL) initialization routine failed. (Exception from HRESULT: 0x8007045A)
at DlibDotNet.NativeMethods.get_frontal_face_detector()
at FaceRecognitionDotNet.FaceRecognition..ctor(String directory)
at FaceRecognitionDotNet.FaceRecognition.Create(String directory)
at Camera_Capture_ConsoleApplication.Program.Main(String[] args) in C:\Users.....\Documents\Visual Studio 2005\Projects\Camera_Capture_ConsoleApplication\Camera_Capture_ConsoleApplication\Program.cs:line 3139
2020-08-26 18:11:43.166	Error	Error in Main Method - Unable to load DLL 'DlibDotNetNative': A dynamic link library (DLL) initialization routine failed. (Exception from HRESULT: 0x8007045A)
at line :
fr = FaceRecognition.Create("C:\WFH_Client\models");
working on some machines and not on some machine
machine config : windows 10 pro 64 bit

@vikrant-bioenable
Could you check this issue?
https://github.com/takuya-takeuchi/DlibDotNet/issues/153

my cpu is intel i5 7th generation...... it works on some machine perfectly...but on some machine showing above error... i had installed VC_redist.x64.exe also. file version - 14.26.28720.3

@vikrant-bioenable
C:\Users.....\Documents\Visual Studio 2005

Are you kidding? Do you use VS 2005?
I doubt your machine does not have VC++ 2017 runtime.

I have installed VC++ 2015-2019 runtime... please see attached image

I am using Visual Studio 2019....just project saved location was vs 2005....... as of your dll not even open in visual studio 2015.....

@vikrant-bioenable
I have a few time to check issue.
Could you give the minimum sample project reproduce issue?

Hello sir , please find my source code build in Visual Studio 2019 - having above mentioned issue at below line -
fr = FaceRecognition.Create(AppRootPath + "\models");
GITHUB LINK -
https://github.com/vikrant-bioenable/DemoProjectFacerecognition.git

Dear Sir.., Please update.. I am stucked here...

sorry, I lost your message.
let me check again.
thanks

Open solution file by VS2019
Debug by F5
Stop on the above break point

And created Hyper-V machine, install only VS2019.
After this, did the avobe steps again.
But there is no issue.

I can not reproduce your issue.

Dear Sir, On my machine also it works very fine.... but on client machine this code fails at fr = FaceRecognition.Create(AppRootPath + "\models");

Just for reference, has the same issue and was solved by using an older build for cpus without AVX instruction set.
See here: https://github.com/takuya-takeuchi/DlibDotNet/issues/153#issuecomment-553400395

Maybe the reason is need to install old Net Framework version like 2.0, 3.0, 3.5,....it's worked for me :D


Refactor ensure_nondefault method for chargeback rate

Fixing: https://bugzilla.redhat.com/show_bug.cgi?id=1552260
Needs to merge with the UI PR: https://github.com/ManageIQ/manageiq-ui-classic/pull/3673
Refactor ensure_nondefault method so that a new default? method is created and improved: super is used instead of default?. The method is not private because we need it in chargeback controller, too.

@miq_bot add_label bug

@miq-bot add_label gaprindashvili/yes

@miq-bot add_label bug

@miq-bot assign @gtanzillo


Creating VM Notebook for an another user of my resource group

Is it possible to create a VM Notebook for another user ?
I have tried but it seems that only the creator has access to the virtual machine

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

ID: d0a39c00-0d4b-eef2-39cc-2ebe0603d854
Version Independent ID: 265dd37f-2299-c762-63be-3c8e5cf504f8
Content: Tutorial: Create your first ML experiment: Setup - Azure Machine Learning
Content Source: articles/machine-learning/service/tutorial-1st-experiment-sdk-setup.md
Service: machine-learning
Sub-service: core
GitHub Login: @trevorbye
Microsoft Alias: trbye

@cconcordet please provide us with the link to the documentation you are following so we can better assist.

Here is the documentation I followed on how to create a VM Notebook :
https://docs.microsoft.com/en-us/azure/machine-learning/service/tutorial-1st-experiment-sdk-setup
I was wondering if it is possible to share the VM with other people from my resource group

#assign:@maxluk

Hello, any update or timeline ?

At Ignite (Nov 4th) we will release updated version of Notebook VM called Compute Instance which will support sharing of the VM between users of workspace.

You may have noticed that this feature hasn't shipped quite yet. Please stay tuned. Thank you so much for your inquiry.
#please-close.


Always use 24-hours locale

Fixed issue https://github.com/firebase/firebase-ios-sdk/issues/1946

Thanks for the contribution @Ramesik!


Update README.md

Add syntax highlight \o/

I think this improves the appearance slightly, but it's not Austral syntax highlighting, which GitHub doesn't support. I'm not sure about using ml and then changing that in the future.

On second thought this is "cheating" but it does improve the visual appearance so I'll merge.

On second thought this is "cheating" but it does improve the visual appearance so I'll merge.

Nice!!


Can a ClockPicker work when it's dynamically loaded?

I'm creating a simple webpage for myself (learning from scratch), which is basically an expandable table with a clockpicker in each row. The clockpicker on the initial row works fine, but on the dynamically loaded rows, it doesn't work.
I'm loading the dynamic rows with a jQuery script, something like this:
<script>
$(function(){
    jQuery('a.newRowButton').click(function(event){

        event.preventDefault();

        var newRow = jQuery(
            '<html stuff>' +
            '<more html stuff/>'
        );

        jQuery('table.table1').append(newRow);
    });
});
</script>

I wouldn't be surprised if there are way better ways to do this, but I'm stil learning as I go here. If you know of a better way to add html code to a page dynamically, I'd love to hear about it.
In the meantime, I'm wondering, is it possible to make the clockpicker work on the dynamically loaded rows as well?

Hi,
have you tried event delegation ?
jQuery doc : https://learn.jquery.com/events/event-delegation/
Modify this line : $('#parent-div').on('click', '.newRowButton', function(event){
};

Well, I found a solution. I fully expect it to be a crappy and buggy one, but for now, it works.
The clockpicker method:
function assignClockPicker() {
	$('.clockpicker').clockpicker({
	    placement: "bottom",
	    align: "left",
	    default: "now",
	    donetext: "Done",
	})
	.find("input").change(function() {
	    handleClockPicker(this);
	});
}
assignClockPicker();

This should be equivalent to having the contents of the assignClockPicker method outside of the method.
Then, at the very end of the function that adds the new row, I simply added assignClockPicker();.
The clockpickers now work on all rows just fine. I would love some feedback on the validity of this construction, because I can't imagine that this is the best, or even a good way to do this.


Adding connectivity check, compilation fix and some code refactoring to verbs

Connectivity check - checking the correctness of the RDMA configuration parameters by pinging on each channel.
Compilation with verbs and without CUDA fix (contrib/verbs only works on GPU #13466)
Code refactoring:
Call done in case of not OK status fix
Replace hardcoded 100 with RDMA_QP_QUEUE_DEPTH

Can one of the admins verify this patch?

This PR fixes the #13466 (contrib/verbs only works on GPU) issue.

@junshi can you please review?

@noaezra , can you confirm here that you're ok with your commit being submitted?

I'm okay with this change, but I don't have a well-informed opinion as to how appropriate it is.  @junshi15 should give his opinion.  Is there a background thread other than the cited bug discussing this change?

@dariavel The patch looks good, except that I have one question above.

@jhseu # "noaezra , can you confirm here that you're ok with your commit being submitted?"
It's ok that my commit is being submitted.

@jhseu @poxvoculi @junshi15
can you please re-run the tests?

Jenkins, test this please.

Thanks @tatatodd . looks like the failure is not related to the PR.
Can we merge or re-run ?

Jenkins, test this please.


As the head tutor, I am able to import new contacts from CSV files to reflect the new batch of students when a new semester starts.

so that I can perform operations on large numbers of contacts at once.


Fix interpolation

Some change in the original Android lib was not added see https://github.com/googlemaps/android-maps-utils/blob/main/library/src/main/java/com/google/maps/android/SphericalUtil.java#L142-L144

@alexpechkarev I created this PR because I found an issue with the interpolation. I compared the original Android function and found this change not being added. Hope you can agree on the change and merge / release a new version with this change. Thanks in advance.

Many thanks, @StanAnsems !


autoresize window on play

function references in hyperx need to look like:
onloadedmetadata=${onLoadedMetadata}

and not:
onloadedmetadata="${onLoadedMetadata}"

also removed some unnecessary torrent.ready checks in the player.

fixes #17

I think this is trivial enough for a merge since no one else is awake :)

Nice fix! I wonder why there's a distinction.


What happens with homekit automations

I have a lot of automations related with the melcloud unit. I think that I delete it from Homekit i will loose a lot of automations. Is posible avoud this?

You need to add device to automations again. I think is not difficult.

Difficult no, but boring and large, yes. In my case I have automations and shorcuts with conditions like 'if open a window, shut down the AC', or inverse 'if the AC is on and open window, first notify and after 30 seconds shut down the AC'. And I have 5 windows... And add scenes and automations when I exit home... At least 40 automations and scenes.. Boring 😔 In verIonn 1.0 I don't see motivation to upgrade, maybe I'll wait to future updates with new features. The last 0.x version works great

If you don’t want run duplicated devices then stay by 0.21.x, In 1.0.0 is only one breaking change.

Ok. Thanks. Is not difficult but for me is a lot of job. And I'm blind and is a slowly job for me. Can I first add the new unit and later delete the old? If it is possible, I think is a possible solution. If I remake the scenes wiyh the new unit before delete the old,the scenes and automation will no automatically deleted. The problem occurs when a scekne is empty or an automation no has a valid condition to start (a dispositive than no rxist)

Yes, you can first add the new device and later delete the old not responsible.

That's good news, it makes it a little easier for me when I decide to update. I suppose that the big change from 0.x to 1.x means that in your roadmap you have planned to add new options and features that will represent an important leap forward and that the publication of version 1.0 is only the first step of what is to come. I will be attentive because right now I need important improvements that encourage me to update. I can also tell you that the plugin in its latest version 0.x has reached an excellent level of excellence. It works perfectly and I use it (and enjoy it) every day. I congratulate you on a perfect job. Right now it is very difficult for me to think about how to improve the plugin from now on, so I will be very attentive.

Its changed version to 1.0.0 because its break previous paired devices.

I know, you say to me a few weeks ago, that any major change (0.x, 1.x...) suposes always delete and connect to home app again the unit.


GoodNight added

GoodNight is in active development and a nice app for automatic temperature/gamma adjustment. It is originally based on GammaThingy.

Thx


Add pytest for imaging_no_curation module

Closing this individual issue as we are working to improve overall test coverage.


[top_earlgrey,doc] Specify open drain / virtual open drain

This addresses the questions and concerns raised in issue #22802.

@OTshimeon and @moidx (or someone from your teams): Could you please check if this is accurate from the silicon side and suffices for the integration teams, respectively?

@jesultra @jettr please use
the type definitions from the pinout table for the purpose of this review https://opentitan.org/book/hw/top_earlgrey/ip/pinmux/doc/autogen/pinout_asic.html.
Thanks

Thanks for your feedback, @jesultra. I see your points; the described model doesn't answer all questions. I'll follow up with the silicon team and update this PR.

@jesultra: I changed the description after iterating with the silicon team. Could you please check if it answers your questions?

Non-lint CI checks intentionally canceled for this doc-only change


Add an option to have the full contents of posts in the RSS feed

I'd like to use m site's feed to syndicate the content to other places, so it needs to contain the full post contents. I know I can adapt the built-in template, but maybe this can be supported with a config option?
I'd be willing to implement this feature as my first contribution.
This makes thue feed rather large, so RSS pagination is also something that might be worth looking into.

Possible dupe of #4071 ?
+1 on implementing this, thanks!

Yep, looks like it's a dupe.


[MGDOBR-809] Add Cloud Provider and Region field to request to create a Bridge

MGDOBR-809
This PR updates the BridgeRequest object to require the user to specify two additional properties:

cloud_provider - The id of the cloud provider from the list available at /api/v1/cloud_providers
region - The name of the region, from the list available at /api/v1/cloud_providers/{id}/regions

By default we configure AWS and us-east-1 as the only supported Cloud Provider/Region combination at this point. This will likely change in the future.
PR owner checklist
Please make sure that your PR meets the following requirements:

[x] Your code is properly formatted according to this configuration.
Run mvn clean verify -DskipTests so that imports are correctly set.
[x] Your commit messages are clear and reference the JIRA issue e.g: "[MGDOBR-1] - $clear_explanation_of_what_you_did"
[x] The layers in the kustomize folder have been updated accordingly.
[x] All new functionality is tested
[x] Pull Request title is properly formatted: MGDOBR-XYZ Subject
[x] Pull Request contains link to the JIRA issue
[x] Pull Request contains link to any dependent or related Pull Request
[x] Pull Request contains description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket

PR reviewer(s) checklist

[ ] Check if code and Github action workflows have been modified and if the modification is safe
[ ] If the modification is safe, add the safe to test label

How to trigger pipelines and use the bots:

Run the end to end pipeline
Annotate the pull request with the label: safe to test. If you want to run the pipeline again, remove and add it again.

Rebase the pull request
Comment with: /rebase.

Deploy BOT
Comment with `/deploy <dev,stable> when the PR has been merged to deploy to a target environment.

@sutaakar I'd like your guidance on the best way to update the integration tests to include the new requirement of being able to specify cloud provider/region on the BridgeRequest.
I could hard-code it into the BridgeSteps class, but my instinct is to make it part of the And statement of the BDD tests e.g And create a new Bridge "mybridge" in cloud provider "aws" in region "us-east-1"
WDYT?
cheers,
Rob

@robpblake
And create a new Bridge "mybridge" in cloud provider "aws" in region "us-east-1"
Looks good to me :+1:

A nitpick, when I try to create a Bridge using body:
{
  "name": "ksuta-test"
}

then manager throws exceptions:
{
  "kind": "Errors",
  "items": [
    {
      "kind": "Error",
      "id": "31",
      "href": "/api/v1/errors/31",
      "code": "OPENBRIDGE-31",
      "reason": "Cloud Provider '{id}' is not valid."
    },
    {
      "kind": "Error",
      "id": "21",
      "href": "/api/v1/errors/21",
      "code": "OPENBRIDGE-21",
      "reason": "Region cannot be null or empty"
    },
    {
      "kind": "Error",
      "id": "21",
      "href": "/api/v1/errors/21",
      "code": "OPENBRIDGE-21",
      "reason": "Cloud Provider cannot be null or empty"
    }
  ]
}

OPENBRIDGE-31 shouldn't be thrown as provider is not specified.

Similar for:
{
  "name": "ksuta-test",
  "cloud_provider": "aws"
}

throws:
{
  "kind": "Errors",
  "items": [
    {
      "kind": "Error",
      "id": "32",
      "href": "/api/v1/errors/32",
      "code": "OPENBRIDGE-32",
      "reason": "The Region '{region}' for Cloud Provider '{id}' is not valid."
    },
    {
      "kind": "Error",
      "id": "21",
      "href": "/api/v1/errors/21",
      "code": "OPENBRIDGE-21",
      "reason": "Region cannot be null or empty"
    }
  ]
}

Otherwise looks ok

@robpblake I suspect the issue is caused by doubling-up on the validation.
You're using

https://github.com/5733d9e2be6485d52ffa08870cabdee0/sandbox/pull/899/files#diff-72df43d9d7d769e1649ec80043ee87e9f5c9b7b56ab0fe1a5f7b3092b08157baR14
https://github.com/5733d9e2be6485d52ffa08870cabdee0/sandbox/pull/899/files#diff-72df43d9d7d769e1649ec80043ee87e9f5c9b7b56ab0fe1a5f7b3092b08157baR31
https://github.com/5733d9e2be6485d52ffa08870cabdee0/sandbox/pull/899/files#diff-72df43d9d7d769e1649ec80043ee87e9f5c9b7b56ab0fe1a5f7b3092b08157baR26

The first is passing through our mechanism where you provide a dynamic payload with ExternalUserException's.
The other two are causing vanilla ConstraintViolation exceptions to be thrown that are mapped to OPENBRIDGE-21 (UnclassifiedConstraintViolationException used when a ConstraintViolation does not have a dynamic payload containing an ExternalUserException).

@manstis @sutaakar Thanks for the feedback and pointers. I finally got around to pushing up the fix. Can I ask for another review from you both please.
@kelvah I think this is the final version of the PR, so ready for your approval as well.
cheers,
Rob

using wrong parameter in api/smartevents_mgmt/v1/cloud_providers/something causes NPE:
curl -X 'GET' \
  'http://localhost:8080/api/smartevents_mgmt/v1/cloud_providers/awsq' \
  -H 'accept: application/json'

...\nCaused by: java.lang.NullPointerException\n\tat com.redhat.service.smartevents.manager.api.models.responses.CloudProviderResponse.from(CloudProviderResponse.java:40)\n\tat com.redhat.service.smartevents.manager.api.user.CloudProviderAPI.getCloudProvider(CloudProviderAPI.java:62)\n\tat com.redhat.service.smartevents.manager.api.user.CloudProviderAPI_Subclass.getCloudProvider$$superforward1(Unknown Source)\n\tat com.redhat.service.smartevents.manager.api.user.CloudProviderAPI_Subclass$$function$$1.apply(Unknown Source)\n\tat io.quarkus.arc.impl.AroundInvokeInvocationContext.proceed(AroundInvokeInvocationContext.java:54)\n\tat io.quarkus.hibernate.validator.runtime.interceptor.AbstractMethodValidationInterceptor.validateMethodInvocation(AbstractMethodValidationInterceptor.java:68)\n\tat io.quarkus.hibernate.validator.runtime.jaxrs.JaxrsEndPointValidationInterceptor.validateMethodInvocation(JaxrsEndPointValidationInterceptor.java:35)

Otherwise looks ok

Example 1
POST
curl -X 'POST' \
  'http://localhost:8080/api/smartevents_mgmt/v1/bridges' \
  -H 'accept: application/json' \
  -H 'Authorization: Bearer ... \
  -H 'Content-Type: application/json' \
  -d '{
  "name": "manstis-b1",
  "cloud_provider": "",
  "region": ""
}'

Response
{
  "kind": "Errors",
  "items": [
    {
      "kind": "Error",
      "id": "21",
      "href": "/api/smartevents_mgmt/v1/errors/21",
      "code": "OPENBRIDGE-21",
      "reason": "ConstraintViolationImpl{interpolatedMessage='Region cannot be null or empty', propertyPath=createBridge.bridgeRequest.region, rootBeanClass=class com.redhat.service.smartevents.manager.api.user.BridgesAPI_Subclass, messageTemplate='Region cannot be null or empty'}"
    },
    {
      "kind": "Error",
      "id": "21",
      "href": "/api/smartevents_mgmt/v1/errors/21",
      "code": "OPENBRIDGE-21",
      "reason": "ConstraintViolationImpl{interpolatedMessage='Cloud Provider cannot be null or empty', propertyPath=createBridge.bridgeRequest.cloudProvider, rootBeanClass=class com.redhat.service.smartevents.manager.api.user.BridgesAPI_Subclass, messageTemplate='Cloud Provider cannot be null or empty'}"
    },
    {
      "kind": "Error",
      "id": "21",
      "href": "/api/smartevents_mgmt/v1/errors/21",
      "code": "OPENBRIDGE-21",
      "reason": "ConstraintViolationImpl{interpolatedMessage='The supplied Cloud Provider details are not valid', propertyPath=createBridge.bridgeRequest, rootBeanClass=class com.redhat.service.smartevents.manager.api.user.BridgesAPI_Subclass, messageTemplate='The supplied Cloud Provider details are not valid'}"
    }
  ]
}

OPENBRIDGE-21 is the generic UnclassifiedConstraintViolationException.
Example 2
POST
curl -X 'POST' \
  'http://localhost:8080/api/smartevents_mgmt/v1/bridges' \
  -H 'accept: application/json' \
  -H 'Authorization: Bearer ...   \
  -H 'Content-Type: application/json' \
  -d '{
  "name": "manstis-b1",
  "cloud_provider": "bananna",
  "region": "a"
}'

Response
{
  "kind": "Errors",
  "items": [
    {
      "kind": "Error",
      "id": "33",
      "href": "/api/smartevents_mgmt/v1/errors/33",
      "code": "OPENBRIDGE-33",
      "reason": "Cloud Provider 'bananna' is not valid."
    }
  ]
}

You probably wanted both errors to be returned?
You should not return in your CloudProviderConstraintValidator after finding one error but let the whole method play out.

Thank you for a PR @robpblake . Some questions from me.

Thank you for a PR @robpblake . Some questions from me.

Thanks @jomarko  - nice feedback. I'll address your points.
I've spoken with @manstis about this today. I'm going to revert to a previous implementation given some inconsistencies that this approach creates in our openapi spec. We then have a broader task to look at validation more generally.
cheers,
Rob

@manstis @sutaakar @kelvah I think we're there...this is the least bad/most consistent option I think we can have until we've given some though to @kelvah 's doc on validation in general.
Could I ask for one last round of reviews from you all?
cheers,
Rob

Hi @robpblake could you please fix the merge conflict so this can be merged?
@kelvah Did you want to review this?
My PR adds support for Bridges to be updated; but I'll need to make a change to it after this PR is merged to prevent the Cloud Provider and Region from being updated.


Dynamic Environment and Adaptive Learning Strategies

Hi,
The Minecraft AI project is a fantastic demonstration of reinforcement learning. I have a few advanced suggestions to enhance the agent's learning and adaptability:
Suggested Enhancements:

Dynamic Environment:

Adaptive Obstacles: Introduce moving obstacles and changing environmental conditions (e.g., weather effects) to challenge the agent's adaptability.
Randomized Maze Layouts: Generate different maze configurations for each episode to prevent the agent from memorizing the layout.

Advanced Reward Structure:

Hierarchical Rewards: Implement a multi-tiered reward system that includes intermediate checkpoints and sub-goals to guide the agent's progress.
Exploration Incentives: Provide rewards for exploring new areas to encourage thorough investigation of the maze.

3Adaptive Learning Strategies:

Curriculum Learning:Start with simpler mazes and gradually increase complexity as the agent's performance improves.
Meta-Learning Allow the agent to adapt its learning rate and strategies based on performance feedback.

Improved State Representations:

Augmented Visual Input: Include additional sensory inputs, such as depth perception or object recognition, to enhance the agent's understanding of the environment.
-Feature Extraction: Use advanced techniques like convolutional neural networks (CNNs) to process raw pixel data more effectively.

Benefits:

Enhanced agent robustness and generalization to new environments.
More efficient learning through structured rewards and adaptive strategies.
Improved performance in complex and dynamic scenarios.

Thank you for considering these suggestions to take the project to the next level!
Ruby Poddar

Hi @rubypoddar , do you want to work on any of those enhancements?


Redesing MyLibraryPage to new Xbox's Games&Apps page

(Image credit: Windows Central)
Features

[x]  Sort games.
[ ] Tabs in Full library section.
[ ] Filter games.
[ ] User can create games groups.

Desing

[ ] Add icons to sections in the NavigationBar.
[ ] Redesing search button.

The sort options available is:

Sort: A to Z;
Release Date;

The sorting from release date is not right. This is problem is from Dart's sort method from DateTime, need to find other way to sort that.

[ ] Sort right from release date.

The new filter design is rolling out for Xbox Insiders.
Consider implementing it.

Demo image
Demo image

Probably the tabs will substitute the game filter for limitations reasons.

As said the tabs is used to filter, but this will change and the filter will have a dedicated button and the tabs will no longer exist until find a usage.

The release of the next alpha version is coming, and the features and bugs related here will be transferred to yours specific issues.


Offset in y axis

Using Inkscape 1.1 I get a warning while running about transforms (maybe related) and upon generation notice that the gcode is offset downwards by about a centimetre.
The points in this screen shot should be at each corner of the canvas for example.

The offset is precisely 11.25mm. I can work around this by manually remembering to offset prior to gcoding, but naturally a proper fix (or info on what I may be doing wrong?) would be helpful.

Welcome!
It looks like you're using an outdated version of the extension. Try installing the latest 2.x version ;)

🤦 Thank you, you're right, I originally downloaded from https://jtechphotonics.com/?page_id=1980 so was on V1.1_beta.
Jus ttested and it works perfectly.
Apologies and thanks.


Update bazel-rules-curl

Fixes an issue of not being able to include openssl headers. 3rdparty/bazel-rules-curl#12

@CodingCanuck please, take a look at https://github.com/3rdparty/bazel-rules-curl/pull/12, so you know what this PR does (my bad for not including this into PR Description!)

@CodingCanuck please, take a look at 3rdparty/bazel-rules-curl#12, so you know what this PR does (my bad for not including this into PR Description!)

That's very helpful context! That (as well as something like a standalone explanation) would be great to include in the description of the final commit that you merge in.

Merged this into #179!


hyper::server::conn::AddrIncoming "required features" doc is wrong

https://docs.rs/hyper/0.14.4/hyper/server/conn/struct.AddrIncoming.html says:

This is supported on crate feature server and (crate features http1 or http2) only.

Actually, the tcp feature also needs to be enabled.
https://github.com/hyperium/hyper/blob/v0.14.4/src/server/conn.rs#L77-L78
If I just enable the http1 and server features, the hyper::server::conn::AddrIncoming item does not exist.

What's the "E-easy" fix?
I thought it might be just as simple as replacing the #[cfg(feature = "tcp")] pub use ...; with cfg_feature! { #![feature = "tcp"] pub use ...; } so that the re-export has a doc(cfg(...)), but the docs page still has the banner with the original list of features, ie without tcp. I guess rustdoc only looks at doc(cfg())s on the definition, not a re-export, even when viewing the re-exported path.
So I tried replacing the #[cfg(feature = "tcp")] mod tcp; in server/mod.rs with cfg_feature! { #![feature = "tcp"] mod tcp; }. That still didn't change the banner on AddrIncoming itself, but it did add banners on every method and trait impl of AddrIncoming noting that they require the tcp feature. Ideally there'd be just the one banner on the type itself, that said "This is supported on crate feature server and (crate features http1 or http2) and tcp only", but I guess this is better than nothing.
Is there a better solution?


LoadHF can not load from compressed files

dataset https://huggingface.co/datasets/GEM/xlsum is implemented over files compressed as *.tar.bz2.
Currently, these can not be loaded in Streaming mode, as shows in the attached Notebook
ShowNotImplemented.pdf
I am suggesting a fast patch in a coming pull request.

Welcome.
You need to accept the pending invitation and then you can push aPR.

Note that you as an external contributor you must commit with sign off.

suggesting this pull request for a fix of issue 286:
https://github.com/IBM/unitxt/pull/288

resolved in https://github.com/IBM/unitxt/pull/298


Update PPA for Ubuntu 20.04

Can you please update PPA for Ubuntu 20.04? Thanks!

Which PPA are you using? This one should have a package for 20.04:
https://launchpad.net/~plasma-applet-qalculate/+archive/ubuntu/ppa

Hmm, it looks like this is somehow outdated and does not work anymore. I will see if I can fix it.

Look like the libqalculate21 dependency is pulling in some outdated libicu packages. Maybe @RogueScholar can help here?

My personal PPA has been updated, disco is no longer supported. There are now packages of 0.8.1 for bionic, eoan and focal: https://launchpad.net/~dschopf/+archive/ubuntu/plasma-applet-qalculate/+packages

Yes, your personal PPA  works well (but with libqalculate20 instead of libqalculate21), I switched to it - thanks!