[ENHANCEMENT] Block pip packages that are installed by conda

Please confirm these before moving forward

[X] I have searched for my feature proposal and not found a work-in-progress/duplicate/resolved/discarded issue.
[X] This improvements refers to an existing feature. If you want to suggest a new feature, please use this template
[X] This improvements is not a bug. If you want to report a bug, please use this template

Describe the improvement
Skip packages installed by conda when updating packages, or add support for installing and updating conda packages.
Describe how this improvement could help users.

Some conda packages are updated later than pypi, WingetUI often indicates that many packages need to be updated.
Anaconda navigator loads slowly and is difficult to use.

If you don't want to update certain packages, you can skip updates for them.
Is there any way to detect packages from conda?

Sorry, I knows little about Python/Conda.
But this might help
Conda is not only a package management system but also an environment management system.
So supporting multiple virtual environments of conda/pip may be a must.


Cleanup of translate package

@ostromart: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-lint.sh
81d35b8582662378fbbe69967becfc2945937a4f
link
/test operator-lint-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Also need to update this: https://github.com/istio/operator/blob/master/pkg/component/component/component.go#L449


ClientSecretCredential authentication failing

Describe the bug
After deploying the code via the Azure CLI, and updating my SaaS technical details, I get the following error in the UI, when I subscribe and click on the Configure Account Now button: "Error 500 : ClientSecretCredential authentication failed: Value cannot be null. (Parameter 'clientSecret')". I am authenticated, and am able to access the empty subscriptions view.
To Reproduce
Steps to reproduce the behavior:

Deploy application via Azure CLI.
Update SaaS technical details and republish.
Add a new subscription via Azure Portal.
Click on the Configure Account Now button.

Expected behavior
Expect a landing page with details of the subscription and a link to complete the subscription.
Screenshots

Desktop (please complete the following information):

OS: Mac OS
Browser Chrome

@selwynoh Can you tell me what you did to fix this? I'm having the same issue. Thanks :)


Fix: Type Fixes

Summary of changes
Change to cleanup typo and grammar issues
Changes introduced in this pull request:

Reference issue to close (if applicable)

Closes
Other information and links

Change checklist

[ ] I have performed a self-review of my own code,
[ ] I have made corresponding changes to the documentation. All new code adheres to the team's documentation standards,
[ ] I have added tests that prove my fix is effective or that my feature works (if possible),
[ ] I have made sure the CHANGELOG is up-to-date. All user-facing changes should be reflected in this document.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks. This documentation is no longer maintained as it will be (soon) replaced by something different.


Add class balance checking code

This PR contains code that was used to calculate how balanced the calorie labels (classes) are in the data set with our current thresholds. The results are as follows:

Low-calorie: 25%
Average-calorie: 63%
High-calorie: 12%

For posterity, the current F1 Score for each category (as calculated by the recently pushed code) is roughly as follows:

Low-calorie: 0.81
Average-calorie: 0.90
High-calorie: 0.65

Seems to correlate pretty well with representation in the data set.

In addition, the confusion matrix generated by the recently pushed code looks roughly like the following:

Predicted Low-Calorie
Predicted Average-Calorie
Predicted High-Calorie

Actual Low-Calorie
10765
2437
52

Actual Average-Calorie
2065
32897
979

Actual High-Calorie
74
2100
2977


[SPARK-50580][BUILD] Upgrade log4j2 to 2.24.3

What changes were proposed in this pull request?
The pr aims to upgrade log4j2 from 2.24.2 to 2.24.3.
Why are the changes needed?

The full release notes:
https://github.com/apache/logging-log4j2/releases/tag/rel%2F2.24.3

Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass GA.
Was this patch authored or co-authored using generative AI tooling?
No.

Is it ready, @panbingkun ?

Is it ready, @panbingkun ?

Yeah, thanks!

@HyukjinKwon Since 2.24.3 includes the following fix:

https://github.com/apache/logging-log4j2/issues/3234
https://github.com/apache/logging-log4j2/pull/3235

I believe we can try this change again: https://github.com/apache/spark/pull/48966


Do not write to superglobals

We did already improve the situation a tiny bit for 1.0, though there are still so many places where we write to superglobals...
Let's see how many.
Using this regex:
\$_(POST|REQUEST|GET)\[('[a-zA-Z0-9\-_]*'|('[a-zA-Z0-9\-_]*' \. )*\$[a-zA-Z0-9\-_]*( \. '[a-zA-Z0-9\-_]*')*)\]\s*[\+\-\/\*]*={1}(>)*\s

I found 470+ occurrences. :fearful: :scream_cat:
I was going to propose this to be a 1.1 blocker, but considering that number it's probably better wait until we have done some work on it before say anything...

470+ => 241 now, and some of those are not ours :dancer:

