[BUG] Images are loaded incorrectly in production

Description
Images load correctly during development, but when the React app is served from the Resources folder, incorrect images are displayed due to identical file names with different extensions.
Steps to Reproduce

Import two images with identical file names but different file extensions from separate folders.

import PlaceholderAvatar from "src/assets/images/icons/default-avatar.png";
import PlayerAvatar from "src/assets/images/default-avatar.jpg";

Build the react app. Below shows the files are all stored under /static/media/ folder.

ResourcesHelper.LoadResource is unable to load the correct image.

Potential Cause
Currently, the files in Resources folder are loaded without file extension.
https://github.com/ReactUnity/core/blob/fef8c873b41af0922c3cffe6e98d37dc0efee34f/Runtime/Types/AssetReference.cs#L168-L188
https://github.com/ReactUnity/core/blob/fef8c873b41af0922c3cffe6e98d37dc0efee34f/Runtime/Helpers/ResourcesHelper.cs#L55-L63

My current workaround is to either change the file names or specify the assetModuleFilename in webpack config using webpack-merge.
{
   output: {
      assetModuleFilename: "static/media/[name].[hash][ext]",
    }
}

In the new version of @reactunity/scripts package, extension names will be added to the end of the file names to avoid name collisions, because Unity ignores extensions in Resources.


sqlite: appears to be confused by sqlc.arg or parentheses

Version
Other
What happened?
I have been using sqlc for over a year with sqlite, by using the mysql engine and holding it in the right way so that the generated code would be compatible with sqlite.
Now that sqlite is supported, I tried swapping the engines. I get syntax errors, which I reduced to the bits of code below.
This is with sqlc 1.16.0. I think the template is out of date: I could not pick this version, nor could I pick sqlite as the engine.[
Relevant log output
sqlc generate failed.

line 3:13 mismatched input '(' expecting {')', ','}
# package db
query.sql:1:1: mismatched input '(' expecting {')', ','}

Database schema
Not needed, really. The error happens when parsing the query.

SQL queries
-- name: GetPeople :many
SELECT ID FROM people
WHERE (LENGTH(sqlc.arg(name)) = 0);

Configuration
{
  "version": "1",
  "packages": [
    {
      "path": "db",
      "engine": "sqlite",
      "schema": "query.sql",
      "queries": "query.sql"
    }
  ]
}

Playground URL
https://play.sqlc.dev/p/a375863b4700d7104acad69d85e9c9b55f6611eb3a8214af946e490339b38e59
What operating system are you using?
Linux
What database engines are you using?
MySQL
What type of code are you generating?
Go

Possibly related to https://github.com/kyleconroy/sqlc/issues/1881, which shows errors when sqlc.narg is used with sqlite. Though the errors look different.

This may have been resolved with the merge of https://github.com/kyleconroy/sqlc/pull/2274. I know it's been awhile but if it's possible to test your engine switch with the latest sqlc dev build and report results that would be ideal. I will try to test myself once a new version of the playground ships.

Indeed it works! I've fully switched from using the mysql engine as a workaround to the sqlite engine :)


feat: what

libraryB
0.0.3-aplha
Associated Task: #234324
✨Features

91a9c61 Thanks ncqwer ! - errororororo
6467e8c Thanks ncqwer ! - what
6800842 Thanks ncqwer ! - what

/generate-commit

/generate-commit


Better Tensorboard documentation

Would be nice to have a notes page

This is largely done and replaced by our example library


Do not bother closing SSL engine inbound if the outbound has already been closed.

Do not bother closing SSL engine inbound if the outbound has already been closed.
Motivation:
Invoking the javax.net.ssl.SSLEngine.closeInbound() method will send a
fatal alert and invalidate the SSL session if a close_notify alert has
not been received.
From the javadoc:
If the application initiated the closing process by calling
closeOutbound(), under some circumstances it is not required that the
initiator wait for the peer's corresponding close message. (See section
7.2.1 of the TLS specification (RFC 2246) for more information on
waiting for closure alerts.) In such cases, this method need not be
called.
Always invoking the closeInbound() method without regard to whether or
not the closeOutbound() method has been invoked could lead to
invalidating perfectly valid SSL sessions.
Modifications:
Added an instance variable to track whether the
SSLEngine.closeOutbound() method has been invoked. When the instance
variable is true, the SSLEngine.closeInbound() method doesn't need to be
invoked.
Result:
SSL sessions will not be invalidated if the outbound side has been
closed but a close_notify alert hasn't been received.

@mrb-twx can you please address my two comments and sign the ICLA:
http://netty.io/s/icla
Please ping me once done

Sorry, completely misread that.

Squashed and cherry-picked into 4.0 (e146873740f4143b93f3d16e8e1e264b7f86e285) , 4.1 (58dc7f790231428edf2502ee4c4ca6ff29280832) and master (a4b51dd66b08dcdf4495a831d74ccb11e22877dd)

@mrb-twx thanks!


GRIB FILES FAIL TO DOWNLOAD

Expected Behavior

./Run.sh should run without error and generate output
Current Behavior

Run.sh fails on EXTRACTING NAM DATA step
### EXTRACTING NAM DATA INTO CALMET INPUT FILE FORMAT Traceback (most recent call last):
  File "./Create3DDAT.py", line 230, in <module>
    gidPRMSL = varNames.index("prmsl")+1  # Pressure reduced to mean sea level
ValueError: 'prmsl' is not in list

