Add sklearn metrics

Example of classification plot with accuracy

for your curiosity: 😄
This is the plot for an SVR, linear kernel, fitting the normalized logarithms. The score is the RMSE.

This is a the same but for linear regression. As you see it still goes wild, but at least the features are selected at the minimum

for your curiosity: 😄
This is the plot for an SVR, linear kernel, fitting the normalized logarithms. The score is the RMSE.

This is a the same but for linear regression. As you see it still goes wild, but at least the features are selected at the minimum

@carloszanella I didn't trust that spike too much and I checked deeper: It was a bug in the end 🐛
It was just for pipelines though. I'll push the fix and a test against it in this same PR. Here you have the plot with the correction. Way more reasonable.

@carloszanella I didn't trust that spike too much and I checked deeper: It was a bug in the end 🐛
It was just for pipelines though. I'll push the fix and a test against it in this same PR. Here you have the plot with the correction. Way more reasonable.


sorting option for Hash fields

This pull request addresses https://github.com/mongoid/mongoid/issues/1572
This pull request adds a "sorted" option for Hash fields.  When :sorted => true, Hash fields are completely sorted on storage and search.  This way mongoid can support ruby's definition of Hash equality, i.e.
{"A" => 1, "B" => 2} == {"B" => 2, "A" => 1}

even though mongodb does not.
I'm not sure I've done all of the appropriate testing necessary, because of how large the codebase is, and how new I am to it.  In particular I'm not sure what I need to do to restrict this option to Hashes, or where the tests for that should reside.
If this would be better suited as a gem, let me know.  I'm new to open source dev.

Thanks for the pull... After some further consideration I am going to say that this is better off as a gem, or to tell people to use Mongoid's custom field serialization in their apps for something like this... I'd like Mongoid to just behave as Mongo does.
Thanks for the work though!


Blocks get deleted when trying to stack them.

Describe the bug
Blocks get deleted when trying to stack them in the "E" inventory after you fill your inventory from the crafting tab.
To Reproduce

Fill your entire crafting tab with blocks until they get stacked in your hotbar
Press E to open your inventory
Drop or shift click on your hotbar items

Environment info:

OS: Windows
Minecraft version: 1.20.2
Mod Version 1.12.1 beta jar from modrinth

Other mods:
Soduim

They also get deleted when pressing "Q"

Will need to rewrite the entire inventory system to use the actuall chests and blocks instead of creating virtual invetory


Write a Swift program to find the smallest three elements in an array

Description
Write a Swift program to find the smallest three elements in an array
Input  : arr[] = {10, 4, 3, 50, 23, 90}
Output : 3 4 10

Input  : arr[] = {12, 13, 1, 10, 34, 1}
Output : 1 1 10

How to contribute

Comment !assign to assign this issue to yourself
Fork this repository
Create a new branch
Save the solution in program/program/find-the-smallest-three-elements-in-an-array/FindTheSmallestThreeElementsInAnArray.swift
Commit the changes
Create a pull request

Assign it to me!  I'll handle this

!assign

!assign


the preview fires twice

hello friend,
a huge thank for acode overall. it's a great tool to write software on the go.
when pressing the ▶ button on either html or js files, the file loads.
but when pressing back 🔙 i see the same file loaded in another window: browser for html, or console for js.
since the development is in a phone, i assume there're not plenty of resources to waste.
could this be resolved?
thanks in advance,
alex

Fixed in v1.9.0

thanks @deadlyjack for considering my report.
i have 1.9.0 (931), with this problem still active.

it was indeed fixed in a later commit of 1.9.0 and it works much faster now.
thank you very much friend.


Force specific IP to node because infrastructure sometimes requires specific IP's to be forever.

Just recently due to to Canada getting "rogered" I had to rebuild some of my cluster and it turns out one of my nodes needed to be reinstalled during my RCA.  Is there a way to guarantee that a node can start with a specific IP?  Some infrastructure cannot have IPs shifting around.

Hi @Leopere the node IP will be the same as long as his wireguard key doesn't change. You can keep a backup of the /etc/netbird/config.json file to avoid IP changes in the future

Correct however that's not always possible for example during the Rogering Canada received it was massively disruptive and a number of things broke and during an RCA it was necessary to upgrade from the old Wiretrustee namespace to Netbird in this scenario the IP was forced to roll over.  It would be massively inconvenient for let's say an Active Directory Domain Controller is bound to the Netbird IP among others and to have to meddle with keys in a secure way.
Also perhaps the config is somehow compromised on the machine which would mean that you would have to continue to use the old key.  Is it technologically difficult to allow me to determine the IP of a Netbird node through the panel?

Ok, I see your point.
It is possible to allow such an option via the panel or API, we will discuss and prioritize that.
Regarding issues with the name change, we would like to get more information as the client was prepared to migrate files between /etc/wiretrustee to /etc/netbird and it may be a major bug if you faced issues.

For now I'll follow your advice but just hoping that it makes it at some point might be a good feature!

Ok, I see your point.
It is possible to allow such an option via the panel or API, we will discuss and prioritize that.
Regarding issues with the name change, we would like to get more information as the client was prepared to migrate files between /etc/wiretrustee to /etc/netbird and it may be a major bug if you faced issues.

My problem was caused by the Rogering* that Canada received and I wouldn't really know how to reproduce a country losing internet to really reproduce. All of my other 25 various style nodes recovered fine.

Hi, @Leopere, I intended to send you a message instead of closing the issue. My apologies.
The initial issue seems to have been resolved, but I wanted to ask you if with the DNS feature you still have the need for forcing the IP address, if so, could you explain a bit your use case so we can properly take into consideration?

it was mostly that sometimes a node dies and occassionally I don't have backups to specific nodes /etc/ directories as its typically okay if we just blow away a node and start fresh on it.  However the way Netbird was working was you would have to keep the private key of the machine if you wanted to continue using it's IP address and it would be critical in some cases such as active directory servers for example that the IP addresses used never change.  In some cases using Gluster FS is also a trick with changing locations and IP's of nodes in a cluster. I will often use the wireguard/netbird mesh to avoid having to trust gluster's encryption/security layer.


Remove unused header files

Found that those things have been done in Optimizeincludes , this pr should be closed.


Fix add.oil

Mul is also missing checks for negative values
Perhaps the old add should be renamed uadd, don't know if it should be updated to handle intify. Then rename current mul umul
:x
read
:y
$x
read
$y
# intify
+
$x
-
$x
+
$y
-
$y
# check y's sign
explode
$y
$scratch
je
$scratch1
$-
$yn
$yp
:yp
je
$y
$0
$pr
$incy
:incy
-
$y
+
$x
jmp
$yp
:yn
je
$y
$0
$pr
$decy
:decy
+
$y
-
$x
jmp
$yn
:pr
write
# $x = 0
:0
$x
return
:-
"-
:scratch

:scratch1

Interesting. You also use an "intermediary" syntax, but a very different one from mine.
True, mul.oil can't handle negative values either, at the time explode didn't exist, and even though explode isn't technically needed to check for the sign, it helps a lot.

The intermediary representation is just something I threw together that'd be easy to assemble.


Fix code scanning alert-Incorrect conversion between integer types

What type of PR is this?
/kind bug

What this PR does / why we need it:
Fix code scanning alert-Incorrect conversion between integer types
Which issue(s) this PR fixes:
Fixes #549
Fixes #548
Fixes #547
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

good job 👍


Switch to the vault package repository for CentOS 8 now that it is EOL.

This PR "fixes" the failing CentOS 8 packaging workflow (see here) by telling CentOS 8 to use vault packages as there are no "current" packages now that it is EOL. This will presumably allow anyone still running CentOS 8 to install a Routinator package.
Note: This PR doesn't attempt to address the separate question of whether or not it makes sense to keep packaging for CentOS 8 or if we should package instead or additionally for Rocky Linux, for example (and whether or not we would need to also extend packages.nlnetlabs.nl with a separate RockyLinux repository as well).
A successful run of the packaging workflow using the changes in this PR can be seen here.

Oops, I forgot to wait for the package test which is failing. Investigating...

Oops, I forgot to wait for the package test which is failing. Investigating...

I resolved this by using the CentOS compatible AlmaLinux LXD image instead.


Astral Codec's "Translation Probe Thicket" gets semi-deleted by The Outsider

What Happened?
When running both The Outsider and Astral Codec at the same time:

The Translation Probe Thicket signal is not visible

Intended behavior when just Astral Codec (+ Cheat & Debug) is running:

Incorrect behavior when The Outsider + Astral Codec (+ Cheat & Debug) are running:

The probe itself is often invisible (when I first mentioned this in Discord it was visible, but not any more)

The "Unidentified Signal Nearby" notification always works, but trying to scan the signal does not. The probe is also still solid, and still projects holograms when you pick up the nearby scroll, etc, so you could say it's "mostly there".

