[T7][T15-C4]

Ready for review

@JamesHuangUC @mchen14 I have some general remarks:

Avoid using public data members unless essentially required.
Update your Unit testing to reflect your new code.
Be on top of your integration testing and modifying the tests to suite your task manager.
Create issues for all your user stories and assign people to each one of them. This will allow us to easily see the amount of work each person has done. We will also look into the quality of code alongwith with the significance of the code within the complete project.
You can attach labels to prioritize your issues and you can add more issues for a complex user story once you start working on it.
Please mark your v0.2 of your project using labels before coming to lab this Thursday.


Update protagonist to 2.0.0-pre.4

This release is not needed due to caret dependency on Protagonist.


Feature: add clusterimage with kubevela

What is the problem this feature will solve?
add clusterimage with kubevela
If you have solution，please describe it
Fixed by: https://github.com/labring/cluster-image/pulls
What alternatives have you considered?
No response

Fixed by: https://github.com/labring/cluster-image/pulls


1.19.4 Crashes when bees headbutt. Cannot load worlds without mod after removal.

Here is my latest log and crash report.
latest.log
crash-2023-06-14_16.04.25-server.txt

Update: Not being able to load without the mod was a Prism (client) issue. The headbutt attempt then crash still occurs.

Sorry that this took so long to address, I've pushed an update to the 1.19.4 version that should fix the issue. Let me know if you run into it again.


Systemd: include network.target in After=

On a system where the service bridge is configured by NetworkManager albatross fails to restore VMs after a reboot due to the bridge not being up:
Jan 03 15:02:20 solsort albatrossd[1877]: albatrossd: [ERROR] failed to create [vm: stub-resolver]: interface service does not exist

Likely, adding network.target to the list in After=... in the systemd service files will fix this. I have added network.target on the machine and will confirm whether this fixed the issue (after I have done some upgrades).

Thanks for your report -- is this something we should add to the systemd file?

Yes, I think that would be a good idea.
For what it's worth it was not sufficient with my setup (bridge managed by NM) and I have added NetworkManager-wait-online.service as well.

would you mind to push a PR? maybe putting the NetworkManager-wait-online as well (and describe in the README that if one doesn't use network manager to comment that bit out?)


Generate run/measurement CSVs for Mixtral T3k demo test

Added benchmark support to perf test.
Missing demo_with_prefill.py now.

Merged


Changes in subject not shown clear enough

If for example a class is split in Latin and French and only the French part of the class actually has a substitution, it appears like the whole class has a substitution. Fortunately, we're not lacking data to detect such a situation - we can use the (so far unused) data field course_long (defined in drawing.js:175).
It's formatted the following way: "SUBJECT_CLASS_TEACHER" - We only need to parse this and check for differences. If there's a difference, we should add some indication about this change.

fixed with https://github.com/Antricks/substitution-table/commit/ce729c2466221aba90a79a2783204d6ee6615b59


”Bolintin-Deal” is listed in the drop-down menu

Description:
"Bolintin-Deal" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "Bolintin-Deal" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Bolintin-Deal, GIURGIU".


Allowed to delete Gobal DNS provider when they are attached to Global DNS entries.

Rancher server version - Build from master 01/22
Steps to reproduce the problem:
Create a Global DNS Provider .
Add a Global DNS entry which uses this Global DNS Provider.
Delete Global DNS Provider .
Global DNS Provider gets deleted successfully.
Expected Behavior:
Global DNS Provider should not be deleted since there are Global DNS entries associated with it.

Tested with latest build from master:
We are not allowed to delete Gobal DNS provider when they are attached to Global DNS entries.
User is presented with following error messages:
Error deleting Cannot delete the provider until GlobalDNS entries referring it are removed


Performance Data and Intel variables added

Performance Data Added
Intel Var. added

images are not showing up in readme in branch. is that going to  be fixed when it goes live? @shreejam21

images are not showing up in readme in branch. is that going to be fixed when it goes live? @shreejam21

It will show once its merged- the img sources use the main branch -> which has no images yet. Once merged it should show up


Monitor

Adds Prometheus ServiceMonitor
Delegates as many defaults as possible to ZITADEL
Uses ZITADEL v2-alpha.43 by default

IMO it would also make sense to include the possibility to configure TLS and therefore also the scheme for the serviceMonitor.
# values.yaml
metrics:
  enabled: false
  serviceMonitor:
    enalbed: false
    tlsConfig: {}
    # Set a different scheme than http, default or when omitted is http
    # scheme: https

# templates/servicemonitor.yaml
spec:
  endpoints:
  - interval: "{{ .Values.metrics.serviceMonitor.interval }}"
    {{- if $serviceMonitor.scheme }}
    scheme: {{ $serviceMonitor.scheme }}
    {{- end }}
    {{- if $serviceMonitor.tlsConfig }}
    tlsConfig:
      {{ $serviceMonitor.tlsConfig | toYaml }}
    {{- end }}

@DoGab I added more config options, including TLS and HTTP scheme.


[Feature request] Please allow br.setNextRequest() in the Test section of a request

I have checked the following:

[X] I've searched existing issues and found nothing related to my issue.

Describe the feature you want to add
Hi,
It would be very nice to get bru.setNextRequest() in the "Tests" section of a request too. In the "Tests" section I check the correctness of the response. In my opinion this is the right part were to decide if the response is so bad, that it does not make sense to continue with the collection. As consequence I what to execute the bru.setNextRequest() to skip some requests or terminate the collection run at all.
As the current request is executed as all scripts (post response, tests) are finished I do not see any conceptional problem to set the next request in the test section too.
Kind Regards
Thomas
Mockups or Images of the feature

Hi @tho-gru-38 I have created a PR with the changes to allow the use of bru.setNextRequest() at the test scripts. Let's hope that the maintainers approve it.

Hello, just passing by looking for a feature, shouldn't this be closed ?

As this feature is implemented, it can be closed.


Update initializer to work with modules

The following lines must be added to the pom.xml to be able to run under the module system:
            <plugin>
                <artifactId>maven-compiler-plugin</artifactId>
                <version>3.8.1</version>
            </plugin>

We could also add comments in the Main file:
// If you want to run under the Java Platform Module System (JPMS), add a
// "requires com.speedment.runtime.application" in your "module-info.java" file

Should the module system be default or should it be an option in the initializer to run without the module system?

Good question. The best thing would be a radio button with "Use Java Module System" and then we could generate a template module-info.java file and add the maven-compiler-plugin in the pom.xml.
Second best (and presumably easier): Always adding the maven-compiler-plugin would be ok (does not harm if JPMS is NOT used) and having comments as proposed could guide the user to implement her own module-info.java file.

Having a check box would highlight that Speedment is JPMS capable.

Here is the new logic for the pom.xml file: Unconditionally add the following section under <build><plugins>:
            <plugin>
                <artifactId>maven-compiler-plugin</artifactId>
                <version>3.8.1</version>
            </plugin>

If the module system is selected, a new file module-info.java shall be added in the directory src/main/java whereby the contents shall look like this:
module $ArtifactId {
    requires com.speedment.runtime.application;
    requires com.speedment.enterprise.datastore.runtime; // If $DataStore.enable
    requires com.speedment.enterprise.plugins.enums; // if $Plugins.enums
    requires com.speedment.enterprise.plugins.json; // if$Plugins.JSON

    // If $Plugins.spring (all of the below)
    requires com.speedment.enterprise.plugins.spring.runtime;
    requires spring.boot;
    requires spring.web;

    requires java.annotation;
    requires spring.webmvc;
    requires spring.beans;
    requires spring.context;
    requires spring.core;
    requires spring.boot.autoconfigure;
}

Where:
$ArtifactId is the specified "ArtifactId" in the initializer
$DataStore.enable is the specified check box in the initializer
etc.

all the database modules and join have been broken out as separate modules now. So the module-info should look like this:
module mymodule {
    requires com.speedment.runtime.application;
    requires com.speedment.runtime.connector.mysql; // Depending on which DB was selected
    requires com.speedment.runtime.join; // Always add this. Could be removed if joins are not used
}

The generator automatically includes all open db:s so the POM remains the same.
The Main file needs to be expanded:
.withBundle(JoinBundle.class)

I've updated the initializer and the generator to accomodate the new module system. It is ready for launch as soon as the JPMS feature is completed.


[6.0] mask out tests that are failing on Linux

Explanation: Wraps a handful of tests in #if os(macOS) blocks so that they only execute on macOS.
Scope: Test-only change blocking CI on Linux
Issue: None
Original PR: https://github.com/swiftlang/swift-docc/pull/996
Risk: Very low. This is a test-only change that changes the platforms that some tests run on. The tests are still executed in CI.
Testing: Tests still pass, CI is passing now.
Reviewer: @patshaughnessy @mayaepps

@swift-ci Please test


Fixup after 0.5.0 release

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/cannoli-prb/167/
Test PASSed.


packaging for andriod causes errors packiging fails. please try to fix soon as we need our app working for our school project

UATHelper: Packaging (Android (ASTC)): C:/Users/julia/Documents/Unreal Projects/wellbeing_app/Plugins/OpenAI-Api-Unreal/Source/OpenAIAPI/Private/OpenAICallGPT.cpp(117,3): error: misleading indentation; statement is not part of the previous 'if' [-Werror,-Wmisleading-indentation]
UATHelper: Packaging (Android (ASTC)):                 _payloadObject->SetNumberField(TEXT("presence_penalty"), FMath::Clamp(settings.presencePenalty, 0.0f, 1.0f));
UATHelper: Packaging (Android (ASTC)):                 ^
UATHelper: Packaging (Android (ASTC)): C:/Users/julia/Documents/Unreal Projects/wellbeing_app/Plugins/OpenAI-Api-Unreal/Source/OpenAIAPI/Private/OpenAICallGPT.cpp(115,2): note: previous statement is here
UATHelper: Packaging (Android (ASTC)):         if (!(settings.logprobs == 0))
UATHelper: Packaging (Android (ASTC)):         ^
UATHelper: Packaging (Android (ASTC)): 1 error generated.
UATHelper: Packaging (Android (ASTC)): Took 9.8692925s to run dotnet.exe, ExitCode=6
UATHelper: Packaging (Android (ASTC)): UnrealBuildTool failed. See log for more details. (C:\Users\julia\AppData\Roaming\Unreal Engine\AutomationTool\Logs\C+Program+Files+Epic+Games+UE_5.1\UBT-wellbeing_app-Android-Development.txt)
PackagingResults: Error: misleading indentation; statement is not part of the previous 'if' [-Werror,-Wmisleading-indentation]
UATHelper: Packaging (Android (ASTC)): AutomationTool executed for 0h 0m 13s
UATHelper: Packaging (Android (ASTC)): AutomationTool exiting with ExitCode=6 (6)
UATHelper: Packaging (Android (ASTC)): BUILD FAILED
PackagingResults: Error: Unknown Error

same siuation


Using Apostrophs in top-level definitions in module not working

Description
When using an apostroph in a name in a top level definition the compiled javascript the compiled module unuseable as the compiled javascript is syntactically invalid
Repro steps

Define module in F#

module Bar
let bar x = ...
let baz y = ...

let foo' z = 1

fable

Expected behavior
Export of bar, bazand foo'
Actual behavior
Javascript along this lines will be generated
exports.foo' = exports.bar = exports.baz = undefined;

//real definition ...

exports.foo' = foo_;
//# sourceMappingURL=Matrix.js.map

Known workarounds
Dont use apostrophs in names
Related information

Branch 0.3.22

Unfortunately, because of the way exports work in ES6 (they must be valid JS names) this can't be fixed. I've just added a check so an error is thrown a compile time if an invalid JS name is being used for a root member. I hope this is not much of an inconvenience.

If I understand this correct?
export["some weird' identfier"] = function () {...}
won't work, correct?
Then what about name mangling? so that
let foo' = 1
let bar'' = 2

becomes
var foo$apostroph =1;
var bar$apostroph$apostroph = 2;

and consequently then on the caller site as well.

We can consider this (something similar is already being done for overloads), but one of Fable's goals is to make it easy to create modules that can be easily consumed from JS and this can complicate things in this regard. Please consider the problem doesn't affect only apostrophes but also any character not allowed in JS variable names.

How about any UTF-16 code point other than [A-Za-z0-9_] gets converted to "$uxxxx", where xxxx is the code point's hexadecimal representation, similar to the "\uxxxx" encoding of JSON?  For example,
let ``My name$ is Earl`` = 17

becomes
var My$u0020name$u0024is$0020Earl = 17;

It is ugly but completely predictable, and reserves the ability for other kinds of escaping via $ by using a character other than 'u' after the $.  Something like this is a necessary price for F#'s greater flexibility with variable names.

Since there seems to be interest on this, I'm reopening the issue and changing the tag to discussion to know the opinion of others. I think the dilemma here is:

Allow developers to use any name in public members of root modules even if these names won't be (easily) callable from JS.
Force developers to use valid JS names for public members of root modules and warn them at compile time otherwise.

a) yup its ugly ;-)
b) I am completely for it. The reason being that using apostroph in F# is pretty common. IF one really wants to write libraries to be consumed from pure JS he/she should refrain from using non standard identifiers. On the other hand if one writes a Fable lib (to be consumed by Fable itself) then it should be possible to just that. Just btw. I think that Purescript has a pretty similar philosophy on this issue.

