Models question

Is there any possible way to create table based of a model, I think I can create a schema that follow the model, but that also means I have to maintain the both the schema create table AND the model types, which isn't the end of the world for maybe having a 1 - 3 models but once you get a decent number of models, I think it could be quite tedious to maintain both.
If  it is possible to create a table from its model, that would be great to hear and also if any of this does not make sense, feel free to ask for more clarfication

Hi, thanks for asking. Currently, there's no way to generate a table from a model class. However, it's a nice feature to have. Honestly, I don't have enough experience in introspecting a database schema, but I will try my best to learn it and hopefully implement it in this library. If you or anyone else know how to do this, please let me know, I'm open for PR! 😃
I'm thinking of generating migration files from a model class, not creating the tables directly. Then, you can run the migration whenever you want to create the table. But if you have a better strategy to do this, I would love to know.

I think any strategy is better than none at all 😃, I have actually no experience on creating any ORM at all. However I've used alot in my time. Almost all of them have there own custom types. that you can plug into a model. I would presume that is for a good reason, as databases in generel tend to have more specific types than (what's provided in javascript) string, integer, etc. Hopefully that is some kind of hint / step in the right direction, with that being said. I'm totally clueless and only take what I just said with a grain of salt.
Thank you for replying so fast, I really do apperciate & good luck, I'll be keeping a close eye out for any like-wise feature.

Here is an example project handling migrations specifically, if you need any inspiration for this.
<EMAIL_ADDRESS>

No problem, thank you very much @Yofou and @wyrd-code 😃

I think any strategy is better than none at all 😃, I have actually no experience on creating any ORM at all. However I've used alot in my time. Almost all of them have there own custom types. that you can plug into a model. I would presume that is for a good reason, as databases in generel tend to have more specific types than (what's provided in javascript) string, integer, etc. Hopefully that is some kind of hint / step in the right direction, with that being said. I'm totally clueless and only take what I just said with a grain of salt.
Thank you for replying so fast, I really do apperciate it & good luck, I'll be keeping a close eye out for any like-wise feature.

I have seen often in Node ORMs there is a "sync" feature, which is intended just to speed up development process and basically drops everything and recreate the tables based on the defined models/entities.
Also they support generating migrations based on entities/models changes.
I think both strategies are useful, but probably generating migrations based on models changes is higher priority one and "sync" is just a nice to have (and it can be dangerous, since you can delete all your data if you push it to production by accident).


Serve the flag icons from Fides/privacy center

The mobile phone input flag icon is coming from a 3rd party domain. Let's have this served from the privacy center like the other logos.

Currently pulling flags from https://gitlab.com/catamphetamine/country-flag-icons

Note that it's not "the flag icon" as in the US flag, but an icon for every country.
The quickest hack is to bundle them into the app: including all flags. I think this is fine because we have Next doing the build which chunks assets automatically.

Verified on UAT


Creating a new Profile.md

created a new branch, created a file, and made a commit.

add nwe file and delete de new branch


Check for warnings after compilation

Motivation
Currently forge suppresses warnings when building projects (https://github.com/gakonst/foundry/issues/304). This adds relevant methods that we can use to detect warnings there.

I assumed that this portion spat out both warnings and errors? :thinking: Maybe this assumption is incorrect https://github.com/gakonst/ethers-rs/blob/0ddcb4c84cc3c4ec2ba65fcce280bd40caaa481b/ethers-solc/src/artifacts.rs#L709-L718
I definitely want it to emit warnings as well in this PR if this is not the case

oh my bad, yeah warnings are also stored in errors, doh :)

Committed a minor fix, think it would technically be possible for it to say "Compiler run successful with warnings" even though you also have high severity errors if you had both errors and warnings


tests: lwm2m_registry: Fix license to Apache-2.0

Use Apache-2.0
Signed-off-by: Jarno Lämsä<EMAIL_ADDRESS>

@SeppoTakalo please review.

Marking this DNM until the release meeting later today where we will discuss whether #53929 should be reverted because it is an enhancement and was merged post-feature freeze by mistake.


[Wallet] Add tailwind wallet

[ ] auto connect doesnt work

connect wallet
refresh page
expected: auto connect
actual: app asks to reconnect the wallet

[ ] reject is not picked up by app

connect wallet
create a trade on the app
open wallet dialogue and reject it
expected: CTA to pick up that user rejected and return to the state of "buy"
actual: CTA stays on loading mode waiting for user response


AB#4956

Proposed changes

Related issues

Checklist

[x] I consider the submitted work as finished
[x] I tested the code for its functionality
[ ] I wrote test cases for the relevant uses case
[ ] I added/update the relevant documentation (either on github or on notion)
[ ] Where necessary I refactored code to improve the overall quality

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

@DallasP9124 and @r-joyce this change is pretty simple. I don't see anything wrong with the code.
Going to request a quick review on this and a few other bug-fix PRs today. I think I can handle UI changes like these after this week.


inconsistent ordering of tags in GateFamily._default_description

Description of the issue
the order of tags in GateFamily._default_description() is not guaranteed to be consistent, because both _tags_to_accept and _tags_to_ignore are frozensets which get passed to list() in the f-string. Because this description is included in GateFamily._value_equality_values_(), this can lead to (what should be) identical GateFamilies being evaluated as not equal.
How to reproduce the issue
might require some repetition/modification to reproduce on different platforms, but this usually works me:
gate_fam1 = cirq.GateFamily(cirq.X, tags_to_accept=frozenset("abcd"))
gate_fam2 = cirq.GateFamily(cirq.X, tags_to_accept=frozenset("dcba"))

print(gate_fam1.description)
print(gate_fam2.description)
print(gate_fam1 == gate_fam2)

which will print the something like
Accepts `cirq.Gate` instances `g` s.t. `g == X`
Accepted tags: ['c', 'a', 'd', 'b']
Accepts `cirq.Gate` instances `g` s.t. `g == X`
Accepted tags: ['b', 'a', 'd', 'c']
False

where the two descriptions show the accepted tags in different orders and so the gate families aren't equal
Cirq version
1.1.0.dev20221025155635

Hi! Is this fix still needed? Could I please be assigned?
Thanks!

@chinmayee-vw Yes, this is still outstanding.  I've assigned it to you.

Yeah, we should only include the user-provided description in _value_equality_values_, which may be None, not the generated default description since that can be ordered differently.

@chinmayee-vw Do you still plan on working on this issue?

@tanujkhattar yes! still working on it.

@tanujkhattar I can take this issue up.


Moved object-assign to correct dependencies hash as dependencies was listed twice

related to https://github.com/aexmachina/ember-notify/issues/54#issuecomment-129957857

Haha, LOL. Thanks!


Support array_any_value

Is your feature request related to a problem or challenge?

Description
Returns the first non-null value in the list.

Example
list_any_value([NULL, -3])

Result
-3

Alias
list_any_value

Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Additional context
list_any_value duckdb

take

@pingsutw @Weijun-H  Can I take a poke at this issue if you are not working on it ? Has been stale for a while.

It seems that @pingsutw is not working on it. Please feel free to pick up @athultr1997

take


Add play simple tone sequence function

Current a sound updating is only possible in update() with 25ms step. However, when porting games from other system sometimes we need play much faster changed tone sequences. How about add a api function like PlaySimpleTones(uint16_t *seq, uint32_t volume) and a bool state to tell the sound updating callback to switch between normal sound or simple tone state? The Sequence could be a 16bit uint array [xxxhz,yyyyms,xxxxhz,yyyyyms], then we could have easy to use sound effects.

I found the solution to do it without changing the API... Just disable the 1ms sound update callback and replace it with my own one...


[BUG] AttributeError: 'dict' object has no attribute 'to'

/home/v/swarms/tests/models/test_idefics.py::test_run failed: idefics_instance = <swarms.models.idefics.Idefics object at 0x7f6a4c128a60>
def test_run(idefics_instance):
    prompts = [["User: Test"]]
    with patch.object(
        idefics_instance, "processor"
    ) as mock_processor, patch.object(
        idefics_instance, "model"
    ) as mock_model:
        mock_processor.return_value = {
            "input_ids": torch.tensor([1, 2, 3])
        }
        mock_model.generate.return_value = torch.tensor([1, 2, 3])
        mock_processor.batch_decode.return_value = ["Test"]

      result = idefics_instance.run(prompts)

tests/models/test_idefics.py:58:

self = <swarms.models.idefics.Idefics object at 0x7f6a4c128a60>
prompts = [['User: Test']], batched_mode = True
def run(self, prompts, batched_mode=True):
    """
    Generates text based on the provided prompts.

    Parameters
    ----------
        prompts : list
            A list of prompts. Each prompt is a list of text strings and images.
        batched_mode : bool, optional
            Whether to process the prompts in batched mode. If True, all prompts are
            processed together. If False, only the first prompt is processed (default is True).

    Returns
    -------
        list
            A list of generated text strings.
    """
    inputs = (
        self.processor(
            prompts,
            add_end_of_utterance_token=False,
            return_tensors="pt",

      ).to(self.device)

        if batched_mode
        else self.processor(prompts[0], return_tensors="pt").to(
            self.device
        )
    )

E       AttributeError: 'dict' object has no attribute 'to'
swarms/models/idefics.py:106: AttributeError

Upvote & Fund

We're using Polar.sh so you can upvote and help fund this issue.
We receive the funding once the issue is completed & confirmed by you.
Thank you in advance for helping prioritize & fund our backlog.

@vyomakesh09 I have just optimized this method and made it much better


Error with command sf after install plugin-lightning-dev

Summary
In VSCode
After install @salesforce/plugin-lightning-dev
sf command return
AuthDecryptError: Failed to decipher auth data. reason: Unsupported state or unable to authenticate data.
Code: AuthDecryptError
Steps To Reproduce:

sf plugins install @salesforce/plugin-lightning-dev

after install

sf doctor

AuthDecryptError: Failed to decipher auth data. reason: Unsupported state or unable to authenticate data.
Code: AuthDecryptError
after uninstall  @salesforce/plugin-lightning-dev

sf doctor
All is OK

VS Code Version:
1.93.1
SFDX CLI Version:
@salesforce/cli/2.58.7 win32-x64 node-v20.17.0
OS and version:
Windows 11

After installing plugin-lightning-dev when starting VSCode the following error appears :
Salesforce Extensions for VS Code encountered a problem while configuring your environment. Some features might not work. For details, click Help > Toggle Developer Tools or check the Salesforce CLI logs in ~/.sfdx/sfdx.log.

Sorry, I closed the issue by mistake

Command :

sf lightning dev app

