mGRID

WIP; DO NOT MERGE;
mGRID-related native code

Refer to this link for build results (access rights to CI server needed):
/job/libnd4j_PR/16/

Refer to this link for build results (access rights to CI server needed):
/job/libnd4j_PR/18/

Refer to this link for build results (access rights to CI server needed):
/job/libnd4j_PR/19/

Refer to this link for build results (access rights to CI server needed):
/job/libnd4j_PR/20/

Refer to this link for build results (access rights to CI server needed):
/job/libnd4j_PR/21/

Refer to this link for build results (access rights to CI server needed):
/job/libnd4j_PR/23/

Refer to this link for build results (access rights to CI server needed):
/job/libnd4j_PR/24/

Refer to this link for build results (access rights to CI server needed):
/job/libnd4j_PR/25/

Refer to this link for build results (access rights to CI server needed):
/job/libnd4j_PR/26/

Refer to this link for build results (access rights to CI server needed):
/job/libnd4j_PR/29/

Refer to this link for build results (access rights to CI server needed):
/job/libnd4j_PR/30/

Refer to this link for build results (access rights to CI server needed):
/job/libnd4j_PR/32/

Refer to this link for build results (access rights to CI server needed):
/job/libnd4j_PR/33/

Refer to this link for build results (access rights to CI server needed):
/job/libnd4j_PR/42/

Refer to this link for build results (access rights to CI server needed):
/job/libnd4j_PR/43/

Refer to this link for build results (access rights to CI server needed):
/job/libnd4j_PR/44/

Refer to this link for build results (access rights to CI server needed):
/job/libnd4j_PR/45/

Refer to this link for build results (access rights to CI server needed):
/job/libnd4j_PR/46/

Refer to this link for build results (access rights to CI server needed):
/job/libnd4j_PR/47/

Refer to this link for build results (access rights to CI server needed):
/job/libnd4j_PR/48/

Refer to this link for build results (access rights to CI server needed):
/job/libnd4j_PR/49/

Refer to this link for build results (access rights to CI server needed):
/job/libnd4j_PR/50/

Refer to this link for build results (access rights to CI server needed):
/job/libnd4j_PR/51/

Refer to this link for build results (access rights to CI server needed):
/job/libnd4j_PR/52/


A&R -> 'Old' data isn't showing in export.

When i run and download an export, any of the 'old' data, ie. prior to 2022, isn't actually in the export. it shows years and fiscal years 2017 and future, but there is no data for these line items.

Hi @Christopher-walsh22 - Nicole has highlighted that this ticket is a duplicate. Can you please confirm before I close? Thanks!

it is a duplicate to #237 and #269. can close.


Update /etc/hostname if exists on RHEL

If /etc/hostname exists on RHEL it should be updated, otherwise the
hostname change is considered to be transient by hostnamectl.

👍


no instance of overloaded function "std::swap" matches the specified type

What is the issue you have?
Cannot compile in Visual Studio 2017 after including json.hpp

Please describe the steps to reproduce the issue. Can you provide a small but working code example?
This line - 18474	inline void swap(nlohmann::json& j1, generates the following error
Severity	Code	Description	Project	File	Line	Suppression State
Error (active)	E0493	no instance of overloaded function "std::swap" matches the specified type

Did you use a released version of the library or the version from the develop branch?
Tried 3.01 and 3.12 develop

If you experience a compilation error: can you compile and run the unit tests?

I'm able to cmake the project but unable to compile after including it... I've tried:
ISO C++ latest draft and ISO C++ 17. I'm not able to go to any earlier versions due to dependencies.

Weird. I blew away the project, remade the project and everything worked.
#JustMicrosoftThings ...

I am not using MSVC myself, so I cannot really file a bug report on this.

Here's the deal -- I did a little thinking on this and I think that it is possible that the JSON file being referenced could affect this bug. When I remade my project I rebuild the config file as well and I did it in a slightly different way. It may be something to look at ...


[SCO-1678] Add programable retry mechanism

I have not added any tests as this does not change current logic or behaviour.

I have not added any tests as this does not change current logic or behaviour.

I believe it does make sense to add a test case where this new feature is used. E.g. add a test case where the client should retry even in case of 404.

Accidental PR closing, my bad :) Reopened it right away.


Log error when pixel fails to load

If there is an error while loading the pixel from the server, the current behavior is that snowplow fails silently. This change logs an error.

Thanks @fivetran ! Could you sign our CLA? https://github.com/snowplow/snowplow/wiki/CLA

Scheduled for debug mode as this doesn't make sense for production

I signed the CLA

Confirmed!

Scheduled to 3.0.0

This PR won't quite fit with plans for #381 as we won't be using console.log for the new debug mode. We will emit errors/warnings when the pixel fails to load those. Closing this in favour of the wider debug mode coming in v3.


[P2PW-1172] Settings screen

JIRA
Block by #81

Missing: worked settings: Currency, faceId, appearance, hide zero balances


File generated when disk is full are corrupted

I just noticed we have few empty TS in our graphite server
Actually, files was empty, without headers:
CorruptWhisperFile: Unable to read header (/opt/graphite/storage/whisper/stats/application/com/sensiolabs/insight/http/response/status-code/200.wsp)

I guess (not sure) it happens because the disk was full. So only an empty was created.
How to detect that?
find  /opt/graphite/storage/whisper/ -type f -empty

How to fix that?
find  /opt/graphite/storage/whisper/ -type f -empty -delete

I've run into this same issue. :(  I'm going to delete those corrupt files and start again.
It appears that I've lost a bunch of data, due to this series of events:

The files were created when the disk was full, without headers.
I fixed the disk issue
Since there was no header on many of the files, all further metrics were lost.

I only discovered this because I checked disk usage in graphite/storage and saw that my carbon-cache/carbon-cache-a/console.log_2015_06_25.log file was 7GB.
:sob:

Yes, you should be monitoring your disk for capacity. Any attempted creates will fail and further metrics will be lost. Existing whisper files will resume writes once capacity is available (assuming the metrics are still in the buffer). This is a non-issue and no different from virtually any other service that has to write metadata when writing files to UNIX.


Utilise Form Request Validators (Old 344)

In gitlab by @A-Lawrence on Aug 31, 2015, 01:07
To stop cluttering up our models and controllers with so much junk, let's use them.
For reference: http://laravel.com/docs/5.1/validation#form-request-validation

In gitlab by @A-Lawrence on Dec 20, 2015, 18:51
Removed ~98469 label

In gitlab by @A-Lawrence on Jan 11, 2016, 01:08
This has been started in commit 8700ae7195669d2604b631972f18d8120ae3b248

In gitlab by @NFarrington on Jan 14, 2016, 14:05
@A-Lawrence Can we decide which classes need this change at some point? Otherwise we can close this and simply bear it in mind for future.

Closing as this can be kept in mind during future development.  Specific classes can have the features added as a new issue.


incorrect input for ReluGrad layer

Hi, I have a simple linear model with 3 fc layers and 3 relu layers -
and the corresponding training model is generated by onnxruntime -
As highlighted in the red circle, instead of output of the last Relu layer, the output of last Gemm layer is going to the ReluGrad.
Although, the behavior of the other two intermediate relu/grad layers seems ok, i.e. the Relu Outputs become inputs of ReluGrad,
But the last layer looks buggy.
Code for generating training artifacts:
model = onnx.load("orig.onnx")
artifacts.generate_artifacts(model,
                             requires_grad = ["fc1.weight","fc2.weight","fc3.weight"],
                             loss = artifacts.LossType.CrossEntropyLoss,
                             optimizer = artifacts.OptimType.AdamW,
                             artifact_directory = "./")

Please advise how to fix this.

Closing this issue, as it is answered here.


Declarative check function

It’s kind of annoying to have to import check from hTest every time the default check doesn’t work well. It should be possible to pass an object literal like {deep: true, epsilon: 0.2, looseTypes: true} and have it produce the right checking function behind the scenes.

@DmitrySharabin One thing for the to-dos is to document this functionality.


Images are getting displayed as landscape while taking in portrait mode

I am getting strange result when taking pictures in portrait mode, Picture was getting displayed in landscape. This is happening when we try to take picture while holding camera in the air (Portrait mode). I believe Camera view changes angles which are not respected while taking these type of pictures.This same issue is happening in your demo and playground application as well

List of Devices which were used to produce this issue
Device # 1
Brand: Google
DeviceModel: Nexus 5
Product: hammerhead
Manufacturer: LGE
Android Version: 6.0.1

Device # 2
Brand: samsung
DeviceModel: SM-G530H
Product: fortuna3gxx
Manufacturer: samsung
Android Version: 4.4.4

Device # 3
Brand: samsung
DeviceModel: SM-G530H
Product: fortuna3gxx
Manufacturer: samsung
Android Version: 5.0

Closing as duplicate of issue #15. If you feel that this is a separate issue, please explain in detail why and provide the complete set of steps to reproduce the problem.

Its almost close to issue #15 I am getting EXIF orientation as ExifInterface.ORIENTATION_UNDEFINED (0)


Check that no file contains trailing ws

rustfmt allows trailing spaces in string literals unfortunately.
bors r+
🤖

bors r+

bors r+


Feet / Miles selection box shows up even when editing is disabled.

At first I was really confused why the text "Feet" showed up when I mouse over speed and senses.
But after a bit of digging I found out that if you enable editing this text lets you choose between miles and feet.
It would probably make sense to not have this show up on a locked (un-editable) sheet. Especially since the the text doesn't really do anything then.

Ah, yeah no, that shouldn't show up when not in editing mode. An oversight on my part.

Ah, yeah no, that shouldn't show up when not in editing mode. An oversight on my part.

Fixed via https://github.com/zeel01/MonsterBlocks/commit/6484f5e050bdcdcfd06fc2bcec2feb236efce68c


Migrate Data from webflow base to postgres production

Detailed Description

Production will start working once it has data of all photographers and orgs in production postgres db
Context

It's essential to make the turn to the new platform
Possible Implementation

Extract JSON file from webflow and implement as sequelize seeder.

#38 done


2017/05/23

VSCode Version: Code - Insiders 1.13.0-insider (7403245d3d42edab8d0c728cf56b8a2db097db7f, 2017-05-23T05:14:19.493Z)
OS Version: Windows_NT ia32 6.1.7601
CPUs: AMD A4-5000 APU with Radeon(TM) HD Graphics     (4 x 1497)
Memory (System): 2.72GB (0.50GB free)
Memory (Process): 163.30MB working set (163.31MB peak, 135.60MB private, 39.59MB shared)
Load (avg): 0, 0, 0
VM: 0%
Initial Startup: yes
Screen Reader: no
Empty Workspace: no
Timings:

Component
Task
Time (ms)

main
start => app.isReady
1099

main
app.isReady => window.loadUrl()
330

renderer
window.loadUrl() => begin to require(workbench.main.js)
1616