@cosinekitty  F# is UTF-8
The following character classes are vaild for identifiers

Letter, Uppercase \Lu
Letter, Lowercase \Li
Letter, Titlecase \Lt
Letter, Modifier \Lm
Letter, Other \Lo
Number, Letter \Nl
Punctuation, Connector \Pc
Mark, Nonspacing \Mn
Mark, Spacing Combining \Mc
Other, Format \Cf

This was a bit more complicated than expected but it should be fixed now, so we cannot only use apostrophes but also write function names in Japanese ;)
It was complicated because it only affected root members in the final JS file (not nested ones), including function, class and module names. I just hope I managed to cover all cases. I've followed @cosinekitty's suggestion and replaced JS non-valid ident chars with the unicode point (in hexadecimal to make it shorter) prefixed by $.
I'll publish shortly a new version with the fix :+1:

@alfonsogarciacaro I added a couple of comments on lines in the commit. Not sure you receive notification for those.


ClojureScript :require-macros runs ns tests inside shadow compiler jvm

This can cause tests related to clj macroexpansion to fail wrongly, not sure why

This can cause tests related to clj macroexpansion to fail wrongly, not sure why


Nested Optionals get ignored by the LLM

[x] This is actually a bug report.
[x] I am not getting good LLM Results
[x] I have tried asking for help in the community on discord or discussions and have not received a response (I did get one, just not one that made much sense against my further investigation).
[x] I have tried searching the documentation and have not found an answer.

What Model are you using?

[ ] gpt-3.5-turbo
[ ] gpt-4-turbo
[x] gpt-4
[ ] Other (please specify)

Describe the bug
When a pydantic model has a property whose type is Union[ModelA, ModelB], any properties typed Optional within either ModelA or ModelB are ignored by the LLM.
To Reproduce
class InterventionRequest(BaseModel):
    """
    Use when you want the user to intervene.
    """
    request: str

class Tool(BaseModel):
    """
    Use this tool to get users from the database.
    """
    order1: Optional[str] = Field(
        default=None,
        description="The order to sort the users by. Should be in format <field>:<order>, e.g.: 'field1:asc'. Can only include one order. Leave blank to return all users in no particular order.",
    )
    filter1: Optional[str] = Field(
        default=None,
        description="Use this when you want to filter based on a value. Format <field>:<operator>:<value>. Can only include multiple filters (comma separated). Example: 'email:ct:test@test.com,score:gte:50'. Operators: eq (equals), neq (not equals), ct (contains), nct (does not contain), stw (starts with), edw (ends with). Leave blank when not filtering by field value.",
    )
    fields: Optional[str] = Field(
        default=None,
        description="Use this to set the fields that will be returned. Should be in format <field1>,<field2>,<field3>. Can only include multiple fields (comma separated). Example: 'email,score'. Leave blank if you want to return all fields.",
    )
    count: Optional[int] = Field(
        default=None,
        description="The number of users to return. Maximum is 10_000.",
    )

class ToolCall(BaseModel):
    choice: Union[InterventionRequest, Tool] = Field(
        description="The choice of what to use. You can either request an intervention by the user, or use the tool."
    )

Expected behavior
The Tool would simply be filled as expected whenever we pass ToolCall as a response_model. What happens instead is that all fields whose type is Optional are consistently ignored.
Extra notes
I have been able to demonstrate that the issue is not with prompting or complexity of the pydantic model. I did:

And got:

Obviously, this is not a huge issue, as there is a workaround. I just wish it wouldn't ignore the Optionals, as that would make the writing more pythonic.

Hmm I don't really get this problem tbh. I suspect either
(1) the prompt itself doesn't provide enough information or clear instructions
(2) The model is being asked to generate a rather complex schema. You can honestly use pydantic to do post processing if you need the data in a specific format if it helps.
from typing import Optional, Union
from pydantic import BaseModel
from openai import OpenAI
import instructor

client = instructor.from_openai(OpenAI())

class GoogleSearch(BaseModel):
    user_query: str
    start_date: Optional[str] = None
    end_date: Optional[str] = None
    email_filters: Optional[list[str]] = None

class GetCalendar(BaseModel):
    relevent_emails: list[str]
    start_date: Optional[str] = None
    end_date: Optional[str] = None

class ModelChoice(BaseModel):
    steps: list[Union[GoogleSearch, GetCalendar]]

def generate_plan(query: str):
    return client.chat.completions.create(
        model="gpt-4o-mini",
        response_model=ModelChoice,
        messages=[
            {
                "role": "system",
                "content": "You are a helpful assistant that plans queries well. Today is 2024-11-08.",
            },
            {
                "role": "user",
                "content": query,
            },
        ],
    )

print(generate_plan("Go fetch my calendar for the next 6 days thanks"))
#<EMAIL_ADDRESS>start_date='2024-11-08', end_date='2024-11-14')]
print(generate_plan("Fetch my emails from<EMAIL_ADDRESS>for the last 2 days"))
#<EMAIL_ADDRESS>start_date='2024-11-06', end_date='2024-11-08')]

Going to close this issue for now


Images to pre

Codecov Report

Merging #60 (d256e9f) into pre (02cdb7b) will decrease coverage by 2.87%.
The diff coverage is 43.92%.

@@            Coverage Diff             @@
##              pre      #60      +/-   ##
==========================================
- Coverage   71.47%   68.60%   -2.88%
==========================================
  Files          29       29
  Lines        2507     2768     +261
==========================================
+ Hits         1792     1899     +107
- Misses        715      869     +154

Impacted Files
Coverage Δ

lib/utils/wits_overflow_data.dart
70.37% <9.52%> (-4.44%)
:arrow_down:

lib/screens/question_and_answers_screen.dart
33.03% <10.52%> (-0.93%)
:arrow_down:

lib/screens/post_question_screen.dart
39.59% <26.31%> (-0.90%)
:arrow_down:

lib/forms/question_comment_form.dart
70.43% <40.00%> (-7.59%)
:arrow_down:

lib/forms/question_answer_comment_form.dart
85.48% <45.83%> (-9.52%)
:arrow_down:

lib/forms/question_answer_form.dart
80.34% <48.00%> (-8.91%)
:arrow_down:

lib/widgets/answers.dart
55.65% <48.07%> (-5.12%)
:arrow_down:

lib/widgets/question.dart
76.96% <56.66%> (-8.55%)
:arrow_down:

lib/widgets/comments.dart
81.10% <61.40%> (-16.20%)
:arrow_down:

Continue to review full report at Codecov.

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


no-incompatible-modifiers rule

Proposal: Create a rule that disallows certain combination of modifiers, such as
test.only.skip(...)
test.before.only(...)
test.before.skip(...) // ?
test.before.beforeEach()
test.before.after()
test.before.serial()
// others?

Value: Earlier feedback when linting in the editor
Let me know if you think that this is an unnecessary rule or not (it probably is, or very close to).

I would say unnecessary, same as https://github.com/sindresorhus/eslint-plugin-ava/pull/55#issuecomment-201120065. It should be a runtime error, not a linter one.
People are not likely to make these mistakes often (they make no sense), so there isn't a lot of value.

Agree these are pretty rare cases, just wanted to have your thoughts on this ;)


Sparse Index not building properly

Hello,
I have found a small error in your building of the sparse index, There is a small mistake in your _calc_idf funtion for sparse indexes.
In this function you are using an eps value to replace the negative idf values. However your code is a little bit faulty.

Firstly you are using r_freq to determine negative idf value, why not just use the idf value itself as a check?
your statement:

if r_freq > 0.5:
       continue

Is not correct because what is happening here is that if the r_freq > 0.5 (or idf is negative) it skips the rest of the code under the if statement including:
if idf < 0:
   negative_idfs.append(word)

So those words whose idf's are negative never appear in the idf dictionary. You can confirm this by checking your idf dictionary for a word which exists in more than 50% of the corpus.
I would suggest changing the _calc_idf function to:
    def _calc_idf(self, nd):
        """
        Calculates frequencies of terms in documents and in corpus.
        This algorithm sets a floor on the idf values to eps * average_idf
        """
        # collect idf sum to calculate an average idf for epsilon value
        idf_sum = 0
        # collect words with negative idf to set them a special epsilon value.
        # idf can be negative if word is contained in more than half of documents
        negative_idfs = []
        for word, freq in nd.items():
            idf = math.log(self.corpus_size - freq + 0.5) - math.log(freq + 0.5)
            #r_freq = float(freq) / self.corpus_size
            if idf < 0:
                negative_idfs.append(word)
                continue
            self.idf[word] = idf
            idf_sum += idf
        self.average_idf = idf_sum / len(self.idf)

        eps = self.epsilon * self.average_idf
        for word in negative_idfs:
            self.idf[word] = eps

This will ensure that the negative idf values are replaced with the value of epsilon and that they are present in the idf dictionary.

The reason I put the statement:
if r_freq > 0.5:
       continue

is because words with so high frequency across all the documents are almost always stopwords, noise and they don't carry any importance for the document, especially for question answering and fact-checking in a very large corpus. If you build the inverted index including these very high frequency words, there's also an efficiency problem, because you will have to calculate tfidf scores for a much larger number of documents on average. You can change the r_freq threshold depending on your data, and I think it's a good idea to refactor the code and pass it as a parameter instead of having it as a fixed value. I will make it a configurable parameter in the next version of the framework.

I completely agree that stopwords shouldnt be included in the index as it will bring the efficiency down.
However:

Arent you already removing the stopwords during preprocessing?

I have run some numbers for a corpus size of 10000.

As you can see where ever r_freq > 0.5 idf is always negative so infact you are not storing any negative idf values and they are never replaced by average epsilon value so that part of your code is in effect never being run.

