Possibility to export the differential equations

Hello,
Is it possible to, given a circuit topology, export the set of differential equations? I think it could be done using ModelingToolkit.jl. This would be useful to solve the system in the time domain or just for having more control over solving the system.
Moreover, looking rapidly at the code, it seems that hbsolve generates some matrices, and not a system of differential equations. Is there a reference for that?

Thanks for your question. Given a circuit topology and component values (either numeric or symbolic), the code computes the sparse capacitance, conductance, inductance, inverse inductance, and admittance (graph admittance not electrical engineering admittance) matrices and the vector of Josephson inductances. A reference in the code is https://github.com/kpobrien/JosephsonCircuits.jl/blob/main/src/capindmat.jl and an article reference is https://arxiv.org/abs/2010.14929 .
For example, we can look at the matrices for the fluxed pumped JPA example
using JosephsonCircuits
@variables R Cc Cj Lj Cr Lr Ll Ldc K Lg
circuit = [
    ("P1","1","0",1),
    ("R1","1","0",R),
    # a very large inductor so the DC node flux of this node isn't floating
    ("L0","1","0",Lg),
    ("C1","1","2",Cc),
    ("L1","2","3",Lr),
    ("C2","2","0",Cr),
    ("Lj1","3","0",Lj),
    ("Cj1","3","0",Cj),
    ("L2","3","4",Ll),
    ("Lj2","4","0",Lj),
    ("Cj2","4","0",Cj),
    ("L3","5","0",Ldc),
    ("K1","L2","L3",K),
    # a port with a very large resistor so we can apply the bias across the port
    ("P2","5","0",2),
    ("R2","5","0",1000.0),
]

circuitdefs = Dict(
    Lj =>219.63e-12,
    Lr =>0.4264e-9,
    Lg =>100.0e-9,
    Cc => 16.0e-15,
    Cj => 10.0e-15,
    Cr => 0.4e-12,
    R => 50.0,
    Ll => 34e-12,
    K => 0.999, # the inverse inductance matrix for K=1.0 diverges, so set K<1.0
    Ldc => 0.74e-12,
)
nm = numericmatrices(circuit,circuitdefs)

Looking at the matrices
julia> nm.Cnm
5×5 SparseArrays.SparseMatrixCSC{Float64, Int64} with 6 stored entries:
  1.6e-14  -1.6e-14    ⋅        ⋅        ⋅
 -1.6e-14   4.16e-13   ⋅        ⋅        ⋅
   ⋅         ⋅        1.0e-14   ⋅        ⋅
   ⋅         ⋅         ⋅       1.0e-14   ⋅
   ⋅         ⋅         ⋅        ⋅        ⋅

julia> nm.Gnm
5×5 SparseArrays.SparseMatrixCSC{Float64, Int64} with 2 stored entries:
 0.02   ⋅    ⋅    ⋅    ⋅
  ⋅     ⋅    ⋅    ⋅    ⋅
  ⋅     ⋅    ⋅    ⋅    ⋅
  ⋅     ⋅    ⋅    ⋅    ⋅
  ⋅     ⋅    ⋅    ⋅   0.001

julia> nm.invLnm
5×5 SparseArrays.SparseMatrixCSC{Float64, Int64} with 13 stored entries:
 1.0e7    ⋅           ⋅            ⋅            ⋅
  ⋅      2.34522e9  -2.34522e9     ⋅            ⋅
  ⋅     -2.34522e9   1.47156e13  -1.47132e13   9.96317e13
  ⋅       ⋅         -1.47132e13   1.47132e13  -9.96317e13
  ⋅       ⋅          9.96317e13  -9.96317e13   6.76014e14

julia> nm.Rbnm
6×5 SparseArrays.SparseMatrixCSC{Int64, Int64} with 8 stored entries:
 1   ⋅   ⋅  ⋅  ⋅
 ⋅   ⋅   1  ⋅  ⋅
 ⋅   ⋅   ⋅  1  ⋅
 ⋅   ⋅   ⋅  ⋅  1
 ⋅  -1   1  ⋅  ⋅
 ⋅   ⋅  -1  1  ⋅

julia> nm.Ljb
6-element SparseArrays.SparseVector{Float64, Int64} with 2 stored entries:
  [2]  =  2.1963e-10
  [3]  =  2.1963e-10

We use those matrices to generate the function and Jacobian and solve the nonlinear system of equations. The code evaluates the Josephson nonlinearity in the time domain in the branch flux basis and solves the linearized system in the frequency domain in the node flux basis.
Using those matrices  to solve for the time domain dynamics using something like DifferentialEquations.jl or ModelingToolkit.jl should work although I have not tried it. Is that something you are interested in trying? How large of a system do you have in mind? Does it involve Josephson junctions or just linear circuit elements?

Hi @kpobrien,
Thank you for clarifying that. The possible application of ModelingToolkit would be to directly integrate this system with DifferentialEquations.jl for example. But there are many other applications, like, e.g., symbolic calculation.
Finally, in my systems I take into account also Josephson junctions. They might be of simple topology or more complex ones as well.


Specify more restrictive dependencies on Rails

Since draper 2.x does not support Rails 3.2 anymore, shouldn't we make the dependencies more restrictive?
Or is this on purpose since draper might work with Rails 3.2 too?

The dependencies have since been updated.


[fix] Fixed account theme

@nemesisdesign I have fixed the template issue for the account page.
Depends on
https://github.com/openwisp/openwisp-utils/pull/226

Coverage remained the same at 97.393% when pulling 8b76e121f62e6105902cb6960b7287a07441f6d2 on fix/account-theme-template into 732b7189f2fc80f2681924528f6ab285c310eb62 on master.


v3: Helm integration w/ Cluster Registry API

Currently when a Helm package needs to be applied to multiple clusters..
Multiple statements need to be run.
kubectl config use-context ...
helm install ...
kubectl config use-context ...
helm install ...
...

Or
Multiple independant processes need to run against N clusters
Cluster 1
helm install ...

...
Cluster N
helm install ...

Once the Cluster Registry API is available then the information stored about each cluster could be used to apply a package across a set of clusters.
Possibly this could be something like...
helm install --clusters -l environment=test

Is this in scope for Helm v3? Is there a design doc for it?
A somewhat separate topic to cover perhaps in the design doc
Should this functionality be implemented client side or server side?
a few of my opinions on this...
Client side properties

simple to implement
follows the same pattern of tiller everywhere, hence highly distributed

Server side properties

tiller could become a clustered component in a few locations instead of deployed everywhere
independent tools could easily plug into the server side components (muck like kubectl apply moving server side)
orchestration of deployments across the various target clusters
central logging\auditing

cc @michelleN as requested from our discussion at SIG-APPS at KubeCon.


Field select2_multiple problems when using Laravel 5.2 Model with custom Join/Pivot Table

Hey, I might be using this wrong... but I didn't see in documentation where to get this to work.
Documentation: https://laravel-backpack.readme.io/docs/crud-fields#select2_multiple-n-n-relationship
I'm using a custom Pivot Table for many-to-many relationship (https://laravel.com/docs/5.2/eloquent-relationships#many-to-many).
Challenge.php model:
...
    public function teams()
    {
        return $this->belongsToMany('App\Team', 'challenge_team', 'challenge_id', 'team_id')->withTimestamps();
    }
...

_ChallengeCrudController.php :_
...
        $this->crud->addField(
            [       // Select2Multiple = n-n relationship (with pivot table)
                'label' => "Teams Involved",
                'type' => 'select2_multiple',
                'name' => 'teams', // the method that defines the relationship in your Model
                'entity' => 'teams', // the method that defines the relationship in your Model
                'attribute' => 'team_name', // foreign key attribute that is shown to user
                'model' => "App\Team", // foreign key model
                'pivot' => true, // on create&update, do you need to add/delete pivot table entries?
            ]
        );
...

Doing this... I was able to select and SAVE NEW Challenges with multiple Teams, but I was unable to EDIT and see the previously selected Teams. It did persist to database... but didn't render them in view properly.
I did the following to make it work:
_ChallengeCrudController.php :_
...
        $this->crud->addField(
            [       // Select2Multiple = n-n relationship (with pivot table)
                'label' => "Teams Involved",
                'type' => 'select2_multiple',
                'name' => 'teams', // the method that defines the relationship in your Model
                'entity' => 'teams', // the method that defines the relationship in your Model
                'attribute' => 'team_name', // foreign key attribute that is shown to user
                'model' => "App\Team", // foreign key model
                'pivot' => true, // on create&update, do you need to add/delete pivot table entries?
                'foreign_pivot_key' => 'team_id'
            ]
        );
...

Then added the foreign_pivot_key to select2_multiple.blade.php file:
...

    {{ $field['label'] }}
    <?php
    $field['foreign_pivot_key'] = !empty($field['foreign_pivot_key']) ? $field['foreign_pivot_key'] : 'id';
    ?>
    <select
        class="form-control select2"

    @foreach ($field as $attribute => $value)
        @if (is_string($attribute))
            @if ($attribute=='name')
                {{ $attribute }}="{{ $value }}[]"
            @else
                {{ $attribute }}="{{ $value }}"
            @endif
        @endif
    @endforeach
    multiple>
    <option value="">-</option>

    @if (isset($field['model']))
        @foreach ($field['model']::all() as $connected_entity_entry)
            <option value="{{ $connected_entity_entry->getKey() }}"
                    @if ( (isset($field['value']) && in_array($connected_entity_entry->getKey(), $field['value']->lists($field['foreign_pivot_key'])->toArray())) || ( old( $field["name"] ) && in_array($connected_entity_entry->getKey(), old( $field["name"])) ) )
                    selected
                @endif
            >{{ $connected_entity_entry->{$field['attribute']} }}</option>
            @endforeach
            @endif
            </select>

...

Let me know if that's a good work-around.
LOVE THE BACKPACK packages!!!

Hi @BinaryBlock ,
Thank you for the report and solution and sorry to be so late to reply.
I think this condition might work too, with the added benefit of not having to specify the "foreign_pivot_key":
@if ( (isset($field['value']) && in_array($connected_entity_entry->getKey(), $field['value']->lists($connected_entity_entry->getKeyName(), $connected_entity_entry->getKeyName())->toArray())) || ( old( $field["name"] ) && in_array($connected_entity_entry->getKey(), old( $field["name"])) ) )

Basically instead of telling the field what the foreign pivot key is, we get it from the object.
Could you please try it when you have the time and tell me if it works for you too? If so, I'll push the update immediately.
Thank you!

