Support multimodal (GPT-4V) directly in OpenAIClient

This PR is part of the #1975
As the [Major Update 1] in "Multimodal Orchestration", it enables the OpenAIClient to load images and to communicate with OpenAI directly.
Hence, all ConversableAgent can use GPT-4V in the llm_config. The multimodal client will read the images before sending to OpenAI.
PR Summary:

We check the model name and llm_config in client.py, if it is a vision model, we will convert messages into multimodal format.
We move "img_utils.py" from the contrib folder to the main autogen folder to resolve circular import error. Workflow for testing is also updated.
Similarly, we moved the HTML parse utility function into the new "autogen/multimodal_utils.py" file to resolve the circular import error. We will also move the audio features into the same file in the future audio PR.
Test cases path updated accordingly.

Why are these changes needed?

Related issue number

Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[X] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[X] I've made sure all auto checks have passed.

Codecov Report
Attention: Patch coverage is 93.33333% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 19.79%. Comparing base (08ba070) to head (574b845).

Files
Patch %
Lines

autogen/oai/client.py
83.33%
2 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #2026       +/-   ##
===========================================
- Coverage   37.53%   19.79%   -17.75%
===========================================
  Files          65       65
  Lines        6913     6931       +18
  Branches     1521     1549       +28
===========================================
- Hits         2595     1372     -1223
- Misses       4092     5393     +1301
+ Partials      226      166       -60

Flag
Coverage Δ

unittests
19.78% <93.33%> (-17.76%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

So as you know I'm a big fan of this PR -- but am super nervous about introducing a prompting DSL that diverges from other prompting template DSL (e.g., Guidance), overlaps with HTML, cannot be disabled, and has no escaping conventions.
Anyhow I like everything else in this PR, and am willing to accept when you think it's ready.

So as you know I'm a big fan of this PR -- but am super nervous about introducing a prompting DSL that diverges from other prompting template DSLs (e.g., Guidance), overlaps with HTML, cannot be disabled, and has no escaping conventions.
Anyhow I like everything else in this PR, and the DSL stuff appears already to have been merged... So I am willing to accept when you think it's ready.

Yes, this PR is irrelevant to the HTML-style parser. I will create another PR to disable the HTML parsing by default after this PR. I just modified the ConversableAgent to accept OpenAI style multimodal input, see this example (also added to the notebook).
# Here is another way to define images, without using the HTML tag and using the OpenAI format directly.
image_agent = ConversableAgent(
    name="image-explainer",
    max_consecutive_auto_reply=10,
    llm_config={"config_list": config_list_4v, "temperature": 0.5, "max_tokens": 300, "cache_seed": 42},
)

user_proxy = autogen.UserProxyAgent(
    name="User_proxy",
    system_message="A human admin.",
    human_input_mode="NEVER",  # Try between ALWAYS or NEVER
    max_consecutive_auto_reply=0,
    code_execution_config={
        "use_docker": False
    },  # Please set use_docker=True if docker is available to run the generated code. Using docker is safer than running the generated code directly.
)

user_proxy.initiate_chat(
    image_agent,
    message=[
        {"type": "text", "text": "What's the breed of this dog?"},
        {"type": "image_url", "image_url":
         {"url": "https://th.bing.com/th/id/R.422068ce8af4e15b0634fe2540adea7a?rik=y4OcXBE%2fqutDOw&pid=ImgRaw&r=0"}}]
)

I was still waiting others to formalize the "Message" in autogen, but there are many blockers for that. Maybe I should just go ahead and create a simple ImageMessage class.
Guidance and Gemini are also using special characters for images, such as "<|_image:xxxxx|>", but it would have less chance of collision with HTML tags.

So as you know I'm a big fan of this PR -- but am super nervous about introducing a prompting DSL that diverges from other prompting template DSLs (e.g., Guidance), overlaps with HTML, cannot be disabled, and has no escaping conventions.

Anyhow I like everything else in this PR, and the DSL stuff appears already to have been merged... So I am willing to accept when you think it's ready.

Yes, this PR is irrelevant to the HTML-style parser. I will create another PR to disable the HTML parsing by default after this PR. I just modified the ConversableAgent to accept OpenAI style multimodal input, see this example (also added to the notebook).

# Here is another way to define images, without using the HTML tag and using the OpenAI format directly.

image_agent = ConversableAgent(

    name="image-explainer",

    max_consecutive_auto_reply=10,

    llm_config={"config_list": config_list_4v, "temperature": 0.5, "max_tokens": 300, "cache_seed": 42},

)

user_proxy = autogen.UserProxyAgent(

    name="User_proxy",

    system_message="A human admin.",

    human_input_mode="NEVER",  # Try between ALWAYS or NEVER

    max_consecutive_auto_reply=0,

    code_execution_config={

        "use_docker": False

    },  # Please set use_docker=True if docker is available to run the generated code. Using docker is safer than running the generated code directly.

)

user_proxy.initiate_chat(

    image_agent,

    message=[

        {"type": "text", "text": "What's the breed of this dog?"},

        {"type": "image_url", "image_url":

         {"url": "https://th.bing.com/th/id/R.422068ce8af4e15b0634fe2540adea7a?rik=y4OcXBE%2fqutDOw&pid=ImgRaw&r=0"}}]

)

I was still waiting others to formalize the "Message" in autogen, but there are many blockers for that. Maybe I should just go ahead and create a simple ImageMessage class.
Guidance and Gemini are also using special characters for images, such as "<|_image:xxxxx|>", but it would have less chance of collision with HTML tags.

Yes I like that tag format much better... it doesn't collide with html. Out of curiosity, do you know if those tokens are treated special in their tokenization?

@afourney  Yes, usually images are treated separately in the model. There is often an "image encoder model" (separate of the LLM) to process the image, and this "encoder" can be thought as a "tokenizer". Usually, the process will convert the image URI string back to an image, cut the image into $k$ patches, and then encode the $k$ patches into $k$ tokens, and then put them back with the text inputs (concatenation of the tokens).


Increase wait time between polling for traces

Increase wait time between polling for traces and wait the full amount of timeout time when waiting for a trace that shouldn't exists.
This will help reduce RPC calls and avoid hitting API quotas

Removed the numbers in favor of TimeSpans.


README.md edit

Added a more 'non-technical' description of what Moment.js does.
Hopefully it'll help some of the newer developers out there get a better understanding of what Moment.js can do for them in quick paragraph.

Thanks for contributing.  However:

Please rebase to collapse this to a single commit.
Please be consistent on style, "Moment.js", not "Moment JS".
The second bullet sounds like natural language parsing, which is not a feature we support or intend to.
The phrasing in the fourth bullet is off.  Please be consistent.
The last bullet refers to the same feature set as the first bullet

Additionally, the momentjs.com documentation page could use some of this kind of wording, in several places.  We have a separate repo for that here: https://github.com/moment/momentjs.com
Specifically, we have a lot of text about how to use it, but not much in terms of general description or why it is needed.  A getting started guide would also be helpful.  All of these things belong in the docs repo.
Thanks.

Thanks for the feedback! I've made the suggested changes and I'll take a look at some of the other docs you suggested and see if I can help out there.
As you can tell, I'm just getting my feet wet, as they say.

Well, we really do need more help around all areas, not just the code.  Docs, guides, issues, project management, etc.  So, very happy to receive items like this. :)

Looks like during rebase it got triplicated. :)

OOOPS!  =/ Should be good now.

Readme looks ok, but looks like your PR is pulling in a previous commit.  Fix if you can, or LMK and I'll just merge the one item manually.

Hmm. Not sure how to fix. Sorry to be a pain in the ...!

It looks like it should be good to go! Take a look and let me know if that is what you need it too look like.

Looks ok to me.  But seems lacking somehow.  @maggiepint any thoughts on expansion?  I like the intent.

I wrote it as a brief overview so I didn't want to make it too wordy. I'd be happy to expand on it if you have some direction, however.

Well, fundamentally when describing moment you have to answer two questions:
What does it do?
Why wouldn't I just use the built-in date object?
I would say that this PR addresses most of what moment does. It does not address the extensive internationalization support that Moment has, which is IMO one of its strongest features.
It also doesn't say anything about the improvements it makes to the date object. I don't think it has to enumerate everything it fixes (that would be a long list), but it could maybe phrase things in a way that causes the uninformed reader to feel the need to google "problems with javascript date".
It would behoove us to make a guide on the website about stuff in the date object that moment fixes that we could potentially link to later.
@mj1856 any of this seem like what we're missing?

It does mention the languages, just not as a bullet point.
I agree on your points Maggie, and I think we can do a guide on that at some point and link here.  Since this change is just in the readme, I think keeping it short and sweet is probably good.
Re-reading it again, I really don't like it actually.  My reasons are:

Too many idiomatic catchphrases ("breeze", "heavy lifting", "super-charge")
Emphasis on "dates"  (we do times too)
"Read and parse dates"...  are reading and parsing different things?  And we don't really parse dates, we parse strings that happen to contain representations of date and time values.
"easily calculate dates" - the phrasing is off here too. One might "perform calculations on a date", but one doesn't "calculate a date" in isolation.  The difference is that one phrasing implies a starting and ending value with some manipulation in between, and the other implies creation.
"multitude of formats" makes me think "oh really, which ones?" Right now we only have direct support for ISO8601, the ASP.NET JSON crap, and the rest is custom formatting.  There's an issue open for RFC2822, but that's it.  Sure, you can format output any way you like it via format token specifiers and locale-specific defaults, but you can't easily list the multitudes of formats moment supports.

Sorry to be so critical.  I do appreciate the effort - I just don't think this adds a whole lot in its current format.  It reads more like an advertisement than anything else.  Also, by having a bulleted list, one might think that everything in the list is all that there is.
@PatrickFranz - If you want to take another stab at it, I'd suggest going through the documentation, section by section. Summarize each significant function in one or two words, and sort from most useful to least.  If the list gets too long, drop a few of the lesser important items and replace with something like "and much more..."   Use a natural tone of voice, but try not to add any hype.  Let the reader decide for themselves if it's valuable to them or not.
P.S. - Part of my reasoning is that I've been interacting with a few folks lately on other web sites who are very anti-library and seem to be on a campaign to discourage use of moment.js.  It's been respectful discourse, but I don't want to give them any additional ammunition.  Moment serves a very particular purpose and if someone is happier writing their own code against the Date object, more power to them. :smile:

OK. Sorry for wasting your time.

Not a waste at all.  It's good for us to think through these things.  We've been discussing ideas like this quite a bit, such as adding the new guides section on the website.
I really do appreciate your efforts, and if you want to try again that would be welcomed.
Thanks,
Matt


Text output functions (WScript.Echo / console.log / print) corrupt non-ASCII characters on output.

