ADIOS2 output with ParaView visualization, adios2stream and ADIOS2DataCollection

This PR provides seamless integration of ADIOS2 output datasets. This PR is ready for review from the mfem community and I'd be happy to address potential feedback and accommodate for the community's preferences.
Overall, this PR provides:

The adios2stream and ADIOS2DataCollection classes for seamless integration of ADIOS2 library outputs v2.5.0 similar to the existing mfem's stream abstraction interface.

Paraview, as of upcoming 12/10/2019 MPI nightly release coverage using the ADIOS2VTXReader is documented under: config/run_adios2stream.py

Currently the ADIOS2 dependency is supported with mfem CMake-based builds only

To test this PR branch:

Install adios2 v2.5.0 (macOS users can do brew install adios2).

Configure mfem with cmake -DMFEM_USE_ADIOS2=ON -DADIOS2_DIR=/path/to/adios2 and build

cd mfem-build/examples copy or soft link the file config/run_adios2stream.py to run current examples with ParaView support. ./run_adios2stream.py ex9p has transient behavior and would generate the directory mfem_out_ex9p which contains the generated adios2 bp directories for each case.

Download ParaView nightly, go to Version>nightly and select the one from 12/10/2019 or more recent with MPI support.

Open any of the bp datasets from 3., select the ADIOSVTXReader and the variable to visualize (e.g. solution) in the variable drop down menu and click on the play button for the transient animation in ex9p. Running in parallel requires server/client mode launching mpirun -n 4 pvserver.

Main motivation/long-term goals:

Provide flexibility to tackle I/O bottlenecks in MPI environments, examples: set arbitrary number of substreams (N-to-M) where N is MPI processes and 1<M<N. Particularly useful when N is large (1024 or above) to reduce interaction with the filesystem at scale. See #1008

Provide all mfem generated outputs, spatial and temporal data, under a single directory containing binary data (BP format), see #44 . Data interaction can be done through ADIOS2 C++11 e.g. Python APIs, and the command-line bpls utility.

Provide Paraview and VTK out-of-the-box serial and parallel (pvserver) visualization using an automated schema. This work is mainly done in VTK on the VTK::IOADIOS2VTX module reader.

Current Shortcoming/Limitations:

Some AMR cases: ex6p, ex15p
Support for mixed elements: (*-mixed meshes)
Higher-Order elements (to explore). Currently relying on RefinedGrid.
Temporal-varying mesh not tested.
Need more tests at scale and miniapps.
VisIt support (VisIt is currently using an older VTK version).
Currently file-based support only, memory-to-memory insitu requires extra work in VTK

PR
Author
Editor
Reviewers
Assignment
Approval
Merge

#1188
@williamfgc
@tzanio
@rcarson3 + @jakubcerveny
02/16/20
⌛due 03/08/20
⌛due 03/15/20

How hard would it be to add a DataCollection type interface for ADIOS2? I use that interface quite often within ExaConstit as seen here in the ExaConstit driver. As an application user, I've found that this interface makes it very easy to support different data formats for various post-processing needs that might pop-up.

@rcarson3 thanks for the feedback. It shouldn't be hard, all functions in this PR trace back to the adios2stream class. If that's preferred I'd be happy to add an adios2 data collection.

@williamfgc if you could add that DataCollection interface I could definitely bring in these changes into one of my branches of MFEM and test it out within ExaConstit and let you know about any comments I might have on using it.

@rcarson3 sounds great, allow me a few weeks for the changes and testing with ParaView to be ready as I'll be on and off due to the holiday break. In the meantime, feedback is welcome for further changes to the current PR. I'll keep you posted. Thanks!

I removed the ready-for-review label not to trigger our 3-week review process, but please feel free to review informally and provide feedback to @williamfgc.

@tzanio @rcarson3 finally got the time to focus on this pending PR. I added the ADIOS2DataCollection interface as requested. Please feel free to review, this is no longer WIP. Looking forward to your feedback. Thanks!

Please address Travis failures -- the code should build fine without ADIOS if MFEM_USE_ADIOS2 is not set.

@tzanio thanks! The issues are addressed, there is only one optional check I don't know how to address or if it's an issue at all. Please see below:
Running test 'branch-history'... FAILED
=== BEGIN error log from branch-history.err
This branch adds/modifies a large file: mesh/mesh.cpp (size 280K)
Commit 201cbedf adds/modifies a large file: mesh/mesh.cpp (size 284K)
=== END error log from branch-history.err

This check is OK. Thanks @williamfgc !

@williamfgc I've been able to run the example problems just fine. Overall, the code seems fine. One thing you might want to look at is how PR #1296 added higher order elements for the Paraview datacollection. You might be able to do something like that here. I'm going to try testing it within ExaConstit next, and see if anything comes up during that testing.

@rcarson3 thanks for the feedback. I'll look into PR #1296 . Let me do a separate PR for it. Please let me know if you run into issues with this PR or ParaView viz when running ExaConstit. Thanks!

@williamfgc I was able to run in ExaConstit with relatively decent success. However, I did run into a few problems. First, I found that if I'd declared an Adios2DataCollection like so:
ADIOS2DataCollection adios2_dc(MPI_COMM_WORLD, collection_name, pmesh);

I ran into issues where if the destructor for this object occurred after the MPI_Finalize() call was made the program would crash. I found that I had to do something like the below to fix the crash.
ADIOS2DataCollection *adios2_dc = new ADIOS2DataCollection(MPI_COMM_WORLD, collection_name, pmesh);

with the necessary delete later on before MPI_Finalize();
It might be nice to have this documented with that data collection class, so  users can see how to properly handle/create that object within the generated Doxygen documents.

@rcarson3 thanks for the feedback, that's a very good point. The C++ destructor with a copy to the MPI communicator gets called after MPI_Finalize is scope is the same as MPI_Init/MPI_Finalize. I will  add it to the constructor doxygen header. Thanks!

@rcarson3 and @jakubcerveny, do you need more time to review this?

@tzanio I don't think I need more time. I've got a few points above that still need to be addressed, but overall I'm happy with the PR. It runs fine on a number of different platforms and seems to work within larger applications such as ExaConstit.
Although, the PR does touch most of the examples and adds Adios2 output as a feature to them. We'd probably want to restrict those changes to only a few examples like one where the DataCollection output is used and another where the mesh print option is used.
@williamfgc it sounds like in a future PR you might add the higher-order element support in maybe in that same PR you could also implement a load functionality to it. Afterwords, you should be able to test, at least your DataCollection, methods in a similar framework\way that's being done for the VisitDataCollection in the unit test suite.

I installed ParaView, compiled/installed ADIOS2, ran ex9p through the script and obtained directories like ex9-p-periodic-cube.mesh_o2.bp. However I'll probably need a little more info on how to visualize the results in ParaView.
I'm running mpirun -n 4 pvserver on the background and I opened the *.idx file in one of the result directories. I chose "solution" as the image dimension (?) and pressed Play. And I'm getting some warnings and errors:

@jakubcerveny thanks for reviewing. A couple of things:

please try opening directly the .bp directory, ParaView accepts opening directories since a few versions ago.
make sure you select the ADIOS2VTXReader instead of the ADIOS2CoreReader. The ADIOS2VTXReader is only available with Paraview MPI variants in https://www.paraview.org/download/ .

The detailed instructions are in my first post of this PR, please find a screenshot with the same file and ParaView version you're referring to. Let me know if you have any questions. Thanks!

@williamfgc It works now, thanks!

Can you please resolve the merge conflicts

@tzanio @rcarson3 @jakubcerveny the latest commit resolve the conflicts with master. Please let me know what you decide on which examples adios2-related functionality should be added. It will keep the current configuration on a branch in my fork for future testing. Thanks for your feedback.

We agreed with @tzanio that we want the ADIOS2 output only in examples where VisIt output is, to prevent copying the same code. Examples 5, 9, 15 and 16 should cover all the use cases, is that right?

@jakubcerveny thanks, I'd accommodate to the request. Also, I use adios2stream in example 12 to illustrate saving temporaries. Is it OK to keep this example?

If ex12p shows something unique then I'd keep it. Thank you.

Can you please resolve the merge conflicts?

Thanks for the quick response @williamfgc.
@rcarson3 and @jakubcerveny -- can you take another look and approve?

@williamfgc Do you plan to merge adios2datacollection.{c,h}pp with datacollection.{c,h}pp? On second thought, we do already have conduitdatacollection.* so there is a precedent for a new module... what do you think @tzanio?

@williamfgc Do you plan to merge adios2datacollection.{c,h}pp with datacollection.{c,h}pp? On second thought, we do already have conduitdatacollection.* so there is a precedent for a new module... what do you think @tzanio?

I don't have a strong preference. Separate files are OK.

The license needs to be updated though...

I'm not sure we really need the run_adios2stream.py script, but otherwise I'm OK.

I'm not sure we really need the run_adios2stream.py script, but otherwise I'm OK.

@williamfgc, what's the purpose of that script?

@tzanio the script is just for information purposes and for running the examples covered with ParaView automatically (I thought config would be a good place)
# Usage: Run from mfem-build/examples
# ./run-adios2-examples.py example
# Example:
# To run examples ex9p: ./run_adios2stream.py ex9p
# To run all examples: ./run_adios2stream.py all

The end result is a directory with all the possible examples combinations from the comments or each example file.
I don't have a preference, just thought it's convenient to provide a python script to generate all the supported cases at once rather than doing it manually. Let me know if you want me to remove it.

@tzanio the script is just for information purposes and for running the examples covered with ParaView automatically (I thought config would be a good place)
# Usage: Run from mfem-build/examples
# ./run-adios2-examples.py example
# Example:
# To run examples ex9p: ./run_adios2stream.py ex9p
# To run all examples: ./run_adios2stream.py all

The end result is a directory with all the possible examples combinations from the comments or each example file.
I don't have a preference, just thought it's convenient to provide a python script to generate all the supported cases at once rather than doing it manually. Let me know if you want me to remove it.

My vote is to remove it. I understand that it is useful, but we don't have other project-specific scripts like that.

@tzanio sounds good, I'll go ahead and remove it.

One option is to keep the script under tests/adios2. It can be useful for testing?

Merged in next for testing...

Re-merged in next for testing...

LGTM

Future improvements:

[ ] Support make-based build system
[ ] Support high-order output and ParaView visualization


Please invite me to the GitHub Community Organization

Name
Sakshi Sharma
Discord Username (if applicable)
No response
Additional Context
want to join

Bro idk nothing about it.   Do you know anything about ledger?
On Mon, Mar 27, 2023, 12:12 AM github-actions[bot] @.***>
wrote:

It's great having you contribute to this project Welcome to the community
🤓
If you would like to continue contributing to open source and would like
to do it with an awesome inclusive community, you should join our Discord
Server http://discord.eddiehub.org and our GitHub Organisation
http://github.eddiehub.org - we help and encourage each other to
contribute to open source little and often 🤓 . Any questions let us know.
—
Reply to this email directly, view it on GitHub
https://github.com/EddieHubCommunity/support/issues/7147#issuecomment-1484463978,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6QM2DD6T5RPCFH7D2K4XQLW6EHSFANCNFSM6AAAAAAWIUQISM
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


