crates.io version appears to be missing definitions.rs

👋
I've been working on a toy renderer and started to do an overhaul over the weekend. It's been awhile since I last updated so I updated Ash to the latest version which, at the time of this writing I believe is 0.37.3+1.3.251.
Currently, as least in CLion(my IDE), it reports a bunch of unresolved references including things like ash::vk::CommandBuffer, as well as other key components when I grab Ash as a Crate. That being said, things weirdly still appear to build and run without issue so far.
As far as I can tell the issue has something to do with the code that's on crates.io; if I specify this repository as the source, things appear to be fine and the definitions are found.
Should I just be pointing to Github from now on? Or is there some other issue on my end I need to look into? (apologies, still a bit new to Rust)
Thanks!

In general, if CLion generates errors but rustc doesn't, it's a bug in CLion. Using rust-analyzer instead of its built-in Rust support will be more robust.

Wouldn't it be surprising if you're the first of 225.000 downloaders of the 0.37.3 crate to report this, while all others silently use the repository source as suggested to work around this?
No, especially if the crate still compiles a critical file cannot just be missing. It must be the IDE messing up, or (though very unlikely) Rust still having the compiled ash/ crate somewhere in your target/ folder while the file got deleted or renamed via an accidental hot-key while browsing the source file after a jump-to-symbol.
(I modify files that way by accident from time to time)
The source of the crates.io package can be seen on docs.rs, and it is confirmed to contain definitions.rs: https://docs.rs/crate/ash/0.37.3+1.3.251/source/src/vk/definitions.rs
I will close this as it is not an ash issue, but you can try to cargo clean the project and clear out crate sources from ~/.cargo to have them redownloaded.

Apologies! I just wasn't sure where to start looking for the problem given how weird it seemed and as I hadn't run into this before.  I had tried completely cleaning and wiping .cargo prior to writing which didn't help. Also tried re-installing Rust without success.
Didn't know docs.rs was synced with the crate, will keep that in mind from now on.
I'll open an issue with Jetbrains then, also tried rust-analyser just now like Ralith suggested which also helps confirms things.
Thanks!

Did you also try wiping out ~/.cargo and target/, after first shutting down CLion, and only starting it again after the removal of those folders has completed?

Did you also try wiping out ~/.cargo and target/, after first shutting down CLion, and only starting it again after the removal of those folders has completed?

Good point, I actually hadn't tried shutting down CLion + wiping those folders. Unfortunately, just tried now with no luck either.
Oh well, no worries. I'll just source from Git for the time being or switch to VSCode/rust-analyser until they can fix their plugin.

My first guess is that definitions.rs comes in at a little over 50k lines now. Perhaps that's too much though it doesn't explain why a git revision is fine.
More likely they keep a broken file/compilation cache elsewhere?

That certainly sounds like a possibility. Well hopefully it gets fixed soon.
Will let you know if they happen to come back to me with anything interesting.


Missing documentation on how to configure the maintenance control configuration using ARM templates.

Missing documentation on how a maintenance control configuration can be linked to a scale set using an Azure Resource Manager template. I can't find any documentation how to link this. Also isn't present in the ARM template export from the Azure Portal when linked using the Portal (the microsoft.maintenance/maintenanceconfigurations is present, so is the scaleset, but no link between the two).

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

ID: 859bb60a-6201-b72f-2abb-b0b9fc06504e
Version Independent ID: 0b5bd7af-a437-a729-7e06-70182b8463d8
Content: Overview of Maintenance control for OS image upgrades on Azure virtual machine scale sets - Azure Virtual Machines
Content Source: articles/virtual-machines/virtual-machine-scale-sets-maintenance-control.md
Service: virtual-machine-scale-sets
GitHub Login: @ju-shim
Microsoft Alias: jushiman

@ppaschedag we'll review your feedback and get back to you shortly. Thanks.

@ppaschedag
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@ju-shim
Can you please check and add your comments on this doc update request as applicable.

Hello,
Thank you for the request. Currently, you are right, there is no clear documentation. I'll work with the right PMs to get some documentation up, but there is no immediate update possible right now. I'll circle back to this issue once we have something ready.
Thank you for your patience.

Hello,
Just to update this thread. Me and the PM team have been working on a new document to address this need. We currently have a draft in place and I am hoping we can get it reviewed and published this month. Thank you for your patience.
Julia (VM/VMSS writer)

Is there any progress on this, it seem it is still not published?
I would like to get this done in my ARM templates, so the documentation would be appreciated!

@ppaschedag - The template is being published shortly. You will find it under the following section as "ARM Template" once it's live sometime today: https://docs.microsoft.com/en-us/azure/virtual-machines/virtual-machine-scale-sets-maintenance-control
Thank you for your patience.
Julia (VM/VMSS writer)

#please-close

The example is not complete, without example values it is pretty hard to figure out. Also the "scope" property is not mentioned in the docs, but this seems to be required for an extension resource. Without it deploying the ARM results in a "bad request".
I got it to work with by setting the scope, however the logic about why the resourceId has to have the same value eludes me.
{
"type": "Microsoft.Maintenance/configurationAssignments",
"apiVersion": "2021-09-01-preview",
"name": "[variables('maintenanceConfigurationAssignmentName')]",
"location": "westeurope",
"scope": "[variables('scaleSetId')]",
"properties": {
"maintenanceConfigurationId": "[variables('maintenanceId')]",
"resourceId": "[variables('scaleSetId')]"
},
"dependsOn": [
"[variables('maintenanceId')]",
"[variables('scaleSetId')]"
]
}


Add outde school

Add outde (Ho Chi Minh City Open University) to support list

@buingocminhhuong I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host outde.edu.vn not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


feat: Add customizable directories

This PR adds a var for the working directory to be able to save the config on another location then /home/homeassistant. In my case it is a separate zfs dataset that i put in /config on all my linux containers.
This PR also adds a var for the logging file location. I personally want to put this on /var/log/ so i can scrape it more easily with rsyslog.
I have also fixed some typos and added some optimizations. Please have a look at the different commits.

Hey @timvy,
thanks for your PR, great.


ambiguous versions of binObs_py.F

Two versions of binObs_py.F are in the source tree:
1. ./src/Components/misc/scat/binObs_py.F
2. ./src/Shared/GMAO_Shared/GMAO_pyobs/binObs_py.F
This could result in unintended behavior depending on how code is compiled/linked.

Anton - I changed the naming in scat to be unambiguous


Flow fails during writing powered verilog

Description

This MPW user_project_wrapper design: https://github.com/The-OpenROAD-Project/caravel_user_project/blob/Procenne-Digital-Design/secure-memory/openlane/user_project_wrapper/config.tcl
Previously its worked fine. Latest OpenLane throws error and failed to complete writing powered verilog stage.
Expected behavior

Complete the flow.
Environment

Kernel: Linux v3.10.0-1160.49.1.el7.x86_64
Distribution: centos 7
Python: v3.6.8 (OK)
Container Engine: docker v20.10.16 (OK)
OpenLane Git Version: a0b19fa68b79eff3e29b674106b06d20aab637f1
pip: INSTALLED
python-venv: INSTALLED
---
PDK Version Verification Status: OK
---
Git Log (Last 3 Commits)

