Deleting a lot of characters causes corruption.

The following code seems to result in corruption:
  function steve()
     end_of_file()
     local c = 10000
     while( c > 1 ) do
        c = c -1
        delete();
     end
  end

This was actually found via "HOME Ctrl-space END Ctrl-w" (i.e. select who file, and then Ctrl-w to cut).  But it turns out deleting a single character, 10,000 times, is sufficient to cause corruption.

OK with the previous commit we can now reliably trigger this corruption.
Append this to kilo.lua:
  --
  -- Cause corruption.
  --
  function on_init()
     -- Create an input-file
     os.execute( "cp kilo.c kilo.c.tmp" )

     -- Open it
     open( "kilo.c.tmp" )

     -- Move to end of file
     end_of_file()

     -- Delete a lot of lines.
     local c = 10000
     while( c > 1 ) do
        c = c -1
        delete();
     end

     -- All done
     --exit()
  end

Now launch:
   ./kilo /dev/null --eval=on_init

Immediately the display is corrupt.
TODO:

Add exit()
Run under valgrind.

Current state of this bug - still present.
To reproduce:
  ./kilo --config ./valgrind.lua /dev/null  --eval=valgrind

This:

Loads the additional Lua configuration-file valgrind.lua
Opens the file /dev/null for editing (!)
Invokes function valgrind() on startup.

Running make valgrind; Ctrl-q Ctrl-q Ctrl-q ; less valgrind.out shows the output from valgrind.
I've traced through the code several times, and reverted the contents of editorUpdateSyntax, editorUpdateRow and a few more functions to @antirez's original implementation but still the bug appears.
The code seems to be correct, but I'm 90% sure there is a subtle off by one, or other corruption bug in editorDeleteRow.  This is suspicious to me:
editorUpdateRow(row);
row->size--;

Surely size should be decreased before the row is updated?    Anyway reversing that order doesn't help.
Will ponder some more.  Would be nice if this could be confirmed as a bug in the original program, rather than something I've introduced too!

Ahah.
The problem isn't corruption, per se.  The problem is entirely rendering.
To reproduce:
   $ make && ./kilo  --config ./valgrind.lua --eval valgrind /dev/null

At this point the editor will be open, and it will show gibberish.

Pressing "up" or "page-up" will continue to show gibberish.

BUT:

Press "HOME" or "END" and you see all is actually OK.

The problem must be that we're not updating the screen-position correctly, somewhere.
To resolve we can change delete_lua to add the current display offset, etc:
        editorSetStatusMessage("x=%d,y=%d", E.cx, E.cy );  // add colloff + rowoff + & etc.

Now I feel better.

Added some debugging before I left for work.
E.colloff becomes negative, after the valgrind function runs, which is the root cause for the display corruption.  It also explains why pressing HOME or END reinstates the display, because they indirectly reset the row/col offsets.

I tried to reproduce this in original kilo (i.e. the one by antirez) and failed. E.coloff doesn't become negative.

Yeah I couldn't reproduce it in the original either.
There are only two places in the code where E.coloff is decremented, and both look sane to me.  I have a fix:
 @@ -1613,6 +1613,15 @@ int delete_lua(lua_State *L)
              E.cx--;
      }

 +    /*
 +     * BUGFIX: Ensure we don't walk off the screen,
 +     * and draw gibberish.
 +     */
 +    if ( E.coloff < 0 )
 +        E.coloff = 0;
 +    if ( E.rowoff < 0 )
 +        E.rowoff = 0;
 +
 if (row) editorUpdateRow(row);

But I'd like to understand how it is getting into this state first.

This is where it all goes wrong:
    if (E.cx >= E.screencols)
    {
        int shift = (E.screencols - E.cx) + 1;
        E.cx -= shift;
        E.coloff += shift;

        editorSetStatusMessage( "Shift is %d", shift );
    }

The status message shows "Shift is -2".


Support must-revalidate

@khoazero123 are you interested in writing some docs and tests for this? If not I might just close this until someone can pick up the remaining work which would be required to land it.


huggingface[major]: upgrade pydantic

integration tests pass: https://github.com/langchain-ai/langchain/actions/runs/10707626581/job/29688555303


chore: add docs on handling of jwt

Fixes RVT-4003

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1353  👈
#1352
#1351
#1350
#1348
#1347
#1345
#1344
#1343
#1342
#1341
#1340
#1339
#1338
#1337
#1336
#1335
#1334
#1333
#1332
#1329
#1328
#1327
#1322
#1321
#1320
#1319
#1318
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @NathanFlurry and the rest of your teammates on  Graphite


Cannot load module 'Web3swift' as 'web3swift'

Hello. There was a problem with the module. I tried to solve this problem in various ways, but it does not work. Who had a similar problem, please tell me how to solve it.
P.S: I read the post on stackOverflow, and it said that there might be a problem in cocoa pods or Xcode or swift version. I use xCode 10.1, swift 4.2, cocoapods v.1.5.3, in Podfile i'am use pod 'web3swift', '~> 2.0'

use pod 'web3swift', :git => 'https://github.com/bankex/web3swift.git', '~> 2.0'

@v57 thx, but I can't pod install, I have error [!] Invalid Podfile file: syntax error, unexpected '\n', expecting =>.

Oh sorry, its pod 'web3swift', :git => 'https://github.com/bankex/web3swift.git'


fix: typo on update abort log

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
This pull request fixes a typo:
from info('No licenses where updated, let's abort ');
to info( 'No licenses were updated, let's abort');
Checklist

[x] I have squashed my commits into a single one with a message that aligns with the contributing guidelines
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works

Coverage remained the same at 90.816% when pulling fcf6095a4ba3c11e5a5e5548f90c44ed04eeb3e8 on geromegrignon:fix/update-log-typo into e4432857e61361d140b4b5b02ece051ebfaa195b on FantasticFiasco:master.

Hi @geromegrignon.
Thanks for finding and fixing the issue! I'll add you to the list of contributors in README.

Thanks for this great project, I'm currently adding it to all my repos


How to use css style when export html to xls?

Hi, first nice work!
I made a dynamic report using nodejs and angular 1.6. I can export a html table to xls, but when i need to use css style, this is not working, as you can see in the picture below. I tried a lot of things before i came here, if is not issue, pardon me.

Data for enhancement of the new API: https://github.com/SheetJS/js-xlsx/blob/master/tests/write.js#L14-L19


Refresh org will fail if one of the individual refreshes change

Describe the bug
If you have a GitHub organization where one of the individual workflows would fail refresh (ex. the workflow no longer exists on GitHub) then this will fail and stop the rest of the workflows from being refreshed. An individual workflow refresh failure should not fail a refresh org.
One issue with this is that we cannot delete these workflows that no longer have a github reference, so it will always fail and the user cannot fix this.
To Reproduce
Steps to reproduce the behavior:

Go to my workflows
Refresh an organization with a workflow where the corresponding GitHub repository no longer exists
Will fail

Expected behavior
Should pass, but maybe have some notification that some workflows could not be refreshed.
┆Issue is synchronized with this Jira Story
┆Issue Type: Story
┆Sprint: Backlog
┆Issue Number: DOCK-1403

Going to remove with #3538


RuntimeError: Expected to mark a variable ready only once

RuntimeError: Expected to mark a variable ready only once. This error is caused by one of the following reasons: 1) Use of a module parameter outside the forward function. Please make sure model parameters are not shared across multiple concurrent forward-backward passes2) Reused parameters in multiple reentrant backward passes. For example, if you use multiple checkpoint functions to wrap the same part of your model, it would result in the same set of parameters been used by different reentrant backward passes multiple times, and hence marking a variable ready multiple times. DDP does not support such use cases yet.
I replace torch.nn.Sequential(*layers) with SublinearSequential(*layers) and meet an error

Can you attach a minimal case that can reproduce this error?


Requesting to add support for picking multiple images

Please add support for selecting multiple images at once using the image_picker plugin.

duplicate of https://github.com/flutter/flutter/issues/16054

Closing as dup - thanks @d-silveira for spotting.


Abandon branch pruning if an arg name is redefined.

This makes it so that branch pruning for a given input arg is
abandoned if an input arg is redefined in the user code as its
value potentially becomes non-const (and no const-prop analysis
yet to figure it out).
Fixes #4163

flake8 error:

The problem of argument redefinition is solved in #4148 by using get_definition (the relevant tests work in that PR), which I believe is the right way. I suggest merging that PR first, and limiting this PR to the other portions.

Thanks @ehsantn, good point. As this is intended to land on the 0.44 series to fix regression(s), I'll pull those bits of code in to this so as to not add new features to a release series.


Decode layout.json with unicode characters

https://forums.flightsimulator.com/t/msfs-livery-tools/590459/21?u=poetalirico

It seems to be something with the Il-103 package, a character not decoded. I’m downloading it to discover what is happening. You can set your scan depth to 0 at the settings to continue working with your F/A-18 livery, but it won’t find files out of that specific package folder (they aren’t needed in most cases, though). It should be easy to solve, if it is what I think it is, but I can’t make a promise on the exact time.
Extracting textures is usually a fast process, and the progress bar should bounce to one side and the other. I’ll look into a way of restoring the GUI even if an unexpected error occurs.

Solved by 93857f9265ed5efe6f3b78937486d5b7e21c6b10


adds root path if enabled and needed

This adds an option to add the trailing slash for the root path, if needed.

Awesome!


Progress margin-bottom deleted

In 4.0.0-alpha.6
Progress works with margin-bottom: X (default 20px) in previos bootstrap versions.
This change make more difficult migrations between BS3 and BS4.
To make it easier I think we should persist this margin-bottom.

Leaving it without the margin given our utilities. Thanks though!


pagination since_id is ignored when max_id is set

For example, if I have the following statusIDs.
100, 99, 98 ... 1
If I make the following call:
/api/v1/timelines/public?since_id=74&max_id=78
I would expect the following results:  [77, 76, 75]. This is also what I get when I call the api directly.
However, what I currently see is : [77, 76, 75, 74, 73, 72, ... etc]
I don't understand why go-mastodon deliberately sets since_id to an empty string if a max_id is specified. I would like to be able to paginate through the results until I reach the since_id that I specified, but that's not currently possible because when I specify a max_id, then my since_id value is ignored...
Instead of https://github.com/mattn/go-mastodon/blob/master/mastodon.go#L311
func (p *Pagination) setValues(params url.Values) url.Values {
	if p.MaxID != "" {
		params.Set("max_id", string(p.MaxID))
	} else if p.SinceID != "" {
		params.Set("since_id", string(p.SinceID))
	}
	if p.Limit > 0 {
		params.Set("limit", fmt.Sprint(p.Limit))
	}

	return params
}

why not?:
	if p.MaxID != "" {
		params.Set("max_id", string(p.MaxID))
	}
if p.SinceID != "" {
		params.Set("since_id", string(p.SinceID))
	}

This issue is discussed at #86

@pacmessica I am sorry to be late.
I fixed it with #90 !
Please check it!