failed to detect package manager

Apologies in advance if this isn't the proper place to share this error.

@pheuter  Does your package.json file inside /packages/background directory?

@pheuter  Does your package.json file inside /packages/background directory?
You should create directory with name background-functions/ inside root directory mentioned in form.

Appreciate the response, I'll go ahead and close this issue, being helped now over Slack 👍


sprint challenge

@clintKunz

MVP

[x] Comprehension Questions
[x] Build the HTML and CSS to create the missing navigation and header
[x] Link up About and Home pages
[x] Box colors
[x] Build the rest of the About page layout to match the design
[x] Pull request

Stretch

[ ] Build a page of your choosing from the navigation items. Come up with content and images that fit the theme.
[ ] Introduce CSS animations to your site.
[ ] Build a contact page and create a form with several inputs of your choosing
[ ] Add responsive breakpoints to your code by using media queries

What Looks Great

MVP complete! Nice work
Nice semantic HTML
Great use of Flexbox throughout both pages

What Could Be Improved

On the nav bard you could line up the anchor tags (based on baseline) with the logo as seen in the design file
Frequent and descriptive commits

Rating (1-3)
2


ADOrganizationalUnit: Adding Server parameter

Pull Request (PR) description

This Pull Request (PR) fixes the following issues

Fixes #563
(Should be done to all AD resources though)

Task list

[ ] Added an entry to the change log under the Unreleased section of the
file CHANGELOG.md. Entry should say what was changed and how that
affects users (if applicable), and reference the issue being resolved
(if applicable).
[ ] Resource documentation added/updated in README.md.
[ ] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help.
[x] Comment-based help added/updated.
[ ] Localization strings added/updated in all localization files as appropriate.
[ ] Examples appropriately added/updated.
[ ] Unit tests added/updated. See DSC Community Testing Guidelines.
[ ] Integration tests added/updated (where possible). See DSC Community Testing Guidelines.
[ ] New/changed code adheres to DSC Community Style Guidelines.

This change is

Closing PR as it is abandoned. Please reopen if you wish to continue work on this.


[FEATURE] rebuild last healthy replica during node drain

Is your feature request related to a problem? Please describe.

Currently with the introduction of the node drain helper, if there is a storage class with just one replica, and that happens to be on the node being drained, then longhorn instance-manager pods will not be evicted even when the VM is stopped.
Describe the solution you'd like

With the introduction of faster replica rebuilds in longhorn v1.4.0: https://github.com/longhorn/longhorn/issues/4783 we should be able to trigger a replica rebuild for such a scenario, by evicting the replica.
This should allow us to safely migrate data for such workloads and hopefully keep them online.
Describe alternatives you've considered

Additional context

@Vicente-Cheng @starbops I think we have this issue in the upgrade path too.

There is a related upstream issue for doing this automatically: https://github.com/longhorn/longhorn/issues/2238

I've been playing around with the code and just wanted to centralize all of my notes here regarding requirements/caveats.
Requirements:

Longhorn version >= 1.4.0, which is not a problem for Harvester iirc we already have checked in a newer version of Longhorn.
Longhorn setting fast-replica-rebuild-enabled set to true. This appears to be the default in one of my fresh installations of Harvester.
snapshot-data-integrity needs to be set to fast-check on the affected volumes for the draining node (or this setting can be set to ignore if the global Longhorn value for this setting is fast-check)
Node ActionHandler should require the force option as it currently does if the unhealthy VM volume's snapshot-data-integrity is not set to fast-check or if fast-replica-rebuild-enabled is set to false. In this case, it will behave as it currently does where it will shut down the VMs.
The nodedrain controller should retain its behavior of shutting down the workloads if the force annotation is used.

Known caveats:

I can't find a way to evict on a replica-by-replica basis, but I'm able to evict all replicas on the node.

@WebberHuang1118 Please check if this is still an issue or we need to modify a Longhorn setting.


Changing order of nodes

I made this demo: https://codesandbox.io/s/festive-germain-1js7yj?file=/App.svelte
In the demo, I am rendering stars from an array. Also I am changing order of stars, to display dragged start on top:
let handleDragStart = e => {
	// save drag element:
	dragItemId = e.detail.target.id();
	// move current element to the top:
	const item = list.find(i => i.id === dragItemId);
	const index = list.indexOf(item);
	list.splice(index, 1);
	list.push(item);
};

But I don't see that on canvas. Does the library support that? Or I am doing something wrong?

Your code sandbox link is corrupted, it says "something went wrong". Can you share another one?

Updated. Same link https://codesandbox.io/s/festive-germain-1js7yj?file=/App.svelte

I think it must follow the order that I have in the template. It is predicable and allows writing in app in a "svelte" and declarative way. If possible, a user should use as less Konva API directly as possible.