Yes, for the given threshold of 0.5 the rest of the code never runs. That's why I'm planning to replace the hardcoded threshold value with a variable parameter that can be set as an argument when you intitialise the sparse retriever. For some small collections this threshold maybe should be omitted / set to 0 for best results.


Slider inside section with jQuery slideToggle

Hi, and first of all thanks a lot for all that work! Slick is really amazing.
I have a page with different sections which open and close with native jQuery slideToggle().
I use Slick inside one of these sections. I am surely missing something obvious, for there is one thing I am not able to fix with my current setup, and is the first slide being "unstyled" for the time it needs to switch to the second (slider is set to autoplay).
Also, this happens if you first open and close another section before the one with the slider in it. It does not happen always though, but kind of randomly.
I apologize for my bad english...
I have a fiddle here with this set

Check this fiddle
You can not set the slider section to display: none; at the start, slick will not init.
Trick is to stop autoplay when display is set to none.

Please refer to:
http://stackoverflow.com/questions/26677649/slick-slider-loaded-with-displaynone-creates-bad-initiation/26680322#26680322

@mazzaker , @simeydotme thanks a lot to you both!!
I was stuck on where to apply those methods, both solutions work great, I really appreciate your kind help and time spent...thanks again guys!!


'getMap' is not defined in GoogleMaps plugin, when using push notification plugins

Hi,
I need help. I have been getting this error and I was able to isolate this issue to occur when I am using Pushwoosh or OneSignal plugins. They happen to be push notification plugins.
How do I get past this issue:
'getMap' is not defined in GoogleMaps plugin.....plugins/cordova-plugin-googlemaps/www/googlemaps-cdv-plugin.js (133)
Thanks.

I forgot to add, this only occur in Android. Everything is fine with iOS.


Running test in shuffled order breaks JUnit output

Running CppUTest with shuffled order (-s option) to output JUnit XML test format results (-ojunit option) is currently broken: the XML file for each group will only show the last test result for that group.

This is related to a known issue (#168) with JUnit output that it doesn't work right when a group doesn't run all of its tests in order.  Unfortunately the fix for that issue isn't simple to implement, the concept of a TestGroup needs to be refactored to be an actual object type and not just be a string prefixed to the test name itself.

Ah, that explains it. Thank you.


Reset request body on retry

Fixes a bug where requests with a body (event) are not always retried due to the underlying net/http throwing *url.Error because of body and content-length mismatch (body is empty and not reset upon retry). See #773 for details.
The tests did not catch this as they were not doing behaving as a real HTTP server (just mocking RoundTripper) and also not carrying an event body (which would not trigger this behavior).

[x] Changed retry tests to use a network connection and HTTP (test) server
[x] Reset body (if not nil) during retries
[x] Set GetBody func only if it is not already set (the sdk already does this currently)

Closes: #773
Signed-off-by: Michael Gasch<EMAIL_ADDRESS>

Linter workflow seems broken.

Thank you so much @embano1 !! Both for the detailed bug report and the follow-up fix. I will cherry-pick this into the current release as a bug fix and we will get this out. 🙏 🙏 🙏


Gender chart colors

Description
I swapped out the order of the keys in our gender chart to get the right order of colors.
Checklist

[x] I have performed a self-review of my own code

Turns out this is an order of operations issue. The way we import the data has to flow with the order we import the keys. Nice work Kimberlee!


Initial PR for moving subnet creation out of Azure Common

PR still needs to merge in multi-cluster / WAF

This issue addresses -- https://github.com/microsoft/bedrock/issues/1110


Update credits

Full credits for this should go to Manon, rather then me as well.

OK, if you say so. Again, thanks for the inspiration! Any additional feedback on the library is welcome.


Point of Sale app never progresses past the QR screen

I followed the instructions in the README
1) git clone https://github.com/solana-labs/solana-pay.git
2) cd solana-pay/point-of-sale
3) yarn install
4) yarn start
5) http://localhost:1234?recipient=caPsEFqXeu4upjqorwqhhTjmnhtob9EBUsAx7HbXpqw&label=Your+Store+Name
6) Scan QR code with Phantom iOS wallet
7) Send SOL

The SOL gets sent, I can see the transaction just fine on my phantom wallet, however, the Point of Sale app never leaves the QR code page. I am assuming it is supposed to show a spinner or a confirmation page at this point.
Thank for you for any help and apologies if I misunderstood what that PoS app is supposed to do !

Closing this issue as I think this is more of a discord appropriate question.

Okay after wathcing some youtube videos, I think this is a real issue.

When running from the cloned repo on localhost, you're using devnet which means your Phantom wallet must also be configured to use devnet. Is it?

Nope, that was my problem. And it's right there in the README. I just skimmed over that as I already have a wallet 🤦🤦🤦🤦🤦🤦.
Apologies for posting this without giving the README a more thorough read through.
Thanks for the response, much appreciated. This is great.

No problem, glad it's working!


Zulip browser client doesn't work with Safari 13

https://github.com/zulip/zulip/blob/753deab0872c3113ad81f1e7577f390feab782da/static/js/realm_logo.js#L109
I am encountering a JavaScript error because MediaQueryList doesn't have the 'addEventListener' method on my browser (Safari 13). Perhaps we should try to detect whether the browser supports this and use 'addListener' otherwise.
This code seems to have been added in 7e883add5be47a0e320d26e89917cce9c2f1a2e4

https://caniuse.com/mdn-api_mediaquerylist_eventtarget_inheritance agrees.
Cc @aero31aero

I don't have a development environment set up, and doing so for a front-end-only change seems like overkill. Is there some way I can try out the change by only setting up the web frontend?


Configure for the inline snapshot indentation

Hi , thank you for the awesome tool!
Currently,cargo insta review generates inline snapshots in the form like:
assert_snapshot!(
    test.execute("select id, value from t1"),
    @r###"
+----+-------+
| id | value |
+----+-------+
| 2  | 102   |
+----+-------+
| 3  | 103   |
+----+-------+
| 4  | 104   |
+----+-------+

(3 rows)
"###
);

I wonder if it's possbile to make the following output with project-level configuration:
assert_snapshot!(
    test.execute("select id, value from t1"),
    @r###"
    +----+-------+
    | id | value |
    +----+-------+
    | 2  | 102   |
    +----+-------+
    | 3  | 103   |
    +----+-------+
    | 4  | 104   |
    +----+-------+

    (3 rows)
    "###
);

Maybe related to https://github.com/mitsuhiko/insta/issues/457

Thanks for the issue!
It should give reasonable indentation already. Could you post a reproducible example? The source can just be a literal string.

Starting from this test:
#[test]
fn test_foo() {
    assert_snapshot!(
        "aaa\nbbb\nccc\nddd",
        @""
    );
}

After review:
#[test]
fn test_foo() {
    assert_snapshot!(
        "aaa\nbbb\nccc\nddd",
        @r###"
    aaa
    bbb
    ccc
    ddd
    "###
    );
}

If starting from oneline:
#[test]
fn test_foo() {
    assert_snapshot!("aaa\nbbb\nccc\nddd", @"");
}