renderer
require(workbench.main.js)
2717

renderer
create extension host => extensions onReady()
2262

renderer
restore viewlet
3145

renderer
restore editor view state
2574

renderer
overall workbench load
4076

main + renderer
start => extensions ready
8183

main + renderer
start => workbench ready
9958

:warning: Make sure to attach these files: :warning:
-C:\Users\Administrator\c931_main.cpuprofile.txt
-C:\Users\Administrator\c931_renderer.cpuprofile.txt

@hyphhhhyp Please attach those profile files


d3.tsv, d3.csv, d3.dsv return error

d3.tsv, d3.csv, d3.dsv returns error
Steps to reproduce:

Add console.log to drag-zoom-2.component.ts in d3-ng2-demo project:

ngOnInit() {
    let d3 = this.d3;
    console.log('d3.csv:', d3.dsv);
    console.log('d3.csv:', d3.csv);
    console.log('d3.csv:', d3.tsv);
    let d3ParentElement: Selection<HTMLElement, any, null, undefined>;
    //.....

Start local server:

npm run start

Expected result: Application is loaded successfully without errors. Code of dsv,csv,tsv methods displayed.
Actual result: Application is loaded with errors below:
client?93b6:76[default] /Users/vcernomschi/d3-ng2-demo/src/app/d3-demos/drag-zoom-2/drag-zoom-2.component.ts:59:30
Property 'dsv' does not exist on type 'typeof "/Users/vcernomschi/d3-ng2-demo/node_modules/d3-ng2-service/src/bundle-d3"'.errors @ client?93b6:76
client?93b6:76[default] /Users/vcernomschi/d3-ng2-demo/src/app/d3-demos/drag-zoom-2/drag-zoom-2.component.ts:60:30
Property 'csv' does not exist on type 'typeof "/Users/vcernomschi/d3-ng2-demo/node_modules/d3-ng2-service/src/bundle-d3"'.errors @ client?93b6:76
client?93b6:76[default] /Users/vcernomschi/d3-ng2-demo/src/app/d3-demos/drag-zoom-2/drag-zoom-2.component.ts:61:30
Property 'tsv' does not exist on type 'typeof "/Users/vcernomschi/d3-ng2-demo/node_modules/d3-ng2-service/src/bundle-d3"'.

Thanks will investigate.

d3.dsv, d3.csv, and d3.tsv are not valid methods/members of the d3-dsv module. Please refer to the API documentation.
The service/related typescript definitions correctly throws an error in my IDE, when trying to even use them as suggested in your ngOnInit. By implication running them will lead to an error.


[Bug] [admin]  When using a highly available K8s cluster, the jobId for the same task is the same every time it is executed.

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
The logic in Flink's source code: When high availability is used, unless manually configured with PipelineOptionsInternal.PIPELINE_FIXED_JOB_ID, the default is to generate jobId based on the cluster id.

When Dinky submits tasks to Kubernetes, the cluster id is the fixed task name, which results in the same jobId for the same task being executed each time.

When the HistoryServer retrieves results, it does not re-fetch jobIds that already exist, which prevents the retrieval of results for newly submitted tasks.

On the other hand, Dinky's JobRefreshHandler will overwrite the information of tasks that have failed/canceled in the HistoryServer with the same jobId as tasks that are currently running.
What you expected to happen
The jobId should change for each new instance of the submitted task
How to reproduce

Utilize the Flink Kubernetes cluster and configure high availability in the cluster settings, setting jobmanager.archive.fs.dir to the address specified by the HistoryServer.
Launch the HistoryServer to ensure it operates normally.
When submitting tasks to the Kubernetes application, it is observed that the job id for the same task remains fixed upon each submission.

->

The task jobs obtained by the HistoryServer do not update with the completion of each task.
The task statuses in the operations center all change to failed or canceled states in the HistoryServer, triggering alerts (even though the tasks are running normally), and refreshing is ineffective. Disabling the HistoryServer and then refreshing allows the task statuses to return to normal.

Anything else
No response
Version
1.2.0
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I temporarily added a timestamp to the clusterId, and the functionality tested runs without issues. However, I am unsure whether this approach might have any potential drawbacks.


WIP - Hypergrid3

work-in-progress — not to be merged yet
This PR uses queued fetch and resolves a number of issues noted on Monday as well as retiring a lot of code (see pink in diff). More to come!
I'm posting this now just to:

show progress
solicit comments
you can pull it and build & run it if you like

@neilslinger @texodus reminder to add me (@joneit) as a repo "collaborator" so I can among other things specifically request reviews (per this github help page), etc. thx

@texodus the default for the new max column width is 400 (pixels). if you have a different value in mind, set base_grid_properties.columnAutosizingMax in hypergrid.js. You can make the tree column max different from this general default by setting base_grid_properties.treeColumnAutosizingMax.


Should proc macro crates always be compiled in release even if target crate is being compiled in debug?

I have a crate that contains 1800 structs (auto-generated bindings), each with a #[derive(Deserialize, Serialize)] attribute to generate serde impls. A debug build takes ~5m to compile this crate, while a release build takes only ~3m30s because the compiler spends much less time in expanding the serde impls, as verified by -Z time-passes and RUST_LOG=trace.
By having the code generator generate expanded impls of Deserialize and Serialize itself, these times dropped to ~3m10s and ~3m for debug and release respectively.
I know this is 1800-struct crates are few and far between, so this is an extreme edge case. But the small increases in time from each expansion add up to the ~1m30s difference between debug and release.
But from more of a technical standpoint, it seems that proc macro crates have no reason to be compiled with their dev profile since their only purpose is to be loaded by rustc itself and their expansion should always be the same. Does it make sense to always compile proc macro crates with their release profile?

That seems like it would significantly increase overall build times for crates that don't have thousands of structs.

Only "clean" builds though -- I think this is similar (but perhaps a smaller step) to https://github.com/rust-lang/cargo/issues/1359.
Essentially this is a superset of that where crates we build ~once we could also optimize once and result in faster code even in debug mode.
cc @nnethercote -- relates to https://github.com/rust-lang-nursery/rustc-perf/issues/239

@sfackler Yes, it should absolutely be measured across all crates in crates.io or similar. Can crater be used to do this kind of experiment?
@Mark-Simulacrum Optimizing all dependencies will have a noticeable effect at runtime of the target crate, especially in terms of backtraces and general debuggability. I don't think I'd want to compile all my dependencies in release while I'm working on my crate. But proc macros only contribute at compile-time and make no difference at runtime (they're not even compiled for the target arch), so there should be no observable difference apart from compile-time in how they're compiled.

Yes, https://github.com/rust-lang/rust/issues/48683 would cover this. Seems like it would be opt-in rather than automatic, but that's fine.


What is the difference between Streaming Market Data and Streaming Quote Data Websockets ?

What is the difference between Streaming Market Data and Streaming Quote Data Websockets ?


PIE794 fixer removes wrong code and doens't detect one use case

ruff 0.0.218
ruff --select PIE794 foo.py
class foo:
    bar = 1
    bar = 2

→ not detected at all
ruff --select PIE794 --fix foo.py
class foo(object):
    bar = 1
    bar = 2

→ changes into:
class foo(object):
    bar = 1

this is wrong. bar should be 2
it leaves 4 trailing spaces behind (W293)

Thank you! All correct. Will fix.

(Will hopefully get to this later today.)

I think it's not clear that bar = 1 should be removed instead of bar = 2. I know that semantically, it's correct, in that bar = 2 is the value that's resolved when you run the code. But I feel like this check is more likely to occur in situations in which you redefine a field that you already defined? In which case, the second instance would be the "error"?

I fixed the other two issues in #1794. We can keep discussing since the last thing is sort of subjective.

I think it's not clear that bar = 1 should be removed instead of bar = 2. I know that semantically, it's correct, in that bar = 2 is the value that's resolved when you run the code. But I feel like this check is more likely to occur in situations in which you redefine a field that you already defined? In which case, the second instance would be the "error"?

hm, I don't want to risk a behavior change by using the autofixers, so I would change that to the semantically-correct version. If it was wrong before the linter shows you this and you can act upon it.

Yeah that's a reasonable argument. Okay, I'll do another pass on this.


[documentation] Fix Vale errors, warnings and suggestions

Description
PR https://github.com/Quansight/qhub/pull/871 and https://github.com/Quansight/qhub/pull/873 integrated the Vale CI to the GitHub Actions and added the Google developer documentation style guide for language checks.
However, we have a lot of errors, warnings and suggestions generated and we did not get the opportunity to fix all of them. Currently, the CI runs only on modified lines, hence we don't generate a lot of noise but for local usage, we would ideally like it to be fixed.
Recommended steps
To have a stab at fixing these issues, here are some recommended steps:

Install Vale CLI: https://docs.errata.ai/vale/install
Run Vale on the documentation source: vale docs/
Find and fix the issues in batches and open a PR after a significant change.
If you discover an anti-pattern, make a fix on the Vale configuration hosted over tests/vale/styles/

If Vale detects a typo issue, that isn't actually one, add it to the vocab.txt.
If Vale detects grammatical issues, find the relevant YAML that defines the style and add exceptions.

For more background on working with Vale, please refer: https://docs.errata.ai/vale/styles

@HarshCasper - I am interested in this issue, so please assign it to me. I will start working on it starting next week. Thanks!

All Vale errors have been fixed and merged.


[WIP] Constants in invoke

Simple fix to close #447.
Didn't seem that complicated, so I suspect I've overlooked something. I would appreciate it if someone could look this over and point me in the right direction.
If this is somehow correct then there are a few things that should maybe be cleaned up

[ ] Clean up/delete the new example file (since I basically just copied an existing thing)
[ ] ? prohibit constant arguments in the output of an invoke ??? (probably best to do in the parser, but could happen in later logic)

Oh, maybe I was wrong! This looks reasonable. Can you add a test for this under tests/parser/ which will just round trip the example and see if the parser and printer do the right thing?

Looks like the printer gets it slightly wrong:
 54  53│        invoke my_add(
   55  54│          left = y.out,
   56    │-         right = 32'd1
   57    │-         )();
       55│+         right = 32'd1,
       56│+       )();
   58  57│      }
   59  58│    }

It adds a comma which will cause the parser to expect another argument

Ignore the indentation thing, that's my fault

This should get the printer in agreement with the rest of it

Cool, merge it when ready.

Should I remove the file I added to the examples folder? or leave it there with the appropriate .expect?

Oh yeah, remove the examples file.

Done. Looks like some of the tests are failing because they're expecting the old printing with invoke arguments

Aaand that should hopefully make everything and everyone happy


make the lesson flow

current setup worked well for slides, as lesson material it may need some better flow.
Ideas on how to do this are very welcome :)

@samumantha do you have some idea about this now or waiting for input ? what do you think of a lifecycle diagram for software code