a0b19fa 2022-11-01T19:09:25+02:00 Add Wire Length Checker (#1463) - Mohamed Gaber -  (grafted, HEAD, tag: 2022.11.02)
---
Git Remotes

origin	https://github.com/The-OpenROAD-Project/OpenLane (fetch)
origin	https://github.com/The-OpenROAD-Project/OpenLane (push)

Reproduction Material

Its quite confusing to generate the test case.
Please run this MPW CI design by following below steps:
git clone https://github.com/The-OpenROAD-Project/OpenLane-MPW-CI.git
cd OpenLane-MPW-CI

Edit and save  openlane-tag value to 2022.11.02
./scripts/setup-ci.sh
./scripts/run-design.sh secure-memory secure-memory-wrapper

MPW CI Logs: https://jenkins.openroad.tools/blue/organizations/jenkins/OpenLane-MPW-CI-Public%2Fproduction-small-2/detail/PR-97/6/pipeline/1293
Logs

[STEP 27]
[INFO]: Writing Powered Verilog (logs: logs/signoff/27-write_powered_def.log, logs/signoff/27-write_powered_verilog.log)...
[ERROR]: during executing: "openroad -exit -no_init -python /openlane/scripts/odbpy/power_utils.py write_powered_def --output /openlane/designs/secure-memory-wrapper/runs/RUN_2022.11.08_10.46.30/tmp/signoff/26-user_project_wrapper.p.def --input-lef /openlane/designs/secure-memory-wrapper/runs/RUN_2022.11.08_10.46.30/tmp/merged.nom.lef --power-port vccd1 --ground-port vssd1 --powered-netlist /openlane/designs/secure-memory-wrapper/runs/RUN_2022.11.08_10.46.30/tmp/synthesis/pg_define.v /openlane/designs/secure-memory-wrapper/runs/RUN_2022.11.08_10.46.30/results/routing/user_project_wrapper.def |& tee /dev/null /openlane/designs/secure-memory-wrapper/runs/RUN_2022.11.08_10.46.30/logs/signoff/27-write_powered_def.log"
[ERROR]: Exit code: 1
[ERROR]: Last 10 lines:
[INFO ODB-0127] Reading DEF file: /openlane/designs/secure-memory-wrapper/runs/RUN_2022.11.08_10.46.30/tmp/synthesis/pg_define.intermediate.def
[WARNING ODB-0148] error: Cannot open DEF file /openlane/designs/secure-memory-wrapper/runs/RUN_2022.11.08_10.46.30/tmp/synthesis/pg_define.intermediate.def
[WARNING ODB-0129] Error: Failed to read DEF file
Top-level design name: user_project_wrapper
Found default power net 'vccd1'
Found default ground net 'vssd1'
Found 4 power ports.
Found 4 ground ports.
Modified power connections of 3/3 cells.
child process exited abnormally

[INFO]: Saving current set of views in 'results/final'...
[INFO]: Generating final set of reports...
[INFO]: Created manufacturability report at 'reports/manufacturability.rpt'.
[INFO]: Created metrics report at 'reports/metrics.csv'.
[INFO]: Saving runtime environment...
[ERROR]: Flow failed.
[INFO]: The failure may have been because of the following warnings:
[WARNING]: Skipping Tap/Decap Insertion.

    while executing
"flow_fail"
    (procedure "try_catch" line 17)
    invoked from within
"try_catch $::env(OPENROAD_BIN) -exit -no_init -python $::env(SCRIPTS_DIR)/odbpy/power_utils.py write_powered_def --output $arg_values(-output_def)  --..."
    (procedure "write_powered_verilog" line 55)
    invoked from within
"write_powered_verilog -output_nl $netlist_name -output_pnl $powered_netlist_name -output_def $powered_def_name"
    (procedure "run_lvs" line 13)
    invoked from within
"run_lvs"
    (procedure "run_lvs_step" line 10)
    invoked from within
"[lindex $step_exe 0] [lindex $step_exe 1] "
    (procedure "run_non_interactive_mode" line 52)
    invoked from within
"run_non_interactive_mode {*}$argv"
    invoked from within
"if { [info exists flags_map(-interactive)] || [info exists flags_map(-it)] } {
    if { [info exists arg_values(-file)] } {
        run_file [file nor..."
    (file "./flow.tcl" line 411)

Gah... The worst script in the codebase

With OpenLane tag: 2022.11.30 the log as follows:
[STEP 27]

[INFO]: Writing Powered Verilog (logs: logs/signoff/27-write_powered_def.log, logs/signoff/27-write_powered_verilog.log)...

[ERROR]: during executing: "openroad -exit -no_init -python /openlane/scripts/odbpy/power_utils.py write_powered_def --output /openlane/designs/secure-memory-wrapper/runs/RUN_2022.11.30_22.19.27/tmp/signoff/26-user_project_wrapper.p.def --input-lef /openlane/designs/secure-memory-wrapper/runs/RUN_2022.11.30_22.19.27/tmp/merged.nom.lef --power-port vccd1 --ground-port vssd1 --powered-netlist /openlane/designs/secure-memory-wrapper/runs/RUN_2022.11.30_22.19.27/tmp/synthesis/pg_define.v /openlane/designs/secure-memory-wrapper/runs/RUN_2022.11.30_22.19.27/results/routing/user_project_wrapper.def |& tee /dev/null /openlane/designs/secure-memory-wrapper/runs/RUN_2022.11.30_22.19.27/logs/signoff/27-write_powered_def.log"
[ERROR]: Exit code: 1
[ERROR]: Last 10 lines:

  File "/usr/local/lib/python3.6/site-packages/click/core.py", line 1659, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/local/lib/python3.6/site-packages/click/core.py", line 1395, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python3.6/site-packages/click/core.py", line 754, in invoke
    return __callback(*args, **kwargs)
  File "/openlane/scripts/odbpy/power_utils.py", line 222, in write_powered_def
    assert power.name == reader.name
AttributeError: 'OdbReader' object has no attribute 'name'
child process exited abnormally

Hi everyone,
I get the same issue (the power_utils.py one). Reading the file 27-write_powered_verilog.log, I found this error :
openroad>         read_verilog /home/mpotereau/DigitalFlowTest/gf_spi_test/openl  ane/user_project_wrapper/runs/23_01_10_16_10/tmp/synthesis/pg_define.v
[ERROR STA-0164] /home/mpotereau/DigitalFlowTest/gf_spi_test/openlane/user_project_wrapper/runs/23_01_10_16_10/tmp/synthesis/pg_define.v line 561, syntax error, unexpected $undefined, expecting ';' or ','
I look a the line 561 of the pg_define.v file and found :
defparam spi_reg0.REG_ADDR = 32'sb00000000000000000000000000000000;
In my opinion, this is a faulty translation of the original line in my code :
spi_register #(8,3,0) spi_reg1 (
I rewrite my code as :
spi_register #(4'b1000,4'b0011,4'b0000) spi_reg1 (
and the step 27 is now successful.
I don't know what generate the pg_define.v file, but I think the issue is there and not in the power_utils.py part.
@vijayank88 Do you use parameter and do you pass them in decimal format ?


Minor correction for cross-platform filepath

This test fails on Windows 7 x64 without this small adjustment.
This commit is similar to https://github.com/tarent/loginsrv/commit/8c99657f77f73d2d5b784e673eb4cd6694e3664f.

Coverage remained the same at 90.545% when pulling 400f042e081868e4f4a5b421ef6b99c70fbe64f9 on magikstm:Cross-Platform-Filepath-Caddy-Setup-Test into cd718ade49acce53135107c302de94f75895c91f on tarent:master.


chore: adds deejross as reviewer in OWNERS file

Signed-off-by: Jennifer Power<EMAIL_ADDRESS>Description
This PR updates the OWNERS file to add deejross as a reviewer.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] 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)
[X] This change requires a documentation update

How Has This Been Tested?
N/A
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

/retest

/retest
Retesting. Known bug when processing operator deps appearing in E2E.

/lgtm


[$500] IOU - Original amount is copied when system message shows the latest amount for paid edited request

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: 1.4.20-2
Reproducible in staging?: Y
Reproducible in production?: Y
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: Applause - Internal Team
Slack conversation:
Action Performed:
Precondition: User A and B have no unsettled IOU

[User B] Navigate to a different conversation
[User A] Request money from User B
[User A] Change the requested amount
[User B] Navigate to the main chat with User A
[User B] Click Pay elsewhere without going to IOU report
[User B] Go to IOU report
[User B] Right click on the paid system message > Copy to clipboard
[User B] Paste the content

Expected Result:
The copied amount is the same as the amount in the system message, which is the latest amount
Actual Result:
The copied amount is the original requested amount, while the system message shows the latest amount
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/297d2fa6-bbe1-4976-bfa9-b8f07dbc952c
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01447abde9208e3a7c
Upwork Job ID:<PHONE_NUMBER>524594176
Last Price Increase: 2024-01-01

I have some possible workarounds!
(Client-side)
-> ensure that the action dispatched to update the requested amount is correctly modifying the application's state.
-> ensure that the component displaying the system message is reactively bound to the updated state.
-> ensure that the messages concerning payment updates are correctly processed and that their payload correctly updates
the relevant state. ( Assuming websockets or polling )
(Server-side)
-> When User A updates the requested amount, the back-end should properly handle this update in the database.
-> Ensure that any API endpoint responsible for fetching the payment request data returns the latest state.
-> Verify robust transaction handling in the backend.
-> Use appropriate database locking mechanisms to prevent race conditions, especially if multiple updates can happen in quick succession
In conclusion, by focusing on robust state management, reactive data binding, and ensuring API and database consistency, the issue can be systematically identified and resolved. I am confident it's one of the things listed above!

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~0139f2296710c2db01

I have some possible workarounds! (Posting again in case you didn't see the previous one since you didn't have my contributor details)
(Client-side)
-> ensure that the action dispatched to update the requested amount is correctly modifying the application's state.
-> ensure that the component displaying the system message is reactively bound to the updated state.
-> ensure that the messages concerning payment updates are correctly processed and that their payload correctly updates
the relevant state. ( Assuming websockets or polling )
(Server-side)
-> When User A updates the requested amount, the back-end should properly handle this update in the database.
-> Ensure that any API endpoint responsible for fetching the payment request data returns the latest state.
-> Verify robust transaction handling in the backend.
-> Use appropriate database locking mechanisms to prevent race conditions, especially if multiple updates can happen in quick succession
In conclusion, by focusing on robust state management, reactive data binding, and ensuring API and database consistency, the issue can be systematically identified and resolved. I am confident it's one of the things listed above!

Proposal
Please re-state the problem that we are trying to solve in this issue.
The original amount is copied in the clipboard when the system message shows a different amount.
What is the root cause of that problem?
https://github.com/Expensify/App/blob/fe49e4f7fb9677a5834b53cb776f79fb20f893ce/src/pages/home/report/ContextMenu/BaseReportActionContextMenu.js#L58-L63
In this line, we memoized "reportAction" for the context menu. This could be the root of the problem because we are retaining old information for the context menu.
What changes do you think we should make in order to solve the problem?
We should ensure that the information in the context menu is up-to-date. For example, we can pass only the ID of the report instead of the full report to the context menu and retrieve information only when we press actual actions. Alternatively, we can update our memoized hook dependencies to make sure that the information is up-to-date.
What alternative solutions did you explore? (Optional)

Proposal
Please re-state the problem that we are trying to solve in this issue.
Original amount is copied when system message shows the latest amount for paid edited request
What is the root cause of that problem?
We are getting the amount from originalMessage here
https://github.com/Expensify/App/blob/689c22710e9a8f0457014630c8919ef344bacb0d/src/libs/ReportUtils.ts#L4103-L4104
But It has the non-updated the original value.
What changes do you think we should make in order to solve the problem?

We can get calculate from via getMoneyRequestReimbursableTotal same as we did to display the report action
https://github.com/Expensify/App/blob/689c22710e9a8f0457014630c8919ef344bacb0d/src/libs/ReportUtils.ts#L2008-L2009
but in case the iouReport doesn't exist we will fallback to the originalMessage amount
const {amount: amountFromMessage, currency} = originalMessage.IOUDetails ?? originalMessage;
        const amount = iouReport ? getMoneyRequestReimbursableTotal(iouReport as OnyxEntry<Report>) : amountFromMessage;

What alternative solutions did you explore? (Optional)

@FitseTLT when the account B is about to pay, should we get an error feedback? To try again later, with the updated value?

@eVoloshchak - you have some proposals to review!

@eVoloshchak another bump here.

Looks like this might have been resolved, I'm unable to reproduce the issue
@rrrshtt, @FitseTLT, could you double-check if this is still reproducible for you? Thanks

Yep Solved in BE. Can close it

Issue not reproducible during KI retests. (First week)

Closing per https://github.com/Expensify/App/issues/33816#issuecomment-1896420104 and https://github.com/Expensify/App/issues/33816#issuecomment-1899316832


Adding queries to look for abnormal sch task creation and launch

Required items, please complete
Change(s):

Added three different queries for scheduled task hunting purposes.
Hunting Queries/Microsoft 365 Defender/Persistence/rare_sch_task_launch.yaml
Hunting Queries/Microsoft 365 Defender/Persistence/rare_sch_task_with_activity.yaml
Hunting Queries/Microsoft 365 Defender/Persistence/sch_task_creation.yaml

Reason for Change(s):

Additional threat hunting queries to look for rare scheduled task creation and execution.

Hello @JouniMi, Thanks for raising this PR. This PR will be investigated, and we will update you about the same before 09-Oct-2024. Thanks...!!


Enhance UI: Minimize Scroll Bar for Improved Aesthetics

The current presence of a visible scroll bar is detracting from the overall aesthetics of the website. I suggest minimizing or styling the scroll bar to improve the website's visual appeal. Assigning this task to me for GSSoC contribution would enable me to enhance the UI by addressing this issue effectively

hey @Pratik2401 yeah sure you can work upon it

As I was working on these current issue. I noticed that the send message in get in touch section works even if the information is empty so can I add the issue to exsiting or create a new one


gaugetest ImportError: No module named constants

Hi All,
this time I decided to use the library under xubuntu, but after installation when I was trying to launch gaugetest (the same for other test scripts) the interpreter throw me the error ImportError: No module named constants
Any ideas how to fix this?
This is the log
warning: Debugger speedups using cython not found. Run '"/usr/bin/python" "/home/jh/.p2/pool/plugins/org.python.pydev_<IP_ADDRESS>708151115/pysrc/setup_cython.py" build_ext --inplace' to build.
pydev debugger: starting (pid: 3395)
Traceback (most recent call last):
File "/home/jh/.p2/pool/plugins/org.python.pydev_<IP_ADDRESS>708151115/pysrc/pydevd.py", line 1621, in
main()
File "/home/jh/.p2/pool/plugins/org.python.pydev_<IP_ADDRESS>708151115/pysrc/pydevd.py", line 1615, in main
globals = debugger.run(setup['file'], None, None, is_module)
File "/home/jh/.p2/pool/plugins/org.python.pydev_<IP_ADDRESS>708151115/pysrc/pydevd.py", line 1022, in run
pydev_imports.execfile(file, globals, locals)  # execute the script
File "/home/jh/eclipse-workspace/gauge-master/gaugetest.py", line 13, in
import gauge
File "/home/jh/eclipse-workspace/gauge-master/gauge/init.py", line 15, in
from gauge.constants import CLAMP, ERROR, inf, OK, ONCE
ImportError: No module named constants

Did you compile Cython code?
$ python setup.py build_ext --inplace

Thanks a lot, now it works,
but when I try to run visualizer.py I got next error that is probably not an installation issue,
how can I run this visualizer script?
File "/home/jh/eclipse-workspace/gauge-master/tools/visualizer.py", line 240, in
main(g, 5)
File "/home/jh/eclipse-workspace/gauge-master/tools/visualizer.py", line 226, in main
surf = disp.render(at)
File "/home/jh/eclipse-workspace/gauge-master/tools/visualizer.py", line 79, in render
ratio = self.gauge.current(at) / float(self.gauge.max)
AttributeError: 'gauge.core.Gauge' object has no attribute 'current'

That is a different bug.  Would you open a separate issue?


replacedBy do not relate to modified message but initial message

As an exemple the initial message has been edited 3 times and is displayed 3 times :
this is due to subject being initial message and not previous modification
https://timbl.com/timbl/Public/Archive/solid/solidos/2023/03/09/chat.ttl
:HIYcpVGcR0CSkSMdkXzkntPj7LUU77jhurmjJ3-c-Lw
    dct:created "2023-03-09T18:26:18Z"^^xsd:dateTime;
    dct:isReplacedBy
        :cH0b-n7PH0-iYBMc8FddVQYsBvJ4KY8t7VPfFYdPeDk,
        :Edw9klVd9gvn0ErODpbQL1PEt9RWqKUelFZMRuZQ3yw,
        :pjlb4nvOUO_dNHqA8-gAe-AT9_DUUnIIdzGtE8WWh3g;
    sioc:content
        "Tim Berners-Lee:  [chat-threads] (https://github.com/SolidOS/solid-ui/tree/chat-threads\\) updated for lint, some tests. CI succeeded";
    sioc:id <matrix:///e/HIYcpVGcR0CSkSMdkXzkntPj7LUU77jhurmjJ3-c-Lw>;
    foaf:maker n5:this.

it should be :
:HIYcpVGcR0CSkSMdkXzkntPj7LUU77jhurmjJ3-c-Lw
    dct:isReplacedBy :cH0b-n7PH0-iYBMc8FddVQYsBvJ4KY8t7VPfFYdPeDk,
:cH0b-n7PH0-iYBMc8FddVQYsBvJ4KY8t7VPfFYdPeDk
        dct:isReplacedBy :Edw9klVd9gvn0ErODpbQL1PEt9RWqKUelFZMRuZQ3yw,
:Edw9klVd9gvn0ErODpbQL1PEt9RWqKUelFZMRuZQ3yw,
        dct:isReplacedBy :pjlb4nvOUO_dNHqA8-gAe-AT9_DUUnIIdzGtE8WWh3g.

Hmmm

https://spec.matrix.org/v1.6/client-server-api/#event-replacements
         The original event must not, itself, have a rel_type of m.replace
        (i.e. you cannot edit an edit — though you can send multiple edits for a single original event).

I prefer our way of modeling it to matrix way, preserving the chain of versions.

See also Matrix change MSC2676 Message editing.


Title does not render in VSCode based Jupyter notebooks

Despite setting the title argument, the displayed graphic does not appear to illustrate the title.

VS Code: 1.33.1
Python ext:  2019.3.6558
plotly_express: 0.1.3

Unfortunately VSCode has an older version of plotly.js baked in and so Plotly Express plots won't look right until this is fixed... We're working on getting this resolved but it might be a little while :)
See https://github.com/Microsoft/vscode-python/issues/4953 and the upstream https://github.com/nteract/minimal-plotly/issues/13

