Loop mode - slideTo method transition direction

This is a:

[x] bug

Swiper Version: 3.4.1.

Platform/Target and Browser Versions: Windows, CHROME.

What you did
This is the Swiper params that I use:

var params = {
	initialSlide: 0,
	direction                    : 'vertical',
	slidesPerView                : 2.5,
	loopedSlides                 : 4,
	threshold                    : 50,
	loop                         : true,
	control                      : another swiper instance,
	controlBy                    : 'container',
	prevButton                   : DOM Object,
	nextButton                   : DOM Object,
	touchMoveStopPropagation     : true,
	mousewheelControl            : true,
	speed                        : 800,
	centeredSlides               : true,
	autoplayDisableOnInteraction : false
}

Expected Behavior
If I call slideTo() from 0 to 1 (from a number lower than another), transition direction should be left to right, or top to bottom.
If I call slideTo() from 1 to 0 (from a number greater than another), transition direction should be right to left, or bottom to top.
Actual Behavior
In loop mode, if I call the slideTo method from slide 0 to slide 1, the transition direction is right to left, or bottom to top.
If I call slideNext(), transition direction works as expected (left to right, or top to bottom), and from my point of view this is the right behavior .
Thank you for your work!
It's an amazing project!

Issue is closed because of outdated/irrelevant/not actual/needed more information/inactivity.
If this issue is still actual and reproducible for latest version of Swiper, please create new issue and fill the issue template correctly:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with issue


Add discharge from piping as a source model

A common release scenario is from piping, such as from a PSV. This should include liquid, gas, and two-phase based on the K method.
The user would be responsible for figuring out the total K for the system using some standard methods, such as Crane's, and the rest would follow from some standard assumptions (e.g. ideal gas, isentropic, etc.)

I go back and forth as to whether this is necessary. It could be helpful, but it also doesn't strictly have anything to do with dispersion modelling, and a line has to be drawn somewhere as to what is in scope for this module and what should be left to the user to figure out.

I have decided that this is out of scope -- the code is for dispersion modeling not release modeling more generally. If I did want to go down the road of defining different types of releases, I think that should be a different library.


Mismatch between code and paper in the gradient penalty algorithm

After comparing your code and your WGAN-GP paper, there seems to be a mismatch. When you perform the gradient penalty, you do the following:
    differences = fake_data - real_data
    interpolates = real_data + (alpha*differences)
    gradients = tf.gradients(Discriminator(interpolates), [interpolates])[0]
    slopes = tf.sqrt(tf.reduce_sum(tf.square(gradients), reduction_indices=[1]))
    gradient_penalty = tf.reduce_mean((slopes-1.)**2)
    disc_cost += LAMBDA*gradient_penalty

while in the paper it seems that you are describing the following (note the differences in the first two lines):
    differences = real_data - fake_data
    interpolates = fake_data + (alpha*differences)
    gradients = tf.gradients(Discriminator(interpolates), [interpolates])[0]
    slopes = tf.sqrt(tf.reduce_sum(tf.square(gradients), reduction_indices=[1]))
    gradient_penalty = tf.reduce_mean((slopes-1.)**2)
    disc_cost += LAMBDA*gradient_penalty

You are nevertheless still sampling from a line joining fake and real data, so it should not make much difference.

The 1st case:
interpolates = real + alpha*(fake-real) = (1-alpha)real + alphafake
The 2nd case:
interpolates = fake + alpha*(real-fake) = alpha * real + (1-alpha)*fake
The two results are equal since the variable alpha is random float number between 0 and 1.
I wish this helps.


Vite ^3.0.0 launching tab on <IP_ADDRESS>

Describe the bug
I noticed my host changed from localhost to <IP_ADDRESS> when running Vite locally with default launch options. However, the release notes indicate that I should have experienced the oppositeL https://github.com/vitejs/vite/blob/main/packages/vite/CHANGELOG.md#main-changes

I recreated this on different machines by simply following https://vitejs.dev/guide/#scaffolding-your-first-vite-project and choosing vanilla javascript options (npm create vite@latest, vanilla, vanilla, cd {project folder}, npm install, npm run dev)
If I visit https://stackblitz.com/edit/vitejs-vite-sxspdt?file=package.json&terminal=dev, it appears to launch via localhost, which is confusing to me.
I read this - https://vitejs.dev/guide/migration.html#architecture-changes-and-legacy-options. The issue here is the default open behavior is launching a browser tab with the <IP_ADDRESS> host instead of localhost. It isn't just about correcting the console display option.
Setting the --host localhost option still displays AND launches at <IP_ADDRESS>.
Tested this behavior on two boxes (windows + mac). Both exhibit same behavior.

Reproduction
https://github.com/Jabronironi/vite-3.0.0-host-issue-example
System Info
I was able to reproduce this error on a Mac and Windows 11 machine.

Used Package Manager
npm
Logs
System:
OS: Windows 10 10.0.22000
CPU: (12) x64 Intel(R) Core(TM) i7-10710U CPU @ 1.10GHz
Memory: 4.62 GB / 15.79 GB
Binaries:
Node: 16.15.1 - C:\Program Files\nodejs\node.EXE
npm: 8.12.1 - C:\Program Files\nodejs\npm.CMD
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

This is intended. See NOTE of https://vitejs.dev/config/server-options.html#server-host.
To disable this behavior set dns.setDefaultResultOrder('verbatim') as explained there or upgrade Node.js to 17+.


prefix issues not resolved in https://github.com/microbiomedata/nmdc-schema/pull/447 yet

see #447

Why does prefix linter recommend KEGG_* in some cases and KEGG.* in others

Can this be closed?


[DOCS] Dark mode for docs website

What problem does this feature solve?
It's very difficult to read the docs for long time due to light theme. Please made a dark theme toggle
What does the proposed API look like?
.

Duplicate of https://github.com/vuejs/vuejs.org/issues/552


Error Apache Trust Manager - implementation of the X509TrustManager

Security and trust
Apache Trust Manager
Error Security
Your application uses an insecure implementation of the X509TrustManager interface with an Apache HTTP client, which causes a security vulnerability. See this article in the Google Help Center for more information, including the deadline to fix the vulnerability.

Lcom/zxy/tiny/core/HttpUrlConnectionFetcher$TinyTrustManager;
sv: deadline: 02/17/2022

Additional Information
To properly process SSL certificate validation, change your code in the checkServerTrusted method of your custom X509TrustManager interface to select CertificateException or IllegalArgumentException whenever the certificate presented by the server does not meet your expectations. In particular, be aware of the following issues:

Check that the exceptions thrown by checkServerTrusted are not included in the method. If they were included, this would cause checkServerTrusted to exit normally, and the application would trust a malicious certificate.
Don't use checkValidity to examine the server's certificate. The checkValidity method checks if a certificate has expired, but does not determine if it is trusted.
You can also use network security settings to customize the behavior of your application's certificates in a less error-prone way.

https://blog.csdn.net/luckyleaf666/article/details/119546361


cub::DeviceHistogram::HistogramEven: Incorrect result with integer levels and samples

The problem appears when upper_level - lower_level is not exactly divisible by number of bins. Appears in 1.16 and in https://github.com/NVIDIA/cub/pull/487 too.
Reproduction: the program as is provides [1, 0, 4, 1, 0, 0, 2] as a histogram, but correct one (obtainable by switching to using T = float;) is [1, 5, 0, 2, 1, 0, 0].
#include <cstdlib>
#include <vector>

#include <cub/device/device_histogram.cuh>//<cub/cub.cuh>   // or equivalently <cub/device/device_histogram.cuh>
#include <cuda/std/type_traits>

//using T = float;
using T = int;

// Declare, allocate, and initialize device-accessible pointers for input samples and
// output histogram
int      num_samples = 10;//32 << 20;    // e.g., 10
T*       d_samples;      // e.g., [2.2, 6.1, 7.1, 2.9, 3.5, 0.3, 2.9, 2.1, 6.1, 999.5]
int*     d_histogram;    // e.g., [ -, -, -, -, -, -]
int      num_levels = 8;     // e.g., 7       (seven level boundaries for six bins)
T        lower_level = 0.0;    // e.g., 0.0     (lower sample value boundary of lowest bin)
T        upper_level = 12.0;    // e.g., 12.0    (upper sample value boundary of upper bin)

int main()
{
    std::vector<T> s {2, 6, 7, 2, 3, 0, 2, 2, 6, 999};
    std::vector<int> h(num_levels - 1);

    s.resize(num_samples);
    cudaMalloc(&d_samples, num_samples * sizeof(T));
    cudaMalloc(&d_histogram, (num_levels - 1) * sizeof(int));
    cudaMemcpy(d_samples, s.data(), num_samples * sizeof(T), cudaMemcpyHostToDevice);

    // Determine temporary device storage requirements
    void*    d_temp_storage = NULL;
    size_t   temp_storage_bytes = 0;
    cub::DeviceHistogram::HistogramEven(d_temp_storage, temp_storage_bytes,
        d_samples, d_histogram, num_levels, lower_level, upper_level, num_samples);

    // Allocate temporary storage
    cudaMalloc(&d_temp_storage, temp_storage_bytes);

    // Compute histograms
    cub::DeviceHistogram::HistogramEven(d_temp_storage, temp_storage_bytes,
        d_samples, d_histogram, num_levels, lower_level, upper_level, num_samples);

    // d_histogram   <-- [1, 5, 0, 2, 1, 0, 0];

    cudaMemcpy(h.data(), d_histogram, (num_levels - 1) * sizeof(int), cudaMemcpyDeviceToHost);

    for(int a : h)
        std::cout << a << ", ";

    std::cout << std::endl;

    return 0;
}

Instead of bin = static_cast<int>((common_sample - min) / scale); it should be bin = static_cast<int>((common_sample - min) * bins / (max - min)); as the scale is of integral type and is therefore wrong. The problem that I can foresee is an overflow in multiplication, that can be avoided by extending arithmetics.


How about add response file support for long command line argument list