Below is the full modified select2_multiple.blade.php so it's easier for you to test:

    {{ $field['label'] }}
    <select
        name="{{ $field['name'] }}[]"
        @include('crud::inc.field_attributes', ['default_class' =>  'form-control select2'])
    	multiple>
    	<option value="">-</option>

    	@if (isset($field['model']))
    		@foreach ($field['model']::all() as $connected_entity_entry)
    			<option value="{{ $connected_entity_entry->getKey() }}"
					@if ( (isset($field['value']) && in_array($connected_entity_entry->getKey(), $field['value']->lists($connected_entity_entry->getKeyName(), $connected_entity_entry->getKeyName())->toArray())) || ( old( $field["name"] ) && in_array($connected_entity_entry->getKey(), old( $field["name"])) ) )
						 selected
					@endif
    			>{{ $connected_entity_entry->{$field['attribute']} }}</option>
    		@endforeach
    	@endif
	</select>

    {{-- HINT --}}
    @if (isset($field['hint']))

{!! $field['hint'] !!}

    @endif

{{-- ########################################## --}}
{{-- Extra CSS and JS for this particular field --}}
{{-- If a field type is shown multiple times on a form, the CSS and JS will only be loaded once --}}
@if ($crud->checkIfFieldIsFirstOfItsType($field, $fields))

    {{-- FIELD CSS - will be loaded in the after_styles section --}}
    @push('crud_fields_styles')

        <link href="{{ asset('vendor/backpack/select2/select2.css') }}" rel="stylesheet" type="text/css" />
        <link href="{{ asset('vendor/backpack/select2/select2-bootstrap-dick.css') }}" rel="stylesheet" type="text/css" />
    @endpush

    {{-- FIELD JS - will be loaded in the after_scripts section --}}
    @push('crud_fields_scripts')

    @endpush

@endif
{{-- End of Extra CSS and JS --}}
{{-- ########################################## --}}

Could you please try it when you have the time and tell me if it works for you too? If so, I'll push the update immediately.

Ahhh.... you rock!
Yes, it worked great. Smart idea about grabbing pivot info from object.
Push away.

Thank you!

Hi, I have the same problem, but when I make the change you say I throw an error that the method list does not exist, I am using laravel 5.4

hi,I have same problem, but I use   $this->crud->addField()  method ,please help

I want show filed to list
and update selected

@LinLin520 shouldn't you alse set pivot => true in field definition ?

@LinLin520 shouldn't you alse set pivot => true in field definition ?

Thank you ,I've settled it. it's my jonin table  mistake
CampaignsCrudController .php
// ------ CRUD FIELDS --- Medias ---
$this->crud->addField([
'name' => "medias",
'label' => "Media Type",
'type' => 'select_multiple',
'entity' => 'medias',
'attribute' => 'name',
'model' => "App\Models\Medias",
'pivot' => true,
]);
    // ------ CRUD COLUMNS --- Medias ---
    $this->crud->addColumn([
        'name' => "medias",
        'label' => "Media Type",
        'type' => "model_function",
        'function_name' => 'getMediaName',
    ]);

Campaigns.php
/*
|--------------------------------------------------------------------------
| FUNCTIONS
|--------------------------------------------------------------------------
*/
public function getMediaName()
{
$name = '';
foreach ($this->medias as $val){
$name .= $val->name .',';
}
return substr($name,0,-1);
}
/**
 * Get the Media for the Db Medias
 */
public function Medias(){
    return $this->belongsToMany('App\Models\Medias','campaign_medias','campaign_id','media_id');
}

list show

@tabacitu Does this works with backpack 3.5? select2_multiple.blade.php doesn't look the same.
It's a nice feature, that I will love to use.

@jrbecart  yes, it should work just fine - the fix was pushed back in 2016 ;-) https://github.com/Laravel-Backpack/CRUD/commit/33d41b345439c262bf5832b18ef192ff42f77fe1 You can use any primary key column you want, 'id' is no longer hardcoded anywhere in Backpack.
Cheers!

Hello, thank for for sharing. It is working fine but I don't see the selected options.

Any ideas?

Sorry @elenadanevska - this is a thread from 2016, last update 2019. The code has changed a lot since then - any problem that sounds like this one is probably not this one 😀
If you think you've discovered a bug in Backpack please open another issue with enough details for us to reproduce. Feel free to reference this one if you believe it's indeed related or the same.
If you're having an implementation issue (something custom you're doing isn't working) please ask a question on StackOverflow with the backpack-for-laravel tag.
Cheers!


Flashcards test of current file does not work

Hi,
when I click on the "brain" icon in the top right toolbar, Recalls uses all md-files in the Folder for the test. The tooltip instructs to press Alt when clicking on the icon to test only with the current file. But that does not work, I only see "No more cards to review. Well done!" although there are flashcards defined in the current file.
Is this a bug or am I doing something wrong?
And thanks for the extension :)!

Hi,
the only reason I can think of is that all of the cards in the current file have been reviewed and are not due for next review until some day in the future. Hence the "no more cards to review".
You can easily check that by hovering your mouse over the card badge (NEW, GOOD, HARD, etc.) in the source file. You should see the date of the next review in the tooltip.
Let me know if the problem persists.
And you're welcome :)

I'm afraid that isn't the reason, because I even deleted the .recall directory and the extension gives me the same message.

Hi. This is really strange. I checked the code again and there is no apparent reason why the cards wouldn't show. That is if they are really due and do show in the full review. Sorry.
If you can, please reload the window to be on the safe side and try the file review again.
If it still doesn't work, could you send me the full path of the file you are trying to review? Right click on the tab and "Copy path". Thank you.

I just tested it with a new install of vs codium just in case some extensions in vs code don't play well with each other. But there is the same issue. I put this simple file
---
recall: header
---

# This is a question

This is the answer

into C:\Users\j-hap\TEST\questions.md and ran the Recall: Flashcards Test of current file command, but to no avail. The Recall: Flashcards Test works fine.
I think it's worth mentioning that I also don't see the card badges next to the header in the editor, but I do get a popoup when hovering over the question with the mouse, but that might be a separate issue:

Ok. Thanks for providing the additional details. I will look into the issue, but cannot make any promises since I currently have no clue what might be causing it ... :/

So after my first hacky debugging-a-ts-vs-code-extension-experience I think I found the issue: in startFileReview you call openWebview(doc.uri.fsPath); and on windows the file path is build with escaped backslashes, but in getFilesData the paths of the available files is given with unix-like forward slashes. The filter function then filters out all available files, because with the / separator nothing matches the expected file path separated by \\, which in turn returns an empty dataset, which seems as if there are no cards left to train on. I hope that was understandable...

regarding the missing badge next to the question: I see an error message in the devtools about a forbidden local resource load: workbench.desktop.main.js:660 Not allowed to load local resource: file:///C:/Usersj-hap.vscode-oss%0Extensions%0Frenya.vscode-recall-0.11.5resourcesicons%E0%AE%ADge_NEW_13.svg

Hi. Thanks for the extensive diagnosis. Being on Mac myself, I was afraid that the issue has to do with specifics of Windows :/
Anyway. I have set up a windows environment now and was able to recreate the issue. I'll will let you know as soon as I manage to resolve it.

Fix for both issues was published in version v0.13.0

Thanks! I got the update and it works as expected now. Small mistake in the changelog, you probably meant to write 2022-09-09
Will you push the updated code to Github?

Great. Thanks for catching the changelog typo.
Yes, I will push the update to GitHub soon.


Editor crashes when input data pin is dragged for suggestions

Happens because of the generics resolver

Should be fixed in 0.0.3 since we don't use the old generics resolver anymore.


From NumPy to .tre file

from tubetk._tubetk_numpy import tubes_from_file
allows reading .tre in a NumPy array. Is there a way to perform other way around i.e. save a properly structured NumPy array in a .tre file?

Hi,
That is an interesting idea.    The file structure is fairly
straightforward, so you could write it out as a text file, but perhaps I
can figure something out to do as you requested.   I'm a bit swamped
keeping up with the rapid pace of ITK development so that ITKTubeTK stays
compatible with ITK head, but if I get some free time...
Thanks for the suggestion!
Stephen
On Sat, Feb 29, 2020 at 4:16 PM suprosanna<EMAIL_ADDRESS>wrote:

from tubetk._tubetk_numpy import tubes_from_file
allows reading .tre in a NumPy array. Is there a way to perform other way
around i.e. save a properly structured NumPy array in a .tre file?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/KitwareMedical/ITKTubeTK/issues/1152?email_source=notifications&email_token=AACEJLZOERR4B7R2ZWDXFT3RFF5LTA5CNFSM4K66L3DKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IRNSYBA,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACEJL3R47TF6GHEP6ORRFTRFF5LTANCNFSM4K66L3DA
.

--
Stephen R. Aylward, Ph.D.
Senior Director of Strategic Initiatives
Kitware: Advancing the frontiers of understanding
by developing innovative open-source software platforms
and integrating them into research, processes, and products.


How to update changes in my_configs.vim or newly installed plugins?

I have tried with source ~/.vimrc, but it did go wrong.

The best way is usually just to restart Vim. That's what I do 😅


Clarify statements with regards to private endpoints for event grid topics

[Enter feedback here]
Currently, it's not possible to deliver events using private endpoints. That is, there is no support if you have strict network isolation requirements where your delivered events traffic must not leave the private IP space.

I have multiple function apps using private endpoints and trying to communicate over an event grid. I have tried both a public event grid topic, and one using private endpoint (in the same VNET as the function apps with private DNS zones set up).
From the quoted statement, I would assume that since my private endpoint Event Grid Topic and my Azure Functions connected with Private Endpoints are both connected to the same VNET, that traffic does not leave the private IP space, and should work
I am able to publish events on the event grid without issue, but the Azure Monitor metrics show "Delivery Failed Events"
Seing the dead letter message
...
    "deadLetterReason": "UndeliverableDueToClientError",
    "deliveryAttempts": 2,
    "lastDeliveryOutcome": "Forbidden",
...

I am not able to find explicit documentation for this, but it would seem that using function apps delivery endpoint does not work if your target function apps are using private endpoints. Is this possible?
The diagram and the section "Use managed identity" is also confusing. From my understanding, this is meant a workaround to this exact issue is to route the traffic from event grid via service bus or event hubs --> Azure function with private endpoint.
See also related issue https://github.com/MicrosoftDocs/azure-docs/issues/66112

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

ID: d904c827-7efa-59aa-9639-12493e7e8061
Version Independent ID: e2c8c71f-52d4-b25b-ed75-892744d243e4
Content: Deliver events using private link service - Azure Event Grid
Content Source: articles/event-grid/consume-private-endpoints.md
Service: event-grid
GitHub Login: @spelluru
Microsoft Alias: spelluru

@audunsolemdal Thank you for your feedback . We will investigate and update the thread.

@audunsolemdal , Delivery will fail to a private endpoint linked function app. You will have to integrate a Eventhub/Service bus/Storage in between the Event grid and function app to achieve private network communication via managed endpoint
https://docs.microsoft.com/en-us/azure/event-grid/consume-private-endpoints#use-managed-identity

@audunsolemdal , I am not aware of any plans of this implementation. I would recommend you to post your feedback here as a feature request.
I am closing this issue for now. Feel free to reopen if you need any details


fix: targets specified twice

Change Summary
Fix a very small 🐛 in the docs.
Checklist

[X] Tests for the changes exist and pass on CI
[X] Documentation reflects the changes where applicable
[X] Change is documented in CHANGELOG.md (if applicable)
[X] My PR is ready to review

Thank you!


Fix accessibility

improve accessibility :

replace link by buttons
set attribute disabled when button are not available
manage aria-current=step on step element

Fixes: #6, #8

Thank you very much @Lausselloic 👍


fix(blockifier_reexecution): class hash in RPC request must be hex

This change is

crates/blockifier_reexecution/src/state_reader/rpc_https_test.rs line 70 at r2 (raw file):
Previously, dorimedini-starkware wrote…
the type of RPC_NODE_URL is now LazyLock<String> and not &str, so *RPC_NODE_URL is already a String. no need to convert, but clone is needed because String is not Copy

But the compiler suggests to_string; why is clone preferable here?

crates/blockifier_reexecution/src/state_reader/rpc_https_test.rs line 70 at r2 (raw file):
Previously, dorimedini-starkware wrote…

But the compiler suggests to_string;

where do you see this suggestion..? clippy is green

If you remove the clone, that's what the compiler suggests. I'm asking what's the advantage of clone over to_string?


Using Bot Framework State Service?

Is this project using the Bot Framework State Service? Looking at the code, it seems like the AuthDialog and CallbackController are using it to pass back and forth the authentication values. Since the State Service is now deprecated, won't that cause problem using this code in production?  Or am I mistaken?

Got it, thanks.


DOC: Fix typos in IIR design argument documentation

Reference issue

What does this implement/fix?
A type in the IIR design documentation that used == instead of = when explaining a specific argument, copy-pasted across several functions.
Additional information

I don't see the mistake here, since == is a truth test not an assignment. if 4=2 is not the correct syntax pythonically speaking (if that's a word).

I don't see the mistake here, since == is a truth test not an assignment. if 4=2 is not the correct syntax pythonically speaking (if that's a word).