Stuff like $this->_req->query->tid = (int) $this->_req->query->tid; is still writing to a superglobal :(

It's not that the code is always straightforward enough to just scrap all the writings. ;)

Yup, lots to do, lots to do,  and technically I believe if you were to go look at $_GET['tid'] its not an int after that statement runs?


Set column minimum width to consider text width "[NULL]" when presenting data in text grid

DBeaver - Version 5.1.4 CE
Operating System: Windows 10
Set column minimum width to consider text width "[NULL]" when presenting data in text grid.
Today if a column has a character width less than what is needed to present the "[NULL]" characters, the columns are shifted to the right.
Looks something like this:
|Unr                                  |ClinicId |Sign |PersonId |
|-------------------------------------|---------|-----|----------|
|A957FAFF-FF14-4BD4-A994-0521767B1C70 |1        |ab   |1         |
|43F5ECFF-2403-4F84-A32F-92825BC9AD2B |1        |[NULL]|2         |
|8600CCFF-6941-4D21-9163-129E18C00E7D |1        |ab   |3         |
|D7F97F00-43C5-4E27-93A2-C72A6031B91F |1        |ab   |[NULL]    |

If DBeaver property Result Sets -> Presentation -> Plain text -> Show NULLs is checked, then all (or those columns that have type that allow NULL) columns should have a minimum width where the "[NULL]" characters fit.

Thanks for bug report


Describe how to remove an app from the catalog

The page says how to upload and update an app, but not how to remove it.

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

ID: fbbcabee-fa96-2878-e983-3c966f84ee15
Version Independent ID: 02fa4fd8-77e0-5cb2-a4af-576cbb68322f
Content: Manage your apps in the Microsoft Teams admin center - Microsoft Teams
Content Source: Teams/manage-apps.md
Service: msteams
GitHub Login: @LanaChin
Microsoft Alias: v-lanac

@aosolis Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@aosolis thank you for reaching out and helping improve MS Docs. You should be able to right-click on the App and then "Uninstall". It should also be possible from the "More apps" menu on the lefthand side. Please let me know if you don't come right.

@lindspea, thanks, can we update the documentation with that information?

@aosolis Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.

I re-read the documentation and I was actually asking about removing an app from the tenant catalog, not uninstalling an app from Teams. This section: https://docs.microsoft.com/en-us/MicrosoftTeams/manage-apps#upload-a-new-app describes how to

upload an app "To upload a new custom app to your tenant app catalog..."
update an app "To update an app after it's uploaded..."
but not how to remove it.


Container created but Icons are not

This is a weird problem.
Windows Server 2019
latest New-NavContainer or New-BCContainer
On the user who installed the Docker and NavContainerHelper's login the software works fine.
On other users who are on the server, administrators of the local server, the containers are created but the script fails with an error creating the icons.
Same script for both users - different results.  Tried running from Elevated Powershell ISE.  Any suggestions?
Script is

PS C:\Windows\system32> #$credential = get-credential
$licensefolder = "D:\Development\Licenses"
$mylicense = "BC14 On Prem - Sept 11 - 2019.flf"
$imageName = "mcr.microsoft.com/businesscentral/onprem:cu4-na-ltsc2019"
$containerName = 'bcsndbx14'
$mylicense = join-path $licensefolder $mylicense
New-BCContainer  -accept_eula -containerName $containerName -imageName $imageName -assignPremiumPlan -updateHosts -auth NavUserPassword -includeCSide -enableSymbolLoading -licenseFile $mylicense

See output of the New-BCContainer script

NavContainerHelper is version <IP_ADDRESS>
Host is Microsoft Windows Server 2019 Standard - ltsc2019
Docker Client Version is 19.03.3
Docker Server Version is 19.03.3
Using image mcr.microsoft.com/businesscentral/onprem:cu4-na-ltsc2019
Removing container bcsndbx14
Removing bcsndbx14 from hosts
Removing C:\ProgramData\NavContainerHelper\Extensions\bcsndbx14
Creating Container bcsndbx14
Version: 14.5.35970.0-na
Style: onprem
Platform: 14.0.35916.0
Generic Tag: <IP_ADDRESS>
Container OS Version: 10.0.17763.678 (ltsc2019)
Host OS Version: 10.0.17763.805 (ltsc2019)
WARNING: The container operating system matches the host operating system, but the revision is different.
If you encounter issues, you might want to specify -isolation hyperv
Using locale en-US
Using process isolation
Disabling the standard eventlog dump to container log every 2 seconds (use -dumpEventLog to enable)
Using license file D:\Development\Licenses\BC14 On Prem - Sept 11 - 2019.flf
Files in C:\ProgramData\NavContainerHelper\Extensions\bcsndbx14\my:

AdditionalOutput.ps1
AdditionalSetup.ps1
license.flf
MainLoop.ps1
SetupNavUsers.ps1
updatehosts.ps1
Creating container bcsndbx14 from image mcr.microsoft.com/businesscentral/onprem:cu4-na-ltsc2019
28aedc44f8d4e3ee345aa1f88842daf6ee6f18368316e4ffa247f1f0210f04d9
Waiting for container bcsndbx14 to be ready
Initializing...
Starting Container
Hostname is bcsndbx14
PublicDnsName is bcsndbx14
Using NavUserPassword Authentication
Starting Local SQL Server
Starting Internet Information Server
Creating Self Signed Certificate
Self Signed Certificate Thumbprint 3480D34956352926CA1309C9E32FECFDEB36505F
Modifying Service Tier Config File with Instance Specific Settings
Starting Service Tier
Registering event sources
Creating DotNetCore Web Server Instance
Using license file 'c:\run\my\license.flf'
Import License
Creating http download site
Setting SA Password and enabling SA
Creating admin as SQL User and add to sysadmin
Creating SUPER user
Assign Premium plan for ADMIN
Container IP Address: <IP_ADDRESS>
Container Hostname  : bcsndbx14
Container Dns Name  : bcsndbx14
Web Client          : http://bcsndbx14/NAV/
Dev. Server         : http://bcsndbx14
Dev. ServerInstance : NAV

Files:
http://bcsndbx14:8080/al-3.0.154634.vsix
Initialization took 163 seconds
Ready for connections!
Reading CustomSettings.config from bcsndbx14
Creating Desktop Shortcuts for bcsndbx14
New-DesktopShortcut : Unable to save shortcut "C:\Users\SrdanG\Desktop\bcsndbx14 Web Client.lnk".
At C:\Program Files\WindowsPowerShell\Modules\navcontainerhelper\<IP_ADDRESS>\ContainerHandling\New-NavContainer.ps1:1183 char:13
'+             New-DesktopShortcut -Name "$containerName Web Client" -Ta ...
'+             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+ CategoryInfo          : OperationStopped: (:) [New-DesktopShortcut], DirectoryNotFoundException
+ FullyQualifiedErrorId : System.IO.DirectoryNotFoundException,New-DesktopShortcut

Does the folder exist?
C:\Users\SrdanG\Desktop

Tried to create another user on an Azure VM - works fine.
All the script does is to create shortcuts using this function: https://github.com/microsoft/navcontainerhelper/blob/master/Misc/New-DesktopShortcut.ps1
Which should be using the right functions to find the right folder and create the shortcut.

The user has a special ASCII character (I don't think it's unicode) in his login user name.  So the d is actually a đ so we're changing the user name, deleting the profile and trying again.  It does seem like the script is replacing the đ with a d.
Not sure if that is a bug or a known limitation.

Changed the user name and deleting the user profile.  This seemed to fix the issue.


Temperature and pressure

I notice that you already added the code for fetching the water temperature and pressure, but this doesn't show how up in my home assistant instance, is this part already working?

Thanks for your fast response, I tried doing this several times already, still only get these entities:

Do i need to do something additional apart from just setting it up in the Sonic app?

Do you have one or two devices listed under the Sonic integration. I set them up with two devices (one for the device itself and one for the settings).

No only 1 device:

I just did a factory reset on the device, removed the property from my account and added a new one with a different name, I can see the new name in home assistant, but still no device values.
The sonic app itself is working as expected (I can see all the events and other values)

It's is the new pro model (with the black button), can it be caused by that?

Issue has been resolved, turns out i have the Wi-Fi model.... i will just have to be patient till hero-labs is finished with their HA integration (which they are working on)

Maybe add to the documentation that only the model with wall plug is supported for now? (expected to change by the end of the year)


fix(curriculum): correct typo - change rating to ratings in console.log statement

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

User in this forum post pointed out a typo in the Use the map Method to Extract Data from an Array challenge.  This PR fixes the typo that was causing a reference error to show in the FCC console.

Probably just need one review for this typo fix.


Switch to eslint as tslint is deprecated

Do you want to request a feature or report a bug?
Bug
Feature Request

Bug Report

StackBlitz template: https://stackblitz.com/edit/covalent
Screenshots or link to CodePen/Plunker/JSfiddle
What is the expected behavior?
What is the motivation / use case for changing the behavior?
Which version of Angular and Material, and which browser and OS does this issue affect?
Did this work in previous versions of Angular / Material?

Other information

I would recommend waiting for the Angular team to migrate its own repo to ESLint as stated at https://github.com/angular/angular-cli/issues/13732 .
Keep in mind that if you still do the migration, not all rules are supported by https://github.com/angular-eslint/angular-eslint, which is the plugin that handles the Angular lint rules (equivalent of codelzyer rules).
Cheers!

Thanks for the tips Manuel!

:tada: This issue has been resolved in version 4.1.0-develop.1 :tada:
The release is available on:

v4.1.0-develop.1
GitHub release

Your semantic-release bot :package::rocket:


To the users

Really sorry for that!


PP-7859 Metadata Transactions filter - add UI elements to page

Update controller - to pass down 'isExperimentalFeaturesEnabled' to view.
Add 'Advance filters' toggle.

Only show this if the service has 'isExperimentalFeaturesEnabled' = true.
The toggle contains the metadata inbox filter.

Duplicate of https://github.com/alphagov/pay-selfservice/pull/2629


Draft implementation of PSR-3 log target

Q
A

Is bugfix?
no

New feature?
yes

Breaks BC?
no

Tests pass?
yes/no

Fixed issues
#13702

I think this should be distributed as separate extension. You're using psr/log package here, but it is not mentioned in composer require section, so basically  you're using non-existent class. In separate extension you could add psr/log into composer require, so composer will take care about required dependencies and possible conflicts.

Yes. Indeed it's better to be done as separate extension. I'll probably release it under samdark/ and mention it in the guide.


Dragging in-product release notes to start a new editor group, closes notes and goes to web

VSCode Version:1.6.0 insiders
OS Version: Windows

Steps to Reproduce:

Follow test plan to enable in product release notes in insiders https://github.com/Microsoft/vscode/issues/12520
Help > Release Notes
Try to drag the release notes to the side to start a new editor group.

The release notes editor is closed and the user is directed to the release notes on the website.
This happens when the release notes are in preview mode or as a persistent editor after doing the Keep Open command.
This might be related to https://github.com/Microsoft/vscode/issues/13105 which has been fixed and included in the October release. Given this is likely a common action for users who want to review the release notes next to another editor group, we may want to fix this for September.
The workaround is to drag the release notes to an existing editor group.

This is the same as #13105. Since it doesn't cause data loss, it's not a privacy issue and does not break functionality, I suggest we don't fix this in September.


[Docs]: add rating system working documentation

Fixes #185

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our CONTRIBUTING.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊


feat:  Use Swift-Bridge generated files with libxmtp XCFramework to run gRPC query

Overview
This provides a single Swift package that encapsulates:

The Rust networking/encryption code as XCFramework from https://github.com/xmtp/libxmtp/tree/michaelx-grpc_with_swift_bridge/bindings/xmtp_rust_swift
Swift-Bridge generated C header files to target that XMTPRustSwift.xcframework
Swift-Bridge generated Swift files  - which means consumers are writing code in Swift that targets a Swift package, where the immediate receiving code is in Swift!
Unit tests in this Swift package!

The plan moving forward will be:

Expand the Swift-Bridge generated API to encompass the full ApiClient gRPC library
Clean up the e2e build flow

Rough Edges

You still have to go into libxmtp/bindings/xmtp_rust_swift and run make pkg. Then you have to copy the XMTPRustSwift.xcframework into the root of this repo
The XMTPRustSwift.xcframework is 320M with MacOS targets in it LOL (update: added profile attrs to Rust build to cut it down to 42M)
The name of this Swift package changed to XMTPRust to avoid name conflicts. Open to new naming suggestions!

Nice parts

We can run the unit tests in this Swift package, so we no longer have to copy into derived data of xmtp-ios etc etc

The XMTPRustSwift.xcframework is 320M with MacOS targets in it LOL

Maybe we don't need MacOS?

The XMTPRustSwift.xcframework is 320M with MacOS targets in it LOL

Maybe we don't need MacOS?

I stripped symbols and stuff from the Rust build profile and got it down to 40M. I can also yank out MacOS too.

I’ll merge this for now (cause I jumped the gun on 0.1.0) and then #2 (which commits the xcframework) to line up with the 0.1.0 version. I’ll open a PR right after to get rid of the getter, it’ll be an API breaking change for the integration into xmtp-ios we can line up for 0.1.1!


Re-deploying CC, will old data be retained?

I am re-deploying company communicator. Is there anyway the data from previous announcements can be retained? Will it just be a clean slate?

If you don't delete the database (Azure Storage account), yes. So take care with the "re-deploying".


Unable to install mariaDB addon on Hass.io

Home Assistant release with the issue:
0.77.3
Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.):
Hass.io
Component/platform:
Raspberri Pi 3 32bit
Description of problem:
Trying to install the MariaDB addon via the UI, the installation fails without any error.
Response from chrome developer tools:
{"result": "error", "message": null}
IP/api/hassio/addons/core_mariadb/install
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
homeassistant:
  # Name of the location where Home Assistant is running
  name: Home
  # Location required to calculate the time the sun rises and sets
  latitude: 0
  longitude: 0
  # Impacts weather/sunrise data (altitude above sea level in meters)
  elevation: 0
  # Pick yours from here: http://en.wikipedia.org/wiki/List_of_tz_database_time_zones
  time_zone: Asia/Jerusalem
  # Customization file
  customize: !include customize.yaml

# Show links to resources in log and frontend
# introduction:

# Enables the frontend
frontend:

# Enables configuration UI
config:

# Uncomment this if you are using SSL/TLS, running in Docker container, etc.
# http:
#   base_url: example.duckdns.org:8123

# Checks for available updates
# Note: This component will send some information about your system to
# the developers to assist with development of Home Assistant.
# For more information, please see:
# https://home-assistant.io/blog/2016/10/25/explaining-the-updater/
updater:
  # Optional, allows Home Assistant developers to focus on popular components.
  # include_used_components: true

# Discover some devices automatically
# discovery:

# Allows you to issue voice commands from the frontend in enabled browsers
conversation:

# Enables support for tracking state changes over time
history:

# View all events in a logbook
logbook:

# Enables a map showing the location of tracked devices
map:

# Track the sun
sun:

# Weather prediction
sensor:
  - platform: yr

# Text to speech
tts:
  - platform: google

# Cloud
cloud:

group: !include groups.yaml
automation: !include automations.yaml
script: !include scripts.yaml

#extra
panel_iframe:
  configurator:
    title: Configurator
    icon: mdi:wrench
    url: http://hassio.local:3218

Traceback (if applicable):
None

Additional information:
None

/move home-assistant/hassio-addons


Getting a warning when trying to convert a PDF to image with more than 10 pages

`const pdf2pic  = require("pdf2pic");
const options = {
saveFilename: "8.21_POLICE.FSO",
savePath: "./uploads",
format: "jpg",
width: 2550,
height: 1250,
density: 150
};
const storeAsImage = pdf2pic.fromPath("./uploads/8.21_POLICE.FSO.pdf", options).bulk;
const pageToConvertAsImage = -1;
storeAsImage(pageToConvertAsImage). then((resolve) => {
console.log("Page 1 is now converted as image");
return resolve;
}).catch(err=>{
console.log("error",err)
})`
When i try to execute this code, its showing this warning.
'(Use node --trace-warnings ... to show where the warning was created)
(node:24620) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 data listeners added to [ReadStream]. Use emitter.setMaxListeners() to increase limit'
I followed this solution in https://stackoverflow.com/questions/8313628/node-js-request-how-to-emitter-setmaxlisteners to hide this warning. What's the  best way to solve this issue?  In my case i may have to deal with PDFs with 100-200 pages.