great, thanks!

closable?


IVP problem gives wrong solution?

I'm trying to solve an ODE on [0,T] with terminal value u(T) = -\alpha, of the form:
u' = -u^2/(2-\gamma^2*u)
The solution I get is wrong. I've added a comparison with Chebfun to show what I'm trying to do. The top plot is from chebfun, the lower one is from ApproxFun. Inspecting the ODE shows that the ApproxFun solution is clearly wrong (u is supposed to be decreasing when it is negative).

MATLAB code:
T = 2; alpha = 3; gamma = 0.1;

dom = [0 T];
N = chebop(@(t,u) diff(u)+u.^2./(2-gamma.^2.*u), dom);
rhs = 0;
N.rbc = @(u) u+alpha;

u = N\rhs;

Julia code:
T = 2.; α = 3; γ = 0.1

x = Fun(identity,[0.,T])
d = domain(x)
D = Derivative(d)
B = rdirichlet(d)
N = D+I^2./(2-γ^2*I)

A = [B; N]
b = -α

u = A\b

Hi Asbjørn,
since your problem is nonlinear in u, you will need to specify the operator as an anonymous function and use the newton solver with a reasonably accurate initial guess. The approach for nonlinear ODEs is outlined here for a BVP.

Okay. Is there a plan to implement time-steppers for nonlinear IVPs? Chebfun did this in 5.1, and they claim it works better than the global methods.
http://www.chebfun.org/docs/guide/guide10.html#102-initial-value-problems-and-solveivp

I could see the case for the following override:
u=ode45((t,u)->-u^2/(2-γ^2*u),Interval(0.,2.))

This provides very crude overrides to the methods from the ODE.jl package. A better solution would be to use the output from these overrides as initial guesses for Newton iteration. As can be seen at the end, the norm of the residual is nowhere near the set relative tolerance.
using ApproxFun, ODE, Plots

import ODE: ode23, ode45, ode78, ode4, ode4ms, ode23s, ode4s

for ode in (:ode23, :ode45, :ode78, :ode4, :ode4ms, :ode23s, :ode4s)
    @eval function $ode(fn, t0, y0, d::Space; reltol=eps(), kwargs...)
        t0added=false
        for logn = 4:20
            t = points(d, 2^logn)
            t0 ∉ t && (unshift!(t,t0); t0added=true)
            t, u = $ode(fn, y0, t; points=:specified, reltol=reltol, kwargs...)
            cf = t0added ? Fun(ApproxFun.transform(d,u[2:end]), d) : Fun(ApproxFun.transform(d,u), d)

            maxabsc=maxabs(cf.coefficients)
            if maxabsc==0 && maxabsfr==0
                return(zeros(d))
            end

            if length(cf) > 8 && maxabs(cf.coefficients[end-8:end]) < reltol
                return chop!(cf,reltol/10)
            end
        end
        warn("Maximum length "*string(2^20+1)*" reached")

        t = points(d, 2^21)
        t0 ∉ t && (unshift!(t,t0); t0added=true)
        t, u = $ode(fn, y0, t; points=:specified, kwargs...)

        t0added ? Fun(ApproxFun.transform(d,u[2:end]), d) : Fun(ApproxFun.transform(d,u), d)
    end
end

sp = Chebyshev(Interval(2,0))

u = ode45((t,u)->-u^2/(2-.1^2*u),2.0,-3.0,sp;reltol=eps(),abstol=realmin())

pyplot()

plot(u)

