add jenkins mkcloud result analyzer

:+1: really cool and helps a lot

:+1: agree

Great idea but new scripts should be Ruby or Python (or occasionally bash).  Use of Perl was already deprecated by team consensus, and the justification was documented.  So this would have got a -1 from me except my review was too late :-/


checksum.h - missing include

https://github.com/lammertb/libcrc/blob/master/include/checksum.h
needs the line:
#include <stddef.h>
...for the "size_t" parameters in the function calls.  I think it's included by default in a lot of cases, but I've got a stripped-down bootloader that threw an error without this inclusion.

I see this too.

+1

Fixed in commit https://github.com/lammertb/libcrc/commit/fb02cee2b9dace391a8151e3e42b6802e4c2fa41


Namespace error

Trying to get something, anything, to work with my new Oculus Quest 2.  I've read the first half of the book (quite enjoying it!), but I can't seem to get any of the examples to work, using either 2020 or 2019.  For example:
Assets_UWVRP3Assets\Chapter11\Scripts\RenderCubemapWizard.cs(5,36): error CS0246: The type or namespace name 'ScriptableWizard' could not be found (are you missing a using directive or an assembly reference?)
Can you point me in the right direction?  I know VR is constantly evolving, Unity and Oculus are pretty dynamic, but would love to get something working.  I don't think the problem is the new Quest, but rather the way newer versions of Unity handle namespaces.

Hello. Thank you for reporting this error. I have pushed a fix to the repository.
The problem was the file RenderCubemapWizard.cs is a Unity Editor script so it must be in a folder named "Editor". This file was a last minute addition and we missed that it gives an error when you build.
Alternatively you could just delete that script from your project as its an example taken mostly from the Unity Manual here (https://docs.unity3d.com/ScriptReference/Camera.RenderToCubemap.html)
Best to you and many virtual successes :)

Thanks.  Now I'm getting
ImportFBX Errors:
Couldn't read file
/home/tac/g/tacman/Unity-2020-Virtual-Reality-Projects-3rd-Edition-/UVRP3Projects/Assets/ANGRY
MESH/Snowman/Sources/Meshes/Snowman.FBX.
Unexpected file type
It goes away after I install the Snowman Asset.  But something's still not
quite right.  I'll report back after I've played around some more.
This repo is really large.  I'm in rural Virginia, on a very slow internet
connection, so cloning this repo took more than an hour.  Perhaps another
iteration could use the github large file storage?
On Fri, Nov 6, 2020 at 6:15 PM Jonathan Linowes<EMAIL_ADDRESS>wrote:

