[FEA] Support Negative upper bound on current row in windowing

Is your feature request related to a problem? Please describe.
Exec <WindowExec> cannot run on GPU because not all expressions can be replaced
    !Expression <SpecifiedWindowFrame> specifiedwindowframe(my_range_frame, -12345, -1) cannot run on GPU because Upper-bounds behind current row is not supported. Found: -1

Describe the solution you'd like
Support upper bounds that can go behind the current row.
Describe alternatives you've considered
None
Additional context
Customer query

@kuhushukla which window operations is this for? I don't think it makes a lot of difference, but I want to make sure that we are testing the correct things, and it might reduce the total impact to the processing being done.
@mythrocks do you remember why we could not support this out of the box?  In practice I think we could replicate this is essentially doing a lead/lag like operation before the window operation. But I am not really 100% sure on that.

Sorry for the delay.  This took some recollecting.
TL;DR:  The CUDF primitives I've written do not support negative bounds.  It is doable with some non-trivial effort.
The primitives convert the range values into real row-offset values in the sort-by columns.  This was easy to do in the ROWS case.  In the RANGE case, the maths will be a little confusing.  I'd be delighted to work on it, depending on priority.


Warning when using ADC + API Key

Moving forward API key will have preference over ADC.
I believe this error message is unnecessary and possibly confusing for API key users.
Should it be removed?
https://github.com/googleapis/google-cloud-java/blob/f16bc31f2e0e9e3febab45a214e48688d46b49eb/java-translate/google-cloud-translate/src/main/java/com/google/cloud/translate/TranslateOptions.java#L166

@clundin25 Thanks for reporting this issue! I agree we should remove this warning. In addition, is the current authentication order correct?  In which explicitly set Credentials has preference over API keys.
Also moving this issue back to google-cloud-java as this is a handwritten layer on top of GAPIC(We do have a few of them in the mono repo).


Russian localisaition of mod