Return :
Error (AuthDecryptError): Failed to decipher auth data. reason: Unsupported state or unable to authenticate data.
sf log :
"name":"sf:oclif","msg":"AuthDecryptError: Failed to decipher auth data. reason: Unsupported state or unable to authenticate data.\n    at SfCommandError.from (file:///C:/Users/xxx/AppData/Local/sf/node_modules/@salesforce/plugin-lightning-dev/node_modules/@salesforce/sf-plugins-core/lib/SfCommandError.js:39:16)\n    at LightningDevApp.catch (file:///C:/Users/xxx/AppData/Local/sf/node_modules/@salesforce/plugin-lightning-dev/node_modules/@salesforce/sf-plugins-core/lib/sfCommand.js:315:47)\n    at LightningDevApp._run (C:/Users/xxx/AppData/Local/sf/node_modules/@salesforce/plugin-lightning-dev/node_modules/@oclif/core/lib/command.js:305:29)\n    at async Config.runCommand (C:/Users/xxx/AppData/Local/sf/client/2.58.7-ed27925/node_modules/@oclif/core/lib/config/config.js:424:25)\n    at async run (C:/Users/xxx/AppData/Local/sf/client/2.58.7-ed27925/node_modules/@oclif/core/lib/main.js:95:16)\n


Incompatibility with Industrial Revolution

Bright Ore seems to be incompatible with Industrial Revolution. It causes certain items to have missing textures. I tested all the other mods I downloaded, and Bright Ores seems to be the only mod causing the issue.

Hmm... Please offer me these information:

Industrial Revolution version
Bright Ore version
Minecraft version
Is there any else mod installed?

Could you please try only loading these mods?

Fabric API (latest)
Bright Ore (latest)
Fabric Language Kotlin (latest)
Industrial Revolution (latest)

Thanks!


Homepage Navigation Component

User Story
As a website visitor, I want to be able to navigate to different sections of the website from the homepage using a clear and easy-to-use component, such as a navbar or a side bar.
Acceptance Criteria

The component is located prominently on the homepage and is easy to find.
The component includes links to the main sections of the website, such as Events, Membership, Sponsors, About Us, etc.
The component is consistent across all pages of the website.
Must include navigation links to different pages: Events, Membership, Sponsorship, and Login Portal

Priority
High
Story Points
5

Design


Bls for layer vrf

Integrated BLS for layer VRF
Fixed seed used for GenKeyPair
Fixed size of BLS sig

@antonlerner please check the seed initialization (I understood you already did something of relevance)
@noamnelke please review the rest


Sorry:) Revert "Wmhack2018"

Reverts commons-app/apps-android-commons#1536

Can this branch be deleted now?


Ban class with empty type parameter list

TypeScript Version:  nightly (2.3.0-dev.20170316)
Code
class X<> {}
function f<>() {}
const x: X<> = new X<>();

Expected behavior:
All are errors.
Actual behavior:
All but class X<> {} are errors. We should consistently fail when a type parameter list is empty.

I'd like to give this a shot if no one is already on it :)


[FEATURE] Add Module tainting functionality (force codebuild rerun)

Is your feature request related to a problem? Please describe.
During module development it is sometimes required to force a module to redeploy, even when no code changes are present and the MD5 hash doesn't change. This is because some modules e.g. ones that build docker images and pull resources into the container build context will change, when the code itself hasn't.
This is also the case for when there is a CF/CDK resource drift that has occurred and you need to rerun cdk deploy to detect the changes, but don't want to change the code to force a re-trigger as nothings has changed.
Therefore it is necessary to sometimes "taint" a module that would force seedfarmer to rerun the codebuild job, even if it detects no changes.
To do this currently, different teams do things like changing the readme's or in worse scenario's, removing the SSM parameters for a module.
Describe the solution you'd like
A command to taint a module to force rerunning of codebuild & all downstream dependencies if force redeploy is enabled.
seedfarmer taint --deployment <deployment> --group <group> --module <module>

Describe alternatives you've considered
Currently to enable tainting we've implemented the following script that allows specifying a module to taint and it updates the deployspec md5 to only zeros.
import argparse
import boto3
import json

def check_parameter_exists(ssm, parameter_name):
    try:
        ssm.get_parameter(Name=parameter_name)
        return True
    except ssm.exceptions.ParameterNotFound:
        return False

def update_ssm_parameter(deployment, group, module):
    ssm = boto3.client("ssm")

    project_prefix = "addf"

    print(f"Target: Deployment: {deployment}, group: {group}, module: {module} not found")

    # Construct the parameter name
    parameter_name = f"/{project_prefix}/{deployment}/{group}/{module}/md5/deployspec"

    if not check_parameter_exists(ssm, parameter_name):
        print(f"Parameter {parameter_name} does not exists")
        print(
            "Ensure you have the credentials loaded for your target environment and not the toolchain account"
        )
        exit(1)

    # New parameter value
    invalidated_value = json.dumps({"hash": "00000000000000000000000000000000"})

    try:
        # Update the parameter
        response = ssm.put_parameter(Name=parameter_name, Value=invalidated_value, Type="String", Overwrite=True)
        if "Version" in response:
            print(f"Successfully updated parameter: {parameter_name}")
            print(f"New version: {response['Version']}")
            print(f"New value: {invalidated_value}")
            print(
                "Module has been tainted. Rerun `$ make dry-run` to test and `$ make deploy` to trigger a redeployment"
            )
        else:
            print("Failed to update parameter")

    except Exception as e:
        print(f"An error occurred: {str(e)}")

def main():
    parser = argparse.ArgumentParser(
        description="Taints module by updating deployspec MD5 value to force SeedFarmer to retrigger"
    )
    parser.add_argument("--deployment", required=True, help="Deployment")
    parser.add_argument("--group", required=True, help="Group name")
    parser.add_argument("--module", required=True, help="Module name")

    args = parser.parse_args()

    update_ssm_parameter(args.deployment, args.group, args.module)

if __name__ == "__main__":
    main()

Additional context
Obviously using this script isn't ideal and having first party support for this kind of taining is extremely useful. It would also allow using this feature properly via the toolchain -> target deployment role mapping path instead of directly on SSM in the target account.

Thanks for this, @a13zen .  As we have discussed, the support for tainting is some that we have not embraced due to our declarative manifest paradigm, but we will investigate.


Update repositories to use the environment_template/environment.sh file as reference

This has been shown viable with RASberry.

Extending this, it should now use the properties.sh file additionally to hold descriptive information on the location.


Week 0 Plans, Team, Jobs Review Ticket - Dylan Luo

Scrum Team GitHub pages (2 points)

[ ] Web Project: Project Description, Project Plans/Ideas/Wires, Time box

[ ]  Team Roles described, Project Owner or Sponsor described, Scrum Board started

Individual GitHub pages (3 points)

[ ]  GitHub page for Test Prep study, requiring 5.1, 5.2 notes and actions( See "GitHub pages actions" at the bottom of each TPT page on cspcoders (Links to an external site.))

[ ]  GitHub page that documents Data Structures project, using code snippets for key learnings, and links to  GitHub for code and Replit for runtime

[ ]  GitHub page that documents intended Create Task project for College Board, snippets, and links to Runtime

Individual GitHub pages (3 points) Score: 2.75/3

[x] (1/1) GitHub page for Test Prep study, requiring 5.1, 5.2 notes and actions( See "GitHub pages actions" at the bottom of each TPT page on cspcoders (Links to an external site.))
Comment: Dylan had all the required notes and actions and was able to link it to a jekyll page.
[x] (0.75/1) GitHub page that documents Data Structures project, using code snippets for key learnings, and links to GitHub for code and Replit for runtime
Comment: Dylan's animations did not work, otherwise everything was documented and worked as intended
[x] (1/1) GitHub page that documents intended Create Task project for College Board, snippets, and links to Runtime
Comment: Dylan had a Create Task that fulfilled all college board requirements.


Qu in using

Hello! About 2 days ago I've tried to use dialog calendar.
Like that:
@dp.message_handler(state=Wallet.Graph_state)
async def painting_Graph(message: Message):
    currency = message.text
    await message.answer("Please select a date: ", reply_markup=await DialogCalendar().start_calendar())

@dp.callback_query_handler(dialog_cal_callback.filter())
async def process_dialog_calendar(callback_query: CallbackQuery, callback_data: dict):
    selected, date = await DialogCalendar().process_selection(callback_query, callback_data)
    if selected:
        await callback_query.message.answer(
        f'You selected {date.strftime("%d/%m/%Y")}', reply_markup=markup)

Can I use it in state machine or it incompatible with it?

@djskeletik, this help you https://ru.stackoverflow.com/questions/1398006/Проблема-с-aiogram-calendar-и-fsm

@djskeletik  in your example you are using state only in message_handler which suppose to show inline keyboard.
So at this point its not related to calendar yet. Can you explain what is not working?
Calendar provides keyboards & a series of callbacks. Overall it depends on your FSM use

thanks, I already figured it out, yesterday I wanted to delete it, but
somehow my hands did not reach)
Пн, 29 авг. 2022 г. в 15:21, Levman5 @.***>:

@djskeletik https://github.com/djskeletik, this help you
https://ru.stackoverflow.com/questions/1398006/Проблема-с-aiogram-calendar-и-fsm
—
Reply to this email directly, view it on GitHub
https://github.com/noXplode/aiogram_calendar/issues/4#issuecomment-1230212135,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATAOIC7ZBYNOHO2D5NCLPW3V3STOJANCNFSM57OXQ5PQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


Fix #918, Add send message with status API

Describe the contribution
Fix #918 - Add send message with status API
Testing performed
Built with tests, ran tests, passes
Expected behavior changes
Added API
System(s) tested on

Hardware: cFS Dev Server
OS: Ubuntu 18.04
Versions: bundle main + this commit

Additional context
Supports CF message flow control
Third party code
None.
Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman - NASA/GSFC

CCB 2020-09-30:

Will affect CF 3.0 implementation
This implementation might not be the most efficient but it's effective
There's already an event message that indicates problems with this but an app would have to subscribe to events and then do string operations to catch it.
Is there a requirement for this API?

Have splinter after CCB meeting

Closed in favor of #926 approach


propose new logo for desktop or smartphone

i would like to contribute for make this software logo, you can see my work https://utopian.io/@telurpuyuh
if anything you want for the logo, you can catch me at<EMAIL_ADDRESS>thank you.

Thanks. Will be in touch. I don't want to ask you for something that I may or may not end up using. Especially because you will end up spending your time on it.


Client Corruption State

Describe the bug
The CCS is a collection of issues that all leads to the same problem, quirky unpredictable weapon, it happens because of a few reasons, for month we had no clues but now we have everything we need.

I am currently rewriting rendering but will work on this personally right after I finished

Reproducibility

[x] I reproduced this issue with as few other mods as possible installed.
[x] I am unable to reproduce this issue consistently without modifications to the mod.

Screenshots or/and videos
Video Provided by @boubouleuh
Video Provided by @Koud-Wind
Logs or/and crash reports
Log Provided by @Creat789
Additional context
#244 #236 #201 #157 #151 #148

That video is not considered CCS to some extent

All of the videos I put are definitely caused by the CCS


Add support for tracker 2.0.