Note: all of the text output functions named above are implemented as the same native function, WScriptJsrt::EchoCallback, with multiple JS name bindings.
'NO-BREAK SPACE' (U+00A0)
becomes
á 'LATIN SMALL LETTER A WITH GRAVE' (U+00E0)
¡ 'INVERTED EXCLAMATION MARK' (U+00A1)
becomes
í 'LATIN SMALL LETTER I WITH GRAVE' (U+00E1)
– 'EN DASH' (U+2013)
becomes
? 'QUESTION MARK' (U+003F)
Same transformation for EM-DASH (U+2014)
Here's a test:
console.log("\u00a0\u00a1\u2013\u2014");
// U+00A0 ( ) no-break space
// U+00A1 (¡) inverted exclamation mark
// U+2014 (—) em dash
// U+2013 (–) en dash

Output from ch: áí??
Output from node:  ¡–— (correct output, so apparently not a problem with conhost on Windows)
From looking at a hex dump of the output, the first two cases are apparently incorrectly rendered as a single byte (since the code point is 0 except the bottom 8 bits, we might be mistakenly interpreting that as a code point which is valid to output as a single byte):
> ch test.js | xxd
00000000: a0a1 3f3f 0d0a                           ..??..

Note a0a1 in the first two bytes (the ISO/IEC 8859-1, i.e. Latin-1, one-byte encoding of those codepoints, but not a correct UTF-8 encoding), and 3f (?) in the second two bytes. Final 2 bytes are CRLF.
> node test.js | xxd
00000000: c2a0 c2a1 e280 93e2 8094 0a              ...........

Note c2a0 and c2a1, the correct UTF-8 encodings of U+00A0 and U+00A1, followed by e28093 and e28094, the correct UTF-8 encodings of U+2013 and U+2014 respectively. (LF line terminator at the end.)
In the latter two cases, I think this is internally converting to ? as a replacement character for an unknown character.
Discovered when looking into https://github.com/Microsoft/ChakraCore/issues/2656

/cc @tcare see #2712


Updated Bengali Python course

What does this PR do?
Add resource(s)
For resources
Description
Hello Sir,
I have added 2 very useful python courses in Bengali .It is useful for all the beginners as well as for the python developers.
Why is this valuable (or not)?
This is a Valuable course.
How do we know it's really free?
As the whole playlist is available on YouTube. It is totally free.
For book lists, is it a book? For course lists, is it a course? etc.
It is a course list.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

@sohansai I have made the changes. Can you please check those

@eshellman Please take a review of my PR & please merge it.


Clear all previous votes / reset votes

Thanks for building this powerup, it's very useful! This is a enhancement request, instead of a bug.
It would be great if there was an option to clear all existing votes on the cards, rather than having to rely on every person individually undoing their previous votes from a previous lean coffee session at the start of a new lean coffee session.
I'm aware that technically there shouldn't be any leftover cards from a previous session, but in practice, for some of our lean coffee sessions, we find that instead of deleting the existing list of cards and starting completely afresh, it is more efficient to leave the previous cards in place, reset the votes, and use the new votes to set the agenda for the current session.

Hi, @sjhale 🙂 I imagine this would make sense at the list level (rather than the whole board), so you could keep seeing the votes on the cards that were actually discussed). WDYT?
It makes sense as a use case, I'd say. It shouldn't be too hard to implement, but I'll need to check. Thanks for the feedback!

@tatablack I can't believe how quickly you incorporated this! Thanks very much - it's exactly what I was after! :)


Useful info in AWS Instance utilization.

Policy Template Name
AWS Instance CloudWatch Utilization
Issue Type
Feature Request
Use Case (Feature Request -or- New Policy Template Request Only)
What I have to identify instances on this report are instance IDs and IP addresses. (It says hostname and DNS name, but those are just IPs for me.) It would be useful to have the 'Name' and maybe 'env' tags. I might also like to know the instance type or just plain how much it cost.
In some cases I have reports for instances that no longer exist. That's still useful in case I want to adjust the conditions of their creation and destruction, but it makes it extra hard to figure out what something was after the fact.
The reports I get have about a hundred instances in them. I don't know how you'd want to summarize that, but I'd love to have an alternative to going through it line by line.

Hi @kyleha-flexera   It looks like you may be using an old policy version of that policy.  Have you tried the AWS Inefficient Instance Utilization using CloudWatch policy yet.  It doesn't have the tags, but it has instance type.  AWS doesn't have names on the API output of instances,  but there is a name tag.  We can look into adding these changes.  I don't know about another way to present the data.  you can send your comments about new ways to visualize the data to<EMAIL_ADDRESS>which goes to the product manager.


sql server: problem with "for xml" into query

Hi all,
I've noticed a problem using "for xml" into query.
I've tried this code
Yii::$app->db->createCommand('select top 1 * from <tableName> FOR XML auto')->execute() also using queryAll(), queryScalar(), etc. and I get
General error: 4004 General SQL Server error: Check messages from the SQL Server [4004] (severity 16) [(null)] The SQL being executed was ...
If I copy/paste the query from the error log into a db client, it works.
I use yii <IP_ADDRESS> with php 5.6 under a debian.
Thanks

I created a simple action
public function actionTest(){
        Yii::$app->db->createCommand('select top 1 * from soggiorni FOR XML auto')->execute();
}

called it via browser and I saw this error:
Database Exception – yii\db\Exception
SQLSTATE[HY000]: General error: 4004 General SQL Server error: Check messages from the SQL Server [4004] (severity 16) [(null)]
The SQL being executed was: select top 1 * from soggiorni FOR XML auto
Error Info: Array
(
    [0] => HY000
    [1] => 4004
    [2] => General SQL Server error: Check messages from the SQL Server [4004] (severity 16) [(null)]
    [3] => -1
    [4] => 16
)
↵
Caused by: PDOException
SQLSTATE[HY000]: General error: 4004 General SQL Server error: Check messages from the SQL Server [4004] (severity 16) [(null)]
in /var/www/report-camping/vendor/yiisoft/yii2/db/Command.php at line 994

If I copy/paste the query into a client, I obtain:

What I need is the xml as text field (in the real use case, I need a comma separated values by traversing the xml, but it's too complex to post here. I just reproduces the error).
I also tried to change the action, I saw this solution reading about the same problem in a lavarel forum:
public function actionTest(){

        Yii::$app->db->createCommand('SET QUOTED_IDENTIFIER ON')->execute();
        Yii::$app->db->createCommand('SET ANSI_WARNINGS ON')->execute();
        Yii::$app->db->createCommand('SET ANSI_PADDING ON')->execute();
        Yii::$app->db->createCommand('SET ANSI_NULLS ON')->execute();
        Yii::$app->db->createCommand('SET CONCAT_NULL_YIELDS_NULL ON')->execute();
        Yii::$app->db->createCommand('select top 1 * from soggiorni FOR XML auto')->execute();

    }

Same error.
thanks


Remove nette/utils to allow easier install

In https://github.com/TomasVotruba/unused-public/pull/86 one call to Nette\Util\Arrays::flatten() was not removed. This PR removes the reference and also removes the dependency to nette/utils on the whole.

Thank you :clap:

Thanks, too :)


Publishing a binary verson?

Hi,
This looks like a really cool, lightweight solution I've been looking for. However I don't have access to VisualStudio, would you be willing to publish a binary version of this in the "Releases" section?
Thanks

Hey there. I would be happy to post a binary. However that's a really old program I wrote. I would rewrite it using native code.
Here's a rar containing the exe I'm using now (I'm at work so can't really make a release) It's a build of the C# version of the logger
https://www.dropbox.com/s/01j86v2my8u33mu/ClipboardLog.rar?dl=0
I'm using Windows 7, don't know if you'll run into issues on different versions. Let me know if that works!
You will find ClipboardLog.txt in the MyDocuments folder.


Handle exception when message has wrong schema

The python-streaming-data-types module changed to throw exceptions if the schema is wrong, this change allows us to support it.

Codecov Report

Merging #62 (7770d19) into master (a16973f) will decrease coverage by 0.63%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #62      +/-   ##
==========================================
- Coverage   93.99%   93.36%   -0.64%
==========================================
  Files          11       11
  Lines         433      437       +4
  Branches       58       59       +1
==========================================
+ Hits          407      408       +1
- Misses         20       22       +2
- Partials        6        7       +1

Impacted Files
Coverage Δ

forwarder/parse_config_update.py
87.50% <100.00%> (-3.68%)
:arrow_down:

Codecov Report

Merging #62 (7770d19) into master (a16973f) will decrease coverage by 0.63%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #62      +/-   ##
==========================================
- Coverage   93.99%   93.36%   -0.64%
==========================================
  Files          11       11
  Lines         433      437       +4
  Branches       58       59       +1
==========================================
+ Hits          407      408       +1
- Misses         20       22       +2
- Partials        6        7       +1

Impacted Files
Coverage Δ

forwarder/parse_config_update.py
87.50% <100.00%> (-3.68%)
:arrow_down:


A solution

Not sure why I did this thing in the first place :-) I suppose we all think about elevator algorithms while waiting for one.
I'd say half the challenge is distilling the actual requirements from the text, which is a bit verbose and not entirely clear about everything (and may or may not coincide with a real elevator, as opposed to the writer's assumptions) - or maybe it's just me. I wrote down the short list of actual logical rules I found in the text, but then had to update it with additional ones that were uncovered only by failing tests.
I suspect that if you'd give the well-defined list of rules up front (less than 10 lines) it would be a somewhat different experience. But perhaps that's part of the challenge... it did take longer than I expected, and I guess that's the point.
Anyway... here's a working solution.

btw it's nice to see the different kinds of solutions people came up with... can you post the original solution too? (perhaps in separate branch)

Nice! Thanks for your thoughts. I agree that the requirements could be more clearly communicated. I tried to only give the requirements inline with doctests that illustrated them. Maybe there needs to be a summary upfront like you say.
Others have submitted solutions in pull requests which I have closed, in order to keep the pull request tab clear for changes to the challenge itself. The first one was here: https://github.com/mshang/python-elevator-challenge/pull/2

I found the other solutions, though it took a while - perhaps you can link to them directly in the docs. And I was referring to your own solution that you can add to the bunch :-)


KAFKA-9295: revert session timeout to default value

Updated:
Since https://github.com/apache/kafka/pull/10803 is opened to increase default session timeout to 45 seconds, I reverted my previous change to use default session timeout setting. Thanks.
===============
Increase session timeout to 30 secs to make the test reliable.
I had a PR, to duplicate multiple KTableKTableForeignKeyInnerJoinMultiIntegrationTest tests and duplicate another KTableKTableForeignKeyInnerJoinMultiIntegrationTest tests to increase the session timeout to 30 seconds as comparison. After 3 runs, I found all 30 seconds tests passed, but the original 20 seconds session timeout failed sometimes. That's why this PR to increase session timeout to 30 seconds is created.
But, there's also 1 more thing I noticed. Not sure if it's a bug or not.
We have this declared:
table1.join(table2, tableOneKeyExtractor, joiner, materialized)
            .join(table3, joinedTableKeyExtractor, joinerTwo, materializedTwo)
            .toStream()
            .to(OUTPUT,
                Produced.with(serdeScope.decorateSerde(Serdes.Integer(), streamsConfig, true),
                              serdeScope.decorateSerde(Serdes.String(), streamsConfig, false)));