Currently no ideas and not working on it. Assignments were leftover from last course. Sorry if this caused confusion

I am closing this because it is not really clear how to solve it but as I am now going through lesson and preparing, I will hopefully arrive at better lesson flow.


Capturing Calibration Constant WM data

Could we please have a look at the best way to allow users to write their standard calibration data to file. At the moment, you get a display of it (via Interpretations - Reference Materials - Calibration Constant WM; see first image below), but there is no way to write that summary data anywhere:

But if we instead go to the next option down the menu (Interpretations - Reference Materials - Weighted Means), which is in general less immediately useful, we have options for writing those means to file:

Is there some way to unify this functionality across both menu options?

This value is written to the RefMat Summary report, which is generated with one click (along with all other reports) using the "Generate all reports" button under reports.  Will this suffice?  In attached file this value is in row 210
1014_2_2022_May_09_09.32_RefMatSummaryReport.csv

Yeah, I didn't know about that! I have avoided using "Generate all reports" because I didn't now which templates would be applied to my RefMat and Unknowns data (and the template needs to be matched to the type of Task you're using, otherwise inapplicable column-headers vanish and never return). Jim has told me that "Generate all reports" uses whatever the "default" template is, so now I know! And I see that report can be generated in isolation via Reports - Summary Expressions and Values - Reference Materials.
Although... I can see that the RefMat summary is not reported correctly, relative to the display data in Calibration Constant WMs:

The field labelled "exterr68" in the CSV is actually the 1sigma internal error of the mean in Squid3 (and expressed as absolute in the CSV, not percentage as in Squid3):

Furthermore, the 1sigma external spot-to-spot error in Squid3 is not actually reported anywhere in the CSV.
Jim @bowring I think we need to have a look at these, with respect to the differences between the "true" RefMat summary-block, and the output of Isoplot's WeightedAverage function. I have described them somewhere in here: https://github.com/CIRDLES/Squid/issues/582#issuecomment-801641139

the 1s external spot to spot is row 21
"4cor_206Pb238U_Ext1SigmaErr_Pct | Note: RM dataset: Calculated external (spot-to-spot) 1sigma uncertainty  (expressed as a percentage) derived from WtdAv_4cor_206Pb238U_CalibConst  subject to a user-defined minimum value."

Ah yes, I see! Thanks


this next to object spread in JSX element incorrectly highlighted

VSCode Version: 1.15.1
OS Version: OS X 10.12.6

Steps to Reproduce:

this next to object spread in JSX element incorrectly highlighted

If you have this directly preceded by an object spread, the TM scope is variable.other.object.tsx:

If you add a space between the spread and this, you get variable.language.this:

Reproduces without extensions: Yes

Please share a stripped down version of the the source code in those examples so I can investigate.
Thanks

import * as React from 'react';

class Test extends React.Component {
  foobar() {
    return {id: 'foobar'};
  }

  render() {
    const incorrect =

;
    const correct =

;

    return null;
  }
}

Ha, it actually manifests in GitHub's highlighting too.

Thanks. I've opened https://github.com/Microsoft/TypeScript-TmLanguage/issues/502 to track this upstream


deploy 'production' branch

Codecov Report

:exclamation: No coverage uploaded for pull request base (dev@628bcc4). Click here to learn what that means.
The diff coverage is n/a.

@@          Coverage Diff           @@
##             dev     #341   +/-   ##
======================================
  Coverage       ?   98.68%
======================================
  Files          ?      152
  Lines          ?     2213
  Branches       ?        0
======================================
  Hits           ?     2184
  Misses         ?       29
  Partials       ?        0

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 628bcc4...add8c3e. Read the comment docs.

Did you finish the server setup?

at first i need a build under /home/deploy/foodsaving-frontend/production to switch the path's in the nginx config.
Through this PR circleci pushes the production branch in to this directory
next steps would be

/etc/nginx/sites-available/foodsaving-world: set directory from master to production
/etc/nginx/sites-available/foodsaving-world-dev: set directory from dev to master
$ nginx -t
$ systemctl reload nginx
$ rm -r /home/deploy/foodsaving-frontend/dev
Github: set defaul branch back to master
Github: protect production

Looks good. You should first remove dev from the deployed branches in circle.yml first and then push it to production and master branches. After that, you should see the content on the server in the respective directories.
After that, you do the nginx changes.


Automated cherry pick of #4126: Add OFSwitch connection check to Agent's liveness probes

Cherry pick of #4126 on release-1.7.
#4126: Add OFSwitch connection check to Agent's liveness probes
For details on the cherry pick process, see the cherry pick requests page.

The unit test case "TestOFBridgeIsConnected" is failed, because ofnet is stuck at waiting for tlvMap reply in the versions earlier than this patch is merged https://github.com/antrea-io/ofnet/pull/31 . To resolve the ut failure, we should modify some functions like this,
func newFakeOFSwitch(stream *util.MessageStream, app ofctrl.AppInterface) *ofctrl.OFSwitch {
	dpid, _ := net.ParseMAC("01:02:03:04:05:06:07:08")
	connCh := make(chan int)
	sw := ofctrl.NewSwitch(stream, dpid, app, connCh, 100)
	return sw
}

func sendTlvMapReply(stream *util.MessageStream) {
	reply := &openflow13.TLVTableReply{
		MaxSpace:  248,
		MaxFields: 62,
		TlvMaps: []*openflow13.TLVTableMap{
			{
				OptClass:  0xffff,
				OptType:   0,
				OptLength: 16,
				Index:     0,
			},
			{
				OptClass:  0xffff,
				OptType:   1,
				OptLength: 16,
				Index:     1,
			},
		},
	}
	tlvReplyMessage := openflow13.NewNXTVendorHeader(openflow13.Type_TlvTableReply)
	tlvReplyMessage.VendorData = reply
	stream.Inbound <- tlvReplyMessage
}

// TestOFBridgeIsConnected verifies it's thread-safe to call OFBridge's IsConnected method.
func TestOFBridgeIsConnected(t *testing.T) {
	stream := util.NewMessageStream(&fakeConn{}, nil)
	b := NewOFBridge("test-br", GetMgmtAddress(ovsconfig.DefaultOVSRunDir, "test-br"))
	var wg sync.WaitGroup
	wg.Add(1)
	go func() {
		defer wg.Done()
		sw := newFakeOFSwitch(stream, b)
		b.SwitchConnected(sw)
	}()
	go func() {
		select {
		case <-time.After(time.Millisecond * 100):
			sendTlvMapReply(stream)
		}
	}()
	wg.Add(1)
	go func() {
		defer wg.Done()
		b.IsConnected()
	}()
	wg.Wait()
}

Thanks @wenyingd for the suggestion, will apply it.

/test-latest-all

/test-latest-e2e


[1.323] Invisible and discarded surfaces block GI

Invisible and discarded surfaces block both realtime and baked GI (screenshot)
Manually disabling visibility of those surfaces before building lighting avoids this bug

I'll look into it, I think I already know what's causing this ..

hmm .. I can't seem to reproduce this?
Are you using Unity version 5.4.0f3? Are all brushes in the same model?

How did you make the surfaces "discarded"? Did you assign the "discarded" material manually?

Yep, 5.4.0f3, all brushes the same model. I selected all the faces manually and discarded them via the surface tools. The invisible brush is just a subtractive one.

I've emailed you an example scene, hopefully it makes it easier to reproduce.

Hmmm ... that seems to be an unity bug (unless I'm misunderstanding something) .. the meshes are set to not cast shadows. I'll workaround this

This is fixed in version 1.323
This version is available on the update site https://prenominal.nl/updates/
I'll close this ticket when it's available from the asset store


Needs correction

[Enter feedback here]
This page needs small correction.
Current line: The regular expression is written so that the bot will recognize the cancel trigger whenever a user responds with help, support, or advice.
It should be:
The regular expression is written so that the bot will recognize the help trigger whenever a user responds with help, support, or advice.
its a typo  i guess

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

ID: c0e6663c-4d8c-5c93-ae66-8b556af3ad4d
Version Independent ID: 5acb7423-f108-1bea-8888-ef620308e3f0
Content: Tutorial: Add help functionality to a bot - Bot Composer
Content Source: composer-docs/tutorial/tutorial-add-help.md
Service: bot-composer
GitHub Login: @WashingtonKayaker
Microsoft Alias: kamrani

We're actively working on this article, and a fix will be out with the next release.


Jenkins : Please add ability to pin the repo to a Git commit

It would be nice to be able to pin the repo to a specific SHA when referring to the pipeline repo

Pinning to a commit is a bad idea IMO. If you need this then it's better to fork the repo and tag it yourself and reference that tag.


[DO NOT MERGE] Add beta label to pages showing new taxonomy navigation

Dependent on #1109 .

We've cherry-picked this commit into https://github.com/alphagov/frontend/pull/1109.


Validation method

I'm working on a .validate() method that would validate a DataFrame for certain characteristics.
Most of the examples are taken from this stackoverflow answer, but there are a few others that I think are important to add (such as regex matching and uniqueness over multiple columns).
We could just include PandasSchema to the project, but it's a bit wordy when creating the schema and it doesn't validate for uniqueness across multiple columns (which is a pretty big deal in my specific use-cases). PandasSchema mentions this and alludes to doing more than just column level validation, such as this comment.
My schema looks like this:
schema={
		('firstname', 'lastname'): 'unique',
		'login': 'len<=8',
		'phonenumber': 'regex:((\(\d{3}\) ?)|(\d{3}-))?\d{3}-\d{4}'
	}

and then somewhere in the chained methods one would call
df.validate(schema)

What's nice about PandasSchema is that it outputs the rows that don't match the validation; mine doesn't at the moment. I'm thinking about throwing exceptions, though, so at the program execution stops if the DataFrame doesn't match validation. One could also have different schemas and pepper them throughout the method chain to ensure the DataFrame is transforming correctly. My main concern is ensuring that a DataFrame is 'correct' enough to save to a database table with a set schema.
The main item I'm currently struggling with are whether I should throw Exceptions, and if not, what should the validation do? I think outputting the values that don't match the schema is reasonable, but I guess I don't see how that helps so much in the method chain.

@szuckerman I just realized I let this ball drop.
We now have a data dictionary thingy, and I think you could add another custom accessor that allows for validation of a dataframe. Or it could just be a function, just as you provided. What do you think?
Btw, I'm so happy you brought up PandasSchema; the code shown on their examples are really awesome!


Receptors features not showing up for right hemisphere hOc1

In the tutorial yesterday we realized that when searching for receptor densities for hOc1 in the current live version of siibra-explorer, only the left hemisphere brings up the „real“ receptor feature. In hOc1 right, it just appears as an ebrains dataset feature, not a as receptor feature with fingerprints etc.
In siibra-python,  I get a receptor feature for both left and right hemispheres, so I guess the bug is in the API or explorer:
import siibra
atlas = siibra.atlases['human‘]
siibra.get_features(atlas.get_region('hoc1 left'),'receptor’)
# -> one feature delivered
siibra.get_features(atlas.get_region('hoc1 right'),'receptor’)
# -> same feature delivered, but not found in siibra-explorer

Related to https://github.com/FZJ-INM1-BDA/siibra-api/issues/41

can no longer reproduce as of 2.7.6.
I suspect this is an issue, which is a combination of

poisoned cache
siibra-api acting up


bundle-update command?

It looks like bundle-install command just replaces registered plugins rather than updating, correct me if I wrong. I found this because of my slow internet connection. After I run bundle-install and re-opened kakoune I found that some plugins were missing. It turns out that git clone command in my case sometimes succeeds and most of the time timeouts. So it would be nice to have real bundle-update command which would git pull each plugin.

Yeah, that would be ideal. However, since kak-bundle allows for installation of plugins with any method, from git to symlinks to even mercurial if desired, there's no guarantee that git pull would be correct with the current design.
A possible solution that I came up with was for git pull to be the default whenever bundle-update is called, and this default could be changed with a bundle-updater command - similar to the existing bundle-install-hook and bundle-cleaner commands.
What do you think?

A possible solution that I came up with was for git pull to be the default whenever bundle-update is called, and this default could be changed with a bundle-updater command - similar to the existing bundle-install-hook and bundle-cleaner commands.

That's really nice idea. git pull is sensible default for update command.

I just noticed that bundle-install has plugin name completion. This is great you did amazing job. This partially solves my problem with slow connection as I can re-install a single plugin, however some plugins may grow in size over time and re-installing even a single one can take considerable amount of time. Average users, me including, tend to update all with a single command and expect no breaking changes, so even if git pull will timeout it won't leave a plugin in incomplete state, contrary to git clone from scratch operation.

Sorry for the long wait - I don't have nearly as much free time nowadays as I'd like.
The bundle-update and bundle-updater commands have been implemented. Let me know if it works for you, because it seems to work on my end.

Yes it works as expected. Thanks for your time and work.


feat: Add rotation to iOS (the sequel)

Continuation of #540. Added a new variable to the InputImageMetadata class, cameraLensDirection, it is used only on iOS to determine if the image is mirrored (for example when using the front camera) or not.

The example must change for this to work, sadly I am unable to run the example app on my computer. On android things can stay the same however on iOS the rotation compensation must be calculated based on the orientation of the device. Like this:
`int getRotationCompensation(int orientation, int sensorOrientation, CameraLensDirection cameraLensDirection) {
if (Platform.isAndroid) {
return cameraLensDirection == CameraLensDirection.front
? (sensorOrientation + orientation) % 360
: (sensorOrientation - orientation + 360) % 360;
}
return (360 - orientation) % 360;
}`. Orientation can be calculated using the native_device_orientation package.
IOS does not require different coordinate translation when the device orientation changes, it can be done with just x * canvasSize.width / imageSize.width; for x, and y * canvasSize.height / imageSize.height; for y. This applies for all orientations.
This works when the camera is locked to portrait up in iOS. If the camera is not being locked, then a rotation of 0 in the InputImage will do.
Finally, the InputImage must be called like this now,
InputImage inputImage = InputImage.fromBytes(             bytes: image.planes[0].bytes,              metadata: InputImageMetadata(               size: Size(image.width.toDouble(), image.height.toDouble()),                rotation: InputImageRotationValue.fromRawValue(rotation)!,                format: InputImageFormatValue.fromRawValue(image.format.raw)!,                bytesPerRow: image.planes[0].bytesPerRow,               cameraLensDirection: InputImageCameraLensDirectionValue.fromRawValue(InputImageCameraLensDirection.back.rawValue)!               )             );. In Android the cameraLensRotation is not used so any value will output the same result. :)

I need you to test with the example app and edit it if needed before merging this.

I need you to test with the example app and edit it if needed before merging this.

It doesn't run in my computer (m1 macbook pro), I tried many times. The alternative is my parents computer which will probably melt if I tried to run it haha.

I will give it a try

Thank you!

I will give it a try

Thank you!

@juanmartin8a : Thanks for using this plugin and your contribution.
I merged your PR into this branch:  feature/rotation
and I was able to compile it and play with it, but unfortunately the imageOrientationFromRotation:cameraLensDirection: method  in this file is breaking things for our example app.
Since we are not setting MLKVisionImage.orientation,  by default the display orientation of the image is .up:

and with your method you are introducing different rotations that are not handled in the example app and as a result it is not detecting any objects/faces/etc because the rotation is messed up.
I believe most of the devs consuming this plugin are doing the image rotation as we have it in the example app and explained in the README:
https://github.com/flutter-ml/google_ml_kit_flutter/tree/feature/rotation/packages/google_mlkit_commons#creating-an-inputimage
If we release your changes we will break a lot of projects once they pull the latest version with your changes.
If you were to fix the example app to work with your changes we will merge them and release them. In the meantime we will keep them for a while in the feature/rotation branch.
Let me know if you are not interested in contributing more so we could archive this branch.
Let me know your comments.

I have an M1 as well, why you can not run it?

It was something about the compile size being too big and it being a common problem in the ARM architecture. I'm going to try again today and will let you know :)

maybe it was this issue:
https://forums.developer.apple.com/forums/thread/738275
got that one when I ran the app in Xcode 15, but I fixed it: https://github.com/flutter-ml/google_ml_kit_flutter/commit/2d3b4f81e69393fda0455995fb7cbd7a6329d371
Pull the latest code from feature/rotation

Hi! Sorry for the late reply. The example works! I pulled from the develop branch and will be working on making the rotation backwards compatible.


Serializing Structs

I have noticed that a large number of classes and structs in SharpDX are unable to be serialized conventionally as they are not marked as serializable and was wondering if there was any reason why they weren't, and if not, if you could do so as a project that I have been working to migrate from XNA to SharpDX requires some data to be serialized.

Because [Serializable] attribute is not available on PCL (and basically on Windows Store Apps) so you should no longer rely on this attribute as it is not portable.


fix: colored header hover highlight

Fix to headers colored by GBB.DB.ColorOnLevel not being affect by the highlight font object.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#8
#7  👈
#4
#6
working-trunk

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @juemrami and the rest of your teammates on  Graphite

Merge activity

May 22, 4:45 AM EDT: @juemrami started a stack merge that includes this pull request via Graphite.


Add pending keyword to tests

This PR should fix the remaining open items in #133.

@ErikSchierboom , did my merging the rename pull first screw up this pull request?

@ricemery Probably. I'll do a rebase tonight.

Sorry about that! This is the second time I have merged your changes out of
order. I'll hang my head in shame for the day!
On Wed, Dec 14, 2016 at 12:22 AM Erik Schierboom<EMAIL_ADDRESS>wrote:

@ricemery https://github.com/ricemery Probably. I'll do a rebase
tonight.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/exercism/xscala/pull/240#issuecomment-266961846, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABeHG2j3qLZWA6O7Iuk-ixg-PZIq5O8Kks5rH5lHgaJpZM4LMFXI
.

@ricemery No problem. I've rebased and update the PR.


react-native link fix for android and ios refs #1192

In version 1.4.0 following issues were introduced after the project restructuring PR merge -> https://github.com/airbnb/react-native-maps/pull/1164. This PR solves all these issues,
Issues

react-native link not working on android.
IOS build fails cause of missing MapKit reference (Was renamed to AirMap
module-resolver js build error.

The path should be also updated in Installation instructions in docs/installation.md.

@misogl path updated in Installation instructions 👍

Updated the path in installation docs #1199

I believe the iOS instructions also need the path update. See: https://github.com/airbnb/react-native-maps/blame/master/docs/installation.md#L55

@joncursi path updated in IOS instructions.

:+1:

@joncursi Will the new code be released any time soon?

@PolGuixe it's not up to me, this is not my repository.

I can confirm that this patch fixes the 'empty red-bordered box' issue for me on Android.

Thanks for the PR! Closing this out since the additions have been merged through other PR's

@christopherdro This is not dup PR! The below diff is probably contained only this PR.
// package.json
  },
      "rnpm": {
      "android": {
-       "sourceDir": "./android"
+       "sourceDir": "./lib/android"
      }
    }
  }

@Nkzn Good catch. Opened up a follow up PR.
https://github.com/airbnb/react-native-maps/pull/1313

@christopherdro thx 🤝


memory error when training ResNet or Inception from scratch (dataset of 200k images and image generator is using 20k at a time)

All,
I've about 200,000 images (each image of size 115x115x3) that 'am trying to classify into 10 classes. All 10 classes are not part of the ImageNet dataset and I think 200k images is a fairly large dataset. Therefore, I've been trying to use ResNet or Inception v4 and train the models from scratch (i did make a few modifications to account for the fact that my images are just 115x115).
Using the suggestions in issue #68 and #2708 I set up my image generator to accept 20,000 images at a time. I made sure that each of the 20,000 images contained all 10 classes and is a representative sample of my entire dataset. To make my life easier I saved the 20,000 images as .npy files. Below is pseudo-code
datagen = ImageDataGenerator(
    featurewise_center=False,
    samplewise_center=False,
    featurewise_std_normalization=False,
    samplewise_std_normalization=False,
    zca_whitening=False,
    rotation_range=45, # randomly rotate images in the range (degrees, 0 to 45)
    width_shift_range=0.1, # randomly shift images horizontally
    height_shift_range=0.1, # randomly shift images vertically
    horizontal_flip=True, # randomly flip images
    vertical_flip=True) # randomly flip images

# read the first 20,000 images
X_firstSet = np.load("0_images.npy")
datagen.fit(X_firstSet)

for e in range(200): # total of 200 iterations
    for ithsample in range(10):
        X = np.load(str(ithsample)+'_images.npy') # reads images that are stored as 0.npy, 1.npy, etc.
        y = np.load(str(ithsample)+'_labels.npy') # reads labels that are stored as 0.npy, 1.npy, etc
        # split into train and validation
        X_train, X_validation, y_train, y_validation = train_test_split(X, y, test_size=0.2, random_state=42, stratify=y)
        model.fit_generator(X_train, y_train,
                           validation_data=[X_validation, y_validation],
                           batch_size = 32, nb_epoch = 1,
                           verbose=2, show_accuracy=True)

I tried running the above on a Quadro P6000 (24 GB of GPU memory) but I get a Memory Error message and code stops running usually after the first epoch. Below is the detailed error message:
Traceback (most recent call last):
File "/home/paperspace/myScripts/12partdiff/train_using_resnet.py", line 118, in <module>
model.fit_generator(train_datagen.flow(X_train,y_train, batch_size=batch_size),
File "/home/paperspace/anaconda3/lib/python3.5/site-packages/keras/preprocessing/image.py",
line 493, in flow
save_format=save_format)
File "/home/paperspace/anaconda3/lib/python3.5/site-packages/keras/preprocessing/image.py",
line 842, in __init__
self.x = np.asarray(x, dtype=K.floatx())
File "/home/paperspace/anaconda3/lib/python3.5/site-packages/numpy/core/numeric.py", line
531, in asarray
return array(a, dtype, copy=False, order=order)
MemoryError

I installed Keras from master and just to be sure I also followed the suggestion in #8249 and set up my code to use GPU before using any keras import functions.
Do I have to upgrade my machine or is something wrong in my code? Odds are that I will end up with a much larger dataset (around 500k images). Any suggestions will be appreciated.
Thanks

Just a heads up, I was having the same error when I tried to normalize my image data (it was read in as RGB int values from 0-255, so I was trying to apply X_train/255.). I was using the stanford cars dataset, which meant around 8000 training images, and 8000 test.
I didn't really want to have to give up data augmentation, but I found a way to get it to run on my laptop without any Memory Errors (Intel® Core™ i9-8950HK CPU @ 2.90GHz × 12 ).

The main solution was that, instead of normalizing the input data directly, I applied a Scaling step right after every batch normalization. This only normalized the data when it was needed, and not the entire 8000 images at once. This is the code I used https://github.com/flyyufelix/DenseNet-Keras/blob/master/custom_layers.py, and a section of my conv_block code looked like this:
X = Conv2D(F1, (1, 1), strides = (s,s), padding = 'valid', name = conv_name_base + '2a', use_bias=False, kernel_initializer = glorot_uniform(seed=0))(X)
X = BatchNormalization(epsilon = eps, axis = 3, name = bn_name_base + '2a')(X)
X = Scale(axis=3, name = scale_name_base + '2a')(X)
X = Activation('relu')(X)

I also altered the main code so that it only read in and saved the test images after training (I also dumped the training image data when I was finished using it). This further cut the memory requirements in half.

I hope this helps!


Build on linux and windows with GitHub Actions

Use GitHub Actions to build on Ubuntu 20.04 and Windows.

now with working artifact upload for Windows and Linux

Looks super useful, thanks!


Timetable icons are Inconsistent in colour and size

The icons that appear when you click on an element in your timetable are are Inconsistent in colour and size.

Definitely can fix!

The colour is fixed, the size is not.

they are "technically" the same size, but because of how the icons are designed they do look different. I've made a bit of a fix ea0a736a4e02d37af9054a20e29d0698534db668 so hopefully that looks better.


Scan commands not implemented?

Hi, first of all thanks for your awesome work on this project.
I noticed that the scan commands are commented out:
https://github.com/seppo0010/rlite/blob/454eaa73427791fe055bec8292bd8d84c70e000c/src/hirlite.c#L4113
Any timeline or plan for implementing these? Are there any alternative commands I should use instead?

Hi, thanks for the compliment.
I did not do the scan commands, because Redis's implementation relies on some internal structure to return the iteration integer, and I don't have anything similar to it.
If I was going to implemented, it would basically return all matched elements every time, and it'd be just a wrapper around KEYS/SMEMBERS/ZRANGE/HGETALL.

Thanks for the explanation!


Send instructions about installation ... to participants

could they do this in the session instead? or does it take too much time?
some won't do it in advance anyway, some will ask whether they need to update their R and how to do it, some will encounter issues and need help, some will install it in their onedrive instead of local, etc.

if instructions need to be sent in advance, I can do that on ~5 september.
just the link to the set up page right?

How do the other workshops handle this?
I think we can send the instructions in advance and plan the workshop with the software already installed. Participants who didn't install it can still do so, as it is a self paced workshop anyways.

I will make them install Git within the workshop given they sometimes struggle. But Aaron, for the Julia workshop, also wanted to send instructions in advance.
We can do that for both your workshops. the other workshops don't need installation.
I'll need the instructions before Sept 5

is that enough for you?
Instructions for all attendees
You need to have admin rights to your computer in order to install software:

All attendees: please install R and RStudio: https://posit.co/download/rstudio-desktop/
Attendees to the Julia workshop: please install Julia, VSCode and Julia extension for VSCode following these instructions: https://formal-methods-mpi.github.io/Workshop.jl/stable/0_preparation/preparation/
Other software (namely Git) will be installed during the workshops.

Maybe add a mention of posit cloud, in case participants don't want or can install anything? A Short Intro can be found in the tip box in this chapter: https://nickhaf.github.io/r_tutorial/qmd/setup/setup.html#installation

All attendees: please install R and RStudio following these instructions: https://nickhaf.github.io/r_tutorial/qmd/setup/setup.html#installation
should cover all then. Though I did tell them upon acceptance that they needed admin rights - for the Git workshop, they'll need it and need RStudio installed... (to match the current material)

Should i remove the mention of posit cloud totally, so they already install R for the git workshop?

can be good to know for them (for later people who will take the workshop independently).
Perhaps I can go back to
All attendees: please install R and RStudio: https://posit.co/download/rstudio-desktop/ in the instructions and you don't change a thing in the tutorial?

Okay, let's do that.

@MalikaIhle : Can I close this?

yes!


Does not work

Copied your launch.json, still get this obscure error when using import

me too

I tried, success.

There is an official recipe for that now which you should go with: https://github.com/Microsoft/vscode-recipes/tree/master/Electron


Add image preview to card changes hook

Enable the cards image (cover?) to be shown when the card is changed,  eg; card moved
currently when a card is changed not only does it show the person who changed it,  but also their profile picture,  which could in turn be changed to the cover-  thanks !

if you mean card cover by the first image attachment put onto a card then maybe? I don't know if attachments are sent within card data when a webhook event is given from trello

Only ADD_ATTACHMENT_TO_CARD returns an image, so it is included in the rewrite here: https://github.com/trello-talk/TrelloBot/commit/42249a1
Regarding displaying the card cover in other card-related events, Trello does not return the cover in it's webhook requests. From #2:

... I would rather not make any additional requests to Trello while parsing the event.


[incubator/honeydipper] deprecating chart

Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] DCO signed
[ ] Chart Version bumped
[ ] Variables are documented in the README.md
[ ] Title of the PR starts with chart name (e.g. [stable/mychartname])

Merging manually. Chart is being deprecated and no longer builds.

https://github.com/helm/charts/issues/24310


fix: Refactors message decoding to abort as soon as a suitable decoder found

Problem
See the description of the original issue: #1369
Solution
Use Promise.any() instead of tracking the successfully decoded message state (didDecodeMsg)
Notes

Resolves #1369

npm run fix ok
npm run check ok
npm run test ok

looks good - thanks for this!
apologies for the delay in review, the team was mostly away last few weeks . pls resolve conflicts and once the CI passes, we'll be happy to merge!

No problem at all! Merged and resolved conflicts!


probot stale

issue #64
I have created a .yml file with suitable configuration to remove stale issues.
I have limited it only to pull request so it wont check for opened issues.
After 15 days of inactivity the pull request becomes stale and after further 7 days it will be closed.

@inishchith what do you think ??

@ahegde3 Can you please keep only the commits which are relevant to this PR?

yeah.I will make the change

I have updated commit history

I have done the asked changes

@ahegde3 Please remove the comments and update the 1st line with the link to usage section of the Probot repository.

Have updated as per request


About visualization

Hi, I used metrics_waymo_test_240421-140450/npz for the visualization, but I got an error. Then I printed the information inside and found that it was inconsistent with the information in demo.npz. May I ask how did you generate the file demo.npz? The saved result after I conducted main.sh reasoning with waymo and nuscenes was inconsistent with the content of demo.npz you provided. Looking forward to your reply!

Datasets have different conventions in terms of data structure. It is hard to provide a demo code that works for all. Feel free to adapt the demo.py or main.py to fit the customized or various datasets.

Ok, I see. Thanks again for your reply


Maintenance

Do you intend to continue maintaining this package?

this package is no longer maintained although feel free to use Masonite ORM which is basically a drop in replacement for Orator


More Views than Proxies

Hey @Airkek,
very great tool. Just a quick question, why do I get more viewers than proxies?
I tested the tool with 10 proxies but ended up with like 80 and more livestreams viewers.
Is there any setting to use 1 proxy for 1 viewer?
Thank you for your time 👍

The software rotates the proxies therefore you get more viewers. Are u using paid proxy or free ones?


New data APIs 2: cached latest-at queries

Static-aware, key-less, component-based, cached latest-at APIs.
The overall structure of this new cache is very similar to what we had before. Effectively it is just an extremely simplified version of re_query_cache.
This introduces a new temporary re_query_cache2 crate, which won't ever be published.
It will replace the existing re_query_cache crate once all the necessary features have been backported.

Fixes #3232
Fixes #4733
Fixes #4734
Part of #3379
Part of #1893

Example:
let caches = re_query_cache2::Caches::new(&store);

// First, get the results for this query.
//
// They might or might not already be cached. We won't know for sure until we try to access
// each individual component's data below.
let results: CachedLatestAtResults = caches.latest_at(
    &store,
    &query,
    &entity_path.into(),
    MyPoints::all_components().iter().cloned(), // no generics!
);

// Then, grab the results for each individual components.
// * `get_required` returns an error if the component batch is missing
// * `get_optional` returns an empty set of results if the component if missing
// * `get` returns an option
//
// At this point we still don't know whether they are cached or not. That's the next step.
let points: &CachedLatestAtComponentResults = results.get_required::<MyPoint>()?;
let colors: &CachedLatestAtComponentResults = results.get_optional::<MyColor>();
let labels: &CachedLatestAtComponentResults = results.get_optional::<MyLabel>();

// Then comes the time to resolve/convert and deserialize the data.
// These steps have to be done together for efficiency reasons.
//
// Both the resolution and deserialization steps might fail, which is why this returns a `Result<Result<T>>`.
// Use `PromiseResult::flatten` to simplify it down to a single result.
//
// A choice now has to be made regarding the nullability of the _component batch's instances_.
// Our IDL doesn't support nullable instances at the moment -- so for the foreseeable future you probably
// shouldn't be using anything but `iter_dense`.
//
// This is the step at which caching comes into play.
//
// If the data has already been accessed with the same nullability characteristics in the
// past, then this will just grab the pre-deserialized, pre-resolved/pre-converted result from
// the cache.
//
// Otherwise, this will trigger a deserialization and cache the result for next time.

let points = match points.iter_dense::<MyPoint>(&mut resolver).flatten() {
    PromiseResult::Pending => {
        // Handle the fact that the data isn't ready appropriately.
        return Ok(());
    }
    PromiseResult::Ready(data) => data,
    PromiseResult::Error(err) => return Err(err.into()),
};

let colors = match colors.iter_dense::<MyColor>(&mut resolver).flatten() {
    PromiseResult::Pending => {
        // Handle the fact that the data isn't ready appropriately.
        return Ok(());
    }
    PromiseResult::Ready(data) => data,
    PromiseResult::Error(err) => return Err(err.into()),
};

let labels = match labels.iter_sparse::<MyLabel>(&mut resolver).flatten() {
    PromiseResult::Pending => {
        // Handle the fact that the data isn't ready appropriately.
        return Ok(());
    }
    PromiseResult::Ready(data) => data,
    PromiseResult::Error(err) => return Err(err.into()),
};

// With the data now fully resolved/converted and deserialized, the joining logic can be
// applied.
//
// In most cases this will be either a clamped zip, or no joining at all.

let color_default_fn = || {
    static DEFAULT: MyColor = MyColor(0xFF00FFFF);
    &DEFAULT
};
let label_default_fn = || None;

let results =
    clamped_zip_1x2(points, colors, color_default_fn, labels, label_default_fn).collect_vec();

Part of a PR series to completely revamp the data APIs in preparation for the removal of instance keys and the introduction of promises:

#5573
#5574
#5581
TODO
TODO
TODO

Builds on top of the static data PR series:

#5534

Checklist

[x] I have read and agree to Contributor Guide and the Code of Conduct
[x] I've included a screenshot or gif (if applicable)
[x] I have tested the web demo (if applicable):

Using newly built examples: app.rerun.io
Using examples from latest main build: app.rerun.io
Using full set of examples from nightly build: app.rerun.io

[x] The PR title and labels are set such as to maximize their usefulness for the next release's CHANGELOG
[x] If applicable, add a new check to the release checklist!

PR Build Summary
Docs preview
Examples preview
Recent benchmark results
Wasm size tracking

Github won't let me link these two issues:

Fixes #4733
Fixes #4734

Reminder to close them manually after...


Feature/model destroy options

:tada: This PR is included in version 2.11.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Feature  add express validation to routes

Pull Request Test Coverage Report for Build 350429622

57 of 61 (93.44%)  changed or added relevant lines in 12 files are covered.
12 unchanged lines in 6 files lost coverage.
Overall coverage decreased (-2.5%) to 51.952%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/routes/fetchUserStrategy.ts
4
8
50.0%

Files with Coverage Reduction
New Missed Lines
%

src/routes/fetchCompetitionSchedule.ts
1
85.71%

src/routes/fetchMatches.ts
1
85.71%

src/routes/fetchMatchData.ts
2
84.21%

src/routes/fetchScouterUIDs.ts
2
84.21%

src/routes/fetchScouterSuggestions.ts
3
76.92%

src/routes/fetchStrategy.ts
3
76.0%

Totals

Change from base Build 350113305:
-2.5%

Covered Lines:
308

Relevant Lines:
551

💛  - Coveralls


Refactor Chunks and ChildValues API to work iteratively

Change exposes WalkRefs and WalkValues which has optional shallow or deep
traversal. This change also removes broken concurrency feature and dead
code from the walk package

@rafael-atticlabs  PTAL wasn't sure if it would be best that I merged upstream changes prior to the PR or to do the PR after merging the upstream changes, wanted it to be fairly easy to track whats going on.

@rafael-atticlabs  PTAL merged changes from upstream back in

Current coverage is 88.13% (diff: 82.01%)

Merging #2599 into master will increase coverage by 0.17%

@@             master      #2599   diff @@
==========================================
  Files           236        236
  Lines         17735      17660    -75
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits          15599      15564    -35
+ Misses         1897       1856    -41
- Partials        239        240     +1

Diff Coverage
File Path

••         25%
go/types/type.go

••••       40%
go/types/struct.go

••••       40%
go/types/map.go

•••••      50%
go/types/number.go

•••••      50%
go/types/list.go

•••••      50%
go/types/string.go

•••••      50%
go/types/set.go

•••••      50%
go/types/bool.go

•••••      57%
go/dataset/dataset.go

••••••     66%
go/types/blob.go

Review all 21 files changed

Powered by Codecov. Last update bc40db4...02a5eed

mostly lg. back to you.

@rafael-atticlabs  missing previous comments. That said I adjusted the code according to your comments IRL. PTAL

@rafael-atticlabs  I ended up needing to change Walk API to support skipping, because photo-index demonstrated its utility. I also re-enabled the skip tests, which test skipping. PTAL

lgtm w/ comment fix & changes discussed in meatspace.
thanks for sticking with this! this is really nice now.

@rafael-atticlabs  thanks for the PR next stop js codebase!


Update payload

https://ctftime.org/writeup/25809

https://www.onsecurity.io/blog/server-side-template-injection-with-jinja2/

group_concat trick to extracts all tables https://www.youtube.com/watch?v=YA8wcG1GL8A

https://github.com/ryan-cd/ctf/tree/master/2021/0x41414141/shjail

command injection bracket glob
summary common Linux commands with usage

https://ctftime.org/task/14603

python command injection


TPM module renders

Are you going to move those commits to another branch/PR as they are unverified?

I have moved all unverified commits to the new branch - TPMpinput_v2

create a new clean branch and cherry-pick commits to verify them or commit your changes in a different way
keeping in mind the verification status of your commits

This commits are corrected and moved to the new MR.


Added support for objects with cyclic references

Removed Context from DbStoragePlainFile to decrease coupling storage with android classes. Storage needed only path to directory with db files.
Added support for objects with cyclic references and added tests for it.

I disabled cyclic dependencies support intentionally:
https://github.com/pilgr/Paper/blob/development/paperdb/src/main/java/io/paperdb/DbStoragePlainFile.java#L51
Kryo docs about this
https://github.com/EsotericSoftware/kryo#references
It's disabled because of an issue in CompatibleFieldSerializer which is used by default to bring compatibility support for changed classes. More info is here https://github.com/EsotericSoftware/kryo/blob/master/src/com/esotericsoftware/kryo/serializers/CompatibleFieldSerializer.java#L39
I had significant amount of crashes in my app because of that. So I'd prefer to keep stable compatibility support against cyclic dependencies support. I guess cyclic dependencies should never happen in proper data model architecture.
P.S. Removing context from DbStoragePlainFile is nice idea! But please make changes in the development branch.

Let's imagine such case in development branch
class Person {
...
private List groups;
...
}
public class Group {
...
private List contacts;
...
}
And implement bidirectional references between contacts and groups of contacts.
person.getGroups().add(group);
person1.getGroups().add(group);
person1.getGroups().add(group1);
person2.getGroups().add(group1);
group.getContacts().add(person);
group.getContacts().add(person1);
group1.getContacts().add(person1);
group1.getContacts().add(person2);
Saving any person object will lead to stack overflow.
In my opinion cycle references should not be cause for users not use this database.
With proposed changes saving objects with cycle references would not be responsibility for serialization algorithm. References are only stored in
Map<String,String> cycleReferences = new HashMap<>();
which is not issue for kryo to serializer.
What if we can propose this functionality by calling:
Paper.book().unsafe().write(key, value); // where value - object with cycle references.

What if you miss one of those lines?
person1.getGroups().add(group);
group.getContacts().add(person);

You come up to data inconsistency.
Even in your case, you save change
public class Group {
...
private transient List contacts;
...
}
save list of all Person and then, on read, fill in each Group instance with available Person.
But anyway such data structure looks weird.
Paper been designed to be as simple as possible and I want to keep it simple.

Maybe put a note in README.md to say circular references not supported ? I understand the design decision but It took me some time to reach this page.

@martinbonnin makes sense, thanks!


bin/obj Folders

Remove them and add to .gitignore

manually deleted them from repo (https://github.com/MSiccDev/LinkTools/commit/80fee9988413b64ab004097ca244c09406925fb8, https://github.com/MSiccDev/LinkTools/commit/2d423a826927088f5155d55dda47491d66fe56cf, https://github.com/MSiccDev/LinkTools/commit/e06e86c387648a8d8057a13331c8311870835de0, https://github.com/MSiccDev/LinkTools/commit/87b4decd0787b931ab18717c055c65ca46a93084)


make GAF of GO annotations in contig files

As described in email - create a GAF from the GO annotations in the *.contig files so we don't have to carry on editing in Artemis to use evidence not available in Canto. I'm rashly assuming it can go in pombe-embl/external_data/external-go-data/, to be included in the main GO file generation.
When it's done, ping me and I'll delete all the annotations from the contig files and we can enjoy the glorious GAF-based future.

I've had an attempt at making a GAF file from the annotation in the contigs. It looks OK at first glance but I'm still checking.
pombase-build-contig-files-only.gaf.txt

I'm now doing a test load of the contigs into a fresh Chado database but with the GO loading disabled.  Then I'll load the GAF file I created earlier to check that we have all the annotations in the GAF file.

I don't know if this indicates some duplication but:
cat pombase-build-contig-files-only.gaf.txt |wc
21455
grep "GOid=GO:" *.contig |wc
16093
This could be explained if you expanded the 'piped' extensions into individual annotations.
But definitely not to few!

I'm now doing a test load of the contigs into a fresh Chado database but with the GO loading disabled.

clever!

It looks good to me!

This could be explained if you expanded the 'piped' extensions into individual annotations.

Yep, that's happening.  The extensions are expanded when loading into Chado.

cat pombase-build-contig-files-only.gaf.txt |wc
21455

That count will include ND lines which aren't helpful in this case.

I'm now doing a test load ...

Nice. Assuming it passes your test-load checks, good to ... er, GO ;)

I'm now doing a test load of the contigs into a fresh Chado database but with the GO loading disabled. Then I'll load the GAF file I created earlier to check that we have all the annotations in the GAF file.

That more or less worked.
A couple of annotations didn't end up in the GAF file because they on pseudogenes: SPCC553.05c/wtf6 and SPCC285.06c/wtf17
Now I know that I'll tweak the GAF writer to include them next time I export.
The other issue is that there are some multi-transcript genes where there are annotations on more than one transcript (SPAC27F1.09c/prp10).  The GAF writer ignores transcripts and so in the GAF file it looks like all the annotation is attached to the gene, and is repeated:
PomBase    SPAC27F1.09c    prp10           GO:0005634      PMID:10759889   HDA             C       U2 snRNP-associated protein SF3B1       sap155|SF3b155  protein taxon:4896      20050518      PomBase
PomBase    SPAC27F1.09c    prp10           GO:0005634      PMID:10759889   HDA             C       U2 snRNP-associated protein SF3B1       sap155|SF3b155  protein taxon:4896      20050518      PomBase
PomBase    SPAC27F1.09c    prp10           GO:0005634      PMID:10759889   HDA             C       U2 snRNP-associated protein SF3B1       sap155|SF3b155  protein taxon:4896      20050518      PomBase

When we read that GAF file all the annotation gets attached to the first transcript (SPAC27F1.09c.1) because the GAF reader doesn't know any better.
There's an open related issue: https://github.com/pombase/pombase-chado/issues/830
Is there preferred way to specify a transcript ID in a GAF file?
A temporary fix is to leave those multi-transcript annotations in the .contig files and move the rest to a GAF fie.

Is there preferred way to specify a transcript ID in a GAF file?

Spec is here: http://geneontology.org/docs/go-annotation-file-gaf-format-2.2/
... but I'm not sure it's very helpful. It's hinting that transcript isoforms can be distinguished using column 17, but it doesn't directly address the case where annotations refer to the protein products of different transcripts from one gene (i.e. our use case). Most of what it has about transcripts seems to be for functional ncRNAs; otherwise it seems to assume we'd have separate protein identifiers for transcript isoform products.
We'll probably have to ask via a ticket or email. Or we could see what happens if we stick one of our transcript isoform IDs in column 17 and pass it off as also an identifier for the protein it encodes.
Tangent: the GYAD/GPI spec is similarly opaque.

A temporary fix is to leave those multi-transcript annotations in the .contig files and move the rest to a GAF file.

For the small number we're dealing with, that would be OK with me. Life will be easiest if you can give me a list, although in a pinch I could probably remember which genes are affected.

I would go ahead. we can open a ticket about the need to fix the column 17 ones.
Let me know when I can edit... I already have a list of fixes, additions, removals etc  (not log file ones)

That all sounds awesome :D

I'm wondering how come there is GO annotation in the contigs associated with multi-gene transcripts. I presumed this would all be from publications and so would be in Canto.

Probably because the annotation can't be attached to a transcript in Canto, only to a gene.  Some of the annotation in the contigs is transcript specific.

The load has finished.  There are quite a few things to fix:

I forgot to remove the ND lines from legacy_go_annotations_from_contigs.txt so they got loaded - whoops
the qualifiers column wasn't loaded from legacy_go_annotations_from_contigs.txt
the column 17 values aren't loaded

I should be able to fix those problems before the next load (on Friday night) so I won't undo all the changes.
I'm about to remove the ND lines and then I'll work on the other problems.

Other problems:

I forgot to include the annotation from pseudogenes in the GAF file - I'm fixing that now
I'm not sure qualifiers like this are being handled correctly: "from=GO:0140053|GO:0005739"

but maybe it's better?  (let's chat on the next call):

http://dev.pombase.kmr.nz/gene/SPAP8A3.11c
https://www.pombase.org/gene/SPAP8A3.11c

Is it worth fixing from entries in the contig files?

answer from meeting: nah, it turned out correct in the GAF so it's all sorted. Yay!

I opened 3 new tickets
https://github.com/pombase/curation/issues/3137
https://github.com/pombase/curation/issues/3136
https://github.com/pombase/canto/issues/2495
to continue when this one closes tomorrow (hopefully)


Don't work

hey i installed the mod with Thunderstore but nothing happen when i press "P" ingame.

hey i installed the mod with Thunderstore but nothing happen when i press "P" ingame.

The mod is depreceted and won't be updated since it has been substituted with this one which uses a mod loader made for SonsOfTheForest called RedLoader.


noVNC connection timed out

ISSUE TYPE

Bug Report

COMPONENT NAME

noVNC

CLOUDSTACK VERSION

cloudstack-agent <IP_ADDRESS>
cloudstack-management <IP_ADDRESS>
cloudstack-usage <IP_ADDRESS>
cloudstack-common <IP_ADDRESS>
CPVM systemvmtemplate-4.19.0-kvm
CONFIGURATION

N/A
OS / ENVIRONMENT

Ubuntu 22.04.4 LTS
SUMMARY

I encountered the same issue as #8303. I manually replaced ${VNCPORT} with 8443 in /etc/iptables/rules.v4, and then re-executed /opt/cloud/bin/setup/postinit.sh
After the modification, the VNC console page can be opened, but it is always connected
I watched a lot of tutorials, but nothing worked
STEPS TO REPRODUCE

sed -i 's/${VNCPORT}/8443/g' /etc/iptables/rules.v4
sh /opt/cloud/bin/setup/postinit.sh
tail -f /var/log/cloud/cloud.out

EXPECTED RESULTS

noVNC can be loaded normally

ACTUAL RESULTS

The noVNC console kept connecting until it timed out

This is the log

/var/log/cloud/cloud.out
2024-05-05 10:05:54,362 INFO  [cloud.consoleproxy.ConsoleProxyResourceHandler] (Thread-70:null) Get resource request for /resource/noVNC/vnc.html
2024-05-05 10:05:54,367 INFO  [cloud.consoleproxy.ConsoleProxyResourceHandler] (Thread-70:null) Sent file /resource/noVNC/vnc.html with content type text/html
2024-05-05 10:05:56,402 INFO  [cloud.consoleproxy.ConsoleProxyNoVNCHandler] (qtp18310625-36:null) Get websocket connection request from remote IP : <IP_ADDRESS>
2024-05-05 10:05:56,481 INFO  [cloud.consoleproxy.ConsoleProxyNoVncClient] (Thread-72:null) Connect to VNC server directly. host: 38.xx.221.14, port: 5901
2024-05-05 10:05:56,505 INFO  [consoleproxy.vnc.NoVncClient] (Thread-72:null) Connecting to VNC server 38.xx.221.14:5901 ...
2024-05-05 10:09:01,071 INFO  [cloud.consoleproxy.ConsoleProxyGCThread] (Console Proxy GC Thread:null) Dropping com.cloud.consoleproxy.ConsoleProxyNoVncClient@4b621b07 which has not been used for 184 seconds
2024-05-05 10:10:56,482 ERROR [cloud.consoleproxy.ConsoleProxyNoVNCHandler] (Connector-Scheduler-371c7f21-1:null) Error on websocket
org.eclipse.jetty.websocket.api.CloseException: java.util.concurrent.TimeoutException: Idle timeout expired: 300001/300000 ms
        at org.eclipse.jetty.websocket.common.io.AbstractWebSocketConnection.onReadTimeout(AbstractWebSocketConnection.java:564)
        at org.eclipse.jetty.io.AbstractConnection.onFillInterestedFailed(AbstractConnection.java:172)
        at org.eclipse.jetty.websocket.common.io.AbstractWebSocketConnection.onFillInterestedFailed(AbstractWebSocketConnection.java:539)
        at org.eclipse.jetty.io.AbstractConnection$ReadCallback.failed(AbstractConnection.java:317)
        at org.eclipse.jetty.io.FillInterest.onFail(FillInterest.java:140)
        at org.eclipse.jetty.io.ssl.SslConnection$DecryptedEndPoint.onFillableFail(SslConnection.java:591)
        at org.eclipse.jetty.io.ssl.SslConnection.onFillInterestedFailed(SslConnection.java:419)
        at org.eclipse.jetty.io.ssl.SslConnection$2.failed(SslConnection.java:170)
        at org.eclipse.jetty.io.FillInterest.onFail(FillInterest.java:140)
        at org.eclipse.jetty.io.AbstractEndPoint.onIdleExpired(AbstractEndPoint.java:407)
        at org.eclipse.jetty.io.IdleTimeout.checkIdleTimeout(IdleTimeout.java:171)
        at org.eclipse.jetty.io.IdleTimeout.idleCheck(IdleTimeout.java:113)
        at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515)
        at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
        at java.base/java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:304)
        at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
        at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
        at java.base/java.lang.Thread.run(Thread.java:829)
