tempest.scenario.test_server_basic_ops.TestServerBasicOps.test_server_basic_ops fails with PortNotFound

What version (commit) do you encounter this issue on?
master (when both Liberty and Mitaka) however it is more common since the move to Mitaka
What type of issue is this (bug, feature, documentation, regression)?
bug
What is the expected behavior?
Pull request and periodic jobs run to test the codebase. The final step of the job is to run tempest. These tempest tests should complete successfully.
What is actually happening?
The tempest test tempest.scenario.test_server_basic_ops.TestServerBasicOps.test_server_basic_ops fails.
07:58:09 ==============================
07:58:09 Failed 1 tests - output below:
07:58:09 ==============================
07:58:09
07:58:09 tempest.scenario.test_server_basic_ops.TestServerBasicOps.test_server_basic_ops[compute,id-7fff3fb3-91d8-4fd0-bd7d-0204f1f180ba,network,smoke]

An extract of tempest's debugging information for one of these failures is
07:58:09 Captured traceback:
07:58:09 ~~~~~~~~~~~~~~~~~~~
07:58:09     Traceback (most recent call last):
07:58:09       File "/opt/tempest_c2c90574a78c7704f06e8fac279c4f1d11365993/tempest/scenario/manager.py", line 108, in delete_wrapper
07:58:09         delete_thing(*args, **kwargs)
07:58:09       File "/opt/tempest_c2c90574a78c7704f06e8fac279c4f1d11365993/tempest/lib/services/compute/floating_ips_client.py", line 62, in delete_floating_ip
07:58:09         resp, body = self.delete(url)
07:58:09       File "/opt/tempest_c2c90574a78c7704f06e8fac279c4f1d11365993/tempest/lib/common/rest_client.py", line 290, in delete
07:58:09         return self.request('DELETE', url, extra_headers, headers, body)
07:58:09       File "/opt/tempest_c2c90574a78c7704f06e8fac279c4f1d11365993/tempest/lib/services/compute/base_compute_client.py", line 53, in request
07:58:09         method, url, extra_headers, headers, body)
07:58:09       File "/opt/tempest_c2c90574a78c7704f06e8fac279c4f1d11365993/tempest/lib/common/rest_client.py", line 645, in request
07:58:09         resp, resp_body)
07:58:09       File "/opt/tempest_c2c90574a78c7704f06e8fac279c4f1d11365993/tempest/lib/common/rest_client.py", line 764, in _error_checker
07:58:09         message=message)
07:58:09     tempest.lib.exceptions.ServerFault: Got server fault
07:58:09     Details: Unexpected API Error. Please report this at http://bugs.launchpad.net/nova/ and attach the Nova API log if possible.
07:58:09     <class 'neutronclient.common.exceptions.PortNotFoundClient'>

What were you attempting to do when you encountered this error (give as much detail as possible)?
These occurences have been found during pull request and periodic testing. The have happened when Tempest is run at the end of the build.
How can we reproduce this issue?
I am unaware of specific steps, this failure is intermittent.
How can we verify your issue is addressed?
The build summary should no longer report failures due to this issue.
If this is a large issue, what can be done immediately to alleviate the issue while working on a longer-term solution?
What is your evaluation on the importance of this issue, and why?
This slows the merger of new code, it may be specific to dev testing but should be considered a prio-2
What branches / milestones do you believe this issue should target?
The list below is what is currently supported for backports
Remove branches (lines) you do not wish to target.

[x] Master
[x] liberty-12.2

DocImpact: This section is to be completed by the issue owner as part of the issue's resolution.
Does your change require documentation?

[ ] Yes, and I submitted an issue to Docs issues
[ ] Yes, and I contributed a documentation patch to docs-rpc
[x] No documentation required for this change

Here is the actual tempest failure output, isolated to just the specific failing tests:
http://paste.openstack.org/show/525933/
Here appears to be a launchpad bug filed against tempest for this issue:
https://bugs.launchpad.net/tempest/+bug/1586931
And here is the current fix proposed to tempest master (submitted 2016-07-02):
https://review.openstack.org/#/c/336842/

Yay - problem has been fixed upstream in master/mitaka/liberty
master: https://review.openstack.org/#/c/336842/
mitaka: https://review.openstack.org/#/c/338184/
liberty: https://review.openstack.org/#/c/338200/
@git-harry @hughsaunders we just need to track when OSA includes this in their sha bumps, and then ensure we include those sha bumps in our sha bump at some point :)

While the neutron side of this issue has been fixed, the problem has now been pushed to nova.
Excerpt from a recent nova api log:
2016-08-24 16:57:38.196 2188 DEBUG keystoneauth.session [req-eac3c216-7480-4d5f-966d-50b5975916a7 6d871cbec69e4eff897e4f05081a9128 d3d34c3e228740b8bb2f8e99b815393c - - -] REQ: curl -g -i -X GET http://<IP_ADDRESS>:9696/v2.0/ports/67f35c46-6417-45ef-9c9a-de3d568efb5f.json -H "User-Agent: python-neutronclient" -H "Accept: application/json" -H "X-Auth-Token: {SHA1}98a9246aab1074c66b7ca536523302ecb79f3037" _http_log_request /openstack/venvs/nova-13.3.1/lib/python2.7/site-packages/keystoneauth1/session.py:248
2016-08-24 16:57:38.688 2188 DEBUG keystoneauth.session [req-eac3c216-7480-4d5f-966d-50b5975916a7 6d871cbec69e4eff897e4f05081a9128 d3d34c3e228740b8bb2f8e99b815393c - - -] RESP: [404] Date: Wed, 24 Aug 2016 16:57:38 GMT Content-Length: 132 Content-Type: application/json; charset=UTF-8 X-Openstack-Request-Id: req-6a704a7c-92c1-454f-baf4-3b5bf77b4814
RESP BODY: {"NeutronError": {"message": "Port 67f35c46-6417-45ef-9c9a-de3d568efb5f could not be found.", "type": "PortNotFound", "detail": ""}}
 _http_log_response /openstack/venvs/nova-13.3.1/lib/python2.7/site-packages/keystoneauth1/session.py:277
2016-08-24 16:57:38.689 2188 DEBUG neutronclient.v2_0.client [req-eac3c216-7480-4d5f-966d-50b5975916a7 6d871cbec69e4eff897e4f05081a9128 d3d34c3e228740b8bb2f8e99b815393c - - -] Error message: {"NeutronError": {"message": "Port 67f35c46-6417-45ef-9c9a-de3d568efb5f could not be found.", "type": "PortNotFound", "detail": ""}} _handle_fault_response /openstack/venvs/nova-13.3.1/lib/python2.7/site-packages/neutronclient/v2_0/client.py:263
2016-08-24 16:57:38.691 2188 ERROR nova.api.openstack.extensions [req-eac3c216-7480-4d5f-966d-50b5975916a7 6d871cbec69e4eff897e4f05081a9128 d3d34c3e228740b8bb2f8e99b815393c - - -] Unexpected exception in API method
2016-08-24 16:57:38.691 2188 ERROR nova.api.openstack.extensions Traceback (most recent call last):
2016-08-24 16:57:38.691 2188 ERROR nova.api.openstack.extensions   File "/openstack/venvs/nova-13.3.1/lib/python2.7/site-packages/nova/api/openstack/extensions.py", line 478, in wrapped
2016-08-24 16:57:38.691 2188 ERROR nova.api.openstack.extensions     return f(*args, **kwargs)
2016-08-24 16:57:38.691 2188 ERROR nova.api.openstack.extensions   File "/openstack/venvs/nova-13.3.1/lib/python2.7/site-packages/nova/api/openstack/compute/floating_ips.py", line 170, in delete
2016-08-24 16:57:38.691 2188 ERROR nova.api.openstack.extensions     instance = get_instance_by_floating_ip_addr(self, context, address)
2016-08-24 16:57:38.691 2188 ERROR nova.api.openstack.extensions   File "/openstack/venvs/nova-13.3.1/lib/python2.7/site-packages/nova/api/openstack/compute/floating_ips.py", line 67, in get_instance_by_floating_ip_addr
2016-08-24 16:57:38.691 2188 ERROR nova.api.openstack.extensions     context, address)
2016-08-24 16:57:38.691 2188 ERROR nova.api.openstack.extensions   File "/openstack/venvs/nova-13.3.1/lib/python2.7/site-packages/nova/network/neutronv2/api.py", line 1424, in get_instance_id_by_floating_address
2016-08-24 16:57:38.691 2188 ERROR nova.api.openstack.extensions     port = self._show_port(context, fip['port_id'], neutron_client=client)
2016-08-24 16:57:38.691 2188 ERROR nova.api.openstack.extensions   File "/openstack/venvs/nova-13.3.1/lib/python2.7/site-packages/nova/network/neutronv2/api.py", line 896, in _show_port
2016-08-24 16:57:38.691 2188 ERROR nova.api.openstack.extensions     raise exception.PortNotFound(port_id=port_id)
2016-08-24 16:57:38.691 2188 ERROR nova.api.openstack.extensions PortNotFound: Port id 67f35c46-6417-45ef-9c9a-de3d568efb5f could not be found.
2016-08-24 16:57:38.691 2188 ERROR nova.api.openstack.extensions
2016-08-24 16:57:38.710 2188 INFO nova.api.openstack.wsgi [req-eac3c216-7480-4d5f-966d-50b5975916a7 6d871cbec69e4eff897e4f05081a9128 d3d34c3e228740b8bb2f8e99b815393c - - -] HTTP exception thrown: Unexpected API Error. Please report this at http://bugs.launchpad.net/nova/ and attach the Nova API log if possible.
<class 'nova.exception.PortNotFound'>

Bug 1586931 Has been fixed in Nova but backport to liberty is still pending.

@hughsaunders would you mind putting this in a new bug instead of reopening this one, it throws off all the reporting and makes work harder to track.


Implementation of 'likely' branches

Hello. I've tried to implement 'likely' branches idea, but I have a question. At this time, performance simulator puts 'nop' after branch-likely instruction, because we can obtain branch-target only while decoding branch instruction itself. But, as I understood, functional simulator doesn't want 'nop' being there. What should I do?

But, as I understood, functional simulator doesn't want 'nop' being there. What should I do?

we have to use target mispredict flush to have the following pipe:
Branch  F D E B W
PC+4      F *
Target      F D E M W

Could you please provide IPC results for default SQRT and SQRT with a likely branch?

Could you please provide IPC results for default SQRT and SQRT with a likely branch?

`init:    li           $a0, 145
sqrt:    li           $t0,  0x40000000
move    $t1,  $0
L0:       or         $t2,  $t1,  $t0
srl        $t1,   $t1,   1
sltu      $t3,  $a0, $t2
bne      $t3,  $0,   L1
         subu     $a0, $a0, $t2
         or          $t1,  $t1,  $t0

L1:         srl         $t0,  $t0,  2
bne        $t0,  $0,   L0
move     $v0,  $t1
end:      b             end`
IPC: 0.316456
`init:    li           $a0, 145
sqrt:    li           $t0,  0x40000000
move    $t1,  $0
L0:       or         $t2,  $t1,  $t0
srl        $t1,   $t1,   1
sltu      $t3,  $a0, $t2
bne      $t3,  $0,   L1
         subu     $a0, $a0, $t2
         or          $t1,  $t1,  $t0

L1:         srl         $t0,  $t0,  2
bnel       $t0,  $0,   L0
move     $v0,  $t1
end:      b             end`
IPC: 0.342466