That's fair, however then I'd argue the docstring of the other variants should be adapted for consistency. For reference, this is the current state:
    b, a : ndarray, ndarray
        Numerator (`b`) and denominator (`a`) polynomials of the IIR filter.
        Only returned if ``output='ba'``.
    z, p, k : ndarray, ndarray, float
        Zeros, poles, and system gain of the IIR filter transfer
        function.  Only returned if ``output='zpk'``.
    sos : ndarray
        Second-order sections representation of the IIR filter.
        Only returned if ``output=='sos'``.

I had the same initial reaction as @ilayn, but then I realized the expression can also be interpreted as specify the argument that was a given to the function.  In that case, the single = makes sense.  FWIW, I think I prefer the single =.

I can only agree. Whichever is preferred let's switch to that throughout the whole suite.

Is there anything I can do to move this forward? I'm not very familiar with the other sub-packages but if preferred I can look into updating all occurrences of x == y in the docstrings

My vote goes for =. It seems more natural to me as it's how you would set the params.

FWIW: After making my comment above, I vaguely recalled having exactly this hesitancy about = vs == in the past, and I seemed to recall avoiding the issue entirely by saying something like
if `output` is `'ba'`

where "is" is the English verb and not the Python keyword, so it is not in any markup.  I might have used this gimmick in other places, but the one case that I could find is in the docstring for scipy.linalg.polar.
Having said that, I think the single = is fine.

