P200 connections becoming “corrupt” when installing new versions.

Sometime in the last few betas my connections to 3 P200s have started crashing and becoming unable to connect after installing a new beta.
I tend to install the latest betas pretty regularly especially as I get my setup ready for V3.  Somehow something has corrupted and now none of my connections to P200s connect.  Creating a new connection works.  I’ve tried reinstalling older betas but it seems the “damage” has been done.  The original programmed connection never connect then crash.  Log shows E22 connection refused.
Similar to the issue when the module was first updated.

When you say "unable to connect", is that only with Companion? Are you still able to connect to the P200 web ui or via some other application?

Yes still able to talk to the cams via another control software, cam control, the dashboard, etc.  the companion module just tries to load then crashes.  On another PC loaded with companion and the config imported I am able to add a new instance and it connects ok.Thanks!Sent from my iPhoneOn May 15, 2023, at 20:39, bryce @.***> wrote:
When you say "unable to connect", is that only with Companion? Are you still able to connect to the P200 web ui or via some other application?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Hey @bryce-seifert.  I think I may have found a lead for this issue.  I finally noticed that some of my instance names previously had a space in the cam ie CAM 2 vs CAM2.  When I go to import now they default to CAM_2 with the underscore without any interaction from me.  Wondering if it is something with the instance naming in the install scripts?
Thanks!

Intersting. The instance rename is happening at the Companion level, not within the module itself. But that being said, I don't think this should affect the connection. Can you send the support bundle via slack, after the module crash occurs?

@obcocav The latest companion beta should have the fix for the upgrade error. Let me know if this resolves the issue

@bryce-seifert.
Unfortunately like I mentioned I think the problem from earlier in the beta cycle is back.  I am unable to even load the GUI in the last couple of builds.  It takes 15 min or so the first time I launch companion for the “syncing” screen to appear but it never finishes loading even after a restart.  One time the GUI did pop up but all modules were listed as “unknown” status and companion crashed.  Since then I get to the sync page and it never finishes.  @Julusian @krocheck

Looks like 5889 is the last usable build I can get to open

could you send me your db file<EMAIL_ADDRESS>

Looks like its solved with the latest updates!  Thank you!


Remove final TODO, update  name

Removes one final TODO from the docs files and renames the docs project name.

Codecov Report

Merging #230 (d393130) into main (b4841e4) will decrease coverage by 42.71%.
Report is 1 commits behind head on main.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main     #230       +/-   ##
===========================================
- Coverage   80.65%   37.95%   -42.71%
===========================================
  Files          37       35        -2
  Lines        3066     2951      -115
===========================================
- Hits         2473     1120     -1353
- Misses        442     1765     +1323
+ Partials      151       66       -85

Flag
Coverage Δ

integration-test
?

unittests
37.95% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 30 files with indirect coverage changes

Thanks Matt!

Thanks for updating the name.
The centralized make-docs script has no knowledge of Beyla as a project yet. Is the repository keeping the same name or is it also being renamed to be grafana/beyla? If it isn't being renamed, I'll need to update the make-docs script to support the updated name.

I think we are renaming it, let me do that so that there are no issues.

It appears as if I don't have permission to do so, let's wait for @mariomac to come back from vacation as he did the rename last time.

Thanks for confirming. I'll make the changes now and the update-make-docs.yml workflow will open a PR for us here tomorrow :)


[terra-date-time-picker] Does not honor time in maxDateTime (assume minDateTime as well)

Bug Report
Description
The terra-date-time-picker calls onChange as long as the date is within the minDateTime/maxDateTime range, even if the time makes those values exceed the minDateTime/maxDatetime.
Steps to Reproduce
With onChange set to print some text to the console

Set an max date time halfway through some date (D;T) for a maxValue
Pick a valid date time (D-1;T+1), see onChange fire
Change the date to an invalid date (D+1,T+1), onChange does not fire
Change the date to D (D; T+1), so that the time is invalid, onChange fires, we expect it not to

Expected Behavior

onChange is not called when a dateTime outside of min max bounds is entered
onChangeRaw is called when a dateTime outside of min max bounds is entered

@ Mentions
@benbcai

I think there's a related issue with DateTimePicker and initial values.
We have a common workflow where the initial date and the minimum date are both set to now (D;T).
Whenever the component gains focus, the value is immediately cleared.
This seems to be because underneath the DateTimePicker uses the DatePicker, and the DatePicker strips the time off of the selected date before performing it's comparisons to determine if the selected date is valid, but doesn't strip the time off of the minimum and maximum.
So, you end up comparing D;T to D (the minimum), which fails, so it considers the value as invalid and clears it.

I would expect the date picker to either strip time off of both selected date and min/max date time, or neither.

@prmeznar After discussing with our team on how to handle the time aspect in terms of the min/max date/time, it turns out that there are some (edge) cases that are difficult for terra-date-time-picker to manage and would lead to an undesired user experience. For that reason, we plan to remove the minDateTime and maxDateTime props in the next major version. What we recommend consumers to do now and going forward is to use form validation to perform validation on the date/time value and show an error or alert when appropriate.
We still plan to keep the minDate and maxDate props in terra-date-picker. There is still a small change that we will make in the isDateOutOfRange function shown in your above screenshot to make it work as expected when only either the minDate or maxDate (but not both) is passed in.

After further discussions with the consumers and terra team, it was decided to not completely remove the minDateTime and maxDateTime props but instead rename the minDateTime and maxDateTime props to minDate and maxDate respectively. If the time portion is included in these props, the time portion would not be taken into consideration when determining the minimum and maximum time. The minDate and maxDate props would straight up be passed down from the date-time-picker to the date-picker and would let the date-picker handle these props. This decision would keep the consistency of having these two props in both the date-picker and date-time-picker and also stays consistent with the includeDates and excludeDates where the time portion is not taken into consideration and would just be passed down directly to the date-picker to reflect the disabled/enabled dates in the calendar picker.


Unpublish withdrawn content items with state disparity

Trello ticket: https://trello.com/c/aoTVNMQz/741-withdrawn-content-items-should-be-labelled-as-such-in-the-publishing-api-medium
This branch contains a data migration to unpublish several editions from Whitehall that are in the "withdrawn" state, but in the "draft" or "published" state in the Publishing API.
It also bumps the dependency on gds-api-adapters as this is needed to unpublish drafts.

@steventux kindly looked at this, and discussed the missing unpublishings with me, with the concern that the explanation will appear on the frontend, so the name of the migration file might not be appropriate.
In my local database, there is only one item with a missing unpublishing 5fee02a4-7631-11e4-a3cb-005056011aef. Looking at the corresponding document on the frontend crashes the frontend [1].
1: https://www.gov.uk/government/publications/badgers-report-action-taken-under-licences-a01-and-a26

Thanks to Liz, the document mentioned above with the missing unpublishing has now been fixed, so I have removed the code to cope with this.
I have also rebased this branch, and fixed the conflicts.
@steventux, could you take another look?

This looks ok, I'd be tempted to read the content id data from file(s) as the migration is obscured by this.

👍 . I had similar misgivings about the content ids being in this file, but at least it's encapsulated and you don't have logic to read in a file within a migration.


add test for search with seq_no_primary_term

Description
Add test spec for search with the seq_no_primary_term param as suggested by @dblock here: https://github.com/opensearch-project/opensearch-go/pull/574#issuecomment-2195683900
Issues Resolved
N/A
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Thanks! Would love more test coverage around _search schema if you have time.

Looks like GHA was delayed so the tests didn't run on this PR. Will monitor.
https://www.githubstatus.com/


Adds CorpusBLEU metric

Adds the CorpusBLEU metric based on Sylvains implementation in the fastai NLP course BLEU notebook

Added missing self.get_grams

Thanks!


Add jupyter book bibliography

Add bibliography section in introduction and updated content in jupyter book.

Looks good, thanks Arijeet!


load_s_colors source

I'm trying to make a few "circles" that are centered at a point.
I thought it would be interesting to rewrite the PointShader to accept:

A face color  (likely set to transparent)
An edge color
An edge width

However, I somewhat got stuck reading the PointShader looking at load_s_colors and load_s_texcoords.
It didn't seem that these were defined anywhere obvious to me.
Any tips on where to look?

We actually have (short-term) plans to add support for more marker shapes (square, star, etc.), and also include edge_color and edge_width. Likely via a new PointMarkerMaterial. You are very welcome to make a start tho! :)

However, I somewhat got stuck reading the PointShader looking at load_s_colors and load_s_texcoords.
It didn't seem that these were defined anywhere obvious to me.

Buffer bindings automatically have a loader function created for it. This is mostly to handle limitations with vec3. So if you define the binding, you can make use of a load_<name of binding>().
https://github.com/pygfx/pygfx/blob/c5c588eecb228134aeaba62f9c1b96c46691adab/pygfx/renderers/wgpu/shader/base1.py#L506-L522