After review:
#[test]
fn test_foo() {
    assert_snapshot!("aaa\nbbb\nccc\nddd", @r###"
    aaa
    bbb
    ccc
    ddd
    "###);
}

I've a tool using ast-grep to replace @ to - to make cargo fmt works (https://github.com/mitsuhiko/insta/issues/499):
#[test]
fn test_foo() {
    assert_snapshot!("aaa\nbbb\nccc\nddd", -r###"
    aaa
    bbb
    ccc
    ddd
    "###);
}

cargo fmt:
#[test]
fn test_foo() {
    assert_snapshot!(
        "aaa\nbbb\nccc\nddd",
        -r###"
    aaa
    bbb
    ccc
    ddd
    "###
    );
}

replace - back to @:
#[test]
fn test_foo() {
    assert_snapshot!(
        "aaa\nbbb\nccc\nddd",
        @r###"
    aaa
    bbb
    ccc
    ddd
    "###
    );
}

Sorry for being slow — am I understanding correctly that the current output and the desired output have the same indentation? And the difference is the opening delimiter?
(Am on phone so possibly I'm missing something...)

Thank you for quick reply!
The desired output is
#[test]
fn test_foo() {
    assert_snapshot!(
        "aaa\nbbb\nccc\nddd",
        @r###"
        aaa
        bbb
        ccc
        ddd
        "###
    );
}

while I get
#[test]
fn test_foo() {
    assert_snapshot!(
        "aaa\nbbb\nccc\nddd",
        @r###"
    aaa
    bbb
    ccc
    ddd
    "###
    );
}

I think a couple of things here:

Specifically the indentation: this is selected at the same level as the macro. Which I think generally works OK? But open to alternatives.

Though I'd be very hesitant to add a configuration, we want to have a simple interface, so we should come up with a rule that works across most cases. Is there a better rule than the current one?

Formatting: I agree the formatting is occasionally not be ideal. cargo fmt works within macros iff the macro contents is valid rust, which @ isn't.

Ideally we could have syntax valid rust syntax, and then cargo fmt would work. Currently the difficulty is that we need to distinguish between assert_snapshot!(name, expr) and assert_snapshot!(expr, inline_value), and so we need something that isn't a valid rust expression. So I can't think of a way around it
Your ast-grep approach is clever!

Indeed. These maybe project-specific preference. The better way could be keeping the default behavior and format using ast-grep in project desiring different style.

so we need something that isn't a valid rust expression. So I can't think of a way around it

rustfmt is requiring a syntactical valid expression, but not semantically valid expression. Thus there is room to find such patterns. For example, my using -r###""### utilize the fact that unary minus to a string is valid in syntax but invalid in type. Symbols like *, !, ?(postfix) are also possible.

(great, will reply in #499)

To reply to:

Maybe further, in assert_snapshot!(expr, string_lit), we can always treat it as inline snap if the second argument is string literal because it's not likely assert_snapshot!(name, expr) has a simple string literal as expr (comparing a string lit with string lit).

I would vote to stay away from any ambiguities like this; confusion there would be very expensive...


Not working on Android 8.0 Oreo

Facebook login fails with "Sorry something went wrong" message
Google login just hangs after entering the password

With 8.0, for me in OAuthManagerModule.java
private WritableMap accessTokenResponse()
receives a OAuth2AccessToken that looks more like a payload from a URL separated by '&' and the GSON parser crashes on receiving it in about line 404, since the raw response is not JSON, and it would not contain the "user_id" even if it was JSON
Map accessTokenMap = new Gson().fromJson(accessToken.getRawResponse(), Map.class);

I'm seeing possibly the same issue on Android 6.0. Logcat:
10-16 10:30:37.892  3051  3051 E AndroidRuntime: FATAL EXCEPTION: main
10-16 10:30:37.892  3051  3051 E AndroidRuntime: Process: (snip), PID: 3051
10-16 10:30:37.892  3051  3051 E AndroidRuntime: com.google.gson.JsonSyntaxException: java.lang.IllegalStateException: Expected BEGIN_OBJECT but was STRING at line 1 column 1 path $
10-16 10:30:37.892  3051  3051 E AndroidRuntime: 	at com.google.gson.Gson.fromJson(Gson.java:900)
10-16 10:30:37.892  3051  3051 E AndroidRuntime: 	at com.google.gson.Gson.fromJson(Gson.java:853)
10-16 10:30:37.892  3051  3051 E AndroidRuntime: 	at com.google.gson.Gson.fromJson(Gson.java:802)
10-16 10:30:37.892  3051  3051 E AndroidRuntime: 	at com.google.gson.Gson.fromJson(Gson.java:774)
10-16 10:30:37.892  3051  3051 E AndroidRuntime: 	at io.fullstack.oauth.OAuthManagerModule.accessTokenResponse(OAuthManagerModule.java:403)
10-16 10:30:37.892  3051  3051 E AndroidRuntime: 	at io.fullstack.oauth.OAuthManagerModule.access$100(OAuthManagerModule.java:45)
10-16 10:30:37.892  3051  3051 E AndroidRuntime: 	at io.fullstack.oauth.OAuthManagerModule$1.onOAuth1AccessToken(OAuthManagerModule.java:128)
10-16 10:30:37.892  3051  3051 E AndroidRuntime: 	at io.fullstack.oauth.OAuthManagerFragmentController.loaded10aAccessToken(OAuthManagerFragmentController.java:147)

I don't think it's OS-specific, although I don't know what the issue actually is.

@stuartlangridge  Sorry for the insufficient detail, I was experiencing this on the iOS side.


Refused packet makes go_ora panicking

Company's security rules block direct connection to oracle server when the VPN is on.
The connect packet is refused, and the actual refused packet is 12 bytes long. This cause a panic in function newRefusePacketFromData when trying to get message beyond the 12th byte.
panic: runtime error: slice bounds out of range [:383] with capacity 16

goroutine 1 [running]:
github.com/sijms/go-ora/network.newRefusePacketFromData(0xc000012100, 0xc, 0x10, 0xc)
        C:/Users/me/go/src/github.com/sijms/go-ora/network/refuse_packet.go:44 +0x165
github.com/sijms/go-ora/network.(*Session).readPacket(0xc0000021e0, 0x0, 0x0, 0x0, 0xfd)
        C:/Users/me/go/src/github.com/sijms/go-ora/network/session.go:270 +0x10c
github.com/sijms/go-ora/network.(*Session).Connect(0xc0000021e0, 0xc0000021e0, 0x203000000000000)
        C:/Users/me/go/src/github.com/sijms/go-ora/network/session.go:76 +0x2a9
github.com/sijms/go-ora.(*Connection).Open(0xc000048000, 0x18, 0x0)
        C:/Users/me/go/src/github.com/sijms/go-ora/connection.go:232 +0x1a5
main.main()
        C:/Users/me/go/src/github.com/sijms/go-ora/examples/hello_ora/main.go:35 +0x10a

Here is a fix for not panicking and return an error.
I have added hello_ora program to test connection. I have put it into the subfolder examples/hello_ora.
hello_ora
        hello_ora check if it can connect to the given oracle server, then print server banner.

Usage:
        hello_ora oracle://user:pass@server/service_name

On successful connection:
go run . 'oracle://user:pass@server/service_name'

Successfully connected.

Oracle Database 11g Express Edition Release <IP_ADDRESS>.0 - 64bit Production
PL/SQL Release <IP_ADDRESS>.0 - Production
CORE    <IP_ADDRESS>.0      Production
TNS for Linux: Version <IP_ADDRESS>.0 - Production
NLSRTL Version <IP_ADDRESS>.0 - Production

On refused connection:
go run . 'oracle://user:pass@server/service_name'
Can't open connection: connection refused by the server

yes actually I didn't implement refuse package
thank you

now I need to fix redirect_packet also. but I cannot receive one to analyze
did you receive redirect packet before

When the VPN is on, I can't connect to a server on my local lan. What an intrusive VPN/FW!
Then the response to ConnectPacket is 0,12,0,0,4,0,0,0,1,114
I don't know the is packet is longer. The func readPacketData stops at the 12th byte, as indicated in the first word of the buffer.
Otherwise, when not connected to the vpn, the response to Connect is Accept and then Data.
May be I can configure my local server to get a Redirect, but how?

according to #23 they are using RAC to get redirect packet

I think reconnect data is send in DataPacket after RedirectPacket


How do I know if a user is already logged in when my app starts?

Hi
I don't want to force the user to log in each time my (Android) App starts.
Basically my App offers Facebook and Google Login the first time it starts.
Once the user logged in with Facebook or Google the app should remember this.
Next time the user starts my app he should be already logged in.
The react-native-facebook-login calls an onLoginFound callback which tells my app if the user is already logged in to my app.
How do I achieve the same with react-native-google-sign-in ?
@stami

@BerndWessels You may want to see https://github.com/joonhocho/react-native-google-sign-in/blob/master/js/GoogleSignIn.ios.js for all supported APIs.
See GoogleSignIn.android.jos as well if you are using Android.

What should i prefer, if I am not using google and facebook authentication. I am using my own database using login API. Can you please suggest how to maintain the session?


Fix undo button

#34
Before:
Menu is over hovering box, so that can not click undo button.

After:
Remove z-index from menu.

Great, thanks!

Hi!
It seems to be buggy again 😕:

Thanks!

Hi!
It seems to be buggy again 😕:

Thanks!

Agree: Undo Message Sent still doesn't work


PKGBUILD set build type to Release and strip exe/dll for Release build

PKGBUILD set build type to Release
PKGBUILD Update mingw-w64-x86_64-scopehal-apps to v0.1
For Release build strip exe/dll
Build success on my fork

Stripping exe/dll have improved a lot package at end with up to 10x size improvement
See https://github.com/bvernoux/scopehal-apps/actions/runs/3072804571

Issue for glscopeclient-windows-portable & glscopeclient-windows.msi

shaders directory => New shaders are missing (especialy the new one *.spv)

Todo add copy of shaders from scopehal-apps/build/lib/scopeprotocols/shaders

masks directory is missing

Todo add copy of masks from scopehal-apps/src/glscopeclient/masks

Shall be fixed by https://github.com/bvernoux/scopehal-apps/commit/ffd618ae4649ffa1c9b0b9d04681c5da10012aee

PR cancelled
We should find a way to strip the exe/dll but to keep the debug  symbol in external files in a portable way for all platforms with Cmake
See comment from Andrew on IRC:
<azonenberg> See here https://stackoverflow.com/questions/866721/how-to-generate-gcc-debug-symbol-outside-the-build-target
<azonenberg> this way we'd only need one type of binary
<azonenberg> and we package the .debug files separately


ESP CYD2USD

Hi
I have uploaded code to CYD 2 USB but its not working I have uloaded also this another one but screen is flipped and not workig good

we only support the USB C and Micro USB Variants of CYD
we do NOT support the CYD with Micro USB and USB C in one
Thanks

Are you planning create for this one too??


Make Eclipse support more resilient

It seem that the Eclipse plugin needs to be applied before the testset plugin. It would be helpful if that check whether Eclipse is applied could be deferred to a later stage in the execution and accept any order.
In our project we apply quite a number of plugins and it starts to become tricky to set them up in the proper order for everything to work.

Hey, @remmeier — In previous releases, I had some patching codes in my build to work around how source trees were getting built by the Eclipse plugin.  But I'm commenting because I've stopped using the Eclipse plugin entirely since Eclipse BuildShip became stable.
If you haven't tried BuildShip yet, I'd encourage you to give it a go.  If you haven't, though, may I ask why you're still using the Eclipse plugin?  I may be able to help, here.


Refactored plugin tree

Coverage remained the same at 78.189% when pulling 56a5a1250351aac0d413f09b3c2e8975047358b1 on czpython:feature/3.6.x/plugin-tree-refactor into<PHONE_NUMBER>0ee9f8c5c078bda8e5cfdd9a639f53 on divio:release/4.0.x.

@kinkerl
The plugin architecture will only have a single field for tree related stuff which is the position field.
We don't have an internal documentation for this at the moment.
There will be no replacement for fix-tree because positions are absolute.


setupegg.py develop won't invoke Cython

Seems to be working now

Need to find someone who knows more about setuptools and what's going on here. Not urgent (anyone running in develop mode should be able to run Cython themselves) but...

Wish I could help  . . . but I'm probably not that person. I'll push it out to a few Pythonistas I know tomorrow and see what turns up.
BTW, really dig the pandas package and am super-impressed with the development since inception way back on Google Code.
Looking at the open items for a place to contribute currently.

Fixed by PR #572


Constitution + jael updates

This PR contains two sets of changes.
First, it updates the zuse-constitution and jael logic to work more cleanly with the current version of the contracts. Aside from a few tweaks, I did a quick look-over of the existing logic, and it all still seems to match the contracts.
Secondly, it makes jael track continuity... sort of. When a ship indicates a continuity breach on-chain (or we've just never met it before), its live aka continuity flag will get set to |. It is then up to @joemfb's modified ames to send a %meet task to jael when we finally manage to connect with that ship again, setting the flag to &.
Changes to this flag get sent as part of the existing %pubs subscription, which has been updated to always send full public key state for a ship (now [live=? life=life pubs=(map life pass)]) rather than just the changes.
...That continuity flag is stored as part of jael's state-relative, even though it really is absolute state (ie can't be deduced from other parts of the state). It would be more correct to store a list of [ship life]s we've "met" alongside the Ethereum event logs, but that also feels a bit janky.
For now, I feel like this slight architectural mismatch isn't a big deal. This particular piece of state might need more work in the near future, I'd rather wait for the dust to settle before refactoring this.
I have lightly tested this and expect it to be correct based on that. No stress-testing was done though, but the logic here isn't too fragile.

I hope everyone realizes that this means —or will eventually mean!  — we never have a global breach again.
Sent from my iPhone

On Aug 29, 2018, at 5:24 PM, Fang<EMAIL_ADDRESS>wrote:
This PR contains two sets of changes.
First, it updates the zuse-constitution and jael logic to work more cleanly with the current version of the contracts. Aside from a few tweaks, I did a quick look-over of the existing logic, and it all still seems to match the contracts.
Secondly, it makes jael track continuity... sort of. When a ship indicates a continuity breach on-chain (or we've just never met it before), its live aka continuity flag will get set to |. It is then up to @joemfb's modified ames to send a %meet task to jael when we finally manage to connect with that ship again, setting the flag to &.
Changes to this flag get sent as part of the existing %pubs subscription, which has been updated to always send full public key state for a ship (now [live=? life=life pubs=(map life pass)]) rather than just the changes.
...That continuity flag is stored as part of jael's state-relative, even though it really is absolute state (ie can't be deduced from other parts of the state). It would be more correct to store a list of [ship life]s we've "met" alongside the Ethereum event logs, but that also feels a bit janky.
For now, I feel like this slight architectural mismatch isn't a big deal. This particular piece of state might need more work in the near future, I'd rather wait for the dust to settle before refactoring this.
I have lightly tested this and expect it to be correct based on that. No stress-testing was done though, but the logic here isn't too fragile.
You can view, comment on, or merge this pull request online at:
https://github.com/urbit/arvo/pull/795
Commit Summary
Update constitution and jael logic to work with latest contracts
Include continuity flag in relative public key state
Add %meet task to jael for indicating era connectivity
Misc. jael notes and improvements
File Changes
M sys/vane/jael.hoon (133)
M sys/zuse.hoon (35)
Patch Links:
https://github.com/urbit/arvo/pull/795.patch
https://github.com/urbit/arvo/pull/795.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Network breach headaches are dead. Long live node breach headaches!


Not compatible with @quasar/app v2.0.0

App · ⚠  Extension(@quasar/qwindow): Dependency not found - @quasar/quasar-app-extension-colorize. Please install it.
App · ⚠  Extension(@quasar/colorize): is not compatible with @quasar/app v2.0.0. Required version: ^1.0.0

@lyf1350 Thanks for making me aware of this. I'll try to get to this on the weekend.

Today with the latest quasar version the compatibility problem still exists as it was described by @lyf1350 .

Sorry guys. I am working on this now. Hop to have some fixes released today or tomorrow.

ok, running a bit behind on this, so hopefully some time this week.

I only wanted to give some feedback. The timing is completely up to you. Thank you very much for all your quasar extensions.

@kanich yes, it may not be this week. But, soon I hope (hopefully before end of month).

@kanich Just letting you know, I have been working on this. Hope to have a release soon. For now, it'll be in the next branch

Thanks for the information. I was trying to build-up some experiments with next branch but I failed on for me unusual code structure. I shall wait for new npm extension release.

Thanks for the information. I was trying to build-up some experiments with next branch but I failed on for me unusual code structure. I shall wait for new npm extension release.

@kanich Yes, it's a work in progress and has some issues I need to iron out first.

@kanich Just letting you know I have rewritten the drag/drop. It all works on Firefox (finally!)
Still ironing out some minor issues and have to do some extensive testing with mobile.
This repo will get renamed to quasar-ui-qwindow
Look for a release (hopefully!) some time next week.

@hawkeye64 Thank you for the status info.

Fixed in v1.0.0-beta.1


Weights and Biases quick question

Since they framed detection as a YOLO problem, I assume they have weights and biases. (eg. the classic equation for regression: y = xW + b, where W is the weights and b is the bias). Does this just mean that the 'weights' are the weights saved into the h5 file? Does that file also include the bias?
I'm asking because I am trying to plot these in a histogram to visualize in Tensorboard and I am not sure where exactly to find this information in the code (i.e. where in the code to put tf.summary.histogram("weights", weights)). I tried just setting histogram_freq=1 but Tensorboard doesn't like generators. Still relatively new to tensorflow and tensorboard.

Includes both weight and bias.

@experiencor is this case, why in https://github.com/experiencor/keras-yolo2/blob/master/Yolo Step-by-Step.ipynb do we have:
x = Conv2D(64, (3,3), strides=(1,1), padding='same', name='conv_2', use_bias=False)(x)


The manualGrouping doesn't work

material-react-table version
v1.2.0
Describe the bug and the steps to reproduce it
When enable manual grouping seems to be doesn't work.
Your Minimal, Reproducible Example - Sandbox (Recommended) or code snippets
Here's the codesandbox:
https://codesandbox.io/s/mrt-manual-grouping-issue-r8r5cn?file=/src/TS.tsx
Just toggle the commented field manualGrouping.
Screenshots or Videos (Optional)
With manualGrouping true

With manualGrouping false

Do you intend to try to help solve this bug with your own PR?
No, because I don't have time to dig into it
Terms

[X] I understand that if my bug cannot be reliably reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

In what way are you trying to use the manualGrouping prop? How are you then grouping you're data manually? The manualGrouping is supposed to turn off the built-in grouping functions, so this is working as intended.
From the TanStack docs:

Enables manual grouping. If this option is set to true, the table will not automatically group rows using getGroupedRowModel() and instead will expect you to manually group the rows before passing them to the table. This is useful if you are doing server-side grouping and aggregation.

Hi @KevinVandy, Sorry I'm new to react-table.
I've found that the manual grouping is quite difficult to implement.
Do you have any example of that?
I've found an example from react-table documentation regarding grouping but it doesn't give me any idea how I can implement it into the material-react-table.

Hello, @rohmanhm , have you successfully addressed the issue of manual grouping in Material-UI's React Table? I've been searching for documentation on this in Tanstack's resources and the Material React Table (MRT) library, but haven't found any.
@KevinVandy , would you be able to provide an example of how to implement manual grouping when making API calls with pagination?


ZeroMQ Adapter

Create a ZeroMQ Adpater using aiozmq. Should take a similar approach to #14.
The adpater should support on_connect like CryoStreamAdapter, so something like:
class MyDevicesZeroMqAdapter(...):
    async def on_connect(self) -> AsyncIterable[int]:
        i = 0
        while True:
            yield i
            i += 1

This should produce incrementing numbers forever over a ZeroMQ stream.

A basic version of this exists as of #25 .

Closed with #137


Tray option to start DoA

Probably need configuration to DoA game path. Or can it be started via steam somehow?

steam://rungameid/ ftw!


Update script tunes june 2022 round two, electric boogaloo

Updates standup scripts now that metal legs are on all robots
Pre-PR Checklist:
Have you

[x] Updated NUbook if necessary (add link to NUbook PR here)
[x] Added/updated tests for your changes, including regression tests for bug fixes
[x] Updated relevant module READMEs
[x] Added/modified documentation directives in relevant code
[x] Added a descriptive title and relevant labels to the PR

You should be fired for naming your branches this way.

Tested. Works well.


Advice about a dataset with conditions

hi @joschif, congrats on the publication and thank you for porting the analysis in a package!
I would like to ask for an advice for analyzing a 10x multiome dataset (control and condition with multiple biological replicates) using Pando.
I want to analyze each cell type separately (which should be fine as mentioned in #49), but since im interested into differences between control and condition within the same cell type, im not sure what would be the best way to call infer_grn(). I was thinking about:
infer_grn(genes = cell_type_specific_DEGs) and interpreting the network as condition specific, but im not sure if that makes sense?!
What do you think about it, do you have any advice? Is there any use case of pando in the experimental design similar to the one I have?
Would be very thankful for support!
Amel

Hi @AmelZulji, when doing case-control comparisons we've usually run Pando on both conditions together and then tested for differential module activity or reg region accessibility. I personally would probably not constrain the GRN to DEGs, because a lot of important regulators will not be DE and so it might be hard to get a well-connected GRN

Thank you for the answer, @joschif!
When it comes to module activity, do you first build module activity with AddModuleScore() as mentioned here: https://github.com/quadbio/scMultiome_analysis_vignette/blob/main/Tutorial.md#section-3-gene-regulatory-network-reconstruction.
If yes, what test would you use to test differentia module activity (I am having 2 groups with 5 biological replicate each)?
Thank you in advance for help!

Hi @AmelZulji, yes, that AddModuleScore() is definitely one way to do this. You could then test for diff activity very simply with a t-test/ANOVA. Since you have replicates, I would probably suggest to use pseudobulk approaches.

Thank you, @joschif! One last question: in order to avoid bias from number of cells, pseudobulk profiles should be generated using averages, or?


LFS-386: Create an environment variable for BioPortal API key

For local testing: set environment variable BIOPORTAL_APIKEY to the desired API key.
(Sasha: if you ever need to test it on Windows that likely means adding this variable in the System Properties/Environment Variables dialogue)

I think I should be able to see the list of available vocabularies?


cigien: Watch randmcnallygpsupdates.com

cigien requests the watch of the watch_keyword randmcnallygpsupdates\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

randmcnallygpsupdates\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Spevacus in Charcoal HQ


Switching to the combat screen

Switch to the combat screen after finishing the interaction with combat NPCs.

added scene switching from dialogue to combat. currently pushed into 'moiz' branch


[Question]: Visual comparisons Feature Parity

Your question
Hi Just wanted to check what the approach will be in terms of feature parity with the JavaScript version of playwright.
I was looking for some basic visual testing and noticed the JS version offers this via Pixelmatch
test-snapshots
Is the plan that the dotnet version will have feature parity with JS version or do we need to look at other implementations to try achieve the same feature of the Java script version.

Playwright Library has feature parity across languages. Visual regression testing is a part of the @playwright/test, a test runner for Playwright on Node.js, so it is missing on .Net.
It 1.18, you will likely see https://playwright.dev/docs/next/api/class-locatorassertions/ for .Net. And only after that, (1.19?), we are likely to add visual assertions into that set.

Thanks for the reply.
is there a backlog or some roadmap where we can upvote features we would like to see ported before others or is features done on a case by case basis

I would echo the OPs question. We're looking to adopt Playwright .Net into our testing pipeline and we were very excited to see Visual comparisons in the docs, and then sad when we realised it wasn't in the .Net flavour. Is there a feature upvoting site we can contribute to to help prioritise this higher?

Just going to give this a bump now 1.18 is out
are we likely to see this feature enabled within playwright or does anyone know good open source alternatives that will play nicely with playwright ?

Updating this for future travels who get their question marked as answered with this post
Thanks to @vmachacek
You can use this to solve some of the issue with a few small lines of code
Verify.HeadlessBrowsers
and this blog that show how easy it is to use
snapshot-testing-with-verify

hey, any updates on the topic? i'm migrating my selenium-based framework to playwright in .net, visual testing is one of the important points for the nearest future - would love to be able to use playwright for this instead of external tooling. really sad to see it in js/ts and not in .net :(

Hello Playwright team!
Can we hope for this feature anytime in the near future?
We have been using Playwright for some time now, and we have come across this requirement and looks like we are in trouble as it's not supported in c#.
We would really appreciate any response/suggestions.

It is a pity this is not supported in dotnet version. It would provide a huge benefit in a lot of the tests. I wanted to migrate to Playwright and this is a huge dealbreaker. Hope this gets implemented soon.

Any updates on this?
This feature is the missing piece that would allow us to migrate from tools like Ranorex and others to the easy and awesome playwright.

Also looking for an update on this as we're starting a new project soon that would really benefit from this.
Would really like to use Playwright with C#

I remembered this issue from late 2021 when I wrote my first set of playwright tests.
It would still be nice to have this feature.

I was just looking for this exact functionality in the docs. I've been doing some searching in the repo, and it looks like ToMatchSnapshot is used internally, or there's at least a PR to rename existing functionality to ToMatchSnapshot?
https://github.com/microsoft/playwright-dotnet/pull/2928
Can this be made public?

The ToMatchSnapshot method inside this codebase is by far not as powerful as the one upstream in Node.js is. We don't feel confident making it public as of today. I recommend copying it into your own code base for now until we have figured out e.g. asset handling etc.

Hi,
I'm really waiting or this feature too. I hope it will be appear very soon :)


Greenkeeper/@nestjs/testing 7.0.1

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


What's the issue with zIndex in Android React Native

Please use https://github.com/react-native-community/discussions-and-proposals to propose changes or discuss feature requests.
We kindly ask that issues of this type are kept in that dedicated repo, to ensure bug reports and regressions are given the priority they require.
Maintainers may flag an issue for Stack Overflow or kindly ask you to move the discussion to https://github.com/react-native-community/discussions-and-proposals at their own discretion.

For Discussion

How come zIndex still doesnt work within react native on android when two sibling views are supposed to overlap. If I'm right there was a pr in 0.42.x that was supposed to fix this.

Closing as this doesn't seem to meet the bar for a Discussion issue. It can be improved by linking to issues that have been opened about this, having a clear goal about what is being proposed, etc.


Version Packages

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.
Releases
<EMAIL_ADDRESS>Minor Changes

cb9e3bf: Added in-field button tokens
Token Diff
Tokens added (19):

in-field-button-edge-to-disclosure-icon-stacked-extra-large
in-field-button-edge-to-disclosure-icon-stacked-large
in-field-button-edge-to-disclosure-icon-stacked-medium
in-field-button-edge-to-disclosure-icon-stacked-small
in-field-button-edge-to-fill
in-field-button-fill-stacked-inner-border-rounding
in-field-button-inner-edge-to-disclosure-icon-stacked-extra-large
in-field-button-inner-edge-to-disclosure-icon-stacked-large
in-field-button-inner-edge-to-disclosure-icon-stacked-medium
in-field-button-inner-edge-to-disclosure-icon-stacked-small
in-field-button-outer-edge-to-disclosure-icon-stacked-extra-large
in-field-button-outer-edge-to-disclosure-icon-stacked-large
in-field-button-outer-edge-to-disclosure-icon-stacked-medium
in-field-button-outer-edge-to-disclosure-icon-stacked-small
in-field-button-stacked-inner-edge-to-fill
in-field-button-width-stacked-extra-large
in-field-button-width-stacked-large
in-field-button-width-stacked-medium
in-field-button-width-stacked-small

Run report for 74e8e843
Total time: 47s | Comparison time: 43.2s | Estimated loss: 3.7s (8.0% slower)

Action
Time
Status
Info

🟩
SetupNodeTool(18.13.0)
3.4s
Passed

🟩
InstallNodeDeps(18.13.0)
18.7s
Passed

🟩
SyncNodeProject(tokens)
0.4ms
Passed

🟩
RunTarget(tokens:build)
23.3s
Passed

🟩
SyncWorkspace
0ms
Passed

🟩
RunTarget(tokens:generateDiffResult)
1.7s
Passed

Touched files
.changeset/swift-gifts-dress.md
packages/tokens/CHANGELOG.md
packages/tokens/package.json


Explore button menu goes outside of screen

When I click on the Explore button in the navbar (top-right), the menu goes outside of the screen

https://github.com/astrolabsoftware/fink-science-portal/commit/cac3f601984fc1abcd50a965da726ab50fe05db8


websocket: throw an error rather than crash if specify invalid url

Following code will cause app crash:
new WebSocket('an URL not startswith ws/wss/http/https');

The problem caused by https://github.com/facebook/react-native/blob/master/Libraries/WebSocket/RCTSRWebSocket.m#L318. There is no any checks before the native assert, will caused app crashed.
I add check on WebSocket's constructor, will throw an error if specify invalid url.
Test plan (required)
I've add test case.

It looks like this PR has been abandoned so I am going to close it. Please feel free to reopen if I am wrong and you are still working on it!


Bug in GPU settings

Algorithm runs very slow even setting the parameters: {"tree_method ": "gpu_hist", "predictor": "gpu_predictor"}.
To make it run faster (all in GPU) its necessary to set one additional parameter: {"updater": "grow_gpu_hist"}
XGB version: '1.3.0-SNAPSHOT'
HW: DGX Station V100
To reproduce the experiment:
import time
import xgboost as xgb
from sklearn.datasets import make_classification

# Load a Dataset and sent to XGBoost DMatrix
train, target = make_classification( n_samples=10000, n_features=500, random_state=2021)
xdata = xgb.DMatrix(train, label=target )

print( "This runs in cpu, single core, very slow" )
params = {
    "objective": "binary:logistic",
    "eval_metric": "logloss",
    "nthread": 1,
}
start_time = time.time()
model = xgb.cv( params, xdata, num_boost_round=10, verbose_eval=True, seed=2021 )
print( time.time() - start_time, 'seconds' )
print()

print( "This is supposed to run in GPU but still run very slow" )
params = {
    "objective": "binary:logistic",
    "eval_metric": "logloss",
    "tree_method ": "gpu_hist",
    "predictor": "gpu_predictor",
    "nthread": 1,
}
start_time = time.time()
model = xgb.cv( params, xdata, num_boost_round=10, verbose_eval=True, seed=2021 )
print( time.time() - start_time, 'seconds' )
print()

print( "Add the parameter {'updater': 'grow_gpu_hist'} make it runs x25+ faster" )
params = {
    "objective": "binary:logistic",
    "eval_metric": "logloss",
    "tree_method ": "gpu_hist",
    "predictor": "gpu_predictor",
    "updater": "grow_gpu_hist",
    "nthread": 1,
}
start_time = time.time()
model = xgb.cv( params, xdata, num_boost_round=10, verbose_eval=True, seed=2021 )
print( time.time() - start_time, 'seconds' )

am I missing something ?

run log:
This runs in cpu, single core, very slow
[0]	train-logloss:0.48580+0.00162	test-logloss:0.49674+0.00078
[1]	train-logloss:0.36871+0.00179	test-logloss:0.38832+0.00284
[2]	train-logloss:0.29466+0.00121	test-logloss:0.32335+0.00454
[3]	train-logloss:0.24379+0.00155	test-logloss:0.28102+0.00411
[4]	train-logloss:0.20733+0.00182	test-logloss:0.25302+0.00488
[5]	train-logloss:0.18037+0.00149	test-logloss:0.23304+0.00548
[6]	train-logloss:0.15909+0.00141	test-logloss:0.21936+0.00546
[7]	train-logloss:0.14295+0.00150	test-logloss:0.21186+0.00534
[8]	train-logloss:0.12968+0.00119	test-logloss:0.20573+0.00586
[9]	train-logloss:0.11803+0.00095	test-logloss:0.20203+0.00657
22.743130922317505 seconds

This is supposed to run in GPU but still run very slow
[16:43:26] WARNING: /opt/conda/envs/rapids/conda-bld/xgboost_1603491637460/work/src/learner.cc:529:
Parameters: { tree_method  } might not be used.

  This may not be accurate due to some parameters are only used in language bindings but
  passed down to XGBoost core.  Or some parameters are not used but slip through this
  verification. Please open an issue if you find above cases.

[16:43:30] WARNING: /opt/conda/envs/rapids/conda-bld/xgboost_1603491637460/work/src/learner.cc:529:
Parameters: { tree_method  } might not be used.

  This may not be accurate due to some parameters are only used in language bindings but
  passed down to XGBoost core.  Or some parameters are not used but slip through this
  verification. Please open an issue if you find above cases.

[16:43:31] WARNING: /opt/conda/envs/rapids/conda-bld/xgboost_1603491637460/work/src/learner.cc:529:
Parameters: { tree_method  } might not be used.

  This may not be accurate due to some parameters are only used in language bindings but
  passed down to XGBoost core.  Or some parameters are not used but slip through this
  verification. Please open an issue if you find above cases.

[0]	train-logloss:0.48580+0.00162	test-logloss:0.49675+0.00078
[1]	train-logloss:0.36871+0.00179	test-logloss:0.38832+0.00284
[2]	train-logloss:0.29466+0.00122	test-logloss:0.32335+0.00454
[3]	train-logloss:0.24379+0.00156	test-logloss:0.28103+0.00411
[4]	train-logloss:0.20733+0.00182	test-logloss:0.25303+0.00488
[5]	train-logloss:0.18037+0.00149	test-logloss:0.23304+0.00548
[6]	train-logloss:0.15909+0.00141	test-logloss:0.21936+0.00546
[7]	train-logloss:0.14295+0.00149	test-logloss:0.21186+0.00534
[8]	train-logloss:0.12968+0.00119	test-logloss:0.20573+0.00586
[9]	train-logloss:0.11803+0.00095	test-logloss:0.20203+0.00657
25.442907333374023 seconds

Add the parameter {'updater': 'grow_gpu_hist'} make it runs x20+ faster
[16:43:51] WARNING: /opt/conda/envs/rapids/conda-bld/xgboost_1603491637460/work/src/gbm/gbtree.cc:81: DANGER AHEAD: You have manually specified `updater` parameter. The `tree_method` parameter will be ignored. Incorrect sequence of updaters will produce undefined behavior. For common uses, we recommend using`tree_method` parameter instead.
[16:43:51] WARNING: /opt/conda/envs/rapids/conda-bld/xgboost_1603491637460/work/src/learner.cc:529:
Parameters: { tree_method  } might not be used.

  This may not be accurate due to some parameters are only used in language bindings but
  passed down to XGBoost core.  Or some parameters are not used but slip through this
  verification. Please open an issue if you find above cases.

[16:43:51] WARNING: /opt/conda/envs/rapids/conda-bld/xgboost_1603491637460/work/src/gbm/gbtree.cc:81: DANGER AHEAD: You have manually specified `updater` parameter. The `tree_method` parameter will be ignored. Incorrect sequence of updaters will produce undefined behavior. For common uses, we recommend using`tree_method` parameter instead.
[16:43:51] WARNING: /opt/conda/envs/rapids/conda-bld/xgboost_1603491637460/work/src/learner.cc:529:
Parameters: { tree_method  } might not be used.

  This may not be accurate due to some parameters are only used in language bindings but
  passed down to XGBoost core.  Or some parameters are not used but slip through this
  verification. Please open an issue if you find above cases.

[16:43:52] WARNING: /opt/conda/envs/rapids/conda-bld/xgboost_1603491637460/work/src/gbm/gbtree.cc:81: DANGER AHEAD: You have manually specified `updater` parameter. The `tree_method` parameter will be ignored. Incorrect sequence of updaters will produce undefined behavior. For common uses, we recommend using`tree_method` parameter instead.
[16:43:52] WARNING: /opt/conda/envs/rapids/conda-bld/xgboost_1603491637460/work/src/learner.cc:529:
Parameters: { tree_method  } might not be used.

  This may not be accurate due to some parameters are only used in language bindings but
  passed down to XGBoost core.  Or some parameters are not used but slip through this
  verification. Please open an issue if you find above cases.

[0]	train-logloss:0.48616+0.00073	test-logloss:0.49651+0.00121
[1]	train-logloss:0.36969+0.00117	test-logloss:0.38857+0.00065
[2]	train-logloss:0.29469+0.00119	test-logloss:0.32381+0.00228
[3]	train-logloss:0.24365+0.00174	test-logloss:0.28044+0.00301
[4]	train-logloss:0.20758+0.00162	test-logloss:0.25214+0.00306
[5]	train-logloss:0.18133+0.00160	test-logloss:0.23501+0.00250
[6]	train-logloss:0.16061+0.00241	test-logloss:0.22192+0.00273
[7]	train-logloss:0.14409+0.00244	test-logloss:0.21323+0.00408
[8]	train-logloss:0.13147+0.00211	test-logloss:0.20759+0.00397
[9]	train-logloss:0.11955+0.00205	test-logloss:0.20454+0.00410
0.685046911239624 seconds

@titericz After a very confusing debugging session, I found you that typed "tree_method ", with an extra space at the end.

That was fun, I looked into the implementation of std::set_difference.  ;-)

https://github.com/dmlc/xgboost/pull/6769 might be able to provide better error messages.  suggestions are welcomed.

Thanks @trivialfis . At least my bug may help with better error messages ;)


added app status validation

Fixes

https://issues.redhat.com/browse/HAC-3332
Description

Fixed graph node validation and added app status validation
Type of change

[ ] Feature
[ ] Bugfix
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[x] Other (please describe): Test Case Fix

Screen shots / Gifs for design review

How to test or reproduce?

Browser conformance:

[x] Chrome
[ ] Firefox
[ ] Safari
[ ] Edge

/retest

/retest

/retest

/retest

/retest

/retest

/lgtm

/approve


Target pod monitoring from loadbalancer

The meridio lb pod and nsp pod are not monitor the target pod reachability through nsm interface. I have tested it with disabling the nsc interface on the target pod.

https://github.com/Nordix/Meridio/issues/194


Return golang errors instead of nvml.Return values

These changes move from using nvml.Return values to using native Golang error values, making code more idiomatic.
Note that this requires that error.Is is used instead of checking for equality in the returned value.
For example:
current, pending, err := device.GetMigMode()
if errors.Is(err, nvml.ERROR_NOT_SUPPORTED) {
	fmt.Printf("MIG is not supported for device %v\n", i)

} else if err != nil {
	log.Fatalf("Error getting MIG mode for device at index %d: %v", i, err)
}
fmt.Printf("MIG mode for device %v: current=%v pending=%v\n", i, current, pending)

would become:
current, pending, err := device.GetMigMode()
if errors.Is(err, nvml.ERROR_NOT_SUPPORTED) {
	fmt.Printf("MIG is not supported for device %v\n", i)

} else if err != nil {
	log.Fatalf("Error getting MIG mode for device at index %d: %v", i, err)
}
fmt.Printf("MIG mode for device %v: current=%v pending=%v\n", i, current, pending)

Your two example's in the description are the same .. I think you wanted one of them to check euqility instead of using errors.Is().
That said -- what will happen if someone does do err == nvml.ERROR_NOT_SUPPORTED? Will it be false even when errors.Is(err, nvml.ERROR_NOT_SUPPORTED) would be true?


Django-natural-keys

Closes: #2900
What's Changed

~Add django-natural-keys aka natural_keys as a library dependency~  No longer a library dependency

~Currently I'm overriding/extending roughly 50% of the code base in django-natural-keys to add features and fixes that we need (primarily handling for Location's variadic natural key and the fact that ContentType is not a NaturalKeyModel but we have models that have natural-keys referencing it). I think a number of my extensions here would make sense to at least try contributing upstream, although it doesn't appear to be a very active project presently.~
[x] TODO: review all of my overrides/extensions and see if there's a smaller/simpler set of changes that could achieve the same results; the current changes have been added piecemeal as I encountered various limitations and issues.

~Change Nautobot's BaseModel to inherit from NaturalKeyModel and RestrictedQuerySet to inherit from NaturalKeyQuerySet.~ Additionally add a new BaseManager as derived from NaturalKeyManager.

Change all model objects declarations as needed to ensure that they correctly are based on BaseManager.

Add a custom implementation of natural_key_slug based around URL-encoding of strings to produce a "slug-like" string that is lossless and can still be converted back to the original natural key on demand.
Add a ModelTestCases.BaseModelTestCase class in core that includes a couple of natural-key tests, and extended a number of existing test_models.py test files to use this base class where doing so would be reasonably simple.

[ ] TODO: add basic tests for all models as a way to assert that all models' natural-keys are working correctly. (We get pretty good coverage of this for many models already when using --cache-test-fixtures and running tests, as that uses dumpdata/loaddata with natural-keys enabled...)

Add a custom natural-key implementation for Location because it has a self-referential/recursive natural key that needs some special handling to avoid infinite recursion when trying to determine the potential scope of the natural key.
For now, exclude Prefix and IPAddress from having a natural_key implementation because those models don't actually have a reasonable natural key as presently defined.

>>> DeviceType.objects.first().natural_key()
['Arista', 'Arista DeviceType 15']

>>> DeviceType.objects.get_by_natural_key("Arista", "Arista DeviceType 15")
<DeviceType: Arista DeviceType 15>

>>> DeviceType.objects.first().natural_key_slug
'Arista&Arista+DeviceType+15'

>>> DeviceType.objects.get(natural_key_slug='Arista&Arista+DeviceType+15')
<DeviceType: Arista DeviceType 15>

TODO

[x] Explanation of Change(s)
[x] Added change log fragment(s) (for more information see the documentation)
[x] Attached Screenshots, Payload Example
[ ] Unit, Integration Tests
[x] Documentation Updates (when adding/changing features)
n/a Example Plugin Updates (when adding/changing features)
[x] Outline Remaining Work, Constraints from Design

[ ] Investigate how hard it would be to add URL patterns for natural-key-slugs for all models in any sort of automated way?

OK! I managed to get rid of django-natural-keys as a dependency without adding too much more to our own code, and I at least personally find the reimplemented code easier to understand than django-natural-keys's original implementation. Hope y'all agree with me on that!
Also I made a stab at adding/updating some developer documentation.


Use StringEncoding in TextManager

This issue introduces the StringEncoding described in the following design doc: https://hackmd.io/Ybil_epnQPiBczPcbGRqCw?view
It adds the ability to define constant, manual, direct, and formatted string encodings in TextManager. This is related to #3053 and should close it when done.

This was only partially done by #4051 and #4062


fix: locking on linux and osx

Should fix https://github.com/ccarpita/kvstore/issues/1
Also, fix a number of shell style warnings.

@matthewpersico if you get a chance, can you test this in your environment and let me know if this addresses the issue?

I'll try to pull this down this weekend and review.

Approved! Works fine. On Cygwin, no less. :-)

If you merge back into master, I can then send you a PR with my other changes.


[D3D12]CommandList->Close gives E_INVALIDARG with RenderDoc attached

This is my own engine, which works well without RenderDoc, but with RenderDoc attached, CommandList->Close throws E_INVALIDARG which is rather weird because Close() doesn't have any parameters. However the flow of my app may be weird because I initially close a command list that may have no commands in it ! I can provide a build of my app if someone wants to reproduce this. PIX for Windows works fine btw.

RenderDoc version: 1.5
Operating System: Windows 10 2004
API: D3D12
RTX 2060, latest driver

Hi there, your issue has been closed due to not correctly following the issue template. Please read the documentation about filing issues and open a new issue which does follow the issue template. Do not reply to this issue with more information or with a correctly formatted issue, as that will only lead to confusion.
I try to address all github issues and fix bugs quickly, but badly formatted issues or poorly described bug reports especially are a significant drain on project resources. Taking a few minutes to file a good issue makes a huge difference and will get it addressed faster - everybody wins!
In github when you first create a new issue the form is pre-populated with a template which you should read over and follow. As linked above there is documentation going into more detail about what makes a good issue.


Unable to load aiy-voicebonnet

Hi,
I build the Debian package aiy-overlay-voice_1.0-1_all.deb but I can't load the overlay.
sudo dtoverlay aiy-voicebonnet
* Failed to apply overlay '1_aiy-voicebonnet' (kernel)

This is what I have in dmesg:
[Wed Jan 24 21:59:30 2024] OF: overlay: WARNING: memory leak will occur if overlay removed, property: /soc/i2s@7e203000/status
[Wed Jan 24 21:59:30 2024] OF: overlay: WARNING: memory leak will occur if overlay removed, property: /soc/i2c@7e804000/status
[Wed Jan 24 21:59:30 2024] OF: overlay: WARNING: memory leak will occur if overlay removed, property: /soc/i2c@7e804000/aiy-io-i2c@52/compatible
[Wed Jan 24 21:59:30 2024] OF: overlay: WARNING: memory leak will occur if overlay removed, property: /soc/i2c@7e804000/aiy-io-i2c@52/reg
[Wed Jan 24 21:59:30 2024] OF: overlay: WARNING: memory leak will occur if overlay removed, property: /soc/i2c@7e804000/aiy-io-i2c@52/type
[Wed Jan 24 21:59:30 2024] OF: overlay: WARNING: memory leak will occur if overlay removed, property: /soc/i2c@7e804000/aiy-io-i2c@52/status

I'm running kernel 6.1.74-v8+.
uname -a
Linux rpi3b02 6.1.74-v8+ #1725 SMP PREEMPT Mon Jan 22 13:35:32 GMT 2024 aarch64 GNU/Linux

Any idea how I could fix this?
Thanks :+1:

Here with the versbose.
DTOVERLAY[debug]: using platform 'bcm2835'
DTOVERLAY[debug]: overlay map loaded
run_cmd: which dtoverlay-pre >/dev/null 2>&1 && dtoverlay-pre
DTOVERLAY[debug]: loading file '/boot/firmware/overlays/aiy-voicebonnet.dtbo'
DTOVERLAY[debug]: wrote 2017 bytes to '/tmp/.dtoverlays/0_aiy-voicebonnet.dtbo'
DTOVERLAY[debug]: wrote 2017 bytes to '/sys/kernel/config/device-tree/overlays/0_aiy-voicebonnet/dtbo'
* Failed to apply overlay '0_aiy-voicebonnet' (kernel)
run_cmd: which dtoverlay-post >/dev/null 2>&1 && dtoverlay-post

Same result with kernel 6.1.21-v8+.

I found your aiy-voicebonnet-soundcard-dkms_3.0-1+nmu1_all.deb but I can't install it as well.
DKMS make.log for aiy-voicebonnet-soundcard-3.0 for kernel 6.1.21-v8+ (aarch64)
Thu 25 Jan 15:46:59 EST 2024
make: Entering directory '/usr/src/linux-headers-6.1.21-v8+'
warning: the compiler differs from the one used to build the kernel
  The kernel was built by: gcc (Debian 10.2.1-6) 10.2.1 20210110
  You are using:           gcc (Debian 12.2.0-14) 12.2.0
  CC [M]  /var/lib/dkms/aiy-voicebonnet-soundcard/3.0/build/snd-aiy-voicebonnet.o
  CC [M]  /var/lib/dkms/aiy-voicebonnet-soundcard/3.0/build/rt5645.o
  CC [M]  /var/lib/dkms/aiy-voicebonnet-soundcard/3.0/build/rl6231.o
In file included from /var/lib/dkms/aiy-voicebonnet-soundcard/3.0/build/snd-aiy-voicebonnet.c:25:
/var/lib/dkms/aiy-voicebonnet-soundcard/3.0/build/rt5645.h:12:10: fatal error: sound/rt5645.h: No such file or directory
   12 | #include <sound/rt5645.h>
      |          ^~~~~~~~~~~~~~~~
compilation terminated.
make[1]: *** [scripts/Makefile.build:250: /var/lib/dkms/aiy-voicebonnet-soundcard/3.0/build/snd-aiy-voicebonnet.o] Error 1
make[1]: *** Waiting for unfinished jobs....
In file included from /var/lib/dkms/aiy-voicebonnet-soundcard/3.0/build/rt5645.c:32:
/var/lib/dkms/aiy-voicebonnet-soundcard/3.0/build/rt5645.h:12:10: fatal error: sound/rt5645.h: No such file or directory
   12 | #include <sound/rt5645.h>
      |          ^~~~~~~~~~~~~~~~
compilation terminated.
make[1]: *** [scripts/Makefile.build:250: /var/lib/dkms/aiy-voicebonnet-soundcard/3.0/build/rt5645.o] Error 1
make: *** [Makefile:2012: /var/lib/dkms/aiy-voicebonnet-soundcard/3.0/build] Error 2
make: Leaving directory '/usr/src/linux-headers-6.1.21-v8+'

Going further, I'm able to build the module but when I try to load it I got two unknown symbols.
[ 9883.028212] snd_aiy_voicebonnet: Unknown symbol rt5645_sel_asrc_clk_src (err -2)
[ 9883.028297] snd_aiy_voicebonnet: Unknown symbol rt5645_set_jack_detect (err -2)

Finally loaded!
lsmod | grep snd_aiy
snd_aiy_voicebonnet    16384  2
rt5645                159744  2 snd_aiy_voicebonnet
snd_soc_core          274432  5 snd_soc_bcm2835_i2s,vc4,snd_soc_hdmi_codec,rt5645,snd_aiy_voicebonnet

aplay -l
**** List of PLAYBACK Hardware Devices ****
card 1: aiyvoicebonnet [aiy-voicebonnet], device 0: Google AIY Voice Bonnet SoundCard HiFi rt5645-aif1-0 [Google AIY Voice Bonnet SoundCard HiFi rt5645-aif1-0]
  Subdevices: 1/1
  Subdevice #0: subdevice #0

arecord -l
**** List of CAPTURE Hardware Devices ****
card 1: aiyvoicebonnet [aiy-voicebonnet], device 0: Google AIY Voice Bonnet SoundCard HiFi rt5645-aif1-0 [Google AIY Voice Bonnet SoundCard HiFi rt5645-aif1-0]
  Subdevices: 0/1
  Subdevice #0: subdevice #0

Here is the working procedure for Debian Bullseye and Debian Bookworm.

But nothing on the Raspberry Pi Z W even if the module is loaded.

Here is the working procedure for Debian Bullseye and Debian Bookworm, works on 32-bit and 64-bit.
  I have everything installed, but the overlay will not load
dtoverlay -l  Shows nothing

@goldyfruit Were you able to load the overlay with this method? I have everything installed, but the overlay will not load dtoverlay -l Shows nothing

Nop, usually dtoverlay don't like to be load by the user.


No passphrase dialog is shown.

Getting error "Encrypted private key detected, but no passphrase given"
...and no passphrase dialog is shown (don't want to save it in config for security reasons)

I am getting the same error, but while using a ssh key?
Encrypted private key detected, but no passphrase given
Settings:

{

    "remotefs.remote": {
        "dev": {
          "privateKeyPath": "<path to key>",
          "scheme": "sftp",
          "host": "<host name>",
          "username": "<username>"",
          "rootPath": "<root path>""
        }
      }

}

@TMorville how did you solve your error?

You don't need to hardcode the passphrase in sftp.json, just add "passphrase": true and a vscode popup will prompt when you hit save to enter your passphrase.


Disable ASCII style for progress bar

I think it looks nicer that way:

vs.

It doesn't look... particularly beautiful on my machine. I think it's safer if we use ASCII.

Hmm ok, on my Mac it looked way better :) But in that case, I'll close it.


The readme.md file of YanFei-CUIot-MZ-6 is empty

The readme.md file of YanFei-CUIot-MZ-6 is empty, \vendor\platform\Fibocom-L610

This readme.md file of  YanFei-CUIot-MZ-6 has been updated.


Fork and test PR / CI

Fork this repo, create a pull request, see if the pull request runs CI, set up circleci on fork if necessary, document the process in a GETTING_STARTED.md file

close due to project shutdown


Server is not running due to typo in settings.py file

First, we saw that vagrant up not running the server. than we tried to run the runserver command manually and saw that we are getting this error:
raise ImportError(msg) ImportError: Module 'AcadeMeData.apps' does not contain a 'AcadeMeDataConfig' class. Choices are: 'AcademedataConfig'.
So we understood the issue and fixed it.
PR link for the fix

PR #36  has been merged.
Done.


Add Sekolah Terpadu Pahoa

Official site is https://pahoa.sch.id . A school in Indonesia that cater pre-school up to senior high school.

@gustovar First stage of verification passed, official domain for Sekolah Terpadu Pahoa is sekolah.pahoa.sch.id as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@gustovar Thank you for your patience while waiting for a review and processing your request. Pull request merged.


feat: integrate dagger contracts

Integrate dagger contracts from nim-dagger-contracts repo.
Add dagger-contracts, nim-web3, and all of nim-web3’s transitive deps as submodule deps to nim-dagger. Note: nim-web3 and its transitive deps may no longer be needed when we switch to nim-ethers.
Add a testContracts nimble task to test all of the contracts functionality. Namely, this spins up an ethereum simulator, deploys the contracts (in dagger-contracts), runs the contract tests, and finally, regardless of success/error, kills the ethereum sim processes. The nimble task can be run with ./env.sh nimble testContracts.
We also tested nim-dagger-contracts as a submodule dep of nim-dagger, and while the tests run as expected, the preference is to merge nim-dagger-contracts inside of nim-dagger for ease of parallel development. There’s also a high probability that nim-dagger-contracts is not being used as a dep by other projects. Are there any strong objections to this?
Co-authored-by: Michael Bradley<EMAIL_ADDRESS>

@markspanbroek, removed nim-web3 in favour of nim-ethers.


Add appVersion for uchiwa

"home" key is needed for ci testing now, otherwise testing will fail.

/assign @lachie83

/ok-to-test
/lgtm


password with special characters is difficult to connect with

The Rust URL encoding will encode the user password to https://url.spec.whatwg.org/#userinfo-percent-encode-set
However, PG itself points to this as the encoding: https://datatracker.ietf.org/doc/html/rfc3986#section-2.1
What works for me is to use jq -sRr '@uri' or more specifically
#!/usr/bin/env bash
set -euo pipefail

PGPASSWORD="${PGPASSWORD:-''}"
PGHOST="${PGHOST:-'localhost'}"
PGPORT="${PGPORT:-5432}"
password="$(echo -n "$PGPASSWORD" | jq -sRr '@uri')"
if echo $PGHOST | grep '/' >/dev/null ; then
  host='localhost'
else
  host="$(echo -n "$PGHOST" | jq -sRr '@uri')"
fi
echo "postgres://$PGUSER:${password}@${host}:$PGPORT/$PGDATABASE?sslmode=disable"

What I am doing is not passing any PG args to datanymizer other than --dbname as the url and also giving the url to pg_dump with pg_datanymize --dbname "$DATABASE_URL" -- $DATABASE_URL

hmm, also saw that strategy not work in a different case and reverted back to CLI args and it worked there.


unix: use fchmod() in uv_fs_copyfile()

This commit introduces fchmod() in uv_fs_copyfile() to set the mode of the destination file.
Refs: https://github.com/nodejs/node/issues/15394

CI with no related failures: https://ci.nodejs.org/job/libuv-test-commit/459/

Doesn't this effectively assume that the user and group are already set the same and that the OS doesn't support ACL?

Doesn't this effectively assume that the user and group are already set the same

Are you saying we should call fchown()? If so, I don't think we should by default.

and that the OS doesn't support ACL?

We don't currently handle ACLs.

Are you saying we should call fchown()? If so, I don't think we should by default.

Yes, I thought it sounded strange to be setting the group permissions, but not the group. Although I see python's shutil.copy2 is documented as having the same behavior (with a red warning box), so perhaps this is just the expected behavior?


panorama width and height vs. minPitch and maxPitch

Hi.
I set:
#panorama {
width: 100%;
height: 600px;
}
and it works with a full desktop screen but it leaves a blank white space on the bottom if the screen height is more than 600 pixels
I tried to set: height: auto;
but it does not work in the sense that the panorama is not shown at all.
I noticed another issue with the above panorama settings: reducing the width of the available screen (i.e. using a smartphone), the minPitch parameter setting ("minPitch": -30) seems not to work any longer and the geometry of the shown scene seems to change if compared with the full desktop screen.
Is there anything I missed?
Thanks,
Alex

