Dotnet SDK auto-updates in Windows

This is potentially more of a question regarding intended best practices and definitely isn't a bug, so apologies if this is the wrong place to ask.
Our development teams are using .NET 6 on Windows 10. When a new SDK version is released, Windows auto-updates. I don't recall auto-updates being applied before, I believe it was introduced in .NET 5, and this causes issues for us because a new version is not simply added - the old version is replaced. This messes things up when the global.json in the project roots specify a rollForward property of disable, to ensure that we're developing using the same versions as our build agents / app servers.
I appreciate this is configurable - apparently via the Receive updates for other Microsoft products when you update Windows setting in Windows - however I probably want updates for those other products.
Is there a recommended way to control this for dotnet SDKs specifically?

@richlander PTAL

Good question. I'll ask.
Do you know that you can roll-forward on patch version. Is that acceptable to you?

@drk-mtr, we do provide the ability to not receive MU updates for .NET so you dont need to block all other updates. Take a look at this blog post about .NET updates coming to MU, and specifically the section on blocking these updates.

I read SDK vs Runtime not .NET vs no .NET.

Thanks for the responses all, really appreciated!
@jamshedd Thanks - I have applied that registry change and will keep an eye on what happens. I'm not clear on whether this relates to the runtimes, the SDKs or both.
@richlander Do you know that you can roll-forward on patch version. Is that acceptable to you? I think quality and release controls are good enough on dotnet that this would be fine in 99.999999% of cases, but because I'm paranoid I like to try and ensure that nothing can change in CI pipelines unless we allow it to change. So I'm probably being OTT for the vast majority of cases, but my personal preference is to disable rollForward.

@drk-mtr The registry entries applies to a specific update bundle, e.g. updates for .NET 6.0 or .NET 5.0. It doesn't allow for partially blocking specific components (child packages of the update bundle) like the SDK or one of the runtimes.
For CI scenarios, the zip copies of the SDK/runtime can be useful. For example, when we build the SDK itself, we acquire and create a local install of the runtime that won't get patched by MU, so we can retain full control, especially since we have tests that we want to run on older runtimes, e.g. we have tests for backwards compatibility for 3.1 and 5.0 when building the 6.0 SDK.

Thanks again all! Lots of good info here - I'll have a play and will most likely recommend latestPatch as best practice.

I've just started having a play around and I've realised that using latestPatch will result in our build pipelines breaking every time the feature is incremented, which is quite often.
The ideal for me would be that the build agents have multiple versions of the latest LTS available by default i.e. latestPatch, latestFeature and latestMinor. This would avoid additional (potentially time-consuming) steps to copy across zips, or tasks to install a different SDK during build.
Using latestFeature for now as a compromise but not entirely comfortable with it.

I've just started having a play around and I've realised that using latestPatch will result in our build pipelines breaking every time the feature is incremented, which is quite often.

Can you elaborate? I'm not following.

@marcpopMSFT  Patches of the SDK do not contain breaking changes and every change is reviewed and approved individually. For the most part, the changes that are approved are high impact or security in nature. You're right that it's possible for something to slip through that we missed but I'm not generally aware of that happening to the patch releases.

Unfortunately that's exactly what happened with 7.0.202. See https://github.com/dotnet/runtime/issues/83526. It contained a change that actually broke assembly loading for me, which means I'm stuck on 7.0.201 for the time being. Thankfully the registry setting in https://github.com/dotnet/core/issues/7457#issuecomment-1129554412 works beautifully and I no longer have Windows Update trying to uninstall 7.0.201. 😁

@marcpopMSFT

If you do want more control, Jacques suggestion of using the file-based installs will work but note that you'll be responsible for updating at that point and so could end up building with missing security fixes.

Which method should I be using for file-based installs? The docs don't mention it. (I can't see a comment by a Jacque in this thread either.)

It contained a change that actually broke assembly loading for me

Similar situation for us. Breaks happen, we'd like repeatable builds across developer machine and build agents by using the exact same version.

I appreciate this is configurable - apparently via the Receive updates for other Microsoft products when you update Windows setting in Windows - however I probably want updates for those other products.
Is there a recommended way to control this for dotnet SDKs specifically?

I have the same request as the initial request. Can we reopen this issue?

I think the current bugged .NET 6 SDK release 6.0.422 illustrates the need for control over the installed SDKs. Every time I reboot my development machine I need to uninstall 6.0.422 and reinstall 6.0.421 because of https://github.com/dotnet/roslyn/pull/73536 as reported by https://github.com/dotnet/roslyn/issues/73510 .


Don't try to unlock audio on iOS when set to disableAudio

When window['PhaserGlobal'].disableAudio or SoundManager.prototype.noAudio, we should avoid trying to unlock audio, as there may not be a SoundManager.prototype.context initialized.
Temp. workaround: set window['PhaserGlobal'].disableWebAudio.
(Low-impact issue since throwing from the touch handler doesn't stop any important code from running.)

This is now fixed in dev.


CPU issue_SQLTools Oracle Driver

Type: Bug
activation on startup
Extension version: 0.1.7
VS Code version: Code 1.80.2 (Universal) (2ccd690cbff1569e4a83d7c43d45101f817401dc, 2023-07-27T21:05:41.366Z)
OS version: Darwin x64 22.6.0
Modes:

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz (12 x 2600)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
15, 26, 52

Memory (System)
32.00GB (0.65GB free)

Process Argv
--crash-reporter-id 78f3c48c-bc48-4786-99d2-976663d5f723

Screen Reader
no

VM
0%

Hi, do you mean the plugin crashed when you open a sql file, is the file very large?

Hi, do you mean the plugin crashed when you open a sql file, is the file very large?

hello!
no, as a matter of fact, i don't open any sql files with the extension at all. it has been happening precisely on startups.. ;((

maybe you can see the error information in the output panel of sqltools

maybe you can find the performance issue as the following says or upload the profiles file so i can analyse
https://github.com/Microsoft/vscode/wiki/Performance-Issues


Time Conversion

Apply moment.js to Javascript
Convert and display current time to destination time
Link JS function to HTML element "destination current time"

mvp/time-conversion

Finished time conversion API


[W6.1h][T10-B2]Yap Zuo Xuan

added Printable interface for name,email,phone,address to implement. Also added(Printable... printables) method.

@wishingmaid Some comments added. Please close the PR after reading comments.
Ideally the getPrintableString(Printable … printable) method should be used somewhere.


Build failing for some instances of Linux x86_64, clang, libc++

Helo.
I'm packaging CppCommon into https://conan.io/center/
Conan is a package manager for C/C++ and the Conan Center is a central repository for recipes. They have a extensive pipeline that exercise the packaging of the library in many different platforms (72, in the case of CppCommon).
I'm getting some errors in libc++, clang, linux, x64 builds.
    static_assert((alignof(Impl) == StorageAlign), "ExceptionsHandler::StorageAlign must be adjusted!");
    ^             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 error generated.

All of them seem to be on exceptions_handler.cpp:640:5. Some other build logs can be found in this github comment.
My goal is to pack CppServer as a Conan recipe, but before it's better to make CppCommon available.
I'm also proposing a fix: #7 but I thought it would be clearer to highlight the issue here.

Fixed. Thanks!


PHP Support

I think we should drop support for all non current versions of PHP.
http://php.net/supported-versions.php
@tsdogs  @thyseus what do you think?
Dropping support for me means:

We remove the php version from travis tests.
We are allowed to use newer language features.

I think we should lean to the version constraints that the Yii Framework pretends.
Which is >PHP 5.4 actually and may be risen to > PHP 5.6 for Yii 2.1.0 iirc.

Yes stick to what Yii2 Framework does. (I had an error on UserSearch.php line 96 if I remember correctly) on php 5.6 on debian)
Lately I also had some troubles with phpunit on php 5.6 they seem to have dropped support for the php 5 series...

I also encountered the problem with phpunit and 5.6. I really wonder why yii calls code inside phpunit on every request, even when not in debug mode. I did not find the time to investigate this yet.

I solved it by using a more recent composer.json from yii2 app templates (using no-dev though)
But at the end updated to 7.0 as it's the one I'm using on dev machine.

Please merge this: https://github.com/dektrium/yii2-user/pull/933

Will do tomorrow


Fix broken to method

As documented in #283, calling .synthesize() after .to() raises a ValueError because met.metamer was no longer a leaf tensor. This is because, for some reason, Tensor.to is considered a differentiable operation. I'm not sure why that's desirable behavior, but this PR adds a solution: when moving tensors, we call tensor.data.to (instead of tensor.to) and then check whether the original version required a gradient and, if so, add it back. This is similar to how we behave in load.
You can find some discussions of this issue: on the pytorch forum, and on stackoverflow (one, two).
Additionally,:

the warning being raised was because I was trying to set self.model, instead of self._model, which was raising an AttributeError because model doesn't have a setter method. self.model.to was still being called (and modified in place), which was why met.model(met.metamer) worked without a problem. Replacing self.model with self._model solves this problem.
moves this self._model.to out of Synthesis and into the child classes (eigendistortion, metamer), since not all synthesis objects will have model attributes.
This is wrapped in try/except because we do not require that models have a to method (see model requirements). it is possible that a model will not have a to method but still behaves correctly, basing all their tensor dtype and devices on their input. so we simply raise a warning.
adds tests for this behavior (just call synthesize() after to() to our existing tests)

closes #283

Note that in Synthesis.to(), for tensors I'm essentially doing:
attr = getattr(self, k)
attr = attr.data.to(*args, **kwargs)
if getattr(self, k).requires_grad:
    attr = attr.requires_grad_()
setattr(self, attr, k)

which is weird because I'm basically resetting the tensor to its data. however, I can't seem to get around that. Replacing the .to line with either of the following  fails:
- attr = attr.data.to(*args, **kwargs)
+ attr = attr.to(*args, **kwargs)

- attr = attr.data.to(*args, **kwargs)
+ attr.data = attr.data.to(*args, **kwargs)

The first fails because attr.grad is None (because attr is no longer a leaf tensor) and the second fails because of a type issue with the (Adam) optimizer when synthesize() is called before and after the to() call (and they both fail in the same way if requires_grad is explicitly checked or not).
So I guess we're stuck with this solution.

While testing these, realized there were several other related changes to make:

I document that models don't have to be torch.nn.Module, but didn't test that. Now I do (and made a change to validate_model so it passes).
I document that metrics can be objects, but didn't test that. Now I do.
Realized that metrics must always return non-negative numbers, so added that to validate_metric.
In several places, there were functions that accepted a tensor as input and initialized another tensor always with dtype=torch.float32. These functions would thus fail if the input had a different dtype. They now initialize the new tensor with the same dtype as the input.


MultiSpeaker setup

Have you try this on multi-speaker way ?

@bshall Can we also replace Encoder and Decoder with Transformers ?

Hi @rishikksh20, sorry about the delay on this. I only noticed this issue now.
I have tried a multi-speaker setup (about 10 speakers) using one-hot codes for each speaker. It works pretty well but I think there is a small degradation compared to the single speaker model. In my experience fine-tuning the acoustic model on a small amount of target data seems to work better. I haven't experimented with using speaker embeddings for a zero-shot model though so can't comment on how well it performs in that setting.
I'd imagine that using Transformers would be fine. I don't think such heavy machinery is required though. I have done some experiments training the Hifi-GAN directly on the soft units (augmented with the pitch contours) and this seems to work well. It also simplifies the pipeline since it makes the acoustic model unnecessary.