norm(u'+u^2/(2-.1^2*u))

Newton iteration (using the most recent commit) gives essentially machine precision. Code updated above.

That works well (and gives a higher precision than the Chebfun defaults),
thank you both.
I'll let you close the issue, in case you want to keep it open for
implementing ODE-overrides.
On Fri, 10 Jun 2016 at 11:25 Richard Mikael Slevinsky <
<EMAIL_ADDRESS>wrote:

Newton iteration (using the most recent commit) gives essentially machine
precision. Code updated above.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/ApproxFun/ApproxFun.jl/issues/335#issuecomment-225147931,
or mute the thread
https://github.com/notifications/unsubscribe/AFV6DTKNicoUv2Ckni0USRHq9E9yv2MBks5qKTuDgaJpZM4IyNNR
.

How about we put these into a new repository, ApproxFun/ApproxFunODE.jl, for the time being?  We can also consider wrapping:
https://github.com/luchr/ODEInterface.jl

I've created
https://github.com/ApproxFun/SpectralTimeStepping.jl
which right now contains just an Example.  At some point I'll move the BDF codes over there.


Add a --test-pool-regexp option

Takes a regular expression as argument and only runs against those hosts whose hostnames match. If combined with -T/--test (soon to be something different, maybe -T/--test-pool-size) will only run against a certain number of hosts before pausing to ask if it's OK to continue based upon the "test results"

How about --host-regexp instead

Implemented in development branch


"index_left" exception when running scripts on linux

this goes back to building_id - need to double check it's still working on mac


WYSIWYG editor removes HTML tags

Describe the bug
The WYSIWYG editor inconsistently removes html tags and otherwise damages the source content, regardless of what HTML sanitizer is being used.
To Reproduce
Steps to reproduce the behavior:

open the Using Custom HTML Sanitizer example in the documentation.
Insert some valid HTML code in the Markdown area.

Some code that will be destroyed

This code will be damaged by the WYSIWYG editor.

Switch from Markdown to WYSIWYG, then back. Observe that some of the content has been changed.

Some code that will be destroyed

This code will be damaged by the WYSIWYG editor.

Repeat steps 2 and 3 in the Basic Example that does not use a custom sanitizer. The same content is changed in the same way.

Expected behavior
Expected HTML code to be retained, and only changed by the sanitizer specified.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows
Browser Chrome
Version 86.0.4240.111

@breezystatic77 Paragraph is created by default in the Markdown editor. So you don't need to create a paragraph using the
 HTML string, and it's correct that the P tag disappears when converted from WYSIWYG to Markdown. However, the part where the line breaks before and after
 seems to be a bug and needs to be fixed.

Also, if you type something like this:
fdsfdsfdsa
DFSAfds
gfdhdgjhjhg
jhjhhfgjhjgf
JHGhjgjhg
and switch to markdown it will produce something like this:
fdsfdsfdsa
DFSAfds
gfdhdgjhjhg
jhjhhfgjhjgf

JHGhjgjhg


Skip validation on unattached blob.

A record can have an Active Storage attachment that (no longer) points to a blob. When we attempt to validate the content-type and size of the blob we get a delegation error.
The solution in this PR is to skip validation when the attachment is not attached.
This may solve #890 and other upload related spec suite failures.

Thank you!!! Really appreciated.


stranslate: Update to version <IP_ADDRESS>3

Closes #XXXX

Relates to #XXXX

[x] I have read the Contributing Guide.

Already updated automatically in d5069c6.


help: Add change-your-time-zone.

#8889
@rishig This is just the time zone doc, separated out into it's own pull request.

@rishig quick ping

I think likely we should just remove this feature. Tim, thoughts?

I basically disagree; I think instead we should just change the model for the setting.  I think it's not much work.  So @Lyla-Fischer I think you should ignore this for now and we can come back to it.

Because the source repository for this pull request was deleted, this pull request has been automatically closed by GitHub as an unintended side effect of renaming our default Git branch. If this change is still desired, please open a new pull request, referencing this one in the description.


stuck in understanding

Challenge Waypoint: Understand Boolean Values has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/46.0.2490.80 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:
function welcomeToBooleans() {

// Only change code below this line.

  return false;

// Only change code above this line.
}

welcomeToBooleans();

@kranthi123 please describe your issue in more detail.

@kranthi123 - I encourage you to stop by the help chat room and ask for assistance there.
You can find the chat room at https://gitter.im/FreeCodeCamp/Help

Closing this since there is no issue described. Please feel free to reopen with further details!
Thanks and happy coding!

i can help u got it right  but u forgt to change the return false; and rewrite it at the bottom of all of it with the {} brackets  and it should pass  so basically take and write it like this
function welcomeToBooleans() {
return false;;  <-- change that from false
}


CLI update issues

Bug Report or Feature Request (mark with an x)
- [x] bug report
- [ ] feature request

Updating CLI from 1.0.0 to >1.1.0 or above throws error and does not install the compiler-cli properly on windows 10
Versions.
@angular/cli: 1.1.1
node: 6.10.3
os: win32 x64
@angular/animations: 4.2.0
@angular/common: 4.2.0
@angular/compiler: 4.2.0
@angular/compiler-cli: 4.2.0
@angular/core: 4.2.0
@angular/forms: 4.2.0
@angular/http: 4.2.0
@angular/platform-browser: 4.2.0
@angular/platform-browser-dynamic: 4.2.0
@angular/router: 4.2.0
@angular/cli: 1.1.1
@angular/language-service: 4.2.0

Repro steps.
Update a project from angular cli 1.0.0 to latest version on windows:
rm -rf node_modules dist
npm install --save-dev @angular/cli@latest
npm install

The log given by the failure.
@angular/compiler-cli does not install properly, it needs to be reinstalled. (throws error: compiler-cli was not installed properly)
Additionally throw error on install and when installing new packages:
Error: Cannot find module 'code-point-at'
    at Function.Module._resolveFilename (module.js:469:15)
    at Function.Module._load (module.js:417:25)
    at Module.require (module.js:497:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (D:\ng\test-project-ng4\node_modules\wide-align\node_modules\string-width\index.js:3:19)
    at Module._compile (module.js:570:32)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)

@Ploppy3 no one else has reported this issue... can you retry please. I was just able to walk through this scenario successfully.  (I admit that I do not have a windows environment).

Unfortunately I don't have any project to update anymore, all 3 of them broke in the same way described above. Though, between projects' update I updated npm to the latest hotfix to check if it was related to npm, then I installed the latest version of node for the same reason but still the next projects broke.
The only thing I used in the project other than the tools you provide in the CLI was npm-check-updates. Could it be related in any way?
Talking about that. What's the proper way to update the dependencies of a cli project? Do you work on a tool to do this automatically?

After much more investigation it seems that npm 5 is the root of the problem. The introduction of the package-lock.json seems to create many of the issues I'm getting. I can't tell if it's only npm5 or the combination of the cli and npm as I'm only using npm for angular.
I reinstalled npm & node completely, both up to date to the latest version and the problem persist.

Try delete node_modules and package-lock.json, run npm install again.

I realized too that deleting the package-lock.json fixes the issue momentarily. Just until the next time you use npm.

@Brocco Is it possible that the 'code-point-at' package is a missing dependency ?
Adding it to the dev dependencies fixes some issues that I get.

I'm having a similar problem. Just globally reinstalled the latest Angular CLI and typescript. Started a fresh "ng new", and when I "ng serve" I get the angular cli compiler error.

As @Ploppy3 mentioned package-lock.json is a real pain. It will prevent anything from updating. You can change a package version in package.json but nmp i will ignore it until you delete the lock file. And it wasn't the issue with the older versions of npm.

Restarting my computer fixed the issue for some reason.

NPM 5 was release recently and seems to have many bugfixes related to the lock file.
I couldn't test yet, please try and tell me if it helps.

@brandonwilliams this is insane... it did work after restarting :(


BCE device CAN BUS

Hello. I use the traccar 3.10 software. And i was wondering if the last version of traccar suports the BCE Can bus protocol. Beacause we buyed some BCE Devices and we can't read the can data.
Thanks!

As far as I can see, only basic location data is supported at the moment.

Any chance to support all the data from the protocol?

Please send an email to discuss it. Address is in profile.


OpenVPN tries to use PAP authentication

When trying to integrate OpenVPN with RADIUS authentication we see the connection is using unencrypted PAP as the authentication method. Please update documentation with steps to enable encrypted OpenVPN and RADIUS authentication.

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

ID: 6298aba8-c953-c7c1-3630-421639631b70
Version Independent ID: b552f7ae-0c8f-2bfc-a47e-131807b7e613
Content: Integrate NPS with VPN Gateway RADIUS authentication for MFA
Content Source: articles/vpn-gateway/vpn-gateway-radius-mfa-nsp.md
Service: vpn-gateway
GitHub Login: @AhmadNYasin
Microsoft Alias: genli

@stuhad This is something simple that you set on the OpenVPN Client.
Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@TravisCragg-MSFT I'm downloading the ovpn config from the Azure Virtual Network Gateway P2S portal, if there is a way to alter the config please document it

@stuhad For the network policy on the RADIUS server for the Azure VPN, do you have PAP selected as an option under Authentication Methods of the Constraints Tab in the Properties?
Also what conditions do you have specified for the Policy on the RADIUS server ?

I'm not really sure why you are asking me this as we ** absolutely do not** want to us PAP, but yes if we set PAP as an Authentication Method then we see successful connections from the Azure Virtual Network Gateway (VNG) OpenVPN Peer to Site (P2S) VPN.
If we set any other Authentication Method such as MS-CHAP-v2 then the NPS server rejects the authentication attempt because the Azure VNG OpenVPN P2S VPN only seems to try connecting with an Unencrypted authentication mechanism.

@cocallaw did you need any further information?

I spoke with the engineering team, and they confirmed that with a P2S VPN configured with RADIUS and OpenVPN, currently PAP is only authentication method supported between the GW and RADIUS server.

#please-close

@cocallaw We are in the exact same situation at the moment, using RADIUS and OpenVPN. How is it acceptable to only support PAP? Is there any other way to use MSCHAPv2?


Purpose of endDate field

What is the purpose of the endDate field?
When using a custom rule and specifying an end date, it is included in the rrule using the UNTIL parameter. However, it appears that this is not happening when the endDate field is used.
In my view, when a custom rule is employed, the endDate should be disregarded and replaced by the rule's set options. However, if no custom rule is used, the value from the endDate field should be utilized as the UNTIL parameter for the rrule.
This approach would also prevent conflicts between the custom rule options and the endDate. For instance:

The endDate occurring before the end date set in the custom rule.
Occurrences happening after the endDate when using the "After X occurrences" option.
The endDate having a value while the custom rule is configured to never end.

The endDate is intended as the end of the event in question and doesn't concern the UNTIL of the rrule.
For example, an event might start on 2023-01-10 at 13:00 and finish an hour later at 14:00. The end date would determine that this event lasts for an hour, and doesn't impact the generated rrule. You could use this end date on your front-end to display an event as necessary.
If an endDate is not required, it can be disabled in the plugin config with hideEndDate: true.
Forgive me if I've misinterpreted the question here - if there is a bug in how this is working please feel free to reopen this with details.

First, thank you for your prompt response.
I apologize for the misunderstanding regarding the purpose of the endDate field. Thank you for providing clarification!
To summarize, does this mean that the only way to specify an end date for a period is by utilizing a custom rule?

Yes - you'd need to use a custom rule to define that. Without that, the Rrule would assume it's an indefinite recurrence.


Release Google.Cloud.Redis.V1 version 2.3.0

Changes in this release:

Commit ac367e2: feat: Regenerate all APIs to support self-signed JWTs

Created release for Google.Cloud.Redis.V1-2.3.0


Scripts not recognizing DevOps User and Key

When running docker compose for the 11-docker-compose/02-replicated-pair example in getting started, I noticed that the script does not detect that I have these PING_IDENTITY_DEVOPS_USER and PING_IDENTITY_DEVOPS_KEY set.  I have run the proper configuration commands and they are in my .bash_profile and are also shown in my env listing.
To Reproduce
Run docker compose in the directory 11-docker-compose/02-replicated-pair (locally).
Expected behavior
Should have detected PING_IDENTITY_DEVOPS_USER and PING_IDENTITY_DEVOPS_KEY values.  Not sure why this is not seeing the values from my environment.
Screenshots
pingdirectory_1    | ##################################################################################
pingdirectory_1    | #    DevOps User/Key
pingdirectory_1    | ##################################################################################
pingdirectory_1    |          PING_IDENTITY_DEVOPS_USER : --- empty ---
pingdirectory_1    |           PING_IDENTITY_DEVOPS_KEY : --- empty ---
Additional Info
When I looked at the code I see that you are doing a && test of both of these, meaning that both need to be missing before that section of the code will kick in, but you may want to have that as a || check so if either of these values are not set, it will use the temporary ones.
Line 23 of pingcommon/hooks/17-check-license.sh

Just adding this output from env:
PING_IDENTITY_DEVOPS_KEY=3e***-masked--aaa
PING_IDENTITY_DEVOPS_TAG=:edge
<EMAIL_ADDRESS>

I was able to resolve this by putting the variables for the DEVOPS items above into the docker-compose.yaml file so they are passed into the docker environment.  Probably be good to add that note to the documentation.


cargo build/run: Infinite Blocking for rand Dependency

Hello,
I'm just getting started with Rust, and unfortunately I appear to have already run into an issue that I can't seem to find support for.  While doing the "Guessing Game" tutorial in the Rust documentation, I ran into a snag while trying to include the "rand" crate in my Cargo.toml file:
[dependencies]

rand = "0.3.0"

When I try to execute cargo build or cargo run, I get the following message, which doesn't seem to go away no matter how long I wait:
    Blocking waiting for file lock on the registry index
I've tried starting a new project and just including the dependency for the default "Hello, world!" program, and I run into the same issue.  Here are my cargo and rustc versions:
cargo 0.15.0-nightly (298a012 2016-12-20)
rustc 1.14.0 (e8a012324 2016-12-16)
Does anyone know what could be causing this?  Sorry to be introducing myself to the community this way, but I wasn't sure what else to do about the issue.  Thank you!
(P.S.: Everything seems to run fine without the dependency, as long as I don't have any references to it in my Rust code.)

Thanks for the report, and definitely sounds like something is awry! You wouldn't happen to be working in an IDE would you? Also, what platform are you on?

Thanks for listening!  I have a MacBook Pro with OS X El Capitan (Version 10.11.6). I've been using IntelliJ as an IDE, but I encountered the same issue when creating a project from the command line and just creating a project using cargo new [projectname] --bin.

Hm so what this error should mean is that there's a concurrent Cargo somewhere else which is actively using the build directory. If that's not the case then there's definitely a bug in Cargo! We've had bugs with NFS mounted home dirs and such in the past, but those should be handled now.
Are you sure that when you see this message there are 0 other instances of Cargo running on the system?

(sometimes IDEs run cargo in the background, but it should work if you execute it locally w/o the IDE)

It starts working again if I just take out the dependency.  I just now confirmed that again.  Then when I put the dependency back, it starts blocking again.

Here's my entire project folder, if that helps.
guessing_game_2.zip

Yeah without the dependency Cargo doesn't need to update the registry index so it won't need to attempt to acquire this lock.
cc @matklad, do you know if IntelliJ uses cargo as a library or something like that and may hold a lock for a long time?

I've just tried quitting IntelliJ.  Still blocking.

@russodd you're sure there's no other Cargo running in the background?

@alexcrichton How would I check that?

if IntelliJ uses cargo as a library or something like that and may hold a lock for a long time?

Nope, we call cargo metadata only, and we actually try to make sure that we don't have concurrent invocations of cargo.

@russodd I normally run ps aux | grep cargo

@matklad ok cool, thanks for the clarification!

@alexcrichton Okay, so there does seem to be some stuff happening, but I don't know what started it:
drusso            2582   0.0  0.1  2485636  10748   ??  S     2:09PM   0:00.06 /Users/drusso/.rustup/toolchains/stable-x86_64-apple-darwin/bin/cargo metadata --verbose --format-version 1 --all-features
drusso            2581   0.0  0.0  2461780   7540   ??  S     2:09PM   0:00.04 /Users/drusso/.cargo/bin/cargo metadata --verbose --format-version 1 --all-features
drusso            1754   0.0  0.0  2493944   7672   ??  S    10:35AM   0:00.05 /Users/drusso/.rustup/toolchains/stable-x86_64-apple-darwin/bin/cargo metadata --verbose --format-version 1 --all-features
drusso            1753   0.0  0.0  2461780   4548   ??  S    10:35AM   0:00.03 /Users/drusso/.cargo/bin/cargo metadata --verbose --format-version 1 --all-features
drusso            2667   0.0  0.0  2434840    768 s003  S+    2:17PM   0:00.00 grep cargo

Yep, that's definitely Cargos spawned by IntelliJ :)

And yet they're running even when IntelliJ is closed...  I guess nothing kills them.

@russodd if you kill them (kill $pid where $pid is the second column of numbers) then a build typically proceeds as normal, right?

Maybe we should see the stack trace before we kill them?

@alexcrichton Yes, killing them seems to work.  Sorry, @matklad, I didn't see your post before I did it.

@russodd ah ok in that case, want to just keep an eye and see if this happens again? I think this isn't necessarily a bug in Cargo so I'm going to close, but feel free to ping here/on IRC if you need help debugging this if it happens again!

@alexcrichton Okay.  Thank you for your help!

I think this isn't necessarily a bug in Cargo

Hm, are you sure? Under what circumstances can Cargo deadlock (because this looks like a deadlock similar to the one we had with git dependencies before). We try to avoid concurrent Cargos in IntelliJ Rust, but this is only a best effort: there may be several instances of the IDE running, or you can invoke cargo on the command line.

Heh in theory Cargo deadlocks under no circumstances! Put another way I don't know of any existing issues of Cargo deadlocking that would cause this.
Let's keep our eyes peeled to see if this happens again and see if stack traces can illuminate what's happening.

@alexcrichton I'll take a look at this now. Looks like I see a potential deadlock, but need to stick more printfs to be sure!

Ok, so I was unable to get a dedlock experimentally or theoretically. However I've managed to get a frozen Cargo another way!
If during an index update I turn off the wifi, the Cargo gets stuck on the select with an infinite timeout inside libgit2. If I enable wifi later it does not unstuck. I would expect that dropping an internet connection should result in an error after some timeout.

Select:
❯ strace -p 11408                                                                                ~/projects/cargo
strace: Process 11408 attached
select(5, [4], [], [4], NULL

Stacktrace
#0  0x00007fe29fc7a9a3 in __select_nocancel () from /home/matklad/.nix-profile/lib/libc.so.6
#1  0x00007fe2a12f51f0 in wait_for () from /nix/store/kwk80fmkrhqardxgv3psd441a6awfg91-libgit2-0.24.3/lib/libgit2.so.24
#2  0x00007fe2a12f54b4 in curls_read () from /nix/store/kwk80fmkrhqardxgv3psd441a6awfg91-libgit2-0.24.3/lib/libgit2.so.24
#3  0x00007fe2a1333071 in bio_read () from /nix/store/kwk80fmkrhqardxgv3psd441a6awfg91-libgit2-0.24.3/lib/libgit2.so.24
#4  0x00007fe2a088131c in BIO_read () from /nix/store/qa72yjcxzn48s558rpjp2c82hqjg03gp-openssl-1.0.2j/lib/libcrypto.so.1.0.0
#5  0x00007fe2a0bdbb9a in ssl3_read_n () from /nix/store/qa72yjcxzn48s558rpjp2c82hqjg03gp-openssl-1.0.2j/lib/libssl.so.1.0.0
#6  0x00007fe2a0bdd4fd in ssl3_read_bytes () from /nix/store/qa72yjcxzn48s558rpjp2c82hqjg03gp-openssl-1.0.2j/lib/libssl.so.1.0.0
#7  0x00007fe2a0bd9f54 in ssl3_read_internal () from /nix/store/qa72yjcxzn48s558rpjp2c82hqjg03gp-openssl-1.0.2j/lib/libssl.so.1.0.0
#8  0x00007fe2a133336d in openssl_read () from /nix/store/kwk80fmkrhqardxgv3psd441a6awfg91-libgit2-0.24.3/lib/libgit2.so.24
#9  0x00007fe2a132abfc in recv_stream () from /nix/store/kwk80fmkrhqardxgv3psd441a6awfg91-libgit2-0.24.3/lib/libgit2.so.24
#10 0x00007fe2a136cbf4 in http_stream_read () from /nix/store/kwk80fmkrhqardxgv3psd441a6awfg91-libgit2-0.24.3/lib/libgit2.so.24
#11 0x00007fe2a136e301 in git_smart.recv_cb () from /nix/store/kwk80fmkrhqardxgv3psd441a6awfg91-libgit2-0.24.3/lib/libgit2.so.24
#12 0x00007fe2a136fcf8 in recv_pkt () from /nix/store/kwk80fmkrhqardxgv3psd441a6awfg91-libgit2-0.24.3/lib/libgit2.so.24
#13 0x00007fe2a1370d51 in git_smart.download_pack () from /nix/store/kwk80fmkrhqardxgv3psd441a6awfg91-libgit2-0.24.3/lib/libgit2.so.24
#14 0x00007fe2a134bbee in git_remote_download () from /nix/store/kwk80fmkrhqardxgv3psd441a6awfg91-libgit2-0.24.3/lib/libgit2.so.24
#15 0x00007fe2a134c98e in git_remote_fetch () from /nix/store/kwk80fmkrhqardxgv3psd441a6awfg91-libgit2-0.24.3/lib/libgit2.so.24
#16 0x000055af88acecdd in git2::remote::Remote::fetch (self=0x7ffd3b140048, refspecs=..., opts=..., reflog_msg=...)
    at /home/matklad/.cargo/registry/src/github.com-1ecc6299db9ec823/git2-0.6.3/src/remote.rs:171
#17 0x000055af88657545 in cargo::sources::git::utils::fetch::{{closure}}::{{closure}} () at /home/matklad/projects/cargo/src/cargo/sources/git/utils.rs:596
#18 0x000055af886a60f6 in cargo::util::network::with_retry<(),git2::error::Error,closure> (config=0x7ffd3b14f770, callback=...)
    at /home/matklad/projects/cargo/src/cargo/util/network.rs:22
#19 0x000055af886578a3 in cargo::sources::git::utils::fetch::{{closure}} (f=...) at /home/matklad/projects/cargo/src/cargo/sources/git/utils.rs:595
#20 0x000055af88655bc9 in cargo::sources::git::utils::with_authentication<(),closure> (url=..., cfg=0x7ffd3b140728, f=...)
    at /home/matklad/projects/cargo/src/cargo/sources/git/utils.rs:416
#21 0x000055af88657346 in cargo::sources::git::utils::fetch (repo=0x7ffd3b140e90, url=..., refspec=..., config=0x7ffd3b14f770)
    at /home/matklad/projects/cargo/src/cargo/sources/git/utils.rs:585

Ah, and we can clearly say that the original issue is not a deadlock, because it involves only a single cargo: there are 4 entries in the ps output, half of which are rustup wrapper scripts. So, only two Cargos are involved. But, if we look at the time, we see that one of them has been working (most likely stuck) for hours!
@russodd have you turned the WiFi / suspend your computer or something similar?

That'd definitely explain it though! Stuck during a registry update would certainly prevent all future Cargos from proceeding.
@matklad how you ran across that is beyond me.

Here's the select: https://github.com/libgit2/libgit2/blob/1db3035d74a8f734d65b4e779d3e4b22cfd90ebb/src/curl_stream.c#L224

I've tried to investigate further what's going on but I still fail to undestand why this select does not return. Looks like the connection is actually open all this time... at lest, this is what /proc/net/tcp says:
  sl  local_address rem_address   st tx_queue rx_queue tr tm->when retrnsmt   uid  timeout inode
 277: 6400A8C0:AA5C 71FD1EC0:01BB 01 00000000:00000000 00:00000000 00000000  1000        0 88872 1<PHONE_NUMBER>000000 36 4 0 10 -1

@alexcrichton I am at the end of my unix-fu, so I think it's your turn to create an issue at libgit/cargo, and I really need some sleep now :)

Hm, I also wonder why exactly do we need to hold a file lock during fetch operation? As always, it's better to minimize critical sections. Can we fetch the data from the network first, and lock the repository only for actually updating the tips?


[#76] Display actual Identity info on backup component

👍 👍  LGTM, some minor liner fixes need to be done for CI to pass.
You can run ./script/linter before committing, to make sure there are no liner issues.
I'll be happy to merge it once linter is fixed.

I will run the lint.sh file and make the appropriate changes.


Customers walk using gdx-ai

Currently implemented:
The map contains spawnpoint locations for chefs, customers, lights. It also contains all possible locations a customer would go to including the 4 submit stations and queueing spots.
The spawnpoint location for the customer is used to spawn them. The chef spawn locations are not used as of yet.
The customer looks for an available objective, trying to go for the stations first, otherwise they queue up. No customers will spawn with a full queue.
Customers as of yet do not leave when given order. This is because our orders should be tied to their respective submit stations which is not currently implemented.

Customers don't move from their first objective, meaning they stay in the same place after fulfilling an order. Maybe call a function on submit station that makes all the customers move one objective along.

I acknowledge that the tests fail, however that is because the behaviour of customermanager was altered. It will be further altered in a later PR, and therefore this will not be fixed atm.


Tracking issue for nivo and react 17

tracking https://github.com/plouc/nivo/issues/667
this prevents using react 17 on node 16.x
Ideally, nivo completes the migration to react-spring, but I'm assuming we will need to rip out nivo in favor of another graphing library.

👏🏻 👏🏻


Volumes are not deleted when an a node is deleted.

Whenever a node is deleted from Jenkins the OpenStack plugin should terminate the instance and also expected it to delete its attached volume.
We noticed this does not happen as the attached volumes are not deleted. This causes the cluttering of the OpenStack project quota.
Is there any option to do this? I could not find a way to do this via the configuration page (Manage Jenkins).

Hi, the plugin does not anything to attach any volumes. Can you elaborate on your use case? What kind of volumes? How do you attache them?

We use the plugin to create OpenStack instances from an image (image ID used for slave template). Each of the instances gets its own volume attached (cloned from the image).
When we delete a slave, the plugin terminates the instance but the volume that was attached to that instance is not deleted and remains unattached.
This causes the OpenStack project's Volume Storage to be filled with unallocated volumes which have to be manually deleted.
When i manually create an instance from image - the "Boot from image (creates a new volume)" option - in the Horizon dashboard, i see there's a checkbox for the "Delete on Terminate" option. If i enable this checkbox, the volume will be deleted automatically when the instance is terminated.
Can we achieve the same behaviour via the OpenStack plugin?

@remzix, When I provision via plugin it never creates a volume. What you describe surely seems like "Boot from image (creates a new volume)" behavior instead of plain old "Boot from image" I rely on. Is that desirable for you - to create a new volume for every slave?

@olivergondza thanks for the info. we noticed that we did not use the delete_on_termination:true option in our image. it now works.
closing this as not a plugin issue.

Thanks, had the same issue.

@remzi-e Where did you change this setting? I cant find it anywhere in image options/metadata

@remzi-e @DEvil0000 How can this be configured? I tried adding delete_on_termination:true to the metadata of my image, but the save button is disabled in Horizon.

@DEvil0000 @remzi-e @joinghel @tstrul1 Did anyone of you solve the issue, if so, how? Setting this option in image metadata does not seem to fix the problem.

@glothriel in our image metadata:

we added or adapted (I don't remember): block_device_mapping:
[{"guest_format": null, "boot_index": 0, "delete_on_termination": true, "no_device": null, "snapshot_id": "cca35882-d038-4798-abdc-b6b70c17af2c", "device_name": "/dev/vda", "disk_bus": "virtio", "image_id": null, "source_type": "snapshot", "tag": null, "device_type": "disk", "volume_id": null, "destination_type": "volume", "volume_size": 60}]
The important part is: "delete_on_termination": true.


Adding OSM Poland Telegram group

I have not run npm run test as I don't have Node installed - thanks for doing this.

Thanks @lukaszkruk 👍


Transition Duration Config V9 🐛

🐛 Bug Report
Noticing some differences in how configs / durations work from v8 -> v9. (Some maybe intended, however if you could point me in right direction would be super appreciated).
You can see the differences clearest between the two codesandboxes included below.
To Reproduce
Here's a codesandbox w/ the latest beta. Some things i've noticed:

The timeout property no-longer changes render duration (doesnt seem to do anything at all?)
Messages no longer animate out together (if multiple were tapped in succession).

https://codesandbox.io/s/messagehub-31iqh
Expected behavior
Here's how the demo worked on v8 (notice that the timeout is having impact on config - this is my ideal / expected behavior)
https://codesandbox.io/embed/7mqy09jyq
Environment

react-spring v9.0.0
react v16.8.x (or react-native v0.58.x)

Not sure what's wrong with v9.0.0-beta.34 on the old example, but I have a newer example using the latest canary right here: https://codesandbox.io/s/issue-948-puurn
That example comes from the pull-808 branch, which applies the following changes:
diff --git a/demos/hooks/notification-hub/index.js b/demos/hooks/notification-hub/index.js
index 50b78ab..87f390d 100644
--- a/demos/hooks/notification-hub/index.js
+++ b/demos/hooks/notification-hub/index.js
@@ -4,6 +4,8 @@ import { X } from 'react-feather'
 import { useTransition } from 'react-spring'
 import { Main, Container, Message, Button, Content, Life } from './styles.js'

+const DEBUG = false
+
 let id = 0

 function MessageHub({
@@ -15,26 +17,29 @@ function MessageHub({
   const [cancelMap] = useState(() => new WeakMap())
   const [items, setItems] = useState([])

-  const transitions = useTransition(items, item => item.key, {
+  const transition = useTransition(items, {
+    key: item => item.key,
     from: { opacity: 0, height: 0, life: '100%' },
-    enter: item => async next =>
-      await next({ opacity: 1, height: refMap.get(item).offsetHeight }),
-    leave: item => async (next, cancel) => {
-      console.log('  one')
-      cancelMap.set(item, cancel)
-      await next({ life: '0%' })
-      console.log('  two')
-      await next({ opacity: 0 })
-      console.log('  three')
-      await next({ height: 0 })
-      console.log('  done!')
+    enter: item => async (next, stop) => {
+      if (DEBUG) console.log(`  Entering:`, item.key)
+      cancelMap.set(item, () => {
+        if (DEBUG) console.log(`  Cancelled:`, item.key)
+        stop()
+        setItems(state => state.filter(i => i.key !== item.key))
+      })
+      await next({
+        opacity: 1,
+        height: refMap.get(item).offsetHeight,
+        config,
+      })
+      await next({ life: '0%', config: { duration: timeout } })
+      cancelMap.get(item)()
     },
-    onRest: item => {
-      console.log('onRest', item)
-      setItems(state => state.filter(i => i.key !== item.key))
+    leave: item => async next => {
+      if (DEBUG) console.log(`  Leaving:`, item.key)
+      await next({ opacity: 0, config })
+      await next({ height: 0, config })
     },
-    config: (item, state) =>
-      state === 'leave' ? [{ duration: timeout }, config, config] : config,
   })

   useEffect(
@@ -45,15 +50,18 @@ function MessageHub({

   return (
     <Container>
-      {transitions.map(({ key, item, props: { life, ...style } }) => (
-        <Message key={key} style={style}>
+      {transition(({ life, ...style }, item) => (
+        <Message style={style}>
           <Content ref={ref => ref && refMap.set(item, ref)}>
             <Life style={{ right: life }} />
             <p>{item.msg}</p>
             <Button
               onClick={e => {
                 e.stopPropagation()
-                cancelMap.has(item) && cancelMap.get(item)()
+                if (cancelMap.has(item)) {
+                  if (DEBUG) console.log(`  Cancelled item:`, item)
+                  cancelMap.get(item)()
+                }
               }}>
               <X size={18} />
             </Button>

ah ok thanks!

Another thing I noticed, is in the sandbox you linked, the elements aren't removed from the dom (but they were in v8).

That will be fixed in the next canary 👍

rad, thanks!

HI all, @aleclarson I tried your example and is working, the only problem is with version 9.0.0-rc3,
If you try you can see that the items close by cross button are not working properly, do you know why this happens?


Add some kind of automated functional testing

What would you like Loftsman to do?
It would be nice to add some functional testing alongside the unit tests run by actions. We could do this using KIND.
Why is this needed?
for qa purposes, having some real tests against a real cluster without requiring we do so manually is needed. Testing actual installs, upgrades, etc.

Initial functional testing capability will go out with 1.1.0, but not really an official path, e.g. not wired up to actions, etc. Let's continue to track here

Initial review on the functional testing script so we can address in this issue:
https://github.com/Cray-HPE/loftsman/pull/31#discussion_r621300291
https://github.com/Cray-HPE/loftsman/pull/31#discussion_r621303862
https://github.com/Cray-HPE/loftsman/pull/31#discussion_r621311029


Design hibernate regions

This RFC proposes to hibernate a peer when it reaches a stable state to reduce CPU waste.
The proposal here may need to be formalized and verified using TLA+. @nolouch would you like to help me with it?

@BusyJay OK, I will try to.

PTAL

LGTM.

PTAL


Traffic Accident Prediction

Description
The Traffic Accident Prediction project aims to develop a machine learning model that predicts the likelihood of traffic accidents based on historical data, such as accident records, weather conditions, traffic volume, and road characteristics. By analyzing these factors, the model can provide insights into high-risk areas and conditions, helping local governments and traffic authorities implement targeted safety measures and improve traffic management strategies.
Problem it Solves
Traffic accidents are a significant public safety concern, leading to injuries, fatalities, and economic losses. Traditional methods of analyzing traffic incidents often lack predictive capabilities, making it challenging for authorities to proactively address potential risks. This project seeks to solve this problem by providing a data-driven approach to identify and predict high-risk scenarios, allowing for better allocation of resources and implementation of preventive measures.
Proposed Solution
The proposed solution involves:

Data Collection: Gathering historical accident data, weather conditions, traffic patterns, and road information from reliable sources.
Data Preprocessing: Cleaning and preparing the data for analysis, handling missing values, and encoding categorical variables.
Model Development: Using Scikit-learn to develop a predictive model (e.g., Random Forest, Logistic Regression) to estimate the likelihood of accidents based on the input features.
Explainability: Integrating the ExplainableAI package to provide insights into the model's predictions, highlighting the key factors contributing to accident likelihood.
Deployment: Creating a user-friendly interface for local governments to input current conditions and receive risk assessments.

Alternatives Considered

Rule-based Systems: While a rule-based system can provide immediate safety guidelines based on fixed conditions, it lacks the adaptability and predictive power of a machine learning approach.
Traditional Statistical Models: Although traditional regression models can analyze historical data, they may not capture complex patterns as effectively as modern machine learning techniques.

Additional Context
The project can benefit from collaboration with local traffic departments for access to comprehensive datasets. Visualizations can be incorporated to present insights from the model in an understandable format, enabling stakeholders to make informed decisions. Additionally, regular updates and model retraining can ensure the system remains accurate and relevant as traffic patterns evolve.
ADD LABELS GGSOC EXT 24 AND Hacktoberfest
ASSIGN ME THIS PROJECT

yeah create this project, have use of explainableai package.
it can help in - for model training, llm explanations for predictions, etc.
have all ur codebase in /examples dir, eg. /examples/your_project_name

@ombhojane pull request has been created. Kindly review it. Thank you!

@ombhojane pull request has been created. Kindly review it. Thank you!

@ombhojane


$dirty not being set on certain directives

http://plnkr.co/edit/n642ZGLz8VNgT5RjNO2L?p=preview
When changing the uif-textfield, the $dirty property is not set to true.
It IS set to true for the form, but not for the ng-model property bound to the uif-textfield.
Apparently this also occurs at uif-dropdown.
We need to verify this for all components.

The best solution seems to have the name property on the input element. Right now, this attribute is not copied to the input element. We could do this manually, but that's harder to maintain.
We could use https://github.com/angular/angular.js/blob/master/src/ng/compile.js#L2747 to copy all attributes from the directive to the input element.

Instead of a single issue, can we please create seperate issues for each directive? Found using one issue to track all directives was a PITA and a mess in the commit references... also hard to track if someone was already working on one directive as you couldn't tell who was working on one vs. another.

Instead of a single issue, can we please create seperate issues for each directive? Found using one issue to track all directives was a PITA and a mess in the commit references... also hard to track if someone was already working on one directive as you couldn't tell who was working on one vs. another.

I completely agree, but it does not apply to each directive. I don't want to create an issue for each directive, only if it requires an update.
I've created two issues for uif-textfield and uif-dropdown.

Closing this one as we have seperate issues for each directive that's affected.


Make repo and mount optional in app specs

@paetling Ran into a case when creating example specs where I wanted to run a Python fileserver, which is a one-liner requiring no code. https://github.com/gamechanger/example-dusty-specs/blob/master/apps/fileserver.yml

clean out the default settings in test/unit/utils.py   Should not be setting repo or mount for anybody now.  Other than that, :ghost: :cow:


fix gateway print log panic

Ⅰ. Describe what this PR does

Codecov Report

Merging #167 (dfbfa1f) into master (2986258) will not change coverage.
The diff coverage is 0.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##           master     #167   +/-   ##
=======================================
  Coverage   43.34%   43.34%
=======================================
  Files          50       50
  Lines        5373     5373
=======================================
  Hits         2329     2329
  Misses       2647     2647
  Partials      397      397

Flag
Coverage Δ

unittests
43.34% <0.00%> (ø)

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

Files Changed
Coverage Δ

pkg/trafficrouting/network/gateway/gateway.go
54.26% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

/lgtm

/approve

/approve


Fix dependency resolution

Fixes Issue #740

Upgrades maven-dependency-tree (this PR supersedes #1165)
When analyzing an aggregate build (in Maven) if a child module has not been built and as such, the JAR file does not exist, a warning will be printed and a dependency will be added that contains the evidence from the build - but does not point back to a specific file.
Fixed bug that would cause pom.xml dependencies without an identifier to be incorrectly merged with any other JAR.

Have test cases been added to cover the new functionality?
yes

Can one of the admins verify this patch?


Gain Sub-Element Not Included in All AudioBlockFormats

In the standard. The gain sub-element of AudioBlockFormat is listed as common to all implementations of AudioBlockFormat. As far as I can see here, it is only implemented in the AudioBlockFormatObjects class.

Hi,
So far this library only implements features from BS.2076-1, which only includes a gain parameter in the Objects type. We're planning on updating this soon.


fix(lidar_centerpoint): fix constexpr related bugs

Description
These are fixes based on clang-tidy CRITICAL warnings ( = clang compile error)
For example,
constexpr variable 'detection_yaw_sin' must be initialized by a constant expression

Related links
Parent Issue:

Link

How was this PR tested?
Notes for reviewers
None.
Interface changes
None.

Effects on system behavior
None.

@knzo25
Agreed 😮💨
They would be constexpr from C++26


multi: export channel status field in migration25

Fixes #6558.
Previously, in migration25.OpenChannel, there was a private field chanStatus used to keep track of the channel status. The following migrations, migration26 and migration27 also have their own OpenChannel defined, with migration26 inherited from migration25, and migration27 inherited from migration26. The private field chanStatus, however, is NOT inherited and each of the migrations uses its own. This is fine for reading and writing as, under the hood, the chanStatus is just a uint8 value. Because each migration has its own fetcher and putter, it can safely access its private field to read and write it correctly.
The issue pops up when we use the method migration25.FundingTxPresent(). Because it's evaluating its channel status using its own private field chanStatus, this field would always be the default value(ChanStatusDefault), leading the statement !c.hasChanStatus(ChanStatusRestored) to always be true. Thus a restored channel will be mistakenly considered to have funding tx present, causing failures in reading the channel info in the following migrations.
We fix this by exporting the ChanStatus field so its value can be set by following migrations.
Affected migrations
Migration 26 and 27 will fail at decoding the funding tx step if we are the channel funder and the channel is restored. If they succeed, it means either,

we are not the channel funder, or,
the channel is funded externally, or,
the channel is not restored

And the data has been migrated correctly. Otherwise, the migrations will halt and lnd will quit with an error.

Got one confirmation from @ziggie1984 that this pr fixed the issue. Thanks again for reporting the issue and sharing the testnet channel db file!


Update openstack-ansible SHA to 11.2.14

This commit updates the openstack-ansible SHA to the latest tag.
Paramiko recently released version 2.0.0, this is installed by
bootstrap-ansible.sh and causes the script to fail due to missing
dependencies. This is addressed in openstack-ansible kilo but is not
included in the latest tag. deploy.sh has been modified to install the
missing packages. This is required until the next openstack-ansible tag
is released.
Issue: https://github.com/rcbops/rpc-openstack/issues/1037

Initial 👍  pending results of gate

:+1: an issue should be created to track the removal of that section targeted for a couple of weeks from now.

@tomc603 @prometheanfire I removed the SHA bump from this patch because the OSA tag doesn't include the required commit so that can be handled separately by https://github.com/rcbops/rpc-openstack/pull/1045

that works for me, even better, once 11.2.15 is out we can just git revert this

👍


Bump vitest and vitest c8 coverage

Attempt to fix https://github.com/danielroe/nuxt-vitest/issues/171
Does this look like it should solve our issues?

Would be great to use Vitest 0.31 with this. Hope this gets merged soon.

I'm not sure why the tests failed honestly, so @danielroe might have something to add. You can still use 0.31, just manually add vitest-environment-nuxt as well into your project pnpm i -D vitest-environment-nuxt should work, I'm using it and have vitest 0.31 installed.

Thanks. Did had to force it though.

The issue is that vitest 0.31 breaks the component mocking feature of this library and we're waiting for that to be fixed upstream.

@danielroe is that still the case? I see https://github.com/vitest-dev/vitest/pull/3438 was merged. Wondering if it's safe to unpin the vitest package here.

I appreciate the PR - but let's track in https://github.com/danielroe/nuxt-vitest/pull/157. It requires more work to get things working with the latest version of Vitest....


Request:Enabling Web Access for Documentation Search in "gpt-engineer"

I'm wondering if it's possible to enable the "gpt-engineer" to access the web for searching documentations.

I'm wondering if it's possible to enable the "gpt-engineer" to access the web for searching documentations.

Definitely, supports for embeddings, pdfs, and web search is a must, else it would be very hard to get something working with updated frameworks apis.

Doesn't the latest gpt model support functions now?
It should be possible in theory to implement web access.

Supporting paid-only features sounds like a poor choice to me.

Doesn't the latest gpt model support functions now?
It should be possible in theory to implement web access.

Supporting paid-only features sounds like a poor choice to me.

I don't think it is a paid feature.

Hi!
Yes, almost anything is possible these days, the latest feature provided by openai (not a paid feature AFAIK) are functions.
That would potentially allow for us/you to create a "search google" function.
See #58 for a first approach.


[Create New Issue]: DSA : Height Balanced Tree

Description
I will be explaining the concept of height-balanced trees and their implementation using C++
Domain
DSA
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign


Best approaches to automatically plot IEnumerables (Lists, Queues, etc.)

I remember some time back, I have found some examples here how to automatically plot my data to a FormsPlot control, but I cannot find these examples anymore in the latest Cookbook, might be I just did not try searching hard enough! :)
So practically I am asking for examples from simple to advanced cases when we want to "auto plot" data from a List, Queue, or other collections. I often collect data from hardware in such "buffer" collection, and it would be great to see approaches how to "bind" such collection to a FormsPlot. So as the collection grows/changes, what are the different ways/tricks to re-plot? Most often the plotting frequency is less than 1 Hz, so in my case speed is not a factor.
Thanks for advises and any links where I can see such examples!

Hey @bukkideme, the cookbook demonstrates features that can be implemented in console applications. User-control-specific examples can  be found in the demo program.
Currently the user controls do not bind data objects - they just take in double[] and display it whenever formsPlot1.Render() is called. That said, you may find these two examples in the demo application useful:
Live Data
This demo uses a fixed-size array and frequently updates its values and re-plots

Growing Data
This demo shows how to plot growing data

Hope it helps!


Changes usage of _ASSERT for SUCCESS error checking

Description
Adds new macro for _ASSERT when it is used specifically for checks against SUCCESS. The new macro is called _ASSERT_SUCCESS and compares two inputs against a success return code (0).
Additionally, this PR adds a macro for when _ASSERT  is used with conditional .and. through the use of _ASSERTS (which can be renamed to another thing if preferred).
My original intention with this PR was to create a macro for conditional .and. and conditional .or. but particularly with .or. there are multiple instances with varying number of arguments. I tried experimenting with having optional arguments but was unsuccessful. Because of this, the implementation of the _ASSERTS one is limited in usage to only two arguments so perhaps it might be better to just leave the _ASSERT_SUCCESS in this PR and remove the other macro?
Related Issue

Motivation and Context

Usage of a specific _ASSERT case can be converted to a macro.
How Has This Been Tested?

Tested in intel by changing the error code from good to bad to ensure macro worked as expected. When bad, error message was provided as expected. When good, result was zero diff.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Trivial change (affects only documentation or cleanup)

Checklist:

[x] I have tested this change with a run of GEOSgcm (if non-trivial)
[x] I have added one of the required labels (0 diff, 0 diff trivial, 0 diff structural, non 0-diff)
[x] I have updated the CHANGELOG.md accordingly following the style of Keep a Changelog

I believe the latest push addresses all the suggestions.

I believe the latest push addresses all the suggestions.

@Gvilla1000-nasa I think we'd also discussed making the non-userRC .and. asserts into two separate asserts? Though I suppose that could be a separate PR.

I believe the latest push addresses all the suggestions.

@Gvilla1000-nasa I think we'd also discussed making the non-userRC .and. asserts into two separate asserts? Though I suppose that could be a separate PR.

Working on it right now. Found a missing .and. with error checking as well


SL-13 Add param to turn off DEXSeq.gff output save

Hi,
There is currently no param that disables saving the output DEXSeq.gff. This PR adds a param called skip_dexseq_gff_save to the nextflow.config, and included this param in the modules.config to skip saving DEXSeq.gff if this param is specified.
PR checklist

[x] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs- [ ] If necessary, also make a PR on the nf-core/rnasplice branch on the nf-core/test-datasets repository.
[ ] Make sure your code lints (nf-core lint).
[x] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

Thanks, I have now changed the config files to specify using enabled as suggested.

Thanks, have now fixed the typo causing this.


Show brotli filesize

Would be nice to show filesize when compressed with brotli, like rollup-plugin-filesize can do: https://github.com/ritz078/rollup-plugin-filesize#showbrotlisize

@aomarks .. I've added support for brotli. Check version 1.2.x

Confirmed, thank you!


Add lint configurations

Stack:

#8
#7
#5
#4 ⬅
#3

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.


Fix a data race exposed by race checker

When adding the -race flag to go test along with -cover, a couple of
data races were exposed by the tool. This fixes those data races, and
introduces 'make cover' as a target in the Makefile.
The wg.Wait() was being executed before anything as added to the
WaitGroup. Therefore, it was possible that the channel might close
before any work was done. This makes sure that the WaitGroup has
something to wait on before calling Wait.
Signed-off-by: Brad P. Crochet<EMAIL_ADDRESS>

Not sure if we care to commit the coverage.out, but if we don't, add it to the gitignore.

Not sure if we care to commit the coverage.out, but if we don't, add it to the gitignore.

It's already in the ignore file here


IMDS Credentials Provider

Describe the feature
https://docs.aws.amazon.com/sdkref/latest/guide/feature-imds-credentials.html
~All SDKs seem to support this credential provider type and we should too.
One thing to note, CRT already supports this and has a swift binding, so perhaps all we need to do is expose that.
Use Case
N/A
Proposed Solution
No response
Other Information
No response
Acknowledgements

[ ] I may be able to implement this feature request
[ ] This feature might incur a breaking change

Duplicate of #1094


Bug in PR 82

I posted about an error/bug under the actual PR#82, might have been the wrong place.
But the LABS are no longer working :(
I posted my bug report here:
https://github.com/MicrosoftLearning/eShopOnWeb/pull/82
It this part in Program.cs that breaks running the example, why do we need SQL, and KeyVault?
else{
    // Configure SQL Server (prod)
    var credential = new ChainedTokenCredential(new AzureDeveloperCliCredential(), new DefaultAzureCredential());
    builder.Configuration.AddAzureKeyVault(new Uri(builder.Configuration["AZURE_KEY_VAULT_ENDPOINT"] ?? ""), credential);
    builder.Services.AddDbContext<CatalogContext>(c =>
    {
        var connectionString = builder.Configuration[builder.Configuration["AZURE_SQL_CATALOG_CONNECTION_STRING_KEY"] ?? ""];
        c.UseSqlServer(connectionString, sqlOptions => sqlOptions.EnableRetryOnFailure());
    });
    builder.Services.AddDbContext<AppIdentityDbContext>(options =>
    {
        var connectionString = builder.Configuration[builder.Configuration["AZURE_SQL_IDENTITY_CONNECTION_STRING_KEY"] ?? ""];
        options.UseSqlServer(connectionString, sqlOptions => sqlOptions.EnableRetryOnFailure());
    });
}

Please fix, I need the labs for next time teaching AZ-400 as an MCT

Had closed it by mistake

@theDiverDK can you please check if https://github.com/MicrosoftLearning/AZ400-DesigningandImplementingMicrosoftDevOpsSolutions/issues/552 resolves your issue? Thanks!

Hi @LuizMacedo
Yes, adding
UseOnlyInMemoryDatabase = true
ASPNETCORE_ENVIRONMENT = Development
Did get the website to work.
But it is not mentioned in the guide for the lab i was trying to do:
https://microsoftlearning.github.io/AZ400-DesigningandImplementingMicrosoftDevOpsSolutions/Instructions/Labs/AZ400_M09_L16_Monitoring_Application_Performance_with_Application_Insights.html

Hi @theDiverDK, thanks for your feedback. I own the guide and courses related to DevOps, including AZ-400. The parameters were added in the application (eShopOnWeb) so there is no need to mention them in the lab steps. All hosted lab instructions will be updated soon to reflect it. The issue was that I updated the application, and usually, it takes time to reflect the update in the lab hoster environment. Please, feel free to open new issues or PRs if you see something else we can do to improve our course. Your feedback is appreciated, thanks!

Hi @LuizMacedo
I am a bit confused (most likely due to english is not my main language)
What is the difference between 'lab steps' and the 'hosted labs'?
Any ideas when the labs like https://microsoftlearning.github.io/AZ400-DesigningandImplementingMicrosoftDevOpsSolutions/Instructions/Labs/AZ400_M09_L16_Monitoring_Application_Performance_with_Application_Insights.html will be updated to be reflected in the guide?
In about 3 weeks time i'll be teaching the AZ-400 course, and would like the labs to work without having to inform the students about needed fixes.
For the actual course, i'll be using GoDeploy as a lab provider.
Thanks for your time :)

Hi @theDiverDK no worries, it's not my first language too =), my fault not explaining that well!
Lab steps are what you see in GitHub with the step-by-step, for example, the one you shared https://microsoftlearning.github.io/AZ400-DesigningandImplementingMicrosoftDevOpsSolutions/Instructions/Labs/AZ400_M09_L16_Monitoring_Application_Performance_with_Application_Insights.html.
When I say hosted labs, I'm referring to the labs offered by the lab provider (in your case, GoDeploy). These lab providers rely on our GitHub labs for their instructions. Whenever we make updates to the labs or approve PRs, it may affect the providers who will need to reimport or update their environment and steps. Next time, I will make sure to inform the lab providers better. For major changes like the upcoming AZ-400 in mid-April, we usually inform with 2 or 3 months before the update, but in this case, I missed to inform about the approval of a minor PR. It was my mistake, and I apologize for that.
You don't need to inform the students about the needed fix. We updated the eShopOnWeb application with all you need to just follow the labs as it is today. If for any reason they have issues, make sure about the variables, but should be smooth.
Let me know if you have questions. Thanks.

Hi @LuizMacedo
Thanks for the clarification, I actually had understood you correctly :)
I'll try the lab tomorrow again.
Thanks :-)

Hi @LuizMacedo
I have just tested the lab guide:
https://microsoftlearning.github.io/AZ400-DesigningandImplementingMicrosoftDevOpsSolutions/Instructions/Labs/AZ400_M09_L16_Monitoring_Application_Performance_with_Application_Insights.html
And I still get the same error, but adding the 2 configuration strings fixed the issue as before.
Didn't you say:
We updated the eShopOnWeb application with all you need to just follow the labs as it is today
I deleted the setup in Azure and Devops, and followed the guide and i got the 500.03 error again :(
By the way when I clone the repo from https://github.com/MicrosoftLearning/eShopOnWeb.git
The branch called 'dependabot/nuget/Microsoft.AspNetCore.Authentication.JwtBearer-8.0.2' is selected, not Main. Is that on purpose?

@theDiverDK it was added here #88. You always should use the main branch. That branch was created by DependaBot.

Hi @LuizMacedo
Thanks for the update, i will test it tomorrow.
I wonder why Devops selects a non ‘Main’ branch?

@theDiverDK the other branches will be merged as soon as I test the new version bump suggested by Dependabot and the main will be the only available. That's a good question. I don't know why GitHub shows that as default. I'll find out, but main is the default.

Hi @LuizMacedo
This has not been fixed in #88, which updated 2 bicep files, but in Module 9 Lab 16 no bicep files are used to deploy the web app.
I have create a Pull Request: https://github.com/MicrosoftLearning/AZ400-DesigningandImplementingMicrosoftDevOpsSolutions/pull/558
Where I have tried to add the needed steps to add a AzureAppServiceSettings@1 task to the deploy pipeline, that add's the needed app setting for the web app to actually work.
Hope my proposal can be used.
p.s. Sorry if my english is not correct, I am from Denmark ;-)


isValid function

Is there a function I can call which will return true or false if the form is valid or not?

This seems part of the HTML 5 form validation, check MDN for this


Steps chosen / choice penalty

Hi,
thanks for the implementation!
I'm still unfamiliar with it, would it be possible to extract or store the steps (or choice) taken during the optimization as it is doable with normal dwt implementations?
Or, equivalently, adding penalty terms for the different choices?

Hi @cfrancesco
Thanks for your interest in my code.
With sDTW, I don't think it's possible to get the "best path" using the cost matrix. The reason for that is the sDTW algorithm actually returns a value that corresponds to all possible alignment between the two inputs, weighted by their probability (see the comment from the original authors here: https://github.com/mblondel/soft-dtw/issues/10#issuecomment-383377409).
This question is probably best posted in the original sDTW repo.

Ok, thank you! Then I'll investigate how to add penalties rather than inspecting the optimal path.
I'll post any update, should there be any.

No problem! Sure please do.
Also, please consider starring this repository if you find it helpful :)


Add test to see if HTML shim is needed

Should be as simple as 'hidden' in document.createElement('a') (that is how the HTML5Shiv is doing it)


One any of object

entity of type "object" that includes "oneOf" " or "anyOf" but no "properties" falls through the cracks.
This PR fixes this.
Traceback (most recent call last):
  File "jsf/src/jsf/schema_types/object.py", line 40, in generate
    return super().generate(context)
  File "jsf/src/jsf/schema_types/base.py", line 49, in generate
    raise ProviderNotSetException()
jsf.schema_types.base.ProviderNotSetException

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "jsf/src/jsf/schema_types/object.py", line 40, in generate
    return super().generate(context)
  File "jsf/src/jsf/schema_types/base.py", line 49, in generate
    raise ProviderNotSetException()
jsf.schema_types.base.ProviderNotSetException

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "jsf/src/jsf/schema_types/object.py", line 40, in generate
    return super().generate(context)
  File "jsf/src/jsf/schema_types/base.py", line 49, in generate
    raise ProviderNotSetException()
jsf.schema_types.base.ProviderNotSetException

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "mockdatageneration/prototype/test.py", line 19, in <module>
    data = faker.generate()
  File "jsf/src/jsf/parser.py", line 143, in generate
    return self.root.generate(context=self.context)
  File "jsf/src/jsf/schema_types/object.py", line 42, in generate
    return {o.name: o.generate(context) for o in self.properties if self.should_keep(o.name)}
  File "jsf/src/jsf/schema_types/object.py", line 42, in <dictcomp>
    return {o.name: o.generate(context) for o in self.properties if self.should_keep(o.name)}
  File "jsf/src/jsf/schema_types/object.py", line 42, in generate
    return {o.name: o.generate(context) for o in self.properties if self.should_keep(o.name)}
  File "jsf/src/jsf/schema_types/object.py", line 42, in <dictcomp>
    return {o.name: o.generate(context) for o in self.properties if self.should_keep(o.name)}
  File "jsf/src/jsf/schema_types/object.py", line 42, in generate
    return {o.name: o.generate(context) for o in self.properties if self.should_keep(o.name)}
TypeError: 'NoneType' object is not iterable

Looks fine to me. Thanks.


Why use ObservableOnSubscribe instead of extends Observable itself

ObservableOnSubscribe is used in Observable.create() method and will do that cancelation check for your inside of it. So it's redundant to use ObservableOnSubscribe IMO. I propose to directly use Observable as super class and it can reduce allocations compare to Observable.create() : ObservableCreate + ObservableOnSubscribe +  Emitter -> CustomObservable.  But it's sure a big change for this code base. Do you interested in this approach ? Or you prefer using ObservableOnSubscribe in some particular reason?

Good point.
The reason why I used ObservableOnSubscribe was - I just followed same approach as (legacy) RxBinding used.
I strongly agree with your opinion - I'll tag this issue as enhancement, please feel free to send a PR if you interested.

I think it is better way in rxjava 2 for creating a hot source like firebase ChildEventListener
   @NonNull
   public static Flowable<RxFirebaseChildEvent<DataSnapshot>> observeChildEvent(@NonNull
       final Query query) {

    return Flowable.create(emitter -> {

        final ChildEventListener childEventListener = new ChildEventListener() {

            @Override
            public void onChildAdded(DataSnapshot dataSnapshot, String previousChildName) {
                if(!emitter.isCancelled())
                emitter.onNext(
                        new RxFirebaseChildEvent<>(dataSnapshot.getKey(), dataSnapshot, previousChildName,
                                RxFirebaseChildEvent.EventType.ADDED));
            }

            @Override
            public void onChildChanged(DataSnapshot dataSnapshot, String previousChildName) {
                if(!emitter.isCancelled())
                emitter.onNext(
                        new RxFirebaseChildEvent<>(dataSnapshot.getKey(), dataSnapshot, previousChildName,
                                RxFirebaseChildEvent.EventType.CHANGED));
            }

            @Override
            public void onChildRemoved(DataSnapshot dataSnapshot) {
                if(!emitter.isCancelled())
                emitter.onNext(new RxFirebaseChildEvent<>(dataSnapshot.getKey(), dataSnapshot,
                        RxFirebaseChildEvent.EventType.REMOVED));
            }

            @Override
            public void onChildMoved(DataSnapshot dataSnapshot, String previousChildName) {
                if(!emitter.isCancelled())
                emitter.onNext(
                        new RxFirebaseChildEvent<>(dataSnapshot.getKey(), dataSnapshot, previousChildName,
                                RxFirebaseChildEvent.EventType.MOVED));
            }

            @Override
            public void onCancelled(DatabaseError error) {
                if(!emitter.isCancelled())
                emitter.onError(new RxFirebaseDataException(error));

            }
        };

        emitter.setCancellable(() -> query.removeEventListener(childEventListener));

        query.addChildEventListener(childEventListener);

    }, BackpressureStrategy.BUFFER);
}


Remove jQuery Migrate by updating code that requires it

https://github.com/jquery/jquery-migrate

Will fix with Girder.


Handle nil run list option in knife bootstrap

IMO it's a bug in knife-ec2 but it worked before and there could be lots of plugins relying on the behavior so it needs to just work.
Fixes https://github.com/chef/chef/issues/4131
@btm

:+1:

:+1:
Fixing in knife-ec2 in https://github.com/chef/knife-ec2/pull/374.


OSSM-1107 Add jwksResolverExtraRootCA feature

This PR is related to OSSM-1107 and Istio PR 17176 https://github.com/istio/istio/pull/17176
Add a new SMCP spec field as
.spec.security.certificateAuthority.jwksResolverExtraRootCA

Signed-off-by: Yuanlin Xu<EMAIL_ADDRESS>

Is there any suggestion about a new integration test ? which component(s) need to be covered when setting that field ?

/retest

/retest

Why did you merge it now? Did you test it?

I wak checking tide and it was pending for a label. I will avoid adding that ok-to-merge label next time.


Search for wordcount formatters with extension

Autoload expects the autoload scripts to have the '.vim' extension.
Fixes #1896.

thanks!


n8n pod failing to configure - no encryption key

According to values.yaml,
n8n:
  encryption_key: # n8n creates a random encryption key automatically on the first launch and saves it in the ~/.n8n folder. That key is used to encrypt the credentials before they get saved to the database.

I'm using helm on microk8s. With default values.yaml file,
$ microk8s helm -f values.yaml install n8n open-8gears/n8n --debug
results in pod with CreateContainerConfigError, specifically
  Type     Reason             Age              From               Message
  ----     ------             ----             ----               -------
  Warning  Failed             3s (x3 over 4s)  kubelet            Error: secret "n8n" not found

No .n8n folder is created.
I managed to bypass the issue by typing encryption key into values.yaml manually.

Looks like #34 solves the issue.
@Vad1mo I would appreciate if you could look into the request and merge it if possible. I'm happy to provide any help needed.


Device Tunnel - NRPT Table

Is it possible to use the nrpt DNS table with device tunnel vpn profile. I can configure it, but it doesn't work. can you give me an example or an advice.

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

ID: e7453068-95eb-2a42-fd0b-553ae593adb0
Version Independent ID: b39b2cb7-25b1-2af3-4954-dbcc04c836e9
Content: Configure Windows 10 Client Always On VPN Connections
Content Source: WindowsServerDocs/remote/remote-access/vpn/always-on-vpn/deploy/vpn-deploy-client-vpn-connections.md
Service: unspecified
GitHub Login: @shortpatti
Microsoft Alias: pashort

@donky-bison, can you provide more details as to what you are doing?  What is the high-level scenario?

We have an onprem Skype for Business (SFB) deployment. The SFB client has to connect over the Internet to the SFB Edge Server and not over the VPN Tunnel to get connected to the sfb pool. To reach this Goal, we have to disable the dns Resolution for the internal SFB Servers.
We have now implemented the dns role on the vpn Server, which forwards the sfB Client query for the internal sfb servers to nowhere.  The VPN Clients uses the VPN Servers as DN Server.
If we could control the dns behavior for certain FQDN with NRPT, this would be nice. We could then configure nrpt to resolve the dns Names for our internal SFB Servers by a public DNS.

@donky-bison, I apologize for the delay, I have the engineering team working on providing more details.

Hi @donky-bison, I believe you can use NRPT dns table.  Let me verify with a couple of engineers.

@MihaiSP, can you take a look at this issue?


feature request: changing window focus with hints

Before chunkwm, I used hammerspoon. The only feature I really miss is their window hints [1]. The two use cases I have are:

I can switch directly to the window I want, compared to a series of left/right/up/down moves.
I can switch focus to floating windows hidden under tiled windows.

There's nothing stopping me from using hammerspoon in conjunction with chunkwm. It just seems a little inelegant to run both is all.
[1] http://www.hammerspoon.org/docs/hs.hints.html

Not likely to happen in chunkwm. At least not by me.


Investigate whether it's possible to integrate vello renderer

Currently plugin uses CPU for rendering text. I wish to bring cosmic-text to 3d world so it should be performant. Instead of doing CPU text rendering we could try to investigate whether it's possible to use GPU rendering, vello renderer may be an option to do so. It seems vello integrates well with bevy: https://github.com/linebender/vello/blob/main/examples/with_bevy/src/main.rs
General idea how GPU text rendering works:

for every font, and every font size for that font, e.g. "Arial 16", we create a "sprite sheet" (actually FontAtlas, which holds a handle to a TextureAtlas, which holds handles to one or more Image)
for every NEW UNIQUE "character" (actually a subpixel-positioned glyph), we use cosmic-text to render that "character" to the "sprite sheet", and we keep track of the "sprite" (rectangle coordinates within the "sprite sheet"), with a mapping between "character" and "sprite".
cosmic-text also calculates the layout of the text as normal, so we have access to the positions of each "character", relative to the whole text box's position
so we effectively send to the GPU, the sprite data, and the position on the render surface (e.g. the window), just like we would with a sprite from a sprite sheet.

Let's investigate the ways to improve performance once (if) we face this issue.


Initial nprlib with viewdirection node

This changelist introduces an NPR (non-photorealistic rendering) data library to MaterialX, initially consisting of a single viewdirection node.
The NPR data library is designed for the portable expression of artistic, non-physically-based materials, with common examples being cartoon shading, architectural diagrams, and edge highlighting.  In future versions of MaterialX, we expect this data library to be extended with additional primitives and artist-facing graphs as they are requested by the graphics community and approved by the MaterialX TSC.
For shader generation, MaterialX initially supports the NPR data library in GLSL, ESSL, MSL, and OSL, with MDL being omitted for now, based on intentional restrictions in its design philosophy.

This changelist includes a handful of very simple NPR examples, including Edge Brighten, Environment Map, and Starfield, in order to test implementations and evaluate parity across languages.
Artists are encouraged to create their own examples, and we'd be happy to include them in the future!

Very excited to see this work towards nprlib. Thank you!!


Unexpected operation performed during replay after rejecting an external function

import { Bot, session } from "grammy";
import { conversations, createConversation } from "@grammyjs/conversations";

const bot = new Bot();

bot.use(session({ initial: () => ({}) }));
bot.use(conversations());
bot.use(
	createConversation(async (conversation, ctx) => {
		console.debug("Received /start");
		await ctx.reply(`Entered conversation`);
		await conversation.wait();
		await conversation.external({
			task: () => Promise.reject("fail"),
		});
	}, "start")
);

bot.command("start", async (ctx) => await ctx.conversation.enter("start"));

bot.catch((err) => console.error(err?.error || err));
bot.start();

Expected log when sending '/start' many times:
Received /start
fail
Received /start
fail

but got this:

Received /start
Received /start
fail
Received /start
Error: Unexpected operation performed during replay (expected 'wait' but was 'ext')! It looks like the conversation builder function is non-deterministic, or it relies on external data sources.
    at ConversationHandle._replayOp (/home/chaotik/git/trontrack/node_modules/.pnpm/@grammyjs+conversations@0.6.1_grammy@1.8.3/node_modules/@grammyjs/conversations/out/conversation.js:301:19)
    at ConversationHandle.external (/home/chaotik/git/trontrack/node_modules/.pnpm/@grammyjs+conversations@0.6.1_grammy@1.8.3/node_modules/@grammyjs/conversations/out/conversation.js:427:41)
    at file:///home/chaotik/git/trontrack/mre.js:13:22
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async run (/home/chaotik/git/trontrack/node_modules/.pnpm/@grammyjs+conversations@0.6.1_grammy@1.8.3/node_modules/@grammyjs/conversations/out/conversation.js:126:13)
    at async runUntilComplete (/home/chaotik/git/trontrack/node_modules/.pnpm/@grammyjs+conversations@0.6.1_grammy@1.8.3/node_modules/@grammyjs/conversations/out/conversation.js:175:26)
    at async /home/chaotik/git/trontrack/node_modules/.pnpm/@grammyjs+conversations@0.6.1_grammy@1.8.3/node_modules/@grammyjs/conversations/out/conversation.js:225:26
    at async /home/chaotik/git/trontrack/node_modules/.pnpm/grammy@1.8.3/node_modules/grammy/out/composer.js:61:13
    at async /home/chaotik/git/trontrack/node_modules/.pnpm/@grammyjs+conversations@0.6.1_grammy@1.8.3/node_modules/@grammyjs/conversations/out/conversation.js:142:9
    at async /home/chaotik/git/trontrack/node_modules/.pnpm/grammy@1.8.3/node_modules/grammy/out/composer.js:61:13

Received /start
Error: Unexpected operation performed during replay (expected 'wait' but was 'ext')! It looks like the conversation builder function is non-deterministic, or it relies on external data sources.

Node.js v17.7.1

I have a vague idea what could be the issue, I'll look into it soon:tm:

@x066it can you review #8?

Now after first fail it starts failing immidiately after each invoking /start, not waiting for user reply in the middle of conversation.
I don't think it should be like this.


gst-msdk/av1e: do not use matroska mux/demux

Avoid matroskamux/demux since it writes timestamps
to header, and will be different in each r2r iteration.
Signed-off-by: U. Artie Eoff<EMAIL_ADDRESS>

@yefengxx please test this on AV1e capable platform to ensure the new gstreamer generated command line works as expected.

@yefengxx please test this on AV1e capable platform to ensure the new gstreamer generated command line works as expected.

Hi @yefengxx any progress on test results for this change?

Close.  Superseded by #367


Slash Command Permissions

closes #44

[x] API endpoints
[x] Types
[x] Set the command ID inside command classes while syncing
[x] Permissions prop within commands
[x] Permissions syncing per guild
[x] Tests

Syncing mockup
creator
    .syncCommands({
      syncPermissions: true // true by default
    });

// or

creator
    // Syncs command permissions, reading the ID of stored commands and
    // bulk updating them per guild
    .syncCommandPermissions();

Command Mockup
module.exports = class HelloCommand extends SlashCommand {
  constructor(creator) {
    super(creator, {
      name: 'hello',
      description: 'Says hello to you.'
      defaultPermission: false,
      permissions: {
        '<guild_id>': [
          {
            type: ApplicationCommandPermissionType.USER,
            id: '<user_id>',
            permission: true
          }
        ]
      }
    });
    this.filePath = __filename;
  }
}

<3 I was literally about to get started adding this feature since I needed it for my project and realised you did it. Thank you :) Loving what you built!


Can't import css files in build

Using version 2.0.0
While the dev server is working perfectly fine, as soon as I try to build my deck which has components that use an external stylesheet the build fails. the output I get is:
rm -rf dist && mdx-deck build index.mdx

[mdx-deck] building

✔ [mdx-deck]
  Compiled successfully in 3.73s

 DONE  Compiled successfully in 3730ms

webpack:///./node_modules/style-loader/lib/addStyles.js?:23
        return window && document && document.all && !window.atob;
        ^

ReferenceError: window is not defined

While it does produce a dist folder, the contents of it does not work to display the presentation.
Digging into this I found #178 and noticed that the webpack config used by mdx-deck actually does include a style loader. And again, it works perfectly fine for the dev server, just breaks for building.
At a loss as to why this wouldn't work for builds.

Should be fixed in 2.0.4, but if you still run into issues feel free to reopen this


Wuma Menu props not loading up?

Im trying to restrict the props but NONE of them show up on the Wuma restrictions tab, theres a lot of props in my server so it would take forever to go to esch one is the spawnmenu and copy to clipboard and fix for this? or any other method at all would be great thanks.

This is intended behavior. If you need to restrict ALL props then the best way is just setting the prop limit to 0. Otherwise you will have to create a lua script to do this more effectively.


fix: group membership and enable changing project namespace

This addresses a few things:

Enable changing the namespace of a project
Fix a problem when removing a user from a group removed it from all other groups
Prevent removal of the owner from the group if there is no other owner

/deploy renku-ui=ciyer/3023-namespaces

You can access the deployment of this PR at https://renku-ci-ds-165.dev.renku.ch

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

25 of 29 (86.21%)  changed or added relevant lines in 3 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.2%) to 89.173%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

components/renku_data_services/project/db.py
7
8
87.5%

components/renku_data_services/namespace/db.py
12
15
80.0%

Files with Coverage Reduction
New Missed Lines
%

components/renku_data_services/crc/models.py
1
85.71%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
5683

Relevant Lines:
6373

💛  - Coveralls

Tearing down the temporary RenkuLab deplyoment for this PR.


Initial/anonymize name passed to ui-avatars

See https://github.com/laravel/jetstream/pull/940 for similar change and reasoning.

Thanks!