Caused by: java.util.concurrent.TimeoutException: Idle timeout expired: 300001/300000 ms
        ... 8 more

It looks the kvm host uses public ip
Can you check if 38.xx.221.14:5901 is open from

the host itself
the CPVM
any other servers

It looks the kvm host uses public ip
Can you check if 38.xx.221.14:5901 is open from

the host itself
the CPVM
any other servers

CPVM and host are two public ip
host ip: 38.xx.221.14
CPVM ip: 38.xx.221.245
host itself:
Proto Recv-Q Send-Q Local Address           Foreign Address         State       PID/Program name
tcp        0      0 38.xx.221.14:5900       <IP_ADDRESS>:*               LISTEN      2785/qemu-system-x8
tcp        0      0 38.xx.221.14:5901       <IP_ADDRESS>:*               LISTEN      2783/qemu-system-x8

CPVM:
Proto Recv-Q Send-Q Local Address           Foreign Address         State       PID/Program name
tcp        0      0 <IP_ADDRESS>:8443            <IP_ADDRESS>:*               LISTEN      2615/java
tcp        0      0 38.xx.221.245:8443      <IP_ADDRESS>:33727    ESTABLISHED 2615/java
tcp        0      0 <IP_ADDRESS>:43112      38.xx.221.14:5901       ESTABLISHED 2615/java