The green scroll on the nearby bench is also often invisible (I wasn't checking this before so maybe it always was?)

but interacting with it still works fine.
What was supposed to happen?
Ideally, the mods would peacefully coexist without such senseless violence against innocent probes.
Of course this goes way beyond what most mods are expected to support, but it would be nice if this "just worked" when my randomizer is randomizing both of these mods (and more). On Discord xen said it'd be "nice to have that not happen", so it's at least worth recording in an issue.
For the time being, my randomizer will be working around this by checking for the Unidentified Signal Nearby state instead of the signal being scanned, and warning the player about these invisible things when they have both mods installed.
Platform
Steam
Mods
I've tested this with several sets of mods, and I'm pretty sure the minimum needed to reproduce is just Astral Codec, The Outsider and their direct dependencies.
But in case it matters, the "maximum" mod set I run with during rando development these days (and thus what some of these screenshots were taken with) is
[
  "dgarro.CustomShipLogModes",
  "dgarro.SuitLog",
  "Ixrec.ArchipelagoRandomizer",
  "SBtT.TheOutsider",
  "clubby789.OWClock",
  "Walker.AstralCodex",
  "GameWyrm.HearthsNeighbor",
  "Heelio.EotEBrighterDreams",
  "Glitch.AltDebugMenu",
  "xen.NewHorizons",
  "JohnCorby.VanillaFix",
  "xen.CommonCameraUtility",
  "Vesper.UnityExplorer",
  "xen.EnableMeditation",
  "MegaPiggy.OWRichPresence"
]

Logs
I haven't found any unique error messages in the logs on wakeup, on Signalscope usage, or on walking up to the probe.

Thanks for fixing the signal!
Unfortunately the probe and the scroll are still invisible.

Should we reopen this issue or make a new one?

They work for me

Okay fine I'll double check

The sky in your screenshot looks different, did you just not visit the chime yet?


Unable to uncheck 'Allow requestors to approve their own changes' under Completion options

I was not able to find the switch to be able to uncheck the "Allow requestors to approve their own changes".
There is no switch for update policy.
$requiredReviewer = az repos policy required-reviewer create     --blocking true
--branch "main"     --branch-match-type "exact"
--enabled true     --repository-id $($repositoryid)
--required-reviewer-ids $reviewerGroupId     --org $adoOrg
--project $adoProject `
--message $reviewerMessage

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

ID: c8131327-0712-d643-9fd5-f172b25bd88e
Version Independent ID: d39c0fdb-90a3-e3f2-92cc-c5135e93bd7d
Content: az repos policy required-reviewer
Content Source: latest/docs-ref-autogen/repos/policy/required-reviewer.yml
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

route to CXP team

Adding the Service team for the Devops issue.
@v-anvashist, @V-hmusukula. Could you please look into this issue and provide an update once you get a chance ?

Thank you for your feedback. This has been routed to the support team for assistance.

any update?

similarly, this one needs attention from API team  @v-anvashist?

Hi @omerzubair,
we dropped mail to API team and waiting for their response. we will get back to you once we received response from the API team.
Thanks!

@omerzubair we received response from API team. They do have this covered in the REST API (Configurations - Create - REST API (Azure DevOps Policy) | Microsoft Docs). Parameter is named “creatorVoteCounts”. Can you check if this will be good enough?

Hi @v-soujanya , thanks for your response. I will certainly try and revert back.
cheers

Hi, all! How can the creatorVoteCounts parameter be used in the AZ CLI to uncheck the "Allow requestors to approve their own changes"?

Hey @v-soujanya
Is there any update on this issue?

@gpolcode Hi, we are owning only Azure CLI service and right now with Az CLI we don't have feasibility for this request. Just to help here I have raised the concern to the API team and posted the response the response.
we received response from API team. They do have this covered in the REST API (Configurations - Create - REST API (Azure DevOps Policy) | Microsoft Docs). Parameter is named “creatorVoteCounts”. Can you check if this will be good enough?
Please try it & let me know if that would work for you.

owning only Azure CLI service and right now with Az CLI we don't have feasibility for this request. Just to help here I have raised the concern to the API team and posted the respo

Its a MS product and we are reliant on AzDO team to help us , so we can provide ability to the clients.
Will really appreciate, if we AzDO team can add this highly requested item on the radar. its been 18 months already


Implement Biosample UI

Implement Biosample angular UI following the corresponding Figma designs done by @ielis

Hi @belkassaby , the Figma design for Biosample is in place.


Add support for connect start/end events.

I've altered the signatures of the connect start and end methods. Notably:

Changed from InetAddress and port to InetSocketAddress. Was just easier to pull this off the route, but don't have strong feelings on this one.
Add Proxy to connectStart. I thought this might be useful for people using proxies.
Changed to use Protocol on connectEnd. This is already a public type and felt easier to work with.

Good stuff.


Fix borgs not getting names on roundstart

About the PR
Title.
Why / Balance
Fixes #33444.
Technical details

Media

Requirements

[x] I have read and am following the Pull Request and Changelog Guidelines.
[x] I have added media to this PR or it does not require an ingame showcase.

Breaking changes

Changelog

:cl:

fix: Now borgs get names on roundstart.

Couldn't we just add a "Preferred Borg Name" on the pre round starts? like in customization

should be on the player one, this gives built borgs names too

should be on the player one, this gives built borgs names too

Wasn't it like that before borg type switching?

They can still rename it, the question is if a build borg should have a random name by default or not.

@slarticodefast
https://github.com/user-attachments/assets/1c2a5407-e338-400f-9878-6f37382f1787

Fuck, wrong branch.


Distinguish Service Roads from Parking Aisle

I don't know the best way of doing this (color, thickness) but have them distinguished would be very helpful.

Service roads with a special service=* tag are now drawn with a different color.  This includes things like service=parking_aisle, service=drive-through, service=alley and probably some others I am forgetting.
I used the color #dcd9b9, which is the same color we use for highway=unclassified, but these service roads are drawn thinly.   (Thick roads are for normal traffic, thin roads are for service traffic).

Well, this is a surprise.  A completely unintended, but useful, side effect is that now railways with a service=* tag (siding, yard, etc.) also use that same color #dcd9b9 for their stroke:


Github Actions & To Say 🙏

Hi @sindresorhus
I found this repo very useful. but i am not that used to working with micro services.
so decided to convert this as a github actions https://github.com/varunsridharan/action-gh-latest-repo/
Thanks for creating a cool project :-)

🙌


Number of error messages during incialization

LG Air Conditioner Model
LQ smar airconditioning unit with model name: PC09SQNSJ
Describe the bug
The basic control od my A/C units through Homebridge works, however number of error messages is shown in Homebridge log. Copy of log here: [https://www.dropbox.com/s/f1i56xkf49wn21z/log.txt?dl=0]
Symptom check

Does your unit appear in the Home app? Yes
Is the current on/off state, temperature, and model being reported? Yes. Model reported differently than on the product sticker. Homebridge shows RAC_056905_WW, but sticker shows PC09SQNSJ.
Does your unit respond to the controls? Yes, even though not to all fucntion control. ONly to the main ones.

Relevant logs
If you feel there are relevant logs, please include them here. Otherwise you can remove this.
See here: https://www.dropbox.com/s/f1i56xkf49wn21z/log.txt?dl=0

Hmm, this seems very strange -- thanks for providing those logs!
Is this still happening for you? What NodeJS version is running for your Homebridge? I see you have several devices -- are all of them broken, or is it a mix of them?

Hello, yes, still happening. I have NodeJS v14.16.0
The main functions do work fine. I mean on/off, mode heating or cooling, temperature setting. However the other functions like swing do generate these errors.

Interesting, I think this might be a recent change on the Homebridge API. I'll need to dig into the changes; I've had my A/C unit turned off for the winter but I'll have it on again soon to test this out. Sorry about that!

Actually, I think I see what happened -- your unit and the unit in #63 support different swing modes than what more basic units (and Homekit) support.
I've pushed a fix that should hopefully resolve the errors you're seeing -- could you try upgrading to the new version of this plugin and confirm?

Hello, I tried 1.10.0, but the logs do still containt similar errors:

Hey! This should be fixed now, could you try updating to the latest version?

Hello, just updated the plugin. All error messages disappeared.
Just having a routing issue towards the server on internet:
[20. 4. 2021 10:18:42] [LgThinqAirConditioner] Error initializing platform Error: getaddrinfo ENOTFOUND route.lgthinq.com
All other plugins do work fine, so I wonder, can it be some bug in the plugin? Or just a routing issue?

Hello, I have good news. It works fine. No disrupting error messages.
Obviously, the last time I had a network issue only.
Many thanks for helping me to solve this!

Awesome! Thanks for checking


Social metatags in the website are wrong

There are a few Metatags (For example the Twitter one) that don't show the information and the picture of the website correctly. You can check as well if works on Facebook.

This is an issue related to SBT Microsite project, instead Freestyle, so, close here and open there


Update README for macOS installation

Captures a set of installation instructions I used to get plotman successfully running on macOS (10.15) Catalina.
Includes community recommendation from ChiaForum to install the development branch to avoid cyclical spawning of plot jobs.

Are updates still needed here?

I'm culling through the PRs to help myself focus on those I still need to help move through.  If you feel this is still important, please do reopen it (or comment and nag me if you can't reopen it, or submit an updated one, or...).


Updating from v4 to v5 causes Gulp tasks not terminate.

Current behavior
I have a project that manage some lyfecicle task with Gulp. After updating this plugin from v4 to v5 when i start a gulp script, the process executes succesfully all gulp tasks but after not terminate.
Expected behavior
I expect that gulp process terminates after executing all tasks, like the behavior with v4 version
Environment

fork-ts-checker-webpack-plugin: [6.0.0]
typescript: [3.9.5]
webpack: [4.43.0]
os: [macOS Catalina and gitlab pipelines]
gulp: [4.0.2]

Same problem with v6

Hi! Could you create a reproduction repository?


[Bug]: Incorrect version sorting on https://registry.bazel.build/

What happened?
Attaching a screenshot of https://registry.bazel.build/modules/re2.
Basically the top part of the page shows a copy-paste line with version 2021-09-01, but the lastest version is 2023-11-01. In fact the recommended version is yanked.

Version
Not relevant, this is a website issue.
How to reproduce
No response
Any other information?
No response

Yeah this module doesn't use semver syntax, i think this is another follow-up to https://github.com/bazel-contrib/bcr-ui/pull/56 likely caused by https://github.com/bazel-contrib/bcr-ui/pull/55.
Maybe only this one module is affected?

Dupe of https://github.com/bazel-contrib/bcr-ui/issues/54


Vendored dependencies

We were exposed to supply chain attacks because our Rust dependencies were not pinned. This PR fixes that by pinning the kimchi stubs dependencies and also incorporating all vendored dependencies available in proof-systems.
Explain your changes:

Vendored all Kimchi stubs dependencies to a separate repository (i.e., kimchi-stubs-vendors).
Incorporated the kimchi-stubs-vendors repository as a dependency through git submodules.
Updated the Kimchi stubs dune script with proper instructions to use the vendored dependencies.
Pulled in all the vendored dependencies available in proof-systems:berkeley.

Explain how you tested your changes:

Verified that our build scripts no longer downloads crates from remote sources, both on CI and locally.
Ensured that Cargo always compiles the pinned dependencies available in our vendored submodules, both on CI and locally.
Checked that the version of the dependencies that Cargo builds matches exactly what we specify in Cargo.toml.

Checklist:

[ ] Dependency versions are unchanged

Notify Velocity team if dependencies must change in CI

[x] Modified the current draft of release notes with details on what is completed or incomplete within this project
[x] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[ ] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[x] All tests pass (CI will check this if you didn't)
[ ] Serialized types are in stable-versioned modules
[x] Does this close issues? List them

Closes #15623

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

I have:
ile "src/lib/crypto/kimchi_bindings/stubs/dune", line 51, characters 0-615:
51 | (rule
52 |  (enabled_if
53 |   (= %{env:MARLIN_PLONK_STUBS=n} n))
....
74 |     cp
75 |     %{read:dune-build-root}/cargo_kimchi_stubs/release/libwires_15_stubs.a
76 |     .))))
error: failed to select a version for the requirement `libc = "^0.2.0"` (locked to 0.2.154)
candidate versions found which didn't match: 0.2.147
location searched: directory source `/home/soc/codes/o1-labs/mina-berkeley/_build/default/src/lib/crypto/kimchi_bindings/stubs/kimchi-stubs-vendors` (which is replacing registry `crates-io`)
required by package `wires_15_stubs v0.1.0 (/home/soc/codes/o1-labs/mina-berkeley/_build/default/src/lib/crypto/kimchi_bindings/stubs)`
perhaps a crate was updated and forgotten to be re-vendored?

while building.
Also, i have the following (unrelated) error:
Error: optcomp: cannot open imported file: /src/config.mlh: /src/config.mlh: No such file or directory
File "src/lib/genesis_constants/genesis_constants.ml", line 1, characters 3-9:
1 | [%%import "/src/config.mlh"]

Do you know where it comes from?
Commands I use to test:
git clone git@github.com:MinaProtocol/mina/ mina-berkeley
cd mina-berkeley
git checkout vendoring
gvm use go1.20
git submodule init
opam switch create ./ 4.14.0
git submodule update --recursive && eval $(opam env)
opam switch import opam.export -y
./scripts/pin-external-packages.sh
make build

Building it one last time to make sure everything is fine using the berkeley branch in proof-systems
!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!approved-for-mainnet


[DOCS-419] Beta integrations updates

Changelog
Updated

Page
Deploy Preview
Comment

Configure Beta Integrations
https://deploy-preview-324--cobalt-docs.netlify.app/integrations/beta/configure-beta/
Added instructions for specific apps

Request a Beta Integration
https://deploy-preview-324--cobalt-docs.netlify.app/integrations/beta/
Updated the list of applications

Preview This Change
To see how this change looks in production, scroll down to Deploy Preview. Select the link that looks like https://deploy-preview-<num>--cobalt-docs.netlify.app/
Variables
Help us support a “Write once, publish everywhere” single source of truth. If you see a line that looks like:
{{% asset-categories %}}
You’ve found a shortcode that we include in multiple documents.
You’ll find the content of the shortcode in the following directory:
https://github.com/cobalthq/cobalt-product-public-docs/tree/main/layouts/shortcodes
That shortcode has the same base name as what you see in the PR, such as asset-categories.html.
Checklist for PR Author
[ ] Did you check for broken links and alt text?
Be sure to check for broken links and Alt Text issues. We have a partially automated process,
as described in this section of our repository README:
Test Links and Alt Attributes.

Hi @eugenerevzin, I added instructions for new applications.


SSR & profressive perf improvements to landing page

This PR tackles performance on the landing page and some general perf fixes to the header and footer using SSR.
General mobile lighthouse scores are up from low 80s to low 90s. General problems still arise around loading gtag and 200kb of fonts.
Try for yourself:
before: https://pr850-f9aa1c0---lit-dev-5ftespv5na-uc.a.run.app/
after: https://pr850-a7e1b8a---lit-dev-5ftespv5na-uc.a.run.app/
General Changes:

homepage

SSRd a bunch of components
Lazy load SVGs and images except those above fold
do not load litdev-aside by default
removes dsd-pending styles on noscript
Some things in the header and footer are SSRd on every page, so nohydrate was removed and hydration helpers loaded every page

litdev-search

now SSRd
moved off of lion-combobox because could not be SSRd
hydrated on focus
now has focus indication with bolded underline

litdev-version-selector

Is now SSRd
mods transform is lazy loaded to work on Node

lazy-svg

lazy loads SVGs that load with <use>
strategies are on idle, on visible, and eagerly
SSRd to prevent layout shift

This is awesome!
Two issues found during manual testing.
Issue 1
The combobox dropdown options currently cause a horizontal scroll to appear.
Repro: Full screen chrome. Open page. Type "dir". Notice a horizontal scroll bar has appeared and it is possible to scroll right.
Screenshot:

Similarly in the mobile view, there is a slight horizontal movement.
Issue 2
Switching between full screen search and mobile search is broken. The first one works, but then the second doesn't seem to.
Repro:
Refresh the page. Type in the search that loaded and see a dropdown. Then resize the page to access the other search bar. Typing in the other search bar does not result in suggestions rendering.
Screenshot:
https://user-images.githubusercontent.com/15080861/170148144-fd654a07-f655-4588-8df4-829c2aedc3b8.mov

Thanks for the review @AndrewJakubowicz, fixed!

Sorry for these nitpicks. Did an A11y pass to ensure the behavior is the same as the prior search.
VoiceOver is working fantastic on Safari using Mac.
Manually tested keyboard behaviors. The only behavior that I've found is different is with the ESC key.
When using arrow keys to navigate the dropdown, or if the dropdown is open. ESC should close it.
If the focus is in the search input and ESC is pressed, it should also clear the input.
Another very subtle, and related issue. If on a narrow screen and interacting with the search. If you press escape it closes the drawer and makes the search dropdown invisible but still clickable.
Repro (shown in video). Type a search and notice the top link in the mobile search. Click with mouse away from drawer and notice it shuts. After re-opening the drawer the mouse indicates there is no link under the search. Now focus the search to show the dropdown and press ESC. The drawer shuts (but the dropdown is just invisible this time). Re-open the drawer and notice you can now click something beneath the search. Clicking it takes you to the link that is there.
I also had Mac VoiceOver assistant active while recording this. Repro on Safari and Mac.
Video repro:
https://user-images.githubusercontent.com/15080861/170372231-90b607da-d395-43bf-a5df-d883833abd90.mov

Resolved!


Pass secret as file path for security

The sliding sync proxy currently only takes its secret via the environment variable SYNCV3_SECRET. When used with systemd this is not considered secure:

Note that environment variables are not suitable for passing secrets (such as passwords, key material, …) to service processes. Environment variables set for a unit are exposed to unprivileged clients via D-Bus IPC, and generally not understood as being data that requires protection. Moreover, environment variables are propagated down the process tree, including across security boundaries (such as setuid/setgid executables), and hence might leak to processes that should not have access to the secret data.

From https://www.freedesktop.org/software/systemd/man/latest/systemd.exec.html#Environment
Please change the sliding sync proxy to accept the path to a secret file.

Hi @V02460! I'd be more than happy to take a stab at this. I could have something working towards the end of the week: does that sound good?

This will hopefully be introduced on PR #446!

For more people coming across this and wish to have more security, I've come up with a clever workaround!
I created a script I simply named run.sh:
#!/bin/sh

## Get environmental variables

# PostgreSQL authentication
psql_username=$(cat /config/secrets/postgresql_username)
psql_password=$(cat /config/secrets/postgresql_password)
export SYNCV3_DB="postgres://${psql_username}:${psql_password}@postgresql:5432/syncserver?sslmode=disable"

# Matrix sync proxy secret
export SYNCV3_SECRET=$(cat /config/secrets/.secret)

# Matrix sync proxy server name
export SYNCV3_SERVER="https://your.homeserver.here"

# Setting bind address
export SYNCV3_BINDADDR=":8009"

## Run sync server
env /usr/bin/syncv3

Then in the same directory I created a directory called secrets where I keep three files for the authentication.
sliding-sync/
|-- run.sh
|-- secrets/
|   |-- .secret
|   |-- postgresql_username
|   |-- postgresql_password

postgresql_username: Username of the PostgreSQL server
postgresql_password: Password for the user on the PostgreSQL server
.secret: The generated secret file.
Then for the docker compose I did:
  sliding_sync:
    image: ghcr.io/matrix-org/sliding-sync:latest
    ports:
      - 8009:8009
    volumes:
      - ./sliding-sync:/config
    entrypoint: sh -c /config/run.sh # Runs the script instead of the usual entrypoint of the docker image.

While this won't be as completely secure, it'll at least help a little.
Hopefully a proper implementation comes soon!


Serializing and deserializing ISQ artifacts

A major limitation of ISQ right now is that it is ephemeral: the tensors must be requantized on every run.
To solve this, this PR enables saving and loading ISQ artifacts into safetensors files.
Progress

[ ] Saving (serialization)

[x] GGUF ISQ quantized
[ ] HQQ ISQ quantized

[ ] Loading (deserialization)

[ ] GGUF ISQ quantized
[ ] HQQ ISQ quantized

RUST_BACKTRACE=1 cargo run --features cuda -- -i plain -m microsoft/Phi-3.5-mini-instruct --load-isq out.safetensors
RUST_BACKTRACE=1 cargo run --features cuda -- --isq Q4K -i plain -m microsoft/Phi-3.5-mini-instruct --isq-artifact out.safetensors


Platform autodetection fails on ARM

Probably raised after merging #1032, ARM platform autodetection seems to be failing on ISAMBARD's ARMs.
Tested on Cavium ThunderX2 ARM v8.1, with Devito 4.
TO reproduce follow the wiki for installing Devito on ARM.
Needs to work on archinfo.py

Linked to #1106

Some more info as I spent half an hour on EPSRC systems, due to some work I had to revive on KNL and SKL.
Cavium ThunderX2 ARM v8.1 doesn't like '/proc/cpuinfo' but only 'cpuinfo'.
Other than that, which can be fixed, the generated code is not compiled due to :
AutoTuner: could not perform any runs
/tmp/devito-jitcache-uid415400263/a81515abc891d92252f8d8962642a2ccea3b9659.c:5:10: fatal error: xmmintrin.h: No such file or directory
 #include "xmmintrin.h"
          ^~~~~~~~~~~~~
compilation terminated.
FAILED compiler invocation:gcc -O3 -g -march=native -fPIC -Wall -std=c99 -shared /tmp/devito-jitcache-uid415400263/a81515abc891d92252f8d8962642a2ccea3b9659.c -lm -o /tmp/devito-jitcache-uid415400263/a81515abc891d92252f8d8962642a2ccea3b9659.so

More uncomfortable than that, is that the generated code cannot be found in the related path,
to play with backdoor.
Just my 2 pennies on an update to this issue, as I was tempted out of nowhere to give a try.
Also updated the wiki as we do not need devitobench anymore. (https://github.com/devitocodes/devito/wiki/[Manual]-Installing-Devito-on-ARM.-Tested-on-GW4-Isambard)

Update on v4.2.
Compiler still doesn;t like
 #include "xmmintrin.h"
 #include "pmmintrin.h"

Doesnt like
MM_FLUSH ***
..
will update more

diff --git a/devito/passes/iet/misc.py b/devito/passes/iet/misc.py
index 14c2dc7..0b6d3c5 100644
--- a/devito/passes/iet/misc.py
+++ b/devito/passes/iet/misc.py
@@ -24,8 +24,9 @@ def avoid_denormals(iet):
     header = (cgen.Comment('Flush denormal numbers to zero in hardware'),
               cgen.Statement('_MM_SET_DENORMALS_ZERO_MODE(_MM_DENORMALS_ZERO_ON)'),
               cgen.Statement('_MM_SET_FLUSH_ZERO_MODE(_MM_FLUSH_ZERO_ON)'))
-    iet = iet._rebuild(body=(List(header=header),) + iet.body)
-    return iet, {'includes': ('xmmintrin.h', 'pmmintrin.h')}
+    #iet = iet._rebuild(body=(List(header=header),) + iet.body)
+    #return iet, {'includes': ('xmmintrin.h', 'pmmintrin.h')}
+    return iet, {}

 @iet_pass

Hi @stuartm-dugeo.  Is that a solution or what?

After this change, I was able to run devito on an A64FX system (it removes the xmmintrin.h from the source files).
They appear to be there to cause denormalised numbers to be set to zero in hardware - so possibly not that important?

FWIW its a quick'n'dirty hack...

Yes, I see.
The main issue is that a more global-ARM solution needs to be explored. For Cavium ThunderX2 ARM v8.1this still is not enough probably.

Thank you very much of course. Would you like to submit a PR on it?

After some discussion with @jaimesouza .
# 1) In devito/devito/archinfo.py
# I am using the follwing get_cpu_info(), insted of the current (latest master) version
@memoized_func
def get_cpu_info():
    try:
        # On linux, the following should work and is super quick
        cpu_info = {}
        with open('/proc/cpuinfo', 'r') as f:
            lines = f.readlines()
        get = lambda k: [i for i in lines if i.startswith(k)][0].split(':')[1].strip()
        cpu_info['flags'] = get('flags').split()
        cpu_info['brand'] = get('model name')
        # TODO: Other info omitted as currently unused
    except:
        # Fallback: rely on the slower `cpuinfo`
        cpu_info = cpuinfo.get_cpu_info()
    # At this point we have some information about the CPU, but we can't distinguish
    # between physical and logical cores yet, as:
    # 1) `/proc/cpuinfo` isn't as detailed on Power architectures as on Xeons
    # 2) `cpuinfo.get_cpu_info` only gives the logical core count
    # So below we try another approach
    # Detect number of logical cores
    logical = psutil.cpu_count(logical=True)
    if logical:
        cpu_info['logical'] = logical
    else:
        # Never bumped into a platform that make us end up here, yet
        # But we try to cover this case anyway, with `lscpu`
        try:
            cpu_info['logical'] = lscpu()['CPU(s)']
        except KeyError:
            warning("Logical core count autodetection failed")
            cpu_info['logical'] = 1
    # Detect number of physical cores
    physical = psutil.cpu_count(logical=False)
    if physical:
        cpu_info['physical'] = physical
    else:
        # Fallback: we might end up here on more exotic platforms such a Power8
        try:
            cpu_info['physical'] = lscpu()['Core(s) per socket'] * lscpu()['Socket(s)']
        except KeyError:
            warning("Physical core count autodetection failed")
            cpu_info['physical'] = 1
    return cpu_info
# 2) In devito/devito/passes/iet/misc.py
# I had to remove some intel specific headers
header = ()
#header = (cgen.Comment('Flush denormal numbers to zero in hardware'),
#          cgen.Statement('_MM_SET_DENORMALS_ZERO_MODE(_MM_DENORMALS_ZERO_ON)'),
#          cgen.Statement('_MM_SET_FLUSH_ZERO_MODE(_MM_FLUSH_ZERO_ON)'))
iet = iet._rebuild(body=(List(header=header),) + iet.body)
#return iet, {'includes': ('xmmintrin.h', 'pmmintrin.h')}
return iet, {}

@georgebisbas In addition to your 2nd edit, I had to remove all occurrences of -march=native from devito/compiler.py to be able to build version 4.2.3 with GCC 9.2.0. After that, I begin running tests, which resulted in #1500.


DFP multiple size support

As per the DFP spec

Android: https://developers.google.com/mobile-ads-sdk/docs/dfp/android/banner#multiple_ad_sizes
iOS: https://developers.google.com/mobile-ads-sdk/docs/dfp/ios/banner#multiple_ad_sizes

Multiple ad sizes
DFP allows you to specify multiple ad sizes which may be eligible to serve into a PublisherAdView.

See #151 for a work in progress. Should be usable for both iOS and Android, although there might still be some inconsistencies between the two.

I would love to get some feedback on the changes I introduced in the 2.0.0 alpha version, so if you could give that a try, that would be great!
https://github.com/sbugert/react-native-admob/releases/tag/v2.0.0-alpha.4

Hi,
I'm happy to give the new version a go.
I do have a question though, does this use the DFP sdk?
I got an email the other day stating that we're using a deprecated version
of the sdk, but I haven't investigated where yet.
On 11 Aug 2017 20:37, "Koen Punt"<EMAIL_ADDRESS>wrote:

I would love to get some feedback on the changes I introduced in the 2.0.0
alpha version, so if you could give that a try, that would be great!
https://github.com/sbugert/react-native-admob/releases/tag/v2.0.0-alpha.4
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sbugert/react-native-admob/issues/128#issuecomment-321901076,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABiVL3T24JTdntSeHP-z6xisa7lMbbKTks5sXK2UgaJpZM4NRUKx
.

Hi @Johan-dutoit, the library uses the "Google Mobile Ads SDK", which is the official library that both serves DFP and AdMob. However, the version of the SDK being used is defined in your project (Podfile and/or build.gradle).
For reference; I use version 7.21 of the Google Mobile Ads SDK for my iOS project.
If you follow the installation instructions it should get you in the right direction:
https://github.com/sbugert/react-native-admob/tree/next#installation

Version 2 is in beta and being tracking on the master branch. See the README on how to install this version, and check the release notes to see what changed.


[Config] Rename local-context to config parampersist

Description
We have a long story in email to ask rename original az local-context to az config parampersist.
This PR is to do this work, and we also add deprecate info for az local-context.
And please note that in this PR, we only change the UE part(command name), the internal class name/ function name were not changed.
Testing Guide
az config parampersist on

Command group 'config parampersist' is experimental and not covered by customer support. Please use with discretion.
 Please use with discretion.
 Please use with discretion.
{
"all": {
"identity_name": "xiaojxu-msi-3",
"resource_group_name": "img_tmpl_customizers000001"
}
}

az config parampersist delete resource_group_name

Command group 'config parampersist' is experimental and not covered by customer support. Please use with discretion.
Param persist value is deleted. You can run az config parampersist show to show all available values.

az local-context off

This command is implicitly deprecated because command group 'local-context' is deprecated and will be removed in a future release. Use 'config parampersist' instead.
Command group 'local-context' is experimental and not covered by customer support. Please use with discretion.
Local context is turned off, you can run az local-context on to turn it on.

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

Config

/azp run

/azp run


Suggestion : Better Menu on small screens

I would suggest having a Hamburger type menu for the episodes instead of just displaying at the top.
This will help for a better navigation on smaller devices like Mobile phones.
If you agree, I would love to work on this.

Implemented in v2


Breadcrumbs trail not working after update to 1.8.1

Seems it broken again after 1.8.1 update.

Do you get any errors?
Do you have Live Preview enabled?
Can you try on 1.9.0 and let me know

See no errors alerts.
Tried both mode "live on/off"
Current 1.10.1 version don't work too.

Oh, found how to enable console to see errors.

Ahh thank you, that error message helps alot.
I think it should be relatively simple to fix.

@Victor-IT @BricoleurSoul I think this is fixed in 1.10.2. Hard to tell because I couldn't reproduce, but I now have a safety check to ensure this will ensure

Show in edit mode ON:
The title of the parent index note has been truncated.

In both mode - trail not working in preview.


Fix small wall placement count preview (thanks to gynt)

Fixes the bug where the small wall placement preview only allowed the player to use half of their stone.
See issue #663.
Special thanks to gynt for finding and reverse-engineering the wall build preview!
Test build:
https://we.tl/t-6SiSwVALlU

Looks like they touched this part of the code in SHC-E, as with 1 remaining stone, you can only place 3 small walls, and then 1 more small wall. Same change works flawlessly in SHC-HD though.

Fixed the SHC-E difference bug. Now it works in both HD and SHC-E.
Test build:
https://we.tl/t-6FrqCx8Hn0

works as intended, tested also other walls to remain the same as before. tested with SHC and SHC-E

This does not work in multiplayer, it lags the game, and only half the wall is placed.

This does not work in multiplayer, it lags the game, and only half the wall is placed.

Note:
The game uses esi+constant to calculate those addresses. I don't know why, since esi seems to be always 0191D768.
E.g. esi+109E70 = 0191D768+109E70 = 01A275D8

Note:
The game uses esi+constant to calculate those addresses. I don't know why, since esi seems to be always 0191D768.
E.g. esi+109E70 = 0191D768+109E70 = 01A275D8

I only checked which addresses should be used. Didn't check where or what the code is doing.

I only checked which addresses should be used. Didn't check where or what the code is doing.

Nice work @LeSpec ! Thanks to that, we know what is going on here.
So, what is going on here is that the code checks twice if building a wall is allowed, 1, at the UI level, and 2, at the game level. It calls the same function at some point for each of these levels.
The current fix grabs the player id and the current wall type from a function that is only called by the UI. This worked in single player, because the UI is always involved (currentUIPlayerID == actionPlayerID). In multiplayer, when your fellow player places something, your UI is not involved, so this trick does not work (currentUIPlayerID != actionPlayerID).
As LeSpec points out, we need the two variables from a different function.
It is tricky, because what LeSpec suggests are addresses that are used after the action is already accepted by the UI, while this fix was specifically created to address the fact that the UI does not allow it at all (and the action is never created).
The current fix is placed inside this function:
getWallTilesThatCanBeBuilt(currentUIPlayerID) // in Single Player at the UI level, but not in Multi Player
getWallTilesThatCanBeBuilt(actionPlayerID) // in Single Player and Multi Player at the code level

As you can see the function is unaware of whether you are placing a low wall or a high wall, so the fix is to make the function aware of that fact by storing that value in memory and using it inside the function. The problem is, that value is in two different locations, depending on where we are at the UI level or the code level.
I think the most maintainable fix is to make this fix hook two locations, but then after getWallTilesThatCanBeBuilt is called, to adjust the outcome of the function in case of low wall placement. getWallTilesThatCanBeBuilt is called from 2 locations that are relevant (total of 3), 00437fd3 (ui level), 00502f85 (code level). The code level is easy to address, we should multiply by 2 if the action is a low wall command (that is in the stack, because it is a parameter to the function we are in). The ui level can keep its "CurrentlySelectedWallID" AOB.

Nice work @LeSpec ! Thanks to that, we know what is going on here.
So, what is going on here is that the code checks twice if building a wall is allowed, 1, at the UI level, and 2, at the game level. It calls the same function at some point for each of these levels.
The current fix grabs the player id and the current wall type from a function that is only called by the UI. This worked in single player, because the UI is always involved (currentUIPlayerID == actionPlayerID). In multiplayer, when your fellow player places something, your UI is not involved, so this trick does not work (currentUIPlayerID != actionPlayerID).
As LeSpec points out, we need the two variables from a different function.
It is tricky, because what LeSpec suggests are addresses that are used after the action is already accepted by the UI, while this fix was specifically created to address the fact that the UI does not allow it at all (and the action is never created).
The current fix is placed inside this function:
getWallTilesThatCanBeBuilt(currentUIPlayerID) // in Single Player at the UI level, but not in Multi Player
getWallTilesThatCanBeBuilt(actionPlayerID) // in Single Player and Multi Player at the code level

As you can see the function is unaware of whether you are placing a low wall or a high wall, so the fix is to make the function aware of that fact by storing that value in memory and using it inside the function. The problem is, that value is in two different locations, depending on where we are at the UI level or the code level.
I think the most maintainable fix is to make this fix hook two locations, but then after getWallTilesThatCanBeBuilt is called, to adjust the outcome of the function in case of low wall placement. getWallTilesThatCanBeBuilt is called from 2 locations that are relevant (total of 3), 00437fd3 (ui level), 00502f85 (code level). The code level is easy to address, we should multiply by 2 if the action is a low wall command (that is in the stack, because it is a parameter to the function we are in). The ui level can keep its "CurrentlySelectedWallID" AOB.

Do we even need the condition that checks PlayerID?
Maybe @LordHansCapon can explain what it does.

Do we even need the condition that checks PlayerID?
Maybe @LordHansCapon can explain what it does.

@LeSpec The function I hooked has the current player invoking this function in the stack at [esp+04] (esp+08 because I pushed ebx before it). I thought making sure that this is the currently controlled player will fix the issue in MP aswell. I guess not.

@LeSpec The function I hooked has the current player invoking this function in the stack at [esp+04] (esp+08 because I pushed ebx before it). I thought making sure that this is the currently controlled player will fix the issue in MP aswell. I guess not.


config: set the mailer URL options in an initializer

We guess the host and the protocol with the information we have on startup.
Signed-off-by: Miquel Sabaté Solà<EMAIL_ADDRESS>

I'm not convinced of this solution either... The thing here is that the mailer will fail on production/staging because there is no default URL options set there. In development, there are some made up default configs for it, but it's far from ideal.
So, my solution has been to set this up in an initializer. A safer solution would be to add a before_action hook into the applications controller with the following code:
def set_hostname
  ActionMailer::Base.default_url_options[:protocol] = request.protocol
  ActionMailer::Base.default_url_options[:host] = request.host_with_port
end

This has the obvious problem that it would be called always, which is an overkill. We can put it inside of the passwords_controller, which is the only place requiring a mailer right now. But I don't know how to feel about this. I guess it could be a good workaround, but then if we use mailers in other places, we have to make sure to set this up for each of the usages. The good thing about the previous code is that it's safer, since it doesn't make any guesses as my patch does.
Another solution, would be to:

Treat the fqdn as a non-secret, and put it inside of the configuration. This is a matter of semantics I guess, but the fqdn of your deployed configuration is not a secret. This looks like legacy code from the times where we didn't have a configuration in place.
Tell the user to specify if they plan to deploy Portus with SSL in development. In production/staging we would still force users to go for SSL, but in development, we might want to provide an environment variable (following configuration semantics, so like PORTUS_USE_SSL) that tells Portus what to do. This would be safer than assuming that if the developer has specified the environment variable PORTUS_MACHINE_FQDN, then SSL is in place.

I've only checked this with sendmail, not `smtp, but this shouldn't matter I think.


Generate CompressionInfo for OrderedMap PathStructs

This is necessary because the query path for OrderedMaps ends at the container of the ordered list rather than the ordered list while the input type is the ordered map. This means that the marshalled/unmarshalled JSON doesn't match the path. Thus this information is required in order to wrap the incoming/outgoing JSON such that the value matches the path.
Specifically CompressionInfo comprises of a "pre" part and a "post" part. The "post" part is what's used to wrap the JSON when marshalling (e.g. doing a replace), and the "pre" part is used to wrap incoming JSON (e.g. Lookup) so that it can be unmarshalled to its parent GoStruct.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

14 of 14 (100.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 84.226%

Totals

Change from base Build<PHONE_NUMBER>:
0.06%

Covered Lines:
1682

Relevant Lines:
1997

💛  - Coveralls


Some gender-neutral terms in my field

I am new to this, so forgive me if this has been brought up before; either way, I would appreciate any help.
I have been using the Gender Bias calculator for a few years now, and I really like it - but: there are a few 'male' terms that, in my field, are not so much gender-related as they are academic career-related (see below). For example, someone who has many publications, grants and/or manuscripts is typically someone who is further along in their career, not more likely male. (I realize that there is a gender disparity at the top of academia, but if I praise someone for their publications or grants, it doesn't mean that they are male: case in point, the last 10 letters I wrote to support promotion of women surgeons all skewed heavily male.
(If the algorithm skews male for a female candidate, is that seen as a compliment??)
Is there a way to alter the calculator for specific fields or study or profession?
Thanks
"experiment\w*"
"scholarship"
"result\w*"
"publication\w*"
"publish\w*"
"manuscript\w*"
"project\w*"
"grant"

Thanks for responding!
I cannot disagree with you, and I am certainly not suggesting that the
algorithm should be tweaked to better satisfy my needs, or heaven forbid,
my own biases. That said, context matters, and the letters that I write
vary from recommendations for trainees to support letters for peers.
Ultimately, you program makes me try to be a better letter writer, and its
transparency is helpful: it is easy enough to omit the trigger words that
are related to the person's career stage (research v. teaching, for
example) when running the letter through the program.
Again, I appreciate your taking the time to respond.
Francois Luks
On Tue, Nov 5, 2024 at 5:40 AM Stuart Lowe @.***> wrote:

@francoisluks https://github.com/francoisluks The original research
that this was based on makes the point that "letters for male applicants
included more reference to 'his research,' 'his ability', or 'his career,'
whereas letters for female applicants included more reference to 'her
teaching,' or 'her training.'".
Changes to the word list (or additional, subject-specific, word lists)
should come from proper research into the area. Just because we'd like
various words to be neutral, unfortunately, doesn't make them that. This
should be evidence-based as much as possible.
Yes, ideally none of these word would be used more for women or men. Then
it wouldn't matter which you used. But, in a society which has long
favoured men and disadvantaged women, using language that (unconsciously or
not) furthers the existing biases is a problem. It isn't that "skew[ing]
male for a female candidate" is a "compliment". It is about you, as someone
writing a recommendation letter, being aware when your wording is
disadvantaging women (and advantaging men).
Yes, you can fork this project and create your own personal word lists if
you want. It is only a basic web page so doesn't require anything fancy to
host it if you want your own.
—
Reply to this email directly, view it on GitHub
https://github.com/slowe/genderbias/issues/6#issuecomment-2456825877,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BKLOBV7OZNLPSUGQFE6WHXDZ7COBXAVCNFSM6AAAAABMCWG37KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJWHAZDKOBXG4
.
You are receiving this because you were mentioned.Message ID:
@.***>

I'm glad it is useful to you. Another person opened an issue to tell me it wasn't useful!
If you download a zip of the files you could make your own version specific for your field. Perhaps edit https://github.com/slowe/genderbias/blob/main/dictionaries/medical.json to create your own custom dictionary then make sure that dictionary JSON file is an  in https://github.com/slowe/genderbias/blob/main/index.html.


[WEB] Client Side Error Handling for Web in User Authentication Flow

This ticket is kind of a catch all - basically going through the code and updating every situation of which there is possibility for a client error.
Current Behavior: Right now, I don't believe there is a current Behavior for error handling client side errors.
Expected Behavior: Using inline error messages to handle client side errors

Checking design inconsistencies, possibly mobile / desktop web variations are different, this design seems wrong with a red "placeholder text" or "input value" instead of a validation error under the input.

Updated the issue description, removed the screenshot to avoid confusion, talked to designers and the conclusion is to stick to one consistent appraoch from the designs (red error text under the input and red underline)


Split query compat tests into separate tests

Description
Closes #2054.

Readiness checklist

[ ] I added/updated unit tests.
[ ] I added/updated integration/compatibility tests.
[ ] I added/updated comments and checked rendering.
[ ] I made spot refactorings.
[ ] I updated user documentation.
[ ] I ran task all, and it passed.
[ ] I ensured that PR title is good enough for the changelog.
[ ] (for maintainers only) I set Reviewers (@FerretDB/core), Assignee, Labels, Project and project's Sprint fields.
[ ] I marked all done items in this checklist.

By separating the tests, it takes longer on test-integration-pg and test-integration-tigris.
Currently on main test-integration-pg takes around 12min to 13min, in this PR it takes about 15min.
On main test-integration-tigris takes 7min to 8min, in this PR it takes 9min.


Fix transaction & subscription spelling in documentation.

trancastion is now transaction
subbscriptions is subscriptions

Super thank you

Thank you, @AlexXanderGrib!


Preprocessor to generate python code from notebook

I think the current preprocessors for notebooks work by pickling and depickling the code.
Can we instead have a preprocessor that would work by generating a python file that strips out everything except

imports
class defining the train & predict methods

I created a simple preprocessor which I've been using in my demos.
https://github.com/jlewi/kubecon-demo/blob/master/fairing_util.py
Could we get this checked into the main repo so that users don't have to write it themselves?

/assign @abcdefgs0324


swapchain recreation error

Program: ...ild\windows\app\bin\debug\AMD64\vulkan_samples.exe
File: C:\Users\AkumaRRR\Desktop\Vulkan-Sample\framework\vulkan_sample.cpp
Line: 568

Expression: render_context && "Render context is not valid"

Can you elaborate? Hard to help with so little information.

What did you do to trigger the error?
What platform?
What implementation (GPU, Vendor, driver)?

I simply run the sample in debug mode and this is the result:

Win10 22H2
RTX 3060 Ti (driver 531.14)
ucrtbased.dll!common_assert_to_message_box<wchar_t>(const wchar_t * const expression, const wchar_t * const file_name, const unsigned int line_number, void * const return_address) Line 388	C++
ucrtbased.dll!common_assert<wchar_t>(const wchar_t * const expression, const wchar_t * const file_name, const unsigned int line_number, void * const return_address) Line 424	C++
ucrtbased.dll!_wassert(const wchar_t * expression, const wchar_t * file_name, unsigned int line_number) Line 444	C++
vulkan_samples.exe!vkb::VulkanSample::get_render_context() Line 568	C++
vulkan_samples.exe!vkb::Platform::update() Line 199	C++
vulkan_samples.exe!vkb::Platform::main_loop() Line 152	C++
vulkan_samples.exe!WinMain(HINSTANCE__ * hInstance, HINSTANCE__ * hPrevInstance, char * lpCmdLine, int nCmdShow) Line 54	C++

All other samples works fine.

@SaschaWillems i likely know the offending line will test and submit a patch for this
This sample may not create a render context in the same way as other samples
But the offender is the on post draw hook in platform

I reproduced this bug and fixed it with #726

Thx!


Watch emit is not consistent

I wrote a script using ts.createWatchCompilerHost and with visitors im modifying emits and for first one it works great but when i save second time my edit from last time affects results. For example when i see @CustomElement decorator i delete that decorator and emit some lines of code. Next time that file compiles with watch since that decorator is deleted neither code i actually try to emit works or decorator comes back to me. I'm curious whats right way for this kind of edits.
Deleting decorator from decorator array
const SpliceDecorator = (
  node: ts.Node,
  Condition: (x: ts.Decorator) => boolean
) => {
  if (!node.decorators) return;
  const Dec = node.decorators.findIndex(Condition);
  if (Dec > -1) {
    const R = node.decorators[Dec];
    const Decs = Array.from(node.decorators);
    Decs.splice(Dec, 1);
    if (Decs && Decs.length) {
      node.decorators =  ts.createNodeArray(Decs);
    } else {
      delete node.decorators;
    }
    return R;
  }
};

Updating emited code / class
    const AssignTag = ts.createProperty(
      undefined,
      [ts.createModifier(ts.SyntaxKind.StaticKeyword)],
      "Tag",
      undefined,
      undefined,
      ts.createStringLiteral(Key)
    );
    let Nodes: any[] = [AssignTag];
    Nodes = Nodes.concat(node.members);
    node = ts.updateClassDeclaration(
      node,
      node.decorators,
      node.modifiers,
      node.name,
      node.typeParameters,
      node.heritageClauses,
      ts.createNodeArray(Nodes)
    );

I cut some unrelated parts if some things look unnecessary its probably because of this.

Can you provide a minimal repro of the code you're using for the APIs?

https://github.com/EmpatiLab/Linker/tree/master/Source


Config api_key_cmd did not return a value when executed

I love this plugin but I can't make it work.
I use Plug to install it:
    Plug 'jackMort/ChatGPT.nvim'
    Plug 'MunifTanjim/nui.nvim'
    Plug 'nvim-lua/plenary.nvim'
    Plug 'nvim-telescope/telescope.nvim'

I have a secret.txt.gpg in my home directory that was encrypted with the following command:
gpg --encrypt --recipient [my email] secret.txt
The contents of the secret.txt are $OPENAI_API_KEY=xxxxxxxxxxxxxxxx
And I try to get the OPENAI_API_KEY like so:
local home = vim.fn.expand("$HOME")
require("chatgpt").setup({
    api_key_cmd = "gpg --decrypt " .. home .. "/secret.txt.gpg"
})

The error I'm getting is Config api_key_cmd did not return a value when executed
What am I doing wrong here ?
Thanks!

secret.txt should only have the plain openai api key. Try again after removing $OPENAI_API_KEY= from your secret.txt file

I have tried that but I got the same error message.

I also have the same error. I am using the security bin from macOS to get the key from keychain. The same command prints the key in a fresh session. Can share the command when I am back at my laptop.

@psteinroe as a workaround, I set OPENAI_API_KEY in my zshrc:
export OPENAI_API_KEY=$(security find-generic-password -w -s openai)

The workaround that I found for this is using Lua to print the value to stdout:
local function getKey()
    local cmd = 'gpg --decrypt ~/.lsp/chatgpt/credential.txt.gpg'
    local handle = io.popen(cmd)
    local result = nil
    if handle then
        result = handle:read("*a")
    end
    return result
end

require('chatgpt').setup({
    api_key_cmd = getKey
})


warning in funModeling::freq

Hey Pablo!
Thank you for the package, I just discovered it but I think it will be so useful.
I got that warning when using freq. It says to report, so here I am.
Cheers
Gaelle
Warning message:
The <scale> argument of guides() cannot be FALSE. Use "none" instead as of ggplot2 3.3.4.
ℹ The deprecated feature was likely used in the funModeling package.
Please report the issue at https://github.com/pablo14/funModeling/issues.
This warning is displayed once every 8 hours.
Call lifecycle::last_lifecycle_warnings() to see where this warning was generated.

Hi! Thank you so much for this report :)
Now I don't have the time to support the library, I'm open to accepting PR


Average cumulative returns by quantile: performance improvement

Call utils.common_start_returns just once as it is very computing
intensive.
Split the calculation of average cumulative returns from the plotting code
so that it's possible to reuse the same calculated average returns in the two
subsequent calls to plot_quantile_average_cumulative_return

Seems like you need to rebase.

Done

LGTM :+1:


Feat: bmt integration tests

is there still a plan for this PR? @bvrooman

is there still a plan for this PR? @bvrooman

Yes - I will give this PR some TLC when I get a chance! It's purpose is to ensure the BMT functionality at an integration level.


homepage h1

@d88naimi - can you address the second portion of the screenshot? We need "The power of Zesty's headless CMS" to be an H2 (keep current CSS styling)
This is actually on the "headless CMS page" : https://kfg6bckb-dev.webengine.zesty.io/solutions/headless-cms/


Add threat grid tests

Added tests to be somewhat consistent with the current ones. Not sure if there's a better approach.

lgtm :+1:

Yeah, this looks fine to me.


Upgrade Axios to address recent vulnerabilities

Versions prior to 0.21.1 contain Server-Side Request Forgery (SSRF) vulnerability. See this advisory for details.

Update dev dependencies to reflect latest log4js-node.

43630d7


Enable schema.org structured data markup for Google (from 1975)

Problem Description
We need to provide a schema.org markup for Google search results to show rich description of each page.
Expected Behavior/Solution

Each LUX page will provide schema.org data for the Google crawler to process.

Requirements

Mapping provided by @rs2668: https://deploy-preview-540--linked-art.netlify.app/cookbook/mappings/schema.org/

Needed for promotion
If an item on the list is not needed, it should be crossed off but not removed.

[x] Wireframe/Mockup - Heather - N/A
[ ] Committee discussions - Sarah
[ ] Feasibility/Team discussion - Sarah
[x] Backend requirements - N/A
[ ] Frontend requirements- TBD
[ ] Questions

List of questions for discussions. Answers should be documented within the issue.

UAT/LUX Examples
Example endpoints from LUX to use for development and testing, if applicable
Dependencies/Blocks

Blocked By

https://git.yale.edu/lux-its/lux-web/issues/1976 (old tix)
https://github.com/project-lux/lux-frontend/issues/73 (new tix)

Blocking

https://git.yale.edu/lux-its/lux-web/issues/1687
new tix: https://github.com/project-lux/lux-frontend/issues/24

Related Github Issues
Issues that contain similar work but are not blocking or being blocked by the current issue.
Related links

Introduction to structured data markup in Google Search
Generate structured data with JavaScript
Rich Results Test

Wireframe/Mockup
N/A

Ok to close


Learning Packages - Broadlink Switch

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
0.35.0
Python release (python3 --version):
3.4.2
Component/platform:
Switch.Broadlink
Description of problem:
When called lear_command service, it show the received package as a persistent notification, but the package code is partially showed (it is bigger than appears) and there is no way to select or copy it.
Expected:
We need to copy the code and paste it in the configuration.yaml
Problem-relevant configuration.yaml entries and steps to reproduce:
switch:
  platform: broadlink
  host: RM IP
  mac: 'MAC'
  switches:
    tvsony:
      friendly_name: 'Main TV'
      command_on: 'JgBUAFATExQnEyYUJxMUEyYUFBMnExQTExQTExQAAx5NFBIVJhQmFCcTFBMnExMUJxMTFBMTExQUAAMeTRQSFCcTJxQnExMUJhQSFSYUExMTFBQTFAANBQAAAAA='
      command_off: 'JgBUAFATJxMmFCcTJxQSFCgTEhQnExMUFBMTFBMAAw5OEycUJhMnFCcTExQnExMTJxQUExIUExQUAAMNTRQnEycTJxQmFBMUJhQTFCcTExQTExQTFAANBQAAAAA='

Call broadlink.learn_command service
wait RM or Mini3 red light and press the remote button to learn
a persistent notification with the package will be showed

Traceback (if applicable):

Additional info:

I had the same problem.
Try looking in the Hass window (the text that you see in the command line prompt after typing hass) it will show the full command.

I solved this by setting for info the switch.broadlink logging.
But I think it would be easier getting the package from the frontend.

I dont understand how to get this, i used the all in one installer script - I dont type "hass" to run anything and I cant copy from the persistent notification. Please advise


Include required jars for AD-auth

https://github.com/flyway/flyway-docker/issues/138

Hi,
Since 10.17.0, Flyway has now been shipping with the required jars for AD Auth within the cli


AlertRules in compiled YAML manifests are reordered on each compilation

See #projectsyn-ig (don't want to post customer info here)
Actual Behavior

Each time the catalog is recompiled, alertrules are reordered, causing a new catalog push

Expected Behavior

Catalog recompilation without actual changes do not cause catalog pushes

https://github.com/appuio/component-openshift4-monitoring/blob/70b05a9d2ca1dfbc34e1d3f7e55499d046b51b34/component/rules.jsonnet#L148 Is this supposed to sort rules or groups?

Fixed in #35


Consider checking in the sources for generated bindings

Silk.NET does not currently check in the generated bindings for things like Vulkan or OpenGL.
This presents a couple scenarios that are less than desirable:

You can't see the bindings without cloning the repository
You can't open the solution in Visual Studio (or another IDE) and have it "just work"
You can't trivially diff the API surface between two given releases
You can't easily perform historical builds
etc

The generated sources are overall small and Git is highly efficient at compressing the source files and the diffs of the files over time. For example, the current repo in compressed form is only 9.80 MiB (representing all 397 commits across all 5 branches).
As such, it would be great if Silk.NET could begin checking in the generated bindings and updating them whenever it targets a new baseline version of the respective specs.

The Working Group has decided to check in these files and will implement the change shortly.


Ordered CQL Expressions in Logic Highlighting HTML

Which fqm-execution feature is this request related to?
Logic highlighting and html generation
Describe the solution you'd like
Re-order the CQL expressions into a more intuitive format. Currently, the CQL expressions are semi-arbitrarily ordered based on the ingestion of libraries as read from the measure bundle. Suggest explicitly ordering the CQL expressions, likely putting population definitions at the top. Function definitions could be at the bottom or in some other consistent logical order.
Additional context
Example of current html:

Population definitions are mixed in with other definitions, seemingly at random.

Just to provide another example, this is an example of what the order of expressions currently is:

This feature would change it so that definitions used as measure populations are first, "helper definitions" are in the middle, and function definitions are at the bottom, like so:

We have implemented this in #242. If there is concern on how this will affect users of fqm-execution, we will consider changing the re-ordering to be "opt in" rather than the default behavior


Usage with Next.js 13 ./app

Hi, currently trying to integrate the Sanity Studio inside my site with <NextStudio ... . My project is using the new ./app directory and "sanity": "3.0.0-rc.0" + "next-sanity": "1.0.8",.
Currently getting this error.

I can of course install styled-components as a dependency, and that appears to let it attempt to render.

But that breaks all of my other pages in ./app.

My branch testing this is here: https://github.com/alvarlagerlof/portfolio/pull/610

Might need to add sanity and next-sanity in next.config.js?
https://beta.nextjs.org/docs/api-reference/next.config.js#servercomponentsexternalpackages

Might need to add sanity and next-sanity in next.config.js?
beta.nextjs.org/docs/api-reference/next.config.js#servercomponentsexternalpackages

That worked better, but for now it seems I am blocked by lack of a code-input plugin on v3.

You should be able to use code-input on v3 using:
npm install --save-exact @sanity/code-input@studio-v3

https://github.com/sanity-io/code-input#usage

Nevermind, I still can't use both parts without styled-components.

@alvarlagerlof could you push the changes @steven-tey suggested to https://github.com/alvarlagerlof/portfolio/pull/610 and I can give it a look?

Yes! It's pushed now to this. Current state is the studio doesn't render without installing styled-components and the rest breaks when I do so.

Thanks, I'll give it a go and use your branch as base @alvarlagerlof 🙌

Hi @alvarlagerlof, we just released v3 which supports Next 13 ./app for Studio embeds 🎉

Hi @alvarlagerlof, we just released v3 which supports Next 13 ./app for Studio embeds tada

Cool. I am still blocked it seems because @sanity/code-input and @sanity/portable-text-editor both require server components. The code editor also hasn't been updated for "use client".

It's not necessary to mark those dependencies as use client.
Refer to https://github.com/sanity-io/nextjs-blog-cms-sanity-v3/tree/main/app/studio/[[...index]]
You still need to install the styled-components dependency, but you don't need to setup SSR for it.

I did follow that, and it fixed the css, but the code input still tried to run client things.

Here you go @alvarlagerlof https://github.com/alvarlagerlof/portfolio/pull/646

I'm having a similar issue are we sure this is fixed?

I'm having a similar issue are we sure this is fixed?

Just install styled-components as an extra dependency and this will go away. They haven't included it in their package yet.

@alvarlagerlof I should have noted - i installed "styled-components": "^5.3.6"

styled-components is now a peer dependency: https://github.com/sanity-io/next-sanity#next-sanitystudio-peer-dependencies


[docs[[dex] Add slippage page in fern

Preview to the page : https://paradex-preview-f9dc26b0-97b3-4c84-b3d7-44daa2bc190b.docs.buildwithfern.com/documentation/trading/slippage

@aweaver89 pending last part of your comment

So let's just remove that last text block for now.  This will make it so that we can get this up.  We already went through what slippage is above so this part isn't as urgent.

@aweaver89 pending last part of your comment

So let's just remove that last text block for now. This will make it so that we can get this up. We already went through what slippage is above so this part isn't as urgent.

actually @filbert-nicholas took care of this quickly and changed the last heading to How to .... instead of Steps to ....
Going to double check quickly but should be gtg!


dispatch init fails when targeting usgovvirginia

Tool
Name: Dispatch
Version: 1.5.3
OS: Windows
Describe the bug
I've been whitelisted into QnA Maker in Azure Government which now allows me to attempt to use the Virtual Assistant Template in Azure Government.  The deploy_congnitive_models.ps1 script from that template calls:
dispatch init --name <dispatch name> --luisAuthoringKey <key> --luisAuthoringRegion <gov region> --dataFolder <folder>
This call hangs when passed --luisAuthoringRegion usgovvirginia and the error in deploy_cognitive_models_log.txt says, "Invalid region"
To Reproduce
Steps to reproduce the behavior:

From PowerShell 6 command line, connect to Azure Government by entering:
az cloud set --name AzureUSGovernment
Login in to your Azure Gov subscription using your Azure Gov user account:
az login
run the dispatch command  by entering:
dispatch init --name <dispatch name> --luisAuthoringKey <key> --luisAuthoringRegion usgovvirginia --dataFolder <folder>

Expected behavior
The command should complete without hanging or reporting "Invalid region" in log
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
[bug]

@managedcoder, for gov, one would need to use "--gov true" parameter.  Let me know if you are still having troubles getting it to work with the gov parameter.

Thanks so much!  That did the trick and dispatch init now completes successfully

@managedcoder, thanks!

@tsuwandy can you or someone familiar with the tool update dispatch's readme with the --gov flag and any other new flags?


Fix transparent tr-pass

I plan on writing an autest for this, which will require privilege to run to exercise the transparent case.
This fixes #9146

Oops didn't push to fork.


'Dart' booster type is ignored in case of using parameter Xgb_model in train

Hello,
I've encountered with strange behaviour
num_round = 150
bst0 = xgb.train(plst, dtrain, num_round, watchlist, early_stopping_rounds=40)
bst0.save_model(args.model + '.150')
num_round = int(args.ntree)
bst = xgb.train(plst, dtrain, num_round, watchlist, early_stopping_rounds=40, xgb_model=bst0)
bst.save_model(args.model)
Stdout for bst0:
....
[143]   test-auc:0.8941
[18:10:56] src/tree/updater_prune.cc:74: tree pruning end, 1 roots, 14 extra nodes, 0 pruned nodes, max_depth=4
[18:10:56] src/gbm/gbtree.cc:575: drop 21 trees, weight = 0.869565
[144]   test-auc:0.894481
[18:11:02] src/tree/updater_prune.cc:74: tree pruning end, 1 roots, 20 extra nodes, 0 pruned nodes, max_depth=5
[18:11:02] src/gbm/gbtree.cc:575: drop 0 trees, weight = 1
[145]   test-auc:0.89451
[18:11:09] src/tree/updater_prune.cc:74: tree pruning end, 1 roots, 18 extra nodes, 0 pruned nodes, max_depth=5
[18:11:09] src/gbm/gbtree.cc:575: drop 0 trees, weight = 1
[146]   test-auc:0.895184
[18:11:16] src/tree/updater_prune.cc:74: tree pruning end, 1 roots, 16 extra nodes, 0 pruned nodes, max_depth=6
[18:11:16] src/gbm/gbtree.cc:575: drop 28 trees, weight = 0.869565
[147]   test-auc:0.895616
[18:11:23] src/tree/updater_prune.cc:74: tree pruning end, 1 roots, 18 extra nodes, 0 pruned nodes, max_depth=6
[18:11:23] src/gbm/gbtree.cc:575: drop 0 trees, weight = 1
[148]   test-auc:0.893348
[18:11:29] src/tree/updater_prune.cc:74: tree pruning end, 1 roots, 18 extra nodes, 0 pruned nodes, max_depth=5
[18:11:29] src/gbm/gbtree.cc:575: drop 0 trees, weight = 1
[149]   test-auc:0.895612
[18:11:36] src/tree/updater_prune.cc:74: tree pruning end, 1 roots, 18 extra nodes, 0 pruned nodes, max_depth=7
Stdout for bst - and there're no dart weights - bst has 'gbtree' booster type:
[0]     test-auc:0.895676
Will train until test-auc hasn't improved in 40 rounds.
[18:11:44] src/tree/updater_prune.cc:74: tree pruning end, 1 roots, 16 extra nodes, 0 pruned nodes, max_depth=6
[1]     test-auc:0.895739
[18:11:52] src/tree/updater_prune.cc:74: tree pruning end, 1 roots, 14 extra nodes, 0 pruned nodes, max_depth=7
[2]     test-auc:0.895768
[18:11:59] src/tree/updater_prune.cc:74: tree pruning end, 1 roots, 16 extra nodes, 0 pruned nodes, max_depth=7
[3]     test-auc:0.895871
[18:12:06] src/tree/updater_prune.cc:74: tree pruning end, 1 roots, 14 extra nodes, 0 pruned nodes, max_depth=7
[4]     test-auc:0.895949
Thanks!

Old bug, no answer. Closing


[FLINK-6087] Fix file-path filtering in ContinuousFileMonitoringFunction

The Files Filter is only applied to directories, so there is no way to filter individual files. This fixes it.

It would be good to also have a test for this.
R: @kl0u could you please have a look? Now it's not much but with adding a test it's more than just one line. 😉


Update conda build env to openjdk 20

@leshikus can you take a look at this? We want to update to openjdk 20 in conda for developers and public builds per @vlad-penkin . Older openjdk is fine on the build containers as long as it is only for CI.


Does this support ranges to display multiple sliders?

Does this support ranges to display multiple sliders?
Really awesome work!

:+1:

Apologies for the late response. Alas, ranges are not supported. It would add significant complexity and confusion to both the public API and the implementation to support both a normal value slider and a range version in one control.
The only suggestion I can make is that you could find an open source range slider and retrofit the ASValuePopUpView on top of it. One complication I can envisage is that when the two sliders are close together, extra logic will need to be provided to prevent the popUpViews from overlapping.
Good luck in your mission and please extract as much code from this project as needed if it will be of benefit.
All the best,
Al


Prevent prevent file descriptor leak in ShrinkDebugFile()

Looks good; personally I'd have just put if instead of else if so that it just checks if the file is not null and closes it regardless of if the other function has run properly.
Course I'd have just written the first if slightly differently anyway and there would only be 2 calls to fclose instead of 3 lol


use byte array operation instead of sturct.unpack

FYI
But I don't know, this modification make faster.

Sorry, I recently could not take time for checking the pull req.
I understand your modification and cannot find any problems there. But after merge this branch, test/test_fst.py is failing. I'll try to investigate that.
FYI, we can execute tests by following steps.
$ git clone https://github.com/mocobeta/janome.git
// test_fst.py does not need dictionary data, we can just run.
$ cd janome
$ python tests/test_fst.py

// other tests need to build dictionary.
// download mecab-ipadic from here, and extract to janome/ipadic directory
// http://sourceforge.net/projects/mecab/files/mecab-ipadic/2.7.0-20070801/
$ cd janome/ipadic
$ tar xzf mecab-ipadic-2.7.0-20070801.tar.gz
// make the dictionary (byte data and source codes). this takes some minutes.
$ ./build.sh mecab-ipadic-2.7.0-20070801
// execute tests
$ cd ..
$ unzip ipadic/sysdic.zip
// this takes about one minute first time to compile dictionaries.
$ python tests/test_dic.py

struct.unpack is no longer used


flaky test: Microsoft.AspNetCore.SignalR.Tests.HubConnectionHandlerTests.SendToOthersInGroup

Seen this fail a couple of times now. Never in isolation, always as part of a random ci run.
Failed   Microsoft.AspNetCore.SignalR.Tests.HubConnectionHandlerTests.SendToOthersInGroup(hubType: typeof(Microsoft.AspNetCore.SignalR.Tests.HubT))
Error Message:
 System.TimeoutException : Operation in SendToOthersInGroup timed out at C:\projects\signalr\test\Microsoft.AspNetCore.SignalR.Tests\HubConnectionHandlerTests.cs:1199
Stack Trace:
   at System.Threading.Tasks.TaskExtensions.<OrTimeout>d__6`1.MoveNext() in C:\projects\signalr\test\Microsoft.AspNetCore.SignalR.Tests.Utils\TaskExtensions.cs:line 60
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task task)
   at Microsoft.AspNetCore.SignalR.Tests.HubConnectionHandlerTests.<SendToOthersInGroup>d__42.MoveNext() in C:\projects\signalr\test\Microsoft.AspNetCore.SignalR.Tests\HubConnectionHandlerTests.cs:line 1199
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
[xUnit.net 00:00:13.3133086]     Microsoft.AspNetCore.SignalR.Tests.WebSocketsTransportTests.WebSocketsTransportSendsUserAgent [SKIP]
Skipped  Microsoft.AspNetCore.SignalR.Tests.WebSocketsTransportTests.WebSocketsTransportSendsUserAgent
Total tests: 1211. Passed: 1199. Failed: 1. Skipped: 11.

Happened here again:
https://ci.appveyor.com/project/aspnetci/signalr/build/1.0.3965

https://ci.appveyor.com/project/aspnetci/signalr/build/1.0.3954

https://ci.appveyor.com/project/aspnetci/signalr/build/1.0.4059

This might be a test client bug.


chore: JSON form autogenerate control tooltip text changes

Description
Fixes #12206
Type of change

Change in copy of tooltip text

How Has This Been Tested?
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:red_circle: Total coverage has decreased
// Code coverage diff between base branch:release and head branch: task/json-form-modify-tooltip

Status
File
% Stmts
% Branch
% Funcs
% Lines

:red_circle:
total
55.55 (0)
37.01 (-0.01)
35.18 (0)
55.85 (-0.01)

:red_circle:
app/client/src/utils/WorkerUtil.ts
88.98 (-0.78)
70.59 (-1.96)
100 (0)
92.38 (-0.95)

:red_circle:
app/client/src/utils/autocomplete/TernServer.ts
52.71 (-0.23)
40.83 (-0.84)
36.21 (0)
56.74 (-0.25)

/ok-to-test sha=41b3d08

@ashit-rath this reminded me, do we want to add a beta tag here?
Also is the documentation on track cc @dilippitchika @Pranay105

@somangshu Is this a normal practice to place a beta tag on new widgets?

@ashit-rath
So the decision depends on any limitations we are already aware of, in that case the beta label need to be place on the widget

@somangshu  Can the 50 fields limit be considered as a limitation for now, if not then can you think of any?

None for now, 50 fields on one form also is a bit of extreme.
@dilippitchika do you have any thought, should we have the beta tag?


fix: exported / referenced plugin same instance

It's important that the exported plugin reference and the one used in the exported configs are the very same – else one will get an error like:

ConfigError: Config "node/flat/recommended-module": Key "plugins": Cannot redefine plugin "n".

This since two plugins are only allowed to have the same key if they are === to one another. See: https://github.com/eslint/eslint/blob/0583c87c720afe0b9aef5367b1a0a77923eefe9d/lib/config/flat-config-schema.js#L373-L375
This caused a regression for me in https://github.com/voxpelli/eslint-config/blob/638bbad2580bc4246527a6f0d989dc8194547475/base-configs/node.js#L10

🤔 What a strange bug!

Yeah, it’s a bit odd that flat configs allows a redeclaration of a plugin but only if that plugin is === to the existing one.
Creates odd expectations.


Suggestion to add multi-level relations to ActiveRecord::extraFields()

Currrently there is no possibility to dynamically manage relations that should be included to respond in case when relation level is bigger than one.
For example we have model Business and you can define Locations relation like this:
    public function extraFields()
    {
        $fields = parent::fields();
        $fields[]='locations';
        return $fields;
    }

And then you can include it to respond using  ?expand="locations"
But if we want to include Locations together with related location Schedules, then that doesn't work.
I suggest to implement multi-level relations (using dot notation):
    public function extraFields()
    {
        $fields = parent::fields();
        $fields[]='locations';
        $fields[]='locations.schedule';
        return $fields;
    }

And then it will be possible to use it this way: ?expand="locations, locations.schedule"
I think it would be nice, because it's very often needed to manage relation records that should be included to respond.
Is it possible?

+1

+1

+1

+1

+1

+1

+1

Where in code YII2 looks for "expand" parameter to choose which extra fields must be shown?

Where in code YII2 looks for "expand" parameter to choose which extra fields must be shown?

https://github.com/yiisoft/yii2/blob/master/framework/rest/Serializer.php#L159

+1

This problem make me cry, really!
I want to get list of "items" with author.
And in author I want to get Full name extra field.
But it's not working at all. Expand not using custom fields.

+1

For the moment, I've worked around this problem by overruling the fields() call in my ActiveRecord model.
class MultiLevelActiveRecord extends \yii\db\ActiveRecord
{
    public function fields()
    {
        $fields = parent::fields();

        // Add multi-level expanded fields
        $expandFields = explode(',', Yii::$app->request->getQueryParam('expand'));
        foreach ($expandFields as $field)
        {
            if (strpos($field, strtolower($this->formName()).'.') === 0)
            {
                $fields[] = substr($field, strlen($this->formName()) + 1);
            }
        }

        return $fields;
    }
}

That enabled me to use:
?expand=category,author.name
But only based on exact modal names, and not based on relations. It's dirty, but does the job for me.

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

http://domain/restapi/users/1/posts/123 (#3319) +1

+1

I'm currently implementing something to handle this issue. I use scopes, so I define each scope in the model and the client requests the scopes to be returned. The request is
?scope=profile,orders,locations

and in the model I define them as arrays like in fields(). I personally don't think that the client should be able to define exactly what to be returned it should be limited to scopes defined on the server. It can be easily used with oauth server(in my case) which limits the visible scopes for each client.

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

Please, use :+1: emoticon to the first message in this issue instead of adding more "+1" comments. Thank you!

@SilverFire i think the reason is because reactions emojis doesn't get attention.

We have enough members asking for this feature, so we already know that we must take care about this issue. Clicking "+1" is OK to let us know, that you wanna this kind of enhancement too.

I'm not locking the conversation. We will be happy to hear more feedback from you

There are currently two PR's from @mdmunir with two different approaches on how to solve this feature request #11493 and #11476 I think the people here should check them and see which oneis easier to implement and use.

+1

+1 this is the answer what i need, thanks you.

+1

I would like to suggest an addition that would complement the resource level configuration options. I would also suggest that we allow configuration in the application URLmanager. My proposal can be seen here. http://www.yiiframework.com/forum/index.php/topic/72222-enhancement-suggestion-configurable-extra-patterns-in-mainphp/

+1

+1

+1


Enhancement Unify error code and enable customed err msg for BE internal errors

Proposed changes
Issue Number: #11874
Problem summary
This PR unified error code in Status and allows developers to create BE internal errors with more detailed error message, and freely determine whether to capture stacktrace if compile BE with ENABLE_STACKTRACE on.
    auto status = Status::Error<SOME_ERROR_NEED_TO_STACKTRACE>("this is error msg");
    // don't capture stacktrace this time
    auto status1 = Status::Error<SOME_ERROR_NEED_TO_STACKTRACE, false>("this is error msg");

// be/src/common/status.h
// always don't capture stacktrace
template <ErrorCode code>
static constexpr bool capture_stacktrace() {
    return code != E_OK
        && code != E_END_OF_FILE
        && code != SOME_ERROR_NO_NEED_TO_STACKTRACE
        ...
        ;
}

Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[ ] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[ ] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[ ] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 35.72 seconds
load time: 450 seconds
storage size:<PHONE_NUMBER>8 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221201171555_clickbench_pr_56358.html


Internal package error

I'm getting an internal package reference error.  This seems like it could be due to merging the samples into one repo.
XPS-13:~/go/src/github.com/Azure/azure-sdk-for-go-samples$ dep ensure
XPS-13:~/go/src/github.com/Azure/azure-sdk-for-go-samples$ go test -v ./resources/.
resources/cleanup.go:12:2: use of internal package github.com/Azure/azure-sdk-for-go-samples/vendor/github.com/Azure-Samples/azure-sdk-for-go-samples/internal/config not allowed
resources/deployment.go:15:2: use of internal package github.com/Azure/azure-sdk-for-go-samples/vendor/github.com/Azure-Samples/azure-sdk-for-go-samples/internal/iam not allowed
Thank you for your contribution and feedback! Help us review faster by providing the following information:
This issue is a: (mark with an x)

[ ] bug report
[ ] request for new sample
[ ] documentation issue or request
[ ] regression (a behavior that used to work and stopped in a new release)

Steps to reproduce:
Errors and log messages:
Expected behavior:
OS and Go versions:
Further info:

The sample repo is using dep as dependency management tool, so please put the repo in the following path to avoid this kind of error.
$(GOPATH)/src/github.com/Azure-Samples/azure-sdk-for-go-samples

@ArcturusZhang  I am getting the same error even after running with dep and putting azure-sdk-for-go-samples in  this $(GOPATH)/src/github.com/Azure-Samples/azure-sdk-for-go-samples
directory.

Hi @hetalsonavane now the sample repo has been switched to go modules as dependency management. With go modules, you could put your code wherever you want


Extracting/streaming "scripts"

Just wondering if it would be possible to extract the "scripts" that are referenced in some of the extracted csv files. For example, "map_script.csv" references scripts named "sc_map_", but those assets aren't (yet) present in the export.
Thank you for your work on the project =D

The latest version should already had it. They are in Serial/Res/Map

Oh! I just completely missed that. Thank you =D


Don't invoke XCTContext when running in the Testing framework.

Fixes #883

It does fix the issue! Thanks for the quick fix!


Added a songs tab to the music screen

The songs tab lists all available songs. This allows the user
to play songs that are not part of an album.
This should provide a fix for issues #160
AFAICT it doesn't yet provide a fix for issue #7.

Is the Songs tab on the Artist overview working for you? I don't get any songs there.

Didn't check well enough. I'll fix this ASAP.

Should be fixed now.


integration_test_macos plugin does not build

Steps to Reproduce
$ cd ./packages/integration_test/example/
$ flutter build macos

Fails
[!] The name of the given podspec `IntegrationTestMacOS` doesn't match the expected one `integration_test_macos`

integration_test_macos.podspec name should be integration_test_macos instead of IntegrationTestMacOS.
https://github.com/flutter/plugins/blob/89794a635a5efad6705e3d7ccc29e8139318d947/packages/integration_test/integration_test_macos/macos/integration_test_macos.podspec#L5

@dnfield Do you know why integration_test_macos is in the iOS-only integration_test directory?
https://github.com/flutter/plugins/tree/89794a635a5efad6705e3d7ccc29e8139318d947/packages/integration_test.  I was expecting something like

integration_test

integration_test
integration_test_macos

or

integration_test

ios
macos

but it's

integration_test

ios
example
integration_test_macos

macos

It was started as a federated plugin.

@dnfield So should it look like https://github.com/flutter/plugins/tree/master/packages/path_provider with each platform as sibling directories?

I think it'll look like that eventually yes - @stuartmorgan was looking at this

This is what a quick semi-federation looks like; it should indeed have been done one of the ways you listed, but instead the federated macos plugins was added inside the existing plugin.
We should just make that plugin not be federated. Also, the macOS implementation doesn't actually do anything; we could likely just remove it for now.


Feature/about page

About us - looking good, great job! The success page:

The footer needs to be set at the down side of the page. Also can you link the success page to the submit buttons of the order now and careers? So when people click either "book table" or "apply now" they are redirected to the success page?

@annie-uchiha Thank you for pointing it out! Can you take a look now?


move source directory

The current bitbox is difficult to use from TypeScript.
Please modify the package so that it does not include the source file before compilation.

Thank you for the PR. I'm not a user of Typescript, so I'm not really sure how to evaluate this. I'll try to see if any of the other committers for this repo have the prerequisite knowledge to evaluate this PR.


Feature request: replay last executed command

It would be really nice to be able to easily replay the last command that was executed by Hexagon.
Not really sure how the flow should be, but it would be valuable :)

hexagon now provides replay (r) command to do this 🙌🏻