Issue #11 updated RecordType details by Name to Developer Name

updated getRecordTypeInfosByName to getRecordTypeInfosDeveloperName to avoid issue with names in different languages.
#11

The only thing that scares me about this is that it changes the behavior of a method that might already be used by legacy code. Any code that already uses getRecordTypeId() will break with this change, as it will now return null when it used to return an Id. Is this an intended consequence? And if it is, we need to put out a notice that code refactoring will be needed when upgrading CSUtils.

I agree with you @btamburrino but I thought it will not have impact on existing builds. Do you want me to add this as a new method or add note on the read me ?.

It will have an impact on existing builds. Right now, existing builds are passing in the Label of the Record Type to this method: Id recordTypeId = CSUtils.getRecordTypeId('Account', 'Business Account'); This would return the Id of the Business Account Record Type.
If the CSUtils class in that repo is updated with this one, suddenly that call returns null, because now it wants the Developer Name. The code would have to be changed to Id recordTypeId = CSUtils.getRecordTypeId('Account', 'Business_Account'); for it to work.
There is no right or wrong answer to your question. It is probably best, in the long run, to do what you did here with an update in the README regarding legacy code. It's preferable to creating a new method or overloading this one with an optional Boolean parameter. It is rare that we update the CSUtils class in an existing codebase so I might be overstating the issue. Thoughts, @ktatta?

There is generally no reason to upgrade CSUtils class but in a situation where that happens in an existing code base, it is a deliberate decision. However, I do agree that it is best to do the following: 1) create another implementation getRecordTypeIdByDevName

I like breaking things, so take my suggestion with a big grain of salt. I'm sad that Apex's concept of deprecation is mostly for managed packages.
I think getRecordTypeId should be renamed getRecordTypeIdByLabel and getRecordTypeIdByDevName be added. This will break getRecordTypeId if blindly copied. Add a comment to the old method stating that the name has been intentionally changed and that getRecordTypeIdByDevName should be used. This lets people who want to update CSUtil make a choice, rename the method back or do the right thing and move to using DevName.


Provide Progress of the encoding progress.

hi @qanuj what are your plans for this pr exactly?

When I record a video it has few problems. e..g the duration is missing and shows N/A which means shows no progress in % of encoding completed (added onProgress event as well). This fixes both. I am bit torn on If I should have the getSeekableBlob in this  or let client implement it.
I did try that and since there is no access to update the capturedData with modified seekableData, I feel its best to keep this process in convertEngine with ability to configure via options plus check if the function (getSeekableData) is available.
I did test this in my project and seems to be working alright.

the ffmpeg.wasm plugin was upgraded to use v0.12 and they now provide progress events:
// Listen to progress event instead of log.
ffmpeg.on('progress', ({ progress, time }) => {
    messageRef.current.innerHTML = `${progress * 100} % (transcoded time: ${time / 1000000} s)`;
});

if we add progress events for converters I would suggest to start there.


Fuzz test for deleting rows from tables

What type of enhancement is this?
Tech debt reduction, Other
What does the enhancement do?
Looks like we missing the fuzz test for deleting rows from tables.
It's an important API that should be carefully tested.
Implementation challenges
@WenyXu Could you provide some docs or code to help developers that would be interested in implementing this feature? Thank you.

Upon observing GrepTime uncovering code issues through fuzzing, I am intrigued. With some prior experience in fuzzing, I have been contemplating integrating fuzzing tests into Kvrocks. Here are my inquiries:

Fuzzing essentially tackles a vast search problem. Many fuzzers pinpoint code issues by randomly generating inputs that meet specific criteria, such as csmith and sqlsmith. However, not all randomly generated inputs are necessarily meaningful; some may hold more significance and warrant further exploration beforehand. Consequently, certain fuzzing tools steer their input generation process using metrics like changes in code coverage: the notion being that if a newly created input can traverse previously unexecuted code segments, it likely holds greater value. This form of fuzzing is commonly known as coverage-guided mutation-based fuzzing, exemplified by afl++ and libfuzzer. Will GrepTime adopt this approach?

Another focal point of fuzzing involves identifying dependable test oracles. Numerous fuzzer tools employ differential testing: for instance, supplying inputs separately to MySQL and MariaDB then contrasting the outcome disparities between them. Nonetheless, an anticipated challenge lies in variations across different DBMS implementations potentially yielding numerous false positives (even among MySQL and databases asserting compatibility with MySQL). Addressing this issue might necessitate substantial software engineering efforts to alleviate or discover innovative workarounds like sqlancer. What strategy does GrepTime intend to pursue here?

A fuzzer could unearth a multitude of errors; however, distinguishing genuine findings from false positives within extensive fuzzer outputs poses a challenge. Moreover, many errors identified by multiple fuzzer tools may stem from the same root cause—manually sifting through tens of thousands of outputs to pinpoint dozens of distinct bugs could prove arduous. At this juncture, we might require additional tools for filtering and simplifying lengthy SQL inputs produced by these utilities. Does GrepTime harbor any distinctive approaches to tackle this predicament?

I am a beginner in Rust and am quite interested in greptime. Can I contribute to this?

I am a beginner in Rust and am quite interested in greptime. Can I contribute to this?

Hi @hanxuanliang, thank you for your interest. Fuzz tests could be a little tricky. The first challenge is how to validate deleted values. Do you have any ideas?

I have reviewed the current test-fuzz structure, and the general approach is as follows:

A new fuzz_delete target needs to be added.
Define a FuzzInput to serve as the input for the fuzz test.
Implement a delete_expr, which defines the expressions required for the delete operation.
Finally, assemble execute_delete, which completes the following sequence: create-table -> insert-rows -> delete-rows -> drop-table.

I have reviewed the current test-fuzz structure, and the general approach is as follows:

A new fuzz_delete target needs to be added.
Define a FuzzInput to serve as the input for the fuzz test.
Implement a delete_expr, which defines the expressions required for the delete operation.
Finally, assemble execute_delete, which completes the following sequence: create-table -> insert-rows -> delete-rows -> drop-table.

Cool, almost there.