38.xx.221.14:5901

@z-mio
can you run telnet 38.xx.221.14 5901 in the CPVM ?

@z-mio can you run telnet 38.xx.221.14 5901 in the CPVM ?

@weizhouapache

然后分享登录 /var/log/cloud.log打印 CPVM ?

@weizhouapache
ok, here's the full log
cloud.log

normally kvm host uses private IP
When open a vm console, ACS adds a route in the CPVM for kvm host
2024-05-06 10:42:13,131 DEBUG [resource.consoleproxy.ConsoleProxyResource] (Thread-14:null) Successfully executed process [53393] for command [/bin/bash -c ip route add 38.xx.221.14 via <IP_ADDRESS> ].
2024-05-06 10:42:13,133 DEBUG [resource.consoleproxy.ConsoleProxyResource] (Thread-14:null) addRouteToInternalIp: added route to internal ip=38.xx.221.14 via <IP_ADDRESS>

unfortunately it causes your issue with vm console, as your kvm host uses a public IP.

Not a typical setup
route may not be added if the kvm host is in the same subnet as public/private IP of CPVM.
marked this as a bug

@weizhouapache
Thank you very much.
I changed the public IP (38.xx.221.14) of the host to a private IP (<IP_ADDRESS>), and I was able to connect to the VPC.
(This issue had been bothering me for 3 days 😭

@weizhouapache
Thank you very much.
I changed the public IP (38.xx.221.14) of the host to a private IP (<IP_ADDRESS>), and I was able to connect to the VPC.
(This issue had been bothering me for 3 days 😭

@z-mio
thanks for the update
I will close this issue

@weizhouapache
Thank you very much.
I changed the public IP (38.xx.221.14) of the host to a private IP (<IP_ADDRESS>), and I was able to connect to the VPC.
(This issue had been bothering me for 3 days 😭

@z-mio , great that you have figured it now. If you have a location in the documentation that could be improved , feel free to submit a PR or point to it.


tiny-cuda-nn install fails, different error than in the notes

When I run "pip install ninja git+https://github.com/NVlabs/tiny-cuda-nn/#subdirectory=bindings/torch" I get this error.
Collecting git+https://github.com/NVlabs/tiny-cuda-nn/#subdirectory=bindings/torch
Cloning https://github.com/NVlabs/tiny-cuda-nn/ to c:\users\andy\appdata\local\temp\pip-req-build-3eo2lr_w
Running command git clone --filter=blob:none --quiet https://github.com/NVlabs/tiny-cuda-nn/ 'C:\Users\Andy\AppData\Local\Temp\pip-req-build-3eo2lr_w'
Resolved https://github.com/NVlabs/tiny-cuda-nn/ to commit bd1f7272eda2bb6047c3dcfb253086dbf41bc593
Running command git submodule update --init --recursive -q
Preparing metadata (setup.py) ... done
Collecting ninja
Obtaining dependency information for ninja from https://files.pythonhosted.org/packages/b6/2f/a3bc50fa63fc4fe9348e15b53dc8c87febfd4e0c660fcf250c4b19a3aa3b/ninja-<IP_ADDRESS>-py2.py3-none-win_amd64.whl.metadata
Using cached ninja-<IP_ADDRESS>-py2.py3-none-win_amd64.whl.metadata (5.4 kB)
Using cached ninja-<IP_ADDRESS>-py2.py3-none-win_amd64.whl (312 kB)
Building wheels for collected packages: tinycudann
Building wheel for tinycudann (setup.py) ... error
error: subprocess-exited-with-error
× python setup.py bdist_wheel did not run successfully.
│ exit code: 1
╰─> [43 lines of output]
C:\Users\Andy\AppData\Local\Temp\pip-req-build-3eo2lr_w\bindings/torch\setup.py:5: DeprecationWarning: pkg_resources is deprecated as an API. See https://setuptools.pypa.io/en/latest/pkg_resources.html
from pkg_resources import parse_version
Building PyTorch extension for tiny-cuda-nn version 1.7
Obtained compute capability 61 from PyTorch
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2022 NVIDIA Corporation
Built on Wed_Sep_21_10:41:10_Pacific_Daylight_Time_2022
Cuda compilation tools, release 11.8, V11.8.89
Build cuda_11.8.r11.8/compiler.31833905_0
Detected CUDA version 11.8
Targeting C++ standard 17
running bdist_wheel
C:\Users\Andy\Anaconda3\envs\nerfstudio\lib\site-packages\torch\utils\cpp_extension.py:476: UserWarning: Attempted to use ninja as the BuildExtension backend but we could not find ninja.. Falling back to using the slow distutils backend.
warnings.warn(msg.format('we could not find ninja.'))
running build
running build_py
creating build
creating build\lib.win-amd64-cpython-38
creating build\lib.win-amd64-cpython-38\tinycudann
copying tinycudann\modules.py -> build\lib.win-amd64-cpython-38\tinycudann
copying tinycudann_init_.py -> build\lib.win-amd64-cpython-38\tinycudann
running egg_info
creating tinycudann.egg-info
writing tinycudann.egg-info\PKG-INFO
writing dependency_links to tinycudann.egg-info\dependency_links.txt
writing top-level names to tinycudann.egg-info\top_level.txt
writing manifest file 'tinycudann.egg-info\SOURCES.txt'
reading manifest file 'tinycudann.egg-info\SOURCES.txt'
writing manifest file 'tinycudann.egg-info\SOURCES.txt'
copying tinycudann\bindings.cpp -> build\lib.win-amd64-cpython-38\tinycudann
running build_ext
building 'tinycudann_bindings._61_C' extension
creating build\dependencies
creating build\dependencies\fmt
creating build\dependencies\fmt\src
creating build\src
creating build\temp.win-amd64-cpython-38
creating build\temp.win-amd64-cpython-38\Release
creating build\temp.win-amd64-cpython-38\Release\tinycudann
"C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.29.30133\bin\Hostx64\x64\cl.exe" /c /nologo /O2 /W3 /GL /DNDEBUG /MD -IC:\Users\Andy\AppData\Local\Temp\pip-req-build-3eo2lr_w/include -IC:\Users\Andy\AppData\Local\Temp\pip-req-build-3eo2lr_w/dependencies -IC:\Users\Andy\AppData\Local\Temp\pip-req-build-3eo2lr_w/dependencies/cutlass/include -IC:\Users\Andy\AppData\Local\Temp\pip-req-build-3eo2lr_w/dependencies/cutlass/tools/util/include -IC:\Users\Andy\AppData\Local\Temp\pip-req-build-3eo2lr_w/dependencies/fmt/include -IC:\Users\Andy\Anaconda3\envs\nerfstudio\lib\site-packages\torch\include -IC:\Users\Andy\Anaconda3\envs\nerfstudio\lib\site-packages\torch\include\torch\csrc\api\include -IC:\Users\Andy\Anaconda3\envs\nerfstudio\lib\site-packages\torch\include\TH -IC:\Users\Andy\Anaconda3\envs\nerfstudio\lib\site-packages\torch\include\THC "-IC:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v11.8\include" -IC:\Users\Andy\Anaconda3\envs\nerfstudio\include -IC:\Users\Andy\Anaconda3\envs\nerfstudio\Include /EHsc /Tp../../dependencies/fmt/src/format.cc /Fobuild\temp.win-amd64-cpython-38\Release../../dependencies/fmt/src/format.obj /MD /wd4819 /wd4251 /wd4244 /wd4267 /wd4275 /wd4018 /wd4190 /EHsc /std:c++17 -DTCNN_PARAMS_UNALIGNED -DTCNN_MIN_GPU_ARCH=61 -DTORCH_API_INCLUDE_EXTENSION_H -DTORCH_EXTENSION_NAME=_61_C -D_GLIBCXX_USE_CXX11_ABI=0
format.cc
C:\Users\Andy\AppData\Local\Temp\pip-req-build-3eo2lr_w/dependencies/fmt/include\fmt/format-inl.h(11): fatal error C1083: Cannot open include file: 'algorithm': No such file or directory
error: command 'C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.29.30133\bin\Hostx64\x64\cl.exe' failed with exit code 2
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
ERROR: Failed building wheel for tinycudann
Running setup.py clean for tinycudann
Failed to build tinycudann
ERROR: Could not build wheels for tinycudann, which is required to install pyproject.toml-based projects
I'm quite new to this all and it seems a different error than those mentioned in the notes, so I hope some can interpret what the problem(s) is/are to get me further. Thanks!

hi, i met this error too. How did you solve it?

@ShaYito If I remember it well, I put in the command export LIBRARY_PATH="/usr/lib/wsl/lib:$LIBRARY_PATH just before I did the tiny-cuda-nn install and that fixed it for me.

I get the same bug in windows. What is the solution for it?

@ShaYito If I remember it well, I put in the command export LIBRARY_PATH="/usr/lib/wsl/lib:$LIBRARY_PATH just before I did the tiny-cuda-nn install and that fixed it for me.

I have this exact problem. How did you get it solve?
I put this command but it wasn't recognize

same problem, also in windows and the export LIBRARY_PATH did not work as well

same problem, also in windows and the export LIBRARY_PATH did not work as well

I already solve this problem but I don't remember exactly how. But looks like if you follow the clip. it won't work since there's update in the installation instruction page. SO you need to follow instruction in the page not clip. Good luck

@chocolateriderx what do you mean by "not clip"? if you could provide a link that would be great!

it worked for me now using WSL2 and this link: https://gist.github.com/SharkWipf/0a3fc1be3ea88b0c9640db6ce15b44b9 without any problems:))


User deletion

Hello, I encountered an issue with HomeKit guest users. When adding a guest with access to the nfc lock, everything works fine and the keys register in homekey.json. The problem is when I try to delete the user, because then the program doesn't wipe its credentials from homekey.json and the key still can be used to validate even though it was deleted from the home. Can you provide a fix for that? I've noticed that when the deletion happens, the program receives the accessory_driver unpair command and logs its execution, but that's it.

Hi,
Thanks for your report. I'm gonna take a look into it soon.

Hi again,
I've added immediate handling for "unpair" events into the develop branch.
Would be glad if you could try it out and report back.

Hello, I've tested the implementation and it seems solid. Everything works as expected. Thank you!


Use port below 1024 - moonraker behind reverse proxy

Hi.
I have moonraker behind reverse proxy and I use it over HTTPS. But at configuration stage I cannot use port with number lower than 1024. Why is there such a validation?
Mainsail have no problem with moonraker on port 443.

Hi we are not connecting via standard HTTP or HTTPS, instead we are using Websocket.
As an example, I connect to Mainsail via port 80 but I do have Moonraker on port 7125 for websoket.
You should be able to view this info in mainsail.cfg
https://moonraker.readthedocs.io/en/latest/configuration/#server

I connect both to mainsail and moonraker on same port and both are
available on port 443 including websockets ;)
sob., 8 kwi 2023, 13:26 użytkownik Marc-Olivier Arsenault <
@.***> napisał:

Hi we are not connecting via standard HTTP or HTTPS, instead we are using
Websocket.
As an example, I connect to Mainsail via port 80 but I do have Moonraker
on port 7125 for websoket.
You should be able to view this info in mainsail.cfg
https://moonraker.readthedocs.io/en/latest/configuration/#server
—
Reply to this email directly, view it on GitHub
https://github.com/marcolivierarsenault/moonraker-home-assistant/issues/106#issuecomment-1500871393,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJQSEYQ3LK466MDHOJY7DLXAFDP7ANCNFSM6AAAAAAWXM2FYY
.
You are receiving this because you authored the thread.Message ID:
<marcolivierarsenault/moonraker-home-assistant/issues/106/1500871393@
github.com>

have you tried entering the IP (with no https://) and port 443 ?
If that does not work, can you upload a copy of your mainsail config file and your Home Assistant logs (that covers an attempt to configure the integration)

That's the problem, I can't use port 443 ;)

Ooo interesting.
Let met look at that
Thanks did not realized that.
Le sam. 8 avr. 2023 à 08:49, wizu @.***> a écrit :

That's the problem, I can't use port 443 ;)
[image: moonraker]
https://user-images.githubusercontent.com/1247507/230722047-0d99ec0e-3035-4094-9d0e-4f2536cdaeea.png
—
Reply to this email directly, view it on GitHub
https://github.com/marcolivierarsenault/moonraker-home-assistant/issues/106#issuecomment-1500885243,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAUDC2QBPCTOAS3U44FGYCDXAFNF7ANCNFSM6AAAAAAWXM2FYY
.
You are receiving this because you were assigned.Message ID:
<marcolivierarsenault/moonraker-home-assistant/issues/106/1500885243@
github.com>

--
Marc-Olivier Arsenault

@wizu, we just shipped the patch, should be available in the next release.
In the mean time, you can get the latest code by downloading the main version
https://moonraker-home-assistant.readthedocs.io/en/latest/support/latest.html

Now it works well. Thanks.


Option # 1 - Implementation Completed - Moved code of coin gecko from Trakx.Common to Trakx.CoinGecko.ApiClient

@RafaelMeloTrakx should we close this one without merging it?