See https://github.com/Microsoft/vscode-python/pull/5428 for progress!


Added top rented page

closes #32
https://github.com/user-attachments/assets/7d4cdb56-3c4a-4414-8705-0cdc102d9578

oKAY I'm on it

I have updated the bg but I am not able to grasp the vercel issue. can some mentor help me on this?
@SaranshBangar

@ShivanshPlays Just look into how to use key property in mapping

@SaranshBangar Please check, the error is solved now

@SaranshBangar please check


spacing seems to come from grid that operator is originally made with

Hi!
Not sure when this one crept in, or how we haven't hit it until now.  I'm assuming this isn't expected behaviour...?
def test_spacing_from_new_grid():
    from devito import Grid, Function, Operator, Eq
    grid = Grid(shape=(10, 10), extent=(9, 9))
    u = Function(name='u', grid=grid, space_order=1)

    # A bogus operator that just assigns the x spacing into the array
    # Note, grid.dimensions[0].spacing here is not a number, it's the symbol h_x
    op = Operator(Eq(u, grid.dimensions[0].spacing))

    # Create a new grid with different spacing, and a function defined on it
    grid2 = Grid(shape=(5, 5), extent=(9, 9))
    u2 = Function(name='u', grid=grid2, space_order=1)
    op(u=u2)

    # Fails: the h_x that was passed to the C code is from `grid`, not `grid2`
    assert u2.data[2, 2] == grid2.spacing[0]

Thanks!

@tjb900 fixed by #1523 . Closing the issue. Thanks for the report!


Cannot set up 2FA with Microsoft Validator

Describe the bug
I try to set up Microsoft Validator on PyPI's manage account page option Set up 2FA with an authentication application (TOTP) and it does not work.
Expected behavior
Application set up.
To Reproduce
Following the steps here, I enter the app, click in Add Account, scan the QR Code in the link https://pypi.org/manage/account/totp-provision from PyPI.org, and the app generates a number with 6 characters and a space in the middle. "XXX XXX". I input this on PyPI's website and click Set up application.
Scenario 1 (I put the 6 digit number with the space)
I get the validation message: Please match the requested format.. I assume this behavior is expected, BUT I don't see any expected format in the page.
Scenario 2 (I put in the 6 position number without the space)
I always get Invalid TOTP code. Try again?no matter how many times I try.
My Platform
I am using Linux Ubuntu 19.10 with Firefox 74 64-bit.
Additional context
Don't need.

Hi, I'm guessing you're actually using Microsoft Authenticator instead, correct?
I just tried setting up 2FA with that app on Android and it worked for me.
Can you check that the time on your mobile device is up-to-date?

I assume this behavior is expected, BUT I don't see any expected format in the page.

Good catch, I filed a separate issue for this here: https://github.com/pypa/warehouse/issues/7585

@di Indeed my mobile device is 2 minutes ahead of current time in Google. I will try to put automatic time on my Android settings and try again.

@staticdev Also a great idea, I created https://github.com/pypa/warehouse/issues/7588.
Glad that your issue was resolved!


added fake pressure