We can randomly invoke the compact/flushing of the target table(You can follow the instructions in this issue: https://github.com/GreptimeTeam/greptimedb/pull/3363).
I recommended executing a SELECT SQL to ensure the row was deleted (i.g., ... delete-rows -> compact table(randomly, optional) -> flush table(randomly, optional) -> insert rows(randomly, optional) -> validate deleted value -> drop-table.).

I have reviewed the current test-fuzz structure, and the general approach is as follows:

A new fuzz_delete target needs to be added.
Define a FuzzInput to serve as the input for the fuzz test.
Implement a delete_expr, which defines the expressions required for the delete operation.
Finally, assemble execute_delete, which completes the following sequence: create-table -> insert-rows -> delete-rows -> drop-table.

Cool, almost there.

We can randomly invoke the compact/flushing of the target table(You can follow the instructions in this issue: feat: flush or compact table and region functions #3363).
I recommended executing a SELECT SQL to ensure the row was deleted (i.g., ... delete-rows -> compact table(randomly, optional) -> flush table(randomly, optional) -> insert rows(randomly, optional) -> validate deleted value -> drop-table.).

compact/flushing, this step, can I interpret it as: After the insert, because the LSM architecture requires that rows successfully inserted can only be deleted through compact + flush. So I understand the process should be: create-table -> insert-rows -> compact (must?) -> flush (must?) -> select-count-rows -> delete rows -> select-count-rows -> drop-table.
At the same time, I found that there is no whereExpr provided in selectExpr, so does it mean that deleteExpr only needs to delete the entire table's data?

compact/flushing, this step, can I interpret it as: After the insert, because the LSM architecture requires that rows successfully inserted can only be deleted through compact + flush.

Nope, We want to cover more cases.

So I understand the process should be: create-table -> insert-rows -> compact (must?) -> flush (must?) -> select-count-rows -> delete rows -> select-count-rows -> drop-table.

For compact (must?) -> flush (must?)  steps can be considered as optional.
The whole picture will be like this:
create-table -> (insert-rows -> compact (optional) -> flush (optional) -> delete rows-> validate deleted)* -> drop-table.
And the (insert-rows -> compact (optional) -> flush (optional) -> delete rows-> validate deleted)* means these steps can be repeated multiple times.

At the same time, I found that there is no whereExpr provided in selectExpr, so does it mean that deleteExpr only needs to delete the entire table's data?

I prefer to implement the where clause for the SELECT first. The select count operation is still not good enough.

During the process of reading tests-fuzz-README, I encountered a problem. The specific compilation error is in the following file. What aspects can I look into to find a solution?
test-fuzz-errors.txt

What about cargo clean and try again?

The issue still persists. My current platform is:
platform: Mac M2 Max
OS: macOS 14.4.1


fix(no-promise-reject): new Promises and throw statements are now also checked

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

#848  👈
#847
#846
main

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

Join @RebeccaStevens and the rest of your teammates on  Graphite


Ft/support native esm

This PR Enables knex to support native esm for knexfiles, migrations and seeds.
It enables the dynamic import of knexfiles, migrations, and seeds using the import() syntax.
It inspects the consuming app's package.json to determine if the application is configured to interpret .js files as esm or cjs, and uses the according stub.
This PR drops support for node 8, as node 8 is no longer receiving updates https://blog.risingstack.com/update-nodejs-8-end-of-life-no-support/
and updates package.json  and README to reflect that.

@ExProbitasFiducia : Howdy, and thx for putting this PR together!  I'm just catching up w/ things, so it's probably going to take me a couple of days to read through everything.
However, after a quick glance, I'm not entirely sure what the scope of this PR is.  From what I can tell, it looks like it's doing several different things:

Introducing native support for ESM
Updating the syntax in several places to use async / await
Deprecating the --esm flag.  (it's unclear to me if the new code will be 100% compatible to the previous --esm flag behavior)
Revising the Migrator because...?  (more async / await revisions?)
Etc.

Would it be possible to split this PR into multiple smaller PRs that each have a narrowly-defined scope?  For example: for the "native support for ESM" task, it seems like the PR should only include the following changes:

Adding the ability to infer if the consuming package is a "module".  (Note: if I recall, there is already a separate library that can handle this step.  It might be the esm module that has been mentioned elsewhere)
If the consuming package is a "module", then add support for the "mjs" and "cjs" extensions.

Thanks in advance!


examples: add stats to jquery and seneca examples

The react example has to wait. The way it's built is using webpack and
installing opossum as an npm module.
Fixes: https://github.com/bucharest-gold/opossum/issues/47

Coverage remained the same at 99.432% when pulling 18be6ba8e300fe1d040320a91b58619eed15174f on 47-examples-update into 36196781bdb6239d293eb93525d785639b841c9e on master.


Audit and make documentation for command JSON.GET consistent

The documentation of the command [JSON.GET](https://dicedb.io/commands/jsonget/) present in the docs could have become stale. We need your help to ensure that the documentation is complete, correct, and consistent.  Completing this issue will give you much-needed exposure to how the command is implemented and the different parameters it accepts. Hence, this is a great way to build a good understanding of the project and its functionality. While still making a significant contribution. Here's what you need to do:

Go through the documentation of the command and run all the examples, making sure they all work as expected
If the same command is present in Redis, then the expected output of the DiceDB command is the same as the Redis output
If the command is not in Redis, then as per your judgment, raise an issue with this template or fix the documentation and raise a pull request.
The documentation should contain the following sections in the following order

short and concise introduction paragraph about the command covering what it does
Syntax
Parameters: List all the parameters the command accepts, and ignore the section if there are no parameters
Return values: List all possible return values and under what condition
Behaviour: Describe the behavior of the command, what it does, how it does it, some internal specifics if any
Errors: List all possible errors the command can throw and under what condition
Examples: List all possible examples of the command and the expected output. Assume CLI implementation

Note: The title of the sections should be the exact strings mentioned above, like "Examples", "Return values", etc.
Note: The description of the command in Frontmatter of the command.md file is the first paragraph (introduction) of the command.
Go through the DiceDB/dice repository and explore the command implementation

to understand all possible return values
to understand all possible error values and conditions
to understand different parameters the command accepts

Please use the documentation of the SET command as the reference point and structure the
documentation of this command in a very similar way, including the following points

if you see any "Conclusion" section, remove it
the headers (h1, h2, h3) should be appropriately used
the CLI prompt should be <IP_ADDRESS>:7379> and not anything else
Use markdown tables for parameters and return values (ref SET command documentation)
wrap any command or paramter in backticks (`) to highlight them
the section header should be capitalize as used in the SET command documentation

If you find any inconsistencies, please fix the documentation and raise the PR.
The core idea of this exercise is to ensure that the documentation is consistent, correct, and complete.
Make sure you comment on the other issues you created (if any) as a comment on this issue and also any PR (if any) that you created.
Thank you for picking this up and contributing to the DiceDB. It means a ton.

Hi @arpitbbhayani
I would like to pick this up.
Can you please assign this to me?

Hi @arpitbbhayani @JyotinderSingh
I would like to pick this up. Can you please assign this to me?

Assigned

Behaviour mismatch

When a key exists but path in json data does not exists

In Redis we get an error

In Dice we get nil


Cookiecutter

This PR introduces the following:

Repo is structured according to cookiecutter guidelines
Pylint and DSLinter is configured, and return perfect score
Repo still contains main.py for manual execution of the ML pipeline, this will be removed in later editions

Unnecessary configuration files and artifacts from the cookiecutter repo generation process have been removed. Any other mention of individuals have been changed to that of the team


Template Updates: Wellbeing

Context
Part of 1.7 release is updating existing templates to avoid custom animations and unnecessary prepublish checklist suggestions.
Summary
Wellbeing Template revisions

remove custom animations and replace them with effects available in editor
add arrows and cta circle as stickers
adjust blob shapes as directed

Relevant Technical Choices
Used my own judgement for what direction everything should fly in.
To-do

User-facing changes
See figma for revised template: https://www.figma.com/file/NHHMwIfxMnz39NxqkrFb7R/Stories-Post-Stable?node-id=7357%3A546501
List of changes can be found here: https://github.com/google/web-stories-wp/issues/6976
Before
https://user-images.githubusercontent.com/10720454/114105754-39edfa80-9882-11eb-8288-a355157f51ef.mp4
After
https://user-images.githubusercontent.com/10720454/114104162-3442e580-987f-11eb-8363-dcbaebe41657.mp4
Testing Instructions
Use the wellbeing template from dashboard/explore templates and see that the template has been updated in line with Figma. There should be no more custom animations and PPC suggestion about clickable link element size should be resolved on page 4.
QA

[ ] This is a non-user-facing change and requires no QA

This PR can be tested by following these steps:

UAT

[ ] UAT should use the same steps as above.

This PR can be tested by following these steps:

Reviews
Does this PR have a security-related impact?
No
Does this PR change what data or activity we track or use?
No
Does this PR have a legal-related impact?
No
Checklist

[ ] This PR addresses an existing issue and I have linked this PR to it in ZenHub
[ ] I have tested this code to the best of my abilities
[ ] I have verified accessibility to the best of my abilities (docs)
[ ] I have verified i18n and l10n (translation, right-to-left layout) to the best of my abilities
[ ] This PR contains automated tests (unit, integration, and/or e2e) to verify the code works as intended (docs)
[ ] I have added documentation where necessary
[ ] I have added a matching Type: XYZ label to the PR

Fixes #6976

This looks great, just a couple of questions:

Should this background animation be removed?

Yes! Those should be gone now, i missed a few of 'em earlier.

I think the arrow sticker here needs to be white

It should be white now:

Small fixes to do:
Overall

All animation durations are too quick. Let's change all of them to 2000ms

Page 1

Remove logo animation

Page 8

Remove animation from number 3

Page 9

Remove animation from title and numbers

Thanks!

Overall

All animation durations are too quick. Let's change all of them to 2000ms

Updated!

Page 1

Remove logo animation

Page 8

Remove animation from number 3

Page 9

Remove animation from title and numbers

(Percy's having issues right now and won't let us log in, the visual diff that it is catching is an intentional update of the template).


Use field initialization shorthand if possible

Use field initialization shorthand if possible.
Also this PR fixes a bug that attributes on a shorthand field get removed.

Thansk!


Update dependency: jwt_verify_lib

Signed-off-by: Daniel Grimm<EMAIL_ADDRESS>Description: update jwt_verify_lib to support RS384 and RS512 tokens
Risk Level: low
Testing: upstream unit tests
Docs Changes: none
Release Notes: none
Fixes https://github.com/istio/istio/issues/15380
This had previously been merged into the 1.2 release track but is now missing from the 1.3 track due to the switchover to envoy filters. This only targets release-1.3 because it has been merged upstream (https://github.com/envoyproxy/envoy/pull/8212) and I expect it to be automatically pulled into master.

cc @istio/release-managers-1-3


Bazel initial commit

Forgot to mention: I took the liberty of editing the import_relativity test to make it verifiable.  See the expected outcomes in test/expect.


swallow SJAX errors during in-browser translation

Fixes https://github.com/evanw/node-source-map-support/issues/161.
Similar to what @evanw did in https://github.com/evanw/node-source-map-support/commit/bbe8c603ab17418953629ff26b188a40a9015882, to prevent "translation" from failing when trying to load a resource from a different domain.

@alonsogarciapablo is this not needed anymore?

Awesome 👍


Definition of gene could be more precise

Currently the definition of gene is:
interactor for genetic interaction.
However I think it does not reflect the use the term in the current curation standards.

Replaced with an adaptation of the definition of gene as given by the sequence ontology (http://www.sequenceontology.org/browser/current_release/term/SO:0000704):
"A genomic region (or regions) that includes all of the sequence elements necessary to encode a functional transcript. A gene may include regulatory regions, transcribed regions and/or other functional sequence regions."


Widget background colour now matches ui components

This fixes Item 55: Background at times is a different colour to
background of items in mapcore panel.

Before

After

The test fails due to the changes on the search result and not related to this pull request.


If I remove a catalog then add another with same name, the new one does not show up

Rancher versions: 5/25
Steps to Reproduce:

Create a local catalog named "catalog1" and point it to a valid url and Save
Under the catalog menu go to that catalog
Go back to manage catalog and delete catalog1 and add Catalog at same time
Name it "catalog1" and point it to a different valid url

Results:  Old catalog gets deleted but the new one is not added.  Need to do all deletes first then Adds on Save

Update: This occurs because on delete and then recreation of the same catalog, the create catalog endpoint is hit instead of the update catalog endpoint. Thus, the catalog added with the same name and different URL is seen as a duplicate.

It's the delete of a new entry with the same name that causes this; changed to not send a delete when removing a row that didn't come from the server in the first place.

Version - 1.6.1-rc10
Verified fixed


Improve validation error message

Right now, it isn't clear from this error which DoFn is causing this error/where it is. This will dump the whole DoFn's name and file path.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

R: @lostluck


App keeps crashing

The app is keeping crashing when I open a folder or a disk.

@rokmakovec
Does this only happen in details mode or also in grid mode?
If it doesn't happen in grid mode it's already tracked in #3547. (:

@rokmakovec thanks for the report. Please add the app version and log files to the issue description.


Resolve #1321, cleans up leader

Closing this, as am creating new PR against master.

You don't have to create a new PR, you can just reset the PR branch to some other commit and force push.


Better error handling

Traceable error codes would be nice. Need to simulate them on individual basis and using categories.
Trying to avoid using exceptions.

Failed to not use exceptions. Seems like the most reasonable approach as dealing with return values everywhere turned wonky.


Contact Flow Export for Connect integration defaults to us-west-2

When exporting the contact flow, the Get Customer Input bot with bot defaults to stating the bot is in us-west-2 even if deployed in another region.

Sorry about that @domcatalano .. yes, it's a bug that slipped through the net. Fixed in next release coming soon.


Investigate where netcdf-c breaks thread safety

The global mutex is in no way ideal when reading/writing from multiple threads, and could be split into several (global/per file), or replaced by RWlocks.
This requires investigation into where netcdf does something thread unsafe, and limit the locking to this part. Should also investigate where HDF5 might be problematic
One could also integrate bindings to https://github.com/Parallel-NetCDF/PnetCDF, but this limits the formats to CLASSIC
https://github.com/Unidata/netcdf-c/issues/1373

Constraints from Rust should also be taken into account when investigating (multiple reader, no writers), which could allow relaxing of the locking

There is currently no locking (in netcdf-c) of the global NCList, which contains the file-handles. One could either lock when opening/closing files, or try to upstream a RwLock protecting this structure.

Caching of variables and attributes requires a separate lock when reading/writing variables/groups.

Continuing from #42 which seems to overlap with this one. HDF5 does support single-writer / multiple-readers (e.g. http://docs.h5py.org/en/stable/swmr.html). So maybe this is something that could be supported on newer HDF5 based netcdf files. I saw somewhere that using hdf5 to write, and especially adding new variables, to a netcdf file is likely to make the file unreadable by netcdf. But reading should be fine.

Multiple readers should be supported out of the box, but some initialisation might be required on first access of the variable. I think we can have a lock surrounding opening/closing files and smaller locks on variable access

@gauteh Do you have a suitable benchmark we could use for testing parallell performance?

I can try to make something self-sufficient - but I don't think I will
manage before after newyear. Are we talking about concurrent access from
threads or from processes? I am little bit confused about the difference
between this issue and the parallel read issue.
man. 30. des. 2019, 16:28 skrev magnusuMET<EMAIL_ADDRESS>
@gauteh https://github.com/gauteh Do you have a suitable benchmark we
could use for testing parallell performance?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mhiley/rust-netcdf/issues/43?email_source=notifications&email_token=AAAN362CLA2GC6JZDYS3NUTQ3IHRLA5CNFSM4J4HZULKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEH2RPRY#issuecomment-569710535,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAN366IEF6GZ2SOEAFPYQ3Q3IHRLANCNFSM4J4HZULA
.

And to same file struct? Or one struct per thread to same netcdf file?
man. 30. des. 2019, 16:31 skrev Gaute Hope<EMAIL_ADDRESS>
I can try to make something self-sufficient - but I don't think I will
manage before after newyear. Are we talking about concurrent access from
threads or from processes? I am little bit confused about the difference
between this issue and the parallel read issue.
man. 30. des. 2019, 16:28 skrev magnusuMET<EMAIL_ADDRESS>
@gauteh https://github.com/gauteh Do you have a suitable benchmark we
could use for testing parallell performance?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mhiley/rust-netcdf/issues/43?email_source=notifications&email_token=AAAN362CLA2GC6JZDYS3NUTQ3IHRLA5CNFSM4J4HZULKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEH2RPRY#issuecomment-569710535,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAN366IEF6GZ2SOEAFPYQ3Q3IHRLANCNFSM4J4HZULA
.

Are we talking about concurrent access from threads or from processes?

I believe we are mostly interested in threads

the difference between this issue and the parallel read issue

The parallel access requires an MPI communicator, and parallell instances of the same program. This is not exposed at all in the current form of the library

Ok, I don't have anything using MPI at the moment. But I can create some
benchmarks for multi-thread concurrency.
man. 30. des. 2019, 16:39 skrev magnusuMET<EMAIL_ADDRESS>
Are we talking about concurrent access from threads or from processes?
I believe we are mostly interested in threads
the difference between this issue and the parallel read issue
The parallel access requires an MPI communicator, and parallell instances
of the same program. This is not exposed at all in the current form of the
library
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mhiley/rust-netcdf/issues/43?email_source=notifications&email_token=AAAN363R67BQZPINTLUDRBTQ3II3BA5CNFSM4J4HZULKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEH2SETQ#issuecomment-569713230,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAN36YMSDRXXYW4LJY55X3Q3II3BANCNFSM4J4HZULA
.

And to same file struct? Or one struct per thread to same netcdf file?

I am not really sure whether Variable should be Send/Sync yet. I'll need to dig into netcdf-c and see.

Tried making a small example now using latest master. I'm not able to pass a Arc<netcdf::File> between threads anymore due to UnsafeCell on groups: https://github.com/gauteh/rust-netcdf/blob/thread-perf/tests/concurrency.rs
I also noticed you are working on some big changes in #51, which is probably good to get in before resolving this..?

Yeah, I've pretty much redone entirely how this crate parses the netcdf file, which will remove the UnsafeCell and pull the crate to a one-to-one mapping towards netcdf. I don't think there is any strong opposition against merging, so I'll go ahead and do just that.

Great!

Some observations from using a non-threadsafe HDF5 library: Segfaults and aborts, even when working on unrelated netCDF files. HDF5 should be considered highly unsafe for multithreading.

I guess that settles the discussion on HDF5. I asked developers of Hyrax (an official opendap server) about their HDF5 interface. It seems to be more thread-safe, but I am not sure if this refers to the HDF5 reading library or possible an interface in-between. If somehow HDF5 can be made thread-safe w.r.t. at least reading that will put the performance much higher than netcdf-libs in other languages (depending on use case of course).
https://github.com/OPENDAP/hdf5_handler/issues/13

Maybe using https://github.com/aldanor/hdf5-rust could help ? Note that I haven’t used that crate, but readme says « provides thread-safe Rust bindings and high-level wrappers for the HDF5 library API. »
HTH.

Maybe using https://github.com/aldanor/hdf5-rust could help ? Note that I haven’t used that crate, but readme says « provides thread-safe Rust bindings and high-level wrappers for the HDF5 library API. »
HTH.

It also relies on the official HDF5 library, but provides thread-safety through a global lock. This means that a single process can only sequentially use any of the (unsafe) functions from HDF5.

We have https://crates.io/crates/hdf5file, but quite a lot of work is neccessary to get this up to spec, and gain performance relative to a linked thread-safe hdf5

For CDF-1,2,5 we could realistically create a safe dispatcher, I already have a CDF-parser written in nom

netcdf-c does not make any reasonable guarantee regarding thread safety, so a global mutex is needed. If multi-reading is necessary, consider using MPI (requires some additional support in this library), or copy the approach of dars


Login page

1- Login with buttons images are not rendering.
2- It's a very basic design we can make it a good looking.

Just want clarification, This issue needs to sove or is addressed already

okay Thanks


README.md typos and grammar fixes

Hey, I went over the README and did some cleanup because I'm really obsessive about this stuff, I'm sorry! I did this to the transformers README too.

Are we doing code reviews here? Changes look good to me!

LGTM, thanks !


Commandbox install hangs

Installing package [github:cfsimplicity/lucee-spreadsheet]
Cloning Git URL [https://github.com/cfsimplicity/lucee-spreadsheet.git]
SLF4J: Class path contains multiple SLF4J bindings.
SLF4J: Found binding in [jar:file:/C:/Users/ananda/.CommandBox/lib/runwar-3.6.1-SNAPSHOT.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: Found binding in [jar:file:/C:/Users/ananda/.CommandBox/lib/slf4j-nop-1.6.4.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: See http://www.slf4j.org/codes.html#multiple_bindings for an explanation.
SLF4J: Actual binding is of type [org.slf4j.impl.SimpleLoggerFactory]
....[here it hangs]

Sorry, I'm not really au fait with the CommandBox integration which was added by @murphydan Perhaps he can help or someone from Ortus?

I was just able to do it without a problem.

My guess is that this has more to do with the CommandBox setup on your computer in general or permissions and nothing to do with lucee-spreadsheet. This is just trying to install straight from the github repo and not using Forgebox or anything. Can you install anything from a github repo using CommandBox?
@danielosan - are you on the CFML slack (https://cfml.slack.com)? There is a box-products channel that you could ask on as well.

Hi thanks for quick reply.  Not sure what is the cause of the problem. I am using commanbox daily (3.7.0 on Win10-64) and hardly have problems. Maybe some conflict with lib in classpath. Had this before with OSGi bundles.
Since I had to ntegrate it quickly I downloaded it and installed it manually into the coldbox app. Works well. Thanks for the great work!! Because using the Lucee extension is no option for us.


how can I set input and output device to use?

I would like to auto-connect to a server and use the local microphone and speaker for communication. The script works fine but I can't hear or speak. any help would be apprechiated! regards

@utillity Please let us know how it worked, I'd like the same, please.

Hi, sorry, did not get it to work and gave up as there does not seem to be a true shell client with realtime communication support out there currently.
Von: Ways<EMAIL_ADDRESS>Gesendet: Freitag, 29. Mai 2015 16:11
An: mattvperry/mumble-ruby
Cc: Tilfried Weissenberger
Betreff: Re: [mumble-ruby] how can I set input and output device to use? (#42)
@utillityhttps://github.com/utillity Please let us know how it worked, I'd like the same, please.
—
Reply to this email directly or view it on GitHubhttps://github.com/mattvperry/mumble-ruby/issues/42#issuecomment-106818717.


Unable to load DLL 'OpenCvSharpExtern' in 4.2.0

Summary of your issue
System.DllNotFoundException: Unable to load DLL 'OpenCvSharpExtern': The specified module could not be found. (Exception from HRESULT: 0x8007007E)
Environment
Windows 10 1903 18362.535 64-bit
VS 2019 16.4.2
Target .NET Framework 4.7.2
OpenCvSharp4.Windows <IP_ADDRESS>91223
What did you do when you faced the problem?
Created a new solution (class library), ran a test for the code below.
Example code:
    public class Class42
    {
        public Class42()
        {
            new Mat();
        }
    }

Output:
System.TypeInitializationException: The type initializer for 'OpenCvSharp.Mat' threw an exception. ---> System.TypeInitializationException: The type initializer for 'OpenCvSharp.NativeMethods' threw an exception. ---> System.DllNotFoundException: Unable to load DLL 'OpenCvSharpExtern': The specified module could not be found. (Exception from HRESULT: 0x8007007E)
  Stack Trace:
    NativeMethods.redirectError(CvErrorCallback errCallback, IntPtr userdata, IntPtr& prevUserdata)
    NativeMethods.LoadLibraries(IEnumerable`1 additionalPaths) line 79
    NativeMethods.cctor() line 40
    --- End of inner exception stack trace ---
    NativeMethods.core_Mat_sizeof()
    Mat.cctor() line 714
    --- End of inner exception stack trace ---
    Mat.ctor()

What did you intend to be?
No exception.
The exact same setup works with OpenCvSharp4.Windows <IP_ADDRESS>91216.
Also, OpenCvSharpExtern.dll files are present in both [project]\bin\Debug\dll\x86 and [project]\bin\Debug\dll\x64. I opened them in Dependencies and didn't see any errors; don't know what to look for beyond that.

:thinking:
Well... it may work if you delete the bin/ folder and try again.
I also tried, but it worked. This is a VS solution I made for test the issue.
OpenCvSharpClassLibraryTest.zip

I tried deleting the bin folder; that didn't help.
I then tried the solution you posted. After I built and ran it, I got the same exception (console app and test projects). Then I noticed that there wasn't a dll folder in Release for the console app, so I did a Clean and Rebuild. That did create a dll folder in Release with the appropriate content. However, running the console app again resulted in the exception again.
Finally, I ran Process Monitor set to show file system activity for ConsoleApp1.exe and ran the console app. Inspecting the output revealed that shortly after the app successfully opens OpenCvSharpClassLibraryTest\ConsoleApp1\bin\Release\dll\x86\OpenCvSharpExtern.dll, it tries to open OpenCvSharpClassLibraryTest\ConsoleApp1\bin\Release\OpenCvSharpExtern.DLL and fails. I assume this is the point where the exception is caused.
Do you have any idea why it would try to find the dll in the base folder instead of in the dll
\... folder?

Inspecting the output revealed that shortly after the app successfully opens OpenCvSharpClassLibraryTest\ConsoleApp1\bin\Release\dll\x86\OpenCvSharpExtern.dll

x86??? Your environment is x64, but the x86 DLL is loaded...
OpenCvSharp controls the target DLL file path for C# DllImport by loading the dll/[x64 or x86]/OpenCvSharpExtern.dll in advance with the Win32API LoadLibrary function. If the platform setting is AnyCPU, the program must determine whether the native DLL to be loaded at runtime should be x64 or x86.
As a workaround, manually copying dll/x64/OpenCvSharpExtern.dll to the same folder as the exe file may make the program work.

I've checked in Task Manager, and the console app is 32-bit. According to this, the reason is that "Prefer 32-bit" is checked in the project properties.
Copying the 64-bit dll to the exe folder causes the program to crash with a System.BadImageFormatException (as might be expected). Copying the 32-bit dll to the exe folder results in a new exception: System.DllNotFoundException: Unable to load DLL 'OpenCvSharpExtern': A dynamic link library (DLL) initialization routine failed. (Exception from HRESULT: 0x8007045A).
After I unchecked "Prefer 32-bit" and rebuilt the solution, the console app showed as 64-bit in Task Manager. Without copying any dlls, running it caused the original exception. After I copied the 64-bit dll to the exe folder, I got the "initialization routine failed" exception again.
I'm guessing there's something different about OpenCvSharpExtern.dll between <IP_ADDRESS>91216 and <IP_ADDRESS>91223 that's causing this, but I can't figure out what.

I've used Debug Diag to analyze the crash, and it reports that

the assembly instruction at OpenCvSharpExtern!xphoto_oilPainting2+c65836 in E:\Projects\OpenCvSharpClassLibraryTest\ConsoleApp1\bin\Release\dll\x64\OpenCvSharpExtern.dll has caused an unknown exception (0xc000001d) on thread 0

Thank you. xphoto_oilPainting2 ?? I can't find any problems there, but in the next version I will try to disable it for now.
https://github.com/shimat/opencvsharp/blob/master/src/OpenCvSharpExtern/xphoto.h#L308

No problem. I've enabled native code debugging in the project properties for the console app, caught the exception and looked at the disassembly at the point where the exception was thrown (which, in hindsight, I should have done immediately). It turns out exception 0xC000001D means Illegal Instruction, and the instruction in question is vpxor, which, according to Intel 64 and IA-32 Architectures Software Developer’s Manual, is an AVX/AVX2 instruction (and my CPU does not support AVX).
I suppose the question now is why does DLL initialization code need to run AVX instructions, and why doesn't it check for CPU support before trying to run them?
Also, 0xc65836 is a 12MB offset, so the problem might not actually be in xphoto_oilPainting2. If I had a debug build of OpenCvSharpExtern.dll and its PDB file, I might be able to see which function contains the offending code.

Yes i have same issue with some pc that have old cpu pentium N that cant load OpenCvSharpExtern i try to use geterror and return me 1114 = ERROR_DLL_INIT_FAILED, new cpu dont have issue with same code

I've tried building the project (commit tagged <IP_ADDRESS>91223) to get a debug build of OpenCvSharpExtern.dll, but I have not succeeded. First, VS complained about a missing opencv_world420d.lib file, so I got the one from opencv-4.2.0-vc14_vc15.exe here and put it in opencv_files_420\win-x64\x64\vc16\staticlib. After that, when I tried building an x64 Debug build, I got 149 unresolved external symbol errors for various functions for OpenCvSharpExtern.

I'm sorry but the project settings of the Debug configuration is incomplete, as I always use the Release configuration.
opencv-4.2.0-vc14_vc15.exe does not include opencv_contrib, so you need to build OpenCV yourself in order to build OpenCvSharpExtern.

My cmake log of building OpenCV
  CPU/HW features:
    Baseline:                    SSE SSE2 SSE3
      requested:                 SSE3
    Dispatched code generation:  SSE4_1 SSE4_2 FP16 AVX AVX2 AVX512_SKX
      requested:                 SSE4_1 SSE4_2 AVX FP16 AVX2 AVX512_SKX
      SSE4_1 (13 files):         + SSSE3 SSE4_1
      SSE4_2 (1 files):          + SSSE3 SSE4_1 POPCNT SSE4_2
      FP16 (0 files):            + SSSE3 SSE4_1 POPCNT SSE4_2 FP16 AVX
      AVX (4 files):             + SSSE3 SSE4_1 POPCNT SSE4_2 AVX
      AVX2 (26 files):           + SSSE3 SSE4_1 POPCNT SSE4_2 FP16 FMA3 AVX AVX2
      AVX512_SKX (3 files):      + SSSE3 SSE4_1 POPCNT SSE4_2 FP16 FMA3 AVX AVX2 AVX_512F AVX512_COMMON AVX512_SKX

AVX is supposed to be a dispatched (optional) feature, but it is actually required??

I have this problem as well.
OpenCV should check if AVX is available when loading the libraries, however there seems to be a bug in an initializer which runs an AVX instruction which than causes the crash: _https://github.com/opencv/opencv/issues/15690_
This is related to OpenCV 4.1.1 and 4.1.2.
If I understand it correctly, it should have been resolved for 4.2 - but it does not seem that way.

opencv-4.2.0-vc14_vc15.exe does not include opencv_contrib, so you need to build OpenCV(with contrib) yourself in order to build OpenCvSharpExtern.

I tried doing this, but I was unable to build a debug version of OpenCvSharpExtern.dll equivalent to your release version. (I was able to build, with much trickery, a ~20MB debug version of OpenCvSharpExtern.dll which didn't exhibit the original problem, presumably because it didn't include the AVX-containing function.)
It seems this problem is another instance of opencv/opencv#15690, as @janecek1980 mentioned.

I made DEBUG version of OpenCV4.2.0 lib files.
https://drive.google.com/open?id=1MnY0sR13y2lLrN1x0vGplbxedMhMcat5
(The DEBUG lib files are too large to upload to GitHub unless I use Git LFS.)

I made DEBUG version of OpenCV4.2.0 lib files.

I've built OpenCvSharpExtern.dll using these. However, with this DLL, I can not get VS to break on the actual offending instruction, despite having native code debugging enabled. It just shows the DLL not found exception, like it did previously when I had native code debugging disabled (and I've verified in the output window that it's repeatedly loading and immediately unloading the correct DLL). I have no idea why this is happening. Analyzing in DebugDiag didn't help either.

There is simil issue with emgu project when use opencv 4.2 library
https://github.com/emgucv/emgucv/issues/297

I ran into a similar issue, turned out that the Windows 10 version I was using was Windows 10 N which doesn't include the Microsoft Media Foundation framework. This has been introduced as a dependency in the latest builds of OpenCV.
Worth checking? You can download the framework to install on the N versions of Windows 10.

I'm trying to create web service with face detection on netcoreapp3.1 https://github.com/Pavelyev/CoreFaceDetector
Project is being published with Self-Contained mode and win-x64 runtime.
On my machine it works fine (Microsoft Windows 10 Enterprise LTSC (10.0.17763 N/A Build 17763))
But on server (Microsoft Windows Server 2012 R2 Standard (6.3.9600 N/A Build 9600)) it throws following exception on new CascadeClassifier(path) call:
 ---> System.DllNotFoundException: Unable to load DLL 'OpenCvSharpExtern' or one of its dependencies: A dynamic link library (DLL) initialization routine failed. (0x8007045A)
   at OpenCvSharp.NativeMethods.redirectError(CvErrorCallback errCallback, IntPtr userdata, IntPtr& prevUserdata)
   at OpenCvSharp.NativeMethods.LoadLibraries(IEnumerable`1 additionalPaths)
   at OpenCvSharp.NativeMethods..cctor()

 option in publish profile doesn't affect the result
Is there anything I can do?

@Pavelyev as far as I understand, current version of OpenCVSharp won't work on processors without AVX2 instruction set. If you can't change your server to something with more modern CPU, the only solution (aside from actually managing to fix it in OpenCV(Sharp) sources) is to downgrade to something like 4.1 or 4.0 version.
If you will do this, please tell what version don't have this issue.
I'm planning to downgrade OpneCV in my project too, but don't have suitable PC to reproduce the issue, so can't know for sure which version started this.

I used the OpenCvSharp4.Windows with Version <IP_ADDRESS>90326 and that worked for me. (Before we had and older version of the deprecated Nuget package OpenCvSharp3-AnyCpu which lead to an services crash. In the first step i updated this package which lead to the "Missing OpenCvSharpExtern" exception.) Hope this helps.

Is there any official response from the authors maintaining OpenCvSharp?
I am trying to make this work for OpenCvSharp4, UWP, x86 build.

I have released the new OpenCvSharp(4.3.0). Does the problem still occur in 4.3.0?

I have released the new OpenCvSharp(4.3.0). Does the problem still occur in 4.3.0?

thank you and Stay at home :dango:

Thank you, works for me even on older CPUs.

still i facing the problem sir. Hope to get help from you. Thanks

Please try reinstalling NuGet. I don't know why, but sometimes it fixes the error.
Update-Package -reinstall

If it doesn't solve your problem, try to check the dependencies of OpenCvSharpExtern.dll with a tool such as DependencyWalker.

@shimat  This problom occured when I deployed app to Azure Function, while my app run normally on local machine. Version is <IP_ADDRESS>00421

Migrated project from NetFramework 4.7.2 to NetCore 3.1 and now dll/[x86|x64] folder is missing just after executing build in VisualStudio. Dlls are moved to runtimes folder as mention aboved. WindowsLibraryLoader's static constructor throws exception that it cannot find Extern.dll and swallows it immediately for some reason.
at OpenCvSharp.WindowsLibraryLoader.LoadLibrary(String dllName, IEnumerable`1 additionalPaths)
   in C:\projects\opencvsharp\src\OpenCvSharp\PInvoke\WindowsLibraryLoader.cs:line 119

Manually creating dll\x64 folder and copying libraries resolves issue, but how to fix it to make everyting working automatically again?

Actually problem with netcore and netstandard is because of this commit
However I don't understand why this rules should be skipped.
If you don't specify exact runtime identifier in project runtimes folder is created all the time, but WindowsLibraryLoader doesn't search there at least by default.
And even if you set exact runtime identifier in project file, OpenCvSharpExtern.dll will be located in the same folder as application, but still won't load because LoadLibraryInternal method always adds dll to the path, and we never have dll because of that commit.

I believe that WindowsLibraryLoader is not necessary for .NET Core. The native DLLs are automatically path-resolved with "runtime/[RID]/native".
https://github.com/shimat/opencvsharp/blob/master/nuget/OpenCvSharp4.runtime.win.nuspec#L28
.NET Framework cannot properly select x86 and x64 DLLs at runtime, and there is no path for native DLLs such as runtimes\win-x64\native in the .NET runtime specification, so I prepared WindowsLibraryLoader.
Many people have commented on this issue, but it works fine for me.

I checked with ProcessExplorer and yes, Extern.dll is loaded by itself , but after that exception is thrown and swallowed. Looks like TryPInvoke() loads it as inteded.
So maybe you should skip https://github.com/shimat/opencvsharp/blob/master/src/OpenCvSharp/PInvoke/NativeMethods/NativeMethods.cs#L92 for .netcore (maybe only when additionalPaths variable is empty, if that'll be safer).

Thanks 🙆♂  https://github.com/shimat/opencvsharp/pull/947

Hi,
I am still getting this issue on OpenCVSharp4.Windows <IP_ADDRESS>00524. When targeting either .net 4.6.1 or 4.7.2. Was this fix included in this version?

I found out that when i use
[DeploymentItem("Resources")]
in my unittest it cannot find the dll. As soon as i remove it, everything works again.

In Microsoft Visual Studio 2019, be sure that you get the Runtime DLLs with using Nuget Packet Manager : Install-Package OpenCvSharp4.runtime.win -Version <IP_ADDRESS>00524
Have a nice coding...

Hi, I have same error like that. I've try to some version from 4.4 to 4.2, but still error, when I've changed to version 4.1, It run perfect. I don't know why

@mpolicki any chance of getting a copy of your fixed OpenCvSharpExtern.dll? Currently am locked into very versions and need to deploy asap onto machines with CPU without AVX2 support. If there was any way for you to share your fix it would be hugely appreciated. Thanks so much.

I tried doing this, but I was unable to build a debug version of OpenCvSharpExtern.dll equivalent to your release version. (I was able to build, with much trickery, a ~20MB debug version of OpenCvSharpExtern.dll which didn't exhibit the original problem, presumably because it didn't include the AVX-containing function.)
It seems this problem is another instance of opencv/opencv#15690, as @janecek1980 mentioned.

@BenSmithYoti I'm sorry, I no longer have any of those files.

@mpolicki Oh well - was worth a shot. Thanks for the rapid response, really appreciated.

@shimat first thanks so much for this excellent resource, absolutely awesome!
I am wondering if you might be able to point me in the right direction. It seems the .Net Framework version support drops off on the more recent nuget packages. I wonder if this is unavoidable?
I am stuck needing to target net45 and have come across this issue (CPUs without AVX2 support), which seems (I believe?) to boil down to an issue in specific OpenCv releases (?) https://github.com/opencv/opencv/pull/15704 https://github.com/opencv/opencv/issues/15690 etc ?
I would just update the OpenCvSharp nuget references (OpenCvSharp4 and OpenCvSharp4.runtime.win currently on <IP_ADDRESS>00108) which has support for .Net 4.0), but it seems that is the last version that I can use.
Looks like from https://www.nuget.org/packages/OpenCvSharp4/<IP_ADDRESS>00208 upwards its 4.6.1 +
Almost at QA/production and then just today found this AVX2 issue when testing on a Celeron CPU.
Wonder if you have any suggestions at all for any easy ways forward?
Thanks so much for sharing all your excellent hard work!

I was having this problem in production (After Creating the Setup File and Installing it). I've solved it by doing the following steps

Installed the OpenCVSharp Through NuGet Packages
Setting the Platform Target to X64 in Project Properties
Creating the Setup File with the same Platform
Manually Copying the "packages\OpenCvSharp4.runtime.win.<IP_ADDRESS>10817\runtimes\win-x64\native\OpenCvSharpExtern.dll" file into the installation directory.

Issue seems still present on:

.Net Framework 4.8
OpenCvSharp4 <IP_ADDRESS>20608
OpenCvSharp4.runtime.win <IP_ADDRESS>20608
OpenCvSharp4.Windows <IP_ADDRESS>20608
OpenCvSharp4.WpfExtension <IP_ADDRESS>20608
OpenCvSharp4.Extensions  <IP_ADDRESS>20608
I am using Fody Weaver
Windows Desktop Application WPF

Exe compiles fine as soon as a OpenCvSharp method is called:
System.TypeInitializationException: "The type initializer for 'OpenCvSharp.Internal.NativeMethods' threw an exception."
DllNotFoundException: Unable to load DLL 'OpenCvSharpExtern': The specified module could not be found. (Exception from HRESULT: 0x8007007E)

What I have tested so far:

Reinstalling all nuget packages with. Update-Package -reinstall
Sample project with .Net Core 6 works without problems, .Net Framework 4.8 does not
Downgrading to version <IP_ADDRESS>11225 (the lowest version which offers a nuget package for OpenCvSharp4.Extensions)
Setting project to x64 only + disabling all 32bit options.

What I have observed;

Manually moving OpenCvSharpExtern.dll to the build folder --> This works but isn't an option for our customer. Dlls need to be imbedded into the executeable with Fody weaver
The projects build folder still creates a /dll/x64 and /dll/x86 folder which includes opencv_videoio_ffmpeg455_64.dll and OpenCvSharpExtern.dll

I hope this info helps a bit.
Are there any news on how to solve this issue?

OpenCvSharp4 is very perfect,
Can you make an android version, thanks


[h3] Update to 4.1.0

Describe the pull request

What does your PR fix?
Updates h3 to 4.1.0

Which triplets are supported/not supported? Have you updated the CI baseline?
As before, Yes

Does your PR follow the maintainer guide?

Yes

Thanks again!


disqus.com

disqus still getting blocked on something maybe on <IP_ADDRESS> www.disqus.com

#57 :thinking:

@xxcriticxx please test now with latest updates. There's a couple of other domains that disqus is reliant on which were also blocked. And should be fixed now. Please confirm. @funilrys latest fix should sort this now 🙏

@mitchellkrogza ok i check when i get home

Cool let me know if all ok now. :+1:
Kind Regards
Mitchell Krog

Visit me at https://mitchellkrog.com

License My Images From Getty Images Here
http://www.gettyimages.com/search/photographer?family=creative&page=1&photographer=mitchell krog&sort=best&excludenudity=true#license
or From Gallo Images Here
http://galloimages.co.za/Search?q=mitchell krog&p=1&a=1&l=2,1&st=2&dr=on&is=1&token=48034&pp=13&rc=6

disqus.com now works!
@mitchellkrogza whats the list of domains i will add to pihole whitelist

That’s great news. You should always reference the centralized whitelist of
domains from
https://github.com/mitchellkrogza/CENTRAL-REPO.Dead.Inactive.Whitelisted.Domains.For.Hosts.Projects/blob/master/DOMAINS-whitelist.txt

@mitchellkrogza i need to know what you guys whitelisted to get disqus.com to work

Have a look at the commits on CENTRAL REPO
https://github.com/mitchellkrogza/CENTRAL-REPO.Dead.Inactive.Whitelisted.Domains.For.Hosts.Projects/commit/1d182f4186fa24b3f71a4548de85823f247314c9
AND
https://github.com/mitchellkrogza/CENTRAL-REPO.Dead.Inactive.Whitelisted.Domains.For.Hosts.Projects/commit/733d89f5bf58b0985eb88e255c00ea64c60649db

Some interesting developments https://thehackernews.com/2017/10/disqus-comment-system-hacked.html?https=1

Even better : https://www.troyhunt.com/disqus-demonstrates-how-to-do-data-breach-disclosure-right/


Cut 0.4.1 release

Relevant for https://github.com/NixOS/nixpkgs/pull/288956#discussion_r1504514507

Release has been created: https://github.com/threefoldtech/mycelium/releases/tag/v0.4.1

https://github.com/threefoldtech/mycelium/releases/tag/v0.4.2 has been released with a regression fix for the HTTP API


[Dropdown] onChange method getting called during initial render of component

Describe the bug
onChange method getting called during initial render of component
To Reproduce
Component I have used:
<Dropdown           name='account'           required = {true}           placeholder={"Placeholder"}           value={value}           onChange={ setFormData}         >           {'Account Type'}           <DropdownOption value={'customer'}>Customer</DropdownOption>           <DropdownOption value={'provider'}>Provider</DropdownOption>   </Dropdown>
Expected behavior
onchange method should not be called when component gets mounted.

Hi there. This happens because the dropdown value is derived initially from aggregating the child dropdown-options which have selected attributes. Is this causing some sort of problem for you in practice?

Hi there. This happens intentionally because the dropdown initial value is derived initially from aggregating the child dropdown-options which have selected attributes. Is this causing some sort of problem for you in practice?

Yes, it mainly causes error when we have default value and based on that other components are getting rendered. The value gets reset on initial render

Yes, it mainly causes error when we have default value and based on that other components are getting rendered. The value gets reset on initial render

You have to set the initial/default value by setting selected on the child options.


refactor(frame_reader): reduce complixity and enhance readability

FrameReader used to be a large collection of switch statements. This
change reduces that complexity by splitting into the concept of AMQP
and SASL frame internal reader classes. A number of other readability
cleanups occurred including: unrolling a number of if statements to
fail early, as well as registration of Readers.
NOTE: I'd like to further discuss this PR, because while the readability of having multiple internal FrameReaders is nice, this can be optimized even more to create a single registry in the FrameReader class encompassing all frame types we accept. It would reduce readability imho, but probably improve performance. Furthermore, if we can agree on what all *_frame.js files should look like, we can automate the building of the frame registry.

alright, how about that?

As for your note, I'm unwilling to sacrifice the readability of distinct frame classes for perf... yet.  However, I definitely realize that there is quite a bit of cleanup that could happen there.  Perhaps the distinct classes could, instead of providing the readPerformative, provide a field array and a defaults object, we could build the reader from that.

@noodlefrenzy well I was more speaking about the possibility of doing a further refactoring to merge AmqpFrameReader and SaslFrameReaders into a single lookup object, a la:
var registry = {
  '0x00': {       // AMQP Frame
     'open-descriptor-whatever-it-is': Open,
     'close-descriptor...': Close
  },

  '0x01': {      // SASL Frame
     // you get the idea
  }
}

@noodlefrenzy either way, it's sort of a problem for the future, I ran some benchmarks last night and it didn't buy me that much

Ah, that, yeah I'm totally cool with that :)

Want me to cut a new release with these changes?

I'm writing a slew of unit tests to get full coverage for the frame reader, I'd say new release after that?

Fantastic!  Sounds good to me.


Upgrade ANO to DOCA 2.8

@cliffburdick @jjomier if we can merge it today so rmax folks can submit their MR

@e-ago I don't have permission to merge


Test if :2222 is in use, increment until a free port is found

Aside from the hard dependency on lsof, this does what it should so that multiple testing instances can be run at once (though not with the same backing disks).

Nice improvement; this has bitten me before. If you're going to print the port in use, maybe sleep (for five seconds?) if it won't use 2222, since that message will tend to get pushed off the screen in serial-console mode.

Nice improvement; this has bitten me before. If you're going to print the port in use, maybe sleep (for five seconds?) if it won't use 2222, since that message will tend to get pushed off the screen in serial-console mode.

Good call. I moved the message to only be shown if a non-standard port is used, and then set it to sleep for a few seconds.


Tvl pool staking

Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context, including:
https://github.com/paritytech/substrate/issues/14566

What does this PR do?
This PR introduces the TotalValueLocked of all nom-pools
Why are these changes needed?
they are requested in issue 14566
How were these changes implemented and what do they affect?
Its just an extra StorageValue with an effect on execution time on every flow of funds in and out of the nom-pools.

Use Github semantic linking to address any open issues this PR relates to or closes.
Closes #14566

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3394074

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3394725

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3394726

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3394724

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3397880

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3397879

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3399072

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3399073

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3399266

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3399265

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3403500

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3403498

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3403499

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3409048

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3409046

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3409047

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3414730

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3415581

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3415582

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3415635

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3415636

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3415637

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3415867

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3415869

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3416575

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3416577

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3416576

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3423911

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3423912

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3423913

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3424543

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3424542

When I try the runtime upgrade on a Kusama snapshot then upgrade fails because the TVL which is calculated in the migration by summing up all T::Staking::total_balance() of every BondedPool is different from the sum of PoolMember::total_balance() of every PoolMember. I currently have no idea what the reason for this deviation might be.
Here is the output of the runtime upgrade:
runtime::nomination-pools] [19343161] 🏊♂ Upgraded 149 pools [2023-08-23T17:25:38Z ERROR runtime] panicked at 'assertion failed: (left == right)      left:61550133549019336,      right:<PHONE_NUMBER>0739417: TVL deviates from the actual sum of funds of all PoolMembers.'


Multithread results vary widely.

Text results vary widely when processing performed in parallel.  Is this normal?  See example:
`int pageCount = 42;
Bitmap[] b = GetTestBitmaps(pageCount);
string[] results = new string[pageCount];
Parallel.For(1, pageCount + 1,
index =>
{
using (TesseractEngine engine = new<EMAIL_ADDRESS>"eng", EngineMode.Default))
using (Pix image = PixConverter.ToPix(b[index-1]))
using (var page = engine.Process(image))
{
results[index-1] = page.GetText().Trim();
}
});
string total = String.Join(Environment.NewLine, results); // differs from run to run.`
The text output difference is more pronounced when the image load activity occurs in the loop.  I presume this is because the processor load is heavier, but I'm not clear on the real cause.

This may not prove helpful, but I would encourage you to review the code outside of this tesseract specific section.  I've been using this project within a parallel loop just like that, for over a year now, and the results are solidly consistent.

The only thing outside the example is the bitmap fetcher.  This is an obvious place for a potential difference but I actually did a version of the code where I attempted to verify that the bitmaps were the same.


re-enabled rescheduling for resident tasks

The current behavior of marathon when receiving an InstanceChange for a resident task is, that no ScaleRunSpec is propagated.
This change will correct this behavior and also send ScaleRunSpec events if InstanceChange is received for an instance in Condition. UnreachableInactive.


added book catalog implementation

@gooroo-dev review it

@goorpp-dev review it again

@gooroo-dev review it again

@gooroo-dev review it again

@gooroo-dev review it again

@gooroo-dev review it again

@gooroo-dev review it

@gooroo-dev review

@gooroo-dev review

@gooroo-dev review again

@gooroo-dev review

@gooroo-dev review

@gooroo-dev review it

@gooroo-dev review it

@gooroo-dev review

@gooroo-dev review

@gooroo-dev review

@gooroo-dev review

@gooroo-dev review

@g00r00-dev review

@gooroo-dev review

@gooroo-dev review

@gooroo-dev review

@gooroo-dev review

@gooroo-dev review

@gooroo-rev review

gooroo-dev review

@gooroo-dev review

@gooroo-dev review

@gooroo-dev revire

@gooroo-dev review

@gooroo-dev revire

@gooroo-dev review


"HTML Wrap Line Length" parameter does nothing

Seems like this value is not passed to js-beautify properly.

This was merged in about 18 days ago :+1:

Update js-beautify to 1.5.5.

This was already done in https://github.com/Glavin001/atom-beautify/pull/228

I guess the 'Wrap line length' setting should work fine if this issue is closed, but it does not seem to work.
Is this near to be fixed?

I'm using HTML.
I put the line length to 120, but the wrapping is not properly working and some lines keeps longer.

@malkomich Would you mind generating a debug.md file and pasting it into a gist?  This way I can see your sample, beautifier, options, and so forth so that I can reproduce the problem.

Here it is :smile:
https://gist.github.com/malkomich/e5a6c23e21417c5a1c30905cca58c57d

Thanks for that.  It appears the language is HTML, the beautifier is JS Beautify, and that JS Beautify does support text wrapping to a specified length as tested on their web tool.


Append different plot types in one window

When I try to append a scatter plot to a line plot, my scatter plot is drawn with lines which is obviously not desired.
Sample code:
Y = np.linspace(-5, 5, 100)
winLine=viz.line(
    Y=Y * Y,
    X=Y,
    opts=dict(markers=False),
)

viz.scatter(
    X=np.random.rand(25, 2),
    Y=(np.random.randn(25) > 0) + 1,
    win=winLine,
    opts=dict(
        markersize=10,
        markercolor=np.floor(np.random.random((2, 3)) * 255),
    ),
    update='append'
)

Hi @ehsanomidi - this is not quite the way that the 'update' field is supposed to be used. Currently you can only use scatter's update to update scatter plots with more scatter traces and line's update to update line plots with more line traces.
You will be able to override this behavior once #275 is in however by appending a line trace using the line function's update but setting 'traceopts': {'mode': 'markers'} in opts, as we process line plots as scatter plots in 'line' mode so if you override that with 'markers' mode that trace will be handled as a scatter plot.

You can also view #155 for a method to implement this behavior right now by using the generic send functionality

This is now supported via both #275 and #155, closing.


feat(cli): use prompts and progress status for chart commands

Description
This pull request changes the following:

feat(cli): use prompts and progress status for chart commands

Related Issues

Closes #589

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


solves issue #2334, also add a lot of defines for DT_FLAGS_1 that wer…

…e missing to detect for PIE DYNs, so might as well have them
issue #2334

My initial feeling is that the approach taken is incorrect but this will take more time for me to evaluate the original issue.  At first glance your change would appear to allocate fake externals in the EXTERNAL block for all symbols when canExecute is true.  This should only be done when a real memory address is not established (address == Address.NO_ADDRESS).   The EXTERNAL block is intended to be used only for those symbols not accounted for within the GOT/PLT for which relocations may reference.

The real issue is how we process the PLTGOT and migrating symbols into the External space (not EXTERNAL block).  The External space is used to identify Imports associated with the PLT.   I did notice that we did not properly handle the DT_JMPREL relocation table processing which plays a role in this.  I am currently working to fix this issues you have observed with #2334 .  The use of DT_FLAGS_1 is not the solution to this problem.

Sure, i am not familiar enough with the elf parsing codebase (far from it, tbh i installed eclipse and just debugged through the code until i found a point where right before it removed the symbols) to pretend that my approach is the correct one. Just so you know the way i did it (testing for an executable, even a relocatable one (which was the reason of the DF_FLAGS_1) end up creating the correct external references and the blocks in the library correctly point towards the correct symbol in the library. Just reject the proposed merge rq and give me a headsup in the issue if you manage to fix the parsing of JMPREL

Fix has been pushed to master if you would like to try it.  I am going to close this pull request.


Customization: less compiler failure

While I was trying to get the customization downloader to work (turns out the problem I was having is actually #1516), I tried specifying all options in "Customize Variables", and got a less compiler error:
A less error occured trying to build your bundle. Please paste the error below in an issue for us at http://github.com/twitter/bootsrap! thanks!

{"type":"Parse","message":"Syntax Error on line 671","index":24780,"filename":"bootstrap.css","line":671,"column":270,"extract":["}","undefinedundefinedundefinedundefinedundefinedundefinedundefinedundefinedundefinedundefinedundefinedundefinedundefinedundefinedundefinedundefinedundefinedundefinedundefinedundefinedundefinedundefinedundefinedundefinedundefinedundefinedundefinedundefinedundefinedundefined",null]}

Specifying all variables to default can be easily done by running:
$('.download-builder input').each(function() { var t=$(this); t.val(t.attr('placeholder')); });

You don't need to set the variables to defaults -- the default to it automatically (just leave them empty)

Hm... i can't reproduce that. If I type "red" into link color it works for me. Can you give a more specific example

Sorry, I messed it up in my javascript -- I was changing values of the input checkboxes as well. After I pressed reset, it fixed the variable fields but (natrually) not the checkboxes on the rest of the page. If I do $('#variables input').each(function() { var t=$(this); t.val(t.attr('placeholder')); }); to populate every field, it works again.
My apologies


fix(api-kit): Fix failing e2e tests

What it solves
Fixes some failing e2e tests for the api-kit.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 77.417%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
774

Relevant Lines:
929

💛  - Coveralls


[loc] Adding language - Bosnian (bs) to transifex VS Code project

We expect at least three language contributors sign up and product team sign off before adding new language.
Please vote by adding your transifex account name here.
Criteria link: https://github.com/Microsoft/Localization/wiki/How-to-request-a-new-UI-language-for-VS-Code-project

Assign me to translate VS Code to Bosnian. Transifex username: DaTRiX25

I would love to contribute with Bosnian translation as well. Transifex username: elAhmo


Use overlay FS when building projects

Currently, we install a repo, then run TS on it, then run TS on it again. This creates a situation where the results of the first compile can negatively affect the second.
In #131, I attempted to fix this by using git to stage all files after installing (even if ignored), then reverting them. That didn't work because FS/memory cost of that was too high for the runner.
Instead, we can use OverlayFS to do this; we install into the lower layer, then perform all operations on an upper layer. After the first run, we throw away the overlay and create a new one, such that the second compilation sees the repo as being freshly installed.
If tmpfs is disabled, we instead use multiple directories and just reflink copy them. Theoretically we could use copying within the tmpfs too, but I don't believe that tmpfs can do that.
This PR is best viewed without whitespace due to indent changes: https://github.com/microsoft/typescript-error-deltas/pull/143/files?w=1

Going to push this over to a branch in this repo and reopen; want to test that it works in the real world but can't start jobs via my fork.


Added note about not supporting python 3.7

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/convert-model-winmltools.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Support for CMakes Ninja Multi-Config generator

Starting with CMake 3.17 the Ninja Multi-Config generator is supported. That allows to build Release, Debug and other Configurations (e. g. RelWithDebInfo, MinSizeRel) without new configuration and generation.
If your CMake code generates a valid Ninja Multi-Config build system, it will also always generate a valid Ninja build system.
If you build multiple configurations you have to postfix the generated .lib, .dll, .pdb, .so and .a files. That can be done via CMAKE_<CONFIG>_POSTFIX. I propose to set these defaults:

CMAKE_RELEASE_POSTFIX (none)
CMAKE_DEBUG_POSTFIX=d
CMAKE_RELWITHDEBINFO_POSTFIX=rd
CMAKE_MINSIZEREL_POSTFIX=rs

Implementing that would simplify and hardening a build for multiple configurations. Currently you need to do a complete re-configuration to do that.

cc @veblush

@jtattermusch Can you take a look please?

@veblush Among our approximately 50 libraries, gRPC is currently the only one that does not yet support Ninja Multi-Config. It would be nice if this could be implemented in one of the next releases. ;-)


crash on CFXNotificationTokenRegistration keyCallbacks

Step 0: Are you in the right place?

For issues or feature requests related to the code in this repository
file a Github issue.

If this is a feature request please use the Feature Request template.

For general technical questions, post a question on StackOverflow
with the firebase tag.
For general (non-iOS) Firebase discussion, use the firebase-talk
google group.
For backend issues, console issues, and other non-SDK help that does not fall under one
of the above categories, reach out to
Firebase Support.
Once you've read this section and determined that your issue is appropriate for
this repository, please delete this section.

[REQUIRED] Step 1: Describe your environment

Xcode version: 12.4
Firebase SDK version: 7.10.0
Installation method: CocoaPods (select one)
Firebase Component: Analytics, Crashlytics, Performance

[REQUIRED] Step 2: Describe the problem
Steps to reproduce:
What happened? How can we make the problem occur?
This could be a description, log/console output, etc.
Hi, I update the Firebase SDK in my podfile from 7.4.0 to 7.10.0. And one of my users gets this crash:

The user is on iPad with iOS 12.5.2.
The error log doesn't look very clear to me, do you have any idea to help me understand what is happening ?
If you have a downloadable sample project that reproduces the bug you're reporting, you will
likely receive a faster response on your issue.
Relevant Code:
No sample code

The crashing stack trace doesn't include any symbols from Firebase (aside from the Crashlytics crash recorder thread), which means this crash is likely not caused by Firebase. Looks like the crash is caused by an index out of bounds error in one of your collection views.

Thank you for your explanation.


.to_html {AttributeError}'AgGrid' object has no attribute 'to_html'

I was trying to use justpy in order to generate interactive HTML reports. Following the documentation  i wanted to use the .to_html method on the generated webpage object as the following
def grid_test():
    wm_df = pd.read_csv('https://elimintz.github.io/women_majors.csv').round(2)
    wp = jp.WebPage()
    grid = jp.AgGrid(a=wp)
    grid.load_pandas_frame(wm_df)
    return wp.to_html()

But the following error is raised .to_html {AttributeError}'AgGrid' object has no attribute 'to_html'
Is this expected ?

For this to work in JustPy you just need to return wp.
Change the last line to return wp and the page will be served with the grid.
Please take a look at the example in the tutorial: https://justpy.io/grids_tutorial/pandas/
If you need the html presentation for the page this is not available with an AgGrid component since it has no html presentation.

@elimintz  Thanks for your comment but my intentions was not to return the webpage it self. I wanted to return the html content to be dumped later into a .html file. I wanted to utilize justpy model for producing reports with pandas but seems this is not supported
Thanks for your time

You can use JustPy to create an interactive page that is the report and send people the link. Is that use case helpful to you?

Thanks elimintz, i wanted to explore justpy as a pandas renderer ) possibly using ag-grid and plotly. My goal was to export an html that is standalone and can work offline
I tried digging more into the API but seems i can add a <script></script>  tags


Fix: Legend should update with reflow(fixes #276)

What is the purpose of this pull request? (Check any that's applicable)

[ ] Feature
[X] Bug fix
[ ] Dependency updates
[ ] Documentation updates
[ ] Build related changes
[ ] Changes an existing functionality
[ ] Other, please explain:

Does this introduce a breaking change?

[ ] Yes
[X] No

What changes did you make? (Give an overview)
I have added reflowLegend method which will update the legend during the reflow and each graph will be calling this method on panning the graph with reflow. Also, I am checking if the graph has no data or not, and based on the graph if it does not holds any value, then no data view will be displayed and if it holds value, no data view will be removed(if any exists).
Add any screenshots
When the graph is loaded with No data view(legends will be disabled and a no data view will be displayed):

On click of panning button, when the graph is loaded with some data, legend(s) gets updated if it holds any data and the no data view will be removed from the view:

@NarasimhaShenoi Can you update the dev examples in the PR Description like in this PR: https://github.com/cerner/carbon-graphs/pull/258

@Veera81 I have updated the example and the PR descitption.

@NarasimhaShenoi I'm guessing probably you might have tested all scenarios with boundary conditions for all graph types for legend reflow right?

@Veera81 Yes, I have tested reflowLegend all the graphs and it is working fine.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[pylint] type property name too long

Names that would be flagged at current length of 40:
return_headers_and_deserialized_path_list
get_deleted_path_properties_from_generated_code
_list_metric_enriched_series_data_requests
_list_anomalies_for_detection_configuration
_list_anomalies_for_detection_configuration_requests
build_get_incidents_by_anomaly_detection_configuration_request
build_delete_datasource_credential_request
obtain_token_by_refresh_token_on_behalf_of
_get_accounts_having_matching_refresh_tokens
_infer_max_workers_greater_than_one_if_needed
_validate_topic_subscription_and_rule_types

Names that would be flagged at current length of 40:
return_headers_and_deserialized_path_list get_deleted_path_properties_from_generated_code _list_metric_enriched_series_data_requests _list_anomalies_for_detection_configuration _list_anomalies_for_detection_configuration_requests build_get_incidents_by_anomaly_detection_configuration_request build_delete_datasource_credential_request obtain_token_by_refresh_token_on_behalf_of _get_accounts_having_matching_refresh_tokens _infer_max_workers_greater_than_one_if_needed _validate_topic_subscription_and_rule_types

I think we can probably ignore any names that are prefixed with _


run oadp-operator e2e test from the velero repo

execute openshift/oadp-operator e2e tests directly against the velero repo locally or via prow ci
Thank you for contributing to Velero!
Please add a summary of your change
execute openshift/oadp-operator e2e tests directly against the velero repo locally or via prow ci
to test
optional: update the VELERO_IMAGE variable
execute: make -f Makefile.prow test-e2e

locally tested:
    Mongo application DATAMOVER
    /tmp/oadp-operator/tests/e2e/must-gather_suite_test.go:75
------------------------------
[AfterSuite]
/tmp/oadp-operator/tests/e2e/e2e_suite_test.go:198
  > Enter [AfterSuite] TOP-LEVEL - /tmp/oadp-operator/tests/e2e/e2e_suite_test.go:198 @ 10/15/24 13:36:33.389
2024/10/15 13:36:33 Deleting Secrets
2024/10/15 13:36:33 Deleting DPA
  < Exit [AfterSuite] TOP-LEVEL - /tmp/oadp-operator/tests/e2e/e2e_suite_test.go:198 @ 10/15/24 13:36:33.595 (206ms)
[AfterSuite] PASSED [0.206 seconds]
------------------------------

Ran 1 of 33 Specs in 253.674 seconds
SUCCESS! -- 1 Passed | 0 Failed | 0 Pending | 32 Skipped
PASS

tested and idea is working (i.e. unsupported override is used in tests)
spec:
  backupLocations:
    - velero:
        config:
          profile: default
          region: ...
          s3ForcePathStyle: 'true'
          s3Url: '...'
        credential:
          key: cloud
          name: bsl-cloud-credentials-ibmcloud
        default: true
        objectStorage:
          bucket: mateus01022023
          prefix: velero-e2e-...
        provider: aws
  configuration:
    nodeAgent:
      enable: true
      podConfig:
        resourceAllocations: {}
      uploaderType: kopia
    velero:
      defaultPlugins:
        - openshift
        - aws
        - csi
      featureFlags:
        - EnableCSI
      logLevel: debug
  podDnsConfig: {}
  unsupportedOverrides:
    awsPluginImageFqin: ''
    azurePluginImageFqin: ''
    gcpPluginImageFqin: ''
    kubevirtPluginImageFqin: ''
    nonAdminControllerImageFqin: ''
    openshiftPluginImageFqin: ''
    resticRestoreImageFqin: ''
    veleroImageFqin: 'quay.io/konveyor/velero:latest'

But it still not clear to me how this will work in CI. Along CI image name, we also need OADP operator installed to run the tests

we also need OADP operator installed to run the tests

hopefully ocp/release repo test config can make it so operator is already installed, if not we can follow up later.

tested and idea is working (i.e. unsupported override is used in tests)
spec:
  backupLocations:
    - velero:
        config:
          profile: default
          region: ...
          s3ForcePathStyle: 'true'
          s3Url: '...'
        credential:
          key: cloud
          name: bsl-cloud-credentials-ibmcloud
        default: true
        objectStorage:
          bucket: ...
          prefix: velero-e2e-...
        provider: aws
  configuration:
    nodeAgent:
      enable: true
      podConfig:
        resourceAllocations: {}
      uploaderType: kopia
    velero:
      defaultPlugins:
        - openshift
        - aws
        - csi
      featureFlags:
        - EnableCSI
      logLevel: debug
  podDnsConfig: {}
  unsupportedOverrides:
    awsPluginImageFqin: ''
    azurePluginImageFqin: ''
    gcpPluginImageFqin: ''
    kubevirtPluginImageFqin: ''
    nonAdminControllerImageFqin: ''
    openshiftPluginImageFqin: ''
    resticRestoreImageFqin: ''
    veleroImageFqin: 'quay.io/konveyor/velero:latest'

But it still not clear to me how this will work in CI. Along CI image name, we also need OADP operator installed to run the tests

ya..  not sure if we'll do that via prow or via this makefile.  If we DO need to do it via this makefile, wouldn't just a be a make deploy-olm from the oadp-operator dir?  EASY no?

make deploy-olm from the oadp-operator dir? EASY no?

deploy, and setup credentials stuff in prow so to not expose secrets in this makefile

/lgtm

/gentle-ask-to-merge-this-pr Openshift CI, could you please kindly merge this PR as it has already been approved and has the lgtm label. Let's get this one through, shall we?
@kaovilai I think this change broke velero as we don't have DOWNSTREAM_OWNERS file here?
https://github.com/openshift/release/commit/cd8b87b75e923ddfa1011bcbb746726370600e7b#diff-a9035bbaee5af66130c81cfd0b97df589a7d2e22e87ad9d3800851407967c07bR695

It's not broken from before. The change in release only allows us to add owners file without carrying conflicting changes with upstream owners. Upstream owners file may not be in the right format for prow or have our people on it anyways.

Demo of manual label override to continue auto merge


chore: Add Cloud Java as Owners of Java Auth Library

As discussed previously, adding Do Not Merge label here until Sep 1st (9/1/2024).


WIP: Build and publish skupper-router image

Closes #39

@mcressman Would it be possible to copy the qdrouterd RPM spec file directly into this repo, so that rpmbuild can be used to do builds?


Won't let me change scaling settings 1.19

I have the custom player mod and pekhui mod both in the mods folder, both mods are loaded in-game. For some reason, it will not let me change things like adjusting eye height and such, no clue why this is

Update CPM. Pehkui support was added to 1.19 forge in 0.4.4a.

Update CPM. Pehkui support was added to 1.19 forge in 0.4.4a.

k ty, can you give me a link for which version to use? It's ok if no

wait nvm I found it tysm


PDFSimple pdf compression

PDFSimple needs compression functionality in order to be useful.  File sizes are comically large.  Exporting to html and creating pdfs from that is a cumbersome workaround.

Hello!
Could you please describe your question in a little more detail?


Package proposal: x64dbg.vm

Package Name
x64dbg
Tool Name
x64dbg
Package type
ZIP_EXE
Is the tool a console application?
false
Tool's version number
2024-01-06_21-29
Category
Debuggers
Tool's authors
mrexodia
Tool's description
An open-source user mode debugger for Windows. Optimized for reverse engineering and malware analysis.
Download URL
https://github.com/x64dbg/x64dbg/releases/download/snapshot/snapshot_2024-01-06_21-29.zip
Download SHA256 Hash
467e41d68e489aad288cb0d868ea37ceb9dcda3757d9badc6314964d847e26b4
Dependencies
No response
Why is this tool a good addition?
We currently install x64dbg from chocolatey. That version is nearly 3 years old at this point and is clearly not maintained. MrExodia does release via github release very frequently, so issue is to suggest switching to that.

I have also contacted the chocolately admins to ask for a package update there, but honestly, us using the github release would still be much better.

Thanks for the suggestion @stevemk14ebr!
@vm-packages the date seems to be used as the version:

should we use 2024.01.06 as version for the package? This is consistent with the current package.
If we want to update this package automatically, we would need to modify the update script to consider versions with -, as at the moment only . are allowed: https://github.com/mandiant/VM-Packages/blob/main/scripts/utils/update_package.py#L59

A dotted date makes sense as the version

@stevemk14ebr we are not using the chocolatey package, but this url: https://sourceforge.net/projects/x64dbg/files/snapshots/snapshot_$snapshotDate.zip
So modifying $snapshotDate in packages/x64dbg.vm/tools/chocolateyinstall.ps1 would also update the tool (and it is very easy to change it):
https://github.com/mandiant/VM-Packages/blob/79c35a70707ea79f01559eceb76eac000e029bd4/packages/x64dbg.vm/tools/chocolateyinstall.ps1#L11
The only advantage to change to GitHub releases would to benefit from automatic updates (which as said requires changes in the current script). @stevemk14ebr do you know if the maintainer is planing to continue using GH releases? I see only one recent release in GH, but some more in sourceforge.

I just asked, SF is the primary release channel (I was wrong about GH earlier). So we should bump to the latest hosted on SF


Update Statefulset lab to use aws-ebs-csi-driver

Issue #, if available:
Description of changes:

Created a new page (ebs_csi_driver.md) for the driver installation

Updated storageclass.md and yaml file to use the new drivers and to encrypt the EBS volumes by default

Created a new namespace called mysql

Updated the mysql yaml files to use the new namespace

Updated the documentation to use the new namespace

Simplified clean up

Add a new screenshot from the AWS console (EBS) to showcase encryption

Lots of small clean up and fixed typos

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

this is great! thanks!


STITCH-2847 - delays event emission until local changes are applied

Addresses the case where sometimes events are received before the local changes are applied by deferring the emission of the event to the write model container in a few places in the data synchronizer.

Dont worry about the checkstyle issue I think I fix it in my current PR


Added govcloud domain support

Added govcloud domain support

Please update the README to export this env variable just like others

vm.sh is updated to have default domain as "logicmonitor.com" in case LM_COMPANY_DOMAIN is empty or null or " ".
README is also updated.
@choudharypooja please validate once again.


Getting null pointer exception on java.lang.String moe.banana.jsonapi2.JsonApi.type()

I have Contact class with type="contacts"
@JsonApi(type = "contacts")
public class Contact extends Resource implements Serializable {
}
But now I am creating an object of class which is sub-class of Contact. But, I am getting following error,
12-01 13:54:55.641 27318-27318/vocera.uc E/CoreApplication: Uncaught Exception
java.lang.NullPointerException: Attempt to invoke interface method 'java.lang.String moe.banana.jsonapi2.JsonApi.type()' on a null object reference
at moe.banana.jsonapi2.Resource.typeNameOf(Resource.java:426)
at moe.banana.jsonapi2.Resource.(Resource.java:155)
At Vocera. Ac. core. entity_model. Contact.(Contact.java:14)
I am doing something wrong here? Do I need to add same @JsonApi(type = "contacts") to sub-class also?

A @JsonApi annotation is not inheritable. When you have multiple instance of the same type, the annotation should be declared on each of the class. But be cautious this may not be best practice of JSON API specification.


Logging events ppx, examples

Add a ppx to implement the structured event specification described in #4759, provide a couple of examples.
The implementation is in Ppx_coda.Declare_event. A comment there describes usage and the expansion.
(Not sure if expanding the type to a list of strings is the best, maybe the expansion should be a single string.)
The examples are in a new library, Logging_events. Uses of the ppx expand to instances of the type t in that library. The examples there are dummies. If this PR is merged, we'll create actual instances at a later date.

See new comments in #4759.

Will use a new approach.


Init role

Assignee is None

Created by capitanFlint129 at 2022-12-02T12:50:51Z

Assignee is None