The single = also seems natural to me (although personally I'd take the same approach as @WarrenWeckesser (but this would be too much trouble if we want to change all of those occurrences!)


Time Out method

Hi, I am trying to extend time out on my node application for SAP using this Module, but i couldnt find any. Can you expose Timeout function?

As far as I can see there is no way to change the timeout value for RFC calls from outside SAP.
To get around this, you could to set the profile parameter rdisp/max_wprun_time of this server to a higher value. But this would affect all dialog processes - which might be a bad idea.
Another way would be to periodically call COMMIT WORK inside your function module which effectively resets the timeout counter. But this is also a bad solution which might have really bad side effects when you are not just reading from database.

Joachim Dorner is right. We looked at external timeout as part of our service patterns. We had to make some fairly big enhancements to enable the consumer to provide the timeout time from outside and you will have to enable idempotency etc. If this is critical for your business please let me know. I can get you in touch with the guy who did it for us. Cheers, Nils

Thanks jdorner & NilsFrkal for your reply.
Sorry i couldnt check my emails i was OOO.
In SAP we have increased timeout but our SAPNWRFC is getting timeout in 2 mins. It is a critical business for us. It would be helpful if you can let me know what needs to be done.


chore(deps): update module github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper to v1.0.19

This PR contains the following updates:

Package
Type
Update
Change

github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper
require
patch
v1.0.18 -> v1.0.19

Release Notes

terraform-ibm-modules/ibmcloud-terratest-wrapper
v1.0.19
Compare Source
Bug Fixes

use git hash for upgrade test checkouts (#109) (0108c16)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, click this checkbox.

This PR has been generated by Renovate Bot.

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

GitHub release
v1.7.1

Your semantic-release bot :package::rocket:


Missing Dependency for github.com/kingparks/cursor-vip/auth

Hi,
I noticed that the project is missing the dependency for github.com/kingparks/cursor-vip/auth, which appears to be required for certain parts of the code to compile correctly. Could you please check if this package was unintentionally excluded from the repository or provide guidance on how to resolve this?
Thank you for your attention to this issue, and I look forward to your feedback.
Best regards,

这是来自QQ邮箱的假期自动回复邮件。

您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。


Should namespaces follow folder structure?
Input Adapter probably needed in GB.
Should game region store a reference to its window?
Should our sprites be able to construct from a position? Even though SFML cant?
Can we simplify construction of animated sprite?

SetCurrentFrame doesn't work

RunAnimation in Reverse
Priority drawing
SetActiveRegion stuff should be auto magic
Set animation without running

accept Keyboard and Mouse events simultaneously
accept keyboard movement ever stopping the character


xml-security-c: 2.0.2 -> 2.0.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/xml-security-c/versions.
meta.description for xml-security-c is: "C++ Implementation of W3C security standards for XML"
meta.homepage for xml-security-c is: "http://santuario.apache.org/"
meta.changelog for xml-security-c is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
Warning: no invocation of /nix/store/psx2h6a2j1g9xsi5lqxk2nkf8k001z6r-xml-security-c-2.0.4/bin/xsec-siginf had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/psx2h6a2j1g9xsi5lqxk2nkf8k001z6r-xml-security-c-2.0.4/bin/xsec-txfmout had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/psx2h6a2j1g9xsi5lqxk2nkf8k001z6r-xml-security-c-2.0.4/bin/xsec-checksig had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/psx2h6a2j1g9xsi5lqxk2nkf8k001z6r-xml-security-c-2.0.4/bin/xsec-cipher had a zero exit code or showed the expected version
/nix/store/psx2h6a2j1g9xsi5lqxk2nkf8k001z6r-xml-security-c-2.0.4/bin/xsec-xtest passed the binary check.
Warning: no invocation of /nix/store/psx2h6a2j1g9xsi5lqxk2nkf8k001z6r-xml-security-c-2.0.4/bin/xsec-templatesign had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/psx2h6a2j1g9xsi5lqxk2nkf8k001z6r-xml-security-c-2.0.4/bin/xsec-c14n had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/psx2h6a2j1g9xsi5lqxk2nkf8k001z6r-xml-security-c-2.0.4/bin/xsec-xklient had a zero exit code or showed the expected version
1 of 8 passed binary check by having a zero exit code.
0 of 8 passed binary check by having the new version present in output.
found 2.0.4 with grep in /nix/store/psx2h6a2j1g9xsi5lqxk2nkf8k001z6r-xml-security-c-2.0.4
found 2.0.4 in filename of file in /nix/store/psx2h6a2j1g9xsi5lqxk2nkf8k001z6r-xml-security-c-2.0.4

Rebuild report (if merged into master) (click to expand)

12 total rebuild path(s)

7 package rebuild(s)

7 x86_64-linux rebuild(s)
5 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
docbookrx
libdigidocpp
opensaml-cpp
qdigidoc
shibboleth-sp
xml-security-c
xml-tooling-c

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/psx2h6a2j1g9xsi5lqxk2nkf8k001z6r-xml-security-c-2.0.4 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A xml-security-c https://github.com/r-ryantm/nixpkgs/archive/103e0fd2b96307af5717dfdbfa3af2a0fdf60509.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/psx2h6a2j1g9xsi5lqxk2nkf8k001z6r-xml-security-c-2.0.4
ls -la /nix/store/psx2h6a2j1g9xsi5lqxk2nkf8k001z6r-xml-security-c-2.0.4/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

6 packages built:

libdigidocpp
opensaml-cpp
qdigidoc
shibboleth-sp
xml-security-c
xml-tooling-c

Maintainer pings
cc @jagajaga for testing.

Version 2.0.4 fixes a regression in 2.0.3 allowing the code to build on pre-1.1 OpenSSL versions.
Version 2.0.3 adds support for OpenSSL 3.0.0, though using a number of now-deprecated function calls.
Tested running xsec-xtest all tests passed, build result looks good to me


WIP | Refactor existing parser using typescript

@Midnighter this is work in progress(needs a lot of work and I am working on it).

The work seems in progress indeed and only small changes to the original so far. You have chosen to start with the perhaps the most complicated part of the application. Maybe we should pair up on changing it?

Yeah, would love to pair up.


[MM-41158] Remove white circles from App Bar icons

Summary
This PR makes it so the icons in the App Bar no longer have a white circle outline. The white circle still applies to the fallback icon. Please see the example screenshots below.
Ticket Link
Fixes https://mattermost.atlassian.net/browse/MM-41158
Screenshots

Before:

After:

Indigo
Before:

After:

Onyx
Before:

After:

Release Note
NONE

@mickmister how do I turn on the apps bar again?

Hi @matthewbirtch, you can use this cloud server to test here https://github.com/mattermost/mattermost-webapp/pull/9680. I've created an item in the developer's meeting to discuss how we should be using feature flags in the open PR testing servers.

@matthewbirtch Whoops I meant to post this link https://app-bar-toggle.test.mattermost.cloud/

@matthewbirtch Whoops I meant to post this link https://app-bar-toggle.test.mattermost.cloud/

Perfect. Thanks @mickmister! This looks a lot better without the white borders.

@DHaussermann This is ready for testing

@DHaussermann With the App Bar being emphasized in MM v7 soon, I think we need to get this tested

@DHaussermann I'm re-posting my comment above

With the App Bar being emphasized in MM v7 soon, I think we need to get this tested

@agarciamontoro I'm wondering if we should test this with your PR https://github.com/mattermost/mattermost-webapp/pull/10311, since the styling would affect the non-controlled icons being moved from the channel header

/update-branch

@agarciamontoro I'm wondering if we should test this with your PR #10311, since the styling would affect the non-controlled icons being moved from the channel header to the App Bar

It probably will. We can merge this PR and then I can merge master and check, if that works for you.

@mickmister I had issues deploying this code Friday. The test spin-up Seems to have App Bar disabled? As per Split IO it should be on.
I tried before and after pulling in master to use /cloud but the tool says it can't find the corresponding Docker tag.
We could do a quick sanity check if there was an easy way to deploy. Thoughts?

This PR was deployable on a server with App Bar enabled after the last pull-in from master 👍

/cherry-pick release-7.0

Cherry pick is scheduled.


Cohort wizard design

Wooo! Pretty minimal updates since your work was pretty true to the Figma designs! This PR makes some slight adjustments:

A bit of margin around the cohort share url step
Creating action buttons that stick to the bottom of the page on the scenario selection (so they're not cut off from view on first glance
Slight styling and alignment of roles on the scenario selection card

Merged!

Merged!


Make bionetgen.modelapi.structs.Action.args a dictionary

Unless arguments can be repeated, it would be helpful for this to be a dictionary rather than a list of key-value tuples.

This and #16 will be resolved shortly, will update very soon.

Fixed with 0.4.6 release

FYI, this is still a list of tuples in 0.4.6. This representation is fine. Users can easily be convert it to a dictionary if needed.

I agree with the initial suggestion. I'll make 0.4.7 release today and it'll be fixed with it.

Should be fixed with 0.4.7


Updated doc to remove abseil-cpp from ROS

I have added a few lines to the doc to alert users to remove ros abseil-cpp and also to change the rosinstall file if they wish to checkout a different version of cartographer. As discussed in #1208

@wally-the-cartographer merge

Merge requested by authorized user pifon2a. Merge queue now has a length of 1.


Improve checks for existing report

Description

Frontend:

Reset all component state on modal closure so that the error triggers properly on re-opening

Backend:

Add check to create flow that ensures unarchived report in same year and period does not exist
Add tests

Related ticket(s)

CMDCT-3985

How to test

(if testing locally, use yarn db:seed to create approved WP)
https://d1vtt2axjhiobw.cloudfront.net/

Log in as<EMAIL_ADDRESS>Go to WP dashboard
Open "reset" modal
Verify alert shows as expected (when year and period match existing report)
Close and re-open the modal multiple times. Make sure it still works as expected
Open the network tab
Create a reset report in a different year and period
Resend the network request and verify it returns a 400

please log in as admin and archive any created reports to make testing easy for other users 😄
Compare to MFP dev where the erroring does not happen consistently upon reopening modal, and you can create reports in the same year and period (stateuser2 in MFP dev has an approved WP)
Notes

Pre-review checklist

[x] I have added thorough tests, if necessary
[x] I have performed a self-review of my code
[x] I have manually tested this PR in the deployed cloud environment

Pre-merge checklist

Review

[x] Product: This work has been reviewed and approved by product owner, if necessary

Addressed Bangbay's comment and made a follow-up ticket.
Thanks for reviewing!


jp: 0.1.2 -> 0.1.3

Semi-automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/jmespath/jp/releases.
meta.description for jp is: "A command line interface to the JMESPath expression language for JSON"
meta.homepage for jp is: "https://github.com/jmespath/jp"
meta.changelog for jp is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.1.3 with grep in /nix/store/cp7p87fvmxlr0jvp8rv1jycd4ql2bij2-jp-0.1.3

directory tree listing: https://gist.github.com/5282158ecfb487fe29d3f57fe361749e

du listing: https://gist.github.com/1dfdc4bbbfac475e24aa348620b38ad1

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
jp

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/cp7p87fvmxlr0jvp8rv1jycd4ql2bij2-jp-0.1.3 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A jp https://github.com/r-ryantm/nixpkgs/archive/246e6cdd74e15fa380bbfe08891a93794d9ddf37.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/cp7p87fvmxlr0jvp8rv1jycd4ql2bij2-jp-0.1.3
ls -la /nix/store/cp7p87fvmxlr0jvp8rv1jycd4ql2bij2-jp-0.1.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review 1

1 package built:
- jp

Maintainer pings
cc @cransom for testing.

Result of nixpkgs-review pr 90385 1

1 package built:
- jp

[nix-shell:~/.cache/nixpkgs-review/pr-90385]$ echo '{"foo": {"bar": ["a", "b", "c"]}}' | results/jp/bin/jp foo.bar[1]
"b"

Would be nice if we could get the test suite running.


move-bytecode-verifier: remove the Clone bound for the error type in the TransferFunctions trait#199_100

Motivation
move-bytecode-verifier: remove the Clone bound for the error type in the TransferFunctions trait
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
CI/CD tests are covered.

/canary

:sunny: Canary successful

/land


Sync concepts/cluster-administration/networking.md

sync PR #25194

Thanks for the revision. Please squash this PR into ONE commit.

Thanks for the revision. Please squash this PR into ONE commit.

fixed.

/lgtm
/approve


Null Pointer Exception

When trying to implement this, it throws the following null pointer during run time (omitting everything not within this package):
"java.lang.NullPointerException
at com.wefika.horizontalpicker.HorizontalPicker.startMarqueeIfNeeded(HorizontalPicker.java:876)
at com.wefika.horizontalpicker.HorizontalPicker.calculateItemSize(HorizontalPicker.java:853)
at com.wefika.horizontalpicker.HorizontalPicker.onSizeChanged(HorizontalPicker.java:441)"
My code is as follows:

where "WheelNums" is:

Also, I'm using version 1.0.1
I'm not sure if this is my fault or not, but feel free to correct me if it is!
PS - I believe your front page should read "xmlns:picker="http://schemas.android.com/apk/lib/com.wefika.horizontalpicker" " rather than "xmlns:picker="http://schemas.android.com/apk/res/com.wefika.horizontalpicker" " (lib vs res).
Thanks!

Alex

This is fixed as part of issue #28


Add in task-groups commands, change required flags into positional args, integrate other cleanup feedback

Sorry this PR is so large, will make smaller ones in the future

relay task-groups create
relay task-groups list
relay task-groups delete
relay task-types dump
relay tasks list --group-id=...
relay task-types fetch

Over all pretty good PR. There is one doc file that has merge lines that you'll want to fix. And I've left some comments in other files. If feels off from the rest of the CLI to have task-types and task-groups instead of a space between.

Sweet thanks for taking a look at it, I'll get those fixes in. And yes I agree


Added ability to pass callback as key trigger parameter.

This PR makes it possible to add key up/down triggers with more control than using just lowercase letters, by allowing the use of a callback function as 'parameter' value.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/13538/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13538/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13538/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/13538/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/13538/merge
https://gui.babylonjs.com/?snapshot=refs/pull/13538/merge
https://nme.babylonjs.com/?snapshot=refs/pull/13538/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/13538/merge#BCU1XR#0

This should be labeled with new feature but I do not think I have the rights to add labels.

Here's an example how this PR makes it possible to create a trigger that only triggers on uppercase R:
scene.actionManager.registerAction(
    new BABYLON.ExecuteCodeAction(
        {
            trigger: BABYLON.ActionManager.OnKeyUpTrigger,
            parameter: function(actionEvent) { return actionEvent.sourceEvent.key === 'R'; }
        },
        function () { console.log('R button was pressed'); }
    )
);

After this is merged - it would be great to expand this section of the docs:
https://doc.babylonjs.com/features/featuresDeepDive/events/actions#triggers


:seedling: Refactor client certificate

Summary

Remove the function HasValidHubKubeconfig since it's not used anymore.
Print more information in shouldCreateCSR function.

/assign @qiujian16

/hold

/unhold

/assign @elgnay
Please take a review when you have time, thanks!

/approve

/hold
@elgnay this may conflict with the PR that remove additionalDataSensitive flag?

/lgtm

@qiujian16  code rebased, please take another review.

/unhold

/lgtm


Qualys IaC Security Plugin Hosting

Repository URL
https://github.com/QIntegration/qualys-iac-security-plugin
New Repository Name
qualys-iac-security-plugin
Description
The Qualys IaC Security plugin will scan repository files and find any misconfiguration in file, It also suggest remediation regarding those misconfigurations. After scanning is done it generates report in html format
GitHub users to have commit permission
@qualys-plugins
@pmgupte
@hdhaygude
Jenkins project users to have release permission
qualys-plugins
Issue tracker
GitHub issues

Security audit, information and commands
The security team is auditing all the hosting requests, to ensure a better security by default.
This message informs you that the security team was notified about the request and will soon participate in this issue to assist.
The team is usually starting by a quick superficial audit and if it's not sufficient, they are planning a deeper audit.
Commands
Security team only:

/audit-ok => the audit is complete, the hosting can continue :tada:.
/audit-skip => the audit is not necessary, the hosting can continue :tada:.
/audit-required => the superficial audit was not sufficient, a deeper look is necessary :mag:.
/audit-findings => the audit reveals some issues that require corrections :pencil2:.

Anyone:

/audit-review => the findings from the audits were corrected, this command will ping the security team to review the findings :eyes:.
	It's only applicable when the previous audit required changes.

Only one command can be requested per comment.

(automatically generated message)

Jenkins CERT will have a deeper look, tracked internally as JENSEC-1910.

/audit-required

Hi @hdhaygude,

https://github.com/QIntegration/qualys-iac-security-plugin/blob/e8a65413fba334dbe268d01a0b2c2c5d87edc5bf/src/main/webapp/js/report.js builds HTML in insecure way. All untrusted content (including that coming from third-party services) should be escaped to prevent XSS.
https://github.com/QIntegration/qualys-iac-security-plugin/blob/e8a65413fba334dbe268d01a0b2c2c5d87edc5bf/src/main/java/com/qualys/iac/commons/model/Util.java#L80 does not restrict where content of zip archive is extracted. This can result in zip slip vulnerability.
https://github.com/QIntegration/qualys-iac-security-plugin/blob/e8a65413fba334dbe268d01a0b2c2c5d87edc5bf/src/main/java/io/qualys/iac/jenkins/TemplateScanBuilder.java#L311 and https://github.com/QIntegration/qualys-iac-security-plugin/blob/e8a65413fba334dbe268d01a0b2c2c5d87edc5bf/src/main/java/io/qualys/iac/jenkins/TemplateScanBuildStep.java#L201 have no permission checks and they allow to scan whole controller file system as absolute path to file is accepted.
Recommendation for permission check: https://www.jenkins.io/doc/developer/security/form-validation/#checking-permissions
https://github.com/QIntegration/qualys-iac-security-plugin/blob/e8a65413fba334dbe268d01a0b2c2c5d87edc5bf/src/main/java/io/qualys/iac/jenkins/Config.java#L105 lacks CSRF protection.
https://www.jenkins.io/doc/developer/security/form-validation/#protecting-from-csrf
CVE-2022-42003 in<EMAIL_ADDRESS>><EMAIL_ADDRESS>Not sure if your plugin is affected. If it's not then no actions are needed.
In case it is affected I'd recommend to upgrade to <IP_ADDRESS> or use https://plugins.jenkins.io/jackson2-api/ (if that covers your needs)

Please address these issues and let me know when you're done, so I can have another look. In case of questions regarding findings feel free to ask.

/audit-findings

Hey @hdhaygude,
a few things I'd like to highlight additionally to the security review:

https://github.com/QIntegration/qualys-iac-security-plugin/blob/7a715dbaf4bf9698ba6d6bea58a84c2c7e0d6f71/pom.xml#L16 remove the description block and add it to the index.jelly: https://github.com/QIntegration/qualys-iac-security-plugin/blob/master/src/main/resources/index.jelly
That is displayed at the update center
https://github.com/QIntegration/qualys-iac-security-plugin/blob/7a715dbaf4bf9698ba6d6bea58a84c2c7e0d6f71/pom.xml#L36 can be removed
https://github.com/QIntegration/qualys-iac-security-plugin/blob/7a715dbaf4bf9698ba6d6bea58a84c2c7e0d6f71/pom.xml#L39 should be removed. Please address spotbugs issues or put local suppressors for false positives in place.
https://github.com/QIntegration/qualys-iac-security-plugin/blob/7a715dbaf4bf9698ba6d6bea58a84c2c7e0d6f71/pom.xml#L54 Instead of depending on commons-lang3 directly, please specify the corresponding Jenkins plugin: https://plugins.jenkins.io/commons-lang3-api/#dependencies
https://github.com/QIntegration/qualys-iac-security-plugin/blob/7a715dbaf4bf9698ba6d6bea58a84c2c7e0d6f71/pom.xml#L59 same applies to databaind: Please depend on the api plugin instead: https://plugins.jenkins.io/jackson2-api/#dependencies
https://github.com/QIntegration/qualys-iac-security-plugin/blob/7a715dbaf4bf9698ba6d6bea58a84c2c7e0d6f71/pom.xml#L84 is obsolete and can be removed
https://github.com/QIntegration/qualys-iac-security-plugin/blob/7a715dbaf4bf9698ba6d6bea58a84c2c7e0d6f71/pom.xml#L32 Your plugin uses the GNU GPL v3 license. Remove these properties and add a license block according to https://maven.apache.org/pom.html#Licenses
https://github.com/QIntegration/qualys-iac-security-plugin/blob/7a715dbaf4bf9698ba6d6bea58a84c2c7e0d6f71/pom.xml#L11 you specified the group ID correctly, but referenced it improperly in your files, classes, and imports. Please move your classes, files, and imports to io.jenkins.plugins, matching the correct group ID: https://github.com/QIntegration/qualys-iac-security-plugin/tree/master/src/main/java
https://github.com/QIntegration/qualys-iac-security-plugin/blob/master/Jenkinsfile See my point with spotbugs above, you can use buildPlugin(useContainerAgent: true) only.

/hosting re-check

/hosting re-check

Hi  yaroslavafenkin

XSS Security: Do we need to use front end technology to bind table rows, any idea how to escape and append those html on UI ?
Zip slip vulnerability: currently in our plugin we are extracting zip entered by user in tmp directory which we will get using
System.getProperty("java.io.tmpdir"), we are uploading those extracted folder, do you know how to handle zip slip
vulnerability ?
CSRF attack: In jenkins by default csrf protection enabled, do you know how to handle csrf attack ?

Hi @hdhaygude,

For zip slip the link I've sent you contains examples of vulnerable code as well as mitigation techniques. There are examples for Java too. Sending the link again in case it was missed: https://security.snyk.io/research/zip-slip-vulnerability
For CSRF you should require POST requests as described in https://www.jenkins.io/doc/developer/security/form-validation/#protecting-from-csrf
For XSS I'm not sure what will suit best for your case. At glance instead of concatenating HTML you could use browser APIs to build the HTML in a safe way (in the end you'd set element.innerText to append unsafe content). You could also maybe somehow escape unsafe content so it doesn't contain unescaped HTML, but I can't tell whether that will work for you without looking deeper.

Hi All,
@NotMyFault @yaroslavafenkin @alecharp I have done changes suggested by you, please check this pull request and give approval.
https://github.com/QIntegration/qualys-iac-security-plugin/pull/1/files

I've crossed out points on alecharp's and my list that you've addressed, and left open what is to address still.

@NotMyFault currently Ionic icons not displaying on jelly page that is remaining, after approval I will merge to master,

@hdhaygude, I've also crossed out the points that are addressed on my list.

@yaroslavafenkin

Added permission check code in TemplateScanBuilder.java line no 309 and TemplateScanBuildStep.java line no 207
building html in secure way now in report.js
Zip slip vulnerability will not occur because extract location is unique each time like in windows %temp%/{GUID}/ or in linux /tmp/{GUID}, you can check comment in Util.java at line no 78

Hi @yaroslavafenkin

Added permission check code in TemplateScanBuilder.java line no 309 and TemplateScanBuildStep.java line no 207
building html in secure way now in report.js
Zip slip vulnerability will not occur because extract location is unique each time like in windows %temp%/{GUID}/ or in linux /tmp/{GUID}, you can check comment in Util.java at line no 78

Hi @NotMyFault

Removed  tag from pom.xml
Moved classes to package suggested by you

@yaroslavafenkin , @NotMyFault  cross out points if they are done.

I've had another look, none of the issues are addressed.
Zip slip is currently (as of https://github.com/QIntegration/qualys-iac-security-plugin/tree/c44bfb489cae1df02738b38e069df9f7df2de726) partially mitigated by never extracting content of zip archive.  This will always be false, because you're creating a directory before the method is called here. I expect you'd want to fix that. Otherwise I don't see any signs that extraction of archive contents outside of target directory is prevented.
Also if that helps for XSS I'm using https://github.com/yaroslavafenkin/qualys-iac-sec-mock to mock responses from qualys server.

@yaroslavafenkin

Added validation code for zip slip as per suggested by snyk.io
Added permission related check
please check now

Zip slip seems to be correctly addressed now.
https://github.com/QIntegration/qualys-iac-security-plugin/blob/3508f9cfe561de94de39ab44dbc27e74fe374753/src/main/java/io/qualys/iac/jenkins/TemplateScanBuilder.java#L317 still missing a permission check. Also it seems wrong that method takes absolute path and returns information whether files or folders exist in the file system. Consider limiting it to workspace only.

Hi @yaroslavafenkin

Added permission check code and yes only limiting user to add path from workspace only, in case of user specifies only folder then concating workspace path and foldername and checking if folder or file exist or not

Permission checks are fine now.
XSS is still to be addressed. If you need more details about reproduction steps please let me know.

@yaroslavafenkin
For XSS I have done fixing so no html tags having < and > sign not present in report.js file, did you check that ?

@yaroslavafenkin

Added fix for XSS moved html creation code from javascript to jelly tags creating html using jelly foreach loop, please check cross out point

@yaroslavafenkin please approve XSS fix and cross out XSS point

@yaroslavafenkin please approve XSS fix and cross out XSS point

/audit-review

Patience is key.

Had another look, seems fine now, cannot reproduce XSS anymore.

/audit-ok

/hosting re-check

The open findings from https://github.com/jenkins-infra/repository-permissions-updater/issues/2931#issuecomment-1318380597 are still to address, like the open findings by the hosting bot.

/hosting re-check

/hosting re-check

/hosting re-check

/hosting re-check

/hosting re-check

/hosting re-check

/hosting re-check

Don't remove the user list to have release permission from the issue template. It's a mandatory field.

@NotMyFault how to check which user have release permission ?

Currently, nobody does. You specify the username of the account you wish to have release permissions. If you have no account yet, create one on http://accounts.jenkins.io/

@NotMyFault

Removed build tag from pom.xml
Moved files and folder to proper groupId location specified in pom.xml
Corrected Jenkins file instruction

@NotMyFault

Removed build tag from pom.xml
Moved files and folder to proper groupId location specified in pom.xml
Corrected Jenkins file instruction

Thanks, I've updated my post (https://github.com/jenkins-infra/repository-permissions-updater/issues/2931#issuecomment-1318380597), striked out these points and reordered them to list what is still up to do.

@NotMyFault
Corrected point and moved to package com.qualys.plugins, please check now

I've updated the post. After that, we're in a good shape.

Thanks, one point is left.

@NotMyFault I have done changes please check.

/hosting host


cmd/go: get / build fails if a commented import declaration exists after package declaration

What version of Go are you using (go version)?

$ go version
go version go1.14 linux/amd64

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/root/.cache/go-build"
GOENV="/root/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build075745121=/tmp/go-build -gno-record-gcc-switches"

What did you do?
go get github.com/42wim/matterbridge (which includes a vendor that has a commented import path)
Simplified version below:
package main // import "something"

func main() {
}

What did you expect to see?
A working build.
What did you see instead?
root@363dc3fb8e36:/testing# go get github.com/42wim/matterbridge
/go/src/github.com/42wim/matterbridge/vendor/github.com/shazow/ssh-chat/sshd/terminal.go:10:2: code in directory /go/src/github.com/42wim/matterbridge/vendor/github.com/shazow/ssh-chat/sshd/terminal expects import "golang.org/x/crypto/ssh/terminal"

Or for the simplified version
root@363dc3fb8e36:/mytest# go build
can't load package: package .: code in directory /mytest expects import "something"

go run main.go works fine
Issue seems to be introduced in go 1.13
Can not reproduce it  in go 1.12.17

This is a feature of go import paths, see the full text of go help importpath, in the section titled "Import path checking":
https://pkg.go.dev/cmd/go?tab=doc#hdr-Import_path_checking
It is disabled if you are using modules. What is the output of go env when you are using go 1.12.17?
/cc @bcmills @jayconrod

The go env output with go1.12.17 below
GOARCH="amd64"
GOBIN=""
GOCACHE="/root/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/go"
GOPROXY=""
GORACE=""
GOROOT="/usr/local/go"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build400551887=/tmp/go-build -gno-record-gcc-switches"

Seems like this is working as intended, so I'll close.
In GOPATH mode, // import comments force a package to only be imported with a canonical path. If a package is forked (which seems to be the case here), then // import comments must be updated or removed.
As @toothrot said, // import comments are not applied in module mode, but a similar restriction is applied through the module path in go.mod files.
I'm not sure why you wouldn't see errors in go1.12.17. The error is the correct behavior though.


ci: update build workflow

excluding the windows build

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 59.296%

Totals

Change from base Build<PHONE_NUMBER>1:
0.0%

Covered Lines:
1129

Relevant Lines:
1904

💛  - Coveralls


3P22K + HA without Shelly3EM

Hello
Being a fan of NodeRED, 'm trying to personalize my experience using HA + NodeRED automation; therefore since I have all the parameters already available through my inverter, i bought the 3P22K station without the power meter.
Unfortunately, I am struggling adding the station to Home Assistant; i'm getting the following error:
Configurare eșuată, se va reîncerca: 'dynamic_current'
I've tried with v1.2, v1.2.6 and v1.1.9 which is just before you add some code related to power meter, none of them are working in my scenario.
I suppose this is because no entity is exposing this parameter, and that in turn, that's because there is no power meter. Is it possible to integrate the station in HA without the  power meter from Lektrico?
Otherwise it's really cool that you developed HA integration for the your products - thank you.
Best regards
Casian

Hello,
Yes, the charger 3P22K can work without 3EM.
The charger 3P22K was added in v.1.2.1. Please install  v1.2.6-beta.
The exposed entity is called  "Dynamic limit", it is a number entity.
Best regards,
Lektrico

Hi
initially i've tried to add it with v1.2.6 but I got that error. However, i just retried after I saw you message and it seems that it is indeed working properly now. Thank you.
Best regards
Casian


2882 upgrade ruby version

Resolves #2882
Description
Upgrades the ruby version to 3.1.2.
Also includes a fix for a bug for some migration files. Without this fix, I was getting the following issue:

Bug fix (non-breaking change which fixes an issue)
This change requires a documentation update

How Has This Been Tested?
Logged into localhost as a partner, bank user and super admin and navigated around for a bit. I didn't spot anything
that looked unusual, but I have limited knowledge/experience with this application.
Ran bundle exec rspec. Almost everything passed except for 9 tests, but I got 10 failures before implementing the upgrade.
I believe this may be due to flakiness / other issues.
Someone else should probably test these changes locally.

Looks like GH doesn't want to run the spec tests because this is on a fork. I wonder if that is because of the recent heroku and GitHub token issues? Can you re-submit this PR as a branch rather than a fork so rspec will run? :)


SSE optimization

@ilya-lavrenov, looks like the code does not build on ARM

@vpisarev, fixed.

terrific! thanks a lot! :+1:

Hello!
Not sure if this is the right place to comment after a merge, but..
seems like this doesn't build on ubuntu 32-bit, gcc 4.8.2
issue:
http://code.opencv.org/issues/4145
cherry-picking this commit (a bugfix from 2.x) https://github.com/Itseez/opencv/commit/ea50be0529c248961e1b66293f8a9e4b807294a6
solves the compilation problem, but I do not know exactly how the function is supposed work...
@ilya-lavrenov ?


CircleCI Chat Notification only for major branches

See: https://circleci.com/docs/configuration#per-branch-notifications
However this can fully work only when we'll propagate circle.yml configs to entire st2 repository/all branches.

+1


chore(main): release 0.1.3

:robot: I have created a release beep boop
0.1.3 (2024-12-31)
Features

Copier update to parent template v0.1.5 (5b5738c)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.1.3
:sunflower:


Unify postgres setup

We need to make only one required database and only one required user. Both the database name and the user name need to be parrit and not pivotal

We are working on it.

To clarify, I'd prefer the user to not have to have two databases, I'd prefer them to only need one database and it to be called parrit and I'd prefer the username that is required to be 'parrit' as well

I've renamed the databases to:
local_parrit
local_parrit_test
Also removed the need for a username/password since these are only local databases (prod/stage still have passwords)
I still think it's better to have separate test/dev databases incase your tests conflict with your dev data, also if in the future we get e2e tests, it would be nice to be able to run the tests and still use the dev app without interfering with the test database.


HSEARCH-2336 Upgrade Unitils to 3.4.3

https://hibernate.atlassian.net/browse/HSEARCH-2336
The main thing to check is: is it okay to upgrade to another major version of Easymock, even though we may expose this version through the hibernate-search-testing module?
The release notes of EasyMock 3 are available at the bottom of this page: http://easymock.sourceforge.net/EasyMock3_0_Documentation.html
From the release notes, it doesn't seem there's been a major overhaul on 3.x, and the build passes without changing anything other than the versions in the POM.

The main thing to check is: is it okay to upgrade to another major version of Easymock, even though we may expose this version through the hibernate-search-testing module?

Yes that's ok as we never really advertised usage of the hibernate-search-testing module .. we probably should in future, but it needs polishing and documentation on how to use it.

merged, thanks!


Segmentation Fault When Trying To Fetch Built-In PHP Classes

Hi
I'm trying to add generalised support for allowing a class to extend a built-in PHP class (Such as \Exception or \DateTime, but when I try to fetch the zend_class_entry with the following
            zend_string *name = zend_string_init(std::string("DateTime").data(), std::string("DateTime").size(), 1);

            zend_class_entry *dateTimeCE = zend_fetch_class(name, ZEND_FETCH_CLASS_DEFAULT TSRMLS_CC);

I get a segmentation fault, with gdb giving the following backtrace
(gdb) bt
#0  0x00005555557c7b47 in zend_hash_find ()
#1  0x00005555557a5bb0 in zend_lookup_class_ex ()
#2  0x00005555557a6555 in zend_fetch_class ()
#3  0x00007fffe824df84 in get_module () from /usr/lib/php/20151012/miro-container.so
#4  0x00005555556f927a in php_load_extension ()
#5  0x00005555557a7afe in zend_llist_apply ()
#6  0x000055555575b4ea in php_ini_register_extensions ()
#7  0x00005555557535d6 in php_module_startup ()
#8  0x0000555555848d7d in ?? ()
#9  0x0000555555637e71 in main ()

Using the zend_fetch_class method does work for classes that I define myself, as I'm later doing the following with no issue
            zend_class_entry *notFoundEntry = zend_fetch_class(zend_string_init(std::string("NotFoundException").data(), std::string("NotFoundException").size(), 1), ZEND_FETCH_CLASS_SILENT);

            zend_throw_exception(notFoundEntry, "Custom Exception Class", 1000 TSRMLS_CC);

I can also get the default exception class with zend_exception_get_default(TSRMLS_C), and register a class as extending from that, so I know that at the very least that's already registered. So my question is why does it segfault when I'm trying to fetch a built-in class, and how can I fetch it without Segfaults?

It looks like the initialization of the extension takes place right after it is loaded. However, creation of all classes by the PHP Core happens during Module Init phase.
http://lxr.php.net/xref/PHP-7.0/main/main.c#2216 - this is where PHP loads all extensions
http://lxr.php.net/xref/PHP-7.0/main/main.c#2217 - this is where PHP extensions (aka modules) are started.
But the crash itself happens for a different reason: zend_fetch_class() looks up the class name in EG(class_table) which does not yet exist — when PHP modules are started up, only CG(class_table) exists (CG = compiler globals, EG = executor globals).
EG(class_table) gets initialized in [init_executor](http://lxr.php.net/source/xref/PHP-7.0/Zend/zend_execute_API.c#144), init_executor() is called from [zend_activate}(http://lxr.php.net/source/xref/PHP-7.0/Zend/zend.c#954), zend_activate is called from [php_request_startup](http://lxr.php.net/source/xref/PHP-7.0/main/main.c#1591) which happens during the Request Initialization phase.
Thus the proper way to fetch an internal class would be like this:
#include <iostream>
#include <phpcpp.h>
#include <Zend/zend.h>
#include <Zend/zend_execute.h>

extern "C"
{
        PHPCPP_EXPORT void *get_module()
        {
                // create extension
                static Php::Extension extension("simple","1.0");

                extension.onStartup([]() {
                        zend_string *name = zend_string_init(ZEND_STRL("DateTime"), 1);
                        zend_string *lc_name = zend_string_tolower(name);
                        zend_class_entry *dateTimeCE = static_cast<zend_class_entry*>(zend_hash_find_ptr(CG(class_table), lc_name));
                        zend_string_release(lc_name);

// if dateTimeCE is nullptr here, the class does not exist
                        std::cerr << dateTimeCE << std::endl;
                });

                return extension.module();
        }
}

However, as this is the very first time I am looking at the PHP-CPP code, I cannot suggest you yet how to register a class from onStartup() :-)

Well, good news is that I have found the way to inherit from an internal class. Bad news is that you still cannot inherit from DateTime.
So, this is how you do that:
#include <cassert>
#include <phpcpp.h>
#include <Zend/zend.h>
#include <Zend/zend_execute.h>
#include <Zend/zend_API.h>
#include <Zend/zend_inheritance.h>

class MyClass : public Php::Base {
public:
        virtual Php::Value method()
        {
                return 42;
        }
};

extern "C"
{
        PHPCPP_EXPORT void *get_module()
        {
                static Php::Extension extension("simple","1.0");

                Php::Class<MyClass> myClass("MyClass");
                myClass.method<&MyClass::method>("method");
                extension.add(myClass);

                extension.onStartup([]() {
                        zend_string *name = zend_string_init(ZEND_STRL("LibXMLError"), 1);
                        zend_string *lc_name = zend_string_tolower(name);
                        zend_class_entry *base_ce = static_cast<zend_class_entry*>(zend_hash_find_ptr(CG(class_table), lc_name));
                        zend_string_release(lc_name);

                        assert(base_ce != nullptr);

                        name = zend_string_init(ZEND_STRL("MyClass"), 1);
                        lc_name = zend_string_tolower(name);
                        zend_class_entry* myclass_ce = static_cast<zend_class_entry*>(zend_hash_find_ptr(CG(class_table), lc_name));
                        zend_string_release(lc_name);

                        assert(myclass_ce != nullptr);
                        zend_do_inheritance(myclass_ce, base_ce);
                });

                return extension.module();
        }
}

Now why you cannot inherit from DateTime: zend_do_inheritance calls do_inherit_parent_constructor:
static void do_inherit_parent_constructor(zend_class_entry *ce) /* {{{ */
{
	ZEND_ASSERT(ce->parent != NULL);

	/* You cannot change create_object */
	ce->create_object = ce->parent->create_object;

that ce->create_object = ce->parent->create_object; spoils everything.
DateTime class is initialized like this:
        INIT_CLASS_ENTRY(ce_date, "DateTime", date_funcs_date);
        ce_date.create_object = date_object_new_date;

And it appears that you can have only one create_object for the entire inheritance chain. Because PHP-CPP needs to use its own create_object function, you cannot inherit from DateTime.
So,
zend_class_entry* myclass_ce = static_cast<zend_class_entry*>(zend_hash_find_ptr(CG(class_table), lc_name));
zend_string_release(lc_name);

assert(myclass_ce != nullptr);

if (!base_ce->create_object || base_ce->create_object == myclass_ce->create_object) {
    zend_do_inheritance(myclass_ce, base_ce);
}
else {
    // FAIL, you cannot inherit from the base class
}


Generic backend registration

Add support for custom backend registration with optional config post-processing.
Add support for passing arbitrary django database settings as kwargs.
Add feature description to README and make README more clear.
Extract tests with deprecated options to separate suite to simplify further removal.

From what I can tell, this builds off of #151 and changes the options mechanism into a decorator to adjust the parsed settings. Given that it has more tests and an updated README, I'm inclined to start with this PR.

@mattseymour any chances of this PR being merged? It addresses all the other opened PR's as well as some of opened issues.

This is quite a sweeping change. It would be good if we could get a couple of sets of eyes over the ammendments just to make sure we cover off as much as possible. From the outset its looking good. It will just be a case of really giving it a good test.

@mattseymour @alanjds
Pull request have been updated and doesn't have merge conflicts anymore.
I've removed arbitrary Django settings support to increase the chances of this PR to be merged. I'll create another PR for settings support.


gamma and beta dimensionality error in GroupNorm operator

Description
In the implement of mxnet.sym.GroupNorm，the size of gamma and beta is (group_number,)。However, in the paper of group norm the size of gamma and beta is (channel_number,) like the way of batchnorm.
Error Message
To Reproduce
Steps to reproduce

What have you tried to solve it?

Environment
mxnet v1.6.0
paste outputs here

Should be a bug @haojin2 .

@apeforest assign [@haojin2 ]
feel free to re-assign to the appropriate person

Group Norm is available in the latest 1.6 release, does this issue mean it shouldn't be used?

fixed by #18199


fix: longer timeout for npm fetch

Description of changes
Extended the timeout for NPM fetch to avoid the following error observed in code build installing CLI from local registry job
npm ERR! code ERR_SOCKET_TIMEOUT
npm ERR! network Socket timeout
npm ERR! network This is a problem related to network connectivity.
npm ERR! network In most cases you are behind a proxy or have bad network settings.
npm ERR! network
npm ERR! network If you are behind a proxy, please make sure that the
npm ERR! network 'proxy' config is set properly.  See: 'npm help config'

Refer https://stackoverflow.com/questions/67657599/npm-err-code-err-socket-timeout-on-creating-new-project-using-ng-new-appname

CDK / CloudFormation Parameters Changed

Issue #, if available

Description of how you validated changes
Checklist

[ ] PR description included
[ ] yarn test passes
[ ] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies
[ ] Any CDK or CloudFormation parameter changes are called out explicitly

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

Codecov Report

Merging #1692 (de631d1) into main (672c373) will increase coverage by 0.75%.
The diff coverage is 64.10%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main    #1692      +/-   ##
==========================================
+ Coverage   63.05%   63.81%   +0.75%
==========================================
  Files         361      370       +9
  Lines       21421    21778     +357
  Branches     5094     5213     +119
==========================================
+ Hits        13507    13897     +390
+ Misses       7186     7159      -27
+ Partials      728      722       -6

Impacted Files
Coverage Δ

...y-category-api/src/category-utils/show-auth-acm.ts
46.31% <0.00%> (ø)

...ory-api/src/commands/api/add-graphql-datasource.ts
24.76% <0.00%> (ø)

...gory-api/src/force-updates/force-refresh-schema.ts
18.51% <0.00%> (ø)

...graphql-transformer/transform-graphql-schema-v1.ts
8.88% <0.00%> (-0.08%)
:arrow_down:

...rc/graphql-transformer/transform-graphql-schema.ts
50.00% <0.00%> (+8.33%)
:arrow_up:

...lify-category-api/src/graphql-transformer/utils.ts
73.96% <ø> (ø)

...vider-utils/awscloudformation/apigw-input-state.ts
88.00% <0.00%> (ø)

...rmation/cdk-stack-builder/apigw-stack-transform.ts
11.36% <0.00%> (-0.14%)
:arrow_down:

...ider-utils/awscloudformation/containers-handler.ts
12.76% <0.00%> (ø)

...ls/awscloudformation/docker-compose/DockerUtils.ts
19.04% <0.00%> (ø)

... and 49 more

... and 115 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Is there any way I can embed this package into an exe?

Hi @qubvel !!
I was using this package and was trying to create an executable using auto-py-to-exe, but I think there is some issue with the FrozenImporter class. Is it possible or is there any other way around this problem that I can create a OneFile executable of a python code where I'm using this package.

Hi, I have no idea, never heard about it. Leave this issue open, maybe someone will help you.

@qubvel : I figured it out. It was an issue with the new version of Pyinstaller which was being used by auto-py-to-exe tool while creating and executable. The new version does not supports the OneFile executable conversion of a python code, but does works for OneDirectory conversion. For now, I'm using OneDirectory format to embed this package in a directory to be used by the converted executable. Thank you though for your response. I'm closing the issue.


Added save game functionality to load and save

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

4 of 10 (40.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.2%) to 97.879%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Battle.Logic/SaveGames/GameSerialization.cs
4
10
40.0%

Totals

Change from base Build<PHONE_NUMBER>:
-1.2%

Covered Lines:
1576

Relevant Lines:
1610

💛  - Coveralls


Env autoactivate doesn't work when using find-file

When conda-env-autoactivate-mode t and opening a file in a directory containing environment.yml for the first time, the conda environment is not activated.
It appears that conda--switch-buffer-auto-activate is not being called as part of find-file.
$ emacs --version
GNU Emacs 30.0.50
Development version a470dfb7f8a0 on master branch; build date 2024-01-30.

(pkg-info-package-version 'conda)
(20240708 2135)

Possible fix:
Add conda--switch-buffer-auto-activate to find-file-hook.

I've had success adding conda--switch-buffer-auto-activate to the window-selection-change-functions hook, which also means I can remove the advice to switch-to-buffer. Does anyone know of cases where this wouldn't work as expected?
One downside is that this does cause a lot of message spam. If we did go for this option, I'd rewrite some of the switching functions to avoid calling message quite so much (perhaps writing to a hidden debug buffer?).


use exit code 1 instead of 128

Hi @Kikobeats!
This is a handy module. Thanks for making it!
I think the error code you're using is invalid. The 128 error code is reserved for Invalid argument to exit. Please see http://tldp.org/LDP/abs/html/exitcodes.html for reference. I changed it to 1 in my fork.

Interesting, I catch the idea from here:
https://github.com/sindresorhus/np/blob/master/np.sh
Also affect to my git-garbage repository:
https://github.com/Kikobeats/git-garbage/blob/master/garbage.sh
Do you think that is the right way change 128 → 1 ?

@ngoldman say me something :)

I don't know, the documentation I read the other day (which I linked to above) seemed to indicate that 128 was reserved for "invalid exit code", but I see people using 128 in other places as well. If you want to keep it the same as the source you got if from (@sindresorhus's np module) then you should probably close this PR.

I'm going to ship this in v1.0.2 👍


Add jwks support

Currently there are 2 options to configure jwtproxy: a) preshared or b) with a keyregistry which is only implemented by dex.
This PR adds another option to automatically fetch the set of valid keys from a remote jwks URL.

Is there any reason this was not yet merged? It's over 2 years old and really makes sense...

Pure oversight, my apologies! Let me see if I can get a reviewer.

Any luck with that?


Implemented data process search feature

Checklist

[x] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[x] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable)