Description
Adding fake_pressure

Codecov Report

Merging #134 (02037eb) into master (5536e3a) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 02037eb differs from pull request most recent head 78e9024. Consider uploading reports for the commit 78e9024 to get more accurate results

@@           Coverage Diff           @@
##           master     #134   +/-   ##
=======================================
  Coverage   34.03%   34.03%
=======================================
  Files          67       67
  Lines        3570     3570
=======================================
  Hits         1215     1215
  Misses       2355     2355

Continue to review full report at Codecov.

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

@MiltFra @kshxtij @arjunnaha I think this is done, just one thing to sort: line 110 of main.cpp is causing an error because I've copied Temperature's constructor, but Pressure's constructor is different. Temperature takes a logger, Pressure doesn't (why?) and Pressure takes a temp_pin, I don't know what that is. Can someone who understands this code either finish this off or explain to me what to do? Cheers

@MiltFra @kshxtij @arjunnaha I think this is done, just one thing to sort: line 110 of main.cpp is causing an error because I've copied Temperature's constructor, but Pressure's constructor is different. Temperature takes a logger, Pressure doesn't (why?) and Pressure takes a temp_pin, I don't know what that is. Can someone who understands this code either finish this off or explain to me what to do? Cheers

Pressure uses the new and correct way of handling loggers. Please adapt temperature to fit this style. Should be quick

@MiltFra @kshxtij @arjunnaha I think this is done, just one thing to sort: line 110 of main.cpp is causing an error because I've copied Temperature's constructor, but Pressure's constructor is different. Temperature takes a logger, Pressure doesn't (why?) and Pressure takes a temp_pin, I don't know what that is. Can someone who understands this code either finish this off or explain to me what to do? Cheers

Pressure uses the new and correct way of handling loggers. Please adapt temperature to fit this style. Should be quick

Okay but I still don't know what to pass it for the temp_pin @kshxtij @MiltFra


Support samsung browser

Do we have it in Can I Use?

@ai http://caniuse.com/#compare=samsung+4

Awesome. Browserslist use Can I Use browsers. So we need only add it to docs. Do you want to send PR?

@ai yep :smile:


Update wholesome.js

Anytime 🙃
On Thu, 27 May, 2021, 9:29 pm Srujan Deshpande, @.***>
wrote:

@.**** approved this pull request.
How could we forget that!?!
Thank you
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/srujandeshpande/DrinkWaterBot/pull/46#pullrequestreview-670367950,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASU7PE54BIGTVCD52ITIQNLTPZT5DANCNFSM45USLR4Q
.


Swap out some outdated repo urls in documentation

I noticed that rand and rustfmt are no longer located in rust-lang-nursery.
Rather than updating rand to github.com/rust-random/rand, I've swapped it out for regex in this PR. Some considerations:

Preference for github.com/rust-lang over github.com/rust-random to reduce the perception of unnecessarily endorsing a project that isn't already endorsed by being in Cargo's org.
Ruled out libc because make-your-own-libcpocalypse is not a fun detour for anyone following along and experimenting with git dependencies.
Ruled out everything with a -rs suffix (git2-rs, flate2-rs, backtrace-rs, futures-rs) out of personal distaste.
Went for something that has comparable name recognition to rand, i.e. ruled out hashbrown.

We could alternatively use dummy URLs like github.com/user/example everywhere but I feel that maintaining this every several years is not a big deal and is worth it for the character of the documentation.

r? @Eh2406
(rust-highfive has picked a reviewer for you, use r? to override)

Thanks!
@bors r+

:pushpin: Commit 05f66046608aaf4fe6944ace2e2d3734955ac224 has been approved by ehuss

:hourglass: Testing commit 05f66046608aaf4fe6944ace2e2d3734955ac224 with merge 74d7f07adbbe457a066d541701ac3c724fbf0efe...

:sunny: Test successful - checks-actions
Approved by: ehuss
Pushing 74d7f07adbbe457a066d541701ac3c724fbf0efe to master...


Updates to the annual review governance document and template for LFDT

Replaces PR #289 that was incorrectly handled in GitHub.  Some review of the PR was done in #289 and the comments made there have been addressed in this PR.
Per that PR, changes made include:

Hyperledger references to LFDT
lifecycle "stage" references --> "status" to be consistent with the Lifecycle document
Change some references to Maintainers to TSC, where appropriate.
Added "TBD Link" for a couple of links that are TBD (e.g., Discord channel).  Other links were updated and will likely be correct, but a link checker should be used on the published site to find invalid links.
Fixed Markdown lint warnings
General text cleanup

See https://github.com/LF-Decentralized-Trust/governance/pull/24


fix: broken mount_to_body in CSR mode

I added the following to my Cargo.toml and can confirm that it fixed the blank page render issue for me.
leptos = { git = "https://github.com/leptos-rs/leptos", features = ["csr", "nightly"], branch="fix-csr"}
Thanks for the quick turnaround!


Doc improvements: pyfunction usages and acquiring Python object

I found some details making me a little confused when reading the docs, which I think can be improved.
The first is about the argument parsing for pyfunction macro. I searched for how to add keyword arguments to function and surprised to find that the related article is the arg macro under "Method argument" section. And later I found that there's one small section in function documentation referring to the method argument section. I think it's more intuitive to put the documentation of specifying argument behavior under the page for normal function, and then add a referring link on the method argument section.
The second is about how to acquire the Python generally in PyO3. There are documentations to the usage of py: Python object in the GIL page, but it seems that there's no guide of how to acquire a Python object (e.g. for creating a Python list). In my case, I want a function to return objects of different type from the same function, so I have to use a PyObject as a return type (no sure if this is the designed way), then the code will look like
// just for demonstration
#[pyfunction]
fn some_func(input: u8) -> PyResult<PyObject> {
    match input {
        0 => Ok(py.None()), // how to get a `py`?
        1 => Ok(PyList::new(py)),
        2 => Ok(PyDict::new(py)),
        _ => Err(...)
    }
}

Then after looking around the docs, I found some example directly put py: Python as the first argument of the function, so I tried
#[pyfunction]
fn some_func(py: Python, input: u8) -> PyResult<PyObject> {
    match input {
        0 => Ok(py.None()), // how to get a `py`?
        1 => Ok(PyList::new(py)),
        2 => Ok(PyDict::new(py)),
        _ => Err(...)
    }
}

and it works! I'm not sure if this is the idiomatic way to do this, so it's better to document how to get a py object. Thanks!

The argument parsing section refers to the "method arguments" chapter, but there, "A Python parameter can be specified as part of method signature, in this case the py argument gets injected by the method wrapper, e.g." is basically hidden in "Instance methods", that should be much more prominent.

Thanks for pointing that out! Maybe we can add a section in the GIL page to organize possible ways to acquire the py by referring to here? And the description here is a little be ambiguous, does the py parameter has to be the first one or anywhere is ok?

Thanks, this is excellent feedback :)
It is mentioned in the api documentation.  but not really anywhere in the guide. This is something I initially found annoying as well, I remember having to scour both the docs and the guide to learn how to do things.

Maybe we can add a section in the GIL page to organize possible ways to acquire the py by referring to here?

100% agree 👍. Where did you first start looking when you tried figuring this out?

The first page I looked at is the GIL page because I know the Python is related to the GIL lock. This page is also where the Python object is introduced. It feels right to me to directly describe how to obtain this object after it's first introduction.

And maybe the title of that page can be changed to GIL, mutability and object representations? I was expecting to see the docs of builtin Python types (PyLong, PyDict, etc) in this page at first.


Change testing strategy

test all branches against staging, and test master against both staging and dev

Closing as not relevant change. Though, in order to get the other tests green I'll make all regular builds to work against staging using env vars, temporarily: only for the following PRs: https://github.com/neuromation/cookiecutter-neuro-project/pull/84, https://github.com/neuromation/cookiecutter-neuro-project/pull/97, https://github.com/neuromation/cookiecutter-neuro-project/pull/125, https://github.com/neuromation/cookiecutter-neuro-project/pull/127, https://github.com/neuromation/cookiecutter-neuro-project/pull/129, https://github.com/neuromation/cookiecutter-neuro-project/pull/131


fix architecture selection on Create VM screen

sorry for the messy commit history, relevant change is here: https://github.com/utmapp/UTM/pull/57/commits/21fb3e8e1ef918aed1fc449f75fb1a98eec31131

Can you resubmit with just that commit? You can force push to the branch to resubmit.

Fixed by #61


AutoMapperJs is not defined

Hi,
I've been trying to get automapper-ts to work with my Node Express backend. Everything works fine but I can't seem to implement external Profile files.
As seen in the screenshot, I keep getting AutoMapperJs is not defined. Not sure how to resolve this.
UserProfile.ts

Mapper.ts (I took out config.addProfile(new UserProfile()), if I leave it in, it would give me the error stated above)

export class UserController extends BaseController {

    private _userRepository: IUserRepository = new UserRepository(User);
    private _autoMapper: Mapper = new Mapper();

    @Post('register')
    async register(@Body() newUserParams: NewUserParams): Promise<UserVm> {
        // ...more code here, simplified
        const user = await this._userRepository.create(newUser);
        const result: UserVm = this._autoMapper.mapper.map('IUser', 'UserVm', user.toJSON());
        return <UserVm>result;
    }
}

The above block is where I implemented automapper-ts and it's working beautifully. Now I'm thinking it would be nicer if I can get the Profile to work. Any suggestions?

+1

I have similar issue when trying to create custom converter
import { Page } from 'page/models/page.model';
import { PageVm } from 'page/models/view-models/page-vm.model';
import 'automapper-ts';