@rishikksh20 hi, did you try soft-unit for multispeaker setup for any-to-many voice conversion? if so, did you success? i'm trying just using one-hot codes for multi speaker setup now, but suffering from speaker identity degradation. even though result speech speech is quite audible.

yes I feel the same with my training


Check to make sure only one AWX token exists

Recreates the original PR https://github.com/ansible/eda-server/pull/280, which for some reason it can not be reopened after the migration to a public repo.
Only grab the first token. So checking to make sure we have at least one token makes sense.
Jira: https://issues.redhat.com/browse/AAP-12376

@Alex-Izquierdo I think we should also fix this related issue
https://github.com/ansible/eda-server/blob/ed741364238c4e618946ab99a3b46b87c35d9d7f/src/aap_eda/wsapi/consumers.py#L113
Since it blindly assumes that there is a token available when the ansible-rulebook comes back on the websocket channel looking for parameters and the missing token causes an exception to be thrown. For now I think the assumption is you always need a token as we make more changes we can parse the rulebook and see if it has run_job_template action and then prompt for a token but even in that case we need this code path to be blocked. There is a stack trace for this code in many JIRA tickets, the server dying ends up closing the websocket connection and the ansible rulebook ends saying error exchanging web socket data. https://issues.redhat.com/browse/AAP-11677 https://issues.redhat.com/browse/AAP-11681
Since we have the activation instance here we should mark it as broken from the server side itself if the token is missing. The  server provides the token to ansible-rulebook.
https://github.com/ansible/eda-server/blob/ed741364238c4e618946ab99a3b46b87c35d9d7f/src/aap_eda/wsapi/consumers.py#L102
Since we have the activation id we should be able to set it to Failure from the server side itself instead of having the ansible-rulebook say that web socket connection closed.

@Alex-Izquierdo There needs to be another check here for the token
https://github.com/ansible/eda-server/blob/ed741364238c4e618946ab99a3b46b87c35d9d7f/src/aap_eda/tasks/ruleset.py#L44
I think we need to figure out the token based on the user_id that created the activation since if multiple users create the activations we need to find on whose behalf the worker is running the activation.
https://github.com/ansible/eda-server/blob/ed741364238c4e618946ab99a3b46b87c35d9d7f/src/aap_eda/wsapi/consumers.py#L316
Maybe this function https://github.com/ansible/eda-server/blob/ed741364238c4e618946ab99a3b46b87c35d9d7f/src/aap_eda/wsapi/consumers.py#L308 can come out and be placed in a shared location and be called from multiple places. We can pass in the activation id instead of the message. It could also throw an exception saying "No tokens defined"
I think there are at least 3 places we need to block it

From the UI side when the activation is created the first time (exists)

When the activation is started by a worker before we call K8S or podman to run the activation (missing)

When the ansible-rulebook comes on the websocket channel asking for the payload (missing)

I think the UI also has code to ensure that only 1 token is created I think this PR is guarding on the backend so that a REST API call doesn't end up adding an extra token.

Hi @mkanoor I totally agree with all that you say, we must fix the usage of the token. But it looks like that is out of the scope of this PR which its goal is to prevent to have more than one token. I propose to create a new issue or update some existing one (like this one ) and address all these issues in a different PR.

Except for the above minor comments, LGTM. Works as expected.


Avoid spikes in temperature data

I'm not sure why it started happening but I'm seeing extreme spikes in temperature data.

Potentially ignore volitile readings or check the CRC when reading (as suggested in the Arduino forum).

CRC is already checked when parsing the file. A failed CRC check should result in a -\/ which will end up as an error in the log.
class ParserTest extends Specification {
  "Fails to extract temperature with failed CRC check" >> {
    val output =
      """|72 01 4b 46 7f ff 0e 10 57 : crc=57 NO
         |72 01 4b 46 7f ff 0e 10 57 t=23125
      """.stripMargin
    Parser.parse(output) must be_-\/.like {
      case error: CrcFailure => ok
    }
  }
}

case class RecordTemperature(host: Host, input: TemperatureReader, output: TemperatureWriter, error: PrintStream = System.err) extends Runnable {
  def run(): Unit = {
    input.read.fold(error.println, temperatures => {
      output.write(Measurement(host, now(), temperatures)).leftMap(error.println)
    })
  }
}

Not quite as high as the above but the datasheet says 85 is a starting temp. Probably worth filtering out 85 degrees.

I was experimenting with the fix (since I still get spikes occasionally) and I thought of something.
If a change is classified as a spike when it's more than X% change, is that a change in degrees Celsius or Kelvin? If it's Celsius, what happens when the temperature is around 0C? I.e. if the temp goes from 0.1 to 0.2 C, it's obviously not a spike, but it would be read as a 100% increase, and then tagged as a spike, no?
And by the way, without the fix installed, I get very large spikes (up to hundreds of degrees); withe the fix, I still get spikes but of a lower value (but still over the set threshold, i.e. if the temp is 18C I get a spike to 40C...)

Thanks for the thoughts.
It's very difficult to gauge cause and effect here. I'm planning to tidy a few other things up just in case they are causing problems (like making sure no exceptions could be thrown causing weird behaviour), then I think I'm going to log every measurement and cross reference that to what's in the RRD database. That way I can see if the cause is with RRD. I'm also thinking of changing the way its logged to RRD as there's a slim chance more clients could increase the odds of spikes... not sure yet.
To answer your question though, everything is in centigrade and the comparison is between the last value and current. You can see it in the CSV and the % difference.

Closing for now as recent work seems to have fixed this... at least, I haven't seen it for a while. If I see it again or it's reported, I'll reopen.


Sysctl lense fails for keys that contain colons (":")