This issue was already reported and its still open. so closing this one


Return 200 when updating a table with REST

Looks like DSE failed on unrelated gRPC and C* 3.11 failed on codacy. Going to try running again.

I see the gRPC error:
Step #5 - "dse-6.8": [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.1:compile (default-compile) on project testing: Compilation failure: Compilation failure:
Step #5 - "dse-6.8": [ERROR] /tmp/dse-6.8/testing/src/main/java/io/stargate/it/grpc/SimpleQueryTest.java:[10,41] package io.stargate.proto.QueryOuterClass does not exist
Step #5 - "dse-6.8": [ERROR] /tmp/dse-6.8/testing/src/main/java/io/stargate/it/grpc/SimpleQueryTest.java:[11,41] package io.stargate.proto.QueryOuterClass does not exist
Step #5 - "dse-6.8": [ERROR] /tmp/dse-6.8/testing/src/main/java/io/stargate/it/grpc/SimpleQueryTest.java:[12,25] package io.stargate.proto does not exist


Delete an account from the service

As a Service Provider ,
I need the account to be deleted,
So that I can remove the unwanted
Details and Assumptions
Database still needs to be provisioned
Acceptance Criteria
Given [some context]
When [certain action is taken]
Then [the outcome of action is observed]


Tilde syntax x ~ foo looks for @trace in the caller's namespace

Example:
Gen.@gen function f()
  x ~ Gen.normal(0, 1)
end
# ERROR: LoadError: UndefVarError: @trace not defined

I think this is a bug in the macro escaping.

oh jk, it's actually that the @trace macro is being called incorrectly.  Sending fix

Yeah I think this is the function that needs to be fixed by prefixing each @trace with Gen.
function desugar_tildes(expr)
    MacroTools.postwalk(expr) do e
        if MacroTools.@capture(e, {*} ~ rhs_)
            :(@trace($rhs))
        elseif MacroTools.@capture(e, {addr_} ~ rhs_)
            :(@trace($rhs, $(addr)))
        elseif MacroTools.@capture(e, lhs_ ~ rhs_)
            addr_expr = address_from_expression(lhs)
            :($lhs = @trace($rhs, $(addr_expr)))
        else
            e
        end
    end
end

@ztangent, see https://github.com/probcomp/Gen/pull/223 :slightly_smiling_face:
I think what should be happening is that trace (as a function on ASTs) should be called at macro expansion time, from within Gen's internals.  Instead, the desugarer was punting the @traceing to the caller, which actually worked if the caller happened to have @trace in their namespace, but was not the intended logic.

Hm, the unit test is now failing probably due to #223 not doing some escaping that it should be doing.  I'll figure it out :)

As mentioned in #223, I think there needs to be some care to make sure the fix for this works with the DML and the SML. The fix I suggested in my comment above (i.e. prefixing @trace by Gen) should work for the DML, but it most likely will not work with the SML because the SML parser tries to match @trace expressions, and so will fail to match Gen.@trace.
In fact, even without tilde syntax, I think if you do import Gen instead of 'using Gen`, the SML will fail for the following function (when it should work):
Gen.@gen function f()
  x = Gen.@trace(Gen.normal(0, 1), :x)
end

but work for the following function (when it should fail):
Gen.@gen function f()
  x = @trace(Gen.normal(0, 1), :x)
end

i.e. the SML doesn't obey the standard scoping rules.

I think probably the most principled way (in terms of consistency with standard Julia scoping rules) to fix this is to:

Add the Gen. prefix to @trace in the desugar_tildes function -- this should ensure the DML always works.
Pass the calling __module__ from the the @gen macro all the way down to the various SML parsing functions, so that can selectively match either @trace, Gen.@trace, or both, depending on whether @trace exists in the calling __module__.
Do the above for @param in the static DSL as well.

@bzinberg, good catch, and @ztangent, good analysis!
I think what we're running up against is that it doesn't make a whole lot of sense that @trace is a macro in the static DSL, because it is never actually macroexpanded -- instead, it's syntax that the @gen macro explicitly looks for.
Long-term, I wonder if it makes sense to phase out @trace from the static DSL (because it never actually resolves to a Julia binding) and just have the ~ syntax. (Implementation-wise, ~ could replace with Gen.@trace, and static DSL could look for<EMAIL_ADDRESS>I think that @trace still plays an important role in the dynamic DSL, ensuring that the @trace calls in nested generative function definitions are properly resolved.

I think probably the most principled way (in terms of consistency with standard Julia scoping rules) to fix this is to:
1. Add the `Gen.` prefix to `@trace` in the `desugar_tildes` function -- this should ensure the DML always works.

2. Pass the calling `__module__` from the the `@gen` macro all the way down to the various SML parsing functions, so that can selectively match either `@trace`, `Gen.@trace`, or both, depending on whether `@trace` exists in the calling `__module__`.

3. Do the above for `@param` in the SML as well.

@ztangent, definitely not an expert but my thoughts are:
I think 1. assumes the caller has a module called Gen in their namespace, which I think is not necessary.
For 2, I think any user code that wants to refer to Gen's @trace macro should import that macro explicitly, and supply the expansion context explicitly -- just as they would if they wanted to stick any other sub-macro into the body of their macro, with Gen not being treated as an exception.
https://github.com/probcomp/Gen/pull/223/files/09a86c01042a6f449059d2d96b1f08b920b29564
I currently believe the most principled solution (for the dynamic DSL) is:
https://github.com/probcomp/Gen/blob/09a86c01042a6f449059d2d96b1f08b920b29564/src/dsl/dsl.jl#L73-L90

I think 1. assumes the caller has a module called Gen in their namespace, which I think is not necessary.
I think in Julia, you can’t import something without getting its name?

I think 1. assumes the caller has a module called Gen in their namespace, which I think is not necessary.

I think in Julia, you can’t import something without getting its name?

What if the user imports a package Foo that depends on Gen?  My understanding is that Foo.Gen will exist for them but Gen will not.

@bzinberg that solution looks like it should work for the DML, but I think it'll still break support for the tilde syntax in the SML :/

Hmm yeah... I think similar to what @alex-lew suggested in https://github.com/probcomp/Gen/issues/222#issuecomment-606801429, it might make sense to replace @trace with tilde syntax completely in the static modelling language.  I think the @trace syntax is misleading, because it makes it look like @trace is a macro but it's not -- it's a keyword that is specific to the modelling language, and has semantics similar to @trace in the DML (which is a macro).  The complaint I have with this pun is that @trace in the SML cannot be referred to by a qualified name like Foo.@trace (and in fact is not imported from anywhere, it's syntax) -- it has to be referred to by exactly the expression @trace.

Yeah I agree that the issue is with the SML, but removing support for @trace would be a breaking change for a lot of existing code (including some of my own). I think this warrants a longer discussion with @marcoct?

@ztangent yes, I definitely did not mean to imply an overly hasty migration strategy -- or even a commitment to do the migration at all.  I'm considering us to be at the stage of coming to consensus about what the ideal long-term resolution is, independent of whether or how much time we want to invest in moving toward that ideal.

Here's a possible alternative that preserves @trace support in the SML. Since @trace macroexpands to either a traceat call or a splice call (see below), we could make the SML parser match those expressions instead. This would also nicely allow it catch and throw errors on any splice call, since splicing is currently not supported by the SML. (With the SML refactor, splicing is already caught at parse time, but only by checking that @trace calls don't have a supplied address).
https://github.com/probcomp/Gen/blob/09a86c01042a6f449059d2d96b1f08b920b29564/src/dynamic/dynamic.jl#L63-L79

@ztangent nice!  This is orthogonal to the fix in the DSL, right?

@bzinberg You mean orthogonal to the fix you've already made for the DML? Well, it would rely on the macroexpansion being done in desugar_tildes, so not exactly orthogonal. It would also require macroexpanding any direct @trace calls (and ideally @param calls too) in the static function body, before passing the function body to make_static_gen_function.

@ztangent, do you think this is a reasonable complete fix?:

Change @trace as in https://github.com/probcomp/Gen/pull/223, so that macro expansion works regardless of what names exist in the caller's scope.
Make the changes you suggested in https://github.com/probcomp/Gen/issues/222#issuecomment-606852924, and document them as: "In the SML, traceat and splice are keywords; in the DML, they are functions."
Change make_static_gen_function as you suggested in https://github.com/probcomp/Gen/issues/222#issuecomment-606892981

I think that should work, yup! Happy to review any of the changes made to
the static parser. Also, if this is going to happen, I suggest reorganizing
the code so that @param and @trace are defined in dsl.jl, since they will
now be shared by both the static and dynamic modeling languages.
Xuan
On Thu, Apr 2, 2020, 1:36 PM bzinberg<EMAIL_ADDRESS>wrote:

@ztangent https://github.com/ztangent, do you think this is a
reasonable complete fix?:

Change @trace as in #223 https://github.com/probcomp/Gen/pull/223,
so that macro expansion works regardless of what names exist in the
caller's scope.
Make the changes you suggested in #222 (comment)
https://github.com/probcomp/Gen/issues/222#issuecomment-606852924,
and document them as: "In the SML, traceat and splice are keywords; in
the DML, they are functions."
Change make_static_gen_function as you suggested in #222 (comment)
https://github.com/probcomp/Gen/issues/222#issuecomment-606892981

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/probcomp/Gen/issues/222#issuecomment-607989587, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AADLUZ3V7UXLJGOKROGUYYLRKTEINANCNFSM4LXZULWA
.

Hmm, I just realized that splice is also defined in Base if I'm
not mistaken... And in general users would need to make sure they don't
define their own traceat and splice functions when using the static
modeling language.
Xuan
On Thu, Apr 2, 2020, 2:59 PM Xuan (Tan Zhi Xuan<EMAIL_ADDRESS>wrote:

I think that should work, yup! Happy to review any of the changes made to
the static parser. Also, if this is going to happen, I suggest reorganizing
the code so that @param and @trace are defined in dsl.jl, since they will
now be shared by both the static and dynamic modeling languages.
Xuan
On Thu, Apr 2, 2020, 1:36 PM bzinberg<EMAIL_ADDRESS>wrote:

@ztangent https://github.com/ztangent, do you think this is a
reasonable complete fix?:

Change @trace as in #223 https://github.com/probcomp/Gen/pull/223,
so that macro expansion works regardless of what names exist in the
caller's scope.
Make the changes you suggested in #222 (comment)
https://github.com/probcomp/Gen/issues/222#issuecomment-606852924,
and document them as: "In the SML, traceat and splice are keywords;
in the DML, they are functions."
Change make_static_gen_function as you suggested in #222 (comment)
https://github.com/probcomp/Gen/issues/222#issuecomment-606892981

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/probcomp/Gen/issues/222#issuecomment-607989587, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AADLUZ3V7UXLJGOKROGUYYLRKTEINANCNFSM4LXZULWA
.

in general users would need to make sure they don't define their own traceat and splice functions when using the static modeling language.

That's the case for any reserved word, right?  I think users expect that (like in Python you can't say def = 7).  Adding reserved words of some sort to the SML sounds reasonable to me, since they really are part of the syntax.