So, I think (and checked the code), we should run tableOneKeyExtractor first, then joiner, and then joinedTableKeyExtractor if we needed, then joinerTwo.
But in the failed case:
Did not receive all 1 records from topic output- within 60000 ms

I found we got all 4 data and entered tableOneKeyExtractor
<during rebalancing...>
tableOneKeyExtractor:-1
tableOneKeyExtractor:2
tableOneKeyExtractor:-2
tableOneKeyExtractor:1
<all streams are running>
joiner: value1 2.22,20
joinedTableKeyExtractor:value1=2.22,value2=20

The expected result (run from my local env)
tableOneKeyExtractor:1
tableOneKeyExtractor:2
tableOneKeyExtractor:-1
tableOneKeyExtractor:-2
joiner: value1 1.33,10
joiner: value1 2.22,20
joinedTableKeyExtractor:value1=1.33,value2=10
joinedTableKeyExtractor:value1=2.22,value2=20
joiner2: value1 value1=1.33,value2=10,waffle

You can notice the joiner: value1 1.33,10 doesn't get run in the failed case, and that's why the following joinedTableKeyExtractor and joiner2 not run. Do you have any thought about that? I'm thinking it might because the 3 streams are still under rebalancing, so the streaming processing might get terminated if the stream drop suddenly. And that's why the data lost. Do you think it's possible? Any thought about it?
You can check this jenkins log here (search "standard error"). Some keywords are:
starting is starting 3 streams
all running is that 3 streams reach running state
tableOneKeyExtractor, joinedTableKeyExtractor, joiner2 should be clear
value1 is joiner
Thank you very much.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@ableegoldman , do you have any thought? Thank you.

1 more thing to add, because I can still see
reason: removing member KTable-FKJ-Multi-98311d03-31f2-4b20-bfcf-0cfc516741f5-StreamThread-1-consumer-06247087-0bc9-4f74-b0a4-a617c638e427 on heartbeat expiration

So, the session timeout increasing is to avoid this unexpected rebalancing happen, to make the test reliable. I think it makes sense. Just concerning the data lost issue mentioned above. Thank you.

Hey @vvcephei @bellemare , any thoughts on Luke's analysis above? I'm not super familiar with the FKJ so it's difficult for me to say whether this sounds like a possible data loss bug or not. That said, the symptoms do seem quite suspicious -- no other test is suffering from this degree of flakiness due to rebalancing, and even if the test does incur some rebalances here and there, it should still be able to process at least one record I would think.

Hard to say. Also the logs are truncated. However, it seems there is some issue in the beginning that maybe some topic where not created?
[2021-05-27 08:36:11,424] WARN [Consumer clientId=KTable-FKJ-Multi-e7433918-5346-4bc4-b9b8-41bc75e16fa9-StreamThread-1-consumer, groupId=KTable-FKJ-Multi] Error while fetching metadata with correlation id 2 : {KTable-FKJ-Multi-KTABLE-FK-JOIN-SUBSCRIPTION-RESPONSE-0000000017-topic=UNKNOWN_TOPIC_OR_PARTITION, KTable-FKJ-Multi-KTABLE-FK-JOIN-SUBSCRIPTION-RESPONSE-0000000030-topic=UNKNOWN_TOPIC_OR_PARTITION, KTable-FKJ-Multi-KTABLE-FK-JOIN-SUBSCRIPTION-REGISTRATION-0000000009-topic=UNKNOWN_TOPIC_OR_PARTITION, KTable-FKJ-Multi-KTABLE-FK-JOIN-SUBSCRIPTION-REGISTRATION-0000000022-topic=UNKNOWN_TOPIC_OR_PARTITION} (org.apache.kafka.clients.NetworkClient:1092)

Note the timestamp of 08:36:11.
After the truncated logs continue, the timestamp is 10:20:17 ... -- it's an almost 2h gap ...

Thanks @mjsax ! I'll further investigate it and let you know~
@ableegoldman , since #10803 is already merged into trunk, now the default session timeout is 45 seconds. I think we can merge this PR to revert previous session timeout increasing workaround. Thank you.


Syntax Highlight: Simple Network Management Protocol

Which Syntax?:
IOS
Expected behavior and actual behavior:
Highlight. Actual, none?
Steps to reproduce the problem:
Review config containing SNMP configuration.
New Feature Description:
Would like to see SNMP configuration lines highlighted. Highlighting that it is an SNMP line, if changes were added / removed,

Examples please

Hi!
Sorry about the delay. Are you looking for example output to try playing around with, or an example of how I would like it colored?

Ah yes, I should be clearer :)
The example commands are enough... links to the command reference is even better.

https://www.cisco.com/c/en/us/td/docs/ios-xml/ios/snmp/command/nm-snmp-cr-book/nm-snmp-cr-s5.html
there is the SNMP command reference.

Done.


feat(explore): new datasets have autocomplete filters enabled if UX_BETA is set

SUMMARY
If UX_BETA feature flag is set to True, all new datasets will have autocomplete filters enabled by default. The existing datasets will not be modified.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
https://user-images.githubusercontent.com/15073128/126776554-078ca129-964f-4e6a-8973-442a62d8cbb5.mov
TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[x] Has associated issue: closes #15694
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

CC @junlincc @graceguo-supercat

/testenv up FEATURE_UX_BETA=true

/testenv up FEATURE_UX_BETA=true

We'll address Ville's comment in a follow up PR


Yehor/pred 832 transferable models support

Updated README file with detauls for standalone scoring

Coverage increased (+0.06%) to 82.739% when pulling b360fd8317534aaee76f1e5b67b417643b416364 on yehor/PRED-832_transferable_models_support into ffd64d2db89548eb1e2cc11b19e5e286b1d7a4b1 on master.

Coverage increased (+0.06%) to 82.739% when pulling b360fd8317534aaee76f1e5b67b417643b416364 on yehor/PRED-832_transferable_models_support into ffd64d2db89548eb1e2cc11b19e5e286b1d7a4b1 on master.

Coverage increased (+0.06%) to 82.739% when pulling b360fd8317534aaee76f1e5b67b417643b416364 on yehor/PRED-832_transferable_models_support into ffd64d2db89548eb1e2cc11b19e5e286b1d7a4b1 on master.

Coverage increased (+0.06%) to 82.739% when pulling b360fd8317534aaee76f1e5b67b417643b416364 on yehor/PRED-832_transferable_models_support into ffd64d2db89548eb1e2cc11b19e5e286b1d7a4b1 on master.

Coverage increased (+0.06%) to 82.739% when pulling b360fd8317534aaee76f1e5b67b417643b416364 on yehor/PRED-832_transferable_models_support into ffd64d2db89548eb1e2cc11b19e5e286b1d7a4b1 on master.

Coverage increased (+0.06%) to 82.739% when pulling b360fd8317534aaee76f1e5b67b417643b416364 on yehor/PRED-832_transferable_models_support into ffd64d2db89548eb1e2cc11b19e5e286b1d7a4b1 on master.

//CC @datarobot/predictions

@jendavies

@nimnull  Make sure you can run make test successfully when you updated the README.  The inside test this is a command that tests that RST is valid for PyPi

@jendavies applied all suggestions. Does it look good now?

okay with me

Coverage increased (+0.06%) to 82.739% when pulling 1f1cb0eeb8c10718b92083529796587500a81ebf on yehor/PRED-832_transferable_models_support into ffd64d2db89548eb1e2cc11b19e5e286b1d7a4b1 on master.

Updated with recent remarks

Coverage increased (+0.06%) to 82.739% when pulling bf4716b9cd06690b529e3a292d42894af293b136 on yehor/PRED-832_transferable_models_support into ffd64d2db89548eb1e2cc11b19e5e286b1d7a4b1 on master.


outdated dependencies

This library has a lot of stale dependencies.
Some of them have security vulnerabilities.
With bb9a18634e7f I've committed the current package-lock.json, so we freeze the current situation and start from there.

Tested each, updated and published new version.


Unit Tests to form validators/filters and issues about aliases.

In this request we will add Unit Tests to the filters and validators methods and resolve the issues #7 and #8 . And review all related comments to translate to english.

Coverage increased (+7.5%) to 22.656% when pulling a609d014ea2ba8a6868eb2ec30fd1e82a8aebc16 on form-methods-alias into 249cfcef2b21d69836ab9a34f59986c4c55f185d on master.

Coverage increased (+9.8%) to 24.964% when pulling 12385ca2680f5514aab83ddf3830432ccff626ad on form-methods-alias into 249cfcef2b21d69836ab9a34f59986c4c55f185d on master.


Remove deprecation warnings from rasa/utils/train_utils.py

Context:
In preparation for the 3.0 release, all deprecation warnings need to be removed.
These were belatedly identified in rasa/utils/train_utils.py in connection with DIET classifier:

update_deprecated_sparsity_to_density()

    if WEIGHT_SPARSITY in config:
        rasa.shared.utils.io.raise_deprecation_warning(
            f"`{WEIGHT_SPARSITY}` is deprecated."
            f"Please update your configuration file to use"
            f"`{CONNECTION_DENSITY}` instead.",
            warn_until_version=NEXT_MAJOR_VERSION_FOR_DEPRECATIONS,
            docs=DOCS_URL_MIGRATION_GUIDE_WEIGHT_SPARSITY,
        )
        config[CONNECTION_DENSITY] = 1.0 - config[WEIGHT_SPARSITY]

update_deprecated_loss_type()

    if config.get(LOSS_TYPE) == SOFTMAX:
        rasa.shared.utils.io.raise_deprecation_warning(
            f"`{LOSS_TYPE}={SOFTMAX}` is deprecated. "
            f"Please update your configuration file to use"
            f"`{LOSS_TYPE}={CROSS_ENTROPY}` instead.",
            warn_until_version=NEXT_MAJOR_VERSION_FOR_DEPRECATIONS,
        )
        config[LOSS_TYPE] = CROSS_ENTROPY

_replace_deprecated_option()

    if isinstance(new_option, str):
        rasa.shared.utils.io.raise_deprecation_warning(
            f"Option '{old_option}' got renamed to '{new_option}'. "
            f"Please update your configuration file.",
            warn_until_version=warn_until_version,
        )
        return {new_option: config[old_option]}

    rasa.shared.utils.io.raise_deprecation_warning(
        f"Option '{old_option}' got renamed to "
        f"a dictionary '{new_option[0]}' with a key '{new_option[1]}'. "
        f"Please update your configuration file.",
        warn_until_version=warn_until_version,
    )

create_common_callbacks()

        if tensorboard_log_level == "minibatch":
            tensorboard_log_level = "batch"
            rasa.shared.utils.io.raise_deprecation_warning(
                "You set 'tensorboard_log_level' to 'minibatch'. This value should not "
                "be used anymore. Please use 'batch' instead."
            )