Hey buddy, I'm a Russian-speaking person. I noticed that your "Kawai Dishesh" mod doesn't have my language. So I decided to volunteer to translate your mod. If possible, please add my translation of your mod. I am also ready to cooperate and discuss with you the problems of Russian translation of your mod. Thank you for your wonderful mod and I will be very grateful for cooperation with such a talented mod developer! (Link to the mod localisaition https://drive.google.com/file/d/1x93D5roJAdUWAE9VjYKnQ3iNzzH_OQsU/view?usp=sharing)

Hi, I will gladly add the Russian Translation, Thank you so much for it!

Привет, я с удовольствием добавлю русский перевод, большое спасибо за него!

Don't forget to check this file again as well, I made some small corrections with some words a couple of minutes ago. But overall, the translation is finished. The only thing I can't translate is the villagers' phrases. When the player has a kawaii effect or neko effect. But if you want I can send you directly in the chat room translated each phrase that villagers say. And then you put them in the Russian localization of the mod. In fact, the translation I did for version 1.18.2

You look so kawaii !! "Ты выглядишь так няшно !!"

Oh, i see, probably i can change to make it more easy to add translation to messages

Tho, in the new versions, you can set the messages directly throught config

Oh, i see, probably i can change to make it more easy to add translation to messages

That would be a good solution, I'm waiting for the changes for 1.18.2. And then I'll be sure to send you translations of the villagers' phrases. If I have time, I will try to help with Russian localization of the mod for the newest versions. For I noticed that it has its own innovations. Thank you for your cooperation with me


run pyflakes / flake8

running pyflakes gives me:
$ pyflakes ot/*/*.py ot/*.py examples/*.py
ot/optim.py:9: '.bregman.sinkhorn_stabilized' imported but unused
ot/utils.py:96: undefined name 'reduce'
examples/demo_OTDA_classes.py:6: 'numpy as np' imported but unused
examples/demo_barycenter_1D.py:12: 'mpl_toolkits.mplot3d.Axes3D' imported but unused
examples/demo_barycenter_1D.py:14: 'matplotlib.colors.colorConverter' imported but unused

when running flake8
$ flake8 ot/*/*.py ot/*.py examples/*.py

you'll see that you have a lot of pep8 style violations.

Hello, pyflakes errors have been corrected in commit https://github.com/rflamary/POT/commit/e2733d354c87bdc3cbc58a5e3e838e688fafe627.
pep8 needs much more work and will be addressed in the future.

thanks for the quick fix !


Passing parameters to computed column

How do I pass a parameter to a computed column?
In my controller I have:
public function assignModal(LeadAssignDataTable $lead_assign_data_table, Lead $lead)
{
    return $lead_assign_data_table->with('lead', $lead)->render('leads.assign');
}

This passes the lead to my datatable class, used by the html function here:
public function html()
{
    return $this->builder()
        ->setTableId('lead-assign-table')
        ->columns($this->getColumns())
        ->minifiedAjax()
        ->ajax(route('leads.assign', $this->lead))
        ->responsive()
        ->autoWidth(false)
        ->orderBy(0, 'asc');
}

I also have a computed action column:
public function dataTable($query)
{
    return datatables()
        ->eloquent($query)
        ->addColumn('action', 'leads.assign_action');
}

protected function getColumns()
{
    return [
        Column::make('name'),
        Column::computed('action')->title(''),
    ];
}

I can currently access the lead from the leads.assign_action view using request('lead'). This feels kind of hacky to me, I'd like to use the lead as a variable instead.
Here is the view file:
<form method="post" action="{{ route('leads.assign', request('lead')) }}" class="d-inline-block" data-ajax-form>
    @csrf
    <input type="hidden" name="salesperson_id" value="{{ $id }}">
    <button type="submit" class="btn btn-primary">
        Assign
    </button>
</form>

I'm just wondering if there is a way to pass the lead as a variable into the computed view?

@redbastie You got any solution ?

I ended up just using callback:
        ->editColumn('action', function (User $user) {
            return view('assigns.create_action', [
                'lead' => $this->lead,
                'salesperson' => $user,
            ]);
        });

Yes, I also use callback for complex cases. Actually, most of the time. :)


Update security_finding class to extend findings

Currently security_finding was extending base event instead of expected findings class.
This will add the malware profile to the class as it's included in the findings class.

Signed-off-by: Johan Cornelissen<EMAIL_ADDRESS>

Hey, there's no expectation for security_finding class to extend the findings class, extending the base_event is perfectly fine. But looking into the future, it's probably a good idea to utilize findings as the base.
Also, you raise another good point, the class can very well use a Malware profile. However, it will require more adjustments to class structure as the attributes defined in the malware profile  are already defined in the class itself. I'll rework it and create a new PR to get those changes in.
For now, closing this PR. I'll tag you in the new one.


ASN.1 X25519 key serialization is non-standard and doesn't match current consensus

OpenSSL's X25519 code currently uses OIDs:
1 3 6 1 4 1 11591 15 1      : X25519
1 3 6 1 4 1 11591 15 2      : X448

[Edit: I originally transcribed this from an older commit. Fixed.]
And then it uses an id-ecPublicKey-based serialization for PKCS#8 with the ECPrivateKey structure in contents, like all the X9.62 ECC stuff.
Rich can correct me if I've got the details wrong, but I believe the current consensus from IETF 96 is:

X25519, X448, Ed25519, Ed25519ph, Ed448, and Ed448ph each get distinct top-level AlgorithmIdentifier OIDs, not using id-ecPublicKey, etc.
Private keys are not serialized with ECPrivateKey. Instead the contents in PKCS#8 would be just the octet string raw private key serialization or something of the sort. (I'm not sure if anyone's decided if this serialization would include the public key or not... doesn't hugely matter as long as we pick one rather than mimick ECPrivateKey's silly OPTIONAL field.)

Basically, declare that X25519 and friends are totally different animals from the X9.62 universe (they really are) and not try to shoehorn one into the other.
On the BoringSSL side, my expectation was to map each of these into distinct EVP_PKEY types that are completely detached from EC_KEY and friends. Saves side-channel trouble with bn_correct_top and avoids getting mixed up with EC_KEY assuming one primitive is good for both key exchange and signing. (We haven't actually routed any our curve25519 stuff to EVP_PKEY yet. Haven't implemented Ed25519/X.509 certificates yet and our X25519/TLS code does not use EVP_PKEY.)
OpenSSL shouldn't serialize things in a non-standard form. Since X25519 was just added for TLS which doesn't serialize X25519 this way[*], it'd probably be best to:

Switch NID_X25519 to be unassociated with any OID. The OID can be filled in in a future release once things are settled.
Make it impossible to use X25519 EC_KEYs with i2d_ECPrivateKey, X509_PUBKEY, PKCS8_PRIV_KEY_INFO, and friends. That too can be filled in once things have settled. (The OID change alone might do this?) Actually, I expect there won't be an ECPrivateKey serialization at all, so you'd probably want to just keep that not working.

[*] Fixed-ECDH + X25519 ciphers aside. But those are gone in TLS 1.3 and depend on X25519 certificates which don't exist yet.

This should be fixed in current master.


Add Firework Sikky by Sikky

Song Name
Firework Sikky
Artist Name
Sikky
Source
Youtube
Youtube Link
aRo20UlOtk0
Direct File Link
No response
Song ID
502268
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 75206202

accept

accept


Add support for google cloud print

Add support for google cloud print for a server side print.

This client library cannot support the Google Cloud Print API, because the Cloud Print API does not follow the same format as the APIs supported by this library. This library can only support APIs that follow the format of the APIs found here.

@jmdobry do you have a suggestion where should it fit?

As far as I know, there isn't any Node.js client library that supports the Cloud Print API. Your options are:

Formulate the http request manually using something like request
Write the client library yourself


Remove dashboard from release

In ISO6 the old dashboard should not longer be included. This means:

Removing it from the builds (RPM)
Removing the status file handler from the server
Adding a redirecting from dashboard to console

Hey team! Please add your planning poker estimate with Zenhub @arnaudsjs @FloLey @Hugo-Inmanta @sanderr


Added Option Page to allow display of display multiple projects

Added Option Page.
Allows you to display multiple project types and links in the HelpExplorer Window or the first one found for your selected project.

The WriteCapabilitiesToFile(value) only runs in debug mode and returns the following example files. This is easier than trying to get values out of the string[] capabilities while running to add to values to the projecttypes.json file.
CSharp_Capabilities.txt
MauiBlazor_Capabilities.txt
MauiCore_Capabilities.txt
VSIX_Capabilities.txt
WindowsXAML_Capabilities.txt
WPF_Capabilities.txt
AspNetCore_Capabilities.txt

Used some of the Capabilities found in the output files to update the projecttypes.json file.

Awesome. Thank you!!!

Mads, if you can decide what Microsoft url's should be the ones used for each project and file types I will add them and test.  The ones I used may or may not be the best ones based on your PM's requirements.

Perhaps we can get @jasonchlus getting the right URLs for the content and add them to this file?

Mads, I do need you to test something to be sure it is not my notebook. But in Visual Studio Enterprise 2022, when I go to the options page it does not load, if I cancel then open again it loads file, This seems to happen on more than HelpExplorer options on my machine.  If you see the same then I will open bug report.

I've never seen that happen before. Strange

So, does it happen on yours?

I meant to say Loads fine not (file)

I would send some screen shots but Syncfusion is updating on my system and I cannot run VS right now.

First loaded and even though it allows me to click HelpExplorer and General the results do not change.

Then if I click away to and then back it stays the same as what the click away screen was.

Then on the third click away and back it works and shows the correct options.  I also noticed in the first try if I cancel instead of clicking away the reopen the options from screech it displays correctly the first time.

I should mention that it does this in the exp instance of VS 2022, I have not tried with HelpExplorer installed to see if it is an issue with the debugger or the exp instance of VS 2022.


Support chain rules for multiphase evaluation

@M4tteoP last time it worked very well to ask you for test cases :) It would be good if you can give this a look for adding more test coverage, or verifying existing coverage is enough. Unlike normal evaluation, there are many different cases for chains such as root variables being before chain variables, being after chain variables, there being multiple chains, etc. Notably, my understanding is it's fine to only differentiate root vs non-root as chains should only apply if the entire chain matches, but not sure (especially after seeing that recent "flow action" PR).

Superseed by #719


lib/dll postfix

Is it possible to allow adding a postfix to the libraries via a cmake option?
I ask because having Debug/Release configurations with the same DLL names, the wrong DLL can be picked by the Windows search order.
Being able to add a custom pre/postfix to the generated lib and dll names will be very helpful.

Apologies, I see that this already exists via MATERIALX_LIBNAME_SUFFIX


Phase 2: Full installer.

When beyond phase 1. It needs a proper installer. Shouldn't require anything from the user.
Feedback and people are a must for this project so once beyond initial phase 1. Phase 2 should be getting a proper installer.

I may be able to help out with a Drag/Drop Mac binary release.  Let me know.

The idea here was to make sure the libusb driver mostly installed. Was apparently a headache for Scorch and he switched over to making installers that integrated the libusb stuff in them. And it seemed like the amount of questions he got about that stuff dropped. I'm thinking maybe this needs to be expanded to providing solid binary installers for the major OSes.

This is actually fixed circa 0.4.0 since the new release has #124 finished.


Duplicate concurrent socket reconnects after connection is lost

Hi there, I've encountered an issue with the reconnect behaviour whenever a client (with active subscription(s)) loses its connection with the server. For instance, when the server shuts down manually.
I've managed to trace it back to #onSocketError which may try to reconnect  more than once if called again when #createSocket succeeded while reconnecting. Once the duplicate sockets successfully connect again the client becomes unstable and behaves unreliable.
Validating whether #isReady is true before calling #connect in onSocketError seems to fix the problem, but I'm not sure whether this is a proper change as I'm not familiar with the codebase.
Environment:

Node.js Version: v17.9.0
Redis Server Version: v7.0.2
Node Redis Version: v4.2.0
Platform: Windows 11

Unless you will share a bit more details about the creating connection part that you have (code / flow) it would be hard to help.
But in general, I would recommend using the common events for determining the client's connectivity status.
and once you have the ready event being called, use connect (wrap as a promise if you want and wait for it to resolve)
If you can share more details I will try to help pinpoint the issue. But maybe someone got it 🤷🏾

Hey @MatanYemini, sorry for the late reaction!
I've created a small example of the problem with the latest version of this module. Hopefully this helps to pinpoint the issue.
The logs are from a single client which tries to reconnect with the server more than once and at the same time. The client needs to have at least one active subscription before the server shuts down. Once the server is back online the client successfully reconnects with more than one socket as can be observed from both server and client logs. The problem could also occur if the client only connects to a server that requires a password, thus without performing any action other than connect.

This has been fixed by PR #2480


Can't use array tool

When I use the array tool, the menu pops up, but nothing happens when I click the "Generate" button

Turns out this was happening because of a problem because of a bug fix that broke the 'generate- array'. It should be working now for new files as of commit - c2d6af4


Associated record deletion

In Rails, and I presume other frameworks, it's not uncommon for records to automatically destroy some or all of their associated records when they themselves are destroyed. So, a deletion of one nominal record may delete many other records as well.
Currently, it appears JSON API has no way to communicate this to the client in the response to a DELETE, so orphaned records may be left on the client side that have been destroyed on the server side.

Currently the hinted direction for your use-case would be to add this information within top-level meta of the response document.

A server MUST return a 200 OK status code if a deletion request is successful and the server responds with only top-level meta data.


CAMEL-10571 SObject tree creation, Composite API

Don't merge look at #1334 first.
@dhirajsb would you mind taking a look at this?
This pull requests adds support for SObject tree creation using Salesforce
Composite API. It includes the commit from #1334 which it is based on.
So please review pull request #1334 before merging this in, and if you can
notify me through comments I'll rebase if #1334 gets merged in.
This functionality allows user to create multiple, up to 200, records
including parent-child relationships, up to 5 levels using one API call.
For instance:
Account account = ...
Contact president = ...
Contact marketing = ...
Account anotherAccount = ...
Contact sales = ...
Asset someAsset = ...
// build the tree
SObjectTree request = new SObjectTree();
request.addObject(account).addChildren(president, marketing);
request.addObject(anotherAccount).addChild(sales).addChild(someAsset);
final SObjectTree response =
template.requestBody("salesforce:composite-tree", tree,
SObjectTree.class);
The returned response in the output message is the identical object
received in the body of the input message updated with identifiers
received on successful record creation or populated errors on failure.

I've removed the unused import in bb96e36

Also try to fix CS errors, the other PR had an error
[INFO] Starting audit...
[ERROR] /Users/davsclaus/workspace/camel/components/camel-salesforce/camel-salesforce-maven-plugin/src/main/java/org/apache/camel/maven/CamelSalesforceMojo.java:898:13: Boolean expression complexity is 8 (max allowed is 6). [BooleanExpressionComplexity]
Audit done.

@davsclaus will do, thank you :+1: (forgot to run verify goal the sourcecheck profile)

I've rebased on the master and included suggestions from the review

Thanks for the PR it has been merged. Do you mind closing this?

Thanks @davsclaus :+1:


Bug fix broken document links

data/document_exclusion_list.json - This holds the list of quest ids along with their project numbers that need to be excluded as they are not publicly available this month.
helpers/project_helpers.rb - A new function is added called 'get_h1_project_document_details' that serves document data without the ones from the document_exclusion_list.json file.

Coverage decreased (-0.1%) to 19.048% when pulling 75e50fb963f2858fa58ddbde3fb174fdd1d113e4 on bug-fix-broken-document-links into 52d5f51ab5da608e503af597b3e814c13b068fa2 on master.


[E2E] Split e2e package to each e2e test

Description
TestMain() is called once in the entire package so that we could not create the multiple go test files having TestMain(). This pr is to separate e2e package to each test package and update the docs.
Issue reference
#846
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests
[x] Unit tests passing
[x] End-to-end tests passing
[x] Extended the documentation
[ ] Specification has been updated
[ ] Provided sample for the feature

@nmlc I changed the structure and updated the docs. thanks for your feedbacks.


Refactoring | find appropiate name for the current global store

Description
Currently, we have two types of stores. One is built using the Vuex library, which only covers some parts of the native wallet, while the other is a new store built using the Pinia library. Although the Pinia store is growing, it currently only has a single data point.
Before proceeding with the migration of the old Vuex stores to the new Pinia store, we need to better define the names and structure of the new storage system.
This global store was renamed to appnav because it suits better the current usage of it. This change is made in this branch which is part of this PR
Consider

Remove the store entirely

Closing as this will be addressed as a part of #347


Sample stacks do not work

When using the sample stacks with pack version 0.4.1 the following error occurs:
ERROR: invalid builder 'cnbs/sample-stack-build:alpine': builder index.docker.io/cnbs/sample-stack-build:alpine missing label io.buildpacks.builder.metadata -- try recreating builder

seems like the sample builders are missing a newly required label io.buildpacks.builder.metadata

Seems like you are trying to use a stack as a builder. What configuration/command are you trying to execute?
It should be cnbs/sample-builder:alpine for the builder.

Oh - I forgot to create the builder locally. It worked after creating the builder with:
 pack create-builder cnbs/sample-builder:alpine --builder-config builders/alpine/builder.toml --no-pull

and then using it with --builder cnbs/sample-builder:alpine.


add NewTask flag to avoid crash

Fixes the following error: 'Calling startActivity() from outside of an Activity context requires the FLAG_ACTIVITY_NEW_TASK flag. Is this really what you want?'
Saw this happen on a simulator that did not have the PlayStore installed. Probably something that will not happen on real devices but you never know.

Thank you for the PR! Was there a specific version of Android this happened on? When I used the plugin on an emulator without the Play Store or Google services installed it just opened the app in the browser.

Hello @edsnider , I've just run your plugin on Android emulator and I've encountered the same issue described by DarkSpikeX.
Answer to your question : Android11 was running (API 30).

Same thing for me, Emulator Antroid 11 API (31). Crashing with the same exception.

We were inspired by this library and made a continuation for MAUI, maybe it will be useful to someone - https://github.com/oscoreio/Maui.AppStoreInfo
This also includes your PR


Trigger Type restriction

I want to restrict trigger type selection in editor (actually I want only calendar triggers and don't want event, logon etc triggers). Is that possible?
Best regards
Kirill

To be clear, you want to limit the type of trigger that can be created when clicking the New button or you want to disable the editing of certain types of triggers that are already added? Neither are possible right now, but it would be an easy addition.

First.
But if user edit existing trigger, he should be able to change it's type only to limited set.

Would you expect to set the restrictions from TaskEditorDialog, TaskPropertiesControl or TriggerEditDialog?

TaskEditorDialog would be best.

TaskScheduler.zip
TaskSchedulerEditor.zip
The two files linked here include a prerelease of v2.7 which includes the enhancements you've requested. Please test and confirm that the changes meet your requirements.

Yes it works perfectly! Exactly the way I wanted, thank you very much!

Released in v2.7.0 today. You can find example code on the Triggers page of the wiki.


Support attribute descriptions

When defining attributes, it would be extremely useful if supporting descriptions could also be specified. These supporting descriptions would be included in the web UI with each attribute input field, and could provide some instructions about what the attribute does and what it's possible values and uses are.

Hey @gmac ! The task source code is included in the UI - why not just add a comment above each attribute with a description / acceptable values, etc?


Move and update swiftenv zsh completions

What's the reason/justification for this change?
This change would break the completion completely and result in it no longer being included, check the logic at https://github.com/kylef/swiftenv/blob/master/libexec/swiftenv-init#L105-L111.

Ah I hadn't seen that sourcing logic. I assumed it was requiring homebrew's completion directory and user configuration to function.
Before this swiftenv completion didn't work for me at all. Let me check this out again knowing that it should work without the updated homebrew formula.

Just checked your configuration in your dot files. You are using bash? https://github.com/keith/dotfiles/blob/b5736bddd02cd90ab9ae7da59c27e1e35099a183/langs/swift.bash#L5 because this PR mentions ZSH.
Either way, it's working for me on both bash, zsh and fish:

That bash file is sourced from zsh. It's bash so it works with both.

Ah. The problem is that sourcing of the zsh file doesn't work when installed via homebrew. Since SWIFTENV_ROOT is ~/.swiftenv but the completions are in $(brew --prefix swiftenv)/share/zsh/site-functions/swiftenv.zsh

I think because of homebrew's infrastructure for completions, shown in the homebrew PR, it makes sense to allow the user to set up the completions in this case. But that would require the rename for zsh which makes it more annoying for sure.

Okay, I'll have a play around with it via Homebrew installed. I think the logic to figure out the source path could just be adapted to figure out the source path relative to current source instead of using SWIFTENV_ROOT.

This now throws an error for me:
% swiftenv<TAB>
_swiftenv:read:2: option valid only in functions called via compctl
Usage: swiftenv completions <command> <args...>

Some of the updates from this PR could be applied to the zsh file, without the rename, to remove compctl which seems to be the "old way" of doing things http://zsh.sourceforge.net/Doc/Release/Completion-Using-compctl.html


libfuse/libdokan: enable teams for admins on prod

And enable them for all users on staging.
(To be merged after we deploy prod.)

Not mentioned, but this also enables teams by default for all staging/devel users.

Not mentioned in the subject, but it's mentioned on the first line of the PR description.  (It was already enabled for all devel users before this PR.)


Calling from web browser takes long time

I am trying to make calls from browser to a mobile by loggingin through demo page provided. I could login and make the call, but when I dail an audio call it remains 40 seconds in connecting status and then changes to ringing status to make the call. So it takes approx 45 seconds for me to get call to mobile after dailing from browser. Are there any special setting that i need to make the application faster? My internet speed is 7.8Mbps and I am checking this on  Chrome browser.

@raady07 I found this issue with asterisk. If possible/applicable to your situation, please try modifying src/tinyMEDIA/src/tmedia_session_jsep.js the way I did:
tmedia_session_jsep01.onIceCandidate = function (o_event, _This) {
    var This = (tmedia_session_jsep01.mozThis || _This);
    if (!This || !This.o_pc) {
        tsk_utils_log_error("This/PeerConnection is null: unexpected");
        return;
    }
    var iceState = (This.o_pc.iceGatheringState || This.o_pc.iceState);

    tsk_utils_log_info("onIceCandidate = " + iceState);

    if (!tmedia_session_jsep01.ice_timer) {
	tmedia_session_jsep01.ice_timer = setTimeout(function() {
		console.log("ICE timer...");
		tmedia_session_jsep01.onIceGatheringCompleted(This);
		tmedia_session_jsep01.ice_timer = null;
	},2000);
    };

    if (iceState === "complete" || (o_event && !o_event.candidate)) {
        tsk_utils_log_info("ICE GATHERING COMPLETED!");
        tmedia_session_jsep01.onIceGatheringCompleted(This);
    }
    else if (This.o_pc.iceState === "failed") {
        tsk_utils_log_error("Ice state is 'failed'");
        This.o_mgr.callback(tmedia_session_events_e.GET_LO_FAILED, This.e_type);
    }
}

it almost half the time

Instead of using google ICEservers, I created a local ICE server and the issue is resolved for the moment. I am not sure if it is a permanent solution, but the call connection is faster and getting connected in 3~4 seconds. I had been using this setup since 3 months and there were no problems till now.

I change the code  setTimeout(..., 2000) to setTimeout(..., 0). and It become faster.

Thank you

40 secs delay occurs because of delay in ICE gatherings, Reasons: Required ports might be blocked on your network or you might be under a VPN. In either of the cases, chrome waits till timeout (default is 40 secs).
Decreasing timeout to 0 sec can be dangerous because you are not giving time for ICE negotiation and this might result in one-way audio issues.


Add watch to atoms

Hi :)
We use condition_variable::wait_for  in our code.
It would be nice to be able to use immer atoms for this but if I'm not misunderstanding we would have to do some sort of busy-waiting because we cant access the internal lock.
Clojure agents/atoms/vars/refs have the add-watch function.
https://clojuredocs.org/clojure.core/add-watch
Would there be a possibility of adding this the the atoms?

Hmmm, but you want the lambda registered with add-watch to be called in the thread that calls add-watch or in the thread that changes the atom? In the latter case, it does not replace the use-case for condition_variable::wait_for. But in the former case, this is defo not easy in C++, and you would need to pass an executor or alike to add-watch.
In either case, there would be some overhead that I'm not sure is appropriate for atom itself.
When dealing with "observable" data, I use cursors in Lager: https://sinusoid.es/lager/cursors.html
Note, however, that there are some caveats with using Lager cursors in a multi-threaded context. If you want I'd be happy to dig deeper in your use-case (we can also open a consulting tab if this is part of a commercial project).
Cheers!

Hmmm, but you want the lambda registered with add-watch to be called in the thread that calls add-watch or in the thread that changes the atom? In the latter case, it does not replace the use-case for condition_variable::wait_for. But in the former case, this is defo not easy in C++, and you would need to pass an executor or alike to add-watch.

Yes you're right, I would expect add-watch to call the lambda on the "atom changing thread" not the "waiting thread".
Guess I need something that blocks, not registers a lambda.

In either case, there would be some overhead that I'm not sure is appropriate for atom itself.

Dont complect on my account :)

When dealing with "observable" data, I use cursors in Lager: https://sinusoid.es/lager/cursors.html
Note, however, that there are some caveats with using Lager cursors in a multi-threaded context. If you want I'd be happy to dig deeper in your use-case (we can also open a consulting tab if this is part of a commercial project).
Cheers!

I have looked at Lager and Rx for observable data, I think either would do the job well, but I was hoping to be able to just use immer. I will have to look closer at lager cursors.
Thanks!  Closing this for now, let me know if I can be of further aid.


Upgrade ff for curves.

Description

Upgrade ff for curves.
Pending algebra hotfix 391 & 392
Main change

use #[derive(MontConfig)] for all Fq/Fr.
use MontFp, QuadExt, CubicExt.
remove some [rustfmt::skip] for clean.
upgrade test code for new ff.

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

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

Thanks a lot!

Quick question: would it be possible to port over some of the sage scripts in this PR: https://github.com/arkworks-rs/curves/pull/95/
The main aim is to check that the generator we specify in the Rust files actually is a generator.
If not, I can file a quick issue to add it and we can address it in a followup PR

Sure, I get the generator in the parameters from sagemath, like:
k = GF(258664426012969094010652733694893533536393512754914660539884262666720468348340822774968888139573360124440321458177, modulus="primitive")
k.gen()

But there are some big numbers(mnt4_753) that I can't get quickly, so I use previous GENERATOR * R.inverse() to get it, so I'm not sure how to write this sage script, need you followup.
And, I will add this test_constants (algebra/test-curves) to all Fq/Fr with previous constants and it will feel better.

But there are some big numbers(mnt4_753) that I can't get quickly, so I use previous GENERATOR * R.inverse() to get it, so I'm not sure how to write this sage script.

For the numbers that are hard to factorize, one can try consulting other sources (like libsnark), and copying the generators from there. Also, you only really need a partial factorization of MODULUS - 1; you can use a tool like https://www.alpertron.com.ar/ECM.HTM to break it down to large factors, and then the sage script can verify that GENERATOR^LARGE_FACTOR != 1.

Is it like in bls12_377/fr.rs, and add to all Fq/Fr ?

Sorry, I should have linked the earlier sage scripts, I meant more like this one. We can make a scripts folder in each crate, and add sage scripts for both the base_field and the scalar_field.

And, I will add this test_constants (algebra/test-curves) to all Fq/Fr with previous constants and it will feel better.

Finally, regarding the test_constants test, we can instead use the strategy here: https://github.com/arkworks-rs/algebra/blob/a40bcae1f5a9759c5202b70bd8d353e75086bdb5/test-curves/src/bls12_381/tests.rs#L15
here mont(6, 4) specifies the number of limbs for Fq and Fr, respectively. So, for example for bn254, we'd add mont(4, 4).

Cool, I will instead use mont to test Fq and Fr. And I will add this scripts/xx.sage.

Great, thanks!

Hi pratyush,
can you hotfix this line: https://github.com/arkworks-rs/algebra/blob/master/test-templates/src/lib.rs#L294
it missing generate_field_test!($($tail)*);
BTW, I found that Fp12Parameters is not changed to Fp12Config like others, is it special?

Done in this PR: https://github.com/arkworks-rs/algebra/pull/393
I fixed both issues, thanks for catching this!

Done in this PR: https://github.com/arkworks-rs/algebra/pull/393
I fixed both issues, thanks for catching this!

Here https://github.com/arkworks-rs/algebra/blob/master/test-templates/src/fields.rs#L457 need to fix.
Replace with let mut serialized = vec![0u8; buf_size + 1]; because this buf_size is calculated by EmptyFlags, but here will use SWFlags, so add one size for test.

I had renamed Params to Config. And here https://github.com/sunhuachuang/r1cs-std , also fixed the rename and test code in r1cs-std (curve will deps), when algebra#393 is merged, I will make a PR to r1cs-std fix-ff branch.

Thanks a lot!

Merged #393


This wraps the annotation with a <details> with a summary and disclosable group stats. I just need to double check that the formatting doesn't look terrible.

After a couple of very-manual tests, it looks like this:

It looked a bit like that first line was too strong on its own, so I tweaked a little:

Thoughts, @ticky?

(I might sneaky merge this so we can try it out!)

Looks legit! I was bolding it only because it made sense in context, but with this in place it looks legit.


feat: whiskers

Adds Whiskers for building the theme links (and README in general).

Indeed! I forgot to mention it but yeah instead of editing the README directly, edit the raycast.tera file and then run just build to build README. You will need both Whiskers and Just installed :)


Delete Tailwind CSS 👋

Is this issue still open? If it is, can u let me know what all css is to be removed?


Doesn't start

I did this on a VM but also replicated on real hardware.  Then it would show the startup UI but freeze.
Here's a few screenshots of the log:

Also compiling Hyprland also fails:

That's all! Also I'm pretty sure I've checked every page for a solution but couldn't find any.

How is that a hyprland issue?
Compiling fails because you didn't read the wiki and did not clone with --recursive.

People who stumble into this in the future: Void does not use systemd so you need seatd or elogind to have adequate permissions to run Hyprland. This is probably what happened for the Reporter.

This ^^^ + Everything is explained in Void's documentation. READ.


Compilation fails if $script starts with a number

If you rename the script file to anything that starts with a number (like 321.ps1) then run.\321.ps1 -setupthe compilation will fail and spit a few pages of errors like:
PS C:\Users\Alex\Desktop\> .\321.ps1 -setup
Add-Type : c:\Users\Alex\AppData\Local\Temp\nbt05ch1.0.cs(43) : Identifier expected
c:\Users\Alex\AppData\Local\Temp\nbt05ch1.0.cs(42) :
c:\Users\Alex\AppData\Local\Temp\nbt05ch1.0.cs(43) : >>>   public class 321 : ServiceBase {
c:\Users\Alex\AppData\Local\Temp\nbt05ch1.0.cs(44) :     private System.Diagnostics.EventLog eventLog;
      // EVENT LOG
At C:\Users\Alex\Desktop\321.ps1:861 char:5
+     Add-Type -TypeDefinition $source -Language CSharp -OutputAssembly ...
+     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidData: (c:\Users\Alex\A...tifier expected:CompilerError) [Add-Type], Exception
    + FullyQualifiedErrorId : SOURCE_CODE_ERROR,Microsoft.PowerShell.Commands.AddTypeCommand

Add-Type : c:\Users\Alex\AppData\Local\Temp\nbt05ch1.0.cs(43) : } expected
c:\Users\Alex\AppData\Local\Temp\nbt05ch1.0.cs(42) :
c:\Users\Alex\AppData\Local\Temp\nbt05ch1.0.cs(43) : >>>   public class 321 : ServiceBase {
c:\Users\Alex\AppData\Local\Temp\nbt05ch1.0.cs(44) :     private System.Diagnostics.EventLog eventLog;
      // EVENT LOG
At C:\Users\Alex\Desktop\321.ps1:861 char:5
+     Add-Type -TypeDefinition $source -Language CSharp -OutputAssembly ...

Sorry but I don't know C (of any variant) well enough to be able to suggest a fix.
Thanks VERY much for this template - it's really saved my backside!

Forgot to say: PSService.ps1 is the script in question. Doh!

Service names beginning with a digit are unusual, but legal indeed Ex: 1394ohci
Bug fixed in commit c3a6d83, by prefixing the C# class name with the string "Service_".
This makes sure the class name begins with a letter in all cases, and thus prevents compilation failures.


In console error paperSpigot 1.15.2

In console error paperSpigot 1.15.2
Version RealScoreboard: 8-2-2020 Build 1
https://pastebin.com/cmmeJhuA

You got some problem with the Scrollable String feature.
Are you using the latest build? Does the error also appear on Spigot?


Adding travis

First draft at adding CI via Travis. Two notes:

If this PR is merged, we will have Travis support
This is a first draft of how to execute multiple test suites concurrently. The log output is suboptimal due to how make parallelizes processes so this may be improved in the future.

@joshgrantsauce looks good tome man. Looks like it's missing sauce credentials for UI automation. but other than that, is it ready for merging?

@nikolay-advolodkin Yep, that's the missing piece here is managing credentials. Not sure how we want to do this (a dedicated user for testing? use our own? store via Travis? etc)


Use URIUtils to encode URL parts in SJS

Since java.net.URLEncoder is not available on ScalaJS, and I could not
find a suitable replacement which would not result in a new dependency,
I separated the affected code into separate source directories. The SJS
code now uses JavaScript URIUtils.

LGTM, thanks @randm-ch!

Just released this along with some dependency updates as v1.2.0

Awesome, cheers! 🥳


In Memory Driver

source
Currently, Mongo does not work with Ubuntu 22.04, see Mongo x86 Support & Mongo ARM Support. This means that mongo-memory-server also does not work in those scenarios. For people looking to use DocMQ in an in-memory environment, this leaves no alternatives.
Proposal
Add support for a native JS in-memory driver as part of the DocMQ core. Here are a list of JS drivers I could find on NPM, along with the features that DocMQ needs.

Find/Sort At a minimum, find/sort features make it easier to update records without writing a lot of custom code inside of the driver's take, ack, etc methods.
changes api A method to retrieve the list of changes at the point of change. This minimizes local polling and offers better performance in the worker
types TS support would be ideal, but isn't a hard requirement. Having access to the types will minimize errors.

Library
Stars
Activity
Types
Find/Sort
Changes API

lowdb

external
lodash
no

LokiJS

external
yes
yes

ForerunnerDB

no
yes
yes

PicoDB

no
yes
yes

TyrDB

no
yes
yes

mango-db

yes
yes
yes

As of Sept 2022, Loki seems to be the most logical choice given the stars/support and the fact it's being maintained again. LowDB does have more support, but requires lodash for find/sort features and does not offer any kind of changes API. While GitHub stars (or npm download numbers) are not a perfect indicator of project success, it does tend to identify projects with a larger audience.

@TAKANOME-DEV for visibility

I looked in the 3 main packages, LokiJS and ForerunnerDB have plenty of features and ForerunnerDB has search and sort like MongoDB.
Which one do you think we should go with?

I'll see about putting a prototype together today/tomorrow using Loki. Forerunner has more features, but DocMQ doesn't really need much as long as we can (1) find the next "jobs" to run (visible < now) and (2) list future runs (visible > now, sort ASC). As part of the #8 work, I did turn the driver layer into a test suite, making it easy to build out the driver & tests.

Finished off the Loki driver, and will look into making this a full release in the morning. There's some minor cleanup to do, but all tests are passing.

Currently, it doesn't leverage any indexes, though there should eventually be ones on ref, ack, deleted, and visible for performance
I'm not a fan of the .find().where() syntax, but this seems to be a limitation in how Loki clones objects. As part of serialization, dates are converted to strings. It did not look like there was a way to change how Loki clones objects
The types are a little messy, though some of this is because we're using external typings. To make it work, we needed a LokiDoc which is a modified QueueDoc with Loki's restrictions, and a WithLoki<T> that attaches Loki specific information to the QueueDoc object.
I really don't like the DROP_ON_CLONE. The more I look at it, the more it should probably be a generalized 'preserveFieldson theBaseDriver` that can be used by mongo, loki, postgres, and other drivers

Wow, that's great :clap: , I need to dive into this, I've never used queue before, I have so much to learn :sweat_smile:
Thank you Jakobo for your hard work and support :fire: :rocket:
If you have any resources on queuing and any related topics and where to start for a beginner, I would really appreciate it :pray:

Sure. The underlying design is to a Message Queue, and there are a couple great youtube videos & articles on the topic:

https://www.youtube.com/watch?v=oUJbuFMyBDk
https://www.youtube.com/watch?v=J6CBdSCB_fY
https://hevodata.com/learn/message-queues/ (non-video)
https://medium.com/must-know-computer-science/system-design-message-queues-245612428a22 (non-video)

As you get more into the reading, DocMQ is a Priority Queue, with our ordering driven by the visible field. Do let me know here or on discord if you have followup questions!


[Compute] securityType has only one enum value and modelAsString is false

Problem
I got a problem when invoking get method of VMSS in Python SDK. It also applies to VM.
securityType has only one enum value in swagger definition.
        "securityType": {
          "type": "string",
          "description": "Specifies the SecurityType of the virtual machine. It is set as TrustedLaunch to enable UefiSettings.

 Default: UefiSettings will not be enabled unless this property is set as TrustedLaunch.",
          "enum": [
            "TrustedLaunch"
          ],
          "x-ms-enum": {
            "name": "SecurityTypes",
            "modelAsString": false
          }
        }

Python code
vmss = client.virtual_machine_scale_sets.get(resource_group_name, name)

The value of securityProfile in HTTP response according to logs is
      "securityProfile": {
        "encryptionAtHost": true
      }

However, vmss.virtual_machine_profile.security_profile.security_type evaluates to 'TrustedLaunch'. Then it causes an error in subsequent statements.
security_type is a constant in Python SDK.
https://github.com/Azure/azure-sdk-for-python/blob/b5cb735efec7525bc3e2f83ee7094ea4626ac686/sdk/compute/azure-mgmt-compute/azure/mgmt/compute/v2020_12_01/models/_models_py3.py#L5706
security_type = "TrustedLaunch"

Expected behavior
Set ModelAsString=True so that Python SDK will not always set it to TrustedLaunch.
My requirement is that Python SDK can work. I read the swagger and relevant document again. I don't think the current swagger definition is wrong. Setting it to true should be able to fix the Python SDK issue. But even it is false, Python SDK should not treat it as a constant since it is not marked required in swagger. Either updating swagger or fixing Python SDK can solve customer's issue in Azure CLI.

checking this, will provide update here after investigation.

there're IcM related to it https://portal.microsofticm.com/imp/v3/incidents/details/238804462/home

quick update: a fix is currently in progress, will provide an update here once it gets rolled out.

The fix has been rolled out to: https://pypi.org/project/azure-mgmt-compute/22.1.0/

This issue has been solved by PR https://github.com/Azure/azure-cli/pull/18788, so I close it


Restrict Test Reason and Result Code Dropdown Options for Test Data

As an ECMPS user, I want the options in the Test Reason and Result Code dropdown restricted so that I can choose from applicable options
Acceptance Criteria:
Given I am creating Test Data for any of the Test Data types in the attached document
I want the Test Reason Code and Test Result Code dropdown options restricted to the applicable options
https://app.zenhub.com/files/287570343/22315a08-c8d2-4c8b-800e-8e006fe24986/download
Note: Check that we are using the relationships API endpoint for Reason and Result Code (Test Summaries Relationship API)
Extension & exemption codes mdm endpoint:
https://api.epa.gov/easey/dev/master-data-mgmt/extension-exemption-codes
Testing
Test Case: TC1221
Tester:

Linearity

RATA

QA Reporting Instructions for Misc.

Leak

missing QA test reason
Should not have initial
Should not have Recert

Other Test

Missing QA test reason

Pems

missing qa test reason

Dry Gas Meter

Missing QA test reason

Mass flow meter:

Missing Qa test reason

Barometer calibration

missing qa test reason

Quaterly gas audit

missing qa test reason

Temperature sensor

missing qa test reason


Add pencil icon

Adds pencil icon

edit-4 basically

Yeah true, I a bit doubting what the right name is .. 🤔

@locness3 Do you think we need to call it edit-4?

No idea

Also shouldn't it be rounded? I personally don't like the shape of this icon. Maybe you could do something more like Octicon's pencil, but... 2px rounded?


Failed to configure SSL certificates in ArcGIS Server. Importing CA certificate failed.

The error in subject is bound to happen if the the certificate is already installed. But if you are unregistering and reregistering a server in a site (eg. for maintenance purposes) this should just skip it or have the option to force overwrite it.
In addition to this. running the server_node recipe to reregister or rejoin the site, it will try to import the certificate 11 times before fatal failing the hole procedure.
I can manually remove the certificate before running the server_node, but please provide an option to skip this step if its already installed or an option to force overwrite the existing certificate (alias).
Thanks,

Should probably mention that we are using Esri cookbooks version 3.6.1, Chef Client 14.14.29, and ArcGIS Enterprise 10.7

Hi @thk70,
When the cookbook checks to see if the ssl certificate exists, it looks for "entryType":"PrivateKeyEntry" in the response from ArcGIS Server:
https://github.com/Esri/arcgis-cookbook/blob/ae704a00cef52d34057569e36214f8669d539266/cookbooks/arcgis-enterprise/libraries/server_admin_client.rb#L267
However, it appears that ArcGIS Server 10.7 and 10.7.1 do not include a "entryType":"PrivateKeyEntry" in the response, but 10.8 and 10.8.1 does.
10.7/10.7.1 response is missing "entryType":"PrivateKeyEntry":
DEBUG: Request: POST https://machine.domain.com:6443/arcgis/admin/machines/machine.domain.com/sslcertificates/machine.domain.com

DEBUG: Response: 200 {"aliasName":"machine.domain.com","issuer":"CN=domain Issuing CA, DC=domain, DC=com","subject":"CN=*.domain.com","subjectAlternativeNames":["DNSName: *.domain.com"],"validFrom":"Thu Mar 04 11:33:25 PST 2021","validUntil":"Sat Mar 04 11:33:25 PST 2023","keyAlgorithm":"RSA","keySize":2048,"serialNumber":"4f000038dd5677db65798833470000000038dd","version":3,"signatureAlgorithm":"SHA256withRSA","keyUsage":["Digital_Signature","Key_Encipherment"],"md5Fingerprint":"444ed35efc8ba2796f375f9fa6f9b216","sha1Fingerprint":"bf9286a4c54630e20469ac4ddbda4c510ca938c3","sha256Fingerprint":"12210799e6e735b5a4d25df7114cc5b651b5ea4afe716992c5ac9f69fc5c04a4"}

DEBUG: Request: POST https://machine.domain.com:6443/arcgis/admin/machines/machine.domain.com/sslcertificates/importExistingServerCertificate
DEBUG: Response: 200 {"status":"error","messages":["Importing CA certificate failed. "],"code":500}
ERROR: Failed to configure SSL certificates in ArcGIS Server. Importing CA certificate failed.

10.8/10.8.1 response that includes "entryType":"PrivateKeyEntry", in this case importing certificate is skipped:
DEBUG: Request: POST https://machine.domain.com:6443/arcgis/admin/machines/machine.domain.com/sslcertificates/machine.domain.com

DEBUG: Response: 200 {"aliasName":"machine.domain.com","entryType":"PrivateKeyEntry","issuer":"CN=domain Issuing CA, DC=domain, DC=com","subject":"CN=*.domain.com","subjectAlternativeNames":["DNSName: *.domain.com"],"validFrom":"Thu Mar 04 11:33:25 PST 2021","validUntil":"Sat Mar 04 11:33:25 PST 2023","keyAlgorithm":"RSA","keySize":2048,"serialNumber":"4f000038dd5677db65798833470000000038dd","version":3,"signatureAlgorithm":"SHA256withRSA","keyUsage":["Digital_Signature","Key_Encipherment"],"md5Fingerprint":"444ed35efc8ba2796f375f9fa6f9b216","sha1Fingerprint":"bf9286a4c54630e20469ac4ddbda4c510ca938c3","sha256Fingerprint":"12210799e6e735b5a4d25df7114cc5b651b5ea4afe716992c5ac9f69fc5c04a4"}

We will fix this in the next release of the cookbooks, however in the mean time feel free to use the following workaround:
Replace line 281 in cookbooks/arcgis-enterprise/libraries/server_admin_client.rb:
https://github.com/Esri/arcgis-cookbook/blob/ae704a00cef52d34057569e36214f8669d539266/cookbooks/arcgis-enterprise/libraries/server_admin_client.rb#L281
with:
JSON.parse(response.body)['entryType'].nil? || JSON.parse(response.body)['entryType'] ==entry_type

Thanks,
Cameron K.

@thk70
We have implented a fix for this in the latest release, v370!
https://github.com/Esri/arcgis-cookbook/releases/tag/v3.7.0
Thanks,
Cameron K.


Joining a room directly via URL is not playing correctly

This is being caused by browsers restrictions on auto playing audio/video without any user input (click, typing, anything) on the page first.
My only idea to fix it is to entirely change the PasswordRequestWindow.tsx component to also have a username field so there will always be a user input on password and non-password rooms before the video starts playing.

This is fixed in #14.


Improve debugging support in dockerfiles and fix guitool doctest errors

Use the develop branch of wildbook-ia in dockerfile
I regularly build the wbia-provision with outdated wildbook-ia by
mistake because it's using the master branch, while the latest changes
are in the develop branch.  So I think it's better to use the
develop branch in the dockerfile.  For production, it's probably best
to use tags or what's released on pypi.

Display command line being run in dockerfile
We have some really long command lines in the dockerfiles and it's
sometimes difficult to know exactly where something failed, for example:
RUN . /virtualenv/env3/bin/activate \
 && cd /wbia/flann \
 && /bin/bash run_developer_setup.sh \
 && cd /wbia/hesaff \
 && /bin/bash run_developer_setup.sh \
 && cd /wbia/brambox \
 && /virtualenv/env3/bin/pip install -e . \
 && cd /wbia/lightnet \
 && /virtualenv/env3/bin/pip install -r requirements.txt \
 && /virtualenv/env3/bin/pip install -r develop.txt \
 && cd /wbia/wbia-plugin-flukematch \
 && /bin/bash unix_build.sh \
 && cd /wbia/wbia-plugin-curvrank \
 && /bin/bash unix_build.sh \
 && cd /wbia/pydarknet \
 && /bin/bash unix_build.sh \
 && cd /wbia/pyrf \
 && /bin/bash unix_build.sh \
 && cd /wbia/vtool \
 && /bin/bash run_developer_setup.sh

In the middle of this I see something like:
Traceback (most recent call last):
  File "<string>", line 1, in <module>
ImportError: No module named utool

and with /bin/bash -xc around the whole command line, you get:
+ cd /wbia/pyrf
+ /bin/bash unix_build.sh
Traceback (most recent call last):
  File "<string>", line 1, in <module>
ImportError: No module named utool

Fix flake8 errors in guitool/app_item_model.py
wbia/guitool/api_item_model.py:91:5: E306 expected 1 blank line before a nested definition, found 0
wbia/guitool/api_item_model.py:198:13: E306 expected 1 blank line before a nested definition, found 0
wbia/guitool/api_item_model.py:1092:5: F401 'wbia.guitool' imported but unused
wbia/guitool/api_item_model.py:1094:5: F821 undefined name 'guitool'
wbia/guitool/api_item_model.py:1124:11: F821 undefined name 'guitool'
wbia/guitool/api_item_model.py:1139:11: F821 undefined name 'guitool'

Fix guitool import in guitool/api_item_model.py doctests
When running the wildbook-ia doctests:
--- Error: 117 / 117 ---
    * REASON: NameError
    DOCTEST DEBUG INFO
      XDoc "/wbia/wildbook-ia/wbia/guitool/api_item_model.py::simple_thumbnail_widget:0", line 4 <- wrt doctest
      File "/wbia/wildbook-ia/wbia/guitool/api_item_model.py", line 1086, <- wrt source file
    DOCTEST PART BREAKDOWN
    Failed Part:
        1 >>> # ENABLE_DOCTEST
        2 >>> import wbia.guitool
        3 >>> from wbia.guitool.api_item_model import *  # NOQA
        4 >>> guitool.ensure_qapp()  # must be ensured before any embeding
        5 >>> wgt = simple_thumbnail_widget()
        6 >>> ut.quit_if_noshow()
        7 >>> wgt.show()
        8 >>> guitool.qtapp_loop(wgt, frequency=100, init_signals=True)
    DOCTEST TRACEBACK
    Traceback (most recent call last):

      File "/virtualenv/env3/lib/python3.6/site-packages/xdoctest/doctest_example.py", line 553, in run
        exec(code, test_globals)

      File "<doctest:/wbia/wildbook-ia/wbia/guitool/api_item_model.py::simple_thumbnail_widget:0>", line rel: 4, abs: 1086, in <module>
        >>> guitool.ensure_qapp()  # must be ensured before any embeding

    NameError: name 'guitool' is not defined
    DOCTEST REPRODUCTION
    CommandLine:
        python -m xdoctest /wbia/wildbook-ia/wbia/guitool/api_item_model.py simple_thumbnail_widget:0

It's because we probably changed import guitool to
import wbia.guitool and now guitool is not defined.  Changing to
import wbia.guitool as guitool works.

FYI: Not sure if you are using xdoctest or the old doctest runner inside utool. But if you are using xdoctest then any instance of >>> ut.quit_if_noshow() can be changed to >>> xdoctest: +REQUIRES(--show) to remove dependencies on utool.
The second case is a builtin xdoctest directive that will only run the following lines in the doctest if the --show parameter is on the command line. This is the same thing that ut.quit_if_noshow() was doing.

FYI: Not sure if you are using xdoctest or the old doctest runner inside utool. But if you are using xdoctest then any instance of >>> ut.quit_if_noshow() can be changed to >>> # xdoctest: +REQUIRES(--show) to remove dependencies on utool.
The second case is a builtin xdoctest directive that will only run the following lines in the doctest if the --show parameter is on the command line. This is the same thing that ut.quit_if_noshow() was doing.

@Erotemic I'm using run_doctests.sh which uses xdoctest.  There are lots of instances of .quit_if_noshow() in wildbook-ia, should I change them too # xdoctest: +REQUIRES(--show)?  Or maybe remove all the .quit_if_noshow() and change run_doctests.sh to include --show?  @bluemellophone Any preferences?

FYI: Not sure if you are using xdoctest or the old doctest runner inside utool. But if you are using xdoctest then any instance of >>> ut.quit_if_noshow() can be changed to >>> # xdoctest: +REQUIRES(--show) to remove dependencies on utool.
The second case is a builtin xdoctest directive that will only run the following lines in the doctest if the --show parameter is on the command line. This is the same thing that ut.quit_if_noshow() was doing.

@Erotemic I'm using run_doctests.sh which uses xdoctest. There are lots of instances of .quit_if_noshow() in wildbook-ia, should I change them too # xdoctest: +REQUIRES(--show)? Or maybe remove all the .quit_if_noshow() and change run_doctests.sh to include --show? @bluemellophone Any preferences?

I prefer to use xdoctest wherever possible.  For the tests, however, I'd like to prioritize headless testing since we will be moving the GUI to an optional plugin in the future.  For now, though, we can keep the support with xvfb and start building it as a more extensible testing platform for any GUI tests we want to run.


Scan the latest antrea-ubuntu Docker image for vulnerabilities

We use https://github.com/arminc/clair-scanner, a CLI utility built for
the Clair tool to perform the scan, then we analyze the results and send
an email notification if necessary.
We configure the Github CI so that the scan is run as a daily cron job.
Fixes #218

Probably a lot of room for improvement, but hopefully this is a good starting point.

Thanks @tnqn I addressed the typos.

Marking this as ready for review. I tested the Github action in my fork.

LGTM

/skip-all


Problem with @libsql/client in React Native

Path: /sdk/ts/quickstart
Hello, I am following the instruction in "Turso Quickstart (TypeScript / JS)":
First, install the @libsql/client, and restart the Metro server.
npm install @libsql/client
Second, create the client:
`import { createClient } from "@libsql/client";
const client = createClient({
url: "libsql://...",
authToken: "...",
});
`
Introducing my libsql address, and my token generate in the Turso Dashboard.
Here if I launch the app in the Android Emulator, i get the following error:

"Syntax Error"
@libsql/core/api could not be found within the project or in these directories:
node_modules

could you please, explaine if these if a package problem? because i have tried these both in Reac Native and Expo React Native, and have the same problem in both.

Hey @Miguel-Salazar-Dev
Our story with React Native isn't great because of the node file system dependencies. I'd recommend you join our community on Discord as there are some previous discussions about this that may help.


Secret key support to Docker registry token based authentication

Why not simplify the local deployment of a private Docker registry by allowing a "secret key" or rootcertbundle to sign tokens in the token based authentication method?

Does the image support the Swarm secrets?


Android Client URI Use

Is it possible to have a 3 line example of how to call the URI on Android?
I have no idea what type of Intent Action I should use and there is no documentation about it anywhere.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 9effefa0-ac98-1fdd-6174-f06d524622f5
Version Independent ID: 5d405e1c-d2eb-9c72-3cf6-817930cad4f6
Content: Remote Desktop clients URI scheme
Content Source: WindowsServerDocs/remote/remote-desktop-services/clients/remote-desktop-uri.md
Product: windows-server-threshold
GitHub Login: @lizap
Microsoft Alias: elizapo

Thanks for the suggestion - we'll see what we can do.

To make sure I understand what you're asking for - we do have an example call in the article: rdp://full%20address=s:mypc:3389&audiomode=i:2&disable%20themes=i:1
Are you asking for an example that shows that call in context?

Where and how is this URI used within the Android client?? That is, can this be used in the 'Desktop' item?? What are the defaults for the URI??

Closing due to inactivity.  Please reopen to pursue further.  Thanks!


Docs bug: "recent cryptanalysis" link is broken

Is there a (more stable) mirror that can be linked to instead?

Guess I didn't realize I used the NIST's beta website link.
Here's the fixed one: https://csrc.nist.gov/News/2017/Recent-Cryptanalysis-of-FF3

thank you!


It would be nice to have the github link in the bot's description

I put it in the Playing field, which feels terrible, or we can make an !help command, which feels better.

Yeah, there doesn't seem to be an obvious place. I assume the Notes are per-User so if we add something there it doesn't show up for everyone else. I'm surprised there isn't something we can add that will show up when someone right-clicks the bot and choose profile. But perhaps there is not. !help or !about or both sounds like a good idea.


Chrono Widget 0.03

Display minutes:seconds when less than 1 hour is left.

I guess this isn't ready to merge yet as it's using graph?
Please can you let me know next time if that's the case? Saves me merging accidentally :)

No, Active Pedometer uses graph, but there is no PR for this yet.
This does not, can be merged.

Ok - I think the issue is you did a PR from your GitHub master branch.
When you do a PR it keeps updating it based on your branch - so it looks like you committed the active pedometer and that's updated this PR as well.
Closing for now - maybe you could create a branch with just the chrono widget on and PR from that, or just reopen this PR when you're happy with the active pedometer as well

Ah, did not realize that Active Pedometer was in here, too. Sorry.


[Ready for review] Fix drawer menu

Fix issue #63
Hey @trallard, I have found the bug and added customization as follow:
https://github.com/trangology/community-handbook/blob/00ebf3edbb3125634c711c3e067dd16419e932ec/docs/stylesheets/custom.scss#L322
However, after compiling the custom.scss file to the custom.css one, the css file was formatted in the new style that inserted into this file 55 lines of code, and I am not so sure if it's still okay for this PR could be merged. 🤔
So let me know if there is something needed to be changed. Thanks!

Hey @trangology  thanks for the contribution!!! It looks great
There is one minor fix needed before merging this though. Now the nav title in here is too light and hard to read

The botched selector is md-nav__title[for=__drawer] which is currently taking the value of var(--md-primary-bg-color) it should be fg  - I would go for the dark one 😉  to differentiate from the links below

@trallard Hmm, I just realized that my fix broke the UI of the large screen resolution 😭

😬😬 yes the Dracula one has a bit more definitions.
I will be online tomorrow in case you need some help or want to do some pair programming

Absolutely down for a pair programming session. Just give me your available time slots with your timezone either here, or ping me on Discord. My timezone is Moscow Time (GMT+3), and will fix my schedule based on yours :grin:

I am BST based and will be online from 6:30pm GMT+3 since we have the Americas sprint.
Feel free to ping on Discord

@trangology let me know if you are going to make further changes, else I will merge this

I have been struggling with Git since our conversation ended. Will get it done in 15 mins 🤞

@trallard I am done! Check it out 😀

✨  Thank you so much


Feat:Add support in OTA agent for controlling number of data blocks.

Add support in OTA agent for controlling number of data blocks requested from streaming service.

Description

This change adds support in OTA agent for controlling number of data blocks requested from streaming service. A new parameter representing number of data blocks is added as a config that is sent with every data block request. For example if this parameter is set as 16 then OTA streaming service will send 16 Data Blocks in response.
This change also restarts the timer as soon as the requested number of data blocks are received instead of waiting for the request timer timeout. This helps in speeding up the OTA process by avoiding unnecessary wait for timer to expire.
Checklist:

[x] I have tested my changes. No regression in existing tests.
[x] My code is Linted.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Looks like you will need to uncrustify those files.


trySendMessages no longer requires parameters. Fix startup request

Automated message from Dropbox CLA bot
@borisyankov, it looks like you've already signed the Dropbox CLA. Thanks!


Update Insights operator utils library version to 1.8.2

Update Insights operator utils library version to 1.8.2

Updated already


Changed get_course_info to count instead of fetch all.

Pending changes:

This breaks the current sorting (lines 207 and 231)
 all_backups.sort(key=lambda x, y: int(-5*(int(x.server_time > y.server_time) - 0.5)))

The frontend needs to adapt to the new changes.

We really only care about sending the first 10 file contents, so if we can do this without reading everything, it's still an optimization. (Not sure if it'll be enough)

A few students can't use the web site at all because of this call timing out. It'd be great if we can get this working soon - we're pretty close!

Does the frontend still need to adapt to these changes?

@Some3141 can you shed some light on this sorting? I'm confused why we can't just use a order in the queries...

@moowiz  re: frontend - Yes - but it'll be much easier to do once the call works.

@moowiz - I think I might not have realized I could order by time.
Also, is this ready to merge?

Okay, everything looks good to me.

@Some3141 - This isn't ready to merge. You can test by starting the server and trying to view the dashboard.

[ ] This breaks the current sorting (lines 207 and 231)
all_backups.sort(key=lambda x, y: int(-5*(int(x.server_time > y.server_time) - 0.5)))

[ ] The frontend needs to adapt to the new changes.

LGTM.


Pass Picker Widget State to builderHeader

Is there a way to reference a picker state inside of the builderHeader:?

Starting with version 2.0.1, you can do this:
final picker = PickerWidget.of(context);


Fixed bug for inserting to back of Dataframe

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Modin/37/
Test FAILed.


How to add parameter to devenv.nix?

I like to use a nixpkgs channel for most packages, but for some, I want the latest version, so I've added an extra input.
Now I'd like to have that available in devenv.nix as:
{ pkgs, pkgs-latest, ... }:

When I used to import the file manually I could do it:
let
    pkgs = nixpkgs.legacyPackages.${system};
    pkgs-latest = inputs.nixpkgs-unstable.legacyPackages.${system};
in {
    devenv.shells.default = (import ./devenv.nix { inherit pkgs pkgs-latest inputs config; });

But recently I needed to access config.env.DEVENV_ROOT so I had to change it to:
devenv.shells.default = {
  imports = [ ./devenv.nix ];
};

Which I guess is better anyways, but now I don't know how to get the pkgs-latest passed into there :thinking:

I'm in a flake with flake-parts, and tried some things, but not successfully so (see the commented parts):
      imports = [
        inputs.devenv.flakeModule
        ({ lib, config, ... }: {
          config = {
            # devenv.modules = [
            #   ({ inputs', ... }: { pkgs-latest = inputs'.nixpkgs-unstable.legacyPackages; })
            # ];
          };
        })
      ];
      perSystem = { config, self', inputs', pkgs, system, ... }: # perSystem docs: https://flake.parts/module-arguments.html#persystem-module-parameters
        let
          pkgs = nixpkgs.legacyPackages.${system};
          latest = inputs'.nixpkgs-unstable.legacyPackages;
        in
        {
          # devenv.shells.default = (import ./devenv.nix { inherit pkgs latest inputs config; });
          devenv.shells.default = {
            imports = [
              # ({ config, ... }: { config.pkgs-latest = inputs'.nixpkgs-unstable.legacyPackages; })
              ./devenv.nix
            ];
          };
        };

https://devenv.sh/common-patterns/#getting-a-recent-version-of-a-package-from-nixpkgs-unstable

Thanks.... that solves my current problem :+1: :pray: (I should RTFM)
Will come back if I find another use-case for this :)


Overlay fail but underlay works

Hello, please consider this as a possible bug.
Overlay file: overlay.pdf
Source file: source.pdf
This generates a bad result (overlay looks inverted):
qpdf --overlay ./overlay.pdf -- ./source.pdf ./result_overlay.pdf
This generates a correct file:
qpdf --underlay ./source.pdf -- ./overlay.pdf ./result_underlay.pdf
Maybe related to https://github.com/qpdf/qpdf/issues/559
Related with https://github.com/LeoFCardoso/pdf2pdfocr/issues/40

I am not able to reproduce this with qpdf 11.2.0. Both files render identically (based on machine comparison, not visual inspection).

Thank you for the fast reply.
Here are more technical details. I'm using macos.
macleonardo-2016:Downloads leonardo$ qpdf --version
qpdf version 11.2.0
Run qpdf --copyright to see copyright and license information.
macleonardo-2016:Downloads leonardo$ qpdf --overlay ./overlay.pdf -- ./source.pdf ./result_overlay.pdf
macleonardo-2016:Downloads leonardo$ qpdf --underlay ./source.pdf -- ./overlay.pdf ./result_underlay.pdf
macleonardo-2016:Downloads leonardo$ md5sum result_*.pdf
305058f51e04d28af0234669145d41e0  result_overlay.pdf
78997697199b35329dd83b69ad81035c  result_underlay.pdf
macleonardo-2016:Downloads leonardo$ uname -a
Darwin macleonardo-2016 21.6.0 Darwin Kernel Version 21.6.0: Mon Dec 19 20:44:01 PST 2022; root:xnu-8020.240.18~2/RELEASE_X86_64 x86_64

result_overlay: result_overlay.pdf

result_underlay: result_underlay.pdf

There was an important piece of missing information here....overlay.pdf looks blank but contains hidden text. You have to actually select text to see the effect. I didn't get this from the description, but when I decoded the overlay file and looked at it, I saw that it was filled with hidden text. If I hit CTRL-a after performing your operations, then I can see it. Now I have enough information to look into this. Thanks.

This one is quite interesting. I appreciate your taking time to report it. I have fully analyzed this and understand what's going on.
There's an error with the source file that qpdf is not taking into consideration. It would be possible to change how qpdf does overlay and underlay to make it not susceptible to this problem. In a nutshell, PDF allows you to push graphics state onto a stack and restore it. qpdf and numerous other tools do this to isolate a block of code from changes made by other code in the content stream. The source file here modifies the state stack in an unbalanced way, which means that qpdf's isolation of the original page is incomplete. There would be a way for qpdf to isolate the original page in a stronger fashion. I'll look into that fix. It would cause a change to the way overlay/underlay is implemented for all files, but I think it would be good for underlay/overlay to work even when there are errors on the source page.
Here's a detailed explanation of what's going on.
In PDF content, the q operator pushes the current graphics state (including scaling, etc.) onto a stack, and the Q operator restores the old graphics state by popping it off of the stack. In this way, surrounding a block of content by q...Q isolates any changes it makes to the graphics state from other changes. The way qpdf's overlay and underlay feature works (and this would be true of pretty much any tool that is not a full PDF interpreter) is that it prepends q before the entire content and then appends Q to it, thus isolating the existing content. Then it adds a little code either before or after (depending on underlay or overlay) that draws the item being underlaid or overlaid.
The problem is that the original source has unbalanced q and Q operators. The original source does nothing but draw an image. Somewhat strangely, it does this by inverting the entire page, then drawing the image inverted on the page. This is most likely done by some software so that it can think of coordinates as 0,0 at the upper left, which is typical of image file formats, rather than 0,0 at the lower left, which is used by PDF and PostScript (and usually mathematical Cartesian coordinate systems).
The content stream from the original source is this, with comments added by me:
stream
1 0 0 -1 0 1658 cm    % flip the page about the bottom edge, then shift up by the height, resulting in vertical flip
q
1648 0 0 -1658 0 1658 cm   % draw the image the same way: flip about the bottom, then shift
/I6 Do
Q
q    % Note: unbalanced q operator

In the overlay case (the one that's broken), qpdf ends up with the following, again comments inserted by me
q  % qpdf adds this to isolate the original content stream
% original content stream begins here
1 0 0 -1 0 1658 cm
q
1648 0 0 -1658 0 1658 cm
/I6 Do
Q
q
% original content stream ends here
Q  % qpdf adds this to end isolation of original content stream
% qpdf adds this to draw the overlay
q
0.9999 0 0 0.9999 0.03951 0 cm
/Fx1 Do
Q

If you look at the q and Q operators, you can see the problem:
q   % BEGIN 1
1 0 0 -1 0 1658 cm
q   % BEGIN 2
1648 0 0 -1658 0 1658 cm
/I6 Do
Q  % END 2 (back to 1)
q  % BEGIN 2

Q % END 2 (back to 1)
q % BEGIN 2 -- changes from "1" are still in effect, hence the inversion of the page!
0.9999 0 0 0.9999 0.03951 0 cm
/Fx1 Do
Q % END 2

The new code that qpdf adds to draw the overlay is this /Fx1 Do where /Fx1 is a "form xobject", which is a self-contained block of code that is more strongly isolated than is achieved with just q and Q. The fix would be to have qpdf put the original page in a form xobject as well as the overlay/underlay page. That would fully protect it from pages that are unbalanced in this way. The fact that underlaying the source works shows that this approach would work.
I'll take a look at the code and see how big a fix that would be.

Yes, I'll be able to fix this for 11.3.0, which I plan on releasing today if possible.

There's an error with the source file that qpdf is not taking into consideration.

Note that, prior to ISO standardisation, unbalanced q/Q operators were considered perfectly kosher. In fact, the possibility was specifically mentioned in the PDF 1.7 standard, rather than just allowed by omission, if I remember correctly.


Unity URP 2022.3.36f1 Compile Error on import

Assets\TrueTrace\Resources\Utility\Atmosphere\AtmosphereGenerator.cs(4,7): error CS0246: The type or namespace name 'CommonVars' could not be found (are you missing a using directive or an assembly reference?)

Sorry, I uploaded the wrong file, fixing it now


Reorganize By Namespace

#291
This is a feature that allows for the automatic reorganization of files by creating folder structures based on the namespaces defined within the files.

On thing i was thinking about, should the folder names be fix lowercase?

@waldo1001 i don't know how to forward that PR? can you help me here?

@christianbraeunlich @jwikman I hope one of you two can help me. How do I proceed with this Pull Request now?

@Spycoclown I cannot do much, I'm afraid.
Only @waldo1001 can merge PR's.
Just a quick comment from my side, don't leave unnecessary comments in the code. ;-)

First of all - thanks SO MUCH for this PR.  I'll try to handle this today ..

This is a wonderful update!
It is possibile to exclude a part of namespace from this function?
We usually create a namespace with this format:
...
(e.g. Contoso.BaseApplication.Sales.Customer)
So I would like to exclude the first two parts from the reorganization function.

Yes, I know :(
Don't worry, I'll wait to hear from you.


feat: improve uninstall script to find the installed version

This commit improves the experience of using the uninstall script by
automatically finding the version of kepler-operator installed. So, user
need not explicitly specify the version to installed (but they can
should they choose to) to uninstall.
It also adds a --help option to the script to print the usage

cc: @vimalk78 @vprashar2929

can we add an option to find resources even if find_installed_version fails? for those clusters where operator installation fails halfway.


Create SECURITY.md

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
Please describe the motivation of this PR and the goal you want to achieve through this PR.
Modification
Please briefly describe what modification is made in this PR.
BC-breaking (Optional)
Does the modification introduce changes that break the backward-compatibility of the downstream repos?
If so, please describe how it breaks the compatibility and how the downstream projects should modify their code to keep compatibility with this PR.
Use cases (Optional)
If this PR introduces a new feature, it is better to list some use cases here, and update the documentation.
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects, like MMDet or MMPreTrain.
The documentation has been modified accordingly, like docstring or example tutorials.

A lot of unnecessary information is being included in this. Show people the supported versions, rather than talk about them. Show people how to report a vulnerability rather than talking about discussing vulnerabilities.

Hi @KOSASIH,
We'd like to express our appreciation for your valuable contributions to the mmdetection. Your efforts have significantly aided in enhancing the project's quality.
It is our pleasure to invite you to join our community thorugh Discord_Special Interest Group (SIG) channel. This is a great place to share your experiences, discuss ideas, and connect with other like-minded people. To become a part of the SIG channel, send a message to the moderator, OpenMMLab, briefly introduce yourself and mention your open-source contributions in the #introductions channel. Our team will gladly facilitate your entry. We eagerly await your presence. Please follow this link to join us: https://discord.gg/UjgXkPWNqA.
If you're on WeChat, we'd also love for you to join our community there. Just add our assistant using the WeChat ID: openmmlabwx. When sending the friend request, remember to include the remark "mmsig + Github ID".
Thanks again for your awesome contribution, and we're excited to have you as part of our community!


support connection properties

Hi,
flink-jdbc-driver does not support connection properties currently. Thus,  SessionClient connected to flink-sql-gateway cannot change Environment at session level.
For example, In my senario, I deployed a flink sql gateway , I use beeline to connect it and submit flink sql query. The query always submit to that one Flink Cluster based on flink-sql-gateway's configuration. The execution configuration like time-characteristic/max-parallelism cannot change as well. This is tremendous  inconvenience.
I looked through flink-sql-gateway code. Actually it support overriding deployment/execution/tableconfig params at session level.
POST http://localhost:8083/v1/sessions
{ "planner":"blink", "properties": { "deployment.jobmanager":"helloflinkonk8s.test.rest.shbt2.k8s.flink.ads.qihoo.net:80", "execution.parallelism": 1000, "execution.max-parallelism": 1000   }   }
Then, I looked through flink-jdbc-drive code. It just ignore connection properties. I improved it, and now it works. I'm using beeline :

submitting to a remote flink cluster:

!connect jdbc:flink://localhost:8083?planner=blink&deployment.jobmanager=helloflinkonk8s.ultron:8081&execution.max-parallelism=1000

submitting to a yarn-session flink cluster:
!connect jdbc:flink://localhost:8083?planner=blink&deployment.executor=yarn-session&deployment.yid=application_1574648431818_877975

submitting query for yarn-per-job mode:
!connect jdbc:flink://localhost:8083?planner=blink&deployment.executor=yarn-per-job

I submitted a pull request , pls review


Refactor:changed ci workflow for janush.app

What has been introduced:

npm dependecines caching in cloud-deploy and web-deploy jobs has been added to speed them up a bit

the following custom actions to cloud-deploy job:

Cognito custom action - this action deploys janush autogenerated app stack and extracts: identity pool id, user pool id and user pool web client id data. Extracted values are exported then to the environment variables available for subsequent steps within the current job (cloud-deploy)

Amplify custom action - this action deploys janush autogenerated app ci stack and extracts: s3 bucket name, amplify app id and amplify app branch name data. Extracted values are exported then to the environment variables available for subsequent steps within the current job (cloud-deploy)
Adding this custom actions makes main.yml file more readable and ensures single responsibility rule for each of jobs in workflow. It means that data obtained within one job are obtained in that particular job and exported for further usage in other places. In case of urge to modify those actions, the following steps must be done:

Appropriate index.ts must be modified accordingly to individual needs

Modified index.ts file should be converted to index.js file using e.g. tsc command

To generated index.js file all dependencies used in its imports should be attached. One can use the @vercel/ncc library for that purpose to generated such a file automatically.

action from gitHub marketplace to cloud-deploy job - this action is used to get environment variables created by custom cognito action in cloud-deploy job and set them in repository secrets for further usage in web-deploy job. Thanks of that, secrets generated in one job can be exported safely to another one. In other to make that action work it is required to generate personal access token in codeandpepper github account and add it to secret with name TOKEN in janush.app repository, it must be done otherwise that action will not work.

What should be introduced in future:
There are still places especially in web-deploy job where one can introduce another custom actions to extract some logic form main.yml file making it even more readable.

I guess these pictures shouldn't be in the project. @przemyslaw-rzasa-cp removed them before they were published

I guess these pictures shouldn't be in the project. @przemyslaw-rzasa-cp removed them before they were published. At the moment they are on s3.

I think you're right. I created my branch from main before @przemyslaw-rzasa introduced his changes with pictures on s3. Then I created pr and sorted out conflicts but there was no conflicts with these images hence they are still there. I will check it and eventually remove what is required.


Update gir with removed type version heuristics / update gir-files with new versions

See https://github.com/gtk-rs/gir/pull/1229 / https://github.com/gtk-rs/gir-files/pull/85

Thanks!


SIGABRT using commit d4dc4f2a50f39b1c05efd955a6e9de0db2b197bc

[ 0][t 3][1627604606.817476034][MessagesManager.cpp:33091][#51][!Td][&*it != nullptr]   chat xxxxxxx server message 4706480 update_message

I pushed a likely fix, but do you have a log with an example of requests, which lead to the issue?

No, that error happened twice today, randomly. I can't reproduce it in any
way.
On Fri, Jul 30, 2021, 8:54 PM Aliaksei Levin @.***>
wrote:

I pushed a likely fix, but do you have a log with an example of requests,
which lead to the issue?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tdlib/td/issues/1641#issuecomment-890089009, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACBUY3CFHF53EUT3XTIGPODT2LYNRANCNFSM5BIEHAXQ
.

This should be fixed in master.


Mixins and ES6

First of all, :+1: for this. I haven't got time to use it yet, but the docs and demo/tutorial look impressive etc.
I totally understand the possibilities of mixins, but I'm hesitant about it because there's no real ES6 way for this yet, and as we know ES6 is here. I wonder if you have any thoughts on this issue and if you're worried that some core functionality will be rewritten as things take shape (or worse, a project is abandoned because of it).
Some references;
https://medium.com/@dan_abramov/mixins-are-dead-long-live-higher-order-components-94a0d2f9e750
https://github.com/rackt/react-router/issues/659
Cheers

No, mixins in Blaze Components were designed to work well with ES6 and JavaScript. As described in the README:

JavaScript is a single-inheritance language and instead of Blaze Components trying to force fake multiple-inheritance onto a language, it offers a set of utility methods which allow the component to interact with its mixins and mixins with the component. The code becomes more verbose because of the use of methods instead of overloading, overriding or extending the existing elements of the language or objects, but we believe that results are easier to read, understand, and maintain.

We decide to simply use a system on top of what JavaScript provides, without any mingling of JavaScript internals. See the diagram in the README.
Also, it is an optional feature. You do not have to use mixins. You can vanilla JavaScript or ES6. If that is good enough for you, great. If not, Blaze Components offer a non-intrusive API to provide mixins for you. It is mostly there so that if community develops mixins that those mixins can interoperate, instead of each implementing some fake ways manually.
So mixins as they are in Blaze Components are "real in ES6". They work with ES6. Maybe they should be named something else, just to not confuse things when there will be official mixin support in ES2099. But what is here is now and working, and behaves as a mixin. If there will be some mixins in the future, they will be not conflicting with these mixins (except in the name).
Also, mixins are not the core reason this package exist. This package exist mostly to make UI in Meteor more reusable. If there will be other mixins available, then that will be just another way to provide more composability. And we can always then just make internal implementation of Blaze Components mixins use official mixins.
I also talked with Dave Herman about ES6 and mixins do not seem to be anywhere near. So I do not think we should worry about this now. And in meantime we can use these non-intrusive mixins. :-)

So mixins are offered now because nothing better exists. But you can decide not to use them and use only inheritance. I think that for UI composability is better, but we will see how things will go. This is the idea of the project. To create a community developing reusable components. And we will all learn what works and what does not. How you compose or inherit event handlers? How you compose or inherit animations? How you compose or inherit template content? Still many open questions. But at least now we have a space around which we can start trying things out.

(Closing as answered, but discussion can continue.)


Fix memory leak for multiple runs in the same process

As reported on #2767 by @agis, there's a memory leak in RSpec when having multiple runs in the same process.
Causes
After a lot of investigation, I've managed to locate the leak's causes and those caused by the rspec-core's gem are resolved with the introduced changes of this PR.
RSpec::Core::World - @filtered_examples
The world's instance variable @filtered_examples' keys are classes of RSpec::Core::ExampleGroup generated for a run's example groups (a.k.a. RSpec::ExamplesGroups::Foo).
Since the instance variable is not being cleared between runs, it keeps references to these classes (along with their "expensive" state - each group class contains all of its examples along with their reporter/loader/formatter etc).
Note: even though the RSpec::ExampleGroups.remove_all_constants does remove the constants, the references above keep them alive.
RSpec::Core::AnonymousExampleGroup
Examples that are added to the RSpec::Core::AnonymousExampleGroup upon their initialization (ex. RSpec::Core::SuiteHookContext) don't currently get cleared between runs.
RSpec::Core::SharedExampleGroup::Registry
The RSpec::Core::SharedExampleGroup::Registry maintains references to RSpec::Core::ExampleGroup classes that were generated by previous runs (as keys in the @shared_example_groups hash) preventing them from being garbage collected.
Benchmarking
Introducing the changes of this PR to the reproduction script made by @agis confirmed the fix.
In the previous state, the memory keeps growing and on the 10.000th iteration has reached ~800Mb.
With this PR's fixes, the memory reaches a plateau pretty early and on the 10.000th iteration is at ~39Mb.
External causes
Besides rspec-core, I found other "external" causes for memory leaks depending on the codebase's used libraries.
I'll try to open the proper PRs for those as well. Until then, find below some info for each of them along with workarounds in case someone finds them helpful.
rspec-mocks & rspec-rails
The RSpec::Mocks::Configuration also keeps references to RSpec::Core::ExampleGroup classes.
The following rspec-rails section
      def self.initialize_activerecord_configuration(config)
        config.before :suite do
          # This allows dynamic columns etc to be used on ActiveRecord models when creating instance_doubles
          if defined?(ActiveRecord) && defined?(ActiveRecord::Base) && defined?(::RSpec::Mocks) && (::RSpec::Mocks.respond_to?(:configuration))
            ::RSpec::Mocks.configuration.when_declaring_verifying_double do |possible_model|
              target = possible_model.target

              if Class === target && ActiveRecord::Base > target && !target.abstract_class?
                target.define_attribute_methods
              end
            end
          end
        end
      end

registers a before suite hook in RSpec's configuration which in turn alter's RSpec::Mocks::Configuration
      def before_verifying_doubles(&block)
        verifying_double_callbacks << block
      end
      alias :when_declaring_verifying_double :before_verifying_doubles

I believe that some of these blocks are being defined through RSpec::Core::ExampleGroup class instances (when their SuiteHookContext examples execute) and since they live forever in the RSpec::Mocks::Configuration instance, they keep the references to their RSpec::Core::ExampleGroup alive forever.
Workaround (monkey patch):
module RSpec
  module Mocks
    def self.reset
      @configuration = nil
    end
  end

  module Core
    class World
      alias_method :original_reset, :reset

      def reset
        original_reset
        RSpec::Mocks.reset
      end
    end
  end
end

ActiveRecord
ActiveRecord also keeps RSpec::Core::ExampleGroup class instance references as keys in its @@already_loaded_fixtures class variable here:
      if run_in_transaction?
        if @@already_loaded_fixtures[self.class]
          @loaded_fixtures = @@already_loaded_fixtures[self.class]
        else
          @loaded_fixtures = load_fixtures(config)
          @@already_loaded_fixtures[self.class] = @loaded_fixtures
        end
        ...

Workaround (monkey patch):
module RSpec
  module Core
    class World
      alias_method :original_reset, :reset

      def reset
        original_reset

        ActiveRecord::TestFixtures.class_variable_get(:@@already_loaded_fixtures)&.delete_if do |klass, _|
          klass < RSpec::Core::ExampleGroup
        end
      end
    end
  end
end

ActiveSupport
ActiveSupport also keeps a record for each class for which it has loaded its hook here:
    def run_load_hooks(name, base = Object)
      @loaded[name] << base # <- base can be of RSpec::Core::ExampleGroup
      @load_hooks[name].each do |hook, options|
        execute_hook(name, base, options, hook)
      end
    end

The block above is being executed for bases that are RSpec::Core::ExampleGroup classes thus references to them live forever.
Workaround (monkey patch)
module RSpec
  module Core
    class World
      alias_method :original_reset, :reset

      def reset
        original_reset

        ActiveSupport.instance_variable_get(:@loaded)&.each do |_, bases|
          bases.delete_if do |base|
            base.is_a?(Class) and base < RSpec::Core::ExampleGroup
          end
        end
      end
    end
  end
end

Closes #2767

Closing due to inactivity during the monorepo migration.


Add new study - using Trial Registry - search registry independent

When adding a new study by using the trial registry you have to choose the registry first and to enter the exact ID. Would it be possible to have a registry independent search of ID or title (through study characteristics like in MDR)?

it requires to update the new MDR code. I would suggest to leave everything as it is now and complete the overall integration of the new back-end with the new front-end. Once it's done, I will update the new MDR code and add an additional endpoint to search studies avoiding using registry ID.


provideStore not working since update of angular to rc

Doing this in the boot.ts file:
`import {bootstrap} from "angular2/platform/browser";
import {HTTP_PROVIDERS, Http} from "angular2/http";
import {MainApp} from "./main/app";
import "rxjs/BehaviorSubject";
import 'rxjs/add/operator/map';
import {provideStore} from '@ngrx/store';
import {users} from "./users/reducers/user.reducer";
import {UserService} from "./users/services/user.service";
//register the store
bootstrap(MainApp, [
HTTP_PROVIDERS,
UserService,
provideStore({users})
])
.catch(err => console.error(err));`
produces the following error:
"Uncaught Invalid provider - only instances of Provider and Type are allowed, got: [object Object]"
This was working fine before RC.
Cheers
Jusef

I dont think this is an issue with ngrx/store.
did you clean your node_modules? (old angular should be gone)
you still reference import {bootstrap} from "angular2/platform/browser";
But for rc0 it changed to import {bootstrap} from '@angular/platform-browser-dynamic';

Yup, that is it. Thank you very much and apologies for waisting your time, still trying to get to the grips with all the new updates.


400: This action is restricted for safety reasons at this time.

Hello,
The demo is working but 90% of the time I got a 400 error This action is restricted for safety reasons at this time.
BTW congratulation for building this so simple test bed !

Duplicate of #74


run 'demo_deep_pyramid.m', get  error using caffel.....,

Error using caffe
Expected 3 arguments, got 2
Error in init_cnn_model (line 60)
caffe('init', cnn.definition_file, cnn.binary_file);
Error in demo_deep_pyramid (line 43)
cnn = init_cnn_model('use_gpu', USE_GPU, 'use_caffe', USE_CAFFE);
However I could run dp-dpm succeed, with matlab2015,caffe-0.999,Tesla K40

It seems the usage of Caffe in init_cnn_model.m is outdated and was written before the caffe.Net class was introduced to Caffe's Matlab interface.
Here is a more up to date example on instantiating a caffe network instance.
Example:
net = caffe.Net(cnn.definition_file, cnn.binary_file, 'test');
Further changes may still be needed that update how dp-dpm uses this net object.

@kashefy Thanks, I have solved this problem, this code is fit to caffe-1.0,the caffe_init,. include two arguments


Add contact information

Hi Nima, I see you've said you'd like to learn from people but you've not included any contact information. So wanted to recommend you add some (idk, maybe at least an email) in there so people can connect with you more easily. Have a nice life 👋.

Hi @aderchox  I cant because of security reason, however if there is any thing some one wants to ask/answer they can send them here, I answer as soon as I can.
Best wishes.

That's weird, you could create an email with any made up names and put it on your account as your contact information and no one will find out who you are, anyways I won't insist, it's your own choice :).