Could you please provide IPC results for default SQRT and SQRT with a likely branch?

init:    li     $a0,  145

sqrt:    li     $t0,  0x40000000
         move   $t1,  $0

L0:      or     $t2,  $t1, $t0
         srl    $t1,  $t1, 1
         sltu   $t3,  $a0, $t2
         bne    $t3,  $0,  L1

         subu   $a0,  $a0, $t2
         or     $t1,  $t1, $t0

L1:      srl    $t0,  $t0, 2
         bne    $t0,  $0,  L0
         move   $v0,  $t1

end:     b      end

IPC: 0.316456
init:    li     $a0,  145

sqrt:    li     $t0,  0x40000000
         move   $t1,  $0

L0:      or     $t2,  $t1, $t0
         srl    $t1,  $t1, 1
         sltu   $t3,  $a0, $t2
         bne    $t3,  $0,  L1

         subu   $a0,  $a0, $t2
         or     $t1,  $t1, $t0

L1:      srl    $t0,  $t0, 2
         bnel   $t0,  $0,  L0
         move   $v0,  $t1

end:     b      end

IPC: 0.342466

Could you please tell how many iterations were performed?
My concern is that you have a bogus flush on the second iteration. You have a target in BTB already, and you know that branch is taken from branch prediction, but if I understand correctly your implementation, you flush on each likely branch instance.

Could you please tell how many iterations were performed?
My concern is that you have a bogus flush on the second iteration. You have a target in BTB already, and you know that branch is taken from branch prediction, but if I understand correctly your implementation, you flush on each likely branch instance.

Yes, that's my fault.
For 'bnel' I have fixed 50 instructions and 146 cycles, when for 'bne' number of cycles is 158.

Thanks.

Could you please tell how many iterations were performed?

By "iterations" I mean loop iterations.

By "iterations" I mean loop iterations.
Nearly 32 iterations.

Could you please provide IPC results for default SQRT and SQRT with a likely branch?

Updated information about IPC and mispredictions rate in my comment.

I've made some changes in misprediction analysis method in order not to miss mispredictions detected on Decode stage. But I have a question. Should we count mispredictions purposely caused by 'likely' branches? These 'short' mispredictions in order to decode target.

I've made some changes in misprediction analysis method in order not to miss mispredictions detected on Decode stage. But I have a question. Should we count mispredictions purposely caused by 'likely' branches? These 'short' mispredictions in order to decode target.

The bug is that I count these mispredictions twice: on Decode and Branch stages.

Should we count mispredictions purposely caused by 'likely' branches? These 'short' mispredictions in order to decode target.

I think yes. Please print the target (short) mispredictions and direction mispredictions separately.

62 iterations
init:    li     $a0,  145

sqrt:    li     $t0,  0x40000000
         move   $t1,  $0

L0:      or     $t2,  $t1, $t0
         srl    $t1,  $t1, 1
         sltu   $t3,  $a0, $t2
         bne    $t3,  $0,  L1

         subu   $a0,  $a0, $t2
         or     $t1,  $t1, $t0

L1:      srl    $t0,  $t0, 2
         bne    $t0,  $0,  L0
         move   $v0,  $t1

end:     b      end

IPC: 0.401606
mispredict: detected on decode stage - 0%; on branch stage - 86.6667%
init:    li     $a0,  145

sqrt:    li     $t0,  0x40000000
         move   $t1,  $0

L0:      or     $t2,  $t1, $t0
         srl    $t1,  $t1, 1
         sltu   $t3,  $a0, $t2
         bnel   $t3,  $0,  L1

         subu   $a0,  $a0, $t2
         or     $t1,  $t1, $t0

L1:      srl    $t0,  $t0, 2
         bnel   $t0,  $0,  L0
         move   $v0,  $t1

end:     b      end

IPC: 0.404858
mispredict: detected on decode stage - 6.66667%; on branch stage - 86.6667%

Hello. What should I add or edit?

Are unit tests working for your code?


Doesn't work in partial rendered via JQuery without a page refresh

(Nathan, I feel like I'm peppering you with these. I'm sorry!)
So this is weird one.
I have a list of objects that gets rendered via a partial for each object. When a new object is created it fires off some javascript and prepends the new object to the list without refreshing the page (via JQuery).
The title of each object is editable via on_the_spot, and I can click on any title in the list and it is editable.
However, when a new object is inserted into the list and I click on the title nothing happens. Yet, if I click any other title of an object in the list the on_the_spot function works. If I then refresh my page and click on the title of the new object it works.
When I investigate the outputted HTML code it seems to be identical:
Code for the new object that is inserted into the list - Doesn't work:

The description

Code for an existing object in the list - Works:

The description

The only discernable difference I can spot is the inclusion of a "title=" param in the code that works.
The partial also includes some other javascript to expand and collapse the object in the list. This works fine which would seem to rule out any javascript issues in the actual partial or the way it is rendered.
Any insight would be greatly appreciated.

No problem.
The code used to initialse the on-the-spot fields is
$('.on_the_spot_editing').each(initializeOnTheSpot);

Not sure how familiar you are with jQuery, but this means that new elements added to the DOM will not be editable (since the function is only applied to elements inside the DOM after loading).
So what you need to do, when inserting (prepending) your new object, you also have to make sure that that specific elements gets initialised, using that code.
So either do
$('.on_the_spot_editing').each(initializeOnTheSpot);

again, or use some more specific selector.
Does that help?

Nathan,
That's brilliant - thanks. Complete fixed the bug.
B


V 2.0.0 from develop.

Merge develop branch to master and tag 2.0.0.

Codecov Report

Merging #1 into master will increase coverage by 9.61%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##             master     #1      +/-   ##
==========================================
+ Coverage     90.38%   100%   +9.61%
- Complexity       20     30      +10
==========================================
  Files             1      1
  Lines            52     78      +26
==========================================
+ Hits             47     78      +31
+ Misses            5      0       -5

Impacted Files
Coverage Δ
Complexity Δ

src/Container.php
100% <100%> (+9.61%)
30 <11> (+10)
:arrow_up:

Continue to review full report at Codecov.

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


Trainer: GUI for trainer related patches / enemy data editing

As mentioned at https://github.com/nipkownix/re4_tweaks/issues/243#issuecomment-1181395554
Added a quick GUI for editing data of cEm instances, also added some trainer patches to it as well, currently includes an Invincibility patch & a 1-shot-kill weak enemies patch - both patches are linked up to debug flags so the tool menu flag editor should be able to enable them too (maybe in future we could add a way to bind certain flags to hotkeys too, or something like that)
E: also added a infinite ammo patch, thanks to @Mister-Curious for providing the bytes for it!

This is currently a draft PR since we're still waiting on #257 to be tested/merged, which this includes some of the GUI changes from, once that's merged I'll probably redo this branch again.
Only really tested with 1.1.0 so far, but everything is sig scanned etc, so hopefully should work on other versions too.
Also interested in any requests for things to add to this section, or any ideas on best way to save/load cEm changes, not really sure how to handle that atm.
(main issue is that there's no way to uniquely ID a cEm... afaik cEms can get loaded into any slot, so can't rely on the index of it to know where to load modded values into, also cEms can be created/destroyed any time during play, so the cEm being changed might not even exist when level is loaded)

when i can figure out a way to save the changes it will be perfect, by far the biggest and best update so far

I hope you discover a way to save that, luis webber make it save using the INI

Thanks for this. Its looking good. I did provide you with a big list of edits in your inbox on the forums a while ago. Were some of those not working? I don't see a lot of those present here:
No Enemy Spawn
No BGM
Invincibility
Infinite Ammo
Disable Collision
Always XL Attache Case
No Leon Model In Inventory
Bodies do not disappear
All the code is in that message thread. Glad to see this coming to life!

Looks awesome!

(main issue is that there's no way to uniquely ID a cEm... afaik cEms can get loaded into any slot, so can't rely on the index of it to know where to load modded values into, also cEms can be created/destroyed any time during play, so the cEm being changed might not even exist when level is loaded)

Yeah, I was about to mention that.
Not sure if trying to save this stuff is the best way to go, as I imagine there are too many things we have to predict/assume.
Unless we went with something more generic such as if EmTypeXX, apply Speed *= 2 or something like that.
Depends on what people are expecting to achieve with this, I guess.

we modders want to be able to edit the size and speed of our esl enemies, it's the most wanted function for us, we just want to edit 1 time and save for our mods to be complete

And who knows being able to do the ESL and maybe the ITA and ETS too because this trainer can put this in game would be perfect

Are ESL, ITA, ETS, etc, file extensions? If so, it would be better to find a way to edit those files directly, instead of trying to save memory changes like this.

Yes they are file extension, I don't know how Raz0r did it but in compenion it saves the ini in notepad to keep the changes

we modders want to be able to edit the size and speed of our esl enemies, it's the most wanted function for us

I would say that while this is a feature some have requested, it is certainly not 'the most wanted function'. There are plenty of things we still need that would be much more useful than that.

Are ESL, ITA, ETS, etc, file extensions? If so, it would be better to find a way to edit those files directly, instead of trying to save memory changes like this.

Yes they are file extensions, and there are tools for these files (and ample tutorials) on how to use them.

that's what's missing in our mods Raz0r managed to save it but with it's encrypted it's not possible to find out, but that would make our mods more complete they would

Looks awesome!

(main issue is that there's no way to uniquely ID a cEm... afaik cEms can get loaded into any slot, so can't rely on the index of it to know where to load modded values into, also cEms can be created/destroyed any time during play, so the cEm being changed might not even exist when level is loaded)

Yeah, I was about to mention that. Not sure if trying to save this stuff is the best way to go, as I imagine there are too many things we have to predict/assume. Unless we went with something more generic such as if EmTypeXX, apply Speed *= 2 or something like that. Depends on what people are expecting to achieve with this, I guess.

Its not a problem be something more generic, change the enemy speed and size is one of the most request thing by modders, luis webber is the only one who do that and never shared with the comunity and we always try to discover a way but no success by years. Me, JM, kTeo, DarkxVoid, Scoovy, Adam, Ashe, Waly, Higo, Leo Lima, every of these modders are trying figure out how to do that and is very frustrating. kTeo have discovered two codes for enemy size for the Garrador and Mega but only that two and nothing about the speed. Its not a problem to me if be generic or more complexy way to do that, can be very nice to have it simple or not, when i see this complexy way that emoose made i'm have cryed, this is important for the mods alot. Now i will speak portuguese for Nipkownix because i know he is Brazilian. Mano, resumindo aqui em portugues pq fica mais facil de eu explicar, tamo geral tantando descobrir isso a um século ja e nada de achar a velocidade e tamanho dos inimigos, quando vi isso dai q o emoose fez eu fiquei maluco, adorei, se criar algo mais simples ta bom d+ já, vai todo mundo já ficar feliz.

