Question about hits and frames parsing

Hi aEnigma,
I'm trying to figure out how to correctly parse the hits and frames of skills when it mixes damaging effects AND non-damaging ones.
I have three different cases to explain my trouble. First let's take Shine Surprise Attack +1 (702490), which have 3 effects, and only one table of hits/frames :

"attack_frames": [[12,  22]],

In this case, frames are available for damaging effects only (effect 1, which is physical damage).
Now let's consider Eleven's Gigagash (510751), which have 2 effects and two different table of hits/frames :

"attack_frames": [[42,  48,  54,  60,  66,  72,  78,  84,  90], [42,  48,  54,  60,  66,  72,  78,  84,  90]],

Oddly in this case, it seems there is BS chain frames for the non-damaging effect 33 (imperil). BTW, one of the effect of the Shine example is also an imperil with ID 33.
Now another example with DKLeon's Dark Servant Blade (509923) :

[[0], [0], [70,  76,  82,  88,  94,  100,  106], [112]],

There is frames for the non-damaging effects, but here with 0 value.
There must be an algorithm to correctly parse the frames of the damaging skill. However, I really cannot figure out what it is. In the Surprise Attack +1 case, the game must ignore there is two non-damaging effects to find the attack frames, while in the other cases similar effects do have frames.
Do you have any idea for an algorithm to achieve what I'm looking for ?

I'm interested by the answer too. I was supposing the the frames data was
only for damaging effects, until I saw those [0] frames ^_^
On Thu, Oct 10, 2019 at 8:31 AM Patrick Mingard<EMAIL_ADDRESS>wrote:

Hi aEnigma,
I'm trying to figure out how to correctly parse the hits and frames of
skills when it mixes damaging effects AND non-damaging ones.
I have three different cases to explain my trouble. First let's take Shine
Surprise Attack +1 (702490), which have 3 effects, and only one table of
hits/frames :

"attack_frames": [[12, 22]],

In this case, frames are available for damaging effects only (effect 1,
which is physical damage).
Now let's consider Eleven's Gigagash (510751), which have 2 effects and
two different table of hits/frames :

"attack_frames": [[42, 48, 54, 60, 66, 72, 78, 84, 90], [42, 48, 54,
60, 66, 72, 78, 84, 90]],

Oddly in this case, it seems there is BS chain frames for the non-damaging
effect 33 (imperil). BTW, one of the effect of the Shine example is also an
imperil with ID 33.
Now another example with DKLeon's Dark Servant Blade (509923) :

[[0], [0], [70, 76, 82, 88, 94, 100, 106], [112]],

There is frames for the non-damaging effects, but here with 0 value.
There must be an algorithm to correctly parse the frames of the damaging
skill. However, I really cannot figure out what it is. In the Surprise
Attack +1 case, the game must ignore there is two non-damaging effects to
find the attack frames, while in the other cases similar effects do have
frames.
Do you have any idea for an algorithm to achieve what I'm looking for ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/aEnigmatic/ffbe/issues/18?email_source=notifications&email_token=AAHWD3LWSQDZSFOTFZYUM4DQN3D2PA5CNFSM4I7IOT5KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HQ2VU5Q,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHWD3MKDVCI23WON7UCNPTQN3D2PANCNFSM4I7IOT5A
.

TBH, I don't really know, the game will probably either use a previous value or a default value...

I'll try to test if there is the same number of frames tables than effects in the skill.
If yes, I'll use the frames tables located at the index of the effects.
If no, I'll use the frames at the beginning of the table, assuming all the frames are for the damages.
This would be okay for these three specific skills. Need to implement this and test it a bit with more skills.
@lyrgard Read on reddit your health is good again, good news ! I'll try to implement quickly my idea to extract the frames of the damaging effects. I'll come back to you with the results. Btw, I'm currently implementing a parser to transform aEnigma data dump into french readable text. If one day my code does interest you for any reason, let me know.

It is indeed working for the three skills I took as examples. But I quickly found one which is NOT working : Saber of Adversity (509912) from Wild Rose Firion...
There is 5 effects :
        "effects": [
            "No effect",
            "Increase physical damage against [...] for 3 turns",
            "Physical damage (2x, ATK) to one enemy",
            "Physical damage (10.8x, ATK) to one enemy",
            "Gain 3 uses [...] for 2 turns"
        ],

And the frames are :
"attack_frames": [[0], [0], [110,  120,  130,  140,  150,  160,  170,  180,  190], [200]],
Only 4 tables, not 5. I could refine my algorithm to exclude the effects which are parsed to "No effect", but this is getting really clumsy...

And here I have it, Serena's Deliverance (233160) :
        "effects": [
            "No effect",
            "Increase magic damage against Reapers by 50% to caster for 2 turns",
            "Magic* damage (4.5x, SPR) to one enemy"
        ],

And the frames :
"attack_frames": [[50], [12], [12]],
Unline Firion's case, the "No effect" have frames attached to it. The "No effect" has the exact same configuration (effect ID 24, 6 zeros as parameters) for both Firion's and Serena's cases.
Feel free to share if you have any idea, I must face it, I do not understand -_-

Hi guys,
Unfortunately there is no real logic way to get the attack_frames...
I parse those data since 2 years and I always need to made some adjustments because the same effect isn't showing the same result for the frames...
Sometime it's a 0, sometimes it doesn't show and even worse sometime it has a real value...
I already try lot of things to improve the way to get the frames but never find the perfect solution because the data implemented in FFBE are a real mess...
I think the only to understand how they manage to use those data in game it's to reverse engineering the code itself...
But if you manage to find a way to get the good values in any cases I'm interested :)
Bismark

I propose to try to make some rules, and see if we find cases that doesn't match.
What about :
nbDamageEffect = count of effects that do damage
if (attack_frames.lenght == nbDamageEffect) {
    Assign each frame to a damage effect, in same order
} else {
    Assign each frame to each effect, in same order, until no more frame.
}

Is there skills that don't match that ? If I didn't checked wrong, it should work for all examples above

@lyrgard I'm currently testing a slightly different version of your algorithm, but which might have the exact same result (except for a small detail you did not include) :
frames = empty table
lastIndexOfDamageEffect = find the last index of effect that do damage, but is not DOT (!= 139)
if(attack_frames.length > lastIndexOfDamageEffect) {
  for all effects {
    if(effect do damage but is not DOT) {
      concat to frames local variable the frames corresponding to the effect index
    }
  }
} else {
  assign each frame to a damage effect, in same order
}

My else case correspond to your if case exactly. But in your else case (= my if case), you do have the risk of adding "0 0 ..." in front of Firion's frames. You need afterwards to exclude the frames you assigned to non-damaging effects. That why I have an inner if test.
Up to now, I did not find any case which is not correct under this algorithm. But I suspect there might be problems with delayed attacks and/or jumps. DOT's seems to always be after the immediate damages, so excluding effect 139 seems to do the trick.

I don't understand why you say I would add "0 0" in front of firion's frames.
To be more clear, in my else, I said "Assign each frame to each effect, in same order, until no more frame.", but in reality, I just drop frame information for not damaging skills. It was just to explain how the attack frames are matched to the attack effects.
Jump and DOT probably can't have frames, so just treat them as non damaging effect for that algorithm (and that's why it's so hard for them to fix Reberta, the idea is to do a chaining jump, but I think the code doesn't allow jump to have frames in FFBE).
Delayed attacks can have frames, but it should have its own frames in the attack_frames. Just consider them as if they were on a separate skill, for chaining purpose.

@lyrgard Ho, okay, then what you do is similar to what I do. I'm stacking the damaging frames into a local variable, and you are assigning the frames to the damaging effects (based on index), then construct the frames based on damaging effects only. That should output the exact same result.
If you find any skill which is not working properly, please let me know.

Closing this ticket as it seems the discussions on this topic will not continue further. Cheers all !
@ygalland It seems my algorithm is working. I could not find any skill which frames are not correctly parsed with it.

Ok, I think I found how it works, I found an algorithm that works for all cases, and is logical enough.
You take the attack_frames array, and if there are more effects that attack frames, you increase it's size by repeating the last value. Then you have the attack frames for each effect. And obviously, all not attack/heal and so on effects will just ignore it

I'll use a function like that to retrieve the data now :
function getValueAtIndex(array, index) {
    if (!Array.isArray(array)) {
        array = [array];
    }
    if (index < array.length) {
        return array[index];
    } else {
        return array[array.length - 1];
    }
}


OCPCLOUD-2703: OWNERS: update subcomponent

Update subcomponent, our way of organizing them changed.

/lgtm cancel
This is actually a Machine API provider

Ah good catch, I went by name! Updated.


Incorrect cities for us/co/archuleta

Hello,
Looking at the data for us/co/archuleta I noticed that the city is always set to ARCHULETA. That's actually the county name and there are a number of cities within the county. Unfortunately, the correct city names don't seem to exist in the source data.
As an example, here is 77 BEUCLER PL. When you type it into Google Maps, it takes you to 77 BEUCLER PL, PAGOSA SPRINGS, CO:
https://www.google.com/maps/place/77+Beucler+Pl,+Pagosa+Springs,+CO+81147/@37.2298902,-107.1856546,14.23z/data=!4m5!3m4!1s0x873dd1dd7c3738db:0x2410396fefbc78b1!8m2!3d37.2274441!4d-107.1718542
I checked the parcel source as well, and that also has no city field.
Since these addresses are technically incorrect, should they stay as part of OpenAddresses?

@rbrtmrtn Thanks for reporting this. I've created a Pull Request to remove the city field. As you found, I don't see any fields with the proper city field so this source will remain, however the addresses won't contain a city field in the output.

Thank you for taking a second look at it @ingalls. Glad to hear about the resolution!


ci: set main branch to be protected

I hereby agree to the terms of the CLA available at: https://databend.rs/dev/policies/cla/
Summary
Summary about this PR

I think it would be better to set the main branch to be protected

Thanks!


With Kerberos login when we trying to publish in Tableau server using our Kyvos custom connector we are not able to see "Server Run As account" in Authentication option

Describe the Bug
We have our own ODBC based custom Kyvos connector which is listed in Tableau & now we are enhancing the connector to support Kerberos Authentication as well using connection dialog v2
We tested it on Tableau desktop and it is working fine as expected, however when we tried to publish in Tableau server but we are not able to see "Server Run As account" in Authentication options. We are seeing Prompt and Embedded as options. We are expecting to see "Server Run As account" in Authentication options when we are connected using Kerberos.
We also tried by adding property in manifest file <customization name="CAP_AUTH_KERBEROS_IMPERSONATE" value="yes"/> but it didn't work.
Screenshots

For you reference we are attaching our connector files:
tableau_connector_sk.zip
About me
Srinadh Kunam
Kyvos Insights

Hi @srinadhkunam
Is your Tableau Server setup on Linux or Windows ? For Linux machine, you have to additionally run the tsm commands mentioned here.

Hi @devanshsoni9
We have both Linux and Windows Tableau Servers. We are seeing the issue in both when we tried to publish from desktop.

Hi @srinadhkunam
I think you might be already doing it but if can you confirm that the tsm commands mentioned here are being run ?

Hi @devanshsoni9
I validated again & when we tried to publish using Spark SQL connector, I'm able to see "Server Run As account" as below

