(DO NOT MERGE)(MODULES-2243) Remove LCM RefreshMode Change

Removes setting the LCM RefreshMode in the Invoke DSC Resource script and
have it bail out if it is set to anything else than Disabled.
Creates a Puppet type that will set the LCM RefreshMode to one of the
supported values

In 9dd238f we probably want a .fixtures.yml as well to get those deps installed with rake spec_prep
fixtures:
  forge_modules:
    powershell: "puppetlabs/powershell"
    stdlib: "puppetlabs/stdlib"
  symlinks:
    dsc: "#{source_dir}"

Overall this is really close to merge as of e17e680.
I'm pretty satisfied with local tests with multiple resource manifests, like this one:
dsc_service { 'w32time':
  dsc_name  => 'w32time',
  dsc_state => 'Running'
}

dsc_registry { 'registry_test_binary':
  dsc_ensure    => 'Present',
  dsc_key       => 'HKEY_LOCAL_MACHINE\SOFTWARE\TestKey',
  dsc_valuename => 'TestBinaryValue',
  dsc_valuedata => 'BEEF',
  dsc_valuetype => 'Binary',
}

dsc_registry { 'registry_test_dword':
  dsc_ensure    => 'Present',
  dsc_key       => 'HKEY_LOCAL_MACHINE\SOFTWARE\TestKey',
  dsc_valuename => 'TestDwordValue',
  dsc_valuedata => '1',
  dsc_valuetype => 'Dword',
}

dsc_registry { 'registry_test_string':
  dsc_ensure    => 'Present',
  dsc_key       => 'HKEY_LOCAL_MACHINE\SOFTWARE\TestKey',
  dsc_valuename => 'TestStringValue',
  dsc_valuedata => 'Cats',
  dsc_valuetype => 'String',
}

dsc_service { 'w32time2':
  dsc_name  => 'w32time',
  dsc_state => 'Stopped'
}

Let's get #38 in first since we're going to want the rake task spec_prep working properly on Windows.  Then I believe you have a WIP test to update this PR with @jpogran, and some minor cleanup based on comments, and I think we can :shipit:


fix: emscripten build on windows

Since family detection is performed for cmd.exe, it recognizes msvc instead of clang and passes the wrong compilation options (-nologo).

cc @thomcc Could you review this please?


KomikCast

Request for source indonesian
Komikcast (Manga, Manhua & Manhwa)
#https://komikcast.io/

any news?

Actually i already built komikcast.io sources.
But the manga list title showing "Untitled", i still need to figure it out bcs i need to create parser or helper.
I'll send you, dm me at discord @kurotenshi

Check this link
I can't pr this one, bcs i'm overriding the template.rs
Please close this issue
Thanks

wdym you can't pr it bc you're overriding template.rs?

wdym you can't pr it bc you're overriding template.rs?

I'm overriding, parse_manga_listing
from
let title = manga_node.select(self.manga_title).attr("title").read();
bcs komikcast using class instead of attr, so i'm changing it. That's why i won't merged it
let title = manga_node.select("h3.title").text().read();

you can just make a template change that allows disabling the attribute selector?

It's from mangastream, but modified. I'll make a parser for it


Update cli args documentation

Updating the cli documentation after 0.5 release.

I was originally just going to fixup -f vs --force and --framework. But I ended up just updating all the args for tye commands since I noticed quite a few are out of sync with what we have.


Library code should be a virtual document

Hi VS Code dev here :wave:
Have the following code
num1 = 1.5
num2 = 6.3
# Add two numbers
sum = float(num1) + float(num2)

Go to references on float -> notice a new editor appears with editable content.
This feels like a very bad experience since this is library code and should be read only.
As @jrieken suggest Python should be a virtual document provider, as a virtual document fits this use case much better.
fyi @luabud

hey @isidorn! During housekeeping I transferred this to the Pylance repo and then talked about this with the team. We were thinking whether it wouldn't make more sense for Pylance to return the content with a flag saying it should be read only or not (and then VS Code would handle that appropriately)?
For context, it's not uncommon for users to want to edit their stub files (.pyi) that are open when they use "Go to references" (folks in our team actually do this all the time). The main exceptions are usually builtins.pyi  like in the float example, and probably anything under site-packages.

@luabud thanks for discussing with the team.
You understand the needs of the python users much better than me, and if you decided that editing stub files makes sense then we should allow that. As for builtins.pyi and site-packages if you can mark them as readonly that would make sense to me. And currently the mechanism to mark something as readonly should be through your document provider. You probably already have a ContentProvider, and that provider returns TextModels, and each of those TextModels can decide if it is readonly or not. At least, that is how it works in core when I implemented it for debug, and I think the same should be exposed to extensions.

We're not a TextDocumentContentProvider for python files. We just provide URIs that VS code opens using the default content provider. I don't think it would be worth the effort to start doing this as we've had no up votes on this issue.
I think in order for this to work we'd have to create a custom URI scheme for built in files. Then we'd be a content provider for just that scheme.

thanks all for the additional context! I'll close this as out of scope then 😊


Crashlytics does not show actual caller

"net.ypresto.utils.timbertree.CrashlyticsTree.e" is used for caller and all of e() calls are recorded on same issue.

fixed!


fix($urlMatcherFactory): unquote all dashes from array params

I noticed when using an array of uuids in a URL that the URL Matcher would only unquote the first dash it found. The fix was pretty straight forward and I have added tests.

Closes  #1717


[Azure App Configuration sdk] Running into 401 issue when there is a trailing '/' in store endpoint

Package Name:  @azure/app-configuration
Package Version:  1.3.1
Operating system:
[ ] nodejs

version:  16.9.0

[ ] browser

name/version:

[ ] typescript

version: 4.0.2

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
When using the App configuration client and passing a configuration store endpoint with a trailing '/' and a credential from, am running into a 401 error.
To Reproduce
Steps to reproduce the behavior:

Using the App configuration client, pass a store endpoint with a trailing '/' and a credential.

Expected behavior
The Azure app configuration SDK should handle any trailing '/' in the store endpoint.

@minhanh-phan can you take a look?

Hey @MaryanneNjeri,
We've tested the SDK with your case, and it is working fine.
Can you share more details on the error?

Hey @HarshaNalluru when using the SDK and passing the app configuration store endpoint with a trailing slash something like "https://store-name.azconfig.io/", run into this 401 error, with the response
{
  "name": "RestError",
  "statusCode": 401,
  "request": {
    "streamResponseStatusCodes": {},
    "url": REDACTED, // Censored
    "method": "GET",
    "headers": {
      "_headersMap": {
        "accept": "application/vnd.microsoft.appconfig.kvset+json, application/json, application/problem+json",
        "user-agent": "AzurePipelines.AzureAppConfiguration.Push/5.0.0 azsdk-js-app-configuration/1.3.1 core-http/2.2.4 Node/v16.19.0 OS/(x64-Windows_NT-10.0.22621)",
        "x-ms-client-request-id": "5a0e62c5-d2cb-4d51-8c86-b49a4f338526",
        "authorization": "REDACTED", // I’ve censored the Bearer token.
        "cookie": "REDACTED"
      }
    },
    "withCredentials": false,
    "timeout": 0,
    "requestId": "5a0e62c5-d2cb-4d51-8c86-b49a4f338526"
  },

The error object doesn't also give a lot of information on what the error could be.

Looks like the error you see is not much informative, would you run the test with "AZURE_LOG_LEVEL=verbose" environmental variable and provide us the logs, that might be more helpful in understanding the problem.

@HarshaNalluru also the response headers indicate that
www-authenticate': {
      name: 'www-authenticate',
      value:  Bearer error="invalid_token", error_description="Authorization token failed validation"'

the token is invalid

You said the problem is from the trailing "/" in the description.
Have you tried with an endpoint where there is no trailing "/" to confirm your theory?
The problem is probably from using a wrong credential. Can you verify?

@HarshaNalluru Yes without the trailing "/" in the endpoint authentication is successful

@MaryanneNjeri Though we can't repro the issue, I am inclined to make the fix for you. Will keep you posted.
In parallel, I want to understand the root cause better.
Since we can't repro the issue on our side, can you provide the location of your resource? maybe that has to do something with it.

@MaryanneNjeri, Thanks for reporting.
@azure/app-configuration/v/1.4.1 has bene published to the npm


kernel: event modification functions return previous value

Update the return value of functions that modify the internal event
state from void to uint32_t, so that calling code can determine
whether the event was already in a given state, or if the call modified
it.
This simplifies the usage of struct k_event as an alternative to
atomic_t that users can block on.
Implements https://github.com/zephyrproject-rtos/zephyr/issues/57216
Signed-off-by: Jordan Yates<EMAIL_ADDRESS>

I do note (more pedantry) that this locks us into a 32 event state word; supporting more would now be an incompatible API change. Might plausibly be worth typedef'ing a k_event_state_t or whatever so we can move to uint64_t in the future.

I believe this was already the case given the events parameter is already uint32_t instead of a typedef.


certificate error for repos.php.earth

Hi Team,
I am using below approach on Alpine linux to build my PHP container.
Approach:
apk add --no-cache wget ca-certificates
&& wget -O /etc/apk/keys/phpearth.rsa.pub https://repos.php.earth/alpine/phpearth.rsa.pub
&& echo "https://repos.php.earth/alpine/v3.8" >> /etc/apk/repositories
However, I started receiving below error since yesterday while building docker container. Can you please help me on this and what is this issue. I am not getting this error while accessing site in browser.
Error:
#9 4.239 fetch https://repos.php.earth/alpine/v3.8/x86_64/APKINDEX.tar.gz
#9 4.530<PHONE_NUMBER>93196:error:14007086:SSL routines:CONNECT_CR_CERT:certificate verify failed:ssl_clnt.c:1132:
#9 4.533 ERROR: https://repos.php.earth/alpine/v3.8: Permission denied
#9 4.533 WARNING: Ignoring https://repos.php.earth/alpine/v3.8: No such file or directory

Cross-referencing due to related: phpearth/alpine#32
Cross-referencing due to related: phpearth/PHP.earth#37


Error: unexpected end of JSON input

Bug report
Describe the bug
I am trying to create a new supabase instance using CLI on a ubuntu 20.04 (arm) and I am receiving:
Error: unexpected end of JSON input
To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:

install supabase cli 0.10.5
Create an empty directory supabase-local

ubuntu@ubuntu:~/supabase-local$ git init
Initialized empty Git repository in /home/ubuntu/supabase-local/.git/
ubuntu@ubuntu:~/supabase-local$ supabase init
Error: unexpected end of JSON input

Expected behavior
I guess it should create a supabase directory.
System information

OS: Linux Ubuntu 20.04 - arm
Version of supabase-cli 0.10.5

Hey @titogarrido, this is related to #41 - the containers won't run properly on aarch64 Linux. I'll close this issue in favor of that one.


byte overview documentation

Doesn't take much, but we should document the byte overview

resolved by 83917bc349a8d4efe986a7aef991d3386bc78dbf


code for user-defined mask

Hello!
I am trying to use your code for "Null-text Inversion for Editing Real Images using Guided Diffusion Models".
In particular, since I have an inpainting mask, I am trying to generate an image using a user-defined mask (like shown in fig. 8 or fig. 14 of "Prompt-To-Prompt Image Editing With Cross-Attention Control").
The code for using user-defined mask is missing, so I was trying to implement a way to do that.
Did you just apply the given mask instead of the one computed from the prompt in LocalBlend?
Could the following code represent what you did (resizing the mask to 64x64, repeating over the 2 channels, applying the mask to the latent space)?
class LocalBlend:
    ...
    def __init__(...)
        ...
        mask = np.array(Image.fromarray(mask).resize((64, 64), Image.NEAREST))
        mask = mask[None,None,:,:]
        mask = mask.repeat(2, axis=0)
        self.mask = torch.from_numpy(mask).cuda()

    def __call__(...)
        ...
        mask = self.mask
        mask = mask.float()
        x_t = x_t[:1] + mask * (x_t - x_t[:1])

The code I wrote works (example in the image), I am just wondering if it follows the way you intended to do it.
As you can see, using the given mask code above allows me to edit just the pie on the left, instead of all the pies:

@fabrizioguillaro what if the mask didn't match the position of the pie? like the mask is on the right.. would it still give reasonable results?


Optimize dpp sort module

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: yangxudong:x: brucedingYou have signed the CLA already but the status is still pending? Let us recheck it.


he following OpenGL versions/extensions are required but are not supported by your GPU driver:

OpenGL15

OpenGL20
GL_ARB_framebuffer_object
GL_ARB_texture_float
GL_ARB_half_float_pixel

Hi @endergamer123. This is not suitable for a bug report. I am sorry but the error is telling you that your video card isn't good enough to run Terasology.
It is possible we can make it easier to run Terasology one day, but not likely any time soon. In the meantime maybe you'll end up with access to a newer computer sometime and can play on there. Good luck!


Issues

Hi,
following the readme, I instantiated at top level here: val logger = KotlinLogging.logger {}
and then I use wherever I need it like: logger.warn{ "omg" }
But whenever I run some tests, I always get the following:

SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".
SLF4J: Defaulting to no-operation (NOP) logger implementation
SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details.

why?

I think you are missing the actual logging implementation (from
readme): Important
note: kotlin-logging depends on slf4j-api. In runtime, it is also required
to depend on a logging implementation. More details here
http://saltnlight5.blogspot.co.il/2013/08/how-to-configure-slf4j-with-different.html
.
On 10 Nov 2017 13:28, "Giuseppe Barbieri"<EMAIL_ADDRESS>wrote:

Hi,
following the readme, I instantiated at top level here
https://github.com/kotlin-graphics/assimp/blob/master/src/main/kotlin/assimp/assimp.kt#L7:
val logger = KotlinLogging.logger {}
and then I use wherever I need it like: logger.warn{ "omg" }
But whenever I run some tests, I always get the following:
SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".
SLF4J: Defaulting to no-operation (NOP) logger implementation
SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further
details.
why?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/MicroUtils/kotlin-logging/issues/28, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ACj0QZjrZh2cPp1snSKS0WG2Wxg6-s7Tks5s1DNhgaJpZM4QZbWJ
.

Uh, sorry I was blind..
Anyway, following that link, adding this compile group: 'org.slf4j', name: 'slf4j-simple', version: '1.7.25' to Gradle seems working.
However now I get a lot of noise for something external to my program..
https://gist.github.com/elect86/e3a74336984f1354f448b9eae72b1bde
Any idea how could I solve?

I think it is not possible with slf4j simple, but with other frameworks
such as log4j you can filter messages. Note that warn messages usually
indicates a problem that might be better to solve.
On 10 Nov 2017 14:49, "Giuseppe Barbieri"<EMAIL_ADDRESS>wrote:

Uh, sorry I was blind..
Anyway, following that link, adding this compile group: 'org.slf4j',
name: 'slf4j-simple', version: '1.7.25' to Gradle seems working.
However now I get a lot of noise for something external to my program..
https://gist.github.com/elect86/e3a74336984f1354f448b9eae72b1bde
Any idea how could I solve?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/MicroUtils/kotlin-logging/issues/28#issuecomment-343465024,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACj0QSOSznx5mVV4Ji1auHaUAG0sZI6Cks5s1EZJgaJpZM4QZbWJ
.

@elect86 slf4j-simple is just that - simple :) Try logback instead, it is pretty straight forward to configure and is the de-facto standard for slf4j impl: https://logback.qos.ch/manual/configuration.html