_check_confidence_setting()

    if component_config[MODEL_CONFIDENCE] == LINEAR_NORM:
        rasa.shared.utils.io.raise_deprecation_warning(
            f"{MODEL_CONFIDENCE} is set to `{LINEAR_NORM}`. We "
            f"introduced this option in Rasa Open Source 2.3.0, "
            f"but have identified multiple problems with it based "
            f"on user feedback. Therefore, `{MODEL_CONFIDENCE}={LINEAR_NORM}` "
            f"is now deprecated and will be removed in Rasa Open Source `3.0.0`."
            f"Please use `{MODEL_CONFIDENCE}={SOFTMAX}` instead."
        )

Definition of Done:

[ ] the deprecation warnings are resolved.
[ ] @samsucik assigned reviewer

There is also this is diet_classifier.py:
# TODO: This should be removed in 3.0 when softmax as
#  model confidence and normalization is completely deprecated.
message_sim = train_utils.normalize(
    message_sim, self.component_config[RANKING_LENGTH]
)

There is also this is diet_classifier.py:
# TODO: This should be removed in 3.0 when softmax as
#  model confidence and normalization is completely deprecated.
message_sim = train_utils.normalize(
    message_sim, self.component_config[RANKING_LENGTH]
)

I have removed softmax as a loss type but I can't see a deprecation warning about softmax as model confidence. I'm just wondering if we should involve removing this code in this issue because softmax as model confidence will still exist after the changes I've made. Any ideas about that?

@dakshvar22 maybe you can clarify the above-linked comment -- I think we're not removing softmax as model confidence/normalisation?

Yes, the TODO in the code is outdated. softmax as a value for model_confidence should not be removed.
The issue with normalization (the code snippet above) should be handled separately because there are a few different ways to fix it in 3.0 and will need some feedback from CSE before we implement a solution.
I'd recommend to leave it out of this issue. I'll create another issue for it.


[kafka_consumer][wip] enable fetching consumer offsets from kafka

What does this PR do?
This PR enabled collection of kafka consumer offsets from kafka.
This PR aims to:

supporting collection of consumer offsets from Kafka, in addition to ZK.
not break backward compatibility by keeping a single metric space for both sets of offsets collected, and their derived metrics (consumer_lag).
where available still support the self discovery of consumer groups, topics and partitions. Work still ongoing to support this when consumer offsets are not written to ZK.

We can now collect the metrics from either source and append the source:kafka or source:zk tag accordingly to the metrics, including kafka.consumer_lag. So both should be available. For customers using one or the other (consumer offsets in kafka vs zookeeper), all dashboards and monitors should continue to work as expected. No changes needed. For people in a migration or mixed setup, we still have to evaluate the impact of having the two metrics. The inclusion of an additional tag canonical is in consideration to label the canonical offsets (again, with customers in the process of migrating).
Although the new KafkaClient has pretty decent support for an async implementation, that is currently out of the scope.
Motivation
Support for newer Kafka's and their new-style consumers storage of offsets in kafka itself.
Testing Guidelines
An overview on testing
is available in our contribution guidelines.
Versioning

[ ] Bumped the version check in manifest.json
[ ] Updated CHANGELOG.md

Additional Notes
This is still a WIP. I am trying to limit the changes to the interface as much as possible, while adding all required support. The main change thus far to the configuration is the addition of a kafka_consumer_offsets boolean option at the instance level: if set, then we will attempt to collect offset from kafka (as well as ZK if the zk_connect_str is still set in the YAML). If zk_connect_str is not set we will assume only kafka offsets are wanted.
We have also introduced the kafka_retries option at the init level to configure the maximum number of retries per failed query we will make to kafka after a failed query. Kafka sometimes has it's things like a consumer coordinator change, or a leader change, etc... during these times the metadata collection may fail. Tweaking this value helps us be more resilient to these conditions. In case we ultimately fail to collect the metrics, there's not much to worry about, these issues are transient and should be resolved by the next iteration.

 [kafka_consumer] support querying consumer offsets from Kafka


In medium screen, adjust to 640px so it can contain two columns

Before
Before fixing, it can only contain one column in a medium screen, the rest of blank space are too large.

After
After fixing, it can contain two columns (centered) in a medium screen

Why
https://github.com/h5bp/h5bp.github.io/blob/main/assets/style.css#L100-L102
The grid unit is 300px, with left and right margin 10px each. So if we want to wrap two colums, max-width should be calculated as 300px * 2 + 4 * 10px = 640px. I think the original value 620px is a typo.
What do you think?

THANKS!


Update s6-overlay

Not sure if you are interested in getting this updated, but I was looking at both running Litestream and implementing s6-overlay in a container. So the work I was doing anyway might help someone else out.
Some notes about choices made:

For easy offline testing I added a docker-compose.yml file.
This uses the atmoz/sftp image to have a place for Litestream to push updates to. I was going to use MinIO, but that requires either a fix for benbjohnson/litestream#219 to land or a 0.4.0 release that includes benbjohnson/litestream#374 (and possibly benbjohnson/litestream#379).
Simply docker-compose up from the directory and start requesting the website.

This runs the Go application as the CMD rather than as a supervised service.
The old finish file that used s6-svscanctl needed to be replaced with one using /run/s6/basedir/bin/halt, per the documentation. But the same documentation says:

If a service should bring the container down, you should probably run it as a CMD instead; […]

This also made the overall s6-overlay setup a little simpler. If you do want to keep it as a s6 supervised service, cherry-pick away the last commit in this PR (6529407) to instead have a s6-overlay version 3 compatible service running myapp.

Thanks for the inspiration that this repo has given me. And am looking forward to be able to use MinIO with 0.4.0 when that gets cut ☺

hey @Zegnat. Sorry for such an incredibly late reply. I'm not maintaining this s6-overlay example repo any more as most folks use the -exec flag that was added to Litestream instead of using a supervisor like s6. I did have trouble with getting started with s6 and finding examples so I'm sure your PR will be helpful to folks out there. Thanks!

@benbjohnson thanks for getting back on this! I will merge my changes into my own main branch then and simply keep that fork updated for as long as I used it 👍
I understand that -exec will cover most usecases, but I think Litestream also makes for a great illustration on how to get started with s6-overlay.
Have you considered marking this repository as archived on GitHub to illustrate the not-maintained status?

@Zegnat Yeah, that's a good idea. I'll go ahead and move it to archived.


[dvc][server][controller][vpj][test] Add PubSubConsumer with Pub Sub integration.

Summary, imperative, start upper case, don't end with a period

Begin replacing KafkaConsumerWrapper with PubSubConsumer throughout the consumption code. PubSubConsumer interface just replace topic, partition terms with PubSubTopic and PubSubTopicPartition.
Now, ApacheKafkaConsumer and SharedKafkaConsumer is the implementation of PubSubConsumer. If we plan to support other Pub Sub systems, we will implement a specific implementation of PubSubConsumer. I keep KafkaConsumerWrapper and name of SharedKafkaConsumer until we finally get rid of 'kafka' term inside class/method/comment existing in our current ingestion flow.
Replace string topic and partition on interfaces of StoreIngestionTask, KafkaConsumerService, and their factories and related test classes.
PubSubTopicRepository is put into KafkaPubSubMessageDeserializer for avoiding changing the interface of calling chain from VeniceStoreIngestionService to PubSubConsumer Instance. Ideally, PubSubMessageDeserializer should be inside PubSubConsumer to generate PubSubMessage<byte[], byte[]> and PubSubTopic. Other components should only deal with PubSubTopic and PubSubTopicPartition.

How was this PR tested?

internal CI
Does this PR introduce any user-facing changes?

[*] No. You can skip the rest of this section.
[ ] Yes. Make sure to explain your proposed changes and call out the behavior change.

Hi Hao, thanks for the great work. I think this is very close to being ready to merge. I left some super minor comments, and it seems like you have a spot bugs violation. Let's sort these out and merge this 🚀


Add flag

Hey its anyway to add flag method in runServer?

https://github.com/jpenilla/run-paper/issues/18


Fix missing filter for repo owner

The original query was not filtering by the owner, so it included all 'ubn' repositories for a particular language.

This change is

Current coverage is 3.56% (diff: 60.00%)

Merging #112 into develop will not change coverage

@@           develop      #112   diff @@
========================================
  Files           36        36
  Lines         8391      8391
  Methods          0         0
  Messages         0         0
  Branches         0         0
========================================
  Hits           299       299
  Misses        8067      8067
  Partials        25        25

Powered by Codecov. Last update a521b1e...cd205ad


initial version

@garyb can you take a look at https://github.com/safareli/purescript-number-input-halogen/pull/1
I have moved configuration into halogen props.


Add edu-vd.ch

Institution/School Name
Département de la formation, de la jeunesse et de la culture (DFJC) du canton de Vaud
Instiution/School Website
https://edu-vd.ch (https://my.edu-vd.ch)
Email Users

[x] Students
[x] Academic/Teaching Staff
[x] Other/Support Staff
[ ] Alumni

Education Levels

[ ] Post-graduate research
[ ] Graduate School
[ ] College (University) or Undergraduate School or equivalent
[ ] Community College or equivalent
[ ] High School or equivalent
[x] Elementary School or equivalent

@Imbwbl Please provide us with:

the school official website URL
the school street address, including city and country
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

Please provide us with a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.
You can provide us with any of the following:

A copy of instruction (a hard-copy or electronic one) given to you by the school that tells you to use an email address in this email domain
An URL of a page at the official school website that says that students use this email domain
A reference to a recognized journal where an article was published by someone from the school, with explicit affiliation to the school, and e-mail address from the email domain
A WHOIS data that link the email domain to the school somehow
A copy of official letter from the school representative saying that the school provides e-mail addresses in this domain to the students
Anything else that can prove this fact in similar way, i.e. can be verified independently.

@Imbwbl You might miss my previous comment. Please react.

@Imbwbl You might miss my previous comment. Please react.

We did see it, let us some time to collect all these things.

@Imbwbl Hello! Do you have any update on the matter?

@Imbwbl I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the aforementioned issue. Thank you for your understanding!


Color coding HTTP Verbs

I saw a demo of the solution today and thought that a good idea would be to color code the HTTP verbs similar to what swagger UI supports.
I am talking about this interface:

And using this:

I think that from a UI/UX perspective, it makes it easier to consume.

Thank a lot for the suggestion! Great Idea. Will be added.

@jbogarin it was now handled in #133. Thanks again!


Error when importing an Xpansion study with existing sensitivity directory

When I import an Xpansion study such that it already has a folder user/expansion/sensitivity, I get an error when I try to get into the Xpansion tab of the study with the following message :
Échec du chargement de la configuration d'Xpansion
Status : 500
Exception : ValidationError
Description : Unexpected server error
AxiosError: Request failed with status code 500
Whenever I remove the sensitivity folder, the error disappears and I can get to the Xpansion tab normally.
An example study