Original reporter: sray@<hidden>
Here is my origin post for cabal-install [https://github.com/haskell/cabal/issues/1681]
After install a lot of hackages, it seems impossible to regenerate an overall local document index. As for my case, with 160+ hackages, the command line argument list is more than 50k characters long, where the limitaion on Windows is 32768.
As Mikhail suggested on the link above, this problem could be solved by adding support for response file [http://gcc.gnu.org/wiki/Response_Files]

This will (I have tried it) fail with something like "-title=hscolour-1.22: Colourise Haskell code." as the spaces in this param get split up.

@randen
What about -title="hscolour-1.22: Colourise Haskell code."?

@Fuuzetsu, sorry this has become more complicated that I had thought.
@23Skidoo, my lazy, but non-standard thing I did temporarily was to force the response file creator to signal the appropriate parsing by using a newline (carriage-return, whatever--any of the various CR/NL combos) to separate each param in the response file, then the consumer can just split back out with "lines".  Not really standard, but, except for being able to provide on of CR or NL in a param value, it covers more cases than splitting on any white-space.


Select doesn't go away once selected, on web mobile viewport

Current Behavior
If you go to the Select docs in Chrome, change the viewport to be Mobile Touch, and select the item that's also overlaying the Trigger, the overlaid items do not disappear
https://tamagui.dev/docs/components/select
Expected Behavior
Once you select any item, the overlaid items should disappear
Tamagui Version
The URL says: https://tamagui.dev/docs/components/select/1.40.0

Platform (Web, iOS, Android)
Web, tested in Chrome Mobile touch Dev Tools

Reproduction
On Chrome, open the Dev Tools, go into Mobile Touch emulation mode

Open the Select docs: https://tamagui.dev/docs/components/select/1.40.0

Click to open the Custom Select

Click on the same Item that is overlaid with the Trigger

You should notice that the Select does not go away

If you select an Item that is not overlaid with the Trigger, the Select does go away, as expected

System Info
No response

check solutions in this issue

Thanks for the solution!
Correction: the above ref should be placed on Select.Item


Add Electron tests

zmq-prebuilt will be tested against Electron:

0.37.8: Atom 1.10, no need to rebuild on linux and os x
1.1.3: latest version matching node's ABI, no need to rebuild on linux and os x
1.4.3: latest version, needs rebuilding

Yeah! 🎉
✅ says we are good to upgrade in nteract

Hoorah!


[LSM] LSMLiquidStakeEnabled flag on host zone

Context and purpose of the change
Since not all host zones will have LSM enabled, we need a flag on the host zone to indicate whether they have it enabled and whether LSM liquid stakes are supported.
Brief Changelog

Added bool field LsmLiquidStakeEnabled on host zone
Added check in LSM liquid stake that it's enabled
Extended register host zone command to include parameter

Testing Instructions

Start dockernet

make start-docker

Confirm LSM is currently enabled by viewing the field lsm_liquid_stake_enabled on the host zone

strided q stakeibc list-host-zone

Build a proposal file to disable LSM (call it proposal.json)

{
    "description": "Proposal to disable LSM",
    "hostZone": "LSM",
    "enabled": false,
    "deposit": "10000001ustrd"
}

Submit the proposal and vote. Make sure to vote before before the voting period ends (30 seconds)

strided tx gov submit-legacy-proposal toggle-lsm proposal.json --from val1 --gas auto -y
strided tx gov vote 1 yes --from val1 -y
strided tx gov vote 1 yes --from val2 -y
strided tx gov vote 1 yes --from val3 -y

Wait for the proposal to pass and then view the host zone again, now lsm_liquid_stake_enabled should be set to false

strided q stakeibc list-host-zone

Update the proposal.json to the following (to re-enable LSM)

{
    "description": "Proposal to enable LSM",
    "hostZone": "LSM",
    "enabled": true,
    "deposit": "10000001ustrd"
}

Submit the proposal and vote

strided tx gov submit-legacy-proposal toggle-lsm proposal.json --from val1 --gas auto -y
strided tx gov vote 2 yes --from val1 -y
strided tx gov vote 2 yes --from val2 -y
strided tx gov vote 2 yes --from val3 -y

Wait for the proposal to pass and then view the host zone again, now lsm_liquid_stake_enabled should be set to true once again

strided q stakeibc list-host-zone

@ethan-stride @riley-stride Added gov route to enable LSM for host's


[HUDI-1003] Handle partitions correctly for syncing hudi non-parititioned table to hive

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contributing.html before opening a pull request.

What is the purpose of the pull request
This PR aims to create the table normally without partitions, when sync hudi non-parititioned table to hive.
JIRA: https://issues.apache.org/jira/browse/HUDI-1003
Brief change log

Judge whether to reset partitionFields to empty or not according to HIVE_PARTITION_EXTRACTOR_CLASS_OPT_KEY, during building HiveSyncConfig.

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Codecov Report

Merging #1720 into master will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master    #1720      +/-   ##
============================================
- Coverage     18.21%   18.21%   -0.01%
  Complexity      859      859
============================================
  Files           348      348
  Lines         15356    15359       +3
  Branches       1523     1524       +1
============================================
  Hits           2797     2797
- Misses        12201    12204       +3
  Partials        358      358

Impacted Files
Coverage Δ
Complexity Δ

...n/scala/org/apache/hudi/HoodieSparkSqlWriter.scala
46.74% <0.00%> (-0.85%)
0.00 <0.00> (ø)

Continue to review full report at Codecov.

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

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contributing.html before opening a pull request.

What is the purpose of the pull request
This PR aims to create the table normally without partitions, when sync hudi non-parititioned table to hive.
JIRA: https://issues.apache.org/jira/browse/HUDI-1003
Brief change log

Judge whether to reset partitionFields to empty or not according to HIVE_PARTITION_EXTRACTOR_CLASS_OPT_KEY, during building HiveSyncConfig.

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ]  Has a corresponding JIRA in PR title & commit
[ ]  Commit message is descriptive of the change
[ ]  CI is green
[ ]  Necessary doc changes done or have another open PR
[ ]  For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

My bad. Accidentally closed

@leesf Please take a review again.

@leesf Please take a review again.

Thanks for the update, we could also move the test to TestHiveSyncTool, wdyt?

@leesf Please take a review again.

Thanks for the update, we could also move the test to TestHiveSyncTool, wdyt?

Sure, I think so.

@leesf I add a method getTablePartitionKeys in HoodieHiveClient for the test use, I am not sure if it is reasonable，but should be needed. Please take a look.

@leesf I add a method getTablePartitionKeys in HoodieHiveClient for the test use, I am not sure if it is reasonable，but should be needed. Please take a look.

Thanks for the update, left minor comment, LGTM overall, please also take a look at the travis failure.

@leesf I add a method getTablePartitionKeys in HoodieHiveClient for the test use, I am not sure if it is reasonable，but should be needed. Please take a look.

Thanks for the update, left minor comment, LGTM overall, please also take a look at the travis failure.

OK, thanks.


Uuid Validator

I realise there's an open road to ZF3 PR but here's what I'd consider some low-hanging fruit in case you're planning any other 2.x releases.
NOTE: I'd be happy to upgrade this to the new format once the new API for validators is stabilised in that other PR. So if you don't think you'll release this in a new 2.x release let me know and I'll rebase the work against the ZF3 WIP branch.

Thanks, @gsomoza !

It's nothing! Happy to help a little bit :)


Added option top overwrite the default classes globally

This helps me when I need to have multiple quickedit elements on the page and want to be able to do 2 things:

Not having to pass a classes element to each component
When using the component without compiling Vue, so that these can be defined on a runtime level.

Thanks for your PR, I'll look into this. I would have liked to discuss this in an issue.

Cleaning up, closing unused PRs


Fix agent saving issue

Fixes and closes mitre/caldera#2907

Also closes https://github.com/mitre/caldera/issues/2953


K-Nearest Neighbor (kNN) Functionality

Find the K-nearest neighbors to a specific point or a set of points

KNN is implemented in PointRDD by Jinxuan Wu.


listProductFirmware returns "response" instead of parsed response

was expecting the promise to return:
[ fw_1, fw_2, ... etc]
instead promise returns:
{ statusCode: 200, body: [fw_1, fw_2, ... etc] }
whichever we use, we should document it

was expecting the promise to return
[ fw_1, fw_2, ... etc]

best i can tell pretty much all GET and HEAD requests return a basic response object (e.g. { { statusCode, body }

whichever we use, we should document it

lol another one i logged a few days back 🤝
https://app.clubhouse.io/particle/story/45778/improve-jsdoc-data
...if you have specific requests, feel free to add them to that story.
but, yes, we definitely need to do better across the board. thanks for the report  :pray::+1:


Support for azure acr helm repositories

This adds some basic support for helm repositories hosted on azure container registry (not OCI but classic ones. Add a new element to RepositorySpec to state that is externally managed and runs the az-cli command instead of the helm one.

Just to solve conflict? In that case I prefer merge from upstream. Any problema with that?

@javiplx That's fine, too!

Error is connection related while building image, but I have no permission to retry.

Triggered a rerun of the failed job.


[RFC] Error handling -- partial data

Description
For previous episodes on error handling, visit https://github.com/apollographql/apollo-kotlin/issues/4711
Goal
Simplify the error handling which is currently based on 3 different properties of ApolloResponse:

response.data for GraphQL data
response.errors for GraphQL errors
response.exception for network exceptions

This is confusing and ideally, we want ApolloResponse to be a result-like class with either data or an exception. There is a lot of brain muscle around result-like classes. Kotlin stdlib has Result, Arrow has Either, Rust has Result, etc... So it feels like a natural fit here.
Proposal: @catch client directive
Inspired by Relay error handling, introduce a @catch client directive that allows user to opt-in error handling on a given field.
If this field or any sub-field contains an entry in the errors array then this error is exposed to the user and no data is shown for that field.
Advantages: data contains everything needed to display the UI. There's no need to go back to the errors array because the error is now inline at the position where it happens. We could keep the errors arrays for advanced use cases but in the very large majority of use cases, it shouldn't be needed. We can rely on data or exception again.
A @catch field is modeled in generated models as a sealed interface and matching result types:
Schema:
type Query {
  product: Product!
}

type Product {
  name: String!
  price: Float!
}

Query:
{
  product @catch {
    name
    price
  }
}

Kotlin codegen:
class Data(val product: ProductResult)

sealed interface ProductResult
class ProductSuccess(val product: Product): ProductResult
class ProductError(val error: Error): ProductResult

class Product(val name: String, val price: Double)

Examples
Product error
Query:
{
  product @catch {
    name
    price
  }
}

Error Response:
{
  "errors": [
    {
      "message": "Cannot resolve product",
      "path": ["product"]
    }
  ],
  "data": {
    "product": null
  }
}

Kotlin test:
val response = apolloClient.query(Query()).execute()
val product = response.data.product
check(product is ProductError)
check(product.error.path == listOf("product"))

price error
If a nested fails, the error is exposed on the closest enclosing @catch field:
{
  "errors": [
    {
      "message": "Cannot resolve product price",
      "path": ["product", "price"]
    }
  ],
  "data": {
    "product": {
      "name": "Champagne",
      "price": null
    }
  }
}

Kotlin test:
val response = apolloClient.query(Query()).execute()
val product = response.data.product
check(product is ProductError)
check(product.erro.path == listOf("product"))

I like @catch! It allows you to decide if/where you want to deal with partial results.
To be discussed: for apps that never want partial results, we could think of a global setting where data is a Result too, to avoid having to set @catch on all queries.
Tangential remark: if we feel errors vs exception contributes to the confusion, we could remove (well, deprecate first!) errors, let GraphQL errors only be surfaced in exception as ApolloGraphQLException?


Allow Fargate platform version to be set

Fargate v1.4.0 is out, however the LATEST version still defaults to v1.3.0. In order to use v1.4.0 you need to set the platform_version variable to 1.4.0

Thanks a lot for the PR @shmick! 🚀
I have updated the docs and bump up the version.


Boo.Lang ported to DNXCORE

Hey, thanks for the PR!  This looks really good; just a couple notes:

In several places you added a new using clause to include namespaces that are only used inside conditional code.  Can you make those using clauses conditional too?

workspace.xml is massive and appears to be composed mostly of historical data.  Is there any way to clean that up a little?

Having Boo run on .NET Core would be great, and this looks like an important first step.  Thanks for putting in the effort to get the ball rolling on this! 😀

Hi,
Sure, I'll fix it today.

Awesome, thanks!

Hi, I did the changes but it is still failing. Do you have an idea why? I am not also sure, but seems to me that after compilation in VS some other files have been regenerated...

Hi, I did the changes but it is still failing. Do you have an idea why? I am not also sure, but seems to me that after compilation in VS some other files have been regenerated...

The conditionals look good, but why did you change the parser?
Don't worry about the failing tests; the build machine's test runner is b0rked.

The conditionals look good, but why did you change the parser?
Don't worry about the failing tests; the build machine's test runner is b0rked.

Hi,
I didn't do anything special. I just executed nant and I think this is the reason. Should I remove these changes? I am not sure. Why executing nant changed the parser. Was it regenerated properly before?

Yeah.  As a general rule of thumb, if you don't know why a change is there and it had nothing to do with what you were working on, you'd be better off reverting it rather than committing it.  It looks like this changes the semantics of the parser in at least one case.

All right, that latest looks good.  Merging. 😃

Also, feel free to drop by the Boo dev group and talk about your work on this.


fix: support for WOFF2 and TTF

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


add badges and tooltips for deleted products

I want to merge this change because it adds information badges to deleted product variants in order details.
PR intended to be tested with API branch:
Pull Request Checklist

[x] This code contains UI changes
[x] All visible strings are translated with proper context including data-formatting
[ ] Attributes [data-test-id] are added for new elements
[ ] Changes are mentioned in the changelog
[ ] The changes are tested in different browsers and in light/dark mode

Test environment config

API_URI=https://stable.staging.saleor.cloud/graphql/

Task linked: SALEOR-4136 Handle removed variants in Order Fulfilment page

I want to merge this change because:

it disables fulfill button in order view if all of the unfulfilled products are deleted from the database (partial fulfill is possible if some of the products are still in the database),
it adds badges on unconfirmed orders,
it adds badges on refund view.

Branch has been rebased onto newest 3.0 release.
Screenshots

Added badges on return/replace view and possibility to return removed products.
Before:

After:


[AgentService] Fix invalid argument passing on reruns

Looks like the author of https://github.com/microsoft/azure-pipelines-agent/pull/2353 confused slice method with splice.
Changing process.argv values causes the script to lose some of the passed arguments. Thus, subsequent runs of the listener process are performed with a different set of arguments. This issue only affects linux and darwin agents.
Agent service for windows does not support passing parameters and does not need to be changed.

Tested it and everything works as expected


Change text color for extra variables and dimensions

When extra variables and dimensions are detected, the text displayed is the same color as the surrounding text. It's a good idea to change the color of the output text (maybe red?)

fixed by d9ba7eb.


New board support

Updating board list and descriptions with new assets to support new 4chan boards.

Hey, how did you get the project to build? I had an unspecified error when trying to build vpx_lib.

The project didn't work for me out of the box so there was a bunch of things I had to adjust on my side. IIRC my vpx_lib issues went away when I downloaded VS2012 because of some missing dll.
Do you have any other output? Trying to jog my memory.

I have VS2015 Community. When I try to build vpx_arm I get to the step "Assembling vp9_reconintra_neon_asm.asm.s" and then get "error MSB6006: "cmd.exe" exited with code -1073741515". Trying to build vpx_win32 I get a lot of messages about the yasm tool being missing (those messages aren't present when trying to build the ARM version). Got any ideas?
Also, where'd you get those board images? They're perfect.

Ah, I see. Yeah, the cmd.exe exit code -1073741515 was caused by a missing dll called msvsdis110.dll (%PROGRAMFILES%\Microsoft Visual Studio 11.0\Common7\IDE\msvcdis110.dll), but that error is obfuscated unless you try building it via the developer console. You could try downloading the VS2012 Redistribution Package or VS2012 Express to get that dll.
I never got the vpx_win32 variant building so I only tested it on the actual phone. I tried downloading yasm and cygwin and pointing at those tools, but it wouldn't take. Would have to see how pantaloons set up his build most likely.
As for the pictures, I pulled /trash/ and /aco/ from the board's sticky. The rest were images I searched for that were inspired by the general topics on the board at the time.

Interesting. Thanks for that! I'll see if I can get it building, I would love to fix the captcha problem. Hope pantaloons takes notice and updates the app on the store. At the very least I can have an updated version!

Just in case you're curious, the VS2012 redist didn't work, I had to install VS2012 Express. However now the nestegg library is complaining about requiring a retarget to WP8.1, did you have to do that? It doesn't bother to explain why that would be required.

Also, when the app runs it fails to load any threads because it can't find System.Xml.Xpath. I tried removing the reference to Xpath and it doesn't seem like it's used anywhere in the code but regardless of how many times I clean and rebuild it still crashes when entering Post.Merge(APIPost) from Post.cs:206. Did you encounter this?

It prompts for a retarget when you open the solution for the first time, so if you accidentally said yes that could be part of it.
As for the Xml path issue, I never encountered it myself. You may need to download a certain .NET or Phone SDK version to reference it. I can't say for sure without a more specified error.
I wouldn't remove a reference for the sake of consistency with either future syncs or potential pull requests.
-----Original Message-----
From: "Alex Van Liew"<EMAIL_ADDRESS>Sent: 1/11/2016 6:23 PM
To: "pantaloons/4charm"<EMAIL_ADDRESS>Cc: "RailsRoyale"<EMAIL_ADDRESS>Subject: Re: [4charm] New board support (#14)
Also, when the app runs it fails to load any threads because it can't find System.Xml.Xpath. I tried removing the reference to Xpath and it doesn't seem like it's used anywhere in the code but regardless of how many times I clean and rebuild it still crashes when entering Post.Merge(APIPost) from Post.cs:206. Did you encounter this?
—
Reply to this email directly or view it on GitHub.

All the literature I've found online indicates that System.Xml.Xpath simply can't be used in Windows Store or Windows Phone apps, . I see the package in NuGet but it won't install because it's not targeting the full .NET framework. My copy of VS2015 community just says "referenced component cannot be found" but I think HTMLAgilityPack depends on it (which is why it's strange that that installation went fine). What version of VS are you using?

I'm using VS2013 Ultimate, but I don't think that's the root of it.
I did have a problem with NuGet already existing in the directory (.nuget) which is an old version which caused my package manager to throw a fit. I deleted that folder and updated the NuGet manager in VS itself which then fixed all of my dependent packages like HTMLAgilityPack.
If you have the pre-existing .nuget folder you could remove it from the directory temporarily and see if NuGet allows some sort of update that could potentially fix your problem.

VS2015 didn't complain about NuGet at all, weird that yours did. Deleting the folder didn't help but I found a package by Microsoft System.Xml.XPath.SL4 that appears to add the appropriate assembly (it's a copy of an assembly found in the Silverlight 4 SDK), and HTMLAgilityPack mentions that a reference to that particular assembly is required for WP in its readme. That's pretty much all I needed (other than the file from VS2012), now it works. Thanks for your help. I'm going to make a pull request with a few minor changes I made to make it build, would you try them on your machine to see if it still works on earlier versions of VS?

Go ahead. I'll give it a shot in the next few days and see if it builds when I have time.
Ideally pantaloons will get on at some point and we can update the solutions and targets to 8.1 and perhaps a Win10 variant.
I hope he merges this change at some point because I have a potential solution for the captcha problem but I would like his opinion to see if it's appropriate.
-----Original Message-----
From: "Alex Van Liew"<EMAIL_ADDRESS>Sent: 1/12/2016 1:53 AM
To: "pantaloons/4charm"<EMAIL_ADDRESS>Cc: "RailsRoyale"<EMAIL_ADDRESS>Subject: Re: [4charm] New board support (#14)
VS2015 didn't complain about NuGet at all, weird that yours did. Deleting the folder didn't help but I found a package by Microsoft System.Xml.XPath.SL4 that appears to add the appropriate assembly (it's a copy of an assembly found in the Silverlight 4 SDK), and HTMLAgilityPack mentions that a reference to that particular assembly is required for WP in its readme. That's pretty much all I needed (other than the file from VS2012), now it works. Thanks for your help. I'm going to make a pull request with a few minor changes I made to make it build, would you try them on your machine to see if it still works on earlier versions of VS?
—
Reply to this email directly or view it on GitHub.

I took a peek at updating to 8.1 and it's a bit more involved than one would hope. The "Retarget to Windows Phone 8.1" option retargets it as a Silverlight app, and as luck would have it you can't use System.Net.Http in a Silverlight app, only in a Runtime app. I wasn't able to find any docs about how to actually upgrade to a Runtime app, though, just differences. Upgrading to Silverlight would mean rewriting a good chunk of the HttpClient code.
I have my own thoughts about a captcha fix. We should probably discuss them in #12, though. I'll comment my thoughts and findings there once I write them up.


undefined:2:1230: property missing ':' when using background with Data URI

It appears that the parsers is not parsing correctly CSS which contain Data URIs. In the below there are 2 examples where one succeed and another one fails.
The only difference is that the classes with dataURI are on 2 seperate lines.
CSS version: 3.0.0

const css = require('css')

const cssContent1 = `
.tox-rtc-remote-image {background:url("data:image/svg+xml;charset=UTF-8,%3Csvg%2width%3D%2236%22%20height%3D%2212%22%20viewBox%3D%220%200%2036%2012%22%20xmlns%3D%22http%3A%2F%2Fwww.w3.org%2F2000%2Fsvg%22%3E%0A%20%20%3Ccircle%20cx%3D%226%22%20cy%3D%226%22%20r%3D%223%22%20fill%3D%22rgba(0%2C%200%2C%200%2C%20.2)%22%3E%0A%20%20%20%20%3Canimate%20attributeName%3D%22r%22%20values%3D%223%3B5%3B3%22%20calcMode%3D%22linear%22%20dur%3D%221s%22%20repeatCount%3D%22indefinite%22%20%2F%3E%0A%20%20%3C%2Fcircle%3E%0A%20%20%3Ccircle%20cx%3D%2218%22%20cy%3D%226%22%20r%3D%223%22%20fill%3D%22rgba(0%2C%200%2C%200%2C%20.2)%22%3E%0A%20%20%20%20%3Canimate%20attributeName%3D%22r%22%20values%3D%223%3B5%3B3%22%20calcMode%3D%22linear%22%20begin%3D%22.33s%22%20dur%3D%221s%22%20repeatCount%3D%22indefinite%22%20%2F%3E%0A%20%20%3C%2Fcircle%3E%0A%20%20%3Ccircle%20cx%3D%2230%22%20cy%3D%226%22%20r%3D%223%22%20fill%3D%22rgba(0%2C%200%2C%200%2C%20.2)%22%3E%0A%20%20%20%20%3Canimate%20attributeName%3D%22r%22%20values%3D%223%3B5%3B3%22%20calcMode%3D%22linear%22%20begin%3D%22.66s%22%20dur%3D%221s%22%20repeatCount%3D%22indefinite%22%20%2F%3E%0A%20%20%3C%2Fcircle%3E%0A%3C%2Fsvg%3E%0A");}

.mce-spellchecker-word {background:url("data:image/svg+xml;charset=UTF-8,%3Csvg%20width%3D'4'%20height%3D'4'%20xmlns%3D'http%3A%2F%2Fwww.w3.org%2F2000%2Fsvg'%3E%3Cpath%20stroke%3D'%23ff0000'%20fill%3D'none'%20stroke-linecap%3D'round'%20stroke-opacity%3D'.75'%20d%3D'M0%203L2%201%204%203'%2F%3E%3C%2Fsvg%3E%0A");}

`
css.parse(cssContent1); // Object {type: "stylesheet", stylesheet: Object {source: undefined, rules: [, ], parsingErrors: []}}

const cssContent2 = `
.tox-rtc-remote-image {background:url("data:image/svg+xml;charset=UTF-8,%3Csvg%2width%3D%2236%22%20height%3D%2212%22%20viewBox%3D%220%200%2036%2012%22%20xmlns%3D%22http%3A%2F%2Fwww.w3.org%2F2000%2Fsvg%22%3E%0A%20%20%3Ccircle%20cx%3D%226%22%20cy%3D%226%22%20r%3D%223%22%20fill%3D%22rgba(0%2C%200%2C%200%2C%20.2)%22%3E%0A%20%20%20%20%3Canimate%20attributeName%3D%22r%22%20values%3D%223%3B5%3B3%22%20calcMode%3D%22linear%22%20dur%3D%221s%22%20repeatCount%3D%22indefinite%22%20%2F%3E%0A%20%20%3C%2Fcircle%3E%0A%20%20%3Ccircle%20cx%3D%2218%22%20cy%3D%226%22%20r%3D%223%22%20fill%3D%22rgba(0%2C%200%2C%200%2C%20.2)%22%3E%0A%20%20%20%20%3Canimate%20attributeName%3D%22r%22%20values%3D%223%3B5%3B3%22%20calcMode%3D%22linear%22%20begin%3D%22.33s%22%20dur%3D%221s%22%20repeatCount%3D%22indefinite%22%20%2F%3E%0A%20%20%3C%2Fcircle%3E%0A%20%20%3Ccircle%20cx%3D%2230%22%20cy%3D%226%22%20r%3D%223%22%20fill%3D%22rgba(0%2C%200%2C%200%2C%20.2)%22%3E%0A%20%20%20%20%3Canimate%20attributeName%3D%22r%22%20values%3D%223%3B5%3B3%22%20calcMode%3D%22linear%22%20begin%3D%22.66s%22%20dur%3D%221s%22%20repeatCount%3D%22indefinite%22%20%2F%3E%0A%20%20%3C%2Fcircle%3E%0A%3C%2Fsvg%3E%0A");} .mce-spellchecker-word {background:url("data:image/svg+xml;charset=UTF-8,%3Csvg%20width%3D'4'%20height%3D'4'%20xmlns%3D'http%3A%2F%2Fwww.w3.org%2F2000%2Fsvg'%3E%3Cpath%20stroke%3D'%23ff0000'%20fill%3D'none'%20stroke-linecap%3D'round'%20stroke-opacity%3D'.75'%20d%3D'M0%203L2%201%204%203'%2F%3E%3C%2Fsvg%3E%0A");}

`
css.parse(cssContent2); // undefined:2:1230: property missing ':'

Another issue is with keyframes here:
https://github.com/TinkoffCreditSystems/taiga-ui/issues/485

after some digging, I found that each style line has more than one semicolon (;) will break to more than one line which leads to different errors

We are facing issues with this too. We use devextreme UI components and their theme css file gives this error.
D:\TempProjects\angular12Devextreme>ng build --configuration=production √ Browser application bundle generation complete. √ Copying assets complete. × Index html generation failed. undefined:1:111670: property missing ':'
Hope for a fix soon.

Any updates?

Last release was a year ago and there are loads of open PRs, so I wouldn't expect much from this library. @alan-agius4 maybe it's a good idea to fork this project since it's no longer maintained?


Bannered Mare counts as trespassing at night

Describe the bug
The Bannered Mare was locked and counts as trespassing at night. Hulda gives me a bounty, so it isn't just a locked door, the game thinks I am trespassing.
To Reproduce
Steps to reproduce the behavior:
Enter the Bannered Mare at night (I'm assuming after 8 pm) It was 1 am when I tried
Expected behavior
Not to be trespassing.
Screenshots
If applicable, add screenshots to help explain your problem.
Save
saves.zip
Crash log
If applicable, add a save to reproduce your problem. You can find them at My Documents/My Games/Skyrim VR/SKSE/ where they'll be named something like crash-2021-12-20-23-21-37.log.
Save
If applicable, add save to reproduce your problem.
Desktop (please complete the following information):

OS version: Windows 10
RAM: 32GB Ram
CPU: i7-11800H
GPU: Nvidia RTX 3070
Storage type Narsil is installed on: SSD
Antivirus MalwareBytes, Windows Defender

Additional context
Add any other context about the problem here.

I went to the Bannered Mare on your save and it seemed to be locked, might be why it was seen as trespassing? I went in during the day and waited inside though and wasn't kicked out, so maybe with the removal of the trespass mod I added last minute this gets fixed. Will keep this here until the next hotfix update.

oh FYI, if a door of a shop/inn is closed, sneaking in will count as trespassing, that's by design. The issue would be more on why it's locked, but I don't see anything in my esps that could cause the door to lock itself, so I wonder if it's just vanilla behaviour?

I encountered the locked door also.  The side door wasn't locked, and they didn't seem to care that I was inside.  However, the alchemy shop did a weird thing.  I was buying stuff in the middle of the day.  After I finished, the vendor lady walked around the counter and started asking me to leave.  So, I left.  I'll grab a screenshot or make a save if it happens again.  Or just wait till the fix.

Oh it's possible you entered through the back door, which is why they would be kicking you out. I'm temporarily removing that mod anyway because there's some locations it covers that have entirely different layouts in Holds.


Contract setup can fail for one but succeed for the other party

Following scenario:

Maker and Taker go into contract setup
Either party sends Msg2 but does not receive Msg2 (before the timeout)
The party that received both messages will continue to publish lock, the party other party will fail in contract setup and transition to SetupFailed upon restart (and then does not get any updates on the Cfd anymore)

This is critical, because the party for which the contract setup failed won't get any information on locked up funds in case this happens. Eventually, when the party for which it worked published commit and cet the party for which the setup failed will receive back the funds, but that just randomly happens without the party knowing.
First, we could argue, that we should remove Msg2 from the contract setup and do it afterwards. But that does not really solve the problem as a whole, because we would then still have the problem for Msg1.
What we need, is to persist all the information we have before we send Msg1, because after doing so the other party is able to publish the lock transaction. So, upon restart we have to monitor for lock, in case we have sent Msg1 (irregardless of the contract setup failing).
I think it is OK to keep Msg2 in the mix, although we could also move that part of the code outside the contract setup.
We could generalize this problem into:
Once you signed something and sent it to the other party we have to make sure that we have that state persisted so we can reason about it. We could solve part of this by extending the broadcast in the wallet.

Tape-workaround: Add Ack messages to both actors at the end and only publish if ack was received

@itchymax @bonomat @thomaseizinger
I'll see to add the ACK messages and discard the complete setup (on both maker and taker) side in case we did not get the ACK from the other party.
This is a temporary solution that could be exploited by a counterparty that removes sending the final ACK. No money can be "stolen", but the rogue counterparty could "hold the funds hostage". I think this is good enough for now. In our event-model refactor we already introduce error scenarios where we pass out the incomplete DLC for monitoring, so it should be easier to add a more complete fix.

Removed from Umbrel milestone becuase #724 was implemented as workaround. More proper fix is expected to be implemented as part of the refactoring towards event model. Keeping the ticket open until that goes in.

Having a final ack-message exchange has minimized failures during contract setup. But eventually we should still tackle this in a more resilient way (dealing with monitoring of half-finished DLCs). Not sure when to schedule that in though.

closed in favor of https://github.com/itchysats/itchysats/issues/2563


Consolidate contrib docs and fix link

Description

Fixes link to testing doc
Consolidates the two places we document contributions

GitHub Issues
Some discussion on updating contrib docs in #407 and the initial PR was in #408

Coverage remained the same at 77.118% when pulling dfa38a05ff7f400890e5d524791342b640b86d78 on tophatmonocle:contribution-typos into d2a524df8efea020019c964355cbf44512b3f360 on Miserlou:master.

I prefer having it in a single document, sorry about that. I always try to minimize the amount of cruft and legacy in a project..

I prefer having it in a single document, sorry about that. I always try to minimize the amount of cruft and legacy in a project..

Would you prefer removing the 2nd contrib.rst doc then? I can consolidate in the other direction :)

Oh man, I didn't even know about that file.
This is in fact directly related to #243 - basically somebody needs me to teach me how and why to publish to ReadTheDocs..


feat(sql_views): added views as datasets for SQLAlchemy DBs

Tested with MS SQL Server, this solution should work with other SQLAlchemy manageable DBs.
Now the ingestion recipe accept, optionally, two more options:
include_views: False
include_tables: True

To assure retro-compatibility they can be omitted, and in such case the default arguments are those ones expressed up here.
Checklist

[X] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[-] Links to related issues (if applicable)
[-] Tests for the changes have been added/updated (if applicable)
[X] Docs related to the changes have been added/updated (if applicable)

I tested this on Snowflake as well and it worked! Thanks @vlavorini for putting this together


ERROR while training - runpy.py", line 193, in _run_module_as_main     "main", mod_spec)

Hi there,
i'm trying to train ludwig with images, my dataset.csv looks like this:
image_path,class
dataset/chest_xray/train/NORMAL/IM-0115-0001.jpeg,normal
dataset/chest_xray/train/NORMAL/IM-0117-0001.jpeg,normal
my model_definition.yaml:
input_features:
-
name: image_path
type: image
output_features:
-
name: class
type: category
training:
epochs: 5
and when i try to train i get the error:
Using full raw csv, no hdf5 and json file with the same name have been found
Building dataset (it may take a while)
Traceback (most recent call last):
File "c:\users\anaconda3\envs\ludwig\lib\runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "c:\users\anaconda3\envs\ludwig\lib\runpy.py", line 85, in run_code
exec(code, run_globals)
File "C:\Users\Anaconda3\envs\ludwig\Scripts\ludwig.exe_main.py", line 9, in
File "c:\users\anaconda3\envs\ludwig\lib\site-packages\ludwig\cli.py", line 94, in main
CLI()
File "c:\users\anaconda3\envs\ludwig\lib\site-packages\ludwig\cli.py", line 60, in init
getattr(self, args.command)()
File "c:\users\anaconda3\envs\ludwig\lib\site-packages\ludwig\cli.py", line 70, in train
train.cli(sys.argv[2:])
File "c:\users\anaconda3\envs\ludwig\lib\site-packages\ludwig\train.py", line 764, in cli
full_train(**vars(args))
File "c:\users\anaconda3\envs\ludwig\lib\site-packages\ludwig\train.py", line 252, in full_train
random_seed=random_seed
File "c:\users\anaconda3\envs\ludwig\lib\site-packages\ludwig\data\preprocessing.py", line 403, in preprocess_for_training
random_seed
File "c:\users\anaconda3\envs\ludwig\lib\site-packages\ludwig\data\preprocessing.py", line 525, in _preprocess_csv_for_training
random_seed=random_seed
File "c:\users\anaconda3\envs\ludwig\lib\site-packages\ludwig\data\preprocessing.py", line 62, in build_dataset
**kwargs
File "c:\users\anaconda3\envs\ludwig\lib\site-packages\ludwig\data\preprocessing.py", line 90, in build_dataset_df
global_preprocessing_parameters
File "c:\users\anaconda3\envs\ludwig\lib\site-packages\ludwig\data\preprocessing.py", line 165, in build_data
preprocessing_parameters
File "c:\users\anaconda3\envs\ludwig\lib\site-packages\ludwig\features\image_feature.py", line 215, in add_feature_data
data[feature['name']][i, :, :, :] = img
ValueError: could not broadcast input array from shape (1152,1422,1) into shape (1858,2090,1)
Can you please help me understand what i'm doing wrong?
Thank you very much!
Regards,
João

either you reshape your images during preprocessing as explained here or all the images you provide have to be of the same size.Moreover, the size of your images is way too big, depending on the task I suggest not to exceed 256 x 256.

either you reshape your images during preprocessing as explained here or all the images you provide have to be of the same size.Moreover, the size of your images is way too big, depending on the task I suggest not to exceed 256 x 256.

Thank you for your reply.
so i've tested out what you've said and still got the same error. i've changed the model_definition.yaml to this:
input_features:
-
name: image_path
type: image
height: 256
width: 265
output_features:
-
name: class
type: category
training:
epochs: 5

I just updated that section to reflect this possible misunderstanding.
To resize the image you have to use the preprocessing part of the model definition:
input_features:
  -
    name: image_path
    type: image
    preprocessing:
      height: 256
      width: 265

thank you for your help!
i've changed the model_definition.yaml for what you've said and now i'me getting this error:
Using full raw csv, no hdf5 and json file with the same name have been found
Building dataset (it may take a while)
Traceback (most recent call last):
File "c:\users\anaconda3\envs\ludwig\lib\runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "c:\users\anaconda3\envs\ludwig\lib\runpy.py", line 85, in run_code
exec(code, run_globals)
File "C:\Users\Anaconda3\envs\ludwig\Scripts\ludwig.exe_main.py", line 9, in
File "c:\users\anaconda3\envs\ludwig\lib\site-packages\ludwig\cli.py", line 94, in main
CLI()
File "c:\users\anaconda3\envs\ludwig\lib\site-packages\ludwig\cli.py", line 60, in init
getattr(self, args.command)()
File "c:\users\anaconda3\envs\ludwig\lib\site-packages\ludwig\cli.py", line 70, in train
train.cli(sys.argv[2:])
File "c:\users\anaconda3\envs\ludwig\lib\site-packages\ludwig\train.py", line 764, in cli
full_train(**vars(args))
File "c:\users\anaconda3\envs\ludwig\lib\site-packages\ludwig\train.py", line 252, in full_train
random_seed=random_seed
File "c:\users\anaconda3\envs\ludwig\lib\site-packages\ludwig\data\preprocessing.py", line 403, in preprocess_for_training
random_seed
File "c:\users\anaconda3\envs\ludwig\lib\site-packages\ludwig\data\preprocessing.py", line 525, in _preprocess_csv_for_training
random_seed=random_seed
File "c:\users\anaconda3\envs\ludwig\lib\site-packages\ludwig\data\preprocessing.py", line 62, in build_dataset
**kwargs
File "c:\users\anaconda3\envs\ludwig\lib\site-packages\ludwig\data\preprocessing.py", line 90, in build_dataset_df
global_preprocessing_parameters
File "c:\users\anaconda3\envs\ludwig\lib\site-packages\ludwig\data\preprocessing.py", line 165, in build_data
preprocessing_parameters
File "c:\users\anaconda3\envs\ludwig\lib\site-packages\ludwig\features\image_feature.py", line 213, in add_feature_data
user_specified_num_channels
File "c:\users\anaconda3\envs\ludwig\lib\site-packages\ludwig\features\image_feature.py", line 110, in _read_image_and_resize
num_channels))
ValueError: Image C:\Users\Anaconda3\envs\ludwig\data\pneumonia\dataset\chest_xray\train\PNEUMONIA\person1_bacteria_1.jpeg has 3 channels, unlike the first image, which has 1 channels

@absint0o So it looks like some of your images only have 1 channel (greyscale) while others have 3 channels. This is expected behavior. There are two solutions here:

Set num_channels: 3 in the preprocessing section. Ludwig will append channels with zeros for the images that have <3 channels.
Fix the number of channels for all your images in the training data.

@absint0o So it looks like some of your images only have 1 channel (greyscale) while others have 3 channels. This is expected behavior. There are two solutions here:
1. Set num_channels: 3 in the preprocessing section. Ludwig will append channels with zeros for the images that have <3 channels.

2. Fix the number of channels for all your images in the training data.

Hi there,
so i have more images with 1 channel than images with the 3 channels so i've set my num_channels to 1 so it ignores the images with 3 channels.
There's any way that in preprocessing ludwig converts the 3 channel images to 1 channel image?
offtopic subject: Ludwig is amazing! congrats to all!

@absint0o thank you! At the moment if you specify 1 channel and the image had 3 channels, Ludwig uses just the first channel.
I just created a pull request that applies greyscaling instead of just using the first channel: https://github.com/uber/ludwig/pull/339

Just merged the PR. Now if you specify num_channels: 1 and your images have more than 1 channel, they are converted to greyscale.

@absint0o thank you! At the moment if you specify 1 channel and the image had 3 channels, Ludwig uses just the first channel.
I just created a pull request that applies greyscaling instead of just using the first channel: #339

Thank you for your consideration.
i have another two question:
it´s possible to improve a previous trained model without creating a new one?
it's possible to re-enforce the model accuracy by "telling" which prediction was correct and incorrect?
i also would like to make a sugestion: it would be great if ludwig had a forum so people can debate and discuss more than just errors or issues.
thank you very much!
Regards

@absint0o you can both resume the training of a model with --model_resume_path or load a model to use as initialization of a new model with --model_load_path.
You can find the, admittedly not well advertised, forum on Gitq.

@absint0o you can both resume the training of a model with --model_resume_path or load a model to use as initialization of a new model with --model_load_path.
You can find the, admittedly not well advertised, forum on Gitq.

sorry, it's on the user guide and i've missed it out!
i also have to define the path for the dataset as i normally do when do a new training right?
thank you and i'll check the ludwig forum!

Remove \model at the end of the path ;)

Nice! Thank you very much, I will try it out!


renga status fails after an update

running renga status after a renga update results in:
renga status
Traceback (most recent call last):
  File "/Users/rok/Projects/renga-python/renga/cli/_graph.py", line 177, in add_tool
    tool = CommandLineTool.from_cwl(cwl)
  File "/Users/rok/Projects/renga-python/renga/models/cwl/_ascwl.py", line 39, in from_cwl
    'Got {0} but need {1}'.format(class_name, cls.__name__)
TypeError: Got Workflow but need CommandLineTool

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/rok/.virtualenvs/renga-demo/bin/renga", line 11, in <module>
    load_entry_point('renga', 'console_scripts', 'renga')()
  File "/Users/rok/.virtualenvs/renga-demo/lib/python3.6/site-packages/click/core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "/Users/rok/.virtualenvs/renga-demo/lib/python3.6/site-packages/click/core.py", line 697, in main
    rv = self.invoke(ctx)
  File "/Users/rok/.virtualenvs/renga-demo/lib/python3.6/site-packages/click/core.py", line 1066, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/Users/rok/.virtualenvs/renga-demo/lib/python3.6/site-packages/click/core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Users/rok/.virtualenvs/renga-demo/lib/python3.6/site-packages/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "/Users/rok/.virtualenvs/renga-demo/lib/python3.6/site-packages/click/decorators.py", line 64, in new_func
    return ctx.invoke(f, obj, *args[1:], **kwargs)
  File "/Users/rok/.virtualenvs/renga-demo/lib/python3.6/site-packages/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "/Users/rok/.virtualenvs/renga-demo/lib/python3.6/site-packages/click/decorators.py", line 17, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/usr/local/Cellar/python/3.6.5/Frameworks/Python.framework/Versions/3.6/lib/python3.6/contextlib.py", line 52, in inner
    return func(*args, **kwds)
  File "/Users/rok/Projects/renga-python/renga/cli/status.py", line 38, in status
    status = graph.build_status(revision=revision)
  File "/Users/rok/Projects/renga-python/renga/cli/_graph.py", line 315, in build_status
    self.add_file(filepath, revision=revision)
  File "/Users/rok/Projects/renga-python/renga/cli/_graph.py", line 223, in add_file
    self.add_tool(commit, cwl, file_key=file_key)
  File "/Users/rok/Projects/renga-python/renga/cli/_graph.py", line 181, in add_tool
    commit, path, file_key=file_key, cwl=cwl
  File "/Users/rok/Projects/renga-python/renga/cli/_graph.py", line 140, in add_workflow
    self.G.add_edge(input_map[source], tool_key, id=input_id)
KeyError: 'step_1/output_stdout'

It seems like you have run the older version of the code. See https://github.com/SwissDataScienceCenter/renga-python/blob/development/renga/cli/_graph.py#L176-L183


Fix bug in Mapper.load1kVromBank

This is used by MMC3 and MMC5.  I haven't actually noticed any issues while playing, though I suspect it's because if the MMC3 game I've been playing (Crystalis) even uses this function at all, it gets by with only using the Tile data, which is being copied correctly.

I haven't studied how vrom is used vs how the precompiled tiles are used, so maybe it's obvious, but if the vrom isn't used at all, then it probably shouldn't even be copied.  Otherwise, I'm not clear on where one is used vs the other.

Honestly, it's been 10 years since I wrote this, so I haven't a clue. It looks good in principle though, and doesn't break any MMC3 games I tried. Thanks!

@shicks Would you like to be a maintainer? I have so little time to maintain this, and it takes a lot of time to spin up all the NES internals in my head for every pull request. It would be a shame for it to fall into disrepair due to that.
No obligation to do work - just the power the merge pull requests and cut releases if you want to!
Same invitation goes to out to @malignantz and @BenShelton who have made some good contributions recently. :)

@bfirsh I'd be happy to maintain, I'm actually going to be using this library to continue a pet project of mine soon (hopefully) so will be happy to help out.

@BenShelton Awesome! Thank you!

I've actually got my own fork github.com/shicks/jsnesx that's diverging quite a bit - the main focus is to be a debugging and disassembly tool.  Not sure what difference that makes to you.  If you want to make me a maintainer, that's fine with me.

Oh nice! I didn't see that.
Is there anything you think would be worthwhile upstream? Any debugging tools/hooks would be useful, I'm sure.
Made you a maintainer, anyway! Feel free to use, or not. :)


ts2phc "nmea: unable to find utc time in leap second table" error

Running ts2phc -s nmea sometimes gets the error
nmea: unable to find utc time in leap second table

Looks like this is caused by this bug in ts2phc, which was fixed after the release of linuxptp 3.1.1
https://github.com/richardcochran/linuxptp/commit/63fc1ef4fd5e5fc45dd4de3bf27920bb109a4357?diff=unified

Line in logs before nmea: unable to find utc time in leap second table is something like nmea delay:<PHONE_NUMBER>147466182 ns.  This number of nanoseconds is about the difference between now and 1970. This means that here
https://github.com/richardcochran/linuxptp/blob/6feb76186ac39eab2fc011053f1cf46bfc94c8e4/ts2phc_nmea_master.c#L216
m->rmc_time was (incorrectly) zero.  This fields is set in monitor_nmea_status from the ts field of struct nmea_rmc. The failing call to mktime will cause this not to be initialized.

Better link is
https://sourceforge.net/p/linuxptp/code/ci/63fc1ef4fd5e5fc45dd4de3bf27920bb109a4357/


Updated deprecated syntax category names

Updated deprecated name strLit  and numLit  to str  and num

Thanks, I pushed a different fix that removes the duplication between the snippets and the full code


NameIDFormat fix

https://github.com/bergie/passport-saml/issues/338
When identifierFormat: null the passport-saml generated metadata with a tag <NameIDFormat/> which causes an error in AD FS: The value 'NameIDFormat' must be an absolute URI.
Setting the identifierFormat: urn:oasis:names:tc:SAML:1.1:nameid-format:unspecified allows importing the metadata file successfully.

@sibelius This look OK to me. What do you think?


Acsessing web ui from lan

I'm trying to find a way to run this on my home server and access it from different computers on my LAN but because it sets up the web ui in loopback it is not possible.
I know I can set up reverse proxy to do it but I want to do it without extra applications.
I tried to find were in the code it sets it up but I just couldn't find it.
Is it possible to add support for this to be accessible from LAN?
Thanks

sed -i 's|^    webui.launch(share=share)|    webui.launch(server_name="<IP_ADDRESS>", share=share)|g' /fastsdcpu/src/frontend/webui/ui.py
execute the command above

This discussion helped me because I wanted to do the same thing.
By the way, the following appears at the startup.

To create a public link, set share=True in launch().

I tried adding -s to make share=True, so my web UI has been published to the whole world!
Either loopback only or publish externally is extreme, so it would be helpful to support an option to publish locally.

There is a much simpler solution - you can export GRADIO_SERVER_NAME=<IP_ADDRESS> before starting web UI.


[WIP] @orta: Lets users dismiss bid confirmation view by tapping checkmark

I saw a user do it at an auction once – it makes sense – it's the only saturated part on the screen, and she had been bidding already. Power user I guess :smile:
Fixes #427.

Let's get a test on this one, could be hard to remember why it exists in the future

with that, you're cool to merge :+1:

Cool cool.

Trigger a gesture recognizer in tests is proving to be tricky-isn. Gonna refactor into a loading VC view model.

Bit hairy, this. Lots  of non-obvious dependencies.

OK, so this PR got a lot bigger. I've done the following:

Created a view model for LoadingViewController and placed all network logic within it.
Extricated a lot of dependencies within the network models. These are now combined under the FulfillmentController protocol, to encourage loose coupling.
Added tests for new classes.
Tried to improve code commenting.

Still needs:

[ ] Adds tests for green checkmark a.k.a "let's get a test on this one."

While I'm at it, PlaceBidNetworkModel, BidCheckingNetworkModel, and BidderNetworkModel could use some tests. They currently have none :fearful:

Cool, looks good :+1:


Minor fixes

Please review carefully--I am not set up to actually run these changes, and there are no tests.
(Relatedly, it'd be nicer not to check in your go.mod pion replace directives.)

I just checked the doc https://golang.org/pkg/time/#Timer.Reset
Reset should be invoked only on stopped or expired timers with drained channels.

if !t.Stop() {
	<-t.C
}
t.Reset(d)

Does it mean that we can't use Reset() like this?

Huh. Good question. @ianlancetaylor, what is the preferred way to create a timer and then repeatedly kick it to some point further in the future, in performance sensitive code?

Sorry for the noise, Ian. I was confused about the interaction with the timer.Stop proposal.

@cedricfung please take another look. The new code is still significantly more performant: ~93ns/op instead of ~270ns/op, and no allocs instead of 208B/3allocs.

But I feel that <-timer.C will block the loop until timer expires?

sorry, my misunderstanding.


Issue Title: "Add Cursor-Following Animation"

@DeadmanAbir
Issue Description: The website currently lacks interactive design elements that enhance user experience. I propose adding a cursor-following animation to create dynamic interactions where design elements move along with the mouse cursor, providing a more engaging and visually appealing experience.
I’d like to work on adding this feature. Please assign this issue to me.

@DeadmanAbir Issue Description: The website currently lacks interactive design elements that enhance user experience. I propose adding a cursor-following animation to create dynamic interactions where design elements move along with the mouse cursor, providing a more engaging and visually appealing experience.
I’d like to work on adding this feature. Please assign this issue to me.

show us a preview of what you are trying to add or an example link.

@DeadmanAbir
https://github.com/user-attachments/assets/bd9a82b6-a3e8-4780-bc55-3f527768a292
like this, and there are many other styles for cursor animation

@DeadmanAbir
cursor6.mp4
like this, and there are many other styles for cursor animation

Hey while this thing looks good in web app, but it will going to be a complete mess when it comes to mobile app view, because in mobile the hover features doesn't work properly.

@DeadmanAbir that's ok,


Profiles Editor: dirty state not obvious in editor

Testing #217139
I liked that we're using the typical editor tab/titlebar dots to indicate dirty state in the editor. It wasn't obvious in the tree which item was dirty though. Perhaps we should use the dot or some other decoration on the italicized list item here?

It seems we do not show any indication in the explorer tree neither. Should this be a consistent experience everywhere?


Add local Run Inference with Gemma notebook

Adds a notebook example to run a Beam pipeline with RunInference and a Gemma open model locally.

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

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] 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 or the workflows README to see a list of phrases to trigger workflows.

R: @damccorm
Dataflow pubsub example is in-progress


fix(File): Doesn't show warning when rename file name to same name

What this PR does 📖
1. Doesn't show warning when rename file name to same name
https://github.com/Satellite-im/Uplink/assets/63157656/aa551775-ac60-444f-967c-c07fee131834
Which issue(s) this PR fixes 🔨

Resolve #983

Special notes for reviewers 🗒
Additional comments 🎤

neat, thanks! 🎉

what happens if a file is given the same name as a folder (and vice versa)? Allowing that could be problematic


determine malicious behavior and course of actions

Description

the node currently does not act on malicious miners. the only action taken right now are

when receiving a ballot, if the smesher has already published a ballot in the same layer
when receiving a ballot that has inconsistent votes

vote for multiple blocks within hdst
vote support/against for the same block
vote abstain on a layer but voted support/against for blocks in that layer

tortoise will consider such ballot has weight/height 0. are these action sufficient for mainnet launch?
also, there are other behaviors that should also be considered malicious. for example, a smesher that

publishes syntactically invalid atx/proposal/ballot
publishes multiple ATXs in the same epoch
publishes two ATXs of the same sequence number
...

we should create a list of behaviors that we consider malicious and decide the course of actions for them.

In particular there is a large hole that needs to be closed before genesis in atxs https://github.com/spacemeshos/go-spacemesh/blob/develop/activation/handler.go#L126-L129 . Otherwise same post can be reused for any number of atxs.
There is a solution that was used for ballots we can reuse the same for atxs, but as noted above it needs to be a part of some larger work so that we don't miss other holes like this.

@countvonzero @dshulyak what's the state on this? I think at some malicious behavior checks have already been implemented.
Is there still something that needs to be done and I can help with?

it would be great to generate a more comprehensive list of malfeasance behaviors (i think @noamnelke may have a doc for that) first, and go through that list to determine

what are we supposed to do in each scenario
what we are, if any, doing in each scenario
come up with a cohesive way to manage these if possible.

there are several issues and all of them are placeholders, i will close atleast some of them


[8.x] Allow passing a closure to Collection's when and unless

This allows passing a Closure as the conditional to when and unless:
return $this
    ->getOptions()
    ->when(fn ($options) => ! $options->has('share'))
    ->put('share', true);

Here's a real-world example, converting @jbrooksuk's Forge code snippet from this:
$jobData = $payload['data'];

if (! isset($jobData['initiated_by'])) {
    $jobData = array_merge($payload['data'], array_filter([
        'initiated_by' => request()->user()->id ?? null,
    ]));
}

return ['data' => $jobData];

...to a collection pipeline:
return collect($payload['data'])
    ->unless(fn ($data) => $data->has('initiated_by'))
    ->put('initiated_by', request()->user()->id ?? null)
    ->filter()
    ->pipe(fn ($data) => ['data' => $data->all()]);

:smiley:

P.S. While at it, I realized that unless didn't have support for the proxy (like when already does), so I added it.
This necessitated making $callback optional. Do we consider that a breaking change?

I realized this is a BC change, since you may have passed in a variable that could potentially hold a closure:
function foo($callback = null)
{
    $collection->when($callback)->map($callback);
}

:cry:


Pass the ReadClient to ReadClient::Callback::OnDone.

This allows a single callback to be used across multiple read clients
as needed, and is symmetric with how WriteClient and CommandSender work.
Problem
chip-tool needs to be able to tell which of multiple subscriptions it's dealing with has completed.
Change overview
See above.
Testing
Code compiles.  No other changes for now.

This seems to be the undo of #14013 (however partial since some changes were still needed ... eg #32257 looks to deal with lifecycle for OnError/OnDone).
Overall adding this comment to keep some track of history via github linking, however longer term we should try to remove the coupling that we have.


Evaluate Hosting Providers

Options to consider (not comprehensive):
https://www.elephantsql.com/plans.html
https://vercel.com/pricing
https://www.netlify.com/pricing/#core-pricing-table
https://pages.cloudflare.com/

Netlify

free plan offers generous resources
built-in form handling and identity services
easy setup and deployment of static sites and web applications
user-friendly interface


Torchscript execution failures (hangs, access violation, Fatal error. Internal CLR fatal error. (0x80131506) )

Repeated execution of Torchscript modules leads to consistent failures of one of these varieties, apparently due to memory corruption (within about 30 seconds of runtime).
This has now been narrowed down to a fairly minimal repro. This hangs or gives CLR fatal error within about 20 seconds, repeatedly running a medium sized (26mm parameter) neural network and interleaving with exercising .NET garbage allocation.
This is running latest version of Torchsharp (0.100.3) on Windows 11 with CUDA 12.2. The TorchScript file is too large to easily upload (56MB) and possibly this would happen with any TorchScript module.
    public static void ReplicateFailure()
    {
      var module = torch.jit.load<Tensor, (Tensor, Tensor, Tensor, Tensor)>(@"c:\temp\fail.ts", DeviceType.CUDA, 0).to(ScalarType.Float16);

      for (int bs = 1; bs < 133; bs += 3)
      {
        Tensor input = torch.tensor(new float[bs * 64 * 135], ScalarType.Float16, new Device("cuda:0"), false)
                            .reshape(new long[] { bs, 64, 135 });
        for (int j = 0; j < 100; j++)
        {
          using (var dx = torch.NewDisposeScope())
          {
            module.call(input);
          }
          ExerciseGC(j);
        }
        Console.WriteLine(bs + " done loop this batch size ");
      }
    }

    public static void ExerciseGC(int index)
    {
      Console.WriteLine("in " + index);
      object[] objs = new object[20_000];
      for (int i = 0; i < 20_000; i += 999)
      {
        objs[i] = new byte[i * 10];
      }
      Console.WriteLine("out");
    }

Does it work better if you dispose the input tensor at the bottom of the loop? You're allocating a lot of GPU memory, as far as I can tell, and not disposing of it.

In the code, the single call to the Torchscript module is immediately enclosed with a NewDisposeScope(), I believe this will release the GPU memory every time thru the loop, correct?

In the code, the single call to the Torchscript module is immediately enclosed with a NewDisposeScope(), I believe this will release the GPU memory every time thru the loop, correct?

It will release the memory produced by the module (i.e. its output) at the bottom of the inner loop, but not the input tensor that is allocated in the outer loop.

Yes, understood. I had missed the outer allocation.  Unfortunately the code you provide still hangs within 30 seconds. Is the next thing to try to replicate with a smaller Torchscript file that is accessible to both of us, and see if that can also replicate the problem? I have no problems at all with any of the non-Torchscript features of Torchsharp on my system.

Yes, it's hard to diagnose without more details. Unfortunately, I'm going to be out of the office until August 7th, but maybe we can continue this then?

Understood, thanks, I'll try to have data points to share upon your return.

Here's a more minimal repro of this problem that makes Torchscript not currently usable for me.
Repeated execution of unit test TestLoadJIT_3 will surface this problem, resulting in CLR errors, access violations and hangs.
For example, the following always fails within a minute or two of execution (on CPU, using explicit release of Tensor memory).
    int xx = 0;
    jit.ScriptModule<Tensor, Tensor> mx = torch.jit.load<Tensor, Tensor>("l1000_100_10.script.dat");
    while (true)
    {
      using var input = torch.ones(1000);
      using var output = mx.call(input);
      Console.WriteLine(xx++);
    }

Here's the tail output of one sample abend (after about 60 seconds):
242146
Fatal error. 242147
242148
Internal CLR error. (0x80131506)
242149
242150

And it works when you run it using PyTorch, presumably?

Interesting...
The problem is that when returning the result, the C++ code calls into managed code to allocate a 'TensorOrScalar' struct, which then has to be pinned in order to avoid being moved.
However, in IndexedPinnedArray.cs, line 44, the call to AddrOfPinnedObject() sometimes returns IntPtr.Zero for a valid, allocated handle to a pinned object. This will take some sleuthing to figure out.

Hi all,
@NiklasGustafsson thanks for this great package.
I was wondering if there is any update on this? Or if in the meantime somebody found out at least an hacky workaround.

I come back to this one every now and then when I'm bored... :-) Still not any closer to figuring out what's going on.

Thank you for the update! I was also going to ask, this is definitely an impediment for my work too.
I'm sorry it's proving so much a mystery. Here's one radical thought - maybe you could try upgrading to more recent Pytorch (2.1) and CUDA (12.x) versions. This could have two benefits, it might fix the problem if it was an old bug or at least behave differently and give more clues. Additionally, a package upgrade would be very helpful (especially since the current one does not support modern hardware such as H100,  reporting "no kernel found for device" when Torchsharp is run.

Upgrading to 2.1 isn't that radical, per se, but the move to CUDA 11.8 or 12 that it requires will be a lot of work (we're on 11.7 right now). It's something that needs to be done, but it will take some preparation and a few solid days of dedicated time.
It also requires help from someone who has the authority to take ownership on behalf of the .NET Foundation, because every CUDA version change requires a package name change, and that requires ownership transfer and some coordination.

Understood, good luck. CUDA 11.7 is only 14 months old but already feels ancient! For example, Pytorch 2.1 installation page seems to indicate that the minimum requirement is CUDA 11.8.

I got started on the move, but I'm running into the 'couldn't load nvfuser_codegen' warning on Windows again, which I thought had been addressed, so that will take some figuring out. Also, it'll be important to be on the latest and greatest NVIDIA drivers.

Great news. It's true we get this nvfuser error too, but it seems benign as far as I can tell. And looking online, it seems likely that this nvfuser is or has been deprecated in Pytorch.

It's still there in 2.1.0, and it's not benign -- it seems to disable the CUDA backend when it fails to load. Either that, or there's some other issue that disables it. I hope it's the same problem -- I'd rather solve one than two.

I can definitively report that in my environment (Windows) I get this warning/error but CUDA definitely still works, so you perhaps have a different cause.
[W C:\actions-runner_work\pytorch\pytorch\builder\windows\pytorch\torch\csrc\jit\codegen\cuda\interface.cpp:47] Warning: Loading nvfuser library failed with: error in LoadLibrary for nvfuser_codegen.dll. WinError 126: The specified module could not be found.
(function LoadingNvfuserLibrary)
Pytorch version  : <IP_ADDRESS>
Training device : cuda:0

Installing the CUDA 12.1 Toolkit seems to take care of both problems.

Status update on 2.1.0 work: I have it working on Windows, with CUDA, but I'm having a challenging time getting the Linux and MacOS builds to build. libtorch has moved to C++ 17, and enabling it creates problems because CLANG (unlike MSVC) doesn't simply ignore the command line option for the one file that is C rather than C++, so it rejects it.
The problem is in the src/Native/CMakeList.txt file, where I don't know how to add the compiler option for just C++, excluding C.

Good news!
Regarding your challenge, GPT4 produced a plausible solution, but I'm not qualified to judge if it's just a hallucination..
.................
Building cross-platform C++ projects can indeed be challenging due to differences in compiler behavior and available features. To address your issue with setting compiler options for C++ files only, you can use the $<COMPILE_LANGUAGE:CXX> generator expression in CMake. This expression evaluates to true for C++ sources.
You can set the C++ standard for your target using the target_compile_features function, and for setting specific compiler options, you can use target_compile_options. Here is an example of how you might adjust your CMakeLists.txt:
cmake
if (WIN32)
    [...]
else()
    add_definitions(-Werror) # treat warnings as errors
    add_definitions(-D_GLIBCXX_USE_CXX11_ABI=0)
    add_compile_options(-Wno-unused-local-typedef)
    add_compile_options(-fPIC)
    add_compile_options(-fvisibility=hidden)

    # Set C++ standard
    set_target_properties(your_target_name PROPERTIES
        CXX_STANDARD 17
        CXX_STANDARD_REQUIRED ON
    )

    # Add C++ specific compile options
    target_compile_options(your_target_name PRIVATE
        $<$<COMPILE_LANGUAGE:CXX>:-std=c++17>
    )
endif()

In this example:
Replace your_target_name with the actual name of your target.
CXX_STANDARD is used to set the required C++ standard.
CXX_STANDARD_REQUIRED ensures that CMake will fail configuration if the required standard is not supported.
target_compile_options is used to add compile options conditionally for C++ files.
These commands ensure that the C++ standard and compiler options are applied only to C++ files, not to C files. Adjust the options as needed for your project’s requirements.

So, v0.101.0, with libtorch 2.1.0 support is out. I have not checked whether it addresses the problem you reported in this bug.

I can confirm that this problem persists. I will continue to look into it. I understand that it's blocking.

It makes me wonder if the method we're using to pass data between TorchScript and .NET is too clever -- calling back to .NET to allocate arrays of TensorOrScalar, instead of allocating them in native code and then freeing up.

Okay, so by allocating and freeing memory in native code, instead of pinning managed objects, it both faster and can run your repro case (and a few others) 1,000,000 times.
It takes a very long time (15 minutes) to do so, so I'm going to have to use a lower number of iterations for the checked-in unit test.

Really great news on both the 2.1.0 upgrade and this fix, many thanks.

Since this is a really significant bug to have fixed, I will make another release just for this.

Addressed in v0.101.1, just release to NuGet Gallery.


expose mac->write_phy_reg() and read_phy_reg(), with mutex (IDFGH-10731)

Is your feature request related to a problem?
I have several PHYs on the SMI bus. Only one of them is under control of ESP-IDF and its network stack.
The other PHYs are under my control. I access them like so:
// ...
mac = esp_eth_mac_new_esp32(&esp32_emac_config, &mac_config);
// ...
err = mac->read_phy_reg(mac, phyad, regad, &data);
// ...

This is not a documented API.
This access may fail with ESP_ERR_INVALID_STATE ("phy is busy") when ESP-IDF's network stack already accesses the SMI bus in that instant.
This also happens in the other direction: if I have an ongoing SMI access, ESP-IDF's access attempt may fail. ESP-IDF seems mildly tolerant of such failure but I've seen it hallucinate an Ethernet connection coming back up after I pulled the network cable. This was with an IP101G (like the Ethernet Kit) and with me nearly filling the SMI bus with nonsense reads.
Describe the solution you'd like.
I would like

these methods to become public documented API
these methods to not fail but synchronize bus access using a mutex or similar suitable operation

Describe alternatives you've considered.
My own code explicitly handles ESP_ERR_INVALID_STATE by retrying, after some FreeRTOS delay, until the call succeeds.
Right now I am trusting that ESP-IDF will not fail or behave erratically under any intermittent failure to poll the PHY.
I have stress-tested ESP-IDF for how it handles congested access to the SMI bus. This is not the use case. My use case is occasional communication on the SMI bus, upto dozens of accesses per second.
Additional context.
No response

While we're at it, I'd like to have a recursive lock on the SMI bus, so I can explicitly lock+unlock it for multi-transaction operations like read-modify-write or MMD access (802.3 Annex 22D: Clause 22 Access to Clause 45 Registers)

https://www.linecall.de/wp-content/uploads/2016/11/802.3-2015_SECTION2.pdf
https://www.ieee802.org/3/efm/public/nov02/oam/pannell_oam_1_1102.pdf

@crackwitz thanks for the feature request. Could you please be more specific and describe the use case in more details? Currently, it sounds me quite specific and I don't think it would be used by majority of ESP32 users.

Our use cases may be considered specific in that all of this is beyond strictly establishing an Ethernet link for the ESP32.
Some use cases for direct SMI access to PHYs:

configure registers for interrupt output and LED control (polarity, modes like speed/link/act)
configure (enable/disable) Auto-Negotiation
read negotiated link speed
choose to include/exclude link speeds for Link Speed Advertisement
prevent PHY from trying to establish a link

Even if those things were modeled as interfaces to PHY drivers, I am not sure that I as the user could safely access those. As far as I can see, anything ultimately accessing the SMI bus is currently under exclusive control of the ESP-IDF network stack, so those accesses would never fight for access to the SMI bus. Our use cases require the application to access to those PHYs.
My current options are:

carry a patch for ESP-IDF that adds a (recursive) mutex in the SMI access functions, or
figure out a way to query FreeRTOS for when ESP-IDF will access the bus (I see accesses every two seconds), and avoid the bus around that time

Thanks for the reply. I understand the types of features to be configured in PHY via SMI. However, it's still hard to me to imagine a real device where multiple PHY's are used and managed by ESP32 but the only one is used for data interface to ESP32.

It's an industrial piece of electronics that manages the data path between different ports. One could call it a "media converter" because the ports have *MII in common but aren't all RJ45 Ethernet.

So where the data path is connected to from these "ports"? Sorry for asking such questions but I need to understand to the matter so we could prioritize the feature request. I assume you understand that priority goes to requests which have potential to be used by wider audience.

@crackwitz Just one note regarding

My current options are:
carry a patch for ESP-IDF that adds a (recursive) mutex in the SMI access functions, or

If I understand the issue correctly, it's not necessary to patch IDF. You can wrap the write_phy_reg, read_phy_reg functions like so:
/* declarations */
/* these two variables store the original read_phy_reg and write_phy_reg pointers */
esp_err_t (*read_phy_reg_inner)(esp_eth_handle_t eth_handle, uint32_t phy_addr, uint32_t phy_reg, uint32_t *reg_value);
esp_err_t (*write_phy_reg_inner)(esp_eth_handle_t eth_handle, uint32_t phy_addr, uint32_t phy_reg, uint32_t reg_value);
/* these two functions are the new read_phy_reg and write_phy_reg implementations */
static esp_err_t read_phy_reg_wrap(esp_eth_handle_t eth_handle, uint32_t phy_addr, uint32_t phy_reg, uint32_t *reg_value);
static esp_err_t write_phy_reg_wrap(esp_eth_handle_t eth_handle, uint32_t phy_addr, uint32_t phy_reg, uint32_t *reg_value);
/* PHY access mutex functions */
static void acquire_mutex(void);
static void release_mutex(void);
static void create_mutex(void);

/* part of your ethernet init code */
void main() {
  create_mutex();
  mac = esp_eth_mac_new_esp32(&esp32_emac_config, &mac_config);
  /* save the pointers to the original functions */
  read_phy_reg_inner = mac->read_phy_reg;
  write_phy_reg_inner = mac->write_phy_reg;
  /* replace them with the wrappers */
  mac->read_phy_reg = read_phy_reg_wrap;
  mac->write_phy_reg = write_phy_reg_wrap;
  /* ... the rest of the initialization... */
}

/* register access wrappers */
static esp_err_t read_phy_reg_wrap(esp_eth_handle_t eth_handle, uint32_t phy_addr, uint32_t phy_reg, uint32_t *reg_value)
{
  acquire_mutex();
  esp_err_t result = read_phy_reg_inner(eth_handle, phy_addr, phy_reg, reg_value);
  release_mutex();
  return result;
}
static esp_err_t write_phy_reg_wrap(esp_eth_handle_t eth_handle, uint32_t phy_addr, uint32_t phy_reg, uint32_t *reg_value)
{
  acquire_mutex();
  esp_err_t result = write_phy_reg_inner(eth_handle, phy_addr, phy_reg, reg_value);
  release_mutex();
  return result;
}

static void acquire_mutex() { ... }
static void release_mutex() { ... }
static void create_mutex() { ... }

This way, the implementation of PHY register access stays in IDF, but you can wrap it with your own functions which would acquire a mutex. Since both IDF and your code will call mac->read_phy_reg, mac->write_phy_reg, the calls will invoke the wrappers (read_phy_reg_wrap, write_phy_reg_wrap above).

mac->read_phy_reg = read_phy_reg_wrap;
mac->write_phy_reg = write_phy_reg_wrap;

Thank you! That looks like it will absolutely solve our issue! I never considered replacing the function pointers in that internal structure!
I'll evaluate this in the coming days and most likely close the issue then.

  mac->read_phy_reg = read_phy_reg_wrap;
  mac->write_phy_reg = write_phy_reg_wrap;

Works like a charm. Thanks!


Adds support for connection-less presentation requests

Summary of Changes

Updates AFJ packages adding better support for connection-less presentation requests
Refactors proof request screens to handle connection-less proof requests along with error handling and messaging
Refactors credential offer screen with better error handling and messages
Updates QR scanner to handle connection-less presentation requests
Updates language tags

Related Issues
#138
Pull Request Checklist
This is just a reminder about the most common mistakes. Please make sure that you tick all appropriate boxes. But please read our contribution guide at least once; it will save you a few review cycles!
If an item doesn't apply to your pull request, check it anyway to make it apparent that there's nothing to do.

[ ] All commits contain a DCO Signed-off-by line (we use the DCO GitHub app to enforce this).
[ ] Updated LICENSE-3RD-PARTY.md for any added dependencies or vendored components.
[ ] Run prettier: npm run style-format
[ ] Updated documentation for changed code and new or modified features.

If you have any questions to any of the points above, just submit and ask! This checklist is here to help you, not to deter you from contributing!
PR template adapted from the Python attrs project.

@amanji Ready to merge?

If there are no objections from the community 👍.

Silence is golden. Merge it!


hal::i2c_arbitration_handler wrapper

This class manages arbitration laws during the transaction function and provides a handler in the event that an arbitration laws exception is thrown. This can only have a multi-controller bus architectures which are quite rare. This class should only be used in a situation where you have a multi controller I do see bus architecture.

This is not useful. I2c drivers should wait for the bus to become available before progressing. Its not deterministic, but neither is having multiple controllers on the same i2c bus.


F-Droid

F-Droid is Android package manager and repository for FOSS apps.
@Swedneck seems to have ran a mirror in IPFS, but I am unable to access it currently.

https://forum.f-droid.org/t/ipfs-mirrors/4489

Yeah sorry about that, it's offline along with a bunch of stuff on my old server, sadly.

Added an F-Droid doc here: https://github.com/ipfs/package-managers/commit/5b3211439ede132faad66dcf213abeef28b3fb4a


feat: pass env variables for reflection service and DJ core

Summary
Becomes easy when these components are separately deployed
Test Plan

[ ] PR has an associated issue: #
[ ] make check passes
[ ] make test shows 100% unit test coverage

Deployment Plan

@sadath-12 thanks for this PR. Can you expand on the PR description a bit? Is the purpose here so that you can have one centrally stored set of the same environment variables that you set for each of the different service types? You still intend to run these as separate services right? Also, should we add a DJ_QUERY_ prefix to the query service settings object?
cc: @agorajek and @shangyian, existing deployments would need to add these prefixes to our env vars once this change is deployed.

@sadath-12 thanks for this PR. Can you expand on the PR description a bit? Is the purpose here so that you can have one centrally stored set of the same environment variables that you set for each of the different service types? You still intend to run these as separate services right? Also, should we add a DJ_QUERY_ prefix to the query service settings object?
cc: @agorajek and @shangyian, existing deployments would need to add these prefixes to our env vars once this change is deployed.

yes , we would deploy dj components separately which might not necessarily all be in same localhost .
Also I was intending to add env variables as preference which would not change the default.
For query service it seemed fine since it doesn't call other services

@sadath-12 I'm not sure why the prefix is needed? If these are all separate deployments, shouldn't each have their own .env file?

@shangyian I dint notice the feature of pydantic to automatically pick up env variables first . so I think this pr might not be necessary

+1 to what @shangyian said.


Release v0.7.0

v0.7.0 (2016-09-21)

PR #410/#405 - @zollman - Custom Watcher/Auditor Support. (Dynamic Loading)
PR #412 - @llange - Google SSO Fixes
PR #409 - @kyleberry - Fixed Report URLs in UI.
PR #413 - @markofu - Better handle IAM SSL certificates that we cannot parse.
PR #411 - @zollman - Many, many new watchers and auditors.

New Watchers:
* CloudTrail
* AWSConfig
* AWSConfigRecorder
* DirectConnect::Connection
* EC2::EbsSnapshot
* EC2::EbsVolume
* EC2::Image
* EC2::Instance
* ENI
* KMS::Grant
* KMS::Key
* Lambda
* RDS::ClusterSnapshot
* RDS::DBCluster
* RDS::DBInstace
* RDS::Snapshot
* RDS::SubnetGroup
* Route53
* Route53Domains
* TrustedAdvisor
* VPC::DHCP
* VPC::Endpoint
* VPC::FlowLog
* VPC::NatGateway
* VPC::NetworkACL
* VPC::Peering

Important Notes:

Because the watchers are dynamically loaded, you can now delete any you don't wish to use.
New permissions required:

cloudtrail:describetrails
config:describeconfigrules
config:describeconfigurationrecorders
directconnect:describeconnections
ec2:describeflowlogs
ec2:describeimages
ec2:describenatgateways
ec2:describenetworkacls
ec2:describenetworkinterfaces
ec2:describesnapshots
ec2:describevolumes
ec2:describevpcendpoints
ec2:describevpcpeeringconnections,
iam:getaccesskeylastused
iam:listattachedgrouppolicies
iam:listattacheduserpolicies
lambda:listfunctions
rds:describedbclusters
rds:describedbclustersnapshots
rds:describedbinstances
rds:describedbsnapshots
rds:describedbsubnetgroups
redshift:describeclusters
route53domains:listdomains

Contributors:

@zollman
@kyleberry
@llange
@markofu
@monkeysecurity

Holding off on merging.  Working on an problem where duplicate audit issues are being recorded.


Fix bogus deprecation warnings with GCC 10

@msimberg  Thanks for that! Is that a gcc10 bug or is it our fault?

@msimberg Thanks for that! Is that a gcc10 bug or is it our fault?

GCC bug, see https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99318.


fix(back-office): revert function app node version change for prod deploy

Pull Request Template
Describe your changes
A prod deployment is required to clear up the old Appeals BO resources. The change to the Function App Node version is not ready for production yet, so has been reverted. This can just be deployed from this branch.
Useful information to review or test

Type of change 🧩

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] Other (please explain in the description section above)

Checklist before requesting a review

[ ] My code follows the style guidelines of this project
[ ] My code changes do not include any hardcoded secrets or passwords
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] My Tech Lead is aware of any infrastructure changes that will cost money

Deployment complete, branch no longer required.


minor improvements to gosec parser

Adding line, confidence, formatting for code which can be already in backtick form.
Adding ref to being built documentation in references field.
Please submit your pull requests to the 'dev' branch.
When submitting a pull request, please make sure you have completed the following checklist:

[x] Your code is flake8 compliant (DefectDojo's code isn't currently flake8 compliant, but we're trying to correct that.)
[ ] If this is a new feature and not a bug fix, you've included the proper documentation in the ReadTheDocs documentation folder. https://github.com/DefectDojo/Documentation/tree/master/docs or provide feature documentation in the PR.
[ ] Model changes should include the necessary migrations in the dojo/dd_migrations folder.
[ ] Add applicable tests to the unit tests.

@madchap looks good for me too!! thanks for the PR.