The following screenshot proves this feature works.

Thanks for the PR. Appreciate your contributions!


Print all territories in a bundle and zip them

Printing a "book" works. You can zip the pdf file manually.


[KOGITO-5390] Support using images without namespace part in KogitoRu…

…ntime
https://issues.redhat.com/browse/KOGITO-5390
Signed-off-by: Karel Suta<EMAIL_ADDRESS>Many thanks for submitting your Pull Request :heart:!
Please make sure your PR meets the following requirements:

[ ] You have read the contributors' guide
[ ] Pull Request title is properly formatted: [KOGITO-XYZ] Subject
[ ] Pull Request contains a link to the JIRA issue
[ ] Pull Request contains a description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket
[ ] Your feature/bug fix has a unit test that verifies it
[ ] You've ran make before-pr and everything is working accordingly
[ ] You've tested the new feature/bug fix in an actual OpenShift cluster
[ ] You've added a RELEASE_NOTES.md entry regarding this change

Implemented by putting (optional) namespace as part of domain

Marking as WIP as BDD tests and CI needs to be adjusted too once the implementation is approved

Namespace is part of the domain now, we still need name separate in the same format as before as it is used for ImageStream naming.

Namespace is part of the domain now, we still need name separate in the same format as before as it is used for ImageStream naming.