Maybe if we can find the code that handles creating cEm from the ESL list, we could tie the changes to the index inside the ESL, assuming that the cEms that people want to modify are all from ESL that should probably work.
I've never really looked into the ESL stuff before though, so no idea where that's handled in the code yet.

Mano, resumindo aqui em portugues pq fica mais facil de eu explicar, tamo geral tantando descobrir isso a um século ja e nada de achar a velocidade e tamanho dos inimigos, quando vi isso dai q o emoose fez eu fiquei maluco, adorei, se criar algo mais simples ta bom d+ já, vai todo mundo já ficar feliz.
Ah e já agradeço muito, Thank You!

Só fiquei meio confuso pq Mister-Curious falou ali que existem algumas tools que editam aqueles arquivos lá. Essas tools não servem pra mudar essas coisas? Se aquele luis webber conseguiu, imagino que ele editou os arquivos, ao invés de editar a memória do jogo. Estranho ele não ter compartilhado com o resto da comunidade...

Nunca achamos qual arquivo mexe com o tamanho, o kTeo achou pelo HEX como mudar o Garrador e Mega. Quanto ao Luis ele é apenas egoista, diz que é programador e que passaria as coisas, mas era promessa atrás de promessa por anos, bloqueou a DLL dele pra ninguém usar e nunca passou nada pra comunidade e assim que todos descobrem algo q ele ja tinha a anos ele em seguida revela como faz pra parecer um bom samaritano, enquanto isso todos aqui se esforçando pra não chegar a nenhum lugar. Com isso os mods dele sempre estão no topo, como o Life in Hell, Rising of Evil e Arrange e a gente se esforçando pra não ter metade do reconhecimento. Bem ao contrário de nós que soltamos tudo que aprendemos na Remodding, tem tudo la, só procurar por coisas do kTeo, minhas, do Curious, mas nada do Luis dês de anos.

Maybe if we can find the code that handles creating cEm from the ESL list, we could tie the changes to the index inside the ESL, assuming that the cEms that people want to modify are all from ESL that should probably work.
I've never really looked into the ESL stuff before though, so no idea where that's handled in the code yet.
E: oh looks like ESL might be related to EM_LIST stuff I added in way back, https://github.com/nipkownix/re4_tweaks/blob/master/dllmain/Game.h#L194
Would still need to find where the cEm is created first though.

Looking foward to it.

Its not possible to use the last 4 Columns of the ESL for edit the size and speed? This bytes is blank

Hmm nice idea, maybe possible to add that, if we can find all the parts that read from the EM_LIST stuff.
So far I've seen two methods, EmSetEvent seems to read all the data out of EM_LIST and creates a new cEm from it, and GetEmIdFromList seems to return the EmXX ID from EM_LIST, and game either passes that ID to EmReadSearch which I guess creates a new cEm from it, or passes it to EventMgr::EvtRead.
EmSetEvent might be pretty easy to update, but not sure about the second one though, not really sure how that second method could be getting stuff like position etc out of the EM_LIST, if it's only being given the ID...

OK

Hmm nice idea, maybe possible to add that, if we can find all the parts that read from the EM_LIST/ESL stuff.
So far I've seen two methods, EmSetEvent seems to read all the data (position, rotation, HP) out of EM_LIST and creates a new cEm from it, and GetEmIdFromList seems to return the EmXX ID from EM_LIST, and game either passes that ID to EmReadSearch which I guess creates a new cEm from it, or passes it to EventMgr::EvtReadXXXX.
EmSetEvent might be pretty easy to update to read stuff from those bytes, but not sure about the second one though, not really sure how that second method could be getting stuff like position etc out of the EM_LIST, if it's only being given the ID...
There are only 4 bytes there though, scale & size would need 8 bytes to store full float values for them. I guess we could split them into 2 bytes each and maybe hold a percentage value for them instead, or maybe see if there's a 16-bit half-float impl. we can add in, not sure if many hex editors support half-floats though, so guess percentage would be easier to edit.

Ok try percentage

OK

Hmm nice idea, maybe possible to add that, if we can find all the parts that read from the EM_LIST/ESL stuff.
So far I've seen two methods, EmSetEvent seems to read all the data (position, rotation, HP) out of EM_LIST and creates a new cEm from it, and GetEmIdFromList seems to return the EmXX ID from EM_LIST, and game either passes that ID to EmReadSearch which I guess creates a new cEm from it, or passes it to EventMgr::EvtReadXXXX.
EmSetEvent might be pretty easy to update to read stuff from those bytes, but not sure about the second one though, not really sure how that second method could be getting stuff like position etc out of the EM_LIST, if it's only being given the ID...
There are only 4 bytes there though, scale & size would need 8 bytes to store full float values for them. I guess we could split them into 2 bytes each and maybe hold a percentage value for them instead, or maybe see if there's a 16-bit half-float impl. we can add in, not sure if many hex editors support half-floats though, so guess percentage would be easier to edit.

Ok try percentage, try to put 00 be Default size and speed

Seem to have it mostly working, here's a test build:
dinput8-emListParamExtension.zip
The first 2 unused bytes in highlighted part of https://github.com/nipkownix/re4_tweaks/pull/266#issuecomment-1183939200 are treated as speed percentage, last 2 as size percentage, (eg set to 200 to make them 2x, 50 for 0.5x, etc)
Only tested with 1.1.0 but hopefully sigs will match other versions too.
If you find an enemy that isn't being affected by it let me know!

Amazing, i will test it now!! Thank You so much!

https://www.youtube.com/watch?v=w5HY5FQ1jVI ITS WORKING, IM HAPPY ASF!!

 Only speed works when spawning by ganado plaga)
