Data Validation

Is there a single source of truth with regards to Congressional metrics? For example, for the 115th Congress, congress.gov lists 547 members, Wikipedia lists 541 members, and legislators-current.yaml lists 540.
Close to releasing a Congressional 'BI dashboard' using the data from this repo, but first would like to verify various metrics, though not sure what the best processes and resources for doing so would be. Are there any guidelines, and/or has anyone gone through a similar process with their own project(s)?
For context, here are a few examples of the various metrics being displayed, as well as some of the resources that are being referenced:

Thus far has been a process of constant spot checking, but would be very useful if there were some type of public Congressional reporting utilities to facilitate.
(Apologies if this is not the appropriate place for such an open ended question, just not sure where else to look for discussion related to 'civic hacking' or 'open gov' - haven't found many active communities thus far.)

It depends what you want to display: Congress.gov lists any member who has served during the 115th Congress, while @unitedstates/congress-legislators has a file for current members and one for those who are no longer in congress. Not sure about Wikipedia, but I would not call it a canonical source.

I see, legislators-current.yaml makes sense then, thank you.
As this app is intended for a general audience, Wikipedia is embedded throughout, for both legislator and committee profiles, like so:

As a layperson, i think Wikipedia (+ social media) can add a valid dimension to the overall profile of public officials, but not sure how that data might conflict with canonical records (nor even entirely sure what qualifies as canonical, aside from congress.gov and govtrack.us).
I'm currently linking to, and using data from: sunlight foundation, propublica, ballotpedia, countable, crowdpac, opensecrets, us vote foundation, and vote smart (in addition to .gov sites), but not sure what type of discrepancies a casual public user might encounter between them and this repo.


No way to set program language

Plugin and program are semi-translated to russian. I would like to use English language completely. Currently I don't see any way to do it

You can run DBeaver with command line parameter -nl en (command line parameters are readme.txt).
Also if you use Windows installer you can choose language there.

Thanks for help, that helped, more entries have become english, but some are still russian

That's because Eclipse caches some UI elements.
You can cleanup UI settings by deleting file ~/.dbeaver4/.metadata/.plugins/org.eclipse.e4.workbench/workbench.xmi

I use DBeaver4.0.2 Windows installer and choose English at the first step,but after all set up, it come out semi-translated to simplified Chinese. What's wrong? My os is win10 64bit run in traditional Chinese.

Most likely you have the same problem (try to delete workbench.xmi file).
I'm going to remove all localizations in next version. They are seriously out of date and have to be patched/reorganized.


Search Preview in Locally built Dejavu doesn't work

I tried to build a Searching UI with Dejavu, so I tried to Search Preview function. It doesn't work in my local built Dejavu, however it does work when I output the Index to a JSON file and upload to appbased.io.
by the way, my index was created with FSCrawler

@zydisney did you followed the second step mentioned here correctly, i.e. Adding batteries submodule?
Also, it will be helpful if you can share some errors from the browser console.
Another thing to keep in mind using Search Preview is that it only works if you have connected to a single index. If you try connecting using regex / comma-separated list of indexes then this option would be disabled.

@zydisney closing this issue for now. Let us know if you are still facing this issue.


Filter events before reply block

[x] Add failing testcase showing messages make it into reply block
[x] Ensure reply doesn't get event(message)
[x] Ensure submessage event(message) doesn't make it out with no reply block
[x] Remove no-longer needed event(message) filtering in DefaultWasmVMContractResponseHandler.Handle
[x] Update other tests as needed

Ready for review.
Added a few tests to assert the "message" events were not returned from DispatchSubmessages, nor ever returned to the Reply call.
Fixed/removed a few other tests that expected to find messages.


Added proxy support to Playlist

Issue seems to be fixed already -- please re-open if this isn't the case :)


fix: ensure correct version of torch is always installed based on BUI…

Description
Previously torch was being installed in the requirements.txt file, this would pull in the CUDA12 version of torch, which isn't appropriate for other BUILD_TYPE configurations.  This PR moves all of the pytorch related packages to BUILD_TYPE specific files so that we always install the correct pytorch package to match the BUILD_TYPE.
Hopefully this will help with issues such as #2737 and #1592.
Notes for Reviewers
Given that pytorch needs "special handling" based on the build_type, this would be a good candidate to pull the install of pytorch out of the requirements files and into the libbackend.sh library to better handle the torch, torchaudio and torchvision installations.  Many python backends have requirements-TYPE.txt files purely for handling the complexities of the torch installation, so it is very easy to get it wrong.
Signed commits

[x] Yes, I signed my commits.

@cryptk I think this is a good idea. However, I think that dependabot may not be picking up our "accelerator specific" files, and torch is obviously pretty important to track (even if in most cases, we're probably stuck on whatever version the project _requires)
Based on https://stackoverflow.com/questions/75899186/how-to-configure-dependabot-to-check-multiple-files it looks like we might be better off renaming these files, simply to invert the naming to something like cublas11-requirements.txt
Do you mind double checking me here and potentially doing the rename as a part of this PR since it's already basically touching everything?

@dave-gray101 once I have some time I'll take a look at your recommendation, I've been pretty swamped lately

This would be great, because at the moment I cannot use this project on anything but CUDA powered hardware. CPU only AMD 8700GE or CPU & iGPU won't work with the current docker images, even after rebuilding.
Since AMD is also attempting to mainline the driver stack for their (Xilinx) AI silicon for Kernel 6.12, a build type specific proecss would be helpful

Hi @gymnae if you are using docker, try using sha-e676809-hipblas-ffmpeg as the image name...  I can't find which gfx#### code the 8700ge is at the moment.  If I have some spare time this week I might try applying this patch to my local instance, but I've had issues with other changes in the past so no guarantees until the patch can get merged to the repo.  Cheers

Hi @gymnae if you are using docker, try using sha-e676809-hipblas-ffmpeg as the image name... I can't find which gfx#### code the 8700ge is at the moment. If I have some spare time this week I might try applying this patch to my local instance, but I've had issues with other changes in the past so no guarantees until the patch can get merged to the repo. Cheers

Thank you, I will try that. The 8700GE has a 870M iGPU, which is gfx1103 and currently not natively supported by ROCm, but an override works, at least in so far that ROCm is happy

Seems to break on mamba:
 #51 108.9 Initializing libbackend for mamba
#51 108.9 virtualenv created
#51 108.9 virtualenv activated
#51 108.9 activated virtualenv has been ensured
#51 108.9 starting requirements install for /build/backend/python/mamba/requirements-install.txt
#51 108.9 finished requirements install for /build/backend/python/mamba/requirements-install.txt
#51 108.9 starting requirements install for /build/backend/python/mamba/requirements.txt
#51 108.9 make: Leaving directory '/build/backend/python/mamba'
#51 108.9 Using Python 3.10.12 interpreter at: /usr/bin/python3
#51 108.9 Creating virtualenv at: venv
#51 108.9 Resolved 3 packages in 3ms
#51 108.9 Installed 3 packages in 13ms
#51 108.9  + packaging==24.1
#51 108.9  + setuptools==72.1.0
#51 108.9  + wheel==0.43.0
#51 108.9 error: Failed to download and build `causal-conv1d==1.4.0`
#51 108.9   Caused by: Failed to build: `causal-conv1d==1.4.0`
#51 108.9   Caused by: Build backend failed to determine metadata through `prepare_metadata_for_build_wheel` with exit status: 1
#51 108.9 --- stdout:
#51 108.9
#51 108.9 --- stderr:
#51 108.9 Traceback (most recent call last):
#51 108.9   File "<string>", line 14, in <module>
#51 108.9   File "/build/backend/python/mamba/venv/lib/python3.10/site-packages/setuptools/build_meta.py", line 368, in prepare_metadata_for_build_wheel
#51 108.9     self.run_setup()
#51 108.9   File "/build/backend/python/mamba/venv/lib/python3.10/site-packages/setuptools/build_meta.py", line 497, in run_setup
#51 108.9     super().run_setup(setup_script=setup_script)
#51 108.9   File "/build/backend/python/mamba/venv/lib/python3.10/site-packages/setuptools/build_meta.py", line 313, in run_setup
#51 108.9     exec(code, locals())
#51 108.9   File "<string>", line 20, in <module>
#51 108.9 ModuleNotFoundError: No module named 'torch'

I think I've got most of the fixes out, let's see if CI is now liking it

Should be available in master images as CI builds and publish the images -  @bunder2015 @gymnae  if you can test that now everything works as expected that'd be appreciated!

Hi @mudler I waited for quay to output the hipblas images, but they didn't seem to get uploaded.  I tried to build them manually with docker compose build, but it couldn't find the rocm clang.
The sliver of logs suggested that BUILD_TYPE was blank despite setting it in the docker-compose.yaml file...  So I tried adding --build-arg BUILD_TYPE=hipblas and it appears that apt couldn't find hipblas-dev or rocblas-dev to install...
Cheers

Hi @mudler, also wanted to try the new images, but it appears the hipblas image build failed

sha-ad5978b-hipblas-ffmpeg popped up on quay, but I'm still getting the "operator torchvision::nms does not exist" message...
Cheers

sha-ad5978b-hipblas-ffmpeg popped up on quay, but I'm still getting the "operator torchvision::nms does not exist" message... (still no idea why I ran into issues building the image manually)
Cheers

ok cool - I could finally reproduce here and https://github.com/mudler/LocalAI/pull/3194 should fix it. I tested manually, but need to double check after the CI builds the images as I did jumped in the container I had here and recreated the venv manually.


Modification of CI Workflow to enhance Coverage reporting

Summary

The problem is that the coverage report is being displayed on the command line, and it would be ideal to have a GitHub formatted output that shows up in code files in the PR that highlights the lines of code that were not covered in the unit test CI workflow. Doing so, it would assist reviewers in ensuring that the code coverage is high.
The unit_tests workflow has been modified to:

Create a coverage file only if the specified OS and python version finish the test
Run a step to process the file and extract the file names and the missed lines/ranges and highlight the lines using workflow commands

Resolves: #288
Release Note

Documentation

@mihaimaruseac I had to create a new PR , because the last one closed due to me reverting my changes on the PR branch. The issue was that the tests achieved 100% coverage, that's why it didn't post anything. I'm assuming because someone implemented more tests.
I tried adding random lines of code, and it managed to detect them.

tests achieved 100% coverage, that's why it didn't post anything. I'm assuming because someone implemented more tests

Yeah, we reached 100% coverage in #291. Sorry for not mentioning it.


Missing resources in kubectl get/describe --help

kubectl get --help doesn't list vms, vmrs and ovms as resources.
vms, vmrs, and ovms are not listed in kubectl describe --help too.

I'm not familiar with KubeVirt and kubectl that much yet to say if anth else is missing, but I would appreciate it if --help was useful and complete.

Closing this, as there is nothing we can do from the kubevirt side.


[Issue] - Health Checker - Current Auth Certificate shows False

On Exchange Server 2016 latest patches
v<IP_ADDRESS>9
Provide the version of the script that has the issue.
Current Auth Certificate result = false even if it is valid checked by monitorExchangeAuthCertificate.ps1 and
(Get-AuthConfig).CurrentCertificateThumbprint | Get-ExchangeCertificate | Format-List Subject, Thumbprint, NotAfter, NotBefore
Internal Server Certificate = True
A clear and concise description of the issue.
Should be True since certificate is configured and valid.
A clear and concise description of what you expected to happen.
Internal Transport Certificate	True
Current Auth Certificate	False
Next Auth Certificate	False
SAN Certificate	True
Namespaces...
Certificate
Lifetime in days	1779  << cert
If applicable, add the exception that wasn't handled.
Invalid results for the auth certificate
Add any other context about the problem here.

@jtsai8585 Are you looking at the correct cert? In the Certificates section, you will see Current Auth Certificate: False for all certificates except the one cert that is the auth cert.
In the post you provided that Internal Transport Certificate: True and SAN Certificate: True which is NOT typical for the Auth cert.

I see what you mean now.  Each cert have all four types listed.
Thanks.


Implementation about denoising auto encoder

Hello! I notice that denoising auto encoder is used as benchmark. And I am interested in the implementation of that AE. Could you provide the network definition file?

Sure. I have placed it here. I had to rename it to a .txt file so github would accept it, but it is a prototxt net definition file.

Thank you!


update scythe-crush-warning to v1.1.0

Fixed a bug where the overlay is incorrectly being displayed inside the Nightmare instance

Sorry for dismissing your review, I've just updated the plugin after Hueycoatl CAs


set out_depth and pad_depth in PoolParameters if depth_window == 1

Closes https://github.com/tensorflow/tensorflow/issues/8313

Can one of the admins verify this patch?

@tensorflow-jenkins test this please


Big GPU consumption with retina tiles on browser zoom to 75-33 %

Browser: Chrome 50, 52
OS: Linux (reproducible on Windows too)
Leaflet version 1.0.0-rc.1
Issue is not reproducible with leaflet 0.7.7
Play test - http://playground-leaflet.rhcloud.com/qiwa/1
Steps -

open link with retina tiles with map for all window - 100% width and 100% height
open chrome task manager (Shift + ESC)
reduce browser zoom to 50%
see GPU memory consumption (medium - near 120 mb, peak - more then 512)

Result - big memory consumption, on old or device with limited gpu memory - destroyеd map layer
Problem: GPU hardware acceleration was added to tiles. It's consist from 2 pieces will-change:

opacity property on tile element from (commit 6b67df023  make animations significantly smoother with will-change ) [https://github.com/Leaflet/Leaflet/commit/6b67df023234186e70647334c381fd66e4838721]
translate3d properties in addTile https://github.com/Leaflet/Leaflet/blame/master/src/layer/tile/GridLayer.js#L752

If you are on a Retina device you already have 4x the number of pixels of a normal device (say 3840x2040 pixels). If you even zoom the browser to 50%, now you have 16x the number of pixels of a normal device (say 7680x4080 pixels).
Obviously having a browser tab display 7680x4080 pixels will use that much GPU memory. But Retina devices are new and have enough memory to handle this. If not, just do not zoom the browser window to 50%, it will make your memory usage 1/4.
The GPU hardware acceleration is a key feature of Leaflet and the added properties are one reason why 1.0 is slightly faster then 0.7.

Wouldn't it be easy to avoid issues like this with a simple flag to disable GPU hardware acceleration?

@kaynz http://leafletjs.com/reference.html#global-l_disable_3d

The main issues here is the tight case for retina tiles (on non-retina devices initially but it's can be fixed by override zooms) and acceleration on raster tile elements.
I fix it for our project by set .leaflet-fade-anim .leaflet-tile style to will-change: init; (it's recommended to not use widely will-change on mdn - https://developer.mozilla.org/en-US/docs/Web/CSS/will-change )
And manually change translate3d to translate for tile dom elements. Other optimizations dont cause additional gpu memory consumption and not show additional gpu usage in compare with 0.7.7
And the tweak actually work well in chrome and i dont see any changes (but may be it's can be issue in other browser if we will remove that premature optimization for tiles

@davojta why is it better than L_DISABLE_3D?

@hyperknot because L_DISABLE_3D will remove all accelerations (from panes and container elements) which is reduce performance in whole map app (because of translate3d style makes accelerated layer if we remove that style, all layers will be without hw acceleration)

I fix it for our project by set .leaflet-fade-anim .leaflet-tile style to will-change: init;

Can you make that into a pull request for the leaflet repo? I'd be nice to try it out.

@davojta but you said:

And manually change translate3d to translate for tile dom elements.

On which tiles or panes or elements exactly do you use translate3d and on which ones do you use translate? Yes, making a pull request would be really good.

@davojta Any news on this?

need to review my fix, i  stuck with needed optimizations
I switch off will change property as mentioned in https://github.com/Leaflet/Leaflet/issues/4730
also i made fall back from translate3d to translate by monkey patching leaflet
and actually found bug in our tile rendering mechanics (actually we use retina tiles for no reason)
So these fixes completely fixed our perf issues

Does it make sense to avoid translate3d in L.Marker and L.Tooltip?

No, it doesnt. Translate3d is working well with small area elements like
tooltips and markers. The main problem is full screen elements which very
heavy to process on gpu due to big texture size created by acceleration
layer.
On Wed, Jan 18, 2017, 11:25 AM kaynz<EMAIL_ADDRESS>wrote:

Does it make sense to avoid translate3d in L.Marker and L.Tooltip?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Leaflet/Leaflet/issues/4578#issuecomment-273412618,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA2i0jm74dmkTZbo57Y7juNVlKKoG7efks5rTcyKgaJpZM4Ig-uv
.


[Amazon] Rename amazon build flag

https://jenkins.code.rdkcentral.com/job/github-triggered/817/ started


toggle visibility not re-showing hidden window on macOS

https://discordapp.com/channels/748161877045149706/748162434237202472/764446424631869471
also applies to the relevant hotkey

closing until/unless re-reported


[website] Include US government web design standards in the list of websites using Prism!

Woohoo!!!!
https://playbook.cio.gov/designstandards/

How exciting! :tada:

That's awesome!
Do they have any logo, though? Also, which of the existing websites will we remove? (Or we can add a third line by displaying back Typeplate and adding Raspberry Pi for example. What do you think?

We could include a small screenshot or something. It’s a government website, this is big! We won't let a lack of logo stop us in including it :D
I’m afraid TypePlate is a bit small compared to the other websites in the list, which affects the impression the whole list makes. I’m on the fence re:raspberry pi, it's well known, but not as well known as e.g. sitepoint or MDN. We can remove WebPlatform, since they're not using it everywhere, and it's also smaller than the rest of them. Or just add a third row with 1 item. Up to you, both are fine with me!

Here we go!

If we’re not gonna use a screenshot, perhaps we could just have styled text instead of an image of styled text?

I'm not sure I wanna add the Merriweather font just for 4 words on the homepage... Can't we just consider this a logo?

Oh, I didn’t realize you were using the font from the website! Of course then.
Another option would be to use a screenshot like this:

with background-size: cover;

I'll give it a try tomorrow ;)

Thanks!

Nice idea, it is still readable and it looks great IMO. If you want us to use backgrounds instead of image tags for the other logos too (I made a special case of this one), just tell me ;)

I like it too, thanks!
Image tags are more semantic, but we'd need object-fit support to do this with image tags, which is not supported by IE/Edge :/

Ok, let's leave it that way for now! ;)


Provide node test patches for #7452

@YJDave You can grab my patches and move them to your branch, look at them, run them, and squash.

Thanks!!
Included this commits in #7452 :)

Ok, @YJDave, since you've gotten the commits on the other PR, I'm closing this.  Good luck!


Metadata block interference patterns

When you have big trees with large blocks of the same colour metadata, interference patterns emerge. This seems to be caused by tiny white lines between the blocks. I wonder if this would be solved by rounding all coordinates before drawing the blocks so that they are full integers?

yeah i think that would be a good solution. I'm not sure if the boxes are trimmed or there are white lines drawn


Set server-side pagination rendering with Laravel 5.6

Issue Type (delete the irrelevant ones)

[x] Question

Specs
Latest
What browser?
Expected Behavior
Chrome
I'm currently implementing this using laravel 5.6 and its working fine, should I use laravel's paginate() in serverside?

@neverender24 you might have better luck asking this question in laravel forums or stack overflow. Issues here specifically deal with the plugin itself.
closing.

@neverender24 you find a solution ?


OneDrive Integration not Single Tenant compatible

Preflight Checklist

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Describe the bug
Registering the app as Single Tenant app in a Azure AD will result in an error (pop-up window stating access denied) every time a user tries to log in (Azure AD error code 50194).
The reason for this seems to be that microsoft only allows multi tenant apps to use the https://login.microsoftonline.com/common endpoint, single tenant app are required to use a tenant specific endpoint (i.e. https://login.microsoftonline.com/${Tenant_UUID}).
The ability to use single tenant configurations would be greatly appreciated in a self hosted scenario.
Error Message in Azure AD Logs:
Application '{appId}'({appName}) is not configured as a multi-tenant application. Usage of the /common endpoint is not supported for such applications created after '{time}'. Use a tenant-specific endpoint or configure the application to be multi-tenant.

Login started working after setting the Azure AD App to multi tenant.
To Reproduce

Register a drawio server as an app in Azure AD
In Azure AD App settings set "Supported account types" to "Accounts in this organizational directory only"
Try logging into OneDrive --> Error
Reconfigure Azure AD App to multi tenant ("Accounts in any organizational directory")
Wait for AD to sync (takes a few minutes)
Try logging in --> works

Expected behavior
Login works.
draw.io version (In the Help->About menu of the draw.io editor):
20.6.0
Desktop (please complete the following information):

OS: Windows 10
Browser MS Edge, Firefox

Thanks for the report. Support for single tenant will be added to the next release using window.DRAWIO_MSGRAPH_TENANT_ID

Internal commit https://github.com/jgraph/drawio-dev/commit/dc03290718514b4fe470c45836ef1d332930c63f

Released in 20.6.1


KOGITO-6493 Upgrade mvnw version

The (build) kogito-examples check has failed. Please check the logs.

The (build) kogito-apps check has failed. Please check the logs.

Jenkins rerun kogito-apps tests

Jenkins rerun kogito-examples tests

The (build) kogito-apps check has failed. Please check the logs.

Jenkins rerun kogito-apps tests

Jenkins rerun kogito-apps tests

The (build) kogito-apps check has failed. Please check the logs.

Jenkins rerun kogito-apps tests

The (build) kogito-apps check has failed. Please check the logs.

Jenkins rerun kogito-apps tests

The (build) kogito-apps check has failed. Please check the logs.

Jenkins rerun kogito-apps tests


[feature request] Derive some useful traits

I want to suggest deriving Clone and PartialEq for charts and containing structs. Serialize and Deserialize for example could also be feature gated, if necessary. What do you think?

Hi @koopa1338 , thanks for the suggestion. What are the justifications that we would want a chart to implement Clone and PartialEq?

When using leptos you are using a lot of move closures, which it makes a real hassle to make the chart responsive in leptos.
PartialEq may be an edge case and I only need it to detect if a chart has changed, maybe there is an api to catch those events?
In general, I would recommend deriving traits from the std if it doesn't hurt performance, and it makes sense, otherwise feature gate the derives.


design is not obvious, needs refactoring

@rultor release, tag is 0.0.26

@rultor release, tag is 0.0.26

@yegor256 OK, I will release it now. Please check the progress here.

@rultor release, tag is 0.0.26

@yegor256 Done! FYI, the full log is here (took me 8min).


controller.validateTrigger=validateTrigger.change is not handling blur

Unfortunately we still need the blur detection for required field handling, so we either need to automatically include blur as a trigger, or allow the user to specify two triggers.

Apologies- I'm not clear on what you mean here. Please include an example with the gist and a description of the expected behavior.

Sure, the issue is shown here: https://gist.run/?id=f5725cdc0f807450fe4555794449d851
It's not a bug as-such, it's just highly undesirable.
If you tab over the required fields, no error appears, although as you type it obviously does.
The problem is that people want change events to provide more immediate feedback, and we've removed the most immediate feedback for required fields.
I'm not wild on 'assuming' people also want blur events to be handled, but with 'change' I'm can't think why they wouldn't.
This example was a test of the option binding, and demonstrates nicely why the event type needs to be assignable on a per-component basis. The option doesn't trigger as it changes because there's no blur event, so in theory you need the change event (but only for this field), and strictly speaking you want it to validate on the binding event, as this allows throttle and debounce (which is an issue on option fields if the user holds up/down keys).
This is why I was wondering if the validateTrigger should have the option 'validateTrigger.bind' as its default?

got it- a changeOrBlur validate trigger? would that do it?

Possibly. How will this react with the binding behaviours? Are we doubling up on them?
I know it's a bit weird, but it may be better to have changeOnly, for the unlikely event that someone ever wants to not catch the blur?
I can see us getting inundated with issues saying that change is ignoring required fields...


Nitpicky stuff (12/2022)

[x] Dis(in)genuine currency press(both versions) trigger twice when parent discarded.

[x] Mind Twist, 2nd action; nothing is logged.

[x] Calling the Cavalry should be more specific about what is happening, with a source:this and a better title.

[x] Vivene Goldsun misevaluates when deciding if costs are payable or not, works properly when picking a good to discard.
( "<" vs ">")

Dis(in)geniune Currency Press fixed by #1607


[Feature Request] F5/M5 Livid Solver

Title say everything

Odin won't duplicate any features we believe a viable mod has without adding any additional functionality.
In this instance the viable mod is Skytils and I don't think we have any additional functionality to add.

Skytils livid solver works badly, half of the time it doesn't mark the mob and its box is hard to see


Downgrading from 3.3.2 to 2.15.1 errors about toltec-deletions not being available

Describe the issue
It appears that toltecctl isn't properly cleaning up opkg state when reenabling on an older OS version.
To Reproduce
Steps to reproduce the behaviour:

Use codexctl to downgrade from 3.3.2 with toltec enabled to 2.15
Run toltecctl reenable

Expected behaviour
A clear and concise description of what you expected to happen.
Screenshots

System information:
toltecctl status output:
reMarkable: ~/ toltecctl status
Enabled: Yes
Supported: Yes
Branch: testing
Model: rm2
OS: <IP_ADDRESS>9

Additional context
Upon inspecting /opt/lib/opkg/status there is only one entry for toltec-deletions, which has been changed to rm2os2, but it should be rmallos2. Changing this allows toltecctl reenable to work as expected.

This also appears to happen on the rM1


[TRI-2668] logger.trace should have the output as attributes

If you return data from logger.trace it should appear in the OTEL attributes from the span.
const bar = await logger.trace('key', async() => { return foo });

The same as we do for attempts.
TRI-2668

@matt-aitken I would like to give it a attempt

Hi there, this is affecting us pretty much currently, since it's hard to debug complex runs when something goes wrong without having the context of what the data response was.
Would love to create a PR, just not sure where to start/what would have to be changed to archive this.


build(package.json): bump ver

Pull Request Test Coverage Report for Build 274

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

Totals

Change from base Build 265:
0.0%

Covered Lines:
52

Relevant Lines:
104

💛  - Coveralls


Casting is broken for multi-audio streams

Have you read the FAQ and checked for duplicate open issues?
Yes
What version of Shaka Player are you using?
4.2.2
Can you reproduce the issue with our latest release version?
Yes
Can you reproduce the issue with the latest code from main?
Yes
Are you using the demo app or your own custom app?
Demo, custom
If custom app, can you reproduce the issue using our demo app?
Yes
What browser and OS are you using?
MacOS Ventura Beta, Chrome 106.0.5249.103  - whatever is the default on the page
Android 13, Native Chromecast SDK - receiverID (default CAF: CC1AD845)
iOS 16, Native Chromecast SDK cast to receiverID (default CAF: CC1AD845)
What are the manifest and license server URIs?
Any multi-audio stream, DASH, HLS, e.g.
https://storage.googleapis.com/shaka-demo-assets/angel-one/dash.mpd
What configuration are you using?  What is the output of player.getConfiguration()?
Default on https://shaka-player-demo.appspot.com/demo/#uilang=en-GB;asset=https://storage.googleapis.com/shaka-demo-assets/angel-one/dash.mpd;panel=HOME;build=uncompiled;debug
What did you do?

Play locally:
https://storage.googleapis.com/shaka-demo-assets/angel-one/dash.mpd
Open the menu within the player (...) then click Cast
Start casting

What did you expect to happen?
Casting happens on the Chromecast receiver
What actually happened?
Casting fails with error code: 3016
If you refresh the page (Chromecast stays connected), then I get:
Cannot read properties of undefined (reading 'sourceBufferExtraFeatures')

Most probably this is a receiver-side side issue, but as far as I know, it is using Shaka Player by default.

Quick status update on this:
It seems everything is back to normal from yesterday.
I don't really know what was changed on the CAF v3 receiver side, but definitely, there was a glitch in the matrix as I wasn't the only one experiencing the issue on Monday (10.10.2022).
I am closing the ticket. Thanks!


[Event Request] codeunit 1248 "Process Bank Acc. Rec Lines" - OnBeforeImportBankStatement

Can you add event OnBeforeImportBankStatement to be able to override the code to get DataExchDef?
        StartDateTime: DateTime;
        FinishDateTime: DateTime;
        // >>>>>>>>>>>>>>>>>>
        RelatedRecord: RecordId;
        DataExchDefCode: Code[10];
        // <<<<<<<<<<<<<<<<<<
    begin
        // >>>>>>>>>>>>>>>>>>
        /*
        BankAcc.Get(BankAccRecon."Bank Account No.");
        BankAcc.GetDataExchDef(DataExchDef);

        DataExch."Related Record" := BankAcc.RecordId;
        DataExch."Data Exch. Def Code" := DataExchDef.Code;
        */
        OnBeforeImportBankStatement(BankAccRecon, DataExchDefCode, RelatedRecord);
        if DataExchDefCode = '' then begin
            BankAcc.Get(BankAccRecon."Bank Account No.");
            BankAcc.GetDataExchDef(DataExchDef);

            RelatedRecord := BankAcc.RecordId;
            DataExchDefCode := DataExchDef.Code;
        end;

        DataExch."Data Exch. Def Code" := DataExchDefCode;
        // <<<<<<<<<<<<<<<<<<

        if not DataExch.ImportToDataExch(DataExchDef) then
            exit(false);

    [IntegrationEvent(false, false)]
    local procedure OnBeforeImportBankStatement(BankAccRecon: Record "Bank Acc. Reconciliation"; var DataExchDefCode: Code[10]; var RelatedRecord: RecordId)
    begin
    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.


[RHBPMS-4105] concurrent JGit test

Concurrent JGitUtil access test.
Thread 1 : Keep committing new files to a repo
Thread 2 : Keep reading (resolvePath) an existing file in the repo.
It runs successfully in Linux (Fedora 22) but it constantly fails in Windows 7.
The loop counts (200 for committer thread. 2000 for reader thread) are arbitrary values that I can reproduce the issue constantly but try to minimize the test time. You may need to increase the values to reproduce the issue in a faster Windows machine.

Thanks @tkobayas. I'll take a look on this.

-1 for it; it should use byteman to force concurrency issue


Fix issues found by OAP integration

Signed-off-by: Gao Hongtao<EMAIL_ADDRESS>

Codecov Report

Merging #200 (747ddb6) into main (e6b01fc) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #200   +/-   ##
=======================================
  Coverage   44.57%   44.57%
=======================================
  Files          84       84
  Lines        8382     8382
=======================================
  Hits         3736     3736
  Misses       4325     4325
  Partials      321      321

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Update index.md

Updated main page with link to Unreal early access page.

Docs Build status updates of commit 9274bb8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Tutorials - Fragment Highlighter & Fragment Edges Tutorial

Description
Tutorial for Fragment Highlighter

What is the purpose of this pull request?

[ ] Bug fix
[ ] New Feature
[X] Documentation update
[ ] Other

Added a commit for Fragment Edges tutorial as well.


Suggestion: make ignored_modules a tuple of symbols, instead of a tuple of modules

On issue I've run into is that JET will complain about (perhaps deeply nested) indirect dependencies, and that means that one has to add those dependencies to the Project toml in order to ignore those modules. It might be really nice if ignored_modules instead accepted Symbols, so that no additional dependencies are required.

I'm just curious, is there any chance that this issue gets fixed? (cc @aviatesk).
We've been trying to remove CUDA as a direct dependency in some packages, and requiring CUDA just pass this test is a bit unfortunate.


Backfill jobs which are lost while Dr.Elephant was down

@akshayrai , kindly review.

Hey @varunsaxena
As this isn't an internal change, we can probably have the design document added in here.

Travis CI is failing due to oraclejdk7 support no longer being available. Will raise another PR to change it to openjdk7

PR #495 fixes jdk 7 build issue post which the build here passes

Discussed with @akshayrai offline. As he is on leave and he has already approved final changes, I will merge the code changes into master by myself.


Limitation by INT_MAX elements

Good day!
I am trying to use sparse.mm for my custom layer for the super-resolution problem, and seems that my tensors are a bit large. When I am implementing the sparse matrix multiplication of my sparse features to the dense weights like this
result = sparse_mm(sparse_tensor_2d, self.weights).to_sparse().coalesce()
I get the
RuntimeError: nonzero is not supported for tensors with more than INT_MAX elements,   file a support request
The size of my sparse_tensor_2d is (33554432 x 16), and the size of the dense self.weights tensor is (16, 64)
Could you tell me, can this problem be solved by changing the INT_MAX? If yes, how can I do it?
Regards, Evgeny

This is a known pytorch issue as far as I can tell: https://github.com/pytorch/pytorch/issues/51871
That said, our sparse_mm function is for sparse times dense matrix multiplication and thus results in a dense matrix. Your use case may be specific but in general, there is no obvious reason to call to_sparse() (which is where nonzero is probably being called) on the result of our sparse_mm.
-> Closing this

This is a known pytorch issue as far as I can tell: pytorch/pytorch#51871
That said, our sparse_mm function is for sparse times dense matrix multiplication and thus results in a dense matrix. Your use case may be specific but in general, there is no obvious reason to call to_sparse() (which is where nonzero is probably being called) on the result of our sparse_mm.
-> Closing this

Thank you!


XML & PHP is not rendering correctly.

Information

Language: PHP + XML
Plugins: none

Description
I can't get XML to display the elements. It also seems to override the PHP code in the display. I have an if / else statement showing but it only renders the XML tag. The XML not rendering element tags happens throughout page in multiple examples.
Code snippet
sample page](https://fuelingphp.com/return-xml-response-in-php/)

The code being highlighted incorrectly.
<?php

header("Access-Control-Allow-Origin: *");
header("Content-Type: text/xml; charset=UTF-8");

if(file_exists('courses.xml')) {
    $courses = file_get_contents('courses.xml');
    echo $courses;
} else {
    echo "<?xml version=\"1.0\"?><Error>404 - File not found!</Error>";
}

?>

That's because your site's source code is this:

--
  | header("Access-Control-Allow-Origin: *");
  | header("Content-Type: text/xml; charset=UTF-8");
  |
  |
  | if(file_exists('courses.xml')) {
  | $courses = file_get_contents('courses.xml');
  | echo $courses;
  | } else {
  | echo "<error>404 - File not found!</error>";
  | }
  |

Note the syntax highlighting GitHub does. <error> is not text, it's an element. Prism strips all elements before highlighting, this is an intended and necessary effect. For more information, see the basic usage section on our website.

THanks for the response, I'm not quite sure that I'm following.
If you scroll down the page, there are pure XML blocks that aren't rendering even though the language type selected is XML.
This content is actually being created from a WP plugin that is using prism. I'd like to reach out to them but could use some guidance on where to direct them.
Are you saying that the XML elements cannot be displayed.. even on the code blocks that are specified as XML?
Thanks again for assistance.

there are pure XML blocks that aren't rendering even though the language type selected is XML.

Same problem. This is the HTML of the code block.

< ? xml version="1.0" ? >
<courses>
    <course id="C-001">
        <title>Fundamentals of Programming</title>
        <credithours>3 + 1</credithours>
        <prerequisites>None</prerequisites>
    </course>
    <course id="C-002">
        <title>Object Oriented Programming</title>
        <credithours>3 + 1</credithours>
        <prerequisites>Fundamentals of Programming</prerequisites>
    </course>
</courses>

Again, all HTML elements inside  will be removed.
The reason < ? xml is escaped is likely that your CMS (or similar) detected that < ? xml ... is not a valid element, and escaped it as text.
To make e.g. the HTML element <course> text, it has to be escaped as <course>.

Are you saying that the XML elements cannot be displayed.

Prism can highlight XML code (example), but it will ignore HTML elements. We highlight text and only text. If your XML code is not given as text but as HTML elements, then Prism won't highlight them.
To put it in more technical terms, we use textContent to extract text, not innerHTML.
One of the reasons no sane syntax highlighter uses innerHTML is security. Users would not be able to highlight HTML code containing scripts without those scripts executing.

Awesome. This makes sense. I can get this corrected. Thank you for the explanation.


[core] Moved the secret_store from S3 to SSM

to: @ryandeivert @Ryxias
cc: @airbnb/streamalert-maintainers
related to: #240
resolves: #240
Background
Using S3 as a secret_store is no longer desirable. Using SSM Parameter store makes visibility into what secrets are set much easier and requires less setup by StreamAlert
Changes

Implemented SSMDriver (a bit of copy&pasta from the S3Driver)
Updated unit_tests so they pass when using this new driver
Removed secrets bucket creation
Added ssm:GetParameter permissions to the alert_processor

Testing

ran ./tests/scripts/pylint.sh - currently there are warnings (i aim to clean this code up but want to ensure this initial commit is good)
ran ./tests/scripts/unit_tests.sh - All created tests now pass, i need to add some for SSMDriver

@jack1902 I've been merging 3.0.0 PRs into feature-3-0-0 so you will have to rebase this one before we can merge

@ryxias rebase on feature-3-0-0

Per a conversation, I will help Jack rebase this
Question is... how?

Looks like I got it yay

Coverage decreased (-0.2%) to 95.911% when pulling 919a6d5a1347709184fa6d11d6900c54a99753c6 on jack1902:outputs/move_secret_store_to_ssm into a97b266486d6d0a46cd76fce25d6f6bea63fb6e2 on airbnb:release-3-0-0.

For the record, the old migration script can still be found here: https://github.com/jack1902/streamalert/blob/fad19f32400196e04967d917991ddcc88fb5d689/scripts/migrate_from_s3_to_ssm.py

I will address @ryandeivert 's comments in a follow-up PR


[Azure.Core TestFramework] Fix Required Optional Environment Variable

Summary
The focus of these changes is to bypass the attempt to sanitize an optional environment variable when no value was present.  Previously, if the variable was not read, sanitizing would result in an exception.
Last Upstream Rebase
Friday, September 4, 10:39am (EDT)
References and Related Issues

[Azure.Core] Test Environment Exception When Optional Environment Variable Not Set (#14869)

Fixes #14869

Add a test to https://github.com/Azure/azure-sdk-for-net/blob/aec86f6742b73deb307f15390500771dbe13c5e2/sdk/core/Azure.Core/tests/TestEnvironmentTests.cs#L13, please

Add a test to
https://github.com/Azure/azure-sdk-for-net/blob/aec86f6742b73deb307f15390500771dbe13c5e2/sdk/core/Azure.Core/tests/TestEnvironmentTests.cs#L13
, please

Done.  Was waiting for CI to finish to determine if there was a dedicated set of tests; it wasn't obvious they were in the Azure.Core solution rather than the TestFramework.

Thank you for fixing this!


Can't change tag names or add new ones

Hi, thanks for this theme it's really cool!
I've been trying to change the tag names and number of tags with no success. I've added a tag5.html file under the tags folder and edited the tag.yml. I understand there is a loop in default.html that should read through the list, but nothing changes. Any tips?
Thanks! 🙃

Case solved, I just needed to add the tags to the .md files that match the ones in the file above!


[ISSUE #4160]remove the final modifier of a method which is already in a final class

Make sure set the target branch to develop
What is the purpose of the change
remove the final modifier of a method which is already in a final class
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Codecov Report

Merging #4163 (6dedcce) into develop (93edad2) will decrease coverage by 0.05%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop    #4163      +/-   ##
=============================================
- Coverage      48.01%   47.96%   -0.06%
+ Complexity      5015     5003      -12
=============================================
  Files            635      635
  Lines          42506    42500       -6
  Branches        5567     5567
=============================================
- Hits           20411    20383      -28
- Misses         19605    19622      +17
- Partials        2490     2495       +5

Impacted Files
Coverage Δ

...he/rocketmq/broker/plugin/MessageStoreFactory.java
15.38% <ø> (ø)

...rocketmq/broker/filtersrv/FilterServerManager.java
20.00% <0.00%> (-14.29%)
:arrow_down:

...va/org/apache/rocketmq/store/FlushDiskWatcher.java
81.25% <0.00%> (-9.38%)
:arrow_down:

...ache/rocketmq/common/stats/MomentStatsItemSet.java
39.13% <0.00%> (-4.35%)
:arrow_down:

...ava/org/apache/rocketmq/filter/util/BitsArray.java
59.82% <0.00%> (-2.57%)
:arrow_down:

...va/org/apache/rocketmq/logging/inner/Appender.java
34.83% <0.00%> (-2.25%)
:arrow_down:

...a/org/apache/rocketmq/filter/util/BloomFilter.java
60.43% <0.00%> (-2.20%)
:arrow_down:

...a/org/apache/rocketmq/store/StoreStatsService.java
35.85% <0.00%> (-1.69%)
:arrow_down:

...rg/apache/rocketmq/remoting/netty/NettyLogger.java
14.96% <0.00%> (-1.37%)
:arrow_down:

.../apache/rocketmq/logging/inner/LoggingBuilder.java
63.92% <0.00%> (-1.27%)
:arrow_down:

... and 9 more

Continue to review full report at Codecov.

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

Coverage decreased (-0.06%) to 51.993% when pulling 6dedcce1f53a3f1344df0e19d50714cc5a2303ab on xyjForCoding:develop into 93edad2cb3de29839ee0c67daf41d2b27cff30e7 on apache:develop.

Coverage decreased (-0.06%) to 51.993% when pulling 6dedcce1f53a3f1344df0e19d50714cc5a2303ab on xyjForCoding:develop into 93edad2cb3de29839ee0c67daf41d2b27cff30e7 on apache:develop.

I don't know what to do now. The codecov report says all checks have failed.


The Caravan map tag still exists after the Caravan has been deleted.

Mod source
Factorio Mod Portal
Which mod are you having an issue with?

[X] pyalienlife
[ ] pyalternativeenergy
[ ] pycoalprocessing
[ ] pyfusionenergy
[ ] pyhightech
[ ] pyindustry
[ ] pypetroleumhandling
[ ] pypostprocessing
[ ] pyrawores

Operating system

=Windows 10

What kind of issue is this?

[ ] Compatibility
[ ] Locale (names, descriptions, unknown keys)
[X] Graphical
[ ] Crash
[ ] Progression
[ ] Balance
[ ] Pypostprocessing failure
[ ] Other

What is the problem?
The Caravan map tag still exists after the Caravan/Aerial Caravan has been deleted.
Steps to reproduce
No response
Additional context
No response
Log file
No response

Fixed for the next version. Thanks for the report.


Network/Packet bug

Not sure if this is specific to Flux Networks or if it is more a bug of SonarCore's packet system: https://pastebin.com/nJzvkkwR the server does not crash from this but the player does get disconnected.
This error occurred seemingly at random so unfortunately I cannot give any steps to reproduce. If I find anything that occurs just before this happens I will update this issue.
SonarCore: 1.12.x-5.0.5
Flux Networks: 1.12.x-3.0.3

Fixed this one in an earlier version


Implement Qobj double validation against general Qobj schema and backend-specific schema.

What is the expected enhancement?
According to #854, qobj emitted in Qiskit has to validate twice. One validation goes against the general Qobj schema. The second validation will go against a backend-specific one.

I think the idea here is validate against the general qobj schema, pull the backend schema using backend.schema() and validate against that.

^ @dcmckayibm yes.

Closing after the meta #851.


Feature/issue 686 map rect serial

Submission Checklist

[ ] Run unit tests: ./runTests.py test/unit
[X] Run cpplint: make cpplint
[X] Declare copyright holder and open-source license: see below

Summary:
Add the serial version of map_rect
Intended Effect:
Add serial map_rect function to stan-math. A MPI parallelized version will follow.
How to Verify:
Run tests:

test/unit/math/prim/mat/functor/map_rect_test.cpp
test/unit/math/prim/mat/functor/map_rect_serial_test.cpp
test/unit/math/rev/mat/functor/map_rect_serial_test.cpp

Side Effects:
Documentation:
Please refer to doxygen documentation in stan/math/prim/mat/functor/map_rect.hpp.
The design follows a map-reduce-combine design. The map step is effectively done by the use by passing in the data in a rectangular format. That is, the first index of the input data structures is used to define what is a job. The reduce is to calculate results and the the combine operation inserts results in the AD tree in a pre-computed form. The gradient pre-calculation is performed to ensure that the MPI version will give exactly the same results (and this scheme is in addition faster for large problems when compared to build a big AD graph instead).
Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company):
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

OK, I hope I addressed your comments adequately.
Thanks for the quick review (on a Sunday).

Sorry---you're right.  That's an Eigen type.  Eigen doesn't automatically call the constructors, so you can size and then set without overallocating on the autodiff stack.  But it also doesn't let you do an emplace constructor.  So yes, to_var is probably easiest here.

This is really clear and well organized.  Thanks!
It's ready to merge as soon as the tests pass.


Found a typo in SVM.md

Hello there,  I found a small typo in SVM.md thought of fixing it. I have gone through Contributing Guidelines.
Here's a snapshot of the typo I found.

I think it must be 'SVM are used mostly for classification type problems'.
I would like to fix this issue.

Hey @SuhasBRao Thank you for going through the contents and informing us about this typo. I think you have already made a PR for this issue and i will take a look at it asap.

Thank you for reviewing the issue.

Hey there, I found few more typos and corrected them. I have made a PR please do review it.

Sure i will review it asap.

@SuhasBRao reviewed the changes. The PR has been merged. Thanks for your contribution.


SsdpClient does not have a stop()

Currently, the SsdpClient does not expose a stop() method. So if one wants to scan for devices for 10 seconds and then stop, the only way you can do that now is to do a setTimeout() and call process.exit(); rendering the use of the SsdpClient limited to only command line tools.
SSDP does have a private _stop() method, which SsdpClient does not use. Is that due to the fact that its original intended functionality is to be used by SsdpServer to stop advertising and unbinding the socket?

When i am using Ssdpclient.stop() with my own ssdpIp, it is throwing an error as
"cannot read addMembership property of null".
Please help.

Which version does this happen on, @debanshur ?

2.7.2


Add example how users can change kernel list and add custom kernels

The first two examples provided focus on the ease of running simulations from different initialisation fields. That is an important aspect of plasticparcels.
However, the other key feature is for the testing of parameterisations. So we should add an example on how users can change the kernel list, and also add their own kernels

This has been fixed in #32 for varying parameters and creating custom kernels, and in #29 for running simulations from different initialisation maps.


Supply marginLeft/marginRight as inputs to makeVList.

Current code, which modifies the returned elements from makeVList,
is fragile: if makeVList internals changed, the callers would have
to change too.
One place remains where the return value from makeVList is modified, in groupTypes.enclose. That will need to be removed in a future commit, because it's not just changing left/right margins.

@kohler, PR #757 already modifies groupTypes.enclose. It removes the call to makeVList and substitutes a call to new function makeImageVList.

@ronkok, Thanks for letting me know. I am working on an alternative to makeImageVList.

Ping on this e.g. @kevinbarabash? Safe; blocking #768.


Small example code issue

At:
(https://learn.microsoft.com/en-us/azure/active-directory/external-identities/authentication-conditional-access#azure-ad-mfa-reset-proof-up-for-b2b-collaboration-users)
Titled:
"Azure AD MFA reset (proof up) for B2B collaboration users"
Under point 2:
The same generic (non-user-specific) example line of code is given.

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

ID: 14c806ae-ffb3-c2db-b1d9-017a2d2f32ba
Version Independent ID: a8a32962-098a-b767-f413-c467f776dc0d
Content: Authentication and Conditional Access for B2B users - Azure AD - Microsoft Entra
Content Source: articles/active-directory/external-identities/authentication-conditional-access.md
Service: active-directory
Sub-service: b2b
GitHub Login: @msmimart
Microsoft Alias: mimart

@vanboven
Thanks for your feedback! We will investigate and update as appropriate.

@msmimart
Yes, that is correct. In point 2 of the article "Azure AD MFA reset (proof up) for B2B collaboration users", a generic example line of code is provided to illustrate how to reset MFA proof-up for B2B collaboration users. The purpose of this example is to demonstrate the syntax and structure of the code that needs to be used for resetting MFA proof-up for B2B collaboration users.
It is important to note that this code is not intended to be a user-specific example, but rather a generic example that can be customized based on the specific requirements of each individual user. As such, users will need to modify the code to suit their specific needs and requirements.

@vanboven
We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation


[layer] Support reshape layer

This patch provides support for reshape layer and basic unittests.
The flatten layer is also updated to use reshape layer internally.
Signed-off-by: Parichay Kapoor<EMAIL_ADDRESS>

:octocat: cibot: @kparichay, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnstreamer.mooo.com/nntrainer/ci/repo-workers/pr-checker/1645-202110201427210.22398209571838-151920734bf75b449c8d0902ff956a03a8ced2b6/.


Fix WeakReference leak

See https://github.com/DataDog/dd-sdk-android/issues/1762 for details.

Closing because refersTo requires API 33+


checkType appears to be false by default in v2.0.10

Docs are saying this:

checkType: This parameter turns on and off whether check field type. Default is false. (The default is true if version < 1.1.4)

But the Parameters.js file has
checkType: false

my bad


Enable building of tests for NESO

This is just a small PR that makes it optional to build unit tests when installing NESO with spack and which makes Google Test a dependency so that CMake doesn't end up downloading and building it itself.

Note that this shouldn't be merged until after some updates to the NESO CMake have been merged in the main repository.


fix: correct parsing issues with quoted strings

An initial stab at fixing the parsing issues seen in issue #7.

The string parser has been brought up-to-date with Go's particular method of quoting.  Example of how things work now from the output with synthetic data (i.e., not directly in an annotated pod, but using the same file that you would):

Note: when merging, we will want to merge and squash.

For reference on parsing spec:
https://cs.opensource.google/go/go/+/refs/tags/go1.17.3:src/strconv/quote.go;drc=refs%2Ftags%2Fgo1.17.3;l=31

The is_hex_digit() check catches those cases; a nul byte fails the check,
so it'll hit the error path and bail.
On Fri, Nov 19, 2021 at 2:44 PM Chris Bandy @.***>
wrote:

@.**** commented on this pull request.
In parseutils.c
https://github.com/CrunchyData/pgnodemx/pull/8#discussion_r753517501:

		break;

	case 't':

		*dst++ = '\t';

		src++;

		break;

	case 'v':

		*dst++ = '\v';

		src++;

		break;

	case '"':

		*dst++ = '"';

		src++;

		break;

	case 'x':

		/* next 2 chars are hex bytes */

		if (is_hex_digit(src[1]) && is_hex_digit(src[2]))

Should this (and the 'u' 'U' loop below) be more defensive about the
length of src? The last two characters the buffer being \x, for example.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/CrunchyData/pgnodemx/pull/8#pullrequestreview-811656311,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARZX2CDG5V7EX3ADOPK2V43UM2ZKFANCNFSM5IKP4Z6Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Add: Canal Showsport

Channel Name
Canal Showsport
Alternative Names (optional)
No response
Network (optional)
No response
Owners (optional)
No response
Country
AR
Subdivision (optional)
No response
City (optional)
Corbdoba
Broadcast Area
c/AR
Languages
esp
Categories (optional)
sports
NSFW
FALSE
Launched (optional)
No response
Closed (optional)
No response
Replaced By (optional)
No response
Website (optional)
https://canalshowsport.com.ar/
Logo
https://canalshowsport.com.ar/wp-content/uploads/2021/11/2021-11-logo-SS-HEADER-blueE.png
Notes
No response

https://iptv-org.github.io/channel?id=CanalShowsport.ar


Update last-week-on-osm -> latest-changes

Moved latest-changes to "Changeset Tools"
Updated description
Changed repository tyrasd --> rene78 (more up-to-date)

Thanks for the feedback! Makes sense. I have just moved it to the end of the list.

Looking good! Thanks for helping with keeping this list up to date 🥇


gdu with ssh won't close, 100 percent cpu usage

GDU v5.25.0
Ubuntu LTS 22.04
Mainline kernel 6.3
This is a weird and somewhat random issue that I have seen lately.  I run gdu on my server which I control with ssh.  Sometimes when logging in and using htop, I see gdu running and consuming 100% of a single core. It's difficult to replicate, I have tried closing the ssh window without closing gdu first, but so far gdu always closes when I try this. I have no idea why it's running sometimes when no ssh connection is active, but I have seen this multiple times.

Is it possible that gdu is started by crond?

There's no reason why it would be. Have not seen this happen lately, it's quite rate.

Most likely exactly the same issue as ncdu had (although I've seen it happen anyway still)
https://code.blicky.net/yorhel/ncdu/issues/141

@Crowley007 did you ever figure this out?
I just read your issue and noticed a couple things in your report:

"on my server which I control with ssh"
"it's running sometimes when no ssh connection is active"

I am not certain how to make sense of those statements. Wouldn't you be seeing the behavior over an active ssh connection? If that is the case, you may want to take a look at the scripts that execute when you login. One is likely .bashrc or .bash_profile in your home directory. Examine the contents of those files to see if there isn't something calling gdu.
Further investigations could be done by checking the logs.
Based on the link posted by @borestad I am also wondering if both of you are running gdu as root.
I think that gdu is a great tool and would like to keep the discussions and development going.

Solution for ncdu was most likely this: https://code.blicky.net/yorhel/ncdu/commit/888bd66
I will check if we can introduce somethink similar.

My workaround the last couple of months has been this bash function that sets a timeout (and fixes the broken terminal)
gdu () {
	timeout --foreground -s SIGTERM 10m $HOMEBREW_PREFIX/Cellar/gdu/*/bin/gdu-go -x -a -i "$HOME/Library/Mobile Documents" "$@" || (
		stty sane &> /dev/null
		tput reset &> /dev/null
		clear
		echo "[timeout]: 10m gdu"
	)
}

I hope this was it.


CAS Auth support

I've been quite liking testing Zulip for an open source project I help manage the backend of, and I've been liking it so much that I'm considering introducing it at work, a not-for-profit college, however all of their auth goes through, as seemingly with most colleges in my experience if they have SSO at all, the CAS protocol, which is very similar to oauth/oidc but older and less well thought out.  CAS does not seem to be in python-social-core, however it is both quite simple to implement (especially if prior oauth/oidc experience, it is utterly trivial compared to those) and there is a variety of existing python libraries for it from django-cas to (the seemingly biggest and most used one) https://github.com/python-cas/python-cas.
If I were to look into adding support for CAS, would it be better to look into adding it into python-social-core or into zulip straight by using python-cas or...?

@OvermindDL1 I would recommend adding it to python-social-auth; that way, the benefits of the integration are available to other apps using that library.  And I think it's probably a similar amount of work; if you look at the architecture of a social_auth backend, it's usually just a few classes and some tests (And they have a nice test framework).
The other approach you could take is combining https://github.com/apereo/mod_auth_cas with https://zulip.readthedocs.io/en/latest/production/authentication-methods.html#apache-based-sso-with-remote-user, which would be a bit more configuration work (and less reusable) but should let you setup CAS auth with Zulip 2.0.2.

I would recommend adding it to python-social-auth; that way, the benefits of the integration are available to other apps using that library.

Very good point.  ^.^

The other approach you could take is combining https://github.com/apereo/mod_auth_cas with https://zulip.readthedocs.io/en/latest/production/authentication-methods.html#apache-based-sso-with-remote-user, which would be a bit more configuration work (and less reusable) but should let you setup CAS auth with Zulip 2.0.2.

Is there a description of the on-wire API for that so I could whip up an nginx setup for it instead, they don't use apache here except in one very old place they are trying to get rid of.
I also just got to thinking about the details, the CAS server here doesn't supply an email (it's a very old protocol) but rather a string ID that needs to be looked up elsewhere for further information.  I already have a server built that has the needed information however and it delegates to the CAS server for initial login before looking up the needed information, I could easily have it act as a proxy.  If there is information on how the remote_user bit is actually implemented then I could have my server emulate that functionality and return the expected information.  :-)

That system just uses the Apache REMOTE_USER variable in the WSGI request to zproject/wsgi.py as its source of truth.
And the model is to post to /accounts/login/sso/ with that variable set.  It's not an HTTP header; those get encoded as e.g. HTTP_REMOTE_USER.

I'm unfamiliar with the (HTTP_)REMOTE_USER header, do you have a link to the standards spec for that, I seem to be unable to find one?  (Only thing I recall about anything related to REMOTE_USER is an old CGI var, though that's a long dead spec at this point and it had quite a share of issues in many implementations.)

And the model is to post to /accounts/login/sso/ with that variable set.

I'm guessing that the (HTTP_)REMOTE_USER is validated some way so that not just any incoming connection can post whatever they wish to it and get magically logged in?  Is it encrypted with a shared key?  Is it able to hold other data than just a singular value like an email (such as email, username, etc...)?
My setup is this, the server that can both perform CAS Auth and lookup a corresponding email address, a user can have multiple assigned email addresses and any one of which can be supplied as a login auth so any of them must be able to allow authentication, the server can supply the list of valid emails for a given user along with the rest of the user's auth (including the email they logged in with).  The auth with such a server would work similar to CAS, I could even implement it 'as' CAS, but giving back an email as the ID in addition to whatever other requested information, such as the list of valid emails associated with the user, different emails are used by a user depending on how and where they are logging in from on campus so it kind of corresponds to their 'current' group, although there is a groups field as well that is a list of what areas they are in and have access to (like IT, Financial Aid, etc...), all of which can be supplied via the CAS protocol as well as it would be fairly easy to implement on that server as it has all the bulk work done already (it's already an integration system between CAS and a dozen other things, adding one more is simple).
The server itself is a custom server written in a low level language (it does report generation as well as a few other CPU heavy tasks initially so it was originally built with efficiency in mind, right now its the fastest thing on campus) with complete control over its socket, so I can send and craft whatever is needed over the network.
So if all I need to do it post to that route with that variable set (in the post data I'm guessing?), then all I'd need is docs about the precise format it needs and expects and how to secure that data so no random person can just log in as they wish (shared key I'm guessing? as long as it's not something horribly insecure as IP address matching or so that's trivial to break on an internal network like this).
I'm wondering again if just implementing CAS on the server itself 'as' a CAS server that just delegates to the other one might be best, hmm...

The key detail for security is that it's not an HTTP header; it's passed in the WSGI request (I think via the environment?).  See for example these docs: https://wsgi.readthedocs.io/en/latest/specifications/simple_authentication.html https://modwsgi.readthedocs.io/en/develop/user-guides/access-control-mechanisms.html

As it is an entirely different server altogether I'm unsure how I could set the environment in another server from that?  Everything runs in docker here, so everything is containerized to begin with (zulip's docker container seems to run quite well, thanks for that!), but even then it is still different physical servers regardless (that server is one of very very few that have access to the backing oracle database).  I'm not really sure how that could be used, especially within a docker environment as docker controls the environment statically on load.  Having this server become a CAS bouncer might be the best task for me then, something else for me to do after spring break next week (assuming I'm not dragged away from home too much by my wife then I'll try to work on getting CAS into python social core).  ^.^

Yeah, if you're CAS server runs on another machine, you can't use REMOTE_USER as your security model.

Heh, not just on another machine, but a few different ones.  The main CAS server gets daily syncs from the backing oracle database, but that CAS server is hosted off-site (a collection of college's shared setup), that CAS server only gives the internal backing ID (PIDM) of the accounts, so when it comes back into our network then we have to look up into our backing oracle database to get more information, of which it is not accessible to almost everything (that's one of the points of my integration server, to act as a central point of truth for all the distinct areas) so that used to bounce through a couple other servers and scripts before my server was setup to centralize it all.
It's an interesting mess that is my job in getting cleaned up.  ^.^
However, does any of the supported auth's in zulip support even a proprietary format, like how the Discourse forums have their own proprietary interface for a 'low level remote auth' (which uses a shared key to encrypt the secrets)?
I'm actually kind of tempted to setup an OIDC/OAuth2 server and wrap CAS behind that here, it would have a lot more uses and it's not the first time I've done something like that...

Just a quick note that we recently added SAML support, which doesn't address this but may be relevant for folks using auth providers that have both interfaces.


Update set-up-mfa.md

[exclude the Azure Windows VM Sign-In app] Hyperlinked page didnt have explanation on how/where to exclude the app in conditional access policy. Adding a link to a page where it does explain about it.

@ravibak : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@Heidilohr
Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@heidilohr @femila - Can you review this old PR and determine whether it needs to be closed or merged?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.

#sign-off


Save Failure With Status Code 502

Everytime my project saving, it always return 502 and I can't continue my work now

I have the same problem and I think it's occurring because of the issue described in #35

Doesn't save - produces 502
When I open it in a separate tab, it shows the old save.

Doesn't save - produces 502
When I open it in a separate tab, it shows the old save.

Related this issue.
https://github.com/didin1453fatih/dbdesigner.id/issues/35


[Removal]: Please check and remove my hotspots from Denylist(2/6)

Hotspot Name
Lucky Quartz Okapi
Hotspot b58 Address
112UNmdN827AzL8LvfJEpyiM4xSR9qvddjARJQmg5p958ANqNsCH
Discord Handle
kuka#3163
Hotspot Manufacturer
Heltec Automation
Removal Reason
My Router‘s bandwidth was updated, and then my Hotspots were denied, please check them again. Thank you.
Modifications
No
Extra forwarders
No
Extra antennas
No
Additional Information

We have taken into account this removal request in the next iteration of the denylist. It may take some time before it's processed. This is an automated comment due to the volume of addition and removal requests. A future tool is in the works to provide more insight into the analysis approach made by the Helium team and community members that maintain this list.


Don't recommend using formats that aren't widely supported.

Welcome! Please use this template for reporting documentation issues on https://developers.google.com/web/.
For questions about Chrome DevTools, checkout the
google-chrome-devtools tag on StackOverflow.
For questions about Chrome, check the google-chrome
tag. If you think you've found a Chrome bug, check out our
feedback page for more details on how to file it.
Page Affected: https://developers.google.com/web/TODO
What needs to be done?
Please provide description of what's needs to be done here. Please be sure to include the link to the original page
and any supporting links/data.

[ ] Appropriate type-something label has been added.
[ ] Appropriate section-something label has been added.

Same question here: #6773 (comment)


Gltf support

Contains
Completes PR #3785 by rebasing the engine to support BiomesAPI.
How to test
Create a world with the BiomesAPI module activated
To check gltf support run command: spawnPrefab "engine:cubesm"
Outstanding before merging

[ ] Convert old models from .md5 to .gltf

Can one of the admins please verify this patch?

This one did the trick! Thanks @Kanonbell :-)
Going to edit the GCI task to point to this PR and also leave a note on the old PR
@GooeyHub: add to whitelist

Hooray Jenkins reported success with all tests good!

Any changes to the heightOffset parameter in the skeletalmesh component don't seem to have any effect on the height of the in-game model. The parameter works fine in the main develop branch, but doesn't work on the gltf branch. Can someone else confirm this issue, just in case it may be an error happening with me locally?

It is entirely possible I broke it along the way

Hooray Jenkins reported success with all tests good!

Found a possible solution for both floating and rotation that needs confirmation. This PR + Behaviors, PR-23 + WildAnimals, PR-46 + changing prefab heightOffset from -0.8 to 0.8

Old PR got rebased with an additional bug fix commit added! And this one got a merge commit added which duplicated the existing commits :-( So closing this one in favor of the old one again - still appreciate your efforts on this @Kanonbell !
Thanks for the bug hunting @casals !


Questions About Attestation Certificate

Is there a standard process for obtaining or generating the attestation certificate?
Just get in to Java Cards, mainly want use it for login to windows lol

You can use openssl to generate your own attestation certificate. It's a X.509 cert. The only thing you have to mention is to use the NIST P256 curve.
Here is the openssl output for the example certificate I used in this projekt:
openssl asn1parse -inform DER -in FIDO-attestation_cert.der
0:d=0  hl=4 l= 316 cons: SEQUENCE
    4:d=1  hl=3 l= 228 cons: SEQUENCE
    7:d=2  hl=2 l=   3 cons: cont [ 0 ]
    9:d=3  hl=2 l=   1 prim: INTEGER           :02
   12:d=2  hl=2 l=  10 prim: INTEGER           :47901280001155957352
   24:d=2  hl=2 l=  10 cons: SEQUENCE
   26:d=3  hl=2 l=   8 prim: OBJECT            :ecdsa-with-SHA256
   36:d=2  hl=2 l=  23 cons: SEQUENCE
   38:d=3  hl=2 l=  21 cons: SET
   40:d=4  hl=2 l=  19 cons: SEQUENCE
   42:d=5  hl=2 l=   3 prim: OBJECT            :commonName
   47:d=5  hl=2 l=  12 prim: PRINTABLESTRING   :Gnubby Pilot
   61:d=2  hl=2 l=  30 cons: SEQUENCE
   63:d=3  hl=2 l=  13 prim: UTCTIME           :120814182932Z
   78:d=3  hl=2 l=  13 prim: UTCTIME           :130814182932Z
   93:d=2  hl=2 l=  49 cons: SEQUENCE
   95:d=3  hl=2 l=  47 cons: SET
   97:d=4  hl=2 l=  45 cons: SEQUENCE
   99:d=5  hl=2 l=   3 prim: OBJECT            :commonName
  104:d=5  hl=2 l=  38 prim: PRINTABLESTRING   :PilotGnubby-0.4.1-47901280001155957352
  144:d=2  hl=2 l=  89 cons: SEQUENCE
  146:d=3  hl=2 l=  19 cons: SEQUENCE
  148:d=4  hl=2 l=   7 prim: OBJECT            :id-ecPublicKey
  157:d=4  hl=2 l=   8 prim: OBJECT            :prime256v1
  167:d=3  hl=2 l=  66 prim: BIT STRING
  235:d=1  hl=2 l=  10 cons: SEQUENCE
  237:d=2  hl=2 l=   8 prim: OBJECT            :ecdsa-with-SHA256
  247:d=1  hl=2 l=  71 prim: BIT STRING

openssl x509 -inform DER -in FIDO-attestation_cert.der -text
Certificate:
    Data:
        Version: 3 (0x2)
        Serial Number:
            47:90:12:80:00:11:55:95:73:52
        Signature Algorithm: ecdsa-with-SHA256
        Issuer: CN = Gnubby Pilot
        Validity
            Not Before: Aug 14 18:29:32 2012 GMT
            Not After : Aug 14 18:29:32 2013 GMT
        Subject: CN = PilotGnubby-0.4.1-47901280001155957352
        Subject Public Key Info:
            Public Key Algorithm: id-ecPublicKey
                Public-Key: (256 bit)
                pub:
                    04:8d:61:7e:65:c9:50:8e:64:bc:c5:67:3a:c8:2a:
                    67:99:da:3c:14:46:68:2c:25:8c:46:3f:ff:df:58:
                    df:d2:fa:3e:6c:37:8b:53:d7:95:c4:a4:df:fb:41:
                    99:ed:d7:86:2f:23:ab:af:02:03:b4:b8:91:1b:a0:
                    56:99:94:e1:01
                ASN1 OID: prime256v1
                NIST CURVE: P-256
    Signature Algorithm: ecdsa-with-SHA256
         30:44:02:20:60:cd:b6:06:1e:9c:22:26:2d:1a:ac:1d:96:d8:
         c7:08:29:b2:36:65:31:dd:a2:68:83:2c:b8:36:bc:d3:0d:fa:
         02:20:63:1b:14:59:f0:9e:63:30:05:57:22:c8:d8:9b:7f:48:
         88:3b:90:89:b8:8d:60:d1:d9:79:59:02:b3:04:10:df
-----BEGIN CERTIFICATE-----
MIIBPDCB5KADAgECAgpHkBKAABFVlXNSMAoGCCqGSM49BAMCMBcxFTATBgNVBAMT
DEdudWJieSBQaWxvdDAeFw0xMjA4MTQxODI5MzJaFw0xMzA4MTQxODI5MzJaMDEx
LzAtBgNVBAMTJlBpbG90R251YmJ5LTAuNC4xLTQ3OTAxMjgwMDAxMTU1OTU3MzUy
MFkwEwYHKoZIzj0CAQYIKoZIzj0DAQcDQgAEjWF+ZclQjmS8xWc6yCpnmdo8FEZo
LCWMRj//31jf0vo+bDeLU9eVxKTf+0GZ7deGLyOrrwIDtLiRG6BWmZThATAKBggq
hkjOPQQDAgNHADBEAiBgzbYGHpwiJi0arB2W2McIKbI2ZTHdomiDLLg2vNMN+gIg
YxsUWfCeYzAFVyLI2Jt/SIg7kIm4jWDR2XlZArMEEN8=
-----END CERTIFICATE-----

Hello, I would like to know how to get the raw hexadecimal form of the certificate - is it the whole ASN structure or some part within? I am stuck with .der certificate, unable to find out what format the applet expects...

Oh, its just the DER representation... I was a bit confused because mine had like two times the size of example certificate, but it had only more data...

Here is a quick no-frills install script for Global Platform Pro: https://gist.github.com/darconeous/adb1b2c4b15d3d8fbc72a5097270cdaf
Installs the attestation cert, too. It just uses the original Google one from the spec.

Yeah its's DER encoded and nice install script. This eliminates the "scriptor" tool.  This is perfect because it is not available for Windows.

Hi Tobias,
you can even shorten this when re-factoring the method to correct extended length handling:
private void handleSetAttestationCert(final APDU apdu) throws ISOException {
        final byte[] buffer = apdu.getBuffer();
	short len = apdu.setIncomingAndReceive();

        final short copyOffset = Util.makeShort(buffer[ISO7816.OFFSET_P1], buffer[ISO7816.OFFSET_P2]),
			ioSz = apdu.getIncomingLength();

        if(copyOffset == 0 && ioSz == attestationCertificate.length) {
		len = Util.arrayCopyNonAtomic(buffer, apdu.getOffsetCdata(), attestationCertificate, (short) 0, len);
		while (len < ioSz) {
			len = Util.arrayCopyNonAtomic(buffer, (short) 0, attestationCertificate, len, apdu.receiveBytes((short) 0));
		}
            attestationCertificateSet = true;
        } else {
		if ((short) (copyOffset + len) > attestationCertificate.length) {
			ISOException.throwIt(ISO7816.SW_WRONG_DATA);
		}
	        Util.arrayCopy(buffer, apdu.getOffsetCdata(), attestationCertificate, copyOffset, len);
		if ((short) (copyOffset + len) == attestationCertificate.length) {
			attestationCertificateSet = true;
		}
        }
}

to
00A4040008A0000006472F000100
800900000001403082013C3081E4A003020102020A47901280001155957352300A06082A8648CE3D0403023017311530130603550403130C476E756262792050696C6F74301E170D3132303831343138323933325A170D3133303831343138323933325A3031312F302D0603550403132650696C6F74476E756262792D302E342E312D34373930313238303030313135353935373335323059301306072A8648CE3D020106082A8648CE3D030107034200048D617E65C9508E64BCC5673AC82A6799DA3C1446682C258C463FFFDF58DFD2FA3E6C378B53D795C4A4DFFB4199EDD7862F23ABAF0203B4B8911BA0569994E101300A06082A8648CE3D0403020347003044022060CDB6061E9C22262D1AAC1D96D8C70829B2366531DDA268832CB836BCD30DFA0220631B1459F09E6330055722C8D89B7F48883B9089B88D60D1D9795902B30410DF

This is also part of the @c-base implementation variant including similar refactorings to Roberts.
Best regards & greeting to BN,
Christian


Attachment problem: read 2 bytes more data than requested

Hello!
There is an attachment in the email which can not be saved properly using the mail-mime-parser library. I try to do it the following way:
$handle = $att->getContentResourceHandle();
while (!feof($handle)) {
    $part = fread($handle, 1024);
    // ...
}
I use loop in order to control the size of the attachment and it works in the general cases. But fread() throws the following error on the special file:

fread(): GuzzleHttp\Psr7\StreamWrapper::stream_read - read 2 bytes more data than requested (8194 read, 8192 max) - excess data will be lost.

This issue can be reproduces using 1.1* and 1.2.0 versions. The example of the email, also the raw original attachment are in the zip file. Can you please look at this and advise something?
Thank you in advance!
example.zip

Hi @beldeveloper,
Looking at this, I can reproduce the warning as well using fread, but writing the stream out still seems to be okay and the file doesn't look corrupted or anything.  Can you confirm that?
I also noticed that using Psr7 streams doesn't produce a warning... changing your code like so:
$stream = $att->getContentStream();
while (!$stream->eof()) {
    $part = $stream->read(1024);
    // ...
}

But it may still indicate an underlying issue, and I'll have to dig deeper to see what's happening and what's causing it.

Hi @beldeveloper,
It turns out the issue is related specifically to fread.  Because the function returns 'bytes', and conversion to utf-8 in zbateson/stream-decoders returns chars, calling for example fread($handle, 1) will read 1 byte, and not 1 multibyte character for example.
For this reason I've deprecated usage of getContentResourceHandle, getTextResourceHandle and getHtmlResourceHandle.  These should not be used to read character streams.  Instead getContentStream, getTextStream and getHtmlStream should be used.
I've also found a bug reading UTF-16LE data while investigating this.  I'm not entirely sure what the underlying problem was that was causing it, but aligning my byte reading to a 4-byte boundary seems to fix it.  This didn't appear to be an issue for non-LE multibyte streams as far as I could tell... but as I don't really know the underlying issue there may be more to the bug.  My testing is pretty comprehensive though and no longer causes an error, so I'm hopeful the alignment is all that was needed.  This is released in zbateson/stream-decorators 1.0.3.
The actual deprecation in code will be a 1.2.1 release, but I might delay the actual release for some additional fixes as there's no specific change to code beyond marking the deprecated methods.
All the best.

Hello @zbateson
Really appreciate your quick help!


ListBox with SelectionMode=Extended issue

When I have list box with multiple items selected and I click on a item that is part of the selection, the item is not selected and multi selection is lost. Default behavior of ListBox is to select clicked item. Also, if you have buttons inside the listboxitem they also don't work.

@ekalchev Can you provide me more information?
PLEASE READ BEFORE POSTING AN ISSUE
Seriously consider creating and linking to a test repo which takes absolutely minimal setup to illustrate how reproduce the problem.
My time is limited and .Zip files, code snippets and partial examples are often a time sink. GitHub also provides great communication and code review tools which can be utilised.
HOW TO POST A GOOD SAMPLE.
Follow these guidelines and I will most likely look at the issue sooner:

Post a full GitHub repository. Not a zip file, half baked snippet etc. If GitHub is new to you consider it a great learning opportunity and chance to get involved.
The repository should have just ONE step max for me to get running, and that is "Restore NuGet Packages". If there are any other missing dependencies, or uncompiling features I will most likely move on to another issue.
I'm not trying to be awkward. I'm just busy and I'm helping a lot of people, not just you, so help me out and I will help you out.

What steps will reproduce this issue?
Write problem description here
Expected outcome
Write expected outcome here
Repo
Link to the Repo
Environment

GongSolutions.WPF.DragDrop v?.?.?
Windows OS ??
Visual Studio 20xx
.NET Framework ?.?

I am using latest version of GongSolutions.WPF.DragDrop
Visual Studio 2015
.NET Framework 4.6
I can see the problem is here
if (itemsControl is MultiSelector) {
((MultiSelector)itemsControl).SelectedItem = null;
((MultiSelector)itemsControl).SelectedItem = item;
} else if (itemsControl is ListBox) {
var selectionMode = ((ListBox)itemsControl).SelectionMode;
try {
// change SelectionMode for UpdateAnchorAndActionItem
((ListBox)itemsControl).SelectionMode = SelectionMode.Single;
((ListBox)itemsControl).SelectedItem = null;
((ListBox)itemsControl).SelectedItem = item;
}
SelectedItem = null
is setting the selected item to null
but
.SelectedItem = item
but this line is not setting SelectedItem to Item. I can see Item has the correct value but it is not getting set to SelectedItem. Probably some check inside SelectedItem setter is preventing this. May be related to WPF 4.5 changes for the list box?

The problem is not in GongSolutions.WPF.DragDrop. I have custom implementation of ICollectionView and Contains method was not implemented which was causing this side effect. Selection now works but I still struggle with GongSolutions.WPF.DragDrop code that handles clicks when there is a multiple selection. Handling mouse down events for the selected items prevent me from having commands trigger for the buttons inside the selected treeview items. Any idea how I can achieve that with GongSolutions.WPF.DragDrop?

This lib does not interfere with click events regardless of if your listbox uses multi selection or not.
As punker suggested, having a repo showcasing your issue would help a lot in supporting you with your issue.

@ekalchev as @Insire wrote, a small app which demonstrates your issue would be nice. Cause multiple selection should work as you can see and test in the main showcase app. also having buttons or combobox or textbox inside items is also possible.

@ekalchev Any news for a repo app?


[Darwin] Add external plugin paths into platform specified by PLATFORM_DIR

The driver currently has logic to infer the location of the platform based on the conventional layout of SDKs within platforms. However, that doesn't work well when SDKs have been built up in other places.
Introduce support for a second convention, that the PLATFORM_DIR environment variable can specify the path to the platform. When present, also add external plugin paths pointing into that platform.
Fixes rdar://138370421.

@swift-ci please test

@swift-ci please test

@swift-ci please test Windows


Changes 31st Jan 2023

Add NcMethodResultError response for when errors are encountered (this also contains the errorMessage moved from the base class). Add guidance in IS-12 documentation to indicate which MethodResult to send when there is an error condition or if the transaction was successful.
All roles are now NcString
NcNamePath is now NcRolePath of sequence
Added 2 more options to NcResetCause (Unknown and ManualReset)
Added 3 more options to NcDeviceGenericState (Unknown, LicensingError and InternalError)
in NcDeviceManager - userInventoryCode, deviceName and deviceRole and now nullable
in NcObject - userLabel is now nullable

Reviewed on the call on the 7th of February 2023.


mobile: Update pom.xml with protobuf-javalite transitive dependency

This is to allow depending on io.envoyproxy.envoymobile:envoy-xds without having to manually add com.google.protobuf:protobuf-javalite dependency.
Risk Level: low
Testing: CI
Docs Changes: n/a
Release Notes: n/a
Platform Specific Features: n/a

/assign @abeyad


Minor UI fixes

As a start, some minor UI fixes:

Removed flickering of folder tree and file list with double buffering. The Win32 controls do support it but it isn't exposed to WinForms. Hence reflection and platform invoke..
Added missing localization entries for the xbin editor. Always with the English text as placeholder. A proper translation is needed.

Nice changes to Game Explorer. Do you advise adding these UI fixes to other editors with long containers (such as map editor) or was it just an issue with the Game Explorer?

Yes, especially tree views and list views in resizable windows benefit greatly. I can make the fix for all of them if you'd like.
Can I open the map editor for M1DE as well? I haven't found it yet.

Sadly no. :(
It's only for Mafia II. You can open it by double clicking a "FrameResource_X.fr" file. In the future I hope to change this with a much more visible approach such as "Open Map Editor", then you can select which SDS to load.

I applied the changes to the rest of the tree views. Can't check if it had a visible effect since I only have M1DE for testing.
The XBIN editor did not benefit at least. Your decision whether/what to merge.

There is actually another significant list here -
Forms/Docking/DockSceneTree.Designer.cs
This uses a tree view for all loaded objects. After that one has been double buffered, I'll merge it.

Since I'm not really a fan of manually setting each control to double buffered, i went the inheritance way. Conveniently there is already a custom TreeView which i extended. I'm closing this PR now. My follow up PR is much cleaner: https://github.com/Greavesy1899/MafiaToolkit/pull/35