I tried to set: height: auto;
but it does not work in the sense that the panorama is not shown at all.

That's an issue with your CSS, not with Pannellum. The viewer has no intrinsic height, so setting the height to auto is the same as setting it to zero height, so of course you can't see it. As far as the CSS goes, the viewer is sized the same way you'd size an empty <div>.

I noticed another issue with the above panorama settings: reducing the width of the available screen (i.e. using a smartphone), the minPitch parameter setting ("minPitch": -30) seems not to work any longer and the geometry of the shown scene seems to change if compared with the full desktop screen.

Quoting from the documentation for minPitch: "Sets the minimum / maximum pitch the viewer edge can be at, in degrees." By reducing the width of the screen while keeping the height constant, you're changing the aspect ratio. As the parameter deals with the edge of the view, this affects how it is applied.

Maybe I missed something important.
I need to display panoramas both on desktop and smartphone screens.
Responsive images will automatically adjust to fit the size of the screen.
As far as I know the following CSS settings should get the result:
#panorama {
width: 100%;
height: auto;
}
How could I get responsiveness otherwise?
Perhaps by adding media query rules?
Any suggestion is welcome.
Thanks,
Alex

As far as I know the following CSS settings should get the result

As I explicitly told you in my last reply, that sets the viewer height to zero.