NAM_data/raw/date/*.grib contain only link to https://nomads.ncep.noaa.gov/ rather than data.
Possible Solution

NOAA have made https changes

Steps to Reproduce (for bugs)

./Run.sh
more  NAM_data/raw/20190206/<filename>

Context

Complete run failure
Your Environment

Operating System and version (desktop or mobile): FOE-LINUX (all users)

Any other relevant information

#4 Run.sh edit to fix this


Include Battery data

Thanks for the nice plug-in,
However, is it possible to include also the power flow to the battery or back from the battery. This would be a nice additional feature.
Thanks a lot

Sorry I don't have a Fronius-compatiable battery system (I have a Tesla Powerwall) so I don't see any battery data in the API to test against.
Do you have any sample data at http://[your-fronius-ip]/solar_api/v1/GetPowerFlowRealtimeData.fcgi?

Battery SOC sits in Inverters.1.SOC
{    "Body" : {       "Data" : {          "Inverters" : {             "1" : {                "Battery_Mode" : "normal",                "DT" : 1,                "E_Day" : null,                "E_Total" :<PHONE_NUMBER>4444439,                "E_Year" : null,                "P" : 7327.86865234375,                "SOC" : 100.0             }          },          "Site" : {             "BackupMode" : false,             "BatteryStandby" : true,             "E_Day" : null,             "E_Total" :<PHONE_NUMBER>4444439,             "E_Year" : null,             "Meter_Location" : "grid",             "Mode" : "bidirectional",             "P_Akku" : 2.0681488513946533,             "P_Grid" : -3886.98,             "P_Load" : -3440.88865234375,             "P_PV" : 7529.881591796875,             "rel_Autonomy" : 100.0,             "rel_SelfConsumption" : 46.956199893719628          },          "Smartloads" : {             "Ohmpilots" : {}          },          "Version" : "12"       }    },    "Head" : {       "RequestArguments" : {},       "Status" : {          "Code" : 0,          "Reason" : "",          "UserMessage" : ""       },       "Timestamp" : "2022-08-26T01:33:09+00:00"    } }

@systemx-xx Hey mate, I saw you made a fork at https://github.com/systemx-xx/homebridge-fronius-inverter-lights and added the SOC support.
If you want to open a PR I'm more than happy to merge it back in.

hey @longzheng I have not had a chance to play around with that since I forked it as displaying the inverter data using lights etc is pretty sub par to be honest. I really wish Apple would open up the ability for custom displays.

Fair enough. I noticed the homebridge-tesla-powerwall plugin uses a fan speed to display energy flow which has the advantage of showing actual values in the overview.

I've been using that plugin since getting a Powerwall and have been pretty happy with it. I wonder if this plugin should also do the same.

Ah thats clever, yeah thats quite a bit nicer.

I've just published a new version 1.5.1 that adds a battery accessory if you enable it in the config.
        {
            "name": "Fronius inverter",
            "platform": "FroniusInverterLightsPlatform",
            ...
            "battery": true
        },

Since I don't have a battery connected to my Fronius inverter I can't test it myself so let me know if there's any issues.

Thanks a lot. It works well. Best regards Thomas Von meinem iPhone gesendetAm 24.06.2023 um 10:20 schrieb Long Zheng @.***>:
I've just published a new version 1.5.1 that adds a battery accessory if you enable it in the config.
{
"name": "Fronius inverter",
"platform": "FroniusInverterLightsPlatform",
...
"battery": true
},
Since I don't have a battery connected to my Fronius inverter I can't test it myself so let me know if there's any issues.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


public url problem...

can you help?

Hey, Gradio had a hiccup today, but it's back on track. Check out its status here https://status.gradio.app/


Strip leading spaces from compiler flags when building OpenSSL on Linux

I've checked this Git style guide. [Yes]
I've checked this CMake style guide. [Yes]
My change will work with CMake 3.2 (minimum requirement for Hunter). [Yes]
I will try to keep this pull request as small as possible and will try not to mix unrelated features. [Yes]

Done. Sorry for the delay.

Testing:

https://ci.appveyor.com/project/ingenue/hunter/builds/25086231
https://travis-ci.org/ingenue/hunter/builds/542192457?utm_source=github_status&utm_medium=notification

Released: https://github.com/ruslo/hunter/releases/tag/v0.23.188


feat: Add Loc support and Form component

SUMMARY:
Adds Form component, LocaleReceiver, LocaleProvider. Size, Validation, Disabled, and Shape contexts. Normalizes shapes, fixes a lot of CSS.
The video:
https://drive.google.com/file/d/1yGlmXsJnbHhgmfUN74IPYK5RZn9V72X6/view?usp=sharing
JIRA TASK (Eightfold Employees Only):
ENG-2792
ENG-2793
ENG-24252
CHANGE TYPE:

[ ] Bugfix Pull Request
[X] Feature Pull Request

TEST COVERAGE:

[ ] Tests for this change already exist
[X] I have added unittests for this change

TEST PLAN:
Pull the pr branch and do yarn and yarn storybook, verify all form stories.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@d9a406d). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #332   +/-   ##
=======================================
  Coverage        ?   89.64%
=======================================
  Files           ?      482
  Lines           ?    12819
  Branches        ?     4241
=======================================
  Hits            ?    11491
  Misses          ?     1303
  Partials        ?       25

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

@dkilgore-eightfold Seems this PR is changing the radio sizes. This is breaking at multiple places across TA so reverting changes for radio label sizes in this PR


optimize_all(fluxes=True) and optimize_single(fluxes=True) do not return fluxes

Even with fluxes=True passed into community.optimize_all(), only maximal growth rates are returned. In fact, community.optimize_single() does not accept the fluxes argument.
Using version '0.22.6'

Yeah, optimize_all shouldn't take the argument either, that is an error in the signature and docs. In most cases you will want to use community.optimize or community.cooperative_tradeoff. See https://micom-dev.github.io/micom/growth_fluxes.html. What were you trying to achieve?

Thanks for the response, that makes sense!
I'm comparing carbon use efficiency (CUE) of single organisms and communities of those organisms. The calculation for CUE requires the rate of some exchange fluxes in addition the biomass flux.
My plan was to construct communities and use optimize_single() to get CUE for each community member independently and then compare with community.optimize(). I'm still able to get the monoculture CUE by loading the models into cobrapy separately, but that's obviously less efficient since the models need to be loaded multiple times.

The main thing I'm trying to do is create subsets of a community without doing multiple model loads. A hacky solution I'm going to move forward with is to just update the abundance profile to reflect the subset I'm interested in.

I'm having success with first setting community._rtol =0 and then community.set_abundance()

Reopening because I still need to fix the signature. Updating the abundance is good but may renormalize unless you use com.set_abundances(..., normalize=False). What also worak is to just knockout all reactions for some taxa, which will work with the context manager (this will require a fix from 0.22.7 which I'll release today).
from micom import Community
from micom.data import test_taxonomy

tax = test_taxonomy()
com = Community(tax)
knocked_taxa = com.taxa[0:2]
with com:
    [r.knock_out() for r in com.reactions.query(lambda ri: ri.community_id in knocked_taxa)]
    sol_knocked = com.cooperative_tradeoff()

# automatically reset on leaving the with block
sol_normal = com.cooperative_tradeoff()

print(sol_normal.members)
print(sol_knocked.members)

This is also a little bit more numerically stable and is the strategy used in knockout_taxa.

Okay, fixed now. Feel free to open a discussion at https://github.com/micom-dev/micom/discussions for a strategy to implement partial models if you have additional questions.


Update where-to-find-tech-jobs.md

add a remote job board
Description

Please include a summary of the change and which issue is fixed. Also include relevant motivation & further context of what the PR is all about. List any dependencies that are required for this change.
Fixes #(Insert Issue Here Without Brackets)
Changes Include

[ ] Bug fix 🐛 (non-breaking change which fixes an issue)
[ ] New feature ✨ (non-breaking change which adds functionality)
[ ] Breaking change 😢 (fix or feature that would cause existing functionality to not work as expected)
[ ] Refactor ⚒ (code change that neither fixes a bug nor adds a feature)
[ ] Tooling ⚙ (Changes to the build process or auxiliary tools and libraries such as documentation generation)
[ ] Performance ⚡ (code change that improves performance)
[ ] This change requires a documentation update 📖

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Checklist

[ ] My code follows the style guidelines of this project
[ ] 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
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[ ] I have checked my code and corrected any misspellings

Screenshots (if any)
(prefer animated gif)

Hi @Bhupesh-V, may I know why it wasn't merged?
Thanks.

Self-promotion

so adding a resource is forbidden?


Updating foundation account gives an index instead of an account

Purpose
Block explorer fails to decode blockSummary, when an update of foundation account is queued, because the dashboard assumes the foundation account to be an account address, but the api exposes the account index instead.
Changes
Show the foundation account index
Checklist

[x] My code follows the style of this project.
[x] The code compiles without warnings.
[x] I have performed a self-review of the changes.
[x] I have documented my code, in particular the intent of the
hard-to-understand areas.

Please link this to the issue @amaurremi created.

Fixes #21


deploy: Fix mounting /var/lib/libvirt from host

closes #222

This change is

There are no tests for this on CI.
Manual tests on dind require building image locally, sending it to kube-node-1, and then starting it by start-dind command of ./build/cmd.sh (preceded by build, copy and copy-dind).
This should allow to use host /var/lib/libvirt in virtlet container, so also keeping it's images store during virtlet daemonset or even node restarts.

Review status: 0 of 4 files reviewed at latest revision, 1 unresolved discussion.

image_skel/start.sh, line 42 at r1 (raw file):

# Removal of the socket below helps but not always.

if [[ -f /var/lib/libvirt/qemu ]]; then

[[ -f /var/lib/libvirt/qemu ]] is basically never true (should use -e and perhaps only do this if /var/lib/qemu is not a mount point) There's chance this hack may not be necessary with /var/lib/libvirt/qemu being mounted from host in daemonsets, but this may break integration tests. (For me the hack doesn't affect Travis but w/o it integration tests were breaking on some test machines)

Comments from Reviewable

Review status: 0 of 4 files reviewed at latest revision, 1 unresolved discussion.

image_skel/start.sh, line 42 at r1 (raw file):
Previously, ivan4th (Ivan Shvedunov) wrote…
[[ -f /var/lib/libvirt/qemu ]] is basically never true (should use -e and perhaps only do this if /var/lib/qemu is not a mount point) There's chance this hack may not be necessary with /var/lib/libvirt/qemu being mounted from host in daemonsets, but this may break integration tests. (For me the hack doesn't affect Travis but w/o it integration tests were breaking on some test machines)

(can test for the mountpoint using mountpoint command)

Comments from Reviewable

Review status: 0 of 4 files reviewed at latest revision, 1 unresolved discussion.

image_skel/start.sh, line 42 at r1 (raw file):
Previously, ivan4th (Ivan Shvedunov) wrote…
(can test for the mountpoint using mountpoint command)

It's imo necessary because we have dual situation - there can be /var/lib/libvirt with potentially started vm on qemu on host - so there is a possibility that this socket will be there, so hack withmv will be useful, and second - there was no /var/lib/libvirt on host - so i'm making it empty in prepare-node.sh, so this mv will fail.

Comments from Reviewable

Review status: 0 of 4 files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

Reviewed 3 of 4 files at r1, 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


fix(python): bin script python support, test argument passing

Corrects multiple issues with running bin scripts from python.
These were discovered in the context of this projen issue: https://github.com/projen/projen/issues/2103

Corrects broken argument marshaling to binary scripts introduced in #3694
Exposes exit code and stderr from script execution instead of failing silently

Additionally, adds more robust test coverage of passing arguments to bin scripts.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

The tests are currently failing due to the jsii-calc/bin/run not being executable in the extracted tarball.
status: 126
stderr: "/bin/sh: 1: /tmp/jsii-kernel-4QI5As/node_modules/jsii-calc/bin/run: Permission denied"

I'm still trying to troubleshoot this. Running npm pack manually produces a tarball with correct -rwxr-xr-x permissions.

Hi @RomainMuller, is it acceptable to leave Windows bin invoke functionality partially implemented? It appears that is the current state of jsii-pacmak, as it doesn't generate executables scripts for Windows, which would need to be added.


Create an API document for people to review & comment

This is part of issue #10. So I'm closing this one for now.


Logging Guide, and README and OVERVIEW examples

Add Logging examples to the list of services covered by gcloud on README and OVERVIEW.

Added Guide (logging.rb docs) to issue title.

@JustinBeckwith We are trying to provide a realistic Ruby example for Cloud Logging entries.write.  Can you suggest which monitored resource would be appropriate? Below is the current list from monitoredResourceDescriptors.list, with the required label keys in the parentheses.
gce_instance (instance_id, zone)
gae_app (module_id, version_id)
cloudsql_database (database_id, region)
gce_disk (disk_id, zone)
vpn_gateway (gateway_id, region)
api (service, version, method, location)
global ()
dataflow_step (job_id, step_id)
app_script_function (invocation_type, function_name)
dataproc_cluster (cluster_id, zone)
bigquery_resource ()
container (cluster_name, namespace_id, instance_id, pod_id, container_name, zone)
http_load_balancer (forwarding_rule_name, url_map_name, target_proxy_name, backend_service_name, zone)
gce_target_pool (target_pool_id, zone)
gce_firewall_rule (firewall_rule_id)
gce_forwarding_rule (forwarding_rule_id, region)
gce_network (network_id)
gce_route (route_id)
gce_reserved_address (reserved_address_id)
gce_autoscaler (autoscaler_id, zone)
gce_backend_service (backend_service_id)
gce_health_check (health_check_id)
gce_url_map (url_map_id)
gce_project (project_id)
gce_snapshot (snapshot_id)
gce_target_http_proxy (target_http_proxy_id)
gce_target_https_proxy (target_https_proxy_id)
gce_target_http_instance (target_http_instance_id, zone)
logging_log (name)
logging_sink (name)
metric (name)
testservice_matrix (matrix_id)

Probably a better question for @mr-salty

@mr-salty For purposes of a documentation example, can you suggest a realistic monitored resource type that a Ruby application developer might reference when writing log entries using the Cloud Logging API?

We decided the safest choice of monitored resource type for a user-generated log entries example was "gae_app". But we can always change it, just let us know.

gce_instance is another possibility but I think gae_app is fine as well.


Report profiling attribute values conflicts

A checkbox called "Report profiling attribute values which conflict with profiling preferences" maybe followed by a button which shows the profiling preferences.
When checked, while validating, if for the profiling attributes defined in the preferences we find in the XML extra values which are not defined in the preferences, we report warnings in the same results tab with the rest of the reported problems.

fixed in : f5da378536e475c2bd2fc3804f57bf3b8e8a7b01


Translations stop functioning after a period of time (Forge-only now)

After a certain period of time (normally around 12 to 24 hours), the translations will suddenly stop working. This is caused by a Java standard library problem, and needs to be worked around.
For now, if translations suddenly pause, just run /unitytranslate debugreload. You may need to run this multiple times.
Why does it only happen on Forge now?
Because of the problem of two of the same library (Apache HttpClient) loading from different modules (UnityTranslate and the Minecraft client), Forge will just straight up crash the game.
Unfortunately, because of this, we cannot bundle Apache's libraries with UnityTranslate on Forge unless we shade and relocate them, which then introduces its own myriad of problems.
So instead, as a workaround, if UnityTranslate can find (and use!) Apache's libraries, it will try to use them, but if not, it will just fallback to the old Java HTTP code, which is itself incredibly broken in its own ways.
Technical explanation
Apparently, Java just leaves some HTTP connections open even if they have failed, and as a result they just don't end up closing/releasing, which just straight up causes a deadlock if there are too many HTTP connections.
The reason why /unitytranslate debugreload fixes this is because I recently moved translation API calls to use a ForkJoinPool, and the reload would reset the ForkJoinPool and forcibly close all HTTP connections in that thread pool (which also explains the log spam if you have -Dunitytranslate.printHttpErrors=true enabled).
Further explanation towards this problem can be found in this Medium post.
Below is some information I stored previously just to explain the problem, if you want to read more about what I considered, here you go.
Discarded Solutions
Using Ktor for HTTP requests
Due to an actual problem with Kotlin on Forge 1.20.1 and 1.20.4 (thedarkcolour/KotlinForForge#86), Ktor is completely out of the question from being used as a solution. I was forced to throw Ktor out the window (e842f607925d42d91042c24a808f8c0c1b8b277e) just to be able to support Forge, and Connector couldn't seem to handle Ktor properly either.
Using OkHttp for HTTP requests
Just like the prior problem with Ktor, OkHttp is not a viable solution due to its Kotlin usage.
Rewriting the HTTP handlers to use the logic provided in the Medium post
This ends up becoming way too much effort for what it's worth, especially because this is an incredibly slow problem to fix (requires multiple concurrent translations, appears to only occur after a period of 12-24 hours or so, etc.), so this is not a viable solution.
Possible Solutions
Rewrite HTTP handlers to use Apache's HTTP Clients API
This is what 1ea2dd8a7dd98135799863c7c7ff898b4ea30011 does, but we need further testing to see if this actually helps.

Reading your post there, most of those issues can be solved with shading+relocating, which doesn't have it's own myriad of problems, and works fine in modded environments, even in cross-plat or mono-jar environments.
I don't know if you're speaking from personal experience or it was something you've heard somewhere, but the gradle shadow plugin works completely fine with most, if not all modding buildtools (even when taking reobfuscation into account).
Shading+relocating also has the added benefit of being able to be minified, wheras JarInJar'd dependencies cannot, further reducing your Jar's size.
If you need examples, I have plenty in this repo here: https://github.com/p0t4t0sandwich/TaterLib

Speaking mainly from personal experience, seeing as I've done it many times before. it's caused a lot of problems for me whenever JiJ was not an option, not particularly the fault of Shadow itself, rather the libraries I usually needed to bundle.
In some few cases, a library would rely VERY HEAVILY on having its package name unchanged (one example is LWJGL, which is why server CUDA support isn't available in UT), which then requires either untangling what in the world is going on (which isn't really feasible), or just throwing it right out the window.
btw, to clarify, the reason behind LWJGL's requirement last time the library needed to be shaded (unrelated to UT) was because of an ImGui library depending on LWJGL, more specifically a very specific version of it that wouldn't work if you excluded it. The Axiom mod previously also had the problem of needing to shade LWJGL, I'm unsure if they still do that now though.
Additionally, with shading, sometimes it would also include some other libraries that I would have to determine if I actually need them and if so then relocate them manually, and if not just throw them out the window, which then repeats the cycle of "will it crash if I relocate it?" and stuff like that. It's not fun.
And unfortunately, the problem with Ktor and OkHttp being unable to be used in Forge environments still wouldn't be solved with shading either way, there's still some other issue relating to classloading on Forge because of that god forsaken module system.
So overall Apache's my best bet, and while I would definitely try to shade-relocate it when I have the time, I would really just. prefer if I didn't have to.

Fair enough yeah, came across this and thought I'd throw in my two sense.
Never come across that package-rename issue specifically, but that sounds like an absolute compatibility nightmare.
Just checked, and I've bundled OkHttp into a project here, and it's worked fine in Forge with no module collision issues. I'd assume because I've bundled and relocated the Kotlin dep as well, which given your env would be it's own hassle to try and relocate without buggering up your own Kotlin deps (you'd need to create a nightmare shadow config and exclude any of your project's paths since you use Kotlin). Further increasing that trial-and-error-relocation phase that you're wanting to avoid.
So yeah, sounds like you're stuck between horrible gradle hell and a hard place lol.
shadowJar#minimize() does help a bit to limit the amount of do-I-need-this-transient-deps, but still leaves a decent amount left over to sift through.
Though if you ever do end up trial-and-erroring some gradle, I'd be down to bounce some ideas around.
(As an aside, I noticed your work on Kilt. I haven't had the chance to properly test it, but I've added some preliminary support to my cursed cross-plat lib, nifty work all around!)

I have rewritten the HTTP handler to use OkHttp now (112706c7e5c9852ed7199080724e94718d374f43), and I'm hoping to god that this solves the problem once and for all.


arch/x86: add SMP support (64-bit mode only)

Title says it all.  Add support for up to 4 CPUs (which is the limit currently imposed by the kernel).
Details in the commits.

There are quite a few sanitycheck failures, though I am quite sure that most (if not all) are related to lingering SMP problems in the test cases and/or a recent regression in the scheduler's support for non-IPI-enabled systems, for which an issue will be filed shortly.
Scheduler IPI support is also coming soon.  It's a very straightforward addition, I'm just being lazy.

Force-push for rebase; some commits have been merged in other PRs.

sanitycheck is lying a bit here, I see sporadic failures in my both of my build environments, but these failures appear to appear qemu_x86_64 as much as qemu_x86_long, so I do not believe the problem is related to the arch-specific code.

@ceolin @andrewboie @andyross @dcpleung @nashif No takers?

-:952: WARNING:NEW_TYPEDEFS: do not add new typedefs
#952: FILE: arch/x86/include/intel64/kernel_arch_data.h:94:
+typedef struct x86_cpuboot x86_cpuboot_t;

Gah, our offset generation requires typedefs, on a rainy day we ought to revise this. Ignore this.
The rest should be addressed, trivial formatting stuff.

The rest should be addressed, trivial formatting stuff.

Done.

Gah, our offset generation requires typedefs, on a rainy day we ought to revise this.

I think we go about this the hard way.  A more conventional method this is to compile a program that includes all the requisite headers and then spits out a header file with manifest constants #defined.  I'm really not sure why we go through all this trouble to inject symbols into the object files, perhaps there's a good reason for it that I'm not aware of ...

A more conventional method this is to compile a program that includes all the requisite headers and then spits out a header file with manifest constants #defined. I'm really not sure why we go through all this trouble to inject symbols into the object files, perhaps there's a good reason for it that I'm not aware of ...

All it's used for AFAIK, is to compute sizeof() various structs, and offsets of struct members, and emit a header file with these values, for use within assembly code. It needs typedefs due to the way it generates names but that could be adjusted without too much hassle I think.

All it's used for AFAIK, is to compute sizeof() various structs, and offsets of struct members, and emit a header file with these values, for use within assembly code.

That's what you'd expect, but instead we use a mess of macros (imagine that) to generate absolute symbols in the object files.  If we generated headers, it'd actually be better, because then the assembler could take advantage of the fact that they're constants.  Consider sequences like
movl $_kernel, %esi
movl _kernel_offset_to_current(%esi), %esi

This should simply be
movl _kernel+_kernel_offset_to_current, %esi

because "address of an object plus an offset" can be represented in an object file, but "the sum of two symbols" can't be.
Anyway, I digress.  As you said, rainy-day stuff.

That's what you'd expect, but instead we use a mess of macros (imagine that) to generate absolute symbols in the object files. If we generated headers, it'd actually be better, because then the assembler could take advantage of the fact that they're constants.

No kidding! (I never dived into the details on how it worked). I agree, seems needlessly complicated given the problem it's supposed to solve. Worth an enhancement GH ticket I think to clean it up someday.
Aside from the two build errors in CI I think this PR is ready to go in.

No kidding! (I never dived into the details on how it worked). I agree, seems needlessly complicated given the problem it's supposed to solve. Worth an enhancement GH ticket I think to clean it up someday.

agree this needs to be enhanced, the offset generation step is legacy and is overly complicates the build process for now good reason. Lets looks into solving this finally.


Are any of the MSI z490 boards compatible?

I see you closed the issue. Did you test this?

I tried it on my mpg z490 gaming plus and it didnt work, i also one of my friends tried it on his mag z490 Ace, so i assumed it wont work for any of the z490 ones

There may be Z490 motherboards for which it will work, but I cannot give you a list. The list on the README are the ones that I'm sure are compatible (though it might still not work correctly due to a bug in MSIRGB, since not all motherboards have been tested).


Make plotting dependencies optional

Adds an optional group for installing plotting and analysis specific libraries to lighten the install of the core framework.

LGTM; do we have some docs changes so that the users know how to enable/disable the plotting option?

Yeah, will add some docs. Just wanted to make sure tests run

LGTM. I've updated the changelog as well


wake-up-api

[Enter feedback here]

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

ID: 2fa51748-908b-c781-39b2-09c961b1637a
Version Independent ID: 51b88c0a-29d3-2ca8-0b54-d96be4779dd2
Content: Legacy actionable message card reference - Outlook Developer
Content Source: docs/actionable-messages/message-card-reference.md
Product: outlook
Technology: o365-connectors
GitHub Login: @jasonjoh
Microsoft Alias: jasonjoh

Can you provide details on your issue?

API instructions -and availability - for marketing policies


Transformer: some fix + language enable

These changes have been merged by other PRs -- closing issue. @pasqLisena -- please send another PR if there are any changes for SPARQL Transformer which are missing.


Unable to resolve your shell environment - Fish Shell

Does this issue occur when all extensions are disabled?: Yes

Version: 1.63.2 (Universal)
Commit: 899d46d82c4c95423fb7e10e68eba52050e30ba3
Date: 2021-12-15T09:37:28.172Z
Electron: 13.5.2
Chromium: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Darwin x64 17.7.0

Steps to Reproduce:

Open VSCode
Error occurs on startup

Everytime VSCode opens, I get the following error: "Unable to resolve your shell environment: A system error occurred (spawn /bin/fish ENOENT). I installed Fish shell via Homebrew and it installed it to /usr/local/bin/fish. /bin includes other shells like bash, zsh, etc but Fish is not listed here. Here are my settings:
	"terminal.integrated.profiles.osx": {
		"Fish": {
			"path": "/usr/local/bin/fish"
		}
	},
	"terminal.integrated.defaultProfile.osx": "Fish",

For some reason, VSCode seems to still be looking at /bin/fish instead of the location defined in settings. This continues to happen when I change the default terminal to something else as well.

if you change the name of the profile to "fish2" does it work? and what do you see when you run select default profile?

Thanks for the response. No, changing the profile name to fish2 does not help and changing the default profile doesn't help either. This only happens on start up though. If I change the default profile to bash, the terminal pops up fine, but if I close and reopen VSCode, I get the same error regardless of which profile is default.

ok I just tried what you have for the path and it works - I think you need the -l args
"fish": {
			"path": "/usr/local/bin/fish",
			"args": [
				"-l"
			],
		},

pls lmk if that does not fix your issue

Still seems to crash on startup

does that path exist on your computer?

Yeah, it also looks like that's a shortcut pointing to /usr/local/Cellar/fish/2.7.1/bin/fish. I also tried using this explicit path and I get the same error. /bin/fish does not exist though because it was installed via Homebrew

We use the $SHELL environment variable which it appears you have set to /bin/fish, or homebrew did 🤷
Look into changing it (via the chsh command?).

I was able to update this on my Mac by updating Users & Groups -> (right click user) Advanced Options... -> Login Shell and now my terminal reflects changes to $SHELL; however, VSCode still isn't noticing those changes. Is there somewhere in VSCode that's setting or caching this path that I could update?

We use the $SHELL environment variable which it appears you have set to /bin/fish, or homebrew did 🤷
Look into changing it (via the chsh command?).

Thanks for the insight, this was super helpful! I was able to run sudo chsh -s /usr/local/bin/fish  in my VSCode terminal and after I logged out and back in again, the error was gone and echo $SHELL reflected the proper path. Thanks for your help!


Unresolved symbols with PDB

For some functions, the symbols are just resolved as fun_<offset>.
Example: kernel32.dll!GetConfigDialogName
I can see the resolved name in IDA which also downloads the PDB:

However, when using wholesym, it doesn't work:
 let symbol_manager = SymbolManager::with_config(SymbolManagerConfig::default());
    let symbol_map = symbol_manager
        .load_symbol_map_for_binary_at_path(module_path, None)
        .await
        .ok()?;

    if let Some(address_info) = symbol_map.lookup_relative_address(addr) {
        log::debug!(
            "Symbol: {:#x} {}",
            address_info.symbol.address,
            address_info.symbol.name
        );

        Some(address_info.symbol.name.clone())
    } else {
        log::warn!("No symbol found.");
        None
    }

Ran with:
let name = addr_to_name_async(Path::new("C:/Windows/System32/kernel32.dll"), 0x3CDF0).await;

This returns fun_3cdf0

Great!


fix(HAC-5855): only suggest branches for component

Fixes
HAC-5855
Description
Using a tag or commit can have unintended consequences. Lets only recommend that users use a branch when setting up their components.
Type of change

[ ] Feature
[x] Bugfix
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

Screen shots / Gifs for design review
Before:

After:

How to test or reproduce?

Browser conformance:

[x] Chrome
[ ] Firefox
[ ] Safari
[ ] Edge

/retest

/retest


Public DB Dumps: Re-tool the dump system to index files on-server directly and serve them, reenable automatic indexer to find and display valid links.

Using ancillary systems in the infrastructure, we now have a platform to make daily DB dumps and updates in a way that is secure, actually sanitizes the data, and transfers it back to Metasmoke for use.
Currently, the full DB backup infrastructure seems to be configured to utilize the S3 of @Undo1.  With the migration to my server farm, we can now serve the dump content directly from server with the proper reference on-disk to serve it.
To that end, the public db dumps system needs to be altered to adapt for this, picking up the public DB dump file (a .sql.gz - GZipped SQL file) and serve the file via MS directly, bypassing the need for S3 to handle the dumps.
Initial assignees for this are @thesecretmaster @ArtOfCode- and @Undo1 as they are most familiar with the Ruby codebase to alter the backup handlers where needed to make this work.
###Cron job timing
The ancillary system runs for the data backups and dumping are currently timed at between 8min and 11min of runtime, and handles transfer of the file back to Metasmoke host.  That cron job currently runs at 00:00 UTC but can be altered to change what time it runs at.  Therefore, by 00:15 - 00:20 UTC, Metasmoke will have a file it can serve and index.
###On Metasmoke Host: Docroot and Naming Convention
Current on-system docroot where the dumps are sent to are: /var/railsapps/metasmoke/shared/dumps
Current naming convention is this for the specific dump files:  dump_metasmoke_clean-$TIMESTAMP.sql.gz (where $TIMESTAMP is the unix epoch time of the DB dump run)
Backup Persistence
All the dump files are also backed up on the ancillary system, however that is not directly reachable from Metasmoke (for security reasons related to the unsanitized data).
For data storage/size purposes we probably should continue to ONLY keep one public MS dump daily available on MS.

Also, for the record, there is an SQL dump I have up there now sanitized.  The script on the ancillary box cleans up daily so around 00:10 there'll be a new SQL dump there.

This is mostly done via the ancillary system - it populates the link data that is accessed by Metasmoke and then provides the links for it to parse via a YAML file.
Known minor issues/caveats:

The backup process currently does not run in an async way such that the Redis and SQL backups are at the same timestamp.
Metasmoke currently sees the timestamps but not the full filenames when it creates the paths - the actual URL(s) point to proper files though on dumps.charcoal-se.org.
There is a small chance the backups explode and aren't updated if there's some type of communication failure between the ancillary backup handler and the main Metasmoke system.

None of those issues are blocking.


Project Description text style is different in All Projects and CLA Project page

Background

Launch Project Management Console in dev environment

Login as Project Manager who has access to projects

Observed that Project Description is displayed in sentence case

Click on the project

Observed that project description is displayed in proper case

Expected Behavior
Text style for the description should be consistent in both the pages

Verified in chrome, firefox and edge browsers
Verified that the Project Description text style is consistent across All Projects and CLA Project pages
Example1

Example2


DEV: Getting crash report when open Connect with plaid option in iOS simulator

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: n/a
Reproducible in staging?: DEV
Reproducible in production?: DEV
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @shubham1206agra
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1714116623509679
Action Performed:
Go to any workspace in which you want to add a bank account.
Click on workflows > Connect Bank Account.
Click on Connect with Plaid .
Expected Result:
No crash report pop up
Actual Result:
Observe a crash report popping up on iOS Simulator
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[x] iOS: Native simulator
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
crash.txt

View all open jobs on GitHub

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

Asked clarifying Q's

Closing since this I cannot reproduce this.


Bump Redis depencency to v0.7.1 (fixes Windows builds)

The redis v0.7.0 crate build is broken for systems without Unix socket support.

Thanks, released in 0.5.1!


WASD to move, the objective is to avoid invisible barriers that will send you back to the start of each level before the timer runs out. I’ve seen a couple of repetitive maze games on sprig and they all seem to lack any real challenge. So in this game I would make it tough by sending you back to the start for any mistake, and even if you do manage to be careful if you aren’t quick, you’ll also lose the game. Have fun!!!
How do you play your game?
WASD TO MOVE
AVOID THE INVISIBLE BARRIERS
REACH THE 🏁 BEFORE THE TIMER ENDS
"i" TO RESTART THE GAME
"k" TO MUTE THE MUSIC

Duplicate of #2612


pubsublite BacklogLocation

Can't compile tests when updating dependency com.google.cloud:pubsublite-beam-io to v0.17.0 due to BacklogLocation file location update. #5699
#
# pubsublite/streaming-analytics
#

[ERROR] COMPILATION ERROR :
[ERROR] /tmpfs/src/github/java-docs-samples/pubsublite/streaming-analytics/src/test/java/PubsubliteToGcsIT.java:[29,47] cannot find symbol
  symbol:   class BacklogLocation
  location: interface com.google.cloud.pubsublite.AdminClient
[ERROR] /tmpfs/src/github/java-docs-samples/pubsublite/streaming-analytics/src/test/java/PubsubliteToGcsIT.java:[117,56] cannot find symbol
  symbol:   variable BacklogLocation
  location: class PubsubliteToGcsIT
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.1:testCompile (default-testCompile) on project pubsublite-streaming: Compilation failure: Compilation failure:
[ERROR] /tmpfs/src/github/java-docs-samples/pubsublite/streaming-analytics/src/test/java/PubsubliteToGcsIT.java:[29,47] cannot find symbol
[ERROR]   symbol:   class BacklogLocation
[ERROR]   location: interface com.google.cloud.pubsublite.AdminClient
[ERROR] /tmpfs/src/github/java-docs-samples/pubsublite/streaming-analytics/src/test/java/PubsubliteToGcsIT.java:[117,56] cannot find symbol
[ERROR]   symbol:   variable BacklogLocation
[ERROR]   location: class PubsubliteToGcsIT
[ERROR] -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException

 Testing failed: Maven returned a non-zero exit code.

err: exit status 1

Fixed with #5699


[Address Book] Add a mockup to create a name for the Address after choosing point in the map.

User (Driver/Passenger) shall add a name for his address during adding a new item for his Address Book.

@NazariyChetvertukha Please clarify the task: Why User needs to create a name for the address choosing IN THE MAP? (why they need a map for a name creation?)

@OlenaBobalo  Actually, we don't need a map to add the name to the point on the map, but, I think, it will be looking "more naturally" for the User when he adds a name right after choosing the point (maybe below the address box on the mockup). It is not a requirement.


Variable global not defined in semantic.js

Commit 4292ab5aa1411ae016f879c469322cc72629eb08
causes following error:
08:07:32.421 ReferenceError: global is not defined
semantic.js:497
1semantic.js:497:1
I see it's defined in the single components, but not in the all in one file.

I've modified the logic here because it was incorrect
https://github.com/Semantic-Org/Semantic-UI/blob/next/src/definitions/modules/dropdown.js#L16


[de] Added many missing translations

Danke

Can you translate ?

json : Name
validation : uuid : not present


Possible fix for Chunk Pregenerator problem

Somebody appeared to fix the problem with CPG generating only plains biomes a few years ago in this comment here. I checked the class that was modified and it appears to be the same in REID and JEID. Any chance this could be looked into?

I also saw this comment, I was planning on looking into it after my PR got merged, just to isolate any potential issues that may arise with implementing this.


Bundle and provide a SCSS file without compiling it into CSS

Description
I'm building a Vue/SCSS library with many SCSS variables and mixins, so I need to be able to bundle and provide a SCSS file without compiling it to CSS.
Is there any way to do it?
Suggested solution
Add option to include a bundled SCSS file.
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

If you're copying scss files without modifications. Could try https://github.com/vladshcherbin/rollup-plugin-copy#readme.

Closing as @fi3ework proposal should work here.


Fix cursor hotspot update

Fixes #581
Depends on #710
Tried to do not update the output when a new cursor image is set and the cursor isn't visible, but in the end IMHO it's not worth it: it complicates the DRM code and the benefits are too low.

This PR is ready for review.


add statement descriptor to plan attributes

Includes specs for a max length of 22 characters, as specified in Stripe's API docs at https://stripe.com/docs/api?lang=ruby#create_plan

Hi @jbender , this took way longer than it should've but I'll be merging this once the specs pass. Thanks for https://github.com/Everapps/stripe-rails/pull/28 !

Adding the Stripe API Plans required params for reference.

https://stripe.com/docs/api#create_plan


Put built files into repo root

So they could be easily included as bower_components/underwear.css/underwear.css (as they currently are in app-shell.html)

Fixed with #23


chore: Add mermaid documentation

Overview

Replace entity relationship diagram image with mermaid documentation
Clean up GitHub action tests workflow to remove unused environment variables
BUGFIX: Update logic in the-usher.js that checks SKIP_KEYS_CHECK env variable. Values coming from process.env are always either strings or undefined. Previous logic only checked if env variable existed (could be set to any value).

Your PR Checklist 🚨
❤ Please review the guidelines for contributing to this repository. Our goal is to merge PRs fast 💨 .

[x] Check your code additions will fail neither code linting checks nor unit tests.
[x] Additional units tests have been added to prove code updates work and fixes are effective
[x] Additional documentation has been added (if appropriate)
[x] Update Assignee field to yourself

Issue Reference

Closes #
Open Questions or Items to Callout

@k1ch  please check out the bugfix for the SKIP_KEYS_CHECK env variable.


Even more "Built with" badges

Built with...
Kotlin, Python, JavaScript, Svelte, Vue, C++, Godot Engine, Unity Engine, Unreal Engine (4 and 5), React, Angular, Swift (the programming language), Scala, etc.
Thanks in advance.

May take a while to do this as it's quite a few badges 👍🏽


Show contributions to other repos separately.

My main issue with GitHub repos is that forks I made to help other people's repositories are mixed up with my own. It would be great if:
We had 2 categories:
Own repos
Repos user contributed to
In the latter, an indicator of how many Pull Requests and Commits were added.

thanks, makes sense! related to #34


Complete assignment requirements

Copying a list of the requirements that our website must have to better keep track of what's been done and what hasn't.

[x] A user should be only able to rate a particular song once. A user should be notified that they cannot rate a song twice if they try submitting the song name again. You can assume that there are no two songs and no two artists with the same name. In other words, you do not need to check for different users entering duplicate songs or artists.
[x] People should be required to register and login with a username and password. If a user enters a username that is already taken, the user should be alerted accordingly and asked to pick a different username.
[x]  Once people are logged in they should remain authenticated via a PHPSESSID token.
[x] The registration form should ask people to enter their password twice to ensure that they are not mistaken about what they entered. If the passwords do not match, the user should be prompted again.
[x] If a user is already logged in and requests the registration page, they should be redirected to the index page with the overview of all rated songs.
[x] Show at the top of each page in your app which user is logged in.
[x] Songs (title, artist, and rating) should be displayed for all users in the database. However, users can only alter their own entries.
[x] The fields in the update view should be pre-populated with the current values.
[x] Upon updating a rating (or other form field value) the user should be redirected to the overview page of all songs. The same should happen if the user cancels the update. The same behavior should be present if a user deletes a song or cancels the deletion.
[x] Validate via the frontend that users cannot leave any form fields blank
[x] Validate that password is at least 10 characters long
[x] Validate that rating is a one-digit integer (1-5)
[x] All SQL queries must be parameterized to avoid SQL injection attacks.
[x] Passwords should not be stored in cleartext but rather hashed and salted for security reasons.
[x] Deploy website on InfinityFree
[x] Set up a certificate for website

closing as completed!


Jelly effect

In some devices when the camera open can be seen a jelly effect.
Has anyone noticed this problem?

I haven't seen this. Would you mind giving more specific details? Which phones, MC version, etc?

Phone Moto G4 Plus, Android Marshmallow


Cocoapods Trunk contains invalid info on this project

If you search for this project via CocoaPods:
pod search TAoverlay
It returns a pod entry that is versioned to 1.2.1, the latest available version on this repo is far less than that. Just a heads up.
Nice project!

Thanks :+1:   I uploaded multiple versions while trying to get cocoapods to work. Not sure how to get that removed. The official version should be 1.0.1. I am looking into this. Any help welcome.

So I pushed v1.3. It should now be similar in both places. It should show up in a bit.

FIXED


Fixed truncate for objects

Thanks for the PR. Do you have any particular use-case for this change? truncate should stay as is, because it's meant to be used with string only. If it somehow gets object passed, we should fix it in this specific place.
We also have serializeException (which can be used for any objects really) that's a safer way to serialize objects and truncate them to a specific size (kinda).

Yes, its happens if server on ajax request returns html.
Thank for answer


The Code Scanner Doesn't always work

Describe the bug
First of all, Thank you for creating this amazing library, and making our life a lot of easier.
I have been trying this project for a while now, and what I have found is that, it works, when it works it's really fast, but it doesn't always work, which is weird, because right now, when I left the office yesterday, it was working perfectly, but now it doesn't, and this is not the first time this happened, I have wasted a lot of hours trying to fix what I was doing wrong only to realize it was not my problem at all, I think there is something wrong with this library internally,
To Reproduce
Steps to reproduce the behavior:

The following is my component

interface ICameraScreenProps {
  children?: ReactNode
  navigation: NavigationProp<RootStackParamList, 'camera'>
}

export const CameraScreen: React.FC<ICameraScreenProps> = ({navigation}) => {
  const setScannedCode = useSetRecoilState(scannedQrCodeAtom)

  const permission = useRecoilValue(cameraPermissionAtom)
  const closeCameraScreen = () => {
    cameraLog.info('Closing Camera Screen...')
    // TAKES YOU BACK ONE SCREEN
    // https://reactnavigation.org/docs/stack-actions/#pop
    navigation.dispatch(StackActions.pop(1))
  }

  const readQrCode = (
    event: NativeSyntheticEvent<{codeStringValue: string}>,
  ) => {
    const code = event.nativeEvent.codeStringValue
    rootLog.info(`The Scanned QR Code: ${code}`)
    setScannedCode(code)
    closeCameraScreen()
  }

  if (!permission) {
    return <SomethingWentWrong text={'Please Provide Camera Permission'} />
  }

  return (
    <Camera
      scanBarcode={true}
      cameraType={CameraType.Back}
      style={[StyleSheet.absoluteFill, styles.cameraContainer]}
      onReadCode={readQrCode}
      showFrame={true}
    />
  )
}

const styles = StyleSheet.create({
  cameraContainer: {
    width: Dimensions.get('window').width,
    height: Dimensions.get('window').height,
    justifyContent: 'center',
    alignContent: 'center',
    zIndex: 9999,
  },
})

This doesn't throw any errors, it just doesn't scan the qr code at all.

Expected behavior
When I open the camera, it shows the frame and animation, and when it works, it's really fast and there is nothing wrong with it, but when it doesn't, it just kills all of my time and I don't know what is going on internally, and I just want to scan a qr code, maybe I am doing something wrong with my code, but what I can't get my head around is the fact that, why does it sometimes work, and why it doesn't in other time.
Smartphone (please complete the following information):

Device: [Samsung Tab A]
OS: [Android 11]

Additional context
I want to stress that, maybe it is my fault, but I wouldn't be creating an issue if it's anything wrong with my code, so am I doing anything wrong here, or is there a step I missed? also, why would it work sometimes, and why it wouldn't in other times,

I am facing the same issue. After opening the scanner if I don't scan for the first time, the scanner is not able to detect it from the second time. However, if I scan on the first attempt it scans everything lightning fast. I have gone through each issues but could not find a reliable solution. Anyone here to help ??

@roshikbangali try to find any performance issues in your code, I found out that one of my useEffect is causing some performance issues, also, try to scan the qr code from exactly above the qr code, I found out that works most of the time, otherwise my problems got fixed instantly.

I've noticed the barcode scanning stops working when navigating between Tab.Screens when using @react-navigation/bottom-tabs
@roshikbangali Did you end up fixing it?

I've noticed the barcode scanning stops working when navigating between Tab.Screens when using @react-navigation/bottom-tabs @roshikbangali Did you end up fixing it?

Yes, I have ended up fixing it. It is now working well.

I've noticed the barcode scanning stops working when navigating between Tab.Screens when using @react-navigation/bottom-tabs @roshikbangali Did you end up fixing it?

Yes, I have ended up fixing it. It is now working well.

Could you share how you solved it please?

I've noticed the barcode scanning stops working when navigating between Tab.Screens when using @react-navigation/bottom-tabs @roshikbangali Did you end up fixing it?

Yes, I have ended up fixing it. It is now working well.

Could you share how you solved it please?

You have to manually restart the library every time you open that scanner page. So for that  I used useFocused from react-navigation-native. When I am focussing on the screen I am calling an useEffect hook and on the useEffect hook I am passing the depedency array as [isFocused] enabling me to load the scanner fresh every time I open the page.

@roshikbangali I also experience this issue. Can you please give more clarity as to what you mean by "load the scanner" - is there a method you are calling for this?

@roshikbangali I also experience this issue. Can you please give more clarity as to what you mean by "load the scanner" - is there a method you are calling for this?

https://github.com/teslamotors/react-native-camera-kit/issues/462#issuecomment-1314168230


docker: Add ubi9 dockerfile

[x] commit message has one of the standard prefixes
[ ] faq.md updated if appropriate
[ ] other documentation is changed or added (if applicable)
[ ] playbook changes run through VPC or QPC (if you have access)
[x] VPC/QPC not applicable for this PR
[ ] for inventory.yml changes, bastillion/nagios/jenkins updated accordingly

ref https://github.com/adoptium/infrastructure/issues/3586

Noting also that the UBI8 file (which this ubi9 one is based on) also has similar "features" to the ones mentioned in my review.
@steelhead31 Do you recall why libnss3.so was passed to yum in this dockerfile? (Ref: https://github.com/adoptium/infrastructure/pull/3169) It looks like that doesn't work on UBI8 either - it's also in the playbooks for RPM-based distributions, although I haven't yet found any it'll work on on aarch64.

Noting also that the UBI8 file (which this ubi9 one is based on) also has similar "features" to the ones mentioned in my review.
@steelhead31 Do you recall why libnss3.so was passed to yum in this dockerfile? (Ref: #3169) It looks like that doesn't work on UBI8 either - it's also in the playbooks for RPM-based distributions, although I haven't yet found any it'll work on on aarch64.

Nope, I can't recall , it will have been something to do with a missing dependency or build from source of some kind, but its possibly redundant now. Id say remove it , and see if we encounter any other issues as a reminder.


Retention policy licence requirement seem wrong, F5 also have right to change the settings

Hello
I have a question about this part of the documentation:
To retain an audit log for longer than 90 days (and up to 1 year), the user who generates the audit log (by performing an audited activity) must be assigned an Office 365 E5 or Microsoft 365 E5 license or have a Microsoft 365 E5 Compliance or E5 eDiscovery and Audit add-on license. To retain audit logs for 10 years, the user who generates the audit log must also be assigned a 10-year audit log retention add-on license in addition to an E5 license.
Audit (Premium) can be obtained via other licences than E5 and E5 add-on.
The retention policy can also be upgraded up to 1 year retention if you have for example and F5 Compliance and Security
Is the quoted sentence is true that ONLY E5 have this feature, or all bundle with Audit (Premium) have this?
(the 10 years retention licence is only available for E5 and not for F5, but this seem overkill when you need only 1 years retention)
Regards

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

ID: e52e1877-191e-5dbe-b805-e755c736b985
Version Independent ID: 69df5ed6-a86f-06d7-1156-e1de211f8e83
Content: Manage audit log retention policies - Microsoft Purview (compliance)
Content Source: microsoft-365/compliance/audit-log-retention-policies.md
Service: o365-seccomp
GitHub Login: @robmazz
Microsoft Alias: robmazz

@adrienmaugard Thanks for your feedback! New issues and pull requests for this content are closed while we update this content to support upcoming feature improvements.


Fix wei/gwei/eth conversions

PR Description

gweiToEth factor should be 9, not 18.
BlockProductionDuty should use weiToEth since the values are wei for execution_payload_value and consensus_block_value coming from the API

Technically, it was working before since was using gweiToEth method, which was wrongly using 18, which is the wei to eth factor, but fixed it so the methods becomes more usable.
Fixed Issue(s)
N/A
Documentation

[X] I thought about documentation and added the doc-change-required label to this PR if updates are required.

Changelog

[X] I thought about adding a changelog entry, and added one if I deemed necessary.

LGTM So the output was correct, but it was correct just because of error in formula, right?

Yes


Insufficient training set sample time, unclear task boundaries

Hi,
so I'm gathering sample data for the micro tasks,
and it seems that running through the whole alphabet leads to only 2 reply-answer solutions before switching to a new task.
However to do any kind of training need to have at least 3 reply-answer solutions, 2 for training and 1 for testing.
Here are some proposed solutions:

at least double the soft limit of the micro tasks
at least halve the alphabet size
(should be done anyways) make a clear indication that there is a task switch

3 should ideally be done in any case, would be easy if for instance all the capitals were taken out of the alphabet, and they were used exclusively for indicating the start of a class of task. (assuming there are less than or equal to 26 variations).  alternatively can simply have for instance A for a different soft-limit micro-task, and B for a different kind of task altogether.

Hi elspru,
you will notice that in later tasks (starting with task group 5), this answer-collecting approach stops working because randomly hitting the correct answer gets more and more difficult (for instance, you need to output a space followed by a correct number in task 5.1).
We're helping the agent by starting with an easy tasks first (1-4) - in a single task instance, there always at least exists a one-to-one mapping between the character on input and the correct answer at that position. You could exploit this fact when you're collecting data for your initial training. But still, your agent will need to be able to cope with the more advanced tasks too, where hitting reward by acting randomly is nearly impossible.
Regarding option 3 - knowing that there was a task [instance] switch - we really prefer if the agent is able to recognize a task [instance] switch on its own, without explicitly being told so. Such an agent should be much more flexible when placed in an unfamiliar environment.

Yes Martin, you bring up a good point,
There is far too much dithering and changing of criteria,
it is really quite ridiculous how there is no standard that is simply expanded upon.
Instead the rules are changed many times without warning.
It's like having to learn a new language every 100 words.
One input, one reward, and one feedback,seems to be the very best format.
I think you should just stick to that for the whole thing.
That would make it much easier to gather training data.
Then wouldn't have to worry about the other things I mentioned.
can't teach something if you don't have anything to train it with,
the random prodding is necessary in the begining now, precisely because there is no solution demonstrated.

It's necessary to show a solver a diverse set of task instances for it to be able to generalize to a solution that works on all of the task instances. This means we actually need to teach a "new language every 100 words" so that the solver can find one unifying, abstract language by some form of generalization.
I think you would not perceive the instance switch as such an interruption if the instances were e.g. images and not temporal sequences. The problem with temporal sequences in real life is that if you have a sequence of them, there may be no clear indication where one temporal sequence ends and another begins. We made the design decision to keep this uncertainty in the tasks. So far we've had reports of multiple teams successfully solving (at least) tasks 1-4, which indicates that it's possible to overcome this uncertainty.

We would like to remind you that we encourage the participants to extend the training datasets, and to use any additional training data you may find useful. This is not forbidden by the rules, quite the contrary: your ideas and suggestions are encouraged, and the outcomes will be considered for not only quantitative, but also qualitative evaluation.
The evaluation still will be performed on the original, non-disclosed evaluation curriculum created by General AI Challenge team.
Best,
Daria,
General AI Challenge team

We would like to remind you that we encourage the participants to extend the training datasets, and to use any additional training data you may find useful. This is not forbidden by the rules, quite the contrary: your ideas and suggestions are encouraged, and the outcomes will be considered for not only quantitative, but also qualitative evaluation.
The evaluation still will be performed on the original, non-disclosed evaluation curriculum created by General AI Challenge team.
Best,
Daria,
General AI Challenge team


Can't open select2 after init

On select2 4.0.2, right after I init select2, I try to do : $(myelement).select2("open") and I have this error : The select2('open') method was called on an element that is not using Select2

This appears to be more of a usage question than a bug report or common feature request.  As we continue to keep the Select2 issue tracker geared towards these two areas, bug reports and common feature requests, we are redirecting usage questions to other communities such as the mailing list, IRC, or Stack Overflow.


Add missing hdf5 host dependency

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@rtburns-jpl it looks like h5py is listed as both a "host" and "run" dependency.
https://github.com/conda-forge/isce3-feedstock/blob/ae1d230ed90c26aca5c3d4cfad780b8f6de6b24e/recipe/meta.yaml#L37
https://github.com/conda-forge/isce3-feedstock/blob/ae1d230ed90c26aca5c3d4cfad780b8f6de6b24e/recipe/meta.yaml#L46
Maybe that could be confusing the algorithm that's used to determine the runtime environment spec?

I believe that is unrelated. Our python dependencies need to be listed in "host" and "run" because our python tests run in build.sh. Run deps aren't available in build.sh, and host deps don't get propagated to run if they're python packages, so we have to list them twice.

OK thanks for the clarification.

host deps don't get propagated to run if they're python packages

weird...


[RFC] Set default initialization selections for BBRegisterRPT

FS6 has a new default initialization method for bbregister (mri_coreg). I've added an updated inputspec to Nipype with nipy/nipype#1811, but, as discussed Friday, I'm trying to avoid depending on unreleased Nipype features.
I'm proposing to use --init-coreg in FSv6.0, and --init-fsl in FSv5.3. By using a modified init trait in both cases, this allows BBRegisterRPT to be called without an explicit init parameter, in either case, which means the check doesn't need to be made at workflow creation.
What do people think about (a) the initialization choices; (b) the method for setting them?

It has been a long-standing nipype policy to avoid setting defaults unless necessary. This way parameters used in an analysis are more explicit when reading workflow code.


TypeError: callback is not a function

Hello, I have problem with first run example:
I did:
~$ cucumber.js ./node_modules/protractor-cucumber/example/features/homepage.feature
I getting:
Feature: Homepage
As a user
I want to visit the homepage
So that I can access the various features on offer
events.js:141
throw er; // Unhandled 'error' event
^
TypeError: callback is not a function
at new World (/home/aleksandr/node_modules/protractor-cucumber/lib/index.js:35:5)
I installed all tools as described in instruction, step by step before.
Thanks for soon answer.

It looks like relates to the latest cucumber.js 0.8.0

I get the same. Just installed today.


listener: Re-target pInternalData on JoltPhysicsCollisionEvent copy/move

Needs this to match the parent structure, not some other object that
may be going to be destroyed.
Closes: #51

Works like a charm for me now.


[sdk] update to Simplicity SDK v2024.6.1-0

This PR just updates the simplicity_sdk to https://github.com/SiliconLabs/simplicity_sdk/commit/aa5ce2e835dfdce8c20fb828f27d3a261946f946 (v2024.6.1-0)
There were no changes to protocol/openthread/platform-abstraction/efr32 between https://github.com/SiliconLabs/simplicity_sdk/commit/a1a37fa7dda1f76ea70229aa4ffe2e47cb066edc (v2024.6.0) and https://github.com/SiliconLabs/simplicity_sdk/commit/aa5ce2e835dfdce8c20fb828f27d3a261946f946 (v2024.6.1-0)

@jwhui please review this when you get a change, Thanks!


Create dataset loader for indo wiki paralel corpora

NusaCatalogue: https://indonlp.github.io/nusa-catalogue/card.html?id_wiki_parallel

Dataset
id_wiki_parallel

Description
Manually aligned parallel corpora from Wikipedia

License
Unknown

duplicated with https://github.com/IndoNLP/nusa-crowd/issues/228


[Fixed] #282

See #282. My pull request delete unexpected output in Sublime Text console after saving files.
Thanks.

merged as 5f81753dae3df2f57d592cb5d42534251d885b56


Add SES lockdown to testrunners

root: Add SES dependency
Add SES lockdown to testrunners

Part of #412
The CLI package still has the obscure error, so I couldn’t add SES to the testrunner there. Do we
want SES added to our applications? (Eg adding it now to the faucet.)

@willclarktech do you get this error locally as well?
@cosmjs/faucet-client: /home/circleci/project/node_modules/ses/lockdown.js:17
@cosmjs/faucet-client: import { assign } from './src/commons.js';
@cosmjs/faucet-client:        ^
@cosmjs/faucet-client: SyntaxError: Unexpected token {
@cosmjs/faucet-client:     at Module._compile (internal/modules/cjs/loader.js:723:23)
@cosmjs/faucet-client:     at Object.Module._extensions..js (internal/modules/cjs/loader.js:789:10)
@cosmjs/faucet-client:     at Module.load (internal/modules/cjs/loader.js:653:32)
@cosmjs/faucet-client:     at tryModuleLoad (internal/modules/cjs/loader.js:593:12)
@cosmjs/faucet-client:     at Function.Module._load (internal/modules/cjs/loader.js:585:3)
@cosmjs/faucet-client:     at Module.require (internal/modules/cjs/loader.js:692:17)
@cosmjs/faucet-client:     at require (internal/modules/cjs/helpers.js:25:18)
@cosmjs/faucet-client:     at Object.<anonymous> (/home/circleci/project/packages/faucet-client/jasmine-testrunner.js:4:1)
@cosmjs/faucet-client:     at Module._compile (internal/modules/cjs/loader.js:778:30)
@cosmjs/faucet-client:     at Object.Module._extensions..js (internal/modules/cjs/loader.js:789:10)
@cosmjs/faucet-client: error Command failed with exit code 1.

Is it because node_modules/ses/lockdown.js contains JS syntax too modern for Node.js 10?

@willclarktech do you get this error locally as well?
@cosmjs/faucet-client: /home/circleci/project/node_modules/ses/lockdown.js:17
@cosmjs/faucet-client: import { assign } from './src/commons.js';
@cosmjs/faucet-client:        ^
@cosmjs/faucet-client: SyntaxError: Unexpected token {
@cosmjs/faucet-client:     at Module._compile (internal/modules/cjs/loader.js:723:23)
@cosmjs/faucet-client:     at Object.Module._extensions..js (internal/modules/cjs/loader.js:789:10)
@cosmjs/faucet-client:     at Module.load (internal/modules/cjs/loader.js:653:32)
@cosmjs/faucet-client:     at tryModuleLoad (internal/modules/cjs/loader.js:593:12)
@cosmjs/faucet-client:     at Function.Module._load (internal/modules/cjs/loader.js:585:3)
@cosmjs/faucet-client:     at Module.require (internal/modules/cjs/loader.js:692:17)
@cosmjs/faucet-client:     at require (internal/modules/cjs/helpers.js:25:18)
@cosmjs/faucet-client:     at Object.<anonymous> (/home/circleci/project/packages/faucet-client/jasmine-testrunner.js:4:1)
@cosmjs/faucet-client:     at Module._compile (internal/modules/cjs/loader.js:778:30)
@cosmjs/faucet-client:     at Object.Module._extensions..js (internal/modules/cjs/loader.js:789:10)
@cosmjs/faucet-client: error Command failed with exit code 1.

Is it because node_modules/ses/lockdown.js contains JS syntax too modern for Node.js 10?

@webmaster128 Yes, I missed this locally because I was running the tests with Node v12.

@webmaster128 Yes, I missed this locally because I was running the tests with Node v12.

@webmaster128 See

https://github.com/Agoric/SES-shim/issues/252
https://github.com/Agoric/SES-shim/issues/22

@webmaster128 See

https://github.com/Agoric/SES-shim/issues/252
https://github.com/Agoric/SES-shim/issues/22

@kriskowal what Node.js versions are supported by the "ses" package? Is this documented somewhere?
Node.js 10 will EOL at the end of April, so we could think about dropping it. But it would be good to get this documented.

@kriskowal what Node.js versions are supported by the "ses" package? Is this documented somewhere?
Node.js 10 will EOL at the end of April, so we could think about dropping it. But it would be good to get this documented.

@willclarktech what about a separate SES CI job that runs the tests in node.js 12 and we have something like
if (provess.env.SES_ENABLED) {
  require("ses/lockdown");
  // eslint-disable-next-line no-undef
  lockdown();
}

@willclarktech what about a separate SES CI job that runs the tests in node.js 12 and we have something like
if (provess.env.SES_ENABLED) {
  require("ses/lockdown");
  // eslint-disable-next-line no-undef
  lockdown();
}

@kriskowal what Node.js versions are supported by the "ses" package? Is this documented somewhere?
Node.js 10 will EOL at the end of April, so we could think about dropping it. But it would be good to get this documented.

We at least informally support Node.js 10 since our partners at MetaMask use that too. cc @kumavis. For those cases, SES ships with a rolled-up CommonJS distribution like ses/dist/lockdown.cjs. In Node.js 10, you get ses/dist/ses.cjs by virtue of the package.json "main" entry when you require("ses"), and on anything more recent, require("ses/lockdown") also works. I think you need to expressly dig up require("ses/dist/lockdown.js") instead of require("ses/lockdown") that far back in time.

@kriskowal what Node.js versions are supported by the "ses" package? Is this documented somewhere?
Node.js 10 will EOL at the end of April, so we could think about dropping it. But it would be good to get this documented.

We at least informally support Node.js 10 since our partners at MetaMask use that too. cc @kumavis. For those cases, SES ships with a rolled-up CommonJS distribution like ses/dist/lockdown.cjs. In Node.js 10, you get ses/dist/ses.cjs by virtue of the package.json "main" entry when you require("ses"), and on anything more recent, require("ses/lockdown") also works. I think you need to expressly dig up require("ses/dist/lockdown.js") instead of require("ses/lockdown") that far back in time.

Hey, what's the status of this? Should be part of our service agreement with Agoric.

This is done and running. The remaining trouble was resolved by dropping support for the now unmaintained Node.js 10.
Does this answer your question @hxrts? I don't know about this service agreement.

Ah, should have read above. Yes it does, though @kriskowal can maybe comment on whether Agoric plans to do anything further.

The best person to discuss the scope of Agoric’s service agreement is Vanessa Pestritto at Agoric.
My expectation is that I am to be available to assist if ever a problem arises with compatibility of a new dependency and to provide help upgrading SES, particularly for breaking-changes that might impact compatibility with CosmJS dependencies.
Notably, we’re planning a version bump (0.+1) to tighten security by forbidding the use of the Node.js domain module in any application that uses SES. Incorporating 0.minor version bump is not automatic and manually upgrading could reveal that some applications have an existing dependency on domain. I expect that I will need to be available to assist in relieving those dependencies.


Update to latest eventing deps

Proposed Changes

Adding latest eventing deps
Needed for #688

/assign @slinkydeveloper

/ok-to-test

Reason:FailedMount
Message:MountVolume.SetUp failed for volume "e2e-kafka-r-binary-plain-v1beta1-token-s729t" : failed to sync secret cache: timed out waiting for the condition

Seems irrelevant.

/retest


listaccounts returning invalid number.

When I type "vertcoind listaccounts"
it shows
{
"" : -20686.04748736
}
I have "" (empty string) account and it should show a positive number.
I get correct result from "vertcoind getbalance" command.
This was working well before the hardfork update. Maybe something's broken.

Is this still a problem?

I cannot reproduce this bug so I am assuming this issue is closed for now.


Improve create_release_pr

We noticed that the chart version for https://github.com/giantswarm/cluster-api-provider-cloud-director-app is not updated by CI automatically. Then I checked the CI logs and found this.

Basically, we have different cases regarding repo name vs app name

AppName = RepoName
AppName = RepoName+"-app"
AppName+"-app" = RepoName

1 and 2 are already handled. 3 is not handled in the current form. This PR handles it too.
Also note that it is not only a problem in https://github.com/giantswarm/cluster-api-provider-cloud-director-app. I checked other providers and they are the same too. Example https://github.com/giantswarm/cluster-api-provider-aws-app
Checklist

[x] Update changelog in CHANGELOG.md.

NEVERMIND


huggingface/text-generation-inference integration

Hi,
I'm curious to know if it's possible to utilize the text generation inference of HuggingFace with Langchain JS. I have found that it can be used in a non-document way, as demonstrated in the following code snippet:
import { HuggingFaceInference } from "langchain/llms/hf";

const run = async () => {
  const model = new HuggingFaceInference({
    model: "huggingface/generation-text-inference",
    temperature: 0.7,
    maxTokens: 50,
  });
  const res = await model.call("Question: What would be a good name for a company that makes colourful socks?");
  console.log({res });
};

run();

However, this leaves me wondering whether HuggingFace inference can be used with Langchain JS, or if there is a specific way to utilize the inference server endpoints, such as /generate and /generate_stream. I would greatly appreciate it if you could provide guidance on this matter.
Additionally, I would like to inquire if there are any plans to enhance the integration of HuggingFace/text-generation-inference or to document its usage in the Langchain JS library. This would be extremely helpful for the community and for developers looking to leverage the powerful features of HuggingFace text generation in their Langchain JS projects.
Thank you for your time and consideration. I look forward to your response and any insights you can provide.

I see, but you are wrong in your code, it is necessary to use model: "huggingface/generation-text-inference" and also specify a fake api key to make it work. I hope there will be better integration/documentation with huggingface/generation-text-inference and langchain js in the future as its a pretty good backend to run inference servers. I'll try to put together a PR in my spare time.


Bizarre length related panic with match statement with attribute in closure, in function call.

Given a these very specific inputs, rustfmt panics. Try as I may, these examples are as minimal as I can make them.
Functions named panics<n> contain code blocks that cause the panic, while functions named succeeds<n> do not.
fn panics1() {
    Some(()).map(|_| {
        #[allow(deprecated)]
        match 1.aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa()
        {
            _ => {}
        }
    });
    // thread 'main' panicked at 'Request to format inverted span: Loc { file:
    // SourceFile(./src/main.rs), line: 4, col: CharPos(28),
    // col_display: 28 } to Loc { file: SourceFile(/Users/a/d/tmp/rustfmt_issue_repro/src/main.rs),
    // line: 3, col: CharPos(8), col_display: 8 }', src/tools/rustfmt/src/missed_spans.rs:86:9
    // note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace.
}

fn succeeds1() {
    Some(()).map(|_| {
        #[allow(deprecated)]
        match 1.aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa() {
            _ => {}
        }
    });
}

fn panics2() {
    foooooooooooooooooooooooooooooooooooooooo.map(|f| {
        #[allow(deprecated)]
        match f.a(aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa)
        {
            _ => {}
        }
    });
}

fn panics3() {
    foo.map(|f| {
        #[allow(deprecated)]
        match f.a(aaaaaaaaaaaaaaaaaaaaaaaaaaaaaa, bbbbbbbbbbbbbbbbbbbbbbbbbbbbbb) {
            _ => {}
        }
    });
}

fn succeeds3() {
    foo.map(|f| {
        #[allow(deprecated)]
        match f.a(aaaaaaaaaaaaaaaaaaaaaaaaaaaaa, bbbbbbbbbbbbbbbbbbbbbbbbbbbbb) {
            _ => {}
        }
    });
}

rustfmt --version
# rustfmt 1.4.8-stable (afb1ee1c 2019-09-08)

Related/possible dupe of #3934, already fixed in the latest version of rustfmt on master via #3935

Thank you @bddap for the report and @calebcartwright for the confirmation, closing as this is fixed on the latest master.


Consider adding safe effect wrapper

If somebody wants to work in it - let me know. We can then establish how this would work

@Andarist could you describe what is the idea behind "safe" effect?

Im not yet exactly sure about the API, but one idea is to:
const safe = effect => (effect[SAFE] = true)

function* someSaga() {
  // where result is of type Error or Result
  const result = yield safe(call(someAPI, arg1))
  // in case of an error would die gracefully instead of failing the parent
  yield safe(fork(otherSaga, arg2))
}

In this variant SAFE would have to be interpreted in the redux-saga's core.
Other way of implementing it would be require implementing equivalent of:
function* safe(effect) {
	try {
		return { result: yield effect }
	} catch (err) {
		return { err }
	}
}

In this variant fork would have to be treated differently, so the task descriptor could get returned to the caller, instead of blocking.

I can try do this

Cool!
Seems to be that second variant of what I have proposed (no SAFE symbol, but rather just a simple wrapper around yield effect) is what we should implement.

I'm using next pattern in my projects, that's kinda 2nd version
function* safe(effect) {
	try {
		return { result: yield effect, error: null }
	} catch (error) {
		return { result: null, error }
	}
}

@Andarist interested in working on this enhancement

@younesmln go ahead, if you need any help - please just ask

@restrry how do you use it?

@jaulz I think the problem in the snippet that it uses spawn

The parent will not wait for detached tasks to terminate before returning and all events which may affect the parent or the detached task are completely independents (error, cancellation).

https://github.com/redux-saga/redux-saga/tree/master/docs/api#spawnfn-args
could you try to use fork instead?

I have tried to implement via @restrry solution however I get the following error:
Error: "fork: argument of type {context, fn} has undefined or null `fn`"

this happens when i try to use it at a core level, basically my goal being to wrap my entire process:
export default function* rootSaga() {
  yield all([
    takeLatest(actions.CREATE_FILE_UPLOAD, safe(fork(uploadFile))),
  ]);
}

I have also tried without fork and even though it felt wrong I've also tried wrapping the takeLatest instead. All don't work and I'm pretty sure had the basically the same error message.
Perhapse you could explain why we return { result, error} if there is one so I can attempt to understand the underlying functionality? Also maybe I'm just using this wrong?
For now I am just going to wrap the code I know is likely to crash in a try catch, my main reason for looking into this however is that I don't plan on ever actioning the crash (its fine if this happens) so I would rather enrich my saga via composition instead of having to dump code into a working function.

@fgfmichael the idea is: don't yield your effects immediately inside your function, let's another "safe" function do it.
Example:
function* safe(effect) {
  try {
    return { response: yield effect }
  } catch (error) {
    return { error }
  }
}

function* fetchProducts() {
  const { response, error } = yield safe(call(Api.fetch, '/products'))
  if (response)
    yield put({ type: 'PRODUCTS_RECEIVED', products: response })
  else
    yield put({ type: 'PRODUCTS_REQUEST_FAILED', error })
}

Hi, Is anyone working on this issue @Andarist?

Hi, everyone)) I'm be honored to take this issue. I already have some code written for exactly the same purpose)
#2320

Hey, @Andarist )
I've created a draft pull request for this issue #2392 .
It contains a few red tests on how this feature could've worked if it was implemented. Its a little different from what was discussed in this issue.
Please take a look, I want your opinion on that.
After we reach an agreement on the api in tests, the implementation itself is a piece of cake)))


Named exports and HMR

I am using Vite + MDX with this plugin. It works fine. But refresh is slow. Not like HMR but a page refresh.
Then I found this https://github.com/vitejs/vite/discussions/4583
Frontmatter add named exports to MDX:
export const title = 'My Title';

export default function MDX() { ... }

The named export title will break HMR, meaning that whenever I modify a MDX file, the page will refresh instead of hot module replacement.
To solve the issue, I think it is better to generate code without named exports:
function MDX() { ... }

MDX.title = 'My Title';

export default MDX;

Thanks for reporting this!
I’m personally not a fan of this solution. I think it’s a bug in the HMR library, not this package.
Have you tried using a name instead? That would output something like:
export const frontmatter = { title: 'My Title' };

export default function MDX() { ... }

So this still outputs an exported variable, but it only happens once, not for every key.
Also this might help, but I need to wait for an answer to be sure.

I’m going to close this as a won’t fix, because it’s a bug in the HMR library, not this one.
One solution you could try is to create a recma plugin for MDX which removes all exports, then assigns all exported variables to MDXContent.
In:
export const foo = 'asd'

export function bar() {}

export default function MDXContent() {}

Out:
const foo = 'asd'

function bar() {}

export default function MDXContent() {}

MDXContent.foo = foo
MDXContent.bar = bar


DB pre-condition branch traversal

Check that all branches are properly traversed for db condition checks
ex: https://github.com/valory-xyz/autonomous-fund/pull/58#discussion_r1052636785

As demonstrated here https://github.com/valory-xyz/price-oracle/pull/24 this is not a bug


How to use “sfdx force:source:pull” when folders other than “main/default” are used and components are added in the scratch org?

I've setup the folder structure for a new project following the patterns identified here. When I add e.g. a new SObject in the scratch org and use:
sfdx force:source:pull
the new components are put in a fixed folder "main/default" - there is no option to nominate the correct folder. So the folder structure is compromised.
(When components are already present locally in folders other than "main/default" the pull does place the changes correctly.)
Is there a work-around for this problem? If there isn't this multiple folder approach needs revising. So far I've wasted several hours trying to find a work around.
(Same question posted here https://salesforce.stackexchange.com/questions/206847/how-to-use-sfdx-forcesourcepull-when-folders-other-than-main-default-are-u.)

I believe this is the way the system is designed. All new changes made in the Salesforce org are put in the "default" folder when using sfdx force:source:pull. This ensures that the CLI always knows where changes go that you have not yet allocated to a different part of your folder structure.
Once you move the file to another part of the folder structure, the CLI retains that information and then is able to maintain that change going forward.
This makes perfect sense to me, until I tell a file where to go, the system doesn't know, hence I have a default place for files the system doesn't know what to do with.

I've been wondering about adding the main/default source folder to the .gitignore file. This would force contributors to move the newly added source folders to an appropriate directory. I worry that the default folder will become a dumping ground for all source files.


Expose homee status

As far as I understand, the integration exposes only sensor values from devices.
I would like to have the option, to see and update the homee status value reflecting if at home or sleeping or ...

@hajo62
I jave prepared a beta release to test an initial implementation of the homee status.
Please follow Beta installation howto and see if it works for you.
The status will be added as an entity to the homee device itself

At the moment, this requires that all devices are imported. when setting up homee

Closed with Release 2.7.0


PUT for create requires extension of LDP

In the section HTTP PUT to create the following is written.

Another useful feature that is not yet part of LDP deals with using HTTP PUT to create new resources. This feature is really useful when the clients wants to make sure it has absolute control over the URI namespace -- e.g. migrating from one pod to another. Although this feature is defined in HTTP1.1 RFC2616, we decided to improve it slightly by having servers create the full path to the resource, if it didn't exist before.

Currently it is not clear how a client would ever know that a server implemented this feature without out-of-band information. Because LDP Containers are not Intuitive Containers, a client has no guarantee that when doing a PUT in a namespace it is even allowed to do so in that part of the name space. Furthermore it has no way of knowing in a standard way that the intermediate directories will be created.
I don't see this listed on the LDP Next Charter, though there is still time to add this feature there if it is felt to be useful enough.

Yes, this is a non-standard improvement to PUT that we could propose as an extension to LDP. It is currently described (specced) in this doc so that people writing apps for SoLiD can speed things up a bit.
In any case, a client can still create the container hierarchy if it is unaware that PUT also creates parent containers.

Yes, this is a non-standard improvement to PUT that we could propose as an extension to LDP

It would help then if it were part of the LDP Next Charter. If you don't put it there it won't even be considered.

In any case, a client can still create the container hierarchy if it is unaware that PUT also creates parent containers.

Why would a client bother trying to do something when there is a chance in 10 or 100 that it will work correctly? Unless the client can know that this will work, there is no reason to guess. Otherwise the client will have to have code such as if (origin=="rww.io" | origin = "data.fm" | .... ) { ... } else { .. } and that code would need to be changed every time a new server came up hosting this extension.

I see your point. I think we can address it in LDP Next, where we plan to have a way for servers to advertise what they support.

Well the easy way to do that is to specify sublcassses of ldp:Container such as the iContainer I proposed. But it still would help to have wider agreement of the meaning, and have a w3c namespace, so you need to get it explicitly in the charter.

Getting things into the charter is not that easy. It requires group agreement, so the best I (or you) can do is to suggest it to the group during the next call.

well in any case, one cannot specify this behavior without indicating that it is available. So one needs to create a subclass of ldp:Container - such as iContainer mentioned in issue 50
http://www.w3.org/2012/ldp/track/issues/50
(but one could extend it, for the more general behavior required here) so that it is clear to clients what they can expect.


VS hangs after switching between local and remote hosts

Connect to remote
Restart VS
Connect to remote
Connect to local
Connect to remote again
Wait about a minute, click cancel

Actual: VS UI is unresponsive
Expected: Connection attempt is canceled

No repro. Closing for now.


bugfix: limit SQL instance returned to match

There was a bug here from Microsoft's end that would return a list of SQL instances if they are partial matches. For instance, this command when ran to retrieve database 'Test' will return a list if you have multiple databases called 'Test', 'TestTemp', 'TestDev'. The only way I found to fix this issue and keep it running on schedule is if we filter the returned result down again to only be the instance which ends in ";$BkupToRestore" (hence ensuring no prefix AND no postfix) . This worked for my case and I don't envision it would be a breaking change, just a bugfix.

Hi @treytro thanks for sharing this great azure automation ps script. I had a unique case that fell outside of what was being handled. Here's the fix I used. Enjoy!
Thanks again

Oh, also here is the corresponding behavior I encountered that was mentioned by MSFT team as well:
https://github.com/Azure/azure-powershell/issues/13832

Great catch @kaburkett - thank you!


Added an alternative and maybe more simple way to reverse strings

@all-contributors please add @RasmusKnothNielsen  for code


Updated ec conceptual docs for format overrides

Updated ec format and kindof quantity markdown files.
Added lines and examples explaining how format definitions and overrides handle unit labels which are null or empty strings.

Why is this targeting the imodel02 branch?

New branch created from master.
New PR #6253