Select nested in a control with isolate scope loses bindings to scope when selecting a value

See this plunk:
http://plnkr.co/edit/hxKZI07DVPa5PhcCKLQ9?p=preview
Basically, I've nested the ui-select directive in another directives template and tried to pass ng-model to it using an isolate scope binding. This works fine as long as you don't try and choose a value (e.g. updates to the value on the scope correctly update the select box)
If you select a value on the nested ui-select, it breaks the binding to scope. Any changes to the selected value from then on seem to hit the wrong scope (a child scope I believe)
In the above plunk, if you leave the top dropdown box as-is and mess with the other ones which are all bound to the same property on the scope everything works fine.
If you then select a value from the top dropdown, no values are updated on the page, and subsequent updates to the value from other controls on the page don't hit the broken dropdown any more.

After some more digging, looks like it might actually be an angular issue - not sure if I'm misunderstanding prototypal inheritance but it appears that the scope that it's writing against is a child scope which is inheriting the parent.
The disconnect happens during $$writeModelToScope in ng-model controller in angular
It looks like the method looks at the property name and tries to index it on the scope, but instead of setting the inherited property value, it writes a new property value with the same name on the child, hiding the parent value.
key = ensureSafeMemberName(element.shift(), fullExp);   ensureSafeObject(obj[key], fullExp);   obj[key] = setValue;   return setValue;
It appears on the ensureSafeObject line, obj[key] points to the parent property, but on the obj[key] = setValue line a new property is created on the child instead of what I would expect which is that the inherited parent value is modified.
Am I just misunderstanding how the inheritance should work or does this look like a bug?

Ok looks like it's more of a misunderstanding - I'll have to manually push the value onto the parent scope as writes in javascript don't look up the prototype chain (makes sense!). Going to try watching the value on scope and pushing it to the parent to see if that works

Ok my bad - turns out if you are wrapping a directive you need to prefix the templates ng-model bindings with $parent so that when it's being written back to it updates the correct object! My confusion was from the fact that the reads were working, so it looked like it was becoming unbound, but in fact it was just reading the child-most written value from the chain!

Had the exact same issue, so if the scope is nested e.g. $scope.main.list you need to have ..as-sortable="sortableOptions" ng-model="$parent.main.list"


How to use this ros_pose_estimation package with realsense D435/D415 camera.

Thanks you for this great contribution. I would like to know how we can use dope pose estimation with realsense D435i/D415 camera, how to build the realsense package in docker container and what all necessary changes are required.
It will be really helpful if someone can help me with this.

Have you had the chance to work through the Quickstart yet? This quickstart will ensure that you can run the DOPE pose estimation node inside your Isaac ROS development environment.
Then, you can use the tutorial here to add RealSense support inside your container. The DOPE quickstart uses images that are fed in from a sample rosbag; simply replace this input topic with one of the camera feeds on your RealSense camera, and you should start seeing estimated poses.
If you encounter any specific issues along the way, please describe them in this thread and we'll be happy to help you.


Feature to be able to download podcasts/audiobooks

Requested Feature
provide a spotify url for a podcast/audiobook and download it from the existing sources.
Possible implementation
No response

was thinking of something like this : https://github.com/tsaridas/spotify-downloader/commit/ed4efeca136f193b2f0a7f902c0db68237d03f8e

was thinking of something like this : https://github.com/tsaridas/spotify-downloader/commit/ed4efeca136f193b2f0a7f902c0db68237d03f8e
I saw that  music youtube downloader  library is adding podcasts as well so it might be an option for the future but it works fine for downloads from youtube videos with some known podcasts. haven't tried audiobooks I guess you might be right there about drm.

This works but the % of available episodes/podcasts is not as high as music. A lot of podcasts are drm protected on Spotify and no one uploads them to YouTube because they would've been taken down.
Theoretically I can add this but noobs would assume that any links works (because they assume it downloads directly from Spotify) and a lot of new "not found" issues would arise. Also the matching algorithm would have to be adjusted.
If you want I can merge your PR if you were to open one :)


Refactoring teams chat message send, closes #3106, #3107, #3123

Closes #3106, Closes #3107, Closes #3123
This pull request refactors the following points in the teams chat message send command:

CommandAction return void instead of Promise<void>
Use odata.getAllItems util function
Add telemetry properties for optional parameters
Rename all instances of 'topic' to 'name' for consistency.
Update ensureChatIdByUserEmails function for readability and consistency
Refactor string to array conversion into a function

Thank you. Will review this shortly

Thank you. Did some minor changes and merged them manually


Gen ignition files with UPI

Finished editing the UPI files such that the prep playbook can now hopefully be used to generate the ignition files for non-CIC OCP installs.

Missed a change


Requirements.txt and setup.py

Creating and updating of requirements.txt and setup.py for external usage of this package.

Done


Highlighting with cterm colors getting overridden

In my (vimscipt) config with previous commit https://github.com/nvim-zh/colorful-winsep.nvim/tree/9a474934a27203d1c2e9943c94a29165dd81823d, i configured colorful-winsep like this:
" use color 2 for the new window border
lua require('colorful-winsep').setup()
hi NvimSeparator ctermfg=2

(I currently use neovim with termguicolors off)
As of https://github.com/nvim-zh/colorful-winsep.nvim/issues/20, the NvimSeparator highlight gets overridden on ColorScheme and WinEnter events (here).
Since the setup doesn't allow for using cterm colors, only gui colors, I can't set this color in the setup, so this is the workaround I am currently using:
local group_postwinsep = vim.api.nvim_create_augroup(
    "NvimSeparatorPost", { clear = true }
)

vim.api.nvim_create_autocmd("WinEnter",
    {
        group = group_postwinsep,
        callback = function(opts) vim.cmd [[ hi NvimSeparator ctermfg=2 ]] end,
    }
)

Not sure if there's a better way to do this, or if there's a good solution for how the view.highlight code is organized?
Thanks for the plugin! I appreciate the how clean it makes splits look.

Sorry, just saw your issues, I'm not sure how to fix it

I try to refactor view.highlight

now you can setting ctermfg
colorful_winsep.setup({
  highlight = {
    fg = "#9898B5",
    bg = "#0D0F18",
    ctermfg = 2
  }
})

Thank you!


vue-cli 3 support

Do you plan to support the new vue-cli 3.0? I do not know how much work is needed to move to this version.

I think this depends on: https://github.com/SimulatedGREG/electron-vue/issues/593

Migration to @vue/cli will be done in a complete rewrite, more news to come. :slightly_smiling_face:

Not to steal SimulatedGREG's thunder, but I made vue-cli-plugin-electron-builder. @SimulatedGREG I think we should work together to make one plugin instead of competing with each other. What do you think?

When can I create a project directly using vue-cli 3?


Dylan's a slacker

Dylan come up with some algorithm stuff because I just made the db very organized!!!!! @ @

Yes sir


Adding multiple lines dynamically for text

I am having a textarea content which needs to be added as text in SVG using SVG TEXT() method.
Content of textarea will be split based on the newline characters present in content and needs to be added as tspan to the SVG element with newline() added.
var inText = input.value.replace(/(?:\r\n|\r|\n)/g, '\n');
var tspanText = inText.split( "\n" );
var tspan;
for( var i = 0; i < tspanText.length; ++i ) {
	//element.tspan(tspanText[i]).newLine();
	element.text(function(add) {
	  add.tspan(tspanText[i]).newLine()
	})
	element.tspan(tspan);
}

Only the last line of the textarea content is added to the text element.

ok forst I will explain to you, why your code is not working and then I will tell you a secret ;)
For every line you call the text() method on the element. Like any other command this sets the value of text - so its getting replaced over and over.
This can be avoided if you call build(true) before the text call which activats the build mode of text. You then can add tspans for every line which doenst get removed like this:
element.build(true)

// in the loop
element.tspan('foo').newLine()

However its still not really useful whats written there.
You should switch the loop and the builder function where you add the text:
    element.text(function(add) {
      for( var i = 0; i < tspanText.length; ++i ) {
          add.tspan(tspanText[i]).newLine()
      }
      add.tspan(tspan);
    })
}

That would work best - but now: the secret
svg.js automatically trade newline-charackters as newline and build up the text element for you.
So just call text("Some Text \nWith newlines in it \nSo beatiful") and let the magic happen.
Btw: Thats all in the docs: https://github.com/wout/svg.js#text

Thanks a lot Ulrich master,
I did the other way by picking the newline and then adding each one as a
seperate tspan.
thanks for the mail.
On Thu, May 26, 2016 at 4:06 PM, Ulrich-Matthias Schäfer <
<EMAIL_ADDRESS>wrote:

ok forst I will explain to you, why your code is not working and then I
will tell you a secret ;)
For every line you call the text() method on the element. Like any other
command this sets the value of text - so its getting replaced over and over.
This can be avoided if you call build(true) before the text call which
activats the build mode of text. You then can add tspans for every line
which doenst get removed like this:
element.build(true)
// in the loopelement.tspan('foo').newLine()
However its still not really useful whats written there.
You should switch the loop and the builder function where you add the text:
element.text(function(add) {
  for( var i = 0; i < tspanText.length; ++i ) {
      add.tspan(tspanText[i]).newLine()
  }
  add.tspan(tspan);
})

}
That would work best - but now: the secret
svg.js automatically trade newline-charackters as newline and build up the
text element for you.
So just call text("Some Text \nWith newlines in it \nSo beatiful") and
let the magic happen.
Btw: Thats all in the docs: https://github.com/wout/svg.js#text
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/wout/svg.js/issues/488#issuecomment-221793951

--
Santhosh Kumar.M
(+81)90-7828-4552

Thanks a lot Ulrich,
That helped a lot and i was able to achieve things.


separate getSchema in Typegoose

Why:
I'm in a middle of refactoring a Mongoose backend to Typescript (which means Typegoose!)
However, there's too many inter-dependent Mongoose-schemas (around 50~70?), which means I can't just move one file to Typegoose - everything has to go at once :(
(I have to move everything to Typegoose models before I can run the app - or the app will break due to missing mongoose schemas)
Anyway, Thank you for your time!

Coverage increased (+0.08%) to 87.087% when pulling dfeb272cd33ca75c8ab1f352bb9d8598cf9e9f15 on devdoomari3:separate-getSchema-in-Typegoose into 24b66fd57dba268340c47a577f284b1f8eb78e37 on szokodiakos:master.

@szokodiakos Could you give some feedback on this PR? Thanks : )


Fix for extracting image filename from label from batch project.