Traceback:
raceback (most recent call last):
  File "/usr/local/lib/python3.8/site-packages/anyio/streams/memory.py", line 94, in receive
    return self.receive_nowait()
  File "/usr/local/lib/python3.8/site-packages/anyio/streams/memory.py", line 89, in receive_nowait
    raise WouldBlock
anyio.WouldBlock

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.8/site-packages/starlette/middleware/base.py", line 41, in call_next
    message = await recv_stream.receive()
  File "/usr/local/lib/python3.8/site-packages/anyio/streams/memory.py", line 114, in receive
    raise EndOfStream
anyio.EndOfStream

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.8/site-packages/starlette/middleware/errors.py", line 159, in __call__
    await self.app(scope, receive, _send)
  File "/usr/local/lib/python3.8/site-packages/ratelimit/core.py", line 82, in __call__
    return await self.app(scope, receive, send)
  File "/usr/local/lib/python3.8/site-packages/starlette/middleware/cors.py", line 84, in __call__
    await self.app(scope, receive, send)
  File "/usr/local/lib/python3.8/site-packages/starlette/middleware/base.py", line 63, in __call__
    response = await self.dispatch_func(request, call_next)
  File "/antarest/core/logging/utils.py", line 104, in dispatch
    response = await call_next(request)
  File "/usr/local/lib/python3.8/site-packages/starlette/middleware/base.py", line 44, in call_next
    raise app_exc
  File "/usr/local/lib/python3.8/site-packages/starlette/middleware/base.py", line 34, in coro
    await self.app(scope, request.receive, send_stream.send)
  File "/usr/local/lib/python3.8/site-packages/starlette/middleware/base.py", line 63, in __call__
    response = await self.dispatch_func(request, call_next)
  File "/antarest/core/utils/fastapi_sqlalchemy/middleware.py", line 56, in dispatch
    response = await call_next(request)
  File "/usr/local/lib/python3.8/site-packages/starlette/middleware/base.py", line 44, in call_next
    raise app_exc
  File "/usr/local/lib/python3.8/site-packages/starlette/middleware/base.py", line 34, in coro
    await self.app(scope, request.receive, send_stream.send)
  File "/usr/local/lib/python3.8/site-packages/starlette/exceptions.py", line 82, in __call__
    raise exc
  File "/usr/local/lib/python3.8/site-packages/starlette/exceptions.py", line 71, in __call__
    await self.app(scope, receive, sender)
  File "/usr/local/lib/python3.8/site-packages/starlette/routing.py", line 656, in __call__
    await route.handle(scope, receive, send)
  File "/usr/local/lib/python3.8/site-packages/starlette/routing.py", line 259, in handle
    await self.app(scope, receive, send)
  File "/usr/local/lib/python3.8/site-packages/starlette/routing.py", line 61, in app
    response = await func(request)
  File "/usr/local/lib/python3.8/site-packages/fastapi/routing.py", line 226, in app
    raw_response = await run_endpoint_function(
  File "/usr/local/lib/python3.8/site-packages/fastapi/routing.py", line 161, in run_endpoint_function
    return await run_in_threadpool(dependant.call, **values)
  File "/usr/local/lib/python3.8/site-packages/starlette/concurrency.py", line 39, in run_in_threadpool
    return await anyio.to_thread.run_sync(func, *args)
  File "/usr/local/lib/python3.8/site-packages/anyio/to_thread.py", line 31, in run_sync
    return await get_asynclib().run_sync_in_worker_thread(
  File "/usr/local/lib/python3.8/site-packages/anyio/_backends/_asyncio.py", line 937, in run_sync_in_worker_thread
    return await future
  File "/usr/local/lib/python3.8/site-packages/anyio/_backends/_asyncio.py", line 867, in run
    result = context.run(func, *args)
  File "/antarest/study/web/xpansion_studies_blueprint.py", line 90, in get_settings
    return study_service.get_xpansion_settings(uuid=uuid, params=params)
  File "/antarest/study/service.py", line 2162, in get_xpansion_settings
    return self.xpansion_manager.get_xpansion_settings(study)
  File "/antarest/study/business/xpansion_management.py", line 302, in get_xpansion_settings
    return XpansionSettingsDTO.parse_obj(json)
  File "pydantic/main.py", line 521, in pydantic.main.BaseModel.parse_obj
  File "pydantic/main.py", line 341, in pydantic.main.BaseModel.__init__
pydantic.error_wrappers.ValidationError: 1 validation error for XpansionSettingsDTO
sensitivity_config -> projection
  field required (type=value_error.missing)

How to test the new behavior :
1- Create a study with an Xpansion configuration
2- Import a json to user/expansion/sensitivity/sensitivity_in (you can do it with the raw api) with missing "projection" key, example: {"epsilon": 14.5, "capex":False}
3- Go back to the xpansion screen
4- Old behavior: "AxiosError: Request failed with status code 500" screen. New behavior, normal screen


Add unit tests to info configmap

Changes
Addressed https://github.com/openshift-pipelines/pipelines-as-code/pull/1177#issuecomment-1458067897 by adding unit tests to pkg/cli/info/configmap.go

Submitter Checklist

[ ] ♽  Run make test lint before submitting a PR (ie: with pre-commit, no need to waste CPU cycle on CI
[ ] 📖 If you are adding a user facing feature or make a change of the behavior, please verify that you have documented it
[ ] 🧪 100% coverage is not a target but most of the time we would rather have a unit test if you make a code change.
[ ] 🎁 If that's something that is possible to do please ensure to check if we can add a e2e test.
[ ] 🔎 If there is a flakiness in the CI tests then don't necessary ignore it, better get the flakyness fixed before merging or if that's not possible there is a good reason to bypass it. (token rate limitation may be a good reason to skip).

Codecov Report

Merging #1189 (8c07266) into main (a080acc) will increase coverage by 0.01%.
The diff coverage is 63.63%.

@@            Coverage Diff             @@
##             main    #1189      +/-   ##
==========================================
+ Coverage   62.97%   62.98%   +0.01%
==========================================
  Files         108      109       +1
  Lines        6634     6625       -9
==========================================
- Hits         4178     4173       -5
+ Misses       2044     2040       -4
  Partials      412      412

Impacted Files
Coverage Δ

pkg/cli/webhook/github.go
57.00% <0.00%> (ø)

pkg/cli/webhook/gitlab.go
58.88% <0.00%> (ø)

pkg/cmd/tknpac/create/repository.go
35.84% <0.00%> (+0.66%)
:arrow_up:

pkg/reconciler/status.go
63.52% <75.00%> (+2.27%)
:arrow_up:

pkg/cli/info/configmap.go
86.95% <100.00%> (ø)

pkg/cmd/tknpac/resolve/resolve.go
64.46% <100.00%> (+0.59%)
:arrow_up:

pkg/provider/github/app/token.go
70.27% <100.00%> (ø)

... and 1 file with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@chmouel if these PR's looks good can we merge ?

how come you pushed to the origin branch btw. Can you configure your origin remote to be read only by any chance? ie https://stackoverflow.com/a/10270189/145125


Formula pre-processing causing failure in latexmath

From ISO 10303-110
Fatal:unexpected:\] Attempt to end mode display_math
	at String; line 7 col 0 - line 7 col 86
	current frame is boxing group due to T_CS[\@@open@column] String; line 7 col 75 - line 7 col 75
	In Core::Stomach[@0x7ff2e933a748] Literal String; from line 7 col 0 to line 7 col 86
	 <= Core::Definition::Constructor[\]] <= Core::Stomach[@0x7ff2e933a748] <= Core::Definition::Primitive[\<EMAIL_ADDRESS><= Core::Stomach[@0x7ff2e933a748] <= Core::Definition::Constructor[\@@defa... <= Core::Stomach[@0x7ff2e933a748] <= Core::Definition::Constructor[\@@defa... <= Core::Stomach[@0x7ff2e933a748] <= Core::Definition::Constructor[\<EMAIL_ADDRESS><= Core::Stomach[@0x7ff2e933a748] <=<EMAIL_ADDRESS><= Core::Stomach[@0x7ff2e933a748] <= Core::Definition::Primitive[\<EMAIL_ADDRESS><= Core::Stomach[@0x7ff2e933a748] <= Core::Definition::Constructor[\@@defa... <= Core::Stomach[@0x7ff2e933a748] <= Core::Definition::Constructor[\@@defa... <= Core::Stomach[@0x7ff2e933a748] <= Core::Definition::Constructor[\<EMAIL_ADDRESS><= Core::Stomach[@0x7ff2e933a748] <=<EMAIL_ADDRESS><= Core::Stomach[@0x7ff2e933a748] <= Core::Definition::Constructor[\[] <= Core::Stomach[@0x7ff2e933a748]
1 fatal error
Math: latexmlmath failed to process equation:
\pmatrix{ \hat{e}_{\xi} \cr \hat{e}_{\eta}
\cr \hat{e}_{\zeta} } = {\bf T} \pmatrix{ \hat{e}_x \cr \hat{e}_y \cr  \hat{e}_z },

Confirmation that latexml works:
math.ml
\pmatrix{ \hat{e}_{\xi} \cr \hat{e}_{\eta}
\cr \hat{e}_{\zeta} } = {\bf T} \pmatrix{ \hat{e}_x \cr \hat{e}_y \cr  \hat{e}_z },

cat math.ml | latexmlmath --strict -

Works.

cat math.ml | latexmlmath --strict - is irrelevant. The LaTeX is fed through:
latexmlmath --strict --preload=amsmath --inputencoding=UTF-8 -- -
And it does indeed fail.

Our problem is that we can have heterogeneous LaTeX; but I propose to deal with this by first parsing LaTeX with AMSMath style, and if that fails, parse it with no style.


Match _Unwind_Reason_Code enum between ARM and Itanium.

Unwind ARM was using _URC_OK and Itanium _URC_NO_REASON, which made it difficult to write code that worked with both versions. This adds the missing one to each, which matches what is done in libunwind:
https://github.com/llvm/llvm-project/blob/master/libunwind/include/unwind.h#L33-L34
Also fixes some mismatched in indentation.

Looks like the ManyManySelectors_optimised test was hanging on the "Windows-2016 Debug-32" CI. Any idea why that might have been? Doesn’t seem to me like this could be related to this change.


The value of the slider is 0, the minimum value is 1, the value of this slider can not be less than the minimum value

My initial value is not 0 but 1 same as min value. So i dont get this.

I didn't understand what problem you are having

I am getting error that is saying:
The value of the slider is 0, the minimum value is 1, the value of this slider can not be less than the minimum value
That is not true. Value of slider is 1 and min value is 1 and slider is saying that my value is 0.

Can you show your code?

Where shared_props.surface.total is defined? It is the value you really passed to the component

Yes that is 1. From the store by default.

I think it's possible that your value is asynchronously assigned.

I was unable to resolve this. Still an issue.
Funny thing is when you set :min=50 and :value=100 it shows that value is 50 and also rewrites model to 50 (by default model value is 100)

So, it is like :min is rewriting model value!!!

When the value is less than the minimum value, the value is set to the minimum value.
Maybe you can try to restore it. https://jsfiddle.net/2xy72dod/1240/

It is not less ...
model value is 100 and min value is 50 and nodel is rest to 50.

how to build your project?
i am getting this error:
ERROR in ./src/vue2-slider.vue
Module parse failed: /myproject/node_modules/vue-slider-component/src/vue2-slider.vue Unexpected token (1:0)
You may need an appropriate loader to handle this file type.
| <template>
|   <div
|     ref="wrap"
 @ ./src/index.js 1:16-40

i did npm install and so on.

Anyway, for some reason if i put model like this:
v-model="some.netsted.thing.model"
i am getting that min value error.
but when i do:
data(){
  return{
    some_model: some.netsted.thing.model
  }
}

and then v-model="some_model"  error is gone.

The value of computed cannot be set. The value of store cannot be directly set. It should be modified by dispatch action or commit mutation.
You can not use v-model, use :value and @callback to set the value of store.
example: https://jsfiddle.net/2xy72dod/2857/


Enabling traefik dashboard

I thought I'd leave this here in case anybody else needed it, since there's no documentation. Feel free to close
To enable traefik dashboard you must first modify /root/coollabs/coolify/install/coolify-template.yml
Edit the traefik service to include this in the ports section
      - target: 8080
        published: 8080
        protocol: tcp
        mode: host

and change the values from false to true
      - --api.insecure=true
      - --api.dashboard=true

Then make sure you're in the /root/coollabs/coolify directory and run:
docker stack rm coollabs-coolify &&
    set -a &&
    source .env &&
    set +a &&
    envsubst < install/coolify-template.yml | docker stack deploy -c - coollabs-coolify

Hey @Soneji! Thanks for the issue, but this is deprecated as Coolify v2 does not use Traefik anymore, but HAProxy.


Editor should NOT reformat the entire Markdown file

If i make any small edit using markdown-editor, it will reformat the entire markdown file! This is very bad.
Why is this bad?

Imagine editing a single line in a Javascript, Python or other source code in the IDE and it reformats the entire file!
It makes diffs very noisy and code reviews very hard. If I change one word, the diff should only have on changed line, and in that line, only one changed word.
It causes unnecessary merge and rebase conflicts.
It pollutes the CVS/git history, making it very hard to see what really changed in a commit.
It disrespects the author's coding style and standards. Markdown files in a project should be treated and respected as source code just like any other file in the project. Edits should behave just like edits to code.

Many projects will not be able to use this plugin because of this problem.
Expectations:

Any change to one line should never result in changes to any another line. The only except to this is a change that must change other lines, e.g. increasing the nesting level of a list item will change all items in that list item.

Changes should always be minimal. If I fix a spelling error, only that word should be changed. The containing block should not be reformatted.

Respect the existing formatting. For example, if a list is formatted as a tight list (without blank lines between items), DO NOT insert blank lines between items and change it into a loose list.

Respect IDE settings, such as Editor:General:Remove trailing spaces on and Editor:Code Style:Markdown

I can understand this scenario, but limited by lute can not support this scenario.

Can you submit an issue against lute? I would do it, but I don't know the details as you do. You could point to this issue when you submit it.


UI DreamServer Upgrade

Hello! first time using Github, so hope i'm doing things well. I've created a new UI graphic version of "DreamServer" site, so hope you like it.
I mainly added some tags to reorder the base and reused the IDs and Classes already present to give them a new style. It is Responsive. Anything new you need I'm at your disposal.

I like this. I am new to github also, so I don't know where to report these things, but when using img2img, there is nowhere in the UI to adjust the strength parameter. I reported this issue on another comment chain related to the original web UI and figured I would mention it here as well.

Would it be possible to refactor this PR to create a more generic system for applying themes? Thus keeping both the original theme, this theme, and add the possibility for different themes in the future.

Added some quick comments. Also the background image should probably be a compressed JPG, for space reasons.

Thank u! Totally agree with your comments, i'm working to solve them.

@SergioDiazF This is very attractive. I particularly like the generous resizable prompt box. It's not clear to me whether the "various updates" commit you made address @TesseractCat's comments? I'll wait for confirmation

Hi, my main concern is this:

Would it be possible to refactor this PR to create a more generic system for applying themes? Thus keeping both the original theme, this theme, and add the possibility for different themes in the future.

Personally I prefer the old style, so I'd like it to be configurable. With a configurable system other users could contribute styles as well.

I think having a fully configurable UI is probably too much complexity to be worth it, but most of these changes are just CSS, so replacing the new index.css with the old would probably get you the old UI back.
If we really do want it to be configurable, at least in part, how about having the server take a --css-path option?
(That said, I also prefer not having the background image - it's pretty busy, and distracts from the generated images, which after all are the point.)

I think having a fully configurable UI is probably too much complexity to be worth it, but most of these changes are just CSS, so replacing the new index.css with the old would probably get you the old UI back.
If we really do want it to be configurable, at least in part, how about having the server take a --css-path option?
(That said, I also prefer not having the background image - it's pretty busy, and distracts from the generated images, which after all are the point.)

Yeah, all I mean by configurable is user-selectable css file.

@SergioDiazF This is very attractive. I particularly like the generous resizable prompt box. It's not clear to me whether the "various updates" commit you made address @TesseractCat's comments? I'll wait for confirmation

I took into account the recommendations of  @TesseractCat, so:

Changed background.png to a light background.jpg image.
added a space between all selectors and their respective brackets in css file.
Changed all CSS comments to Caps.
Cleaned CSS file removing useless commented code.
Restore Seed reset button.

I think having a fully configurable UI is probably too much complexity to be worth it, but most of these changes are just CSS, so replacing the new index.css with the old would probably get you the old UI back.
If we really do want it to be configurable, at least in part, how about having the server take a --css-path option?
(That said, I also prefer not having the background image - it's pretty busy, and distracts from the generated images, which after all are the point.)

Yeah, all I mean by configurable is user-selectable css file.

Totally agree, although the UI design has as main objective to facilitate the use of the tool, it also has a personal touch in between, so integrating different creative looks will cover a larger group of people. And it's not that difficult to implement considering that you only have to modify a CSS file.

Added theme support. To add new themes just include CSS file into /theme folder and add a new <option value="theme_n" selected>Theme name</option> inside <select id="theme-selector">, where "theme_n" is the name of the CSS file.

The background and theme systems seem unneeded. Seem like changes for the sake of changes.

@SergioDiazF Unfortunately there have been a lot of changes to the WebGUI to add new features over the past couple of days, and my attempts to resolve the conflicts in index.html were not particularly successful. I would really like to see the theming go in, and if you have the time and energy for it, could you pull a fresh copy of main and make your changes against that? Also, if at all possible, try to end your lines with LF rather than CRLF? I believe Notepad++ lets you do this.
So sorry. I should have merged earlier, but there were lots of backed up PRs.

@SergioDiazF Unfortunately there have been a lot of changes to the WebGUI to add new features over the past couple of days, and my attempts to resolve the conflicts in index.html were not particularly successful. I would really like to see the theming go in, and if you have the time and energy for it, could you pull a fresh copy of main and make your changes against that? Also, if at all possible, try to end your lines with LF rather than CRLF? I believe Notepad++ lets you do this.
So sorry. I should have merged earlier, but there were lots of backed up PRs.

I completely undestand, So i need to close the pull request right? how can i do that?

@SergioDiazF I hope you're still up for adding the theme changing features? I've caught up with the backlog of feature additions, and I think the web interface will be stable now.


how do i add a warning screen to fnf. and a mouse to the main menu?

What is your question?
im making a mod with jumpscares . and i want to add a warning screen. i also want to add the flx mouse to the main menu just like in yoshi engine.

how about making a danger screen as a cutscene?

how about making a danger screen as a cutscene?

i dont have any cutscenes for the mod

What is your question?
im making a mod with jumpscares . and i want to add a warning screen. i also want to add the flx mouse to the main menu just like in yoshi engine.

The flx mouse you can activate it in the project.xml file, if you don’t know what line is, just tell it to me and I will give you the lines

What is your question?
im making a mod with jumpscares . and i want to add a warning screen. i also want to add the flx mouse to the main menu just like in yoshi engine.

The flx mouse you can activate it in the project.xml file, if you don’t know what line is, just tell it to me and I will give you the lines

kk thanks

this line

oof. well the code is there. but i dont know what to chose. there are 3 lines of it.

which one

Any of them

kk but how do i enable them. what do i type

mouse script :
folder Phych engine/mods/scripts
El mié, 4 may 2022 a las 8:50, Jacks Robotics @.***>)
escribió:

kk but how do i enable them. what do i type
—
Reply to this email directly, view it on GitHub
https://github.com/ShadowMario/FNF-PsychEngine/issues/8429#issuecomment-1117338265,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZAFXN6IYHHTDVAZGPU3ZHTVIJ6BDANCNFSM5UZYPOEQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I'll see if I can make a warning lua script
El mié, 4 may 2022 a las 8:54, Dew Drawins @.***>)
escribió:

mouse script :
folder Phych engine/mods/scripts
El mié, 4 may 2022 a las 8:50, Jacks Robotics @.***>)
escribió:

kk but how do i enable them. what do i type
—
Reply to this email directly, view it on GitHub
https://github.com/ShadowMario/FNF-PsychEngine/issues/8429#issuecomment-1117338265,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZAFXN6IYHHTDVAZGPU3ZHTVIJ6BDANCNFSM5UZYPOEQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

kk

i found out that if you go to titlestate.hx you can change flxg.mouse.visible = false; to true. and i can also edit the warning flashing lights text.


Revert function signature change for NewMsgSubmitProposal

Background
As part of #240, we changed the function signature of the govtypes.NewMsgSubmitProposal from:
NewMsgSubmitProposal(content Content, initialDeposit sdk.Coins, proposer sdk.AccAddress)
to
NewMsgSubmitProposal(content Content, initialDeposit sdk.Coins, proposer sdk.AccAddress, IsExpedited bool)
in order to allow the creation of expedited proposals.  However, because the IsExpedited boolean is required, it breaks all the existing usages of NewMsgSubmitProposal, which is annoying because many of the usages of this function are from other repos that we don't maintain such as ibc-go and wasmd, and as such we would be required to create forks of all of these.
Suggested Design
I suggest we either make the IsExpedited parameter optional and defaulted to false (I'm not sure if there's a way to do this in Go).
If that is not possible, we should revert the function signature to the original form, and just create a new function called govtypes.NewExpeditedMsgSubmitProposal that can be called when needed to create an expedited proposal
Acceptance Criteria
goals & criteria for success
(for example)

all existing and new tests should pass

Yeah the constructor change was an API break that probably shouldn't have happened. Lets make a second constructor as you suggested, and restore prior functionality. (Params with defaults don't exist in golang. This is what the .With... design pattern or function param setting design pattern is for)
Per DM convo, assigning to you


License missing

Hi,
this seems to be a really nice project. Sadly there is no license information. What kind of license do you have in mind for this project?

Thanks for pointing that out. I couldn't decide when creating the repository and then forgot about it.


Search Button

Is your feature request related to a problem? Please describe.
If a user wants to go to a particular project then it is very tedious work to scroll down and find the project.
Describe the solution you'd like
I think we can make a search bar at the top so that the user can search for a project and check it out very easily and fast.
Additional context
I am contributor at gssoc'22.
Please assign this task to me.
Steps Followed✅
Check the box with [x] If you followed all the steps.

[x] Is your feature request related to a problem? Please describe.
[x] Describe the solution you'd like
[ ] Describe alternatives you've considered
[x] Additional context

Code of Conduct✅

[x] I agree to follow this project's Code of Conduct.
[x] I checked the current issues for duplicate problems.
[x] I have gone through project README.

We will take care of UI later on.

I'm Contributor in GSSoC22 . Can u assign this to me.

@Siddhesh777 Sorry about this but I have assigned the same issue to someone else. Feel free to work on any other issue


🐛 [Bug] Encountered bug for nn.LSTM with half dtype

Bug Description
When running the compiled LSTM model for half dtype with torch-tensorrt, I get this errors:
RuntimeError: Input and parameter tensors are not the same dtype, found input tensor with Float and parameter tensor with Half
Here is the test code:
import torch
import torch.nn as nn
import torch_tensorrt

class Model(nn.Module):
    def __init__(self):
        super().__init__()
        self.relu = nn.ReLU()
        self.lstm = nn.LSTM(800, 800)

    def forward(self, input_x):
        x = self.relu(input_x)
        x = self.lstm(x)[0]
        return x

model = Model().eval().half().cuda()
x = torch.randn(50, 50, 800).half().cuda()
script_model = torch.jit.trace(model, x)
trt_ts_model = torch_tensorrt.compile(script_model, ir="torchscript", inputs=[x], enabled_precisions=[torch.half], truncate_long_and_double=True)
res = trt_ts_model(x)

Expected behavior

Environment

Build information about Torch-TensorRT can be found by turning on debug messages

Torch-TensorRT Version (e.g. 1.0.0): v1.4.0
PyTorch Version (e.g. 1.0): 2.0
CPU Architecture:
OS (e.g., Linux): Linux
How you installed PyTorch (conda, pip, libtorch, source):
Build command you used (if compiling from source):
Are you using local sources or building from archives:
Python version:
CUDA version: 11.8
GPU models and configuration: A100
Any other relevant information:

Additional context

This PR might be helpful for type issues: https://github.com/pytorch/TensorRT/pull/2469
However, I would recommend using Dynamo path since which is being actively supported right now.
Thanks!

This PR might be helpful for type issues: #2469 However, I would recommend using Dynamo path since which is being actively supported right now. Thanks!

Thanks for your reply! It seems that this PR do not fix the issues and the bug is stall active.


add links to geo data formatters

Summary of changes
Just to restore AdminUI functionality.
Checklist

[ ] Link to issue this PR refers to:
[ ] Relevant changes are reflected in CHANGES.md.
[ ] Added or changed code is covered by tests.
[ ] Required Grand Central APIs are already merged.

done in https://github.com/crate/crate-gc-admin/pull/169


Add an environment variable flag for -g

Fixes: #100

Any idea on when this patch will land in upstream Docker CE packages?

@Yajo I don't, no. I think you'd want to open an issue in https://github.com/moby/moby for this!


Support parsing a new HomePage carousel item type

This PR also unifies the duplicated parsing logic, optimizes away some unnecessary array creation from using a side-effecting map(), etc.

Thanks for PR!


Faults, Resolutions etc. getting deleted, when Bill-to Customer No. is changed in Service Order

Hello,
when changing Bill-to Customer No. in a Service Order and underlying Service Lines don't start with Line No. 10000, all Faults, Resolutions, Comments, etc. are getting lost.
Version: DE Business Central 19.5 (Platform 19.0.36528.36625 + Application 19.5.36567.36700)
Steps to reproduce the problem:

Search for Services Orders

Click on New

Enter 30000 in Field Customer No.

Under „Lines“ enter 10 in Service Item No.

Click on Line -> Comments -> Faults

Enter some Text in Faults. One line is enough fort he test.

Close Fault Service Comment Sheet

Click on Line -> Service Item Worksheet

Under Lines create an new Line: Type Item, No. 80203

In the upcoming Message choose „Ignore“ and accept with OK.

Create a second Line: Type Item, No. 80204

In the upcoming Message choose „Ignore“ and accept with OK.

Delete the first Line with type Item and No. 80203

Close Service Item Worksheet

Under „Invoicing“ enter 40000 in Field „Bill-to Customer No.“
a.	„Do you want to change the Bill-to Customer No.?“ -> Click yes
b.	„Do you want to update the lines?“ -> Click yes
c.	„If you change Bill-to Customer No., the existing service lines will be deleted and the program will create new service lines based on the new information on the header.
Do you want to change the Bill-to Customer No.?“ -> Click yes

Click on Line -> Comments -> Faults

The comments created in Step 6 are deleted.

The Problem lies in procedure RecreateServLines(ChangedFieldName: Text[100]) of Table Sales Header. All Service Comments getting transfered to a temporary table and deleted afterwards.

In procedure CreateServiceLines(var TempServLine: Record "Service Line" temporary; var ExtendedTextAdded: Boolean; var TempServiceCommentLine: Record "Service Comment Line" temporary) these comments should get recreated but won't when no Service Line with Line No. 10000 ist existent.

Service comments are tied to a Service Item Line and not Service Line, so saving and recreating comments in RecreateServLines is not necessary at all.

Thank you for reaching out. Your request could be reporting a product bug not related to the AL application extensibility. Please reach out to the relevant support channels:

For collaboration in preview or beta versions of AL Language in Visual Studio Code (aka ModernDev), use GitHub (https://github.com/microsoft/al).
For Microsoft partners, for versions already in general availability and supported, file requests through Microsoft Partnersource (https://mbs.microsoft.com/partnersource/).
For ISVs/VARs who have discovered a bug in a in preview or beta version, use MS Collaborate (https://docs.microsoft.com/en-us/collaborate/).
For Business Central customers who have issues or found bugs, please contact your Microsoft partner.
For the new features please submit a product suggestion on https://aka.ms/bcideas


Gradereval

Added updateThesisWithEthesis to enable updating single thesis from ThesisShow.

Coverage decreased (-0.1%) to 60.826% when pulling 5129da932bc664bcdb1e87b7b1b262974ceaea75 on gradereval into e93c7d7551cdb87431a18f7b4c7fdb3b57ada2ff on master.


Audio not working

This is my configuration and I can't listen to anything, even I tried echo enabled. I am using Mirror 93.0.1. I also have provided the Mirror Provider inside the gameobject (along with network identity)

thank you @Metater for you support in discord, problem is solved. For those who are experiencing the same issue, make sure you don't use it in scene (as I considered it like Game Manager instance) , rather use it with your Network Player

It should look like this

It is not required to separate "input" and "output" btw. They can be placed under "voice".
When you are using a player model, just place the "voice_output" game object where your player's mouth is in the world.


Support passing existing connections

The initialization locks the context to a specific engine. See example
We'd like to allow users to connect to an engine directly by reference (similar to SQL alchemy).

Another great example of this use case (this was solved via the Duckdb team)

I'm thinking something like this:
from sqlalchemy import create_engine

engine = create_engine("...")

Then:
%sql $engine

IPython supports variable expansion via $var, but I'm unsure if this will pass the actual object or only its string representation.

It's a bit unclear to me the original issue in the ipython-sql repo. I don't understand if the user wants to re-use an existing engine (what I described in the comment above). Or they want to re-use an engine they created with %sql for later queries. Maybe we can ask him and bring the conversation here?

I think that's the first option.
@machow thoughts?

I think this one will also be solved once an engine can be passed dynamically?  @zzFluke

This is still a nice and helpful feature, even if DuckDB can already read dataframes without it! One use case is if you store your dataframes in a dict, you would want to be able to register them with DuckDB (since it won't find them djrectly anymore).

@Alex-Monahan thanks for the feedback! Do you think it's related to this issue or a completely new one?

The ability to either pass in an engine or refer to the engine variable and call a function directly on the connection would work in this case. I think that is what this proposes, but I am not certain!

Awesome, yes it is!

hey @Alex-Monahan, @machow, this has been fixed!
Available in version 0.5.1. Docs here.
Let us know if you have any issues/feedback!

fyi: @pfmoore @gwerbin @rehno-lindeque @ellisonbg @ckald


update test

SUMMARY:
"please provide a brief summary"
TEST PLAN:
"please outline how the changes were tested"

Seems to be another issue at play
ValueError: Trying to set a tensor of shape torch.Size([64, 1]) in "weight_scale" (which has shape torch.Size([64, 0])), this looks incorrect.

Yup I was trying to figure out why this is showing up now. This PR fixes this. While doing that: https://github.com/neuralmagic/compressed-tensors/pull/178  I decided to also enhance test coverage, hence this PR.


Product Template

The product template is currently calling 'template-product' which doesn't exist in the dist files but does in the src files!

@boyoon thanks for reporting this. Likely a left over from the pre-Slate version. Thanks for reporting! 👍

Somehow the dist folder got add to the gitignore


FAILURE: cmake/build/Debug/h2_proxy_test.exe

Timeout
D1011 23:25:40.512000000  6152 test_config.c:388] test slowdown factor: sanitizer=1, fixture=1, poller=1, total=1
D1011 23:25:40.512000000  6152 dns_resolver.cc:306] Using native dns resolver
I1011 23:25:40.512000000  6152 high_initial_seqno.c:42] Running test: test_invoke_requests first_seqno=16777213/chttp2/fullstack+proxy
D1011 23:25:42.950000000  6152 proxy.c:89] PROXY ADDR:localhost:20718 BACKEND:localhost:19691

https://sponge.corp.google.com/target?id=5303f474-1c25-487d-81c6-5f801a863693&target=github/grpc/c_windows_dbg_native&searchFor=&show=FAILED&sortBy=STATUS
seen in PR: https://github.com/grpc/grpc/pull/13031, this is currently failing on windows master (e.g. on this kokoro windows master test run: https://sponge.corp.google.com/invocation?id=90dedf46-ac15-4f51-93d8-42766aa3d461&searchFor=)
looks like this was failing in older runs (https://github.com/grpc/grpc/issues?q=is%3Aissue+high_initial_seqno+is%3Aclosed), but the Timeout is new

I think this is the same as https://github.com/grpc/grpc/issues/13079 (already fixed).


Wrong Scissor Mode and broken FrameEvents

I believe the old EnableScissor(...) work more correctly than current one.

As you can see, the first screenshot is from commit 08a68d3c2d0ecac060c2a341232ddb901aecd4dc the second one is from 6ed53c713c453c9398428154b993ef99b9145920. It looks the old one works for me.
Currently I have daul monitor setup, both of them are 1080P and DPI is 125%.
Besides, I believe at line 446, it should be
                io.AddKeyEvent(keyItr.Value, Raylib.IsKeyDown(keyItr.Key));

instead of current one. Current one will crash directly.

The key event code was removed, it's not needed.
For your scissor issue, do you init your window in high DPI mode?

The key event code was removed, it's not needed.

That's great!

For your scissor issue, do you init your window in high DPI mode?

The screenshot is capture from the simeple-example project, with no modification of init code.
I can confirm that Task Manager reports DPI Awareness is Per-Monitor (v2)

does it work if you remove the scale from the computation?

does it work if you remove the scale from the computation?

Currently I have no idea about where scale is involved in the computation, however, I can confirm that commit 5c1d75dd188ff02cfa1830704453508612a8adb5 is working correctly on my machine where 0b2812fd8a4ac5a25f3dd3df41c442e56a933d60 and 6ed53c713c453c9398428154b993ef99b9145920 are not.

the scale is the multiplication of the sizes by io.DisplayFramebufferScale
I have tried a similar setup to yours and do not have an issue at 125% scale.
Please try the following.

Set a breakpoint in the scissor function and see if the display size that is sent to the scissor  is in real pixels or scaled pixels?
remove the multiplication by io.DisplayFramebufferScale and see if that changes anything.

that change was to fix another reported bug, so I don't just want to blindly revert it.

I believe that's the problem. For first call, Width and Height passed to the function are 1280 and 800 that matches pixel size I grabbed from screenshot, also, io.DisplayFramebufferScale reports correct scale value that is 1.25. By removing multiplication of io.DisplayFramebufferScale, scissoring works fine on machine. I am not sure what changed about DPI processing in Raylib side, but I can also confirm that by calling Raylib.DrawRectangle(0,0,1280,800,Color.WHITE);, the white rectangle will cover the whole window.
Besides, some system information might be useful here:
OS: Windows 11 Build 22631.2861
GPU: Nvidia RTX 4060
Monitor: 1080P at 125% DPI
Hope information above is useful for the problem.

I have updated the the library with a change that I think will fix this issue. It no longer applies the DPI scale for non-high DPI display mode. This seems to have resolved the issue for me (I was finally able to duplicate it in the C++ version).
Please get latest code and tell me if it fixes the issue. If so then I'll publish an updated nuget package with the fix.

Had the exact same issue with two monitors 175% and 125%. The latest code fixed the scissor issue, but there is still the rendering issue as can be seen in the screenshot.

I can confirm that I am experiencing same behavior as @tibimunteanu describes.
However, I believe this behavior was caused by line 346 at rlImGui.cs. When the window is maximized, I think we should still use Raylib.GetScreenWidth() and Raylib.GetScreenHeight(). Removing the condition || Raylib.IsWindowMaximized() do fix this problem.


fix: create copy of labels for label counts

For counting the labels, create a copy of the labels object as the original object can change during the calculation

Codecov Report

Merging #257 (b806452) into main (4c1316e) will decrease coverage by 0.00%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##             main     #257      +/-   ##
==========================================
- Coverage   81.91%   81.91%   -0.01%
==========================================
  Files          56       56
  Lines        3738     3737       -1
==========================================
- Hits         3062     3061       -1
  Misses        676      676

Impacted Files
Coverage Δ

...h/data_access/file_based/file_based_data_access.py
91.89% <75.00%> (-0.04%)
:arrow_down:

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


ISPN-14236 Benchmark via CLI using the resp protocol

https://issues.redhat.com/browse/ISPN-14236

Integrated into main, thanks @tristantarrant !


In-browser audio encoding profiling

Environment
OS: Ubuntu 17.10 artful
Kernel: x86_64 Linux 4.13.0-45-generic
Packages: 2781
CPU: Intel Core i5-5200U @ 4x 2.7GHz [49.0°C]
GPU: Mesa DRI Intel(R) HD Graphics 5500 (Broadwell GT2)
RAM: 2068MiB / 7877MiB

Chrome Version 66.0.3359.181
The system load average is 0.15 0.21 0.27 as reported by htop, before the tests were run.
Set up

Only Chrome was running in the foreground
CPU and memory profiles collected by Chrome Developer Tool
Test code is available at ztl8702/browser-encoding-perf-test, basically the same codebase as in https://github.com/ztl8702/jitsi-meet/tree/3da6b1bd3ba19987d4090d6176212ee3bb96b1af/react/features/local-recording/recording, but with no UI to start/stop the recording. Instead a function is called in the console to start recording for a certain period and then automatically stops the recording.
I used a phone to constantly play audio close to the built-in microphone of the laptop running the tests
Three scenarios, all end-to-end, recording & encoding real audio for a certain period of time using the browser audio APIs.

Ogg - Browser's built-in codec
Wav - WAVE "encoding" (or packaging): runs in the main thread, the most computationally intensive operation is properly copying PCM bits in memory
Flac - FLAC encoding using libflac.js: runs in a WebWorker. The callback function on the main thread receives the PCM bits from Browser API and pass them as a message to the WebWorker, which does the encoding

Results

Chrome profile dumps and the encoded audio files can be downloaded here.
Each scenario was run for 30 seconds and 60 seconds respectively. (There wasn't enough buffer space for Chrome to record profiling data longer)
A comparison test is run by profiling a Jitsi Meet conference (with audio & video on, and 2 participants) on the same machine.

ogg


CloudFront managed Cache and Origin policies

Just like there are aws.iam.ManagedPolicy.AmazonS3ReadOnlyAccess enums, it would be useful to have accessors for AWS CloudFront managed Cache and Origin policies.
The current list can be found with the following commands on aws cli:
aws cloudfront list-cache-policies --type managed
aws cloudfront list-origin-request-policies --type managed

Thanks @aliismayilov  for that suggestion!


Create project

Create a form to add new projects with all possible initial data.

Duplicate of #16


The wallet keep saying loading block index after downloading the blochain and fallowing the instructions :(

It takes a while, wait some more.

Yeah it takes a while. Threw me off at first too. I'd love to improve the UX and add better loading progress indicators. Your synchronizing with the blockchain will likely take quite a while as well.

@yona5324 generally means your internet connection isnt the best, it'll take a while.. sometimes 40mins to an hour

oke, so lets wait and see if this is the case! Thank you for the comments guys!

The problem with this old style QT ui is the lack of feedback, the splash screen disappears on my mac and the Activity Monitor reports the app as "not responding" which is not true .. I think the qt event loop doesn't run while indexing.. anyway it takes a bit also to get the first network connection but then it will complete the syncing; just wait :)

it's recommended to fully sync the chain yourself. it only takes about 1 day (vs bitcoin core taking about 9 days)

Oke after almost one hour still the same results debug file still trying to connect.


Hi, sorry, total newb here, I'd go to the forums rather than posting an issue if I knew where to find them... anyway, I'm running ubuntu, I've been experimenting with esp-open-sdk for some days now, and I just discovered this esp-open-rtos repo today.
I can (apparently) build and flash the example projects successfully, in both open-sdk and open-rtos, however my problem is that when I attempt to connect to the chip with my USB UART cable via any terminal application (I have tried putty and screen so far) I get nothing but a stream of garbage characters. I've tried many different baud rates, mostly 115200 but also every other one I've seen anyone mention in all my googling. I can communicate with an unflashed factory stock chip at 115200, but anything I have flashed onto the chip results in constantly blinking blue LED and constant stream of garbage to my terminal.
Am I missing some totally obvious step? It is possible that my test chip is fried, I have more chips coming in the mail any day now, but my tester does seem to behave fine when it comes to flashing at least. Any advice would be welcome! Thanks for all of your hard work.

Hello,
you can try calling uart_set_baud or running serial_echo example.

AH, thank you so much! I don't know what my problem is, but serial_echo worked!!! We got boots on the ground.


Update README examples

Description

Adds my Custom Join Messages to the examples list.
Updates the test amounts of existing plugins.

Checklist
The following items should be checked before the pull request can be merged.

[x] Code follows existing code format.

Info on creating a pull request

Make sure that unit tests are added which test the relevant changes.
Make sure that the changes follow the existing code format.

For maintainer
When a PR is approved, the maintainer should label this PR with release/*.

If the PR fixes a bug in MockBukkit, update the patch version. (if the current version is 0.4.1, the new version should be 0.4.2)
If the PR adds a new feature to MockBukkit, update minor version. (if the current version is 0.4.1, the new version should be 0.5.0)

Note that a PR that fixes an UnimplementedOperationException should be considered a new version and not a bugfix.

We should probably readd JacksonChen666's example


gnss launch :  send data ubx file via udp, no data out

System information
**Linux Ubuntu 18.04
**Apollo installed from source
**Apollo version master
###Question
I m trying to test localization module with some data in input , I made changes in modules/drivers/gnss in order to receive data via udp in UBX_BINARY format
I receive my data in the gnss udp server. But the data can't achieve to ubx_parser and by cyber_monitor, the topic /apollo/sensor/gnss/stream_status is published with the following message :

Ins_stream_type is still DISCONNECTED.
==>
The data is sent to 4444 port and the new conf file is :

==> The log file is :

*The server keep waiting to receive data, is there any other changes i should do or any explications?
Thank you

Are you now receiving data via udp and can't parse it?
If you think the problem is "Ins_stream_type is still DISCONNECTED." then you can detect this state in your code first

@daohu527 thank you for your answer.
Yes I am now receiving data via udp but the process is interrupted  with an exception right after updating data_parser in DataParser::parseRawdata. cyber_launch unable to read the binary packet.

There is already an error message, you can solve this problem first!
invalid start type


Coalesce k8sEvent experiment

Shouldn't you add some comments? You can probably crib most of the comments from the snapshot coalescing code.


MAINT Fix CI linter check and fix linting issues

all pre-commit hooks are always skipped on the CI. I guess it's because by default pre-commit only runs on staged changes but in the CI there are no staged changes. I propose to always run on all files (it's fast).
I added new rules for flake8 to ignore imports not at top of file for examples and confs.

commit 1feaefb left 1 linting issue and the CI linter fails.

commit 90cfaef and commit 88cf1db fixed the last linting issues and no the CI linter is green !

Note that although mypy is listed in the dev dependencies and has some config flags in setup.cfg, it's not run by pre-commit and not checked in the CI. I'm not adding that in this PR because there's a huge pile of errors to fix :)


The image for the right angled triangle has wrong labellings (specifically base)

Describe the bug

A clear and concise description of what the bug is.

Possible solution

Describe the solution you thought of.

Screenshots

If applicable, add screenshots to help explain your problem.

i would like to correct the issue. Please assign it to me

Deadline 14 March,2021 (11:59 PM IST)