export class PageToPageVmConverter extends AutoMapperJs.TypeConverter {
  public convert(resolutionContext: AutoMapperJs.IResolutionContext): PageVm {
    const source = <Page>resolutionContext.sourceValue;
...

it gives the error
export class PageToPageVmConverter extends AutoMapperJs.TypeConverter {
                                           ^
ReferenceError: AutoMapperJs is not defined


SecurityLoginExtension need?

Is the extension needed ?
https://github.com/symbiote/silverstripe-seed/commit/657d0a7de87788c4ef72fbfa9d9704d34fd692ae#diff-01e6ceb290b12ddbb88f00ce3178c4f6
Security.php does
if($this->getRequest()->requestVar('BackURL')
			&& !$this->getLoginMessage()
			&& ($member = Member::currentUser())
			&& $member->exists()
		) {
			return $this->redirectBack();
		}

as part of preLogin, so I'm not sure what SecurityLoginExtension is doing that this doesn't already.

How new is the above code? When I last looked at this, framework ignored the BackURL for an already authenticated user and just responded with the login page. This extension makes sure the redirection actually happens. @nyeholt, can you please confirm whether this remains applicable in SS3 and/or SS4?


Setup Error Reporting

We need to see all errors happening in our app (including client-side) and react on them, for that there are couple of ideas:

so what Atom does, when error happens, give users a button to "Report it" (without logging it into github, but with required fields to fill)
when setup Slack notifications about all errors that are happening
create UI for error message with above reporting options

--

[ ] create tasks for points above


Issue 13700 - Rejected valid conversion from slice to fixed size array

https://issues.dlang.org/show_bug.cgi?id=13700

What I meant is transformations like length(e+a .. e+b) => b-a should be done by the constant folder, too.

What I meant is transformations like length(e+a .. e+b) => b-a should be done by the constant folder, too.

I have an objection for the idea. Currently constant folding never drop any runtime evaluated expression. In other words, e - e would never be folded to 0 even though e is a pure expression. If e is a pure function call and the function throws Error, is the elimination really legitimate? Currently I don't have good answer for the question.
Therefore I don't want to support the folding (e+b) - (e+a) ==> b - a in general case.

Therefore I don't want to support the folding (e+b) - (e+a) ==> b - a in general case.

We're not talking about the general case, we're talking about just the cases already present in this pull - eg when e is a VarExp.
Basically you're implementing some folding for array[lower..upper].length but then only actually doing it when checking for this implicit conversion.  I just want that same folding to be available when the expression appears in user code.

Basically you're implementing some folding for array[lower..upper].length but then only actually doing it when checking for this implicit conversion. I just want that same folding to be available when the expression appears in user code.

Ah! You mean that:

if a slice expression array[lower..upper] is implicitly convertible to T[n], the constant folding array[lower..upper].length to n should also work.
And the code should be moved in ArrayLengthExp::optimize().
Right?

OK, I'll update this PR.

Yes, that sounds right.  I would also expect that since folding array[lower..upper].length is equivalent to folding upper - lower that the former would be implemented in terms of the latter and both would work, unless there is a good reason not to.

Hmm, I found a problematic case to move the length detection into constant folding.
From test/compilable/interpret3.d
size_t badslice1(int[] z)
{
    return z[0 .. 3].length;
}
static assert(!is(typeof(compiles!(badslice1([1,2])))));

After the fix, z[0 .. 3].length will be quickly folded to 3 and the CTFE failure will disappear.
On the other hand, following code still fails to compile.
void main()
{
    int[] arr;
    enum len = arr[0 .. 3].length;  // Error: variable arr cannot be read at compile time
}

Because optimize() is not currently called on the enum initializer.
What do you think about those compiler behaviors?

Hmm.  It does seem a little strange to allow getting the length of an invalid slice, but I can't think of why it would matter.  Any attempt to actually create the slice would still have the bounds checks.  I think it's reasonable to allow that test to pass.
A similar case would be 0 * (1 / 0).  It can't be evaluated, but you can still potentially fold 0 * int to 0 without needing to evaluate it.  This currently gives an error.
Another is &*cast(int*)null, which does compile and run just fine.

On the other hand, following code still fails to compile.
Because optimize() is not currently called on the enum initializer.

I think that should work.

After some experimental, I noticed that we cannot move the length detection code to the constfolding without breaking existing behaviors.
Therefore, I won't update this pull request.

That's the difficulty I have with this optimization, it works in some cases but not in others. It should work in all cases or none. I lean towards all cases :-)

it works in some cases but not in others. It should work in all cases or none. I lean towards all cases :-)

An important point is: the length detection should be conditionally enabled.
void foo(int[] arr, size_t i)     // arr and i are runtime values
{
    enum ctLen = arr[i .. i + 4].length;  // Even if this compile time evaluation is possible,
    auto rtLen = arr[i .. i + 4].length;  // runtime range violation should be checked in this code.
    // In other words, optimizer must not rewrite the declaration to: auto rtLen = 4;
}

To move the length detection code into optimizer, at least it should be separated with a new WANTxxx flag. To make it easier, I'm starting to refactor optimize.c. See and merge #4296.

auto rtLen = arr[i .. i + 4].length;  // runtime range violation should be checked in this code.

Outside of test cases, why would anyone actually care about this being caught at runtime?

@yebblies Because arr[i .. i + 4] should be evaluated before the length calculation. It's in the domain of evaluation order.

You could say the same about &*cast(int*)null, but that doesn't segfault.

"null dereference" is historically unchecked so it's inherited behavior from C. But range check normally throws runtime error in D, and it's considered as a side effect. I'm not sure that the range check elimination by optimizer is legitimate.

I still not sure the length detection should work, because current dmd sometimes remove such the sub-expression, but sometimes not.
import core.stdc.stdio;
struct S {
    static int x;
}
S foo() {
    printf("call foo()\n");
    return S();
}
int[2] bar() {
    printf("call bar()\n");
    return [1, 2];
}
void main() {
    int a = foo().x;        // foo is called, even though it's not necessary for the S.x access
    int b = bar().length;   // baris *not* called.
}

So I don't want to break existing behavior, until the proper behavior will be defined.

@WalterBright @yebblies So, can we merge this PR soon for 2.067 release? The corresponding documentation PR is already merged by @andralex . If this enhancement should not be released in 2.067, we need to revert the document change.

Updated for ddmd build.
What's the state of enhancement 13700? I think the discussion in this PR was just only for the implementation. But the semantic and documentation were already apporoved by @andralex (the documentation PR https://github.com/D-Programming-Language/dlang.org/issues/727 was already merged by him).
Unfortunately I can't implement the proposed code (do the new features in optimize.c). What should I do?

"null dereference" is historically unchecked so it's inherited behavior from C. But range check normally throws runtime error in D, and it's considered as a side effect. I'm not sure that the range check elimination by optimizer is legitimate.

On review I agree with you here, that the range check side effect must be preserved.

int b = bar().length;   // baris *not* called.

I think this is a bug.

It looks like the dlang.org side has been merged a long time ago. We should definitely look at this.

@Geod24 - After D has been merged into GCC. ;-)


uve_compiler branch setup does not work

Hello,
I have been trying to execute following commands as documented in the README.md of uve_compiler branch.

git submodule update --init --recursive
./set_repos.sh
./inject_uve.sh
./configure.sh

However, I get such an error while executing the first command.

root@e45b589594d3:/home/UVE# git submodule update --init --recursive
Cloning into '/home/UVE/ext_modules/riscv-gnu-toolchain/riscv-qemu'...
fatal: unable to connect to github.com:
github.com[0: <IP_ADDRESS>]: errno=Connection timed out
fatal: clone of 'git://github.com/riscv/riscv-qemu.git' into submodule path '/home/UVE/ext_modules/riscv-gnu-toolchain/riscv-qemu' failed
Failed to clone 'riscv-qemu'. Retry scheduled
Cloning into '/home/UVE/ext_modules/riscv-gnu-toolchain/riscv-qemu'...
fatal: unable to connect to github.com:
github.com[0: <IP_ADDRESS>]: errno=Connection timed out
fatal: clone of 'git://github.com/riscv/riscv-qemu.git' into submodule path '/home/UVE/ext_modules/riscv-gnu-toolchain/riscv-qemu' failed
Failed to clone 'riscv-qemu' a second time, aborting
Failed to recurse into submodule path 'ext_modules/riscv-gnu-toolchain'

I am running the commands at ubuntu 20.04 with gem5 dependencies installed.
Also, I cannot find rebuild.sh file. Is it replaced with rebuild_gas.sh?
Thank you in advance!

Where is the llvm compiler for UVE?
As presented in the paper, "Compiling for Vector Extensions with Stream-based Specialization", the toolchain for UVE is implemented based on LLVM

Hello,
I have been trying to execute following commands as documented in the README.md of uve_compiler branch.

git submodule update --init --recursive
./set_repos.sh
./inject_uve.sh
./configure.sh

However, I get such an error while executing the first command.

root@e45b589594d3:/home/UVE# git submodule update --init --recursive
Cloning into '/home/UVE/ext_modules/riscv-gnu-toolchain/riscv-qemu'...
fatal: unable to connect to github.com:
github.com[0: <IP_ADDRESS>]: errno=Connection timed out
fatal: clone of 'git://github.com/riscv/riscv-qemu.git' into submodule path '/home/UVE/ext_modules/riscv-gnu-toolchain/riscv-qemu' failed
Failed to clone 'riscv-qemu'. Retry scheduled
Cloning into '/home/UVE/ext_modules/riscv-gnu-toolchain/riscv-qemu'...
fatal: unable to connect to github.com:
github.com[0: <IP_ADDRESS>]: errno=Connection timed out
fatal: clone of 'git://github.com/riscv/riscv-qemu.git' into submodule path '/home/UVE/ext_modules/riscv-gnu-toolchain/riscv-qemu' failed
Failed to clone 'riscv-qemu' a second time, aborting
Failed to recurse into submodule path 'ext_modules/riscv-gnu-toolchain'

I am running the commands at ubuntu 20.04 with gem5 dependencies installed.

Unfortunately, the qemu repository has been moved and its reference has not been updated. However, that error is not fatal on my end.

Also, I cannot find rebuild.sh file. Is it replaced with rebuild_gas.sh?

The README.md file has been updated and this mistake has been fixed.

Where is the llvm compiler for UVE? As presented in the paper, "Compiling for Vector Extensions with Stream-based Specialization", the toolchain for UVE is implemented based on LLVM

The LLVM-based toolchain will be made publicly available soon.


National chars mangled after reconnected session

Describe the bug
Hi guys.
With PL keyboard with British layout and the same on a remote Ubuntu - when I first log in then I get national chards as expected, but after a disconnection (not a logout) every subsequent session/connection has national chars mangled/gone haywire.
Then in such "mangled" connection I logout on the remote Ubuntu, (re)login a new session and all is (back to) normal.
Reproduces every time. Perhaps other languages too.
Application details
freerdp-2.11.2-1.fc38.x86_64
xrdp           0.9.17-2ubuntu2 amd64
Environment (please complete the following information):

client Fedora
server Ubuntu

sounds more like an issue of xrdp?
if what I got is correct (e.g. you connect with xfreerdp ) and that terminates and you connect again?
or are you talking about e.g. loss of network and auto-reconnect kicking in?

Yes, disconnect in orderly fashion but, I'd presume being disconnect due to broken network would be the same valid case.
In think it would be right to generalize: disconnect(chars mangled) VS log out&in(chars a ok).

@lejeczek well, yes and no.
reconnect does reset a lot of stuff within freerdp that might break.
a orderly disconnect with program termination on the other hand guarantees that at next connect the exact same settings apply for connection establishment (given same command line/rdp file)
so, what is relevant here:

FreeRDP (or any RDP client in that case) does send a keyboard layout to the server initially. (that might be respected or ignored by the server)
FreeRDP does reverse map key symbols to scancodes (this is entirely in the client, no interaction with the server happens for that)
These scancodes are sent to the server
The server then applies the keyboard layout set there (might be the one you sent or an entirely different one)

so, an easy way to test this would be to use a different (not FreeRDP based) client for connection and try this disconnect situation. (mstsc if possible, that is not based on FreeRDP and does normally use scancode input too, as there is another method mostly used by mobile clients, the unicode input)

I'll try, though I have no Windows access.
Side note - c&p: fedora->ubuntu works / fedora<-ubuntu does not

so closing as 'nothing we can fix on our end'


Update indirect dep gopkgs yaml v3 to final

dependabot is bugging us about it:
An issue in the Unmarshal function in Go-Yaml v3 causes the program to
crash when attempting to deserialize invalid input.
Signed-off-by: Chmouel Boudjnah<EMAIL_ADDRESS>

is this something we need to backport?

We don't parse invalid yaml directly 😅


async-ify all functions that return a Promise

Closes #15

@cwill747 Can you force a merge here despite the issues CodeClimate is finding? Can't think of a way to refactor to reduce duplication any more.


[Admin] Publish

Docs Build status updates of commit 66341b9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/concepts/browsers-used-by-office-web-add-ins.md
:white_check_mark:Succeeded
View

docs/design/built-in-button-integration.md
:white_check_mark:Succeeded
View

docs/design/custom-tab-placement.md
:white_check_mark:Succeeded
View

docs/design/keyboard-shortcuts.md
:white_check_mark:Succeeded
View

docs/develop/extended-overrides.md
:white_check_mark:Succeeded
View

docs/develop/localization.md
:white_check_mark:Succeeded
View

docs/outlook/add-and-remove-attachments-to-an-item-in-a-compose-form.md
:white_check_mark:Succeeded
View

docs/outlook/use-rest-api.md
:white_check_mark:Succeeded
View

docs/reference/manifest/extendedoverrides.md
:white_check_mark:Succeeded
View

docs/reference/manifest/override.md
:white_check_mark:Succeeded
View

docs/reference/manifest/token.md
:white_check_mark:Succeeded
View

docs/reference/manifest/tokens.md
:white_check_mark:Succeeded
View

docs/toc.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 2387a1e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/concepts/browsers-used-by-office-web-add-ins.md
:white_check_mark:Succeeded
View

docs/design/built-in-button-integration.md
:white_check_mark:Succeeded
View

docs/design/custom-tab-placement.md
:white_check_mark:Succeeded
View

docs/design/keyboard-shortcuts.md
:white_check_mark:Succeeded
View

docs/develop/extended-overrides.md
:white_check_mark:Succeeded
View

docs/develop/localization.md
:white_check_mark:Succeeded
View

docs/outlook/add-and-remove-attachments-to-an-item-in-a-compose-form.md
:white_check_mark:Succeeded
View

docs/outlook/use-rest-api.md
:white_check_mark:Succeeded
View

docs/reference/manifest/extendedoverrides.md
:white_check_mark:Succeeded
View

docs/reference/manifest/override.md
:white_check_mark:Succeeded
View

docs/reference/manifest/token.md
:white_check_mark:Succeeded
View

docs/reference/manifest/tokens.md
:white_check_mark:Succeeded
View

docs/toc.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 4baa7f6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/concepts/browsers-used-by-office-web-add-ins.md
:white_check_mark:Succeeded
View

docs/design/built-in-button-integration.md
:white_check_mark:Succeeded
View

docs/design/custom-tab-placement.md
:white_check_mark:Succeeded
View

docs/design/keyboard-shortcuts.md
:white_check_mark:Succeeded
View

docs/develop/extended-overrides.md
:white_check_mark:Succeeded
View

docs/develop/localization.md
:white_check_mark:Succeeded
View

docs/outlook/add-and-remove-attachments-to-an-item-in-a-compose-form.md
:white_check_mark:Succeeded
View

docs/outlook/use-rest-api.md
:white_check_mark:Succeeded
View

docs/reference/manifest/extendedoverrides.md
:white_check_mark:Succeeded
View

docs/reference/manifest/override.md
:white_check_mark:Succeeded
View

docs/reference/manifest/token.md
:white_check_mark:Succeeded
View

docs/reference/manifest/tokens.md
:white_check_mark:Succeeded
View

docs/toc.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Thanks, Ricky!


Corrected the dot example

The dot example in this file was a copy from the matlab docs. Corrected to dot(f, g) to f.dot(g), as is in the tests and notebook example.

Thanks a lot! Sorry for taking so long.


handling URI, UUID, other string formats from Swagger

Blocked by DRUID integration

No longer dependent on Swagger, but still can check string format using JSON MAP or other (confirm or deny this)


CREATE CREDENTIAL should mention gMSA

The Documentation on sql-docs/docs/t-sql/statements/create-credential-transact-sql.md
should mention, if (and when yes, how) group Managed Service Accounts can be used here.
In fact all pages, which describe anything "credential" should mention, if groupManagedServiceAccounts are supported.

@VanMSFT Hi Van, Can you please process this Issue 1819.  Thanks.

Hi @glatzert! Thank you for bringing up this issue! We will evaluate adding gMSA documentation, and will track this in our ticketing system: User Story 1487988. Since we're now tracking this internally, I'll proceed with closing out this Issue. However, please feel free to contact us if you have any additional information. Thanks again!

Thanks for taking care of it.


drivers: usb: fix stm32 USB clock configuration logic, enable USB on stm32f769i_disco

See commit messages.
Fixes #52420.

@arbrauns Would you mind rebasing?

LGTM


AssertionError with @-ms-viewport inside @media

If you append the following code to tests/four.css
@media (max-width: 900px) {
  @-ms-viewport {
    width: 320px;
  }
}

it will cause an AssertionError exception to be raised. Seems the viewport selector gets replaced with a temp_key but the assertion for the @media selector doesn't know that.

TROUBLEMAKER
u'a[href^="javascript'
TROUBLEMAKER
u'a[href^="javascript'
Traceback (most recent call last):
File "test.py", line 38, in
run()
File "test.py", line 12, in run
p.process(URL)
File "C:\Python\lib\site-packages\mincss\processor.py", line 144, in process
processed = self._process_content(content, self._bodies)
File "C:\Python\lib\site-packages\mincss\processor.py", line 372, in _process_content
assert old in content, old
AssertionError: @media (min-width: 768px) {
.cx {
font-size: 21px;
}
}

I have to be honest. I'm struggling to find time to work on this. Would you like to attempt it yourself?


PNW Census: Pipeline development

This is the issue to create the PNW Census pipeline.
The pipeline will need to:

[ ] validate a single csv file
[ ] incorporate the new "year_month" element from the filename in exactly the same way "year" is currently handled; for the purpose of differentiating between different returns e.g. PNW Census will allow 2024_Jan, 2024_Feb.... where 903 only has 2024; adding the column "year_month" instead of "year" to all files
[ ] produce a cleanfile per input following a pipeline.json to be defined with client
[ ] hash the values for unique identifiers in the census; and apply the same hash to 903 identifiers such that a single data model can be created at the end
[ ] concatenate cleanfiles at the la level
[ ] create a reports output for each list for the region
[ ] make the usual logs and outputs available in the standard places in the infrastructure

3 days


Added Conan Exiles template

Windows only due to Funcom not making a Linux server

This doesn't look right, there are a few issues:

The RCON password shouldn't be a normal setting. You should use the ${{RemoteAdminPassword}} field and using UseRandomAdminPassword.
If the server accepts RCON for administration, is there not an RCON command that stops the server? If so the exit method should be 'string' with an ExitString of that command.

Fixed the RCON issue, found that Conan actually has a non-writeable console thus ctrl-c is what the server software from Funcom uses.

Fixed the RCON issue, found that Conan actually has a non-writeable console. ctrl-c is what the server software from Funcom uses.

If what you normally do is press CTRL+C, you should probably actually be using OS_CLOSE - can you try that and verify?

Finally looked at this again. Didn't have a windows machine to test on. Tried all exit methods and only 'Kill' works now. Assume funcom update changed something

The setting in the manifest for MaxPlayers should use $MaxUsers as the FieldName to make it map to the MaxUsers setting properly. The settings should also have a DefaultValue rather than using the CommandLineVars setting (which can now be left empty)

Finally looked at this again. Didn't have a windows machine to test on. Tried all exit methods and only 'Kill' works now. Assume funcom update changed something

I have been using this template for a week or so with no issues.  Ability to configure some of the server settings via UI instead of the config files would of course be nice.
Only issue I've had so far is "restart" never seems to work (just stops the server and doesn't start it again) - I have a scheduled task to restart the server daily, and I had to add a second scheduled task about 15 minutes after that to make sure it got started back up.