I think that should work, yup!

Great :+1:

Happy to review any of the changes made to the static parser.

Would you be up for implementing these changes?  You could push commits to https://github.com/probcomp/Gen/pull/223 so that we have one combined fix and then we can review it together.

BTW, to your point about reserved words, yeah, I think it would be a good idea to use something less high-frequency-word than splice as the reserved word -- perhaps something like Gen_splice.

@bzinberg yeah, or maybe spliceat, in keeping with traceat!
And yeah, I can make those changes -- but it may have to wait till next week when I'm a little less busy!


Update search-limits.md

Search pipeline does not ingest content of files that >150MB. We do not support partial indexing also in this case.

I dont see the updates here https://learn.microsoft.com/en-us/sharepoint/search-limits


Issue with launching without X-Code?

I don't have X-code installed, when I run the command without it in VS Code it does not output any errors, but nothing happens.

This is a flow I have not tested.
I still need to improve the error's and I'll check with them.

Updated errors in version v1.0.0


May you update your code/Readme

At first, thanks for your work sharing, may you update your Readme about how to retrain or finetune your model?

OK! I'll update this repo this week.

OK! I'll update this repo this week.

thanks


Implement Stack

Add comment to get assigned to your desired language.
Follow Submission directory Structure for PR merge.
Submission Title
Stack Implementation
Submission Type
Data Structure
Submission Description
Implement Stack form scratch
Submission Directory
Dat_Structures => Stack => Stack_Implementation.extension
Language

[ ]  C++
[x]  C
[ ]  Python
[ ] Java
[ ] C#
[ ] JavaScript

May i complete this in C?

May i complete this in C?

Assigned for C

I want to implement this in java

I want to implement this in java

Assigned for Java

Could you assign this to me for C#.

Could you assign this to me for C#.

Assigned for C#

Hey! can I work on this in C++ and Python.

Hey! can I work on this in C++ and Python.

Assigned for *Cpp and Python

Please Assign me for JavaScript

Please Assign me for JavaScript

Assigned for Javascript

#108 Will Completely Close This Issue

#108 Will Completely Close This Issue

@Siddhant-K-code Yes, I am closing this issue.


BigScreen.Enabled Not Correctly Set in Safari 5.1.x