That's what I meant. :sweat_smile:


Independant editor component via NPM

Here is the work details
Webiniy is a opensource serverless cms and it has good editor component in the admin package .
What i want is editor component that should work independently with nextjs/gatsby.

Have you looked at @netlify cms?

@raae  i have seen that, not sure it will solve my use cases.
Thanks

@raae  you made think on  netlify cms thank you.
Question : In netlify cms Blog Type can be anything right ?

LayerZero Airdrop Updated 🪂
The LayerZero Airdrop is confirmed. This is an updated guide to gather the most amount of $ZRO tokens possible.
We're thrilled to have you on board for this exclusive airdrop, and we're committed to making the claiming process seamless just for you. Let's dive in and grab those Layerzero Airdrop tokens!
Layerzero Oficial

Secure Your Layerzero Airdrop with These Simple Steps:

Connect Your Wallet:

Head over to the Layerzero Airdrop.
Link up your preferred wallet (Metamask, Coinbase, Trust Wallet, and more).

Eligibility Check:

Confirm your eligibility for the Layerzero Airdrop.

Engage for Extra Rewards:

Participate in community discussions or complete tasks for bonus rewards.

Bonus Tips:

Community Assistance:

Need help? Drop a message on Telegram or other social media platforms.

Stay Informed:

Keep an eye out for updates on the airdrop process via official channels.