Change how image filename is extracted from labelbox label object, as the url changed for batch projects.
URL examples of each:
Batch
https://r7ecommonuksdata.blob.core.windows.net/labelbox-data/dev_Levenseat_Aluminium_sample_batch/dev_levenseat_aluminium_batch_train/images/levenseat01__20220620172225957978.jpg?se=2025-12-12T17%3A46%3A08Z&sp=rt&sv=2020-04-08&sr=b&sig=gLyD5pRMQfDUd/Xm87M8MjbgkKFK6s6d/WLXxkEbv4U%3D
Dataset
https:/r7ecommonuksdata.blob.core.windows.net/labelbox-data/dev_Levenseat_Aluminium_sample/dev_levenseat_aluminium_val/images/levenseat01__20220620111815427050.jpg?se=2025-09-21T15%3A49%3A36Z&sp=rt&sv=2020-04-08&sr=b&sig=ZE7ive%2BB6UaFpQqxdPSxslDdHTwGWHMvd34ZHz0tMFE%3D
Since Batch has more "/", have to call .name on Path object at the end.
In both cases, filename is
levenseat01__20220620172225957978.jpg
and
levenseat01__20220620111815427050.jpg
respectively

Not needed. Redundant pr.


Develop/feature/td 3678 implement open api endpoints to retrieve my accessed learning resources

JIRA link
TD-3678
Note
This branch must be merged first, any changes in PR repulled into this
https://github.com/TechnologyEnhancedLearning/LearningHub.Nhs.WebUI/tree/Develop/Feature/TD-2895-2-ActivityStatus
Description
Resource Reference Models by Certificates achieved and by activityStatusId
Screenshots

Developer checks
(Leave tasks unticked if they haven't been appropriate for your ticket.)
I have:

[x] Run the formatter and made sure there are no IDE errors
[x] Written appropriate unit tests for the changes, including:

accessibility tests for new views
tests for new controller methods
tests for new or modified API endpoints

[ ] Manually tested my work with and without JavaScript
[ ] Tested any Views or partials created or changed with Wave Chrome plugin and addressed any valid accessibility issues
[ ] Updated/added documentation in Confluence and/or GitHub Readme. List of documentation links added/changed:

doc_1_here

[x] Updated my Jira ticket with information about other parts of the system that were touched as part of the MR and have to be sanity tested to ensure nothing is broken
[x] Scanned over my pull request in GitHub and addressed any warnings from the GitHub Build and Test checks.

hiya @OluwatobiAwe. Please could you review this pr later. please dont review it yet. I am working on it again. i am redoing part of it due to the clash where we both changed the dashboard  proc. but when it is ready please could you review it as I will changing some code you've recently written, so I thought you might like to and you'll know it well to :) thanks

@AnjuJose011 @Swapnamol pushing changes didnt update the pull request. so I've added the commits individually via https://github.com/TechnologyEnhancedLearning/LearningHub.Nhs.WebUI/pull/465/files#diff-44c76c20483cbce02b62cead274c68033051d68cd777561cf9491dac32d8e548  so it says all commits are present now. so seems correct. the changes that are relevant are that in the end I kept Oluwatobi's proc but replaced its logic with mine except for determining the variables for pagination. it makes for a small proc but for simplicity, I think it is the better way to go. one thing I did consider was changing the dashboard proc that is no longer in use to zzz_dashboard_proc so we can still refer to it but don't accidentally use it. but I wasn't sure if this is what you would want to happen for a dead proc and wasn't certain that nothing uses it at all so I  have left it on this occasion.


JMS: update Client to Starlight for JMS 2.0.0 and add examples for Server Side Selectors

Summary of changes:

Updated JMS client to 2.0.0
Use "shaded" version, in order to allow to have a Pulsar version different from the one bundled with the Pulsar Driver (currently 2.8.x)
Add a new example about how to add enable server side filtering

Draft docs about Server Side Filtering
https://github.com/riptano/pulsar-jms-docs/issues/14
Please note that in order to allow server side filtering you have to install a .nar file, just follow the instructions above.

I would like also to point out that the way we are configuring selectors is not very effective as basically the filter allows all the messages to pass.
So the only impact that we can measure here is on the overhead added on the brokers to do the filtering.
If you enable "batching" (batchingEnabled) for the producer then:

the Broker will process the Batch and send it on the client
the Client must process it again (because it doesn't know if it contains all good messages, as the Broker can filter out the batch only if all the messages does not pass the filter)

if you disable batching:

the producer will be slower, and in general results will be worst
the broker has to process all the messages (and let the pass)w
the client will safely not run the Selector over the messages, because it knows for sure that the Messages are good, because the Broker is able to fully discriminate non-batch messages


How to reference in an own project?

How shall this sample project be referenced if it is used as basis for another project?
It might happen the new project has a different licence than this sample project ...

That's fine! Consider this public domain.


New editor layout structure

Minor tweaks in tandem with https://github.com/TryGhost/Ghost-Editor/pull/3 for new layout structure.

Closing in favour of https://github.com/TryGhost/Ghost-Admin/pull/341


.NET Core 3.0 Preview 7 has issues restoring project

Here's how I can reproduce this problem with a Hello-World style program:
$ clean-dotnet-cache
+ rm -rf /home/omajid/.aspnet /home/omajid/.dotnet/ /home/omajid/.mono/ /home/omajid/.nuget/ /home/omajid/.local/share/NuGet /home/omajid/.templateengine
+ rm -rf /tmp/NuGet /tmp/NuGetScratch '/tmp/.NETCore*' '/tmp/.NETStandard*' /tmp/.dotnet '/tmp/clr-debug-pipe*' /tmp/Razor-Server '/tmp/CoreFxPipe*' /tmp/VBCSCompiler '/tmp/.NETFramework*'
$ ls
Program.cs  RestoreFailure.csproj
$ cat Program.cs
namespace RestoreFailure
{
    public class Program
    {
        static void Main(string[] args)
        {
            Console.WriteLine("Hello World");
        }
    }
}
$ cat RestoreFailure.csproj
<Project Sdk="Microsoft.NET.Sdk">
  <PropertyGroup>
    <TreatWarningsAsErrors>true</TreatWarningsAsErrors>
    <TargetFramework>netcoreapp3.0</TargetFramework>
    <IsPackable>false</IsPackable>
  </PropertyGroup>
  <ItemGroup>
    <PackageReference Include="NUnit" Version="3.12.0" />
  </ItemGroup>
</Project>
$ dotnet --info
.NET Core SDK (reflecting any global.json):
 Version:   3.0.100-preview7-012802
 Commit:    ad21996be3

Runtime Environment:
 OS Name:     fedora
 OS Version:  30
 OS Platform: Linux
 RID:         fedora.30-x64
 Base Path:   /usr/lib64/dotnet/sdk/3.0.100-preview7-012802/

Host (useful for support):
  Version: 3.0.0-preview7-27902-19
  Commit:  36f91cf6b6

.NET Core SDKs installed:
  3.0.100-preview7-012802 [/usr/lib64/dotnet/sdk]

.NET Core runtimes installed:
  Microsoft.NETCore.App 3.0.0-preview7-27902-19 [/usr/lib64/dotnet/shared/Microsoft.NETCore.App]

To install additional .NET Core runtimes or SDKs:
  https://aka.ms/dotnet-download
$ dotnet restore

Welcome to .NET Core 3.0!
---------------------
SDK Version: 3.0.100-preview7-012802

----------------
Explore documentation: https://aka.ms/dotnet-docs
Report issues and find source on GitHub: https://github.com/dotnet/core
Find out what's new: https://aka.ms/dotnet-whats-new
Learn about the installed HTTPS developer cert: https://aka.ms/aspnet-core-https
Use 'dotnet --help' to see available commands or visit: https://aka.ms/dotnet-cli-docs
Write your first app: https://aka.ms/first-net-core-app
--------------------------------------------------------------------------------------
/home/omajid/temp/RestoreFailure/RestoreFailure.csproj : error NU1603: RestoreFailure depends on Microsoft.NETCore.App (>= 3.0.0-preview7-27902-19) but Microsoft.NETCore.App 3.0.0-preview7-27902-19 was not found. An approximate best match of Microsoft.NETCore.App 3.0.0-preview7-27912-14 was resolved.
  Restore failed in 3.45 sec for /home/omajid/temp/RestoreFailure/RestoreFailure.csproj.

The error goes away if I remove the PackageReference from my csproj. This isn't specific to NUnit, I can trigger this with pretty much any other package (for example,  Newtonsoft.Json).

Erm. I have <TreatWarningsAsErrors>true</TreatWarningsAsErrors>. PEBKAC.


Certbot always fails the first time, succeeds the second.

Certbot task is successful, however on the first run of the playbook the task always fails. On a second run, the command is successful. Needs to be investigated.

Solution may be to use the letsencrypt module.

I have not recently seen this happen.

Aaand that's why the firewall should be restarted immediately after opening HTTP and HTTPS traffic instead of only as a handler. Installation proceeds without issue.

Fixed in #61


fix leftover managed text

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/bdb65e759f67568222ccc44891221010f1cdf364/

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/49f4fc80b6b47e151b86a8ce6ee192c1debbd274/


Change setting the user-agent to make it work in Python 3.6

osmviz stopped working for me at some point with an HTTP 403 error, it worked again if I replaced the existing hack for useragent with a different approach.
I did not test it extensively with different versions of Python3, but it seems to work better than the previous approach on current Ubuntu Buster.

Thanks for the PR!
Please could you fix the lint error?
src/osmviz/manager.py:42:1: F401 'urllib.request.FancyURLopener' imported but unused

Sure, done now.

Done now

Thanks! Will make a new release soon.

Released in osmviz 3.1.1. Thanks again!


lxc config problem

Hi,
I try to set the listen on IPv4 port 8443 with the command :
lxc config set core.https_address [::]:8443
But after entering a bad address and restart daemon lxd, it can't restart and it's impossible to change again this value. Every command (lxc list, lxc config ...) display an error :
error: Get http://unix.socket/1.0/containers?recursion=1: dial unix ///var/lib/lxd/unix.socket: connect: connection refused
The only way i found to fix it, it's to delete or update the config table in the sqlite file (lxd.db).
The daemon is required to use lxc config set ?

Yes, "lxc config set" talks to the daemon over the unix socket.
What version of LXD are you using and on what Ubuntu release (if Ubuntu) and how did you restart LXD?
Also note that you don't have to restart lxd after setting core.https_address, the daemon will immediately start listening when this is set.

LXD 0.18 on Ubuntu Trusty (14.04).
I restart lxd with the service command : service lxd restart

what do you have in /var/log/upstart/lxd.log?

I have this error :
error: cannot listen on https socket: listen tcp 78.192.XXX.XXX:8443: bind: cannot assign requested address

that doesn't match what you said in the bug report, in the bug report you said you set it to "[::]:8443", here it looks like you actually set it to "78.192.xxx.xxx:8443" which presumably isn't bindable on your host, leading to that failure.

oops, didn't mean to close the issue

Ah yes, sorry, I expressed myself badly, I copied the command help.
Indeed, I have entered an IP address.
The problem is that if the given configuration is not good, on reboot, then it is impossible to reconsider this configuration because the daemon doesn't work.

Yeah, we definitely shouldn't be failing hard at startup if an address can't be bound.
What's also weird is that it should have failed the "lxc config set"... not sure why that one succeeded.


Use go 1.22 if 1.23 is not required.

Hi.
I use goenv. go 1.23 is not yet available in goenv, and it is only available as 1.23rc1 in the go git repositories.
https://github.com/golang/go
https://go.googlesource.com/go
It would make life easier if the version required for building codesearch were 1.22 and not 1.23.

Until this is implemented or you have the newer version of go, you can see if installing from my fork https://github.com/junkblocker/codesearch helps your need.


navbar mobile

The menu for the links to the different sections isn't placed correctly. It starts of correct but then changes to be messed up and flush left. Will have to compare to the original.

The flush left is fixed, but now the navbar is constantly expanded when in mobile form.


Common input-handling idiom for various tooling bits

Brought up in https://github.com/amazon-ion/ion-cli/pull/54#discussion_r1227312405
In several of the subcommands we have the same input-handling pattern:
    if let Some(input_file_names) = matches.get_many::<String>("input") {
        // Input files were specified, run the logic on each of them in turn
        for input_file in input_file_names {
            let file = File::open(input_file.as_str())
                .with_context(|| format!("Could not open file '{}'", &input_file))?;
            let mut reader = ReaderBuilder::new()
                .build(file)
                .with_context(|| format!("Input file {} was not valid Ion.", &input_file))?;
            // do something with the reader
        }
    } else {
        // No input files were specified, run logic on STDIN.
        let mut reader = ReaderBuilder::new()
            .build(stdin().lock())
            .with_context(|| "Input was not valid Ion.")?;
        // do something with the reader
    }

We ought to have shared code for this. We could solve it by building an input stream abstraction over a set of files, but then you run into places like #48 where it convenient or even necessary to have context about what input you are processing at the moment, so perhaps it would be best if the common idiom was some flavor iterator-of-inputs-with-source-metadata.
This must have been solved somewhere else before...
Examples (to/dump/inspect)
https://github.com/amazon-ion/ion-cli/blob/e7feb98d625056f2440a9dba5dcbb2308205bbef/src/bin/ion/commands/beta/to/json.rs#L49-L65
https://github.com/amazon-ion/ion-cli/blob/e7feb98d625056f2440a9dba5dcbb2308205bbef/src/bin/ion/commands/dump.rs#L68-L79
https://github.com/amazon-ion/ion-cli/blob/e7feb98d625056f2440a9dba5dcbb2308205bbef/src/bin/ion/commands/beta/inspect.rs#L118-L162

Fixed by #115.


HTTP calls to an invalid URL exposes an stack trace

What version of Remix are you using?
v1.12.0
Are all your remix dependencies & dev-dependencies using the same version?

[X] Yes

Steps to Reproduce
http --follow --all https://www.shopify.com/\?_data\=rootwew
HTTP/1.1 403 Forbidden
CF-Cache-Status: BYPASS
CF-RAY: 798e74c86c3c11b3-BCN
Connection: keep-alive
Content-Encoding: gzip
Content-Type: application/json; charset=utf-8
Date: Mon, 13 Feb 2023 15:04:23 GMT
NEL: {"success_fraction":0.01,"report_to":"cf-nel","max_age":604800}
Report-To: {"endpoints":[{"url":"https:\/\/a.nel.cloudflare.com\/report\/v3?s=SBSW8rub9EdDe5si331mCz8bi913VPD3DCpBDnmrkUeR%2B1VXboNLlevhnyeC%2Ff996saEKTIlvyASR05GlrO2AIVtF7rYIjgwezqXmqW3wCJceN3KdBxFmEOuXC1WMPe9gQ%3D%3D"}],"group":"cf-nel","max_age":604800}
Server: cloudflare
Server-Timing: cfRequestDuration;dur=236.999989
Strict-Transport-Security: max-age=15552000; includeSubDomains; preload
Transfer-Encoding: chunked
Vary: Accept-Encoding
X-Content-Type-Options: nosniff
X-Dc: gcp-us-east1
X-Request-ID: 5ddd5bc5-80f3-4e5f-a254-b6bfa73488fb
alt-svc: h3=":443"; ma=86400, h3-29=":443"; ma=86400
x-remix-error: yes

{
    "message": "Unexpected Server Error",
    "stack": "Error: Unexpected Server Error\n    at handleDataRequestRR (/app/remix/node_modules/@remix-run/server-runtime/dist/server.js:128:31)\n    at runMicrotasks (<anonymous>)\n    at processTicksAndRejections (node:internal/process/task_queues:96:5)\n    at requestHandler (/app/remix/node_modules/@remix-run/server-runtime/dist/server.js:55:18)\n    at Object.handler (/app/remix/node_modules/@remix-run/express/dist/server.js:39:22)"
}

Expected Behavior
Do not expose an stack trace in production.
Actual Behavior
Extract trace dumped in the response. Internal server structure exposed.

Out of interest I just found the same.
I guess revealing the use of Remix isn't terrible. Especially when the response has a x-remix-error header. I'd agree the stack shouldn't be there though. There seems to be an open discussion about that: https://github.com/remix-run/remix/discussions/5199

I'm getting the same error once deployed on Vercel. No error on local dev
    at handleDataRequestRR (/var/task/node_modules/@remix-run/server-runtime/dist/server.js:128:31)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at requestHandler (/var/task/node_modules/@remix-run/server-runtime/dist/server.js:55:18)
    at Server.default (file:///var/task/build/server-node.mjs:68:20)```

Closed by #5541


Ability to pass JSON schema as a String PR:

Issue

Closes #https://github.com/langchain4j/langchain4j/issues/2013
Change
This pull request introduces the JsonSchemaParser class to convert JSON formatted strings into a structured JsonSchema representation. The JsonSchemaParser provides a convenient method for parsing JSON schema strings, enabling validation and representation of schema properties in a strongly typed manner.
The JSON schema strings should adhere to the JSON Schema specification. Below are examples of valid schemas that can be parsed by JsonSchemaParser:
Simple JSON Schema:
{
    "type": "string",
    "description": "A simple string"
}

Simple nested JsonSchema
{
    "type": "object",
    "properties": {
        "person": {
            "type": "object",
            "properties": {
                "name": {
                    "type": "string"
                },
                "age": {
                    "type": "integer"
                }
            },
            "required": ["name"]
        }
    },
    "required": ["person"]
}

So in case you need to write gemini chat model
Option1:
ChatLanguageModel gemini = GoogleAiGeminiChatModel.builder()
    .apiKey(System.getenv("GEMINI_AI_KEY"))
    .modelName("gemini-1.5-flash")
    .responseFormat(ResponseFormat.builder()
        .type(JSON)
        .jsonSchema(JsonSchema.builder()
            .rootElement(JsonObjectSchema.builder()
                .properties(Map.of(
                    "title", JSON_STRING_SCHEMA,
                    "preparationTimeMinutes", JSON_INTEGER_SCHEMA,
                    "ingredients", JsonArraySchema.builder()
                        .items(JSON_STRING_SCHEMA)
                        .build(),
                    "steps", JsonArraySchema.builder()
                        .items(JSON_STRING_SCHEMA)
                        .build()
                    ))
                .build())
            .build())
        .build())
    .build();

String recipeResponse = gemini.generate(
    "Suggest a dessert recipe with strawberries");

System.out.println(recipeResponse);

Option2
ChatLanguageModel gemini = GoogleAiGeminiChatModel.builder()
                                .apiKey(System.getenv("GEMINI_AI_KEY"))
                                .modelName("gemini-1.5-flash")
                                .responseFormat(ResponseFormat.builder()
                                    .type(JSON)
                                    .jsonSchemaString("""
                                        {
    "type": "object",
    "properties": {
        "title": {
            "type": "string"
        },
        "preparationTimeMinutes": {
            "description": "A preparation time in minutes",
            "type": "integer"
        },
        "ingredients": {
            "type": "array",
            "items": {
                "type": "string"
            }
        },
        "steps": {
            "type": "array",
            "items": {
                "type": "string"
            }
        }
    },
    "required": [
        "title",
        "preparationTimeMinutes",
        "steps"
    ]
}

                                        """)
                                    .build())
                                .build();
String recipeResponse = gemini.generate(
    "Suggest a dessert recipe with strawberries");

System.out.println(recipeResponse);

General checklist

[x] There are no breaking changes
[x] I have added unit and integration tests for my change
[x] I have manually run all the unit and integration tests in the module I have added/changed, and they are all green
[x] I have manually run all the unit and integration tests in the core and main modules, and they are all green

[ ] I have added/updated the documentation
[ ] I have added an example in the examples repo (only for "big" features)
[ ] I have added/updated Spring Boot starter(s) (if applicable)

Checklist for adding new maven module

[ ] I have added my new module in the root pom.xml and langchain4j-bom/pom.xml

Checklist for adding new embedding store integration

[ ] I have added a {NameOfIntegration}EmbeddingStoreIT that extends from either EmbeddingStoreIT or EmbeddingStoreWithFilteringIT
[ ] I have added a {NameOfIntegration}EmbeddingStoreRemovalIT that extends from EmbeddingStoreWithRemovalIT

Checklist for changing existing embedding store integration

[ ] I have manually verified that the {NameOfIntegration}EmbeddingStore works correctly with the data persisted using the latest released version of LangChain4j

@langchain4j , yes it will work for me.

@rwankar please clarify what exactly will work?
Passing schema as a string:

in the native LLM provider format
in the generic LC4j format

@langchain4j , I meant native LLM format. Is there such a thing as a generic LC4j schema?
If a generic schema format could be converted to specific LLM format internally, that would be interesting as one could swap LLMs without changing code. But I would have thought this to be non-trivial.

@rwankar

Is there such a thing as a generic LC4j schema?

Yes, you can define json schema for structured outputs and tools using JsonSchemaElement hierarchy and it will be mapped to native LLM schema by each module.

If a generic schema format could be converted to specific LLM format internally, that would be interesting as one could swap LLMs without changing code. But I would have thought this to be non-trivial.

This works already, that's one of the goals of LC4j to have a generic APIs that work with all model providers.
You can either use POJO as a return type of AI Service or use the low-level ChatLanguageModel API and specify schema manually  using JsonSchemaElement hierarchy and it will work for all model proivders that support structured outputs or tools.

Actually, just ignore what I said. It's just creating confusion.
If I can provide a native String schema (e.g using jsonSchemaString) and I can use Result as the return type of AI Service I'm good. I will parse the JSON String that I get in the content().

As I understand it, users need the option to pass the schema as a string to have fine-grained control over it. Therefore, it shouldn't be mapped back and forth, as there’s a risk of losing some details. I envisioned that we would allow the schema to be specified as a string in the native format of the LLM provider, which would then be passed to the provider as-is, without any translation or modification.

I agree with this.

@rwankar thanks for clarificartion. How do you plan to generate JSON schema string?

@omarmahamid just to make sure we are on the same page: the proposed solution is not what is required, please see the comments above for more details


Implement a Quad-Tree based Partitioner

We need to implement a Quad-Tree based Partitioner. Will discuss more on this on regular group meetings.

Resolved, refer to PR #26.


k8sclient: use InClusterConfig() for kubeclient when run as daemonset

When multus is run inside a container, eg as a daemonset, we can use
InClusterConfig() to automatically get the service account token, the
CA certificate, and the apiserver address, all without a kubeconfig
file.
Signed-off-by: Dan Williams<EMAIL_ADDRESS>@dougbtv @rkamudhan

@dougbtv PTAL; had to change a couple things around to fix the testcases.

@rkamudhan CI failed on what looks like an unrelated timeout, could you log into Jenkins and retrigger?  Thanks!

@dcbw Done. Let's see...

Hi,
we getting error as follows:
The command "golint ./multus/... | grep -v ALL_CAPS | xargs -r false" failed and exited with 123 during .
Your build has been stopped.

Is this related to timeout ?

@dcbw you should have the collaborator right for the multus repo. Please let me know, I can send you the invite again on this..

@rkamudhan @s1061123 seems like everything is OK now?

Big thumbs up from me, just deployed a build of this branch and gave it a test drive -- worked flawlessly. Not a deep test, but, enough to know typical scenarios are still workin'!
Thanks Dan.

+1 LGTM. Tested by @dougbtv. so merge it.


fix(plugin): fix resolution shouldContinue for a plugin

Summary:
This is an update to the tests to catch error states, fixes the
wording in the testing, and covers the update to the NewModel
emit function. This is a blocking issue for me in use, and allows
some features to be implemented to align more fully with
current mongoose.

there are a few other features I would like to add, if this is good work I can discuss further

Code sample:
const VerifyUniquePlugin = (plugin, declaredSchema) => {
  plugin.setName('VerifyUniquePlugin')
  plugin.setDescription('A simple plugin to check if a key is supposed to be unique and enforce it.')
  plugin.on('model:put', 'request:pre', async (request) => {
    const {event, model} = request
    const putQuery = event.item.Item
    let foundValues
    const uniqueItems = Object.keys(putQuery).filter(key => declaredSchema.attributes[key].options.unique)
    if (uniqueItems.length > 0) {
      foundValues = await Promise.all(uniqueItems.map(async itemName => {
        const findQuery = {[itemName]: putQuery[itemName].S}
        const results = await model.scan(findQuery).exec()
        return results
      }))
    }
    return new Promise((resolve)=> {
      const response = {}
      if (!lodash.isEmpty(lodash.flatten(foundValues))) {
        response['reject'] = 'This is a duplicate, can not create another.'
      }
      response['resolve'] = 'Value is unique.'
      resolve(response)
    })

  })
  return plugin
}

Schema
const schema = {
  name: {
    type: String,
    unique: true
  }
}

Model
const builtModel = dynamoose.model('NoDupeModel', schema)
builtModel.plugin(VerifyUniquePlugin, schema)

General
builtModel.create({name: 'John'})
// this one should not create another entry
builtModel.create({name: 'John'})

it also changes how this works
builtModel.batchPut([{name: 'John'},{name: 'John'}])

this case should only create a single model.

Other information:
I understand the Plugins are in very early beta, but I'd very much like to use them. This work increases coverage and fixes what I believe is a miss. If I've made a bad assumption in my plugin, please let me know. As another side note, the plugin docs have the wrong event names included. Should I fix that here, or as a separate docs only commit?
Type (select 1):

[x] Bug fix
[ ] Feature implementation
[ ] Documentation improvement
[ ] Testing improvement

[ ] Test added to report bug
[ ] Something not listed here

Is this a breaking change? (select 1):

[ ] 🚨 YES 🚨
[x] No
[ ] I'm not sure

Is this ready to be merged into Dynamoose? (select 1):

[x] Yes
[ ] No

Are all the tests currently passing on this PR? (select 1):

[x] Yes
[ ] No

Other:

[x] I have searched through the GitHub pull requests to ensure this PR has not already been submitted
[x] I have updated the Dynamoose documentation (if required) given the changes I made
[x] I have added/updated the Dynamoose test cases (if required) given the changes I made
[x] I have ensured the following commands are successful from the root of the project directory

[x] npm test
[x] eslint .

[x] I agree that all changes made in this pull request may be distributed and are made available in accordance with the Dynamoose license
[x] All of my commits and commit messages are detailed, explain what changes were made, and are easy to follow and understand
[x] I have filled out all fields above

As another side note, the plugin docs have the wrong event names included. Should I fix that here, or as a separate docs only commit?

Separate docs only PR would be best!

@hweeks Also! If you are creating public plugins I would love to see them!! This plugins project is in very early stages. I haven't heard about a lot of interest in it so far. I'm glad to see you are interested in it. Please let me know how I can support your work in this area!!
Also in this GitHub organization there is a repo for plugin ideas. Not sure exactly how you are using it, but thought I'd bring it up in case you are interested.


some kind of PYPATH problem with TrainingExample (?)

when I run:
./policy_runner.py --policy Baseline
I get
Traceback (most recent call last):
File "./policy_runner.py", line 70, in <module>
rospy.Subscriber('/drivebot/training_egs', TrainingExample, call_policy_training)
NameError: name 'TrainingExample' is not defined

However, I checked with
rosmsg show drivebot/TrainingExample
it worked.
So, I was a little bit confused.
The same error applies to other policies.
I am using Ubuntu 14.04 LTS.

Hello Mat,
I am so sorry for this. I solved this problem.
It was because I commented out those two lines when I first time met some troubles:
from drivebot.msg import TrainingExample
from drivebot.srv import ActionGivenState
Sorry for this. Now the robots are running on my PC.
Thanks for your help.
I think I will soon have some experiment results, and I will let you know.
Oh, I have to finish a paper this week, so, maybe next week I will work on this.
Cheers,
Dorje

No problem, glad it's running now. Let me know if anything else comes up!


Update cube and tri to run on android.

I was re-basing this onto master and there are a few merge conflicts in cube.c that I'm not sure of the best way to merge. Michael, can you take a look at rebase and re-push if you're able to get the conflicts fixed. I'll try to then get it pushed in a timely manner so we don't hit conflicts again.

Ok done (I think). Please test this on linux as with all the x11/xcb changes I had to merge in I may have broken something.

Question: Can you resize the window for these demos, and if so, is there a problem when the swapchain is recreated?

I'm mostly running on android which makes that difficult to test for me. Tobin may be able to quickly check.

@ianelliottus window resizing on linux appears to be working fine for both cube and tri.
@lentinem there were a number of issues on Linux but I massaged the code and verified that it's working on both linux and android. I created new #445 and assigned to Tony to get his final ok since all of the merge conflicts were with his recent changes.

@tobine I was wondering about resizing on Android.

@tobine I made some comments on that CL.


Focus the window when clicking on a group bar tab

Hyprland Version
Hyprland, built from branch  at commit<PHONE_NUMBER>90f402d1382eb3df149ece6db8b809  ().
Date: 2024-02-26
Tag:
flags: (if any)
System Information:
System name: Linux
Node name: odyssey
Release: 6.6.22
Version: #1-NixOS SMP PREEMPT_DYNAMIC Fri Mar 15 18:25:07 UTC 2024
GPU information:
64:00.0 VGA compatible controller [0300]: Advanced Micro Devices, Inc. [AMD/ATI] Phoenix1 [1002:15bf] (rev dd) (prog-if 00 [VGA controller])
os-release: ANSI_COLOR="1;34"
BUG_REPORT_URL="https://github.com/NixOS/nixpkgs/issues"
BUILD_ID="24.05.20240325.57e6b3a"
DOCUMENTATION_URL="https://nixos.org/learn.html"
HOME_URL="https://nixos.org/"
ID=nixos
IMAGE_ID=""
IMAGE_VERSION=""
LOGO="nix-snowflake"
NAME=NixOS
PRETTY_NAME="NixOS 24.05 (Uakari)"
SUPPORT_URL="https://nixos.org/community.html"
VERSION="24.05 (Uakari)"
VERSION_CODENAME=uakari
VERSION_ID="24.05"
plugins:
Bug or Regression?
Bug
Description
Currently Hyprland doesn't focus the window when a group bar tab is clicked. I never noticed before, but today I did. I don't think this is very strange, maybe it isn't a bug, maybe it's intentional. That is because the issue I describe here isn't actually the what I observed. I saw something weird happen but I can't describe it so I describe this instead :100:.
How to reproduce

Have sex without contraceptives
Create a group of two or more windows.
Lock the group.
Switch to the first window in the group.
Open another window to the side or on another monitor.

Ensure the new window (outside the group) is focused.

Click on the second window in the group.
Observe that the focused window does not change.

The window outside the group remains focused.

Crash reports, logs, images, videos
No response

should be fixed in git

Was it a regression that has been fixed within the last day?

you're on a month old version
https://github.com/hyprwm/Hyprland/pull/5224

Thanks for catching that, very strange.


[Skeleton] Add rounded variant

closes https://github.com/mui/material-ui/issues/33043, from #33044

[x] I have followed (at least) the PR section of the contributing guide.

Just chiming in, we need to update the variant prop type and then it's API docs.

Just chiming in, we need to update the variant prop type and then it's API docs.

Done! nice catch


Change UG

Codecov Report

Merging #290 (ba2864d) into master (7c10293) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #290      +/-   ##
============================================
- Coverage     70.02%   70.00%   -0.03%
  Complexity      826      826
============================================
  Files           131      131
  Lines          2886     2887       +1
  Branches        375      376       +1
============================================
  Hits           2021     2021
  Misses          738      738
- Partials        127      128       +1

Impacted Files
Coverage Δ

.../java/seedu/address/model/applicant/Applicant.java
83.63% <0.00%> (-0.77%)
:arrow_down:

...java/seedu/address/logic/commands/job/EditJob.java
94.80% <0.00%> (ø)

Continue to review full report at Codecov.

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


chore: setup codecoverage

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


SSH doesn't work with driver 'none'

docker-machine version 0.3.0-dev (fec8b4a)
I've created a convenience (shorter) link to the running machine
docker-machine create --url=tcp://agrande-docker-1.cloudapp.net:2376 az1
The SSH doesn't work if trying to go through the link, and env doesn't know anything about the machine:
06:12:31::docker-964 ♨ >  docker-machine ls
NAME               ACTIVE   DRIVER       STATE     URL                                        SWARM
agrande-docker-1   *        azure        Running   tcp://agrande-docker-1.cloudapp.net:2376
az1                *        none                   tcp://agrande-docker-1.cloudapp.net:2376
06:13:36::docker-964 ♨ >  docker-machine ssh az1
Error: Cannot run SSH command: Host "az1" is not running
06:13:44::docker-964 ♨ >  docker-machine ssh agrande-docker-1
Welcome to Ubuntu 14.04.1 LTS (GNU/Linux 3.13.0-36-generic x86_64)

 * Documentation:  https://help.ubuntu.com/

  System information as of Wed Jun  3 22:10:04 UTC 2015

  System load:  0.0               Processes:              229
  Usage of /:   5.0% of 28.80GB   Users logged in:        0

... (omitted)

Last login: Wed Jun  3 22:10:05 2015 from pool-108-29-17-62.nycmny.fios.verizon.net
ubuntu@agrande-docker-1:~$ exit
logout
06:14:04::docker-964 ♨ >  docker-machine env az1
open /Users/aperepelytsa/.docker/machine/machines/az1/ca.pem: no such file or directory
06:14:10::docker-964 ♨ >

This driver is meant for testing.
@nathanleclaire perhaps we should remove this from the docs as being a "usable" driver?  Generic replaces what it did before I believe.

Hmmm, yeah, none and url tend to be really confusing, and we generally don't pay them much attention, so we might want to consider deprecating or clarifying their roles more like you mentioned.

perhaps we should remove this from the docs as being a "usable" driver?

+1

Well, --url is a bit different because the basic idea is to have an existing daemon which is running and configured already, and you just set the port to connect on.  With our TLS setup though, I'm really starting to think it's more trouble than it's worth.

I don't see none anymore (?) in the docs.
Closing this for bookkeeping. Let me know if there is still something to be done.

The original idea for "

No, none is not just for testing.
It was originally intended to be the "default" driver which you could use to just add an existing host in without doing anything, e.g. docker-machine create --url unix:///var/run/docker.sock local
It is not intended to make it so that SSH could work (although maybe it could?). However, it has definitely languished in terms of support for even the simplest use case and needs to be fixed.

docker-1.9@ubuntu14 / docker-machine-0.5@osx is broken now, but used to work with previous versions.
user@osx$ ping dockerhost; ssh dockerhost; telnet dockerhost 2376 - OK
user@osx$ docker-machine -D create -d none --url=tcp://dockerhost:2376 dockerhost
Docker Machine Version: 0.5.0 (HEAD)
Found binary path at /usr/local/bin/docker-machine-driver-none
Launching plugin server for driver none
Plugin server listening at address <IP_ADDRESS>:51018
() Calling RpcServerDriver.GetVersion
Using API Version 1
() Calling RpcServerDriver.SetConfigRaw
() Calling RpcServerDriver.GetMachineName
(flag-lookup) Calling RpcServerDriver.GetCreateFlags
Making call to close connection to plugin binary
Making call to close driver server
(flag-lookup) Calling RpcServerDriver.Close
Successfully made call to close driver server
Found binary path at /usr/local/bin/docker-machine-driver-none
Launching plugin server for driver none
Plugin server listening at address <IP_ADDRESS>:51026
() Calling RpcServerDriver.GetVersion
Using API Version 1
() Calling RpcServerDriver.SetConfigRaw
() Calling RpcServerDriver.GetMachineName
(dockerhost) Calling RpcServerDriver.GetMachineName
(dockerhost) Calling RpcServerDriver.DriverName
(dockerhost) Calling RpcServerDriver.GetCreateFlags
(dockerhost) Calling RpcServerDriver.SetConfigFromFlags
Running pre-create checks...
(dockerhost) Calling RpcServerDriver.PreCreateCheck
(dockerhost) Calling RpcServerDriver.GetConfigRaw
(dockerhost) Calling RpcServerDriver.GetConfigRaw
Creating machine...
(dockerhost) Calling RpcServerDriver.Create
(dockerhost) Calling RpcServerDriver.GetConfigRaw
(dockerhost) Calling RpcServerDriver.GetConfigRaw
(dockerhost) Calling RpcServerDriver.DriverName
Reticulating splines...
(dockerhost) Calling RpcServerDriver.GetConfigRaw
(dockerhost) Calling RpcServerDriver.GetConfigRaw
To see how to connect Docker to this machine, run: /usr/local/bin/docker-machine env dockerhost
user@osx: docker-machine env dockerhost
docker-machine -D env dockerhost
Docker Machine Version: 0.5.0 (HEAD)
Found binary path at /usr/local/bin/docker-machine-driver-none
Launching plugin server for driver none
Plugin server listening at address <IP_ADDRESS>:51054
() Calling RpcServerDriver.GetVersion
Using API Version 1
() Calling RpcServerDriver.SetConfigRaw
() Calling RpcServerDriver.GetMachineName
(dockerhost) Calling RpcServerDriver.GetState
Error running connection boilerplate: dockerhost is not running. Please start it in order to use the connection settings
Any ideas why "none" driver has stopped working?

I think it has been replaced by generic driver
https://docs.docker.com/machine/drivers/generic/


Yet Another Die-Update-Restart Bug

Tbh, idk how to fix this one. This is when you pushed a new update and I restarted the bot.
OS: Ubuntu LTS on a Private Server
Run Method: .die = Restart and Auto-Update
Using: pm2
0|Nadeko   |   Restore completed in 17.73 ms for /home/earth/NadekoBot/NadekoBot.Core/NadekoBot.Core.csproj.
0|Nadeko   |   Restore completed in 5.43 ms for /home/earth/NadekoBot/src/NadekoBot/NadekoBot.csproj.
0|Nadeko   |   Restore completed in 1.86 ms for /home/earth/NadekoBot/src/NadekoBot/NadekoBot.csproj.
0|Nadeko   |   Restore completed in 18.33 ms for /home/earth/NadekoBot/NadekoBot.Core/NadekoBot.Core.csproj.
0|Nadeko   |   Restore completed in 10.04 ms for /home/earth/NadekoBot/NadekoBot.Tests/NadekoBot.Tests.csproj.
0|Nadeko   | Microsoft (R) Build Engine version 15.3.409.57025 for .NET Core
0|Nadeko   | Copyright (C) Microsoft Corporation. All rights reserved.
0|Nadeko   | Services/Impl/BotConfigProvider.cs(197,91): error CS1738: Named argument specifications must appear after all fixed arguments have been specified [/home/earth/NadekoBot/NadekoBot.Core/NadekoBot.Core.csproj]
0|Nadeko   | Build FAILED.
0|Nadeko   | Services/Impl/BotConfigProvider.cs(197,91): error CS1738: Named argument specifications must appear after all fixed arguments have been specified [/home/earth/NadekoBot/NadekoBot.Core/NadekoBot.Core.csproj]
0|Nadeko   |     0 Warning(s)
0|Nadeko   |     1 Error(s)
0|Nadeko   | Time Elapsed 00:00:09.11

I need to look at this this option and I will hopefully later today. For now, just do the following:
cd ~ && wget -N https://gist.githubusercontent.com/shikhir-arora/b7a34e2caf67a8e31ea5cd4f3e0f6421/raw/2a9f9c8e29612164d830de3ca5010e85fa54dea6/nadeko-update-pm2.sh && sudo bash nadeko-update-pm2.sh
`

Building NadekoBot
Microsoft (R) Build Engine version 15.3.409.57025 for .NET Core
Copyright (C) Microsoft Corporation. All rights reserved.

Services/Impl/BotConfigProvider.cs(197,91): error CS1738: Named argument specifications must appear after all fixed arguments have been specified [/home/earth/NadekoInstall_Temp/NadekoBot/NadekoBot.Core/NadekoBot.Core.csproj]

Build FAILED.

Services/Impl/BotConfigProvider.cs(197,91): error CS1738: Named argument specifications must appear after all fixed arguments have been specified [/home/earth/NadekoInstall_Temp/NadekoBot/NadekoBot.Core/NadekoBot.Core.csproj]
    0 Warning(s)
    1 Error(s)

The error still persists. 🤔

Make sure you are in /home/earth when you run it.

Well, ~ is set in /home/earth so running either of these will do the same thing, with the exact same results.
I'll wait and see if Kwoth has anything to say about this, since the build error is in the database. He might need to redo the migration script for BotConfig

Where is your NadekoBot actually installed though? Also, did you make any modifications to the bot at all?

what is this wall of text
@ronelm2000 just update again, i just fixed it

Looking at the change in the commit, I say that this should be raised to rosyln somehow if that pre-change worked for Windows but not Linux.

I didn't notice this was just an hour ago. It makes sense why it didn't work with Linux now that I see it. No worries. I had assumed you had issues prior based on the title of this issue.

The error was so bizzare I didn't know what to name it.


Adding a Filter property to exclude assemblies/types

Rationale
Coverlet currently allow to ignore methods/classes using attribute or specific source files using /p:Exclude property.
It would be useful filter assembly/types in a "similar" way of popular OpenCovers framework.
Proposal
Some examples
To Include assembly but exclude all classes of particular namespace.
dotnet test /p:CollectCoverage=true /p:Filter="+[AssemblyName] -[AssemblyName]Namespace.*"

To include only one class
dotnet test /p:CollectCoverage=true /p:Filter="+[AssemblyName]Namespace.ClassName"

To include all classes ends with "ClassName"
dotnet test /p:CollectCoverage=true /p:Filter="+[AssemblyName]*ClassName"

Details
Filters can be inclusive and exclusive represented by + and – prefix respectively, where exclusive (-) filters take precedence over inclusive (+) filters.
cc: @tonerdo @abe545  @ido-namely @ryanmvt @coenm @mathnogueira

I'll keep issue updated with our discussion!

I intend to look into this over the weekend

Copied from Merged PR comments for clarity:
I'm thinking maybe we could have also an /Include= and say that default for coverlet if attributes are missing is /Include=[] this could address one of my coverage scenario where i need to be sure that only 2/3 of my tens of class are covered for > 95% or fail! This may appear strange but in my scenario i "host" these classes on more "environment host" and i don't need to be sure that all are "well covered", but would be great if i could "enforce" my devs to cover well what i think is very important. What do you think?

#94 merged in an released https://www.nuget.org/packages/coverlet.msbuild/2.0.0. Thanks for your patience guys


Prepare release v1.3.5

Full changes: https://github.com/containerd/containerd/compare/v1.3.4...60cf2f0374671df31d0c61b02fdc4b56fd0dd340
Also cherry-picks commits for .github/workflows/release.yml: https://github.com/containerd/containerd/pull/4282 https://github.com/containerd/containerd/pull/4291 https://github.com/containerd/containerd/pull/4292 https://github.com/containerd/containerd/pull/4303
Required by https://github.com/moby/moby/pull/41115

Do we want to bring the release GH Action from master to this branch before releasing now that we have tested & validated it works?

@estesp Done. PTAL.

@kevpar - PTAL -  Looks like they added options.Root to the Cleanup api. We will want to rev hcsshim to the new proto format here.

@jterry75 Proto isn't modified: https://github.com/containerd/containerd/pull/4329/files


Localisation settings

Hi John,
As I mentioned in the forum, I found I needed to localise this to get the Australian channels I wanted so I went one extra step and added a setting to be able to configure this.
By default it will use the location of your device, as defined in your device settings, check to see if an API with that country codes subdomain returns a 200, then uses that. If not, it will use the default API. Users can also change the skill settings to use either Global (normal API), Australia or NZ based search. These were the only country codes I found that worked but I didn't try to test them all.
Also if you try it out, the settings block for this shows up at the bottom of the list as I left the lowercase 'i' in. We should be doing a case insensitive sort on skill setting titles in that list so I think it's better that we fix that on the backend.
If you're not wanting to complicate your skill with settings, feel free to reject this, I was just halfway there so figured I should wrap it up and contribute it back :)
Thanks again for a great Skill!

Awesome! I just responded to your forum message. Glad to see you've already
got this going. :) I think this is a great addition to the skill. I'll see
if I can find time to do quick testing on it this weekend and merge it.
-j
On Mon, Jun 10, 2019 at 1:36 AM Kris Gesling<EMAIL_ADDRESS>wrote:

Hi John,
As I mentioned in the forum, I found I needed to localise this to get the
Australian channels I wanted so I went one extra step and added a setting
to be able to configure this.
By default it will use the location of your device, as defined in your
device settings, check to see if an API with that country codes subdomain
returns a 200, then uses that. If not, it will use the default API. Users
can also change the skill settings to use either Global (normal API),
Australia or NZ based search. These were the only country codes I found
that worked but I didn't try to test them all.
Also if you try it out, the settings block for this shows up at the bottom
of the list as I left the lowercase 'i' in. We should be doing a case
insensitive sort on skill setting titles in that list so I think it's
better that we fix that on the backend.
If you're not wanting to complicate your skill with settings, feel free to
reject this, I was just halfway there so figured I should wrap it up and
contribute it back :)
Thanks again for a great Skill!
You can view, comment on, or merge this pull request online at:
https://github.com/johnbartkiw/mycroft-skill-iheartradio/pull/1
Commit Summary

use au API, and hack fix triplej search
use local api if available
add user localisation settings

File Changes

M init.py
https://github.com/johnbartkiw/mycroft-skill-iheartradio/pull/1/files#diff-0
(30)
A settingsmeta.yaml
https://github.com/johnbartkiw/mycroft-skill-iheartradio/pull/1/files#diff-1
(10)

Patch Links:

https://github.com/johnbartkiw/mycroft-skill-iheartradio/pull/1.patch
https://github.com/johnbartkiw/mycroft-skill-iheartradio/pull/1.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/johnbartkiw/mycroft-skill-iheartradio/pull/1?email_source=notifications&email_token=ADSLGCWXWCBX2ASKJQIO3R3PZYHBHA5CNFSM4HWRP3X2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GYQMEDA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADSLGCU5N7CYO7MBQJXZZDDPZYHBHANCNFSM4HWRP3XQ
.

Oh I just realised that I left a quick hack in there to make a station name match.
We have a station that iheart lists as "triplej". The STT correctly transcribes "triple j" but with the space it returns no match.
Maybe something for us to think about how to handle as I'm sure it won't be an isolated case. Perhaps just a dictionary that can be added to over time? Eg
{
    'stt transcription': 'station name',
    'triple j': 'triplej',
    ...
}

I'll think about it a bit.
I don't like the idea of a dictionary because it would need to be edited
manually and will continually be out of date as stations change. My initial
instinct is to search for the STT transcription and then if that returns
nothing to re-search with whitespace removed from the transcription. That
would work with this specific case for sure and I would wager a good
percentage of other stations in the same class.
On Mon, Jun 10, 2019 at 7:36 PM Kris Gesling<EMAIL_ADDRESS>wrote:

Oh I just realised that I left a quick hack in there to make a station
name match.
We have a station that iheart lists as "triplej". The STT correctly
transcribes "triple j" but with the space it returns no match.
Maybe something for us to think about how to handle as I'm sure it won't
be an isolated case. Perhaps just a dictionary that can be added to over
time? Eg
{
'stt transcription': 'station name',
'triple j': 'triplej',
...
}
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/johnbartkiw/mycroft-skill-iheartradio/pull/1?email_source=notifications&email_token=ADSLGCUHJN5YVRTIUDTDBBTPZ4FR5A5CNFSM4HWRP3X2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXLYMXY#issuecomment-500663903,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADSLGCXBLVJWUCLWAQCOQ2LPZ4FR5ANCNFSM4HWRP3XQ
.

Yep, I'm all for solutions that don't require constant maintenance!


More badge fixes

Fixes https://github.com/microsoft/pxt-arcade/issues/4294
Fixes https://github.com/microsoft/pxt-arcade/issues/4284
Fixes https://github.com/microsoft/pxt-arcade/issues/4292
Fixes https://github.com/microsoft/pxt-arcade/issues/4297
https://github.com/microsoft/pxt-arcade/issues/4292 actually didn't have anything to do with badges, I think it was broken by the recent code carryover change.

Sorry merged this so I can produce a build.


Flicker with inline suggestions

Version: 1.67.0-insider
Commit: 909602cf04e0fa42528d692676d0d4e15e67bba2

I could only get this repro with these very specific steps:

duplicate line down at line 48 in src/vs/editor/test/common/services/modelService.test.ts
select modelService
type out instantiationService
press cmd+/ (notice that something weird begins to happen because Mock now shows)
do cursor right and cursor left (notice the flicker on cursor right)

https://user-images.githubusercontent.com/5047891/161239402-171243b9-6ad0-4af6-b78a-7f63748cc93f.mp4

Our conclusion was to only trigger inline suggest when the cursor moves because of typing, similar to quick suggest

:+1: The triggering of inline completions being very eager on cursor move I also filed as https://github.com/microsoft/vscode/issues/146529 .
I think you can definitely close both issues with the same change (to trigger only on type), but in this particular video, if you look at timestamp 0:14 or so you can see some sort of flicker, I think the inline completion appears for a frame and then disappears again at the same cursor positon. i.e. there might be a secondary bug in there, but maybe it becomes irrelevant.

This is the flicker at ~0:14. I looks like inline suggest shows out of sync with the cursor, note the : between the ghost text and the cursor. @hediet I am not reproducing this anymore and I wonder if this is fixed in the meantime?

Now, if you move the cursor, the inline completion session is canceled.
But thanks for the flickering screenshot - I think we should not show replacements before the cursor, which is still a bug.

I can still repro the issue where mock shows up unexpectedly
https://user-images.githubusercontent.com/29464607/165823806-dbfaefed-bb85-414f-9969-173cf488be78.mov

Since this behavior exists since the inline completion feature has been introduced, it should be okay to move this bug to next milestone.


fix: Avoid polling the consumer if it says it's paused

There is a bug in StreamProcessor where self.__is_paused, but at the
same time we poll the consumer and get messages.
  File "sentry/runner/commands/run.py", line 455, in basic_consumer
    run_processor_with_signals(processor, consumer_name)
  File "sentry/utils/kafka.py", line 46, in run_processor_with_signals
    processor.run()
  File "arroyo/processing/processor.py", line 323, in run
    self._run_once()
  File "arroyo/processing/processor.py", line 442, in _run_once
    assert self.__consumer.poll(0.1) is None

Breadcrumbs indicate that rebalancing was happening at that time. Could
it be that we were paused, but the consumer got more partitions
assigned? It doesn't make sense since we don't have cooperative-sticky
rebalancing enabled, let's add more logs and prevent the crash.
related: https://github.com/getsentry/arroyo/pull/391

related: #413


[Metadata] Get devices by servicename returns 503 unexpected end of JSON input

🐞 Bug Report
Affected Services

 The issue is located in:
core-metadata
Is this a regression?

Unsure
Description and Minimal Reproduction
When attempting to initialize the device cache, the device-sdk-go makes a REST call to core-metadata, which returns an incorrect HTTP 503 error. This error is only observed when querying the specific device service device, and not all of the devices. See my gist for my current devices.
Also, 503 is the wrong error to return in this case it should be a 500.
🔥 Exception or Error
GET http://localhost:48081/api/v1/device

Response Gist

GET http://localhost:48081/api/v1/device/servicename/edgex-device-rfid-llrp

HTTP: 503

unexpected end of JSON input

Observed via device-rfid-llrp-go service startup:

level=ERROR ts=2021-06-16T17:36:12.091729084Z app=edgex-device-rfid-llrp source=init.go:45 msg="Device cache initialization failed: 503 - unexpected end of JSON input\n"

🌍  Your Environment
Deployment Environment:
Ubuntu 20.04
Docker
EdgeX Version:
Hanoi via edgex-compose repo.
Anything else relevant?
Encountered while running device-llrp-rfid-go service.

After running make clean on edgex-compose and re-discovering my device I no longer see the issue. It is possible this was due to me manually removing keys from Redis.

Per @ajcasagrande note above, will reopen if this issue resurfaces.


Adding value to the template from a Workflow/Action

Hi Florian,
as usual, thank you for your work on this tool, awesome.
I'd like to suggest a feature that could be very useful for templating, for example, an Email.
I've been requested to send a reminder for a record of Appointment entity and I've created an HTML Template, but, as you might know, by standard I cannot insert into the Regarding field the lookup to the appointment record; the problem is that I'd like to insert into the template the start time and the location of the appointment which has to be reminded of and I cannot use the standard tags, like {Start Time(Appointment)}, that i can obtain from my action.
Do you have some guess of how to do it? Maybe there is already a way to solve my problem and I missed it!
Best regards,
Andrea

Hi Andrea,
thanks for getting in touch.
You can do this without any issues, the templating is not bound to the regarding object.
The easiest way would be to customize your E-Mail entity with a new Lookup which points to appointments and to fill it inside your workflow.
Then inside the template, just use ${{Value("yourAppointmentLookup.starttime")}}.
Let me know if this solves your problem.
Kind Regards,
Florian

Yeah I thought about that too but this is bound to the presence of the relation Email-Appointment; I'd like to add the information without creating the relationship also because if I have to obtain another information from another Entity I'd be obliged to add another relation.
If there isn't another way I'll follow your advice.
Thank you,
Andrea

Hi @DigitalFlow,
in the end I went with your first advice, We want to make our template the most user-friendly possibile so the easiest way to make a non-tecnichal user understand what the template is doing is adding a new relationship.
Thank you for your time!
I saw some problems with the copy-paste functionality in the HTML templating tool, sometimes if I paste  the ${{Value("")}} keyword the templating language is not catching it; i don't know if this is a problem of the Unlayer Editor tool or of the Templating Language. I solved this writing by hand all the keywords of the templating tool so it's not an important problem but I'd like to let you know.
As always, thank you.
Best regards,
Andrea

Hi @andreaPorini,
yes, pasting into the unlayer editor causes issues sometimes, as pasting directly from another webpage often includes html  or  tags, which breaks the XTL expressions.
I often paste into Notepad first before pasting into the unlayer editor.
But the best workaround is possibly to paste the XTL templates into unlayer HTML components, because there no special actions happen upon pasting expressions.
So whenever possible, I'd suggest to do it like that.
Writing by hand solves it as well, that's correct.
You're very welcome, I'm happy that you enjoy this tool.
Kind Regards,
Florian

Short follow up @andreaPorini:
As it turns out, your browser is already capable of pasting copied text withour formatting into the templates.
Instead of using Ctrl + V, you can either use Ctrl + Shift + V or right click where you want to paste and click on "Paste unformatted".


⚠ Samsung has degraded performance

In b0725e9, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2045 ms

Resolved: Samsung performance has improved in 2dd9960.


Chore: Upgrade flutter version on example.

Codecov Report

Merging #18 (ed0a858) into main (0a2f088) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #18   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            5         5
  Lines          109       109
=========================================
  Hits           109       109

Continue to review full report at Codecov.

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


Memory leaks

seeing memory leaks when using this

It's quite possible. I'll look into during this week and let you know, thanks.


How can I change the duration of a highlight

Hi,
I want to change the duration of highlights, I want to be able to watch smoke highlights, but then also find out what the player does after the smoke has been deployed.
How can this be done?

A new major version has been released, closing the issue 🧹


Extension Feature Requests: Shortcut to cherry webpage and already favorited indicator

Hi, I have two feature requests for the browser extension.
One is a way to get to the cherry webpage through the extension icon. Some of my other extensions have a right click menu with custom entries so it should be possible to add a small shortcut there. (I'm using Firefox, I don't know if Chrome can do the same)
The other request is adding a small icon (like a star) in the corner of the extension icon so you know if you've already favorited a page. I have other extensions that show a number on their icon (uBlock Origin shows blocked ads, Bitwarden shows how many saved passwords for a page) but I don't know if that's a feature built in to Firefox and if it is limited to just numbers.
Loving everything else about Cherry so far, good job

Implemented. It should be available on Firefox very soon, but will take time on Chrome.
The shortcut is Control+Shift+E (Cmd+Shift+E on macOS) and can be customized on both Firefox and Chrome.
I tried to add a star to the right bottom of the "cherry" icon, but just doesn't look great. I end up use the "leaf" icon as the default icon and "cherry" as the picked indicator icon.


stub out alerts page and link on left menu

This provides a mock alerts list with composite expansion.

I like the drop down idea and the list entries look relatively clean.  Good start, this will be a work in progress as alerts/events have a lot of filtering and a lot of data to present.


Fix uploaded artifacts should be overwritten

Fix Uploaded artifacts should be overwritten https://github.com/go-gitea/gitea/issues/28549
When upload different content to upload artifact, it checks that content size is not match in db record with previous artifact size, then the new artifact is refused.
Now if it finds uploading content size is not matching db record when receiving chunks, it updates db records to follow the latest size value.

Test failure seems to be related to changes


Python example to track the mouse movement?

Does the current python API allow custom actions such as tracking the mouse dragging event to get the continuous mouse coordinates? Thank you.

Use input_event_bindings.viewer rather than input_event_bindings.data_view can do this trick.


Modifications for MAC compile

Modifications for MAC compile.  Using #ifdef _Win32 to bypass to printf functions and Growl mods in notifier.

good first attempt but contains compiler generated files. please redo pr with just your exact changes in .h/.cpp files.


Call to r.snap() waits till a tab is clicked - pending more details and exploration

I am very new to RPA-Python. I am facing a problem where a call to snap() waits till I have clicked a tab. I wanted to know how I can fix this and run the flow without having to click on the tab.
I am trying to run a simple use case where I do the following.

Open a URL
Open another URL in a new tab
Search something in the first tab
Take a snapshot of the first tab
Switch to the second tab
Search something in the second tab
Take a snapshot of the second tab

The problem I am facing is that on step 4 - the call to snap() waits till I have clicked on the first tab. And then the code starts moving again :
import rpa as r

r.debug(True)
r.init()
r.url('https://duckduckgo.com/'>)
r.wait()
r.dom('window.open("' + 'https://www.google.com' + '")')
r.wait()
r.popup()  #Because I want the tab with first URL to be active
r.type('//*[@id="search_form_input_homepage"]', r'r.popup')
r.click('//*[@id="search_button_homepage"]')
r.wait()
r.snap('page', r'D:/URL1.png') #This is where the code stops. At this point,
                               #if I click the tab for the first URL
                               #the snap() function returns

r.popup(r'google.com')
r.type('q', r'r.popup')
r.wait()
r.click('btnK')
r.wait()
r.snap('page', r'D:/URL2.png')
r.close()

I am using rpa version '1.45.0' on Python 3.6.5 on Windows 10 Pro.
A  little digging into the code showed that snap() calls send() which in-turn has the following line:
"while _tagui_started and not _ready(): pass"
I tried debugging and this is where the code waits until I click on the first tab (Looks like tagUI spits out something like, "[RPA][18] - listening for inputs" which causes _ready() to return True and break from the loop), but I have no idea why this is happening.
If I run in headless mode there is no problem. The code does what it is supposed to do.
If I open only one URL the snap works nicely.
Any pointers on what I am doing wrong and how to fix this ?

Hi @pachakra sorry for my very late reply! Was away for months. Did you manage to find a solution?
I believe a possible reason could be during visible mode, when the tab is not active it does not render and could not return the snap binary data back. An idea I can think of is using keyboard() to tab to switch tab, or use visual automation to click on the tab you want. You can try if that works for you. Or perhaps run in headless mode to achieve the automation.

Hi @kensoh - thanks much for the response. As I mentioned (and you suggested as well) running in headless mode works, so I am using that for now. I did try adding keyboard() to switch to the tab but had the same problem. Something like this
r.keyboard('[ctrl][shift][tab]')
I was more curious about the behaviour when headless was set to False. Since I am not blocked for now I will close this.

Hi @pachakra, sure. Thanks for your feedback. It is probably some limitation in Chrome rendering, I can replicate it.


Added support for shark europe devices

Hi,
here is my PR for the European device support from the legacy repository.
I tested it again with my Shark RV1000SEU and reconfirmed that the URLs still work with.

When this pull request is merged, could someone please do the needed adjustments (choosing between US & EU to activate the europe boolean) in HA for this?

Thanks for bringing this over! The code looks good to me, but I am not in Europe and unable to test it. Any additional comments from those able to test it?

When this pull request is merged, could someone please do the needed adjustments (choosing between US & EU to activate the europe boolean) in HA for this?

Yes, this is definitely something we're interested in and can get working on once this is merged.

Should be able to add a region string to the auth & reauth flows in homeassistant/components/sharkiq/strings.json in the HASS repo.
I know there is a way to make it a drop down but I'm not sure off the top of my head.  Should be in the docs.  Worst case, I know of a couple other integrations that do that so we can always look at what they do.
Then we'll just have to build in detection of that region string to the auth flow and set the parameters accordingly.  Nice work @TheLastFrame!  I think this will integrate easily.

@JeffResc any idea why the tests are failing here? Because the europe parameter is optional it shouldn't break the existing implementation.

Resolves #2 upon merge.

@JeffResc any idea why the tests are failing here? Because the europe parameter is optional it shouldn't break the existing implementation.
Should be fully backwards compatible with no breaking changes.

I never used Code Coverage, but as far as I understood the error message, it tries to check where the init function sets the class variables and doesn't know the europe flag yet, so it created an error. (see lines below marked with E at beginning - the first one should be where the europe flag is located)
_____________________ TestAylaApi.test_init__required_vals _____________________

self = <tests.test_ayla_api.TestAylaApi object at 0x7f5e77464b50>

def test_init__required_vals(self):
    api = AylaApi(
<EMAIL_ADDRESS>"mypassword", "app_id_123", "appsecret_123"
    )

    assert api._email ==<EMAIL_ADDRESS>    assert api._password == "mypassword"
    assert api._access_token is None
    assert api._refresh_token is None
    assert api._auth_expiration is None
    assert api._is_authed == False
    assert api._app_id == "app_id_123"
    assert api._app_secret == "appsecret_123"
    assert api.websession is None
E       assert (None,) is None
E        +  where (None,) = <sharkiq.ayla_api.AylaApi object at 0x7f5e77[40]
(https://github.com/JeffResc/sharkiq/runs/5993555819?check_suite_focus=true#step:5:40)d2d0>.websession

tests/test_ayla_api.py:[43](https://github.com/JeffResc/sharkiq/runs/5993555819?check_suite_focus=true#step:5:43): AssertionError

Codecov Report

Merging #16 (bf7b064) into main (e091c36) will increase coverage by 0.42%.
The diff coverage is 38.46%.

@@            Coverage Diff             @@
##             main      #16      +/-   ##
==========================================
+ Coverage   52.41%   52.84%   +0.42%
==========================================
  Files           5        5
  Lines         414      422       +8
==========================================
+ Hits          217      223       +6
- Misses        197      199       +2

Flag
Coverage Δ

unittests
52.84% <38.46%> (+0.42%)
:arrow_up:

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

Impacted Files
Coverage Δ

sharkiq/ayla_api.py
66.19% <26.66%> (+0.01%)
:arrow_up:

sharkiq/sharkiq.py
40.31% <28.57%> (-0.16%)
:arrow_down:

sharkiq/const.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

Looks like the tests need to be updated for this.

I was stumped for a bit on this one, but I found a stray comma that was causing the errors. Seems like its good now!

This looks all set to me, thanks @TheLastFrame!


pull request for e2e test at 01/23/2017 05:56:14

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Quickstart (Chapter 3.8): riscof run ... makes an error

Hello,
I followed the whole Quickstart guide you wrote on: https://riscof.readthedocs.io/en/stable/installation.html#create-neccesary-env-files
And i ran into some issues. One issue I couldn't resolve until now was that if I run the tests with :
riscof run --config=config.ini --suite=riscv-arch-test/riscv-test-suite/ --env=riscv-arch-test/riscv-test-suite/env
I get at an error after the test misalign2-jalr-01.S. Until there every test passed but I think it fails at test I-LB-01.S
Following is outputted on my console:
...
    INFO | /home/JStuessi/Documents/riscof_workspace/firsttest/riscv-arch-test/riscv-test-suite/rv32i_m/privilege/src/misalign-sw-01.S : bb74a4aefaa8c89fb28b876484cfdf9ca020cec1 : Passed
    INFO | /home/JStuessi/Documents/riscof_workspace/firsttest/riscv-arch-test/riscv-test-suite/rv32i_m/privilege/src/misalign2-jalr-01.S : bb74a4aefaa8c89fb28b876484cfdf9ca020cec1 : Passed
Traceback (most recent call last):
  File "/home/JStuessi/.pyenv/versions/3.6.0/bin/riscof", line 8, in <module>
    sys.exit(cli())
  File "/home/JStuessi/.pyenv/versions/3.6.0/lib/python3.6/site-packages/click/core.py", line 1128, in __call__
    return self.main(*args, **kwargs)
  File "/home/JStuessi/.pyenv/versions/3.6.0/lib/python3.6/site-packages/click/core.py", line 1053, in main
    rv = self.invoke(ctx)
  File "/home/JStuessi/.pyenv/versions/3.6.0/lib/python3.6/site-packages/click/core.py", line 1659, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/JStuessi/.pyenv/versions/3.6.0/lib/python3.6/site-packages/click/core.py", line 1395, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/JStuessi/.pyenv/versions/3.6.0/lib/python3.6/site-packages/click/core.py", line 754, in invoke
    return __callback(*args, **kwargs)
  File "/home/JStuessi/.pyenv/versions/3.6.0/lib/python3.6/site-packages/click/decorators.py", line 26, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/home/JStuessi/.pyenv/versions/3.6.0/lib/python3.6/site-packages/riscof/cli.py", line 335, in run
    template = Template(report_template.read())
  File "/home/JStuessi/.pyenv/versions/3.6.0/lib/python3.6/site-packages/jinja2/environment.py", line 1195, in __new__
    return env.from_string(source, template_class=cls)
  File "/home/JStuessi/.pyenv/versions/3.6.0/lib/python3.6/site-packages/jinja2/environment.py", line 1092, in from_string
    return cls.from_code(self, self.compile(source), gs, None)
  File "/home/JStuessi/.pyenv/versions/3.6.0/lib/python3.6/site-packages/jinja2/environment.py", line 749, in compile
    source = self._parse(source, name, filename)
  File "/home/JStuessi/.pyenv/versions/3.6.0/lib/python3.6/site-packages/jinja2/environment.py", line 606, in _parse
    return Parser(self, source, name, filename).parse()
  File "/home/JStuessi/.pyenv/versions/3.6.0/lib/python3.6/site-packages/jinja2/parser.py", line 1038, in parse
    result = nodes.Template(self.subparse(), lineno=1)
  File "/home/JStuessi/.pyenv/versions/3.6.0/lib/python3.6/site-packages/jinja2/parser.py", line 1012, in subparse
    add_data(self.parse_tuple(with_condexpr=True))
  File "/home/JStuessi/.pyenv/versions/3.6.0/lib/python3.6/site-packages/jinja2/parser.py", line 725, in parse_tuple
    args.append(parse())
  File "/home/JStuessi/.pyenv/versions/3.6.0/lib/python3.6/site-packages/jinja2/parser.py", line 523, in parse_expression
    return self.parse_condexpr()
  File "/home/JStuessi/.pyenv/versions/3.6.0/lib/python3.6/site-packages/jinja2/parser.py", line 528, in parse_condexpr
    expr1 = self.parse_or()
  File "/home/JStuessi/.pyenv/versions/3.6.0/lib/python3.6/site-packages/jinja2/parser.py", line 543, in parse_or
    left = self.parse_and()
  File "/home/JStuessi/.pyenv/versions/3.6.0/lib/python3.6/site-packages/jinja2/parser.py", line 552, in parse_and
    left = self.parse_not()
  File "/home/JStuessi/.pyenv/versions/3.6.0/lib/python3.6/site-packages/jinja2/parser.py", line 560, in parse_not
    if self.stream.current.test("name:not"):
AttributeError: 'Token' object has no attribute 'test'

I don't understand the error, do you know what could be the reason for this issue?
Best regards
Julien

Downgrade Jinja2
After several hours trying out to install RISCOF on other Python versions and searching for the error, I found this Post on stackoverflow: https://stackoverflow.com/questions/67659135/flask-token-object-has-no-attribute-test-render-template-error
They said you need to downgrade Jinja2 because it has some issues with python3.6. Therefore you need to uninstall Jinja2 with pip uninstall Jinja2 and reinstall a earlier version like: pip install Jinja2==2.11.3
Best regards
Julien


Understanding the Slice(), Splice() and Split() methods in JavaScript

This fixes #2318
Article checklist
ATTENTION: In your PR - add the ISSUE (topic form) # that this PR corresponds to. You can do this by also adding a comment that states "This fixes #(enter your Idea Content Form #)" to link and close your Idea Content (issue) when the PR is merged.
NOTE: (Please ensure that you have only one open issue + linked pull request at a time. This will ensure that we complete the article in a timely manner from inception to publishing.)
If you have not already please go over our Resources Page for more tips and tricks on how to improve our overall technical writing.
Preliminary Checks - Formatting and Structure

[x] Does your article follow any of the suggested structure formats? - see example formats
[x] Is your article properly formatted in Markdown? - see Github Markdown guide
[x] Have you used the correct folder and file structure? - see Contribution Guidelines
[x] Is your article over 750 words? - Tool: Wordcounter.net
[x] Have you used our preferred heading structure? - H3 (###) and up
[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.
[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.
[x] Have you included a hero image that is 600x400 pixels and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images
[x] In your PR - add a comment that states "This fixes #(enter your Idea Content Form #)" to link and close your Idea Content (issue) when the PR is merged.

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid -  Grammarly
[x] Please place your article through a 3rd party plagiarism checker? We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[ ] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide

Contribution guidelines
For first-time contributors and for more details, see Contributing Guidelines

Hey @twish-254 👋.
Briana here.
Upon further discussion with other reviewers, it has come to our attention that your article does not meet the required standards to be published. Simply because your article has been covered in detail by various sites.
According to our guidelines:
"We lean towards content that offers an original viewpoint/approach around programming-centric topics."
All hope is not lost though. What I can advise you to do is to enhance this article in that you can make it a tutorial by illustrating how to use the .splice() array method in a more unusual, unique, and ingenious way to perform various functions in a web/software app.
At least it will meet the content standards and we will publish it, which will be a win-win for both parties.
In case you have any problem or question, do not hesitate to ping me here, via  Slack or via my personal email (briananzivu@gmail.com).
Regards🚀

Hey @twish-254 .
I will close this pr temporarily so that I can be able to review other ready articles.
This should not worry you. If you want to submit or commit to it, just let me know and I shall open it for you. 😄
Regards.

Demystifying the Slice(), Splice() and Split() methods in JavaScript #2727.pdf
👋 Good afternoon - and welcome to the EngEd program.
There appear to be many sentences in this article that are taken verbatim from third-party sources. We can't publish any content that is not original or properly cited/referenced. There are many free online tools where you can check this (such as quetext.com, and if you're uncertain about the definitions around plagiarism, this is a good resource to reference.
We're really looking for unique/original content

@hectorkambow  i have revised my article and removed the plagiarism.

plagiarism check done ✅
Please proceed with the review @BrianaNzivu

@BrianaNzivu I have made the requested changes

👀
Upon a quick glance - it seems this PR is still quite a bit aways from being completed.
As a rule of thumb - please only submit articles (pull requests) with a complete and fully polished article - ready to be published.
All revisions and edits should be completed on your own forked repo (as to not take up room in the queue).
We will be closing this and please re-submit when it is ready to be published as a new PR (and you can add the duplicate label) and ping the reviewer and myself.
@twish-254
cc @BrianaNzivu


Fix template formatting in main.css

Custom CSS wasn't working because the template formatting was incorrect, likely due to whatever CSS formatter you're using.

Thanks for your contribution!
I didn't notice that the formatter broke the syntax.


[WIP] Add functionality for daily-master to NCU-CI

@sam-github heres my draft for what I have so far.

For context: we're looking to get ncu-ci running against daily builds, because, by definition, any failure on master is either a failure in the infrastructure, or a flaky test.

We're trying to figure out if we should push it forward because its useful and achievable, or perhaps whether its a bit of an improvement already and can be merged, or perhaps it was just an interesting experiment and we should abandon it.
@joyeecheung what do you think?

@sam-github Is there a summary of what this does? AFAICT it just adds some logic so that the command can be used on daily build data and it adds some logic to make it more fail-safe?

The intention was just this,

adds some logic so that the command can be used on daily build data

but maybe some other logic was added as needed.

@sam-github I think this should be good to get in with the commented code removed

ive rebased and removed the commented code.

Can this be landed? I dont have permission to merge.

Can this be landed? I dont have permission to merge.

cc @nodejs/automation-collaborators

Ive rebased this against master so is free to be landed again


[BUG] BinaryClassificationTask prediction does not return one score per session with summary_type='mean'

Bug description
When we train a BinaryClassificationTask and then do
prediction = trainer.predict(eval_paths[0])
prediction

this does not return prediction scores.
In addition, we are able to set  trainer.args.predict_top_k, but we should not be able to do that for BC Task, since this is particularly designed for NextItem Prediction task.
Steps/Code to reproduce bug
Expected behavior
we should be getting a probability score per session.
Environment details

Transformers4Rec version:
Platform:
Python version:
Huggingface Transformers version:
PyTorch version (GPU?):
Tensorflow version (GPU?):

Additional context
using merlin-pytorch:22.12 with the latest main branches pulled.

closing since https://github.com/NVIDIA-Merlin/Transformers4Rec/pull/641 fixing the bug.


JAMES-2296 JAMES-3689 fix removal by recipient in more cases

I don't know why no build was started here, it looks like the CI did not find the PR in the first place.
I will try to reopen, to see how it plays out.

Seems it's building now. Looks like a delay on the CI, go figure^^'

Seems it's building now. Looks like a delay on the CI, go figure^^

Not really, I opened another PR, with the exact same SHA1, hence the build is reported here as well. But this very PR is still not to be found by Jenkins :-)

Merged


"Note" for $adminpass is not correct. It should be local Local Administrator password of ASDK host, not the CloudAdmin default password.

In this article, it says below in the "Note"
"The $adminpass on the script below must be the password of the default "CloudAdmin" user from the production environment when the backup was taken."
The $adminpass should be the Local Administrator password of ASDK host, not the CloudAdmin default password.
If we use cloud admin password, it will report error "The Administrator password provided must match the Administrator password of the host"
The reason is that, in DeploySingleNoddeCommon.ps1 line 533 , there is below command
$credentialSuccess = Invoke-Command -ComputerName 'LocalHost' -Credential $builtInAdminCredential -ErrorAction 'SilentlyContinue' { $true }
This command is to test that the Administrator password matches host administrator password. $builtInAdminCredential comes from below command
$builtInAdminCredential = (New-Object PSCredential -ArgumentList 'Administrator', $AdminPassword)

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

ID: 880d3c99-62f9-e909-e2dc-dfa2ff0cb723
Version Independent ID: d9adc6a8-912e-f789-2833-8610a63c6a4c
Content: Use the ASDK to validate an Azure Stack backup - Azure Stack Development Kit (ASDK)
Content Source: azure-stack/asdk/asdk-validate-backup.md
Service: azure-stack
Sub-service: azure-stack-hub-asdk
GitHub Login: @sethmanheim
Microsoft Alias: sethm

@xizhuang
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@sethmanheim
Can you please check and add your comments on this doc update request as applicable.

Hi, @sethmanheim , can I have an update on this please? Thank you!

#please-close

Hi @Seth @.***>, Thanks for your update!
Regards,
Xiaoyu
From: Seth Manheim @.>
Sent: Wednesday, March 13, 2024 3:54 AM
To: MicrosoftDocs/azure-stack-docs @.>
Cc: Xiaoyu Zhuang @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-stack-docs] "Note" for $adminpass is not correct. It should be local Local Administrator password of ASDK host, not the CloudAdmin default password. (Issue #2374)
@xizhuanghttps://github.com/xizhuang Thank you for reaching out and very sorry for the delay. I will update the documentation today, to specify that this needs to be the local admin password. Please note that the ASDK is still functional, but no longer updated.

Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-stack-docs/issues/2374#issuecomment-1992445528, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMU4IJ5X6LAGSDLEC7OBH6TYX5MPHAVCNFSM6AAAAAAYLIU7PKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOJSGQ2DKNJSHA.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>