How could I get responsiveness otherwise?
Perhaps by adding media query rules?

It depends on your preferences for how you want it displayed. Using media query rules with explicit heights for different break points is one option. Another option would be to use aspect-ratio and max-height to set a maximum height and an aspect ratio to use for narrower screen sizes.


executeTransaction - require as action function attribute

Hi, I've got improvement proposal.
Problem:
When using webpack/node. In executeTransaction, require tries to import not existing @arango package.

it fails with
(node:1859) UnhandledPromiseRejectionWarning: ArangoError: ReferenceError: __webpack_require__ is not defined
    at new ArangoError (/mnt/s/project/node_modules/arangojs/lib/async/error.js:71:21)
    at Object.resolve (/mnt/s/project/node_modules/arangojs/lib/async/connection.js:263:32)
    at callback (/mnt/s/project/node_modules/arangojs/lib/async/connection.js:129:26)
    at IncomingMessage.res.on (/mnt/s/project/node_modules/arangojs/lib/async/util/request.node.js:78:21)
    at IncomingMessage.emit (events.js:203:15)
    at endReadableNT (_stream_readable.js:1129:12)
    at process._tickCallback (internal/process/next_tick.js:63:19)

Solution:
If there was require function passed as argument, we can use it as internal arangodb/foxxx require inside action without error on client's side.

