Assignment 2

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
Adding and removing codes on certain files to get the desired output
What did you learn from the changes you have made?
How to try different codes and get the desired results, and how to resolve certain errors
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
The approach I was initially making to open the all_paths file was wrong and giving me errors, so I changed the path
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
The main file was creating problems which I resolved with the help of Moniz and Pedram
How were these changes tested?
By running the cells and executing the desired results
A reference to a related issue in your repository (if applicable)
Checklist

[x] I can confirm that my changes are working as intended

For future reference - review comments are in original PR (https://github.com/saleha-12/python/pull/2) which was closed.

Hi Anjali,
Thank you for your valuable feedback and guidance. I will certainly keep the points you have mentioned in mind moving forward.
Regards,
Saleha.
Sent from Outlook for Androidhttps://aka.ms/AAb9ysg

From: anjali-deshpande-hub @.>
Sent: Thursday, December 12, 2024 12:50:22 AM
To: saleha-12/python @.>
Cc: saleha-12 @.>; Author @.>
Subject: Re: [saleha-12/python] Assignment 2 (PR #3)
@anjali-deshpande-hub approved this pull request.
I see that you have made it consistent to 2 spaces (except the precoded check_zeros which has 4).
The the official Python style guidelines (PEP 8) recommends indent with 4 spaces.
https://peps.python.org/pep-0008/
I have approved this PR. For future, please use 4 spaces and remember to not close the PR. The review changes will reflect in the same PR.
—
Reply to this email directly, view it on GitHubhttps://github.com/saleha-12/python/pull/3#pullrequestreview-2497683110, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BM5KNIEFX7YXFQGGLHLFGG32FEPZ5AVCNFSM6AAAAABTOQ752SVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDIOJXGY4DGMJRGA.
You are receiving this because you authored the thread.Message ID: @.***>


check enabled value when creating vuln scanner config map

The vulnscan-cronjob-template ConfigMap is a part of vuln scanner resources, but it doesn't check .Values.armoVulnScanner.enabled and .Values.armoKubescape.submit.

Hi @johnmanjiro13 , Thank you for opening the PR.
We are making some major changes in the code (mainly remaining), so to avoid conflicts I'm not merging this PR.

I got it. Thank you for your notification.


Add --hash for CLI and --typescript for wasm-bindgen

Implement feature for  #136 Disable hash for release builds, and default is hash closed.  We can make hash enabled by default if necessary.
Add '--typescript' option for wasm-bindgen and copy d.ts file to dest, and d.ts file is friendly to TypeScript and debug.

Not that I'm a qualified reviewer or maintainer or anything, but: Would it be possible to also apply the --hash flag to the SCSS output? That would complete the "predictable output filenames" story; currently I inline my CSS to work around that.

Thank you for opening this PR. I'll close this in favor of #354 and #357, though. Especially the latter allows configuring the TS file generation, instead of unconditionally creating it.


Disable armeria headers

Description
Disables Armeria server header is HTTP responses.
Issues Resolved
N/A
Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

:exclamation: No coverage uploaded for pull request base (1.2@69031c8). Click here to learn what that means.
The diff coverage is n/a.

@@          Coverage Diff           @@
##             1.2     #663   +/-   ##
======================================
  Coverage       ?   91.56%
  Complexity     ?      601
======================================
  Files          ?       75
  Lines          ?     1826
  Branches       ?      152
======================================
  Hits           ?     1672
  Misses         ?      116
  Partials       ?       38

Continue to review full report at Codecov.

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


Resolve Rails 5 deprecation warnings for *_filter hooks.

In Rails 4.0.0 (2012) DHH renamed the *_filter hooks in
AbstractController to *_action. In Rails 5.0 it now prints
deprecation warnings for *_filter calls as they will be removed
in Rails 5.1.

Any chance to merge this and release? Now that Rails 5 is out since a while. Thanks!


haxm is not installing : The system requirements are not satisfied.

I am trying to install haxm on Android Studio as it is required to run the virtual device emulator.
However, Haxm is not installing. This is the installation log file.
Version: 7.6.5
Output folder: C:\Program Files\Intel\HAXM
Extract: checktool.exe... 100%
Execute: C:\Program Files\Intel\HAXM\checktool.exe --verbose
CPU vendor          *  GenuineIntel
Intel64 supported   *  Yes
VMX supported       *  Yes
VMX enabled         -  No
EPT supported       *  Yes
NX supported        *  Yes
NX enabled          *  Yes
Hyper-V disabled    *  Yes
OS version          *  Windows 10.0.17134
OS architecture     *  x86_64
Guest unoccupied    *  Yes. 0 guest(s)
The system requirements are not satisfied.
<--I appreciate your help-->

Thanks for your feedback. According to your log, could you follow the wiki page to enable VMX and then try again?

Hello again, I have entered the bios and enabled the vmx, now I have new problems in the log file: :/
Version: 7.6.5
Output folder: C:\Program Files\Intel\HAXM
Extract: checktool.exe... 100%
Execute: C:\Program Files\Intel\HAXM\checktool.exe --verbose
CPU vendor          *  GenuineIntel
Intel64 supported   *  Yes
VMX supported       -  No
VMX enabled         -  No
EPT supported       -  No
NX supported        *  Yes
NX enabled          *  Yes
Hyper-V disabled    -  No
OS version          *  Windows 10.0.17134
OS architecture     *  x86_64
Guest unoccupied    *  Yes. 0 guest(s)
The system requirements are not satisfied.

Thanks for your feedback.


UI: Add password validation to Register New Account page

Affected module
UI
Describe the bug

[ ] There is no password validation in the UI on the Register New Account page. It allows me to create an account with a password less than 8 characters.

[ ] The form should not be cleared until a success response from the API.

are we doing validation around password == confirm Password?

Yes, that validation is present.


Constants Inconsistency in New Physics Moist

Recently, as a test, I tried to build the feature/sdrabenh/merge_v10.23.0_MSTRF branch that @sdrabenh is using to track the new Physics with GNU instead of Intel and it threw out a number of issues.
One is a simple Fortran standard violation here:
https://github.com/GEOS-ESM/GEOSgcm_GridComp/blob/d4b64898ef763c52204960981c074f98a3391a74/GEOSagcm_GridComp/GEOSphysics_GridComp/GEOSmoist_GridComp/ConvPar_GF2020.F90#L11525
which should be:
       if(versionx==1 .or. COUPL_MPHYSICS .eqv. .false.) then

But the more exciting issues come after that.
First, we have:
/discover/swdev/mathomp4/Models/GEOSgcm-ScottBranch-GNU/GEOSgcm/src/Components/@GEOSgcm_GridComp/GEOSagcm_GridComp/GEOSphysics_GridComp/GEOSmoist_GridComp/Process_Library.F90:32:26:

   13 |   use Aer_Actv_Single_Moment
      |      2
......
   32 |   real, parameter :: R_AIR   =  3.47e-3   ! m3 Pa kg-1K-1
      |                          1
Error: Symbol 'r_air' at (1) conflicts with symbol from module 'aer_actv_single_moment', use-associated at (2)

The issue here is that both aer_actv_single_moment.F90 and Process_Library.F90 provide R_AIR. You can "solve" this by removing R_AIR from aer_actv_single_moment.F90 but there is still the fact that R_AIR is defined a few times in moist:
cloudnew.F90
351:   real, parameter :: R_AIR = 3.47e-3 !m3 Pa kg-1K-1

Process_Library.F90
32:  real, parameter :: R_AIR   =  3.47e-3   ! m3 Pa kg-1K-1

GEOS_MGB2_2M_InterfaceMod.F90
617:      real, parameter :: r_air = 3.47d-3 !m3 Pa kg-1K-1

cldmacro.F90
68:   real, parameter :: R_AIR = 3.47e-3 !m3 Pa kg-1K-1

aer_actv_single_moment.F90
18:       real        , parameter :: R_AIR     =  3.47e-3 !m3 Pa kg-1K-1

The "good" news is that they are all the same value (well, the MGB2 one is REAL64) but they are all the same. Still, probably best to be defined in one place. NOTE: I'm not sure if this is a constant in MAPL or not (nothing in MAPL screams 0.00347).
For now, let's move past R_AIR and see what GNU flags next...
/discover/swdev/mathomp4/Models/GEOSgcm-ScottBranch-GNU/GEOSgcm/src/Components/@GEOSgcm_GridComp/GEOSagcm_GridComp/GEOSphysics_GridComp/GEOSmoist_GridComp/ConvPar_GF_GEOS5.F90:63:6:

   16 | USE ConvPar_GF_SharedParams
      |    2
......
   63 |   rgas    = 287.,    & ! J K-1 kg-1
      |      1
Error: Symbol 'rgas' at (1) conflicts with symbol from module 'convpar_gf_sharedparams', use-associated at (2)

Again, something similar, rgas is defined too many places. But this one is worse. Moist has a few  values of rgas:
Process_Library.F90
431:       RHO = 100.*PL / (MAPL_RGAS*TE )
1277:                              rgas   => MAPL_RGAS,   &

cldwat2m_micro.F90
57:   real(r8), parameter :: rair        = MAPL_RGAS

ConvPar_GF2020.F90
7862:      PKE      = (PLE/MAPL_P00)**(MAPL_RGAS/MAPL_CP)
7863:      PK       = (PLO/MAPL_P00)**(MAPL_RGAS/MAPL_CP)
7881:        ,   rgas     = 287.               &

ConvPar_GF_GEOS5.F90
63:  rgas    = 287.,    & ! J K-1 kg-1
7506:        ,   rgas     = 287.               &

micro_mg3_0.F90
207:   real(r8), parameter :: rair        = MAPL_RGAS

ConvPar_GF_Shared.F90
68:  rgas    = 287.,    & ! J K-1 kg-1
237:  real, parameter:: rgas  =  8.205e-2 ! atm M^-1 K^-1 ! 8.314 gas constant [J/(mol*K)]

aer_actv_single_moment.F90
446:      real(AER_PR), parameter :: rgasjmol = 8.31451d+00         ! [j/mol/k]

cldmacro.F90
17:         MAPL_RGAS , MAPL_H2OMW, &
65:   real, parameter :: ZVIR = MAPL_RVAP/MAPL_RGAS - 1.
66:   real, parameter :: GORD = MAPL_GRAV/MAPL_RGAS

uwshcu.F90
59:   real, parameter :: r     = MAPL_RGAS             ! Gas constant for dry air

aer_cloud.F90
151:      data rgas_par    /8.31d0/                  ! universal gas constant
169:      DATA rgas_ice /8.314d0/  ! Universal gas constant

Some routines use MAPL_RGAS but some (GF) have their own of just 287. GF2020 uses both!
This is also picking up use of rgas as the universal gas constant in aer_cloud.F90 and aer_actv_single_moment.F90 both of which are different than MAPL_RUNIV.
I think this issue might be more wide spread. For example in GF code you often see:
    real, parameter :: &
            cpg      = 102.45             &
        ,   rgas     = 287.               &
        ,   cp       = 1004.              &
        ,   p00      = 1.e5               &
        ,   g        = 9.80               &
        ,   rocp     = rgas / cp          &
        ,   p00i     = 1. / p00           &
        ,   cpor     = cp / rgas          &
        ,   cpi      = 1. / cp            &
        ,   p00k     = 26.870941          &  !  = p00 ** rocp
        ,   p00ki    = 1. / p00k

and I have to think many of these are MAPL_ constants. I mean, I know g is as MAPL_GRAV and there are a few other g in Moist as well:
gfdl_cloud_microphys.F90
71:    real, parameter :: grav = 9.80665 !< gfs: acceleration due to gravity

aer_cloud.F90
150:      data grav_par    /9.81d0/                  ! g constant
170:      DATA grav_ice /9.81d0/  ! gravity

ConvPar_GF_GEOS5.F90
7509:        ,   g        = 9.80               &

ConvPar_GF2020.F90
7884:        ,   g        = 9.80               &
8388:    real, parameter ::  XG    = 9.80665     ! gravity constant

aer_actv_single_moment.F90
264:                wupdraft           = -9.81*air_den*omega(i,j,k)     ! m/s - grid-scale only
454:      real(AER_PR), parameter :: gravity  = 9.81d+00            ! grav. accel. at the earth's surface [m/s/s]

I'm not sure what to do here. We eventually have to figure out how to get GNU to compile this. But I'm wondering what the "right" thing to do is. Change code to use MAPL_ constants? Only do the bare minimum to let it compile?
CC: @tclune @gmao-ckung

I think it is important that we are all cognizant that the long term plan is to use a "dictionary" for determining the values of physical constants.  (Some are going further and using a dictionary for mathematical constants, but I don't think GMAO needs its own value of pi, e, sqrt2, etc.)   In that era, each component will somehow be able to access a dictionary specified by the parent to retrieve its values.    We will maintain a single dictionary for all of GEOS.
In the near term, though we should at least consolidate.   My vote would be to rely entirely on the MAPL constants layer for now (extending as needed), but am happy to entertain reasons to leave these specified in some other layer.    If something is truly unique to a single component, then it is probably ok to define it there.


#28-Fix-Github-Actions-not-looking-in-correct-directory-for-react-build

Need to merge this to fix the azure builds and allow for Continous Deployment to occur for our react frontend

[ ] The pull request is complete according to the following criteria:

[ ] Acceptance criteria have been met
[x] The documentation is kept up-to-date
[x] Comprehensive tests (if applicable) have been generated and all pass.

[x] The pull request describes the changes that have been made, and enough information is present in the description for any developer to understand what has changed
[x] Commmits have been squashed
[x] The pull request title is of the following format : {issue/bug...}/(Issue Number) - Name of issue. E.g bug/30-Fix-Project
[x] The description uses github syntax to link to the issue. E,g Resolves se701g2/Doto#{Number}
[x] At least two reviewers assigned. One of which must be the assigner of the issue.
[x] If there are merge conflicts, run git rebase as opposed to git merge with master.

Done sorry that I forgot to include that


Video width/height not correct when not specified in the html

Tested under  Firefox and Chrome.
DefaultVideoWidth and videoWidth (and their height counterparts) correctly set the size of the controls, but not that of the actual video being displayed. Depending on whether size is set in the html video tag, it is either that, or the original size of the video that will be used. Entering then exiting full screen fixes the display (ie defaultVideoWidth and videoWidth are then applied) so this seems to be a small bug (order in which the size are calculated?).
Thanks for everything,
Martin

@DerProctophantasmist Is this still an issue? If not, can you close this please?

@DerProctophantasmist Closing old issues. If you'd like to reopen this, let us know. Thanks!


exited: {shutdown,{gproc_app,start,[normal,[]]}}

bin/etorrent console
Exec: /home/user/erlang/etorrent/rel/etorrent/erts-5.7.4/bin/erlexec -boot /home/user/erlang/etorrent/rel/etorrent/releases/1.2.1/etorrent -embedded -config /home/user/erlang/etorrent/rel/etorrent/etc/app.config -args_file /home/user/erlang/etorrent/rel/etorrent/etc/vm.args -- console
Root: /home/user/erlang/etorrent/rel/etorrent
Erlang R13B03 (erts-5.7.4) [source] [64-bit] [rq:1] [async-threads:12] [hipe] [kernel-poll:true]

=INFO REPORT==== 7-Nov-2011::21:50:59 ===
    application: gproc
    exited: {shutdown,{gproc_app,start,[normal,[]]}}
    type: permanent
{"Kernel pid terminated",application_controller,"{application_start_failure,gproc,{shutdown,{gproc_app,start,[normal,[]]}}}"}

Crash dump was written to: erl_crash.dump
Kernel pid terminated (application_controller) ({application_start_failure,gproc,{shutdown,{gproc_app,start,[normal,[]]}}})

```

cat log/sasl/sasl-error.log

```erlang

=SUPERVISOR REPORT==== 7-Nov-2011::21:50:59 ===
     Supervisor: {local,gproc_sup}
     Context:    start_error
     Reason:     {'EXIT',
                     {badarg,
                         [{ets,new,
                              [gproc,
                               [ordered_set,public,named_table,
                                {write_concurrency,true},
                                {read_concurrency,true}]]},
                          {gproc,create_tabs,0},
                          {gproc,start_link,0},
                          {supervisor,do_start_child,2},
                          {supervisor,start_children,3},
                          {supervisor,init_children,2},
                          {gen_server,init_it,6},
                          {proc_lib,init_p_do_apply,3}]}}
     Offender:   [{pid,undefined},
                  {name,gproc},
                  {mfa,{gproc,start_link,[]}},
                  {restart_type,permanent},
                  {shutdown,2000},
                  {child_type,worker}]
```

Oh. The problem here is that your OTP version R13B03 is quite old and gproc now assumes a later version of Erlang. specifically, I think the problem is around the options {read_concurrency, true}, {write_concurrency, true} though I may be mistaken.
The quick fix is to upgrade your Erlang version. I think also this would be a good time to decide upon exactly what old versions we support. I am somewhat of the opinion that R14B01 should be the oldest right now, though I might be mistaken.

Thank you,
generating release version for R14B4 fixes this error.


Add support for dragging a shader to user exported material property

Fixes #81722
This is a quick fix but might have unexpected impacts. I would appreciate any feedback or alternative suggestions.

Thanks!


fix: make ghost-text extmark with pcall

Fixes #257
This issue also happens to me in a large Rust project where the LSP is slow. It tries to show a slightly outdated text edit and errors, but shows the correct text edit instantly after. So maybe this could also be fixed with better scheduling.
Now I just make the extmark with pcall to not render outdated ghost-texts and cause errors. Nvim-cmp also does this.

This introduces another error that I've experienced by using your changes:
   Error  10:14:28 msg_show.lua_error Error executing vim.schedule lua callback: ...e/projects/blink.cmp/lua/blink/cmp/accept/text-edits.lua:89: index out of range
stack traceback:
	[C]: in function 'get_line_byte_from_position'
	...e/projects/blink.cmp/lua/blink/cmp/accept/text-edits.lua:89: in function 'get_from_item'
	.../projects/blink.cmp/lua/blink/cmp/windows/ghost-text.lua:68: in function 'draw_preview'
	.../projects/blink.cmp/lua/blink/cmp/windows/ghost-text.lua:53: in function 'show_preview'
	.../projects/blink.cmp/lua/blink/cmp/windows/ghost-text.lua:34: in function 'callback'
	...rojects/blink.cmp/lua/blink/cmp/windows/autocomplete.lua:283: in function 'emit_on_select'
	...rojects/blink.cmp/lua/blink/cmp/windows/autocomplete.lua:129: in function 'open_with_items'
	.../ibrahimdelice/projects/blink.cmp/lua/blink/cmp/init.lua:57: in function <.../ibrahimdelice/projects/blink.cmp/lua/blink/cmp/init.lua:51>

Ok. I have zero issues at least with rust-analyzer, but another solution is probably necessary.

Thanks! I believe I found the root cause to be from the offset applied to text edits

Seems to work :+1:


Product Summary Description

Product Summary
Description
Team Name: Go Works
Team Members: Vaniya Sharma, Racha Ibrahim, Alisha Velji, Olivia Fumagalli
Product Description
For the final group project we plan to build a responsive website for an online store called Kiddoo Toys.  Kiddoo Toys is an online family business for toys run by one of our team members' family and they currently hope to expand their reach by building a website for consumers to explore and buy the products. Currently, the company has done a great job expanding their reach successfully through social media and having a well built and functioning website will help them create more product awareness in the market.
In scope
The features that we plan to add to our website design so far are

online selling/shopping
a shopping cart
product display page
search bar for the products
about us for the company
contact us form

Out of scope
On of the things that are outside the scope of our design would be to add an online chat option to help consumers navigate through the website. This may not be possible to accomplish due to the time frame provided to us to finish the project.

Hey @imd1005-web-development-winter-2023/goworks
This is super awesome!
Very excited to see Kiddoo Toys come to life.
My only feedback is that you have a lot in scope - which is awesome - but perhaps consider starting on the core key functionality first and get that working great and then bolt on the other features as time permits. This way, if you run out of  time at the end you have the key functionality and design working great.
If you need anything else please don't hesitate to reach out!


Possible way to detect 2 languages?

I am bilingual (Spanish and English), but it seems the STT engine struggles when I switch languages (when I am speaking), could you set two languages to detect?

Unfortunately, that is not possible with the recognizer kikitan uses (WebSpeech API)


documentation Comments issues

There is no method summary comments for ‘ContinueAcquireToken’. Also no comments about params and return value
AcquireTokenByRefreshTokenAsync – Can we add a summary comment that this is not going cache the tokens?
AcquireTokenAndContinue –
// Returns:
//     It contains Access Token, Refresh Token and the Access Token's expiration
//     time.
May be we should say It returns nothing. ContinueAcquireToken which will be called subsequently returns AuthenticationResult which contains Access Token, Refresh Token and the Access Token's expiration time.

Archiving this sample


Publish symbols for assemblies in this package

Code that references this package will generate a SymbolCheck error when verifying that symbols are available for assemblies it uses, such as system.interactive.async.dll and system.linq.async.dll.
See https://dev.azure.com/devdiv/DevDiv/_workitems/edit/2047053

Could you clarify what you mean by "this package"? There are several packages in Rx.NET.
We publish symbols to NuGet so I'm not sure what it is you're asking us to do.
Also, that work item does not appear to be publicly visible. I'm guessing that "devdiv" is a Microsoft Azure DevOps account, so I'm wondering if perhaps you are under the impression that Rx.NET is a Microsoft project? It was a long long time ago (10 years ago), hence the use of System.* namespaces, but it has been a community-supported .NET Foundation project for many years now.
None of the currently active maintainers on this project works for Microsoft. So aside from the permission we have to use System package prefixes and namespaces (required for backwards compatibility), and the fact that the .NET Foundation signs our binaries for us, we don't get to do anything when publishing our packages that any random member of the public couldn't do. And as far as I know, publishing symbols to NuGet is the only option open to us.


NavisViews.GetSavedViews - Revit crashes

Hi Adam,
First of all this is an amazing work and incredibly useful.
I have been using your package to extract information from the Clash Tests for a while and works perfectly. Now I wanted to extract information from the saved viewpoints using your node NavisViews.GetSavedViews and for some reason Revit crashes every time I hit RUN. Do you know what am I doing wrong?
I attach an image of the nodes I am connecting, basically 4:
File Path > OpenNavisFile.OpenFile (with Boolean for the Visible input) > NavisViews.GetSavedViews.

I am using:

Revit 2015.
Navisworks 2015.
Dynamo <IP_ADDRESS>0
Dynaworks16 1.2.0

Many thanks in advance.
Angel

Your using DynaWorks16? It should be DynaWorks15.

Hi Gui, I'm finding similar to Angel here.
I'm running Dynamo 1.3 and Dynaworks 17 in Revit 2017 on a Navisworks Manage 2017 file.
Attempting to retrieve all viewpoints (same nodes as above) crashes Revit every time.
Many Thanks
Oliver


State Change On Hold to Response - No Email

Describe the bug in current situation
While doing regression testing, it was found that when an applicant does NOT have an email on file, and the request is put on hold to collect the outstanding balance from the response state - IAO can only change the state back to call for records, when they should be able to put it back to Response instead.
Link bug to the User Story
Impact of this bug
Moderate impact - can be worked around but will confuse and frustrate users
Chance of Occurring (high/medium/low/very low)
Low - most applicants have and provide email addresses, so this will not be encountered often.
Pre Conditions: which Env, any pre-requesites or assumptions to execute steps?
Steps to Reproduce
Steps to reproduce the behavior:

Create a general request in TEST and ensure there is no applicant email on file
Go through the fee payment scenario by paying a deposit (manually) and leaving an outstanding balance
When request moves to response, change state to On-Hold
Record payment, and attempt to change state - user can only move to Call for Records or Closed.

Actual/ observed behaviour/ results
Expected behaviour
GIVEN a Request is in the On-Hold State from the Response State
AND there is no applicant email on file
WHEN the user activates the request state drop down
THEN They will see the options of Response and Closed
Screenshots/ Visual Reference/ Source
If applicable, add screenshots to help explain your problem. You an use screengrab.

@divyav-aot @sumathi-thirumani-aot - I tested this, and it appears this isn't resolved.
I tested this on EDU-2022-55643 in TEST, where I moved the request from a 'Response' state to the 'On Hold' state. In the 'On Hold' state, the request state drop-down does not show 'Response' as an available option.

I have also tested this in TEST-Marshal on EDU-2022-12396 with the same result.
Is there anything I missed?


[web] create jsx mail landing page figma

Describe what you need
we need to create figma for jsx mail 2.x landing page

i made: https://www.figma.com/file/jA02pnUfYT4QRxYZnMvf3r/jsx-mail?node-id=106%3A120&t=rafvHXVZVCmds6Ie-1


feat: implement waiters

Issue #
Closes #572
Description of changes
Final merge PR for waiters in smithy-kotlin.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Missed some errors in waiter codegen. Cancelling this PR until an in-branch fix can be reviewed.


Middleware breaks assertViewHas assertion in test response

Using LaravelCaffeineDripMiddleware breaks the ability to do $response->assertViewHas() in tests, because it looks like the middleware converts the View object to a string and resets the contents.
This causes the TestResponse::assertViewHas to throw an error: 'The response is not a view.'

@dallincoons I'm open to a PR that would fix this. :)


Add m/s for wind speed data

Kilometers per hour as a unit of wind speed is almost never used in the countries of the former Soviet Union.

@Korb will do!


Logout using instance url and access token

I asked this same question on the node-salesforce GitHub account before realizing you had moved everything to jsforce, so sorry for the duplicates. I am logging in the node.js site using HTML login page with salesforce credentials. It returns the instance URL and access token which is saved in my session.
However, in the docs, it is mentioned that we can only log out using session Id and server URL. Can I not log out using the instance URL and access token? I tried doing that but it threw an error. I want the user to log out when they click the logout button.
Great work on the module though, loved it.

You need to pass loginUrl information in connection options to make the logout() work properly.

I am passing the loginURL in the connection.
This is my code for connection:

var username =<EMAIL_ADDRESS>var password = 'mypassword';
var securityToken = 'mysecuritytoken';
var conn = new jsforce.Connection({
	loginUrl : 'https://test.salesforce.com'
});

conn.login(username, password + securityToken, function(err, userInfo) {
				if (err) { return console.error(err); }
				console.log('AccessToken: ' + conn.accessToken);
				console.log('Instance URL: ' + conn.instanceUrl);
				console.log('User ID: ' + userInfo.id);
				console.log('Org ID: ' + userInfo.organizationId);
				req.session.accessToken = conn.accessToken;
				req.session.instanceUrl = conn.instanceUrl;
				return res.redirect('/dashboard');
			});

This is my logout code:
app.get('/logout', function(req, res) {
       //logout for the salesforce user
	var conn = new jsforce.Connection({
		instanceUrl: req.session.instanceUrl,
		accessToken: req.session.accessToken
    });

	conn.logout(function(err) {
		if (err) { return console.error(err); }
		// now the session has been expired.
	});
	req.logout(); //passport logout for the node.js site
	req.session.destroy();
	res.redirect('/');
});

It gives the following error in the conn.logout method:

{ [ERROR_HTTP_404: Bad_Id] name: 'ERROR_HTTP_404' }

anyone have a fixed on this?

Your logout code should be :
let conn = new jsforce.Connection({
oauth2 : oauth2,     //This is important
accessToken: req.session.accessToken,
instanceUrl: req.session.instanceUrl
});
conn.logout(function(err) {
  if (err) {  console.log('error',err); }
  console.log('logged out');
  req.session.instanceUrl = '';
  req.session.refreshToken = '';
  // now the session has been expired.
  res.redirect('/');
});

I was getting the same error but got it fixed by  oauth2 : oauth2, if we don't pass oauth2 details then jsforce library uses default login url causing which is login.salesforce.com which will cause your app to throw an error as your actual loggedin url and login.salesforce.com won't match


/userinfo does not provide phone number

The /userinfo endpoint does not return the user's phone number at the moment:
https://github.com/cloudfoundry/uaa/blob/3.4.0/server/src/main/java/org/cloudfoundry/identity/uaa/account/UserInfoEndpoint.java#L70
Can this be added?

This issue has been addressed starting with UAA 3.5.0


no permission to access "http://iows.ikea.com/retail/iows/tr/tr/catalog/items/" on this server.

Unfortunately i do get:
You don't have permission to access "http://iows.ikea.com/retail/iows/tr/tr/catalog/items/" on this server.
actually I do need only a code saving products in SQLITE database along with their prices, stock availabilities within specific store(s) and product urls. can you please help me how to do this?

Can you share your code?

well I guess it is about ikea turkey's web page. it is https://ikea.com.tr  -> also no need for country and language code. and I cannot use IOWS at all ... but let me check it once more.

Closing this, if you still need help, please leave a comment.


ISPN-9709 doc edits for Spring integration

https://issues.jboss.org/browse/ISPN-9709

I can merge this already and do some pom modifications in m spring 5 PR

@oraNod Can you create the backport PR please

@ryanemerson backport PR is ready: https://github.com/infinispan/infinispan/pull/6417


ASSERT_ERR(0) in BT libbtdm_app.a during A2DP sink reception (IDFGH-112)

I observed an assert while playing back (= receiving) A2DP data on the ESP32:
ASSERT_ERR(0), in lc_lmppdu.c at line 66
(this is apparently in components/bt/lib/libbtdm_app.a)
After this the watchdog triggers and the system is restarted (which works ok). But of course the BT connection is disconnected and the playback stops.
The assert occurs randomly and only very rarely, after hours of streaming.
The setup is:

ESP32-DevKitC with rev1 silicon
I2S DAC connected (and playing back)
WiFi is disabled on the ESP32
BT Classic is enabled on the ESP32, using BTstack (not Bluedroid) (roughly Oct 24).
BT connection and pairing worked flawlessly for hours, A2DP playback is working.
esp-idf at 2c95a77cf93781f296883d5dbafcdc18e4389656 (Oct 24)

What is asserted with this assert? Is this an invalid input to a BT function? Is this an out of memory condition? Is this an internal error?
Sorry, this is very hard to reproduce. I cannot even claim it is reproducible at all. I am merely hoping that a developer of libbtdm_app.a can look into the assert of gains information from this issue (e.g. that the asserted situation really occurs in real life).
I am already trying to reproduce this with the latest esp-idf/BTstack.

@jovermann , the latest master branch have resolve the problem. It's due to the interrupt react slowly, cause bt codes cannot handle in time. Although the bug is fixed, please check your code, if any code lock interrupt(portENTER_CRITICAL) too long time.

Hi all,
This issue has been fixed. Please pull the latest IDF.
Thanks
Weitianhua

Thanks for reporting, feel free to reopen if the issue still happens. Thanks.


Rename .json files to .geojson.

Thanks you for your contribution.
I have to make some changes first to some reports to ensure their continuity before I merge this commit.


This keeps happening - Microsoft.Data.SqlClient.resources.dll is not found during .NET Core 3.1 debug session in Visual Studio 2019

This keeps happening - Microsoft.Data.SqlClient.resources.dll is not found during .NET Core 3.1 debug session in Visual Studio 2019
"Could not load file or assembly 'C:\Users\user1\Dev\source\repos\repo_name\project_name\bin\Debug\netcoreapp3.1\runtimes\win\lib\netcoreapp3.1\Microsoft.Data.SqlClient.resources.dll'. The system cannot find the file specified."
It is only happening with this specific package, basically when I go to create a new SqlConnection object instance. Everything is working fine some software change on the system that seems to always impact this package.
I'm stumped. Any ideas how to work around this in C#? It seems like it magically fixes itself I repair or reinstall Visual Studio and .NET Core SDKs.

@EnocAnanda just to make it more clear, you have a simple application to open a connection with netcoreapp3.1 TFW and when you start debugging it you get an exception? or simply you can provide us a sample repro and we can investigate the issue.

Closing due to inactivity. Feel free to open a new issue or comment here if you are still experiencing the issue.


bug: ion-tabs above keyboard

Type: bug
Platform: android 4.4 webview
Hi I have a problem with < ion-tabs > moving up when opening keyboard.
This can be fixed by using:
< preference name="Fullscreen" value="true"  />
Is there any other way to solve this issue?
Here is the photo of the issue:

Greetings @Klapsa2503!
We are now requiring all issues to be submitted through our custom issue form so that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
This issue will be closed if not resubmitted through the form. Please fill out the rest of the form so that we may assist you.
Thank you for allowing me to assist you.

This is one of the quirks that android has.
http://ionicframework.com/docs/api/page/keyboard/
You could adjust the android.manefest file in the platform/android/ directory and change the keyboard activity to this.
 <activity android:windowSoftInputMode="adjustResize">

The other option is to let the tabs sit on top, as per their paltform design, and pass a input in the nav-title.
http://ionicframework.com/docs/api/directive/ionNavTitle/


Better UX experience for scrolling

💡Scrolling should be separate from the tabs so the user can navigate menus easier.
⚠all-options relies on hardcoded values
📑someone should find a way to not rely on these values incase the height changes on the body, header or tab in the ul
IDK about the compatibility on mobile devices
Also my prettier plugin changed a bunch of white space. I can re-do this on a fresh copy if needed.

Thanks. This pull request has been integrated manually.


feat: add cmek support in composer_env_v2

Fixes https://github.com/terraform-google-modules/terraform-google-composer/issues/74

@imrannayer I took your comment in account, do I need to do something else ?

/gcbrun


[SPARK-16318][SQL] Implement various xpath functions

What changes were proposed in this pull request?
This patch implements all but one remaining xpath functions: xpath_int, xpath_short, xpath_long, xpath_float, xpath_double, xpath_string.
How was this patch tested?
Added unit tests and end-to-end tests.

Can you guys take a look at this? @cloud-fan @dongjoon-hyun @gatorsmile

Test build #61530 has started for PR 13991 at commit 66fae41.

Test build #61530 has finished for PR 13991 at commit 66fae41.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61530/
Test PASSed.

Merged build finished. Test PASSed.

Could you add some test cases for the boundary values? That means, trying the maximum and minimum values for each different data type. Also, the values that are out of the boundary.

Could you add more test cases for corrupted xml and illegal inputs? So far, the test suite XPathExpressionSuite only covers the basic cases.

Normally, for each function or data type we implement in an enterprise product, we need to document the limit/restriction. That is why I am asking for the boundary values, negative cases and error handling.

+1 for @gatorsmile 's advices

Thanks for the feedback. I will update this later.

For this one I think we should consider supporting only foldable literals for the path component.

Test build #61947 has started for PR 13991 at commit 2d48ae5.

Also - rather than having concrete implementations for all of these, why don't we use RuntimeReplaceable?

Test build #61947 has finished for PR 13991 at commit 2d48ae5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61947/
Test PASSed.

I pushed a new change to this. We now have better error messages and test coverage for those. These expressions also now require foldable paths.
I also changed the test values to make sure we are not restricting the range of values an expression can return (e.g. for XPathInt the test case makes sure it can return a value larger than short). However, I don't think it'd make sense to test overflow behavior because those are mostly undefined throughout Spark SQL.
Re: @rxin's suggestion on using RuntimeReplaceable -- I did try that, but I'm afraid the current implementation of RuntimeReplaceable isn't meant for stacking multiple expressions together, and time checking actually breaks if XPathInt replaces itself with Cast(XPathDouble). I looked into fixing that, but it seemed more complicated and it doesn't actually save much space (it actually takes more code to use RuntimeReplaceable).

Test build #61953 has started for PR 13991 at commit d30d891.

Test build #61954 has started for PR 13991 at commit 02df488.

Test build #61953 has finished for PR 13991 at commit d30d891.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61953/
Test FAILed.

Test build #61954 has finished for PR 13991 at commit 02df488.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61954/
Test FAILed.

Test build #61957 has started for PR 13991 at commit d7d5f8d.

I just added the general xpath function that returns an array of string too.

Test build #61958 has started for PR 13991 at commit 48311bb.

Test build #61967 has started for PR 13991 at commit a86bc04.

Test build #61957 has finished for PR 13991 at commit d7d5f8d.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class XPathList(xml: Expression, path: Expression) extends XPathExtract

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61957/
Test FAILed.

Test build #61958 has finished for PR 13991 at commit 48311bb.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61958/
Test FAILed.

Merged build finished. Test FAILed.

Test build #61975 has started for PR 13991 at commit 24deac9.

Test build #61967 has finished for PR 13991 at commit a86bc04.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61967/
Test FAILed.

Test build #61975 has finished for PR 13991 at commit 24deac9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61975/
Test PASSed.

Merged build finished. Test PASSed.

Test build #61992 has started for PR 13991 at commit fecc293.

Test build #61992 has finished for PR 13991 at commit fecc293.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61992/
Test PASSed.

Test build #62000 has started for PR 13991 at commit 0c60d87.

Test build #62000 has finished for PR 13991 at commit 0c60d87.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class ParseUrl(children: Seq[Expression])

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62000/
Test PASSed.

Test build #3175 has started for PR 13991 at commit 0c60d87.

Test build #3175 has finished for PR 13991 at commit 0c60d87.

This patch passes all tests.
This patch does not merge cleanly.
This patch adds the following public classes (experimental):
case class ParseUrl(children: Seq[Expression])

retest this please

LGTM, pending jenkins

Test build #62070 has started for PR 13991 at commit 0c60d87.

@cloud-fan Jenkins already ran twice successfully before.

The latest result is This patch does not merge cleanly., I just wanna double check it.

I guess whatever generates that message is buggy?

Test build #62070 has finished for PR 13991 at commit 0c60d87.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class ParseUrl(children: Seq[Expression])

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62070/
Test PASSed.

Merged build finished. Test PASSed.

As a follow-up task. Can you take a look at the following query files and add useful tests in your test? Thanks.
.//sql/hive/src/test/resources/ql/src/test/queries/clientpositive/describe_xpath.q
.//sql/hive/src/test/resources/ql/src/test/queries/clientpositive/input_testxpath.q
.//sql/hive/src/test/resources/ql/src/test/queries/clientpositive/input_testxpath2.q
.//sql/hive/src/test/resources/ql/src/test/queries/clientpositive/input_testxpath3.q
.//sql/hive/src/test/resources/ql/src/test/queries/clientpositive/input_testxpath4.q
.//sql/hive/src/test/resources/ql/src/test/queries/clientpositive/udf_xpath.q
.//sql/hive/src/test/resources/ql/src/test/queries/clientpositive/udf_xpath_boolean.q
.//sql/hive/src/test/resources/ql/src/test/queries/clientpositive/udf_xpath_double.q
.//sql/hive/src/test/resources/ql/src/test/queries/clientpositive/udf_xpath_float.q
.//sql/hive/src/test/resources/ql/src/test/queries/clientpositive/udf_xpath_int.q
.//sql/hive/src/test/resources/ql/src/test/queries/clientpositive/udf_xpath_long.q
.//sql/hive/src/test/resources/ql/src/test/queries/clientpositive/udf_xpath_short.q
.//sql/hive/src/test/resources/ql/src/test/queries/clientpositive/udf_xpath_string.q

Actually I created the unit tests based on those.

BTW I have to say Hive's test coverage in this area is very spotty, so I don't actually think it's great to follow, but I used those.

OK. Thanks. Then, it will be good to add more tests for cases that are not covered by those hive tests.

Thanks, merging to master! This doesn't merge clearly to 2.0, @petermaxlee can you submit a new PR against 2.0? thanks!

@cloud-fan thanks for merging!
@yhuai I think the degree to which we want to add more tests also depend on how much we trust the library we are using. XPath (with Query) is almost as complicated as SQL itself.

Here it is for branch-2.0.
https://github.com/apache/spark/pull/14131


Add authentication options

Snowflake documented the newly supported authentication methods at https://docs.snowflake.com/en/user-guide/nodejs-driver-use.html#authentication-options.
The PR updates the ConnectionOptions types to include the new options and introducing a new connectAsync() method that required to connect via Browser Based SSO. Further details at https://docs.snowflake.com/en/user-guide/nodejs-driver-use.html#establishing-a-connection

This is published as version 4.5.0. Thank you.


String interpolation not highlighted correctly

Code like
print("$value");

Is not highlighted correctly. value should be seen as a variable.

@theniceboy I think this is a duplicate of #11 where I raised a similar issue. I had a look at the code but couldn't figure out how this was happening. @UserNobody14 if you could provide something oversight maybe I could look into it

Yeah you’re right it is a dup. Sorry about that!


ci: cache dependencies in all exhaustive jobs

P.S. Maybe we can use sccache-actuon in other jobs as well to cache dependencies?

I don't think there's any need for it, in my experience this does a really good job.

Looks good to me


Material chip list should support being not focusable

Bug, feature request, or proposal:
Feature request
What is the expected behavior?
I should be able to pass in an @Input() to mat-chip-list so that it doesn't have tabIndex = 0 by default. Sometimes we want to display chips within a tooltip. And we don't want user to be able to focus on individual chips.
What is the current behavior?
mat-chip-list is always focusable by default
What are the steps to reproduce?
Create mat-chip-list with non-zero chips inside
What is the use-case or motivation for changing an existing behavior?
To increase use cases for material chip list component
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
All

@jelbourn as FYI

@tinayuangao the chip list should respect a user-provided tabIndex if there is one

This doesn't seem to make the chip appear to be not interactive when you click on the chip, it still has a ripple. Is there a way to have a readonly chip but is not disabled?


[CUR-567] Units overview page to docx generation

Description

List of changes

Issue(s)
CUR-567
How to test

Go to https://deploy-preview-2434--oak-web-application.netlify.thenational.academy/teachers/curriculum/docx-poc/select
Choose a subject via the autoComplete
Download the docx template - linked on the page
Download the rendered document
Check the units overview page is present and correct for each year in keystage

Screenshots

Sample docs generated:
Science - Secondary - AQA - 20-05-2024 16-40-53.docx
English - Secondary - AQA - 20-05-2024 16-54-13.docx
Initial ssues found:

In Y10 and Y11, some units are duplicated. In Science this is likely due to Tier variants, in English this is because these are units with options.
The 'open in new window' icon overlaps the text
The year 11 grid in both documents breaks onto a new page so there's an empty page after the Y11 heading.

Regarding the styling issues @rightsaidjames I see the following. Can you try reinstalling your ms word, seems like some funky things are going on (error message previously).

Ah ok @rightsaidjames so it seems like the fonts weren't embedded in the .docx file. There is an updated one available (the usual location), I've also added unit options and fixed the table wrapping issue.

@orangemug The styling issue is now fixed, but I'm getting some weirdness with the data, especially in KS4 Science:

The units aren't in the right order (e.g. Y10 Science, Y11 Science) compared to the visualiser.
Lots of units report having 2 or more options when there is no optionality for that unit (it seems to be confusing variants with optionality)
For units that do have optionality (e.g. in KS4 English), the number of reported options is incorrect.
In Y10 and Y11 Science, there are units displayed in the Word doc that aren't shown in the visualiser (e.g. Inheritance, genotype and phenotype in Y10 and Gravity in space in Y11.

Sample documents:
English - Secondary - AQA - 22-05-2024 16-59-12.docx
Science - Secondary - AQA - 22-05-2024 16-38-00.docx

This has been merged into #2448 as I needed to make changes across both these PRs in one place, closing


feat: persistent logging for the broker

We currently keep the broker's log in memory, but that's not ideal -- it means the logs will disappear forever if the broker gets restarted.
Whatever persistence solution we use needs to play nice with Heroku, since that's where the broker is deployed.
Previous discussions have been redis or mongodb. Whoever claims this issue gets to decide and do it the way they like!
This is related to (and possibly a duplicate of) https://github.com/electron/bugbot/issues/52

If the logs are just an HTML file / text file the cheaper solution to a full on database is to just dump the logs on an S3 bucket somewhere.  (We can make an S3 bucket on our AWS account, or you can provision one through heroku afaik but AWS would be cheaper lol)


unskip end to end chunk insert op test

Description
This PR unskips the end to end chunk insert op test, as it is now working due to PR 19009.

It appears that the summary tests still fail as during the summary encode the UniformChunk gets converted into a SequenceChunk and are no longer reference equal. I have tried inserting a SequenceChunk into the test instead, but it still does not seem to share a reference to the original chunk, so I have kept these tests skipped for now.


[v1.13] bpf: improve handling for short packets

Manual backport for

[ ] https://github.com/cilium/cilium/pull/25159

Once this PR is merged, you can update the PR labels via:
for pr in 25159; do contrib/backporting/set-labels.py $pr done 1.13; done

or with
make add-labels BRANCH=v1.13 ISSUES=25159

/test-backport-1.13

/test-1.24-4.19

/test-1.26-net-next

/test-backport-1.13

/test-backport-1.13

/test-1.26-net-next


⚠ Bofrost.es has degraded performance

In dd050df, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1423 ms

Resolved: Bofrost.es performance has improved in 437ebf8 after 21 minutes.


Combobox Accessibility

Description
Combobox is not currently using the correct DOM for Aria.
React Spectrum might have worked toward fixing this in their code base.

PickerButton seems to be using the right arias in our Storybook though we haven't mapped all the appropriate events (just click) so that the interaction isn't reflecting a production implementation.
Can we update the goal of this issue to cover adding expected events to the demo as well?
https://opensource.adobe.com/spectrum-css/?path=/story/components-combobox--default


Button title length issue

I noticed that if you set the button titles too long, it causes the button width to extend and look ugly.

Fixed this in the pull request as well


2019 OSX builds - crash during build from selection

Hello! - hope this is the correct place for this :)
I just finished building the plug-ins for Maya 2019 on OSX.
Description
When I import the template biped guide, and 'build from selection' - Maya crashes mid-way into the build.
What is working:

The plugins load fine.
I am able to build random modules just fine.
A template biped rig that was generated on mgear 3.1.1 on windows/OSX Maya 2018 is working fine.

p.s
Build of the plug-ins seem to work perfect on 2018 OSX
To Reproduce

On Maya 2019 (OSX), use the plug-ins from below, import biped guide and build rig.

The error does not say much, not the standard Maya fatal error:

Additional information
built plugins: https://www.dropbox.com/s/imdxbs1s22u1ior/2019-osx-plug-ins.zip?dl=0
scons build log: https://www.dropbox.com/s/glonggxthhfuw3y/scons-build-log.txt?dl=0
Osx 10.14
Maya 2019
SCons 3.0.1 installed with brew
EDIT:
Further testing shows that the crash is related to the arm_2_jnt01 and leg_2_jnt01.
When these modules are deleted from the template, the build completes successfully.

parenting them under the root control - still crashing
unlinking them from the host and references IK etc - still crashing
update guides - still crashing
replacing them with any arm/leg module that i am creating - still crashing
replacing them with other module, like a chain that i am creating - all good.

Investigation continues...
EDIT 2:
Building with 'objects' mode works, any mode above will crash.

So far this actually looks like a 2019 OSX issue, and not related to the plug-ins themselves, but this is just a gut feeling.

@liorbenhorin Thanks for sharing this info.
To be honest, I didn't test it in OSX with Maya 2019
I will try to ask some colleagues to see if someone can help with this
Thanks,
Miquel

I had the same experience on macOS 10.13.6 Maya 2019
It seems to work when Guide Settings > Add Internal Proxy Channels is off, it crushes when is on.
On linux Maya 2019 works fine with Add Internal Proxy Channels on

Ops!  I completely misunderstood the issue :(  (I should not read this things in a rush, sorry)
The problem is not at plugins compile time. but later at guide build time. correct?

@miquelcampos  correct.
Same happens for me:

It seems to work when Guide Settings > Add Internal Proxy Channels is off, it crushes when is on.

@liorbenhorin thanks for the confirmation
Do you know if Maya 2019 OSX has any issue with proxy channels?

I haven't had any issues with Maya 2019 proxy channels on macOS.

This is going to be hard to debug for me since I don't have Maya 2019 in OSX env :(
@liorbenhorin @jollypixel  can you help with this?
Thanks
Miquel

I will definitely take a look as soon as I have a couple of hours

Thanks!

@miquelcampos Yes, of course :)

thanks!

It doesn't crush when this is commented out in arm_2jnt_01/init.py
'''
attribute.addProxyAttribute(
attrs_list,
[self.fk0_ctl,
self.fk1_ctl,
self.fk2_ctl,
self.ik_ctl,
self.upv_ctl,
self.mid_ctl])
attribute.addProxyAttribute(self.roll_att,
[self.ik_ctl, self.upv_ctl])
'''
If I uncomment this Maya crushes whitout even trying to save a temp file
'''
attribute.addProxyAttribute(self.roll_att,
[self.ik_ctl, self.upv_ctl])
'''
I tryied leaving only one object (instead of a list) and it doesn't crush, but the proxy attribute doesn't have a name?!
'''
attribute.addProxyAttribute(self.roll_att, self.ik_ctl)
'''
Yet attribute.addProxyAttribute() works fine when I run it in the script editor.
I'll keep looking

On closer inspection...
// Error: file: /Applications/Autodesk/maya2019/Maya.app/Contents/scripts/others/dynRenameAttrWin.mel line 277: Target objects must be specified in "node.attribute" format //

It works with "Use Classic Channel Names" off and one object instead of a list

Just run this on five objects (the last one had an attribute "test" and works fine.
import pymel.core as pm
from mgear.core import attribute
attribute.addProxyAttribute(pm.selected()[-1].test, pm.selected()[:-1])


Parameter name standardization

Now that parameters are more user-facing, I think the naming convention of parameters should be standardized. I prefer them being camelCase but some of them are PascalCase.

@ryaneberly Thanks! I think there were some unintended changes to setters in ValidName.java.

fixed in dev.  Thanks @KamasamaK .


Support of JsonPolymorphic and JsonDerivedType attributes

The issue or feature being addressed
supersedes #2671
Details on the issue fix or feature implementation

Added checks for System.Text.Json polymorphic attributes in the same handlers, where swagger native attributes are handled. Swagger attributes have priority over System.Text.Json ones.

Added this case to NSwagClientExample integration test

Refactored the way Basic and NSwagClientExample tests run. They both have different outputs for .net6 and .net8/9. I made a different theory for such tests in .net6 mode. The redudant SwaggerEndpoint_ReturnsValidSwaggerJson_Basic_DotNet_8 was deleted, as it is handled by the main SwaggerEndpoint_ReturnsValidSwaggerJson

Before I merge this, could you copy-over the changes to the README about how this works please?
https://github.com/domaindrivendev/Swashbuckle.AspNetCore/pull/2671/files#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5

added Readme

Resolves #2571.

Yeah, this is actually nice! Thanks guys!

Thanks for picking up my PR and finishing the work. Good job!


[BugFix] Fix bug where linear/angular velocities are swapped in GPU sim vs CPU sim

After some debugging @fbxiang and I found out (astonishingly) that physx exposed GPU buffer data of link velocities are in a different order to their CPU sim counterpart...

closes #452


[BUG] Ignore Self Applying to Self

When configure a effect to ignore self, it apply the effect on self.
Here's the item I configure:
fvtt-Item-dominio-radiante-xIKmtisuYJoqOVmT.json
The item apply an effect called Domínio Radiante on the source and an aura of 15 ft. that apply Blind on enemies, but the Blind is applying on the source too.

Please also provide:
Environment:

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Foundry Version [e.g. 0.8.8]
Module Version [e.g 0.1.42]
All Modules Disabled: Yes/No
Module List if no:

Oh, I forgot that!

OS: Windows 11
Browser: OperaGX
Foundry Version: v11.315
Module Version: v0.9.10
All Modules Disabled: No
Module List if no:

Compendium R&C is just a compendium module that I create.

I think it might be a bug in DAE or Midi for StatusEffects key. Try using macro.CE instead for now:

I don't use CE anymore D:
That's the only solution?

Until a proper fix.

Oh, I though that would be the final sulution hehe.
I'll just wait. Thanks for the assistance and for the module!

@MrPrimate any news on this? I’m having the issue and would love even just a script workaround to have the aura successfully ignore self.


Prompt for confirmation when moving overwrites existing file

We fixed this when importing files - we should do the same (prompt for confirmation) when moving notes around

I will look into this once you are done with #124, @jffrymrtn

@plafue sounds like a plan. I should get #124 done by the end of the week (or even sooner). I'll post here when it is :)

Oh cool! thanks! :tada:


icacls commands do not work on the page

icacls : /grant :(M)
icacls : /grant "Creator Owner":(OI)(CI)(IO)(M)
icacls : /remove "Authenticated Users"
icacls : /remove "Builtin\Users"
M : The term 'M' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
At line:1 char:45

icacls y: /grant grpsysJumpHostUsers-Azure:(M)

                                        ~

CategoryInfo          : ObjectNotFound: (M:String) [], CommandNotFoundException
FullyQualifiedErrorId : CommandNotFoundException

OI : The term 'OI' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
At line:2 char:35

icacls y: /grant "Creator Owner":(OI)(CI)(IO)(M)

                              ~~

CategoryInfo          : ObjectNotFound: (OI:String) [], CommandNotFoundException
FullyQualifiedErrorId : CommandNotFoundException

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

ID: 69769604-b1c3-a2b8-318d-354ebf0d82a1
Version Independent ID: 964330fc-1a4f-05ca-502a-a09f26e79f10
Content: Create FSLogix profile container Azure Files Active Directory Domain Services - Azure
Content Source: articles/virtual-desktop/create-profile-container-adds.md
Service: virtual-desktop
GitHub Login: @Heidilohr
Microsoft Alias: helohr

Hello @SkywalkerHogie,
Thank you for your feedback! We will review and update as appropriate.

Hi, it's because you're using PowerShell to run the icacls commands instead of command prompt and they both interpret syntax slightly differently.  I've updated the article to use PowerShell to be more relevant, which also works with command prompt.  The changes should be visible in the next few days.
#please-close


Including the resource interfaces inside the FungibleToken contract interface.

Description
This is more a code review/suggestion than a PR, but I thought it will be easier to read it this way.
This basically puts all the resource interfaces inside the FungibleToken contract interface so developers do not need to import two different contracts when creating FT. Only thing we are giving up is the implementation of the init function in the VaultInfo struct.

Thanks for the suggestion! We'll see what other people think before we include any of this

I'll flag this in the actual PR conversation if that makes sense!


Documentation of showGeneralDialog is contradictory

The documentation of showGeneralDialog states:
https://github.com/flutter/flutter/blob/5d37734ea1032f25dfc022f6130916e28b98e7f3/packages/flutter/lib/src/widgets/routes.dart#L1871-L1874
https://github.com/flutter/flutter/blob/5d37734ea1032f25dfc022f6130916e28b98e7f3/packages/flutter/lib/src/widgets/routes.dart#L1876-L1877
but barrierDismissible actually defaults to false (introduced with https://github.com/flutter/flutter/pull/61503) and barrierLabel doesn't default to anything and must not be null if barrierDismissible is set to true.

CC @darrenaustin

@pedromassangocode Pedro I believe this has been fixed. Wdyt?

Yep, thank you!


prefix-tuning RuntimeError: Trying to backward through the graph a second time (or directly access saved tensors after they have already been freed).

System Info
PyTorch version: 2.0.1+cu117
Is debug build: False
CUDA used to build PyTorch: 11.7
ROCM used to build PyTorch: N/A
OS: Ubuntu 20.04.4 LTS (x86_64)
GCC version: (Ubuntu 9.4.0-1ubuntu1~20.04.1) 9.4.0
Clang version: Could not collect
CMake version: version 3.27.0
Libc version: glibc-2.31
Python version: 3.9.17 (main, Jul  5 2023, 20:41:20)  [GCC 11.2.0] (64-bit runtime)
Python platform: Linux-5.15.0-71-generic-x86_64-with-glibc2.31
Is CUDA available: True
CUDA runtime version: Could not collect
CUDA_MODULE_LOADING set to: LAZY
GPU models and configuration:
GPU 0: NVIDIA GeForce RTX 3090
GPU 1: NVIDIA GeForce RTX 3090
Nvidia driver version: 495.29.05
Versions of relevant libraries:
mypy-extensions==1.0.0
numpy==1.23.5
torch==2.0.1
torchdata==0.6.1
torchtext==0.15.2
torchvision==0.15.2
numpy = 1.23.5
torch  = 2.0.1
torchdata =  0.6.1
torchtext   =0.15.2
torchvision =  0.15.2
Information

[ ] The official example scripts
[X] My own modified scripts

🐛 Describe the bug
I encountered the above error while fine-tuning the model with prefix
here is my fine-tuning script：
CUDA_VISIBLE_DEVICES=0,1 torchrun --nnodes 1 --nproc_per_node 1 examples/finetuning.py \
    --use_peft \
    --peft_method prefix \
    --model_name ../model/llama-2-7b-chat-hf \
    --use_fp16 \
    --output_dir ./output \
    --dataset alpaca_dataset \
    --data_path ./data.json \
    --batch_size_training 16 \
    --num_epochs 3 \
    --quantization

Error logs
Traceback (most recent call last):
File "/home/zxy/llama2/llama2-lora-fine-tuning/llama-recipes-main/examples/finetuning.py", line 8, in
fire.Fire(main)
File "/root/anaconda3/envs/llama2/lib/python3.9/site-packages/fire/core.py", line 141, in Fire
component_trace = _Fire(component, args, parsed_flag_args, context, name)
File "/root/anaconda3/envs/llama2/lib/python3.9/site-packages/fire/core.py", line 475, in _Fire
component, remaining_args = _CallAndUpdateTrace(
File "/root/anaconda3/envs/llama2/lib/python3.9/site-packages/fire/core.py", line 691, in _CallAndUpdateTrace
component = fn(*varargs, **kwargs)
File "/root/anaconda3/envs/llama2/lib/python3.9/site-packages/llama_recipes/finetuning.py", line 237, in main
results = train(
File "/root/anaconda3/envs/llama2/lib/python3.9/site-packages/llama_recipes/utils/train_utils.py", line 84, in train
scaler.scale(loss).backward()
File "/root/anaconda3/envs/llama2/lib/python3.9/site-packages/torch/_tensor.py", line 487, in backward
torch.autograd.backward(
File "/root/anaconda3/envs/llama2/lib/python3.9/site-packages/torch/autograd/init.py", line 200, in backward
Variable._execution_engine.run_backward(  # Calls into the C++ engine to run the backward pass
File "/root/anaconda3/envs/llama2/lib/python3.9/site-packages/torch/autograd/function.py", line 274, in apply
return user_fn(self, *args)
File "/root/anaconda3/envs/llama2/lib/python3.9/site-packages/torch/utils/checkpoint.py", line 157, in backward
torch.autograd.backward(outputs_with_grad, args_with_grad)
File "/root/anaconda3/envs/llama2/lib/python3.9/site-packages/torch/autograd/init.py", line 200, in backward
Variable._execution_engine.run_backward(  # Calls into the C++ engine to run the backward pass
RuntimeError: Trying to backward through the graph a second time (or directly access saved tensors after they have already been freed). Saved intermediate values of the graph are freed when you call .backward() or autograd.grad(). Specify retain_graph=True if you need to backward through the graph a second time or if you need to access saved tensors after calling backward.
Expected behavior
I want to know if I wrote something wrong or other reasons, how to solve it

Encounter the same problem! when the finetuning method turn to ptuning or others, there weren't be this problem. Is there any thing wrong with the peft.PrefixTuningConfig?

@JunoLiusj if you are using it with FSDP unfortunately its not supported, https://github.com/meta-llama/llama-recipes/pull/482


减少冗余代码

see #46

Codecov Report

Merging #47 into master will decrease coverage by 0.12%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #47      +/-   ##
==========================================
- Coverage   78.30%   78.17%   -0.13%
==========================================
  Files           7        7
  Lines         696      692       -4
==========================================
- Hits          545      541       -4
  Misses        111      111
  Partials       40       40

Impacted Files
Coverage Δ

set.go
56.47% <100.00%> (-1.96%)
:arrow_down:

Continue to review full report at Codecov.

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


Fix pinning of Microsoft.AspNetCore.Server.Testing

Follow up to https://github.com/aspnet/Hosting/pull/841

cc @anurse \ @Eilon

Note: this is for the master branch \ 1.0.0 tag.


Feature Request: Set Defaults

Feature Request
Feature Information:
So I noticed in minecraft a lot of the normal settings that are default set in the server properties is turned off on your panel by default so perhaps a option at the start before you click manage you could allow us to lets say right click the instance and have a option that says set default for minecraft or something like that so when you go to create a new minecraft instance it bases it off that said server (only copying a few things like server properties not the whole server folder)
I confirm:

[ X] that I have searched for an existing feature request matching the description.

I don't understand your message at all, maybe you could give a more detailed explanation of what you're trying to do?

Basically by default the control panel has everything turned off right, Default minecraft generates some configs and has them on by default, I think the control panel should have these settings on by default instead of just all set off.  My idea was a addon to that, I think you should be able to config one instance and sorta copy it or as I said set it as the default for all newly created minecraft instances, so lets say you edit Allow-Flight from False to true then you start the server, after you go and set the instance as the default so all your settings including the allow-flight = true will now be the default when you create a new minecraft instance but without copying the world too (this is so you dont have to manually config every single option every time)  If you don't want to go that far then simply having the default panel options set to the default server.properties thats generated by just running a server without the panel by default would be nice

You're after configuration templates. That's happening.
Try and break your messages up into paragraphs more, it's not easy to read :P

Closed as no longer applicable.


Display controller variable in custom widget title

Is the any way to access a variable define in the controller of the widget ?
I'm trying to display the number of elements displayed in the widget content.
Thanks for ADF which is very cool and well done.

No, this is not possible a controller is able to manage the widget content and not the title.


added daily event listener to check if bot has entered the room

Same issue, the bot joins the room but doesn't show up immediately on connection when I speak in the background. As soon as I stop it responds and shows up


Question: How would I preserve ipython history between docker-compose runs with the local setup? (Answer is within)

I'm submitting a ...

[ ] bug report
[ ] feature request
[ ] support request => Please do not submit support request here, see note at the top of this template.
[x] quick Q&A tip, I'll answer it myself then close the issue.

Question:  How would I preserve ipython history between docker-compose runs with the local setup?
When using docker-compose for local development, how would you preserve the ipython history between different runs of shellplus?
Answer: Just two lines in your local.yml
Extending the excellent setup that comes with cookiecutter-django, and adding this little change to the local.yml, is not hard. Make sure you've read these docs first
The quick change, as a .diff
diff --git a/local.yml b/local.yml
index b587373..73684bf 100644
--- a/local.yml
+++ b/local.yml
@@ -3,6 +3,7 @@ version: '2'
 volumes:
#...
+  ipython_data_local: {}

 services:
   django:
#...
     volumes:
+      - ipython_data_local:/root/.ipython/profile_default   # persist ipython data, including ipython history
#...

Explained in more detail
Copying from my gist here
First do whatever you've been doing, to start your ipython shell inside of your docker-compose setup.
For example, I've got a project based on cookiecutter-django,
and this is how I was entering a shell for my project locally:
docker-compose run --rm django python manage.py shell_plus

So after doing that I checked where my ipython history might be.
$ docker-compose run --rm django python manage.py shell_plus
#...

In [1]: !ipython locate profile default
/home/someuser/.ipython/profile_default

Now that I have the path, I can use a volume to persist it using
the docker-compose YAML directives for this.
I can add this to volumes section of my docker-compose YAML file:
volumes:
    ipython_data_local: {}

Then in the section for the Python service:
services:
  django:
    volumes:
      - .:/app  # this was already here
      - ipython_data_local:/root/.ipython/profile_default   # persist ipython data, including ipython history
#.........................`

And that'll do it :-)
Disclaimer
BY THE WAY: you only get certain benefits of Docker or Docker-Compose if you keep local and production almost identical. That's still recommended. production.yaml should work locally, for example. Even so, a local.yaml that is very productive and efficient can be a good idea, especially when you have contributors who might have trouble maintaining their own stable environment. So that's the intended use case of this.
Please do not open shells in production using this trick. Please interact responsibly :)

Related: #1794


Fix for getting local cluster for Helm Chart testing in case testing on shared cluster is suppressed

case testing on shared cluster is suppressed

This issue block helm charts CI. In case everything is GREEN. merge it as soon as possible.


Blender 4.1

Please could Blender 4.1 support be added? It has some fixes that are required for a project I am working on.

You can find more up to date images here: https://github.com/ranchcomputing/blender-cpu-image


Difference issue

Solving issue #29. Use the first difference

Though test_simple_list_with_dicts shows the fix for the specific issue, I think the other two new tests are more interesting. Both test_list_with_dicts and test_list_with_dicts_with_duplicates show really unexpected results.
I think a better general solution would be to cache entries that have been matched (including their index) so they could be ignored on following difference findings.
Maybe the code added (_min_diff) is enough for now, as the other two tests will have the same issues with  _max_diff, just giving different values.

Ah, I left the tests, to document the current behaviour. Maybe a comment could left on each test to indicate that the behaviour should be revisited

Ah, I left the tests to document the current behaviour. Maybe a comment could be left on each test to indicate that the behaviour should be revisited

Yes, add it with link on u comment please.

Both things done


Cannot build ViewableInventory

I am currently running

SpongeVanilla version:  https://github.com/SpongePowered/Sponge/actions/runs/1281098566
Java version: 1.16
Operating System: Win10

Plugins/Mods:
Custom mod

Issue Description
java.lang.IllegalStateException: Plugin has not been set in this builder!

The above error occurs before I can provide a plugin to the builder.
This is probably due to this https://github.com/SpongePowered/Sponge/blob/ece3eb6b59556e1159ae3baca15bef703e5fea73/src/main/java/org/spongepowered/common/inventory/custom/SpongeViewableInventoryBuilder.java#L240 missing a plugin provider
        at org.spongepowered.common.inventory.SpongeInventoryBuilder.build(SpongeInventoryBuilder.java:118)
        at org.spongepowered.common.inventory.custom.SpongeViewableInventoryBuilder.completeStructure(SpongeViewableInventoryBuilder.java:240)

My code
ViewableInventory.builder().type(type).completeStructure()

It's telling you. Plugin has not been set in this builder.
Is there a method called plugin or a method which takes a plugin in the builder?

Im well aware of what it says. The plugin method is after the point in my code (my code is returning a EndStep with you supplying the pluginContainer to the EndStep).
The issue is I cant create the endstep unless a plugin is supplied.
After viewing the error in more detail I found that Sponge internally creates a none viewable inventory, however doesnt provide a plugin to the internal endstep (which I had provided a link directly to originally) which seems to be the cause of the crash

Mose is right, it was this commit that introduced the problem: https://github.com/SpongePowered/Sponge/commit/01a277c19e458303aad17515a9ab3d0530fa43e4


Update View and ViewPropTypes component

Fixes https://github.com/MLH-Fellowship/react-native/issues/192
Summary
The PR is part of an effort to update the code comments to match the current documentation on the React Native website. The project is a part of the MLH Fellowship program and involves the automatic generation of the website docs from code comments and flow types as the end result.
To learn more about the project you can visit the project wiki:

Project Details
RN Docs Standards

Link to the documentation(the source of truth):

view.md

Changes

Update the title and prop description from docs.
Remove unnecessary * from the code comments.
Add Snack player example specified in the docs to the code comments as JSDoc.
Add @type annotation to parse supported datatype by the prop.
Add @platform annotation to specify platforms supported by a prop.
Add @default annotation to parse default value of prop.

Changelog
[Internal]
Test Plan
All changes are made to the code comments and thus there is no need for testing.
Reviewed by @jevakallio

Thanks for this contribution, @ani4aniket, and apologies for not getting around to it until now. This MLH-Fellowship project has been abandoned so we're going to close this PR to focus on active ones. If you feel this is an error or if you'd like to pick this up again, please let me know!


.github/labeler.yml: autolabel jupyter stuff

Description of changes

will update the paths once
https://github.com/NixOS/nixpkgs/pull/244777
https://github.com/NixOS/nixpkgs/pull/171202
are merged
@natsukium @thomasjm
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Priorities
Add a :+1: reaction to pull requests you find important.

Oh, I just saw your feedback about moving things around in those PRs. Let me respond over there...


cos-proxy fails on hook "downstream-logging-relation-changed" because of SSL: CERTIFICATE_VERIFY_FAILED

Bug Description
n SQA testrun ce8325a0-c0fe-46f8-af40-acd7b287c8de, cos-proxy fails to install in hook "downstream-logging-relation-changed".
To Reproduce
To reproduce, deploy cos and then the charmed kubernetes bundle, which includes cos.
This issue seems to happen after Dec 13, after some cos update. This issue is not necessarily reproducible, we have seen this bundle deploy without this.
Environment
The environment is a juju maas controller hosting a charmed kubernetes deployment. This deployment is connected to cos, which is hosted on a microk8s, hosted on the same juju maas controller.
Relevant log output
2023-12-13 21:56:09 DEBUG juju.worker.uniter agent.go:22 [AGENT-STATUS] executing: running downstream-logging-relation-changed hook for cos-loki/0
2023-12-13 21:56:09 DEBUG juju.worker.uniter.runner runner.go:719 starting jujuc server  {unix @/var/lib/juju/agents/unit-cos-proxy-0/agent.socket <nil>}
2023-12-13 21:56:09 DEBUG unit.cos-proxy/0.juju-log server.go:325 downstream-logging:32: ops 2.8.0+8.g26c6e95 up and running.
2023-12-13 21:56:09 DEBUG unit.cos-proxy/0.juju-log server.go:325 downstream-logging:32: Emitting Juju event downstream_logging_relation_changed.
2023-12-13 21:56:09 DEBUG unit.cos-proxy/0.juju-log server.go:325 downstream-logging:32: Emitting custom event <VectorConfigChangedEvent via COSProxyCharm/VectorProvider[filebeat_downstream-logging]/on/config_changed[133]>.
2023-12-13 21:56:09 ERROR unit.cos-proxy/0.juju-log server.go:325 downstream-logging:32: Uncaught exception while in charm code:
Traceback (most recent call last):
  File "/usr/lib/python3.10/urllib/request.py", line 1348, in do_open
    h.request(req.get_method(), req.selector, req.data, headers,
  File "/usr/lib/python3.10/http/client.py", line 1283, in request
    self._send_request(method, url, body, headers, encode_chunked)
  File "/usr/lib/python3.10/http/client.py", line 1329, in _send_request
    self.endheaders(body, encode_chunked=encode_chunked)
  File "/usr/lib/python3.10/http/client.py", line 1278, in endheaders
    self._send_output(message_body, encode_chunked=encode_chunked)
  File "/usr/lib/python3.10/http/client.py", line 1038, in _send_output
    self.send(msg)
  File "/usr/lib/python3.10/http/client.py", line 976, in send
    self.connect()
  File "/usr/lib/python3.10/http/client.py", line 1455, in connect
    self.sock = self._context.wrap_socket(self.sock,
  File "/usr/lib/python3.10/ssl.py", line 513, in wrap_socket
    return self.sslsocket_class._create(
  File "/usr/lib/python3.10/ssl.py", line 1100, in _create
    self.do_handshake()
  File "/usr/lib/python3.10/ssl.py", line 1371, in do_handshake
    self._sslobj.do_handshake()
ssl.SSLCertVerificationError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1007)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/var/lib/juju/agents/unit-cos-proxy-0/charm/./src/charm.py", line 517, in <module>
    main(COSProxyCharm)
  File "/var/lib/juju/agents/unit-cos-proxy-0/charm/venv/ops/main.py", line 436, in main
    _emit_charm_event(charm, dispatcher.event_name)
  File "/var/lib/juju/agents/unit-cos-proxy-0/charm/venv/ops/main.py", line 144, in _emit_charm_event
    event_to_emit.emit(*args, **kwargs)
  File "/var/lib/juju/agents/unit-cos-proxy-0/charm/venv/ops/framework.py", line 340, in emit
    framework._emit(event)
  File "/var/lib/juju/agents/unit-cos-proxy-0/charm/venv/ops/framework.py", line 842, in _emit
    self._reemit(event_path)
  File "/var/lib/juju/agents/unit-cos-proxy-0/charm/venv/ops/framework.py", line 931, in _reemit
    custom_handler(event)
  File "/var/lib/juju/agents/unit-cos-proxy-0/charm/lib/charms/vector/v0/vector.py", line 227, in _on_log_relation_changed
    self.on.config_changed.emit(config=self.config)
  File "/var/lib/juju/agents/unit-cos-proxy-0/charm/venv/ops/framework.py", line 340, in emit
    framework._emit(event)
  File "/var/lib/juju/agents/unit-cos-proxy-0/charm/venv/ops/framework.py", line 842, in _emit
    self._reemit(event_path)
  File "/var/lib/juju/agents/unit-cos-proxy-0/charm/venv/ops/framework.py", line 931, in _reemit
    custom_handler(event)
  File "/var/lib/juju/agents/unit-cos-proxy-0/charm/./src/charm.py", line 395, in _write_vector_config
    r = request.urlopen(dest)
  File "/usr/lib/python3.10/urllib/request.py", line 216, in urlopen
    return opener.open(url, data, timeout)
  File "/usr/lib/python3.10/urllib/request.py", line 519, in open
    response = self._open(req, data)
  File "/usr/lib/python3.10/urllib/request.py", line 536, in _open
    result = self._call_chain(self.handle_open, protocol, protocol +
  File "/usr/lib/python3.10/urllib/request.py", line 496, in _call_chain
    result = func(*args)
  File "/usr/lib/python3.10/urllib/request.py", line 1391, in https_open
    return self.do_open(http.client.HTTPSConnection, req,
  File "/usr/lib/python3.10/urllib/request.py", line 1351, in do_open
    raise URLError(err)
urllib.error.URLError: <urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1007)>

Additional context
No response

An aspect of this issue would go away with #118 merged, but we'd need the recv-ca-cert relation to be able to talk with loki behind tls. Would need to render ca_file in vector config. Alternatively, use "insecureSkipVerify"  (ref).

The specific problem described in this issue is now gone with #118 merged.
Closing.


Allow toggle between directory structure view and "flat" view

Is your feature request related to a problem? Please describe.
Sometimes, I'd like to look recursively at a directory to see if any of the files at any level below are at a certain access tier. It's difficult to navigate through every directory to see every file.
Describe the solution you'd like
I'd like to be able to temporarily turn off the directory structure view and see all files by fully-qualified name (full path) on a single level so I can sort by Access Tier (or other column) and see all files that I need to modify.
Describe alternatives you've considered
Additional context

Duplicate of #19

@codefoster I'm going to ask you a question on the linked issue. Please try to answer there!


JavaScript Algorithms and Data Structures: Introducing Else Statements

Affected page

https://www.freecodecamp.org/learn/javascript-algorithms-and-data-structures/basic-javascript/introducing-else-statements
Your code

function testElse(val) {
  var result = "";
  // Only change code below this line

  if (val > 5) {
    result = "Bigger than 5";
  }

  if (val <= 5) {
    result = "5 or Smaller";
  }

  // Only change code above this line
  return result;
}

testElse(4);

Expected behavior

No error message of ""You should not change the code above or below the specified comments." in console as the code above and below the line has not been changed.
Screenshots

System

Device: Laptop
OS: iOs 11
Browser: Chrome
Version: 100.0.4896.88

Additional context

The error is shown when the code above and below the lines has not been changed and will also throw the same error when code is changed within the above/below boundary. This happens even when Reset All Code is hit.

Hi @lampenny thanks for submitting this report.  The reason the tests are complaining is that they expect to see let result = ""; above the first comment, not var result = "";
We changed this challenge about 6 months ago, so it's possible the old version (which used var) was saved in your browser.

This happens even when Reset All Code is hit.

I couldn't reproduce this, when I reset the code I get:
function testElse(val) {
  let result = "";
  // Only change code below this line

  if (val > 5) {
    result = "Bigger than 5";
  }

  if (val <= 5) {
    result = "5 or Smaller";
  }

  // Only change code above this line
  return result;
}

testElse(4);

Would you mind sharing the code you get after resetting?  If the reset isn't working, that's unexpected.

I am also unable to reproduce the Reset All Code bug.

Hi all, thanks for your help!! @ojeytonwilliams was right in that it should be let and this was merely a bug saved in my browser. As of today, when I tried the lesson again, the Reset All Code now works as it should.
Thanks again all.


Cache k8s versions: 1.19.13 and 1.20.9

Added patch versions to KUBE_BINARY_VERSIONS for caching. kube-proxy images are in kube-proxy-images.json.

@ganeshkumar5699  update the test as you did in this commit? https://github.com/Azure/AgentBaker/commit/304ad9410059f42721dcb0f2f68651dc18fb1dd0#diff-d6aeb8e845af4c32251915a17dc7c887d73cd628262e83118b9dcd65f7728c98

@ganeshkumar5699 update the test as you did in this commit? 304ad94#diff-d6aeb8e845af4c32251915a17dc7c887d73cd628262e83118b9dcd65f7728c98

Thank you. I have added the versions for the kube binaries tests there.


UNREACHEABLE, stock in the first chapter

So I have tried to run the very first ansible command from chapter one, ansible -i hosts-simple 192.###.##.##.nip.io -m ansible.builtin.setup, and I keep getting unreacheable. As you can see below, I tried a ping with the nip.io and it says unknown host but when I ping just the ip, I get ping back. I have tried to use the ansible command by removing the nip.io from both hosts files and still unreacheable. I have run to the same on Ubuntu 24.04 and macOS running the latest Ansible. Any help please?
❯ multipass info ansiblevm
Name:           ansiblevm
State:          Running
Snapshots:      0
IPv4:           <IP_ADDRESS>
Release:        Ubuntu 24.04 LTS
Image hash:     5fe06e10a3b5 (Ubuntu 24.04 LTS)
CPU(s):         1
Load:           0.00 0.00 0.00
Disk usage:     1.8GiB out of 4.8GiB
Memory usage:   197.8MiB out of 953.1MiB
Mounts:         --
/Volumes/techops/Learn-Ansible-Second-Edition/Chapter01 main                                                                                ansible9 10:11:00 PM
❯ ansible -i hosts-simple <IP_ADDRESS>.nip.io -m ansible.builtin.setup
<IP_ADDRESS>.nip.io | UNREACHABLE! => {
"changed": false,
"msg": "Failed to connect to the host via ssh: ssh: Could not resolve hostname <IP_ADDRESS>.nip.io: nodename nor servname provided, or not known",
"unreachable": true
}
/Volumes/techops/Learn-Ansible-Second-Edition/Chapter01 main                                                                                ansible9 10:11:14 PM
❯ ping <IP_ADDRESS>.nip.io
ping: cannot resolve <IP_ADDRESS>.nip.io: Unknown host
/Volumes/techops/Learn-Ansible-Second-Edition/Chapter01 main                                                                                ansible9 10:11:31 PM
❯ ping <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>): 56 data bytes
64 bytes from <IP_ADDRESS>: icmp_seq=0 ttl=64 time=0.564 ms
64 bytes from <IP_ADDRESS>: icmp_seq=1 ttl=64 time=0.629 ms
64 bytes from <IP_ADDRESS>: icmp_seq=2 ttl=64 time=0.692 ms
64 bytes from <IP_ADDRESS>: icmp_seq=3 ttl=64 time=0.961 ms
64 bytes from <IP_ADDRESS>: icmp_seq=4 ttl=64 time=0.693 ms
^C
--- <IP_ADDRESS> ping statistics ---
5 packets transmitted, 5 packets received, 0.0% packet loss
round-trip min/avg/max/stddev = 0.564/0.708/0.961/0.135 ms

Hey, thanks for reaching out - it sounds like, for some reason, the host you are using is unable to take advantage of the https://nip.io/ service - so you could try using an alternative service such as:

https://traefik.me/ (<IP_ADDRESS>.traefik.me)
https://sslip.io (<IP_ADDRESS>.sslip.io)

Try using the host command from your terminal to see if they resolve:

If you don't have any luck with those, then you can try updating the inventory file so it reads something like the following:
testhost ansible_host=<IP_ADDRESS> ansible_user=vmadmin ansible_private_key_file=./example_key

Russ

Thank you so much Russ. I can confirm that it was my firewall blocking nip.io. Also, I had to change permission of example_key to 600.
msg": "Failed to connect to the host via ssh: Warning: Permanently added '<IP_ADDRESS>.nip.io' (ED25519) to the list of known hosts.\r\n@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@\r\n@         WARNING: UNPROTECTED PRIVATE KEY FILE!          @\r\n@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@\r\nPermissions 0664 for './example_key' are too open.\r\nIt is required that your private key files are NOT accessible by others.\r\nThis private key will be ignored.\r\nLoad key "./example_key": bad<EMAIL_ADDRESS>Permission denied (publickey).",
"unreachable": true
B.T.W. with my Ubuntu laptop I get a 10.189.46.*** subnet as opposed to the 192.168.64.*** as I got with macOS.
Once again, thank you.

Hi,
I'm glad you got there, and I hope you enjoy the rest of the book,
Russ


Update README.md with improved formatting and grammar and instructions

Pull Request: Update README.md with improved formatting and grammar and instructions
Description
This pull request aims to enhance the README.md file for the react.dev repository by improving formatting, grammar, and providing additional instructions for contributors. The goal is to make the documentation more readable and user-friendly.
Changes Made

Improved overall formatting for better visual appeal.
Enhanced grammar and language for clarity and coherence.
Added or refined instructions in the README for a smoother onboarding experience for contributors.

Checklist

[x] Updated README.md with improved formatting and grammar
[x] Added or clarified instructions for contributors
[x] Tested the changes locally
[x] Ensured the changes align with the contributing guidelines

This pull request is part of an ongoing effort to enhance the documentation quality and contribute to a more user-friendly development environment.

Thanks, but I prefer the existing version, it matches our tone more.


[ MB-15435 ] Generating UUIDs to avoid collisions

Jira ticket
Summary
This was first addressed in #11033, but while reviewing #11187 I ran into a UUID
collision again. I've posted in #g-testing🔒. I'm looking for some
feedback here to see if this fix works for all the other devseed scenarios.
I'll be doing further testing on Monday and will document on this PR what I
find. I didn't include this as part of #11187 so that I don't hold up that work.
Verification Steps for Reviewers
Run the E2E Populate command in the repository and have it exist successfully.
How to test

Run make db_dev_e2e_populate.
Have a successful run of that command and verify that devseed data
scenarios aren't broken.

Messages

:book:
:link: MB-15435

  Generated by :no_entry_sign: dangerJS against 25028d766096ee803160408f52ee127c16f21433


Enforce checkstyle in the pulsar sql module

Modifications
The checksyle plugin was added to the pulsar sql module to enforce the defined style. All violations were fixed:

Ordering of imports.
Formatting of the code.
Absent Javadoc comments.
Other small issues.

// flaky test #4884
run cpp tests


Adding launcher

Being able to launch the node would be perfect to remap the topics to the ones existing in the external software.

Solved in commit 98eb02803bbbee2eab6d8aac93181dbc31ed56e6.