After some more tinkering, this is not possible to achieve in a sound way in Svelte right now. I have created an issue in Svelte about this (https://github.com/sveltejs/svelte/issues/8547) so we might get the required functionality to make this work in the future. For now, we have to rely on the Konva-native way of ordering for such use cases. I'll keep this issue open to track any progress on Svelte side which might make such a functionality possible. I've also written an extensive doc page in the old svelte-konva docs about this, and will port the important bits to the Konva page to avoid too much confusion about this.


Question: Running on Localhost?

Hi
I am trying to run this via localhost.
 C:\wamp64\www\htdocs1>php bin/plugin blackhole generate localhost/htdocs1 static
ERROR Blackhole failed to start.
ERROR The website must match the location of your Grav installation.
ERROR You must have at least one page.
Any ideas? I can in my browser go to localhost/htdocs1 and see the homepage.

I know it shouldn't require this, but try adding http:// to the start of localhost.

Same result

It looks like you added php to the start of your command. Try it just like this.
bin/plugin blackhole generate http://localhost/htdocs1 static

C:\wamp64\www\htdocs1>bin/plugin blackhole generate http://localhost/htdocs1 static
'bin' is not recognized as an internal or external command, operable program or batch file.
Saw on Grav Help section that Windows machine should add php infront of the command in order to execute.

Nice, I didn't know that. Anyway, I'll see what I can find.
After checking it out, there was one issue that I think was related to #7. Anyway, I'll post a fix soon.


Add NINPO

Hi @iwillbuy
I can't not find your denom on-chain data.
I double checked with "factory/inj1sudjgsyhufqu95yp7rqad3g78ws8g6htf32h88/ninpo"  but no data.
please check more information for add asset

Hi @wannabit-yongjoo
Does this not support CW20 token on injective ?

https://explorer.injective.network/contract/inj1sudjgsyhufqu95yp7rqad3g78ws8g6htf32h88/

There are no dapp that use cw20 tokens yet, so currently we do not support them.


Connections existence seems to be checked by retrieving all the connections in "Custom Connector" type

Describe the issue:
When creating a connection in "Custom Connector" type, existence of the connection is checked by listing all the connections as it seems (cannot see any filter call with the connection name).

How to reproduce:
Create a connection in "Custom Connector" type with an existing connection name.

Released with https://wso2.org/jenkins/job/products/job/product-is/4979/


Add coordinator Setup functions to the Path type

Summary
The coordinator has the following functions:
type Coordinator interface {
    IncrementTime()
    IncrementTimeBy(increment time.Duration)
    UpdateTime()
    UpdateTimeForChain(chain *TestChain)

    Setup(path *Path)
    SetupClients(path *Path)
    SetupConnections(path *Path)

    CreateConnections(path *Path)
    CreateMockChannels(path *Path)
    CreateTransferChannels(path *Path)
    CreateChannels(path *Path)

    GetChain(chainID string) *TestChain

    CommitBlock(chains ...*TestChain)
    CommitNBlocks(chain *TestChain, n uint64)
}

The following functions should be moved to be under the path type:
type Path interface {
    Setup()
    SetupClients()
    SetupConnections()

    CreateConnections()
    CreateChannels()
}

I think we could consider a way to reduce this API further (definitely keeping Setup(), but maybe reducing the redundancy between SetupConnections and CreateConnections)
Problem Definition
The coordinator is an unnecessary type in the setup of the path. It is more natural to simply reference the path type directly. This clears the way to reduce the usage of coordinator which in the long run may be an unnecessary layer of the testing pkg.
Proposal
Change:
    path := ibctesting.NewPath(suite.chainA, suite.chainB)
    suite.coordinator.Setup(path)

to
    path := ibctesting.NewPath(suite.chainA, suite.chainB)
    path.Setup()

NOTE: I recommend we leave the coordinator functions (deprecation) and add the functions to the path and replace our internal usage. After release of v8, then I think we can remove the unnecessary coordinator functions. Because v8 is a massive release that is intermixed with a SDK and cometbft update, I think we should limit unnecessary breaking changes

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged/assigned

Lets add the proposed functions to path and deprecate the legacy functions on the coordinator. I will open a separate issue to remove the deprecated functions. That way we can make this change and choose when to have users update at a reasonable time period

Can I work on this sir ?


How is the easiest way to insert an image just like in the desktop version?

update @blink-mind/***  to v0.0.20-beta.12


NPE on INSERT into column in altered table

CrateDB version
4.5.1
CrateDB setup information
Number of nodes: 3
12 GB heap
Steps to Reproduce
CREATE TABLE IF NOT EXISTS "doc"."0x001" (
   "ts" TIMESTAMP WITHOUT TIME ZONE,
   "stream_id" TEXT,
   "01_01" OBJECT,
   "01_02" OBJECT,
   "01_03" OBJECT,
....
"ts_partition" TIMESTAMP WITHOUT TIME ZONE GENERATED ALWAYS AS _cast(date_trunc('quarter', 'UTC', "ts"), 'timestamp without time zone')
)
CLUSTERED BY ("stream_id") INTO 6 SHARDS
PARTITIONED BY ("ts_partition")
WITH (
   "allocation.max_retries" = 5,
   "blocks.metadata" = false,
   "blocks.read" = false,
   "blocks.read_only" = false,
   "blocks.read_only_allow_delete" = false,
   "blocks.write" = false,
   codec = 'default',
   column_policy = 'strict',
   "mapping.total_fields.limit" = 1000,
   max_ngram_diff = 1,
   max_shingle_diff = 3,
   number_of_replicas = '0-1',
   "routing.allocation.enable" = 'all',
   "routing.allocation.total_shards_per_node" = -1,
   "store.type" = 'fs',
   "translog.durability" = 'REQUEST',
   "translog.flush_threshold_size" = 536870912,
   "translog.sync_interval" = 5000,
   "unassigned.node_left.delayed_timeout" = 60000,
   "write.wait_for_active_shards" = '1'
)

Inserting works ok so far, then altered table:
alter table "doc"."0x001" add column "02_0815" OBJECT;
everything still works ok, but after a view days, we cannot insert into the new column 02_0815 anymore:
error: NullpointerException[null], code: 5000, trace: Java.Lang.NullpointerException
We insert via http and do not always get this error, but most of the time Status 200, and rowcount(s) : -2
Also tried inserting via the AdminUI, same result, NullpointerException[null] !
But: inserting into all the other columns still works !
Tried to create another OBJECT column in this table, same result.
At this point the table has:

707 columns
132 MIO records
Size 19.1 GB
Shards 48

The sys.jobs_log table shows the insert statements, but error column is always null.
We then created another table with just 1 OBJECT column (and the other columns from the first table): inserting works as expected.
Expected Result
INSERTING works or get a decent error message.
Actual Result
error: NullpointerException[null], code: 5000, trace: Java.Lang.NullpointerException

Data we try to insert:
INSERT INTO "0x001" (ts, stream_id, "02_0815") VALUES<PHONE_NUMBER>512,'341af56b2ccd533124e2caa0add16dde',
{"sen-z2Right" = 0,
"sen-z2Left" = 7526,
"sen-Sen60V" = 65,
"WARM-Cnt_lt"=0,
"Temp-MIN"=18,
"Temp-MAX"=38,
"Temp-Current"=28,
"PLA-SearchCnt_lt"=34800,
"PLL-SucAllCnt_lt"=26,
"PLL-FailAllCnt_lt"=1064,
"FISH-PickCnt_lt"=0,
"FDRO-SucAllCnt_lt"=26,
"FDRO-FailAllCnt_lt"=1064,
"ERRH-TOCnt_lt"=0,
"ERRH-TICnt_lt"=0,
"ERRH-RACnt_lt"=0}
);

@heyoka Thanks for the report. Would be really great if you could send us the related stacktrace. While using the HTTP interface this can be gathered by adding error_trace=true to the URL, see https://crate.io/docs/crate/reference/en/4.6/interfaces/http.html#error-handling.

@seut As written above, the contents of the error_trace field in the response is exactly :  java.lang.NullPointerException\\n nothing more

For clarification: the number of columns I posted above is the result of
select COUNT(*)
from information_schema.columns
where table_schema = 'doc' and table_name = '0x001'

707
We do NOT have more than 700 OBJECT(DYNAMIC) columns in the table

Probably related to 4594 - it's also NPE with code 5000.

Tried to replicate this with a restore of the affected table with
NullPointerException[Cannot invoke “Object.equals(Object)” because “sourceValue”

when trying to insert


404 at /desktop/previous-versions/archive-windows/undefined

I found a broken link : https://docs.docker.com/desktop/previous-versions/archive-windows/undefined
I arrived on this page through: https://docs.docker.com/desktop/previous-versions/archive-windows/

Thanks for sharing the issue. Closing this ticket in favor of https://github.com/docker/docker.github.io/issues/13553, but if you're able to provide more details that could help us find the problem, feel free to comment on that ticket.


Error: No ML Server install Path found in SOFTWARE\Microsoft\R Server when using azure-ml-admin-cli

Describe the bug
Encountering an error when attempting to use az login with the azure-ml-admin-cli extension. The error message indicates that no ML Server install path is found. Additionally, there are syntax warnings in mls_client.py.
Related command
az login
Errors
C:\Users\muham.azure\cliextensions\azure-ml-admin-cli\azext_admin\clients\mls_client.py:127: SyntaxWarning: "is not" with a literal. Did you mean "!="?
if res.status is not 200 and res.status is not 201:
C:\Users\muham.azure\cliextensions\azure-ml-admin-cli\azext_admin\clients\mls_client.py:127: SyntaxWarning: "is not" with a literal. Did you mean "!="?
if res.status is not 200 and res.status is not 201:
The command failed with an unexpected error. Here is the traceback:
No ML Server install Path found in SOFTWARE\Microsoft\R Server.
Traceback (most recent call last):
File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\knack/cli.py", line 233, in invoke
File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/init.py", line 664, in execute
File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/init.py", line 731, in _run_jobs_serially
File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/init.py", line 701, in _run_job
File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/init.py", line 334, in call
File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/command_operation.py", line 112, in handler
File "C:\Users\muham.azure\cliextensions\azure-ml-admin-cli\azext_admin_client_factory.py", line 83, in admin_extensions_client
client = AdminExtensionsClient(endpoint, access_token)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\muham.azure\cliextensions\azure-ml-admin-cli\azext_admin_client_factory.py", line 97, in init
self._admin_client = AdminUtilClient(access_token)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\muham.azure\cliextensions\azure-ml-admin-cli\azext_admin\clients\admin_util_client.py", line 35, in init
o16n_path = PathContext().o16n_path
^^^^^^^^^^^^^
File "C:\Users\muham.azure\cliextensions\azure-ml-admin-cli\azext_admin\path_context.py", line 30, in init
raise EnvironmentError('No ML Server install Path found in '
OSError: No ML Server install Path found in SOFTWARE\Microsoft\R Server.
To check existing issues, please visit: https://github.com/Azure/azure-cli/issues
Issue script & Debug output
C:\Users\muham.azure\cliextensions\azure-ml-admin-cli\azext_admin\clients\mls_client.py:127: SyntaxWarning: "is not" with a literal. Did you mean "!="?
if res.status is not 200 and res.status is not 201:
C:\Users\muham.azure\cliextensions\azure-ml-admin-cli\azext_admin\clients\mls_client.py:127: SyntaxWarning: "is not" with a literal. Did you mean "!="?
if res.status is not 200 and res.status is not 201:
The command failed with an unexpected error. Here is the traceback:
No ML Server install Path found in SOFTWARE\Microsoft\R Server.
Traceback (most recent call last):
File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\knack/cli.py", line 233, in invoke
File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/init.py", line 664, in execute
File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/init.py", line 731, in _run_jobs_serially
File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/init.py", line 701, in _run_job
File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/init.py", line 334, in call
File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/command_operation.py", line 112, in handler
File "C:\Users\muham.azure\cliextensions\azure-ml-admin-cli\azext_admin_client_factory.py", line 83, in admin_extensions_client
client = AdminExtensionsClient(endpoint, access_token)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\muham.azure\cliextensions\azure-ml-admin-cli\azext_admin_client_factory.py", line 97, in init
self._admin_client = AdminUtilClient(access_token)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\muham.azure\cliextensions\azure-ml-admin-cli\azext_admin\clients\admin_util_client.py", line 35, in init
o16n_path = PathContext().o16n_path
^^^^^^^^^^^^^
File "C:\Users\muham.azure\cliextensions\azure-ml-admin-cli\azext_admin\path_context.py", line 30, in init
raise EnvironmentError('No ML Server install Path found in '
OSError: No ML Server install Path found in SOFTWARE\Microsoft\R Server.
To check existing issues, please visit: https://github.com/Azure/azure-cli/issues
Expected behavior
The az login command should successfully authenticate without errors related to ML Server.
Environment Summary
OS: Windows
Azure CLI Version: (Provide the version from az --version)
azure-ml-admin-cli Version: (Provide the version from az extension show --name azure-ml-admin-cli)
Additional context
Confirmed that ML Server is installed and path is correctly set in the registry.
Followed troubleshooting steps: syntax fix, reinstalling extension, updating CLI and extensions.

Thank you for opening this issue, we will look into it.


Add support for script_raised_revive callback.

Version: 1.4.3
Date: 2019-12-30
    Features:
    - support script_raised_revive

This addresses the bug report filed here:
https://mods.factorio.com/mod/LightedPolesPlus/discussion/5c7be643c3dad0000c727550

Thanks for releasing so fast!


[WIP] Add microbenchmarks for RpcUtils

What changes are proposed in this pull request?
Add some microbenchmarks for RpcUtils, because the stress benchmarks we have now are too heavy, and the variances are too high to tell if a small change causes performance degradation.
Why are the changes needed?
To gain confidence on the performance impact of a potentially costly change, with smaller variance in benchmark results and smaller scopes.
Does this PR introduce any user facing changes?
Sort of.
The benchmarks are standalone executables, but they are not meant to be used directly by end users.

Codecov Report

Merging #14003 (9cd5b04) into master (65e19e1) will decrease coverage by 25.91%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master   #14003       +/-   ##
=============================================
- Coverage     43.30%   17.39%   -25.92%
+ Complexity     9152     2689     -6463
=============================================
  Files          1413     1416        +3
  Lines         81837    81992      +155
  Branches       9909     9923       +14
=============================================
- Hits          35439    14260    -21179
- Misses        43443    66480    +23037
+ Partials       2955     1252     -1703

Impacted Files
Coverage Δ

...ient/journal/RetryHandlingJournalMasterClient.java
0.00% <0.00%> (ø)

...a/alluxio/master/journal/DefaultJournalMaster.java
25.00% <0.00%> (-4.42%)
:arrow_down:

...ter/journal/JournalMasterClientServiceHandler.java
26.31% <0.00%> (-7.02%)
:arrow_down:

...alluxio/master/journal/raft/RaftJournalSystem.java
0.00% <0.00%> (-59.57%)
:arrow_down:

...rc/main/java/alluxio/microbench/RpcUtilsBench.java
0.00% <0.00%> (ø)

...lluxio/cli/fsadmin/journal/QuorumElectCommand.java
0.00% <0.00%> (ø)

...mon/src/main/java/alluxio/shell/CommandReturn.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...mon/src/main/java/alluxio/util/ExceptionUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...n/src/main/java/alluxio/wire/AlluxioProxyInfo.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../src/main/java/alluxio/wire/AlluxioMasterInfo.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 683 more

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 d67996c...9cd5b04. Read the comment docs.


Line graph for number of alerts per 1 hour is incorrect

As shown below, the line graph keeps dropping to 0 for every point, even though there's non-zero data for every point.

I just browsed to pigeonhole and it looks as expected

Selecting yesterday && today as the date range changed this:

Looking a bit further

Looking at @series here, if I load one day I get:
"[{\"name\":\"Number of alerts per 1h\",\"data\":[[1431612000000,0],[1431615600000,0],[1431619200000,4],[1431622800000,2]...

If I load more than one day the data contains 2 values for each timestamp: "0" and then the actual value:
"[{\"name\":\"Number of alerts per 1h\",\"data\"[[1431525600000,0],[1431525600000,3],[1431529200000,0],[1431529200000,4],[1431532800000,0],[1431532800000,2]:(..)[1431615600000,0],[1431615600000,0],[1431619200000,0],[1431619200000,4],(..)

Turns out influxdb returns the data we display:
:count=>[{"time"=>1431565200, "count"=>19}, {"time"=>1431561600, "count"=>18},(..),{"time"=>1431561600, "count"=>0}, {"time"=>1431565200, "count"=>0},(..)

I can't even.
select count(incident_key) from pigeonhole where time ><PHONE_NUMBER>s and time <<PHONE_NUMBER>s group by time(1h) fill(0)
returns duplicates, while
select count(incident_key) from pigeonhole group by time(1h), fill(0) where time ><PHONE_NUMBER>s and time <<PHONE_NUMBER>s doesn't.
Looking at the query_language the latter is the preferred syntax. PR coming up.

https://github.com/influxdb/influxdb/issues/2583


ui: make experimental mode longitudinal more clear

by using neutral color path (grey/silver) when acceleration near zero. wip draft
https://imgur.com/a/TZASRXg

https://github.com/commaai/openpilot/pull/27391


Bug: Check for updates via menu-bar sub-menu action (macOS)

OS: macOS
Version of IPFS Desktop [0.20.0]

Describe the bug
Using latest version of ipfs-desktop, manually selecting to check for updates locks up the UI when the desktop application is not open.
To Reproduce
Steps to reproduce the behavior(macos):

Start ipfs-desktop
Ensure the menu-bar application is visible, and then close the ipfs-desktop window (the application will still be running, and accessible via the menu-icon)
Click the ipfs-desktop menu-bar icon
Hover over about
Click on 'Check for updates'

The sub-menu may close (I believe this is only on the newest version). If it does, re-open it before the update-checking is done in order to reproduce the bug.

Notice that the UI for the menu-bar ipfs menu locks up

The ipfs-desktop application will re-open in the dock with a pop-up confirmation indicating that you are on the latest version.
Expected behavior
Checking for updates is a non-blocking background operation.
Screenshots
Action:

Locked up UI:

Latest version confirmation popup:

Additional context
When I first ran into this bug, the Dock did not display the ifps-desktop icon. So I was unable to close the confirmation popup, and the UI stayed locked up, with the menu-bar menu remaining open and inaccessible.
combined.log
error.log

Checking for updates is a non-blocking background operation.

On my computer, the menubar does not lock during the update checking as it is a background operation. It locks, indeed, after the dialog shows up. However, I think that is a limitation from Electron as both the menubar and the popup are being rendered from the main process.


How to determine when a control is databound?

Is there any event that tells me when a control is databound? I'm trying to determine when an sap.m.Select list binds against an OData model, but I don't see any sort of attachDataBound event. When I try to grab the getBinding(modelName) from the Select, it's always null.
What's the preferred way to determine when a field has been databound?

Hi Peder,
you can check binding with oSelect.isBound("items")
https://github.com/SAP/openui5/blob/master/src/sap.ui.core/src/sap/ui/base/ManagedObject.js#L3212-L3221
also binding has change event that you can register your event listener once.
https://github.com/SAP/openui5/blob/master/src/sap.ui.core/src/sap/ui/model/Binding.js#L97-L108
BTW, we really appreciate if you keep using this for issues and stackoverflow for your questions.
Best regards,
Cahit

I opened this because I don't know that those are suitable alternatives.
(1) isBound is not an event. I can't poll that property continuously to see if a control is bound
(2) My sap.m.Select returns null when I call .getBinding() in onInit, meaning I can't attach to an onchange event.

So I should clarify, I think I'm asking for ManagedObject to have a new event: attachDataBound. This issue should be an enhancement request, unless there's a suitable workaround.

Ugh. Ignore this issue. I had misread the documentation:
getBinding(sName): sap.ui.model.Binding
Get the binding object for a specific aggregation/property
Thanks.

But that still returns null: dropDownList.getBinding("items") returns nada since dropDownList.mBindingInfos.items.binding is null.
So I really don't know what I'm supposed to do here.


Adds match agnostic lint result identity referencing

With the advent of fuzzy matching, referencing back to the original lint result was not possible using the existing logic for identifying. We needed a way to associate the todo itself with the original lint result, thus providing a deterministic way of finding that lint result in order to mutate the severity, which is the crux of the todo functionality.
This PR adds a new mechanism whereby the consumers pass the lint result identity in when building the todo datum. This allows for referencing the lint result when processing the stable batch, which is the batch targeted to have its severity mutated.
This change has been tested and confirmed to work in both current consumers: eslint and ember-template-lint.

Similar to #276, @rwjblue and I had discussed this in depth before implementation. Merging this through without direct approval after discussing that with him.


opts.path is undefined in webpack plugin

🐛 Bug Report
I am migrating to the latest version of webpack and for the config new LoadablePlugin ({writeToDisk: true}) the file loadable-stats.json is never created, I added a log and I have:
{
  filename: 'loadable-stats.json',
  writeToDisk: true,
  outputAsset: true,
  path: undefined
}

My loadable-stats.json is removed by the clean-webpack-plugin.


Explicitly set file type

Describe the bug
csv file are being converted into txt files. The solution here is to explicitly set thecontent type when we upload to IPFS.

This is done


guides: unable to complete deployed basic-go-modules-example

Steps to reproduce:

Visit https://play-with-go.dev/basic-go-modules-example/
Click the first four steps in sequence
Observe the last command in the second command block fails with:

$ git push -u origin main
fatal: unable to access 'https://gopher.live/mod18af785c7bc01.git/': server certificate verification failed. CAfile: none CRLfile: none

This suggests to me that that guide container is not routing gopher.live to the private nginx instance per the docker-compose setup:
https://github.com/play-with-go/play-with-go/blob/bd8be67d0e39d5f97397dc7f95a2b2ba44028f8c/docker/nginx/nginx.conf#L4-L23
In development this happens because we pass set the PWD_UNSAFE=true environment variable:
https://github.com/play-with-go/play-with-go/blob/bd8be67d0e39d5f97397dc7f95a2b2ba44028f8c/docker-compose.yml#L88
This causes the guide-configured networks to be passed to PWD (but I'm guessing this doesn't mean anything in our production deploy?)

We will look at this once again once we have the production deployment in place.

This can be closed, right Paul?

Yep, thanks.


[MB-4107] Storybook components for OrdersTable and ProfileTable

Description
Add storybook components for OrdersTable and ProfileTable for use in review page later
Setup
make storybook
Code Review Verification Steps

[ ] User facing changes have been reviewed by design.
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change

Screenshots

Warnings

:warning:
This PR does not include changes to unit tests, even though it affects app code.

:warning:

Files located in legacy directories (src/shared or src/scenes) have
been edited. Are you sure you don’t want to also relocate them to the new file structure?
View the frontend file org ADR for more information

Messages

:book:
:link: MB-4107

  Generated by :no_entry_sign: dangerJS against 9ecb9310ca0e75097cdb976b2988e54863943a0a


No longer need to manually propagate grant select

Verified with Citus 9.2 on Hyperscale that "grant select on all tables in schema" works all by itself and does not require run_command_on_workers.

Just verified it's valid on Citus Cloud as well.


[JS] File Uploader

Title and Issue number

Title :
Issue No. :
Code Stack :
Close #<issue_no>

Checklist:

[] I have mentioned the issue number in my Pull Request.
[X] I have commented my code, particularly in hard-to-understand areas
[] I have created a helpful and easy to understand README.md
[] I have updated the Index.html file for my contribution

Have you tried running it in your local


Import data from text/copy&paste [SATURN-1036]

Add ability for users to import data to the data table from text/copy&pasting data.
Update the error messages in the modal for file and text data
Fix small issue with key not being updated so that table will re-render automatically with a new import
Added tool tip for upload button

Closing pull request as work is lower priority and requires some changes. Will open a new pull request with new changes when able to work on it.


401 response on OPTIONS

During testing this plugin we noticed having issues when trying to perform a GET Operation from Browser Chrome. Chrome was execution a preflight operation where it is first requesting OPTIONS to the endpoint before executing a GET. Since OPTIONS does not contain the password in case it was sent within the Headers, OPTIONS is refused with a 401. This can cause a CORS issue on the GET operation.
We found a workaround by explicitely excluding OPTIONS from the jwt-check. But we wonder if this should be respected within this plugin itself :).
This is the example for the workaround:
<your-domain> {
	@exclude-options {
		not method OPTIONS
	}

	route @exclude-options {
		jwtauth {
			sign_key "
<JWTAUTH>
    "
			from_header X-Api-Token
			issuer_whitelist <Your Issuer>
			audience_whitelist <audience>
		}
	}
	respond "hello world"
}

Thanks so much @inventivejon for the feedback. After reading the specification of OPTIONS requests, I decided not to handle it in this plugin. Because the logic of how to respond to an OPTIONS request can be complex at times. And it can be customized through Caddy directives.

I understand your point. Do you think it would help to add it to your README? I guess it will be something a lot of people stumble over. Or do you think the problem is solved the wrong way by the workaround?

Sure, it would be great helpful. I also experienced the same issue before which cost me some time doing research on how to deal with OPTIONS requests with Caddy.
The above workaround looks good to me. But I prefer handling OPTIONS requests separately in the same route before any other directives:
myapp.localhost {
  @options {
    method OPTIONS
  }

  route /protected/* {
    # CORS
    header {
      access-control-allow-origin      "https://myapp.localhost"
      access-control-allow-headers     "content-type,x-myapp-client"
      access-control-allow-methods     "OPTIONS,HEAD,GET,POST,PUT,PATCH,DELETE"
      access-control-allow-credentials "true"
    }

    # respond 204 to OPTIONS requests
    respond @options 204

    jwtauth {
      # ...
    }

    respond "protected resource"
  }

}

Hi @inventivejon, I'm closing this issue since I've already added a link to this issue in the FAQ section in the README.


Is there anyway to embed blocks or any quote in Dataview table?

Sorry I am sort of noob of JS.  I am not sure if this function has been realized already.
There is a file like
Step.md
---
name:  Step Function
equation: some equation
Note:   "![[Step Function#Details]]"
---

If I query like
dataview  table  name, Note from Step

It will output

or

How do I really embed it to get ![[Step Function#Details]]?

I think there was a new embed() function that hasn't been documented yet.

AFAIK in beta version the embed() function works only for images, not for files(sections/blocks). And with embed there's no need to use "!" in the value.


digitalocean module may be crashing before installation completes

Many of our UX study participants have gotten stuck at the cloud installer's "configuring SSH" stage.
We nabbed a couple of logs and what's suspicious is this line:
[Module uProxy] [2016-04-27T14:05:04.465Z] Could not deliver message, no such interface: 81

This comes immediately after:
cloud installer D [2016-04-26T20:47:20.814Z] STDOUT: CLOUD_INSTALL_PROGRESS 90

This is right around the time uProxy will be deleting the digitalocean freedomjs module - there may be a race condition here. That said, in at least one case I do see the social provider trying to connect to the server - that fails because of:
Uncaught TypeError: Cannot read property 'trim' of undefined
[2016-04-27T14:05:16.214Z] Module Failed,[object Object]
[2016-04-27T14:05:16.216Z] No registered source '35'
[2016-04-27T14:05:16.216Z] No registered source '34'
[2016-04-27T14:05:16.216Z] No registered source '33'

At which point the social provider module is totally borked.

In other words, there are a couple of things that could be going on here. I'm having trouble reproducing these errors - my best guess right now is that we should bulletproof the hand off points between the various modules.


Export packages' types under @types org

I know that every material-ui package comes with TypeScript type definitions however there is one case in which it's not perfect to have types and the package in the same place. This situation is developing a package which has @material-ui/* package as a peer dependency.
When for example, I add @material-ui/styles package as a peer dependency then installing all packages using yarn install will not put this package's code in the node_modules dir so in the end tsc and (in my case) Visual Studio Code will both complain about not being able to find a module.
The solution could be adding @material-ui/styles also as the dev dependency (it would be in both places dev and peer deps). However, in such a scenario I will have problem with using package locally (using yarn link and yarn link my-package-name).
For now, I was only able to solve this problem by adding the paths option to the tsconfig.json file which contains alias for the @material-ui/styles package which I have installed in some other place on my local machine. But that's far from perfect as some other person using this package might have it installed in other place.
So the only good solution that comes to my mind is having types being exported to an external package (@types/material-ui-styles) like it was done before introducing the @material-ui org.
What do you think about that? Do you have any other solution to this problem? I can create reproduction repository if it's really needed but I hope that I explained my issue well enough to understand the problem.

If you have a peer dependency then adding it as a dev dependency is indeed the recommended solution. Issues with linking are unrelated to typescript though. Duplication of the react module is caused by the same issue.
I don't really have a good solution for this issue. It's very specific towards your local setup. I would suggest asking on stackoverflow or our spectrum community.
Publishing a separate types package is not something we consider at the moment. Especially since we have no direct access to the @types/ namespace.

Hi,
I am building mirofrontends app with typescript.
I want to use material-ui package as external package for all of my frontends. And I have problems with types.
I share material-ui with systemJs and import maps and have issue with typescript and material-ui like "cannot find module material-ui... " cause this package is not installed directly.
Separate types package is what I am looking for. I found this: https://www.npmjs.com/package/@types/material-ui but looks old :(


[OSD][1.3.1] Add Reports and GANTT plugin

Description
Adding reports and GANNT plugin to OSD 1.3.1 release.
This can be packaged together. I trust they will succeed.
Signed-off-by: Kawika Avilla<EMAIL_ADDRESS>Related Issue
https://github.com/opensearch-project/opensearch-build/issues/1885
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #1914 (0e2a366) into main (9b2d2f4) will increase coverage by 5.39%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##               main     #1914      +/-   ##
=============================================
+ Coverage     94.60%   100.00%   +5.39%
=============================================
  Files           178         6     -172
  Lines          3633       105    -3528
  Branches         27        19       -8
=============================================
- Hits           3437       105    -3332
+ Misses          192         0     -192
+ Partials          4         0       -4

Impacted Files
Coverage Δ

src/ci_workflow/ci_check_list_source.py

...w/bwc_test/bwc_test_suite_opensearch_dashboards.py

src/assemble_workflow/bundle_url_location.py

...st_workflow/bwc_test/bwc_test_runner_opensearch.py

src/manifests/input_manifest.py

src/test_workflow/bwc_test/bwc_test_runner.py

src/manifests/test_manifest.py

src/jenkins/BuildManifest.groovy

src/ci_workflow/ci_check_package.py

src/assemble_workflow/bundle_locations.py

... and 162 more

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 9b2d2f4...0e2a366. Read the comment docs.

Waiting to merge until the build triggers with SQL Workbench kicks off.

 Merging.


⚠ Bofrost.fr has degraded performance

In 6c10ffc, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1370 ms

Resolved: Bofrost.fr performance has improved in f75771b after 9 minutes.


type checking for server and testing code

type checking for server and testing code
refine gulp tasks

Type checking heavily uses externs. Please read and try to understand the behavior of externs. We both use existing externs from public repos as well as our own (as some externs are a bit old and out-of-date). You may modify extern files when necessary. However please try to modify as little as possible the externs and only change extern in a reasonable way.
Github doesn't support multiple assignees @djasmine @Shauro. Please both confirm before I merge.

confirmed

confirmed.

More info here.


Remove study_data.sensspecdetail

🗑 Data to Remove
Remove study_data.sensspecdetail . The database has 2 distinct values for this; one is blank. The other is Results from analysis of clinical samples (50% sample).  Early stage (at minimum 12 months prior to ovarian cancer diagnosis), general population controls.. How this latter one got in there is a mystery.
Regardless, neither BMDB nor the RDF generator use it.
📝 Code Modification?

Biomarker database: no
RDF generator: no

Batch №2 since batch №1 is getting full.
Note: modifications to portal are not allowed; that means this information is in fact used and this table/column should be preserved.
💽 SQL
ALTER TABLE `study_data` DROP COLUMN `sensspecdetail`;

🩺 Testing Methodology

RDF before/after comparison: yes
In-app testing: yes
Other: N/A

🧘 Acceptance
Sign off by @mryan4 by closing this issue

No errors found during testing. This issue is closed.


Fix cases table leaks all cases if start filtering

This bug also exists on 10, so I think I will create a real bug issue from the sub task

This bug also exists on 10, so I think I will create a real bug issue from the sub task

Yes, please

Good. Can you double-check also Tasks and any other DataModel in this project? Thanks

The Task model already works with the and filters, so it should work there


k8s support matrix should be revisited pre GA

Description:
k8s support matrix should be revisited pre GA

I think we should support last 5 k8s versions, similar to what upstream Gateway API supports
but we could reduce CI time, and only run 1st, 3rd and 5th k8s version https://github.com/envoyproxy/gateway/blob/1775624114aedee731bee565a0a100eeee80fc13/.github/workflows/build_and_test.yaml#L83
wdyt @envoyproxy/gateway-maintainers ?


Improve error message readability

In case of message like 'types.SimpleNamespace' object has no attribute '_TpLv_tpec_TpExPr_selected_q_question_TPMDL_1'
we would now show  A problem occurred while resolving variable 'selected_q_question' in module '__main__'
resolves #560

Fantastic!


Bug report, fontbakery.com

Hello,
cannot test fonts online, on loading it returns the attached message
please advise, thanks

Sorry; I fixed it!


added metanoia colorscheme

I could be wrong, but I think you should generate the colorscheme.vim file too for the new color scheme (in this case, metanoia.vim) using the generate script in the repo. Also, please add VimWiki support. The highlight groups are in all the other color schemes you can just copy them from any colorscheme.yml, paste them in yours and assign the appropriate colors. The VimWiki support part is totally upto you though, I suggested it since all the other colorschemes have it 😃

Yeah, you are right, forgot about that. Generated the file and added support for VimWiki now =).

I can't see the VimWiki highlights in the yaml file... If you added them directly to metanoia.vim, when someone makes changes to the yaml files, the code that is exclusively in metanoia.vim will be erased. I did the same mistake with making the VimWiki Headers bold in the .vim files and before I realized I can do it from the yaml file, my request was merged. Now I made another PR and been waiting since :((
PS: Consider making the headers bold. I thought it looks good but it's totally upto you 😃

I added bold headers and fixed yml file. I've changed in yml don't know what happened in the last commit, I tought the VimWiki highlights were there.


disable install of pdb files for ogre, seems broken

When trying to build for Debug on Windows this caused the ogre install target to fail. I guess it has not been tested in combination with static build.
For now I think we should just disable it. Later, if we need the pdb files for debugging I can try to patch ogre (and make a pr upstream).

I can reproduce the problem with the pdb files on Windows.
The PR fixes this and looks fine to me.


add Socksify so we can use SAFE_PROXY with socks+ssh

We want to be able to use socks+ssh on vault
if it is behind a proxy and you have a jumpbox
just like credhub and bosh
https://github.com/starkandwayne/safe/issues/132

~/go_workspace/src/github.com/starkandwayne/safe → go build
# github.com/starkandwayne/safe/vendor/github.com/cloudfoundry/socks5-proxy
vendor/github.com/cloudfoundry/socks5-proxy/socks5_proxy.go:82:47: undefined: ssh.FixedHostKey
vendor/github.com/cloudfoundry/socks5-proxy/ssh_client_config.go:8:54: undefined: ssh.HostKeyCallback
vendor/github.com/cloudfoundry/socks5-proxy/ssh_client_config.go:10:10: unknown field 'Timeout' in struct literal of type ssh.ClientConfig

Doesn't quite build on my machine yet.

This does have the unintuitive quality that if the SAFE_PROXY string is malformed, it just ignores it and acts as if no proxy was specified.

We also need a way to document the usage of this command. Probably need a command that lists the available environment variables and their usage.

but we haven't done that with http_proxy var as well right?

i meant that i could not find anything in the documentation related about http_proxy.
but in any case. where do you want this documented?
in the readme or in safe itself

So, I was taking a dive into this yesterday. I want to integrate this into the targets system - basically, define a target with a proxy. I also sort of want to split the all magical SAFE_PROXY variable into HTTPS_PROXY, SSH_PROXY, and SSH_PROXY_PRIVATE_KEY variables (which would serve as overrides for the selected target). I feel there's a little too much magic in the all-in-one variable that makes its less intuitive for people who didn't write it.
The core code looks good - just a bit of shuffling. It's something I can handle when I get the time.

it seems a bit of standard practice to use the all in one variable
as seen with bosh, credhub, fly, uaa etc

Those examples are all Pivotal tooling; I would expect them to be internally consistent.  Is this a practice that other tools use?  I know most things dealing with HTTP and proxying separate out http_proxy from https_proxy and don't fallback to one or the other.


Add rust search to the repository

This is a drop-in replacement for the python-based search service.

@generall Thanks for the review. I addressed your suggestions.

I must correct myself. What I had overlooked was that there may be queries that are not in the prefix cache which will return a Not Found error, so I changed the code as per your suggestion, @generall. With that said, it might be a good idea to expand the prefix cache to contain common search terms which are not in the word list (e.g. people might search for io_uring).

@llogiq, I took a liberty to make a few changes


Works with protected Sites ?

Is it possible to get screenshots of protected sites (login/sso) with this extension ?

I think that job is done on the linkwarden app, Even though it is possible to get screenshots using the browser api, I dont know if is currently possible for me to do send the screenshots of that to linkwarden, I might as well look into it


Humble request for supportive files for V-COCO dataset

Hi YueLiao,
I'm so sorry for bothering you again. I wondered may I have the "corre_vcoco.npy" file if it's at your convenience? I found that just like the hico dataset, the program also relies on the binary mask to do the validation or testing. Annotation alone cannot bring me there. Thank you in advance!
Bozhou

You can generate the "npy" by yourself following the below rules: The "corre_.npy" is generated by the "train_.json" annotation file. If an interaction is defined between ith class object and jth verb, the value at location (i,j) of "corre_*.npy" is 1 else 0.

You can generate the "npy" by yourself following the below rules: The "corre__.npy" is generated by the "train__.json" annotation file. If an interaction is defined between ith class object and jth verb in training set, the value at location (i,j) of "corre_*.npy" is 1 else 0.

Thanks, I'll try it out. Have a good one~

You can generate the "npy" by yourself following the below rules: The "corre__.npy" is generated by the "train__.json" annotation file. If an interaction is defined between ith class object and jth verb in training set, the value at location (i,j) of "corre_*.npy" is 1 else 0.

Hi YueLiao,
Thanks again for the way you pointed out yesterday. But I'm still trying to figure the mask out. I'm a newbie at this so this might be a dumb question:
The "i-th" and "j-th" you mentioned is not likely the numerical category label of the object and the verb, right? Take the object for example, the largest categorical index is 90 but the shape of the corre_hico is <117, 80>, which means 80 masks available for object series. It makes sense since some objects are not documented and the faultage exists in those 90 indices.
So I wonder which kind of mapping I'm supposed to be using? I guess there might be a list of unordered category ids for the objects and verbs respectively, and the index of the items is the exactly "i" & "j" you mentioned.
I appreciate your help so much!

You can map the 90 categories into 80 categories through the label list in dataset/hico.py first, then generating the corre.npy

You can map the 90 categories into 80 categories through the label list in dataset/hico.py first, then generating the corre.npy

Get, thanks bro!


On Android, OpenSSL-1.0.2d is not available anymore

The current version of OpenSSL is now 1.0.2e (as of 3 December 2015). This cause an error when building for Android (cmake can't find the URL for OpenSSL-1.0.2d).

Hi,
I was wondering, is there any fix for this?
I tried updating CMakeLists.txt with openssl URL pointing to to 1.0.2e with relevant url hash, but the build fails (see output below). Any suggestions?
....
Scanning dependencies of target OPENSSL
[  0%] Creating directories for 'OPENSSL'
[  0%] Performing download step (download, verify and extract) for 'OPENSSL'
-- downloading...
     src='http://www.openssl.org/source/openssl-1.0.2e.tar.gz'
     dst='~/AWS-CPP-ANDROID-BUILD/OPENSSL-prefix/src/openssl-1.0.2e.tar.gz'
     timeout='none'
-- downloading... done
-- verifying file... done
-- extracting...
-- extracting... done
[  0%] No patch step for 'OPENSSL'
[  0%] Performing update step for 'OPENSSL'
[  0%] Performing configure step for 'OPENSSL'
-- Configuring done
-- Generating done
-- Build files have been written to: ~/AWS-CPP-ANDROID-BUILD/OPENSSL-prefix/src/OPENSSL-build
[  0%] Performing build step for 'OPENSSL'
Scanning dependencies of target crypto
[  0%] Building C object crypto/CMakeFiles/crypto.dir/cryptlib.c.o
In file included from ~/aws-sdk-cpp-master-Android/openssl/crypto/cryptlib.h:65:0,
                 from ~/aws-sdk-cpp-master-Android/openssl/crypto/cryptlib.c:117:
~/aws-sdk-cpp-master-Android/openssl/crypto/../e_os.h:64:28: fatal error: openssl/e_os2.h: No such file or directory
 # include <openssl/e_os2.h>
                            ^
compilation terminated.
make[5]: *** [crypto/CMakeFiles/crypto.dir/cryptlib.c.o] Error 1
make[4]: *** [crypto/CMakeFiles/crypto.dir/all] Error 2
make[3]: *** [all] Error 2
make[2]: *** [OPENSSL-prefix/src/OPENSSL-stamp/OPENSSL-build] Error 2
make[1]: *** [CMakeFiles/OPENSSL.dir/all] Error 2
make: *** [all] Error 2

I will try and take at look this today or tomorrow

The best fix for this would involve switching to a direct git clone at a version tag of openssl, but there are substantial structural differences between what's in git at the 1.0.2d tag and what is in the release tarball on their website.  Since the build process was done from the perspective of the archive, this is a non-trivial amount of work.  Until that gets fixed, you can at least bypass the problem by switching the URL in the ExternalProject_Add to:
URL https://www.openssl.org/source/old/1.0.2/openssl-1.0.2d.tar.gz

Yes, I did that for the time being. Thanks.

I think Bret fixed this in the HEAD of mainline. Can you check that the value is correct for you?

This has been updated and pushed out. Closing. If you still have an issue let feel free to reopen issue and let us know.

MinGW + Cmake on win7 :
D:\software\OpenSSL-CMake\build>make
Scanning dependencies of target crypto
[  0%] Building C object crypto/CMakeFiles/crypto.dir/cryptlib.c.obj
In file included from D:\software\OpenSSL-CMake\crypto\cryptlib.h:65:0,
from D:\software\OpenSSL-CMake\crypto\cryptlib.c:117:
D:/software/OpenSSL-CMake/e_os.h:64:28: fatal error: openssl/e_os2.h: No such file or directory
include <openssl/e_os2.h>
                        ^

compilation terminated.
crypto\CMakeFiles\crypto.dir\build.make:62: recipe for target 'crypto/CMakeFiles/crypto.dir/cryptlib.c.obj' failed
make[2]: *** [crypto/CMakeFiles/crypto.dir/cryptlib.c.obj] Error 1
CMakeFiles\Makefile2:84: recipe for target 'crypto/CMakeFiles/crypto.dir/all' failed
make[1]: *** [crypto/CMakeFiles/crypto.dir/all] Error 2
Makefile:82: recipe for target 'all' failed
make: *** [all] Error 2
How can I fix it?


commenting out uipath

JIRA TICKET
(https://tools.hmcts.net/jira/browse/DTSPO-13652)
Change description
Commenting out file uipath
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

No longer needed


Output required in JSON format

It would be a lot better if the output would be in JSON format. I think it would be helpful for a lot of people. Hope you guys can make it happen.

Did not understand this one. Please be clearer when describing your requirements.


1577 spdxlicense generate

Summary
Update the license_list.go to have more permissible inputs for greater SPDXID matching.
The spdxlicense package contains the ID method which interacts with the generated file license_list.go
The current implementation contains - in the keys. This PR removes these - and sanitizes the inputs so that we can match on a wider range of inputs found in the wild.
spdxlicense.ID has also been changed to only consider the generated list and return if a value exists
The logic of how this information is encoded has been temporarily been moved to the format helpers.
Note this is temporary. #1554 will be used to update our license parsing logic so that license creation is done at the same time as package creation.
In a follow up PR encoders or other middle layers of syft should no longer have any concerns surrounding updating/finding the correct SPDXID or Expression as this will be done when packages are created at the cataloger level.
Example:
GPL3 gpl3 gpl-3 and GPL-3 can all map to GPL-3.0-only
By moving all strings to lower and removing the - we're able to return valid SPDX license ID for a greater diversity of input strings.

Things to look at in the AM:

[ ] bzip2-1 <-- This test case is odd when removing the -. We might have to make the capture groups a little better to handle this case
[ ] Cyclonedx License regression for made-up
[ ] CycloneDX test updates with name carried through
[ ] Encoder updates with new license updates as a consequence of ID change
[ ] CLI regressions


[0.3/dx12] use explicit external pass barrier access

Improves (but doesn't fix yet) #3009
PR checklist:

[ ] make succeeds (on *nix)
[ ] make reftests succeeds
[ ] tested examples with the following backends:
[ ] rustfmt run on changed code

The basic thing it does is just using the access flags provided in the explicit external barriers, if any, instead of empty().
@zakorgy - please check it out and tell if it's helping or not

@kvark unfortutnatelly we still have errors, and there are new ones: https://gist.github.com/zakorgy/89fc1d62e9cf61c17d329c9b0d5d06d8
Maybe D3D12 doesn't like the fact that the src stage is D3D12_RESOURCE_STATE_DEPTH_WRITE |  D3D12_RESOURCE_STATE_RENDER_TARGET with this solution in my case?

uh oh. I should have waited for your test before publishing.


[1.16.2] Installing this mod with 'KubeJS-Fabric' causes custom recipes not to load properly.

Title^
Adding recipes through KubeJS with this mod installed makes the recipes added not work,  but after a '/reload' it works like 25% of the time.
Also the recipes added gets added shown twice in REI. That is when a recipe added do work which it likely won't.
It's really hard to explain this but essentially when I load into a world and check for a custom recipe, it sometimes doesn't load. And if it loads (which is rare), it gets shown twice in REI. But then after a '/reload' or relog the recipe is gone again.
This doesn't happen when the mod is removed.
Meaning everything works as intended if this mod is removed.
Example of double REI recipes:

Versions:
Minecraft: 1.16.2
Fabric API: 0.19.0+build.398-1.16
IndRev: 1.6.2-BETA
KubeJS: 1.4.1
Other information:
This is reproducable for me on an empty instance with only Fabric API, Fabric Language Kotlin & KubeJS. And ofcourse Industrial Revolution. With a custom recipe.
- Gbergz

Is there any log output that could be related to this?

I've been sitting trying to figure out what could be causing it for like 3 hours now. Logs say nothing. I've gotten help by AK9 and we're both unsure. Until I decided to remove a mod 1 by 1 until the problem went away.

Can you please test if the issue is fixed with this version?

Just tried it, same issue persists unfortunately.

In your log, are you seeing errors similar to these? https://github.com/KubeJS-Mods/KubeJS-Fabric/issues/3
But pertaining to industrial revolution?

In your log, are you seeing errors similar to these? KubeJS-Mods/KubeJS-Fabric#3
But pertaining to industrial revolution?

Yes but only this line:
[ForkJoinPool.commonPool-worker-3/WARN]: Parsing error loading recipe techreborn:centrifuge/calciumcarbonate_cell net.minecraft.class_151: Non [a-z0-9_.-] character in namespace of location: {fluid:"techreborn:calcium_carbonate",holder:"techreborn:cell"}

I tried tracking this but could not find anything. This is possibly a KubeJS issue, I'd recommend opening an issue on their repository. If it ends up being an issue on my end, I'll happily fix it!


$this.dialog.confirm()

I never get 'confirmed' to fire, escape keypress or enter keypress has the same result which toggles onCancel for me.
    async toDelete (resource) {
      this.$dialog.confirm({
        message: 'Are you sure?',
        onCancel: () => {
          console.log('cancelled')
        },
        onConfirm: () => {
          console.log('confirmed')

It's because the cancel button has the focus, if you tab -> enter it'll work.
I don't know why it doesn't have the focus style tho, gonna work on that.


package causes client errors

hey
I added your package to my project and configured Meteor.settings to include a GA key
However, once the package is added, my app is broken
I get this warning in console (I have iron:router):
Route dispatch never rendered. Did you forget to call this.next() in an onBeforeAction?
I know your code calls this.next(), but something seems to be broken
Anyway, the program execution gets cut and its indication is parts of my app produce errors (unknown variables etc)
I don't really know how we can help each other with that, as I can't isolate the problem
UPDATE:
the minute I opened the issue, I think I figured where the problem is...
in my project, I use a transform function for my users collection, which leads to various code executions
My guess would be your package runs first, calling Meteor.user()  and in turn invokes the transform function. As your package runs before other init code is executed, this leads to complications
The workaround may not be trivial (tinkering with load order, if possible at all), so I think it should be made clear to other people

@ChenRoth Thanks for the update. Were you able to pin-point specific interaction?

yeah, it's basically what I presumed - your package runs a Meteor.user()
call at a very early stage, which invokes my custom transform function on
the users collection.
not sure if you're up for this change, but I guess a viable solution would
be to allow optional override of the transform function (or pass a null
argument to bypass it altogether)
On Wed, May 27, 2015 at 8:09 AM tsemana<EMAIL_ADDRESS>wrote:

@ChenRoth https://github.com/ChenRoth Thanks for the update. Were you
able to pin-point specific interaction?
—
Reply to this email directly or view it on GitHub
https://github.com/okgrow/analytics/issues/28#issuecomment-105758925.


Added Marathi and Hindi languages! 😊

Hey buddy, just added Marathi  and Hindi  languages! Checked the correctness twice and will keep updating it for  improvements.
Please add these in next release!
🙏

Thank you for your contributions, will merge it!

Thanks buddy! 🙏


Play store downloads stuck at "Pending..."

I've tried several images including the process of this repo, however once I'm logged into google account and open play store, I can't download any apps. They just get stuck before downloading with "Pending", eventually failing with "Can't download " "Try again, and if it still doesn't work, see common ways to fix the problem" "Send feedback "Got it"
Anyone else seeing this issue?

Ok so I found out it was my DNS server which was blocking resolution of .gvt..com, which in turn broke play store updates... Not a problem with this project and won't affect other users so I'm closing this issue. Sorry for false alarm. Thank you


Nested named slots not rendering

Version
3.0.5
Reproduction link
https://codepen.io/mirkojov/pen/VwmwPYe
Steps to reproduce
If component named slot is a child component named slot, it does not render.
However, sub-component default slot is rendered. This can be seen in the example if the following section is removed:
<template v-slot:foobar>
Grandchild #foobar
</template>

What is expected?
Same behavior as with default nested slot.
What is actually happening?
Templates not rendering.

@vue/compiler-sfc throws error: Codegen node is missing for element/if/for node. Apply appropriate transforms first.

Hey, is this what you want? https://codepen.io/hcysunyang/pen/GRNRWJB

Hey, is this what you want? https://codepen.io/hcysunyang/pen/GRNRWJB

Yes, this is what I expected. Probably not possible without explicitly specifying slots in the child template?

Yes, this is what I expected. Probably not possible without explicitly specifying slots in the child template?

Yes, you should specify slots explicitly in the child component, and strictly speaking, your usage is wrong, so I close this.

Yes, you should specify slots explicitly in the child component, and strictly speaking, your usage is wrong, so I close this.


[WIFI-2696] Docker compose to include ucentralgw ui

Messed up branching, will create a new branch and PR.


WARNING: erroneous pipeline: no element "kvssink" [NVIDIA Jetson Tx2]

Hi,
I am trying to use KVS on Jetson TX2 with JetPack 4.2. I compiled KVS producer library as instructed and copied files to /usr/lib/aarch64-linux-gnu/.
 Then I rebooted TX2 and it does not work anymore.
The only line of error message is
WARNING: erroneous pipeline: no element "kvssink"

But libgstkvssink.so exists under /usr/lib/aarch64-linux-gnu/gstreamer-1.0.
Here is the gst-inspect-1.0 output.
$ gst-inspect-1.0 kvssink

(gst-plugin-scanner:7820): GStreamer-WARNING **: 16:41:50.069: Failed to load plugin '/usr/lib/aarch64-linux-gnu/gstreamer-1.0/libgstkvssink.so': /usr/lib/aarch64-linux-gnu/gstreamer-1.0/libgstkvssink.so: undefined symbol: _ZN3com9amazonaws7kinesis5video18CredentialProvider12getCallbacksEP17__ClientCallbacks
No such element or plugin 'kvssink'

How can i fix this error?
Thank you!

Hi @ac4289 ,
You need to add directory of libgstkvssink.so to environment variable GST_PLUGIN_PATH or move it to the directory where gstreamer plugins are in.

It's complaining about not be able to find symbol in libproducer.so. the plugin is dynamically linked to the libproducer.so. Can you do export LD_LIBRARY_PATH=/path/to/dir for the directory that has libproducer.so?

It's complaining about not be able to find symbol in libproducer.so. the plugin is dynamically linked to the libproducer.so. Can you do export LD_LIBRARY_PATH=/path/to/dir for the directory that has libproducer.so?

Thank you for helping!
I tried this:
export LD_LIBRARY_PATH=/usr/lib/aarch64-linux-gnu/gstreamer-1.0

Still does not work.

is that where your libproducer.so is?

is that where your libproducer.so is?

Yes

what about libcproducer.so?

libcproducer.so is under /usr/lib/aarch64-linux-gnu/

can you move it to /usr/lib/aarch64-linux-gnu/gstreamer-1.0 too? and do export LD_LIBRARY_PATH=/usr/lib/aarch64-linux-gnu/gstreamer-1.0

LD_LIBRARY_PATH=/usr/lib/aarch64-linux-gnu/gstreamer-1.0

Does not work. T_T

just wanna confirm libcproducer.so and libproducer.so are all in /usr/lib/aarch64-linux-gnu/gstreamer-1.0?

just wanna confirm libcproducer.so and libproducer.so are all in /usr/lib/aarch64-linux-gnu/gstreamer-1.0?

Yes

Hi @ac4289  ,

It worked right after compilation finished.

Given that it worked first time, could you please run the source set_kvs_sdk_env.sh and see if that helps.

Hi @ac4289 ,

It worked right after compilation finished.

Given that it worked first time, could you please run the source set_kvs_sdk_env.sh and see if that helps.

No I already tried that. I just reflashed TX2 with JetPack 4.2. I'll see how it goes.

Ok, reflashed TX2 works with KVS. The difference is that the old OS had TVM installed which requires a change of Python path in .bashrc.
Thank you again for your help!

I'm getting the same error: no element "kvssink"
plugin scanner says: Failed to load libgstx264.so, undefined symbol x264_deblock_h_chroma_422_intra_neon
Did you have this issue? Do you know how to fix it? Thanks.

Once you've successfully built the plugin, you should set the paths per: https://github.com/awslabs/amazon-kinesis-video-streams-producer-sdk-cpp/blob/master/install-instructions-macos.md#setting-the-environment-variables-for-library-path


fix cdpath bug

Fixes the instructions printed by the script so that if you create a new app with create-react-app foo, it will suggest getting there with cd foo rather than cd /some/big/long/path/foo.
This will require a new global-cli deploy to work, but it's backward compatible so that the new init script works with the old global-cli (and just is buggy in the same way the current one is).

One small question, doesn't the process.cwd() change the same way every time? Couldn't we just undo that change by joining it where the path.join(process.cwd(), appName) check is?

It changes the same way every time in the sense that process.chdir always turns the working directory into the directory where the app is created. But that means the original working directory is lost, and the original working directory is the one that the developer is going to be cd'ing from.
For example, if you start in /Users/bob/ and run create-react-app my-app, then process.cwd() will be /Users/bob/my-app in the init script. But if you start in /Users/bob/stuff and run create-react-app ~/my-app, then process.cwd() will also be /Users/bob/my-app in the init script. But in the former case you want to tell the user to cd my-app and in the latter case you want to tell the user to cd /Users/bob/my-app.

Ahh, fair enough! :+1: This LGTM then!


Segmentation Fault (core dumped)

Problem encountered on https://dotnet.microsoft.com/en-us/learn/dotnet/hello-world-tutorial/create
Operating System: Linux (Ubuntu 22.04.1 LTS)
I am very new to programming and was trying to follow the tutorial on the above page for C# when I received an error message in the console "segmentation fault (core dumped)", and it couldn't find the file/directory...
MyComputer:~ $ dotnet new console -o MyApp -f net6.0
Segmentation fault (core dumped)
MyComputer:~ $ cd MyApp
bash: cd: MyApp: No such file or directory

HI @bburch1985. Sorry to hear you're having issues.
Did you install using snap by any chance?


Implement malformed CSV screening in back-end

Ensure incorrectly formatted CSV files are filtered out properly.
Attempting to fetch a username/password pair from this event gives a success message on the HTML but gives no email to the team captain(s)

How exactly is the csv being formed that creates this error?


Add /vinbud/ endpoint.

Scrape the vínbúð opening times. Maybe this should be it's own npm module.

I was hoping the tests were just failing on my machine. But that's not working and I need to move this into it's own module so I'm closing this.


HA 2024.1.5 upgrade, now the sonoff integration errors, cannot uninstall

Had the latest version of the integration installed on 2023.11 Home Assistant and all working fine.  Upgraded to 2024.1.5 and now I see the integration has this error on restart of HA:
Logger: homeassistant.setup
Source: setup.py:251
First occurred: January 29, 2024 at 4:47:07 PM (1 occurrences)
Last logged: January 29, 2024 at 4:47:07 PM

Setup failed for custom integration 'sonoff': Unable to import component: Exception importing custom_components.sonoff
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/loader.py", line 822, in get_component
    ComponentProtocol, importlib.import_module(self.pkg_path)
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1204, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1176, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1147, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 690, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 940, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/config/custom_components/sonoff/__init__.py", line 28, in <module>
    from .core import devices as core_devices
  File "/config/custom_components/sonoff/core/devices.py", line 102, in <module>
    PULSEWIDTH = spec(XPulseWidth, param="pulseWidth", enabled=False)
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/sonoff/core/devices.py", line 88, in spec
    return type(cls.__name__, (cls,), {**cls.__dict__, **kwargs})
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 408, in __init__
    wrap_attr(cls, property_name)
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 380, in wrap_attr
    raise TypeError(f"Can't override {attr_name} in subclass")
TypeError: Can't override _attr_native_max_value in subclass

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/setup.py", line 251, in _async_setup_component
    component = integration.get_component()
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/loader.py", line 830, in get_component
    raise ImportError(f"Exception importing {self.pkg_path}") from err
ImportError: Exception importing custom_components.sonoff

I cannot even delete it to install it again.  When I try and add the integration again, i get his error right away:
Config flow could not be loaded: {"message":"Invalid handler specified"}
When i delete the integration, I restart HA, and this long error above still appears in the notifications.

Logger: homeassistant.loader
Source: loader.py:842
First occurred: 17:49:32 (1 occurrences)
Last logged: 17:49:32
Unexpected exception importing platform custom_components.sonoff.config_flow
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/loader.py", line 842, in get_platform
cache[full_name] = self._import_platform(platform_name)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/loader.py", line 859, in _import_platform
return importlib.import_module(f"{self.pkg_path}.{platform_name}")
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/importlib/init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "", line 1204, in _gcd_import
File "", line 1176, in _find_and_load
File "", line 1126, in _find_and_load_unlocked
File "", line 241, in _call_with_frames_removed
File "", line 1204, in _gcd_import
File "", line 1176, in _find_and_load
File "", line 1147, in _find_and_load_unlocked
File "", line 690, in _load_unlocked
File "", line 940, in exec_module
File "", line 241, in _call_with_frames_removed
File "/config/custom_components/sonoff/init.py", line 28, in
from .core import devices as core_devices
File "/config/custom_components/sonoff/core/devices.py", line 104, in
PULSEWIDTH = spec(XPulseWidth, param="pulseWidth", enabled=False)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/sonoff/core/devices.py", line 90, in spec
return type(cls.name, (cls,), {**cls.dict, **kwargs})
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 408, in init
wrap_attr(cls, property_name)
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 380, in wrap_attr
raise TypeError(f"Can't override {attr_name} in subclass")
TypeError: Can't override _attr_native_min_value in subclass

I was able to resolve this by exec'ing into my docker container, deleting the dir under ~/config/custom_components/sonoff, then when I restarted HA, I was able to delete the integration.  Then I installed it fresh from HACS and I'm back up and running!


Don't require simulation.progress to subtype Function

Ah interesting yeah a callable object would have worked as well. We should change
https://github.com/climate-machine/Oceananigans.jl/blob/7b5cc97c18c8cd7dcc7a866312630df511735189/src/Simulations.jl#L158
to
iscallable(f) = !isempty(methods(f))
...
iscallable(sim.progress) && sim.progress(sim)

See: https://discourse.julialang.org/t/how-to-determine-if-an-object-is-callable/2897
Although to have a callable object with parameters you'd have to define an new type right? (As in https://docs.julialang.org/en/v1/manual/methods/#Function-like-objects-1)
Either way, would be good to move the discussion to an issue on getting rid of model.parameters.
Originally posted by @ali-ramadhan in https://github.com/climate-machine/Oceananigans.jl/pull/639#issuecomment-590149784

My question: what are we defending against with this check? If we only want to ensure that we don't throw an error is progress===nothing, we can simply check for that case and otherwise call progress(sim).

Yeah I guess we want to be able to call progress(sim), doesn't matter what the type of progress is although most likely it'll be a Function or callable object. So I think we just want to blacklist anything we cannot call.
So I think your suggestion makes sense, something like:
iscallable(sim.progress) && sim.progress(sim)

which will work if progress = nothing as
julia> iscallable(nothing)
false

and if the function signature is wrong a reasonable error will be thrown.

Maybe just run sim.progress(sim) without any checking? If the user provided some progress object that can't be called, the line will error and provide a nice stack trace so that the user can figure out what went wrong.

Sounds like a good idea. We'll just have to change the default from nothing to a function like default_progress(args...) = nothing or nothingfunk(args...) = nothing.


Icon state color

Hi, got a small question for you guys
I'm pretty new with HA, I checked the box to color the icon depending on state, but nothing happens when the state changes, I'd like to set the state value to color itself when it's "today"
Do I have to modify it in lovelace directly, or in the costumization.yaml?
Thanks for the help

Hi. I think that out of the box, HA only colorizes binary sensors (that turn on/off). And this one is not that.
So you have to either tweak it in Lovelace (I use a picture entity card for that, or you can play with the styles). Or use the custom garbage collection card mentioned in the readme.


Write cypress test for typing characters into numeric fields

Description
A side effect of our number formatting seems to be that typing characters into the field is impossible. The number formatting rules seem to discard any invalid characters (even minuses? maybe that's a bit much). Right now our cypress test for number formatting test that the minus disappears, but it does not explicitly test the expected formatted value.
We should improve this test to make testing for the number formatting more explicit. See tasks for some suggestions on what to test.
Tasks

[ ] Make sure it is impossible to type regular characters, and that these are stripped away from the field value. This behavior should perhaps also be implemented for fields that are supposed to be numeric according to the JsonSchema, but are missing number formatting. Consider UX as well, as we might want a dismissable soft validation if the user continues trying to type text into numeric fields.

[ ] Consider if we should broaden this issue to implement generic support for limiting the possible input a user can type into a field

[ ] Test various permutations of trying to break the formatting by selecting and overwriting characters produced by the formatting operation (i.e. selecting and trying to change the prefix/suffix/thousands separator/comma). See #294.

Tip: It could be useful to run cy.interceptLayout() and refresh the page to intercept and manipulate the formatting rules in this test, to make sure we cover more use cases.

Closing this, consider these tasks moved to #288 (no need for this duplicate)


App stop working during scan

App hangs on Kokomi's name. This happened to me as well.

Error Log
Could not determine character's name or element
Object reference not set to an instance of an object.
at InventoryKamera.CharacterScraper.ScanCharacters() in B:\Projects\VisualStudio\InventoryKamera\InventoryKamera\InventoryKamera\CharacterScraper.cs:line 23
at InventoryKamera.InventoryKamera.GatherData(Boolean[] formats, Boolean[] checkbox) in B:\Projects\VisualStudio\InventoryKamera\InventoryKamera\InventoryKamera\InventoryKamera.cs:line 94
at InventoryKamera.Form1.<>c__DisplayClass21_0.<StartButton_Clicked>b__0() in B:\Projects\VisualStudio\InventoryKamera\InventoryKamera\InventoryKamera\Form1.cs:line 207

Error Log
Could not determine character's name or element
Object reference not set to an instance of an object.
at InventoryKamera.CharacterScraper.ScanCharacters() in B:\Projects\VisualStudio\InventoryKamera\InventoryKamera\InventoryKamera\CharacterScraper.cs:line 23
at InventoryKamera.InventoryKamera.GatherData(Boolean[] formats, Boolean[] checkbox) in B:\Projects\VisualStudio\InventoryKamera\InventoryKamera\InventoryKamera\InventoryKamera.cs:line 94
at InventoryKamera.Form1.<>c__DisplayClass21_0.<StartButton_Clicked>b__0() in B:\Projects\VisualStudio\InventoryKamera\InventoryKamera\InventoryKamera\Form1.cs:line 207

Yep same here.

I'm having a similar issue while scanning(1.1), it stop halfway through without any error log so I tried a previous version (1.0) but it stopped just before saving the file. I tried this over on other PCs cause I thought maybe something was wrong with mine but had the same issue with both versions. Just to be clear I previously used 1.0 and it worked just fine IDK why it's not working now. Please fix it I love this tool it made my life much easier. I really appreciate what yall are doing.

Hi there. There was a bug when IK tried to scan characters with names that wrapped lines in the top left corner. If this bug happened before 4 characters had been scanned, another bug occurred. These issues are hopefully resolved in the v1.1.1 prerelease.  If they have been, please let us know so we can close this issue.


Add target templates

Resolves #285

This is a neat feature. I’ll give it a run tomorrow morning. It should shrink a lot of our targets.

Tested this in a fairly complicated scenario, works like a charm:
targetTemplates:
  BaseApp:
    type: application
    platform: iOS
    sources:
      #Sources
      - path: Sources/N42Utils
      - path: Sources/AppTemplate/SourceryProtocols
      - path: Sources/AppTemplate/Base
      #Folder And File References
      - path: Carthage/Checkouts
        buildPhase: none
        type: folder
    prebuildScripts:
      - script: sh Resources/BuildScripts/Local/Timestamp.sh > $SRCROOT/.timestamp
                echo "Marked timestamp of run"
        name: Mark Timestamp
      - script: sh Resources/BuildScripts/Phases/iOSPrescripts.sh
        name: iOSPrescripts
      - script: sh $SRCROOT/Resources/Buildscripts/Local/MarkTimestamp.sh "Prescripts"
        name: Timestamp Prescripts
    postbuildScripts:
      - script: sh $SRCROOT/Resources/Buildscripts/Local/MarkTimestamp.sh "Timestamp Build Copy Link Carthage"
        name: Timestamp Build Copy Link Carthage
      - script: sh Resources/BuildScripts/Phases/iOSPostscripts.sh
        name: iOSPostscripts
      - script: sh $SRCROOT/Resources/Buildscripts/Local/MarkTimestamp.sh "Postscripts"
        name: Timestamp Postscripts
    attributes:
      ProvisioningStyle: Manual
      DevelopmentTeam: "$(N42_RELEASE_PROVISIONING_DEVELOPMENT_TEAM)"
    settings:
      base:
        PRODUCT_BUNDLE_IDENTIFIER: "$(N42_BUNDLE_IDENTIFIER)"
        PRODUCT_NAME: $(N42_PRODUCT_NAME)
        INFOPLIST_FILE: "Sources/AppTemplate/Base/Supporting Files/Info.plist"
        # SWIFT_OBJC_BRIDGING_HEADER: "Sources/Base/Supporting Files/Bridging-Header.h"
        CODE_SIGN_STYLE: Manual
        DEVELOPMENT_TEAM: "$(N42_RELEASE_PROVISIONING_DEVELOPMENT_TEAM)"
      configs:
        Release:
          CODE_SIGN_IDENTITY: $(N42_RELEASE_PROVISIONING_PROFILE_SIGNING_IDENTITY)
          CODE_SIGN_IDENTITY[sdk=iphoneos*]: $(N42_RELEASE_PROVISIONING_PROFILE_SIGNING_IDENTITY)
          PROVISIONING_PROFILE_SPECIFIER: $(N42_RELEASE_PROVISIONING_PROFILE)
        Debug:
          CODE_SIGN_IDENTITY: iOS Developer
          PROVISIONING_PROFILE_SPECIFIER: $(N42_INTERNAL_PROVISIONING_PROFILE)
    scheme:
      testTargets:
        - AppTemplateTests
        - AppTemplateUITests
    dependencies:
      - carthage: GRDB
      - carthage: RxGRDB
      - carthage: Alamofire
      - carthage: Differentiator
      - carthage: Dip
      - carthage: Moya
      - carthage: ObjcExceptionBridging
      - carthage: Result
      - carthage: RxBlocking
      - carthage: RxCocoa
      - carthage: RxDataSources
      - carthage: RxGesture
      - carthage: RxMoya
      - carthage: RxOptional
      - carthage: RxSwift
      - carthage: RxUserDefaults
      - carthage: TinyConstraints
      - carthage: XCGLogger
      #Crashlytics
      - framework: Carthage/Build/Crashlytics.framework
        embed: false
      - framework: Carthage/Build/Fabric.framework
        embed: false
      - target: KitTemplate
targets:
  AppTemplate:
    templates: [BaseApp]
    sources:
      - path: Sources/AppTemplate/Release
  AppTemplateDebug:
    templates: [BaseApp]
    settings:
      base:
        SWIFT_ACTIVE_COMPILATION_CONDITIONS: DEBUG_MODULES
    sources:
      - path: Sources/AppTemplate/Debug

@yonaskolb This feature will definitely be featured in the successor to https://www.number42.de/blog/2018/07/24/xcodegen-article.html


we shouldn't need an env var with default value in ConfigMap

The default value is always the same, right? We should be able to hardcode it. The less env vars we need, the better.

Fixed in #11.


Changes in make_wheel.sh to create wheels that are pip installable. Fixes #927

names that wheel file with manylinux1 instead of linux

One minor comment, otherwise looks good!


[PBDEV-1650] fix CRUSH Map update issue by adding waitForOSDsUp()

Checklist:

[ ] Commit Message Formatting: Commit titles and messages follow guidelines in the developer guide.
[ ] Reviewed the developer guide on Submitting a Pull Request
[ ] Pending release notes updated with breaking and/or notable changes for the next minor release.
[ ] Documentation has been updated, if necessary.
[ ] Unit tests have been added, if necessary.
[ ] Integration tests have been added, if necessary.

코멘트대로 정리 필요

처음부터 CRUSH Map을 virtual domain으로 적용시킬 수 있게 되어 해당 PR은 필요 없으므로 close 하였습니다