Moving tracker support from olproxy to olmod.  Only fires in olmod if you have appsettings configured for it.

Forgot to close it, landed in v0.2.7


[REQ] WhisperX support

First, I want to thank the author of this tool for simplifying the process of using OpenAI Whisper. Thanks to you, Fauzan, far more people are able to use the features of Whisper via a clean GUI.
As a feature request, I would love to see support added in your program for the latest enhancements added by WhisperX (https://github.com/m-bain/whisperX), which is a greatly-improved version of OpenAI's Whisper.
WhisperX is by a research group from the University of Oxford, is 70x faster than OpenAI Whisper, requires much less GPU memory running the language models, has a lower word error rate, does not have the hallucinations, drifting and repetitions that standard WhisperAI is prone to. The program detects when there is silence, can also detect when there are multiple speakers and identify each one uniquely, even with overlapping voices. It is also able to produce far more accurate timestamps, down to the level of individual letters in the words.
As it processes a recording, it splits the audio into 30 second chunks then batch processes them simultaneously for a dramatic speed increase. It appears to be different from WhisperJAX (https://github.com/sanchit-gandhi/whisper-jax) in that the released version of WhisperJAX splits the audio for batch processing without proper context, meaning that the cuts sometimes occur in the middle of words, which means that WhisperJAX ends up translating partial words, which generates a higher Word Error Rate. WhisperX does not do this. It scans before splitting, properly detecting the start and stop of words, so cuts happen in the spaces between.
I have been reading that WhisperX does a much better job translating various languages compared to OpenAI's version, which makes me think that proceeding with the current version of Whisper I have been using is fairly pointless, because the results would be inferior to WhisperX and I would need to re-do them later.
The problem is that I have been unable to get WhisperX running properly on my machine. I don't know which version/update of which dependency has broken the installation. I have reinstalled things multiple times and spent many hours trying to troubleshoot it. I know that there are many others experiencing similar problems like me. It would be great if you could provide support for either WhisperX or even Faster-Whisper (https://github.com/guillaumekln/faster-whisper) which is not as advanced as WhisperX, but is an improvement over regular WhisperAI.
Ideally, users would have the option to choose between OpenAI's standard version and a huge improvement like WhisperX. Combining the improvements of WhisperX with your GUI would be wonderful!
More info:
https://github.com/m-bain/whisperX (WhisperX GitHub source)
https://www.slashcam.com/news/single/WhisperX--Free-audio-transcription-with-speaker-re-17704.html
https://web.archive.org/web/20230301023005/https://www.swyx.io/transcribe-podcasts-with-whisper
https://arxiv.org/abs/2303.00747

That is long and very detailed, thank you. And yeah i'll try to add whisperx later on because I think i would like to make it so that the user can choose which backend to be used as an option
I have tried adding it before but it seems that there would need to be lots of refactoring so i decided to use whisper_timestamped for now, although the development has been kinda slow lately because of so much personal stuff that i'm doing right now aside from developing this app.
thanks for the suggestion!


Feature/animatable material label and textfields

AnimatableMaterialLabel, AnimatableMaterialErrorTextField and AnimatableMaterialTextView are displayed within the same viewcontroller

👍 Great work


No ad to show iOS

Describe the bug
I have recently released a mobile game on iOS and I was getting an error saying no ad to show before I released it and now it is released I am still getting this error. The intersticial ad unit was created on admob months ago and I have other apps with working adverts that have been out for a while now so my admob account is not new. Why am I still getting this error? Is it an admob problem?
Thanks
To Reproduce
Steps to reproduce the behavior:

Godot 3.2.3 iOS release app with intersticial

Closed as realised there is an iOS specific GitHub, sorry


Driver login tests

Updated the CI so it runs instrumented testing automatically
Fixed the tests so it removes keyboard before clicking on the button.
Problem that was making the CI emulator crashing was coming from the fact that the emulator is not using a pixel 3.
It could happen that when using another device, the connect button was hidden by the keyboard which was making the test fail.
The feedback from the CI is not really helpful though since it just indicated that the system crashed and that it did not receive the other tests which was misleading in identifying the problem.
A painful way to learn that we need to be careful to make sure that the different interactions are possible when designing our tests. 😄 but at least we have some Instrumented unit testing that are running ! 🥳 🎉
closes #67

Good job for adding automated tests :+1:


docs: try to make plugin/preset config less confusing

Motivation
Make it easier to understand that there are 2 ways to configure content plugins, and most users should use the preset config.
See confusion here: https://github.com/facebook/docusaurus/discussions/5246#discussioncomment-1087287
Preview; https://deploy-preview-5313--docusaurus-2.netlify.app/docs/api/plugins/@docusaurus/plugin-content-docs/

@lex111 @Josh-Cena any opinion on this doc structure?

@slorber looks good to me, that's much better!

@slorber LGTM!

thanks 👍


Move all images to Go 1.21

What this PR does / why we need it:

In https://github.com/openshift-knative/serving/pull/592 we missed a few images, had to run make generated-files.

/retest-required


Consider adding "batch" transactions

Suppose you have a chain of transactions, A -> B -> C, where each one spends outputs from the previous one. (This is only really a thing you want if you have scripts or multi-currency, since for "normal" transactions you can usually just merge them.)
At the moment, such a sequence will take |sequence| slots to fully validate, if we wait for each transaction to be successfully validated before submitting the next one, which is rubbish.
It would be nice to be able to "batch" such transactions.
Concretely, I propose an new alternative TxBody:
data TxBody = .... | Sequence [TxBody]

What does this mean?

The semantics are simple: it's exactly the same as applying each TxBody in sequence. It validates iff the whole sequence does. If part of it fails to validate then the whole thing is rejected.
It's a new kind of TxBody, which allows us to have witnesses over the whole sequence.

This would enable other use cases where multiple parties could coordinate to create transaction batches that interact with resources owned by different parties, and only proceed if the whole batch is signed. This allows e.g. atomic swaps.
The alternative would be data Tx = ... | Sequence [Tx], which would also work fine for batching.

I think this would be relatively straightforward to add. The wallet backend would need to know about them, but I don't expect the frontend would, since users probably don't want to make these very often. Rather, the clients are likely to be "smart contracts"/wallet backend applications.

This increasingly seems like something we'll want. In particular, at the moment you end up needing an administrative transaction to "jump start" use of a new currency. If we want people to use tokens heavily for e.g. roles then we want it to be as lightweight as possible, and having to do multiple transactions is slow and/or racey.
cc @polinavino

Flipping this around: this says that we can retain the property that dependent transactions can always be merged even in the presence of scripts, which is nice!
In fact, currently you can only merge dependent transactions if you're the author of both, since you need to sign the combined transaction which is different from the constituents. Batch transactions would let you merge transactions without disturbing the signatures.

I am not totally convinced of the advantage of having batch transactions. I think the type of functionality you're aiming for is better provided by things like Hydra or Lightning? If i understand correctly.
One thing that does concern me slightly is (due to the addition of the 2-phase validation for running scripts) the possibility of the creator of a block simply omitting a valid transaction that several other transactions depend on.
Here, I mean depend on in a way that does not cause the whole block to be scrapped, but rather phase 2 validation failures that result in more total fees being collected from the transactions in this block. Here, recall that a second-phase failure collects all fee-marked inputs, that could have greater total value than the required script fees that would be collected if all scripts validate.
Could this happen??

I think the type of functionality you're aiming for is better provided by things like Hydra or Lightning?

Could you elaborate?

Here, I mean depend on in a way that does not cause the whole block to be scrapped, but rather phase 2 validation failures that result in more total fees being collected from the transactions in this block.

I'm pretty sure this can't happen. The only way that transactions can depend on each other is via transaction outputs. So the creator of a block could either:

Omit a transaction. At worst later transactions will fail in phase 1 due to spending missing outputs.
Tamper with the outputs produced by an earlier transaction. But that would invalidate the signatures, and the txids, and all the stuff we have to prevent this.

(Also, I don't think this is about batch transactions per se, but rather about the process of validation in general?)

I'm pretty sure this can't happen. The only way that transactions can depend on each other is via transaction outputs. So the creator of a block could either:

Omit a transaction. At worst later transactions will fail in phase 1 due to spending missing outputs.
Tamper with the outputs produced by an earlier transaction. But that would invalidate the signatures, and the txids, and all the stuff we have to prevent this.

(Also, I don't think this is about batch transactions per se, but rather about the process of validation in general?)

Yeah, it is about validation in general. I was picturing a situation where if tx1 is processed, the UTxO will have in it
(hash tx1, ix) |-> (addr1, value1, datahash1)
If it does not get processed, there will not be any entry in the UTxO indexed by  (hash tx1, ix). If tx2 is being processed and trying to spend this entry, it will be phase 1 validation failure. If that entry is indeed in the UTxO, then the rest of the arguments passed to the validator cannot be effected by whether tx1 was processed or not. So, in short, looks like you are right not and this cannot happen.

I am not totally convinced of the advantage of having batch transactions. I think the type of functionality you're aiming for is better provided by things like Hydra? If i understand correctly.

Also never mind about this - I am convinced now that batching functionality could be useful

Alternative proposal sketch:
Instead of adding an additional transaction type to support batching/atomic sequences of transactions, relax the validation rules to allow batching in a single transaction.
For ordinary (non script) transactions, it is possible to batch together several payment into one as exchanges regularly do.
If we were to allow the spending of outputs within the same transaction as they are created then we could support the same 'monoidal' behaviour as before.
@michaelpj pointed out that this has the shortcoming that currently validator scripts can require very specific constraints on a transaction such as there are exactly a particular number of inputs etc. However, I think it's worth considering if such constraints are really needed/are not too intentional.

There might be some way to spend the outputs generated within the transaction, but it will involve some way of being clever about having references to those outputs inside the transaction itself. The usual way, (hash tx, ix) means that the hash of the transaction has to be calculated and put inside that same transaction - which can't happen

It just seems very hard to have this work nicely. Even very simple things will break. Consider e.g. someone who tries to find all the money in some currency by summing up all the outputs. Except - oops, in the merged trasnsaction it gives a completely different answer.
We would require validator authors to program very defensively against this condition, which they probably wouldn't, and which we could never assume that they had, so merging together random transactions would usually be unsafe.
So I think you'd really need to have the inputs/outputs "for each transaction" segregated and only seen by the validators in that tx. And then probably the same applies for the other tx metadata. And then you've just got batch transactions again.

I would really like to have this.
There are many interesting DeFi applications which would benefit from batching. Some, might be impossible to port from Ethereum without batching.
Batching also seems important internally for being able to support rollup like Layer 2s.
Conceptually I could see how validation would
work with batching.
Are there any plans to implement this?

It could be simply a heuristic for the mempool and block propose logic, like the honest nodes will propagate the batch together, and include them into the block together. But even if a node breaks them up, they can be still valid, we don't need to change ledger rules for this I guess.

This will overlap with limited liabilities (which also needs a CIP!).

At the moment, such a sequence will take |sequence| slots to fully validate, if we wait for each transaction to be successfully validated before submitting the next one, which is rubbish.

I was under the impression that there is no such limitation? Where is this encoded in the ledger?
E-mail from Duncan (I am not the one replied to) for reference:
> As a follow-up question: if a node's validation context includes the
> pending transactions that it has validated previously, does that mean
> that it can validate a transaction that attempts to consume an output
> from one of those pending transactions?

Yes indeed! Because as I note above, it's a sequence of transactions,
and the validation context is a ledger state, then yes there can be
dependent transactions. This is perfectly legitimate, and the
transaction propagation mechanism is designed to avoid reordering them.
So it's actually a use case that's supported by the design.

> In other words, is it possible for a chain of transactions to be
> added to the same block?

Yes. Now of course in practice the transactions have to be submitted in
order, so this is easier to arrange if it's one agent that is
submitting them all in a sequence.

This can also be noted from the section about colours from https://raw.githubusercontent.com/cardano-foundation/CIPs/ouroboros-leios/CIP-XXXX/leios-design.pdf (https://github.com/cardano-foundation/CIPs/pull/379)

At the moment, such a sequence will take |sequence| slots to fully validate, if we wait for each transaction to be successfully validated before submitting the next one, which is rubbish.

I was under the impression that there is no such limitation? Where is this encoded in the ledger?

@L-as I read it as if you are submitting a sequence of transactions and want to see they are each valid before submitting the next then it will take a few slots. With them batched it would be a single submission. (but not sure)
Though what you describe about having transactions which depend on transactions which come before it in the same block is possible and already seen on-chain.

I was under the impression that there is no such limitation? Where is this encoded in the ledger?

Indeed the key part was "if we wait for each transaction to be successfully validated before submitting the next one", i.e. if you want to know that transaction T1 was definitely added to the chain before submitting T2.

I don't see the use case for that scenario, though I definitely do want batching.

I'm no longer sure that non-atomic batches are that useful outside of limited liabilities (which changes things).
I do think it's quite non-obvious that the mempool has this property, and it's unclear if it's guaranteed at all. So perhaps we just need better documentation.

Non atomic batches can implement atomic batches potentially. Will put down my thoughts later.

Reason to have at least basic batching:
Some times a batch of transactions can be profitable even if the individual steps aren't all profitable.
Example: UTXO locked with script that allows you (and only you) to relock the funds with a script that allows anyone to consume it.
We'd like to take the funds and use it on something, which is technically possible in two transactions, but what if upstream drops the second transaction and only uses the first? We obviously want to prevent this, and a way to sign the entire batch such that the entire batch has to go through at once would be immensely useful.
It's possible that such a batch should have the same limits as a single transaction, since it acts like a single atomic transaction. The same reasoning as for transaction applies here.

Here's a summary of my current thinking on what we might want to do:

Atomic batches

An atomic batch is a batch transaction that cannot be split apart, and all of the transactions must apply in order for any of them to be applied.
Atomic batches are desirable because:

They restore the monoidal property of transaction bodies: you can now always merge two transaction bodies by making them into a batch.
They can let you avoid scripts, e.g. you can implement a simple atomic swap with just atomic batches and no scripts.
They can allow you to take a series of actions with no possibility of someone else intervening in the middle. As a somewhat contrived example, you could withdraw money from a DeFi contract in a way that would render you vulnerable to liquidation, use it for something, and then put it back where it was, all in one atomic move.

One concern is to ensure atomicity in the face of interference: we don't want other users to be able to disassemble batches, and resubmit only parts of them, as that would violate atomicity. This probably means:
a) we want the batch to be signed
b) the signatures must be resistant to stripping
c) it must not be possible to extract a part of the batch that is a valid transaction by itself
The only sensible way of doing this that I can see is for an atomic batch to be a new kind of transaction body, which is a sequence of transaction bodies. Then the witnesses (including signatures) are shared across the batch and naturally ensure integrity of the entire batch. Since the signatures are for the entire transaction body (the batch) they can't be reused for a subset of them.

Non-atomic batches

A non-atomic batch is a batch that can be broken apart.
Non-atomic batches are useful primarily in cases where you want multiple parties to collaborate in creating batches without two way communication (to e.g. agree on signing the whole thing).
The only real usecase that I know of for non-atomic batches is from the limited liabilities paper, where:

Liabilities are allowed, but only inside batches
Other parties can an incentive to "complete" a partial batch, allowing for a unidirectional matching system

Other than that, so long as nodes typically respect them they allow you to explicitly state the hint that you can implicitly give today by just submitting your transactions to your local node in that order. But if we had atomic batches then you could just use an atomic batch. In the context of Leios possibly this hint becomes more valuable if it can replace the use of colours with some thing more semantic - but again you could just use an atomic batch.
Since the pieces of a non-atomic batch should be valid independently, we must not sign them together. So a non-atomic batch would have to be a new kind of transaction that just consisted of a sequence of transactions.

Non-atomic batches are AFAICT not a suitable replacement to colouring. The important thing to note, is that if you can have liabilities inside them, then inherently the batch must be cancellable. This is problematic. Colours have no such behaviour, and a sequence of same-colour transactions can be potentially infinite with no issue.
Another thing I'm wondering about, is how are fees handled for atomic batches?
I have over the last year spent considerable brain cycles on protocol interoperability, have still not found a good solution.
Are atomic batches really true monoids? What if it doesn't fit in a block? Why would an atomic batch have a different size limit than a standalone transaction?
In the first place, it's not clear to me what the exact reasoning is for the limits.
The core issue of protocol interoperability is that there is no true monoid. Even if two protocols work separately, it is not always possible to combine them due to limits. If the limit is practically unreachable, that's not a worry, but this is far from the case currently.
If any of you want to discuss protocol interoperability I'm up for that (in real-life or digitally).

Colours have no such behaviour, and a sequence of same-colour transactions can be potentially infinite with no issue.

Why are these properties desirable, though? I'm wary of adding new desiderata based on the coincidental properties of the colour system. The stated reason for wanting colours is essentially to allow people to retain the current behaviour where they can submit a sequence of dependent transactions and have them processed in order. Batching seems like a more declarative way to get that behaviour without needing colours.

Are atomic batches really true monoids? What if it doesn't fit in a block? Why would an atomic batch have a different size limit than a standalone transaction?

They are as monoidal as pre-Alonzo transactions used to be, or Bitcoin transactions are. In essentially all cases we have size limits to deal with, so it's never going to be possible to get a true monoid. But we can get a monoid-up-to-limits, which is something. It's pretty convenient for building transactions and it does allow pasting things together in many cases.
It would be cool if we got a true monoid. That would require the individual bits of the batch to have independent limits, though :thinking:

Because often you have an infinite chain of dependent transactions working on the same protocol.
I'm still thinking of ways to get something that is mostly a monoid.

Okay, this is a new goal: support infinite sequences of dependent transactions, emitted at a steady rate (is that right?).
That doesn't seem to me like something that fits into the metaphor of "batches" and is something quite different.


serde 1.0.177 broke #[no_std] build

I am using nightly-2023-06-28 on apple silicon mac.
Compilation of serde on my machine results only recently in a wall of errors of not being able to find Ok and Err definitions, mainly in macros. The first error is about std not being found.
Maybe there is a feature allocated for no_std now?

   Compiling serde v1.0.177
error[E0463]: can't find crate for `std`
  |
  = note: the `thumbv7em-none-eabihf` target may not support the standard library
  = note: `std` is required by `serde` because it does not declare `#![no_std]`
  = help: consider building the standard library from source with `cargo build -Zbuild-std`

error[E0432]: unresolved imports `self::core::cmp`, `self::core::iter`, `self::core::mem`, `self::core::num`, `self::core::ptr`, `self::core::slice`, `self::core::str`
   --> /Users/mali/.cargo/registry/src/index.crates.io-6f17d22bba15001f/serde-1.0.177/src/lib.rs:171:26
    |
171 |     pub use self::core::{cmp, iter, mem, num, ptr, slice, str};
    |                          ^^^  ^^^^  ^^^  ^^^  ^^^  ^^^^^  ^^^ no `str` in `lib::core`
    |                          |    |     |    |    |    |
    |                          |    |     |    |    |    no `slice` in `lib::core`
    |                          |    |     |    |    no `ptr` in `lib::core`
    |                          |    |     |    no `num` in `lib::core`
    |                          |    |     no `mem` in `lib::core`
    |                          |    no `iter` in `lib::core`
    |                          no `cmp` in `lib::core`

error[E0432]: unresolved imports `self::core::f32`, `self::core::f64`
   --> /Users/mali/.cargo/registry/src/index.crates.io-6f17d22bba15001f/serde-1.0.177/src/lib.rs:172:26
    |
172 |     pub use self::core::{f32, f64};
    |                          ^^^  ^^^ no `f64` in `lib::core`
    |                          |
    |                          no `f32` in `lib::core`

// ...
error[E0425]: cannot find function, tuple struct or tuple variant `Err` in this scope
   --> /Users/mali/.cargo/registry/src/index.crates.io-6f17d22bba15001f/serde-1.0.177/src/lib.rs:295:32
    |
295 |             Err(err) => return Err(err),
    |                                ^^^ help: a local variable with a similar name exists: `err`
    |
   ::: /Users/mali/.cargo/registry/src/index.crates.io-6f17d22bba15001f/serde-1.0.177/src/private/ser.rs:274:9
    |
274 |         try!(map.serialize_key(inner_variant));
    |         -------------------------------------- in this macro invocation
    |
    = note: this error originates in the macro `try` (in Nightly builds, run with -Z macro-backtrace for more info)

error[E0425]: cannot find function, tuple struct or tuple variant `Ok` in this scope
   --> /Users/mali/.cargo/registry/src/index.crates.io-6f17d22bba15001f/serde-1.0.177/src/private/ser.rs:275:9
    |
275 |         Ok(SerializeTupleVariantAsMapValue::new(
    |         ^^ not found in this scope

// ...
// ...

error: could not compile `serde` (lib) due to 5071 previous errors

This is probably due to a dependency, closing.


The activity log is "This may take several minutes..." when deleting the container apps environment

OS: Win10
Build Version: 20231006.4
Regression: Previous release (0.5.1)
Repro Steps:

Create a new container apps environment.
Delete the above container apps environment.
Check whether the activity log is "Deleting Container Apps environment 'xxx'... this may take several minutes." when deleting the container apps environment.

Expect:
The activity log is "Deleting Container Apps environment 'xxx'... this may take several minutes." when deleting the container apps environment.

Actual:
The activity log is "This may take several minutes..." when deleting the container apps environment.

More Info:
There is also no log "Deleting Container Apps environment 'xxx'... this may take several minutes." in the Output window.

This is by-design. The old text was very long and hard to read. Since the title says "Delete container apps environment," we can assume that the user will know that is what "This may take several minutes..." is referring to.


use stream3

I can't see any harm in merging this. @substack? Should be okay if we bump the browserify major?

This is probably a good idea. Are there any other pieces we need to update at the same time?

Unfortunately the test/crypto.js and test/crypto_ig.js tests in browserify core fail with streams3:
      TypeError: evalmachine.<anonymous>:15118
          ctor.prototype = Object.create(superCtor.prototype, {
                                                  ^
      Cannot read property 'prototype' of undefined

This error looks like a cyclic dependency issue.

I'll find time and have a look. Probably will copy the stream code in iojs.


[REQUEST] "save runner" option in settings

so when my ob close for any reason and i neede recovery my left check or just when I want to move on from where I left off another time

You know that records are saved in the database? When you select a Wordlist / Config combination that was previously stopped it will recover your starting point ^_^ Is that what you meant?

Oh, this is really great. and that's really a secret trait :D wow so no need save runner bcs looks like it's already been saved xD
i ask this bcs i lose my all runners just %40 completed and i want done this check. rn i know solution for this thx ruri <3


InternalOpenIDError: Failed to verify assertion

I get this error using this repo out of the box.
I login into Quickbooks, then it redirects to the callback URL and fails to validate the signature in the sub open-id dependancy because the local computed hash does much match the openID token's hash.
Is there some config I'm missing in my app card configuration on the QuickBooks Developer portal?  My OpenID URL is set to http://localhost:3000/auth/intuit/return
have tired clearing browser cookies etc.

Same thing is happening to me.

Did either you guys ever resolve this? Also same.

Ok I fixed it. I probably should setup a pull request for this....
node_modules/passport-intuit/package.json dependency for passport-openid needs to be updated to "0.4.x"
   "passport-openid": "0.4.x",
After you update it make sure you go directly into the ./node_modules/passport-intuit folder and run
npm update
Then you can restart the app.js


Style can be problematic when a block has LOGBOOK

When a block has LOGBOOK (turning todo into doing, and after a while into todo or done), the style has problems:

I'm sorry, I didn't understand your meaning. Could you please provide more information?


[Question] Using PPO1 on a cluster

I'm having an hard time trying to use PPO1 (or any other algorithm) on a SLURM-managed cluster.
in particular, I considered the example reported in the documentation
import gym

from stable_baselines.common.policies import MlpPolicy
from stable_baselines import PPO1

env = gym.make('CartPole-v1')

model = PPO1(MlpPolicy, env, verbose=1)
model.learn(total_timesteps=25000)
model.save("ppo1_cartpole")

del model # remove to demonstrate saving and loading

model = PPO1.load("ppo1_cartpole")

obs = env.reset()
while True:
    action, _states = model.predict(obs)
    obs, rewards, dones, info = env.step(action)
    env.render()

and launched it with  mpirun -n 2 python fileName.py
Yet it results in the same code run twice
Did anyone ever experience these kind of issues?

Yet it results in the same code run twice

What do you mean?
Sounds like it is expected, the synchronization is done when computing the gradients.

This is an example of a piece of the output. It seems to me that the same code is just executed twice with no synchronization
********* Iteration 39 ***********
-0.00011 | -0.00607 | 127.74117 | 2.85e-06 | 0.60663
------------------------------------
| EpLenMean | 97 |
| EpRewMean | 97 |
| EpThisIter | 1 |
| EpisodesSoFar | 102 |
| TimeElapsed | 20.1 |
| TimestepsSoFar | 9984 |
| ev_tdlam_before | -0.00719 |
| loss_ent | 0.60663414 |
| loss_kl | 2.846489e-06 |
| loss_pol_entpen | -0.0060663414 |
| loss_pol_surr | -0.00011206418 |
| loss_vf_loss | 127.74117 |
------------------------------------
********* Iteration 39 ***********
Optimizing...
pol_surr | pol_entpen | vf_loss | kl | ent
-3.17e-08 | -0.00590 | 108.54033 | 7.48e-10 | 0.58984
-9.48e-06 | -0.00590 | 108.53023 | 3.53e-09 | 0.58985
Optimizing...
pol_surr | pol_entpen | vf_loss | kl | ent
-1.22e-05 | -0.00590 | 108.51881 | 7.80e-09 | 0.58985
-3.13e-06 | -0.00588 | 114.89399 | 4.77e-09 | 0.58785
-1.34e-05 | -0.00590 | 108.50693 | 7.38e-09 | 0.58986
Evaluating losses...
-2.34e-05 | -0.00588 | 114.88756 | 1.50e-08 | 0.58784
-1.84e-05 | -0.00590 | 108.49850 | 8.54e-09 | 0.58986
------------------------------------
| EpLenMean | 99.8 |
| EpRewMean | 99.8 |
| EpThisIter | 1 |
| EpisodesSoFar | 102 |
| TimeElapsed | 20.4 |
| TimestepsSoFar | 10240 |
| ev_tdlam_before | 0.763 |
| loss_ent | 0.5898608 |
| loss_kl | 8.537363e-09 |
| loss_pol_entpen | -0.005898608 |
| loss_pol_surr | -1.8380582e-05 |
| loss_vf_loss | 108.498505 |
------------------------------------
-1.61e-05 | -0.00588 | 114.88094 | 2.55e-08 | 0.58784
-2.33e-05 | -0.00588 | 114.87449 | 2.28e-08 | 0.58785
Evaluating losses...
-2.83e-05 | -0.00588 | 114.87064 | 1.36e-08 | 0.58786
------------------------------------


Documented s3 & DynamoDb URL flags a little bit more, added test case for awsConfig to show expected format.

Not a big fix, but I wish it was there before. This will assure that potential user will not spend too much time to figure out the logic here.
Signed-off-by: Bartek Plotka<EMAIL_ADDRESS>

Nice! LGTM


Improve UserManager accessibility

I was trying to override the CreateAsync method here  but I lose access to UpdateSecurityStampInternal(TUser user) and GetUserLockoutStore(). It would be helpful to make the stores as well as the updating of the security stamp accessible on derived classes.

I see there is a UpdateSecurityStampAsync that could be used when overriding the Create method, but it would still be nice to have access to the various GetNameOfStore() methods in derived instances of this class.


webpack({devServe:{}}) 出错//typescript node api #14006

https://github.com/webpack/webpack/issues/14006

To expedite the triage process, we need everyone to follow the issue template and instructions.
When you clicked "Create New Issue", the issue form was pre-populated with a template and some instructions. We need you to read those instructions completely, follow them, and then fill in all the fields in that template.
We are not able to assist with issues that don't follow the template instructions as they represent a significantly larger amount of work compared to issues which are correctly specified. Thank you for understanding.


Input size has type str but expected one of int, long

Platform (like ubuntu 16.04/win10):
ubuntu 18.04
Python version:
Python 3.6.4 :: Anaconda custom (64-bit)
Source framework with version (like Tensorflow 1.4.1 with GPU):
Caffe 1.0.0 without GPU
Destination framework with version (like CNTK 2.3 with GPU):
PyTorch 0.4.0 without GPU
Pre-trained model path (webpath or webdisk path):
this link automatically downloads the file
this one gives you the webpage I found the file on
You can download the files from the webpage if you scroll to "Code & DataSet"
Running scripts:
mmtoir -f caffe -n .../train_kingscollege.prototxt --inputShape 224 -w .../weights_kingscollege.caffemodel -o output
I get the final error as TypeError: '224' has type str, but expected one of: int, long
The error leads you to mmdnn/conversion/_script/convertToIR.py at the top.
The complete error message is:
Traceback (most recent call last):
  File "~/anaconda3/bin/mmtoir", line 11, in <module>
    sys.exit(_main())
  File "~/anaconda3/lib/python3.6/site-packages/mmdnn/conversion/_script/convertToIR.py", line 184, in _main
    ret = _convert(args)
  File "~/anaconda3/lib/python3.6/site-packages/mmdnn/conversion/_script/convertToIR.py", line 9, in _convert
    transformer = CaffeTransformer(args.network, args.weights, "tensorflow", args.inputShape, phase = args.caffePhase)
  File "~/anaconda3/lib/python3.6/site-packages/mmdnn/conversion/caffe/transformer.py", line 318, in __init__
    graph = GraphBuilder(def_path, self.input_shape, self.is_train_proto, phase).build()
  File "~/anaconda3/lib/python3.6/site-packages/mmdnn/conversion/caffe/graph.py", line 320, in __init__
    self.load()
  File "~/anaconda3/lib/python3.6/site-packages/mmdnn/conversion/caffe/graph.py", line 327, in load
    self.process_train_proto()
  File "~/anaconda3/lib/python3.6/site-packages/mmdnn/conversion/caffe/graph.py", line 358, in process_train_proto
    self.model.input_dim.extend(self.input_shape)
TypeError: '224' has type str, but expected one of: int, long

(Note: there were some modifications to error message but only in terms of /home/.../anaconda3 and replaced with ~/anaconda3/...)

Note: Adding the following
input: "data"
input_shape {
    dim: 64
    dim: 3
    dim: 224
    dim: 224
}

to the train_kingscollege.prototxt file resulted in the error:
mmdnn.conversion.caffe.errors.ConversionError: the train_val.prototxt should be provided with the input shape

which requires me to use the command in the original post and gives the original error.

I created a pull request to fix the input shape problem.
mmtoir -f caffe -n /Users/kit/Downloads/PoseNet/train_kingscollege.prototxt -w /Users/kit/Downloads/PoseNet/weights_kingscollege.caffemodel --inputShape 3 224 224 -o output

However, it gives the following error.
  File "/Users/kit/github/MMdnn/mmdnn/conversion/caffe/transformer.py", line 318, in __init__
    graph = GraphBuilder(def_path, self.input_shape, self.is_train_proto, phase).build()
  File "/Users/kit/github/MMdnn/mmdnn/conversion/caffe/graph.py", line 439, in build
    parent_node = graph.get_node(input_name)
  File "/Users/kit/github/MMdnn/mmdnn/conversion/caffe/graph.py", line 228, in get_node
    raise ConversionError('Layer not found: %s' % name)
mmdnn.conversion.caffe.errors.ConversionError: Layer not found: label_xyz

Then, I inspect the network. I find it might be some of the layers not well supported in MMdnn.


hotfix/galleries -> dev

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

byuSuite/Apps/Galleries/controller/GalleriesArtworkViewController.swift, line 31 at r1 (raw file):

        super.viewDidLoad()
		//Clear out previous artwork so that the collection view is empty when it first appears
		GalleriesArtworkViewController.artwork = [GalleryArtwork]()

Could you explain what the bug was? Are we saving the artwork statically for some reason?

Comments from Reviewable

Review status: all files reviewed at latest revision, 1 unresolved discussion.

byuSuite/Apps/Galleries/controller/GalleriesArtworkViewController.swift, line 31 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
Could you explain what the bug was? Are we saving the artwork statically for some reason?

The bug was that when you went to a gallery it would load the collection view just fine. If you went out of that gallery and into another one it would keep the old collection view cells visible until the new collection view loaded (because the data, artwork, was not cleared and was loaded instantaneously) . Its static so that the artwork is cached and can be used with the GalleriesArtworkPageViewController and they can refer to the same data.

Comments from Reviewable

Review status: all files reviewed at latest revision, 1 unresolved discussion, all commit checks successful.

byuSuite/Apps/Galleries/controller/GalleriesArtworkViewController.swift, line 31 at r1 (raw file):
Previously, alexanderboswell (Alexander Boswell) wrote…
The bug was that when you went to a gallery it would load the collection view just fine. If you went out of that gallery and into another one it would keep the old collection view cells visible until the new collection view loaded (because the data, artwork, was not cleared and was loaded instantaneously) . Its static so that the artwork is cached and can be used with the GalleriesArtworkPageViewController and they can refer to the same data.

Gah. I remember when we decided to do that... I knew it would come back to bite us :-/ Static variables are never the best solution :p

Comments from Reviewable


Customize ValidateIssuer

Hi,
Thanks for this library!
I was wondering why it is not possible to customize the value of ValidateIssuer (it is hard-coded as true). I'm hitting an issue during dev when running Keycloak in Docker, since the Keycloak host name is one thing to other containers in the stack whereas it's localhost from my machine. It would be much easier if I could override this to false during development.

Nevermind, I figured out I need to customize this via the JwtBearerOptions:
jwtBearerOptions.TokenValidationParameters.ValidateIssuer <- false


SIGILL in <crossbeam_channel::flavors::zero::Channel<T>>::recv

Hi! I am seeing this very weird SIGILL. Unfortunately, I can't give a nice steps to reproduce because there are a lot of moving parts here. I am not even sure that it is a problem with crossbeam-channel. However I think it makes sense to report the issue nonetheless :)
I've recently added crossbeam channel to RLS: https://github.com/rust-lang-nursery/rls/pull/923.
I don't actually send any messages and simply wait for channels to close, taking advantage of the select! macro. Specifically, I see SIGILL in this bit of code:
https://github.com/rust-lang-nursery/rls/blob/0b9254b7dcdf52bba50a6e477d7891fb23adf62b/src/concurrency.rs#L44-L57
The most interesting part is that I get SIGILL only when testing RLS inside rust-lang/rust repository. That is, cargo test inside RLS itself works.
I get rust-lang rust checked out locally to  29ee65411c46b8f701bd1f241725092cb1b347e6 commit, and the src/tools/rls submodule checked out to https://github.com/matklad/rls/commit/746b0f4563eb42783d5c679f27951ca2f79b1080.
With this setup, runnig ./x.py test src/tools/rls sigills.  See the rls commit above for the precise point where it happens.
So, this is definitely not the most self-contained and easy to reproduce bug report, but this is the best I have at this time :-)

Oh, and here's how the failure looks like

Thread 2 "test::fail_unin" received signal SIGILL, Illegal instruction.
[Switching to Thread 0x7fffef9ff700 (LWP 14791)]
0x000055555572334a in <crossbeam_channel::flavors::zero::Channel<T>>::recv ()
(gdb) bt
#0  0x000055555572334a in <crossbeam_channel::flavors::zero::Channel<T>>::recv ()
#1  0x000055555570a753 in rls::concurrency::Jobs::wait_for_all ()
#2  0x00005555556dc719 in rls::actions::InitActionContext::wait_for_concurrent_jobs ()
#3  0x00005555556bee25 in rls::test::harness::expect_messages ()
#4  0x0000555555755d02 in rls::test::fail_uninitialized_request ()
#5  0x00007ffff5ee4f5f in <F as alloc::boxed::FnBox<A>>::call_box ()

I've tried replacing the crossbeam_channel with mutexes and condvars, and now that test blocks forever, which kinda hints that my code is to blame. Can it be the case that crossbeam aborts for some misuses of the API (I see some aborts in the code)? If that is the case, then printing a message might be helpful :)

I cannot reproduce the issue. :(
Does it also happen in debug mode? It'd help to see which line in the code caused SIGILL.
Another thing that'd help would be to see where the other threads got stuck at the moment of SIGILL, too. For example, it would be telling if some other thread was inside zero::Channel::<T>::send() when zero::Channel::<T>::recv() hits the SIGILL.

Oh, it could be because ./x.py test src/tools/rls automatically switches src/tools/rls from branch 746b0f4 to 2b57851. Is there something I can do to avoid that?

Is there something I can do to avoid that?

Don't know of a proper way to do this, I just make a commit on rust-lang/rust repo which updates submodule.

Ok, I've managed to reproduce the issue.
Do you know how to run the test in gdb? This is what I'm getting:
$ /home/stjepan/work/rust/build/x86_64-unknown-linux-gnu/stage2-tools/x86_64-unknown-linux-gnu/release/deps/rls-3ec593da9fd41bf7
/home/stjepan/work/rust/build/x86_64-unknown-linux-gnu/stage2-tools/x86_64-unknown-linux-gnu/release/deps/rls-3ec593da9fd41bf7: error while loading shared libraries: librustc_driver-123213898594acb6.so: cannot open shared object file: No such file or directory

I've used the following:
 LD_LIBRARY_PATH="/home/matklad/projects/rust/build/x86_64-unknown-linux-gnu/stage2/lib/rustlib/x86_64-unknown-linux-gnu/lib:$LD_LIBRAY_PATH" gdb --args ~/projects/rust/build/x86_64-unknown-linux-gnu/stage2-tools/x86_64-unknown-linux-gnu/release/deps/rls-3ec593da9fd41bf7 --test-threads 1 test::fail_uninitialized_request --nocapture

Also, is there a way to run the test in debug mode rather than release?

Don't really know about this. I actually don't understand the specifics of x.py :(
One thing I've tried though is to setup the stage2 as a custom rustup toochain and then do cargo t inside src/tools/rls, but that didn't trigger the behavior.
One thing I realize now is that I've always tested in debug inside rls repo. Maybe --release is important for triggering the issue?

@stjepang yep! Running cargo test --release in the RLS repo triggers the bug! Sorry for making you to compile rustc :D

As a sanity check, test --release works as expected with the condvar version.

I've just published v0.2.3 that should fix this bug.
Can you confirm that it's resolved on your machine, too? :)

Yep, works on my machine!
Thanks for the swift fix!


Rename SolrCloud to SolrCluster

The Solr community is trending in the direction of renaming "Solr Cloud" to "Solr Cluster". If that's the case, then the CRD "SolrCloud" should likely be renamed as well.

https://issues.apache.org/jira/browse/SOLR-12613

The opportune time to make this change in the CRD would be before v0.3.0 is released, because users are already going to have to update the apiGroup of their CRDs. Changing the type will just be an extra step.
Before we implement this, we should be sure that the renaming is going to occur.

Closing as this does not seem to be moving forward in Solr either.


.html() throws an error when node list is empty

As mentioned in the title, cash throws an error when trying to call .html() on an empty node list.
The same goes for .text(), but not .attr("stuff"), for example.
I switched from jquery/jqlite to cash and both of them simply return undefined when the node list is empty :)
I'd be happy to do a PR, but the repo seem to be a bit dead so I'll let you guys deal with it when you have some free time 😄
repro : https://plnkr.co/edit/O3MvsQxcVXWINGhyaqs8?p=preview

This, and many other issues, are fixed in my fork.


support VarPF variantes for options to return created flags

The actuals variants of the flag creation do not report back the generated Flag object. But this would be reqired to do further tweaking according to the public fields of a Flag, like adding of annotations.
Therefore it would be useful to have an additional variant of the option creation methods on a FlagSet, which returns the generated Flag object.
The underlying method VarPF already exists and can directly be used to support appropriate VarPF variants for the type specific methods.
For example
func (f *FlagSet) StringVarPF(p *string, name, shorthand string, value string, usage string) *Flag

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


[WIP][INFINITY-1479] container based test

Try to provide a stable environment for running test automation, so they may be less flaky

I'm going to close this, as we have a working container based solution in https://github.com/mesosphere/dcos-commons/pull/1204.
Lets iterate there to get a proper solution to INFINITY-1479.


Can't find custom attr constructor image UnityEngine.CoreModule.dll

I'm getting Can't find custom attr constructor image<EMAIL_ADDRESS>mtoken: 0x0a00000f due to:  assembly:/opt/Unity/Editor/Data/Managed/UnityEngine/UnityEngine.CoreModule.dll type:UnityAPICompatibilityVersionAttribute member:void {0}(string,string[]) signature:.ctor Unloading 0 Unused Serialized files (Serialized files now loaded: 0)
My .yml config file:
name: UMVC.Editor Tests

on:
  push:
    branches:
      - master
      - release/*
      - staging/*
      - pre-release/*
  pull_request:
    branches: [ master ]

#env:
#  UNITY_LICENSE: ${{ secrets.UNITY_LICENSE }}

jobs:
  test:
    name: Test UMVC.Editor
    runs-on: ubuntu-latest
    strategy:
      matrix:
        unityVersion:
          - 2019.3.12f1
        targetPlatform:
          - StandaloneWindows64
        testMode:
          - editmode
    steps:
      - uses: actions/checkout@v2

      # Set the UNITY_LICENSE environment variable
      - name: Setup license
        # This step:
        #   - Grabs the license file contents from the $LICENSE_FILE_PATH
        #   - Uses substitution to escape newline characters for GitHub Actions set-env.
        #     (from: https://github.community/t5/GitHub-Actions/set-output-Truncates-Multiline-Strings/td-p/37870)
        #   - Set the UNITY_LICENSE env var for all future steps using special GitHub Actions syntax
        env:
          LICENSE_FILE_PATH: ./Licences/Unity_v2019.x.ulf
        run: |
          license=$(<"$LICENSE_FILE_PATH")
          license="${license//'%'/'%25'}"
          license="${license//$'\n'/'%0A'}"
          license="${license//$'\r'/'%0D'}"
          echo "::set-env name=UNITY_LICENSE::$license"
      # Delete unity version file (test runner will crash otherwise)
      - name: Delete ProjectVersion.txt
        run: sudo rm ./${{ matrix.projectPath }}/ProjectSettings/ProjectVersion.txt

      # Run tests - only edit mode supported
      - name: Run tests
        uses<EMAIL_ADDRESS>        with:
          projectPath: ${{ matrix.projectPath }}
          unityVersion: ${{ matrix.unityVersion }}
          artifactsPath: ./testReports/${{ matrix.targetPlatform }}
          testMode: editmode

You can find the log at: https://github.com/loic-lopez/UMVC/pull/16/checks?check_run_id=659008680

@loic-lopez Thanks a lot for reporting your issue with Unity Test Runner.
Unfortunately I am unsure about the cause of this issue. Hoping for more people to encounter a similar issue or offer suggestions.

Looking at this again, you were missing project path in the matrix part.
Furthermore this issue seems to be very much project related and not unity-test-runner related.
I see you have found a different non-unity way to run your test, so I'm closing this issue. Please feel free to reopen if you have more information.

hi, yes it an issue with MSBuildForUnity who generate dlls so I have prebuilded the dll in my project before launching the tests


feat: key management for token holders on ibet-Prime enhancement

draft
close #216

I would implement this after we fix interface, so I will leave this PR draft.

Close as stale.


best practice in defining global deps

<Have you checked issues section which may have your similar issue?>

Issue Overview
I am trying to add Tether (required by Bootstrap 4) - but is not getting picked up in the build.
Issue Description
I am trying to add Tether (required by Bootstrap 4) - but is not getting picked up in the build. This is probably a SystemJS question, but wondering if the starter app has this covered.
Reproducables
<Please explain the way to reproduce your issue, or even better with Plunker link>
Information

Operating System
N/A

Node version
N/A

NPM Version
N/A

Environment
N/A

do you mean this tether?
they support bower package so it can be build using useref
but if you use the npm package, you need to register it here, and import them in ts file (and yes, you need to install its typings too)

bower it is. thanks


Add meltano orchestrate to start current orchestrator

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/2231
Originally created by @DouweM on 2020-08-11 21:43:13

With Airflow, this would be equivalent to running meltano invoke airflow scheduler, but different (future) orchestrators would specify their own run_args.

View 4 previous comments from the original issue on GitLab


Converting models with mutable resource variable with v2 TFLiteConveter

1. System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Colab
TensorFlow installation (pip package or built from source):
TensorFlow library (version, if pip package or github SHA, if built from source): 2.4.1

2. Code
This is how the original model is built:
import tensorflow as tf
from tensorflow.keras.applications import MobileNetV2
from tensorflow.keras.layers import *

def get_student_model():
    base_model = MobileNetV2(weights=None, include_top=False,
                input_shape=(224, 224, 3))
    base_model.trainable = True
    inputs = Input(shape=(224, 224, 3))
    x = experimental.preprocessing.Rescaling(1./127.5, offset=-1)(inputs)
    y = base_model(x, training=True)
    y = GlobalAveragePooling2D()(y)
    y = Dense(512, activation="relu")(y)
    y = Dropout(0.5)(y)
    outputs = Dense(30, activation='softmax')(y)
    model = tf.keras.Model(inputs, outputs)
    return model

This is how the TensorFlow Lite model is generated:
import tensorflow as tf

converter = tf.lite.TFLiteConverter.from_saved_model("mobilenetv2_student_no_true_labels_e_125_t_2")
converter.experimental_new_converter = False
converter.optimizations = [tf.lite.Optimize.DEFAULT]

tflite_model = converter.convert()
open("student_mobilenetv2.tflite", 'wb').write(tflite_model)
print('Model size is %f MBs.' % (len(tflite_model) / 1024 / 1024.0))

3. Failure after conversion
When trying to convert with converter.experimental_new_converter = True the conversion runs into -
ConverterError: <unknown>:0: error: loc("Conv_1_bn/moving_mean"): is not immutable, try running tf-saved-model-optimize-global-tensors to prove tensors are immutable

With tf-nightly (2.5.0-dev20210312) this issue seems to be persisting as well.
As per https://github.com/tensorflow/tensorflow/issues/43833#issuecomment-744087884, I understand with the MLIR converter this is probably still unsupported.
What is more surprising are the outputs of the TFLite models converted with the TOCO converter. I have tried with just the float model i.e. when the conversion is done without any optimization flag set. The results remain fixed.
Here's the Colab Notebook that reproduces this issue: https://colab.research.google.com/gist/sayakpaul/2326ccd94e3d33e1c1ced7dfd7f09519/inference_tflite.ipynb.
Cc: @MeghnaNatraj @abattery

I further looked into the issue and it seems to be stemming from this line, in particular training=True -
y = base_model(x, training=True)

I instantiated a blank model with the utility function provided above and removed the training argument. Doing so allowed me to do the conversion seamlessly. I will try the model again with this change and report back.
Hope this helps someone facing a similar issue.

Hi @sayakpaul
We are working on the mutable variable support. I will notify this thread once the on-going work is done. Thanks for the filing this feature request!

@abattery sure.
But currently, with the TOCO converter, I am able to run the conversion successfully but that indeed results in spurious outputs. Is this expected or a known behavior?

@sayakpaul I am not sure the converted model with TOCO will work correctly for mutable variables.

I see. So, the best bet, for now, would be to implement a similar architecture and train using the same loop, and then run the conversions. Am I correct?

Could you try using the following code snippet with the tf-nightly version and linking the tensorflow/lite/kernels:variable_op_kernels?
converter._enable_tflite_resource_variables = True

The above experimental items are subject to change since it is under development.

@abattery yes, it works now, thanks. Here's the Colab. We need to convert it with the flex ops:
converter = tf.lite.TFLiteConverter.from_saved_model("mobilenetv2_student_no_true_labels_e_125_t_2")
converter._enable_tflite_resource_variables = True
converter.optimizations = [tf.lite.Optimize.DEFAULT]
converter.target_spec.supported_ops = [
  tf.lite.OpsSet.TFLITE_BUILTINS, # enable TensorFlow Lite ops.
  tf.lite.OpsSet.SELECT_TF_OPS # enable TensorFlow ops.
]
tflite_model = converter.convert()
open("student_mobilenetv2.tflite", 'wb').write(tflite_model)
print('Model size is %f MBs.' % (len(tflite_model) / 1024 / 1024.0))

Mutable variable support is available in the nightly when converting from SavedModel only (Other types are coming soon).
Builtin ops are available now for VarHandleOp, ReadVariable, AssignVariable in the nightly also
Only thing needed is to set this flag during conversion:
converter.experimental_enable_resource_variables = True
Please let us know if you're having any issues.
Thanks


fix component imports and rename track event

closes #2151

Code looks good. Failing tests though. Pulling locally to test.

@eatonphil s/b fixed

Coverage increased (+0.02%) to 78.412% when pulling c1651030376c0f9c688db94397fdd4f86cf2334a on mparke:adjust-component-imports into 4bf3cc70aa8bc9ff0ad7e3747771717174aa885d on linode:develop.


ScraperB3Curves_sample

just a sample code for B3 curve webscraper

This looks like just an example file. I will not put this is our repository. Examples will soon be in the documentation.


Union[Unset, datetime.date] is not allowed in header

Describe the bug
The parameter name='IF-MODIFIED-SINCE' produces an error 'Union[Unset, datetime.date] is not allowed in header', which prevents the endpoint from being generated.
OpenAPI Spec File
https://docs.developers.clio.com/openapi.json
Desktop (please complete the following information):

OS: macOS 13.4.1
Python Version: 3.12
openapi-python-client version: 0.15.2

Additional context
The parameter schema is:
"schema": {
"type": "string",   (the format in "YYYY-MM-DD")
"format": "date"   (the format defined by full-date in RFC3339)
}
The client generator is attempting to map this schema to a Python Union type, the union of Unset and datetime.date, which is not allowed or fitting in a HTTP header. Maybe string dates need to be transformed to datetime objects and vice versa? Or the date could be converted to a string before setting the header? At least that's my limited understanding of the situation.

I'm getting a similar issue with this https://raw.githubusercontent.com/wikimedia/mediawiki-extensions-Wikibase/e45d4f725e25a95ec2b034238fd3736f19053a16/repo/rest-api/specs/openapi.json (moving this comment from #924).
In this case it cries about:
Union[Unset, List[str]] is not allowed in header

where I'm assuming the issue is it doesn't understand how to map a list of strings to a string. Looking at the swagger endpoint for this API (https://doc.wikimedia.org/Wikibase/master/js/rest-api/#/items/getItem) it seems they want us to encode a list of strings as If-Match: "1276703231","1276705620"' . But I'm not sure if that's a standard openapi thing or if that's just some hack. If it's the latter it should probably have built in support.

My two cents: This is possibly one of those missing pieces of openapi 3.x support. In other words, this is caused by schema: section of a (e.g. header) parameter. If you give the type directly, rather than under schema: subsection, there is no error.


Updated doc & included Naming Tool

Have broken out the document a bit more to make it easier to digest, incorporated references to highlight these are examples only, added CAF tools references for quick access, and cleaned up charts both in providing standard and to reduce examples vs trying to include all naming resources.

@shbabylo : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

@dominicallen - This looks solid can we merge it.

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@JanetCThomas I'm working to resolve merge conflicts. I need to check the update against what's in master one more time, and then I will merge.

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

I'll merge the PR now. Substantial updates had been made in master in the private repo, resulting in substantial merge conflicts. I would recommend a careful review after the public and private repos sync, to be sure that the version you end up with in master in the private repo is the version you want in all sections.

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.


use x.SQL instead of x.Sql

Since xorm have provide SQL and will deprecate Sql, so we can use SQL instead of Sql.

It seems drone is not work.

Current coverage is 3.14% (diff: 0.00%)

Merging #129 into master will not change coverage

@@            master      #129   diff @@
========================================
  Files           33        33
  Lines         7823      7823
  Methods          0         0
  Messages         0         0
  Branches         0         0
========================================
  Hits           246       246
  Misses        7557      7557
  Partials        20        20

Powered by Codecov. Last update 145648a...7dcc3bc

LGTM

LGTM

@lunny In my opinion we shouldn't use x.SQL unless absolutely necessary. If you're fixing all of them, why not change them to x.Select/x.Update etc?

SQL method allow you custom your SQL statement when ORM can not satisfy your requirements . Select and Update have beend used on other place.


add flameshot

Project URL

https://github.com/flameshot-org/flameshot
Category

Images
Description

Powerful yet simple to use screenshot software.
Why it should be included to Awesome macOS open source applications (optional)
It's one of the best screenshot tools I have seen!
Checklist

[x] Edit applications.json instead of README.md.
[x] Only one project/change is in this pull request
[x] Screenshots(s) added if any
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

@MohamadKh75 Thanks!


Project: Employee AI Assistant

Project Name
Employee Assistant
Description
Employee AI Assistant is a Chat bot built using Azure open ai, Azure Search and python where employees can find answers to their questions regarding healthcare plans and company policies for a fictitious company called contoso.
DataSource:
I have used sample pdf files of a fictitious company called contoso for this use case.
Services used to implement solution:

Azure open ai service(Generator)
Azure search service(Retriever)
Azure blob storage(data source)
python+gradio(App/UX)

Key Features:

Azure AI Search Integrated Vectorization
Key less Auth( All Azure Services use azure ad auth and RBAC to communicate with each other)

Who can benefit from this app:

Customer Service Reps can save tons of time by offloading common employee queries to this chatbot and focus on critical issues which need human intervention.
Internal employees of any company will get more benefit from this app by getting answers to their questions quickly without having to go back and forth with customer service reps.

Technology & Languages

[ ] JavaScript
[ ] Java
[ ] .NET
[X] Python
[ ] AI Studio
[X] AI Search
[ ] PostgreSQL
[ ] Cosmos DB
[ ] Azure SQL

Project Repository URL
https://github.com/vikramarigelaRepo/azure-openai-search-rag-demo.git
Deployed Endpoint URL
No response
Project Video
https://youtu.be/VRIlwguGJvs
Team Members
vikramarigelaRepo

Hello @vikramarigelaRepo, thank you for participating in RAG Hack!
The team is working hard to distribute badges. Please have each team member fill out this form:
aka.ms/raghack/badge-dist
Thank you!


rkyv support

I'm trying to use openraft with 20k+ messages per second, and I seem to be running into quite a bit of overhead simply allocating/copying/serializing stuff. I'm currently using bincode, but would like to switch to rkyv as I believe it will do better.
Unfortunately rkyv doesn't work with serde Serialize/Deserialize (due to fundamental differences around supporting zero-copy), and thus  there would need to be support added into openraft for me to be able to use rkyv to serialize things like VoteRequest/AppendEntriesRequest/InstallSnapshotRequest.
I think it would just be a matter of adding some different derives behind a feature flag for rkyv. Is there interest in this? If so I will try to make a PR.

There is an ongoing PR to make serde an optional feature:

#243

What you need looks similar to this? An optional feature that enables several rkyv derive?
I do not have any experience on rkyv other than reading its API doc:)
A PR is welcomed! :DDD

Adding rkyv support would be a nice addition indeed. It's a very efficient format wrt (de)serialization speed which can easily become a bottleneck.
IMHO the solution outlined in the comment https://github.com/datafuselabs/openraft/pull/243#pullrequestreview-901486343 in the currently open serde PR would be a very simple and good solution wrt maintainablity.

Hi All, I'm looking at adding this feature and ran into a small issue. Is there any issue with moving away from using AnyError directly in some of the messages? An example: https://github.com/datafuselabs/openraft/blob/main/openraft/src/error.rs#L319
The issue being that AnyError is defined in the external crate so we can't easily add the rkyv traits to it

Hi All, I'm looking at adding this feature and ran into a small issue. Is there any issue with moving away from using AnyError directly in some of the messages? An example:
https://github.com/datafuselabs/openraft/blob/db047b978e08fdb6f23bc4c27f94d4bd36ab925b/openraft/src/error.rs#L314-L320
The issue being that AnyError is defined in the external crate so we can't easily add the rkyv traits to it

(Edit) I see now that the AnyError crate is actually apart of drmindrmer's repositories. I guess my question becomes if rkyv support should be added there?
(Edit 2) Looking more closely at this, it doesn't appear that the error's themselves would/should ever be sent over the network so it doesn't look like there would be any reason to add this serialization to them. I should have a pr for this sometime today.

It is possible to add rkyv to AnyError.
But AFAIK, only types that need to be persisted on disk need rkyv support.
Errors in openraft are only used for transport.
Does it have to be encoded in the rkyv format for transport?
E.g., for `send_append_entries():
https://github.com/datafuselabs/openraft/blob/1202d1b059f7e07f0f3353b74f0b2795f063ab56/openraft/src/network.rs#L49-L52
The response and response errors are small structs, maybe it's possible to use some other serialization such as serde to encode responses?


Update the page and create the icon logo

Hello,
I updated the page and create a title logo in browser. Please check it thanks! :)

Good idea with adding the pumpkin favicon :)
Please add yourself to the Contributor list if you haven't already.


Fill coloctapp gaps

From #929  , related to #974
coloctapp
Between September 29, 2021 and February 02, 2022 we have 0 documents. We are missing documents, but must go into PDFs to get them now

@flooie can you please check this source? From what I see we may need to parse the pdfs, since old case information is not available on HTML, as it is in colo

Hmm.  Is it possible the html changes.

CO was one of the worst states. Does this mean it's finally not so terrible?

This new Colorado site seems to have no search filters except for "court". Getting the document url requires more steps/requests. And it uses vlex as the backend. The downloaded opinion PDF comes in a zip, and the document has a vlex link in it

So, I don't know if it qualifies as not being terrible, but at least it will let us look for past opinions without going into PDFs

So, we also have a more recent gap. We are missing every Opinion announced on:

April 4, 2024 (2 opinions)
March 14, 21, 28 (2, 1 and 3 opinions respectively)
February 1, 15, 22, 29 (...)

I don't know why the scraper has been failing...

Command to fill the gaps
docker exec -it cl-django python /opt/courtlistener/manage.py cl_back_scrape_opinions --courts juriscraper.opinions.united_states.state.coloctapp --backscrape-start=09/28/2021 --backscrape-end=02/01/2022

The old scraper went down some  months ago. Most recent colotctapp opinion is from March 7th, 2024, so this is a new gap


Cannot clear selection once value is selected.

downshift version: 1.4.0
node version: N/A
npm (or yarn) version: N/A

Reproduction repository:
https://olrv026pz6.codesandbox.io/
Problem description:
Using the example from the Usage page:

Place the focus on the input element
Type "app"; the drop down list will show a single element, "apple"
Click the drop down item, or press Enter to select the "apple" item
Highlight "apple" in the input field.

At this point, I would expect that hitting Backspace to clear the selection and then using the mouse, hitting Enter, or pressing Tab to leave the input field would clear the value.
However, at this point, the field populates itself with "apple".
My expectation is that the selection would be cleared out given one of those actions.

Hi @casperOne!
Thanks for trying downshift :) This is something you can implement yourself. This kind of behavior is one of those things that has a wide variety of use cases, so I chose a pretty common one when making downshift, and if you want something else to happen, then you can do it yourself: https://codesandbox.io/s/xrzj5l487o
In this example, I added an onChange to the input and had it call clearSelection. I also added a div to render the "Selected Item" so you know what the selected item is. I hope that helps :)
You might also be interested in the Typeahead example: https://codesandbox.io/s/82m2px40q9
Good luck!


Follow doesn't work (on showing intermediary responses)

Per  Showing intermediary redirect responses documentation I should be able to follow redirects. However with the latest httpie on Ubuntu (Ubuntu bash on Windows 10), this doesn't work. On running:
http --follow --all httpbin.org/redirect/3
I get:
HTTPie 0.9.2
HTTPie data: /home/xxxxx/.httpie
Requests 2.9.1
Pygments 2.1
Python 2.7.12 (default, Dec  4 2017, 14:50:18)
[GCC 5.4.0 20160609] linux2
usage: http [--json] [--form] [--pretty {all,colors,format,none}]
            [--style STYLE] [--print WHAT] [--verbose] [--headers] [--body]
            [--stream] [--output FILE] [--download] [--continue]
            [--session SESSION_NAME_OR_PATH | --session-read-only SESSION_NAME_OR_PATH]
            [--auth USER[:PASS]] [--auth-type {basic,digest}]
            [--proxy PROTOCOL:PROXY_URL] [--follow] [--verify VERIFY]
            [--cert CERT] [--cert-key CERT_KEY] [--timeout SECONDS]
            [--check-status] [--ignore-stdin] [--help] [--version]
            [--traceback] [--debug]
            [METHOD] URL [REQUEST_ITEM [REQUEST_ITEM ...]]
http: error: unrecognized arguments: --all

Your version is too old, switch the docs over to that version: https://github.com/jakubroztocil/httpie/tree/0.9.2
--all was introduced in 6e1dbadff94c2ed168119cd17df37d7ec9cb8d05 which came in 0.9.4


Don't attach e-cash outputs to LN offer creation transactions

Jan 13 19:37:15 alpha fedimintd[42194]:   Peer 2: Transaction
Jan 13 19:37:15 alpha fedimintd[42194]:     Output: LN offer for 1000 msat with hash be7b2e8660b5ca7975ebe1805060ee67f2df28237ca402f39a2b1553273d3c7a
Jan 13 19:37:15 alpha fedimintd[42194]:     Output: Mint Coins 0 msat

As context for @okjodom, transaction containing offer output gets created in Client<UserClientConfig>.generate_invoice(). That function only adds the offer output, so the extra 0 ecash output is added somewhere in the self.submit_tx_with_change() call. If you investigate that call stack, outputs get added in MintClient.finalize_change() so that must be the culprit.
Ended up implementing simple fix in https://github.com/fedimint/fedimint/pull/1328. But wanted to explain here so Jodom knows a little better how this works.

Should we use NonZero<T> for all thighs output amount to reject on type system level? @elsirion

Should we use NonZero<T> for all thighs output amount to reject on type system level? @elsirion

@dpc it would require something like NonEmptyTieredMulti. I don't think it's worth the hassle. Afaik NonZero mostly makes sense to save a byte for Options etc.

Oh, right. Do we reject outputs with no amounts and tiers with no elements? Should we? Empty tiers break determinism of the encoding. I'm afk, will check later.

We should canonicalize at encoding and reject non canonical at decoding, it seems to be. Iirc we don't.

I wonder if server should reject zero-amount outputs, too?

On one hand it is allowed in L1 and useful there, on the other seems prone to abuse as we don't have any cost associated with it. But then people can just send find back and forth if they want some way to abuse things.


Changes in regular tests generation to align platform scope

Align platform scope for regular python tests:
find changes in GenerateTestPlugin::pytest_generate_tests
RegularSpecificationProcessor::get_test_configurations is called, to select platform and scenarios.
If scenario is not selected, than item is removed in pytest_collection_modifyitems (do not mark as skipped for consistency with Yaml scenarios and to limit memory consumption).
SpecificationProcessor::process is now common for YamlSpecificationProcessor and RegularSpecificationProcessor,
session.specifications[item.nodeid] is updated separately for regular and yaml tests (in separate pytest_collection_modifyitems) to avoid mixing logic.

Codecov Report

Merging #150 (cb37285) into main (c7510c8) will decrease coverage by 0.18%.
The diff coverage is 82.43%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #150      +/-   ##
==========================================
- Coverage   82.68%   82.50%   -0.18%
==========================================
  Files          55       55
  Lines        3592     3573      -19
==========================================
- Hits         2970     2948      -22
- Misses        622      625       +3

Impacted Files
Coverage Δ

src/twister2/fixtures/common.py
88.88% <ø> (ø)

src/twister2/plugin.py
89.38% <ø> (+0.55%)
:arrow_up:

src/twister2/fixtures/builder.py
33.92% <25.00%> (+1.23%)
:arrow_up:

src/twister2/load_tests.py
93.22% <33.33%> (-3.21%)
:arrow_down:

src/twister2/specification_processor.py
91.83% <88.23%> (-0.38%)
:arrow_down:

src/twister2/yaml_file.py
93.02% <88.88%> (-1.10%)
:arrow_down:

src/twister2/generate_tests_plugin.py
90.00% <100.00%> (-1.55%)
:arrow_down:

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