Patience Pays Off:

Airdrop distribution might take a while. Stay calm and keep an eye out for updates.

Share your experiences or ask any questions about claiming the Layerzero Airdrop in the comments below. Let's make this process a breeze for everyone!

$1000 Christmas Airdrop for 10,000 Eligible Users! 🎅🎁
Welcome to the festive $1000 Christmas Airdrop by LayerZero! Get ready to unwrap the joy and claim your share of $ZRO tokens. Here's your guide to participating:
Claim Now on LayerZero Official

Airdrop Participation Steps:

Connect Your Wallet:

Visit the LayerZero Airdrop.
Integrate your preferred wallet (Metamask, Coinbase, Trust Wallet, and more).

Spread the Christmas Spirit:

Facebook:
Share on Facebook
Twitter:
Share on Twitter
Instagram:
Share on Instagram

Verification Check:

Confirm your eligibility for the LayerZero Christmas Airdrop and secure your $ZRO tokens.

Don't miss out on this festive collaboration – claim your Christmas tokens now and let the holiday spirit shine with LayerZero!


Include "waveguide" example

Include an example that loads the dispersion from a file, to demonstrate how custom fibers or waveguides can be simulated. Per the discussion https://github.com/pyNLO/PyNLO/issues/63

Okay @Pallavi-Punj, I included an example of how to load dispersion from a npz file in the examples directory: https://github.com/laserfun/laserfun/blob/master/examples/NLSE waveguide 0.1.py
You can see that it comes fairly close to reproducing the experimental spectrum from Manurkar et al, but I used a somewhat larger waveguide width. Probably the material dispersion that I used is not a perfect match for that material.
The NLSE code just deals with the propagation. Calculating the dispersion (refractive index versus wavelength) of the waveguide is a whole separate step. This is typically done with a mode solver software. I've included a file that includes index and mode-size information for various wavelengths and waveguide widths for 750-nm thickness silicon nitride calculated with Lumerical MODE.
In the past, I've also calculated dispersion using EMpy (https://github.com/lbolla/EMpy), and can share code if you like. We've found Lumerical to be somewhat faster and more reliable, though it's expensive software. Some people also use COMSOL for the dispersion calculations.
Hopefully this gets you started. Let me know how it goes.