We have something like this in our sysctl.conf:
net.ipv4.conf.ib0:0.arp_filter=1
Augeas cannot parse this. There are related issues for keys containing slashes already (#102 and https://github.com/hercules-team/augeas/issues/668), but this here should be much easier to fix, as : is not part of augeas paths, right?
Thanks in advance.

oh, wrong project, sorry. this should be an "augeas" issue, opened this: https://github.com/hercules-team/augeas/issues/684


Two-factor authentication

Do we want to force the users in the organization to have 2FA enabled?
https://github.com/organizations/onebeyond/settings/security

Already enabled

Thanks @inigomarquinez!


Include Kirundi (Rundi - run_Latn) language in the No Language Left Behind (NLLB) project

🚀 Feature Request
Include Kirundi (Rundi) language in the No Language Left Behind (NLLB) project
Motivation

As researchers and developers from Burundi, we are deeply concerned about the absence of Kirundi (also known as Rundi) in the No Language Left Behind (NLLB) project. Kirundi is the national language of Burundi and is spoken by over 11 million people worldwide, including large diaspora communities. Its exclusion from NLLB significantly hampers our ability to develop inclusive, localized solutions for Kirundi speakers globally.

The lack of Kirundi in NLLB creates several critical issues:

Limited access to information: Kirundi speakers struggle to access vital information in their native language, especially in areas like health, education, and technology.

Hindered software development: Local developers face significant challenges in creating applications and services tailored to Kirundi speakers, limiting innovation and economic growth in our region.

Digital divide: The absence of Kirundi in major language models like NLLB widens the digital divide, leaving our community behind in the rapidly advancing world of AI and natural language processing.

Cultural preservation: Without proper representation in language models, there's a risk of Kirundi losing its digital presence, potentially impacting its long-term preservation and evolution.

Pitch
We propose the inclusion of Kirundi in the NLLB project. This addition would:

Enable accurate translation to and from Kirundi, facilitating better communication and information exchange for millions of speakers.

Empower local developers to create more sophisticated, language-specific applications and services.

Enhance natural language understanding capabilities for Kirundi, opening doors for advanced AI applications in areas such as voice recognition, text-to-speech, and sentiment analysis.

Contribute to the digital preservation of Kirundi, ensuring its relevance in the digital age.

Align with NLLB's mission of language inclusivity and bridging the gap for underrepresented languages.

Alternatives
While alternatives are limited, some developers have attempted to:

Use closely related languages like Kinyarwanda as a proxy, but this leads to inaccuracies and doesn't fully capture the nuances of Kirundi.

Develop smaller, less efficient language models specifically for Kirundi, but these lack the resources and scale of NLLB.

Rely on human translation, which is time-consuming, expensive, and not scalable for large-scale applications.

These alternatives are insufficient and emphasize the need for Kirundi's inclusion in a comprehensive project like NLLB.
Additional context

Kirundi is not just a language; it's a carrier of our culture, history, and identity. Its inclusion in NLLB would be a significant step towards digital equity and would open up numerous opportunities for innovation and development in Burundi and for Kirundi speakers worldwide.

We have a growing tech community eager to leverage advanced language models. The inclusion of Kirundi in NLLB would catalyze numerous projects and potentially transform our digital landscape.
Furthermore, Burundi's unique linguistic situation, with Kirundi as the primary language alongside French and English, presents an interesting case study for multilingual societies and could provide valuable data for improving NLLB's capabilities in similar contexts.
We are ready and willing to collaborate in any way possible to facilitate this inclusion, including providing language data, expert knowledge, and testing support.

Hi @labKnowledge!
First, Rundi is actually present in the NLLB-200 model. Please take a look at the NLLB paper, where run_Latn is included in the language list on page 15. Also you could try our translation interface co-branded with UNESCO and Huggingface, https://huggingface.co/spaces/UNESCO/nllb, where translation for Rundi is supported.
Second, currently, there are no specific plans of releasing a new version of NLLB.
However, there are specific steps you could undertake to improve the translation for Kirundi in the future releases of other multilingual translation models.

Translate the NLLB-Seed dataset into Kirundi. Its inclusion has been reported to greatly improve translation quality for Wikipedia-like domains. You can contribute it to the Open Language Data Initiative: https://oldi.org.
Collect other parallel texts for Kirundi that could serve as training data. You could include references to them to the Rundi language card at OLDI.
If you have parallel data, you could fine-tune an NLLB model (using e.g. this tutorial) to improve this Kirundi proficiency.


Update discipline-munit to 1.0.2

Updates org.typelevel:discipline-munit from 1.0.1 to 1.0.2.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.typelevel", artifactId = "discipline-munit" } ]

labels: test-library-update, semver-patch

Codecov Report

Merging #368 (f87caa0) into master (b2c7503) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #368   +/-   ##
=======================================
  Coverage   62.84%   62.84%
=======================================
  Files          38       38
  Lines        1281     1281
  Branches        8        8
=======================================
  Hits          805      805
  Misses        476      476

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 b2c7503...f87caa0. Read the comment docs.


enable beta test on float64 values

cf. #1123

@mattjj could it be better to remove jit in beta sampler? Kind of replacing
@partial(jit, static_argnums=(3, 4))
def _beta(key, a, b, shape, dtype):
  _check_shape("beta", shape)
  a = lax.convert_element_type(a, dtype)
  b = lax.convert_element_type(b, dtype)
  shape = shape or lax.broadcast_shapes(np.shape(a), np.shape(b))
  key_a, key_b = split(key)
  gamma_a = gamma(key_a, a, shape, dtype)
  gamma_b = gamma(key_b, b, shape, dtype)
  return gamma_a / (gamma_a + gamma_b)

by
def _beta(key, a, b, shape, dtype):
  _check_shape("beta", shape)
  a = lax.convert_element_type(a, dtype)
  b = lax.convert_element_type(b, dtype)
  shape = shape or lax.broadcast_shapes(np.shape(a), np.shape(b))
  key_a, key_b = split(key)
  a = np.broadcast_to(a, shape)
  b = np.broadcast_to(b, shape)
  gamma_a = gamma(key_a, a, shape, dtype)
  gamma_b = gamma(key_b, b, shape, dtype)
  return gamma_a / (gamma_a + gamma_b)

This way, we only compile gamma 1 time.
Another possibility is to concatenate a and b (after broadcasting)
def _beta(key, a, b, shape, dtype):
  _check_shape("beta", shape)
  a = lax.convert_element_type(a, dtype)
  b = lax.convert_element_type(b, dtype)
  shape = shape or lax.broadcast_shapes(np.shape(a), np.shape(b))
  a = np.broadcast_to(a, shape)
  b = np.broadcast_to(b, shape)
  ab = np.stack([a, b], axis=-1)
  gamma_ab = gamma(key, ab, shape + (2,), dtype)
  gamma_a, gamma_b = gamma_ab[..., 0], gamma_ab[..., 1]
  return gamma_a / (gamma_a + gamma_b)

Either way will make beta sampler faster in the tests.


fix(sample): fix minmax sampling behaviour

Brief Information
This pull request is in the type of:

[x] bug fixing
[ ] new feature
[ ] others

What does this PR do?
This fixes the minmax sampling behaviour to follow the standard as described here.
Fixed issues
No open issue, but the problem was discussed in the original PR for this feature: https://github.com/apache/echarts/pull/19279#issuecomment-2131424868
Details
Before: What was the problem?
I generated some fake data, 16000 data points for each graph (green random walk, yellow sine wave, and blue pseudo-random).
I added two artificial "spikes" for each graph: a downward spike at index 7000, and an upward spike at index 9000.

No downsampling
Original minmax downsampling implementation

The original minmax implementation appears to act just like the "max" downsampling method for data points that are above y=0, and thereby it misses out on any downward spikes.
After: How does it behave after the fixing?

No downsampling
New minmax downsampling implementation (2pts per px)

The new implementation more closely follows the actual trend, and ensures that it captures all up and down spikes.
Brief example of the minmax downsampling method, as described here:

Example data: [4, 1, 7, 8, 3, 6, 5, 9, 2, 7, 8, 1]
Frame size of 3: [(4, 1, 7), (8, 3, 6), (5, 9, 2), (7, 8, 1)]
Identify the min and max within each frame: [(4, 1, 7), (8, 3, 6), (5, 9, 2), (7, 8, 1)]
Keep the order of the min and max within each group, and those become the resulting downsampled set: [1, 7, 8, 3, 9, 2, 8, 1]

Discussion Point A: the link above suggests using two frames for each pixel, whereby each frame results in two data points, for a total density of 4 data points per pixel. Personally I think this is a bit overkill, as it results in less performance for minimal graph quality increase.
The current implementation in this PR has 2 data points per pixel, however this could be halved to 1 data point if you agree. The benefit of 1 data point per pixel is that the graph is more likely to allow graph animations, whereas the higher density may be above the threshold for animations still to play.
See comparison below:

No downsampling
New minmax downsampling implementation (2pts per px)

As suggested by link (4pts per px)
Doubled rate for 1pt per px

I suggest opening the images in fullscreen for a better comparison.
Discussion Point B: Regarding the code, I referenced the lttb method for implementing this minmax method in a similar way (i.e. having a separate function for downsampling), since the regular downsampling method would not work due to the minmax method requiring the ability to output two samples per frame. It may be needed to add an extra condition to the downsample method selection logic, to check if the rate is high enough (>2) to warrant downsampling, since this method isn't as efficient as the others. However, this will depend on the decision for Discussion Point A above.
Document Info
One of the following should be checked.

[x] This PR doesn't relate to document changes
[ ] The document should be updated later
[ ] The document changes have been made in apache/echarts-doc#xxx

Misc
ZRender Changes

[ ] This PR depends on ZRender changes (ecomfe/zrender#xxx).

Related test cases or examples to use the new APIs
N.A.
Others
Other information

@snukhulov Can you help review this PR? Thanks!

@snukhulov Can you help review this PR? Thanks!

The proposed minimax algorithm is more powerful than the previous one, it resolves discussed problems


Automated cherry pick of #4685: fix: next marker format

Cherry pick of #4685 on release/2.14.
#4685: fix: next marker format

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.14@705d2c9). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.14   #4691   +/-   ##
==============================================
  Coverage                ?   6.87%
==============================================
  Files                   ?     523
  Lines                   ?   88959
  Branches                ?       0
==============================================
  Hits                    ?    6116
  Misses                  ?   82336
  Partials                ?     507

Flag
Coverage Δ

#aFlag
6.87% <0%> (?)

Impacted Files
Coverage Δ

pkg/cloudcommon/db/db_dispatcher.go
0% <0%> (ø)

Continue to review full report at Codecov.

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

/lgtm

/approve


Bug: Users are able to access feed  without verifying

What happened?
Currently

users are able to access feed page make posts and all without verifying there emails,
after submitting register form users are redirected to any page feeling like a error
causing user data to not update in Database

Expected:

Users shouldn't be allowed to visit /feed unverified and redirecting them to waiting page asking them to verify there by clicking on the link sent to their respective emails
after registering users should be redirected to verify waiting page asking them the same

proposing Solution:

Can add a condition checking verified field, if not verified redirecting them to waiting page
same can be done with register form submit
and once user is verified, we can redirect him to /feed

Note: This solution will require resetting auth users and user DB. After implementing it...

Add screenshots
No response
What browsers are you seeing the problem on?
No response
Record

[X] I agree to follow this project's Code of Conduct
[X] I want to work on this issue

after submitting register form users are redirected to any page feeling like an error.

I am not able to replicate that.

causing user data to not update in Database.

Currently the database is designed so that, the accountId of user has to be present in users table to insert the data in posts. If not implemented correctly, you can work on that.

and once user is verified, we can redirect him to /feed.

[!IMPORTANT]
User has to redirect to the /verify page first because according to the current system, user details have been updated on verify page.

after submitting register form users are redirected to any page feeling like an error.

Currently the database is designed so that, the accountId of user has to be present in users table to insert the data in posts. If not implemented correctly, you can work on that.

and once user is verified, we can redirect him to /feed.

I'm not sure if we are on the same page here but   :
lets see -

once you submit register form - are you experiencing any redirects ?? ---it's no for me
you able to access /feed page without verifying on by clicking on the link sent to your email - its yes for me

Note: we are creating user document in verify function - and that is being called after user authentication
And its not related to posts Tho!
If I'm explaining it correctly - if user doesn't click on verify link sent to their emails users won't get added to database

@IrfanshaikH-7 assigning this to you. Please research on it and give a proper solution for this issue


Case conversion of names is not culture-independent

On machines where the current culture is set to Turkish, case conversion of identifiers result in the letter I being lowercased to the dotless-i (ı) variant instead of i.
I noticed that the code uses culture-specific ToLower(), ToUpper() instead of ToLowerInvariant(), ToUpperInvariant()
Generated docker-compose.yml causes issues with docker compose due to non-English letters not being allowed.
This does not appear to break the tool for now, and it can be remedied with a simple search and replace.

Generated docker-compose.yaml

Generated appsettings.json

Steps to reproduce
If you are on Windows, install the Language pack for Turkish and set just the Regional format to Turkish.
Or you can just add these lines to the Program.cs of craftsman:
CultureInfo.DefaultThreadCurrentCulture = CultureInfo.GetCultureInfo("tr-TR");
CultureInfo.DefaultThreadCurrentUICulture = CultureInfo.GetCultureInfo("tr-TR");

Have a ProjectName/ContextName/DatabaseName and/or Entity/Property name that begins with the letter I such as Ingredients.
Further technical details
Craftsman version (dotnet tool list -g): 0.26.4
Similar issues in other projects
https://github.com/CommunityToolkit/dotnet/issues/230

Appreciate the callout on this! Will try and take a look at it soon

👋 the newest v0.27.0 release should resolve this, but I wasn't able to easily test this myself, so I'm going to keep this open for now. Let me know if you're still having issues and I can dig into it some more!


Double-Sided materials intermittently write to depth buffer

Description:
The back faces of an object using a material that is set as double-sided will intermittently write to the depth buffer.  The material's "doubleSided" property is explicitly set to false in the material file, then set to true at run-time.
To Reproduce
The scene consists of multiple objects, all using the same material, but different instances of it.  Some of the instances have doubleSided set to true, some are set to false.
When viewing the scene from various viewpoints, some back faces that should be writing to the z-buffer will not, resulting in geometry showing through that should not.
Expected behavior
Back faces using a material with "doubleSided" set to true should always write to the z-buffer.  The first image, below, shows a roof exhibiting the problem.  The second image is mostly correct, with the back faces of the main roof rendering into the depth buffer.
Screenshots

Desktop (please complete the following information):

OS: Windows 10
GPU: AMD FirePro D500 (FireGL V)
Backend: OpenGL

Additional context
I verified the issue using RenderDoc to verify that the back-faces were not always rendering to the depth buffer.  The symptoms lead me to believe that the doubleSided attribute is not always being respected during the depth buffer pass, possibly because some of the objects in the scene use instances of that material with doubleSided enabled, some with it disabled.
Here is the material definition file:
material {
    name : Lit_Textured,
    requires : [
        uv0
    ],
    shadingModel : lit,
	doubleSided : false,
	colorWrite : true,
	depthWrite : true,
	depthCulling : true,
	culling : back,
    parameters : [
        {
			type : float4,
			name : baseColor
        },
        {
            type : sampler2d,
            name : albedo
        },
        {
            type : sampler2d,
            name : roughness
        },
        {
            type : sampler2d,
            name : metallic
        },
        {
            type : sampler2d,
            name : normal
        },
        {
            type : sampler2d,
            name : ao
        }
    ],
}

fragment {
    void material(inout MaterialInputs material) {
		float4 rawNormal = texture(materialParams_normal, getUV0()).xyzw * 2.0 - 1.0;
		float3 normal;
		normal.x = rawNormal.x * rawNormal.w;
		normal.y = rawNormal.y;
		float2 normXYSqr = normal.xy * normal.xy;
		normal.z = sqrt(1.0 - normXYSqr.x - normXYSqr.y);
        material.normal = normal;
        prepareMaterial(material);
		float4 texColor = texture(materialParams_albedo, getUV0());
		texColor.xyz = texColor.xyz * texColor.xyz;
        material.baseColor = materialParams.baseColor * texColor;
        material.roughness = texture(materialParams_roughness, getUV0()).r;
        material.metallic = texture(materialParams_metallic, getUV0()).r;
        material.ambientOcclusion = texture(materialParams_ao, getUV0()).r;
    }
}

This might be fixed by https://github.com/google/filament/pull/1936.  We were not correctly honoring the culling state associated with the material instance.

@BStringhamVRSK would you be willing to try the above fix?

Sure!  It might take me a little bit to work around our build process to inject my own libraries, but I will let you know what I discover.

That appears to have worked.  Glad to confirm the fix.

Thanks for checking!

Fixed by #1936


Delte accessability promise

Since Orbit is deprecated, I don't expect this to happen anymore, so I suggest deleting these lines.

But it's back, and accessible!

That's good news.


duplicate title tags being rendered - title not updating when navigating pages

Describe the bug
My titles are not updating when I navigate the site.
Reproduction
I have a component for meta tags:
<script>
	export let title = 'chovy\'s boondocking guide';
	export let path = '/';
</script>

<title>{title} - chovy's boondocking guide</title>

I use it like this on a page:
<script>
	import Meta from '$components/meta.svelte';
	const title = 'Mobile Internet Access for Boondocking';
	const path = '/guides/mofi';
</script>

<svelte:head>
	<Meta {title} {path} />
</svelte:head>

If I refresh the page the title is correct, but navigation via links without refresh does not update the title.
Logs
nothing relevant here

System Info
System:
    OS: Linux 5.11 Manjaro Linux
    CPU: (8) x64 Intel(R) Core(TM) i7-1065G7 CPU @ 1.30GHz
    Memory: 1.55 GB / 15.39 GB
    Container: Yes
    Shell: 5.8 - /usr/bin/zsh
  Binaries:
    Node: 16.3.0 - ~/.nvm/versions/node/v16.3.0/bin/node
    Yarn: 1.22.11 - /usr/bin/yarn
    npm: 7.15.1 - ~/.nvm/versions/node/v16.3.0/bin/npm
  Browsers:
    Chromium: 92.0.4515.107
    Firefox: 90.0.2
  npmPackages:
    @sveltejs/adapter-static: ^1.0.0-next.13 => 1.0.0-next.13
    @sveltejs/kit: next => 1.0.0-next.139
    svelte: ^3.34.0 => 3.41.0

### Severity

annoyance

### Additional Information

_No response_

bump

This is unrelated to adapter-static and prerendering. You need to put the component that renders the title in the layout
Duplicate of https://github.com/sveltejs/kit/issues/1540

my component handles the data.


BUG: Mesh and espnow interoperability issue (IDFGH-657)

Environment

Development Kit:      ESP32-DevKitC
Kit version (for WroverKit/PicoKit/DevKitC): [v1|v2|v3|v4]
Module or chip used:  [ESP32-WROOM-32|
IDF version (run git describe --tags to find it): v3.3-beta1-223-ga62cbfe
Build System:         [Make]
Compiler version (run xtensa-esp32-elf-gcc --version to find it): 1.22.0-73-ge28a011
Operating System:     [Linux]
Power Supply:         [USB]

Problem Description
I have an esp8266 sending an espnow message to an esp32 which is part of a mesh.  On receiving this data, the esp32 sends the data to root. If the node is root then the message sends fine.  If it is not root then the send fails.  Also, during the startup process, the node also sends a message to root and root sends a response message back.  Both these work fine so it appears that calling the mesh send in the espnow receive callback when the node is not root is is the issue.
I have raised this on the forum here
https://www.esp32.com/viewtopic.php?f=21&t=9279
Expected Behavior
Send should work correctly and send message to root
Actual Behavior
Depending on the tos value I give during the send I get different error message
MESH_TOS_P2P; I get ESP_ERR_MESH_TIMEOUT
MESH_TOS_DEF; I get ESP_ERR_MESH_ARGUMENT
MESH_TOS_E2E; I get ESP_ERR_NOT_SUPPORT
Steps to repropduce
Code to reproduce this issue
https://www.esp32.com/viewtopic.php?f=21&t=9279
Debug Logs
https://www.esp32.com/viewtopic.php?f=21&t=9279
Other items if possible

[ ] sdkconfig file (attach the sdkconfig file from your project folder)
[ ] elf file in the build folder (note this may contain all the code details and symbols of your project.)
[ ] coredump (This provides stacks of tasks.)

The receiving callback function also runs from WiFi task. So, do not do lengthy operations in the callback function. Instead, post necessary data to a queue and handle it from a lower priority task.

See IDF-DOC


[feat]: Add a UI to the Start9 registry link

Prerequisites

[X] I have searched for existing issues that already suggest this feature.

Describe the Feature Request
Links to the marketplaces i.e. https://community-beta-registry.start9.com display a not found error when viewed via a standard browser instead of selected as marketplace in the change registry section of StartOS.  Currently the marketplace URL returns:
<!DOCTYPE html>
<title>Not Found</title>
Not Found

/

Please add a UI so opening the URL in a standard browser returns useful information.
Describe the Use Case
This is helpful in cases where the registry URL is posted / sent and someone else opens the URL in a browser instead of in StartOS Add Custom Registry.
Describe Preferred Solution
There should be a configuration in the Marketplace setting that returns useful information.  It would be nice if marketplaces could automatically display what services are in them.  For example:
If I go to https://registry.start9.com it could automatically display something like https://marketplace.start9.com/marketplace displays today instead of the Not Found as described above.
And if I go to https://community-beta-registry.start9.com it displays the same UI, but shows the services available within that marketplace.
Describe Alternatives
Have UI display a configurable HTML message.  Here is a first draft default message with a proposed link to a picture to include:
The URL https://community-beta-registry.start9.com is designed to be opened within the StartOS marketplace configuration page.  To access this marketplace, go to the Marketplace tab and click on CHANGE and enter the URL there.
<PICTURE>
Here are some links that may be useful:

More information on Start9 Marketplaces is available at: https://docs.start9.com/latest/user-manual/dashboard-overview#marketplace-tab
Frequently Asked Questions: https://docs.start9.com/latest/support/faq/
More information on StartOS is available at: https://start9.com/

Anything else?
No response

I think this issue will need to be transferred to the registry or website repos.  Perhaps it could be lumped in with https://github.com/Start9Labs/start9.com/issues/17


20221207 performance

Performance pass for service bridge Port view

Codecov Report

Merging #32 (29e75df) into prep-release (fe5c463) will decrease coverage by 0.35%.
The diff coverage is 0.00%.

@@               Coverage Diff                @@
##           prep-release      #32      +/-   ##
================================================
- Coverage         27.80%   27.45%   -0.36%
================================================
  Files                41       41
  Lines              1478     1497      +19
================================================
  Hits                411      411
- Misses             1067     1086      +19

Impacted Files
Coverage Δ

src/django_devicectl/models/devicectl.py
54.54% <0.00%> (-1.95%)
:arrow_down:

src/django_devicectl/rest/views/service_bridge.py
0.00% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


MicMute not stating on boot

Hi,
Each time I boot I do not see the program running when booting even if I selected it to do so.

I have to manually click on the icon in order to start it.
Not a huge problem, but it's annoying xD
Not sure if it's a bug or if I have to do something else.
Hope you can tell me anything about it.
Thanks
BTW, great program ;)

Can you open task scheduler and check if there is a task named MicMute like in the picture below:

Hi,
Yes, I checked Task Scheduler and I think there was an error with a previous installation from a similar program.
I saw a task named MicMute but the Location was pointing to another folder and another .exe file. ( I didn't screenshot that :_(  )
Now I have deleted that and reinstalled yours and it appears and works perfectly.

First it appeared "MicSwitch" when I reinstalled yours, and after rebooting it appeared "MicMute" and it loads in the startup perfectly.
I think everything is fine, thanks :)

I'm glad it worked out :D


How to serve the app by nginx?

Hey! Everything works. Now I looking for some way to serve the app through Nginx. Which is the best way to start and daemonize babel-node process in production?

Hi! Cool. I'm just working on an improvement and once thats done I'll get back to you with the Nginx issue.

1/ I'm using pm2 manager to run node app with bin/start.js as starting point (maybe server-polyfl.js can be used but my file suits me better)
start.js:
require('babel-register')
module.exports = require('./server')
2/ created config for nginx/sites-available:
upstream app_myaudit_xyz {
server <IP_ADDRESS>:3000;
keepalive 80;
}
server {
#listen 80;
#listen [::]:80;
#server_name ~^(www.)?myaudit.xyz$;
server_name myaudit.xyz www.myaudit.xyz;
root /home/app/www/dist/public;
location / {
try_files $uri @proxy;
}
location @proxy {
proxy_pass http://app_myaudit_xyz;
proxy_http_version 1.1;
proxy_set_header Upgrade $http_upgrade;
proxy_set_header Connection 'upgrade';
proxy_set_header Host $host;
proxy_cache_bypass $http_upgrade;
}
}
I believe this info can lead you to a working solution

@pasechnik nice answer 👍


CI: change pyscipopt installation to use pip and pin version

Description
Please include a short summary of the change.
Pyscipopt 5.2.1 is the latest version (released october 31 :ghost: 2024).
Also update the mac-os version to 13 in CI due to encountering the following error:
The macOS-12 environment is deprecated, consider switching to macOS-13, macOS-14 (macos-latest) or macOS-15.

Issue link (if applicable):
Type of change

[ ] New feature (backwards compatible)
[ ] New feature (breaking API changes)
[ ] Bug fix
[x] Other (Documentation, CI, ...)

Contribution checklist

[x] Add our license to new files.
[x] Check that your code adheres to our coding style.
[x] Write unittests.
[x] Run the unittests and check that they’re passing.
[x] Run the benchmarks to make sure your change doesn’t introduce a regression.

Getting this really weird error and only for mac-os on python3.10
continuous_integration/test_script.sh: line 20:  2849 Abort trap: 6           python -c "import cvxpy; print(cvxpy.installed_solvers())"

But otherwise the scip issue seems to be fixed by changing the installation method (the conda version hasn't been updated in a while so its probably out-of-date, rather than a new version causing issues).

I tried rerunning the failing job but that didn't help. We can merge anyway.


Check if the class \Essence\Essence exists before requiring

Hi,
I think it'd be best to check for the existence of the class \Essence\Essence before calling require_once as one might have a different composer setup (I have a similar one to what’s described here).
// ...
if(!class_exists("\Essence\Essence")) {
	if(!file_exists(__DIR__ . '/vendor/autoload.php')) {
		throw new \Exception('Please install essence in your the module directory.');
	}
	require_once(/*NoCompile*/__DIR__ . '/vendor/autoload.php');
}
// ...

Thanks

Thank you for your help. I have implemented the class check.
The module is now also accessible via Composer


SystemJS semantics for commonjs libraries failing

What works
Using momentjs as an example. Note the following compiles and runs fine if module:commonjs
import * as _ from 'lodash';

Whats broken
With module:system,
the following compiles fine but fails at runtime:
import * as _ from 'lodash';

the following fails to compile but works at runtime:
import _ from 'lodash';

/cc reported by @Mitranim here

a.ts:
/// <reference path='lodash.d.ts'/>

import * as _ from 'lodash';
var r = _.filter([1, 2, 3], (x => x == 1));
console.log(r);

F:\test> tsc -m commonjs a.ts
F:\test> node a.js
[ 1 ]

F:\test> tsc -m system a.ts
F:\test> node a.js
F:\test\a.js:2
System.register(['lodash'], function(exports_1) {
^
ReferenceError: System is not defined
    at Object.<anonymous> (F:\test\a.js:2:1)
    at Module._compile (module.js:460:26)
    at Object.Module._extensions..js (module.js:478:10)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Function.Module.runMain (module.js:501:10)
    at startup (node.js:129:16)
    at node.js:814:3

We believe this should work. @vladima is looking more into it.

Seems to work ok using the latest es6 module loader / systemjs - just tried import * as _ from 'lodash' and it works like charm. Code can be found here.

Thanks for testing :rose:


Added code that predicts the price and  of three cryptocurrencies, and modified README.md

Hey there!,
I am Yashpreeth, a final year student proficient in Python , C++,Java, Solidity and I have a passion in blockchain and cybersecurity having an experience of 2 years
This is my first contribution, and I’m excited to submit a pull request. I’ve created a simple Telegram bot, Crypto Bot, that lets users get real-time cryptocurrency prices with just a click. It supports popular coins like Bitcoin, Ethereum, Litecoin, and more.
In this PR, I’ve added:
The bot’s code (crypto_bot.py).
A README.md with instructions on how to set it up and use it.
Kindly put the labels of WoB24 and gssoc and second level label to the code and first level label for readme file...
Would love your feedback on this—happy to be contributing!

Hii Yash preet thanks for your contribution will look into ur code and readme files and I will merge it  thanks for your contribution from the next time please make an issue and then raise an pr referencing the issue so that it would be helpful for debugging.
Happy contributing 🎊

Ok...
On Sat, Sep 28, 2024, 3:21 PM Harish @.***> wrote:

Hii Yash preet thanks for your contribution will look into ur code and
readme files and I will merge it .From the next time please make an issue
and then raise an pr referencing the issue so that it would be helpful for
debugging.
Happy contributing 🎊
—
Reply to this email directly, view it on GitHub
https://github.com/OpenTekHub/cryptobot/pull/9#issuecomment-2380586121,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUOXDWIMGXGFK4X2BJ4AO23ZYZ3ZXAVCNFSM6AAAAABPAPAPEKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBQGU4DMMJSGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

As well add the labels too...
On Sat, Sep 28, 2024, 3:22 PM Yashpreeth Nijalinge @.***>
wrote:

Ok...
On Sat, Sep 28, 2024, 3:21 PM Harish @.***> wrote:

Hii Yash preet thanks for your contribution will look into ur code and
readme files and I will merge it .From the next time please make an issue
and then raise an pr referencing the issue so that it would be helpful for
debugging.
Happy contributing 🎊
—
Reply to this email directly, view it on GitHub
https://github.com/OpenTekHub/cryptobot/pull/9#issuecomment-2380586121,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUOXDWIMGXGFK4X2BJ4AO23ZYZ3ZXAVCNFSM6AAAAABPAPAPEKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBQGU4DMMJSGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

worked perfectly and working great thank you for your contribution #10


[TECH] use @swc/jest to transpile module in Jest

Information
To prepare the workspace to be full esm compliant we need to use @swc/jest to run jest with native esm module and commonjs module.
Acceptance criteria

[x] All test pass on the CI

done


Use appveyor miniconda

See:

https://github.com/appveyor/ci/issues/359
https://www.appveyor.com/docs/build-environment/#python

Fixed in 287cde43dc583669cf19a075ade36689c4c68c29.


[Question] Documentation about database Access

Hello,
First, thanks for this nice project, seem a good way to deploy MariaDB on Kubernetes !
I don't find in the README, a way to get a remote access to the database,
I have created the demo mariadb object, the database, theuser, the grant and all seem work well, my mariadb pod is running, but when I try remote access, it doesn't work.  permission denied.  (for Root and User).
My question is, how to define the Host of a grant ?
I'm sorry if it's already documented somewhere.
Thank you

Hey there @valentin2105! Thanks for bringing this up.

My question is, how to define the Host of a grant ?

I've just updated the example, you can specify a spec.host in the Grant resource:

https://github.com/mariadb-operator/mariadb-operator/blob/main/examples/manifests/mariadb_v1alpha1_grant.yaml

By default, it will be %
This is a feature to be released in v0.0.20, so stay tuned! You can subscribe to GitHub releases or join our Slack channel

but when I try remote access, it doesn't work. permission denied. (for Root and User).

Curious about this, are you getting permission denied with the Root user? Could you try to check the logs os the MariaDB instance to see if there something relevant there?
Thanks!

It's fixed, thank you @mmontes11
It was by secret's base64 that wasn't created using echo -n.


Add support NextLink and other custom link components

Added support for using custom link components (e.g., NextLink) in the Button component via the new hrefComponent prop. If hrefComponent is not specified, it defaults to <a>.
Details:

Introduced a new prop hrefComponent in ButtonProps.
If hrefComponent is provided, it will render the specified component for links instead of  <a>.
Maintains backward compatibility by defaulting to <a> when hrefComponent is not set.
Ensures flexibility for projects using custom routing libraries like Next.js.

Related Tickets & Documents

Issues: #2007

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

@jadnast Hi, we've already attempted to support this in #1594 and we'll add it to the next branch (that will be 7 release) soon. You can be able to use only component prop, without need of extra props


move elements test added

@labibdotc please make it more consistent with the other test functions in this file


feat: add unit tests for case study card , check render of tools pages & newsletter

Description

This PR includes tests for CaseStudyCard in the components folder ,components/caseStudyCard
Also have added testing for Tools pages pages/tools
Added testing for Newsletter Pages pages\newsletter
Added testing for case study pages components\caseStudies

Related issue(s)
fixes #2063

/rtm


[turborepo]

What version of Turborepo are you using?
1.9.4-canary.7
What package manager are you using / does the bug impact?
Yarn v2/v3 (node_modules linker only)
What operating system are you using?
Mac
Describe the Bug
when trying to ignore the main package.json in my monorepo it does not seem to allow the ignoring
    "build": "turbo run build --ignore='package.json' --filter='...[main]'",

Below is my turbo.json
{
  "$schema": "https://turbo.build/schema.json",
  "globalDependencies": ["**/.env.*local"],
  "globalEnv": ["PORT"],
  "pipeline": {
    "build": {
      "outputs": [
        "dist/**",
        ".next/**",
        "!.next/cache/**",
        "build/**",
        "api/**",
        "public/build/**"
      ],
      "dependsOn": ["^build"]
    },
    "test": {
      "outputs": ["coverage/**"],
      "dependsOn": []
    },
    "lint": {
      "dependsOn": []
    },
    "dev": {
      "dependsOn": [],
      "cache": false,
      "persistent": true
    },
    "clean": {
      "cache": false
    }
  }
}

Expected Behavior
the expected behavior is to ignore any changes to the package.json
To Reproduce
clone the kitchen sink example - https://vercel.com/templates/remix/turborepo-kitchensink
add the build task and the main branch and run yarn build
Reproduction Repo
No response

A couple of notes here:

--ignore has lots of issues and we're aiming to remove it. If you want a particular outcome for ignoring include that inside of turbo.json.
You're not allowed to ignore package.json; we decide based upon our knowledge of the project whether or not to include that file into the hash. We elect to over-partition the hash to be safe. We know that this reduces the frequency of cache hits, but we still do piecewise inclusion of the file if you have a lockfile.

In this case we don't yet have an optimization in place that detects that the added script was not previously part of any turbo graph, so we simply invalidate the entire build. There's an opportunity for improvement here, but the behavior is still "correct" as is, just suboptimal.

Going to close for a couple historical reasons:

--ignore has been deprecated
We have a newer approach around the workspace root more generally that has some overlap with what's described here.
Granular control of which files impact global hashes can be handled using inputs and the new $TURBO_DEFAULTS$ microsyntax.


Release 2.1.2

Milestone: https://github.com/etalab/inspire/milestone/17
Change Log

#484 Make sure we don’t have dangling commas as we hate them (@tusbar)
#483 Get rid of unmaintained react-sticky (@tusbar)
#482 Disable ci build when deploying/pushing to gh-pages (@tusbar)
#468 Ajout de rôles oublié de la norme ISO 19115 (@tmerlier)
#469 Meilleur vérification des traductions (@tmerlier)
#481 Remove unused label style from semantic-ui (@tusbar)
#480 Update preact and react-loadable (@tusbar)
#479 Fix another broken dataset publication translation (@tusbar)

Codecov Report

Merging #485 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #485   +/-   ##
=======================================
  Coverage   21.44%   21.44%
=======================================
  Files         166      166
  Lines        1231     1231
=======================================
  Hits          264      264
  Misses        967      967

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 c3ca8ba...2239356. Read the comment docs.


Corrections to make on chrisproject.org About page

About Page:
The link to Boston University is coded as https://bu.edu/rhcollab/projects/radiology. However, the site’s SSL Cert is for www.bu.edu. The link should be changed to https://www.bu.edu/rhcollab/projects/radiology to avoid an SSL error.
In the overview paragraph, MOC is listed as “the Massachusetts Open Cloud” but should be “Massachusetts Open Cloud”
In the “How does ChRIS work? “ section, FreeSurfer is linked to a search engine result, but why not link directly to https://surfer.nmr.mgh.harvard.edu/fswiki ? There is no guarantee a search engine will always return the same result set. If people want to know more, they can use their preferred search engine. The other two technologies are directly linked to their main site, so why is FreeSurfer different?
In the “How is ChRIS being used today?” section, “Boston Children’s Hospital (BCH)” is mislinked to “FNNDSC”
The “Boston Children's Hospital Team” is nice, but could be improved with a short statement from each person about the project.
see https://github.com/FNNDSC/website/issues/20#issuecomment-739065375

@mairin hi, can I work on this issue.

Hey, @mairin I would like to work on this issue as an outreachy applicant.

here is  a PR for this


Event creation form: rebuild the form UI

Epic:
https://github.com/CoderDojo/community-platform/issues/1230
This ticket is about building a new UI based upon the mockups described in the Epic

The block for description and address are extensible and folded by default.
The structure of the datepicker and timepicker are to be defined (simple inputs vs library)
The event should setup a default session, a default set of 2 tickets and a type of "one-off".
The API endpoint (v3) will unlikely be defined at that point, hence keep it simple for the submitting aspect.

Choices to be made :

datepicker
timepicker
UI handling of visibility of expansible sections (address/description)
links to ?? on the right side


Update session metadata

we introduced a " in the title with the latest changes. can we fix that @AniketSK ?


feat: add boost compatibility

as title
need skip url stability check

Hello @bazelbuild/bcr-maintainers, modules without existing maintainers (boost.compatibility) have been updated in this PR. Please review the changes.


Support for WebView

It would be nice to use such a toolbar also for text inputs within web views. Is this possible in any way?

Perhaps, but I don’t see how this would benefit any significant number of users..

Since this affects any JS based text editor, I think this is something to consider. If you want to use something like ProseMirror (in my case) or Slate, Quill or whatever you have to use a webview for it.

WebView support would be much needed (image an email editor).
Is this possible right now somehow ?


[FEATURE] add website link in about section

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is.
Finding website link is hard for me
Describe the solution you'd like
A clear and concise description of what you want to happen.
I wanna see the website but for that i opened environments section then clicked on view development
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Add website link in about section or maybe in readme
Additional context
Add any other context or screenshots about the feature request here.

Hey!! This is 100% an issue that needs to be resolved imo. I made the same change when I contributed to Sukoon project.
I would like to solve this one too.

@Niravprajapati1 Add it to the README.md file


python3Package.SQLAlchemy: 1.4.16 -> 1.4.17

Motivation for this change
https://github.com/sqlalchemy/sqlalchemy/releases/tag/rel_1_4_17
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Added a release notes entry if the change is major or breaking
[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 125018 at 21db222b run on x86_64-linux 1

24 packages marked as broken and skipped:

cassandra_2_1
cassandra_2_2
gnuradio3_7
gnuradio3_7Packages.ais
gnuradio3_7Packages.gnuradio
gnuradio3_7Packages.gsm
gnuradio3_7Packages.limesdr
gnuradio3_7Packages.nacl
gnuradio3_7Packages.osmosdr
gnuradio3_7Packages.rds
hadoop
hadoop_2_7
hadoop_2_8
hadoop_2_9
impressive
linuxPackages_hardkernel_4_14.virtualbox
linuxPackages_hardkernel_latest.virtualbox
natron
ocropus
odpdown
opentsdb
pig
rainbowstream
spark

2 packages failed to build:

python38Packages.sqlsoup
python39Packages.sqlsoup

1370 packages skipped due to time constraints:

DisnixWebService
R
abcl
acd-cli
adapta-gtk-theme
adoptopenjdk-icedtea-web
aiodnsbrute
alloy (alloy5)
alloy4
amidst
...

20 packages built successfully:

python38Packages.aiocontextvars
python38Packages.alembic
python38Packages.factory_boy
python38Packages.flask_sqlalchemy
python38Packages.google-auth
python38Packages.google-auth-httplib2
python38Packages.google-cloud-testutils
python38Packages.httplib2
python38Packages.httpretty
python38Packages.oauth2client
python38Packages.pytest-randomly
python38Packages.sqlalchemy
python38Packages.sqlalchemy-i18n
python38Packages.sqlalchemy-utils
python38Packages.telethon-session-sqlalchemy
python39Packages.aiocontextvars
python39Packages.alembic
python39Packages.sqlalchemy
python39Packages.sqlalchemy-i18n
python39Packages.sqlalchemy-utils

2 suggestions:

warning: maintainers-missing
Package does not have a maintainer. Consider adding yourself?
Near pkgs/development/python-modules/sqlalchemy/default.nix:47:3:
   |
47 |   meta = with lib; {
   |   ^

warning: unused-argument
Unused argument: pytest_xdist.
Near pkgs/development/python-modules/sqlalchemy/default.nix:12:3:
   |
12 | , pytest_xdist
   |   ^

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.


Corrected GQ -> LQ data acquisition in slices - wmagent branch

2.1.6_wmagent branch version of https://github.com/dmwm/WMCore/pull/11457

This patch has been applied to all the 4 production agents (after removing the temporary workaround).
Given that it has configuration changes, I am going to make a new tag (<IP_ADDRESS>) with these changes in, and the RPM as well.


Minor improvements for use in a local environment

Hi.
Thank you for this simplest chat :smile:.
I like the simplicity of design and features and I hope you continue this way.

Add a default config.json
Fix the listening websocket for a local network
Use CDNJS for KaTeX
Add a package.json

Do you prefer I squash all commits ?
Can I publish on NPM ?
Regards

This looks great! I'll get this merged this afternoon. These commits add the necessarily polish for an npm package, so I think that would be fine.

I think this is ready for npm now. Feel free to post it.

Ok thank you


[libzippp] x64-windows build failure after liblzma use upstream CMakeLists.txt + Add pkgconfig. (#14615)

Issue
[libzippp] x64-windows build failure after commit https://github.com/microsoft/vcpkg/commit/eb895b95aac6fd7485373702f29f508c42a180a0 [liblzma] use upstream CMakeLists.txt + Add pkgconfig. (#14615) on windows, linux (openSUSE Leap 15.2 on WSL2 and Hyper-V is fine)
Host Environment

OS: Microsoft Windows [Version 10.0.19042.630]
Compiler:

VS2017 Visual Studio 2017 Version: 15.9.29  VS Tools Version: 14.16.27023
Microsoft (R) C/C++ Optimizing Compiler Version 19.16.27044 for x64
VS2019 Visual Studio 2019 Version: 16.8.2  VS Tools Version: 14.28.29333
Microsoft (R) C/C++ Optimizing Compiler Version 19.28.29334 for x64

To Reproduce
Steps to reproduce the behavior:

git pull
.\vcpkg.exe --debug install --x-use-aria2 --recurse --keep-going  libarchive[bzip2,core,libxml2,lz4,lzma,lzo,openssl,zstd]:x64-windows liblzma[core]:x64-windows libxml2[core]:x64-windows libzip[bzip2,commoncrypto,core,default-aes,liblzma,mbedtls,openssl,wincrypto]:x64-windows libzippp[core]:x64-windows

Failure logs

Cut and past the appropriate build messages from the console output.
Warning: passed either --printmetrics or --no-printmetrics, but metrics are disabled.
[DEBUG] Feature flag 'binarycaching' = off
[DEBUG] Feature flag 'manifests' = off
[DEBUG] Feature flag 'compilertracking' = off
[DEBUG] Feature flag 'registries' = off
[DEBUG] Feature flag 'versions' unset
[DEBUG] Using vcpkg-root: D:\vcpkg_x64\vs2019\test
[DEBUG] Using installed-root: D:\vcpkg_x64\vs2019\test\installed
[DEBUG] Using buildtrees-root: D:\vcpkg_x64\vs2019\test\buildtrees
[DEBUG] Using downloads-root: D:\vcpkg_x64\vs2019\test\downloads
[DEBUG] Using packages-root: D:\vcpkg_x64\vs2019\test\packages
[DEBUG] Using scripts-root: D:\vcpkg_x64\vs2019\test\scripts
[DEBUG] Default binary cache path is: E:\vcpkg_x64\vs2019\test\vcpkg\archives
Computing installation plan...
[DEBUG] Found path: C:\Program Files\CMake\bin\cmake.exe
[DEBUG] Found path: C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\Common7\IDE\CommonExtensions\Microsoft\CMake\CMake\bin\cmake.exe
[DEBUG] CreateProcessW("C:\Program Files\CMake\bin\cmake.exe" --version)
[DEBUG] cmd_execute_and_stream_data() returned 0 after    24415 us
[DEBUG] CreateProcessW("C:\Program Files\CMake\bin\cmake.exe" -DVCPKG_ROOT_DIR=D:/vcpkg_x64/vs2019/test -DPACKAGES_DIR=D:/vcpkg_x64/vs2019/test/packages -DBUILDTREES_DIR=D:/vcpkg_x64/vs2019/test/buildtrees -D_VCPKG_INSTALLED_DIR=D:/vcpkg_x64/vs2019/test/installed -DDOWNLOADS=D:/vcpkg_x64/vs2019/test/downloads -DVCPKG_MANIFEST_INSTALL=OFF -P "D:\vcpkg_x64\vs2019\test\buildtrees\0.vcpkg_dep_info.cmake")
[DEBUG] cmd_execute_and_stream_data() returned 0 after    12566 us
[DEBUG] CreateProcessW("C:\Program Files\CMake\bin\cmake.exe" -DVCPKG_ROOT_DIR=D:/vcpkg_x64/vs2019/test -DPACKAGES_DIR=D:/vcpkg_x64/vs2019/test/packages -DBUILDTREES_DIR=D:/vcpkg_x64/vs2019/test/buildtrees -D_VCPKG_INSTALLED_DIR=D:/vcpkg_x64/vs2019/test/installed -DDOWNLOADS=D:/vcpkg_x64/vs2019/test/downloads -DVCPKG_MANIFEST_INSTALL=OFF -P "D:\vcpkg_x64\vs2019\test\buildtrees\0.vcpkg_tags.cmake")
[DEBUG] cmd_execute_and_stream_data() returned 0 after    13326 us
The following packages will be built and installed:
libarchive[bzip2,core,libxml2,lz4,lzma,lzo,openssl,zstd]:x64-windows
liblzma[core]:x64-windows
libxml2[core]:x64-windows
libzip[bzip2,commoncrypto,core,default-aes,liblzma,mbedtls,openssl,wincrypto]:x64-windows
libzippp[core]:x64-windows
[DEBUG] CreateProcessW("C:\Program Files (x86)\Microsoft Visual Studio\Installer\vswhere.exe" -all -prerelease -legacy -products * -format xml)
...
Installing package libzip[bzip2,commoncrypto,core,default-aes,liblzma,mbedtls,openssl,wincrypto]:x64-windows... done
Elapsed time for package libzip:x64-windows: 18.73 s
Starting package 5/5: libzippp:x64-windows
Building package libzippp[core]:x64-windows...
[DEBUG] CreateProcessW("C:\Program Files\CMake\bin\cmake.exe" -DPORT=libzippp -DCURRENT_PORT_DIR=D:/vcpkg_x64/vs2019/test/ports/libzippp -DVCPKG_USE_HEAD_VERSION=0 -D_VCPKG_NO_DOWNLOADS=0 -D_VCPKG_DOWNLOAD_TOOL=ARIA2 -D_VCPKG_EDITABLE=0 -DFEATURES=core "-DALL_FEATURES=encryption;" -DCMD=BUILD -DTARGET_TRIPLET=x64-windows "-DTARGET_TRIPLET_FILE=D:\vcpkg_x64\vs2019\test\triplets\x64-windows.cmake" -DVCPKG_PLATFORM_TOOLSET=v142 -DDOWNLOADS=D:/vcpkg_x64/vs2019/test/downloads -DVCPKG_CONCURRENCY=8 "-DGIT=C:/Program Files/Git/cmd/git.exe" -DVCPKG_ROOT_DIR=D:/vcpkg_x64/vs2019/test -DPACKAGES_DIR=D:/vcpkg_x64/vs2019/test/packages -DBUILDTREES_DIR=D:/vcpkg_x64/vs2019/test/buildtrees -D_VCPKG_INSTALLED_DIR=D:/vcpkg_x64/vs2019/test/installed -DDOWNLOADS=D:/vcpkg_x64/vs2019/test/downloads -DVCPKG_MANIFEST_INSTALL=OFF -P "D:\vcpkg_x64\vs2019\test\scripts\ports.cmake")
-- Using cached D:/vcpkg_x64/vs2019/test/downloads/ctabin-libzippp-791bdc43eb18b87e3bdfa087493e3e32217e672c.tar.gz
-- Extracting source D:/vcpkg_x64/vs2019/test/downloads/ctabin-libzippp-791bdc43eb18b87e3bdfa087493e3e32217e672c.tar.gz
-- Using source at D:/vcpkg_x64/vs2019/test/buildtrees/libzippp/src/32217e672c-26e2865e9c.clean
-- Configuring x64-windows
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:106 (message):
Command failed: ninja -v
Working Directory: D:/vcpkg_x64/vs2019/test/buildtrees/libzippp/x64-windows-rel/vcpkg-parallel-configure
Error code: 1
See logs for more information:
D:\vcpkg_x64\vs2019\test\buildtrees\libzippp\config-x64-windows-out.log

Call Stack (most recent call first):
scripts/cmake/vcpkg_configure_cmake.cmake:314 (vcpkg_execute_required_process)
ports/libzippp/portfile.cmake:14 (vcpkg_configure_cmake)
scripts/ports.cmake:136 (include)
[DEBUG] cmd_execute_and_stream_data() returned 1 after  1786896 us
Error: Building package libzippp:x64-windows failed with: BUILD_FAILED
Elapsed time for package libzippp:x64-windows: 1.788 s
Total elapsed time: 1.792 min
RESULTS
liblzma:x64-windows: SUCCEEDED: 9.933 s
libxml2:x64-windows: SUCCEEDED: 6.03 s
libarchive:x64-windows: SUCCEEDED: 1.175 min
libzip:x64-windows: SUCCEEDED: 18.73 s
libzippp:x64-windows: BUILD_FAILED: 1.788 s
SUMMARY
SUCCEEDED: 4
BUILD_FAILED: 1
POST_BUILD_CHECKS_FAILED: 0
FILE_CONFLICTS: 0
CASCADED_DUE_TO_MISSING_DEPENDENCIES: 0
EXCLUDED: 0
The package liblzma provides CMake targets:
...

Please attach any additional failure logs mentioned in the console output.

VS2017
vs2017-config-x64-windows-out.log

VS2019
vs2019-config-x64-windows-out.log

Additional context
Add any other context about the problem here, such as what you have already tried to resolve the issue.

Failure in config-x64-windows-out.log(s)

-- Looking for BZ2_bzCompressInit
-- Looking for BZ2_bzCompressInit - found
CMake Error at D:/vcpkg_x64/vs2019/test/scripts/buildsystems/vcpkg.cmake:582 (_find_package):
By not providing "FindLibLZMA::LibLZMA.cmake" in CMAKE_MODULE_PATH this
project has asked CMake to find a package configuration file provided by
"LibLZMA::LibLZMA", but CMake did not find one.
Could not find a package configuration file provided by "LibLZMA::LibLZMA"
with any of the following names:
LibLZMA::LibLZMAConfig.cmake
liblzma::liblzma-config.cmake

Add the installation prefix of "LibLZMA::LibLZMA" to CMAKE_PREFIX_PATH or
set "LibLZMA::LibLZMA_DIR" to a directory containing one of the above
files.  If "LibLZMA::LibLZMA" provides a separate development package or
SDK, be sure it has been installed.
Call Stack (most recent call first):
cmake/FindLIBZIP.cmake:53 (find_package)
D:/vcpkg_x64/vs2019/test/scripts/buildsystems/vcpkg.cmake:582 (_find_package)
CMakeLists.txt:25 (find_package)
-- Configuring incomplete, errors occurred!

@Neumann-A @strega-nil
I am sure this is an easy fix, most likely the configuration package name changed, but it sure eludes me on where to make the change 😢

CI Pipeline appears to have just used a cached copy instead of recompiling...
For some reason it did not pickup liblzma is a dependency of libzippp and try to recompile libzippp?
2020-11-20T02:14:57.7480814Z                      libxslt:x64-windows:       pass: 587f8ce8cf93e54055d9c96ef250b73a3c73e3d3
2020-11-20T02:14:57.7485920Z                     libzippp:x64-windows:       pass: 6488d6120706fc2c500637b633e66f11be0e8244
2020-11-20T02:14:57.7490999Z                    licensepp:x64-windows:       pass: 55e039259ca9531b8ac1fcc013e7f9323a33a2ee

@StarGate-One also looking into this one, thank you @StarGate-One

CI Pipeline appears to have just used a cached copy instead of recompiling.

that is because lzma is not a default feature of libzip.

That is the problem:
E:/vcpkg_folders/liblzma/buildtrees/libzippp/src/32217e672c-26e2865e9c.clean/cmake/FindLIBZIP.cmake(53):  find_package(LibLZMA::LibLZMA REQUIRED )
basically 90% of the find_package calls in FindLIBZIP.cmake are nonesense. Wonder if it worked before.

@Neumann-A exciting. I'll make sure to fix it.

@Neumann-A @strega-nil Thank you very much for your hard work, I am trying to learn this so I can make changes/patches and submit the PR instead of just creating issues.

I think that is what is so confusing to me, port libzippp has [vcpkg-root]\buildtrees\libzippp\src\32217e672c-26e2865e9c.clean\FindLIBZIP.cmake and in turn looks like it has all the compression libraries in it for windows builds, but it does not look like the oncre created on Linux builds.
It is more fun to research, find the problem then especially rewarding to fix the problem instead of depending on someone else. It is like a puzzle to be solved.

@StarGate-One if you're interested, you can look into it; any questions you have should be asked on the #include<C++> discord, in the #vcpkg channel :)

--trace or --trace-expand are the magic options for cmake / vcpkg_configure_cmake to get enough log info to debug such errors and see what is going on.

@Neumann-A at the risk of sounding dumb, where would I put these options - I tried modifying/adding them to then end of the command line in vcpkg_build_cmake.cmake and vcpkg_configure_cmake.cmake with no results

--trace or --trace-expand are the magic options for cmake / vcpkg_configure_cmake to get enough log info to debug such errors and see what is going on.

@Neumann-A cc: @strega-nil

That is the problem:
E:/vcpkg_folders/liblzma/buildtrees/libzippp/src/32217e672c-26e2865e9c.clean/cmake/FindLIBZIP.cmake(53): find_package(LibLZMA::LibLZMA REQUIRED )
basically 90% of the find_package calls in FindLIBZIP.cmake are nonsense. Wonder if it worked before.

I think you are absolutely correct. I doubt it ever worked correctly.   All rights reserved.
Dump of file .\installed\x64-windows\bin\libzippp.dll
File Type: DLL
Image has the following dependencies:
zip.dll
MSVCP140.dll
VCRUNTIME140.dll
VCRUNTIME140_1.dll
api-ms-win-crt-runtime-l1-1-0.dll

->dumpbin /dependents .\installed\x64-windows\bin\zip.dll
Microsoft (R) COFF/PE Dumper Version 14.28.29334.0
Copyright (C) Microsoft Corporation.  All rights reserved.
Dump of file .\installed\x64-windows\bin\zip.dll
File Type: DLL
Image has the following dependencies:
ADVAPI32.dll
bz2.dll
bcrypt.dll
zlib1.dll

->dumpbin /dependents .\installed\x64-windows\bin\zlib1.dll
Microsoft (R) COFF/PE Dumper Version 14.28.29334.0
Copyright (C) Microsoft Corporation.  All rights reserved.
Dump of file .\installed\x64-windows\bin\zlib1.dll
File Type: DLL
Image has the following dependencies:
VCRUNTIME140.dll
api-ms-win-crt-stdio-l1-1-0.dll
api-ms-win-crt-heap-l1-1-0.dll
api-ms-win-crt-convert-l1-1-0.dll
api-ms-win-crt-runtime-l1-1-0.dll
KERNEL32.dll
Summary

at the risk of sounding dumb, where would I put these options - I tried modifying/adding them to then end of the command line in vcpkg_build_cmake.cmake and vcpkg_configure_cmake.cmake with no results

in the portfile into the vcpkg_configure_cmake call as OPTIONS parameter
somewhere in the portfile there should be a
vcpkg_configure_cmake(<something something>
                      OPTIONS --trace
                      <something something>
```´
alternativly you can use --trace-expand instead of trace

OK @Neumann-A I give it a try a little later. Thank you👍👍👍

It is going to take someone more experience than I to figure out how this can be fixed.

Thank you everyone.

Thank you everyone.


When using inheritance api versions are not correctly versioned

In ASP.NET Core, if you have a base controller with operations, and an inherited controller, and you use versioning, the version is not correctly discovered.

v11.17.2

This still seems to happen in 11.17.7 as far as I can tell.
I have a base controller like this:
[ApiVersion("1.0")]
[ApiController]
public class ApiV1BaseController : Controller
{
}

and an inherited controller:
[Route("api/v{version:apiVersion}/audio-devices")]
public class AudioDevicesController : ApiV1BaseController
{
}

I hope the commit fixes this... what do you think?

It looks sensible to me! I would be happy to try it out if there was a way for me to get a build of the package?

There is a CI nuget feed on MyGet but there will be a release soon.

Will release a new version in 15 min

v11.17.10

Thanks for turning this around so quickly. I am sorry to report it still does not seem to be working :( If I figure out how to get NSwag's code up and running, I may be able to try debugging it further to contribute a fix.

You can clone the project and start/test with one of the sample apps (installer can be unloaded)

And sorry, didnt test it... another option is to copy the processor class code into your project, remove the existing one from the settings and add your own with a fix... then post here the fixes :-)

https://github.com/RSuter/NSwag/blob/master/src/NSwag.SwaggerGeneration/Processors/ApiVersionProcessor.cs
Easy changeable in the middleware settings (GeneratorSettings property).

Outstanding, that sounds like the best option. I will give that a try.

Are you using ASP.NET or ASP.NET Core? Just for your background: The processor should be working correctly for both as it is used for both...

However, this solution isn't very scalable because it directly inspect's the Controller's base type. I could have inherited from a deeper hierarchy and it still may not have picked up the version attribute.

Maybe with a loop which loops through the BaseTypes until BaseType is null...

context.MethodInfo.DeclaringType and context.ControllerType evaluated to the same thing. So the two calls were redundant there.

Yep, when I looked at the code I also wondered if we can just remove this...


Exception name update

Docs Build status updates of commit 1d969d5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/webview2/releasenotes.md
:white_check_mark:Succeeded
View

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

Just a quick addition of an API rename we did, #signoff


https://quickswap.exchange/#/swap for Polygon doesn't let you switch chain if you aren't already connected to Polygon

If you go to https://quickswap.exchange/#/swap in AlphaWallet app and are connected to Polygon all is good. But if you aren't connected to Polygon, tapping "Connect Wallet" will show you an empty list of wallets. This is because the dapp checks if the dapp browser is running MetaMask like this:
if isMobile
    if isMetaMask
        Show connect or switch chain UI or list of wallets to choose from
    else
        Empty list of wallets to choose from (we are here!)
    end
else
    Show connect or switch chain UI or list of wallets to choose from
end

hboon: I can't think of a proper fix or hack around this [1]. So unfortunately we'll just have to leave it as that.
[1] We can pretend to be MetaMask and this dapp will work, but we can't do it only for this dapp and not the rest, so we can't use this workaround.

Can't fix on our end.


Add env-vars-only option

Small suggested change to just create the .env file using npx create-tigris-app. Potentially needs some refactoring, but pushing for discussion.
https://user-images.githubusercontent.com/328367/214553943-9041a9bf-7324-41e0-be32-d32c71a0412c.mp4
This then updates the console experience to be relevant to people with an existing application. They can run the following for an existing app.
npx create-tigris-app@latest --project awesome_nextjs_app --client-id <CLIENT_ID> --client-secret <CLIENT_SECRET>

We'd need to think through:

The CLI current creates a directory for the app name. We can just remove that functionality and assume it's for the cwd only.
It clobbers over an existing .env. So, we probably want to check if a file exists and append the credentials if they don't already exist. Throw an error if they do already exist.

I've been thinking on this and I think it would probably make more sense for this type of functionality to existing within the tigris CLI and not a create-x-app CLI.

I've been thinking on this and I think it would probably make more sense for this type of functionality to existing within the tigris CLI and not a create-x-app CLI.

This functionality is probably not suitable for create-tigris-app whose purpose is to scaffold new applications. If the user just needs the env file, we can add a view in the web console that shows the env file contents that they can copy and paste.


Re-run getimagesize without source_info if false

Certain JPG images fail to get the $image_info variable when
$this->source_info is set in the getimagesize function if the exif data
is not what PHP is expecting. This ends up throwing exceptions instead
of letting them through as though they do not have exif data.
To allow valid images to make it through, we need to check to see if we
can retreive $image_info via getimagesize WITH $this->source_info, if
not (getimagesize returns FALSE), then get the $image_info without the
$this->source_info. Then it will fall back to throwing the
ImageResizeException if it still does not exist.

Why add composer.lock to gitignore?

Because composer.lock should not be committed up in a library. That should be generated each time a new engineer runs composer install so they know they are getting the freshest version of the dependencies available.

@adityapatadia will you tag this, please?

done.


Sample sets: The page should not be static

If things are changed in the individuals page, like adding a new sample set, then this should be reflected in the sample sets page

Instead, I will make it so it is not possible to move individuals to a sample set which has not been defined in the sample sets table


Add WebRTC VAD and update the run.sh recipe.

Add WebRTC VAD and also update the run.sh recipe to be the same as Hirofumi's PR.

@pengchengguo LGTM! Thank you!


Share link broken when shared from guide

When you create a share link while on a guide, the share link won't work.
Fixes:

make it so all share links don't have the guide paramater in the URL
make it so even if it does have a guide parameter, it will still load the shared code

fix 1 is easy so i'm going to try and figure out fix 2 in the meantime

Thanks so much for pointing this out. I think the issue was it was assuming there were no url params when creating the share link. I strip them all out now with commit dc8d82a0 .


Support move_right=None

It is currently not well-defined what the value of the Sweep.move_right attribute should be, if the sweep is not actually moving, i.e. if the next update is at the same position.
This happens e.g. in TDVP, where after the last update of a sweep at i0=0, the same site i0=0 is updated again at the start of the next sweep.
Both TDVP engines set (i0, move_right) == (0, False) for the last update, which contradicts the definition from the docstring of Sweep:
move_right : bool
        Only set during sweep.
        Whether the next `i0` of the sweep will be right or left of the current one.

This causes problems in Sweep._cache_optimize:

...
if self.n_optimize == 2:
    kwargs = {
        'short_term_LP': [i0, i0 + 1],
        'short_term_RP': [i0, i0 + 1],
    }
    if move_right:
        kwargs['preload_RP'] = i0 + 2
    else:
        kwargs['preload_LP'] = i0 - 1
...

, where not move_right is interpreted as "we are moving to the left".
This PR addresses this by introducing support a third option for move_right, namely None, which means "not moving at all, next i0 is the same as the current".
Sweep._cache_optimize could then distinguish three cases.
I have added a "goal" version in the first commit, which breaks backwards compatibility and then introduced backwards compatible wrappers in the second commit which issue an appropriate warning.
We can then undo the changes from the second commit when moving to v1.0.

Left to do:

[ ] Deal with Sweep._update_env_inds, I am not sure what should be done here...
@jhauschild do you know what this should do in case of move_right is None?
[ ] Document the backwards-compatible solution properly
[ ] changelog


stddev strat updated

stddev strat updated
This one works great,
Removed index.php.
Removed unused package.json item.
stats-lite is required for generating stddev though

Is this actually finished? We startet to review though you seem to constantly push commits... :)


GEO2D and ensure_index Support

I was wondering about GEO2D indexes and ensure_index support. As far as I can see, this is not supported in the current txmongo, but I am not sure how hard it would be to support. When I try to create_index a GEO2D index, I get the following exception:
sort_fields must be an instance of filter.sort
Trying to use ensure_index failed as well - this would be a nice to have.
Let me know if I can help in any way. I have a good knowledge of MongoDB but I am not that well-versed in Twisted so I didn't want to just fork start hacking, but I'll be happy to look at it if you point me in the right direction.

Here's a standalone _create_index() that should do what you need.  It accepts sort lists (like pymongo) in addition to filter.sort objects and passes through additional kwargs, which should allow it to be used to create geo indexes, as well as support background=True and other modifiers.  Not yet extensively tested.
from   bson.son               import SON
from   pymongo                import helpers
from   txmongo                import filter

"""Usage: _create_index(db.coll, [('key1', ASCENDING)], background=True) """

def _create_index(collection, sort_fields, unique=False, dropDups=False, **kwargs):
    def wrapper(result, name):
        return name

    if isinstance(sort_fields, filter.sort):
        keys = list(sort_fields["orderby"])
    else:
        keys = helpers._index_list(sort_fields)

    name = u"_".join([u"%s_%s" % item for item in keys])
    index = dict(
        ns=str(collection),
        name=name,
        key=helpers._index_document(keys))
        unique=unique,
        dropDups=dropDups,
    )
    index.update(kwargs)

    d = collection._database.system.indexes.insert(SON(index), safe=True)
    d.addCallback(wrapper, name)
    return d

Thx kientzlecatch, works quite well after fixing imports and syntax :)
But it's still a dirty workaround, depending on pymongo. So, I'd like to see a proper ensure_index implemented. Especially since geospacial indexing is a damn cool feature of the mongo.

I wish I could do it but I'm quite busy with other stuff. So, I'll kindly wait for a pull request.

Hi..I want to know GEO2D index has been available or not?
Thanks

No. I haven't had the time to work on it.

@kientzlecatch Would be so kind as to make a pull request? :)

@rnz0 Do you still have that pull request?
@gleicon I can also review and test this if the PR is there.

thanks @psi29a I don't see the PR anymore. In any case if you want to work on it let me know how I can help.

This looks to be partially enabled, we have GEO2D support but the ensure_index is just a wrapper around create_index.
As of pymongo 3.0, ensure_index is deprecated. So as far as I'm concerned, this can be closed.
Any objections or comments?

Ok for me. I've proceeded with some tests at the time this was opened but didn't had the full grasp of GEO + mongo to submit something meaningful. Ping me if you need any help. Thanks.

+1


Rotate board - orientation left right

Currently the "orientation" property allows "black" or "white", I would like it to also support "left" and "right".
Since I am using this for a chess timer app for when two humans are playing eachother. The ipad is located alongside the chessboard and both players use it to mark their moves. It makes more sense to have your pieces close to you left or right side rather the normal up and down orientation.

I have worked around it by rotating FEN and Moves. I also had to switch color on the light/dark squares. The notation markers are wrong but I can turn them off.
This would be so much nicer if it could all have been done by the widget ;-)


Fix Django Debug Toolbar Visibility and INTERNAL_IPS in Docker Environments

Problem: The Django Debug Toolbar does not display in Docker (tested on MacOS) because socket.gethostname() returns Docker's default IP (<IP_ADDRESS>), not the host's IP.
Solution: Switch socket.gethostname() to host.docker.internal to accurately reference the host machine's IP from within Docker containers. This change ensures the the host's IP is added to INTERNAL_IPS and the Django debug toolbar is visible when developing in Docker.

This might be working on MacOS, but doesn't work on linux:
Python 3.11.7 (main, Feb  1 2024, 05:12:23) [GCC 12.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import socket
>>> socket.gethostbyname_ex("host.docker.internal")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
socket.gaierror: [Errno -2] Name or service not known


Include MNE-ICALabel and PyTorch in Windows installers

New attempt after the failed #168

Not sure if I'm doing something wrong with the package specs for Windows 🤔 the package name is:
win-64/pytorch-1.13.1-py3.10_cpu_0.tar.bz2

in the pytorch channel.

Installation works but testing failed:
Trying to import MNE and all additional packages included in the installer
Traceback (most recent call last):
  File "D:\a\mne-installers\mne-installers\tests\test_imports.py", line 13, in <module>
    import mne_icalabel
  File "C:\Users\runneradmin\mne-python\1.3.0_1\lib\site-packages\mne_icalabel\__init__.py", line 3, in <module>
    from . import annotation  # noqa: F401
  File "C:\Users\runneradmin\mne-python\1.3.0_1\lib\site-packages\mne_icalabel\annotation\__init__.py", line 1, in <module>
    from .bids import mark_component, write_components_tsv
  File "C:\Users\runneradmin\mne-python\1.3.0_1\lib\site-packages\mne_icalabel\annotation\bids.py", line 7, in <module>
    from ..config import ICLABEL_LABELS_TO_MNE
  File "C:\Users\runneradmin\mne-python\1.3.0_1\lib\site-packages\mne_icalabel\config.py", line 1, in <module>
    from .iclabel import iclabel_label_components
  File "C:\Users\runneradmin\mne-python\1.3.0_1\lib\site-packages\mne_icalabel\iclabel\__init__.py", line 8, in <module>
    from .label_components import iclabel_label_components  # noqa: F401
  File "C:\Users\runneradmin\mne-python\1.3.0_1\lib\site-packages\mne_icalabel\iclabel\label_components.py", line 9, in <module>
    from .network import run_iclabel
  File "C:\Users\runneradmin\mne-python\1.3.0_1\lib\site-packages\mne_icalabel\iclabel\network.py", line 7, in <module>
    import torch
  File "C:\Users\runneradmin\mne-python\1.3.0_1\lib\site-packages\torch\__init__.py", line 128, in <module>
    raise err
OSError: [WinError 182] The operating system cannot run %1. Error loading "C:\Users\runneradmin\mne-python\1.3.0_1\lib\site-packages\torch\lib\shm.dll" or one of its dependencies.

Bummer.....

If you really want to go the extra mile you could borrow the dependencies.exe stuff from OpenMEEG to figure out what the DLL can't resolve
https://github.com/openmeeg/openmeeg/blob/993cabd4541720e7885b91e5dfaff1ff85c501ed/.github/workflows/build_and_test.yml#L476-L481
It should walk the dependencies and say what's missing


style(protocol-designer): flip the expand/collapse arrow for TC substep accordions

overview
This PR closes #5911 by flipping TC substep arrow accordions.
review requests
Make a TC profile step
When you view it in the sidebar, the substep arrows should be pointing down when collapsed, and pointing up when expanded (like the top level arrows do)
risk assessment
Very low, style change behind FF

I was trying to figure out why this is only the case for TC substeps. Logic seemed to have been copied from TitledList in components.
What we do in PD inside CollapsibleSubstep is:
contentCollapsed ? 'chevron-down' : 'chevron-up'} />

in TitledList they do
props.collapsed ? 'chevron-down':  'chevron-up'

So logic is the same. I tried to trace back where props.collapsed is coming from, and it's getting fed into a connected component via a redux selector. Decided not to go down that rabbithole for now, but it seems like there might be some weird boolean flip logic happening somewhere.
For now I just flipped the chevron-up and chevron-down icons inside of our component (CollapsibleSubstep)


Search request price

Not sure what type the maxPrice and minPrice should be.
The client side on the AllGoods website converts float values to an integer, but the server side API accepts integers.

Will create a real PR after I get feedback.