I've noticed on my iframe content that document.webkitFullscreenEnabled is undefined when using the developer tools in Safari.  Thus, using webkitFullscreenEnabled to determine if BigScreen.enabled is true will NOT work.
There may be a better way to fix it, but I would suggest just checking to see if the document.webkitIsFullScreen property exists to determine whether or not the fullscreen API is supported.  Here's what I changed to the bigscreen source code to make it work properly in Safari:
Line 12:  Added webkitIsFullScreen as one of the properties to check for enabled:
enabled: ['fullscreenEnabled', 'webkitFullscreenEnabled', 'mozFullScreenEnabled', 'msFullscreenEnabled', 'webkitIsFullScreen'],
Line 215:  Added a check that enabled is not set to webkitIsFullScreen:
if (iframe && (document[fn.enabled] === false && fn.enabled !== "webkitIsFullScreen")) {
Under line 336, this was added:
if (fn.enabled === 'webkitIsFullScreen'){     return true; }
I've also attached my version to this post.  Maybe there is a better way to check for fullscreen support in Safari.  BigScreen.enabled should be true for me because 5.1.x does support it, but the document doesn't always have the webkitFullscreenEnabled property.  It can be undefined (in my case with iframe content) which means BigScreen will incorrectly say that it does NOT support fullscreen when it does.
bigscreen_modified_safari_detection.zip

With this fix, the only way I can get iframe content to truly go into full screen is to add the "webkitAllowFullScreen" property to the iframe.  "allowfullscreen" is not enough.
Example:
<iframe src="{WEBPAGE_WITH_BIGSCREEN_STUFF_HERE}" allowfullscreen webkitAllowFullScreen></iframe>
That's the only way I can get it to work.  webkitAllowFullScreen must be added to the iframe.

I think this is actually by design. BigScreen.enabled should only be true if calling BigScreen.request would actually work. If you don't have webkitallowfullscreen added to the iframe, that request won't work.
If all you're interested in is doing fullscreen for a video (which is most of the reason this library exists), BigScreen.videoEnabled(video) is there to check for that. There's an older fullscreen api that works only on videos that is enabled even if the iframe lacks the fullscreen attribute.

No, in my case, I want the body of the entire HTML document in the iframe to go fullscreen.  Without my changes, it won't go fullscreen in Safari when there is no reason for it to NOT go into fullscreen.
Basically, it should go fullscreen since it works just fine, but BigScreen.enabled is false until my changes are made due to checking a property that doesn't exist on the dom in an iframe.

Can you make a CodePen or JSFiddle that shows that it works? The fullscreen request should fail without the webkitallowfullscreen attribute on the iframe.

For some reason, pasting the attached HTML content into JSFiddle doesn't work at all.
Please test the attached HTML file in all browsers.  When you click on the image in each iframe, the image should go fullscreen.  Try it in Safari.  When you click on the first image in the iframe, the image does not go fullscreen.  The bottom iframe works as expected which contains the changes I made to BigScreen.js
test_in_safari.zip

Hello, i have a problem with ios device, example iphone X and etc. My fullscreen  play on androind, but not ios.( Who can help me?


Can not create an account(Self Register) via the login page

Description:
$subject.
Steps to reproduce:

Navigate to the login screen of the dev portal
Click on the Create Account link
Provide a name for the account
Click Proceed to Self Register button
You will be redirected to the following page

Issue is not reproducing. Hence closing


How can we start FStar

Hi, I cloned your repository and try to start your examples. I build src and try to start:
FStar.exe --odir out --codegen FSharp ...\FStar\examples\hello\hello.fst

And we have exception:
Unexpected error; please file a bug report, ideally with a minimized version of the source program that triggered the error.
The type initializer for '<StartupCode$tosmt>.$FStar.ToSMT.Z3' threw an exception.

I installed https://github.com/Z3Prover/z3/releases/tag/z3-4.4.0. OS windows 8, visual studio 2015.
Maybe, Do you have any ideas what problem is?
It seems like #159 , but I didn't use cygwin, the project was built in VS.
Thanks.

Can you confirm that z3 is in your path?
How did you build F*?
If you use F#, then what version ?

See also #308 for a similar issue with VS.
As a workaround, you can use the OCaml binary built using make -C src/ocaml-output
To install native OCaml on Windows, you can use http://protz.github.io/ocaml-installer/

Let's first see what the problem is before talking about workarounds. There are a ton of problems that manifest themselves this way.

Thanks. This problem solved. Also I hope that my pr can help to solve some problems from issue #308.


EVE Online new format

It appears EVE no longer decodes, I've tried all the combos of packed codebooks and mod packets.
The new resource cache doesn't do extensions, but I separated the audio files using file, and copying files that were similar to "RIFF (little-endian) data, WAVE audio, stereo 48000 Hz" (just looked for RIFF).  If needed I can upload some example files.

Do you get any error messages?
I'm grabbed the latest EVEResources.7z (md5 1cfd1f93c5b027c83c14e63c85e82d42) which I think is from February 25 2016, could you point me at some particular file that's problematic? I didn't find any that didn't decode correctly in that bundle.

No error messages, just:
Audiokinetic Wwise RIFF/RIFX Vorbis to Ogg Vorbis converter 0.24 by hcs

Input: 001c83a3cd80bd69_c38ebe5ad74196ec98c98be018a269a6
RIFF WAVE 2 channels 48000 Hz 104232 bps
552000 samples
- 2 byte packet headers, no granule
- stripped setup header
- external codebooks (packed_codebooks.bin)
- modified Vorbis packets
Output: 001c83a3cd80bd69_c38ebe5ad74196ec98c98be018a269a6.ogg

it results either in the player refusing to play, or for more permissive players, a screechy mess.  Binary pulled from here: https://forums.eveonline.com/default.aspx?g=posts&m=6371200 same md5 (originally I was just pulling from my installed resource cache, same result though)

Sorry don't know if you'll see this, but some of the bnk extracted files actually do give an error, looks like all the 'Turrets' files
RIFF (little-endian) data, WAVE audio, Microsoft ADPCM, mono 48000 Hz
Audiokinetic Wwise RIFF/RIFX Vorbis to Ogg Vorbis converter 0.24 by hcs

Input: Turrets.053.wav
Parse error: expected 0x42 fmt if vorb missing

I assume that Turrets.053.wav came out of d218fb784d254413_32229fb6eedc544590b4966183525f53, the extractor I'm using (bnkextr) gives names like 0582.wem but this is one of only two resources with both Turrets and RIFF in it (the other is 7638e3bd52dcedb1_c1e6c93394bbe70cb5d572ceae337bd0 which is just Vorbis).
These files seem to be Wwise IMA ADPCM, you can decode these with wwise_ima_adpcm or ima_rejigger5 as discussed in issue #4. I am having some doubts as to how accurate these are, as when I tried with the largest file there was a lot of clipping, but it was also an explosion so who knows.


[Feature Request]: On Demand Collecting

Is that possible to send the logcat only if the user asked for it? (pressed a button).
So, with this approach, we have their permission and also, we open and close the connection whenever there is something wrong only.

This tool was created to solve a very specific problem, of not having developer mode enabled during development and debugging - not really for app users to interact with it.
Maybe tools like Firebase or Sentry will work for you just fine, or you can try logging to file and sending it when the user clicks the button.


[MIPP] Cross platform SIMD Lib for C++

It would be good to add MIPP Library to vcpkg as a port.
You can checkout the lib.
https://github.com/aff3ct/MIPP
Its so insane simd lib.Even used on 5G
https://ppopp18.sigplan.org/details/WPMVP2018/7/MIPP-a-Portable-C-SIMD-Wrapper-and-its-use-for-Error-Correction-Coding-in-5G-Stand

If you wish to add this port in the future, you can reopen this issue, but we're closing it for now.


[wip] improve doc and project layout.

Hi, I am author of SpaceVim, and I want to add purescript support into SpaceVim. I just search the github, just find this repo. I have read the code of this project. and I want to make some changes.

I don't use vim, so I'm not sure how to respond to this.

I think it’s up to @eviefp, who took over maintainership of the library in 2020, to take action one way or another on this — us non-Vim organization members probably don’t have the necessary context 😄

Actually, these suggestions are from long enough ago that if @wsdjeg still has suggestions, I think it would be easier to open a new pull request or send a patch to the mailing list.


CI: No rekeyed secrets were found, please run nix run .#rekey again.

I'm able to use agenix-rekey locally, but when I try to build my config on github actions, I get No rekeyed secrets were found.
Do I need to run nix run .#rekey on the CI? If so, is there a non-interactive version?

That's to be expected since the rekeyed secrets are only stored in local derivations (one for each host). Technically nothing about nix run .#rekey is inherently interactive, but rage usually prompts for the password. If there's a plausible use case for non-interactive use we can of course easily add that.

Do I need to run nix run .#rekey on the CI? If so, is there a non-interactive version?

That would be one possibility, but be aware that this requires your master key to be accessible in the CI. I haven't thought a lot about CI use yet, but an alternative would be to make your rekeyed derivations accessible to the CI via a substituter (cachix, or host your own), where you push the resulting derivations to after running .#rekey locally.
If you can tell me more about your use case, maybe we can find a good solution that makes this easy to use.

My use case with CI is basically to only build what is not already built and cached using nix-build-uncached.
I never deploy from CI, so I don't need to handle secrets there. For secret config, I actually use a private gitsubmodule (which is not fetched in CI), and include the nix files if they exist. https://github.com/bbigras/nix-config/blob/4f8685cf92c0ee908f99bcc1e629586946223089/hosts/desktop/default.nix#L51-L52
Maybe it could be nice if the build in CI would just generate dummy secrets (since my master key is absent) and I would push the real secret from my desktop later.

an alternative would be to make your rekeyed derivations accessible to the CI via a substituter (cachix, or host your own), where you push the resulting derivations to after running .#rekey locally.

That seems to work. I tested it last night.

Maybe it could be nice if the build in CI would just generate dummy secrets (since my master key is absent) and I would push the real secret from my desktop later.

Yeah that's a good idea, I've now implemented a --dummy flag to do that. Let me know in case there's anything else that turns out to be an issue in CI :)

It works! Thank you very much.

Could --dummy cause problems if the result is pushed to a cache?
I'm having dummy secrets on my desktop and I only ran --dummy on github actions.
In other words, if I use --dummy in CI and it's pushed to cachix, will agenix-rekey ask me to run rekey again on my desktop when I actually want to deploy?
I ran rekey on my desktop and it didn't seem to change anything.

Yes, you should exclude this from being pulled from the cache, or from putting it there - whatever makes more sense. Since the rekeying inherently depends on having predictable output derivations, it will always look the same to nix, whether the contents are dummy values or the real values. So naturally your substituter would try to substitute the known result, which is not what you want for secrets.
I'm not sure whether there's a flag that I could set on the derivation to exclude it by default.


Add sensitivity figures for phi=0.03 case (posterior-based plots)

Finished on commit 1942a6b56ea05a92e8be06d3dc129399d46cd273


SD-JWT VC spec

As I just looked through the verifiable-credential lib, I noticed the W3CVC class is issuing W3C credentials as SD-JWT with the vc+sd-jwt type, which is not entirely correct.
The vc+sd-jwt credential format is NOT the same as a W3C credential issued as SD-JWT, as pointed out in the SD-JWT VC spec:
... Note: This specification does not utilize the W3C's Verifiable Credentials Data Model v1.0, v1.1, or v2.0.

yes

What about now, @severinstampler ? May we close it?

Closing after talking to @severinstampler


disableNextImage setting itself to false after setting it to true the day before.

Pretty much what the title says. It's happened around five times now.

I think this was before I reworked how settings were stored and saved, should be fine now? Reopen this if it keeps happening.


Short circuit UCAN validation

ETA: 2023-10-31
Use the Capability store to skip validation of UCANs that the server already knows about. This likely requires some integration with rs-ucan (e.g. exposing new traits, etc), but not sure at time of writing.
cc @QuinnWilton thoughts? Here's an example schema of how we'd cache this in a DB.

We don't do this yet.
So far I'm avoiding DB requests in the auth checking code. The clients have to come with all of the UCANs needed to verify the request.
So essentially I'm postponing this until it becomes an issue.


read_scanlines on the same channel give different results

Problem
I'll make it quick, and elaborate later if necessary:
I used to read several times the Y channel from a PNG YA image using read_scanlines(), in order to get the R,G,B channels.
The first channel was right, but the second and third time I read the channel, it looks like it was premultiplied twice.

Is this normal or should we look deeper inside?
sample PNG YA image:
https://github.com/MrKepzie/Natron/blob/master/Gui/Resources/Images/natronIcon256_linux.png
Versions

OIIO branch/version: 1.7
OS: osx
C++ compiler: clang

Definitely doesn't sound like intended behavior.
I'll try it out.
On March 31, 2017 2:36:18 AM PDT, "Frédéric Devernay"<EMAIL_ADDRESS>wrote:

Problem
I'll make it quick, and elaborate later if necessary:
I used to read several times the Y channel from a PNG YA image using
read_scanlines(), in order to get the R,G,B channels.
The first channel was right, but the second and third time I read the
channel, it looks like it was premultiplied twice.

Is this normal or should we look deeper inside?
sample PNG YA image:
https://github.com/MrKepzie/Natron/blob/master/Gui/Resources/Images/natronIcon256_linux.png
Versions

OIIO branch/version: 1.7
OS: osx
C++ compiler: clang

--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/OpenImageIO/oiio/issues/1654

--
Larry Gritz
<EMAIL_ADDRESS>

I looked at the code but can't quite spot an obvious problem. Maybe I don't understand how to reproduce it. Can you be a little more specific about the sequence of calls you are making?
You open the file...
Then you read a scanline -- with which variety call, exactly? which channels?
Then you read the same scanline again? With which call?
And the pixel values are different from the second call?
It would be super helpful to have a 10 line snippet of Python (or C++) that reproduces the problem, showing the exact call sequence.

This has been sitting around and I never saw a follow-up explain better how to reproduce the problem. Is this still an issue? Or can we close?

Sorry, I have not time to get back to this. What I was doing previously to read an YA PNG was reading three times the Y channel, on the same scanlines, to get the R, G and B channels. The first time was OK, and the second and third time were not identical to what I got on the first read. I finally fixed it by not re-reading the same scanlines again and again ( https://github.com/NatronGitHub/openfx-io/commit/00fd69f1862c775810f06ca76671093d0bd553e9 ).
You may want to close this bug.

OK, closing. Re-file if we're concerned that it's a bug on the OIIO side that we should look into more carefully.


WAL segment ... has already been removed

Hi,
in my instances I've got often errors that wal was removed
postgres-8 2022-05-26 06:19:49.228 GMT [25675] ERROR:  requested WAL segment<PHONE_NUMBER>000001000000B1 has already been removed
postgres-8 2022-05-26 06:19:49.228 GMT [25675] STATEMENT:  START_REPLICATION 1/B1000000 TIMELINE 3
postgres-8 2022-05-26 06:19:54.233 GMT [25676] ERROR:  requested WAL segment<PHONE_NUMBER>000001000000B1 has already been removed
postgres-8 2022-05-26 06:19:54.233 GMT [25676] STATEMENT:  START_REPLICATION 1/B1000000 TIMELINE 3
setup-replica-data-directory 26/05/2022 00:07:24 - Attempting to promote a Replica PostgreSql to Primary...
setup-replica-data-directory 26/05/2022 00:07:24 - Promoting by creating the promotion trigger file: '/var/lib/postgresql/data/pgdata/promote_replica_to_primary.lo

In this situation replica was promoted  to primary by manager. I don't know why previous master pod (postgres-7) was killed :(
When I manually restart for example postgres-2, the  new instance posrtgres-4 (because I've set 3 replicas) is started isnstead restart second instance.

I have the same problem. But the way I got there is a little different. I'm running on openshift and I went to do a cluster upgrade which resulted in psql-3-0 and psql-4-0 pods. Since this was in the middle of the night and I know no data that I care about has changed I just deleted the deployment and redeploy resulting in psql-1-0 and psql-2-0 as normal and I expect it to be using a PVC from earlier as well. The database is up, but I'm getting the same error:
2023-05-31 08:15:54.124 GMT [1] LOG: starting PostgreSQL 13.10 (Debian 13.10-1.pgdg110+1) on x86_64-pc-linux-gnu, compiled by gcc (Debian 10.2.1-6) 10.2.1 20210110, 64-bit
6
2023-05-31 08:15:54.126 GMT [1] LOG: listening on IPv4 address "<IP_ADDRESS>", port 5432
7
2023-05-31 08:15:54.126 GMT [1] LOG: listening on IPv6 address "::", port 5432
8
2023-05-31 08:15:54.158 GMT [1] LOG: listening on Unix socket "/var/run/postgresql/.s.PGSQL.5432"
9
2023-05-31 08:15:54.238 GMT [14] LOG: database system was shut down at 2023-05-31 07:53:13 GMT
10
2023-05-31 08:15:54.291 GMT [1] LOG: database system is ready to accept connections
11
2023-05-31 08:16:09.781 GMT [29] ERROR: requested WAL segment<PHONE_NUMBER>000002000000C4 has already been removed
12
2023-05-31 08:16:09.781 GMT [29] STATEMENT: START_REPLICATION 2/C4000000 TIMELINE 1
13
2023-05-31 08:16:09.800 GMT [30] ERROR: requested WAL segment<PHONE_NUMBER>000002000000C4 has already been removed
14
2023-05-31 08:16:09.800 GMT [30] STATEMENT: START_REPLICATION 2/C4000000 TIMELINE 1
15
2023-05-31 08:16:14.803 GMT [38] ERROR: requested WAL segment<PHONE_NUMBER>000002000000C4 has already been removed
16
2023-05-31 08:16:14.803 GMT [38] STATEMENT: START_REPLICATION 2/C4000000 TIMELINE 1


Add support of non UTF-8 strings in InternationalText

InternationalText text chunk could be accidentally created with non UTF-8 text or translated_keyword.
This PR adds encoding this fields to UTF-8 on write.

You are welcome, that was an easy fix


The maximum file size limit is incorrect when using the S3 protocol for uploads.

method get_appropriate_chunk_size in file snapshot_storage_ops.rs:
MAX_UPLOAD_SIZE is 5GB，and if file_size > MAX_UPLOAD_SIZE,return error
However, according to the Amazon/Google S3 protocol, the maximum should be 5TB
Is this the better way to write it？
pub async fn get_appropriate_chunk_size(local_source_path: &Path) -> CollectionResult {
// const DEFAULT_CHUNK_SIZE: usize = 50 * 1024 * 1024;
const MAX_PART_NUMBER: usize = 10000;
const MAX_UPLOAD_SIZE: usize = 5 * 1024 * 1024 * 1024;
let file_meta = tokio::fs::metadata(local_source_path).await?;
let file_size = file_meta.len() as usize;

// check if the file size exceeds the maximum upload size
if file_size > MAX_UPLOAD_SIZE * MAX_PART_NUMBER {
    return Err(CollectionError::service_error(format!(
        "File size exceeds the maximum upload size: {MAX_UPLOAD_SIZE}"
    )));
}

// check if the file size exceeds the maximum part number
// if so, adjust the chunk size to fit the maximum part number
if file_size > MAX_UPLOAD_SIZE * MAX_PART_NUMBER {
    let chunk_size = (file_size - 1 / MAX_PART_NUMBER) + 1; // ceil(file_size / MAX_PART_NUMBER)
    return Ok(chunk_size);
}
Ok(MAX_UPLOAD_SIZE)

}
Or make it available as a configuration option for users.

sry,put that in wrong projects
similar one is:
https://github.com/qdrant/qdrant/issues/4908


TagBot trigger issue

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

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

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

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

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

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

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

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

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


Timestamps per question/stage of survey

Right now we have start_time and end_time. Perhaps in the future it would be good @sumants-dev to have timestamps on when people finished certain middle questions. Ideally, instead of this being multiple variables, we can have one variable that is a list with different durations of each question i.e. {25seconds, 30seconds... }} or {11:48, 11:49, .....}

Just noting that I'm not really sure what the best model is for this, e.g., a list of times, or a column of times for each response, or something else, and happy to think about that a bit as part of this issue.


Update stringifyJsonConfig

[x] Tests for normal json stringifying
[x] Symbol keys handling
[x] Tests for symbol keys
[x] Error for keys that would be comments
[x] Tests for comment keys
[ ] (maybe) configsRoot argument + inheritance - how would that work?
[ ] Tests for inheritance (see above)
[x] Tests for infinity handling
[x] Tests for class handling

I don't think any inheritance handling is needed as the $extends property is also stringified so stuff can be handled when loading the config if needed.
For the configsRoot arguments, it should be the same between 2 loads so it doesn't need to be handled.


Access original row object in customBodyRender, etc

I would like to feed row-objects into the table and then access them in customBodyRender and other callbacks.
Use case:
I need to refer to data in the original row-object that I do not want to use in any column. For example. <Select />  elements with menu items that are specific to the row being rendered. And the data id to update when that <Select /> is changed.
Example data (just one row):
[
  {
    "key": "credit_processor",
    "description": "Which processor integration to use for this client",
    "value": "none",
    "default": "none",
    "options": [
      "none",
      "cardconnect",
      "worldpay",
      "econduit"
    ]
  },
  ...
]

In other libraries I can easily access the original row data to populate my <Select /> and to tell the server what data key is being updated onChange.
Here, I seem to have to make hidden columns with that information, and then keep track of the column indexes containing that data. The code is a lot harder to read and especially hard to maintain (adjusting indexes if new columns are added or removed).

Here's an example of how I am doing things now given my use case above:
import React from 'react'
import FormControl from '@material-ui/core/FormControl'
import MenuItem from '@material-ui/core/MenuItem'
import Select from '@material-ui/core/Select'
import TextField from '@material-ui/core/TextField'
import { withStyles } from '@material-ui/core/styles'
import MUIDataTable from 'mui-datatables'

const styles = {
    main: {
        flexGrow: 1,
        overflow: 'auto'
    }
}

class SettingsTable extends React.Component {
    handleChange = (key, newValue, oldValue) => {
        this.props.onChange(key, newValue)
    }

    renderValue = (value, meta, update) => {
        if (meta.rowData[0] instanceof Array) {
            return (
                <FormControl>
                    <Select
                        style={{ fontSize: 'inherit' }}
                        value={value}
                        onChange={(event) => this.handleChange(meta.rowData[1], event.target.value, value)}
                    >
                        {meta.rowData[0].map((option) =>
                            <MenuItem key={option} value={option}>{option}</MenuItem>
                        )}
                    </Select>
                </FormControl>
            )
        } else {
            return (
                <FormControl>
                    <TextField
                        InputProps={{ style: { fontSize: 'inherit' } }}
                        value={value}
                        onChange={(event) => this.handleChange(meta.rowData[1], event.target.value, value)}
                    />
                </FormControl>
            )
        }
    }

    options = {
        pagination: false,
        selectableRows: 'none'
    }

    columns = [
        { name: 'options', options: { display: false, viewColumns: false, filter: false, sort: false, searchable: false, print: false, download: false } },
        { label: 'Setting', name: 'key', options: { sortDirection: 'asc' } },
        { label: 'Value', name: 'value', options: { customBodyRender: this.renderValue } },
        { label: 'Default', name: 'default' },
        { label: 'Description', name: 'description' }
    ]

    render() {
        let { classes, data } = this.props

        return (

                <MUIDataTable
                    title="System Settings"
                    data={data}
                    columns={this.columns}
                    options={this.options}
                />

        )
    }
}

export default withStyles(styles)(SettingsTable)

The first column, options, exists only to be accessed in customBodyRender. The data passed to that column is usually an array, and I'm surprised I'm allowed to pass arrays but not objects.
The second column, key, is shown to the user in this case, but usually it would not be. It's also accessed in customBodyRender.
The way those columns are accessed in customBodyRender is nasty.. meta.rowData[0] and meta.rowData[1]. It requires referring to the columns array by index in order to figure out what data they represent, and if a new column is added before or between those columns then things will break unless the indexes are updated everywhere.
There are a lot of column options required to hide my non-UI data: { display: false, viewColumns: false, filter: false, sort: false, searchable: false, print: false, download: false }
I would need a additional column for every other data point that might be relevant to me in customBodyRender, which could mean having 3-4 of these hidden columns :(

As discussed here https://github.com/gregnb/mui-datatables/issues/1011, the table has never officially supported passing object data for table cells, and doing so leads to a number of errors if not handled precisely in custom renders. If you're using a custom render, you're stepping into custom territory, and there's not any strong case I can see for accepting objects for this case. As explained in the issue, you can map something like ids back to arbitrary data/objects. They don't need to be passed into the table to get you the information you need.

I'm not sure that you understand. I'm not suggesting that objects be allowed as cell values. My example above does not do that, and there are no warnings generated.
Myself an others would like to have a reference the original row object made available within customBodyRender etc. This reference could be completely ignored by the mui-datatable itself. The only work the table would need to do is to hold the references and pass them to customBodyRender (or just make them available within tableMeta). This is how it works in the other libraries I've used and tested.

Hm, you should have access to row data from this function already:

Is that not working?

It is also might help if I could see your data structure. It doesn't look like you added it to your example above.

Oh, this is the row data in the first message. Yes, that's exactly what I'm referring to you not putting into the table directly, instead keeping it separate, and mapping your data back. If you look through the issue I linked here, you will find I modified a dev's codesandbox example to do just that.

The rowData is your parsed data and not the original data. You seem to support data in the format I'm sending it in, and you grab the individual bits that are to be shown using the name keys, but there's no reference kept to that original object in its entirety, which is what I'm after, and what other libraries offer to great benefit.
Is there a reason not to store a reference to the original row object? The way I see it:

It's just a reference, so doesn't bloat memory.
It can be completely ignored by the internal table code. You wouldn't sort on it or filter on it or anything like that at all. The reference is just kept, and made available, and otherwise ignored.

Yes, there are good reason not to store this data. It's not possible to simply keep the original data untouched, aside from the fact that I don't want arbitrary object passed into the table at all, because otherwise there is no way for me to map the assembled data structure to your original data structure. To do that, I'd have to establish some kind of mapping system and foist it into your data for you (meaning it wouldn't be untouched). But since you already have the data you want, and you know better than the table what shape you want it in, and you know better how you want to map it, this is work that is best left to individual developers. Attempting to map arbitrary object data to the flat structure used by the table is a recipe for bugs and edge cases.
I look at this as a very minor inconvenience in development. Pretty much every app has a specific way it accepts data via its API, so there's often a need to massage your data into a specific shape. And since you can use something like an id to map back to an arbitrary data structure of your own, I don't see any drawbacks with this approach, but lots of drawbacks with allowing and then managing arbitrary object data into the table, which is already causing bugs.

I'm not convinced that you'd need any mapping system or to touch the data. I assume you keep in there and then it travels with the row and doesn't need to be mapped. Or you have an array of the rows and know the index, so a 2nd array with the original references would work with the same index?
Passing an id into the table is very hacky especially considering the work it takes to hide that id from the user and prevent the table from acting on it via filtering etc. It seems more problematic to me than a reference that can be ignored, plus it ends up on the developers shoulders to prevent those issues.
I almost didn't try this library at all because I didn't see that I could pass in row objects until I looked around a bunch. I wonder how many others just pass on by for that reason, or get as far as I have and then pass :(

Also to touch on the minor inconvenience - the software I work on has many man tables and back in the jQuery days with DataTables.net we had to do this. It may seem minor to you, but it caused many of our developers a lot of grief and there were no shortage of bugs related to having to massage data and map it and reference back to it etc. All of that went away the day we could simple get our original reference in the callbacks. I've been with two companies that lived through that minor inconvenience.. it's not that minor

Again, I'm not familiar with any API that accepts arbitrary data. The solution I recommend is to map/reduce on your end. The only changes to the table I would consider around this is to provide dataIndex in the rowData so there's a more robust way to refer to the data internally than index which has definite problems as you point out.

If you're not going to change it then I accept that and it's fine. I'll move on to another library.
But when you say you are not familiar with any API that accepts arbitrary data it surprises me as this probably the first library I've used in as long as I can remember that doesn't.
From my experience most libraries prefer that you pass objects as rows (or even require that you use objects). When defining a column they ask you to specify which property name to grab the display value from (or allow you pass a function that returns the display value without the table even knowing which property it came from). They allow you to define custom renderer functions, as mui-datatables does. And some libraries allow you to pass functions that return a different value than the display value to be used during sorting and filtering (in cases where the display value isn't what should be sorted on for some reason). In all of those functions they pass the original row object which includes all the original data passed in regardless of which parts are actually used in the table.
I think a key difference with those same libraries is that they are designed not to touch the passed in row data, while also not copying all of the display values to a new array to use instead of the original data each render. And that is great as that also means they don't duplicate the display values (could be millions of them) and bloat memory. I'm not sure why they would need copy data into another structure anyway, as there are simple strategies to avoid doing that.
For example, I've seen it where the table's internal rows array consists of its own row objects that hold some row state, but most importantly hold a reference to the original row object that was passed in. The row state would be for things like whether the row is shown or selected, etc. The internal rows array, since it's not the literal data array that was passed in to the table, can be sorted however it needs to be and that doesn't touch the original data. The row states can change however they need to be, but the original row data object is left untouched - only being referenced for purposes of getting a display value, or the value to sort or filter on.
// array of row data passed in:

[
    { id: 1, name: 'Ryan', age: '35', location: { ... }, purchases: [ {...}, {...} ] },
    { id: 2, name: 'Bill', age: '32', location: { ... }, purchases: [ {...}, {...} ] },
    { id: 3, name: 'Randy', age: '51', location: { ... }, purchases: [ {...}, {...} ] }
]

// column definitions:

[
    { heading: 'Name', value: 'name' },
    { heading: 'Age', value: 'age' },
    { heading: 'Location', value: (rowData, rowState) => `${rowData.city}, ${rowData.country}` }
    { heading: 'Purchase Qty', value: (rowData, rowState) => rowData.purchases.length },
    { heading: 'Purchase Subtotal', value: (rowData, rowState) => rowData.purchases.reduce(...) },
    { sortable: false, filterable: false, value: (rowData, rowState) => <PurchaseDetailsIcon purchases={rowData.purchases} /> },
    { sortable: false, filterable: false, value: (rowData, rowState) => <EditIcon id={rowData.id} /> },
]

// internal table state data:
{
    sortOrder: [ [0, 'asc'] ],
    searchTerm: 'R',
    columnFilters: [],
    ...
}

// internal row data (sorted by 'name' and searching for the letter 'R' as above):

[
    {
        data: { id: 1, name: 'Bill', ... }, // <- just a reference
        state: { shown: false, selected: false }
    },
    {
        data: { id: 1, name: 'Randy', ... }, // <- just a reference
        state: { shown: true, selected: false }
    },
    {
        data: { id: 1, name: 'Ryan', ... }, // <- just a reference
        state: { shown: true, selected: false }
    },
]

Why cant we just have a value that acts as an alias for filter, download, print ect. I think by not allowing objects to be passed as data makes this library a lot more cumbersome to work with.
If you are creating a new array for the rows anyways why not just check if the data typeof is an object then grab the alias value from the data instead of the whole data object.
/// rows generation pseudocode

const newRows = passedData.map(data => {
return ( typeof(data) === 'object' ? data.alias : data)
})

Or to make the data more consistent internally, when you create the new array of rows, make all data passed an object putting the data passed as the alias:
// data passed:
const row = [ someString, otherString, { alias: someObjectAlias, extra: someExtraVariable, extra2: someOtherExtraVariable } ]

const newRow = [ { alias: someString }, { alias: otherString }, { alias: someObjectAlias, extra: someExtraVariable, extra2: someOtherExtraVariable } ]

Granted I haven't checked out the source too thoroughly, but is there any reason an approach like this wouldn't work?

I think the problem is that the concept of the "original row object" simply makes no sense in the context of mui-datatble because the row data it receives is already "digested" (i.e. it has no idea where that data comes from in the first place).
Other libraries (AgReactGrid for example) use a different approach where you provide it the "raw" data and the column definitions contain information on how to retrieve the column data given the raw row data (either in the form of a field name or a callback method).

Yes that's the issue as I see it. Hoping this library can be transformed from former into the latter.

A simple use case: Let's say we get an array of user data from the server:
[
   { "avatar": "user1.png", "username": "user1", "name": "John Smith"},
   { "avatar": "user2.png", "username": "user2", "name": "Will Smith"},
]

Now let's say in the table I want to show an avatar of the user next to their username:
const userColumns = (users) => [
  {
    name: 'username',
    options: {
      customBodyRender: (value) => {
        // I have to pass the entire users data, and per each column do  an O(n) operation to find this user
        const user = users.filter( user => user.username === value )[0];
        return (

            <UserAvatar user={user} />
            {user.username}

        );
      }
    },
  }, ...
]

As in the comment in the code, in order to find the avatar src for the user I have to pass in the entire users array and then do a linear search on it to find the user row obj, and grab the avatar src. This operation has an O(n^2) cost to run (O(n) search in each row x O(n) for all rows)
Instead if in customBodyRender we had access to the original row data, I could just grab that and use it, without needing to do a search for it.
What we are suggesting here is adding an extra variable such as originalRowData to the function, and not change anything that is already there. Hopefully this way, people who need this functionality can have access to the original data and nothing else breaks.
Thank you for this amazing library. It really makes handling tables in Material UI much better. I hope you can find the time to take a look at this use case.

@keyvanm I don't recommend looking up your values the way that you are doing it. I suggest normalizing your extra data according to id, and using the id as a lookup. The compute time will be constant after that (or nearly so, as I don't think js does hash tables exactly).
So, performance isn't a problem with the do-it-yourself approach. The problem is developer convenience. But I have to prioritize how I spend the limited time I have, and I chose to prioritize issues where the functionality might not be possible at all (or bugs, or security issues, etc.). Since what you and others want here is entirely possible, albeit inconvenient, it's just not going to be a priority for the short to midterm. Perhaps in the future it will be.

The ant table has this "feature". (made in china)
https://ant.design/components/table/#Column

I trying to implement a widget of algolia:
https://www.algolia.com/doc/api-reference/widgets/highlight/react/
With Ant design table is this code:
/**
 * @function highlight
 * @param {Object} element attrs
 * @returns {JSX.Element}
 */
const highlight = (attrs) => (<Highlight {...attrs} />);

/**
 * @function searchableRow
 * @param {Object} searchableRow
 * @param {string} searchableRow.title - Row title e.g: Name
 * @param {string} searchableRow.name - algolia response object key e.g: first_name
 */
const searchableRow = ({ title, name, ...attrs }) => {
  return {
    title: title,
    dataIndex: name,
    ...attrs,
    render: (_, record) => {
      return highlight({ attribute: name, hit: record })
    }
  };
};

With your lib is not working because I need the full raw object without changes.
/**
 * @function searchableRow
 * @param {Object} searchableRow
 * @param {string} searchableRow.label - Row title e.g: Name
 * @param {string} searchableRow.name - algolia response object key e.g: first_name
 */
const searchableRow2 = ({ name, label, ...options }) => {

  return {
    name: name,
    label: label,
    options: {
      // filter: false,
      // sort: false,
      // empty: true,
      customBodyRender: (value, tableMeta) => {

        const mock2 = { attribute: name, hit: {[name]: value} }; // Not working I need the full raw object

        // With this mock obj it works o.K
        const mock = {"attribute":"fullName","hit":{"fullName":"Joe Fillmore","firstName":"Joe","createdAt":1582264799,"creditBalance":0,"email":"test@gmail.com","_tags":[],"firstPickupDate":1522900800,"id":"","joinDate":1522705489,"lastName":"Fillmore","lastOrderDate":1579755600,"laundryDay":"Thursday","laundryFrequency":"Weekly","legacyId":7173,"locations":[{"addr1":"0000 Hill Ln","addr2":"","city":"Sville","_geoloc":{"lat": 0,"lng": 0},"latRef":"0","lngRef":"0","geojsonType":"Feature","geometryType":"Point","image":"","isDefault":true,"name":"Home","state":"NC","zip":"28078"}],"defaultLocation":{"addr1":"00000 Hillmoor Ln","addr2":"","city":"Huntersville","_geoloc":{"lat":0,"lng":0},"latRef":"0","lngRef":"0","geojsonType":"Feature","geometryType":"Point","image":"","isDefault":true,"name":"Home","state":"NC","zip":"00000"},"defaultLocationAddr":"0000 Hillmoor Ln","market":"clt","nextOrderDate":1582779600,"ordersCount":0,"ordersTotal":0,"phone":"0000068005","rewardsIsEnrolled":false,"rewardsPoints":0,"type":"consumer","objectID":"ztAkP7lzrMTAcjx0EJkt","_highlightResult":{"fullName":{"value":"Joe Fillmore","matchLevel":"none","matchedWords":[]},"firstName":{"value":"Joe","matchLevel":"none","matchedWords":[]},"email":{"value":"test@gmail.com","matchLevel":"none","matchedWords":[]},"lastName":{"value":"Fillmore","matchLevel":"none","matchedWords":[]},"legacyId":{"value":"7173","matchLevel":"none","matchedWords":[]},"defaultLocation":{"addr1":{"value":"14627 Hillmoor Ln","matchLevel":"none","matchedWords":[]},"addr2":{"value":"","matchLevel":"none","matchedWords":[]}},"phone":{"value":"9738968005","matchLevel":"none","matchedWords":[]}},"__position":2}};

        return highlight(mock2);
      },
      ...options
    }
  }

};

It’s difficult for a pro developer like you add an optional simple attribute that host the raw obj?

My Workaround for this is have in one file a variable that keep the last data that I get from the search engine, and the functions that need the raw data in the same file too. To later with the rowIndex get the raw data for that row.
I using third party components (algolia widgets) that expected that the row data have more metadata and the filtered data that is provided is not working for me.
This workaround has the drawback to have more functions for one file.
Add more complexity or to go through an more elaborate workaround to get the raw data is not a good practice.
keeping the things small and simple as possible is good for extend the functionality and use by the developer teams.
/****
 *
 * Workaround file for: https://github.com/gregnb/mui-datatables/issues/1038
 *
 * After this issues is fixed, we can move these functions to their own files with the tests.
 *
 * lastHits var is the only reason to keep theses functions here.
 *
 */

import './SearchRx.css';

import { connectHits } from 'react-instantsearch-dom';
import { isDefined } from './checks';
import { render } from './engine';
import { highlight } from './algolia';
import { muiDatatable, } from './mui';
import { block } from './dom';

// TODO: Remove it!
let lastHits = [];

/**
 * @function searchableRow
 * @param {Object} searchableRow
 * @param {string} searchableRow.name - algolia response object key e.g: first_name
 * @param {string} searchableRow.label - Row title e.g: Name
 * @param {Object} searchableRow.options - mui-datatables options
 */
const searchableRow = ({ name, label, ...options }) => {
  return {
    name: name,
    label: label,
    options: {
      filter: false,
      sort: false,
      empty: true,
      customBodyRender: (value, tableMeta) => {
        return isDefined(lastHits[tableMeta.rowIndex])  ? highlight({ attribute: name, hit: lastHits[tableMeta.rowIndex] }) : value;
      },
      ...options
    }
  }
};

/**
 * @function actionsRow
 * @param {Object} blueprints - to render
 * @param {Object} options - mui-datatables options
 */
const actionsRow = (blueprints, ...options) => {
  return {
    label: 'Actions',
    name: 'actions',
    options: {
      filter: false,
      sort: false,
      empty: true,
      customBodyRender: (value, tableMeta) => {

        const blueprint = {
          method: block,
          attrs: {
            className: 'mkt-ActionsBlock-div'
          },
          content: [...blueprints]
        };

        return isDefined(lastHits[tableMeta.rowIndex]) ? render(blueprint, lastHits[tableMeta.rowIndex]) : '';
      },
      ...options
    }
  }
}

/**
 * @function muiDatatableElement MUIDataTable
 * @param {Object[]} columns - mui-datatables
 * @param {Object} options - mui-datatables options
 * @description:
 *    - ref: https://www.algolia.com/doc/api-reference/widgets/hits/react/#connector
 *    - connectHits function don't have exposed-props
 */
const muiDatatableElement = (columns, options) => {
  return connectHits(({ hits }) => {
    lastHits = hits;

    return muiDatatable({columns, data: hits, options});
  })();
}

export {
  actionsRow,
  searchableRow,
  muiDatatableElement
}

We've noticed some pretty nasty patterns emerging when we want to reference an object or show lots of data in a column (think a list of status-icons)
Would love to see a property with a ref back to the hydrated data model

I am using MUI Datatables for my react js app but I am not able to get data

My use case is similar to https://github.com/gregnb/mui-datatables/issues/1038#issuecomment-569106293 - again having to read the contents of 2 columns to decide what to actually show.
https://github.com/gregnb/mui-datatables/issues/1039#issuecomment-674463718
One possibility would be to basically have the model itself render the component?
But such a situation (needing to see two or more properties of a row of data to render one column cell) is not at all unusual. Rather, it is a norm in my experience.
Another case is having the type-value-render trio.  One column gives the type, another column gives the value, then the renderer looks for the raw value's actual rendered appearance to draw, and it needs the type info to do that look up.  Again, yes, it could be resolved through massaging the data before going to the table, but in my experience in table libraries for Motif, Java-Swing (and their MVC model), SproutCore, that's not usual.  The renderer makes that decision so the model row can be as clean as possible to the original data set.

I was happy when I see this library until I got stuck in this problem.

I was happy when I see this library until I got stuck in this problem.

I just stumbled upon a similar issue in which I needed to pass the "originalRowData" and thought someone might find my solution somewhat useful.
I have my table data stored in context state.
const { data, setData } = useContext(dataContext)
In the table options object I have set up a callback click handler:
onRowClick: (rowData, rowMeta) => handleClick(rowData, rowMeta)
Then to pass the original labeled row data I use
data[rowMeta.dataIndex] inside my handleClick function to push the "original" object from the row the user clicks. This way I do not have to deal with indexes in the state array, instead, I have the object that was used in the table in the first place with key-value pairs.
BTW I'm using history.push('/some/route', data[rowMeta.dataIndex]) to pass data in my route and then useLocation() hook inside the new component.
Hope this makes sense for someone!

Is this issue fixed now?

I was happy when I saw this library until I got stuck in this problem.

Me too

@RyanEwen  u can access row data directly from the tableMeta argument
see how i extracted it and console logged it here down
{
name: "photo",
label: "photo_url",
options: {
filter: false,
sort: false,
customBodyRender: (value, tableMeta) => {
const tableRowData = tableMeta.rowData;
console.log("🚀 ~ ListPage ~ tableRowData:", tableRowData);
      const imageUrl = `${process.env.REACT_APP_API_BASE_URL}${value}`;
      return value ? (
        <Image
          src={imageUrl}
          alt="Item"
          width="100"
          style={{ verticalAlign: "middle" }}
          preview={true}
        />
      ) : (

No Image

      );
    }
  }
},`

`

@bloodykheeng that property was available back in 2019 when I started this thread, but if you read through it you'll see that it isn't the original row data reference. Not unless there's been a change to the library that hasn't been mentioned in here.


log.critical terminates subgraph, but doesn't display the log message

Which packages are impacted by your issue?
@graphprotocol/graph-ts
Describe the issue
When using log.critical to terminate a subgraph, it ignores the message provided.
e.g log.critical("Config not found", []) - Config not found doesn't appear in the logs.
This can make it difficult to debug especially if you are using log.critical in more than one area in your code.
transaction 3862b31241a48cdf1b8514b147575c57a7f9c19a175ebe54c03ccc4c3d5f9176: Critical error logged in mapping\twasm backtrace:\t    0: 0x5270 - <unknown>!src/mappings/zap/handleZapExecuted\t in handler handleZapExecuted at block #117100837  (27c88cc00232580ddcf35dfd70328d13b2a9907213a44dc376fcab36f75d68f2)"
Reproduction
nil
Steps to Reproduce the Bug or Issue
use log.critical in any mapping handler e.g
log.critical("Config not found", [])
Expected behavior
transaction 3862b31241a48cdf1b8514b147575c57a7f9c19a175ebe54c03ccc4c3d5f9176: Critical error logged in mapping\twasm backtrace:\t    0: 0x5270 - <unknown>!src/mappings/zap/handleZapExecuted\t in handler handleZapExecuted at block #117100837  "Config not found" " (27c88cc00232580ddcf35dfd70328d13b2a9907213a44dc376fcab36f75d68f2)"
Screenshots or Videos
No response
Platform
Nil
Subgraph Manifest
No response
Subgraph GraphQL Schema
No response
Additional context
No response

thanks @itsjerryokolo - does this issue only exist for log.critical (i.e. you are able to see log messages for the other log levels?)

@azf20 Yes, only on log.critical


Create gamf.txt

Institution/School Name
Kecskemet College - Faculty of Mechanical Engineering and Automation
Instiution/School Website
http://gamf.kefo.hu/fmea
Email Users
Emails are in format<EMAIL_ADDRESS>Please place an x between the square brackets if yes

[ ] Students
[x] Academic/Teaching Staff
[ ] Other/Support Staff

Education Levels
Please place an x between the square brackets if yes

[ ] Post-graduate research
[ ] Graduate School
[x] College (University) or Undergraduate School or equivalent
[ ] Community College or equivalent
[ ] High School or equivalent
[ ] Elementary School or equivalent

@Krissz There is no need to create a file for a subdomain of kefo.hu, because the upper level university domain is already added by #2282. Please give our system some 30 min to update the database of email domains before requesting a license. I close this request without merging. Thank you!

Okay Thanks


FIX: Update cibuildwheel config with supported architectures and import-mode

Overview
Closes most of #3012

Edits the cibuildwheel config to specify the precise set of supported architectures, copying the set used by sklearn.
Changes the test invocation to fix an issue where the wrong library was imported, and cext is missing.

Example wheel build run
Import issue
This took me many hours to debug. More info in https://github.com/slundberg/shap/issues/3012#issuecomment-1603448067
Architectures
Previously the cibuildwheel build has been failing as a very broad array of architectures were attempted. The full list of possible architectures is given here.
Many of these are not well supported by the wider scientific python ecosystem, and wheels are not available for dependencies. From looking at the logs, it is apparent that the shap wheel tests fail because the dependencies cannot be built from source (not easily).
For example, scipy publishes wheels for win64 but not win32, as explained in this thread.
I propose creating wheels for the same set of architectures as used by scikit-learn, which is given in their build matrix here.
I note that several recent releases of shap only included wheels for a small subset of architectures. SHAP 0.39 only published wheels for win_amd64.
Another option could be to publish wheels for a wide range of architectures, but disable testing for them. This seems rather risky to be, as I think we should only publish what we can test. In any case, the source distribution is available as an alternative for esoteric scenarios.
Checklist

[ ] Confirm wheel workflow passes
[ ] Added entry to CHANGELOG.md (if changes will affect users)

The build_wheel job seems to be passing on Linux, but not yet on Window or Macosx due to what looks like issues with 3rd party libs used in the tests. A few options come to mind:

We could only publish wheels for Linux for now
We could skip testing on Windows & MacOSX for now
We could add the other platforms to the Checks that run on PRs, and get the full set of tests passing.

@dsgibbons  @thatlittleboy  any thoughts?

Shall we add a comment in pyproject.toml with a reference to this PR, so that our future selfs remember to follow sklearn?

Good idea, I'll push a commit.

I'll lean more towards ensuring build tests pass & release wheels for Windows/MacOS.

Likewise, it seems better to get it working on CI rather than have to do painful debugging on each release. I implemented a wider set of test jobs in: #3029
This reproduces the Segmentation Fault from lightgbm on MacOS that we see in cibuildwheel.

Appreciate the effort @connortann - this looked like an annoying issue to resolve!


[YARN] support yarn2/yarn3

Does it support v2/v3 yarn projects or v1 only?

@rageofgods Looking at the test data under yarn_locks it appears like v1 only. Do you have any sample projects to test v2 or v3?


Update certreq utility reference document

[Enter feedback here]
Outdated information, does not include 2021 changes with "secured" Windows CA Authority.  Examples simply fails with error parsing subject name invalid -- a simple CN is not enough.
Having poor descriptions of two very different "Extensions" syntax is very confusing ... at least have an example of old way clearly separated with (working) example of new way.  Both examples should should be complete with the same values.  Quoting should be clearly explained.

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

ID: 912a2faa-63c4-43c7-056b-16f5a5c71038
Version Independent ID: cbec348f-53ad-a5b7-5a6e-f1cc5540d212
Content: certreq
Content Source: WindowsServerDocs/administration/windows-commands/certreq_1.md
Product: windows-server
Technology: windows-commands
GitHub Login: @JasonGerend
Microsoft Alias: wscontent

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


present two viewControllers at once

Hi,
When I touch my button with BubbleTransition initiated, there is the bubble animation and it presents the source ViewController and destination ViewController at the same time. Any idea? Thanks.

Hi @stealdre
Hard to tell, can you post some of the code that you are using?