alright, thanks for the pointer. I will be trying to have a look, even just to learn about the differences between frag and vertex shaders more.
The new points features are really cool! My local branch was 1 week old which really made a difference!!!
You can probably update the example to show off some of the features:
diff --git a/examples/feature_demo/image_plus_points.py b/examples/feature_demo/image_plus_points.py
index ef84a6c..20f97e0 100644
--- a/examples/feature_demo/image_plus_points.py
+++ b/examples/feature_demo/image_plus_points.py
@@ -9,7 +9,9 @@ Show an image with points overlaid.
 # sphinx_gallery_pygfx_test = 'run'

 import imageio.v3 as iio
+import numpy as np
 from wgpu.gui.auto import WgpuCanvas, run
+
 import pygfx as gfx

 canvas = WgpuCanvas()
@@ -30,10 +32,19 @@ scene.add(image)

 xx = [182, 180, 161, 153, 191, 237, 293, 300, 272, 267, 254]
 yy = [145, 131, 112, 59, 29, 14, 48, 91, 136, 137, 172]
+sizes = np.arange(1, len(xx) + 1, dtype='float32')

 points = gfx.Points(
-    gfx.Geometry(positions=[(x, y, 1) for x, y in zip(xx, yy)]),
-    gfx.PointsMaterial(color=(0, 1, 1, 1), size=10),
+    gfx.Geometry(
+        positions=[(x, y, 1) for x, y in zip(xx, yy)],
+        sizes=sizes
+    ),
+    gfx.PointsMaterial(
+        color=(0, 1, 1, 1),
+        size=10,
+        size_space="world",
+        size_mode="vertex",
+    ),
 )
 scene.add(points)

Would you mind opening a PR to improve that example?

yes can do.


Hospitalizations status and information

Hi there, awesome work! Especially considering that MOH does not share anywhere scrape-able on the web this data..
Are you open to add the usual information attached to the MOH publications regarding the hospitalized cases? To be more specific, I mean in addition to number of cases, add the number of hospitalized cases, number of ICU cases (these are cases that are classified as "hard") and number of deaths per date? Also, number of recovered cases. If you're open, I can probably provide a PR with past data so you only need to track new data.
The rationale behind this is that this data can help to track the occupancy of the health care system in response to COVID-19.
Many thanks!

Hey @MaozGelbart, thanks for your feedback!
We just added this graph here


Remove duplicate proxy_redirect directive

The config check run by nginx will fail because this directive is stated twice

Good catch, thanks for this


fix(ipfs-coord): Updating to v7.1.6

:tada: This PR is included in version 1.10.17 :tada:
Your semantic-release bot :package::rocket:


New blog post for the HTTP Add on v0.1.0 release

The KEDA HTTP Add-on release is still pending. I'll convert this out of a draft PR when the release is done
cc/ @tomkerkhove @zroubalik @jeffhollan

Let me know when you are ready!

This will require https://github.com/kedacore/charts/pull/163 to be merged first


_WD_SetElementValue - $_WD_OPTION_Advanced - $sValue is not escaped

Bug report
Describe the bug
_WD_SetElementValue() in $_WD_OPTION_Advanced mode  does not use any StringEscape action on $sValue
https://github.com/Danp2/au3WebDriver/blob/f00d0d6a62f101fb2e8ff184750bdb05baefcfa2/wd_helper.au3#L2617-L2624
How to reproduce
check with: wd_demo.au3: UserFile :
Using file: UserTesting_EscapeString.au3 with following content:
ConsoleWrite("! Start: UserTesting_EscapeString.au3" & @CRLF)

ConsoleWrite("- Test 1:" & @CRLF)
_WD_Navigate($sSession, 'https://www.google.com')
_WD_LoadWait($sSession)
MsgBox($MB_OK + $MB_TOPMOST + $MB_ICONWARNING, "Warning #" & @ScriptLineNumber, "If you see COOKIE accept panel close them before you will continue.")

ConsoleWrite("- Test 2:" & @CRLF)
; REMARK:  __SetVAR($IDX_VAR, $value) will set  $_VAR[$IDX_VAR] = $value
__SetVAR(0, _WD_FindElement($sSession, $_WD_LOCATOR_ByXPath, "//textarea[@name='q']"))

ConsoleWrite("- Test 3:" & @CRLF)
__SetVAR(1, ' " ! @ # $ % ^ & * ( ) - _ + = { } [ ] ; : \ | , . < > / ? ~ ' & " ' " & ' @CR' & @CR & ' @CRLF' & @CRLF & ' @LF' & @LF & ' @TAB' & @TAB & 'END')
;~ _WD_ElementAction($sSession, $_VAR[0], "VALUE", $_VAR[1])
_WD_SetElementValue($sSession, $_VAR[0], $_VAR[1], $_WD_OPTION_Advanced)

ConsoleWrite("! $_VAR[1] -" & @CRLF & $_VAR[1] & @CRLF)

ConsoleWrite("! __WD_EscapeString($_VAR[0]) -" & @CRLF & __WD_EscapeString($_VAR[1]) & @CRLF)
ConsoleWrite("! END: UserTesting_EscapeString.au3" & @CRLF)

results:
__WD_Post ==> Invalid argument [5] : HTTP status = 400
_WD_SetElementValue ==> Invalid argument [5] : Parameters:    Element=ca424034-1f02-4f2a-9af1-b5fee062a7b2    Value=<masked>    Style=1

! $_VAR[1] -
 " ! @ # $ % ^ & * ( ) - _ + = { } [ ] ; : \ | , . < > / ? ~  '  @CR
 @CRLF
 @LF
 @TAB	END
! __WD_EscapeString($_VAR[0]) -
 \" ! @ # $ % ^ & * ( ) - _ + = { } [ ] ; : \\ | , . < > / ? ~  '  @CR
 @CRLF
 @LF
 @TAB	END
! END: UserTesting_EscapeString.au3
+ wd_demo.au3: Finished: UserFile

Expected behavior
String should be escaped the same way as $_WD_OPTION_Standard (internally in _WD_ElementAction)
Screenshots
none
Additional context
none
System under test
not related

REPRO SCIRPT updated, and require: https://github.com/Danp2/au3WebDriver/pull/464

REPRO SCRIPT updated, and require: https://github.com/Danp2/au3WebDriver/pull/464

Please don't do this in the future. You could have easily reproduced this without requiring another PR.

Taking a look at the two options for _WD_SetElementValue  --

$_WD_OPTION_Standard calls _WD_ElementAction, which formats the string with __WD_EscapeString
$_WD_OPTION_Advanced calls _WD_ExecuteScript, which formats the script with __WD_EscapeString, but not the arguments

I wonder if there would be any undesirable effects if we updated _WD_ExecuteScript to format the arguments. 🤔

Do you plan to resolve this ISSUE with me ... before you release new version ?
Or release will not necessarily be related with this issue.

As you can see from #468, my plan is to release the update tomorrow. Currently, we haven't decided on the best method to address this issue. Any fix most likely won't make the next release because --

this issue isn't critical
the fix could produce unwanted side effects

I wonder if there would be any undesirable effects if we updated _WD_ExecuteScript to format the arguments.

Based upon some basic tests, I can conclude that it would not be safe to format the arguments of _WD_ExecuteScript.
without encoding
    __WD_Post: URL=HTTP://<IP_ADDRESS>:4444/session/6d16a944-058c-40df-bac9-5951fb54fecc/execute/sync; Data={"script":"return arguments[0].second;", "args":[{"first": "1st", "second": "2nd", "third": "3rd"}]}
    __WD_Post ==> Success [0] : HTTP status = 200 ResponseText={"value":"2nd"}
    _WD_ExecuteScript ==> Success [0]