Hi Alex,
thanks for the feedback, I'll take a look :)

I will close the issue now. Youre welcome to reopen if there is still an issue.


Convert sensor values as keywords to numbers, colors

Is your feature request related to a problem? Please describe.
My battery sensor does not return number, but values as keywords like high, low, medium
Describe the solution you'd like

Allow colors on specific value (for example high as green, low as red), OR
convert keywords to values (for example high convert to 80 and use colors as on other sensors)

Additional context

The second one is possible already. Please look at the doc and state_map setting

The second one is possible already. Please look at the doc and state_map setting

Does it really work? I see no effect.

entity: sensor.motion_sensor_battery_state
name: Pohybový senzor 2
state_map:

from: High
to: 75
from: Medium
to: 50
from: Low
to: 25

Please reopen if it didn't solve your problem


There was an error! TypeError: this.$cookie.isCookieAvailable is not a function

Why the function does not work?
There was an error! TypeError: this.$cookie.isCookieAvailable is not a function
// check a cookie exist
this.$cookie.isCookieAvailable("user_session")

This should be fixed in (https://github.com/anish2690/vue-cookie-next/pull/5)


OrderBy fails with "Property Value not loaded nor in schema" when sorting on nestled object parameter

Both OrderBy and OrderByDescending fail to parse the correct member name when ordering on parameter's of a nestled object.
Ex: : "Property 'Value' not loaded nor in schema Failed on FT.SEARCH MyIndex * LIMIT 0 100 SORTBY ChildPropertyName DESC"
The reason seems to be that the correct syntax for FT.SEARCH should be like "SORTBY ChildObjectName_ChildPropertyName DESC", but this is lost in translation in the lambda expression, which can be seen in TranslateOrderByMethod where sb.Field does not include the object name.
Code to replicate issue:
var connection = new RedisConnectionProvider(ConfigurationOptions.Parse("localhost:30379"));

// connection.Connection.DropIndex(typeof(TypeA));

connection.Connection.CreateIndex(typeof(TypeA));

connection.RedisCollection<TypeA>().Insert(new TypeA(1, new TypeB(2, "Two")));
connection.RedisCollection<TypeA>().Insert(new TypeA(2, new TypeB(3, "Three")));

var res = await connection.RedisCollection<TypeA>().OrderByDescending(x => x.TypeB.Value).ToListAsync();

foreach (var typeA in res)
{
    Console.WriteLine($"{typeA.Id} - {typeA.TypeB.Id}");
}

Console.ReadLine();

[Document(StorageType = StorageType.Json, Prefixes = new[] { "typeA" }, IndexName = "idx:typeA")]
public record TypeA([property: RedisIdField, Indexed] int Id, [property: Indexed(JsonPath = "$.Value", Sortable = true)] TypeB TypeB);

public record TypeB([property: RedisIdField, Indexed] int Id, string Value);

This will fail with:
"Property 'Value' not loaded nor in schema Failed on FT.SEARCH idx:typeA * LIMIT 0 100 SORTBY Value DESC"
But redoing the query as "FT.SEARCH idx:typeA * LIMIT 0 100 SORTBY TypeB_Value DESC" succeeds.

Aye you put your finger on it @Dreadheart - it's not correctly constructing the property names correctly for sortby for some reason, will need to look into this, it looks like the same thing is happening for aggregations, probably missed a spot where we unwind the path.


feat(ci): nightly checks with connectivity recovery test

Resolves #1465

Successful run (triggered on PR instead of cron job): https://github.com/opentdf/platform/actions/runs/10584743221/job/29329689009


Enable components to observe actions

🚚  Enable components to observe actions https://github.com/spotify/HubFramework/issues/99

Current coverage is 92.85% (diff: 94.44%)

Merging #105 into master will increase coverage by 0.03%

@@             master       #105   diff @@
==========================================
  Files            58         58
  Lines          4038       4059    +21
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           3748       3769    +21
  Misses          290        290
  Partials          0          0

Powered by Codecov. Last update e9d6522...f4781d3

@JohnSundell 🎾


Temperature Written Incorrectly

Where it displays the temperature it displays it as # °C/F but temperature is supposed to be written as #°C/F basically there's just not supposed to be a space but there is

Fixed


Multiline TextBox caret is moved to wrong column when pressing return

Describe the bug
When you have a TextBox with AcceptsReturn="true", pressing return moves the caret down a line but doesn't move it to the first column until a character is entered.
To Reproduce

Go to ControlCatalog, TextBox page.
Focus one of the multi-line text boxes (second column)
Delete all existing text in the text box
Type some text and press return

Expected behavior
The caret is moved to column 0.
Screenshots
https://user-images.githubusercontent.com/1775141/121663084-538c0800-caa6-11eb-80fb-fac8fa339ab2.mp4
Desktop (please complete the following information):

OS: Windows
Version: master (checked back to 0.10.0)

cc: @Gillibald

Seems to be caused by the logic which inserts an additional line in the case of trailing newlines here: https://github.com/AvaloniaUI/Avalonia/commit/1a4fd00f2b77d0a648b00ca680227bb164c53ddd
When this blank line is present, it contains a duplicate of the \r\n in the previous line - these characters are reported as having a width of 13 each and used to place the caret.

Y these should have zero with

Any idea what the correct fix would be?

First of all why does SkPaint.MeasureText return 13.920455 for a \r character?
Given that it does this, should we just not call MeasureText for non-printable chars?
If so what's the best way to detect non-printable chars?

LineBreak characters don't need to be measured so we can just assume zero width here. We can't simply fix this in one line that's why my TextPresenter rework exists. I need to continue my work on that rework.

To detect non printable character you could test the UnicodeCategory.Control property of each character. That will cover most cases.

We can't simply fix this in one line that's why my TextPresenter rework exists. I need to continue my work on that rework.

Understood, just looking for a temporary fix for this issue until that work's finished.


More data-oriented memory layout for leds

Currently, we just have a vector of leds which each has a handful of properties. Splitting these up into a series of vectors (one for each property, with the indices all lining up) could have substantial benefits.
For example, set_range() could be refactored to something like this:
// index bounds checking...

self.led_colors[range].fill(color);

for some pretty substantial performance wins.
Similar gains could be seen in methods like set_all. Could also help in the battle against the borrow checker here and there.

Ran some experiments and found that slice.fill() is only about about 10% faster for common use-cases in this project, often less. Considering how much of a refactor this would take, and the unknown implications on existing methods (basically at get_muts()), I really don't think this is worth it.

One last thought to consider, what if we kept the same LED struct, but instead of that struct owning its own color data, it simply kept a mutable reference to it? Color data would be stored somewhere else in sequential order. Might create some borrow checker and lifetime goofiness, but worth looking into imo

While it might help some by reducing the byte size of each LED struct, thus making things a touch more cache friendly, I just don't think it's worth the effort right now. Perhaps worth revisiting if I find that I can't get things to run fast enough on super low end microcontroller hardware.

Did my research, self-referential structs in rust are a no-go. Could make it work with reference-counted types, but likely at the expense of the gains I'm trying to get here.


Integration tests fail if LANG is not ""

gitea version: 9296baf
to reproduce:
export LANG=de_DE.UTF-8
make test-sqlite

expected: they should pass
affected tests:

TestAPILFSBatch/Download/MetaNotFound (api_repo_lfs_test.go:186 - should be false)
TestAPILFSUpload/AlreadyExistsInStore (api_repo_lfs_test.go:360 - should be false)
TestMirrorPush (mirror_push_test.go:51 - hould have 1 item(s), but has 2; mirror_push_test.go:54 - should be true)
TestCreateNewTagProtected/API (repo_tag_test.go:33, unexpected error)

TestAPILFSUpload/AlreadyExistsInStore (api_repo_lfs_test.go:360 - should be false)

Interesting one, it seems like the pointer does exist, but this is because once the test it done, it doesn't clean this pointer(which is created in the same test). So on the first-run, this test passes, but on all following runs it will fail because this pointer is already created.
I guess this test should clean this pointer afterwards?
PS. this doesn't seem to relate to LANG, it all seems to be bugs whereby you aren't on the first-run(a reason why drone CI doesn't pick up on those errors).


Instructions to get ollama and merlinite working

A howto get ollama to host merlinite from huggingface

@kelbrown20 Here is yet more documentation and I do not want to add unnecessarily to the sprawl, though @jjasghar's how to guide is valuable for new users. Do you have feedback on where we might either a) add this detail to existing docs or b) where we should link to this so as to make sure folks have the benefit of this knowledge.

A howto get ollama to host merlinite from huggingface

You can also just ollama run sroecker/merlinite
I converted the model from HF and used the provided prompt template, which differs from the example here.

You can also just ollama run sroecker/merlinite
Neat! Yeah I figured someone would want to build it locally so they know exactly what they are getting. There's a chance you (or someone else with another fork) may have edited the Modelfile and made the SYSTEM look like this...

SYSTEM """
You are Mario from Super Mario Bros. Answer as Mario, the assistant, only.
"""

I didn't adapt the system prompt, that's something you can easily do
otherwise.
Getting the chat template right is important though:
https://ollama.com/sroecker/merlinite:latest/blobs/840577c5e55f
This should correspond to the Merlinite 7B model card:
prompt = f'<|system|>\n{sys_prompt}\n<|user|>\n{inputs}\n<|assistant|>\n'
On Tue, Apr 23, 2024 at 5:12 PM JJ Asghar @.***> wrote:

You can also just ollama run sroecker/merlinite
Neat! Yeah I figured someone would want to build it locally so they know
exactly what they are getting. There's a chance you (or someone else with
another fork) may have edited the Modelfile and made the SYSTEM look like
this...
SYSTEM """You are Mario from Super Mario Bros. Answer as Mario, the assistant, only."""
—
Reply to this email directly, view it on GitHub
https://github.com/instructlab/community/pull/198#issuecomment-2072642972,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACYR3IZAYCXCP7C44SGJOTY6Z255AVCNFSM6AAAAABGTUYUL6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANZSGY2DEOJXGI
.
You are receiving this because you commented.Message ID:
@.***>

Oh, I'm not saying you did. but these instructions are designed for you to pull directly from the upstream, and not rely on another name space, and know exactly what you're getting.

I wonder if it makes more sense to frame this as "running arbitrary models locally with InstructLab + ollama" instead of "running Merlinite"? With more explanation on e.g. how the Modelfile is developed?
Also I think to make it cohesive with the InstructLab project, a line at the very end explaining the --endpoint-url that allows you to hit the ollama model as part of your generate command would be helpful.

I had completely forgotten about this, and it's no longer accurate. Not to mention it's not Granite, which is what our defaults should be going forward. I'm going to close this.


Many weird characters in metadata and generated sql queries

Hello guys,
I'm opening an issue because I have on my production servers many issues and I don't know if they are related or not, I don't really know if the issue is coming to from the ORM or the DBAL, and I really can't figure things out, so maybe you guys will have any hint that may help me, or maybe you will be able to point me to another issue I did not find using the search engine that talk about the same kind of issue.
Below you will find many errors that are reported by New Relic with the stack trace (sorry, it will be a long post), and I will explain what I notice. I just can't figure the root cause out. There is one thing for you to know: there are no typo in our code. The issues only appear after some hours, not immediately to add a little more weird to the story.
I'm using Doctrine ORM 2.5.10 inside a Symfony 2.8.27 project, using doctrine bundle 1.6.8. We're running php 7.0.20, and using apcu as a metadata/query/result cache.
ReflectionException: Noticed exception 'ReflectionException' with message 'Class Mdm\Bundle[ProductBundle\Entity\Shop does not exist' in /data/mdm/deploy/mdm-catalog-service/releases/20171019093749/vendor/doctrine/common/lib/Doctrine/Common/Persistence/Mapping/RuntimeReflectionService.php:72

…ne/common/lib/Doctrine/Common/Persistence/Mapping/RuntimeReflectionService.php (72)
…71019093749/vendor/doctrine/orm/lib/Doctrine/ORM/Mapping/ClassMetadataInfo.php (924)
…19093749/vendor/doctrine/orm/lib/Doctrine/ORM/Mapping/ClassMetadataFactory.php (721)
…ommon/lib/Doctrine/Common/Persistence/Mapping/AbstractClassMetadataFactory.php (214)
…releases/20171019093749/vendor/doctrine/orm/lib/Doctrine/ORM/EntityManager.php (281)
…9/vendor/doctrine/orm/lib/Doctrine/ORM/Repository/DefaultRepositoryFactory.php (44)
…releases/20171019093749/vendor/doctrine/orm/lib/Doctrine/ORM/EntityManager.php (698)
…log-service/releases/20171019093749/app/cache/prod/appProdProjectContainer.php (3162)
…mdm/deploy/mdm-catalog-service/releases/20171019093749/app/bootstrap.php.cache (2248)
…log-service/releases/20171019093749/app/cache/prod/appProdProjectContainer.php (3072)
…mdm/deploy/mdm-catalog-service/releases/20171019093749/app/bootstrap.php.cache (2248)
…ymfony/src/Symfony/Component/EventDispatcher/ContainerAwareEventDispatcher.php (183)
…ymfony/src/Symfony/Component/EventDispatcher/ContainerAwareEventDispatcher.php (128)
…ndor/symfony/symfony/src/Symfony/Component/EventDispatcher/EventDispatcher.php (45)
…mdm/deploy/mdm-catalog-service/releases/20171019093749/app/bootstrap.php.cache (3243)
…mdm/deploy/mdm-catalog-service/releases/20171019093749/app/bootstrap.php.cache (3205)
…mdm/deploy/mdm-catalog-service/releases/20171019093749/app/bootstrap.php.cache (3359)
…mdm/deploy/mdm-catalog-service/releases/20171019093749/app/bootstrap.php.cache (2562)
…ed at /data/mdm/deploy/mdm-catalog-service/releases/20171019093749/web/app.php (15)

In this one we can see that the class name is wrong, it contains a [ instead of a \: Mdm\Bundle[ProductBundle\Entity\Shop vs Mdm\Bundle\ProductBundle\Entity\Shop
Doctrine\DBAL\Exception\DriverException: Noticed exception 'Doctrine\DBAL\Exception\DriverException' with message 'An exception occurred while executing 'SELECT f0_.id AS id_0, f0_.code AS code_1, f0_.is_publish AS is_publish_2, f1_.text AS text_3, f1_-lang AS lang_4, f1_.id AS id_5 FROM facet f0_ LEFT JOIN facet_translation f1_ ON f0_.id = f1_.id WHERE f1_.lang = ?' with params ["it"]:SQLSTATE[42883]: Undefined function: 7 ERROR: operator does not exist: facet_translation - characterLINE 1: ...s_publish AS is_publish_2, f1_.text AS text_3, f1_-lang AS l... ^HINT: No operator matches the given name and argument type(s). You might need to add explicit type casts.' in /data/mdm/deploy/mdm-catalog-service/releases/20171019093749/vendor/doctrine/dbal/lib/Doctrine/DBAL/Driver/AbstractPostgreSQLDriver.php:91

…leases/20171019093749/vendor/doctrine/dbal/lib/Doctrine/DBAL/DBALException.php (128)
…/releases/20171019093749/vendor/doctrine/dbal/lib/Doctrine/DBAL/in Doctrine\DBAL\DBALException::driverExceptionDuringQuery called at /data/mdm/deploy/mdm-catalog-service/releases/20171019093749/vendor/doctrine/dbal/lib/Doctrine/DBAL/Connection.php
Connection.php (855)
…/releases/20171019093749/vendor/doctrine/dbal/lib/Doctrine/DBAL/Connection.php (899)
…/releases/20171019093749/vendor/doctrine/dbal/lib/Doctrine/DBAL/Connection.php (830)
…93749/vendor/doctrine/orm/lib/Doctrine/ORM/Query/Exec/SingleSelectExecutor.php (50)
…service/releases/20171019093749/vendor/doctrine/orm/lib/Doctrine/ORM/Query.php (321)
…releases/20171019093749/vendor/doctrine/orm/lib/Doctrine/ORM/AbstractQuery.php (969)
…releases/20171019093749/vendor/doctrine/orm/lib/Doctrine/ORM/AbstractQuery.php (924)
…releases/20171019093749/vendor/doctrine/orm/lib/Doctrine/ORM/AbstractQuery.php (739)
…es/20171019093749/src/Mdm/Bundle/CategoryBundle/Repository/FacetRepository.php (23)
…/releases/20171019093749/src/Mdm/Bundle/SearchBundle/Services/QueryService.php (380)
…/releases/20171019093749/src/Mdm/Bundle/SearchBundle/Manager/SearchManager.php (861)
…/releases/20171019093749/src/Mdm/Bundle/SearchBundle/Manager/SearchManager.php (422)
…/releases/20171019093749/src/Mdm/Bundle/SearchBundle/Manager/SearchManager.php (308)
…s/20171019093749/src/Mdm/Bundle/ProductBundle/Controller/ProductController.php (739)
in Mdm\Bundle\ProductBundle\Controller\ProductController::searchProducts called at ? (?)
…mdm/deploy/mdm-catalog-service/releases/20171019093749/app/bootstrap.php.cache (3246)
…mdm/deploy/mdm-catalog-service/releases/20171019093749/app/bootstrap.php.cache (3205)
…mdm/deploy/mdm-catalog-service/releases/20171019093749/app/bootstrap.php.cache (3359)
…mdm/deploy/mdm-catalog-service/releases/20171019093749/app/bootstrap.php.cache (2562)
…ed at /data/mdm/deploy/mdm-catalog-service/releases/20171019093749/web/app.php (15)

Here we can see in the SQL query, in the SELECT clause, a reference to f1_-lang instead of f1_.lang
Doctrine\DBAL\Exception\SyntaxErrorException: Noticed exception 'Doctrine\DBAL\Exception\SyntaxErrorException' with message 'An exception occurred while executing 'SELECT f0_.id AS id_0, f0_.code AS code_1, f0_.is_publish AS is_publish_2, f1_.text AS text_3, f1_.lang AS lang_4,f1_.id AS id_5 FROM facet f0_ LEFT JOIN facet_translation f1_ ON f0_.id = f1_.id WHERE f1_.lang = ?' with params ["en"]:SQLSTATE[42601]: Syntax error: 7 ERROR: syntax error at or near ""LINE 1: ...publish_2, f1_.text AS text_3, f1_.lang AS lang_4,f1_.id AS... ^' in /data/mdm/deploy/mdm-catalog-service/releases/20171019093749/vendor/doctrine/dbal/lib/Doctrine/DBAL/Driver/AbstractPostgreSQLDriver.php:66

…leases/20171019093749/vendor/doctrine/dbal/lib/Doctrine/DBAL/DBALException.php (128)
…/releases/20171019093749/vendor/doctrine/dbal/lib/Doctrine/DBAL/Connection.php (855)
…93749/vendor/doctrine/orm/lib/Doctrine/ORM/Query/Exec/SingleSelectExecutor.php (50)
…service/releases/20171019093749/vendor/doctrine/orm/lib/Doctrine/ORM/Query.php (321)
…releases/20171019093749/vendor/doctrine/orm/lib/Doctrine/ORM/AbstractQuery.php (969)
…releases/20171019093749/vendor/doctrine/orm/lib/Doctrine/ORM/AbstractQuery.php (924)
…releases/20171019093749/vendor/doctrine/orm/lib/Doctrine/ORM/AbstractQuery.php (739)
…es/20171019093749/src/Mdm/Bundle/CategoryBundle/Repository/FacetRepository.php (23)
…/releases/20171019093749/src/Mdm/Bundle/SearchBundle/Services/QueryService.php (380)
…/releases/20171019093749/src/Mdm/Bundle/SearchBundle/Manager/SearchManager.php (861)
…/releases/20171019093749/src/Mdm/Bundle/SearchBundle/Manager/SearchManager.php (422)
…/releases/20171019093749/src/Mdm/Bundle/SearchBundle/Manager/SearchManager.php (308)
…s/20171019093749/src/Mdm/Bundle/ProductBundle/Controller/ProductController.php (739)
in Mdm\Bundle\ProductBundle\Controller\ProductController::searchProducts called at ? (?)
…mdm/deploy/mdm-catalog-service/releases/20171019093749/app/bootstrap.php.cache (3246)
…mdm/deploy/mdm-catalog-service/releases/20171019093749/app/bootstrap.php.cache (3205)
…mdm/deploy/mdm-catalog-service/releases/20171019093749/app/bootstrap.php.cache (3359)
…mdm/deploy/mdm-catalog-service/releases/20171019093749/app/bootstrap.php.cache (2562)
…ed at /data/mdm/deploy/mdm-catalog-service/releases/20171019093749/web/app.php (15)

Here, there is a small weird square just before f1_.id corresponding to a non printable character it seems.
Doctrine\Common\Persistence\Mapping\MappingException: Noticed exception 'Doctrine\Common\Persistence\Mapping\MappingException' with message 'Class 'AppBuncle\Entity\CategoryNewProduct' does not exist' in /data/mdm/deploy/mdm-catalog-service/releases/20171019093749/vendor/doctrine/common/lib/Doctrine/Common/Persistence/Mapping/MappingException.php:96

…ne/common/lib/Doctrine/Common/Persistence/Mapping/RuntimeReflectionService.php (41)
…ommon/lib/Doctrine/Common/Persistence/Mapping/AbstractClassMetadataFactory.php (281)
…ommon/lib/Doctrine/Common/Persistence/Mapping/AbstractClassMetadataFactory.php (311)
…19093749/vendor/doctrine/orm/lib/Doctrine/ORM/Mapping/ClassMetadataFactory.php (78)
…ommon/lib/Doctrine/Common/Persistence/Mapping/AbstractClassMetadataFactory.php (216)
…releases/20171019093749/vendor/doctrine/orm/lib/Doctrine/ORM/EntityManager.php (281)
…ce/releases/20171019093749/vendor/doctrine/orm/lib/Doctrine/ORM/UnitOfWork.php (2510)
…749/vendor/doctrine/orm/lib/Doctrine/ORM/Internal/Hydration/ObjectHydrator.php (273)
…749/vendor/doctrine/orm/lib/Doctrine/ORM/Internal/Hydration/ObjectHydrator.php (494)
…749/vendor/doctrine/orm/lib/Doctrine/ORM/Internal/Hydration/ObjectHydrator.php (165)
…9/vendor/doctrine/orm/lib/Doctrine/ORM/Internal/Hydration/AbstractHydrator.php (150)
…releases/20171019093749/vendor/doctrine/orm/lib/Doctrine/ORM/AbstractQuery.php (978)
…releases/20171019093749/vendor/doctrine/orm/lib/Doctrine/ORM/AbstractQuery.php (924)
…releases/20171019093749/vendor/doctrine/orm/lib/Doctrine/ORM/AbstractQuery.php (727)
…eases/20171019093749/src/AppBundle/Repository/CategoryNewProductRepository.php (25)
…749/src/Mdm/Bundle/ProductBundle/ProductDecorator/NewProductLabelDecorator.php (44)
…19093749/src/Mdm/Bundle/ProductBundle/ProductDecorator/AggregatedDecorator.php (38)
…s/20171019093749/src/Mdm/Bundle/ProductBundle/Controller/ProductController.php (334)
in Mdm\Bundle\ProductBundle\Controller\ProductController::getProductV2 called at ? (?)
…mdm/deploy/mdm-catalog-service/releases/20171019093749/app/bootstrap.php.cache (3246)
…mdm/deploy/mdm-catalog-service/releases/20171019093749/app/bootstrap.php.cache (3205)
…mdm/deploy/mdm-catalog-service/releases/20171019093749/app/bootstrap.php.cache (3359)
…mdm/deploy/mdm-catalog-service/releases/20171019093749/app/bootstrap.php.cache (2562)
…ed at /data/mdm/deploy/mdm-catalog-service/releases/20171019093749/web/app.php (15)

Looks like first example, but the stack trace is not the same. It is still an error in a class name: AppBuncle\Entity\CategoryNewProduct instead of AppBundle\Entity\CategoryNewProduct. This example is very widespread, and it is the same thing every time: a character is replaced by another one, and everytime it is the same pattern: it is replaced by the previous character in the ascii table. In this example, the letter d is replaced by c, I have examples of a ReflectionException concerning a property dmail that does not exist intead of email. If we look the first example, [ is actually preceding \ in the ascii table. I also have another example, still matching the pattern: Mdm\Bundle\C`tegoryBundle\Repository instead of Mdm\Bundle\CategoryBundle\Repository
Doctrine\DBAL\Exception\InvalidFieldNameException: Noticed exception 'Doctrine\DBAL\Exception\InvalidFieldNameException' with message 'An exception occurred while executing 'SELECT s0_.id AS id_0, s0_.shopid AS shopid_1, s0_.market @S market_2, s0_.googletrackingcode AS googletrackingcode_3 FROM shop s0_ WHERE s0_.shopid IN ('00000999', '00000190')':SQLSTATE[42703]: Undefined column: 7 ERROR: column "s" does not existLINE 1: ..._.id AS id_0, s0_.shopid AS shopid_1, s0_.market @S market_2... ^' in /data/mdm/deploy/mdm-catalog-service/releases/20171019093749/vendor/doctrine/dbal/lib/Doctrine/DBAL/Driver/AbstractPostgreSQLDriver.php:72

I don't even know what to think of this one. We can see in the select clause there is s0_.market @S market_2  instead of s0_.market AS market_2
Doctrine\DBAL\Exception\TableNotFoundException: Noticed exception 'Doctrine\DBAL\Exception\TableNotFoundException' with message 'An exception occurred while executing 'SELECT t0_.id AS id_0, t0_.shopid AS shopid_1, t0_.market AS market_2, t0_.googletrackingcode AS googletrackingcode_3 FROM t0_ WHERE t0_.shopid = ?' with params ["00000190"]:SQLSTATE[42P01]: Undefined table: 7 ERROR: relation "t0_" does not existLINE 1: ....googletrackingcode AS googletrackingcode_3 FROM t0_ WHERE ... ^' in /data/mdm/deploy/mdm-catalog-service/releases/20171019093749/vendor/doctrine/dbal/lib/Doctrine/DBAL/Driver/AbstractPostgreSQLDriver.php:75

…leases/20171019093749/vendor/doctrine/dbal/lib/Doctrine/DBAL/DBALException.php (128)
…/releases/20171019093749/vendor/doctrine/dbal/lib/Doctrine/DBAL/Connection.php (855)
…93749/vendor/doctrine/orm/lib/Doctrine/ORM/Query/Exec/SingleSelectExecutor.php (50)
…service/releases/20171019093749/vendor/doctrine/orm/lib/Doctrine/ORM/Query.php (321)
…releases/20171019093749/vendor/doctrine/orm/lib/Doctrine/ORM/AbstractQuery.php (969)
…releases/20171019093749/vendor/doctrine/orm/lib/Doctrine/ORM/AbstractQuery.php (924)
…releases/20171019093749/vendor/doctrine/orm/lib/Doctrine/ORM/AbstractQuery.php (766)
…ases/20171019093749/src/Mdm/Bundle/ProductBundle/Repository/ShopRepository.php (37)
…e/releases/20171019093749/src/Mdm/Bundle/ProductBundle/Manager/ShopManager.php (30)
…s/20171019093749/src/Mdm/Bundle/ProductBundle/Controller/ProductController.php (316)
in Mdm\Bundle\ProductBundle\Controller\ProductController::getProductV2 called at ? (?)
…mdm/deploy/mdm-catalog-service/releases/20171019093749/app/bootstrap.php.cache (3246)
…mdm/deploy/mdm-catalog-service/releases/20171019093749/app/bootstrap.php.cache (3205)
…mdm/deploy/mdm-catalog-service/releases/20171019093749/app/bootstrap.php.cache (3359)
…mdm/deploy/mdm-catalog-service/releases/20171019093749/app/bootstrap.php.cache (2562)
…ed at /data/mdm/deploy/mdm-catalog-service/releases/20171019093749/web/app.php (15)

This one is also very weird, the table name used in the FROM clause is the alias instead of the real table name.
This is it. Any help will be greatly appreciated, any hint or anything. Again, I don't know if this is a bug from doctrine or something else but maybe you guys will be able to clarify this. Unfortunately, since the issues come over time, I cannot give you a git repository with some code to reproduce this.
Best regards,

using apcu as a metadata/query/result cache.

Possibly one of the issues here.
…log-service/releases/20171019093749/app/cache/prod/appProdProjectContainer.php (3072)

This bit looks interesting: is it a simple service lookup? What is being passed to EntityManager#getRepository()?

Hello @Ocramius ,
Thank you for taking some time to review all of this.
In the appProdProjectContainer.php at line 3072 there is this method:
protected function getMdm_Requestlistener_SrvService()
{
    return $this->services['mdm.requestlistener.srv'] = new \Mdm\Service\MdmRequestListener($this->get('doctrine.orm.legacy_entity_manager'), $this->get('mdm.shop.repository'));
}

And the $this->get('mdm.shop.repository') results to a call to this other method in the appProdProjectContainer:
protected function getMdm_Shop_RepositoryService()
{
    return $this->services['mdm.shop.repository'] = $this->get('doctrine.orm.symfony_entity_manager')->getRepository('Mdm\\Bundle\\ProductBundle\\Entity\\Shop');
}

About APCu, for now we are trying to reduce its usage by removing the result cache from it, it seems the servers are becoming more stable. I opened an issue because I wanted to find the root cause to make sure we won't run into it again in the future.

There is no error in the container code provided, entity class' name is ok as well.
In third error log, it would be interesting what the square sign is, but I don't think it would significantly help (unless it's \x00, APC used to have problem with nulls).
In last error log the table name is completely missing.
This doesn't look like something ORM itself would cause, I would definitely blame APCu or other middleware. As far as I know APCu doesn't use any locking mechanism and doesn't have any sort of transactions, couldn't this be caused by high load & number of writes? But anyway, I'd suggest to switch to different storage, i.e. Redis.

Yes, it seems APCu is the one to blame here. We switched the result cache to redis, it's working fine now. Metadata and Query cache are still using APCu.
I don't really know what happens to be honest. There is a lot to read about APCu cache fragmentation, I don't know if it takes part in this issue or not but for now we have removed all usage of TTL with APCu and the issue does no happen anymore. It seems like add/removal of cache entries "corrupts" other entries eventually.
Thank you guys for taking time reading my long original post and giving me a few hints and places to look for.


CNV-28829: Preview feature mechanism for IT - part 1

📝 Description
This PR exposes a new hook called usePreviewFeatures to enable opt-in and out for tech preview feature (currently only create VM from InstanceTypes flow)
🎥 Demo
No visible changes

/retest

/lgtm

/cherry-pick release-4.13


Contract fuckery

I haven't tested extensively but have noticed that:

Contracts don't get removed on failure

Contracts don't get removed on success

During the loop of 1 and 2 you can sometimes buy another contract but it will be the same moon, same contract. (you still lose money buying it.)

Not sure on this one due to the previous errors but all my contracts were on vow. Only tested in 2 saves so take it with a grain of salt
Uploading LogOutput.log…

The hyperlink points to the issue itself

LogOutput.log

I noticed that both times my contract got seemingly stuck (didn't ever go away on completion) it happened to be on Experimentation. Might just be a coincidence though. It'd be nice to have a command that abandons the current contract so that we can take new ones if one gets stuck like that.


The term 'dream' is not recognized as the name of a cmdlet, function, script file, or operable program.

When I run the command I get an error. Using Powershell on Windows 10.
dream : The term 'dream' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the
spelling of the name, or if a path was included, verify that the path is correct and try again.
At line:1 char:1

dream "a pyramid made of ice"

  + CategoryInfo          : ObjectNotFound: (dream:String) [], CommandNotFoundException
  + FullyQualifiedErrorId : CommandNotFoundException

How did you install this on Windows, using pip? virtualenv? maybe you need to specify the full path of the file. I'm using virtualenv on Windows, and running something like C:\some\path\venv\Scripts\python.exe C:\some\path\modified-cli.py

How did you install this on Windows, using pip? virtualenv? maybe you need to specify the full path of the file. I'm using virtualenv on Windows, and running something like C:\some\path\venv\Scripts\python.exe C:\some\path\modified-cli.py

I ran it in the folder and it solved the problem

I ran it in the folder and it solved the problem


catch fake data up with #3730

We removed a number of payday stats fields in #3730, but were still trying to populate those in fake data. This PR also makes db cleaning a little more robust so we can recover from failure during make fake.
Fixes #3780.

@fenryxo Mind reviewing this? Should fix #3780.
cc: @rohitpaulk @rorep et al.

Mind reviewing this? Should fix #3780.

... or @yifeiwu? :-)

I got a new error:
$ make fake
env/bin/honcho run -e defaults.env,local.env env/bin/fake_data fake_data
Traceback (most recent call last):
  File "env/bin/fake_data", line 9, in <module>
    load_entry_point('gratipay==1913', 'console_scripts', 'fake_data')()
TypeError: main() takes at least 1 argument (0 given)
make: *** [fake] Error 1

@fenryxo Try now?

Fixed :+1:

@fenryxo Thanks! Merged. :-)


Add !theme random [Dark|Light]

When doing a random theme, add the ability to specify random within a subset (light/dark) of the theme list.

Per @parithon, we can investigate the manifest for each theme to determine which subset the theme should be included in

Great idea @mholloway24!  I can't believe I had talked myself out of it.  Thankfully @parithon came to the rescue.  We ended up adding this on stream today and it's out the door already.


[Bug] No Sanctum token cookie is saved when using token mode in Safari

Hi, thanks for nice module. I am struggling with the new token mode.
Describe the bug
When I try to log in to app with token mode in current Safari, it won't save Sanctum token cookie and /api/user request throws error 401 Unauthenticated. In Chrome the token cookie is saved and user can login and call /api/user route without any problems.
To Reproduce
Steps to reproduce the behavior:
Try to login in Safari. Mode: token is set in nuxt.config.ts.
Expected behavior
In chrome token is saved correctly and everything works right.
Screenshots
Chrome cookie is saved correctly:

In Safari nothing appear in cookies:

Module information

Version: 0.4.1
Complete configuration of sanctum from your nuxt.config.ts:

export default defineNuxtConfig({
    devtools: {enabled: true},
    css: ['~/assets/scss/main.scss'],
    ssr: false,

    postcss: {
        plugins: {
            tailwindcss: {},
            autoprefixer: {},
        },
    },

    modules: ["nuxt-auth-sanctum"],

    sanctum: {
        baseUrl: 'http://localhost:8000',
        redirect: {
            onLogin: '/', // Custom route after successful login
            onAuthOnly: '/login',
            onGuestOnly: '/',
            onLogout: '/login',
        },
        endpoints: {
            login: '/api/login',
            logout: '/api/logout',
            user: '/api/user',
        },
        globalMiddleware: {
            enabled: true,
        },
        mode: 'token'
    },
})

Nuxt environment:

Version: 3.12.2
SSR Enabled: no
Environment: local

Laravel environment:

Version: 11
Sanctum installed via Breeze: yes
Checklist completed: (yes / no)
Session domain from your config/session.php: [e.g. domain.test]
List of stateful domains from your config/sanctum.php

return [
    'stateful' => explode(
        ',',
        env(
            'SANCTUM_STATEFUL_DOMAINS',
            sprintf('%s','localhost,localhost:3000,<IP_ADDRESS>,<IP_ADDRESS>:8000,::1')
        )
    ),
];

or
SANCTUM_STATEFUL_DOMAINS=localhost,localhost:3000

CORS settings from your config/cors.php

return [
    'paths' => ['*'],
    'allowed_methods' => ['*'],
    'allowed_origins' => [
        env('FRONTEND_URL', 'http://localhost:3000'),
    ],
    'allowed_origins_patterns' => [],
    'allowed_headers' => ['*'],
    'exposed_headers' => [],
    'max_age' => 0,
    'supports_credentials' => true,
];

Edit: It is working on production, so maybe it's related to my local environment.

Hey @michaeldojcar, I tried to reproduce it in Safari, but no luck 😞 I would suggest checking Safari security-related settings or so because I believe it has nothing to do with the module, especially since you mentioned that in Chrome and Prod it's alright.

Closing due to inactivity


Use gcc from cygwin to build on windows

This installs gcc and make from Cygwin to build on Windows. I did not add a step to run the autograder baseline, but we could try whether using python from Cygwin works at some point.

@mstarzinger Thanks a lot!
@ChristianMoesl What do you think? Can you get make grade to work on Windows using @mstarzinger's code?


entt::registry::empty() is deprecated, but (...) has no member named in_use()

Hi,
I have the following code:
if(registry.empty()) {
    consoleLog.push_back({0, "No entities in scene."});
    return;
}

My compiler tells me:
warning: 'empty' is deprecated: use .storage<Entity>().in_use() instead

So I change the code to this:
if(!registry.storage().in_use()) {
    consoleLog.push_back({0, "No entities in scene."});
    return;
}

Now, my compiler tells me this:
error: no member named 'in_use' in 'entt::iterable_adaptor<entt::internal::registry_storage_iterator<entt::internal::dense_map_iterator<__gnu_cxx::__normal_iterator<entt::internal::dense_map_node<unsigned int, std::shared_ptr<entt::basic_sparse_set<>>> *, std::vector<entt::internal::dense_map_node<unsigned int, std::shared_ptr<entt::basic_sparse_set<>>>>>>>>'

What's going on? I'm using single_include/entt/entt.hpp with version v3.12.2.

Just to add, I'm completely new to EnTT, and I'm not trying to use the issues page as a support forum. I may be making a mistake that would be obvious to people familiar with the library.
I still feel like being warned about deprecation, suggested to use a different function, and then finding out that function doesn't actually exist, is a bit odd. Could this be an issue with the single-include header?

I still feel like being warned about deprecation, suggested to use a different function, and then finding out that function doesn't actually exist, is a bit odd

I mean, the warning says storage<Entity>().in_use() and you did storage().in_use().
I find being notified of a change with a description of what to do, doing a different thing and then being surprised why it doesn't work is just as weird.
Probably that function exists on the right storage though:
if(registry.storage<entt::entity>().in_use() == 0u) {
    // no entities currently in use here
}

Thanks for pointing this out to me!
I failed to read the documentation last night, and I misunderstood the use of the function.
I do have an additional question:
Why does it want me to specify type Entity, when there's just one entity type (essentially a uint32)? Can registries store more than just entities? Or is there some non-purely-ECS thing you can do like, for example, creating an entity class that contains the uint32 EnTT needs, but also storing a name string for in-game referencing?

Well, yeah, the entity type is customizable as everything else in EnTT. You can have 64b entities or use a custom class if you like.
That being said, storage() has a different purpose and storage<T>() wants a T. Since you're asking for the entity storage... 🤷♂


Ministry Restriction Note- Note is not getting reset

once ministry resolves a restriction by adding a note,
1st restriction

and then if they tried to resolve another restriction, the previous note is displayed in the new modal
2nd restriction

@HRAGANBC @CarlyCotton  Can you please confirm this bug. Ann created it so I have no history. Is it still a bug. Do you believe this a priority for the PT release?

@michesmith @CarlyCotton Yes, bug. But not critical - it does not prevent Ministry user from erasing the pre-populated text and inputting new ones, so they can live with this one for now.
I've marked it as enhancement, with release TBD

@michesmith @CarlyCotton Yes, bug. But not critical - it does not prevent Ministry user from erasing the pre-populated text and inputting new ones, so they can live with this one for now.
I've marked it as enhancement, with release TBD


Converting problem: header X-MS-TNEF-Correlator is empty, so there is an exception

I don't know if this is an issue of the project or the email server provider, but there is a problem on converting MimeMessage to Email.
I try to use https://ethereal.email to test emails. I use this code to fetch and get the emails via IMAP:
public static void main(String[] args) {
	Properties props = new Properties();
	props.setProperty("mail.store.protocol", "imaps");
	props.setProperty("mail.debug", "true");

	Folder inbox = null;
	Store store = null;
	try {
		// Connect to the server
		System.out.println("Connect to the server");
		Session session = Session.getInstance(props);

		String host = "imap.ethereal.email";
		String username =<EMAIL_ADDRESS>		String password = "4qvcgpNsPhwnQDNgzE";
		String provider = "imaps";

		store = session.getStore(provider);
		store.connect(host, username, password);

		// open the inbox folder
		System.out.println("open the inbox folder");
		inbox = store.getFolder("INBOX");
		UIDFolder uf = (UIDFolder) inbox;
		inbox.open(Folder.READ_ONLY);

		SearchTerm term = new SearchTerm() {

			private static final long serialVersionUID = 1L;

			public boolean match(Message message) {
		    		Calendar c = Calendar.getInstance();
		    		c.add(Calendar.DAY_OF_MONTH, -1);
		    		Date when = c.getTime();
	                        try {
					if (message.getReceivedDate().after(when)) {
					    return true;
					}
				} catch (MessagingException e) {
					// TODO Auto-generated catch block
					e.printStackTrace();
				}
		        return false;
		    }
		};

		Message[] searchMessages = inbox.search(term);

		for (Message msg : searchMessages) {
			long uid = uf.getUID(msg);
			System.out.println("uid: " + uid);
		        MimeMessage mimeMsg = (MimeMessage) msg;
			Email email = EmailConverter.mimeMessageToEmail(mimeMsg);
			String html = email.getTextHTML();
			String text = html.substring(0, html.indexOf(DELIMITER));
			System.out.println(text);
		}
	}
	catch (Exception ex) {
		System.out.println("Thread has a problem...");
		ex.printStackTrace();
	}

	finally {
		// close the inbox folder but do not
		// remove the messages from the server
		try {
			inbox.close(false);
		} catch (MessagingException e) {
			e.printStackTrace();
		}
		try {
			store.close();
		} catch (MessagingException e) {
			e.printStackTrace();
		}
		System.out.println("Thread is closing...");
	}
}

I got this exception:
java.lang.IllegalArgumentException: value is required
	at org.simplejavamail.internal.util.Preconditions.checkNonEmptyArgument(Preconditions.java:15)
	at org.simplejavamail.email.Email.addHeader(Email.java:519)
	at org.simplejavamail.converter.EmailConverter.fillEmailFromMimeMessage(EmailConverter.java:245)
	at org.simplejavamail.converter.EmailConverter.mimeMessageToEmail(EmailConverter.java:56)

I debugged the code and I got that the key of the header is: X-MS-TNEF-Correlator
The value is an empty string.
At this point, only because of a missing header value, I cannot get the email message converted. Isn't it some weird? I can get the other parts of the email by using all the other methods. But here, I think simple-mail should let me have the rest of the email converted even if there are some missing header parts (which I don't even know if we need or not).

You're right, it should be able to convert the email without issue. I'll look into it.

Released as 5.0.0.rc1-SNAPSHOT. Add OSS' snapshots repo to find it in Maven.


Fixes typo in Readme screenshot URL

Fixes typo in Readme screenshot URL.

:+1: That was my bad when I was resolving the merge conflict in #243


Consider undeprecating attributes_for

Please undeprecate attributes_for? This syntax is extremely useful to me, and your suggested hack (Factory.build(:user).attributes) does not work for me. I have a controller spec roughly like this:
MyClass.should_not_receive(:new)
post :create, my_class: Factory.build(:my_class,params).attributes
# fails because you're forcing me to call MyClass.new to setup the test

Besides this problem, attributes returns a bunch of stuff not needed in a scenario like that (such as created_at, updated_at, etc).
I love FactoryGirl so please don't take this as a criticism. I just really rely on attributes_for and I would probably have to create a private fork if you remove this essential feature.

If you're using stubs, why do you need actual attributes? Couldn't you just
post with a hash like my_class: { "attribute" => "value" }?

One issue with deprecating FactoryGirl.attributes_for is when using it to create a new user, with password.
That is, FactoryGirl.create(:user).attributes[:password] is, in most cases, some hashed version of the password, not suitable for submitting to a User.new/User.create. You can work around this in several ways - including merging in a known attribute when you care, or by not using a factory in those cases (I've done the later).

I too agree to not deprecated this method, see my comment here: https://github.com/thoughtbot/factory_girl/commit/a883315bc4775f5b9996a378d6c4ebe0ebf44a5e#commitcomment-878888

@jferris yes, I could do a hash like that but then I'm repeating information that I'm hoping to keep DRY in the factory definitions. That particular code involves some decision making based on the contents of the hash so I can't just pass a random hash. This was also just an isolated example, I use attributes_for in several other scenarios across various projects. @stevenharman gave another good one.

+1  I got hit by the user issue too.  attributes_for is too valuable.

Also in favor of not deprecating this method. I use attributes_for to obtain random valid attributes which are passed as part of a larger hash to a request class which delegates those attributes to the related class. The workaround seems messier and leaves an unwanted model as a side-effect.

Please restore this method as I have the user issue as well. There are valid reasons that make attributes_for useful.

+1 for UNdeprecate this useful method. As it was explained in previous posts, build().attributes does NOT functionally replace attributes_for().

+1 here too. An important difference between the two methods is that attributes_for skips over any ActiveRecord validations and associations. It can be useful to use attributes_for to get a hash of attributes on a model without needing validations or FactoryGirl's associations.

Another +1! My argument is exactly the same as @itspriddle's.

+1
I'm curious, what was the reason to deprecate it in the first place?

+1
i have troubles with that, because new syntax (FactoryGirl.build(:basic_user).attributes) and old way (Factory.attributes_for :basic_user) returns different results:

Factory.attributes_for :basic_user:
{:first_name=>"John", :last_name=>"Doe"<EMAIL_ADDRESS>:birthdate=>Mon, 20 Jan 1997, :password=>"password", :password_confirmation=>"password", :password_current=>"password", :admin=>false, :email_confirmed=>true, :license_agreement=>true, :time_zone=>"Bratislava", :facebook_uri=>"http://www.facebook.com/...", :twitter_username=>"...", :website_uri=>"http://....com/", :biography=>"All your base\n\nare belong to us.", :current_login_ip=>"<IP_ADDRESS>", :account_type=>"regular"}

FactoryGirl.build(:basic_user).attributes:
{"id"=>nil, "email"=>nil, "birthdate"=>nil, "activation_code"=>nil, "created_at"=>nil, "updated_at"=>nil, "single_access_token"=>nil, "login_count"=>0, "failed_login_count"=>0, "last_request_at"=>nil, "current_login_at"=>nil, "last_login_at"=>nil, "current_login_ip"=>"<IP_ADDRESS>", "last_login_ip"=>nil, "rss_token"=>nil, "license_agreement"=>true, "under13ok"=>false, "can_review_academic"=>true, "sponsor_id"=>nil, "cached_slug"=>nil, "active"=>true, "type_of_view"=>0, "is_dashboard_visited"=>false, "auto_smarthinking"=>false, "gear_notification"=>false, "onetime_sso_token"=>nil, "onetime_sso_token_created_at"=>nil, "super_admin"=>false, "user_views_count"=>0, "invited_by_id"=>nil, "inviter_bonus_given"=>nil, "fb_profile_id"=>nil, "use_fb_photo"=>nil, "birthdate_from_fb"=>false, "facebook_uri"=>"http://www.facebook.com/...", "twitter_username"=>"...", "website_uri"=>"http://...", "censored"=>false}

FactoryGirl.create(:basic_user).attributes:
{"id"=>1, "email"=>nil, "birthdate"=>nil, "activation_code"=>nil, "created_at"=>Fri, 20 Jan 2012 15:47:25 CST -06:00, "updated_at"=>Fri, 20 Jan 2012 15:47:25 CST -06:00, "single_access_token"=>"MoqP0HX8DVRzF26BZXRB", "login_count"=>0, "failed_login_count"=>0, "last_request_at"=>nil, "current_login_at"=>nil, "last_login_at"=>nil, "current_login_ip"=>"<IP_ADDRESS>", "last_login_ip"=>nil, "rss_token"=>nil, "license_agreement"=>true, "under13ok"=>false, "can_review_academic"=>true, "sponsor_id"=>nil, "cached_slug"=>nil, "active"=>true, "type_of_view"=>0, "is_dashboard_visited"=>false, "auto_smarthinking"=>false, "gear_notification"=>false, "onetime_sso_token"=>nil, "onetime_sso_token_created_at"=>nil, "super_admin"=>false, "user_views_count"=>0, "invited_by_id"=>nil, "inviter_bonus_given"=>nil, "fb_profile_id"=>nil, "use_fb_photo"=>nil, "birthdate_from_fb"=>false, "facebook_uri"=>"http://www.facebook.com/...", "twitter_username"=>"...", "website_uri"=>"http://.....com/", "censored"=>false}

@saks - I was just about to report that. Thanks!

and we didn't have to get Google and Wikipedia to go dark.  :)

Thanks guys!

awesome thank you @thoughtbot!!


Most plugins don't show results on Linux

Most plugins don't seem to return any results on Linux and instead, you can only search the web. For example, entering "gif testing" as displayed on the gif plugin page, doesn't display any gif, but only this:

The same occurs with:

the file system navigation plugin
Emoj
IMDB
Caniuse
IP
Kill

What works (besides opening a web search in the default browser):

searching and launching apps
calculator
smart converter
(updated) displaying a map inside the app works (I didn't know it could do this since it's not listed in the app description)

Tested in Ubuntu 16.10 64bit with the latest Cerebro 0.2.5.

@hotice can you turn on developer mode from preferences, open dev. tools from tray menu → development → dev. tools (main), reload application by tray menu → development → reload and send me the output from console tab of dev. tools?

Sure, here's the log (logged when attempting to search for a gif):
The provided value 'moz-chunked-arraybuffer' is not a valid enum value of type XMLHttpRequestResponseType.
 When it is installed you should see this:

Can you check what do you see when you type plugins gif?

Well I suck, I thought they are installed by default and I was not familiar with how to check that. So that's it. Sorry for wasting your time, closing this.

@hotice you're not alone so it is my fault that a lot of people can't find them ;)


Facebook SDK integration

Hi ,
would like to ask a tutorial on how to integrate facebook SDK and other native library to call from the tabris js part ?
I want to integrate Facebook SDK event and other SDK such as Google SDK and Firebase SDK but there's no tutorial found on the internet.
also i want to know how to call java /objective c++ method from the JS code ..
thanks
Regards,

also i want to know how to call java /objective c++ method from the JS code ..

You will need to create your own plugin; the docs can be found here

I want to integrate Facebook SDK event and other SDK such as Google SDK and Firebase SDK but there's no tutorial found on the internet.

thanks for the response , is there any way that i can add a jar /aar references ?

There aren't plugins for this available yet. I suggest you use the SDKs available on npm and use it like you would in any other non-browser environment.

also i want to know how to call java /objective c++ method from the JS code ..

You will need to create your own plugin; the docs can be found here

I tried the tabris-plugin-fab sample but i end up with an error below ,
4:05:20 PM - Starting compilation in watch mode...
src/App.tsx(9,21): error TS6138: Property 'main' is declared but its value is never read.
src/App.tsx(15,7): error TS2304: Cannot find name 'custom'.
src/App.tsx(26,42): error TS2339: Property 'show' does not exist on type 'Widget'.
src/App.tsx(32,42): error TS2339: Property 'hide' does not exist on type 'Widget'.
4:05:21 PM - Found 4 errors. Watching for file changes.
i followed all the instruction from the blog ,  i already add in the config.xml but i can't make it work

Also i noticed that AndroidManifest , and other resources /res/values/strings.xml is not updating even if it was declared in plugin.xml , the only thing that works is  element for build.gradle
please see below

<name>Tabris Plugin Floating Action Button</name>

  <config-file target="AndroidManifest.xml" parent="/manifest/application">
    <meta-data
        android:name="com.eclipsesource.tabris.android.HANDLER.com.tabris.js.android.fab"
        android:value="com.tabris.js.android.fab.FloatingActionButtonHandler" />

  </config-file>

  <config-file target="res/values/strings.xml" parent="/resources">
    <string name='test'>test </stirng>
  </config-file>

  <source-file src="src/android/com/tabris/js/android/fab/FloatingActionButtonHandler.kt"
               target-dir="src/android/com/tabris/js/android/fab" />
  <source-file src="src/android/com/tabris/js/android/fab/ImageProperty.kt"
               target-dir="src/android/com/tabris/js/android/fab" />
  <framework src="/Users/cebx/tabris/testapp/tabris-plugin-fab/src/android/build.gradle" custom="true" type="gradleReference"/>

</platform>

whenever i run tabris build android . all resources are not updating other than build.gradle
Thanks

I want to integrate Facebook SDK event and other SDK such as Google SDK and Firebase SDK but there's no tutorial found on the internet.

thanks for the response , is there any way that i can add a jar /aar references ?

There aren't plugins for this available yet. I suggest you use the SDKs available on npm and use it like you would in any other non-browser environment.

also i want to know how to call java /objective c++ method from the JS code ..

You will need to create your own plugin; the docs can be found here

I tried the tabris-plugin-fab sample but i end up with an error below , 4:05:20 PM - Starting compilation in watch mode... src/App.tsx(9,21): error TS6138: Property 'main' is declared but its value is never read. src/App.tsx(15,7): error TS2304: Cannot find name 'custom'. src/App.tsx(26,42): error TS2339: Property 'show' does not exist on type 'Widget'. src/App.tsx(32,42): error TS2339: Property 'hide' does not exist on type 'Widget'. 4:05:21 PM - Found 4 errors. Watching for file changes.
i followed all the instruction from the blog , i already add in the config.xml but i can't make it work

Okay , i solved the problem above, but there's an annoying display when i run the app on the emulator / android

how can i remove the URL input bar on the top.

That's the developer tools UI which can be hidden with a single property change.  Note this is always hidden in production builds.

That's the developer tools UI which can be hidden with a single property change. Note this is always hidden in production builds.

hi, thank s for the reply , what property should i neet to change ?

showUi().  You can see it in action with the JSX snippet in the docs

Closing this as resolved. Please comment if you have any more issues.


Allow for running services in containers as well

Fuge chooses to start all services as processes instead of containers. It would be great to have an option to start services in their containers as well, while of course maintaining as many of the fuge shell features as possible (like watching for changes and log grepping). This would help in dealing with services that rely for example on different versions of node, or maybe even include services that have been written in other languages as well.

we use the docker-compose format, so you can just use docker-compose up to run the services.

Yes, but I can't use fuge shell in that situation, can I?

so actually this does work, and it will pick up services in your docker-compose file that are being run from an image and not a path. At the moment it isn't smart enough to pull the images for those containers itself though (see: #48)

I'm not talking about things like redis or elasticsearch, but about the microservices I'm actually developing on top of that. Basically I'm just looking for a way to run docker-compose up, and then be able to manage the resulting ecosystem, like fuge shell does (starting/stopping individual microservices, rebuilding on file changes, etc). I don't really want anything to be running as a local process if it can be run in a docker container instead. Is such a setup possible with fuge?

@bsander Did you find an answer?
I am having issue trying to get docker-compose up to work with fuge. I assume that I should be able to launch and connect each of the components automatically using compose.
Thanks.

Unfortunately I didn't, I'd still be interested though :)

@bsander I see what you are saying. Generally, no, fuge won't every get the ability to auto generate and run your containers like that. We tried this with a previous version of fuge and with was hacky at best.
For the short term Fuge won't be able to generate and run containers but we are looking at adding plugin ability so that it's something you can write yourself.
I'm going to go ahead and close as answered but feel free to reopen if you have anymore questions.


0.23.0/no-editing-on-profile-page

Purpose
Force a user to go to user settings page in order to update a profile
Changes
Users who change their name on their user profile page do not see that there is an additional step of updating the name parts, and this has caused confusion on multiple occasions such as when Lyndsy was demoing the citation modifications and didn't realize that the name needed changing in multiple ways. This eliminates the ability for a user to make profile modifications on the profile page, and instead takes them to the settings page for such changes.  I had to make some changes to the knockout which wasn't really expecting to have a situation where you would have a view-only version of the page even if the user had the capability to edit their own info. It was almost there, but there were some assumptions based on what we had currently implemented that were not correct.
For a logged in user, the current profile page now looks like this:

Clicking the edit this profile link will take them to:

If you aren't logged in, the profile page for a user will look like:

Side effects
I think this is relatively self-contained. As long as permissions are maintained properly, a user shouldn't be able to edit on the profile page, changes made on the user settings page should show properly in the profile page, and users who aren't the current user should still be incapable of editing another user's info. We don't have profile picture on the setting page, so I maintain the dialog box directing users to gravatar from the profile page if the user is currently logged in.

Nice. How about "Edit your profile"?

Sure


Test for DDC-2988

This test can be merged only after fix issue DDC-2988
http://www.doctrine-project.org/jira/browse/DDC-2988

This looks something related to https://github.com/doctrine/doctrine2/pull/1113

@aivus could you check the failure test, so the PR could be in good shape and the maintainers could think in merge it? Thanks

@shakaran what do you mean?
This PR provides test only which demonstrates bug.

@aivus I mean that travis is failing for this PR

Yes, and this is correct. Because it shows error in code
On 16 Oct 2017 00:16, "Ángel Guzmán Maeso"<EMAIL_ADDRESS>wrote:

@aivus https://github.com/aivus I mean that travis is failing for this
PR
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/doctrine/doctrine2/pull/1307#issuecomment-336742302,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA-XB_DCppxJSDjYuLbjqNq5t1Dq1038ks5ssnZGgaJpZM4DjUWe
.

@aivus sorry, you are right, the travis is showing fine as intended ;) Just trying to push and easy the job to maintainers, so the bug with joinColumns is fixed soon and don't take another 3 years ;)

@aivus @shakaran the thing is that the destination branch of this PR should be master and not 2.4, we'll do the necessary backporting (on 2.5 though).
So would be amazing if you could change that on GH and rebase your PR to sync with the latest state of our repository.

Rebased to master and target branch changed.
In some reasons build is failing with:
1) Doctrine\Tests\ORM\Functional\Ticket\DDC2988Test::testManyToManyFindBy
Exception: [Doctrine\DBAL\Exception\DriverException] An exception occurred while executing 'SELECT t0.id AS id_1 FROM DDC2988User t0 WHERE users_to_groups.group_id IN (?)' with params [1]:
SQLSTATE[HY000]: General error: 1 no such column: users_to_groups.group_id


Resolve CI Pipeline Failures

This pull request integrates essential security keys into the project's configuration and ensures all tests are up to date and passing. These updates are crucial for enhancing the security and stability of the application.
Closes #45

The only thing to do was to add proper secrets to the GitHub repo. After that tests were not noticed to fail at any point

create corresponding task to the changes. This is not the right task for it.


Add ignore for a few of the types

This includes 8 new functions:

Result.ignore
AsyncResult.ignore
ResultOption.ignore
AsyncResultOption.ignore
JobResult.ignore
JobResultOption.ignore
TaskResult.ignore
TaskResultOption.ignore

While I was adding these I also fixed up a few incorrect function parameter names as I saw them (especially in the JobResult and TaskResult modules).
I wasn't sure how to build the gitbook, so would someone be able to clone my branch and make sure it looks good before merging?

The build error is confusing, the tests are passing on my machine so I'm guessing it's something to do with how Fable encodes unit option types? I've never used Fable so not sure how to resolve this issue.

Hmm interesting that Fable sometimes defines unit as undefined and sometimes as null.  I just asked about the Fable build errors in FSSF slack chat.

Seems it will be fixed next fable release, did we want to wait for that or should I just rewrite it to not use Option.map?

I'm.ok with waiting since fable usually releases bug fixes relatively quickly.

So it looks like it's already out https://github.com/fable-compiler/Fable/releases/tag/2.4.6
Try bumping the fable-compiler version to it in the package.json

Thanks, @CameronAavik for the pull request and @TheAngryByrd  & @cmeeren for jumping in :) Will publish the new NuGet version in few hours.

It's up now! https://www.nuget.org/packages/FsToolkit.ErrorHandling/1.2.4


WIP: Testing CI

recheck

recheck

recheck

recheck

@lennybe There's permission issue to view Details

Same issue here to review the details: http://<IP_ADDRESS>:8080/job/SRIOV-ConnectX-5/69/
While waiting for the details to be available, I'm also wondering what's the test case be executed. shall we consider adding the CI tests in this repo?

Same issue here to review the details: http://<IP_ADDRESS>:8080/job/SRIOV-ConnectX-5/69/
While waiting for the details to be available, I'm also wondering what's the test case be executed. shall we consider adding the CI tests in this repo?

Yeah, thanks,
I am working on it, looks like internal firewall issues. Not related to Intel.
I will update you when it will be fixed.

recheck

recheck

recheck

here you go, the latest recheck result can be reviewed now!

@zshi-redhat @ahalim-intel
Permission issue is fixed. You should be able to see 'details' now.

@zshi-redhat @ahalim-intel
Permission issue is fixed. You should be able to see 'details' now.

So is there any configuration you did in this github repo in order to enable the CI tests? Is it worth documenting the process?

@zshi-redhat @ahalim-intel
Permission issue is fixed. You should be able to see 'details' now.

So is there any configuration you did in this github repo in order to enable the CI tests? Is it worth documenting the process?

Yes,
@ahalim-intel added 'mellanox-ci' as a collaborator to the repo.

@zshi-redhat @ahalim-intel
Permission issue is fixed. You should be able to see 'details' now.

So is there any configuration you did in this github repo in order to enable the CI tests? Is it worth documenting the process?

Yes,
@ahalim-intel added 'mellanox-ci' as a collaborator to the repo.

Thanks @lennybe!  what's your plan of adding more test cases in CI?

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

retest

/retest


Removing then adding prompt quickly in inline form causes message to not be visible

If I have form with inline:true prompts then:
$('#atk_form').form('remove prompt', 'name', 'mesage');
$('#atk_form').form('add prompt', 'name', 'message');

causes prompt to be removed and then adds it again. Unfortunately due to animation adding prompt back does not make it appear.
It would be nice if previous animation would be cancelled before doing prompt fade-in.

Hi, please provide a test case as per our contributing guidelines; you can fork this JSFiddle.

Closing the issue until there’s a test case.


Since this seems to be dead - alternatives?

I guess since no commits happened for 2-3 years and no issues are answered anymore, this is dead. What alternatives are there / are you using? Kind of sad since this was an awesome package back in the day...

Opened up a discussion at https://forums.meteor.com/t/best-practise-account-ui-for-meteor-react-react-router/49575


Feature/server client hint dpr

Please review this as well. No conflicts. :)

@anton-karlovskiy I may be wrong, but I feel like I'm missing where the implementation for this example is. Could you check I'm misunderstanding this PR? :)

First commit appears to be a customized version of CRA https://github.com/GoogleChromeLabs/adaptive-loading/pull/46/commits/b1005bec0432af130d882027651185121126bd18 (but other than <img src="/dpr-aware-image" className="responsive" alt="square box" /> not seeing the implementation). Was this the only part we needed?
Second commit is removing files https://github.com/GoogleChromeLabs/adaptive-loading/pull/46/commits/e1114ecbd7eaf469a7d51580712addb68a9ef290
Third is package.json updates https://github.com/GoogleChromeLabs/adaptive-loading/pull/46/commits/5fd4f046f7dcbcbf19137db8910e66446c919e72
Fourth is removing functions https://github.com/GoogleChromeLabs/adaptive-loading/pull/46/commits/30211c0014a811dd09a7d763ca25ff650b897903

@addyosmani Actually the main implementation is node server.js file as image sources in index.html are fed with response from server.js.

Thanks for clearing that up!

After reviewing the initial implementation, this LGTM. We're likely going to do deeper reviews once we begin to build out more complex integrations of the adaptive loading signals and how we balance usage on the server with client hints vs. client-side signals.


Fix internal documentation links

We have links in the workspace that link to other places in the workspace that are currently broken. For example
"The chip group requires the chip component." on https://pf4.patternfly.org/documentation/core/components/chipgroup
"A good example of this approach is the table component." on https://pf4.patternfly.org/contribution

We fixed this a while ago as a part of the gatsby-theme-patternfly-org refactor.


Removing splicing QTL effect sizes to build "variantFunctionalConsequenceFromQtlId" evidence column.

Background
The current interpretation of splicing QTL (sQTL) beta coefficients for the direction of effect assessment is not informative, where negative are interpreted as loss of function and positive ones as gain of function.
For now we propose to remove them from the assessment for the direction of effect (DoE), so we will need to filter them out when building the evidences.
This issue is also addressing the point 1 of the issue raised by @Daniel-Considine here.
Tasks

[ ] Filter out sQTL signals before getting the one with the biggest effect (get_biggest_effect function) in the generation of the variantFunctionalConsequenceFromQtlId evidence column: https://github.com/opentargets/evidence_datasource_parsers/blob/ea3008a6143fd1347a3629d5869fd50438004abc/modules/GeneticsPortal.py#L84
[ ] Run the implementation of DoE in the backend with the new data to check that the direction of effect data is properly shown.
*(Please note that I'm not familiar with the whole process of the evidence generation, so feel free to modify my suggested tasks)

Acceptance tests
When direction of effect data shows no directionality from sQTL evidences.
How do we know the task is complete?

Using an user case raised in genetics slack channel:
On the associations page for Lymphocyte count associated with TYK2 (link), for the variant 19_10351837_C_T, the DoE is loss of function | Risk, where the loss of function comes from the Decreased gene product level derived from the highest effect size of sQTLs. After removing sQTL, this should appear as a gain of function | Risk.

Looking at the ETL code, the value for variantEffect is based on variantFunctionalConsequenceFromQtlId, a value generated during the OTG evidence generation.
@remo87 @Juanmaria-rr Can you confirm this is true? If so, it's up to us to filter out sQTLs in the logic and we'd have it fixed by next release.

Yes @ireneisdoomed, I think that would be the step to include to remove the sQTL effect sizes from the DoE assessment.

In #3282, @addramir and @Daniel-Considine suggested changing the logic of the aggregation. Instead of using the largest effect size, which could be problematic for poorly powered studies, use the lowest p-value. I would try to introduce this change in the same script as well.
@Juanmaria-rr will have some estimates on the impact of this change.
@buniello This will imply changing a tooltip and documentation.


Allow for data value that is not JSON

I have a webhook I need to call that does not accept JSON, only form data. Is it possible to provide a data value that is not JSON?

Hi. I a currently away on holiday. Will have a look at this when I'm back in September.

Hi. Just getting back to resolving some requests and issues on the project now. I am going to see if I can make form data work as part of the efforts.

I have released a new version today (v1.0.4) that has some basic support for urlencoded form data. It may or may not be exactly what you need. I will close the issue, but welcome any further suggestions for improvement if required. I've done some basic testing, but admittedly it's not been wide test coverage. I created a simple endpoint on my side that would accept the data and checked that I could access it, but in more extensive usage some minor issues may still pop - especially regarding special characters that are potentially not handled correctly in the POSTed data.

I have made an update to the release that now correctly encodes all POSTed data


GDScript: Warn when enum variable has no default

The default will always be set to 0, so if it's not a valid value in the enum, the warning is shown.
Solve part of #74123

Thanks!


Proof of Concept for Ephemeral Backstage Demo Environments

As discussed with @benjdlambert,
Proposal- A "Demo Button" can be placed in any strategic location (README, Website, Emails, etc) to allow a potential Backstage User to rapidly create their own Sandbox'ed Demo Environment which has a preset life-cycle of X hours.
The Demo in this PR highlights the "pulse" survey plugin as an example. You can see this in a preview environment here: https://deployment-16817-backstage-gnrn.app.uffizzi.com/
If you find value in this, we can work with you to iterate towards a customized "choose your plug-in" solution and include lead generation tools for potential adopters.

Accept the PR.
Go to Uffizzi UI and Configure Demo Button (see screenshot below.)
Place Demo Button in README, website, etc.
Potential Backstage Users can 1-click to get their own sandbox.

Example functional demo button:


Added JSON output format support

Hi Paul,
I've got some changes for Sitrep:

Support for basic command line flags
A new JSON report format is implemented (--json)
Altered the text output to use the Report object values
Removed the specific view count inheritance properties from the result
(Report generator has a better solution now, plus I think they were not working properly)
Fixed some unit tests (two count numbers where always failing for me)
Updated README.md

Please check my PR & have a nice weekend! 😉
Cheers,
Tib

This is a great set of changes – thank you! In the future, could you split up changes into individual PRs so they make for easier review? It helps my brain if I'm only looking at one task at a time 🙂
I'm not going to merge these changes yet, because I'm keen to retain the inheritance checks if possible. I'm not sure why they would be failing, because they are pretty simple things. Could you perhaps give me an example where they fail so we could work to fix it?

Sure, no problem, I can submit smaller PR's later on. I think this one contains more smaller changes, but they really depend on each-other. 🙃
Consider the following examples:
import SwiftUI

class View {
}

class A: View {
}

class B: SwiftUI.View {
}

/*
// original report:
SwiftUI Views: 0

// new report:
Inheritance:
   ---
   SwiftUI.View: 1
   View: 1
*/

or a more simple use case:
import SwiftUI

class C: NSObject, View {
}

// original report:
// SwiftUI Views: 0

Relying on the .first member of the inheritance chain is not a good way to count protocol inheritances, it can work with classes tho, but I'd rather check everything in the array.
/// How many structs conform to View
var swiftUIViewCount: Int {
    structs.sum { $0.inheritance.first == "View" }
}

That's why I logged everything from the inheritances. I know it's far from perfect, because of the namespace prefixes (eg. SwiftUI.View), but still it's a little bit more reliable solution than the one before. What do you think? 😅

By the sounds of it, the best solution here is not to remove the current view / view controller inheritances, but just to modify what we have. For UIView and UIViewController we're safe, because they both need to come first in the list. For View – although it will rarely be SwiftUI.View (who writes that?), and it will almost always be the first in the list – there's no reason why we couldn't write some code to make it more accurate.

Ok, fair enough. Is it ok if I revert back these 3 properties & log only the stats based on those in the report? So the core functionality won't change, but we can still have the JSON output.

Yes, that would be ideal.

Fantastic – thank you!


#N/A Add a new rule brew_update_formula

Coverage increased (+0.04%) to 90.926% when pulling 9d9820676a0fa01b61b0b8639ad387962a810873 on scorphus:brew-update-formula into 5ec4909d2f9b6fa30a8bc03f2ee16af73e234f0f on nvbn:master.

Thanks!


Add Troubleshooting topic to Test Network for Docker Desktop setting

Signed-off-by: pama-ibm<EMAIL_ADDRESS>New Docker Desktop setting in macOS causes chaincode install to fail.
Added a new troubleshooting topic to tell users how to resolve it.
Type of change

Documentation update

Description
Identified the type of error that can occur and how to resolve it by unchecking  Use gRPC FUSE for file sharing in Docker Desktop preferences

@Mergifyio backport release-2.2

@Mergifyio backport release-2.1

@Mergifyio backport release-2.0


WIP Add the MAINTANERS.md.

The motivation
Community users should using this doc to quickly find the maintainer for their problem in a specific area of ocm.
The 2 options we have, and Pros and Cons
After going throught the graduated projects of cncf,  we have found 2 pattern of MAINTAINER list.
Type1, json or yaml format, IaaS friendly, configuration style file
For examples:

istio
cilium

Usually, the project's infrustructure team will setup a bot or ci tool to sync these teams configuration to github teams, then each repo under the org can leverage github teams to create the codeowners file.
Using cilium for an example:

it has a repo team-manager to sync the configuration files to their github org teams.
it using a codeowners file instead of owners file in their main repo.

The Pros are:

github teams plus codeowners can bring more fine-grained permission management.
the sync from "file changes" to actual projects is smooth, no second operation are needed.

The Cons are:

it's complex to setup.
it needs extra effort in tooling.

Type2, markdown format, human read friendly, document style file
For examples:

kubevela
prometheus

Obviously, it's much easier to setup but it's not suitable when we have a lot of sub projects or work groups or teams.
The path
We will first add a type 2 MAINTAINER.md, in this step, we will discuss and figure out what teams and work-group we need to create.
Next, when we have enough members and maintainers we will invest the more automatic way -- to create a type 1 MAINTAINER list, we may even can recurit from the community.

Some questions:

Is ocm-vscode-extension in our maintain scope?
Should we addon the “Bootstrap Contributor” and “Emeritus Members” in this list? Llke [kubevela](https://github.com/kubevela/community/blob/dc3edce4c78ddb989c2cb9a9e120e75e047f3aa0/OWNERS.md?plain=1#L56C4-L56C20)
Related issue: What’s the process and result of seperating policy?

/assign @qiujian16
/assign @mikeshng

Could we have the format of:
name, company, [list of repos]
to eliminate duplicated contents?

Could we have the format of:
name, company, [list of repos]
to eliminate duplicated contents?

@mikeshng That way we may end with duplicated list of repos.
And I'm not sure whether we want to keep the [overall list] and [maintainers in different areas] in a same page.
For example, in prometheus, it's sperated:

https://prometheus.io/governance/#team-members
https://github.com/prometheus/prometheus/blob/main/MAINTAINERS.md

After we add all maintainers in the list, we will let toc to take a reivew

Should we add the community repo? It's one of our pinned repos.

Should we add the community repo? It's one of our pinned repos.

Good point, a new section of "Community and Docs" added.

tamalsaha is added as the PR is going to merge.

/approve


Refactoring install commands into install phase

Having install commands in other phases violates the semantics of the .travis.yml configuration. So we have refactored them into the install phase.

Note: This pull request was generated by an automated tool developed by The Software REBELs (a.k.a., the Software Repository Excavation and Build Engineering Labs) of McGill University, Canada. It is part of a research project by Keheliya Gallaba under the supervision of Dr.Shane McIntosh. If you have any questions or feedback about this tool, please feel free to contact the author (keheliya.gallaba@mail.mcgill.ca).

Codecov Report

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

@@           Coverage Diff           @@
##           master      #90   +/-   ##
=======================================
  Coverage   79.61%   79.61%
=======================================
  Files         114      114
  Lines        7472     7472
  Branches     1099     1099
=======================================
  Hits         5949     5949
  Misses       1523     1523

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 9de0fec...ef12ef7. Read the comment docs.


Implements multi stage support for commit interaction plot

resolves se-passau/VaRA#430

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- varats/plots/commit_interactions.py  3

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- varats/plots/commit_interactions.py  3

See the complete overview on Codacy


[gstreamer] update to 1.24.7

Fixes https://github.com/microsoft/vcpkg/issues/40923

[X] Changes comply with the maintainer guide.
[X] SHA512s are updated for each updated download.
[ ] The "supports" clause reflects platforms that may be fixed by this new version.
[ ] Any fixed CI baseline entries are removed from that file.
[X] Any patches that are no longer applied are deleted from the port's directory.
[X] When updating the upstream version, the "port-version" is reset (removed from vcpkg.json).
[X] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[X] Only one version is added to each modified port's versions file.

All features passed with following triplet:
x64-windows

Thanks for the update!


clear xintthresh on changes to lower priv modes

For issue #295,  a new proposal to have hardware automatically clear *intthresh when going to a lower mode via SRET and MRET. This is similar to the mstatus.mprv change in priv 1.12. This avoids the issue and also removes a potential source of bugs.

I suggest that the equivalent change is made in the debug spec for when dprv lowers the privilege level. Note, the change to mprv has already been mirrored: https://github.com/riscv/riscv-debug-spec/pull/503.