I'll look into the restart issue today.
I do plan on using AMPs ability to edit config files, just haven't yet


21134 Add entity types to business searches

Issue #: bcgov/entity#21134
Description of changes:

app version = 5.10.21
imported latest Business Lookup shared component
added C, CBEN, CCC and CUL to amalgamation business search

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the bcrs-entities-create-ui license (Apache 2.0).

/gcbrun

In this UI, there are 3 places that Business Search is used:

firm business search (uses updated default types from shared component)
amalgamation business search (added C, CBEN. CCC and CUL)
continuation in extrapro business search (no change needed -- searches for A only)

/gcbrun

Temporary Url for review: https://business-create-dev--pr-702-8de2bfzl.web.app

a bit late to this but looked through the PR. Looks good


Run pip check during test

Closes #5

Checklist

[X] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[X] 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)
[X] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

noarch: python recipes are required to have a lower bound on the python version. Typically this means putting python >=3.6 in both host and run but you should check upstream for the package's Python compatibility.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

noarch: python recipes are required to have a lower bound on the python version. Typically this means putting python >=3.6 in both host and run but you should check upstream for the package's Python compatibility.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Hopefully this can be resolved upstream: https://github.com/man-group/pytest-plugins/pull/174

Hi! This is the friendly automated conda-forge-linting service.
I was trying to look for recipes to lint for you, but it appears we have a merge conflict.
Please try to merge or rebase with the base branch to resolve this conflict.
Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