But when I try to publish using Kyvos connector I'm seeing only Prompt & Embedded.
Can we get into a call to resolve the issue ASAP.
@VasanthiNeelagiri @sidwray

Hi @devanshsoni9 any update for me on this issues.

Hi @srinadhkunam
I looked at the connector files you provided and have the following suggestion based on my testing.

In this file connectionBuilder.js, the AUTHMECH for auth-integrated should be 3.This is needed for the Kerberos authentication.

Let me know if it changes anything, or else we may have to look into the platform code.

@srinadhkunam updated my comment, per spark docs , In case of Kerberos auth,  the value for AuthMech is 1

@devanshsoni9 we are using our own ODBC driver, which is developed by Kyvos team and in the driver code for Kerberos AuthMech is set as 2.

@srinadhkunam - Hi Srinadh, We are still discussing the solutions and timelines to fix the issue. But, before we proceed further, we want to understand a couple of things:

What are your plans on this connector update? When do you anticipate this new connector to be released?
Is it just for internal purposes/Do you want Tableau to bundle this connector Vs release on Tableau Exchange? Please note that we are encouraging our partners to use and submit JDBC Connectors going further

Could you please share more details on the above questions and if any customers are requesting for this upgrade

Hi @VasanthiNeelagiri, Thanks for the update. Below are answers for your questions.

This issue only blocking for me to release this new version of Kyvos connector and new release of Kyvos ODBC driver.
This enhancement we are doing for couple of our join customers requirement. This new version replaces our existing Kyvos connector which is already included in Tableau bundle.

Thanks for the update Srinadh. To fix this, we need to do a fix on our platform side. Also, we would like to update the connector at the same time. Closing this issue and moving the conversation to email. I will involve our Tech Partner team for updating the connector.
Thanks & Regards,
Vasanthi Neelagiri,
Product Manager, Salesforce


Add experimental_peer_skip_client_san_verification option to partial resources

Description
Fixes:
NoMethodError: undefined method `experimental_peer_skip_client_san_verification' for Custom resource etcd_etcd_service_manager_systemd from cookbook etcd

Check List

[x] A summary of changes made is included in the CHANGELOG under ## Unreleased
[x] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable.

@edgarasg this seems related to the change I made in this PR https://github.com/sous-chefs/docker/pull/1229. I also see some issues related to the fixes I also made in that PR. Let me make a new one here to resolve that and then you can rebase once it's released.


Feature Discussion

Discussion on feature scope for https://github.com/oasis-open/odata-rapid/issues/142
Feel free to comment, propose changes, or just change it.

Do we support query options on data modification requests to shape the response of a POST/PUT/PATCH?

We do;-) and according to section 5.1 on system query option in our URL Conventions we explicitly mention that those can be used on GET, PATCH and PUT and that a response of a POST, obviously depending on the return type of that action or collection one is posting to, it is supposed to same rules than GET as well. I.o.w. yes IMHO.

I think we should, created #278 to explicitly state this


Set Volume Fails

Right now when I call set volume I get:
{"status":{"set-volume":"FAIL","req_body":"{'volume': 4}"}}
It appears my request body is slightly different than native app but more investigation required.

Spoke to soon.  json.dumps(payload) fixes it.


Load Spring Context within JavaFx Thread or not ?

This is more to start a discussion:

At the beginning the spring context was loaded from the javafx thread. That was nice but  generates 2 concerns

You'll load some none non UI beans into graphic thread. That's could take some times and liekly not relevant.
It was  more or less  impossible to use splashscreen (with progress feedback) when your spring ctx loading take time (due to previous point)

Then Spring Ctx is now loaded from another thread that generate other concerns (even if now splash-screen is working fine)

All @FXMLView / Controller are loaded by spring (they are flagged as @Component)

Meaning all UI are loaded by a non graphic thread (not sure it's a best practice)

We are struggled when u want to used WebEngine or GlyphFont (controlsfx) because there are checking in their constructor to be instantiate by JavaFx thread.

There is pro and cons in both solution
I don't know how to solve that properly.
Right now I did a dirty hack: where I instantiate some UI bean by delegating to JavaFx thread and blocking current thread until it's finished :-(
Any views ?

Hi
any news ? no body is facing this time of issue ?
Chris

Ah. I think I undestand...
Give me some time.


Created back button to use browser history

ref #327
OP task: #700

It's good to merge, but it does not really solve the original problem because the pages still reloads after going back and scroll position is lost. May be we need to think of something else sometimes later

I made another PR that should complete the feature


scattermapbox to_image error

Exporting a png of a scattermapbox chart using plotly.io.to_image() results in this error:
ValueError:
The image request was rejected by the orca conversion utility
with the following error:
   522: client socket timeout

It is somewhat inconsistent, sometimes I can export 20 images at a time and then stops altogether. I should add that I'm using GeoJson layers in my map. When it succeeds the output looks like this

I'm doing this from a controller in my application: all the other charts get exported with no problems and quite fast.
I tried launchung the orca server manually to see what is going on, and the error message is quite similar to the traceback I get from my webserver
{"severity":"ERROR","textPayload":"525 - plotly.js error","labels":{"fid":null}}
Lastly, I’ve used the basic example of scattermapbox to ensure nothing else was influencing the output and tried exporting a json with plotly, and then exporting it manually with orca.
Data = [
    go.Scattermapbox(
        lat=['45.5017'],
        lon=['-73.5673'],
        mode='markers',
        marker=dict(
            size=14
        ),
        text=['Montreal'],
    )
]

layout = go.Layout(
    autosize=True,
    hovermode='closest',
    mapbox=dict(
        accesstoken=MY_TOKEN,
        bearing=0,
        center=dict(
            lat=45,
            lon=-73
        ),
        pitch=0,
        zoom=5
    ),
)

fig = dict(data=data, layout=layout)

#write json with plotly.io
pio.write_json(fig, 'figure.json', pretty=True)

Then in my terminal:
$ orca graph figure.json --mapbox-access-token MY_TOKEN -o figure.png --verbose
The output was this
gpu process crashed - false
          Chrome version 59.0.3071.115
          Electron version 1.8.4
renderer error - Uncaught TypeError: Failed to execute 'shaderSource' on 'WebGLRenderingContext': parameter 1 is not of type 'WebGLShader'.
          Chrome version 59.0.3071.115
          Electron version 1.8.4
renderer error - Uncaught TypeError: Failed to execute 'shaderSource' on 'WebGLRenderingContext': parameter 1 is not of type 'WebGLShader'.
          Chrome version 59.0.3071.115
          Electron version 1.8.4
renderer error - Uncaught TypeError: Failed to execute 'shaderSource' on 'WebGLRenderingContext': parameter 1 is not of type 'WebGLShader'.
          Chrome version 59.0.3071.115
          Electron version 1.8.4

Edit: I'm on windows

@etpinard @antoinerg Thank you very much for your work!
I'll try v 1.2.1 both on windows and linux and see if it works so you can close this issue

Hey @etpinard and @antoinerg!
I'm running into the same issue as OP where using plotly.io.write_image() to save a Scattermapbox as a .png results in
ValueError:
The image request was rejected by the orca conversion utility
with the following error:
   522: client socket timeout

I'm using

Mac OSX 10.14.2
plotly 3.4.2
orca 1.2.1 (was using v1.1.1 but upgraded because of the above comment; same error on both versions)

The above are on my local machine (not VM) and I checked that WebGL is working properly via the link above.
I also ran through the steps that OP took to save the image as json using plotly.io.write_json() and ran the same orca command from the terminal. I got a different set of output than they did though
<--- Last few GCs --->

[87021:0x7ffa41857600]    72642 ms: Mark-sweep 2050.4 (2124.7) -> 2050.4 (2098.2) MB, 1468.1 / 0.1 ms  (+ 0.0 ms in 0 steps since start of marking, biggest step 0.0 ms, walltime since start of marking 1468 ms) last resort
[87021:0x7ffa41857600]    74143 ms: Mark-sweep 2050.4 (2098.2) -> 2050.7 (2098.7) MB, 1500.6 / 0.1 ms  last resort

<--- JS stacktrace --->

==== JS stack trace =========================================

Security context: 0x3148bf3b14c9 <DedicatedWorkerGlobalScope map = 0x11d45be8d411>

the command never finished running after ~5 mins and I killed it from there.
Do you have any suggestions for what I should do get past this error? Thanks for your time!

I have been facing this timeout issue too. You can see here in create-server.js that Orca will produce a 522 error when a socket timeout occurs: https://github.com/plotly/orca/blob/master/src/app/server/create-server.js#L69
In my case, I think this is happening because I have a lot of data. I have been trying to increase the socket timeout using the requestTimeout option but with no success so far. requestTimeout doesn't even show up as a possible argument under "orca serve --help".

@tombailey I'm sorry to hear you are facing an issue with exporting scattermapbox. Can you give me a rough estimate of how large your dataset is?

requestTimeout doesn't even show up as a possible argument under "orca serve --help".

It should be there as --request-timeout: https://github.com/plotly/orca/blob/a2ce478735857ca6bcc03ebc798de017e87628bb/bin/serve.js#L48-L54
What version of Orca are you running?

@antoinerg
In my case, I am not using scattermapbox, instead a sankey diagram but I think the situation is similar. As a temporary hack, I have upped the timeout by manually my global node_modules.
The 522 error has now been replace by an Electron error as the process seems to be hitting OOM issues:
`#
Fatal error in , line 0
API fatal error handler returned after process out of memory