em26 BOTH WORKS
em27 SPEED ONLY
em28 BOTH WORKS
em29 SPEED ONLY
em2a ??? (bear traps does not work straight away, only when they get activated.
Looks like some Ems overwrite the scale during their emXX_R0_Init function, after our ESL hooks have been ran... maybe could get it to force the scale each frame for ones that have it set, will try having a look.

So for this Em Manager is not saving the changes for "ESL-spawned only" right?

Might have a solution for the EmXX_R0_Init issue, just made it so before we apply the scale/speed changes it calls em->move() if the Em needs to init first, move() should then let it init and change state so it won't use init function anymore, then we update scale/speed after that and hopefully it'll stay in-place.
(some Ems might change scale/speed in other non-init funcs though, can't really do much for those...)
Test build if anyone wants to try it out:
dinput8-emListParamExtension-initFixTest.zip
Should hopefully let more Ems work with it, haven't seen any issues caused by calling em->move() early like this yet.

So for this Em Manager is not saving the changes for "ESL-spawned only" right?

Em Manager should be able to update changes onto the game being played for both ESL & non-ESL Ems, but there's nothing to save it to file or anything like that.

The gdrive link gives me "We're sorry. You can't access this item because it is in violation of our Terms of Service" :/
I'm not really sure about trying to make it work with another DLL mod, depends how he handles the ESL I guess, if he's calling the games normal EmList funcs with it then I guess he'd just need to make sure to include the 0xA/0x1C bytes...
Then again I did add that check to make sure it's from ESL file which would probably stop that working - maybe worth testing a build from before I added that check and see if that works? (the v4 build should be fine)

I can say this works fine with the Companion DLL side-chaining ESL files. It might be worth trying to accommodate the enemy_swap.ini feature that is used by the DLL to allow scaling of enemies that are spawned with this feature. I'm still trying to think of a value in the .ini file that we could use... Perhaps a custom health value or something.
I might ask my friend to look into this too, as he recently created a way to spawn enemies with the AEV & ITA working together (it's not capable of issuing health though, so some may not want to use it), however since it is a hardcoded edit to the binary, it would be compatible with other DLLs. It might be as easy as assigning some ITA bytes to write to those ESL bytes @emoose drafted.

One issue though is the collision not scaling with the enemy, so for example, if you make a smaller El Gigante  the collision is still for a big enemy. Scaling enemy collision isn't easy as there are often shared values across various enemies so scaling for one, might affect another. Might be worth checking into though.

The gdrive link gives me "We're sorry. You can't access this item because it is in violation of our Terms of Service" :/ If it's a small file you can probably just ZIP it and drag+drop here.
I'm not really sure about trying to make it work with another DLL mod, depends how he handles the ESL I guess, if he's calling the games normal EmList funcs with it then I guess he'd just need to make sure to include the 0xA/0x1C bytes... Then again I did add that check to make sure it's from ESL file which would probably stop that working - maybe worth testing a build from before I added that check and see if that works? (the v4 build should be fine)
(If there was a way to know if EM_LIST pointer is from stack or not then I could probably remove that ESL file check altogether, doesn't seem there's any easy way to do that though ;_;)

Test.zip
The qingsheng DLL with a secondary ESL in r103

Emoose the main thing to do now is that we can edit the secondary esl of qingsheng that it uses the spawn of the mercenaries, the fcv file follows the changes for the collision to be right, and also we can edit and save everything through your tool
But I will list very important things to get into this your  trainer
Edit case items
Leon's Speed
door locking
Free Camera
Can raise camera altitude
Teleport between maps
Edit ETS and ITA by tool
Push Ashley to you by a button
Push all enemies around you
Place hotkeys to select weapons
All are great additions that would make it perfect and we'd stop using Raz0r's

Emoose the main thing to do now is that we can edit the secondary esl of qingsheng that it uses the spawn of the mercenaries, the fcv file follows the changes for the collision to be right, and also we can edit and save everything through your tool
But I will list very important things to get into this your trainer
Edit case items Leon's Speed door locking Free Camera Can raise camera altitude Teleport between maps Edit ETS and ITA by tool Push Ashley to you by a button Push all enemies around you Place hotkeys to select weapons
All are great additions that would make it perfect and we'd stop using Raz0r's

Teleport between maps already have on debug but thats fine. Just to be clear to emoose and others why we always want to be out of Raz0r Trainer, its because this Trainer destroy the game memory for some reason, the game crashes alot with trainer opened.

If there is one Raz0r trainer feature I can not live without it is the ability to navigate the Z axis with hotkeys. I use this feature every time I am testing something (sometimes to get a closer look at something, or to test triggerzone boundaries etc). This feature would be VERY useful to modders who create stage mods.

The gdrive link gives me "We're sorry. You can't access this item because it is in violation of our Terms of Service" :/ If it's a small file you can probably just ZIP it and drag+drop here.
I'm not really sure about trying to make it work with another DLL mod, depends how he handles the ESL I guess, if he's calling the games normal EmList funcs with it then I guess he'd just need to make sure to include the 0xA/0x1C bytes... Then again I did add that check to make sure it's from ESL file which would probably stop that working - maybe worth testing a build from before I added that check and see if that works? (the v4 build should be fine)
(If there was a way to know if EM_LIST pointer is from stack or not then I could probably remove that ESL file check altogether, doesn't seem there's any easy way to do that though ;_;)

Emoose, do you have discord?

Not really sure how far I'm going to go with this really... mostly focused on inspecting/editing things that are in-engine, to hopefully learn more about some unknowns, editing files is kinda out-of-scope here.
(I know another modder is doing some great work documenting the file formats though, hopefully they'll release what they have soon)
Other than that anything that interacts with the engine is still on the table though, will probably work on adding some more of the trainer patches from @Mister-Curious soon, and will probably add some kind of fly-mode similar to the one in the debug build too.
(this branch is still depending on https://github.com/nipkownix/re4_tweaks/pull/257 being merged though, if you want to see this get released properly, please help us test that!)

Not really sure how far I'm going to go with this really, mostly focused on inspecting/editing things that are in-engine, to hopefully learn more about some unknowns, editing files is kinda out-of-scope here.
(I know another modder is doing some great work documenting the file formats though, hopefully they'll release what they have soon)
Other than that anything that interacts with the engine is still on the table though, will probably work on adding some more of the trainer patches from @Mister-Curious soon, and will probably add some kind of fly-mode similar to the one in the debug build too.
(this branch is still depending on #257 being merged though, if you want to see this get released properly, please help us test that...)

I can't test things about FPS because my computer is very high end, i don't have slow motions, i will tell to kTeo test the performance more. About the size and speed some of then is not scaling size, kTeo made a list of what is working.

We don't really need to test if it fixes slow-motion, just that the game still plays fine with the fix in place, the more people that can try playing through the game with #257 the better.

Fine i will keep testing

Added a setting in trainer to allow noclip-style keypad-movement using the numpad, 2/4/6/8 will move player around, 1/7 will move player up and down, currently it's not affected by camera direction but hoping to fix that soon.
(camera also doesn't seem to get updated properly when moving with numpad, if you hold down the normal walk-forward key while using it that should update camera for you though, will hopefully have a fix soon)

So, this depends on #257, right? How would I build it locally without merging that one? I wanted to try adding some trainer options myself :p

AFAIK you should be able to just check out this branch and build it, the #267 dependency is mainly just the cfgMenu column code update (it's both included here & in 267, so will probably be a pain once 267 is merged heh, was planning on rebasing/redoing this onto master once 267 is in though)

Ah whoops accidentally left something unused there, just force-pushed to fix it, if you checked it out you might need to checkout again 😅

Ahhh, so that must have been why I was having issues :p
Will checkout again. Thanks!

Ohh I think the game has some code that clears the debug flags each frame, EnableDebugMenu nops that out IIRC, can probably just move that to Trainer_Init instead, will look into it soon.

Just pushed an update to the no-clip stuff that I think will improve usability.
It also keeps the collision off if DBG_PL_NOHIT is set, which I believe was the original intent of this flag.
Let me know what you think. You can just revert the commit if you don't think it is a very good approach.

"Warning: this section contains cheats & patches that may ruin your game experience, or break things in the game, are you sure you want to use the trainer menu?"

Feels good

Maybe some kind of warning that shows up first time they click it would be fine "Warning: this section contains cheats & patches that may ruin your game experience, or break things in the game, are you sure you want to use the trainer menu?", hmm...

Good idea. Just pushed a commit with this.

Updated build here:
https://github.com/nipkownix/re4_tweaks/suites/7489440390/artifacts/307455476
@Mister-Curious

Updated build here:
https://github.com/nipkownix/re4_tweaks/suites/7489440390/artifacts/307455476
@Mister-Curious

I tested, it feels very good. A good thing is put a key to toggle Collision on and off like raz0r Trainer, Default is T.

Hey Emoose you figure out some way to save changes? Its possible save the changes directly on ESL for the enemies showed when you turn on "ESL-spawned"?. And if possible change the EmXX for Hex, its everything on decimal and feels confusing, Mr Curious enemy IDs list is on hexadecimal and the game files too. I see a lot of future ahead for the game with these in-game tools. RE4 Tweaks is shining! Thx for attention.

I tested, it feels very good. A good thing is put a key to toggle Collision on and off like raz0r Trainer, Default is T.

YES please.. A hot key is a must for this (@emoose thanks)!

@emoose  I really liked your trainer on Re4 tweaks
These are the options we use the most in the Raz0r trainer, and we want you to implement these options
Raz0r Trainer Inventory Tab
Pull Ashley to you
Pull all enemies to you
Leon's Speed
Disable One-Key Collision
Another thing that would be very useful is that even if you restart the game, the trainer options remain checked

something I would like to add as feedback (and simple request) is when the collision is disabled, that it maintain collision that is on a slope. That is to say, the trainer should not cancel sloped collision. This would allow players to still climb hills, or use stairs (this is the behavior of Raz0r's Trainer).
Cheers


Server side useCookie problem

Environment
Development
Reproduction
Reproduction
Describe the bug
paymentId: string = "1234"
.
.
.
server/middleware/request-handler.ts
setCookie(event, "paymentId", paymentId, {})
.
.
.
pages/checkToken.vue
const paymentId = useCookie("paymentId", { default: () => "", watch: "shallow" })
.
.
.
paymentId is string but when get paymentId this value automatically convert to number
Additional context
No response
Logs
No response

by default string numbers deserialise into numbers but you can customise this behaviour by providing a custom decode function in the options to useCookie.


multi auto link on operator drop

When dropping an operator onto the graph, automatically connect to all possible inputs and outputs.

The design looks good to me except that I think a binary search is an overkill.

Added Jasmine tests.
Minor changes from last iteration:

onDrop now proactively searches for suggested ops. Previously took results from latest search (called by onDrag event)
Added mock operator schema and mock graph data for a dummy op with 3 inputs / 3 outputs

Changes since last iteration:

Amended Jasmine tests to test multiple links being created at once
onDrop now triggers search for suggested operators/links. Previously took results from last search (called by onDrag event)
Added mock operator schema and mock workflow data for a dummy operator with 3 inputs and 3 outputs

Has this branch been merged properly? It still says "4 commits ahead of master." Sorry, a little new to github.

Has this branch been merged properly? It still says "4 commits ahead of master." Sorry, a little new to github.

Not yet, Zuozhi needs to approve it for the 4 new commits you have created to merge into the master branch. Your branch is "ahead of the master" means there are commits that exists on your branch but not on the master, vice versa.

In general this PR is very good. I have left some more comments, but it's mostly related to coding styles.
I found a bug when testing this feature, when dragging an operator just outside of the threshold range, the highlighting disappears, but when dropping it, the link is still added. Please investigate this bug. Thank you.

In general this PR is very good. I have left some more comments, but it's mostly related to coding styles.
I found a bug when testing this feature, when dragging an operator just outside of the threshold range, the highlighting disappears, but when dropping it, the link is still added. Please investigate this bug. Thank you.

Turns out an "improvement" I made a few commits ago caused this: upon dropping the floating operator, we would recalculate the nearby operators and connect to those. The logic was that the last set of mouse coordinates would yield the most accurate results. However this is sometimes inconsistent with the operators highlighted by the mouseMove event handler (still not totally sure why), even though the selection criteria is the same. I reverted to using the last calculated nearby operators (calculated from the last mouseMove). The highlighted operators are now always the ones that the new operator connects to.

@MysteriousChallenger the changes look good. When I'm testing it, I found another bug, please fix it.
There's also a place where you added ts-ignore, I understand the reasoning, but I prefer not to add ts-ignore. In order to work around the undefined check, please use something like:
const top = inputOps.peek();
if (top) {
// more code
}

@MysteriousChallenger the changes look good. When I'm testing it, I found another bug, please fix it.
There's also a place where you added ts-ignore, I understand the reasoning, but I prefer not to add ts-ignore. In order to work around the undefined check, please use something like:
const top = inputOps.peek();
if (top) {
// more code
}

That was embarrassing: I mixed some variable names and even misplaced an if-else block. It is fixed for sure this time, so help me god. I ran unit tests and tried it myself before this commit.
sorry for taking up more time


postgres config for pScheduler

Still need to add commands to initialize postgress for pscheduler use

If you look at the docker build output, you'll see lots of errors on the pscheduler stuff.
Looks like this sort of stuff will need to be scripted and added to the DockerFile:
see: pscheduler-server.spec; the 'build', 'install', 'pre', and 'post' section all have commands that will need to be executed, I think.

see if running  "pscheduler internal db-update" will do this.

The above command does not work, and gives this error:
    Unable to determine PostgreSQL user.
I assume this script assumes default locations for files, but all files will need to be in something other than /var
Full log of install errors is available here: https://hub.docker.com/r/bltierney/perfsonar4.0-testpoint-docker/builds/bwvswmmk8c4dlw5ebhf59qb/

That error happens when postgresql-load attempts to determine what user is running PostgreSQL by looking for the server in the process table.  Pg needs to be running for the database to be set up.
During the build of the container, try this:
RUN /usr/pgsql-9.5/bin/pg_ctl start
RUN pscheduler internal db-update
RUN /usr/pgsql-9.5/bin/pg_ctl stop

Now it gives this error:
cannot be run as root
Please log in (using, e.g., "su") as the (unprivileged) user that will
own the server process.

I think a script that wraps the init stuff together will be needed.

Probably all you need to do here is run it as postgres.

Step 8/14 : RUN su postgres -c "/usr/pgsql-9.5/bin/pg_ctl start"
---> Running in d64567def4ff
pg_ctl: no database directory specified and environment variable PGDATA unset
Try "pg_ctl --help" for more information.

probably need to do something like this:
https://github.com/zokeber/docker-postgresql/blob/master/Dockerfile
But I'm not sure what all 'pscheduler internal db-update' is doing for us.

I updated the Dockerfile, but now I'm getting this error:
RUN pscheduler internal db-update
Unable to determine PostgreSQL user.

But I'm not sure what all 'pscheduler internal db-update' is doing for us.

That's what builds or updates the database.  The script that does it is structured so you can run it as many times as you want and you'll always wind up with the current version of the database whether you're starting from nothing, have an existing-but-older version or are fully current.
Maybe we should just get together and work on this.

seems to be working now, but more testing needed before closing.


updates from today's discussion

The changes to Readme seem useful. The change to the code is inconsistent with existing code - at the same time some code (e.g. Csharp) is already written that way to use hardcoded type, others (e.g. PHP) do not. So no real issue because it a small inconsistency. @emilybache this could be merged.

Thanks! Apologies for not noticing this PR before now


Add PlanNodeSerde test for aggregation with multiple global groups

@mbasmanova : Realized I should add this test after https://github.com/facebookincubator/velox/pull/7112 review.


inconsistent data tab in editor panel when sql error, error when change from analysis tab

first of all sorry for the long title, I didn't know how to best write it

this is a series of issues that while related, they can be taken apart separately, first of all I find it weird when applying an error SQL query I can go to the data in the map and see the error

however, when I disable the sql the data tab still shows data from before appying the query, so if we show an error in one side I think we should be consistent at least not showing the data

besides this, if instead of directly disable sql you go to "analysis" and then back to "data" instead of showing the sql panel it shows the data panel indeed now with no data but an infinite load

/cc @nobuti for the sql panel
/cc @saleiva for inconsistency issue
/cc @xavijam  for general awareness
not sure who to assign tho, maybe this can be tackled at the same time idk

Closed by #9563.


Cannot build because of USERAUTH when running the helm goal in the fabric8 mvn plugin

[ERROR] Failed to execute goal io.fabric8:fabric8-maven-plugin:2.2.82:helm (helm) on project artifactory: Execution helm of goal io.fabric8:fabric8-maven-plugin:2.2.82:helm failed: Failed to clone chart repo git@github.com:fabric8io/charts.git due: git@github.com:fabric8io/charts.git: USERAUTH fail -> [Help 1]

Stack looks like this:
http://pastebin.com/5xbwqbXV
Debugging shows that this is caused when using a private key which requires a passphrase to connect to git/github.
PR on its way to fix this

I still have this issue, however only when running mvn clean install in the top-level directory. When using mvn install it works, also mvn package fabric8:helm.

Does your private key for github.com require a password? Check the helm
docs real quick I added props you can use to pass in the password from the
CLI for your private key
http://fabric8.io/guide/mavenFabric8Helm.html
On Sunday, January 24, 2016, Roland Huß<EMAIL_ADDRESS>wrote:

I still have this issue, however only when running mvn clean install in
the top-level directory. When using mvn install it works, also mvn
package fabric8:helm.
—
Reply to this email directly or view it on GitHub
https://github.com/fabric8io/fabric8-devops/issues/242#issuecomment-174278675
.

--
Christian Posta
twitter: @christianposta
http://www.christianposta.com/blog
http://fabric8.io

Didn't try that yet but I wonder why I would need my private key when doing a local mvn install (without a fabric8:helm-push).
Btw, would be cool if an ssh-agent could be supported so that the private ssh password doesn't need to be exposed as a property (and because I dont remember which of the many ssh keys I use for github ;-)
Will go now looking for my github key ...


Support for status check app list in Terraform.

Github status checks can be enabled; however, status check patterns which are required for that to work, are not supported.
When you add a required status check, you can select an app that has recently set this check as the expected source of status updates.

https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/defining-the-mergeability-of-pull-requests/about-protected-branches#require-status-checks-before-merging

Nvm, I realize this is isn't buildkite specific :D


INT B-20879 SIT panel & SIT extension fixes and calculation fixes

Agility ticket
Summary
When demoing SIT functionality to HomeSafe, it was found that there were some calculations that were a little off in the SIT panel and when reviewing/editing SIT extension dates/days. In SIT calculations, we had two instances where we were subtracting days (Truss did this for some weird reason) and that was causing the SIT panel to say and show inconsistent information when changing things in the SIT modal.
There were also some suggested UI fixes of wording.
This PR does the following:

added a sitFormatters JSX component that contains reusable functions and renders used by ReviewSITExtensionModal and SubmitSITExtensionModal
refactored both above components to use the shared utility functions to simplify the logic and help further changes needing to be in once place if they occur
adjusted calculations to reflect request changes from the gov
added some stylings that improved the alignment and readability of some components (like those stupid date picker boxes being misaligned)
updated playwright tests
added relevant tests

How to test

Access MilMove as an office user
Find a move or add origin/destination SIT to a move
Approve the SIT items as TOO
You should now be able to see the SIT panel data
I used this tool to calculate days
Keep in mind that we want to include the SIT entry date & the SIT departure date in the calculations
In the Prime Sim, request a SIT extension for that shipment
Go back as the TOO and review the request, change some dates/values around and make sure that nothing is wonky or inaccurate

Screenshots
SIT panel & Edit SIT authorization should reflect the same data
SIT panel

SIT auth edit (AKA SubmitSITExtensionModal

SIT panel & review SIT extension modal should reflect similar data
SIT panel

SIT extension review modal (AKA ReviewSITExtensionModal)

Showing that when I change the approved days, the SIT auth end date also changes. Biggest thing here is the Approved + Requested date should match up when you put input the max number of days that can be approved.
Also showing that it works backwards.
https://github.com/user-attachments/assets/3ccaa9fc-9858-452a-9ff5-ba5de4a6832e

Flaky tests failing


Merge in Tests and changes from Master

Merged in functional and unit tests from master as well as small patches found while testing, see PR #34
Also merged in contribution guidelines from master see PR #37.
After merging I ran the unit anf functional tests to ensure they passed.
Most did but found some issues that I fixed up in commit b14ec355fc364806bffb847e32666f11fc305dc9.
Now tests are passing within v1.0-dev.

Change CRLF to LF in itoolkit.js and ixml.js, then it looks good!

I think this got introduced when merging with master. Added commit to resolve issue.


onMainThread not fired

Hi
I use "SwiftEventBus.post" when user clicks on the notification and open the app.
but in another side "onMainThread" not fired.
is there an issue or I miss up some thing.
can you help me?
Thanks

Can you share more details please? Maybe post some code?

The app is in killed (not running and not in background), and new notification arrived and user clicks on it in notification center,  it will open the app.
And notification handler is fired and read notification data.
this I call
SwiftEventBus.post("receiveChatNotification")
but
SwiftEventBus.onMainThread(self, name: "receiveChatNotification") { (notification) in
            print("receiveChatNotification")
        }
not fired and didn't print anything on log.

Two things that come to my mind:
Are you sure the post gets called?
Who is the listener? AppDelegate or another ViewController? If it's another view controller, make sure it has been loaded, otherwise it doesn't exist, and therefore can't listen to the event, right? :)

so it must make AppDelegate the listener?

Depends, on what kind of logic you want to implement. If you want to show something to the user you need to load your view controller before sending the post. If you want your view controller to be the listener it has to be loaded, right? If an object doesn't exist, it can't do much. :) 💀

I need to open chatViewController after a user makes click on the notification.  with two situations ( an app is running and another viewController appear - an app is not running).
so I need to make appDelegate is the listener. is this best practices?

My advice is that you load your navigation (supposing you need to go through 3 ou 4 screens before reaching that screen responsible) and you either pass the information down the chain (app delegate instanciates root view controller, and passes that info, then root VC passes to VC1, which passes to your destination VC), or you make the post on the app delegate after you know the app delegate has been loaded. I'm not sure what the best practice is, it really depends on your case.
You can also save that info on a variable (static var, user defaults, or even database...) and use it on the view controller when it loads.


Soft link for environment.yml

In the binder directory, have a soft link for environment.yml instead of for requirements.txt.

Closes #11

I think it was meant that #11 would close #10.

Yes, that is how it was meant. Thank you!


Floating point precision is off

The below code produces the output "1.5" when it should be 1.65
module Main

import "std/math.earl"

fn main() {

    let lstf = [2.2,1.1,2.2,1.1];

    println(Math::mean(lstf));
}

main();

@pub fn mean(lst) {
    return (float(List::sum(lst)) / float(len(lst)));
}

1.5

On top of that, .5+.1 != .6 when it should equal it.


Missing details on activation of Azure Hybrid Benefit using the Azure Portal.

Page states "You can choose or change your licensing model by using the Azure portal or..." but there are no precise indication about how to do it.

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

ID: 00f3b642-1d0d-363b-ed6e-a52dd93090e6
Version Independent ID: 5f3d7946-3455-650e-d21b-c2601495e876
Content: Azure Hybrid Benefit - Azure SQL Database
Content Source: articles/sql-database/sql-database-azure-hybrid-benefit.md
Service: sql-database
Sub-service: service
GitHub Login: @stevestein
Microsoft Alias: sstein

@LudoBoggioTotal Thank you for bringing this to our attention. I went through multiple links and ended up without any information on how to accomplish this task. I will have the content team review this and update the documentation with a link to the information or verbiage pertaining to how this can be done.

@LudoBoggioTotal - thanks for the feedback. I added info about how to turn on the AHB in the portal.

#please-close


compare tun2proxy via tun2socks

i tested both on linux and tun2proxy have more speed for me
do you use tun2socks api?
its very good

Internally, tun2proxy makes use of the smoltcp network stack. In my small manual tests, tun2socks was about 20% faster. I suspect the reason is that smoltcp lacks congestion control.


spriv-dis: Use max_digits10 instead of digits10 for float output precision

max_digits10 is needed for binary->text->binary, essentially, when one wants to disassemble and (re)assemble. digits10 is not sufficient and was causing lossiness for this process. One only need try the float32 value 16777215 which should be exactly representable but was getting truncated by spirv-dis. A test has been added for this value. A number of other tests needed to be "fixed" to take this new output precision into account.

I'd much rather resort to forcing hex-floats for cases you care about.
See https://github.com/KhronosGroup/SPIRV-Tools/issues/882
The parser supports hex-float for bit-to-bit accurate recovery of a float number.  The disassembler doesn't support that yet but it would be relatively easy.

Are you trying to fix a portability of the tests or a "real" use case?
For absolute fidelity I'd rather implement hex-float option for disassembly #882.

For what it's worth, reliable base-10 printing and parsing of floating point numbers is a subject of ongoing research.
E.e. http://cseweb.ucsd.edu/~lerner/papers/fp-printing-popl16.pdf
I don't expect implementations to fall in line any time soon.  Hex floats are the way to go here.

While I was looking at floating point rounding issues in constant folding, I noticed that spirv-dis was dropping 2-3 digits of the significand of floats. I thought it strange that we were worrying about rounding mode accuracy while at the same time dropping 2-3 digits in spirv-dis. While graphics is probably not as picky about precision as HPC, it seemed that this would be something that HPC users would eventually bump into and care about so I was trying to be proactive. I have always thought that the ability to disassemble, edit and reassemble SPIR-V was a real strength of our ecosystem, and I thought it would be a good idea to not be causing inadvertant significant changes if possible.
I looked into this and it seemed fairly obvious to me that max_digits10 was a better precision here than digits10.
I don't doubt that hex-float is the way to go for high fidelity. But I thought if we could fix 99.999% of the problems with a one word change, that might be a good thing until something "better" could be settled on and implemented.

Well, I should have anticipated that this change would hit other tests as well :(
@dneto Is the change to hex-float a done deal? Is it imminent? If yes to both, than it probably doesn't make sense spending time fixing these tests. Otherwise, maybe.

@dneto OK. So it seems your thinking is to only do hex-float as an option.
Are you interested in a less-lossy default ie this change? If so, I will continue fixing this PR. If not, we can just drop it. Let me know.

Finally getting back to this.

I don't doubt that hex-float is the way to go for high fidelity. But I thought if we could fix 99.999% of the problems with a one word change, that might be a good thing until something "better" could be settled on and implemented.

I'll by the "why not both" argument.  If you want to continue down this path then go for it. The hex-float option is not imminent.
When there are spurious-looking changes, e.g. when we have a random float value that isn't testing a specific limit, I suggest fixing the constants to be more binary-friendly. (e.g. .375 instead of .275 ).

I'm not sure whether you saw the extra failures found by the bots.  Is there a platform sensitivity? That would be concerning.

Superceded by #1444

#1444 is merged. I'll close this one.


chore: exclude techdocs from the .csv (RHIDP-4196)

What does this PR do?
chore: exclude techdocs from the .csv (RHIDP-4196); also fix a typo in modules/dynamic-plugins/proc-rhdh-example-external-dynamic-plugins.adoc
Signed-off-by: Nick Boldt<EMAIL_ADDRESS>Screenshot/screencast of this PR
N/A
What issues does this PR fix or reference?
https://issues.redhat.com/browse/RHIDP-4196
How to test this PR?
N/A
PR Checklist
As the author of this Pull Request I made sure that:

[x] Code produced is complete
[ ] Code builds without errors
[ ] Tests are covering the bugfix
[ ] Relevant user documentation updated
[ ] Relevant contributing documentation updated

Reviewers
Reviewers, please comment how you tested the PR when approving it.

will you merge it or are you waiting for someone else to do so? I could merge it if you'd like

/cherrypick release-1.3


[Bug] [connector-file] When the data source field is less than the target (Hive) field, the default value is automatically added.

Search before asking

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

What happened
When the data source fields are less than the target (Hive) fields, a null pointer error will be reported.
SeaTunnel Version
seatunnel-2.3.7-release
SeaTunnel Config
{
  env {
    execution.parallelism = 1
    job.mode = "BATCH"
    job.name = "seatunnel_batch_job"
  }

  source {
    JDBC {
      url = "jdbc:mysql://ip:port/?serverTimezone=Asia/Shanghai"
      driver = "com.mysql.jdbc.Driver"
      user = "root"
      password = "***"
      query = "select * from `test`.`testtable` where 1 = 1"
      split.size = 5000
      fetch_size = 2000
      parallelism = 1
      result_table_name = "result_table"
    }
  }

  sink {
    Hive {
      table_name = "test.testtable"
      metastore_uri = "thrift://ip1:9083,thrift://ip2:9083"
      hdfs_site_path = "/usr/local/hadoop3/etc/hadoop/hdfs-site.xml"
      hive_site_path = "/usr/local/hive/conf/hive-site.xml"
      parallelism = "1"
    }
  }

Running Command
./bin/start-seatunnel-flink-15-connector-v2.sh --config config/mysql2hive.config

Error Exception
Caused by: java.lang.NullPointerException
       at org.apache.seatunnel.connectors.seatunnel.file.sink.writer.OrcWriteStrategy.buildSchemaWithRowType(OrcWriteStrategy.java:196)
org.apache.seatunnel.connectors.seatunnel.file.sink.writer.OrcWriteStrategy.getOrCreateWriter(OrcWriteStrategy.java:115)
org.apache.seatunnel.connectors.seatunnel.file.sink.writer.OrcWriteStrategy.write(OrcWriteStrategy.java:74)
....

Zeta or Flink or Spark Version
No response
Java or Scala Version
No response
Screenshots
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

Well, can u offer the table schema in MySQL and Hive. And do you mean that, the missing fields should be null? Rather than some default value?


Update to Swift 2.0

I did not do much apart from running it through the Xcode converter. The main changes are instead of using the full names for calendar components (eg CalendarUnitYear) we now use the enum (eg .Year).

:+1:

I think it is too early to merge, but there is a need to want to try it.
One idea, Create swift-2.0 branch, and merge it to swift-2.0.
What about?

Hi everyone, sorry for the radio silence; I'll be checking this as soon as possible! Thanks for the support and definitely we gotta move to Swift 2.0.

@akosma any update on moving to Swift 2.0?

Xcode 7 with Swift 2.0 is just around the corner! We need you SwiftMoment!

Xcode 7 with Swift 2.0 is here!  Any estimate on a SwiftMoment update?

@akosma Can I help you?

I'm going to merge #27 since it is more recent. Thanks a lot and sorry for the time it took to review!


fix(provider/oracle): fix missing oci-sdk jars in web startScript

Becasue the oci-sdk jars are not published in maven (https://github.com/oracle/oci-java-sdk/issues/25), the dependency are 'dynamically' declared during fetching and unpacking the oci-sdk. If ':front50-web:startScripts' runs before ':front50-oracle:unpackSdk' finished, the oci-sdk jars then could be missing in the generated script. The fix adds dependsOn and mustRunAfter ':front50-oracle:unpackSdk' to front50-web:startScripts

@spinnaker/reviewers
related PRs:

Same comment as https://github.com/spinnaker/clouddriver/pull/3160#issuecomment-441732017

Same comment as spinnaker/clouddriver#3160 (comment)

Made the 'dependsOn' and 'mustRunAfter' conditional as suggested.


Visualize current/all branches doesn't work

When trying to visualize a branch, the following error is encountered:
Error in startup script: can't find package msgcat
while executing
"package require msgcat"
(file "/snap/git-cola/136/usr/bin/gitk" line 12085)
Tried with and without --devmode, with the same results.

Just confirming this on Ubuntu 19.10 with a slightly different printout:
git-cola
Gtk-Message: Failed to load module "gail"
Gtk-Message: Failed to load module "atk-bridge"
Gtk-Message: Failed to load module "canberra-gtk-module"
Qt: Session management error: None of the authentication protocols specified are supported
Error in startup script: can't find package msgcat
while executing
"package require msgcat"
(file "/snap/git-cola/136/usr/bin/gitk" line 12085)

Can confirm this too. I hope to see this fixed.


Developer Portal Security via Open ID Connect

I see that Developer Portal is protected via Username/Pwd and its possible to use AD, ADB2C, Google, Microsoft. Is it possible to use other SSO solutions like Okta to allow access to Developer Portal? In Azure portal in Security section, its listed in Menu but does not state if this applies to Developer Portal access or not. Can it be elaborated here please?
Thanks.

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

ID: a1c96ba5-164a-e66f-a782-d0cd60a500f3
Version Independent ID: c0f01676-6a46-99a7-a56f-3b2d6eb5e965
Content: Access and customize the new developer portal - Azure API Management
Content Source: articles/api-management/api-management-howto-developer-portal.md
Service: api-management
GitHub Login: @mikebudzynski
Microsoft Alias: apimpm

@ashishmundrajll Thank you for your posting your question! We will review and provide an update as appropriate.
@mikebudzynski any thought here from the product side?

You would need to use authentication delegation. Contact Azure support for assistance if needed.
#please-close


Fix IE9 compatibility by expliticly stating empty href urls

TL;DR: In IE9 it was not possible to give focus to accordion items or tabs via the keyboard.
IE9 doesn't recognise a simple href attribute unless you give it a value href="", and it seems to strip them out which results in the link no longer being a tab stop.

Closing this as we actually did do this at one point, but this turns out to trigger routing issues in some instances.
Please open an issue with a reproduction if this is an issue.


Call Dispose/DisposeAsync on BufferedStream instead of Flush/FlushAsync

Dispose and DisposeAsync call flush internally and execute additional cleanup logic.

Thanks for this. Let's be testing the current master for a while at this stage of the async changes.


Package won't install with ocaml 4.06.1 on mac os x

I tried to install lymp with ocaml-4.06.1 on mac OS X, and it seems that the required dependency bson won't build using this version of ocaml. There is no problem with the 4.05 version though.
The following actions will be performed:
  ∗  install bson 0.89.3                      [required by lymp]
  ∗  install lymp 0.2.3
===== ∗  2 =====
Do you want to continue ? [Y/n] y

=-=- Gathering sources =-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=  🐫
[bson] Archive in cache
[lymp] Archive in cache

=-=- Processing actions -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=  🐫
[ERROR] The compilation of bson failed at "ocaml setup.ml -configure --disable-syntax --disable-client --prefix /Users/xxx/.opam/4.06.1".
Processing  1/2: [bson: ocamlfind remove]
#=== ERROR while installing bson.0.89.3 =======================================#
# opam-version 1.2.2
# os           darwin
# command      ocaml setup.ml -configure --disable-syntax --disable-client --prefix /Users/xxx/.opam/4.06.1
# path         /Users/xxx/.opam/4.06.1/build/bson.0.89.3
# compiler     4.06.1
# exit-code    2
# env-file     /Users/xxx/.opam/4.06.1/build/bson.0.89.3/bson-47809-d2c37b.env
# stdout-file  /Users/xxx/.opam/4.06.1/build/bson.0.89.3/bson-47809-d2c37b.out
# stderr-file  /Users/xxx/.opam/4.06.1/build/bson.0.89.3/bson-47809-d2c37b.err
### stderr ###
# File "./setup.ml", line 247, characters 8-26:
# Warning 3: deprecated: String.set
# Use Bytes.set instead.
# File "./setup.ml", line 247, characters 8-11:
# Error: This expression has type string but an expression was expected of type
#          bytes

Just to check, I also tried it with the ocaml/opam:ubuntu-16.04_ocaml-4.06.0 docker image, and the same problem arises as well (as expected).

Thank you very much for the report. Indeed, bson fails to build because of the old oasis setup.ml. I made a PR to fix it (here : https://github.com/MassD/bson/pull/18), I'll let you know how it goes.

Hi @groar , sorry for the delay. I included bson in the sources and published the result in lymp 0.2.4. You can update opam (opam update), the installation should work fine now. Thanks again for reporting :)


Deprecate VideoCodecContext.gop_size for decoders

Update to Issue https://github.com/PyAV-Org/PyAV/issues/1254.
Implemented jlaine's suggestions to include a docstring, a warning, and a test.

Great job, if you can squash down your commits into a single commit with the message: Deprecate VideoCodecContext.gop_size for decoders (fixes: #1254) I'm happy to merge it!

@jlaine Squashed. Your assistance has been sincerely appreciated.

@JoeUgly thanks for your contribution, it has been merged!


unreliable benchmarks results

Seems like results provided by asv are unreliable. Here are few latest runs from our StartupBench runs, compared to runs on my computer:

Both runs on the bottom of image are on my computer, one is the same number of runs as master (100) and other is 800.
Maximum disrepancy between those two runs is ~ 4%.
Working theory so far: I set number of processes to 2, and this is the number of processes of the runner, so it might be we are fighting for resources with GHA runner and other background processes.
I will try reducing the number of processes to 1. If it won't help, it might be another vote for using our own machines.

#132 should fix it
Leaving open until we see the results.

After #132, #159 and #154  Situation got a bit better, seems like results for windows and linux (real use cases, Add) are stable. #172 Will be a workaround that will silence less important benchmarks.
We still have problems with Darwin.

After https://github.com/iterative/dvc-bench/pull/211 situation got much better, though #221 and #220 show that issue is still not resolved.

Closing, afterhttps://github.com/iterative/dvc-bench/pull/296 we no longer use asv. We will reopen if we notice inconsistency in pytest based benchmarks.


update jetty

Update jetty to the latest version, adapting the configuration files (notably start.ini and the init.d script).

When lambdas from Java 1.8 are used in the source, a newer version of the asm library is required (v5).  Without the updated library, the class loading fails preventing the SlipStream server from starting. The latest Jetty versions use an acceptable version of the asm library.

When upgrading to the latest Jetty without changing the configuration files, the following errors appear in the log:
WARNING: The behavior of the argument OPTIONS=Server,resources,ext (seen in /opt/slipstream/server/start.ini) has changed, and is now considered a normal property.  OPTIONS no longer controls what libraries are on your classpath, use --module instead. See --help for details.
WARNING: Property OPTIONS in /opt/slipstream/server/start.ini already set in /opt/slipstream/server/start.ini
WARNING: The behavior of the argument OPTIONS=jsp (seen in /opt/slipstream/server/start.ini) has changed, and is now considered a normal property.  OPTIONS no longer controls what libraries are on your classpath, use --module instead. See --help for details.
WARNING: Property OPTIONS in /opt/slipstream/server/start.ini already set in /opt/slipstream/server/start.ini
WARNING: The behavior of the argument OPTIONS=logging (seen in /opt/slipstream/server/start.ini) has changed, and is now considered a normal property.  OPTIONS no longer controls what libraries are on your classpath, use --module instead. See --help for details.
Unrecognized argument: "start.d/" in /opt/slipstream/server/start.ini

Points to check:

[ ] servlet version is OK
[ ] is --exec needed in jetty.conf?
[ ] is JSP module needed?
[ ] is JSTL module needed?
[ ] verify that logging works correctly


Remove deprecated ocio_config ImageIOConfigModel settings

Changelog Description
Remove deprecated ocio_config ImageIOConfigModel settings
Additional info
With the support of profiles to determine OCIO config in ayon-core this logic has been superseded and should be removed from each addon still having the deprecated settings.
Fixes #46

It works on my side. Thanks.
Question (mostly irrelevant to this PR) Should this renamed to Enable Global Color Management ?

If that screenshot is for the setting on ayon-core addon, then yes - that may be nice for clarity!
If it's for Ayon-houdini, then no.

If that screenshot is for the setting on ayon-core addon

It's from ayon-core.


[Star Wars Saga Edition] Skill display fix

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

Changes / Description

I hate this update.
Anyway, the skill values on the sheet update properly when attributes or level changes now.
Also added the repeatable Force Powers section to the NPC sheet.

Character Sheet Info Roll20 Internal Use only.


First pass at implementing a network management API

This is a tracking bug for work going around /1.0/networks.
We're looking at adding:

Ability to create new bridges managed by LXD
Configure them (IPv4, IPv6, tunneling, port security, backend driver, ...)
Add containers to them
Delete

I'm working on this in my "network" branch, adding all the API bits and all the new configuration keys.
This work will affect both the daemon (big API extension), client (support for "lxc network ..." commands) and "lxd init" (to setup an initial network).

Woohooo!! +1000

Cool. I'm thinking about nat disabling option per container right now without creating extra bridges, for example for default lxdbr0 bridge.
Seems like lxd-bridge iptables rule makes masquerade (POSTROUTING) for full lxd bridge subnet right now. And rejecting just forward per ip didn't work.  And I don't think that split POSTROUTING rules is a right way anywhere beside my test machine.
Anyway, if creation of new bridges will be just easy as editing /etc/default/lxd-bridge...
Basically what I trying to say, it will be wonderful to see new improved network management for sure.


Update container status properly in case of kata_builtin_shim

When we enable kata_builtin_shim, there is no shim process and thus the shim pid is not available for representing the container liveness.

Is the above PR ready to go ?

@tedyu - you mean did anybody actually submit a PR to remove the shim pid.... /cc @bergwolf @egernst - any idea?


vimPlugins.select-and-search: init at 2014-08-04, vimPlugins.indent-detector-vim: init at 2015-07-06

Motivation for this change
closes: #158492
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

#158492 will be prefered


FILE_CONENT property proposal.

We already have a CONTENT property which is used for CONFIG_FILE nodes. I did not want to reuse it to avoid confusion. As for the offsets: Yes, i can just name them OFFSET end OFFSET_END.

Ok, i now reuse the already existing CONTENT property.


some err about caffe op with multiple gpu

It work well when using caffe op with a  single gpu.
Howerer, when using caffe op with multiple gpus, I printed all gradients of  operators above caffe op and find them all zeros

@HrWangChengdu could you have a look.
Could you also provide a simple code snipplet that produce this problem?

data = mx.sym.Variable('data')
data = mx.sym.CaffeOp(data_0=data, num_weight=2,
prototxt='''layer{type:"Convolution" convolution_param { num_output: 128 pad: 1 kernel_size: 3 stride: 1 }}''')
...
model = mx.model.FeedForward( 		symbol=data ........)
model.fit( ..... , kvstore=local_allreduce_device)

Could you run mxnet/example/caffe/caffe_net.py on multi-gpu? That works fine for me.

I run the example . when I use kv-store = 'local' ,it works fine.
But if kv-store = 'local_allreduce_device', train-top-accuracy don't increase.

@mli @tqchen

@HrWangChengdu This sound like some synchronization issue. Caffe probably use the default stream (Does it? I'm not sure) so you need to sync on the default stream at the end of caffe op

@piiswrong thx. I'm checking on this.

@piiswrong I guess this issue could be closed.


Allow to set absolute parameter in signedRoutes

This PR add the $absolute parameter that is already defined in the src/Illuminate/Routing/UrlGenerator.php@route method for the signed routes.
Basically, in my case I need only the path with the query params (because I will override the host), so the only way to do that is with str_replace or something like parse_url($url, PHP_URL_PATH) . parse_url($url, PHP_URL_QUERY); not a big deal, but will be cleaner if we can use the parameter that already exists for the route.

We can't change method definitions on a point release.


[Bug]: --Privileged Flag Not Working on Build anymore After Install

Description
I am facing an issue with Coolify where the --privileged flag is not working as expected. This results in a validation error when running a docker compose command since the latest update.
And when i remove it its working great.
Minimal Reproduction (if possible, example repository)

Be in the latest version : ()
Add Custom Docker Options : --privileged
Trigger a Deploy and observe

Exception or Error
COMMAND:
docker exec y4gg44o bash -c 'SOURCE_COMMIT=320e9a09c8e4a7e4d5210c1c4028eea57a2a36e1 docker compose --project-directory /artifacts/y4gg44o -f /artifacts/y4gg44o/docker-compose.yml up --build -d'
OUTPUT:
time="2024-02-13T14:15:51Z" level=warning msg="The "bnHHN48" variable is not set. Defaulting to a blank string."
time="2024-02-13T14:15:51Z" level=warning msg="The "bnHHN48" variable is not set. Defaulting to a blank string."
validating /artifacts/y4gg44o/docker-compose.yml: services.r0wks0s.privileged must be a boolean
[2024-Feb-13 14:15:51.447080] Oops something is not okay, are you okay? 😢
[2024-Feb-13 14:15:51.453738] time="2024-02-13T14:15:51Z" level=warning msg="The "bnHHN48" variable is not set. Defaulting to a blank string."
time="2024-02-13T14:15:51Z" level=warning msg="The "bnHHN48" variable is not set. Defaulting to a blank string."
time="2024-02-13T14:15:51Z" level=warning msg="The "bnHHN48" variable is not set. Defaulting to a blank string."
validating /artifacts/y4gg44o/docker-compose.yml: services.r0wks0s.privileged must be a boolean
[2024-Feb-13 14:15:51.459631] Deployment failed. Removing the new version of your application.

Version
v4.0.0-beta.213

It is fixed with #1741


Newmodels editor

This PR is not to be merged now. It's just a way to see the changes more easily.

v2.2.0 (correspond to newmodels version) released.

v3.2.0 published to match new newmodels release.
Also updated the codebase from mtasa-resources upstream (it fixed some bugs in the editor etc)


i18n(es): add microsoft azure

What kind of changes does this PR include?

New or updated content
Translated content

Description

What does this PR change?  Adds Microsoft Azure deploy guide in Spanish

Good job @fredoist, @agustinmulet! Thanks for this :raised_hands:


Program New Interface

Write new interface for edit event and past event.

The new design is in the repo, under the Design folder

Split into multiple, smaller issues


App crashes in tcp client mode

I was following the example in your docs, specifically "When MMDAgent-EX serves as the client - Communication Using TCP/IP," but the app crashes with the error shown in the screenshot below.

Further inspection into your source code leads to an attempt that manages to fix the crash in this PR. It avoids calling FD_ISSET with a negative value (m_server_sd = -1) when using TCP client mode. However, I'm not entirely sure if this is the best approach, so I'm opening this issue to discuss further.
For additional context, a colleague tested this on macOS without issues, but I am experiencing the crash on Ubuntu 24.10.

Thanks you for your contribution to MMDAgent-EX!  I confirmed the problem and merged your PR.


Trying to use a shadow with a non-white text value.

I'm trying to apply colour to a font with a baked shadow which needs to be a different colour. When applying a hexcode, the entire draw including the shadow is the applied colour. The expected behaviour would be that the shadow remains the baked colour regardless.
scribble_font_bake_shadow("font", "white_shadow", 0, 1, #f2f0e5, 1, 0, false);
scribble("[#f32b2b][white_shadow]This should be red with a white shadow, but it's all red.");

Scribble bakes the shadow into the font graphic itself. Colour blending is performed by multiplying the colour with the source font. This means the shadow is affected by the blend colour.
This is inteded behaviour insofar as it is a known side effect of the feature.

Update 9.2 addresses this ticket.


Added open and close events to leanModal.js

The objective is to notify any interested part when the modal has been opened and closed. LeanModal will now trigger the "opening" event when it's starting to fade in, an "opened" event after it has been opened,  "closing" event when it's starting to fade out, and a "closed" event after it has been closed.
This was motivated by a lack of elegant options when trying to hide the page's scroll bars after opening the modal.

Doesn't this feature already exist?
On Feb 11, 2015 9:59 AM, "Urik"<EMAIL_ADDRESS>wrote:

The objective is to notify any interested part when the modal has been
opened and closed. LeanModal will now trigger the "opening" event when it's
starting to fade in, an "opened" event after it has been opened, "closing"
event when it's starting to fade out, and a "closed" event after it has
been closed.
This was motivated by a lack of elegant options when trying to hide the
page's scroll bars after opening the modal.
You can view, comment on, or merge this pull request online at:
https://github.com/Dogfalo/materialize/pull/666
Commit Summary

Added events to leanModal.js

File Changes

M js/leanModal.js
https://github.com/Dogfalo/materialize/pull/666/files#diff-0 (5)

Patch Links:

https://github.com/Dogfalo/materialize/pull/666.patch
https://github.com/Dogfalo/materialize/pull/666.diff

—
Reply to this email directly or view it on GitHub
https://github.com/Dogfalo/materialize/pull/666.

You seem to be right, I misinterpreted the ready and complete callbacks. Sorry about that.

I have checked with my partners and we still need to be notified about the showing and closing events prior to the animation's completion. Might I add another callback in the style you implemented the ready and complete callbacks to be called right before the fade in and fade out animations start?

We're going to leave our callbacks as is for now.


🔹[EPIC] Migrate docs to be a gateway

What ?

High Level Goal: Ideally, we will adapt the current docusaurus app for it to become a gateway. This means: the docs can properly embed components, and people can login using their Near to interact with them.

UX Designs: The design should not change

Why ?

Background: A big part of our documentation is dedicated to Near components, which are currently embed as iframes, and thus are mostly non-interactive. Adding the capability of natively including components will not only allow to better embed them, but also to make more interactive documentation. Besides, if we allow people to login, we might even add more features such as automatically deploying smart contracts for the user, or readily including the ability to mint NFTs from the docs, etc.

Move the Needle: I would expect this to increase user acquisition and retention.

How ?

Completion Requirements -> the Definition of Done:

As a user, when I land on docs.near.org I should be able to login with my Near Account.
As a contributor, I should be able to directly embed widgets on any documentation I create.
As a user, I should be able to fully interact with any component included in the documentation.

Suggested Release Milestones
Just one, the docs as a gateway.

Go To Market Needs

A blogpost explaining the new features would be nice

already implemented it


Pull in new content-loader for simplified slugs

As above. Pull in dm-content-loader 3.5.1, which will drop smart quotes from generated slugs.

#666

Sterling work. :+1:
Not sure why the Jinja2 import line is showing as changed?

😈

Nice spot :fire: :imp: :fire:


TiKV panic during go-tpc prepare

Bug Report

What version of TiKV are you using?
v5.0.2
What operating system and CPU are you using?
DBaaS T3.standard
Steps to reproduce

deploy a T3.standard cluster with 2db 6kv
run go-tpc tpcc prepare -T 64 --warehouses 1000

What did you expect?
no panic
What did happened?
TiKV panic with followering log
[2021/06/18 09:37:26.917 +00:00] [FATAL] [lib.rs:465] ["unexpected error: Store(Other(\"[components/raftstore/src/store/peer_storage.rs:980]: failed to get snapshot after 5 times\")), raft_id: 2652, region_id: 2650"] [backtrace="stack backtrace:\n   0: tikv_util::set_panic_hook::{{closure}}\n             at home/jenkins/agent/workspace/optimization-build-tidb-linux-amd/go/src/github.com/pingcap/tikv/components/tikv_util/src/lib.rs:464\n   1: std::panicking::rust_panic_with_hook\n             at rustc/bc39d4d9c514e5fdb40a5782e6ca08924f979c35//library/std/src/panicking.rs:595\n   2: std::panicking::begin_panic_handler::{{closure}}\n             at rustc/bc39d4d9c514e5fdb40a5782e6ca08924f979c35//library/std/src/panicking.rs:497\n   3: std::sys_common::backtrace::__rust_end_short_backtrace\n             at rustc/bc39d4d9c514e5fdb40a5782e6ca08924f979c35//library/std/src/sys_common/backtrace.rs:141\n   4: rust_begin_unwind\n             at rustc/bc39d4d9c514e5fdb40a5782e6ca08924f979c35//library/std/src/panicking.rs:493\n   5: std::panicking::begin_panic_fmt

Some metrics

/type bug
/severity critical

/assign @gengliqi

It's fixed by https://github.com/tikv/tikv/pull/10454.


Improve breakpoints

[x] Merge all groups (breakpoints, types, etc.)
[x] Instead let each breakpoint have settings about should responsive helpers and objects be generated for it or no
[x] Let select which exactly helpers and object should generate responsive classes for that breakpoint (probably, via list)

Something like:
$ekzo-breakpoints: (
  palm: (
    rule: 'screen and (max-width: #{$ekzo-lap - 1px})',
    generate-responsive: (buttons, grid, spacing, tables, widths)
  ),
  lap: (
    rule: 'screen and (min-width: #{$ekzo-lap}) and (max-width: #{$ekzo-desk - 1px})',
    generate-responsive: ('*')
  ),
  retina: (
    rule: 'screen and (-webkit-min-device-pixel-ratio: 2), screen and (min-resolution: 192dpi)',
    generate-responsive: false
  ),
  vetina: (
    rule: 'screen and (-webkit-min-device-pixel-ratio: 2), screen and (min-resolution: 192dpi)'
  )
) !default;

Implemented in a bit another form.
I thinks that Let select which exactly helpers and object should generate responsive classes for that breakpoint  is a bit too granular approach.
Let's see how current implementation will work out.

Mentioned above commits made possible even such granular selection.
By default global $ekzo-responsive-helpers-breakpoints controls everything, which is handy, but on occasion you can specify list of breakpoints for any automatically generated responsive helpers. Quite handy.


Is it possible to query Log Analytics via the az cli with a saved query?

I can’t tell from the documentation, is it possible to run a saved Log Analytics Query from this CLI command?
If not, a useful enhancement would be to enable the use a saved query in addition to the ability to execute queries in-line.  The queries get long and cumbersome to maintain outside of Log Analytics.
If it is, however, possible to run a saved query, would you mind updating the documentation here?  Thanks.

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

ID: f0fd6a58-ac1a-fa45-8d96-579b4af36499
Version Independent ID: 4098ca97-1b85-eb29-18e9-e6f0495fd030
Content: az monitor log-analytics
Content Source: latest/docs-ref-autogen/ext/log-analytics/monitor/log-analytics.yml
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

add to S179

Hi @GitMje, do you mean to read the saved query from a file?

@kairu-ms I was hoping to use a saved query like one from the saved queries or solutions queries.  This would be similar to how azure devops can run team queries via the API.  Rather than having to hard code a query into the az cli command/scripts.
Please see image below:

@GitMje I got it. It's a very good question. Saved query is supported by command group
az monitor log-analytics workspace saved-search -h

You can fetch the query string like this command
QUERY = $(az monitor log-analytics workspace saved-search show -g resource-group --workspace-name workspacename --name query-name --query query)

However you can not use the $QUERY like this
az monitor log-analytics query --workspace workspace-custom-id --analytics-query $QUERY

Because --analytics-query cannot correctly parse a query with "\n" and "\t" now.
We will fix it in next release.

Hi, the above command it isn't supported anymore. Is there any other way to run a simple query, not a saved one, from the CLI?


Adjust min clamp on height of the dialogs

Nowadays, we have dialogs with a height smaller than 240px; they have like 164px.
And when we pass the correct height configuration with CatDialogConfig, the min-height clamp overlaps the height configuration.
It will be nice to have the possibility of having dialogs at a lower height.

Any news on this issue?

I think it is solved. See this commit

Okay, nice and it's released in version 5.3.0 right? I guess the ticket can be closed


DE2019 control effectiveness seems wrong

Hi,
In Dyn_6DoF_v2_0_r2019b/AirborneSystem/FlightControlSystem/AttitudeAndRateLoop/RateLoop/ControlAllocation/allocation the control effectiveness matrix Ba is calculated based on the DE2019 model. When executing the following code (after having loaded the DE2019 aircraft model and having initialized the sim):
alpha = 0.1;
Cl_deltaA =DE2019.Cl_deltaA_0 + DE2019.Cl_deltaA_alpha * alpha + DE2019.Cl_deltaA_alpha2 * alpha^2;
Cl_deltaR =DE2019.Cl_deltaR_0 + DE2019.Cl_deltaR_alpha * alpha + DE2019.Cl_deltaR_alpha2 * alpha^2;

Cm_deltaA =DE2019.Cm_deltaA_0 + DE2019.Cm_deltaA_alpha * alpha + DE2019.Cm_deltaA_alpha2 * alpha^2;
Cm_deltaE = DE2019.Cm_deltaE_0  + DE2019.Cm_deltaE_alpha * alpha + DE2019.Cm_deltaE_alpha2 * alpha^2;

Cn_deltaA =DE2019.Cn_deltaA_0 + DE2019.Cn_deltaA_alpha * alpha + DE2019.Cn_deltaA_alpha2 * alpha^2;
Cn_deltaR =DE2019.Cn_deltaR_0 + DE2019.Cn_deltaR_alpha * alpha + DE2019.Cn_deltaR_alpha2 * alpha^2;

Ba = [[Cl_deltaA,0,Cl_deltaR];[Cm_deltaA,Cm_deltaE,0];[Cn_deltaA,0,Cn_deltaR]]

You get:
Ba =
0.1474             0    0.0090
0.0142   -1.8535            0
         0             0    0.0759

This is unexpected from what I generally know about aircraft control because Cm_deltaA is nonzero and Cn_deltaA is zero. This is strange because you would expect that an asymmetric aileron deflection does not have an effect on the pitch moment (so Cm_deltaA = 0) and that it does have an effect on the yaw moment (so Cm_deltaA != 0). Maybe the m and n subscripts got mixed up somewhere? Or this is correct and just different from what I would expect (very possible)?
I dug a little bit further into it and the fits that are done in Src\Common\initAllSimParams_DE2019.m seem correct so the data it uses might be false. In that case, B_Axx are not correct which are loaded from Lib\6DoF\Control_allocation_V60.mat.

The reason why the Cn_deltaA is zero, is because the FN (yaw moment) was forced to be zero from the FSI model. I do not remember correctly why this was the case but I think it had something to do with the controller. We track a zero sideslip angle and did not want the instability of the yaw moment to affect this. Also the values were relatively small compared to the other coefficients. However, it is true this value is not zero. Bear in mind that the matrix is based on steady state solutions from the FSI, therefore it looks at a deformed wing state. I will send you the updated allocation matrix, if this does not crash the aircraft we can update the library file in the future.


Docker Compose doesn't load .env file and incorrect DATABASE_URL configuration

Overview:
There are two main issues in the current Docker setup:

The .env file is not being loaded, causing missing environment variables during runtime.
The DATABASE_URL uses host.docker.internal:35000, which leads to connection errors within the Docker network.

These issues prevent the backend service from properly connecting to the PostgreSQL database when using Docker Compose.
Environment:

OS: Arch Linux
Docker Version: 27.0.3
Docker Compose Version: 2.28.1

Actions Needed:

[ ] Load .env file: Update the docker-compose.yml to include the env_file directive.
[ ] Update DATABASE_URL: Modify the DATABASE_URL to use the correct hostname within the Docker network (db:5432).

Proposed Fix:

Load .env file:
Add the following to the api service in docker-compose.yml:
env_file:
  - .env

Correct DATABASE_URL:
Update the DATABASE_URL in docker-compose.yml:
DATABASE_URL=postgres://postgres:postgres@db:5432/bloom

Hi, can you confirm if the issue I reported is correct? If so, I'd like to work on it.

The .env file is not being loaded, causing missing environment variables during runtime.
The DATABASE_URL uses host.docker.internal:35000, which leads to connection errors within the Docker network.

Hi @SureshPradhana, thank you for your suggestion, and patience while we were out of office. Can you share the exact connection errors you experienced? They might be specific to your set-up. But we aim to support all environments, so it's important we learn what is going on here.
As for loading env vars, I agree they should be mounted to the volume / loaded at runtime, as you demonstrated. Our current set-up isn't best practice, and this would be more secure and flexible.

Hi @kyleecodes,
Thanks for the feedback. Here are the connection errors I faced:

Issue: Connection refused when using host.docker.internal:35000 for PostgreSQL.bloom-backend   | [Nest] 39  - 08/29/2024, 11:20:24 AM   ERROR [TypeOrmModule] Unable to connect to the database. Retrying (1)...
bloom-backend   | Error: getaddrinfo ENOTFOUND host.docker.internal
bloom-backend   |     at GetAddrInfoReqWrap.onlookupall [as oncomplete] (node:dns:120:26)
bloom-backend   | [Nest] 39  - 08/29/2024, 11:20:27 AM   ERROR [TypeOrmModule] Unable to connect to the database.

i use linux found out that On Linux, host.docker.internal is not available out-of-the-box. Instead, you might need to use the host’s IP address or configure Docker networking to access the host machine.
In a Docker Compose setup, host.docker.internal isn't suitable because PostgreSQL runs in a Docker container. The recommended approach is to use postgres://postgres:postgres@db:5432/bloom to connect to the PostgreSQL service within Docker.

Great points @SureshPradhana. Thanks for your patience while we were out of office.
This might be a Linux specific suggestion? But let's try it! We are currently making updates to our Docker set-up.
I've assigned you the issue if you would like to implement these changes. Please tag me if you have any questions or if you'd like to be unassigned.

@leoseg I'm going to assign you to this issue as you have done some work on it

Okay, I include the fix for this issue in another pull request and linked it there.