We can also do fake-passing as this function takes only params argument:
const action = function(params, require) {
  const { query } = require('@arangodb');
  return query`return true`;
}
db.executeTransaction(collections, action, options);

// inside executeTransation in src/database.ts
let actionString : string = String.raw`${action}`; // to ensure that is a string
const regex = /(?<=^\([\s\w\W]*|^function\s*\([\s\w\W]*)(\,\s*\brequire\b)(?=\s*\)\s*\=\>\s*\{|\s*\)\s*\{)/;
// this regex should work with this scenarios
// function (params, require) {}
// function ({ ids, ...params }, require) {}
// (params, require) => {}
// ({ ids, ...params }, require) => {} // beware of using rest operator with !!webpack!!
// and also without require attribute
// (params) => {}
// function (params) {}

actionString = actionString.trim().replace(regex, ''); // remove require from function attributes

  // ...request ...
    body: {
      action: actionString,
     // ...
  }

actionString produces function (params) {...} and it will use database's require

I think the cleaner solution is to write the function as normal and then turn it into a multi-line string by wrapping it in backticks.
Trying to mess with the parameter list before sending it to the server seems error-prone. We could change the actual implementation in ArangoDB to pass these globals (or at least require) as additional arguments if a workaround is really needed.
That said, have you looked into the streaming transactions API as an alternative?
https://github.com/arangodb/arangojs/blob/v6.12.0/docs/Drivers/JS/Reference/Database/Transactions.md#databasebegintransaction

I'm closing this issue due to inactivity as it's more of a general question than a bug.


Overlap windows

Hello, if you set fixed columns in a table and use a filter this happens. (z-order need to be updated in css in your distribution)
Thanks

thanks for reporting, fixed in dev branch will release soon after testing


wire: UData changes for multiblockproof

These series of commits ready the wire package for supporting multi-block proofs, which aim to reduce the bandwidth requirements for utreexo nodes during ibd.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

115 of 174 (66.09%)  changed or added relevant lines in 3 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.05%) to 54.47%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

wire/batchproof.go
18
33
54.55%

wire/udata.go
95
139
68.35%

Files with Coverage Reduction
New Missed Lines
%

database/ffldb/blockio.go
4
93.68%

Totals

Change from base Build<PHONE_NUMBER>:
0.05%

Covered Lines:
26295

Relevant Lines:
48274

💛  - Coveralls


Implement tip radius analysis pipeline function

We need a pipeline function that carries out tip radius analysis and returns the length scale below which data is unreliable.

Closed with https://github.com/ContactEngineering/SurfaceTopography/pull/115