0   Electron Framework                  0x00000001025ddc5c crashpad::CloseMultipleNowOrOnExec(int, int) + 961820
1   Electron Framework                  0x0000000103002eab crashpad::CloseMultipleNowOrOnExec(int, int) + 11599723
2   libnode.dylib                       0x0000000100922e1c V8_Fatal + 220
3   libnode.dylib                       0x0000000100b562e7 v8::internal::FatalProcessOutOfMemory(char const*) + 599`
On the --request-timeout arg, I see nothing in --help, see below:
`$ orca --version
1.2.1
$ orca serve --help
orca serve
Usage:
$ orca serve -p 9999
Options:
--help [or -h]
Displays this message.
--port [or -p]
Sets the server's port number.
--plotly [or --plotlyjs, --plotly-js, --plotly_js, --plotlyJS, --plotlyJs]
Sets the path to the plotly.js bundle to use.
This option can be also set to 'latest' or any valid plotly.js server release (e.g. 'v1.2.3'),
where the corresponding plot.ly CDN bundle is used.
By default, the 'latest' CDN bundle is used.
--mapbox-access-token [or --mapboxAccessToken]
Sets mapbox access token. Required to export mapbox graphs.
Alternatively, one can set a MAPBOX_ACCESS_TOKEN environment variable.
--topojson
Sets path to topojson files.
By default, topojson files on the plot.ly CDN are used.
--mathjax [or --MathJax]
Sets path to MathJax files. Required to export LaTeX characters.
--inkscape [or --Inkscape]
Sets path to Inkscape executable. Required to export WMF and EMF formats.
--safe-mode [or --safeMode, --safe]
Turns on safe mode: where figures likely to make browser window hang during image generating are skipped.
--request-limit [or --requestLimit]
Sets a request limit that makes orca exit when reached.
--keep-alive [or --keepAlive]
Turn on keep alive mode where orca will (try to) relaunch server if process unexpectedly exits.
--window-max-number [or --windowMaxNumber, --maxNumberOfWindows]
Sets maximum number of browser windows the server can keep open at a given time.
--graph-only [or --graphOnly]
Launches only the graph component (not thumbnails, dash, etc.) to save memory and reduce the number of processes.
--quiet
Suppress all logging info.
--debug
Starts app in debug mode.
--cors
Enables Cross-Origin Resource Sharing (CORS)
`

@tombailey can you render your figure in a browser using plotly.js? If not, I suspect you're hitting this plotly.js issue: https://github.com/plotly/plotly.js/issues/3813
A fix for this will be released this week as part of plotly.js v1.49.0.

@antoinerg I think you are right, in my case it appears it was a circular sankey issue although I couldn't get plotly.js v1.49.0 to render the sankey diagram.
Back to the original issue, a 522 client timeout error would suggest large data or a bug causing Orca to spend too much time rendering the image so the timeout happens. Whether that is because of an infinite loop in my case or large data which might be the case for this original issue.
As @antoinerg mentions, --request-timeout can be used to give Orca more time to render an image assuming the issue isn't caused by an infinite loop.

@tombailey v1.49.0 hasn't been released yet although you can try a bundle built off master temporarily hosted here: https://45455-45646037-gh.circle-artifacts.com/0/dist/plotly.min.js
You can provide that URL to Orca directly via --plotly flag.
It this still doesn't render, could you please open a new issue or append the problematic dataset to the issue: https://github.com/plotly/plotly.js/issues/3813.
Thanks @tombailey


{Feature Request] Add "Automatic Expose All Ports" like shipyard

Add "Automatic Expose All Ports" like shipyard. A very useful feature i found lacking (or at-least cannot find) for single host docker hosts.

Yes, please this option is very important. Could there be an option to add
open all ports maybe by admin at default on host side,  then the user can
then open any port individually on the container side at will
On Feb 3, 2017 10:07 AM, "xInsertx"<EMAIL_ADDRESS>wrote:

Add "Automatic Expose All Ports" like shipyard. A very useful feature i
found lacking (or at-least cannot find) for single host docker hosts.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/portainer/portainer/issues/558, or mute the thread
https://github.com/notifications/unsubscribe-auth/AGsC45YOoFwu99Oata1_RvOzkpzrGsbdks5rY0K8gaJpZM4L2bd1
.

@xInsertx would you like the equivalent of docker run -P ?
@lavvy I'm not sure I understand what you want.

Maybe have the "expose ports" prepopulated with all the ports the DockerFile wants to expose, giving the user the instant visibility and ability to set the corresponding external port?
Rgds,
Neil Cresswell
On 4/02/2017, at 9:58 AM, Anthony Lapenna <notifications<EMAIL_ADDRESS>wrote:
@xInsertxhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_xInsertx&d=DQMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=iG29eSxZFMyMY7Co8p3ArwKoAgNBVU2uJJA26fYgBA4&s=d_rwTabtyoNbqcKS04dnoz-n7A4N4MzOTrZlUL6OSJ4&e= would you like the equivalent of docker run -P ?
@lavvyhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_lavvy&d=DQMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=iG29eSxZFMyMY7Co8p3ArwKoAgNBVU2uJJA26fYgBA4&s=bEzrbjU-h778XXYhCNo08AGShbDxw2vRZz5lhd7PDuQ&e= I'm not sure I understand what you want.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_portainer_portainer_issues_558-23issuecomment-2D277360602&d=DQMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=iG29eSxZFMyMY7Co8p3ArwKoAgNBVU2uJJA26fYgBA4&s=2EY7hlnhTDK8ogtpYJsS414SFWxF6mdDzw6tem0yXmE&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AWGrleUbED-5FhRGU0gKidmbdSv9J5Wxwtks5rY5T4gaJpZM4L2bd1&d=DQMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=iG29eSxZFMyMY7Co8p3ArwKoAgNBVU2uJJA26fYgBA4&s=Wotx3vLyKOOYegRyVlRhzpAp4gilOV9i30t0hnGug4w&e=.

Oh OK,  Neil thats a good option.
On Feb 3, 2017 4:02 PM, "Neil Cresswell"<EMAIL_ADDRESS>wrote:

Maybe have the "expose ports" prepopulated with all the ports the
DockerFile wants to expose, giving the user the instant visibility and
ability to set the corresponding external port?
Rgds,
Neil Cresswell
On 4/02/2017, at 9:58 AM, Anthony Lapenna<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:
@xInsertx<https://urldefense.proofpoint.com/v2/url?u=https-
3A__github.com_xInsertx&d=DQMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-
v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=
iG29eSxZFMyMY7Co8p3ArwKoAgNBVU2uJJA26fYgBA4&s=d_rwTabtyoNbqcKS04dnoz-
n7A4N4MzOTrZlUL6OSJ4&e=> would you like the equivalent of docker run -P ?
@lavvy<https://urldefense.proofpoint.com/v2/url?u=https-
3A__github.com_lavvy&d=DQMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-
v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=
iG29eSxZFMyMY7Co8p3ArwKoAgNBVU2uJJA26fYgBA4&s=bEzrbjU-
h778XXYhCNo08AGShbDxw2vRZz5lhd7PDuQ&e=> I'm not sure I understand what
you want.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub<https://urldefense.
proofpoint.com/v2/url?u=https-3A__github.com_portainer_
portainer_issues_558-23issuecomment-2D277360602&d=DQMCaQ&c=
euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_
QGpbggVOB-PfMJpM&m=iG29eSxZFMyMY7Co8p3ArwKoAgNBVU2uJJA26fYgBA4&s=
2EY7hlnhTDK8ogtpYJsS414SFWxF6mdDzw6tem0yXmE&e=>, or mute the thread<
https://urldefense.proofpoint.com/v2/url?u=https-
3A__github.com_notifications_unsubscribe-2Dauth_AWGrleUbED-
5FhRGU0gKidmbdSv9J5Wxwtks5rY5T4gaJpZM4L2bd1&d=DQMCaQ&c=
euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_
QGpbggVOB-PfMJpM&m=iG29eSxZFMyMY7Co8p3ArwKoAgNBVU2uJJA26fYgBA4&s=
Wotx3vLyKOOYegRyVlRhzpAp4gilOV9i30t0hnGug4w&e=>.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/portainer/portainer/issues/558#issuecomment-277361518,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGsC4y89h3YqZhtC_ciuQnSvJ1uTVz92ks5rY5XzgaJpZM4L2bd1
.

I think Neil's idea is a good idea, but should be treated separately. It would be helpful to pre-populate so users automatically know what ports were exposed.
Shipyard has a button that allows you to automatically assign all exported ports to an unused port from 49153-65535. This means if I start 3 of the same containers with this selected it automaticly maps all exported ports to new ports via this range.

Ah, then -P will do that, so we just need add that function
Rgds,
Neil Cresswell
On 4/02/2017, at 4:01 PM, xInsertx <notifications<EMAIL_ADDRESS>wrote:
I think Neil's idea is a good idea, but should be treated separately. It would be helpful to pre-populate so users automatically know what ports were exposed.
Shipyard has a button that allows you to automatically assign all exported ports to an unused port from 49153-65535. This means if I start 3 of the same containers with this selected it automaticly maps all exported ports to new ports via this range.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_portainer_portainer_issues_558-23issuecomment-2D277413780&d=DQMFaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=Z7w-dsqt6EcerIQGQgkuy1gCIHXwYo2-M7zbeyDWq4k&s=3JpfnU4WyOEKjaqEgW2k0WMdw_Vx_8qiB-mX2rfx-xA&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AWGrlW2CSbnqjippYAC9SglMx0PI-2Dw1-5Fks5rY-2DnwgaJpZM4L2bd1&d=DQMFaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=Z7w-dsqt6EcerIQGQgkuy1gCIHXwYo2-M7zbeyDWq4k&s=oNWF1fUatqhiaXXqQdrhLqXvg1pyEjZMEdqytoeeP0U&e=.


Lambda with cognito-at-edge too big for CloudFront

What happened:
Error: updating CloudFront Distribution: InvalidLambdaFunctionAssociation: The function code size is larger than the maximum allowed size for functions that are triggered by a CloudFront event: 1423536 Max allowed: 1048576
What did you expect to have happen:
No issues with cognito-at-edge version 1.5.0
How to reproduce this (as precisely and succinctly as possible):
package.json
{
  "name": "lambda",
  "dependencies": {
    "cognito-at-edge": "1.5.1"
  }
}

npm install
Then zip files and update lambda.
Environment:
cognito-at-edge version
1.5.1
node --version
v20.11.0
npm --version
10.2.4

Hi @chbk,
We use the CDK's NodejsFunction construct to deploy our test environments and we did not encounter any issues. Presumably esbuild is able to optimize and reduce the package size even with the default settings (no minify, etc.).
Unless you need the bug fix included in the latest version, I would recommend sticking to 1.5.0 for now. Long term, we would like to get rid of most of our external dependencies (axios and pica), but I can't commit to a specific timeline.
Cheers,


Add a description of your program below its name

Humanity couldn't leap without man's small step preceding it.
So, don't force-feed users a documentation before they even know what they're here for. They can only leap into your download and docs when they take the small step of knowing what it is.

We’re trying to move away from the co-y paste, but thanks.


Added support for Django 2

Coverage increased (+0.2%) to 89.809% when pulling 46a79df51b70011c7541c316b31b3deb70301ee0 on CloudVE:django_2_upgrade into e7fe61b3edc6c81a2da5343a1fad7c7bdacea1cf on digi604:master.

Coverage increased (+0.2%) to 89.809% when pulling d82e459368f9a66e32c059d7c8702a312fd1d66b on CloudVE:django_2_upgrade into e7fe61b3edc6c81a2da5343a1fad7c7bdacea1cf on digi604:master.

Coverage increased (+0.2%) to 89.809% when pulling fb065c75e180b0fed30a8751f47d57f8bb57da1d on CloudVE:django_2_upgrade into e7fe61b3edc6c81a2da5343a1fad7c7bdacea1cf on digi604:master.

Thanks!


New place=country nodes are not rendered

I created several new place=country nodes in OHM but all of them don't show up on the map (e.g. https://www.openhistoricalmap.org/node/2090973517#map=20/47.98992/24.58082&layers=O&date=1850&daterange=1800,2021).
Adding place=state works fine and labels that already existed before the desing change a few days ago show up on the map as well.
Thats why I think that this might be a software problem.

Sorry, just figured out that ID suggests "county" instead of "country"

Ha! It happens to the best of us!
Also - have you looked at this for boundary tagging? https://wiki.openstreetmap.org/wiki/Open_Historical_Map/Tags#Boundaries


Add copied indicator

Buttons that don't appear to trigger anything could be confusing to end users.

Would a better place be under the sidebar?

I think the button text being changed to “Copied!” for a bit would be best, as that’s where the user would be looking.

Something like this?
https://user-images.githubusercontent.com/75220768/235812222-51ffbbb8-d4ef-4241-a481-6c18f46d8b6b.mp4

That looks good— what are these bugs you're describing?

That looks good— what are these bugs you're describing?

It may just be Microsoft's screen recorder, but - some of the buttons look a little delayed in the video.

I didn't notice any lag when testing the buttons. However, for some reason, only one input can say "Copied!" at once. I'm not familiar enough with JavaScript to figure out what's causing this.
I did make one change: I noticed that you had created a new attribute, data-bbcode, for BBcode values in each input. I removed this and replaced references to it with the existing value attribute, as it appears to contain the same information.

I did make one change: I noticed that you had created a new attribute, data-bbcode, for BBcode values in each input. I removed this and replaced references to it with the existing value attribute, as it appears to contain the same information.

That's actually used to save the BBCode that would normally be contained in the input box. I did this since Copied! is shown in the Textbox using the value attribute. This allows for using the copy button to not be affected by the text changing to Copied!, which if the user clicked the button 2 times, it would copy Copied! instead of the BBCode value. It's also used inorder to revert value back to normal after Copied! has been shown.
TL;DR : That's very important, I'd suggest it be reverted.

I didn't notice any lag when testing the buttons. However, for some reason, only one input can say "Copied!" at once. I'm not familiar enough with JavaScript to figure out what's causing this.

I believe the reason for this may have something to do with it using JS's builtin settimeout function. I don't fully understand the way that settimeout works yet, but that is most-likely the breakpoint.

That's actually used to save the BBCode that would normally be contained in the input box. I did this since Copied! is shown in the Textbox using the value attribute. This allows for using the copy button to not be affected by the text changing to Copied!, which if the user clicked the button 2 times, it would copy Copied! instead of the BBCode value. It's also used inorder to revert value back to normal after Copied! has been shown.

I considered this before making the change, but I tested it on GitHub Pages and it still works properly as far as I can tell. The text changing to "Copied!" seems to have no effect on the actual value attribute.

Fixed the bug that made only one textbox fill at a time.


[WIP] Finishing up xclbin generation.

With this PR the binary translation adds all the functionality needed to generate the two binary pieces that IREE needs to generate. To get here  a new option, --iree-amd-aie-vitis-install-dir, was needed to point to the Vitis installation directory to pick up the libraries and binaries needed for the final xclbin generation.
This effectively now is capable of replicating the flow in aiecc.py using the following command
SAMPLES_DIR=<iree-amd-aie source dir>/tests/samples
iree-compile  --iree-hal-target-backends=amd-aie  \
 ${SAMPLES_DIR}/matmul_fill_static_i32.mlir \
  --iree-codegen-transform-dialect-library=${SAMPLES_DIR}/matmul_fill_spec_pad.mlir \
  --iree-amd-aie-peano-install-dir=<peano installation directory> \
  --iree-amd-aie-mlir-aie-install-dir=<mlir-aie installation directory> \
  --iree-amd-aie-vitis-install-dir=<vitis installation directory> \
  --iree-hal-dump-executable-files-to=$PWD  --iree-amd-aie-show-invoked-commands

This PR is currently failing at this step https://github.com/nod-ai/iree-amd-aie/pull/44/files#diff-2a27c2baf66b75bcf426d927b55cbf5f3b5d7bfa69b9b7492ab0d3672805bac4R565 . The diagnosis is there are some expected temporary files that the IREE compilation flow isnt preserving. So that command is failing.


AMPI support for MPI_THREAD_SERIALIZED

Original issue: https://charm.cs.illinois.edu/redmine/issues/1044

We currently only claim to support MPI_THREAD_SINGLE, even though MPI_THREAD_FUNNELED and MPI_THREAD_SERIALIZED actually work.
The only real change needed is to actually implement MPI_Is_thread_main().

Original date: 2016-12-08 14:27:26

Support for MPI_THREAD_FUNNELED: ~~https://charm.cs.illinois.edu/gerrit/#/c/2034/~~ https://github.com/UIUC-PPL/charm/commit/77610c35d5bc582ff83cde41b9cd5b39262c3967
MPI_THREAD_SERIALIZED will take more effort: we'd have to support threads AMPI did not create making calls into AMPI. The issue is with global variables which are only installed via AMPIAPI macros and only work on threads created by AMPI. Also migrating threads not created by AMPI is not supported.


Newsfeed cleanup

Please send your pull requests the develop branch.
Don't forget to add the change to CHANGELOG.md.

Note: Sometimes the development moves very fast. It is highly
recommended that you update your branch of develop before creating a
pull request to send us your changes. This makes everyone's lives
easier (including yours) and helps us out on the development team.
Thanks!

Does the pull request solve a related issue?
If so, can you reference the issue?
What does the pull request accomplish? Use a list if needed.
If it includes major visual changes please add screenshots.

hm, you really forgot the changelog? ;)

hm, you really forgot the changelog? ;)

nope, still working on it :-)

Ready for merging @MichMich if you or anyone got no objections?

did u see the latest commentary about multiple sources not producing output from all?  under MMM-NewsFeedTicker topic. both broken

No, was that in the forums?

yes
https://forum.magicmirror.builders/topic/14784/mmm-newsfeed-ticker-hangs/32?page=4

looks like this might have been user error..

Yes, looks like that he really forgot to set the brackets correctly :-)

PR LGTM

[2021-03-23 11:25:38.560] [ERROR] Whoops! There was an uncaught exception...
[2021-03-23 11:25:38.730] [ERROR] TypeError: self.broadcastItems is not a functi
on
at FeedMe. (E:\OneDrive\MagicMirror\modules\default\newsfeed\news
feedfetcher.js:71:9)
at FeedMe.emit (events.js:198:13)
at SAXStream._parser.parser.emit.args [as emit] (E:\OneDrive\MagicMirror\nod
e_modules\feedme\dist\feedme.js:26:29)
at SAXParser.SAXStream._parser.onend (E:\OneDrive\MagicMirror\node_modules\s
ax\lib\sax.js:190:10)
at emit (E:\OneDrive\MagicMirror\node_modules\sax\lib\sax.js:624:35)
at end (E:\OneDrive\MagicMirror\node_modules\sax\lib\sax.js:667:5)
at SAXParser.end (E:\OneDrive\MagicMirror\node_modules\sax\lib\sax.js:154:24
)
at SAXStream.end (E:\OneDrive\MagicMirror\node_modules\sax\lib\sax.js:248:18
)
at XMLFeedParser._final (E:\OneDrive\MagicMirror\node_modules\feedme\dist\xm
lfeedparser.js:163:21)
at callFinal (_stream_writable.js:617:10)
[2021-03-23 11:25:38.821] [ERROR] MagicMirror will not quit, but it might be a g
ood idea to check why this happened. Maybe no internet connection?
[2021-03-23 11:25:38.969] [ERROR] If you think this really is an issue, please o
pen an issue on GitHub: https://github.com/MichMich/MagicMirror/issues
[2021-03-23 11:29:48.657] [ERROR] (node:7100) UnhandledPromiseRejectionWarning:
TypeError: fetcher.url is not a function
at E:\OneDrive\MagicMirror\modules\default\newsfeed\node_helper.js:57:19
at E:\OneDrive\MagicMirror\modules\default\newsfeed\newsfeedfetcher.js:89:5
at process._tickCallback (internal/process/next_tick.js:68:7)
[2021-03-23 11:29:48.674] [ERROR] (node:7100) UnhandledPromiseRejectionWarning:
Unhandled promise rejection. This error originated either by throwing inside of
an async function without a catch block, or by rejecting a promise which was not
handled with .catch(). (rejection id: 7)

This doenst look like the current develop branch, where did you get it from?

why? what you think is wrong?

maybe when I change to ES5 for iOS 9 compatibility

what branch  are you running? what does "git status" say?

[2021-03-23 11:53:26.837] [LOG]   Starting MagicMirror: v2.15.0-develop

Solved, there is an error on ES5 transform, thank you and sorry

But, also I see that at least two of configs: lengthDescription and hideLoading are ignored

Hideloading is there: https://github.com/MichMich/MagicMirror/blob/10dc315f3b96ebb5e980e47a21011d44bb9da979/modules/default/newsfeed/newsfeed.js#L189
lengthDescription is there: https://github.com/MichMich/MagicMirror/blob/10dc315f3b96ebb5e980e47a21011d44bb9da979/modules/default/newsfeed/newsfeed.js#L170


Avoid the flashing problem of the dropdown menu when user is typing in the search bar

Problem: dropdown menu flashes (displaying a loading message) when the user is typing.

Try to only change the text in the dropdown menu, don't make the whole dropdown menu flash like what google does:
https://user-images.githubusercontent.com/54587893/174679921-79838083-7f9a-4ed8-b389-dee70d8bafe5.mov


Set up Jenkins or Buildbot on a GPU machine

I am tired of not having a CI tool yell at me when I break the GPU compilation pipeline.  It must end.  We already have a candidate machine (napoleon.hiperfit.dk), so I just need to figure which CI tool to use.  Travis will continue to serve for everything that does not depend on having a GPU available.

Working on this now.  Check out http://buildbot.hiperfit.dk (will be http://buildbot.futhark-lang.org once DNS propagates).

Seems to work decently, although user management is manual.  I have added napoleon and gpu3-diku-apl as builders and workers.

This all works really well now.  Every commit is built and tested on a gpuN machine, napoleon, and the TITAN machine.  Benchmark results are dumped here for analysis: http://futhark-lang.org/benchmark-results/


Add Exercise to use conditions

Add Exercise to use conditions

fyi we're changing conditions, might want to wait for the new version coming soon

thanks for the headsup @jerop , I will wait for the update.  Any clue which version might have it ?

@kameshsampath it'll be included in v0.16 release that'll be released this week


Tried to convert this jsx file and failed

The javascript file script.jsx.
var React = self.React;

var CommentBox = React.createClass({
  render: function() {
    return (

        Hello, world! I am a CommentBox.

    );
  }
});

The code to convert javascript
package main

import (
	"github.com/mamaar/risotto/generator"
	"io/ioutil"
	"os"
	"log"
)

func requireJsx(fn string) (js string, err error) {
	f, err := os.Open(fn)
	if err != nil {
		return
	}
	gen, err := generator.ParseAndGenerate(f)
	if err != nil {
		return
	}
	b, err := ioutil.ReadAll(gen)
	if err != nil {
		return
	}
	js = string(b) + "\n"
	return
}

func main() {
	_, err := requireJsx("script.jsx")
	if err != nil {
		log.Fatal(err)
	}
}

Error message:
Line 6:12 Unexpected reserved word

Partly my bad. The JSX code has problem. The attribute class should be className.
Changed the script this way and it will work:
var React = self.React;

var CommentBox = React.createClass({
  render: function() {
    return (

        Hello, world! I am a CommentBox.

    );
  }
});

Although the attribute is a mistake, it is totally valid when you actually compile the JSX with React. I think it shouldn't be showing any error.

Hi. Sorry for the late reply.
Thanks for reporting. Unfortunately I'm busy with other projects so Risotto has staled lately. I'm not sure I'll look into this for a while.


tweak ABIs

@homunkulus r+

:pushpin: Commit 10694b4 has been approved by japaric

:hourglass: Testing commit 10694b4 with merge 10694b4...

:sunny: Test successful - status-travis
Approved by: japaric
Pushing 10694b4ad6badaf9a6048925fc7ef866fe74e8bf to master...


Use the same ami as the instance that does the restart.

Right now during rolling-restart bubuku is trying to find some ami according to some filter that is hard coded in the bubuku code.
This PR is making it different - the ami that is used on the instance will be used on all the other instances.
The upgrade procedure is the following:

upgrade ami on one of the instances.
run the rolling restart from that instance.

:+

:+1:

no changes, rebase only

:+1:

:+1:


co19/LibTest/io/Process/run_A01_t03 failing on Windows

There are new test failures on [dartdevc] Containerizing top-level variables to improve stepping times....Reland "[cfe] Improve encoding of set/list literals for unified collections".
Commit in that range that touched the windows process in the VM:

https://dart-review.googlesource.com/c/sdk/+/167041 https://github.com/dart-lang/sdk/issues/43762

(Need to bisect manually, because the bots don't.)
The tests
co19/LibTest/io/Process/run_A01_t03 RuntimeError (expected Pass)
co19/LibTest/io/Process/run_A01_t03 RuntimeError (expected Pass)

are failing on configurations
dartk-strong-win-debug-x64
dartk-strong-win-release-ia32
dartk-strong-win-release-x64
dartk-weak-asserts-win-debug-x64
dartk-weak-asserts-win-release-ia32
dartk-weak-asserts-win-release-x64

cc @askeksa-google who started bisecting.
cc @a-siva who reviewed the suspect CL.

We're seeing it fail again on
dartk-strong-win-release-ia32
dartk-weak-asserts-win-release-ia32

example logs
So at some point it started passing again and has now flipped back.


update: openssl

Name: openssl
CVEs: CVE-2024-2511, CVE-2024-4603, CVE-2024-4741, CVE-2024-5535
CVSSs: n/a, n/a, n/a, n/a
Action Needed: CVE-2024-2511: update to >= 3.2.1-r2, CVE-2024-{4603,4741}: update to >= 3.2.2, CVE-2024-5535: update to >= 3.2.3
Summary:

CVE-2024-2511: Some non-default TLS server configurations can cause unbounded memory growth when processing TLSv1.3 sessions Impact summary: An attacker may exploit certain server configurations to trigger unbounded memory growth that would lead to a Denial of Service This problem can occur in TLSv1.3 if the non-default SSL_OP_NO_TICKET option is being used (but not if early_data support is also configured and the default anti-replay protection is in use). In this case, under certain conditions, the session cache can get into an incorrect state and it will fail to flush properly as it fills. The session cache will continue to grow in an unbounded manner. A malicious client could deliberately create the scenario for this failure to force a Denial of Service. It may also happen by accident in normal operation. This issue only affects TLS servers supporting TLSv1.3. It does not affect TLS clients. The FIPS modules in 3.2, 3.1 and 3.0 are not affected by this issue. OpenSSL 1.0.2 is also not affected by this issue.
CVE-2024-4603: Applications that use the functions EVP_PKEY_param_check() or EVP_PKEY_public_check() to check a DSA public key or DSA parameters may experience long delays. Where the key or parameters that are being checked have been obtained from an untrusted source this may lead to a Denial of Service.
CVE-2024-4741: Calling the OpenSSL API function SSL_free_buffers may cause memory to be accessed that was previously freed in some situations.
CVE-2024-5535: Calling the OpenSSL API function SSL_select_next_proto with an empty supported client protocols buffer may cause a crash or memory contents to be sent to the peer.

refmap.gentoo:

CVE-2024-2511: https://bugs.gentoo.org/930047
CVE-2024-4603: https://bugs.gentoo.org/932317

Added CVE-2024-4603

Added CVE-2024-4741, CVE-2024-5535.

Set to cvss/CRITICAL following score of https://www.cve.org/CVERecord?id=CVE-2024-5535.


Solved Back Press Issue, Enhanced UI, Added Loading Dialog when Loading Data, Added Animated Splash Screen.

Solved Back Press Issue for both MainActivity and StartActivity, Enhanced some of the UI Components, solved constraint layout issues, Added some new fonts for good UI, Created customizable alert dialog when weather data is loading and added animations on splash screen.

@prasoonsoni back press issue is already solved just remove that and create a new pr

Ok, Sure

Hey @prasoonsoni , also we have people assigned working on Enhanced UI #11 #2 , and Animated Splash screen #29.
So we will only be able to accept a PR of Added Loading Dialog.

So should I do that Added Loading Dialog PR Again?

@prasoonsoni ofc, you can do it.


Explicitly check that otp_attempt param is not nil in order to avoid 'RO...

imho the gem should explicitly check that otp_attempt param is not nil before calling
resource.valid_otp?(params[scope]['otp_attempt'])
otherwise an ArgumentError Exception: ROTP only verifies strings might be raised.
This happens for example when email and password params are set but not otp_attempt, in this case I would expect that current_user is nil and not that it raises an exception.
I would have added tests but I have not found any existing controller spec

Thanks Nicola! I modified the pull request slightly, since Devise actually has functionality for checking this sort of thing built in. Nice catch though!

@ShaneWilton
how is it that 0b6e2214111728e69dddf4d4d2ede5bcc691215a was applied and then reverted?
Thx

@pic Sorry about that - we thought there was something wrong with our implementation and didn't have time to fully test or fix it. I'll reopen this so we can still look at addressing your issue.

Thanks @pic! Sorry about the delay in getting this merged, but I appreciate the contribution.


[Bug]: AttributeError: 'HuggingFaceEmbedding' object has no attribute '_model'

Bug Description
After upgrading LlamaIndex to verison 0.10.36, my ingestion pipeline stopped working.
Version
llama-index-core  0.10.36 & llama-index-embeddings-huggingface  0.2.0
Steps to Reproduce
from llama_index.core import VectorStoreIndex
from llama_index.vector_stores.postgres import PGVectorStore
from llama_index.core import (
    SimpleDirectoryReader,
    StorageContext,
)
from llama_index.readers.file import PyMuPDFReader
from llama_index.embeddings.huggingface import HuggingFaceEmbedding
from llama_index.core import Settings
from llama_index.llms.openai_like import OpenAILike
from llama_index.llms.vllm import VllmServer
from llama_index.core.node_parser import SentenceSplitter
from llama_index.core.ingestion import IngestionPipeline

# LlamaIndex embedding model
EMB_MODEL="BAAI/bge-base-en-v1.5" # For better results you can use the "large" variant
DEVICE="cuda:0" # If running out of GPU RAM, switch to "cpu" (although slower)
Settings.embed_model = HuggingFaceEmbedding(
    model_name=EMB_MODEL,
    device=DEVICE
)

# Create the pipeline with transformations
pipeline = IngestionPipeline(
    transformations=[
        SentenceSplitter(
            chunk_size=1024,
            chunk_overlap=20
        ),
        Settings.embed_model,
    ]
)

Relevant Logs/Tracbacks
RemoteTraceback                           Traceback (most recent call last)
RemoteTraceback:
"""
Traceback (most recent call last):
  File "/home/vmuser/miniconda3/envs/impv_rag/lib/python3.11/multiprocessing/pool.py", line 125, in worker
    result = (True, func(*args, **kwds))
                    ^^^^^^^^^^^^^^^^^^^
  File "/home/vmuser/miniconda3/envs/impv_rag/lib/python3.11/multiprocessing/pool.py", line 51, in starmapstar
    return list(itertools.starmap(args[0], args[1]))
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/vmuser/miniconda3/envs/impv_rag/lib/python3.11/site-packages/llama_index/core/ingestion/pipeline.py", line 124, in run_transformations
    nodes = transform(nodes, **kwargs)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/vmuser/miniconda3/envs/impv_rag/lib/python3.11/site-packages/llama_index/core/base/embeddings/base.py", line 450, in __call__
    embeddings = self.get_text_embedding_batch(
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/vmuser/miniconda3/envs/impv_rag/lib/python3.11/site-packages/llama_index/core/instrumentation/dispatcher.py", line 274, in wrapper
    result = func(*args, **kwargs)
             ^^^^^^^^^^^^^^^^^^^^^
  File "/home/vmuser/miniconda3/envs/impv_rag/lib/python3.11/site-packages/llama_index/core/base/embeddings/base.py", line 341, in get_text_embedding_batch
    embeddings = self._get_text_embeddings(cur_batch)
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/vmuser/miniconda3/envs/impv_rag/lib/python3.11/site-packages/llama_index/embeddings/huggingface/base.py", line 149, in _get_text_embeddings
    return self._embed(texts, prompt_name="text")
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/vmuser/miniconda3/envs/impv_rag/lib/python3.11/site-packages/llama_index/embeddings/huggingface/base.py", line 124, in _embed
    return self._model.encode(
           ^^^^^^^^^^^
AttributeError: 'HuggingFaceEmbedding' object has no attribute '_model'
"""

The above exception was the direct cause of the following exception:

AttributeError                            Traceback (most recent call last)
File <timed exec>:2

File ~/miniconda3/envs/impv_rag/lib/python3.11/site-packages/llama_index/core/ingestion/pipeline.py:722, in IngestionPipeline.run(self, show_progress, documents, nodes, cache_collection, in_place, store_doc_text, num_workers, **kwargs)
    718     with multiprocessing.get_context("spawn").Pool(num_workers) as p:
    719         node_batches = self._node_batcher(
    720             num_batches=num_workers, nodes=nodes_to_run
    721         )
--> 722         nodes_parallel = p.starmap(
    723             run_transformations,
    724             zip(
    725                 node_batches,
    726                 repeat(self.transformations),
    727                 repeat(in_place),
    728                 repeat(self.cache if not self.disable_cache else None),
    729                 repeat(cache_collection),
    730             ),
    731         )
    732         nodes = reduce(lambda x, y: x + y, nodes_parallel, [])
    733 else:

File ~/miniconda3/envs/impv_rag/lib/python3.11/multiprocessing/pool.py:375, in Pool.starmap(self, func, iterable, chunksize)
    369 def starmap(self, func, iterable, chunksize=None):
    370     '''
    371     Like `map()` method but the elements of the `iterable` are expected to
    372     be iterables as well and will be unpacked as arguments. Hence
    373     `func` and (a, b) becomes func(a, b).
    374     '''
--> 375     return self._map_async(func, iterable, starmapstar, chunksize).get()

File ~/miniconda3/envs/impv_rag/lib/python3.11/multiprocessing/pool.py:774, in ApplyResult.get(self, timeout)
    772     return self._value
    773 else:
--> 774     raise self._value

AttributeError: 'HuggingFaceEmbedding' object has no attribute '_model'

@dosu Have you got an example about how the HuggingFaceEmbedding object need to be initialized in the current version of LlamaIndex? As I mentioned, my code was working before the upgrade.

I think this is an issue with local embeddings not being compatible with multiprocessing.
I'd suggest not setting num_workers to anything greater than 1

Instead, increase the batch size on the embed model if you want more throughput (assuming you have the memory)

Thanks, @logan-markewich. What I don not understand is why it was working before.

@vecorro did you have num workers set higher than one before? I would be a little surprised that it worked haha

I know we switched to sentence transformers under the hood, maybe that had some effect

@vecorro did you have num workers set higher than one before? I would be a little surprised that it worked haha

YES

You are right @logan-markewich, setting num_workers=1 fixed the issue.

I think this issue should be reopened. Setting num_workers to 1 is not always desirable. In previous versions, it worked without any changes.

Right @JoepdeJong that's what I told @logan-markewich but he said to be surprised that values >1 were working.

@vecorro did you have num workers set higher than one before? I would be a little surprised that it worked haha

Can confirm I was using values > 1 successfully before the latest version.

It seems that the problem I had is related to caching.
https://github.com/jonfairbanks/local-rag/issues/53#issuecomment-2120087530
Not sure what this has to do with num_workers=1.

I have the same issue. It works when num_workers=1 and not workable when number_workers>1, any solution now?


The AssetStoryblok type is missing properties

{
  "id": 16807114,
  "alt": "",
  "name": "",
  "focus": "",
  "title": "",
  "source": "",
  "filename": "https://a.storyblok.com/f/foo/bar.jpg",
  "copyright": "",
  "fieldtype": "asset",
  "meta_data": {},
  "is_private": false,
  "is_external_url": false
}

This is an example of a real JSON payload that was returned by the Storyblok API for an "asset" field. As you can see, it does not overlap entirely with the AssetStoryblok type generated by generate-sb-types:
export interface AssetStoryblok {
  alt?: string;
  copyright?: string;
  id: number;
  filename: string;
  name: string;
  title?: string;
  focus?: string;
}

As you can see, the payload contains the following additional fields: is_external_url, is_private, source, fieldtype and meta_data.

Hi @ryami333 thanks for reporting this inconsistency, much appreciated!
I have filed a PR #122 to fix some of the types for blok fields, including the Asset and Multiasset fields.


Why not average the log probs in DPO trainer?

In the DPOTrainer the _get_batch_logprobs method takes a param called average_log_prob, which if enabled, lets us get back the averaged logprobs for the chosen and rejected sequences. If disabled, we get back the sum of the logprobs. But in the DPOTrainer, it's hard-coded to False.
I was just wondering why this is the case. Couldn't this bias the model towards writing long sequences? Thanks for the help.

Hi @dblakely , thanks, this is a good question
I personally don't have the answer, perhaps we should make that modulable but not sure. I would like to ask this question to @kashif and @natolambert, what do you think of having that parameter configurable?

My experience is training will collapse with average_log_prob.

I tested it as well and am seeing the same thing - the model became pretty degenerative

Another thought. If you average logprob, each token contributes equally to the results. It's kind of sequence level instead of token level reward. If an answer wins, only some token are better.

I have another possible explanation from math.
We understand that $\pi_\theta(y_w) = \pi_\theta(y_w^1) \pi_\theta(y_w^2 | y_w^1) \cdots \pi_\theta(y_w^n | y_w^1\cdots y_w^{n-1})$, where $y_w^i$ denotes the i-th token of $y_w$.
Consequently, $\pi_\theta(y_w | x) = \pi_\theta(x | y_w)\pi_\theta(y_w)/\pi_\theta(x) = \pi_\theta(y_w^1) \pi_\theta(y_w^2 | y_w^1) \cdots \pi_\theta(y_w^n | y_w^1\cdots y_w^{n-1}) / {\pi_\theta(x)}$.
Similarly, $\pi_\theta(y_l | x) = {\pi_\theta(x | y_l)\pi_\theta(y_l)} / {\pi_\theta(x)} = {\pi_\theta(y_l^1) \pi_\theta(y_l^2 | y_l^1) \cdots \pi_\theta(y_l^n | y_l^1\cdots y_l^{n-1})} / {\pi_\theta(x)}$.
Thus, $\log \pi_\theta(y_w | x) - \log \pi_\theta(y_l | x)$
$= \log \pi_\theta (x|y_w) + \sum_{i=1}^{n_{w}} \log \pi_\theta(y_w^i | y_w^1 \cdots y_w^{i-1}) - \log \pi_\theta (x|y_l) - \sum_{i=1}^{n_l} \log \pi_\theta(y_l^i | y_l^1 \cdots y_l^{i-1})$
Assuming $\log \pi_\theta (x|y_w) \approx \log \pi_\theta (x|y_l)$, the calculation in the code would be correct.
Even if the assumption $\log \pi_\theta (x|y_w) \approx \log \pi_\theta (x|y_l)$ is not made, the above derivation indicates that a sum of log probabilities is a more justifiable choice than an average of log probabilities.


acquistion: initial implemenentation of orders

Sets up the invenio records rest configuration for the
acquisition_purchase_order resource.

addresses #603

Requires https://github.com/inveniosoftware/invenio-stats/pull/104 to be released.


Cannot read file error after uninstalling extension

VSCode Version: 1.2.1
OS Version: Windows 7

Steps to Reproduce:

Install JSHint extension
Uninstall JSHint extension

Error message:
Cannot read file C:/Users/Me/.vscode/extensions/dbaeumer.jshint-0.10.12/package.json: ENOENT: no such file or directory, open 'C:\Users\Me.vscode\extensions\dbaeumer.jshint-0.10.12\package.json'.
After I uninstalled JSHint I now receive this error every time I open vscode, even after restarting.

Different extension, the same situation.

What is inside C:/Users/Me/.vscode/extensions? Can you show me a screenshot of the error?

I had 2 extensions: just Python, and MagicPython. I've unistalled first extension. Now the content of the folder you asked:
C:\Users\bialix.vscode\extensions>dir /b
magicstack.MagicPython-0.5.11
And after I did the command above I no longer get the error message, so I'm unable to make screenshot.
It was a window in the bottom of main editor view with the message from first topic of this issue.

Just reproduced again (install/unistall). Icon of the extensions at the bottom left had red dot and tooltip said "Extensions (1 issue)". Clicking on this icon opened window with title "Output" and the text as quoted above. And folder with extensions had no corresponding directory.

No longer receiving the issue anymore, but what @bialix just posted was the same error

Yes, it seems it can cure itself, so I guess it's related to hard disk cache or something like that.

Yes, the errors should eventually go away. It's a part of the install/uninstall procedure.
Closing the issue.


Fix reactivity bug

Using attrs in a render function will become unreactive as soon as the user changes it.
An example of this is an input's value attribute. It will be reactive to the component data, but as soon as the user manually edits the input field, the input's value will no longer match the Vue data.
Instead, it should be using domProps which sets the elements attributes via JavaScript. This will ensure the attributes stay up to date with the component's data.

A StackOverflow question about this issue:
https://stackoverflow.com/questions/53149991/render-function-on-input-removes-reactivity-in-vue/53152831#53152831
Github issue:
https://github.com/vuejs/vue/issues/9027#issuecomment-435797509
Code Sandbox:
https://codesandbox.io/embed/oooq7o2q16?module=%2Fsrc%2FApp.vue

Fixed in rewrite.


Make ontology browsable from the website (e.g. embed OLS)

Maybe it is possible to directly embed OLS using an ; that would be a better usability than warping the user to OLS.

there is a tree viewer widget, but I can't rmbr exactly why, but we decided on autocomplete widget, but I think we can potentially look in adding the tree viewer widget (in addition) down the road

closing this, if we still would like to consider tree viewer, happy to open another ticket in https://github.com/cell-ontology/cell-ontology.github.io/issues


Geodesic issues

Fixes #410

@shreyasbapat

Hello @nimeshvashistha! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file src/einsteinpy/geodesic.py:

Line 19:1: W293 blank line contains whitespace

In the file src/einsteinpy/tests/test_geodesics.py:

Line 46:23: W504 line break after binary operator
Line 55:24: W504 line break after binary operator

Alright @nimeshvashistha , you were finally able to resolve the merge conflicts! So, now we would find why your tests fail. Can you open the CI log and investigate?
Ping me if you are unable to understand which logs to check.

This PR would have merge conflicts due to #435 .

Are you still working on this?

No, but i made some changes which show in #435

No, but i made some changes which show in #435

Okay, closing this then :)
Thanks!!


Trouble to build for no field or method errors in cel-go

Supporting issue: Trouble to build for no field or method errors in cel-go bufbuild/protovalidate-go#30

I fixed my troubles by this demo project! Thank you!


new crash exploit

is this able to get patched? it has been being abused to hell
/give @a dirt{display:''}

Problem with ViaBackwards/ViaVersion.

it is everything where display is not a object

Report on ViaVersion/ViaBackwards github(s)
https://github.com/ViaVersion/ViaVersion/issues
https://github.com/ViaVersion/ViaBackwards/issues

isn't it a kick exploit?

Report on ViaVersion/ViaBackwards github(s)
https://github.com/ViaVersion/ViaVersion/issues
https://github.com/ViaVersion/ViaBackwards/issues

Since its a ViaVersion issue then its probably a good idea to close it right?

isn't it a kick exploit?

No, it is Crash Exploit I'm pretty sure.

its already patched and its was a kick

its already patched and its was a kick

Verification? OP?

its already patched and its was a kick

Verification? What results do you get, OP?

after the exploit was performed the server is not impacted and only the players who "loaded" (were supposed to receive a packet containing data about) the item (having the item in their inventory, being close to someone holding it) were disconnected from the server.
I would classify that as a kick exploit.
A more targeted version is possible by the use of /minecraft:item replace making it so the item is never in the player's hand and so only "loaded" by them.

its already patched.,.,.,


Updates NuGet to 3.2.0

NuGet also changed the location where they publish their binaries (and they're just a straight binary now), so this PR accounts for that change as well.

Thanks :+1:


ActiveQuery create model not through dependency injection

What steps will reproduce the problem?
\Yii::$container->set('app\models\Order', 'app\resources\Order');

$product = Product::findOne(1);
$orders = $product->orders;

//$order not app\resources\Order

What is the expected result?
\Yii::$container->set('app\models\Order', 'app\resources\Order');

$product = Product::findOne(1);
$orders = $product->orders;

//$order == app\resources\Order

What do you get instead?
Additional info

Q
A

Yii version
2.0.8

PHP version
5.6

Operating system
centos7

yii core should call any class through dependency injection(Yii::createObject())

no, it should not. If you want to replace an existing class in the framework, you can set it in Yii::$classMap. No idea why you would do that in your application...


[Oculus] Outdated support

According to your testing, does the issue occur with ONLY Immersive Portals mod installed?
No
Forge Version
43.2.14
ImmersivePortals Version
immersive-portals-2.3.5-mc1.19.2-forge
Latest Log
https://gist.github.com/MrTangoPIG/42b9c8303408b76fbd56bae1dd63f026
Crash Report (if applicable)
https://gist.github.com/MrTangoPIG/dd814b0027e8a783200c372d170f4af1
Steps to Reproduce
1st instance:
Open game
load world
Open shaders options
Apply shader
Crash
2nd instance (and i guess thereafter without disabling the shader pre-launch):
Open game
Crash
What You Expected
?
What Happened Instead
??

[X] I have confirmed the details provided in this report are concise as possible and does not contained vague information (ie. Versions are properly recorded, answers to questions are clear).
[X] I have confirmed this does not happen on the fabric version (If it does please report to here instead).
[X] I have confirmed I'm not using a hybrid server (Magma, Cauldron) or a different (non official) Forge Server release.
[X] I have confirmed this issue is unique and has not been reported already.

Downgrade Oculus.

Installed version to oculus-mc1.19.2-1.2.8a.jar getting crash...
*does oculus need specific rubidium versions (and so on down dependency line)?

seems oculus support on 1,19,2 is still outdated.
https://mclo.gs/K7lKK4o

Yes it is, thus this issue is still open...

I'm having the exact same issue on 1.18.2 right now

Is it a mod compatibility issue?
Yes
Minecraft version
1.19.2
Oculus Version
oculus-mc1.19.2-1.2.8a.jar / oculus-mc1.19.2-1.6.4.jar
Sodium Version
rubidium-0.6.2b.jar
Operating System
Windows 11
What is your GPU?
RTX 4070Ti
Java Version
java <IP_ADDRESS> 64 Bits
Reproduction Steps
Hello, i have a problem, it is possible to solve it? How can i have the three mod without problem of render or crash? :/
With oculus-mc1.19.2-1.2.8a.jar no problem with immersive-portals-2.3.5-mc1.19.2-forge.jar but the render of the bar of Neat mod is BAD

=========================================================
With oculus-mc1.19.2-1.6.4.jar when I DISABLE immersive-portals-2.3.5-mc1.19.2-forge.jar but the render with the bar of Neat mod is fine

=========================================================
With oculus-mc1.19.2-1.6.4.jar crash with immersive-portals-2.3.5-mc1.19.2-forge.jar /!\
Crash Report file and latest.log
crash-2023-07-20_03.57.24-client.txt
latest.log
Additional context
No response

It has been cited by the people on the Oculus/Rubidium server that the actual fault is on the side of this mod.
And they provide a specific example of getTerrainVertexShaderSource; according to them, this method no longer exists in Oculus or Iris.

Hi, any updates on this ?


Delimiter not working

I tried using the delimiter to make the information go to the line below.
But it returns like this in the csv file:
B;A;SL;C;ap;e;a;t;
all in just one line.
My code:
const json2csvParser = new Parser({infos2, delimiter: ';' });
const csv = json2csvParser.parse(dados);
fs.writeFileSync('./dados.csv', csv, 'utf8');

Are you looking to override the end of the row? Delimiter is for columns, which are supposed to be on one line. Maybe you want eol?

If you can add an example JSON and the CSV that you would expect, we can help better.

Closing because of a lack of response.
Feel free to reopen.


Ianmacleod/completion sync error throws 4xx

fixing errors so that we don't run into so many 5xx errors when TGI returns an issue due to input prompt window being too large.
tested with just up locally to make sure behavior is consistent when calling with a prompt that exceeds the max number of tokens specified by TGI tokenizer.

error messages now look like this for completions-sync and completions-stream:


Replace "can support 2xx HTTP status (other than 200)" test

The test can support 2xx HTTP status (other than 200) occasionally fails because it's relying on an external server that seems to have spotty availability.  This causes CI to someone fail for no good reason.  We should update this test to spy/mock the XHR request so that it doesn't need to access the network at all.

You guys might find this useful:
https://libraries.io/npm/terriajs-jasmine-ajax
It's just jasmine-ajax with my three pull requests merged:
https://github.com/jasmine/jasmine-ajax/pulls

Thanks @kring.  I think a simple spy should handle this, but the way loadWithXhr is currently written might make it more problematic than that.  We'll check out your library but I think some minor changes to loadWithXhr may make it unnecessary (in Cesium's case).
I also opened the above PR to temporarily ignore the problematic test because it now seems to be failing most of the time.


Organization contact e-mail

On https://github.com/maputnik there is still the contact from @lukasmartinelli. Is this still a way to contact us? If not we should change or remove that e-mail address:

Yeah I agree, on a slightly related note we should probably add a 'code of conduct' to the repo/org also, which would need email address(es).
What would be really cool is if there was a service that would act as an 'email group' for GitHub. So for example if someone emailed<EMAIL_ADDRESS>that would then forward those emails to the core maintainers. Does anybody know if something like that exists?

Does anybody know if something like that exists?

What about setting up an email address using an email provider and forward incoming emails to maintainers?
This would be possible e.g. with https://posteo.de/en, costs 1€/month.
<EMAIL_ADDRESS>is currently available: https://posteo.de/en/signup


assistant.py error out

Hi,
when I am trying to run assistant.py I am getting following error
python assistant.py
Traceback (most recent call last):
File "assistant.py", line 27, in
from google.assistant.library import Assistant
ImportError: No module named 'google.assistant.library'

missed this:
pip install --upgrade google-assistant-library

@rahul1990bhatia
My sample code works fine. I changed my subscribe key and public key in assistant.py also I entered magic-mirror-device-id and after running assistant.py I receive msgs: pubnub subscription completed and hello from python!! but there is no response from google assistant , google assistant does not work.
PLease help me out. Thanks in advance.


fixes #79 - support multiple Load calls on single Document instance

This destroys a possible use case where Load() is used to ADD data from a file to the existing data.
Not sure if this is relevant. I think the new way is better because "Load" is usually understood as "replace all existing", as opposed to "Import" or "Add".
Anyway I suggest to add a comment to the Load() function and to changelog that all existing data will be replaced.

Hi @Sven-HP - thanks for pointing this out. As you mentioned Load() is generally associated with "replace all existing", so I'm not sure a special note in the API documentation is needed?
But yes, since we're changing behaviour of Load() I can add a note in the release notes for this upcoming version.
If an Append() function is needed in the future we can introduce that.


Added vec_clear and vec_find.  Now if user passes init_len == 0 to vec_new, memory won't be allocated at this point, but when the vector is first used (push, insert, etc).
Implemented vec_clear that just changes the length of the vector to zero reserving all allocated space.
Implemented vec_find a simple search taking f comaprison method as arg.

I'll split my notes into sections:

Inspecting the changes might be easier if changes would be grouped to commits that would be independent of one another. I'd recommend that you try out making use of the VSCode Source Control to make partial commits from a file (only stage a specific section of a file), if needed.

Vec_clear and vec_find

vec_clear: do you need to allow for the user to pass in a function to free allocated memory from within the elements that are "cleared"?

Lazy initialization

Other functions

vec_free: should this incorporate the iteration of elements to free whatever might need to be freed from within them? is it the user's responsibility to iterate the array to deallocate the memory allocated within the elements?
vec_get: should not return an index that does not contain anything, I think. here, it would be when index == src->len.
vec_from: dst->alloc_size bytes is copied regardless of how many elements there are in src. Will garbage get copied? What if src is empty? There should be a check for what the right copy size is.
vec_copy: same issue as above, copy_size is set to src->alloc_size. Should it be dst->alloc_size? Also, if dst->memory is null, a new vector is created but the size is set confusingly with src->len and dst->elem_size. Should the latter be src->elem_size?
vec_resize: is the call to vec_copy redundant?
vec_push: could the argument in vec_resize simply be dst->len * 2?
vec_remove: would you not want to implement same functionality as in vec_pop? the user can just throw the pointer away if that's not needed.
vec_append and vec_prepend: are 1 and dst->elem_size the correct one to use if dst->memory does not exist? I don't quite understand the allocation strategy. also, if you do changes for vec_get, that has implications here.
Some of these may be misunderstandings related to lazy initialization, though.

You are correct.

This is a good question. Let's think about it separately.

4.1 I think it's ok to assume the user will "manually" manage possible data deallocation in the vector.
void deallocator(void *elem)
{
    // Element deallocation.
}

int main(void)
{
    t_vec v;
    ...
    vec_iter(&v, deallocator);
    vec_free(&v);
    ....
}

I see the main use-case for the vector being with unallocated data anyways. However a vec_destructor(t_vec *src, (*dealloc)(void *)) could possibly be implemented as well.
4.2 Vec from is meant to be a constructor, so it allocates a new vector from the contents found in the pointer. In this case the alloc_size will always match.
4.3 This is correct I think. We either copy as much as is available in src or if dst->alloc_size can't fit everything we copy as much as we can ie. dst->alloc_size amount. There is the question should we actually make an error condition for src->elem_size != dst->elem_size. It's a not so obious if we want to do that. If not the data could be sort of "typecast" by copying from a vec of other type to another. Don't know if this is important though.
4.4 Indeed it is, corrected.
4.5 Also corrected.
4.6 Will think on this in a separate PR
4.7 These are also "optimizations" I will consider separately.


Report when debug enabled.

Signed-off-by: Eric Windisch<EMAIL_ADDRESS>

Coverage decreased (-0.6%) to 89.0% when pulling 756d4a7684ed0e4b37174cf2e314348e72c1074d on ewindisch:debug-submit into 44c6123e0e685e2bdea2a4f9be0d5c697c848a06 on iopipe:master.

Coverage decreased (-0.1%) to 89.5% when pulling 222ca38f4394b0d3dbf830eb600bf2cc88cfdb12 on ewindisch:debug-submit into 44c6123e0e685e2bdea2a4f9be0d5c697c848a06 on iopipe:master.

@pselle updated!

Coverage increased (+0.3%) to 89.5% when pulling a6f65a3d2041a2f6e64e1d5171990f54c4918c1f on ewindisch:debug-submit into 988fef3d8041ef433723e5776033dc7b21794d86 on iopipe:master.


lacking unit tests for actions

Test file test/actions/game.actions.spec.ts lacks coverage for three actions:

initializeBidding
calculateBattleResult
declareBomb

there are placeholders already created for this tests. Those might be extended.

https://github.com/gornanization/1k/pull/35


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/63996

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/64604

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/64897

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/84454

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.


VS Code: Designer removes "intercept" section from YAML

The VS Code designer doesn't seem to yet have support for intercept. The main problem is that if intercept is manually added to the yaml-file, the designer completely removes it.
Steps to reproduce:

Create route using designer
Add intercept into the top of the yaml-file and save file
Re-open the designer
Make a change using the designer
Notice how intercept is gone from the yaml-file

But! It seems the designer has a support for adding intercept using route configuration! That is great, and perfect workaround in my situation.
So maybe close this, what do you think?

Manually editing the YAML to this seems to have worked:
- routeConfiguration:
    intercept:
      - id: intercept-6d50
        intercept:
          id: intercept-233
          steps:
            - log:
                message: 'Intercepted: ${body}'

The designer doesn't render it but the good thing is that the designer doesn't seem to be deleting anything.

have u tried this yaml in a java app, which camel version?
I tried this in my test env with camel version 3.20.0, it did not work, still got the error message as you mentioned above "Node should have only have 1 key, was: 2 keys: [id,steps] (Maybe this is an indent problem in the YAML source)."

@mikoskinen Karavan designer uses explicit YAML DSL. Yes, you need to have routeConfiguration and route on the top level.

have u tried this yaml in a java app, which camel version? I tried this in my test env with camel version 3.20.0, it did not work, still got the error message as you mentioned above "Node should have only have 1 key, was: 2 keys: [id,steps] (Maybe this is an indent problem in the YAML source)."

Manually editing the YAML to this seems to have worked:
- routeConfiguration:
    intercept:
      - id: intercept-6d50
        intercept:
          id: intercept-233
          steps:
            - log:
                message: 'Intercepted: ${body}'

The designer doesn't render it but the good thing is that the designer doesn't seem to be deleting anything.

below yaml should work for me. 'id' field made it a bad parsing.
`