Hello. Thank you for reporting this error. I have pushed a fix to the
repository.
The problem was the file RenderCubemapWizard.cs is a Unity Editor script
so it must be in a folder named "Editor". This file was a last minute
addition and we missed that it gives an error when you build.
Alternatively you could just delete that script from your project as its
an example taken mostly from the Unity Manual here (
https://docs.unity3d.com/ScriptReference/Camera.RenderToCubemap.html)
Best to you and many virtual successes :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PacktPublishing/Unity-2020-Virtual-Reality-Projects-3rd-Edition-/issues/1#issuecomment-723341671,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEXIQPYEEIU45WMTQJ6PR3SOR7SZANCNFSM4SYV3V2Q
.

Ok. Maybe you didn't realize the repo has two main folders. The Projects one is large and is a Unity project with -completed- scenes for each book chapter. The Files folder is just the Assets you can use to -begin- each chapter. You don't need to clone the whole thing you can just download what you need.

On Nov 6, 2020, at 8:37 PM, Tac Tacelosky<EMAIL_ADDRESS>wrote:

Thanks. Now I'm getting
ImportFBX Errors:
Couldn't read file
/home/tac/g/tacman/Unity-2020-Virtual-Reality-Projects-3rd-Edition-/UVRP3Projects/Assets/ANGRY
MESH/Snowman/Sources/Meshes/Snowman.FBX.
Unexpected file type
It goes away after I install the Snowman Asset. But something's still not
quite right. I'll report back after I've played around some more.
This repo is really large. I'm in rural Virginia, on a very slow internet
connection, so cloning this repo took more than an hour. Perhaps another
iteration could use the github large file storage?
On Fri, Nov 6, 2020 at 6:15 PM Jonathan Linowes<EMAIL_ADDRESS>wrote:

Hello. Thank you for reporting this error. I have pushed a fix to the
repository.
The problem was the file RenderCubemapWizard.cs is a Unity Editor script
so it must be in a folder named "Editor". This file was a last minute
addition and we missed that it gives an error when you build.
Alternatively you could just delete that script from your project as its
an example taken mostly from the Unity Manual here (
https://docs.unity3d.com/ScriptReference/Camera.RenderToCubemap.html)
Best to you and many virtual successes :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PacktPublishing/Unity-2020-Virtual-Reality-Projects-3rd-Edition-/issues/1#issuecomment-723341671,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEXIQPYEEIU45WMTQJ6PR3SOR7SZANCNFSM4SYV3V2Q
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Make sure you have Git LFS installed and when you are cloning or pulling the repository it has successfully downloaded all the LFS (large files) types identified in the .gitattributes file. See https://github.com/git-lfs/git-lfs/wiki/Installation


Add gradle

Hello 👋
Please found this PR that:

Fix java code for constants (string and integer) that was self-referenced
Fix some misuse of parent method without super
Add compilation with Gradle and Java 1.8

At this time, JAR file can be launched but nothing appears
Let me know if you are interested

Woah.. you trying to make this playable?

Yes, at least first screen is loading and no more warning on startup by importing constants 🥳
The current state

But all mouse event is broken with Java 1.8, time to read documentation I guess 😄

Yes, at least first screen is loading and no more warning on startup by importing constants 🥳
The current state
But all mouse event is broken with Java 1.8, time to read documentation I guess 😄

I really hope you get this back in it's entirety I'd even front the hosting 😂

Despite mixed events systems, it is now possible to login (fake hero is used), visit stores and play few quests.
There is some errors:

Font are too big
Text renderer doesn't show it in multiline
Storage doesn't work

🎉🎉😀
This is so cool, thanks for the PR!
Merged.

https://github.com/reconSuave/RustyDagger/pull/2#issuecomment-1651190182
Is this editable file ? I tried opening up the JAR file in eclipse, but i'm sort of unable to view the source file for database.
I have intention to host this and make this multi-playerable + modifications to make it better. Anyone mind provide some guidance ?


Fix end position like iOS

Hi @ziqq
If I understood your request correctly, you would like support for a horizontal menu offset.
Update 0.9.2 is live with PullDownButton.menuOffset added.
For your provided example you'll need to do the following update:
PullDownButton(
  onTap: ...,
  buttonAnchor: PullDownMenuAnchor.end,
  menuOffset: -16, // an eyeballed value
  buttonBuilder: ...,
)

Maybe will adding this offset as default?

From my experience, it usually is a 16px offset, with only those picker menus using -16px (and even then I am not sure every native picker has -16px). That's why the default value is 16px


Deterministic Reconstruction Error

Upon reading the paper, specifically the part about the deterministic reconstruction with ddim,
I can not seem to understand why the reconstruction is not exact.
You talk about exact reconstruction and also mention reconstruction loss.
Why, if the formulas are deterministic and the output of the network
is also deterministic, is the reconstruction not exact?
Thanks in advance, Anthony Mendil.

I found the reason. If somebody is interested, feel free to reach out to me.

Hello friend, I have the same doubts as you. Could you tell me the reason why it cannot be exact reconstruction?
Thanks in advance.

Hello! To find out the reason why the reconstruction is not exact despite the deterministic diffusion process and neural network, I manually derived the deterministic reverse and diffusion processes. More precisely, I started at the formula for the deterministic reverse process and reversed it to obtain the deterministic diffusion process. When doing so, you find out that an approximation is required in order to make the resulting formula executable. Specifically, m(x_t, t) can not be computed and must be approximated by m(x_t-1, t-1), where m is the model that predicts x_0. This approximation is the reason why the reconstruction is not exact. As this calculation is not included in the paper, I provide my calculations below.


NLog does not function in a partially trusted environment

Bug
NLog version: 4.3.4 (NLog 2.1 works fine)
.Net 4.5, godaddy hosting platform
I am running a WCF service under a godaddy hosting service. When I attempt to upgrade the service to use NLog 4.3.4, my service will not start. I get the following exception:

What is the current result?

An unhandled exception was generated during the execution of the current web request. Information regarding the origin and location of the exception can be identified using the exception stack trace below.
Stack Trace:
[SecurityException: Request failed.]

I would expect that no exceptions "leak" of the NLog. My application/web service should still start up regardless of whether NLog can be configured, or not.
This is not a problem in NLog <IP_ADDRESS> which was the version prior to upgrading to the latest.

Is the exception even thrown when throwExceptions = false

Yes, that is true, it should be the default, and I tried setting
  LogManager.ThrowExceptions = false;

explicitly, and it still has the error.

Is above the full exception posted? I expect some details after . "I get the following exception:"

Hi Julian. That is all I get out of the webserver.
On a side note, if I add the marked code below to LogFactory. The problem goes away.
    private void LogConfigurationInitialized()
    {
      InternalLogger.Info("Configuration initialized.");
      if (AppDomain.CurrentDomain.IsFullyTrusted) // this is the new code
        InternalLogger.LogAssemblyVersion(typeof (ILogger).Assembly);
    }

I am just reading up on why this works. I would have thought the try{}catch{} in LogAssemblyVersion would have sufficed, but for some reason the exception bubbles up from LogConfigurationInitialized().

Any idea which call is raising the exception? And can't get SecurityException catched (there is a catch), see https://github.com/NLog/NLog/blob/04944f6637f5979627113bc6f1eed23de00784bc/src/NLog/Common/InternalLogger.cs#L356-L374

Here is some more stack information:
[Exception: System.Security.SecurityException: Request failed.
at NLog.Common.InternalLogger.LogAssemblyVersion(Assembly assembly)
at NLog.LogFactory.LogConfigurationInitialized()
The action that failed was:
LinkDemand
The type of the first permission that failed was:
System.Security.PermissionSet
The Zone of the assembly that failed was:
MyComputer]
NLog.LogFactory.LogConfigurationInitialized() +168
NLog.LogFactory.get_Configuration() +637
NLog.LogFactory.GetLogger(LoggerCacheKey cacheKey) +1149
NLog.LogFactory.GetLogger(String name) +127
NLog.LogManager.GetLogger(String name) +50
HL7Spy.Service.MyServiceHost..cctor() +66

don't get it. The catch should catch is. It's can be catched as seen in this example: https://msdn.microsoft.com/en-us/library/system.security.securityexception(v=vs.110).aspx

or is it the 'assemby.Fullname`?

It seems to be the way .NET walks the stack for permissions. See here: https://msdn.microsoft.com/en-us/library/c5tk9z76(v=vs.110).aspx
If you place the catch anywhere above LogAssemblyVersion(), it gets caught.
I placed it in the LogFactory.Configuration getter, since I don't think you want security exceptions to bubble up when its being initialized.
                        this.config.InitializeAll();
                        LogConfigurationInitialized();
                    } catch(SecurityException)
                    {
                      // do nothing
                    }

This seems to work when I test with godaddy.
There must be a way of replicating this in a test. I will see if I can figure that out.

or is it the assemby.Fullname in the catch?

No, I tried commenting out everything in the catch, and I still get the exception.

FYI, I have no problems with the fix, but I like to understand it fully so I can recognize the issue in other parts.

Me too. If you would like to do a shared meeting on skype, let me know I can demonstrate it in about 5 minutes.

Just as an FYI, this is not an issue on godaddy anymore since one can set:
<system.web>
    <trust level="Full" />
</system.web>

I had
<system.web>
    <trust level="Medium" />
</system.web>

from back when this was the only level you were allowed

ah, well partially trust is also legacy AFAIK

Me too. If you would like to do a shared meeting on skype, let me know I can demonstrate it in about 5 minutes.

Thanks, but i'm a bit too busy for that ATM

As medium trust is obsolete (http://stackoverflow.com/questions/16849801/is-trying-to-develop-for-medium-trust-a-lost-cause), we won't try to fix this.

Also reported by #1870, and fixed with  #1874


copy files over ocf_datapipes

Pull Request
Description

Move over location, geospatial files
tried to trim out as much as possible

Helps with #58
How Has This Been Tested?
CI test

[x] Yes

Checklist:

[x] My code follows OCF's coding style guidelines
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have checked my code and corrected any misspellings

very much outside the scope of this pr and also I might be misreading this, but I think we never actually use spatial padding? The option for it defaults to false and cannot be controlled from the config I don't think... gonna go make an issue about this

very much outside the scope of this pr and also I might be misreading this, but I think we never actually use spatial padding? The option for it defaults to false and cannot be controlled from the config I don't think... gonna go make an issue about this
UPD: #62

THanks @AUdaltsova and good to get this in another issue.
@dfulu let me know if your happy for me to merge


Fix broken sharing links for versioned views

Fixes #26

Thanks for ALL !!! 👍 💪

Thanks for ALL !!! +1 muscle

No problem. I need to update the CI to publish the new releases automatically, I'm going to notify you here once it's done (probably this weekend).


Can't make the revert command running correctly.

I have a provisioned WildFly installation with two histories:
➤ ./prospero history --dir=wfly-27                                                                                23:11:56
[b5d2ed78] 2023-02-21T15:07:47Z - update
[3a9c0837] 2023-02-12T17:22:06Z - install

I tried to do the revert but failed:
➤ ./prospero revert perform --dir=wfly-27 --revision=3a9c0837
Feature-packs resolved.

ERROR: The following plugin options are not recognized: jboss-reset-embedded-system-properties

I run the above command in a locally built and installed prospero.

@liweinan what was the manifest you used to install wildfly originally? It looks like the version of galleon-plugins artifact in the first installation is too old. We had to update to a 6.3.0 version recently which broke backwards compatibility

@liweinan what was the manifest you used to install wildfly originally? It looks like the version of galleon-plugins artifact in the first installation is too old. We had to update to a 6.3.0 version recently which broke backwards compatibility

The manifest I used is:
➤ ./prospero install --fpl=wildfly --dir=wfly-27-test --channel=examples/wildfly-27.0.0.Alpha2-channel.yaml

I fetched and rebased my prospero main branch into newest and rebuild the project, then I delete the WildFly dir and regenerate it, then I did a revert it works:
➤ ./prospero revert perform --dir=wfly-27-test --revision=452d0d17                                                                     23:51:08
Feature-packs resolved.
Packages installed. 100%
JBoss modules installed. 100%
Configurations generated.
Updates found:
  [*]io.undertow:undertow-core                             2.2.18.Final         ==>  2.2.17.Final

[*] The update list contain one or more artifacts with lower versions then currently installed. Proceed with caution.

Continue with update [y/N]: y

weli@192:~/w/prospero|main⚡*?
➤ ./prospero history --dir=wfly-27-test                                                                                23:51:53
[6c0318d8] 2023-02-21T15:51:51Z - rollback
[289c9776] 2023-02-21T15:50:09Z - update
[452d0d17] 2023-02-21T15:47:44Z - install

So the problem is fixed :D
Thanks!


Describe how to enable shell completions

Shell completions is always a good thing:) Let's share that tmt has it!

Thanks for improving the docs. The shell completion for bash should work out-of-the-box because /etc/bash_completion.d/tmt is included in the core tmt package. But documenting this could be still useful for those who are using other shells or install using pip. Shall we add completion scripts under /etc/ for other shells as well?

In my case, I was using a kind of pip install TMT, because I am usually installing python packages this way (details: pyenv + poetry + POETRY_VIRTUALENVS_CREATE=0). This allows me to get the most recent versions and do not depend on the OS provided python.
I think there is a lot of users of fish and zsh shells. Usually, I see that apps provide a completion command. So you might have something like:
tmt completions bash
tmt completions zsh
tmt completions fish

And then describe how to add those to your shell (i.e. https://kubernetes.io/docs/tasks/tools/install-kubectl/#enabling-shell-autocompletion)

Usually, I see that apps provide a completion command.

This sounds like a very nice idea. Let's track this under #611.


Documentation appears to be down

I attempted to access dendropy.org today, but was directed to the following website instead:

It appears the domain name is no longer registered.
Would you be open to publishing the docs on GitHub Pages instead? I'm happy to try to help in some capacity here.

The issue is resolved, documentation is showing fine right now.

Thank you!


Remove version: Jackett.Jackett version 0.21.580

404 - release deleted
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/Jackett/Jackett/releases/download/v0.21.580/Jackett.Installer.Windows.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 918)


有计划支持DoRA嘛，说是比LoRA效果好

Reminder

[X] I have read the README and searched the existing issues.

Reproduction
参考：

https://paperswithcode.com/paper/dora-weight-decomposed-low-rank-adaptation
https://github.com/catid/dora
https://arxiv.org/pdf/2402.09353v1.pdf

Expected behavior
No response
System Info
No response
Others
No response

https://github.com/huggingface/peft/pull/1474
dora合进peft了

已经支持


Handle zero-case in group scalar multiplication

Description

closes: #94
See doc-comments for more details.
This part of code could as well be moved into to_affine, but this exact behavior might not be expected outside of scalar_mul

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch (master)
[x] Linked to Github issue with discussion and accepted design OR have an explanation in the PR that describes this work.
[x] Wrote unit tests
[x] Updated relevant documentation in the code
[ ] Added a relevant changelog entry to the Pending section in CHANGELOG.md // (will add once approved)
[x] Re-reviewed Files changed in the Github PR explorer

Thanks for the PR! Could you fix the errors in CI and then I'll merge this.

Thanks for the PR! Could you fix the errors in CI and then I'll merge this.

I don't think these errors are coming from my PR, perhaps from the CI setup. I don't mind fixing it, but then it would bring some unrelated lines here.
error: call to `.borrow()` on a reference in this situation does nothing
   --> src/groups/curves/twisted_edwards/mod.rs:340:44
    |
340 |                 let base_power = base_power.borrow();
    |                                            ^^^^^^^^^ help: remove this redundant call
    |
    = note: the type `ark_ec::twisted_edwards::Projective<P>` does not implement `Borrow`, so calling `borrow` on `&ark_ec::twisted_edwards::Projective<P>` copies the reference, which does not do anything and can be removed
    ```

@Pratyush should be fixed now

Requires https://github.com/arkworks-rs/curves/pull/167

@slumber would you like to add a changelog entry for this?

@slumber would you like to add a changelog entry for this?

Yes, will do


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-28-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3860

Questions ?
Please refer to the Backport tool documentation


how to deploy a service in the docker swarm mode

when i deploy a container, there is no choose that the container is service or not?
and how to add label in the container
--label traefik.port=80 \
--network traefik-net \
--label traefik.frontend.rule=Host:mydomain.com

so how to deploy a service in the docker swam?
the docker version:
  Docker version 1.13.0
the docker node
 ID                           HOSTNAME      STATUS  AVAILABILITY  MANAGER STATUS
 00byqhttygdp55iayr52pi96j *  iZ23qa2w53dZ  Ready   Active        Leader
 3hobaje00uvd7sk1qu32n5hid    iZ23fiyhq7eZ  Ready   Active
 95i1hzti53154ijgo5b4i677q    iZ237b2agzgZ  Ready   Active

Support for swarm mode is still a work in progress.  It's being worked on in the swarm-mode branch (https://github.com/shipyard/shipyard/tree/swarm-mode).

Is this the right ticket to subscribe to, in order to be updated when that swarm-mode branch gets merged?
Also, what's the timeline on that?


fix: search input width resizing

Fixed an issue introduced by d78ba5f8f6e700143bdf3af1bfda0c05153daf45
The watcher was called only once since angular.noop will always return undefined.
Fixed this by calling the resize function on each $digest cycle.
Closes #1618
Closes #1617
Closes #1575

Thanks!

@bogdanalexe90 @aaronroberson Do you know when this fix will be released ?

also waiting for the release...

how can I check the version in which this bug was solved?


pip install accelerate in .exe version

When I'm using the .exe (version 20230203.494) I'm getting this message:
Cannot initialize model with low cpu memory usage because `accelerate` was not found in the environment. Defaulting to `low_cpu_mem_usage=False`. It is strongly recommended to install `accelerate` for faster and less memory-intense model loading. You can do so with:

pip install accelerate

Is it possible you can include the installation of that in the .exe?

It is about 3GB and we don't need it / use it. The exe would balloon up. It would be nice if HF didn't send such messages or there is a way to disable it.

I installed it but still get the error, so what exactly do I have to do for this?


Steps that call other steps display that information.

It's quite often case, when one step is calling another one. I'm
thinking of having a tree of hierarchy of calling steps.
Given /I create something/ do
  step %Q<I create something via rest api>
end

And as the result, there might be a kinda tree with links, what other
steps are called. Moreover, I can imagine to have a graphviz picture for
better visualization, but I realize, this is quite complex issue.

I am having an issue having where my substeps are not being picked up as a step at the moment, resulting in a large collection of undefined steps


Master Fader VU Meter

Two questions for you.

Ability to have a VU Meter on the Master Volume column?

Ability to duplicate Master and have one for each of the left and right channels?

On a side note: This is one really cool mixer. Well done.

Pull requests are welcome 😄


feat: add getDateStr prop to customize display outside moment format

I'm so sorry for that! Thanks for your great contribution.


docs(example): Replace autocomplete/instantsearch example with gif

I have submitted another PR on the readme generator (https://github.com/algolia/algoliasearch-client-readme-generator/pull/59) to give more emphasis on the autocomplete.js and instantsearch.js libraries. The goal is to make sure most people are aware of those libs and use them instead of building their UI from scratch.
With that goal in mind, I removed the autocomplete and instantsearch example and replaced them with animated gif. The readme will be updated (in the other PR) with a full explanation of what those libs are doing.

@pixelastic you removed examples that are useful for either testing (part of our release involves testing the build against the examples) and for learning the lib (as a developper I like having samples that uses the library). I don't think we should remove them.

Yes @pixelastic the examples are still useful when developing, having both seems better. GIFS for ease of reading and code for hacking

Arf, I wasn't aware of that, my bad. I'll add them back, maybe with a comment pointing to autocomplete.js / instantsearch.js.

I've re-added the HTML files, adding a small comment inside redirecting to autocomplete/instantsearch.

Here is the error from Travis. Not sure how this is relevant. I'm lost at debugging Travis

LGTM.
The current testing stack is flacky because of the underlying tunneling tool we are using to allow browsers from saucelab to access travis started web servers serving tests. It's a mess. I test locally for now

When you say that you test locally, do you run a saucelab local test, or manual testing on the browsers your have available on your machine?

@pixelastic I use npm run dev then launch virtualbox and access my <IP_ADDRESS> (host) to launch the test.
I have old browsers running with https://github.com/xdissent/ievms/
Also, I just switched back to "ngrok" (it was buggy also for some time) for our tunneling between saucelabs and travis and it's now much more stable in PRS.


Executive Summary

Executive Summary – Group 2
This document outlines our group’s plan for achieving success in Sprint 3 and the Night @ the Museum (N@tM) event.
Planning and Concept
Our group has decided to focus on engaging internet debates that highlight different perspectives on everyday topics. One example is the classic debate on whether cereal should be poured before or after the milk. By exploring relatable and divisive issues, we aim to spark discussions that challenge common assumptions and show how people approach routine activities differently.
These topics will serve as a foundation for an interactive and engaging experience, encouraging the audience to share their viewpoints.
Team Roles
Each team member will contribute to the project, taking responsibility for various tasks and aspects of our plan.

Name
Subject(s)
Role/Contribution

Pranav

Anvay

Kiruthic

Yuva

Rayhaan


Add TPCDS connector in Velox

Discussed in https://github.com/facebookincubator/velox/discussions/8994

Originally posted by majetideepak March  6, 2024
We should implement a TPCDS connector in Velox to generate data and run benchmarks against it.
The implementation stages would be similar to the TPCH connector.
TPCDS dsdgen can also generate delta files. This can be used to test the Lakehouse formats such as Iceberg.

@pdabre12


Network location on side bar doesn't exist.

Explain the bug
I can't access/see devices on the network anymore as the network side bar option is gone.
Logfile
debug.txt
debug_fulltrust.txt
System Information

OS Version: Windows 10 20H2 19042.870
App version: v<IP_ADDRESS>

It's not gone. Someone thought it was a good idea to hide it if no network locations are present -.-
To get it in 1.5 you can add a (fake) network location from file explorer.

I think it should still show the network index stuff like file explorer does

Exactly

Did a winget ( love it ) - ran FILES - saw the network icon ( i had a mapped z drive already) then tweaked a bunch of setting, restarted and they were gone - can no longer see all the networked devices, mapped drive are fine , so is onedrive and sharepoint drives - gonna uninstall and reinstall - just an FYI since i am running a Windows Dev version
OsVersion                                               : 10.0.21354
OsBuildNumber                                           : 21354
WindowsBuildLabEx:  21354.1.amd64fre.co_release.210402-1630

@FlowingSteve create a new issue for this as it not a relate or same issue

This issue should be resolved with #5082.


[Windows] Document how to do the "show window" migration manually

Background
Flutter 3.10 app startup has a race condition that can result in the window not being shown. https://github.com/flutter/flutter/pull/127046 updated the app template to fix this issue. Windows projects will need to be updated after the next Flutter release. An automatic migration was added for projects that have not modified their Windows entry point. However, the automatic migration may not be able to modify projects that have modified their Windows entry point.
Part of: https://github.com/flutter/flutter/issues/119415
See: https://github.com/flutter/flutter/pull/127046#discussion_r1198028570
Solution
After the next Flutter release, document how to do this migration manually.
Example migration:
diff --git a/examples/hello_world/windows/runner/flutter_window.cpp b/examples/hello_world/windows/runner/flutter_window.cpp
index f68aa9c26c0f5..7f66913a0293c 100644
--- a/examples/hello_world/windows/runner/flutter_window.cpp
+++ b/examples/hello_world/windows/runner/flutter_window.cpp
@@ -35,6 +35,11 @@ bool FlutterWindow::OnCreate() {
     this->Show();
   });

+  // Flutter can complete the first frame before the "show window" callback is
+  // registered. The following call ensures a frame is pending to ensure the
+  // window is shown. It is a no-op if the first frame hasn't completed yet.
+  flutter_controller_->ForceRedraw();
+
   return true;
 }

Slight rephrase suggestion:
+  // Flutter can complete the first frame before the "show window" callback is
+  // registered. The following call ensures a frame is pending to ensure the window is shown.
+  // It is an no-op if the first frame hasn't completed yet.

I'm not super familiar with this, but it sounds like a migration guide at https://docs.flutter.dev/release/breaking-changes would be useful. Even if the fix is simple, it brings attention to its necessity. Would that make sense for this? If so, would you be able to write one up?

@parlough Funny you should ask, I just opened https://github.com/flutter/website/pull/9269 🥳


something wrong

Version of Boardwalk:1.1
Version of Android:4.1.1
Device model:tm9s775
Description of the problem:a java error

Out of memory, read faq


[Bug]: Testcontainers ignores Docker proxies settings

Module
Core
Testcontainers version
1.17.5
Using the latest Testcontainers version?
Yes
Host OS
Linux
Host Arch
x86_64
Docker version
Client: Docker Engine - Community
 Version:           20.10.9
 API version:       1.41
 Go version:        go1.16.8
 Git commit:        c2ea9bc
 Built:             Mon Oct  4 16:08:29 2021
 OS/Arch:           linux/amd64
 Context:           default
 Experimental:      true

Server: Docker Engine - Community
 Engine:
  Version:          20.10.9
  API version:      1.41 (minimum version 1.12)
  Go version:       go1.16.8
  Git commit:       79ea9d3
  Built:            Mon Oct  4 16:06:37 2021
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.4.11
  GitCommit:        5b46e404f6b9f661a205e28d59c982d3634148f8
 runc:
  Version:          1.0.2
  GitCommit:        v1.0.2-0-g52b36a2
 docker-init:
  Version:          0.19.0
  GitCommit:        de40ad0

What happened?
It seems that Testcontainers doesn't load HTTPS_PROXY and HTTP_PROXY environment variables inside the container build process whereas Docker successfully include those variables when it builds images itself.
I have an authenticated enterprise proxy which I set up on my local machine using the ~/.docker/config.json file
{
 "proxies":
 {
   "default":
   {
     "httpProxy": "http://myproxyuser:myproxypassword@proxy.example.com:3128",
     "httpsProxy": "http://myproxyuser:myproxypassword@proxy.example.com:3128",
     "noProxy": "localhost,<IP_ADDRESS>,docker-registry.example.com,.corp"
   }
 }
}

as well as using a /etc/systemd/system/docker.service.d/http-proxy.conf file which looks like
[Service]
Environment="HTTP_PROXY=http://myproxyuser:myproxypassword@proxy.example.com:3128"
Environment="HTTPS_PROXY=http://myproxyuser:myproxypassword@proxy.example.com:3128"
Environment="NO_PROXY=localhost,<IP_ADDRESS>,docker-registry.example.com,.corp"

I developed the following project to illustrate my problem : when I run ./build-using-docker.sh https://proxy.example.com/proxy.crt it launches a ./mvnw --settings /home/myuser/.m2/settings.proxy.xml clean package command inside a Docker container run like this docker run -it --rm -u 1000:998 -v /home/myuser/myprojects/testcontainers-proxy-issue:/home/myuser/myprojects/testcontainers-proxy-issue -w /home/myuser/myprojects/testcontainers-proxy-issue -v /home/myuser/.m2:/home/myuser/.m2 -v /var/run/docker.sock:/var/run/docker.sock -e 'MAVEN_OPTS=-Dhttps.proxyHost=proxy.example.com -Dhttps.proxyPort=3128 -Dhttps.proxyUser=myproxyuser -Dhttps.proxyPassword=myproxypassword -Dhttp.proxyHost=proxy.example.com -Dhttp.proxyPort=3128 -Dhttp.proxyUser=myproxyuser -Dhttp.proxyPassword=myproxypassword -Djavax.net.ssl.trustStore=./proxy-data/cacerts -Duser.home=/home/myuser' -v /home/myuser/.docker/config.json:/home/myuser/.docker/config.json -e DOCKER_CONFIG=/home/myuser/.docker eclipse-temurin:17-jdk-alpine sh -cex 'cp /opt/java/openjdk/lib/security/cacerts ./proxy-data/ && keytool -importcert -alias proxy.crt -file ./proxy-data/proxy.crt -trustcacerts -keystore ./proxy-data/cacerts -storepass changeit -noprompt; ./mvnw --settings /home/myuser/.m2/settings.proxy.xml clean package'.
Then Maven launches the tests of the projects which executes findAllTest() test which launches @ExtendWith(OpenldapTestContainerIntegrationTestExtension.class) before, which executes the following code:
        final var imageFromDockerfile = new ImageFromDockerfile()
                .withFileFromPath(".", Path.of("src", "main", "docker"));

The targetted Dockerfile only contains this:
FROM bitnami/openldap:2.6

RUN test -n "$HTTPS_PROXY" || exit 42

Here is the JUnit failing log:
[ERROR] Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 1.959 s <<< FAILURE! - in com.github.anthony_o.testcontainers_proxy_issue.repository.UserRepositoryIntegrationTest
[ERROR] com.github.anthony_o.testcontainers_proxy_issue.repository.UserRepositoryIntegrationTest  Time elapsed: 1.957 s  <<< ERROR!
org.testcontainers.containers.ContainerLaunchException: Container startup failed
Caused by: org.testcontainers.containers.ContainerFetchException: Can't get Docker image: RemoteDockerImage(imageName=<resolving>, imagePullPolicy=DefaultPullPolicy(), imageNameSubstitutor=org.testcontainers.utility.ImageNameSubstitutor$LogWrappedImageNameSubstitutor@4a0df195)
Caused by: com.github.dockerjava.api.exception.DockerClientException: Could not build image: The command '/bin/bash -o pipefail -c test -n "$HTTPS_PROXY" || exit 42' returned a non-zero code: 42

Whereas if I simply run the following Docker command, it succeed:
docker build src/main/docker/ -t testcontainers-proxy-issue --no-cache
Sending build context to Docker daemon  2.048kB
Step 1/2 : FROM bitnami/openldap:2.6
 ---> eb7e0974d701
Step 2/2 : RUN test -n "$HTTPS_PROXY" || exit 42
 ---> Running in e298e31d18f4
Removing intermediate container e298e31d18f4
 ---> 5a87cf389f41
Successfully built 5a87cf389f41
Successfully tagged testcontainers-proxy-issue:latest

Relevant log output
No response
Additional Information
No response

I found the following workaround in the meanwhile: add the proxy variables in the build args in OpenldapTestContainerIntegrationTestExtension.java
for (var varName : new String[]{"HTTPS_PROXY", "https_proxy", "HTTP_PROXY", "http_proxy"}) {
    final String envVariableValue = System.getenv(varName);
    if (StringUtils.isNotBlank(envVariableValue)) {
        // Add argument builder
        imageFromDockerfile.withBuildArg(varName, envVariableValue);
    }
}

According to https://docs.docker.com/engine/reference/builder/#predefined-args those variables are not visible in the build history, which is what I'm looking for.
I think that those variables (as well as other proxy variables defined in https://docs.docker.com/network/proxy/#configure-the-docker-client) should be included in default build arg in ImageFromDockerfile.java if detected in the current environment variables (via System.getenv?) as well as added if proxies are setup in the ~/.docker/config.json in order to fully imitate docker build CLI.


Added goldpinger helm chart

Signed-off-by: Mikhail Naletov<EMAIL_ADDRESS>
What this PR does / why we need it:

add Goldpinger (a tool for network connectivity monitoring between nodes) chart

Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Variables are documented in the README.md

/assign @cpanato

/assign @unguiculus

/ok-to-test
/lgtm


Remove all previously deprecated configuration options

In the next major release (9.0.0), we should remove all configuration options marked with deprecated => true in default_source.rb. The replacement configuration options that currently reference these deprecated options should have their default values updated to no longer reference those options as well.

For our next major release, leave the deprecated configuration options referencing analytic events, as those have been recently deprecated. All other deprecated configuration options should be able to be removed.

Some of these deprecated attributes are called out as such in this document. Let's update it when we work on this ticket.


shader_object layer is triggering validation errors and causing our driver to crash

The cause of the crash, from what I can see, is that the shader_object layer requests that anything that can be dynamic state is requested as dynamic when the device is created. However, a bunch of that dynamic state is never actually set prior to making draw calls.
For example, if I run:
VK_INSTANCE_LAYERS=VK_LAYER_KHRONOS_shader_object:VK_LAYER_KHRONOS_validation ./deqp-vk -n dEQP-VK.pipeline.shader_object_unlinked_spirv.stencil.format.s8_uint.states.fail_keep.pass_keep.dfail_keep.comp_never

I see the following validation errors, followed by a driver crash. The final two errors are related to the missing dynamic state:
VUID-VkDeviceCreateInfo-pNext-06532(ERROR / SPEC): msgNum: -865700978 - Validation Error: [ VUID-VkDeviceCreateInfo-pNext-06532 ] Object 0: handle = 0x55a1e1ab4750, type = VK_OBJECT_TYPE_PHYSICAL_DEVICE; | MessageID = 0xcc66738e | vkCreateDevice():  If the pNext chain includes a VkPhysicalDeviceVulkan13Features structure, then it must not include a VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_DYNAMIC_RENDERING_FEATURES structure. The Vulkan spec states: If the pNext chain includes a VkPhysicalDeviceVulkan13Features structure, then it must not include a VkPhysicalDeviceDynamicRenderingFeatures, VkPhysicalDeviceImageRobustnessFeatures, VkPhysicalDeviceInlineUniformBlockFeatures, VkPhysicalDeviceMaintenance4Features, VkPhysicalDevicePipelineCreationCacheControlFeatures, VkPhysicalDevicePrivateDataFeatures, VkPhysicalDeviceShaderDemoteToHelperInvocationFeatures, VkPhysicalDeviceShaderIntegerDotProductFeatures, VkPhysicalDeviceShaderTerminateInvocationFeatures, VkPhysicalDeviceSubgroupSizeControlFeatures, VkPhysicalDeviceSynchronization2Features, VkPhysicalDeviceTextureCompressionASTCHDRFeatures, or VkPhysicalDeviceZeroInitializeWorkgroupMemoryFeatures structure (https://www.khronos.org/registry/vulkan/specs/1.3-extensions/html/vkspec.html#VUID-VkDeviceCreateInfo-pNext-06532)
    Objects: 1
        [0] 0x55a1e1ab4750, type: 2, name: NULL

Test case 'dEQP-VK.pipeline.shader_object_unlinked_spirv.stencil.format.s8_uint.states.fail_keep.pass_keep.dfail_keep.comp_never'..
VUID-VkGraphicsPipelineCreateInfo-pLibraries-06634(ERROR / SPEC): msgNum: -117483831 - Validation Error: [ VUID-VkGraphicsPipelineCreateInfo-pLibraries-06634 ] | MessageID = 0xf8ff56c9 | vkCreateGraphicsPipelines(): pCreateInfos[0] Fragment Shader and Fragment Output Interface were created with different VkPipelineMultisampleStateCreateInfo.Fragment Shader pMultisampleState:
        pNext: (nil)
        rasterizationSamples: VK_SAMPLE_COUNT_1_BIT
        sampleShadingEnable: 0
        minSampleShading: 0.000000
        pSampleMask: (nil)
        alphaToCoverageEnable: 0
        alphaToOneEnable: 0
Fragment Output Interface pMultisampleState:
        pNext: (nil)
        rasterizationSamples: VK_SAMPLE_COUNT_1_BIT
        sampleShadingEnable: 0
        minSampleShading: 0.000000
        pSampleMask: 0x55a1e5e77330
        alphaToCoverageEnable: 0
        alphaToOneEnable: 0
. The Vulkan spec states: If an element of VkPipelineLibraryCreateInfoKHR::pLibraries was created with VK_GRAPHICS_PIPELINE_LIBRARY_FRAGMENT_SHADER_BIT_EXT with a pMultisampleState that was not NULL, and if VkGraphicsPipelineLibraryCreateInfoEXT::flags includes VK_GRAPHICS_PIPELINE_LIBRARY_FRAGMENT_OUTPUT_INTERFACE_BIT_EXT, pMultisampleState must be identically defined to that used to create the library (https://www.khronos.org/registry/vulkan/specs/1.3-extensions/html/vkspec.html#VUID-VkGraphicsPipelineCreateInfo-pLibraries-06634)
    Objects: 0

VUID-vkCmdDraw-None-07845(ERROR / SPEC): msgNum:<PHONE_NUMBER> - Validation Error: [ VUID-vkCmdDraw-None-07845 ] Object 0: handle = 0x55a1e1a66870, type = VK_OBJECT_TYPE_COMMAND_BUFFER; Object 1: handle = 0x3d000000003d, type = VK_OBJECT_TYPE_PIPELINE; | MessageID = 0x5791f369 | vkCmdDraw():  VK_DYNAMIC_STATE_DEPTH_COMPARE_OP state not set for this command buffer. The Vulkan spec states: If the bound graphics pipeline state was created with the VK_DYNAMIC_STATE_DEPTH_COMPARE_OP dynamic state enabled then vkCmdSetDepthCompareOp must have been called in the current command buffer prior to this drawing command (https://www.khronos.org/registry/vulkan/specs/1.3-extensions/html/vkspec.html#VUID-vkCmdDraw-None-07845)
    Objects: 2
        [0] 0x55a1e1a66870, type: 6, name: NULL
        [1] 0x3d000000003d, type: 19, name: NULL

VUID-vkCmdDraw-pStrides-04913(ERROR / SPEC): msgNum:<PHONE_NUMBER> - Validation Error: [ VUID-vkCmdDraw-pStrides-04913 ] Object 0: handle = 0x55a1e1a66870, type = VK_OBJECT_TYPE_COMMAND_BUFFER; Object 1: handle = 0x3d000000003d, type = VK_OBJECT_TYPE_PIPELINE; | MessageID = 0x7790eb4c | vkCmdDraw():  VK_DYNAMIC_STATE_VERTEX_INPUT_BINDING_STRIDE state not set for this command buffer. The Vulkan spec states: If the bound graphics pipeline was created with the VK_DYNAMIC_STATE_VERTEX_INPUT_BINDING_STRIDE_EXT dynamic state enabled, but without the VK_DYNAMIC_STATE_VERTEX_INPUT_EXT dynamic state enabled, then vkCmdBindVertexBuffers2EXT must have been called in the current command buffer prior to this draw command, and the pStrides parameter of vkCmdBindVertexBuffers2EXT must not be NULL (https://www.khronos.org/registry/vulkan/specs/1.3-extensions/html/vkspec.html#VUID-vkCmdDraw-pStrides-04913)
    Objects: 2
        [0] 0x55a1e1a66870, type: 6, name: NULL
        [1] 0x3d000000003d, type: 19, name: NULL

There are other failures, this just happens to be one of the first tests that runs in a *shader_object* list and the crash prevents further tests running.

@gary-sweet  The validation errors in regards with dynamic state not being set come from the difference between what the spec requires from shader objects and what it requires from pipelines. There are spec changes in progress to remove the VUIDs you are seeing here and have the same requirements from pipelines and shader objects. There are CTS tests here https://gerrit.khronos.org/c/vk-gl-cts/+/12612 that test this for pipelines, can you check if these pass?

@gary-sweet The validation errors in regards with dynamic state not being set come from the difference between what the spec requires from shader objects and what it requires from pipelines. There are spec changes in progress to remove the VUIDs you are seeing here and have the same requirements from pipelines and shader objects. There are CTS tests here https://gerrit.khronos.org/c/vk-gl-cts/+/12612 that test this for pipelines, can you check if these pass?

@ziga-lunarg  From our automated internal CTS testing I can see that we are passing those exploratory tests, so something isn't quite adding up here.

The crash I'm seeing looks like it might be related to the last validation error is listed above.
The crash is caused because VK_DYNAMIC_STATE_VERTEX_INPUT_BINDING_STRIDE is set, but nothing has ever called vkCmdBindVertexBuffers2() in order to set the stride before the first draw call happens. That results in an undefined stride value being used.

I've looked over the tests and a case for VK_DYNAMIC_STATE_VERTEX_INPUT_BINDING_STRIDE is indeed missing, I will add it.

@gary-sweet I am guessing that VK_EXT_vertex_input_dynamic_state is not supported on your hardware?

@gary-sweet I am guessing that VK_EXT_vertex_input_dynamic_state is not supported on your hardware?

@ziga-lunarg We have a mix, older h/w does not support it, newer h/w does.

And is this issue seen on both?

And is this issue seen on both?

Yes, I've just confirmed that.

The crash is caused because VK_DYNAMIC_STATE_VERTEX_INPUT_BINDING_STRIDE is set, but nothing has ever called vkCmdBindVertexBuffers2() in order to set the stride before the first draw call happens

I've looked over the tests and a case for VK_DYNAMIC_STATE_VERTEX_INPUT_BINDING_STRIDE is indeed missing, I will add it.

Are you suggesting that this is somehow allowed in the spec? If so, can you point me at the relevant text?

I'm not sure yet, I think there might be a bug with stride, I will look into it, but there is a spec change in progress to remove a lot of the VUIDs that require setting dynamic state that isn't actually used, it is mentioned here: https://gitlab.khronos.org/Tracker/vk-gl-cts/-/issues/4675#note_453032

If stride is actually used and needed, then it's definitely a bug in the layer.

@gary-sweet After some investigation I see that the test should set the stride with vkCmdSetVertexInputEXT() which is called in vkPipelineConstructionUtil.cpp:4060. Is that not the case for you?

@gary-sweet After some investigation I see that the test should set the stride with vkCmdSetVertexInputEXT() which is called in vkPipelineConstructionUtil.cpp:4060. Is that not the case for you?

Ok, so things are becoming clearer; I do see that being called, and I see that our driver picks that value up. I then also see that we check whether VK_DYNAMIC_STATE_VERTEX_INPUT_BINDING_STRIDE is set and use the value of that instead. In this case, that value was never set.
I now notice some spec description that I hadn't seen before:
vkCmdSetVertexInputEXT says "If drawing using shader objects, or if the bound pipeline state object was also created with the VK_DYNAMIC_STATE_VERTEX_INPUT_BINDING_STRIDE dynamic state enabled, then vkCmdBindVertexBuffers2 can be used instead of vkCmdSetVertexInputEXT to dynamically set the stride."
and vkCmdBindVertexBuffers2 says "If drawing using shader objects or if the bound pipeline state object was also created with the VK_DYNAMIC_STATE_VERTEX_INPUT_EXT dynamic state enabled then vkCmdSetVertexInputEXT can be used instead of vkCmdBindVertexBuffers2 to set the stride."
That's a subtlety that had obviously escaped me previously. Essentially the stride can be set by either of those paths. My driver is currently assuming that if VK_DYNAMIC_STATE_VERTEX_INPUT_BINDING_STRIDE is set, we should find a valid stride set for it, which doesn't actually seem to be a requirement based on those extra spec paragraphs.
I'll modify the driver based on that which will hopefully resolve the issue. I'll post back when I have the result.

Ok - so taking that subtlety into account in the driver resolves the issue.
Apologies for the noise. I'll close the issue now.

Not an issue with the layer after all.


<EMAIL_ADDRESS>Decompress error

Failed to extract files from C:\Users\Tordenflesk\scoop\apps\authy\2.2.3\authy-2.2.3-full.nupkg.
Log file:
~\scoop\apps\authy\2.2.3\7zip.log:
7-Zip 22.01 (x64) : Copyright (c) 1999-2022 Igor Pavlov : 2022-07-15
Scanning the drive for archives:
1 file, 84580957 bytes (81 MiB)
Extracting archive: C:\Users\Tordenflesk\scoop\apps\authy\2.2.3\authy-2.2.3-full.nupkg
Path = C:\Users\Tordenflesk\scoop\apps\authy\2.2.3\authy-2.2.3-full.nupkg
Type = zip
ERRORS:
Unexpected end of archive
Physical Size = 84580957
Characteristics = Local
Sub items Errors: 1
Archives with Errors: 1
Open Errors: 1
Sub items Errors: 1

Duplicate with #10532, please consider closing this issue and leaving a comment under the previous issue.

Dupcalite of #10532


ReceivedHeader: incorrectly parse

I found a strange bug, the problem is probably in the word 'id'
My code:
$message = \ZBateson\MailMimeParser\Message::from($contents, false);
foreach ($message->getAllHeadersByName('Received') as $received) {
	echo 'FromName: ' . $received->getFromName() . PHP_EOL;
	echo 'FromHostname: ' . $received->getFromHostname() . PHP_EOL;
	echo 'FromAddress: ' . $received->getFromAddress() . PHP_EOL;
	echo 'ByName: ' . $received->getByName() . PHP_EOL;
	echo 'ByHostname: ' . $received->getByHostname() . PHP_EOL;
	echo 'ByAddress: ' . $received->getByAddress() . PHP_EOL;
	echo 'with: ' . $received->getValueFor('with') . PHP_EOL;
	echo 'getComments: ' . implode(' | ', $received->getComments()) . '<hr>';
}

Example 1 (working properly)
Code:
$contents = 'Received: from mail.uiii.ac.idd ([111.222.333.444])
by mail.yetiforce.com with esmtps (TLS1.2) tls TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256
(Exim 4.94.2)
(envelope-from<EMAIL_ADDRESS>id 1mrMVK-00079p-W2
for<EMAIL_ADDRESS>Sun, 28 Nov 2021 16:54:15 +0100';

Response:
FromName: mail.uiii.ac.idd
FromHostname:
FromAddress: 111.222.333.444
ByName:
ByHostname:
ByAddress:
with:
getComments:

Example 1 (bug)
Code:
$contents = 'Received: from mail.uiii.ac.id ([111.222.333.444])
by mail.yetiforce.com with esmtps (TLS1.2) tls TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256
(Exim 4.94.2)
(envelope-from<EMAIL_ADDRESS>id 1mrMVK-00079p-W2
for<EMAIL_ADDRESS>Sun, 28 Nov 2021 16:54:15 +0100';

Response:
FromName: mail.uiii.ac.
FromHostname:
FromAddress:
ByName:
ByHostname:
ByAddress:
with:
getComments: [111.222.333.444]

difference: mail.uiii.ac.idd  VS mail.uiii.ac.id

Aah, it's because there's a section called 'id' so it starts a sub-consumer at that point. Should be an easy enough fix to the regex here:
https://github.com/zbateson/mail-mime-parser/blob/fc0e05de6cc3a7fb449c9a5a1922a1b8b96372c7/src/Header/Consumer/Received/GenericReceivedConsumer.php#L91
In most cases the spaces before/after aren't optional except for 'from' which I think is required and always comes at the start but would have to refresh my memory on it.  So it might make more sense to have a 'FromReceivedConsumer' that uses \s*(from)\s+ and the rest have \s+(token)\s+.
Lastly some better testing needs to be added.
Thanks for reporting, @mariuszkrzaczkowski

some solution ?? it blocks my job

some solution ?? it blocks my job

Feel free to submit a PR with a fix.

I have no idea how to fix it and where

if I knew it, I would have created PR a long time ago

In most cases the spaces before/after aren't optional except for 'from' which I think is required and always comes at the start but would have to refresh my memory on it. So it might make more sense to have a 'FromReceivedConsumer' that uses \s*(from)\s+ and the rest have \s+(token)\s+.

I tried what you wrote but it didn't work

Fixed in 2.2.0.


Fixed docblock for errors method on ValidationException

The title said it all.

No, the contract is correct.

This is taken straight from the exception in question:
	/**
	 * Get the validation error message provider.
	 *
	 * @return \Illuminate\Contracts\Support\MessageProvider
	 */
	public function errors()
	{
		return $this->provider->getMessageBag();
	}

	/**
	 * Get the validation error message provider.
	 *
	 * @return \Illuminate\Contracts\Support\MessageProvider
	 */
	public function getMessageProvider()
	{
		return $this->provider;
	}

Here we have 2 methods that return different things, yet somehow they end up with the exact same docblocks. Funny that, isn't it? Care to explain how that's correct?

They are both returning implementations of this contract.

Thanks for clearing that up. Didn't see that.


generate first the description walkthrough. only than the title

https://github.com/Codium-ai/pr-agent/pull/542


Fixes link for installation document

Fixes the link to the installation document from the getting started
guide.
Closes: https://github.com/redhat-developer/odo/issues/1496

:white_check_mark: odo build 2177 completed and artifacts can be found here (for commit https://github.com/redhat-developer/odo/commit/2b276dbb76 by @cdrage)

/lgtm


Is desktop windows application supported?

Hi Is desktop windows application supported by this library? because in documentation home page it featured windows tag

It's supported but unreliable in release mode. Sometimes it prints everything and cuts sometimes it does not.. Even on their Demo app. There is a bug somewhere in the code for Windows. I would not recommend it for production. Until it has been fixed.

yes it work with me
https://www.youtube.com/watch?v=ZjWOiXC3nIk


Update vbatts email address in MAINTAINERS

Noticed after seeing https://github.com/opencontainers/image-spec/pull/831
I used the email address on @vbatts' GitHub profile, please let me know if you prefer something else 😄
edit: This also aligns with MAINTAINERS in runtime-spec and distribution-spec, so that's probably good signal.

LGTM

LGTM.

@vbatts Need to re-LGTM.


UI components not respecting RenderLayers

The visibility of UI entities like text is not affected by the RenderLayers component. This is a problem when trying to prerender a scene with RenderTarget::Image to use it as a mesh texture (the game UI will also be visible on the texture).
Is there a way to make UI components respect RenderLayers?

I modified the render_to_texture example to demonstrate the bug:
https://github.com/paulkre/bevy/blob/ui-renderlayers-demo/examples/3d/render_to_texture.rs
https://user-images.githubusercontent.com/13008492/192303297-2b443737-1545-4139-9c97-153149b3a038.mp4

I just found an acceptable workaround. You can add the UiCameraConfig { show_ui: false } component to your RenderTarget::Image-camera which hides all UI elements for that camera. This helps most of the time but sometimes you might want to render different UI elements in both cameras. In these situations it would be great if RenderLayers worked for UI.

This look like a duplicate of #5721 . Hasn't this been fixed with #7671 @paulkre can you reproduce in 0.10?

Updated example change that demonstrates the issue:
diff --git a/examples/3d/render_to_texture.rs b/examples/3d/render_to_texture.rs
index 81c687e5a..e4473bcae 100644
--- a/examples/3d/render_to_texture.rs
+++ b/examples/3d/render_to_texture.rs
@@ -35,6 +35,7 @@ fn setup(
     mut meshes: ResMut<Assets<Mesh>>,
     mut materials: ResMut<Assets<StandardMaterial>>,
     mut images: ResMut<Assets<Image>>,
+    asset_server: Res<AssetServer>,
 ) {
     let size = Extent3d {
         width: 512,
@@ -141,6 +142,17 @@ fn setup(
         transform: Transform::from_xyz(0.0, 0.0, 15.0).looking_at(Vec3::ZERO, Vec3::Y),
         ..default()
     });
+
+    commands.spawn((
+        TextBundle::from_section(
+            "hello\nbevy!",
+            TextStyle {
+                font: asset_server.load("fonts/FiraSans-Bold.ttf"),
+                font_size: 200.0,
+                color: Color::RED,
+            },
+        ),
+    ));
 }

 /// Rotates the inner cube (first pass)

Same problem. I use two cameras: one for classic game rendering, and second for rendering onto tile map (decals like blood texures, etc..). But when I spawn a TextBundle it renders to screen (which is ok) and to all my tiles (which is bad).
Here's how I spawn text:
world.spawn(
    TextBundle::from_sections([
        TextSection::new("FPS: ", style.clone()),
        TextSection::from_style(style.clone()),
        TextSection::new("\nEntities: ", style.clone()),
        TextSection::from_style(style.clone()),
        TextSection::new("\nAudio sources: ", style.clone()),
        TextSection::from_style(style.clone()),
        TextSection::new(
            "\n\
            \nSpawn weapon: [G]\
            \nSpawn human : [H] group: [+SHIFT]\
            \nSpawn zombie: [J] group: [+SHIFT]\
            ",
            style,
        ),
    ])
);

Here's how I spawn second camera (for tile map blending):
commands
    .spawn(Camera2dBundle {
        camera: Camera {
            target: RenderTarget::Image(target),
            output_mode: CameraOutputMode::Write {
                blend_state: Some(BlendState::PREMULTIPLIED_ALPHA_BLENDING),
                color_attachment_load_op: LoadOp::Load,
            },
            ..Default::default()
        },
        camera_2d: Camera2d {
            clear_color: ClearColorConfig::None,
        },
        transform,
        ..Default::default()
    })
    .insert(RenderLayers::layer(1)); // NOTE: used different render layer


edit:208 Uncaught TypeError: $image.cropper is not a function

I don't use git submodule. I dropped the files into a field. Maybe it has something to do with it?
It gives edit:208 Uncaught TypeError: $image.cropper is not a function in the console log.

You are missing the submodule of this repository. Download the Files from https://github.com/fengyuanchen/cropper
Your site/fields/cropper-Folder will have another cropper folder containing the files you find over at fengyuanchen/cropper

I don't use submodule but I dragged the files from cropper into the folder and that solved this problem.
Now I have some new errors:
{
                        "filename": "background.jpg",
                        "url": "http://localhost/ekobyn.com/content/haha/background.jpg",
                        "root": "C:\wamp\www\ekobyn.com\content\haha\background.jpg"
                    }
VM9477:4 Uncaught SyntaxError: Unexpected token w(anonymous function) @ VM9470:34K.event.dispatch @ panel.min.js?v=2.2.3:4g.handle @ panel.min.js?v=2.2.3:4

So it's really required to download this with the submodule?


Add Continuous Integration

Continuous Integration
For pDSL to evolve and be scalable once multiple contributors work on it we want and need integration with continuous systems that constantly check its integrity.
As a starter we want these CI systems to be supported:

[ ] TravisCI: for Linux and Mac builds

[x] Register repository
[ ] Verify that master compiles successfully ..

[x] .. on Linux
[ ] .. on Mac

[x] Run pdsl_core unit tests on push or PR to master
[x] Run coverage report

[x] Upload coverage report to Coveralls (seems malfunction atm)
[x] Upload coverage report to CodeCov

[ ] Verify that the code is formatted as intended.
[x] README: Link to travisCI build status

[ ] Appveyor: for Windows builds

[ ] Register repository
[ ] Verify that master compiles successfully
[ ] Run pdsl_core unit tests on push or PR to master
[ ] README: Link to AppVeyor build status

[x] Coveralls: since it is the most popular solution and has a nice bot

[x] Register repository
[x] Make Coveralls bot check PRs
[x] Display coverage reports correctly
[x] README: Link to Coveralls coverage status

[x] CodeCov: since it displays coverage better than coveralls (opinionated)

[x] Register repository
[x] Display coverage reports correctly
[x] README: Link to CodeCov coverage status

This issue tracks the progress made on this work item.
Travis CI
Should do the most of the work load.

Compile-check cargo check the project
Run unit tests of its components cargo test --all-features -all
Do the same in release mode since sometimes release can change semantics of code, e.g. debug_assert! is thrown out.
Optionally perform bench mark tests if any using e.g. cargo bench.
Optionally generating self hosted documentation using GitHub pages.
Generate a coverage report and sending it to Coveralls and CodeCov.

AppVeyor
Since AppVeyor can be very slow we should minimize its functionality to the possible minimum.

Compile-check cargo check the project.
Run unit tests of its components in debug mode only.

Coveralls
We should make use of its bot to never degrade in coverage upon new PRs.
Summary
We want protected branches (especially master) so that no one can merge a PR that breaks the masters invariances:

The code should always compile.
Tests should always pass.
Code test coverage should never decrease.
Performance tests should never show degradations unless fixing a correctness issue.

Since https://github.com/Robbepop/pdsl/commit/059a3406843a03f1f03b3bdee4d4ed47560afcfe we have clippy built into the CI pipeline. :tada:

We should add another issue if we want to add Appveyor or Windows CI support and close this issue.


Dockerfile(engine): use alpine-3.14.6 in pingcap public gcr

What problem does this PR solve?

Issue Number: ref #5668
What is changed and how it works?
Use gcr.io/pingcap-public/pingcap/alpine:alpine-3.14.6 instead of alpine:3.16 to fix some security issues.
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
None

image build and sync verified in: https://cd.pingcap.net/blue/organizations/jenkins/tiflow-docker/detail/tiflow-docker/15/pipeline

/merge


Skip synthetic params by guess (no attribute available unfortunately)

Fixes #2

A diff made against fake source gen for two mapping poet versions on current Fabric yarn 20w16a as of comment is here: https://gist.github.com/9ab6317a5e569ba0574266cbcdc978bf
The result seems to be fine

Let someone else check it out before we merge it. Though this doesn't affect yarn (which specifies build 1 to use)


Multilanguage & slugs

Hi,
I have a site based on Camaleon CMS v2.4.3 with multiple languages.
An article is available in each language by its slug as follows:
English version: http://example.com/article1_name
Russian version: http://example.com/ru/nazvanie_statji1
Now i'm using the following code to draw a language selection menu:
def custom_draw_languages()
  lan = current_site.get_languages
  return if lan.size < 2

  res = ["

#{I18n.locale.to_s}
"]

  lan.each do |lang|
    next if I18n.locale.to_s == lang.to_s

    res << "
 #{lang}
"
  end

  res << "

"
  res.join("")
end

I'm accessing the english version of the article http://example.com/article1_name, after that i want to open its russian version using the language menu selector. The problem is that it points to:
http://example.com/ru/article1_name
but it should point to:
http://example.com/ru/nazvanie_statji1
How can i draw the language menu so it outputs the correct slug ?

@milleroff please test using this method:
https://github.com/owen2345/camaleon-cms/blob/master/app/decorators/camaleon_cms/site_decorator.rb#L115
to verify if it renders with the same error.

Thanks for your answer, but it didn't help.
When using the original "draw_languages" function that you pointed to:
<%= raw current_site.draw_languages('', true, '') %>

i get URLs like this:
http://example.com/ru/article1_name?cama_set_language=ru
But as i mentioned in the first post, i need an URL with a translated slug:
http://example.com/ru/nazvanie_statji1

Hello @owen2345 , I'm having the same problem. The cama_url_to_fixed method in the CamaleonCms :: SiteDecorator class does not consider the slug in the new language, is there any method that retrieves the language-based slug?
Thanks,

I have the same problem, have a solution?

See #774


fix: httpprobeport default if empty

@anichari Is this possible in CFT with an allowedpattern instead to not permit empty? If so, that would probably be more ideal. Otherwise, this just says "if user overrides default but leaves blank/empty, proceed with the deployment but set the value back to 50000/default"

@anichari Is this possible in CFT with an allowedpattern instead to not permit empty? If so, that would probably be more ideal. Otherwise, this just says "if user overrides default but leaves blank/empty, proceed with the deployment but set the value back to 50000/default"

The behavior as of now is that we set the probe port to empty as well, which in turn prevents janus from running the cloud health server. If we are sure that going forward we will always need to run the probe server, then we can enforce the default. If not, i'd keep the existing behavior.


1st sub menu button appears when scrolling

Using the Sample code, if you run the Sample app and select the "Home" fragment, if you scroll the list of countries downwards, the first sub menu button appears into view and scrolls to the top. This appears even if the menu is closed.

This is present in commit f42b3f2995f4594fc289386c0994ad0c415c46f9. I've just noticed there is a newer commit so I'll evaluate that first.


Does not work with alphanumeric columns

Would be really useful if updated with the above feature

This might help: http://stackoverflow.com/questions/6692833/jquery-tablesorter-sorting-alphanumeric-columns

If you are interested, I have a fork of tablesorter which uses a natural sort algorithm by default (demo).


⚠ ClamAV has degraded performance

In 62811bd, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 676 ms

Resolved: ClamAV performance has improved in fbc4f66.


New omnibar

Codecov Report

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

@@          Coverage Diff          @@
##           master    #80   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          23     23
  Lines         529    529
  Branches       59     59
=====================================
  Hits          529    529

Impacted Files
Coverage Δ

lib/sky-pages-module-generator.js
100% <ø> (ø)
:white_check_mark:

lib/sky-pages-bootstrap-generator.js
100% <ø> (ø)
:white_check_mark:

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 ec4c668...2d92a44. Read the comment docs.


Add more scale-mixtures

We need to add more scale-mixtures.

For Gaussian scale mixtures, see for instance this paper: http://www2.stat.duke.edu/~mw/MWextrapubs/West1987a.pdf


Add integration tests for Reindex job task

Description
Adds integration tests which create a new search param, reindex resources using it, and then search using the new parameter
Related issues
Addresses [issue AB#73396].
Testing
Created and ran the new integration tests

@feordin Seems like this PR contains other edits apart from the integration tests.

@feordin Seems like this PR contains other edits apart from the integration tests.

Yes, it needed a rebase, should be fixed now.


Tailwindcss classes are not working in Tooltip & Dialog components

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
I set up the project as per the docs about using mui along with the tailwindcss and all classes are working as expected.
but rounded-none tailwindcss class is not working for the tooltip. (other tailwind classes are also not working for the tooltip)

Expected behavior 🤔
i'd love to remove the rounded border by tailwindcss class
Steps to reproduce 🕹
Steps:

created the tooltip component
set the classes tooltip prop by tailwindcss ones

Context 🔦
No response
Your environment 🌎
System:
OS: macOS 12.4
Binaries:
Node: 16.15.1 - ~/.nvm/versions/node/v16.15.1/bin/node
Yarn: 1.22.19 - ~/.nvm/versions/node/v16.15.1/bin/yarn
npm: 8.11.0 - ~/.nvm/versions/node/v16.15.1/bin/npm
Browsers:
Chrome: 103.0.5060.114
Edge: Not Found
Firefox: Not Found
Safari: 15.5
npmPackages:
@emotion/react: ^11.9.3 => 11.9.3
@emotion/styled: ^11.9.3 => 11.9.3
@mui/base:  5.0.0-alpha.88
@mui/icons-material: ^5.8.4 => 5.8.4
@mui/material: ^5.8.7 => 5.8.7
@mui/private-theming:  5.8.6
@mui/styled-engine:  5.8.7
@mui/system:  5.8.7
@mui/types:  7.1.4
@mui/utils:  5.8.6
@types/react: ^18.0.10 => 18.0.10
react: ^18.1.0 => 18.1.0
react-dom: ^18.1.0 => 18.1.0
typescript: ^4.7.2 => 4.7.2

Could be related to https://github.com/mui/material-ui/issues/33017. Can you please provide a codesandbox/stackblitz to showcase the issue?

@mnajdova
thanks for taking a look at this issue
i will provide the showcase by the sandbox.

@mnajdova
Here is the sandbox link
https://codesandbox.io/s/tailwindcss-using-on-the-mui-tooltip-p5t9us?file=/src/App.tsx
thanks

I don't think the codesandbox is working at all, as none of the tailwind classes are working there. Can you try updating your index.tsx to:
import React from "react";
import { createRoot } from "react-dom/client";
import {
  ThemeProvider,
  StyledEngineProvider,
  createTheme
} from "@mui/material/styles";
import { CssBaseline } from "@mui/material";
import "./index.css";
import App from "./App";
import reportWebVitals from "./reportWebVitals";

const rootElement = document.getElementById("root");
const root = createRoot(rootElement!);

const theme = createTheme({
  components: {
    MuiPopover: {
      defaultProps: {
        container: rootElement
      }
    },
    MuiPopper: {
      defaultProps: {
        container: rootElement
      }
    }
  }
});

root.render(
  <React.StrictMode>
    <StyledEngineProvider injectFirst>
      <ThemeProvider theme={theme}>
        <CssBaseline />
        <App />
      </ThemeProvider>
    </StyledEngineProvider>
  </React.StrictMode>
);

// If you want to start measuring performance in your app, pass a function
// to log results (for example: reportWebVitals(console.log))
// or send to an analytics endpoint. Learn more: https://bit.ly/CRA-vitals
reportWebVitals();

and see if it works.

i updated index.tsx like yours
but it seems still not working 😢

In that case, can you maybe just share a link to a github repo I can clone and try locally?

@mnajdova
thanks for your patience.
i missed this part in my code and it was the reason.
const theme = createTheme({
  components: {
    MuiPopover: {
      defaultProps: {
        container: rootElement
      }
    },
    MuiPopper: {
      defaultProps: {
        container: rootElement
      }
    }
  }
});

root.render(
  <React.StrictMode>
    <StyledEngineProvider injectFirst>
      <ThemeProvider theme={theme}>
        <CssBaseline />
        <App />
      </ThemeProvider>
    </StyledEngineProvider>
  </React.StrictMode>
);

so it's working fine at the moment but still not working for the Dialog ones yet.
do i need to raise a new issue for that and close this?
I created the github repository for that
https://github.com/baymax88/mui-with-tailwindcss

How can we work around the issue on next.js project? It would give us an error as document won't be available.

Never mind. I solved it as:
function container() {
return document.getElementById("__next");
}
const theme = createTheme({
components: {
MuiPopover: {
defaultProps: {
container
}
},
MuiPopper: {
defaultProps: {
container
}
},
MuiDialog: {
defaultProps: {
container
},
},
}
});


Revert "Refactor job response file handling to (hopefully) prevent regressions."

Reverts apple/swift-driver#1036
cc: @allevato @artemcm
Sorry that I have to revert this PR. This PR seems to have caused conflicts with our internal repo. We'll investigate how to resolve them and merge this again.

@swift-ci please test

Thanks for fixing this @artemcm !

Thanks for finding a fix without needing to revert it!

@swift-ci please test source compatibility

(Sorry, trying to test out a theory)

Re-closing, sorry for the noise


Clarification: Data Type display on docs serve using mysql adapter

Describe the bug
Apologies, we do not know if this is a bug yet. Hence like to check with the rest of users who may be able to help. After docs generate and serve, we see that the data type are displayed with b'xxxx'. Please see screenshot below:

We do see the same in the generated target/catalog.json file. Not sure if this is adapter specific or configuration issue but we  searched and could not find a solution.
Appreciate some guidance on this. Thank you!
Steps To Reproduce
In as much detail as possible, please provide steps to reproduce the issue. Sample data that triggers the issue, example model code, etc is all very helpful here.
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots and log output
If applicable, add screenshots or log output to help explain your problem.
The output of dbt --version:

The operating system you're using:
Ubuntu 22.04.1 LTS
The output of python --version:
3.9.13
Additional context
Add any other context about the problem here.

@clementchong thanks for opening this issue!
It sounds like target/catalog.json file contains a Python bytes literal rather than a str literal (see here and here for further explanation).
Basically, it has encoded data that needs to be decoded in order to render properly.
Which database and database version are you using (e.g., MySQL 8.0.32)?
Do you know what your "Connection Character Set and Collation System Variable" settings are?

character_set_results
character_set_server
collation_server
character_set_database
collation_database

character_set_results is "the character set used for returning query results to the client. This includes result data such as column values, result metadata such as column names, and error messages." I'm wondering if your setting might explain the bytes encoding like b'varchar(8)', etc.

@dbeatty10 Thank you for the quick and detailed overview. Our MySQL version is 8.0.25. The variable values are as below:

Do you see anything wrong?

I found the below discussion, would try out.
https://stackoverflow.com/questions/40921341/python-mysql-connector-returns-bytearray-instead-of-regular-string-value

I also found the information_schema.columns table having binary collation for the data_type field. However, I am unable to change it.

I also ran into this issue and it appears to be targeted at MySQL 8.0. The data_type column's collation in information_schema.columns is binary in this version, while it is non-binary in MySQL 5.7 and MariaDB 10.5. When a column is defined with binary collation, the Python MySQL connector returns the column value as a byte.
The collation settings for each database versions are as follows:

DATABASE
DATA_TYPE
CHARACTER_SET_NAME
COLLATION_NAME

mysql:5.7
varchar
utf8
utf8_general_ci

mysql:8.0
longtext
utf8mb3
utf8mb3_bin

mariadb:10.5
varchar
utf8
utf8_general_ci

The different collation settings for each database version can be observed by running the following commands and changing the DOCKER_IMAGE environment variables:
$ DOCKER_IMAGE=mysql:8.0; export DOCKER_IMAGE
$ read -d '' QUERY_TEXT << EOF
select column_name,
       data_type,
       character_set_name,
       collation_name
  from information_schema.columns
 where table_schema = 'information_schema'
   and table_name = 'COLUMNS'
   and column_name = 'DATA_TYPE';
EOF

$ docker run --name col-data-type-collation-check -e MYSQL_ALLOW_EMPTY_PASSWORD=yes -d ${DOCKER_IMAGE}

# Wait for database to start up then:
$ docker exec col-data-type-collation-check mysql -e "${QUERY_TEXT}"
COLUMN_NAME	DATA_TYPE	CHARACTER_SET_NAME	COLLATION_NAME
DATA_TYPE	longtext	utf8mb3	                utf8mb3_bin

# Tidy up
$ docker stop col-data-type-collation-check && docker rm col-data-type-collation-check && docker image rm ${DOCKER_IMAGE}

A proposed fix is to remove the b character prefix from dtype in the MySQLAdapter parse_show_columns function  as follows:
import re
...
MySQLColumn(
    table_database=None,
    table_schema=relation.schema,
    table_name=relation.name,
    table_type=relation.type,
    table_owner=None,
    table_stats=None,
    column=column.column,
    column_index=idx,
    # dtype=column.dtype,
    dtype = re.sub(r"^b'|'$", "", column.dtype)
 )

Dear Jeff
Thank you for the quick explanation and proposed fix. It is very helpful!
Regards
Clement
On Sun, May 7, 2023, 00:32 Jeff Moszuti @.***> wrote:

I also ran into this issue and it appears to be targeted at MySQL 8.0. The
data_type column's collation in information_schema.columns is binary in
this version, while it is non-binary in MySQL 5.7 and MariaDB 10.5. When a
column is defined with binary collation, the Python MySQL connector returns
the column value as a byte.
The collation settings for each database versions are as follows:
DATABASE DATA_TYPE CHARACTER_SET_NAME COLLATION_NAME
mysql:5.7 varchar utf8 utf8_general_ci
mysql:8.0 longtext utf8mb3 utf8mb3_bin
mariadb:10.5 varchar utf8 utf8_general_ci
The different collation settings for each database version can be observed
by running the following commands and changing the DOCKER_IMAGE environment
variables:
$ DOCKER_IMAGE=mysql:8.0; export DOCKER_IMAGE
$ read -d '' QUERY_TEXT << EOF
select column_name,
data_type,
character_set_name,
collation_name
from information_schema.columns
where table_schema = 'information_schema'
and table_name = 'COLUMNS'
and column_name = 'DATA_TYPE';
EOF
$ docker run --name col-data-type-collation-check -e MYSQL_ALLOW_EMPTY_PASSWORD=yes -d ${DOCKER_IMAGE}
Wait for database to start up then:
$ docker exec col-data-type-collation-check mysql -e "${QUERY_TEXT}"
COLUMN_NAME	DATA_TYPE	CHARACTER_SET_NAME	COLLATION_NAME
DATA_TYPE	longtext	utf8mb3	                utf8mb3_bin
Tidy up
$ docker stop col-data-type-collation-check && docker rm col-data-type-collation-check && docker image rm ${DOCKER_IMAGE}
A proposed fix is to remove the b character prefix from dtype in the MySQLAdapter
parse_show_columns
https://github.com/dbeatty10/dbt-mysql/blob/main/dbt/adapters/mysql/impl.py#L97-L113
function as follows:
import re
...
MySQLColumn(
table_database=None,
table_schema=relation.schema,
table_name=relation.name,
table_type=relation.type,
table_owner=None,
table_stats=None,
column=column.column,
column_index=idx,
# dtype=column.dtype,
dtype = re.sub(r"^b'|'$", "", column.dtype)
)
—
Reply to this email directly, view it on GitHub
https://github.com/dbeatty10/dbt-mysql/issues/108#issuecomment-1537177062,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AX6SX27QPBLV5PBDRFYB723XEZ4JZANCNFSM6AAAAAAQAILRU4
.
You are receiving this because you were mentioned.Message ID:
@.***>


[QUESTION] I found this text in my .exe file when viewing it in a file editor. Why is it there and can I change it?

So I complied a program and used the tag --follow-imports and it runs perfectly, no issue in that. However I saw this when viewing the files in a text editor (VS Code), there are like, docs for all the modules I used. Why is it there and will the program be affected if I remove them or edit the text? Also you can consider making a discord or subreddit for any future noob question

You are obviously including doc strings, which can be disabled, check out the documentation. There are a couple of Python flags you ought to use. Nuitka commercial has more details on this, if your goal is obfuscucation. Constant data is not hidden in normal Nuitka, but only covered by Nuitka commercial: https://nuitka.net/doc/commercial.html#protection-against-reverse-enginneering

I am leaving this open as a remind to myself that Nuitka needs to make an output that explains what a created output is and is not, and provide links to this kind of information.

I have came back to this, and I was considering, if not I want to first make things more selective by chance. It would be nice if e.g. stdlib was not including the docstrings by default, and that per Yaml and options, this could be selected, and then we would report on the amount of docstrings included for modules and functions, and point to a mnemonic, that explains it better on the site. That will be a bit of development work, but my suspect is that at least for some packages, say numpy and stdlib, we can get this to be the default for standalone compilation at least. Maybe something for a live stream this week.

how to fix this error
[2024-10-13 07:34:10] [INFO    ] discord.client: logging in using static token

Exception Group Traceback (most recent call last):
|   File "C:\Users\Solar\AppData\Local\Temp\onefile_8088_133733036194204284\EliteOptimizer.py", line 15306, in
|   File "C:\Users\Solar\AppData\Local\Temp\onefile_8088_133733036194204284\EliteOptimizer.py", line 952, in init
|   File "C:\Users\Solar\AppData\Local\Temp\onefile_8088_133733036194204284\EliteOptimizer.py", line 1436, in home
|   File "C:\Users\Solar\AppData\Local\Temp\onefile_8088_133733036194204284\discord\client.py", line 869, in run
|   File "C:\Users\Solar\AppData\Local\Temp\onefile_8088_133733036194204284\asyncio\runners.py", line 194, in run
|   File "C:\Users\Solar\AppData\Local\Temp\onefile_8088_133733036194204284\asyncio\runners.py", line 118, in run
|   File "C:\Users\Solar\AppData\Local\Temp\onefile_8088_133733036194204284\asyncio\base_events.py", line 687, in run_until_complete
|   File "C:\Users\Solar\AppData\Local\Temp\onefile_8088_133733036194204284\discord\client.py", line 858, in runner
|   File "C:\Users\Solar\AppData\Local\Temp\onefile_8088_133733036194204284\discord\client.py", line 786, in start
|   File "C:\Users\Solar\AppData\Local\Temp\onefile_8088_133733036194204284\discord\client.py", line 620, in login
|   File "C:\Users\Solar\AppData\Local\Temp\onefile_8088_133733036194204284\discord\http.py", line 816, in static_login
|   File "C:\Users\Solar\AppData\Local\Temp\onefile_8088_133733036194204284\discord\http.py", line 638, in request
|   File "C:\Users\Solar\AppData\Local\Temp\onefile_8088_133733036194204284\aiohttp\client.py", line 1355, in aenter
|   File "C:\Users\Solar\AppData\Local\Temp\onefile_8088_133733036194204284\aiohttp\client.py", line 659, in _request
|   File "C:\Users\Solar\AppData\Local\Temp\onefile_8088_133733036194204284\aiohttp\connector.py", line 557, in connect
|   File "C:\Users\Solar\AppData\Local\Temp\onefile_8088_133733036194204284\aiohttp\connector.py", line 1002, in _create_connection
|   File "C:\Users\Solar\AppData\Local\Temp\onefile_8088_133733036194204284\aiohttp\connector.py", line 1305, in _create_direct_connection
|   File "C:\Users\Solar\AppData\Local\Temp\onefile_8088_133733036194204284\aiohttp\connector.py", line 1061, in _wrap_create_connection
|   File "C:\Users\Solar\AppData\Local\Temp\onefile_8088_133733036194204284\aiohappyeyeballs\impl.py", line 88, in start_connection
|   File "C:\Users\Solar\AppData\Local\Temp\onefile_8088_133733036194204284\aiohappyeyeballs_staggered.py", line 91, in staggered_race
|   File "C:\Users\Solar\AppData\Local\Temp\onefile_8088_133733036194204284\asyncio\taskgroups.py", line 145, in aexit
| ExceptionGroup: unhandled errors in a TaskGroup (1 sub-exception)
+-+---------------- 1 ----------------
| Traceback (most recent call last):
|   File "C:\Users\Solar\AppData\Local\Temp\onefile_8088_133733036194204284\aiohappyeyeballs_staggered.py", line 88, in run_one_coro
| aiohappyeyeballs._staggered._Done
+------------------------------------


Add support for GZipped contents.

Moved this to a new PR because I messed up my upstream while creating the other PR.

This has been left open pending looking at the thread handling, etc. I have new commits building on the Jetty 9 upgrade (which I believe gets rid of the thread pooling anyways) that will land that will allow a user to set their own HTTP client object so closing this.


Create one integration test for each mutation factory functions.

In bigtable/client/mutations.h we have multiple factory functions, i.e.:

[ ] bigtable::SetCell(family, column, timestamp, value)
[ ] bigtable::SetCell(family, column, value)
[ ] bigtable::DeleteFromColumn()
[ ] bigtable::DeleteFromColumnStartingFrom()
[ ] bigtable::DeleteFromColumnEndingAt()
[ ] bigtable::DeleteFromFamily()
[ ] bigtable::DeleteFromRow()

We need to write an integration test for each one, much like we did for bigtable::Filter::*.

There is a file mutations_test.cc, where all the mentioned test cases are covered.
Do I need to do any changes in those test cases?

No need to change mutations_test.cc.  That is a unit test, we need an integration test like bigtable/client/filters_integration_test.cc where we verify that using these operations against the Cloud Bigtable Emulator has the desired effect.

PTAL


fluid pump doesn't notice range addon change

Industrial Foregoing Version: 1.12.3
TeslaCoreLib Version: 1.12.2-1.0.15
if a fluid pump has power, then it doesn't notice when you add/change the range addon.

:wave: Sorry but Industrial Foregoing has ended support for version 1.12. I'm closing this issue as it will no longer be updated for that version. Thanks for playing with the mod.


Streamline data loader for data set collection

[x] reduce redundant code
[x] document options better
[x] yield templates to copy paste during new contributions
[x] adopt style throughout current library

data sets to modify:

[ ] d10_1038_s41467_019_12464_3
[ ] d10_1016_j_cmet_2019_01_021
[ ] d10_1038_s41586_018_0698_6
[ ] d10_1038_s41586_020_2922_4
[ ] d10_1038_s41591_019_0468_5
[ ] d10_1084_jem_20191130
[ ] d10_1101_2020_03_13_991455
[ ] d10_1186_s13059_019_1906_x

HCL, TMS, MCA will also require modification, waiting for changes to cell type handling for these, though. Will address later in separate PR.

data sets to modify:

[ ] d10_1038_s41467_019_12464_3
[ ] d10_1016_j_cmet_2019_01_021
[ ] d10_1038_s41586_018_0698_6
[ ] d10_1038_s41586_020_2922_4
[ ] d10_1038_s41591_019_0468_5
[ ] d10_1084_jem_20191130
[ ] d10_1101_2020_03_13_991455
[ ] d10_1186_s13059_019_1906_x

HCL, TMS, MCA will also require modification, waiting for changes to cell type handling for these, though. Will address later in separate PR.

Merging this to ease further development of data module, while full data repository checks are still running. Will address potential bugs in further PRs.

Merging this to ease further development of data module, while full data repository checks are still running. Will address potential bugs in further PRs.

Note that the anatomical groups work via data set IDs now, moving forward we will probably only use such groups for reproducility of what data sets a model was trained on and can probably reduce this to data set ID lists in a single file. This file would then be in a reproducibility repo of a project, so I would also phase these out of sfaira soon. @le-ander

Note that the anatomical groups work via data set IDs now, moving forward we will probably only use such groups for reproducility of what data sets a model was trained on and can probably reduce this to data set ID lists in a single file. This file would then be in a reproducibility repo of a project, so I would also phase these out of sfaira soon. @le-ander


Add Function to Retrieve Currency Code by Numeric Code

This PR adds a static function, GetCurrencyCodeByNumericCode, to allow retrieval of a currency code based on its ISO-4217 numeric code.
Purpose:
Simplifies the process of finding a currency code without manually iterating through the currency list.
Provides a more efficient and straightforward method for users needing to convert numeric codes to currency codes.
This function will help streamline workflows that rely on numeric codes for currency identification
Summary by Sourcery
Add a new function GetCurrencyByNumericCode to facilitate retrieving currency codes by their numeric ISO-4217 codes, and include a corresponding test to verify its functionality.
New Features:

Introduce a static function GetCurrencyByNumericCode to retrieve a currency code using its ISO-4217 numeric code.

Tests:

Add a test case for the GetCurrencyByNumericCode function to ensure it returns the correct currency for a given numeric code.

@Rhymond please take a look on that when you have time.


[Feat] Can executing 'Deno. cwd()' in the worker return an independent directory?

I prefer an independent environment for executing workers. I currently treat workers as a lightweight container and hope that each worker has their own directory to handle certain tasks in conjunction with Deno's permission management.

Deno.cwd() is only available in deno, not deno_core. That said - Deno.cwd() returns the CWD based on a syscall, so if you have multiple JsRuntime instance in a single process they will all return the same CWD.

thanks！


feat: enable runtime configuration reloads for auth

As part of a longer term goal to support runtime configuration loading the auth team would like to make use of the --config-dir flag and ensure the /etc/auth.d folder is created. (Related: auth - feat: config reloading)
This is a small diff doing three things:

Create the /etc/auth.d directory.
Copies the gotrue-optimizations.service.j2 to also copy the gotrue.generated.env file to the /etc/auth.d directory.
Change the gotrue.service.j2 to use the --config-dir flag set to the newly created /etc/auth.d directory.

What kind of change does this PR introduce?
Feature
What is the current behavior?
Right now gotrue.service is started without the --config-dir flag.
What is the new behavior?
The gotrue.service  will be started with the --config-dir flag, causing the service to reload configuration changes at runtime.
Additional context
I tested this change and did notice the machines I have access to do not have the /etc/gotrue folder created, I also did not see that folder in the ansible configuration files. I chose to be safe and copy the same invalid path. If the path was an oversight and we want to also update the path of the generated config file to something that exists, I'm open to including that change here.

@cstockton I'll try to take a look today, the change in gotrue service seems to have broken the integration test that builds out AMI and tests the gotrue and other services
https://github.com/supabase/postgres/actions/runs/11003641978/job/30553062346#step:11:235
Probably going to be something simple we can do to adjust and I will look at it soon.

@samrose Thanks sam, apologies, I should have mentioned this set of changes has a direct dependency on the recently merged but currently unreleased changes in https://github.com/supabase/auth/pull/1771. If a flag provided to auth doesn't exist, the server will exit.

@cstockton ah ok. Maybe it was the intention just to keep this in draft until the the auth changes release then?

@samrose You're right Sam, I should have been more clear, the intent is to leave this as a draft. I was hoping to get ahead of any feedback you might have on the changes themselves.

@samrose The pull requests that this change depended on have been merged. This is ready for review now, assuming the tests I just re-ran pass, thank you.

How Is there a way to view logs for why GOTRUE is not ready?

Followed up in person with @cstockton

@cstockton  do you need any other help with this? Is it ready to review/merge?

This is ready for merging soon as your team is happy with the changes @samrose

@cstockton already approved from me needs approval also from @supabase/backend

@darora May I please get an approval for this change?


Unable to execute castnow remotely

Hello, I have a shell script (.sh) that executes castnow which works fine but when I try to run that same shell script remotely, it does not work (For example, if I run the shell script from a php script that is hosted on my lighttpd server).  I've also tried various other methods of trying to execute castnow remotely but to no avail and with the same outcome.  It would either not work at all or it would occasionally start playing a video for a few seconds and then just stop.

Can you add an short example of your shell and php script?


Binaries committed by mistake?

kernel/initrd is a binary file that seems to have been committed by mistake. It does not seem to be useful because there is a script kernel/make-initrd that rebuild the initrd.
Could it be removed from the repository?
/cc @sstabellini

Closed via https://github.com/rkt/stage1-xen/commit/484b17344359b7876cafb29bfce7b09030cc72df


Tree: Add ability to configure text autowrap mode for individual cells

#76532 fixed a regression caused by adding support for multiline cells (#61714), but introduced a non-configurable autowrapping, which might not be desirable. This PR restores the old behavior by default (no autowrapping, unless explicit newlines) and adds the ability to customize the mode.
Testing is very welcome, since i can't believe the solution can be so simple.
Closes #77067.

Looks good from my testing.
I encountered this:

Might be an old issue, though.

Thanks!


click on markdown preview links to local files without suffix/extension/extname get their editor tab closed

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.61.0 (git commit 8f68fcb5640c376671dbef79209f577e8d00b8d5)
OS Version: Arch Linux kernel 5.13.13

Steps to Reproduce:

create files blah.md, foo and foo.bar
add markdown links in blah.md to foo and foo.bar
open blah.md as markdown preview
click link to foo.bar, it works (a new editor tab for foo.bar file is opened)
click link to foo, it works, but the editor tab for foo file gets closed and vscode sais a file foo.md is not found

in case it's a problem with the link definitions, here's my blah.md:
- [foo](foo)
- [foo.bar](foo.bar)

i guess someone now relies on the feature that a markdown link to foo tries to open foo.md so i won't request removing that feature. (i do wonder what's the use-case though?)
i tracked down the problem to this function:
https://github.com/microsoft/vscode/blob/8f68fcb5640c376671dbef79209f577e8d00b8d5/extensions/markdown-language-features/src/commands/openDocumentLink.ts#L85-L93
i dont know how to fix that typescript code, but i changed the generated javascript code from:
const tryUpdateForActiveFile = async () => {
	 if (vscode.window.activeTextEditor && (0, file_1.isMarkdownFile)(vscode.window.activeTextEditor.document)) {
		  if (vscode.window.activeTextEditor.document.uri.fsPath === resource.fsPath) {
				await this.tryRevealLine(engine, vscode.window.activeTextEditor, args.fragment);
				return true;
		  }
	 }
	 return false;
};

to:
const tryUpdateForActiveFile = async () => {
	 if (vscode.window.activeTextEditor && (0, file_1.isMarkdownFile) && (vscode.window.activeTextEditor.document)) {
		  if (vscode.window.activeTextEditor.document.uri.fsPath === resource.fsPath) {
				await this.tryRevealLine(engine, vscode.window.activeTextEditor, args.fragment);
				return true;
		  }
	 }
	 return false;
};

added && between (0, file_1.isMarkdownFile)(vscode.window.activeTextEditor.document) just by trial-and-error...

and now clicking on markdown preview link foo wont close it's opened editor tab, and there is no error for missing foo.md.
i hope someone familiar with the typescript code can fix this.

fwiw, i don't know for what the isMarkdownFile function is used here (and i know even less why it's exported to js the way it is), but removing it seems to fix this issue AND and not break anything i could think of testing...
index 41fb441b935..f907babc504 100644
--- a/extensions/markdown-language-features/src/commands/openDocumentLink.ts
+++ b/extensions/markdown-language-features/src/commands/openDocumentLink.ts
@@ -83,7 +83,7 @@ export class OpenDocumentLinkCommand implements Command {

        private static async tryOpen(engine: MarkdownEngine, resource: vscode.Uri, args: OpenDocumentLinkArgs, column: vscode.ViewColumn): Promise<boolean> {
                const tryUpdateForActiveFile = async (): Promise<boolean> => {
-                       if (vscode.window.activeTextEditor && isMarkdownFile(vscode.window.activeTextEditor.document)) {
+                       if (vscode.window.activeTextEditor) {
                                if (vscode.window.activeTextEditor.document.uri.fsPath === resource.fsPath) {
                                        await this.tryRevealLine(engine, vscode.window.activeTextEditor, args.fragment);
                                        return true;


Force coverage to fail under 70%

All Submissions:

[x] Have you followed the guidelines in our Contributing document?

[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

[x] Does your PR follow our Code of Conduct?

[x] Have you added an explanation of what your changes do and why you'd like us to include them?

[x] Does each method or function "do one thing well"? Reviewers may recommend methods be split up for maintainability and testability.

[x] Is this code designed to be testable?

[x] Is the code documented well?

[x] Does your submission pass existing tests (or update existing tests with documentation regarding the change)?

[x] Have you added tests to cover your changes?

[x] Have you linted your code prior to submission?

[x] Have you updated the documentation and README?

[x] Is PII treated correctly? In particular, make sure the code is not logging objects or strings that might contain PII (e.g. request headers).

[x] Have secrets been stripped before committing?

so, it works - but there are modules under 70% --  at least 4 of them in ./agogosml/