I got confused...
The corresponding updates are in master but not yet released. Converting to draft in the meantime...

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe/meta.yaml:

No valid build backend found for Python recipe for package pytest-shutil using pip. Python recipes using pip need to explicitly specify a build backend in the host section. If your recipe has built with only pip in the host section in the past, you likely should add setuptools to the host section of your recipe.

@ocefpaf and @jcfr, I think 3½ years is a record PR time for me. I think this is finally good to go! :joy:


Add shim for SIGRTMIN

Fixes https://github.com/rust-lang/miri/issues/2832.

Very clean. Thanks!
@bors r+

:hourglass: Testing commit b9ab7a28712db205f20f5f4dad10c3721f1230b1 with merge 78eafea3c71948e5714e44fdb3ba75d3f97a08f4...


feat: entities command

Adding the entities command.
We can now retrieve up to 4 different entities matched by their IDs. If there is no match on IDs, the respective entity is dropped. Hence the command returns a set join on all them entities 🤓 Obviously, this is a rather expensive operation, but the cost will hopefully improve in the future.

In this last commit, I updated the mechanism to deal with any amount of entity types (up to max(u8), "ought to be enough for anyone" 😉 ) and also already doing the intersection inside find_matching, as suggested by @tarrencev above 🙏

Merging, since it's only the hurl tests that are failing.


How to make the initial state of RNN fixed when generate

I'm doing some analysis work to evaluate and measure the creativity of Magenta drum model. I want a configuration, in which for one given primer sequence, the drum_rnn model will generate a fixed paired output sequence. I must exclude all random factors.
Is there any way to achieve such random-excluding configuration for drum-rnn model ?

If I understand correctly what you're asking, reducing the temperature to a very low number (e.g, 0.0001) will return the argmax. However, I think it's likely you'll find the outputs are quite boring without some randomness.


Clean up message bus listener for GOOGLE_LOGIN_LISTENER_TOPIC in ProjectSelector

ProjectSelector now uses message bus and GOOGLE_LOGIN_LISTENER_TOPIC event to listen for login changes, but does not detach itself or specify a disposable to automatically detach.