with encoding
    __WD_Post: URL=HTTP://<IP_ADDRESS>:4444/session/30b6951c-dc25-43c6-85ba-9c7d19d7e381/execute/sync; Data={"script":"return arguments[0].second;", "args":[{\"first\": \"1st\", \"second\": \"2nd\", \"third\": \"3rd\"}]}
    __WD_Post ==> Invalid argument [5] : HTTP status = 400 ResponseText={"value":{"error":"invalid argument","message":"Failed to decode request as JSON: {\"script\":\"return arguments[0].second;\", \"args\":[{\\\"first\\\": \\\"1st\\\", \\\"second\\\": \\\"2nd\\\", \\\"third\\\": \\\"3rd\\\"}]}","stacktrace":"Syntax error at :1:51"}}
    _WD_ExecuteScript ==> Invalid argument [5] : Failed to decode request as JSON: {"script":"return arguments[0].second;", "args":[{\"first\": \"1st\", \"second\": \"2nd\", \"third\": \"3rd\"}]}

I believe the existing functionality works as-is, and formatting of data, if required, should occur prior to calling _WD_SetElementValue. My recommendation is to add an entry in the remarks section of the header to document the situation.


Update api-catalog-cli version from 1.1.0 to 1.1.2 and grep search for success message so we can return error otherwise

What happened 👀

Updated version (didn't notice any difference)
grep search for success message, if doesn't exist then exit 1 (return error)

https://docs.mulesoft.com/release-notes/api-catalog/api-catalog-release-notes

Insight 📝
N/A
Proof Of Work 📹
Tested on:

https://github.com/Jollibee-Foods-Corporation/jfc-global-exp-api-doc/pull/355

Failure (path incorrect):
https://github.com/Jollibee-Foods-Corporation/jfc-global-exp-api-doc/actions/runs/10952458848/job/30411172060

Success:
https://github.com/Jollibee-Foods-Corporation/jfc-global-exp-api-doc/actions/runs/10952513309/job/30411320471

No JIRA task created or referenced, just POW of some error checking here.

Thanks @liamstevens111 - I will brings this new version to app docs repo - will let you know the PRs list soon cc @toby-thanathip


htmlOffsetAdjust and matchText enabled gets StringIndexOutOfBoundsException

Hi,
I am getting string index out of bound error if i have htmlOffsetAdjust and matchText enabled, below stack trace
null:java.lang.StringIndexOutOfBoundsException: String index out of range: -1
	at java.lang.String.substring(String.java:1967)
	at org.opensextant.solrtexttagger.TaggerRequestHandler$1.tagCallback(TaggerRequestHandler.java:193)
	at org.opensextant.solrtexttagger.Tagger.advanceTagsAndProcessClusterIfDone(Tagger.java:210)
	at org.opensextant.solrtexttagger.Tagger.process(Tagger.java:152)

Testcase
buildNames("start end");
assertXmlTag("start <td/> end", true);

i have tried to correct HtmlOffsetCorrector but could not get through, finally i had to change TaggerRequestHandler.handleRequestBody Line 186
from
endOffset = offsetPair[1];
to
endOffset = offsetPair[1]<0?inputString.length():offsetPair[1];
can there be better fix for this?

Thanks for reporting... I didn't get a chance to look today but will hopefully look closer tomorrow (Friday).

Thanks, i guess ignoring wrong offset correction might also one solution
              if(offsetPair[0]>=0 && offsetPair[1]>=0 ) {
            	  startOffset = offsetPair[0];
              	  endOffset = offsetPair[1];
              }

I'd rather not put band-aids on the consumption end; I'd rather have reasonable offsets at the source end.  Thus -- fix the bug.  Thanks for sample data... I ran it through a debugger and it appears we need to call startTag.isSyntacticalEmptyElementTag().  It's not completely clear if testing this actually requires more than one tag; if not you can leave assertXmlTag alone.  Can you work on a fix with test?

Thanks applied fix. change to asertXmlTag is not required, i was trying to add testcase according to exact scenario i had encountered but this fix does not need it.
updated pull request.

is there any reason, html offset corrector have not been migrated to apache solr? I was planning to move everything solr 7.4 so trying to find if it has any missing feature.

I wasn't quite comfortable with adding yet another dependency for an optional feature of a fringe feature of everyone using Solr.  And I have doubts about the overall approach... it works, isn't bad but it'd be really nice if (somehow) the HtmlStripCharFilterFactory (or derivative) analysis component handled this sort of thing instead of the tagger itself needing to have such a feature, which feels a bit bolted on.  I don't have the bandwidth to tackle redoing it though.
https://github.com/OpenSextant/SolrTextTagger/issues/28#issuecomment-385785415

BTW a TODO in this project is to extend the hander shipping in Solr 7.4 to provide the htmlOffsetAdjust option.  I had this in mind when I ported it to Solr so that this feature could be added separately.  Perhaps that would be a separate branch which only provides that feature, tests, that, and does nothing else.


connectionCache

The checkIfInstanceIsUp function in proxy.js uses req.cleanedTeamname to check for the team in the cache. The updateLastConnectTimestamp function uses req.teamname to add the team to the cache. I added some logging and noticed that the checkIfInstanceIsUp function never finds the team in the cache and always calls getJuiceShopInstanceForTeamname(teamname) which adds quite a bit of time to every request. updateLastConnectTimestamp seems to add t-teamname and checkIfInstanceIsUp checks just for teamname. Both functions should use the same version of the teamname for the cache.

Oh, this sounds like a really good catch 👍
Do you mind creating a PR with the fix? Would love to take a look at it

Closed via #72


[Feature Request]: Algorithmic Trading Strategy Optimization Using Genetic Algorithms

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
Steps
Data Collection
Collect historical price data for assets to be traded.
Initialize Population
Create an initial population of random trading strategies.
Fitness Function
Define a fitness function to evaluate the performance of trading strategies based on profits and risk.
Selection, Crossover, Mutation
Implement genetic algorithm operations.
Iterate and Evaluate
Iterate through generations to find the best trading strategy.
Use Case
Develop and optimize an algorithmic trading strategy using genetic algorithms to maximize trading profits.
Benefits
No response
Priority
High
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

pls assign it me

you already recieved max point in this repo

@sanjay-kv sir i'm creating issue on Stackoverflow-Analysis repo


feedback for go.ps1?

For kicks, I tried running go.ps1. I'm actually of those people that care about 110_xp, etc - so I have everything on the machine. However, there is no feedback and it is hard to tell what is going on.

never mind, i just had to wait a while


WIP: Set default values for spec constants with SpecId

-1 Some bugs, missing tests, and some style.

This CL depends on #400 .

Rebased as #400 has been merged.
This CL also depends on #408.

rebased to master as #408 is merged.

PTAL @dneto0 @antiagainst

All comments above have been resolved in 5f0626b

+1

-1  Need a test case to show that you've parsed "-42" correctly as a signed int.


sample_encode encode h265 with intra refresh failed

I try to use sample_encode to encode h265 with intra refresh, and it failed.
platform GEN 9
MediaSDK 77f673853c23a91241

command line:
sample_encode h265 -i 1920x1080.yuv -o sample_output.265 -ir_type 1 -ir_cycle_size 4 -ir_qp_delta -20 -ir_cycle_dist 20 -w 1920 -h 1080
error:
ERROR: Hardware device was lost or returned an unexpected error. Recovering...

[ERROR], sts=MFX_ERR_DEVICE_FAILED(-17), SynchronizeFirstTask, SyncOperation failed at /home/wenbin/software/media/src/MediaSDK/samples/sample_encode/src/pipeline_encode.cpp:168

[ERROR], sts=MFX_ERR_DEVICE_FAILED(-17), GetFreeTask, m_TaskPool.SynchronizeFirstTask failed at /home/wenbin/software/media/src/MediaSDK/samples/sample_encode/src/pipeline_encode.cpp:2193

[ERROR], sts=MFX_ERR_DEVICE_FAILED(-17), Run, m_pmfxENC->EncodeFrameAsync failed at /home/wenbin/software/media/src/MediaSDK/samples/sample_encode/src/pipeline_encode.cpp:2497

if i change ir_qp_delta to 0:
sample_encode h265 -i 1920x1080.yuv -o sample_output.265 -ir_type 1 -ir_cycle_size 4 -ir_qp_delta 0 -ir_cycle_dist 20 -w 1920 -h 1080
error:
sample_encode:
/home/wenbin/software/media/src/MediaSDK/_studio/mfx_lib/encode_hw/hevc/agnostic/base/hevcehw_base_legacy_defaults.cpp:1258: static std::tuple<unsigned char, unsigned char> GetDefault::RPL(MfxFeatureBlocks::CallChain<std::tuple<unsigned char, unsigned char>, const HEVCEHW::Base::Defaults::Param&, const HEVCEHW::Base::_DpbFrame (&)[15], short unsigned int, short unsigned int, const HEVCEHW::Base::FrameBaseInfo&, unsigned char (&)[2][15]>::TExt, const HEVCEHW::Base::Defaults::Param&, const HEVCEHW::Base::_DpbFrame (&)[15], mfxU16, mfxU16, const HEVCEHW::Base::FrameBaseInfo&, mfxU8 (&)[2][15]): Assertion `!"L0 is empty - reordering issue?"' failed.
Aborted (core dumped)

@dmitryermilov @daleksan any update about this issue?

@SofyaBalandina , @DenWolf , please share an update.

@yskuridi please take a look from HEVCe side

Hi @xhaihao ,
I found out that the device fail error is a driver issue. Investigation from driver side is required.
Regarding the issue with empty L0, the investigation is in progress.

@yskuridi Thanks for the update.

Hi @xhaihao
There are 2 issues.
The first relates to gpu hang. Investigation from media driver is required.
The second error occurred due to default GopRefDist (=4) and it leads to encoding with B-frames. But IntraRefresh feature doesn't work with B-frames. Check for GopRefDist with Intra Refresh will be added from msdk side. In order to set GopRefDist=1 sample_encode have -r 1 option.
Thanks,
Yuri.

@yskuridi thanks for the update, should we file a driver issue for the GPU hang?

@xhaihao Indeed, I've created one https://github.com/intel/media-driver/issues/1179.


Unit Converter should recognise metre

Microsoft PowerToys version
0.56.1
Utility with translation issue
PowerToys Run
🌐 Language affected
All
❌ Actual phrase(s)
meter
✔ Expected phrase(s)
meter or metre
ℹ Why is the current translation wrong
In most places that the word metre is used (i.e., outside the USA), it is spelt metre, not meter
Refernce: https://en.wikipedia.org/wiki/Metre
Please allow both metre and meter

Good point.

Metre is the standard spelling of the metric unit for length in nearly all English-speaking nations except the United States[2][3][4][5] and the Philippines,[6] which use meter. Other West Germanic languages, such as German and Dutch, and North Germanic languages, such as Danish, Norwegian, and Swedish,[7] likewise spell the word Meter or meter.

If you use the SI "m" it works. Shouldn't it be like that to all conversions? Or am I missing something?

If you use the SI "m" it works. Shouldn't it be like that to all conversions? Or am I missing something?

Yyyyyeah.....I kinda get what your saying. And if I had a few more brain cells, I'd use m instead of metres.
 There's no rhyme or rhythm to the ways I use PowerToys, for example, I frequently use shortcuts such as < for running application search. PowerToys should adapt to all of us, an international audience, instead of us adapting to Mr Franklin's spelling changes to the English language. Don't get me wrong, I read Ben Franklin's abridged biography as a young man, and I'm a big fan...but...sigh, why oh why Mr Franklin???

Ahahah ok fair enough. It's probably my engineer brain talking. And I use / for the apps so I get it.

Not really a translation issue per-say but should be adding in an if statement.

note, this is a famous last words since i made this bet without looking at the code.
But on quick look, UnitsNet is what actual does the to and from so we just need to do a search for metre and replace it with meter.
Right around here is where i bet we'd do it:
https://github.com/microsoft/PowerToys/blob/fa3a5f80a113568155d9c2dbbcea8af16e15afa1/src/modules/launcher/Plugins/Community.PowerToys.Run.Plugin.UnitConverter/InputInterpreter.cs#L158

so we just need to do a search for metre and replace it with meter.

I think you may be mistaken, that is not solving the issue. Need to support both metre and meter.

What does that mean. Meter and metre are the same thing, no?  If you type in 1 metre to feet, 3.28084 feet would be the output, no?

@crutkas

Meter and metre are the same thing, no?
Yes. Right now Run only understands meter, not metre. When I type in metre, which is how 95% of the world's population spells it, then the converter doesn't know what to do with it.

When I type in meter, which is how 5% of the world's population spells it (i.e. USA, thanks to Mr Franklin's Americanization [yes] of the English language), then the converter understands it

@zaakiy we're saying the same thing :) IPowerToys is built by a multi-national group of individuals and community members across the global.
If you type in metre, we swap it under the hood to work as meter.  Why: cause the nuget package we use looks for the phrase "meter".
I wonder if this may have exposed a larger issue here past spelling.  a US cup is different than what a metric cup is which is different than ....https://en.wikipedia.org/wiki/Cup_(unit)

@crutkas

we swap it under the hood to work as meter

I think you are under the impression that this is already happening.
It doesn't - not right now.
Thanks @jaimecbernardo for https://github.com/lncubus/PowerToys/commit/74f18b95112ca8f7f47de547bd4c7b1f1d94b5ad.


feat: support foreign keys

Changes included in this PR:

main.go - print foreign key constraints in schema DDL file but exclude foreign keys from the schema sent to spanner. (This is being handled using config)

toddl.go - convert foreign keys from (schema.Fkey) to Spanner(ddl. Foreignkey) using cvtForeignKeys() function.

mysqldump.go - handle foreign constraints as well as foreign key column constraints.

infoschema.go - Using INFORMATION_SCHEMA.REFERENTIAL_CONSTRAINTS. Added getForeignKeys() to handle foreign keys.
1. Get all the referenced tables for a particular table using getRefTables()
2. Get all the referenced columns for a particular referenced table using getForeignKey(). Then convert it to foreign key(schema.Fkey).

pgdump.go, postgres/infoschema.go - Similar approach as MySQL.

This PR is still working in progress

MySQL

mysqldump

[x] Handle foreign key constraints using toForeignKeys function.
[x] Handle column foreign key constraint.
[ ]  Support foreign key for ALTER TABLE statements

information_schema

[x] Handle foreign key constraints

PostgreSQL

pg_dump

[x] Handle foreign key constraints using toForeignKeys function.
[x] Handle column foreign key constraint.
[x]  Support foreign key for ALTER TABLE statements

information_schema

[x] Handle foreign key constraints

[ ] Refactor the test cases for foreign keys and add new unit tests.
[ ] Update README

Had a quick look. Approach 1 looks good to me.

@nevinheintze @hengfengli Added foreign key support for postgres and refactored code as per you comments. I still need to update the tests (they are failing now) and also update the readme.
I would request you to please go ahead and review the PR. Meanwhile, I will also update some of the unit test cases.

Changes included in this PR:

main.go - print foreign key constraints in schema DDL file but exclude foreign keys from the schema sent to spanner. (This is being handled using config)
toddl.go - convert foreign keys from (schema.Fkey) to Spanner(ddl. Foreignkey) using cvtForeignKeys() function.
mysqldump.go - handle foreign constraints as well as foreign key column constraints.
infoschema.go - Using INFORMATION_SCHEMA.REFERENTIAL_CONSTRAINTS. Added getForeignKeys() to handle foreign keys.

 Then convert it to foreign key(schema.Fkey).

pgdump.go, postgres/infoschema.go - Similar approach as MySQL.

This PR is still working in progress

MySQL

mysqldump

[x]  Handle foreign key constraints using toForeignKeys function.
[x]  Handle column foreign key constraint.
[ ]   Support foreign key for ALTER TABLE statements

information_schema

[x]  Handle foreign key constraints

PostgreSQL

pg_dump

[x]  Handle foreign key constraints using toForeignKeys function.
[x]  Handle column foreign key constraint.
[x]   Support foreign key for ALTER TABLE statements

information_schema

[x]  Handle foreign key constraints

[ ]  Refactor the test cases for foreign keys and add new unit tests.
[ ]  Update README


RoutingPlugin: restore  lost feature (#107939)

When we have waypoints for a route but the tool should not be active, we deactivate the tool after the route is successfully fetched

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

5 of 5 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 99.694%

Totals

Change from base Build<PHONE_NUMBER>2:
0.0%

Covered Lines:
13447

Relevant Lines:
13448

💛  - Coveralls


fix: Support extended chars in instance names

Pick up go-mssqldb v1.5.0 to get the fix

Fixes #405


Thanks for figuring this out

I'm incorporating your values into my weak aura that I use with my friends to figure out which dungeons will collectively increase our rating gain the most. https://wago.io/EkUYyjIUI
Thanks for providing a data source that lets me update this. I'm sure I'll need to update it again in the future as you figure out higher key levels. It would be great to figure out what the pattern is behind the values for higher keys. I'm sure we'll get there over this expansion :)
(You can close this issue since I'm not reporting a bug. I just wanted to share my appreciation)

thank you for the kinds words. I'm glad I could help, and thank you for mentioning me on the wago page, I shall import that WA and give it a try.
as for higher (than 12) keys, see if this helps:
So from what I can gather to calculate the base score of a key
the formula is
125 + (5 if the key level >=7)  + (15 * key level) + (10 * number of affixes)
so for example if the key level is 2 (which means it has 1 affix)
then the base score for it is 125 + (15 * 2) + (10 * 1) = 165
and if the key level is 12 (so 4 affixes) then base score = 125 + 5 + (15 * 12)  + (10 * 4)  = 350
so you can use this formula to calculate base score of any level dungoen  and then use the remaining formulas displayed on the main page of mythicplanner.com to get actual score from completion time.
keep in mind I am not 100% sure about this formula, it made sense with data on raider.io at the time (when I figured this out) the highest key on raider.io was an 11 so I couldn't confirm it. I bet now people have much higher keys so I will confirm that formula when I get the chance. once confirmed or adjusted I will update the main page explanation to include this.
I hope this helps.

Oh awesome. That looks in the realm of possibility based on what the math was last expansion.
For my WA, luckily I don't need min and max scores, just the base.
Do we know if doing the dungeons on any other affix weeks will change the score? Right now I just have it calculating the max score of any affix combo you have for that dungeon.

Min and max are easy to figure out anyways,
Max = base score + 15
Min = base score - 30 (well technically min is 0, but you get some score if you were still within 40% over the timer. the moment you miss the timer you instantly lose 15 points from the base, and then for the next 40% of the total time you lose some until it adds up to another 15 (totaling to the - 30 mentioned above) then 1 millisecond beyond that you get 0 points.
As for the weekly affixes and scoring, as far as I know, they have removed the duel score system, no more tyrannical week score and fortified week score.
This article talks about that a bit https://www.wowhead.com/news/keystone-master-much-easier-in-war-within-new-score-system-basics-346725
the only affixes that rotates weekly are the level 2 one (the Xal'atath's Bargain one) and the fort /tyr you get at level 4 alternating weekly as the used to, but at level 10 or higher you get both of them at the same time anyway. which I guess why they removed the dual scoring system.

looking at runs on raider.io I noticed that keys LVL 12 or higher seem to get an additional 15 points making the formula
125 + (5 if the key level >=7)  + (15 if the key level >=12) + (15 * key level) + (10 * number of affixes)
I'm not really convinced that's the actual formula now, seems like there is probably something more robust on calculating it. because who knows maybe there more additional points when the key is higher than her another key level. I will keep an eye but in all cases I updated the table in the main mythic planner website

For what it's worth, the score calculation from dragonflight was
if rg.keyLevel < 5 then
        rg.newScore = (rg.keyLevel+10)*5 + 30 + (rg.keyLevel)*2
    elseif rg.keyLevel < 10 then
        rg.newScore = (rg.keyLevel+10)*5 + 40 + (rg.keyLevel)*2
    else
        rg.newScore = (rg.keyLevel+10)*5 + 50 + (rg.keyLevel)*2
    end

based on the code from this addon: https://www.curseforge.com/wow/addons/mythic-plus-rating-gain
which I had in my previous version of the weakaura: https://wago.io/EkUYyjIUI/1.0.12
so I wouldn't write off what you came up with as needing something "more robust".


Please help me understand

Do I need a domain purchased for atlantis to be hosted on ?
I am using the COMPLETE EXAMPLE code to do this.
It gives me error about domain and managed_zone.
how can I open Atlantis UI?
Thanks in advance...

OK....got a new domain,
added the load balancer url as A record.
still getting - <IP_ADDRESS> uses an unsupported protocol.
ERR_SSL_VERSION_OR_CIPHER_MISMATCH
error.
Please help

Hi @Searce-Akshat, you indeed need a domain. Regarding the SSL error, there's a FAQ in the docs. You need to register an A record to validate the SSL certificate. This is also shown in the example code. Try to redeploy the module, or simply recreate the SSL certificate after registering the load balancer's IP address as a A record in the managed DNS zone.

Thanks @bschaatsbergen

My atlantis is now hosted on the domain.

Ive created the github webhook and give the hosted url with /events in the end.
still my atlantis is not receiving the alerts and not responding to "atlantis plan".
Please help

@Searce-Akshat I would recommend to carefully read through the Atlantis documentation to properly deploy a working Atlantis instance. This repository simply helps in deploying Atlantis to GCE :)!

Thanks @bschaatsbergen .
It is working now...you can close the issue


Going through the rust dev tools contributing checklist

Filling in some obvious gaps from https://github.com/nrc/dev-tools-team/blob/master/contribution/checklist.md
r? @emilio
cc @rust-lang/dev-tools : would anyone like to review our CONTRIBUTING.md file in general? (and our contribution workflow?) (and probably outside of this PR)

Huh, I can't cc @rust-lang/dev-tools no matter how hard I try...

Thanks for review!
@bors-servo r=emilio

:pushpin: Commit 4bea594 has been approved by emilio

:hourglass: Testing commit 4bea5946a9f059ce1066b7a624c6248b69bae9f4 with merge 57468b2affd432f4da0733d6b6d0246373b07b73...

:sunny: Test successful - status-travis
Approved by: emilio
Pushing 57468b2affd432f4da0733d6b6d0246373b07b73 to master...


W/PushNotificationsAPI( 8126): Failed to register device: NOKResponse(error=Something went wrong, description=Device could not be created)

Hi,
Having some trouble getting pusher beams to work on Android Emulator in flutter for a new app.  I assume it should work fine on the emulator?
Error messages in log:
Connecting to VM Service at ws://<IP_ADDRESS>:54264/eRpYliPUyG0=/ws
[5] W/PushNotificationsAPI( 8422): Failed to register device: NOKResponse(error=Something went wrong, description=Device could not be created)
D/com.pusher.pusher_beams.PusherBeamsPlugin@ba63f03( 8422): PusherBeams started with <instance-id> instanceId
D/com.pusher.pusher_beams.PusherBeamsPlugin@ba63f03( 8422): Returning initial data: null

Not sure if this is related, but when it repeats the message it also includes these lines above the NOK Response line - which all have denied access:
D/TrafficStats( 8997): tagSocket(84) with statsTag=0xffffffff, statsUid=-1
W/p.app.mobile( 8997): Accessing hidden method Lcom/android/org/conscrypt/ConscryptEngineSocket;->setUseSessionTickets(Z)V (max-target-q,core-platform-api, reflection, denied)
W/p.app.mobile( 8997): Accessing hidden method Lcom/android/org/conscrypt/OpenSSLSocketImpl;->setUseSessionTickets(Z)V (max-target-q,core-platform-api, reflection, denied)
W/p.app.mobile( 8997): Accessing hidden method Lcom/android/org/conscrypt/AbstractConscryptSocket;->setUseSessionTickets(Z)V (max-target-q, reflection, denied)
W/p.app.mobile( 8997): Accessing hidden method Lcom/android/org/conscrypt/ConscryptEngineSocket;->setHostname(Ljava/lang/String;)V (max-target-q,core-platform-api, reflection, denied)
W/p.app.mobile( 8997): Accessing hidden method Lcom/android/org/conscrypt/OpenSSLSocketImpl;->setHostname(Ljava/lang/String;)V (max-target-q,core-platform-api, reflection, denied)
W/p.app.mobile( 8997): Accessing hidden method Lcom/android/org/conscrypt/AbstractConscryptSocket;->setHostname(Ljava/lang/String;)V (max-target-q, reflection, denied)
W/p.app.mobile( 8997): Accessing hidden method Lcom/android/org/conscrypt/OpenSSLSocketImpl;->setAlpnProtocols([B)V (max-target-q,core-platform-api, reflection, denied)
W/p.app.mobile( 8997): Accessing hidden method Lcom/android/org/conscrypt/AbstractConscryptSocket;->setAlpnProtocols([B)V (max-target-q, reflection, denied)
W/p.app.mobile( 8997): Accessing hidden method Lcom/android/org/conscrypt/OpenSSLSocketImpl;->getAlpnSelectedProtocol()[B (max-target-q,core-platform-api, reflection, denied)
W/p.app.mobile( 8997): Accessing hidden method Lcom/android/org/conscrypt/AbstractConscryptSocket;->getAlpnSelectedProtocol()[B (max-target-q, reflection, denied)
W/PushNotificationsAPI( 8997): Failed to register device: NOKResponse(error=Something went wrong, description=Device could not be created)

Happens before my PusherBeams.instance.start() call, so maybe a problem setting up the pushnotification libraries correctly?
Anyone else run into this?  Thought I was following the instructions so far, but maybe there's some other incompatibility/initialisation issue.  I'm using the latest plugins. Also tried with the versions listed on the readme page here, same results.
My flutter setup detailed below thanks for any help.

pubspec.yaml: pusher_beams: ^1.1.1

Have my google-services.json file in \android\app\google-services.json

Here's my app build.gradle:

...
dependencies {
    implementation 'com.google.firebase:firebase-messaging:24.0.0'
    implementation 'com.google.firebase:firebase-installations:17.1.0'
    implementation 'com.pusher:push-notifications-android:1.9.2'
}

apply plugin: 'com.google.gms.google-services'

My project build.gradle:

dependencies {
        classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:$kotlin_version"
        classpath 'com.google.gms:google-services:4.4.1'
    }

Trying to run this both through flutter and android studio produces same results.  Manifest allows outgoing connections and verified I can connect to web from the emulator so I don't think it's a connection problem.
<uses-permission android:name="android.permission.INTERNET"/>    <uses-permission android:name="android.permission.ACCESS_NETWORK_STATE" />
<application
        android:usesCleartextTraffic="true"
        android:enableOnBackInvokedCallback="true">
 </application>

flutter doctor -v output:
[✓] Flutter (Channel stable, 3.19.2, on macOS 13.6.4 22G513 darwin-arm64, locale en-NZ)
    • Flutter version 3.19.2 on channel stable at /Development/Projects/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision<PHONE_NUMBER> (3 months ago), 2024-02-27 16:51:22 -0500
    • Engine revision 04817c99c9
    • Dart version 3.3.0
    • DevTools version 2.31.1

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/xxx/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/xxx/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15C500b
    • CocoaPods version 1.15.0

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2023.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)

[✓] VS Code (version 1.89.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.88.0

[✓] Connected device (4 available)
    • sdk gphone64 arm64 (mobile) • emulator-5554             • android-arm64  • Android 14 (API 34) (emulator)

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Any update on this?

Thanks for following up, not quite sure what changed, but finally started building and connecting to the gateway again.  Not sure if the gateway was down, but the error message wasn't particularly helpful, and a bit unclear where the actual error was being thrown.  After switching emulators and rebuilding a bunch of times and giving it lots of time, I finally saw the registering FCM token message and all was good after that.  Sorry I can't be clearer, problems like this always hard to track down.

Apologies for revisiting this topic, but do you have any insights or solutions that worked for you? I’ve tried numerous approaches but still encounter the same error: “Device could not be created.”

@TekYin Sorry, no, and as of about 2-3 weeks ago, I get this now in production and development too - only on android (iphone seems to work fine).  It looks like the pusher libraries for getting/registering the FCM token no longer work or something has gone wrong with pusher registering for remote notifications during the android startup process.
I've had to decide to refactor pusher out of our app and replace it with something else, as notifications are a critical feature for us, and error messages like these with no obvious resolution or cause of the problem just aren't enough for us to rely on for our app.
Good luck, and let me know if you figure out what the problem was.

Managed to get mine working again! I generated a new private key in my firebase project, then uploaded it to my pusher instance, following these instructions.
I'm not 100% sure about whether this is the root cause, but I have a feeling I might have mistakenly uploaded the google.service.json file to Pusher FCM config originally, and not the generated myapp-id-firebase-adminsdk-<key>.json file which includes the firebase private key.  Once I uploaded this new generated private key file in the pusher FCM configuration settings, my app now connects and generates a token successfully!
The docs aren't super clear which file to upload if you've already generated a key already, so give this a try and see if it helps.


replaced wget with requests

wget cannot be used on Windows.

Oops, left some extra code in there, adding another commit.

Are we formally supporting Windows?

I was told some pods don't have access to Google Colab, and got an uninformative error reported by a TA for the loading section of my scripts, so I decided to change them anyway...

I would worry a bit about pulling in large files like this, there are better patterns with requests.
But I see @mrkrause does not share my worries :)

For HCP our workflow depends on tar being available, so people on Windows machines might be out of luck there.

tarfile is still in the stdlib, right?

hm, we don't need/want to read it all into memory though, we need to unpack a fairly complex directory hierarchy.

I think it has an extractall() member function that mimics tar xvf

ok, let's see if anyone complains and i can look into it.


how to use nonblock mbedtls_net_connect

i try to use mbedtls_net_set_nonblock
mbedtls_ssl_read and mbedtls_ssl_write is nonblock
but mbedtls_net_set_nonblock not change
so.how to use nonblock connect?
thank you guys

Hi @DragonQuestHero in order to use a non-blocking socket, you will need to call mbedtls_net_set_nonblock()
I am not sure why you think that mbedtls_ssl_read() and mbedtls_ssl_write() are nonblocking, if you haven't set the socket to be nonblocking.

ok...sorry my english
I use mbedtls_net_set_nonblock() After mbedtls_ssl_read() and mbedtls_ssl_write() will is nonblocking.return MBEDTLS_ERR_SSL_WANT_READ or MBEDTLS_ERR_SSL_WANT_WRITE
but mbedtls_net_set_nonblock() still block.just return YES or ERROR
thanks.

I just want use non-blocking connect
return YES or ERROR or connecting.look like nonblock read() and write()

Hi @DragonQuestHero thank you for your clarifications. Yes, the mbedtls_net_connect() sample code supplied with the Mbed TLS library is using a blocking connect. In order to do a non blocking connect, you will need to have your own network implementation( pretty much a copy of the current net_sockets.c), setting your own bio callbacks. For mbedtls_net_connect(), you will need to create a non blocking socket, and then use connect and select to have the timed connect.
Another option, is to have the default reference implementation of net_sockets.c (assuming your platform supports it), and you will need to create the socket, set it to non blocking, call connect and then select in your own proprietary function


Lilseyi/separate context

Description
Add support for separating the context from the YooptaEditor, this allows users to lift the context up from the editor, and access editor state from other sibling components.
// Still Supported
import { YooptaEditor } from '@yoopta/editor'

// New Alternative
import { YooptaEditorProvider, Editor } from '@yoopta/editor'

Fixes # (issue)
Type of change
Please tick the relevant option.

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[X] I have performed a self-review of my own code
[X] I have commented on my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] I have checked my code and corrected any misspellings

Hey @lilseyi, sorry for delay.
I need to merge this PR before, because PR includes many changes for Core operations and root <YooptaEditor /> component.
So, after the merging, I'll take a look and see the best solution that can be with the new architecture

No worries @Darginec05 , Excited for the new changes, any idea on when your PR will be merged?

No worries @Darginec05 , Excited for the new changes, any idea on when your PR will be merged?

I hope it will be released tomorrow 👀

@Darginec05 I've merged this PR with all the new changes


Remove personal-information and minister disabled forms

Related to OP-2934
Remove forms :

people.person.positions.minister.edit
people.person.personal-information.edit
administrative-units.administrative-unit.ministers.new
people.new-position (keep route and message, but remove form related content)

Remove ember-changeset rules

validations/address.js
validations/contact-point.js
validations/minister.js
validations/secondary-contact-point.js
validations/site.js

Remove components:

contact-details
propagate-contact-details-update

I would remove the, now empty, controllers for the index routes of minister and mandatory. Other than that it looks good to me.

@mirdono You right, these controllers can be removed. But empty function in router are still needed to pass through the /index route. cf. https://guides.emberjs.com/release/routing/defining-your-routes/#toc_index-routes
For example, without the empty function, people.person.positions.minister.index route is not created and we stay on route people.person.positions.minister, then nothing is displayed.
this.route(
  'minister',
  { path: '/bedienaar/:ministerId/' },
  function () {}
);


Drop fundle support

Fundle is too time consuming to maintain and inactive for almost 2 years.
So I'm going to remove tests, documentation for it and do not plan to support related issues.
What do you think?

There's really nothing to support other than make sure you use the proper directories for fish. Outside of that, there is literally nothing to do. Not officially supporting it is still fine as a fundle user.

@nesl247 Thanks for your feedback
The code related to it I found was a test suite but commented. So no reason to pretend we support it when we have no automated way to back this statement.
Moreover, I prefer to focus my energy on fisher support as it's active and widely used in the community.
Note, the file structure to comply with fundle is still there but no guaranty it will work in the future so I prefer to explicitly state there is no official support.

@edouard-lopez Exactly. That's why I meant it's still fine as a fundle user. Fundle is really just a mechanism for downloading/updating/loading, and not really much else. Which is why it doesn't have a lot of activity, it's just not needed.

I manage to fix the failing test  related to fundle install  in #227.
So it's still live for now! :tada:


Password Issue

1.11.2 and quests 2.8.7
This is the error https://pastebin.com/B89WYCDh and this is the quest: https://pastebin.com/bSS9SCWn
The very first time I ran the quest it worked but I needed to reload my server. Tried it again to make sure I could continue using this method and it through the error.

For this error, there was nothing wrong with the teleport event. this was the multiverse command but I ended up adding Beton quests to use their event. The password seemed to have been the problem because I changed the quest to turn in a pass and it works perfectly now.


[Enhancement] PageIndexReader support use MinMaxPredicate to filter page with column stats

Why I'm doing:
What I'm doing:
PageIndexReader use MinMaxPredicate (which is generated by runtime filter) to filter page with column stats
What type of PR is this:

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.4
[ ] 3.3
[ ] 3.2
[ ] 3.1
[ ] 3.0

https://github.com/Mergifyio backport branch-3.4 branch-3.4.0-rc01


[Event Request] page 9657 "Customer Report Selections" - OnElseMapTableUsageValueToPageValue

We would like to add a new event OnElseMapTableUsageValueToPageValue in the following procedure:
local procedure MapTableUsageValueToPageValue()
var
    CustomReportSelection: Record "Custom Report Selection";
begin
    case Usage of
        CustomReportSelection.Usage::"S.Quote":
            Usage2 := Usage2::Quote;
        CustomReportSelection.Usage::"S.Order":
            Usage2 := Usage2::"Confirmation Order";
        CustomReportSelection.Usage::"S.Invoice":
            Usage2 := Usage2::Invoice;
        CustomReportSelection.Usage::"S.Cr.Memo":
            Usage2 := Usage2::"Credit Memo";
        CustomReportSelection.Usage::"C.Statement":
            Usage2 := Usage2::"Customer Statement";
        CustomReportSelection.Usage::JQ:
            Usage2 := Usage2::"Job Quote";
        CustomReportSelection.Usage::Reminder:
            Usage2 := Usage2::Reminder;
        CustomReportSelection.Usage::"S.Shipment":
            Usage2 := Usage2::Shipment;
        else
            OnElseMapTableUsageValueToPageValue(CustomReportSelection, Usage2);  //A108F0075.Event
    end;
end;

Event Signature:
[IntegrationEvent(false, false)]
local procedure OnElseMapTableUsageValueToPageValue(CustomReportSelection: Record "Custom Report Selection"; var ReportUsage: Option)
begin
    //A108F0075.Event
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


[CLOSED] Move librespot into lib.rs and let main.rs be the test binary

 Issue by torarnv
Wednesday Jul 08, 2015 at 18:31 GMT
Originally opened as https://github.com/plietar/librespot/pull/3

I made all the modules public for now.

torarnv included the following code: https://github.com/plietar/librespot/pull/3/commits

 Comment by plietar
Wednesday Jul 08, 2015 at 20:22 GMT

Could you rebase please.

 Comment by plietar
Wednesday Jul 08, 2015 at 20:28 GMT

Does main.rs get built ?
If not you could put it as an example instead: http://doc.crates.io/manifest.html#the-project-layout

 Comment by torarnv
Wednesday Jul 08, 2015 at 20:37 GMT

Rebased. main.rs gets built as it lives in src. If we make it an example it will only build if you do cargo test, not for a regular cargo build, so keeping it in src is probably preferable?

 Comment by plietar
Wednesday Jul 08, 2015 at 20:49 GMT

Hmm, I'm not really sure how cargo behaves when both a main.rs and lib.rs are present. It indeed seem like both actually get compiled.
Could you declare them explicitly, like http://stackoverflow.com/questions/26946646/rust-package-with-both-a-library-and-a-binary
Just call the binary main and the library librespot.
Thanks

 Comment by torarnv
Wednesday Jul 08, 2015 at 21:44 GMT

Can we merge librespot-protocol into librespot now?

 Comment by torarnv
Wednesday Jul 08, 2015 at 20:52 GMT

https://doc.rust-lang.org/stable/book/crates-and-modules.html describes using both lib.rs and main.rs:

Our package now has two crates: src/lib.rs and src/main.rs. This pattern is quite common for executable crates: most functionality is in a library crate, and the executable crate uses that library. This way, other programs can also use the library crate, and it’s also a nice separation of concerns.

But being explicit is probably a good idea. Will fix!

 Comment by plietar
Wednesday Jul 08, 2015 at 21:47 GMT

The reason they are split is that librespot-protocol contains a lot of generated code, and rust is very slow to compile. Merging them increasing the build time by a lot.
Any reasons you want them merged ?

 Comment by torarnv
Wednesday Jul 08, 2015 at 21:54 GMT

Ah, I see.  I started implementing a mock Player for my usecase, and the PlayerState has a PlayStatus that's coming from use librespot_protocol::spirc::PlayStatus;. AFAICT that's the only dependency on the protocol part for now.
I tried adding extern crate librespot_protocol; but that didn't work, I'm guessing it's an internal dependency of librespot the way the Cargo file is set up?

 Comment by torarnv
Wednesday Jul 08, 2015 at 21:55 GMT

Perhaps we can re-export the PlayStatus from spirc.rs?

 Comment by torarnv
Wednesday Jul 08, 2015 at 22:06 GMT

Well, I guess that delegate is needed after all :smile:
src/main.rs:49:57: 49:64 error: mismatched types:
 expected `&librespot::player::Player<'_>`,
    found `&player::Player<'_>`
(expected struct `librespot::player::Player`,
    found struct `player::Player`) [E0308]
src/main.rs:49     let mut spirc_manager = SpircManager::new(&session, &player, username, name);

(Don't worry, I can get away with a cast for now)

 Comment by plietar
Thursday Jul 09, 2015 at 19:06 GMT

Thanks


update status marker

@FBRTMaka updates the circles to markers to get rid of the issues seen

@birdage oh very cool!


BLECentralManager's state publicly available.

This change is also contained in PR #13, but here I tried to make a small, isolated change instead of the single PR combining many features together.

looking good


GCC: __cpp_binary_literals not defined [-Winvalid-pch]

Note: this issue is solved, I just thought I document it here, so other people save searching Dr. Google.
When cotiring our project (https://github.com/OpenMS/OpenMS), this worked really well for VS2015, but failed for GCC, giving a weird error message that the pch is not used:
cc1plus: Warning: .../cotire/..._CXX_prefix.hxx.gch: not used since »__cpp_binary_literals« is not defined [-Winvalid-pch]
When looking for __cpp_binary_literals in GCC7 one finds this to be an intrinsic C14 feature , which should always be there. So one naturally asks: how come its on for PCH but off for normal compile units?
The reason is that somewhere in the G++7 code, it is switched off iff --pedantic is specified as a compile flag.
This is actually the hint that brought the solution: our cmake codebase uses add_definitions(--pedantic) (which cotire does not recognize as compile flag and hence does not use it when building the pch). We should be using add_compile_options(--pedantic) (which fixes the problem).
See also #73 for some background information.
Thus, it's always useful to run make VERBOSE=1 --trace ... and compare the actual commandline arguments for the pch vs any other compilation unit and check for equality.
Not sure if it makes sense to add a sentence of caution to the cotire documentation like:
Do not use add_definitions(...) for compiler flags, since they will not be used during pch generation, which might invalidate the pch. Use add_compile_options(...) instead.

[closing since solved]


Add mybinder link explicity to readme

There is already a badge that takes the user to mybinder, but it would be good to explicitly link to it where the examples are linked.

Already completed


Issue with --with-reference

When trying to build gf18mcu using --with-reference option, I get the following error.
(cd gf180mcu && make -j all)
make[1]: Entering directory '/home/karim/work/pdk/volare/gf180mcu/build/43493de6d35c95f3ed379c41b9b4b47beedb7bd1/open_pdks/gf180mcu'
if test "xgf180mcu.json" != "x"; then \
	FD_PR_COMMIT = `cat gf180mcu.json | grep gf180mcu_fd_pr | grep -v COMMIT | cut -d'"' -f4` ; \
fi
if test "xgf180mcu.json" != "x"; then \
	FD_PV_COMMIT = `cat gf180mcu.json | grep gf180mcu_fd_pv | grep -v COMMIT | cut -d'"' -f4` ; \
fi
if test "xgf180mcu.json" != "x"; then \
	FD_IO_COMMIT = `cat gf180mcu.json | grep gf180mcu_fd_io | grep -v COMMIT | cut -d'"' -f4` ; \
fi
if test "xgf180mcu.json" != "x"; then \
	FD_SC_MCU7T5V0_COMMIT = `cat gf180mcu.json | grep gf180mcu_fd_sc_mcu7t5v0 | grep -v COMMIT | cut -d'"' -f4` ; \
fi
if test "xgf180mcu.json" != "x"; then \
	FD_SC_MCU9T5V0_COMMIT = `cat gf180mcu.json | grep gf180mcu_fd_sc_mcu9t5v0 | grep -v COMMIT | cut -d'"' -f4` ; \
fi
if test "xgf180mcu.json" != "x"; then \
	FD_IP_SRAM_COMMIT = `cat gf180mcu.json | grep gf180mcu_fd_ip_sram | grep -v COMMIT | cut -d'"' -f4` ; \
fi
if test "xgf180mcu.json" != "x"; then \
	OSU_SC_COMMIT = `cat gf180mcu.json | grep gf180mcu_osu_sc_gf12t3v3 | grep -v COMMIT | cut -d'"' -f4` ; \
fi
/bin/bash: line 2: FD_PV_COMMIT: command not found
/bin/bash: line 2: FD_SC_MCU7T5V0_COMMIT: command not found
make[1]: *** [Makefile:509: pv-repo] Error 127
make[1]: *** Waiting for unfinished jobs....
make[1]: *** [Makefile:535: sc-7t-repo] Error 127
/bin/bash: line 2: FD_IP_SRAM_COMMIT: command not found
/bin/bash: line 2: FD_PR_COMMIT: command not found
make[1]: *** [Makefile:561: sram-repo] Error 127
/bin/bash: line 2: FD_SC_MCU9T5V0_COMMIT: command not found
/bin/bash: line 2: FD_IO_COMMIT: command not found
make[1]: *** [Makefile:496: pr-repo] Error 127
/bin/bash: line 2: OSU_SC_COMMIT: command not found
make[1]: *** [Makefile:574: osu-sc-repo] Error 127
make[1]: *** [Makefile:522: io-repo] Error 127
make[1]: *** [Makefile:548: sc-9t-repo] Error 127
make[1]: Leaving directory '/home/karim/work/pdk/volare/gf180mcu/build/43493de6d35c95f3ed379c41b9b4b47beedb7bd1/open_pdks/gf180mcu'
make: *** [Makefile:132: tech-gf180mcu] Error 2

I noticed this section gf180mcu Makefile:
pr-repo:
	if test "x${REFERENCE_JSON}" != "x"; then \
		FD_PR_COMMIT = `cat ${REFERENCE_JSON} | grep gf180mcu_fd_pr | grep -v COMMIT | cut -d'"' -f4` ; \
	fi
	if test "x${GF180MCU_PR_PATH}" != "x" ; then \
		if test -d "${GF180MCU_PR_PATH}" ; then \
			echo "Using existing installation of primitive library from ${GF180MCU_PR_PATH}" ; \
		else \
			echo "Downloading primitive library from ${GF180MCU_PR_PATH}" ; \
			../scripts/download.sh ${PDK_LIB_PR} ${GF180MCU_PR_PATH} ${FD_PR_COMMIT}; \
		fi ; \
	fi

This wouldn't work. FD_PR_COMMIT = <> is illegal sh syntax. It should be FD_PR_COMMIT=<>. The other issue is that ${FD_PR_COMMIT}, in later lines, references the Make variable called FD_PR_COMMIT not the shell variable being created at runtime.

fixed https://github.com/RTimothyEdwards/open_pdks/pull/373


Realtime WebSocket connection fails

Bug report
Describe the bug
All works well other than realtime pg connection. browser throws the following

When I look on AWS ECS on the RealtimeService logs I see a lot of
project=realtime external_id=realtime [error] Auth error: tenant `realtime` not found`

When I look at the realtime.tenants table I only see one entry and the name is realtime-dev so I suspect this is the issue.
If this is the issue, what's a good way to solve this other than editing this field directly in the DB?

https://github.com/supabase/realtime/tree/main#server-setup
realtime-dev may be the default tenant.
You need to use the default tenant or create a new tenant.
Can you try it?

@salmoro realtime-dev may be the default tenant.
https://github.com/supabase/realtime/tree/main#server-setup
You need to use the default tenant or create a new tenant.
Can you try it?

@mats16 I'm having a hard time figuring this out. is this supposed to work out of the box?

@salmoro Sorry, I have identified the cause of the problem.
The subdomain of supabase-realtime is used to determine tenat_id.
The domain name must be set to something like realtime-dev.supabase.internal.
I already fix it on latest version. Please use it.


Bad routing result, turn restriction ignored

On road crossing turn restriction from "Kołłątaja" to "Kościuszki" is ignored. The turn is allowed only for public transport, not allowed for other vehicles.
https://graphhopper.com/maps/?point=51.102976%2C17.03692&point=51.101602%2C17.038465&locale=en-US&vehicle=car&weighting=fastest&elevation=true&layer=Omniscale

Currently turn restrictions are not enabled for GH Maps using the "speed mode" and only possible with the flexibility mode which is not yet production grade, but we are working on both sides.

It's worth mentioning that from my reports the turn restrictions with flexibility mode are working quite well. :+1:
The most notable is the lack of turn restriction with multiple ways as via member (see #446).


Wakeword not recognized after conversation

There is an issue with the WakewordListener where the wake word is not recognized after the conversation.

Fix #320


Error with Lazy arrays with caching

I am trying  to use lazy array for reading. Below is my testing code
bg_files = "/uscms_data/d2/lpctrig/benwu/AutoEncoderSample/Phaes2L1Ntuple/NeutrinoGun_E_10GeV_V7_5_2_MERGED.root"
nocache_data = uproot.lazyarrays(bg_files, "l1PhaseIITree/L1PhaseIITree", PhysicsObt.keys(), entrysteps=100)
print(nocache_data[1]["jetEt"])
cache = uproot.cache.ArrayCache(1024**2)
data = uproot.lazyarrays(bg_files, "l1PhaseIITree/L1PhaseIITree", PhysicsObt.keys(), entrysteps=100, cache=cache)
print(data[1]["jetEt"])

It seems the first reading without cache is working fine. But the second test with cache is giving me error as below. I think I am following the example.

[139.000001 123.500001 108.500001 100.000001  98.500001  97.500001
  94.500001  90.000001  88.000001  87.500001  83.500001  82.500001]

---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-16-4a637b7bbea9> in <module>
      4 cache = uproot.cache.ArrayCache(1024**2)
      5 data = uproot.lazyarrays(bg_files, "l1PhaseIITree/L1PhaseIITree", PhysicsObt.keys(), entrysteps=100, cache=cache)
----> 6 print(data[1]["jetEt"])

~/.local/lib/python3.6/site-packages/awkward/array/table.py in __getitem__(self, where)
     70             if isinstance(where, awkward.util.string):
     71                 try:
---> 72                     return self._table._contents[where][self._index]
     73                 except KeyError:
     74                     raise ValueError("no column named {0}".format(repr(where)))

~/.local/lib/python3.6/site-packages/awkward/array/chunked.py in __getitem__(self, where)
    310         if self._util_isinteger(head):
    311             chunk, localhead = self.global2local(head)
--> 312             return chunk[(localhead,) + tail]
    313
    314         elif isinstance(head, slice):

~/.local/lib/python3.6/site-packages/awkward/array/virtual.py in __getitem__(self, where)
    349
    350     def __getitem__(self, where):
--> 351         return self.array[where]
    352
    353     def __setitem__(self, where, what):

~/.local/lib/python3.6/site-packages/awkward/array/virtual.py in array(self)
    275         if self._array is None:
    276             # states (1) and (3)
--> 277             return self.materialize()
    278
    279         elif self._cache is None:

~/.local/lib/python3.6/site-packages/awkward/array/virtual.py in materialize(self)
    326             # states (3) and (4)
    327             self._array = self.key
--> 328             self._cache[self._array] = array
    329
    330         return array

~/.local/lib/python3.6/site-packages/uproot/cache.py in __setitem__(self, where, what)
     36
     37     def __setitem__(self, where, what):
---> 38         self._cache[where] = what
     39
     40     def __delitem__(self, where):

~/.local/lib/python3.6/site-packages/cachetools/lru.py in __setitem__(self, key, value, cache_setitem)
     19
     20     def __setitem__(self, key, value, cache_setitem=Cache.__setitem__):
---> 21         cache_setitem(self, key, value)
     22         self.__update(key)
     23

~/.local/lib/python3.6/site-packages/cachetools/cache.py in __setitem__(self, key, value)
     47         size = self.getsizeof(value)
     48         if size > maxsize:
---> 49             raise ValueError('value too large')
     50         if key not in self.__data or self.__size[key] < size:
     51             while self.__currsize + size > maxsize:

ValueError: value too large

Another naive question, is Lazy array most suitable for random accessing in root file from uproot?

The error message is saying that at least one of the chunks is larger than the whole cache—the cache can't hold that one item, let alone any others. You've allocated a 1 MB cache (1024**2), a size that was possibly used in one of the tutorials in order to demonstrate the effect of limited cache. As a result of #285, you can now specify memory sizes as a string: "1 MB", or more likely, "1 GB".
I think you asked @nsmith- earlier today about whether lazy arrays are a good idea for random access. Random access is generally slower than sequential access for any system with caching—even a plain C array is slower to access randomly than sequentially because chunks of it get cached in the CPU's L1/L2/L3 caches. Sequentially, you find all but the first item of a chunk when you look for it in cache; randomly, you have to load a new chunk on nearly every request, and loading a chunk is much slower than finding what you need in cache. Lazy arrays, which are ChunkedArrays of VirtualArrays do the same thing at a larger scale, where the "slow chunk-loading" is not copying from RAM into the CPU but reading from disk into your ArrayCache (and then from RAM into CPU).
If it's at all possible to load everything in memory at once, by supplying no cache or cache=None, then do so. You'll definitely get better results if your random-lookup function is in RAM, rather than on disk.
Although chunkedarray[indexes] for an integer indexes array is vectorized—the number of Python instructions scale with the number of chunks, not the number of entries/events—that still might not be satisfactory if you have many chunks due to many files. (If you have only one file, set chunksize=numpy.inf to load it all into one chunk to minimize the number of Python instructions.) If your data are logically a jagged array, i.e. a ChunkedArray of VirtualArray of JaggedArray, then you can concatenate all of the chunks with
awkward.JaggedArray.concatenate(lazyarray.chunks)

If it's logically flat, i.e. a ChunkedArray of VirtualArray of plain old Numpy arrays, then you can concatenate all of the chunks with
numpy.concatenate(lazyarray.chunks)

I hope to unify these sorts of things so that I can provide only one recipe in the future (awkward.concatenate), but this is where things stand right now.


Steps and ingredients in recipes

For some reason steps aren't picked up when getting infromation  about a recipe with steps.
Figure out what's wrong and fix it.

Tables of steps and ingredients need to be manually included when getting a recipe.
Separate repositories might not be a bad idea after all. Need to investigate further, but for now issue is solved.