routeConfiguration:
intercept:
- intercept:
steps:
- log:
message: 'Intercepted: ${body}'
`


Getting child with the tallest height

Hi , I saw that the expandablepageview goes through each child's height before render.. any intention of adding a feature to default the size of the expandablepageview to the height of the child with the largest height? Thanks!

Hi @julian919, ExpandablePageView adjusts its size to the size of currently displayed child, so it will match the size of the biggest child whenever it is currently displayed. I'm not sure if I answered your question, please let me know if there is something else.

@julian919 hello! did you find the solution?

Closing this issue due to lack of activity. Please let me know if there are some additional questions so I can reopen it.


Doc dns-server-cache-snooping-attacks has non-working URL

https://github.com/MicrosoftDocs/SupportArticles-docs/blob/main/support/windows-server/networking/dns-server-cache-snooping-attacks.md#symptoms
The URL has changed to https://simpledns.plus/kb/152/what-is-dns-cache-snooping-and-how-do-i-prevent-it

Thanks. @ToddMaxey. I have updated the link.


Fix indentation in README

There was a tab instead of 4 spaces in an example, but the subsequent lines used spaces. Not a big deal, but still.

Coverage decreased (-10.3%) to 74.138% when pulling cd88b9280fd41d557273290380d359aec2cf452f on iamale:patch-1 into 6fb3a9ad465a4987c58c9c1723a62520f80188da on libwilliam:master.

I'm not sure how I missed this pull request.
Thank you!


Appsetting issue

Repro steps:

Action: appService.appSettings.Add
Error type: BadRequest
Error Message: AppSetting with name 'MICROSOFT_BOT_APP_ID=66cc0dcb-c870-4ac1-b901-00b3635de619' is not allowed.
Version: 0.13.0
OS: win32
VS Code Version: 1.33.1

Call Stack
module.exports.E.l.pipeline (dist/extension.bundle.js:277:1449893)
s (dist/extension.bundle.js:277:112563)
s (dist/extension.bundle.js:165:26036)
dist/extension.bundle.js:165:27276
o (dist/extension.bundle.js:223:7054)
dist/extension.bundle.js:165:24957
$.module.exports.t.requestLibrarySink.t.headersOnly.t.streamedResponse.t.body.function.t.body.pipe.a [as _callback] (dist/extension.bundle.js:1:521656)
$.module.exports.$.init.t._callback.t.callback.t.callback (dist/extension.bundle.js:253:5461)
$.emit (events.js:182:13)
$.<anonymous> (dist/extension.bundle.js:253:18274)
$.emit (events.js:187:15)
IncomingMessage.<anonymous> (dist/extension.bundle.js:253:17116)
Object.onceWrapper (events.js:273:13)
IncomingMessage.emit (events.js:187:15)
endReadableNT (_stream_readable.js:1090:12)
process._tickCallback (int

Hi @surajsingh290
Seems like your key value isn't valid due to the - in them.  But I believe that you can make the key MICROSOFT_BOT_APP_ID and the value 66cc0dcb-c870-4ac1-b901-00b3635de619 like so

But there is a bug that keeps causing the app setting entry to fail if it has failed once (seems like the key is being cached or something) so I will investigate that.
Thanks for reporting!


Add bot filters and more firewall features

I believe that Gate would win with every possible proxy, but it needs some features against attacks, there are couple projects like:
https://github.com/Leymooo/BungeeCord
https://github.com/Elytrium/LimboFilter
which creates feature like falling check - after joining it checks if the player falls/sends movement packets - if not, they are bot and they get kicked.
We noticed that Gate has some CPU spikes under the attack, it could be benchmarked and maybe better structured or it would needs some debug for bad packets or similar things.
Keep up the good job! ❤

amazing features, i have problem with bot attacks. I get bypassed with all plugins.

Taking notes here. As part of the mitigation, we could check if the IP has a fraudulent history by using some of the available API services, e.g.: this one.

Taking notes here. As part of the mitigation, we could check if the a client IP has a fraudulent history by using some of the available API services, e.g.: this one.

Not bad but the problem this api is limited to 5k lookups and only upgrade to 50k it cost 499$.

Checkout the Gate Lite mode in combination with OVH DDoS protection.
https://gate.minekube.com/guide/security.html#ddos-protecting-your-minecraft-server
#113

I am archiving this inactive request for now.


Unable to use a constant record expression in a query comparison

What did you do?
I have a defined record which is a constant like:
  - expr: (1-0.99)
    record: error_budget

When I query error_budget I can see its constant value. Equally if I query (6 * error_budget) I see the expected value.
What did you expect to see?
Records to be returned as if the right hand side of the comparison is resolved.
What did you see instead? Under which circumstances?
When I query like:
errors_per_actions:ratio_rate30m > (6 * error_budget)

I get "Empty query result"
However if I replace in line the error_budget value like
errors_per_actions:ratio_rate30m > (6 * (1-0.99))

Then I get results as I expect.
I'm probably missing something obvious but this looks like something I should be able to do.
System information
Linux 5.4.156-83.273.amzn2.x86_64 x86_64
Prometheus version
prometheus, version 2.36.1 (branch: HEAD, revision: 3c0a100dec2950f23e91f7efbf8867750960807d)
  build user:       root@e59c79158868
  build date:       20220609-15:25:21
  go version:       go1.18.3
  platform:         linux/amd64

Prometheus configuration file
No response
Alertmanager version
No response
Alertmanager configuration file
No response
Logs
No response

https://prometheus.io/docs/prometheus/latest/querying/operators/#vector-matching
Matching metric > number works differently than metric > metric.
metric > metric only works if both sides had same set of labels.
In your case, since you seem to have a single global error_budget metric, you can use empty on clause (on ()) to force comparison, something like errors_per_actions:ratio_rate30m > on() (6 * error_budget)

https://prometheus.io/docs/prometheus/latest/querying/operators/#vector-matching
Matching metric > number works differently than metric > metric. metric > metric only works if both sides had same set of labels.
In your case, since you seem to have a single global error_budget metric, you can use empty on clause (on ()) to force comparison, something like errors_per_actions:ratio_rate30m > on() (6 * error_budget)

Ah of course, I was missing something obvious.
Also a good hint to use a more scoped name or seek to lookup by labels. Thanks!


Connects to bitshares.eu even if I pass a node to bitshares

I'm using python-bitshares and python-graphenelib master checkouts.
It looks like graphenelib opens a websocket connection based on a shared configuration when BlockchainInstance is constructed, and Notify then opens its own websocket connection a little a later.  The shared configuration contains the default node.bitshares.eu endpoint, which is presently down, so things are crashing for me on use.
What version combination is working these days?
bitshares = BitShares(node='wss://api.bts.mobi/ws', num_retries=-1)
notify = Notify(bitshares=bitshares, markets=list(), on_tx=on_tx)

Traceback (most recent call last):
  File "try.py", line 248, in <module>
    notify = Notify(bitshares=bitshares, markets=list(), on_tx=on_tx)
  File "/home/user/.local/lib/python3.8/site-packages/bitshares/notify.py", line 69, in __init__
    BlockchainInstance.__init__(self, **kwargs)
  File "/home/user/.local/lib/python3.8/site-packages/bitshares/instance.py", line 21, in __init__
    AbstractBlockchainInstanceProvider.__init__(self, *args, **kwargs)
  File "/home/user/.local/lib/python3.8/site-packages/graphenecommon/instance.py", line 27, in __init__
    self._blockchain = self.shared_blockchain_instance()
  File "/home/user/.local/lib/python3.8/site-packages/graphenecommon/instance.py", line 75, in shared_blockchain_instance
    self._sharedInstance.instance = klass(**self._sharedInstance.config)
  File "/home/user/.local/lib/python3.8/site-packages/graphenecommon/chain.py", line 56, in __init__
    self.connect(node=node, rpcuser=rpcuser, rpcpassword=rpcpassword, **kwargs)
  File "/home/user/.local/lib/python3.8/site-packages/graphenecommon/chain.py", line 86, in connect
    self.rpc = self.rpc_class(node, rpcuser, rpcpassword, **kwargs)
  File "/home/user/.local/lib/python3.8/site-packages/grapheneapi/api.py", line 46, in __init__
    self.connect()
  File "/home/user/.local/lib/python3.8/site-packages/grapheneapi/api.py", line 84, in connect
    raise(e)
  File "/home/user/.local/lib/python3.8/site-packages/grapheneapi/api.py", line 79, in connect
    self.connection.connect()
  File "/home/user/.local/lib/python3.8/site-packages/grapheneapi/websocket.py", line 28, in connect
    self.ws.connect(
  File "/home/user/.local/lib/python3.8/site-packages/websocket/_core.py", line 226, in connect
    self.handshake_response = handshake(self.sock, *addrs, **options)
  File "/home/user/.local/lib/python3.8/site-packages/websocket/_handshake.py", line 80, in handshake
    status, resp = _get_resp_headers(sock)
  File "/home/user/.local/lib/python3.8/site-packages/websocket/_handshake.py", line 166, in _get_resp_headers
    raise WebSocketBadStatusException("Handshake status %d %s", status, status_message, resp_headers)
websocket._exceptions.WebSocketBadStatusException: Handshake status 503 Service Unavailable

See https://github.com/bitshares/python-bitshares/issues/298?

See https://github.com/bitshares/python-bitshares/issues/298?

thanks !

thanks !


Fix Requests Session verify parameter

Due to bad variable naming, the wrong value was being passed to the
verify session parameter. Let's rename the variables to reflect the
insecure SSL parameter semantics.
CLOUDBLD-203
Signed-off-by: Athos Ribeiro<EMAIL_ADDRESS>Maintainers will complete the following section

[x] Commit messages are descriptive enough
[x] Code coverage from testing does not decrease and new code is covered
[x] JSON/YAML configuration changes are updated in the relevant schema
[x] Changes to metadata also update the documentation for the metadata
[x] Pull request has a link to an osbs-docs PR for user documentation updates
[x] New feature can be disabled from a configuration file

No release notes added:
There is already an entry in the bug fixes for the next release:

Fetch cachito configurations in worker builds to avoid OCP failures due to long user parameters.


Allowing newer versions of Redis to be used

Fixes #12

+1 please

Bump (:

@tonyskn ping ping ping


Witty Cloud ?

Witty cloud board also looks great for this project - or ESP12F + pinout board.
I'll try it out.

An ESP-01/S will also work well,  and so will a WeMos Mini D1(with ESP-12F). The D1's are easier to find online than the witty.


add TLS for argocd ingress

Description of your changes

Add TLS and Certificate for ArgoCD

Fixes #
I have:

[ ] Read and followed Upbound's contribution process.
[ ] Run make reviewable to ensure this PR is ready for review.
[ ] Added backport release-x.y labels to auto-backport this PR, as appropriate.

How has this code been tested

/test-examples


Update slack link

Does this mean our Slack link was down for a week? We need to make sure to
merge the update same day so we don't have down time.
On Thu, Jul 8, 2021 at 3:24 PM valentinewallace @.***>
wrote:

Merged #48
https://github.com/lightningdevkit/lightningdevkit.org/pull/48 into
main.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/lightningdevkit/lightningdevkit.org/pull/48#event-4997458499,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACPUA5IXOOZJBOHPCX7JX3TWYQTJANCNFSM47W4OQ3A
.

Was going to make this change as part of the vuepress migration but it's not ready yet and wasn't 100% sure if merging this would deploy the new site. Using a development branch for the new site until ready.
A week is too long though so shouldn't happen again.

fyi I integrated the update on the develop branch.


Incorrect moon phase being displayed

I noticed that the wrong moon phase is being displayed in the card

The state of the sensor.moon on this day was actually waning_gibbous

This also manifests itself in my case. The language settings are set to German everywhere.

Same problem in the Czech language.

The issue is still there.

I have pasted the moon phase for my location from timeanddate.com as well.

I am also getting this issue! All language is in english but displaying "full_moon!" when my moon entity is showing "Waning gibbous "

Hey guys, this issue was resolved in #134 and you may upgrade to v1.1.0 :)