Exploring a bit around message bus connections might point to a direction that calling connect() without a disposable uses internal component-level disposable (I guess that's a project). But adding explicit one shouldn't be an issue as well.


python3Packages.mediafire-dl: init at 2023-09-07

Description of changes
This adds a package for mediafire-dl:
https://github.com/Juvenal-Yescas/mediafire-dl
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 258468 run on x86_64-linux 1

4 packages built:

python310Packages.mediafire-dl
python310Packages.mediafire-dl.dist
python311Packages.mediafire-dl
python311Packages.mediafire-dl.dist

@ofborg build python311Packages.mediafire-dl python310Packages.mediafire-dl

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1240

Result of nixpkgs-review pr 258468 run on x86_64-linux 1

4 packages built:

python310Packages.mediafire-dl
python310Packages.mediafire-dl.dist
python311Packages.mediafire-dl
python311Packages.mediafire-dl.dist

[x] package path fits guidelines
[x] package name fits guidelines
[x] package version fits guidelines
[x] package builds on x86_64-linux
[x] executables tested on x86_64-linux
[x] meta.description is set and fits guidelines
[x] meta.license fits upstream license
[x] meta.platforms is set
[x] meta.maintainers is set
[x] build time only dependencies are declared in nativeBuildInputs
[x] source is fetched using the appropriate function
[x] the list of phases is not overridden
[x] when a phase (like installPhase) is overridden it starts with runHook preInstall and ends with runHook postInstall.
[x] patches have a comment describing either the upstream URL or a reason why the patch wasn't upstreamed
[x] patches that are remotely available are fetched rather than vendored

Matches a derivation i have in a private flake, LGTM


Added Support For Linux along with Mac

I have updated the setup.sh file so that it will work for both Linux and Mac.
Issue Addressed: #324
@VictorOdede can you please look into it and let me know if any changes required ..!!

@mmirman can you please look at this ...!!

if this closes and issues you need to write "closes #issuenum"

I am also not qualified to review this

@mr-shitij why is reload_shell commented out?

@mr-shitij why is reload_shell commented out?

Because, we already told the user to reload the shell so no need to reload in code; when the user restarts the shell it will work.

@mr-shitij then we should remove the reload shell function instead of commenting it out

@mr-shitij then we should remove the reload shell function instead of commenting it out

I will do it..!!.

@VictorOdede Can You Please Check Now ..!! I have done with changes ..!!

closes #324


bug: $ionicModal causing scroll up in the previous view

Type: bug
Platform: all
Codepen - http://codepen.io/felquis/pen/wBobPp
How to reproduce

Open the codepen
Scroll to the item 12
Click in the item 12
Click in "Open Modal"
Click in "Hide Modal"
1.[here is the problem] Go to the previous view "with the back button"
The previous scroll is lost, should keep in the item 12

Hope it helps improve Ionic.

I think this issue is similar to the following:
bug: Scroll is lost with collection-repeat #2865

+1

Guys, this issues isn't happening after the changes to fix the issue 2935, I'm pretty sure this was happening because that resize trigger when the modal calls show().

Is possible to know what was the fix exactly ?
Thanks

+1

This issue is still happening, checked the 1.1.1 release bundle, the fix is not in the bundle.


Windows Driver Build Failure on windows-2022 Image

Description
When we move our Windows driver builds to run on the windows-2022 image they fail with the following error:

##[error]C:\Program Files\Microsoft Visual Studio\2022\Preview\MSBuild\Microsoft\VC\v170\Microsoft.CppBuild.targets(436,5): Error MSB8020: The build tools for WindowsKernelModeDriver10.0 (Platform Toolset = 'WindowsKernelModeDriver10.0') cannot be found. To build using the WindowsKernelModeDriver10.0 build tools, please install WindowsKernelModeDriver10.0 build tools.  Alternatively, you may upgrade to the current Visual Studio tools by selecting the Project menu or right-click the solution, and then selecting "Retarget solution".

PR: https://github.com/microsoft/msquic/pull/2025
Pipeline: https://dev.azure.com/ms/msquic/_build/results?buildId=224542&view=logs&j=94396e2f-981e-567a-61e4-6daa52649fcc&t=67d38931-2ffe-5107-01d6-fdb4cd94df8c
Virtual environments affected

[ ] Ubuntu 16.04
[ ] Ubuntu 18.04
[ ] Ubuntu 20.04
[ ] macOS 10.15
[ ] macOS 11
[ ] Windows Server 2016
[ ] Windows Server 2019
[X] Windows Server 2022

Image version and build link
Environment: windows-2022
Version: 20210920.1
Included Software: https://github.com/actions/virtual-environments/blob/win22/20210920.1/images/win/Windows2022-Readme.md
Image Release: https://github.com/actions/virtual-environments/releases/tag/win22%2F20210920.1
Is it regression?
Yes, at least from windows-latest
Expected behavior
The build should succeed, just as it does when using windows-latest.
Actual behavior
The build fails.
Repro steps

Run VSBuild@1 task for a Windows driver on windows-2022 image.

Hello @nibanks. Thank you for your report. We will take a look.

Hello @nibanks!
I guess the problem is missing WDK installation in our win22 image. I believe there had been no proper WDK version released by the time we had been preparing the release. Apart from that we are shipping Visual Studio Code 2022 Preview as a part of the image. There is no compatible public WDK release available yet, which means we can not handle this issue right now.

Hmm... do you have any contacts that might provide some ETA for a WDK that works with Visual Studio 2022?

@nibanks Looks like there is a question about it already. I'll try to contact Luke to see what's going on and figure out if there is expected ETA.

The WDK team is working to ensure WDK compatibility with Visual Studio 2022. Preview versions of the WDK that supports Visual Studio 2022 will be made available through the Windows Insider Program (WIP) as soon as its ready for insiders to review.

@Cymoki thanks a lot for the update!
@nibanks as there is nothing for us to do yet (and there will be nothing to do in the near future as well I bet) I am going to close this issue for now. Feel free to reopen / file a new one as soon as the WDK release is available and we will surely add it to the image.

Ok, but this must block 2022 from becoming latest, as it will break all driver development.

@nibanks 2022 is in beta state so it won't become latest till GA.


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v2.6.24 to before v5.15.7",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=faf0b2e5afe7dae072d2715763c7f992b612b628",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=adf098e2a8a1e1fc075d6a5ba2edd13cf7189082"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "faf0b2e5afe7dae072d2715763c7f992b612b628",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "adf098e2a8a1e1fc075d6a5ba2edd13cf7189082",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "sata_fsl: fix UAF in sata_fsl_port_stop when rmmod sata_fsl\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.15.7 by commit adf098e2a8a1e1fc075d6a5ba2edd13cf7189082, it was introduced in version v2.6.24 by commit faf0b2e5afe7dae072d2715763c7f992b612b628. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2021-1002553


Ubuntu 14.04 vagrant box

In official ubuntu 14.04 vagrant after running "npm install -g redis-commander" redis-commander will crash as soon as it's page is opened, unless run from /usr/local/lib/node_modules/redis-commander directory.
Output:
No config found or was invalid.
Using default configuration.
No Save: false
listening on  <IP_ADDRESS> : 8081
Redis Connection <IP_ADDRESS>:6379 Using Redis DB #0

events.js:72
        throw er; // Unhandled 'error' event
              ^
Error: Cannot find module 'cmdparser' from '/home/vagrant'
    at /usr/local/lib/node_modules/redis-commander/node_modules/browserify-middleware/node_modules/browserify/node_modules/browser-resolve/node_modules/resolve/lib/async.js:46:17
    at process (/usr/local/lib/node_modules/redis-commander/node_modules/browserify-middleware/node_modules/browserify/node_modules/browser-resolve/node_modules/resolve/lib/async.js:173:43)
    at ondir (/usr/local/lib/node_modules/redis-commander/node_modules/browserify-middleware/node_modules/browserify/node_modules/browser-resolve/node_modules/resolve/lib/async.js:188:17)
    at load (/usr/local/lib/node_modules/redis-commander/node_modules/browserify-middleware/node_modules/browserify/node_modules/browser-resolve/node_modules/resolve/lib/async.js:69:43)
    at onex (/usr/local/lib/node_modules/redis-commander/node_modules/browserify-middleware/node_modules/browserify/node_modules/browser-resolve/node_modules/resolve/lib/async.js:92:31)
    at /usr/local/lib/node_modules/redis-commander/node_modules/browserify-middleware/node_modules/browserify/node_modules/browser-resolve/node_modules/resolve/lib/async.js:22:47
    at Object.oncomplete (fs.js:107:15)

Same issue here!
Using Ubuntu 14.04 (3.13.0-43-generic)

Same issue here.

Same here

I have this same issue on CentOS 7. Appears the fix is to run:
npm install -g cmdparser and reload Redis Commander.
Forgotten dependency maybe?

I spoke too soon. It randomly works. But eventually crashes. I don't get it.
Rolled back to 0.2.1 until this is fixed... Sigh.

I've the same issue with debian 8, node v0.12.2 and npm v2.7.4
The command npm install -g cmdparser don't fix the problem.

I've changed in package.json the version of<EMAIL_ADDRESS>to<EMAIL_ADDRESS>and that fixed the problem.

Any chance an npm publish could be done for version 3.0.1 that contains this fix?

Yes, please! In the mean time you can run:
npm install -g git+https://github.com/joeferner/redis-commander.git

@ellisthedev Thanks, this works for me.

@ ellisthedev : It works fine, Thanx

Works for me as well :+1:

Also getting this error on OSX
sowens-MBP:~ sowen$ redis-commander
No config found or was invalid.
Using default configuration.
No Save: false
listening on  <IP_ADDRESS> : 8081
events.js:85
      throw er; // Unhandled 'error' event
            ^
Error: listen EADDRINUSE
    at exports._errnoException (util.js:746:11)
    at HTTPServer.Server._listen2 (net.js:1156:14)
    at listen (net.js:1182:10)
    at net.js:1280:9
    at dns.js:85:18
    at process._tickCallback (node.js:355:11)

This problem should be fixed in current version.
If you think its not feel free to reopen issue with updated informations based on latest version.
The last comment does not show a bug in redis commander but another app already running at the request tcp port 8081. There can only be one app using a port at the same time


Refreshing a website on "Market View" page causes an issue with asset image being fetched with "undefined" in url

Subject of the issue
Refreshing a website on specific market page causes an issue with asset image being fetched with "undefined" in url.
Reproduction steps:

Go to "sBTC Futures" page (https://kwenta-git-feat-futures-kwenta.vercel.app/futures/market/sBTC)
Connect your wallet
Refresh a website => error is shown in browser console

Your environment

Optimism Kovan
Chrome v92.0.4515.159

Screenshots

Can't reproduce that one

@clementbalestrat I can reproduce it every time, all you have to do it to go to url above with open browser console.
Check the video to see it:

I'm using Chrome Browser.


Fix IE Issues

IE has some knows issues with Flexbox, so many of the changes here were in response to those issues. Also, IE doesn't support .include() when fetching an image src, so .indexOf() was used in place.

If you want to use newer spec like Array.includes(), there're lots of polyfills out there and cdn services. We use polyfill.io's cdn for polyfilling on PFE: https://polyfill.io/v2/docs/

I agree with Sarah; for functions, using polyfills is the way to go. 👍
Actually, I think we already have a polyfill? https://github.com/zooniverse/liberating-the-liberator/blob/master/src/index.tpl.html#L7 But I never checked if it was working properly for all the features we need, so polyfill.io is probably the more reliable way to go.
The Flexbox issues, however, are a different story - let me take a look.

Interesting. I'm not familiar with using one of these polyfill services, but the docs look thorough, so I can look into using an .includes() through a polyfill if that's a better option.

polyfill.io's defaults cover a lot, but some have to specified like Array.prototype.includes. Check out what we're doing with PFE: https://github.com/zooniverse/Panoptes-Front-End/blob/master/views/index.ejs#L73

Well if that wasn't just the easiest thing I've ever done.

PR Review

This PR fixes compatibility issues with IE11 and MS Edge.
Accessing the localhost on Win7+IE11 and Win10+Edge, we see that...

The main pages - Home, Classifier, About the Collection - look visually OK on a desktop resolution. There are no visible issues with the layout or the like.
Basic interactions with the Classifier page have also tested OK, notably: adding Annotation.

Overall, this LGTM.
Side discussion:
@srallen You're right, it is the same service (visiting the root Heroku domain shows the Polyfill.io website), and since I was the one who added the link (according to Blame), it means I somehow stumbled across the wrong URL for the correct service. To explain this discrepancy, I'm just going to say... a wizard did it.
Status
LGTM, approving & merging. And remember, if this PR inexplicably causes problems in the future: https://youtu.be/sVgVB3qsySQ?t=16s


snmp metrics improvements/fixes

Add custom MIB loading capabilities to provide better metric names in output.
Rescue cases where a failure to float a value may occur.

@mrichar1
Thanks for this.
Could you also submit a PR against the new repo sensu-plugins-snmp


Imagery Layers Manipulation check boxes get out of sync

For example, in the screenshot below, United States GOES Infrared is checked, but is not visible:

Local link: http://localhost:8080/Apps/Sandcastle/index.html?src=Imagery Layers Manipulation.html&label=Showcases
Perhaps this is a Chrome bug.  Firefox is OK.

The behavior I'm seeing (in Google Chrome stable 39 & 40) is that clicking on one of the lower checkboxes fires the event, updates the map, but does not update the actual graphic of the checkmark itself, so long as the checkbox retains input focus.  The checkmark is then "out of sync" until the input focus moves elsewhere (by clicking in the blank area alongside the toolbar, or in a different window.  Note that clicking the Cesium globe does not change input focus).
Here's a reduced version.  Load the Hello World Sandcastle and switch to the "HTML Body & CSS" tab.  Paste in the following CSS and body:

Loading...

        <input type="checkbox"> The top 3 work.

        <input type="checkbox">

        <input type="checkbox">

        <input type="checkbox"> This one is broken

        <input type="checkbox"> and following ones too.

Run this example and try clicking one of the bottom checkmarks.  Note the graphic does not update immediately, it waits for the input focus to change.
Note that I can't turn off Cesium from this reduced example, so I think WebGL or at least a canvas is required to be behind the buggy checkboxes.  Also, the "translateY" in the CSS plays a key role here, the bug won't manifest without that either.  I also tried replacing the table with a stack of divs, but the problem went away then too.

@mramato is this worth @ggetz looking at during the code sprint?

Sure, should be a good first issue.

I suspect a browser bug here.  Note my reduced example above is just DOM elements & style in front of the Hello World globe.  There is no JS or KO interacting with the offending checkbox elements.  Yet, the canvas and the CSS transforms are still required.  Would be good to reduce-away the Cesium globe and leave just a canvas element (with or without a running WebGL context?), and see if a Chromium bug can be filed.

Er, actually I'm not able to reproduce this at all in Chrome 45.  Maybe it's fixed.

I can't reproduce this either.  I'm going to close the issue.