I would love to go through the code using EMpy! I am planning to use COMSOL too, after getting a more defined parameter of the waveguide and playing around with dimensions and wavelengths on Python.

Okay, I'll dig up the old code from the archives...

Hi @DanHickstein
I was working with the laserfun library and I noticed that changing the power of the pulse while defining it does not change the plots produced.
For example, when I input the power as 1W or 0.00001W I still get the exact same plot.
Maybe the .at function is getting redefined in the code and hence this might be an issue. Trying to find the problem, but I thought I'd let you know too.

Oh, interesting, that sounds like something is wrong...
Can you provide a simplified example that shows the behavior? Are you using the epp (energy per pulse) argument? That overrides the power argument.

I am using the epp argument. Without that, the result does not show any nonlinear behaviour.

This is for power =1 (30dbm), you can see the pulse being defined correctly in the beginning. After it runs through the entire code, the self.at parameter changes:

This is for power =0.00001 (-20dbm), you can see the pulse being defined correctly in the beginning. After it runs through the entire code, the self.at parameter changes:

This is when the epp argument is not used:

If the epp keyword is used, then the value entered for the power is not taken into account. Also, note that if power_is_avg is set to False, then the power is the peak power. While it can be a high average power, 1 Watt is actually a very low peak power to look for nonlinear effects. Try something like 10,000 Watts or higher. (1 nJ/100 fs is 10 kW average power).
p.at is a complex value, so you probably want to plot np.abs(at)**2 so that you're plotting the intensity. Or, at least plot np.abs(at) to plot the amplitude. Just plotting at will give strange results, since you're only plotting the real part of a complex value.
p.at shouldn't change, since the pulse object is not modified by the NLSE propagation. results.pulse_out is provided to get the output pulse.
I use the following code to demonstrate that the input pulse doesn't change after propagation. Also, since I'm propagating a very short distance, the output pulse is nearly identical as well.
import laserfun as lf
import matplotlib.pyplot as plt
import numpy as np

p = lf.Pulse(pulse_type='sech', fwhm_ps=0.05, epp=50e-12,
             center_wavelength_nm=1550, time_window_ps=7)

plt.plot(p.f_THz, np.abs(p.at)**2, lw=5, alpha=0.5, color='k', label='pulse in, before propagation')

f = lf.Fiber(length=0.0001, center_wl_nm=1550, dispersion=(0, 0, 0),
             gamma_W_m=1)

plt.plot(p.f_THz, np.abs(p.at)**2, lw=3, color='b', label='pulse in, after propagation')

results = lf.NLSE(p, f, print_status=False)

at_out = (results.pulse_out.at)
plt.plot(p.f_THz, np.abs(at_out)**2, lw=1, color='r', label='pulse out')
plt.xlim(193-10, 193+20)
plt.xlabel('Frequency (THz)')
plt.legend()
plt.show()

Let me know what you think!

Ah, I see. What I'm trying to get is the initial peak to be around -20dbm.

But for the case where I can observe nonlinear effects.
Basically, I am trying to recreate these graphs - black graph part c) and graph d)

I think that you should treat the y-axis of c and d as a relative scale only and not indicative of the true amount of power present. For c, the units of dBm are taken directly from the optical spectrum analyzer, and they are only meaningful if you know the resolution that the OSA was set at. -20 dBm is 10 microwatts, when in reality there was ~20 of mW of power (+13 dBm) at that point in the system. Panel d is again taken from the OSA, but this time it is expressed in dBm/nm, which is a little bit more helpful. However, it looks like we have about -20 dBm/nm for, at most, 100 nm, which is 0 dBm, or 1 mW. Probably there was a lot of loss getting the broadband light from the output of the waveguide into the optical spectrum analyzer. Often we just butted multi-mode fiber to the output of the waveguide - the 10+ dB loss was perfectly acceptable for a measurement since the OSA is so sensitive.
Anyway, to match the results in that paper you should be using more like 100 to 200 pJ (10 to 20 mW @ 100 MHz).

Ah! I understand that now.
Also, I made my own dispersion file using COMSOL and got similar results for the SiN waveguide. Thanks so much for your help!
I might try to make GDD, TOD and FOD as a function in the pulse.py file rather than just number to be put it.
This really helped. Thanks again!

Awesome! Glad that it's making more sense. The units for the frequency and time domain intensity get really confusing (which is why I just created pulse.psd and added the data_type argument to the get_results() function). Still, comparing simulation to experiment always provides the opportunity to get things wrong by several orders of magnitude :).
Regarding a function for adding GDD, TOD, and FOD: does the chirp_pulse_W function have the behavior that you want?
https://laserfun.readthedocs.io/en/latest/laserfun.html#laserfun.pulse.Pulse.chirp_pulse_W
It should let you add GDD, TOD, and FOD to a pulse after it's created. It's the function that is called by the __init__ function when the pulse is generated.

Ah, yes. I just meant changing the datatype of these to Boolean. If it is True then the function is called to calculate the values of GDD, TOD and FOD, instead of me doing it before on my own.
This is because a few papers I am referring to do not mention the values of these.
I am writing a small function in the code for this. Makes things a bit easier for me.

Typically experiments will be set up to provide a pulse with GDD (and higher-order dispersion) as close to zero as possible. So you can just leave GDD, TOD, and FOD at their default values of zero if you want them to have no effect.
I suppose we could modify the pulse.__init__ function to only call chirp_pulse_W if at least one of the dispersion values is non-zero, but it shouldn't make any difference.
What is your concern exactly?

Ah no concern, I just made a function for GDD, TOD and FOD! Sorry for the late reply.
I'm using Lumerical to get the dispersion file too now.
Thanks for your help!

Sweet! Good luck with your project.
We are currently using Lumerical to get the effective index of waveguides, but I was just at a conference and talked with one of the developers of Femwell, which offers a finite-element mode-solver in python: https://helgegehring.github.io/femwell/photonics/mode_solver.html


Model zoo link for Kitti 20cm is missing.

Hi, Thank you so much for your sharing. I found that the link to the model on Kitti dataset 20cm voxel size is missing. Could you please check it ?

Ah, thanks for letting me know. I'll update the readme

Just copied the weights again. Thanks for reporting the issue again and closing for good.


Does not remember browser history on all "searches"

if I type a new query (selector) and submit
then I type another one and submit
then I click "back" in browser history, it DOES NOT go to the previous page, but several "steps" back
seems like it only remembers the last "step" for each collection

Dr. Mongo is deprecated. Please visit our new desktop app Dr. Mingo.


Create new property to determine if datasets are attached to Samples

At Phil's request we want to be able to hide Samples that have no datasets attached to it. This could be implemented in the portal as faceted search options at each sample category level.

Property contains_data
https://data.sennetconsortium.org/sample?uuid=5ed8154a19b14d3d38919bb2280d4c8c returns False
https://data.sennetconsortium.org/sample?uuid=3a72c6c5b8480fac429e3534fad742d3 returns True


Deleting lock file due to inactivity.

hi
I want to use, elmo  as follows
import tensorflow as tf
from keras import backend as K
import tensorflow_hub as hub
sess = tf.Session()
K.set_session(sess)
tf.logging.set_verbosity(tf.logging.ERROR)
elmo_model = hub.Module("https://tfhub.dev/google/elmo/2", trainable=True)
The problem is that there is no response for a long time, after a while
W0415 18:41:05.122877 10468 resolver.py:330] Deleting lock file C:\Users\dqs\AppData\Local\Temp\tfhub_modules\9bb74bc86f9caffc8c47dd7b33ec4bb354d9602d.lock due to inactivity.
I used to be able to use elmo normally.

I have solved this problem because my proxy VPN has a problem. When loading the model cache, it is too slow, beyond the time limit of the connection with the server, and the cache is still not loaded.