Implement split ORCID/local authentication

The previous platform(s) only provided ORCID authentication, but we want to enable both ORCID and non-ORCID authentication for user accounts. This is comprised of:

[x] Enable the Passport modules for ORCID and local auth
[ ] Enable switching between the two authentication methods for a given user account
[ ] Add authorization role for non-ORCID accounts

Based on discussion, deferring this and relying on OrcID. Will re-open if we decide to support non-OrcID users in the future.


Replace turbolinks with basic, custom service worker

Starting point guide here - https://fossbytes.com/get-jekyll-blog-work-offline/

Idea: have a select few navbar images, and the service worker sets it to be one of those? Avoid the HTTP requests.


Move off old GraphQL API

Let's not remove it entirely because we might want to reuse some of the underpinnings for the product's GQL API.

I am removing some old API cruft @pastuxso. Please review.
This is a precursor to reporting data into the product from kernel/CLI (cc @peraltafederico).


Identify accounts by public key + token

This PR implements the ledger, account and transaction logic changes necessary for transactions in tokens other than the default Coda token.
Notable changes:

Accounts are now indexed by a public key and a token ID.

These are packaged as Coda_base.Account_id.t, an opaque type with accessor functions for the pk and token.

Transactions now involve up to 3 accounts: the account paying the fee and signing the transaction, the source of the tokens to be sent, and the recipient of the tokens.

Currently the public key of the first 2 accounts is necessarily equal. When account predicates are implemented, this will be relaxed, and any account may trigger a user command against any other account.

The transaction snark has been rewritten to support these 3 accounts.

This now requires an additional merkle tree update, to support the third account.
More of the logic is also inlined in the main function, with labels that better describe errors where they may occur.

Transactions with differing signing and source accounts always 'succeed' if the signer has enough in their account to pay the fee.

If the source account doesn't have sufficient tokens, then only the fee is charged, and no tokens are moved.

This simplifies the logic in the transaction pool, while still protecting against cheap DoS attacks against the network.
When multiple accounts are able to sign transactions against the same account, this will make racy behaviour more predictable, and avoid throwing out valid transactions with later nonces for any of the signing accounts.

The account creation fee for transactions involving other tokens is charged to the signing account, instead of subtracted from the transaction amount.

This ensures that the creation fee is paid in a token with an accepted value for the network.
The normal behaviour is conserved for transactions and fees in the same token.

The receipt chain and nonce are now updated for the signing account only.

Bugfixes included:

The timed account code has been updated so that fees charged cannot take an account below its current threshold.
The typo in the transaction logic for fee transfers here has been corrected.
Assorted versioned type fixes.

TODOs before merge:

[x] Where new versions of types have been added, remove the old ones and rename the new ones to V1
[x] Rename all the uses of accounts for transactions to the correct one of signer, source, or receiver.

Next steps:

This does not yet implement token minting.

Blocked on protocol state in the transaction snark (#4244).

This does not support paying fees in any token other than the default coda token.

This would need an exchange-rate mechanism, no proposals for this yet.

These changes are not exposed to the user-facing GraphQL APIs

We need minting before users will be able to create other tokens to use.

There are no new tests, since it's not possible to mint non-coda tokens yet.

Checklist:

[ ] Tests were added for the new behavior
[ ] All tests pass (CI will check this if you didn't)
[ ] Serialized types are in stable-versioned modules
[ ] Does this close issues? List them:

Updated. I will add some more tests before I finish for the evening, but this should be otherwise ready-to-go.
Notable changes:

The shape of transactions has been updated to match the form mentioned in this gist linked in  #4447.

The fee_payer_pk is specified inside the transaction, even though it should currently be the same as both the sender_pk and the signer.

This allows adding multiple signatures and predicate evaluation without having to rewrite the transaction snark again.

The token IDs are specified separately from the account public keys, to clear up the code.

It was confusing before to have to do Account_id.create source_pk (Account_id.token_id receiver_pk) when what we really wanted was Account_id.create source_pk token.

The transaction snark now accepts user commands that fail, and only charges the fee without making any other changes.

This ensures that the network is compensated for processing these transactions, since trying to predict whether commands involving multiple accounts will fail isn't viable to do in the pool.
These failures are predicted statically in the Transaction_snark.User_command_failure module, and then verified in the transaction snark.
This fixes #4500.

Accounts are now only created if the account creation fees have been paid.

This fixes #4498.

Account timing checks are now done whenever an account's balance is decreased.

This ensures that funds that are time-locked cannot be spent as fees etc.
This also fixes a bug where the transaction logic/transaction snark would generate a different hash from the sparse ledger implementation for timed accounts.


cypress: create team, view team details, edit description

Overview
Add cypress tests for the team flows.

[ ] Cancel creating a team
[ ] Create a team
[ ] View team details
[ ] Edit a team description
[ ] Add users
[ ] Cancel add users
[ ] Remove users
[ ] Delete team

Resources

Acceptance testing script

this work has been deprioritized.


stripping the 'L' at the end of the Python hex representation for lon…

…g numbers

[Current coverage][cc-pull] is 96.19%

Merging [#47][cc-pull] into [master][cc-base-branch] will increase coverage by <.01%

@@             master        #47   diff @@
==========================================
  Files             8          8
  Lines          1285       1287     +2
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           1236       1238     +2
  Misses           49         49
  Partials          0          0

Powered by Codecov. Last updated by [cad39e8...941c6b0][cc-compare]
[cc-base-branch]: https://codecov.io/gh/p4lang/p4c-bm/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/p4lang/p4c-bm/compare/cad39e8fa51ce02749864b362dddfc3a63a89df3...941c6b0da115edd782cce15dc111e4ab30760b80
[cc-pull]: https://codecov.io/gh/p4lang/p4c-bm/pull/47?src=pr


FIX PROPOSED: RKRelationshipConnectionOperation does not remove relationships

@blakewatters Currently RKRelationshipConnectionOperation is written in such a way that if the JSON value of the connection specifier is nil then it would keep the current relationship. Shouldn't it remove the relationship instead?
Sifted through the code and found that this can be easily implemented by just adding an else condition (as shown below) to the if (shouldConnect) RKRelationshipConnectionOperation.m:193
else
{
    [self.managedObjectContext performBlockAndWait:^{
        if (self.isCancelled || [self.managedObject isDeleted]) return;
        @try {
            if ([self.managedObject valueForKeyPath:relationshipName]) {
                [self.managedObject setValue:nil forKeyPath:relationshipName];
                RKLogDebug(@"Removed relationship '%@' from object '%@'", relationshipName, self.managedObject);
                if (self.connectionBlock) self.connectionBlock(self, connection, nil);
            }

        }
        @catch (NSException *exception) {
            if ([[exception name] isEqualToString:NSObjectInaccessibleException]) {
                // Object has been deleted
                RKLogDebug(@"Rescued an `NSObjectInaccessibleException` exception while attempting to establish a relationship.");
            } else {
                [exception raise];
            }
        }
    }];
}

Up! After two years, the proposed fix has not yet been merged (0:24 dev branch). @blakewatters Are possible unexpected behaviors resulting from this fix?
The fix is necessary when mapping relationship through foreign key.
For example:

[itemMapping addConnectionForRelationship:@"category" connectedBy:@{@"cat" : @"idc"}];

First mapping: everything works well, the connection is added and items 1 and 2 set the relationship with category 201.

Server side: remove item 1 from category 201 (note: the category node is not removed considering a one-to-many scenario - item2 has still cat 201 - otherwise fetchRequest for orphaned category together with cascade rules on CoreData model would have nullified category relationship on item as expected). Payload looks like:

Second mapping: item 1 keeps the relationship with category 201, even if its attribute "cat" is nil. Using the fix proposed by @dhanushram the relationship is correctly nullified.

Thanks.

@Donnit is there a PR for this fix? If so, please link it.

@segiddins I think there is not
@dhanushram did you send a Pull Request?

Well, that would explain why nothing has been merged :)

@segiddins you're absolutely right. Let's wait for the author of the proposed fix.


webview crash

📚 AppMetrica SDK Version
6.3.0
📝 Description

🔍 Reproducing the issue
I don't know how to reproduce this crash
📋 Relevant AppMetrica Log Output or Crash Logs

Expand to view logs

// Your logs go here

📦 Dependencies

Hi, this is a known issue, the solution is described here.


Setting .affine attribute of image layers resets contrast_limits

🐛 Bug
Changing the affine attribute of the image layer resets the displayed contrast_limit ( image looks like default, the .contrast_limits attribute of the layer is unchanged).
To Reproduce
Steps to reproduce the behavior:

Change contrast limits of an image
Set .affine property viewer.layers.selection.active.affine = napari.utils.transforms.Affine()
Image will look like before contrast limit change

Expected behavior
Changing affine matrix should not reset contrast_limits

Hmm, it doesn't reset it for me (using examples/add_grayscale_image.py), but it definitely messes with it! Thanks for the report @jojoelfe.

I have tested it with examples/add_image.py and there the effect is much stronger. Not sure if it's because of RGB or 8bit vs 16bit.

I could reproduce this on v0.4.12 at least using the RGB image example (examples/add_image.py), though Image.contrast_limits did not change only the appearance of the image.
I cannot reproduce it on v0.4.15 or on main, so I think this has been fixed now. I'm struggling to understand the bug or a simple test to check it, so I'm not curious enough right now to git bisect this to find what fixed it, but maybe it would be enough for @jojoelfe to confirm that this is fixed in v0.4.15 or the main branch? If so, please close the issue!


[@RNC/AsyncStorage]: NativeModule: AsyncStorage is null (0.61.5)

I upgraded react-native from 0.59.8 to 0.61.5
Unlinked @react-native-community/async-storage
App builds successfully and splash screen comes. After that it says
[@RNC/AsyncStorage]: NativeModule: AsyncStorage is null
- Run react-native link @react-native-community/async-storage

When I tried linking it says
info Linking "@react-native-community/async-storage" iOS dependency
error Linking "@react-native-community/async-storage" failed.

If I try unlinking again (I already did before as I encountered error while upgrading), it says
info iOS module "@react-native-community/async-storage" is not installed
info Android module "@react-native-community/async-storage" is not installed

Why does it say @react-native-community/async-storage is not installed even after I installed the package?
I tried removing the package and installing again. Same errors

@y-sumanth
Have you checked troubleshooting docs?

I am facing this on Android

I am facing this on Android too.How to resolve this?

If you upgraded from < 0.60 version, you need to unlink all your dependencies and then relay just on cli feature. Please read the docs to get more info.
If you started with version 0.60+, then autolinking should kick in. no react-native link is required. You just need to make sure that the setup you have follows the proper setup. Again, here are the docs for autolinking.
If you're below version 0.60, then you have to install the dependency manually or through react-native link name-of-the-library. We have guides for manual linking here

@Krizzu I'm having the same issue on Android. The only package I have manually installed is react-native-navigation by Wix. Everything else relies on auto-linking.
I also tried to install AsyncStorage v2 to see if that would fix my problem and the issue persists.


[Migrator] Remove closure shorthand prefixing.

In https://github.com/apple/swift/pull/10414, some SE-0110 features
were backed out, so the Migrator no longer needs to add $0 shorthand
prefixing.
rdar://problem/32907276

@swift-ci Please smoke test

@nathawes Can you review for the 4.0 branch?


Wiki - Stories - Universal Rendering is missing a key ingredient for lazy loading

The final part of the first step for the universal rendering is missing a very key part that is necessary for lazy loading to work correctly on the server.
Specifically app.server.module.ts should look like this:
import { NgModule } from '@angular/core';
import { ServerModule } from '@angular/platform-server';
import { ModuleMapLoaderModule } from '@nguniversal/module-map-ngfactory-loader';

import { AppModule } from './app.module';
import { AppComponent } from './app.component';

@NgModule({
  imports: [
    AppModule,
    ServerModule,
    ModuleMapLoaderModule
  ],
  bootstrap: [AppComponent],
})
export class AppServerModule { }

Note ModuleMapLoaderModule is necessary, otherwise node will throw an error every time a page that is supposed to be lazy loaded is attempted. An error akin to: ERROR: Cannot find module './lazy/lazy.module.ngFactory'.
Interestingly this would be difficult to catch if you are not reading the node output since the pages still load only they are being rendered on the client, defeating the whole server side rendering purpose.

@Brocco can you investigate?


Load Balancer mode - user dependency injection

What should we add or change to make your life better?
At present the Load Balancer implementation has a fixed number of modes, and the implementation of these are rigid. however the number of load balancing algorithms/possibilities are infinite some can be complex and data-driven from external eg. service-discovery
IMHO using dependency injection to allow a developer to register implementation of the logic for selection/routing to a endpoint; and thus remove the fixed nature of the LoadBalancer mode of operation.
Why is this important to you?
As a developer I would like to extend/add/replace the mode of operation of the reverse proxy LoadBalancer with my own implementations in a way that has minimal code complexity and is easy to deploy and test in isolation.
ie. LoadBalancer - Green/Blue deployment routing with traffic slip 60% - 40%

Making load balancing more extensible is covered by https://github.com/microsoft/reverse-proxy/issues/106
FYI: we anticipate Green/Blue rolling deployments to be implemented as a distinct feature from load balancing, see https://github.com/microsoft/reverse-proxy/issues/126

@Tratcher Thanks; I was using Green/Blue as an example of where the loadbalancer splits traffic or selects and endpoint based on some computation which is not currently implemented, possibly a confusing example given the name.

Duplicate of #106


dump2data.py fails when box size changes (NPT)

The dump2data.py utility seems to fail when reading trajectories from simulations that were not run at constant volume (such as NPT simulations).  I noticed this when interrupting a constant-pressure simulation and using the dump2data.py program to convert the last frame from the incomplete trajectory to start a new simulation.
I don't need this feature in my own work, so it might be a little while before I get around to fixing this problem.
-Andrew

This issue was (hopefully) fixed by commit e4bf2c6.


Update the Created event with the miningTry value

event Created(
    uint256 indexed tokenId,
    Rarity rarity,
    uint8[2] color,
    uint256 value,
    uint8[4] quadrants,
    uint32 miningPeriod,
    uint256 cooldownDueDate,
    string tokenURI,
    address owner
);

we should need to add the miningTry value on this event.

@jusdy fixed in this commit


Fast API Websocket coroutines throw attribute error when run with newrelic admin

Description
I am using newrelic agent 8.5.0 with my fastapi which houses both rest and websocket apis. The app get deployed in kubernetes and here is how the container command looks like
   containers:
        - name: a-service
          image: 'a-service:latest'
          command:
            - newrelic-admin
            - run-program
            - uvicorn
            - app.main:app
            - --port=8000
            - --host=<IP_ADDRESS>
          resources:
            requests:
              memory: "512Mi"
              cpu: "500m"
            limits:
              memory: "512Mi"
              cpu: "500m"
          ports:
            - containerPort: 8000
              protocol: TCP

The container comes up healthy and runs , but the coroutines throws error with attribute not found error when calling async functions.  The same is working fine when i run the container without newrelic-admin

Hello,
Could you perhaps supply a reproduction or at least some more specifics?
I was unable to reproduce any issue with the latest version of the agent. FastAPI has a minimalist example that I ran some testing against. I was using an async websocket endpoint and both sync and async HTTP routes were working alongside it with no issues.

Hello @sankarpa, without reproduction steps we will close this ticket in 30 days.

Hi @Ak-x  Sorry for getting late,  I am using aioredis pubsub in the websocket functions and the code looks like below.
 pubsub = redis.pubsub()        await pubsub.subscribe(f"{channel_id}") await websocket.accept()
if I don't use newrelic agent , the subscribe function  call works fine .
I get below error if I initialise the newrelic agent and connect to the websocket
AttributeError: 'coroutine' object has no attribute 'subscribe'

@TimPansino  Any updates on this?


20190122 config frags

/test

@ganeshmaharaj - Please can you put a description on this PR? I'm also confused by even the title so maybe that could be reworked too? :)


Long Subtitle text doesn't correctly wrap in DisplayMode "hub" or "banner"

Describe the bug
When using a long Subtitle and in DisplayMode "hub" or "banner" the subtitle doesn't wrap correctly and cuts off on both sides left and right
To Reproduce
Steps to reproduce the behavior:
The following code will demonstrate the issue
AlertToast(displayMode: .alert,
		type: .regular,
		title: "Sample Message",
		subTitle: "Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec gravida eleifend scelerisque. Donec ornare malesuada orci, nec tempus massa lacinia non.\n\nMaecenas sit amet convallis orci. Aenean malesuada lacus sed odio pulvinar elementum non a tellus. Etiam cursus erat sed fermentum lobortis. Phasellus feugiat euismod orci. Donec ultrices fringilla fringilla.")

Expected behavior
The text should be displayed the same way it does with DisplayMode "alert".
Screenshots

This is the toast using "hud"

This is the toast using "alert"
Smartphon:

Device: iPhone 14
OS: 16.3

Which version of AlertToast did you installed?

1.3.9

On 15. Feb 2023, at 12:31, Elai Zuberman @.***> wrote:
Which version of AlertToast did you installed?
—
Reply to this email directly, view it on GitHub https://github.com/elai950/AlertToast/issues/61#issuecomment-1431223762, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB5PAFD4PNI3AIU4XIZRRNDWXS5BNANCNFSM6AAAAAAU4V5MFY.
You are receiving this because you authored the thread.

@elai950 I ran into this and the v1.3.9 release does not contain the commit 638f38f9daf08e17b7caea22d2fcb9c0a418d1b6.

Yea still the issue, modifying the file directly with the commit ^ in your project Pods is an option assuming you commit pods to your repo.

Is this still happening in iOS 16.4? We currently have a report referencing this exact issue running iOS 16.3.1 but we cannot reproduce the issue on our devices running 16.4 or the simulator running 16.2 or 16.4.

@elai950 Could you have a look why the 1.3.9 doesn't contain the commit please?

Its strange that issue is still not resolved


Added "Page Break" to Lexical Playground

New Page Break Plugin
Page Break - breaks page. You can see it when you are exporting this document to PDF.
Its helpfull thing if you are working with PDF document.
On exporting created HTML to PDF

How to create page break?

Using toolbar (Click to + insert button in toolbar, and select Page Break option)
Using slash / menu (Start writing /page and select Page Break)

Additional information
I have loaded scissors icon from there

For icon using CC0 License

Looking great! For the icon, we've been using the bootstrap icons pack, it has a scissors icon as well. Would be nice to keep it consistent.
https://icons.getbootstrap.com/

Ah, ok. I will change it tomorrow soon as possible :)

Looking great! For the icon, we've been using the bootstrap icons pack, it has a scissors icon as well. Would be nice to keep it consistent.
https://icons.getbootstrap.com/

I have replaced scissors icon to bootstraps scissors icon

Just formatting left to pass integrity checks and it's ready to go


$linkedIn->isAuthenticated() always returning false

Hello,
This is not about a "real" issue but I'm stuck with my problem, trying to make a LinkedIn Connect.
I did some LinkedIn connects but right now I don't know where my issue can be.
In the login process everything is going well, but at the end both $linkedIn->isAuthenticated() and $linkedIn->hasError() return false, because the access token might not be recognized.
The only difference I found with my other made LinkedIn Connects (using your lib) is that the urls are encoded so the GET parameters might not work well.
Here is a var_dump($_GET):
array (size=1)
  'code=AAAZyVupt8xF-0-UCdC9-P-HAYaS54Mos7f71JQ-M-OKUlSk1j0KwQ7FRy0Nd0tzbl8lQ7-petcUWA4TrNZ_V9WssMOQEHUWkBXGi-Zz4aFpXJ9cywc&state=eabddd7452008ca50ead94a050a547786' => string '' (length=0)

So I tried to repopulate it correctly as it follows:
array (size=2)
  'code' => string 'AAAZyVupt8xF-0-UCdC9-P-HAYaS54Mos7f71JQ-M-OKUlSk1j0KwQ7FRy0Nd0tzbl8lQ7-petcUWA4TrNZ_V9WssMOQEHUWkBXGi-Zz4aFpXJ9cywc' (length=115)
  'state' => string 'eabddd7452008ca50ead94a050a547786' (length=32)

But $linkedIn->isAuthenticated() is still returning false :/ so it may not be related to the urls encoding.
Any idea how I could get it working?
Thanks

Do you have the same issue with the PSR7 branch?

Hey Nyholm, getting the same issue here. Is there anything that's changed in the package? Can't find the PSR7 branch to test

The PSR7 branch is now merged into master. There is a quite large change it the package between the latest stable version and master. You can see it here: https://github.com/Happyr/LinkedIn-API-client/pull/73

Perfect, thanks man. Appreciate that


Display Attack/Skill Range While Moving

Have an option that'll display the current attack or skill range during the moving phase, allowing the player to make sure their current path is one that will allow the action they wanted.
Attacks tend to be made so that you are at the limit of your range (and hopefully out of
the enemy's), which makes it difficult to properly position characters on the first try. Going back to the movement mode once the destination has been selected is a hassle, so having something akin to a second (or third?) navigator dedicated to showing the action range while moving would help.
Computing this action range is slightly taxing, so it may be preferable to leave this as an easily reachable option.

The new control interface makes it less of a hassle to change destination. It might be a preferable option, since it won't clutter the screen with even more info. Still, options are nice, so I'll leave that issue open for now.


Wrong Documentation on Custom User Models

I have a user model without username and only email and the following settings:
ACCOUNT_AUTHENTICATION_METHOD = 'email'
ACCOUNT_EMAIL_REQUIRED = True
ACCOUNT_UNIQUE_EMAIL = True
ACCOUNT_USER_MODEL_USERNAME_FIELD = None
ACCOUNT_USERNAME_REQUIRED = False
ACCOUNT_EMAIL_VERIFICATION = 'none'

For this to work, i need to add a custom LoginSerializer that set username = None:
class CustomLoginSerializer(serializers.LoginSerializer):
    username = None

Now when I open http://<IP_ADDRESS>:8000/dj-rest-auth/login/ the self describing api tells me in the description Accept the following POST parameters: username, password Return the REST Framework Token Object's key.
It should tell me to use the parameters: email and password not username.
I have found no way to replace the LoginView or to change the description.
Secondly it would be nice in dj-rest-auth to respect the above settings and automatically disable usernames if they are disabled.

I'm also having this same issue. I'll be looking into it further for the next few days and I'll share any solutions I find if any

Hey @jerlendds  Did you find any solutions, please?


How to train my dataset with TrackRCNN?

Thanks for sharing, I downloaded your data set and tested it, but I don't quite understand how your data set is made. Is it feasible to convert to txt format with labelme or labelimg? How to train my own data set? @pvoigtlaender

We have both the txt format and the png format versions which are equivalent (they are described in more detail at https://www.vision.rwth-aachen.de/page/mots).
The mots_tools repository provides a script to convert from png to txt format: https://github.com/VisualComputingInstitute/mots_tools/blob/master/mots_common/images_to_txt.py
It basically just uses the rle encoding provided by coco.
If you want to use your own dataset you have to find a way to export it in either png or txt format.
I didn't hear about labelimg, but we also used labelme in our annotation pipeline. You can read the json format of labelme in with python and then export as png (a few lines of code will be needed for that).

Thank you very much for your reply. I want to label my own data in the format of the MOT dataset with labelme, but I see that your dataset contains ReID. How is this ReID determined?

I'm not sure what you mean with ReID here, in the dataset we label the instances with a consistent ID over time, is that what you mean?
If yes, then just make sure that somewhere (for example the filename of the jsons you write out with labelme) you keep the instance ID and put it in when exporting to the png/txt format.
Be aware that labeling each frame with labelme without the semi-automatic annotation procedure we used will be quite tedious

Sorry, I see rle as a reid. I mean how to define this rle? Also, is the semi-automatic annotation tool mentioned in your paper open source? Thank you very much. @pvoigtlaender

rle just means run length encoding, an efficient encoding of the mask of an object used by COCO. Using the coco tools you can easily convert numpy masks to rles.
The semi-automatic procedure is not just a single tool, the details are explained in the paper. The polygons were drawn using labelme, the rest is not really published at the moment, but we plan to do that (not sure yet when).

Thank you very much for your answers, I will try to do a small part of the dataset and train it。


Migrate RD Licensing servers section

In the intro of this section it should be written "... to a destination server running Windows Server 2016" instead of "...to a destination server running Windows Server 2012 R2."

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

ID: c9ac782d-d585-fa70-6142-4c1524d70c75
Version Independent ID: f280543a-f892-8891-8760-74f8dd12f746
Content: Migrate your Remote Desktop Services deployment to Windows Server 2016
Content Source: WindowsServerDocs/remote/remote-desktop-services/migrate-rds-role-services.md
Product: windows-server-threshold
GitHub Login: @ChristianMontoya
Microsoft Alias: chrimo

Thanks for raising the issue - Christian submitted a fix, so we should be good.


6.2: Created new first-use topic for To-Do Service API documentation

Two links are for assignments 6.1 and 6.3:
6.1: [Back to Landing Page](overview_david.md)
6.3: [API Reference: Task by User ID](api/api_reference_task_by_user_id_david.md)

Rebased PR/feature branch to current main branch.

I've made all the changes you requested. Previously I was only previewing in Visual Studio Code but for this revision I built the pages in GitHub and noticed a couple of things:

To get {{base_url}} to render properly you have to put them in liquid tags like this:

{% raw %}{{base_url}}{% endraw %}
To show the content inside the raw/endraw tags you put the backticks around it like this:
{% raw %}{{base_url}}{% endraw %}

When rendering in GitHub Pages, numbering can be thrown off if using only Markdown. I've been using a combination of HTML and Markdown (and liquid) in my md files because all of it renders as HTML anyway, especially when using GitHub Pages, Jekyll and other static site generators. To fix this using Markdown, use:

1\.
2\.

shell/json code here

3\.

Otherwise it'll start the numbering over again.

I cannot find what you changed in my index.md file. I'm positive it's my fault somehow. It was there initially and now it's gone. Do you have any suggestions?

I appreciate your feedback!

Great update!

I forgot about the {{ xxx }} issue. You applied the correct fix!
I replaced the list item notation (1. 2. 3....) from the escaped period notation you applied. What was happening was that your indented lists weren't indented enough. It looks like the static site generated likes 4 spaces of indent to not lose track. Indenting the sub lists lets the outer numbering not reset.

Force pushed minor revisions required to sync topic to current state of the main branch.


Update Tinkerbell stack images to latest release versions

Update the Tinkerbell stack images to the latest released versions.

App
Version

Rufio
v0.2.1

Tink
v0.8.0

Hegel
v0.10.1

CAPT
v0.5.0

Boots
v0.8.0

These changes introduce fixes that enable removal of disk constraints in the Bare Metal EKS-A provider. The chart that consumes the images will be updated separately.

/retest

These changes need to be synchronized with changes to the EKS-A repository so holding until those are in-place.

Looks like there arent any references to Boots in the files changed section so I believe only the initial PR comment should updated in terms of the Boots version. v0.8.0 -> v0.8.1

Looks like there arent any references to Boots in the files changed section so I believe only the initial PR comment should be updated in terms of the Boots version. v0.8.0 -> v0.8.1

Thanks, fixed a few errors in the PR overview.


CreateTearhseet: remove closeIconDescription prop

What will this achieve?
There is no close icon in the create flows, so this prop should not even be included for this component.

https://github.com/carbon-design-system/ibm-cloud-cognitive/pull/981


fix(curriculum): Combine Steps 103 and 104 in the "Learn Recursion by Building a Decimal to Binary Converter" Challenge

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #56587
Description
This pull request resolves the redundancy between steps 103 and 104 in the "Learn Recursion by Building a Decimal to Binary Converter" challenge. The changes ensure a streamlined learner experience while maintaining the educational intent of the challenge.
Changes Implemented

Combined step 104 into step 103:

Step 103 now requires learners to use getElementById to select the correct element and set its textContent property to the msg value of the current object.

Removed step 104, as its purpose is now fulfilled by the updated step 103.
Updated hints, tests, and descriptions for the combined step.
Adjusted step navigation to ensure step 103 transitions directly to step 105.

Hey @ariel0206qyx
This PR seems to make similar changes as the existing PR 56587. As such, we are going to close this as a duplicate.
If you feel you have additional changes to expand upon this PR, please feel free to push your commits and request this PR be reopened.
Thanks again! 😊

If you have any questions, feel free to ask questions on the "Contributors" category on our forum or the contributors chat room.


uv add and uv lock fail on corporate network

I'm running into some problems trying to use uv add and uv lock commands on a machine running behind a corporate proxy.
I have configured internal pypi mirrors in my ~/.config/uv/uv.toml file:
[pip]
index-url = "<pypi_mirrror_url>"

I then run through the following "hello world" example and get some errors along the way:
➜  uv init hello-uv
Initialized project `hello-uv` at `/home/develop/hello-uv`

➜  cd hello-uv

➜  uv add requests
Using Python 3.12.3 interpreter at<EMAIL_ADDRESS>Creating virtualenv at: .venv
⠼ requests==1.2.3                                                                                error: Failed to download and build `requests==1.2.3`
  Caused by: Failed to extract archive
  Caused by: Invalid gzip header

➜  cat pyproject.toml
[project]
name = "hello-uv"
version = "0.1.0"
description = "Add your description here"
readme = "README.md"
requires-python = ">=3.12"
dependencies = [
    "requests",
]

[build-system]
requires = ["hatchling"]
build-backend = "hatchling.build"

➜  uv lock
⠼ requests==1.2.3                                                                                error: Failed to download and build `requests==1.2.3`
  Caused by: Failed to extract archive
  Caused by: Invalid gzip header

➜  uv pip compile pyproject.toml
Resolved 5 packages in 1.66s
# This file was autogenerated by uv via the following command:
#    uv pip compile pyproject.toml
certifi==2024.7.4
    # via requests
charset-normalizer==3.3.2
    # via requests
idna==3.8
    # via requests
requests==2.32.3
    # via hello-uv (pyproject.toml)
urllib3==2.2.2
    # via requests

The interesting thing is that the uv pip compile pyproject.toml command succeeds, whereas the uv add and uv lock commands fail.
If I subsequently delete the ~/.config/uv/uv.toml file, then the uv pip compile pyproject.toml command fails with the same error:
➜  uv pip compile pyproject.toml
⠸ requests==1.2.3                                                                                error: Failed to download and build `requests==1.2.3`
  Caused by: Failed to extract archive
  Caused by: Invalid gzip header

This would seem to imply that the uv add and uv lock commands are not respecting the index-url configuration.

Can you try removing the [pip] header from your uv.toml?

Settings under [pip] only apply to uv pip; settings at the top-level apply to everything (uv sync, uv lock, and uv pip et al).

Brilliant, that worked! Thanks

@charliermarsh why do we fail with this weird error? because the proxy returns a weird response?

(@maxfirman if you can get more details about what the response from the proxy is that'd be helpful for improving the error message for other users, uv will say more about the requests with RUST_LOG=trace uv add --verbose ...

uv_add_failure.log
@zanieb log file attached

What kind of reponse is returned when you GET https://files.pythonhosted.org/packages/61/79/efc316760a906763de872d7328c9bf8c5af28708a35fdae57fbb4ee005f7/requests-1.2.3.tar.gz?

curl_pythonhosted.log
@zanieb see attached log. The request goes via our proxy and gets redirected to a our Artifactory which ultimately returns a 404. This looks likely to be a proxy misconfiguration on our end. It would definitely be better to have uv fail immediately and surface the 404 error.


Update to 0.8

Update to newly tagged v0.8.0 cosmwasm.

Tested manually and everything checks out, but it would be nice to have CI.
Plus, I will have to update this a few times with newer cosmwasm versions

Let's wait to merge until we have 0.8 final for cosmwasm

Jap. I wonder where out CI is gone … Let's see

Arg, Windows build error since blake3 v0.1.5 -> v0.3.4 (pulled in by the Wasmer 0.17 update)

We don't need to support Windows (only OSX and Linux can run the chain).
How about removing windows from the CI and then making an issue to fix this soon?

We don't need to support Windows (only OSX and Linux can run the chain).
How about removing windows from the CI and then making an issue to fix this soon?

We use AppVeyor in order to be able to differentiate between meta CI config and the resulting project's CI config. AppVeyor uses Windows by default. We could try to reactivate https://github.com/CosmWasm/cosmwasm-template/pull/17 in order to use AppVeyor+Ubuntu.

Oof, this is not a quick fix.
I'll run the tests locally and then merge?

And revisit the CI after merge

I tested this locally and it works. Feel free to merge @webmaster128


Adjust redirect location to not use landing URL

The auth redirects should ideally use the host of the current JupyterHub instance instead of the landing path of the current profile/environment (/devbooks/) . This will allow us to easily support multiple instances and avoid some confusion.

Resolved in 831462a


docs(developer): registration process enhancement by incl. status endpoint

Description
Updated registration details for status change
Why
Documentation enhancement
Issue
#255
Checklist
Please delete options that are not relevant.

[x] I have followed the contributing guidelines

reopen

@Phil91  I dont have the needed rights to solve the conflict. Can you please have a short look at it - maybe we can solve it easily.

@Phil91 I dont have the needed rights to solve the conflict. Can you please have a short look at it - maybe we can solve it easily.

I'll take over

@Phil91 I dont have the needed rights to solve the conflict. Can you please have a short look at it - maybe we can solve it easily.

@jjeroch conflict is resolved.


Inject caches internally for better testing

Realized that using these global caches inside tests is a pretty bad practice. Updated the internal API so that they can be injected.

Disabled --quiet and can see the failure:
Test Case '-[Tests.SnapTests test_complexBuilder]' started.
/Users/travis/build/GitHawkApp/StyledTextKit/Pods/Snap.swift/Snap/Core/Application/Actions/CompareImages.swift:52: error: -[Tests.SnapTests test_complexBuilder] : failed - ≠ Images are not equal
Test Case '-[Tests.SnapTests test_complexBuilder]' failed (0.104 seconds).


Friendship features

Friendships models:

GET /api/friendships/<user_id>/following
GET /api/friendships/<user_id>/follower
Modified Testcase class method for authenticated user APIClient

Modified router for API UI. Use /api instead of root


Improving the disparity filter interface

Several changes to minimize the need for explicit parameter tuning when using with either StereoBM or StereoSGBM and to make the filter more easy to use in general.

Looks good, thank you!
Please, squash all commits into one.

@mshabunin I replaced the code in the tutorial with a link to GitHub and squashed all the commits

:+1:

Is it possible that this change introduced this error when building:
no member named 'MODE_SGBM_3WAY' in 'cv::StereoSGBM' found in the file disparity_filtering.cpp from the samples folder?

@Arqu , please use latest master branch version of opencv repository when building latest version of opencv_contrib.
This mode has been added recently:
https://github.com/Itseez/opencv/blob/master/modules/calib3d/include/opencv2/calib3d.hpp#L1559

Thanks, I'll rebuild it when I get some spare time, I've managed to skip the module in the build process for now. I probably didn't use the latest source as I've used the 3.0.0 branch instead of master.


fix: promises should resolve correctly

Fixes use cases where code like this might never resolve:
scrollIntoView([HTMLElement], {
  scrollMode: 'if-needed',
  block: 'nearest'
}).then(() => { console.log('done') })

Fixes https://github.com/stipsan/scroll-into-view-if-needed/issues/344

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Initial libnetwork rest api

Defines and implement http handler for "/networks" URLs
Addresses part of requirements tracked by Issue #5
Addressed @mavenugo and @mrjana  comments

Signed-off-by: Alessandro Boch<EMAIL_ADDRESS>

@mavenugo, I took care of your comments.
Please take a look.

@aboch Thanks for addressing the comments. The code looks clean and concise.
props for solid test coverage. :clap:
Few missing endpoints, but they are not blockers. But can be addressed in another PR.
:+1: LGTM.

@mrjana I took care of your comments.
Please take a look.

LGTM


configmaps: Track hash of cluster provided CA certs (PROJQUAY-5174)

The config.openshift.io/inject-trusted-cabundle: 'true' annotation will inject content into the config map after Kustomize has generated its resources. This means that Kustomize will not be aware of changes made by Openshift to these config maps
In order to redeploy QuayRegistry resources when cluster CA are rotated, this PR implements a mechanism to track the changes through hashing of the CA as an annotation on the Quay resources

/cherrypick redhat-3.9


[MESOS-10199] Fix Host field in client headers

This PR replaces empty host with actual IP/Port of the message recipient in Host field. IP+Port has been used as per RFC guidelines.
An earlier PR made host field empty https://reviews.apache.org/r/27865

Ran make check in Linux. HttpServeTest.Discard test seems flaky, it fails with or without this change.
Also verified agent-master registration/communication behind nginx proxy.

Thanks for fixing this, a couple of things, maybe @vinodkone or @asekretenko can have a look after these updates:

Can you follow the existing logic for the http client?  See: https://github.com/apache/mesos/blob/1.11.0/3rdparty/libprocess/src/http.cpp#L1092-L1108
Can you remove the extra .gitignore file? It's not related to this change and we like to keep our commits tidy with only a single logical change involved
Can you add or update one of the existing tests in libprocess?

Can you follow the existing logic for the http client?  See: https://github.com/apache/mesos/blob/1.11.0/3rdparty/libprocess/src/http.cpp#L1092-L1108

I find the existing logic redundant in this case, as message.to.address is an Inet address not a URL, which already includes an IP and port. message.to.address essentially includes the following code block already:
 else if (request.url.ip.isSome()) {
    headers["Host"] = stringify(request.url.ip.get());
  }
// Add port for non-standard ports.
if (request.url.port.isSome() &&
      request.url.port != 80 &&
      request.url.port != 443) {
    headers["Host"] += ":" + stringify(request.url.port.get());
  }

Please let me know if you meant something else.

Can you remove the extra .gitignore file? It's not related to this change and we like to keep our commits tidy with only a single logical change involved

Done

Can you add or update one of the existing tests in libprocess?

Done
cc: @vinodkone @asekretenko

Is there any CI job which verifies build for my sanity? I haven't been successful in running all tests due to the flaky HttpServeTest.Discard

@zorro786  If you believe HTTPDiscard test is flaky you can run the test suite by filter out that test (e.g., make check GTEST_FILTER=-"HttpServeTest.Discard")

Thanks @vinodkone. I have run the tests on master branch, two tests fail:
HttpServeTest.Discard and HttpServeTest.Pipelining.
Same tests fail on this PR change too. Rest all pass.

LGTM. I merged it. Thanks for the fix @zorro786


Several packages produce 'missing a bundled dependency "@aws-cdk/util”' warnings on 'yarn add'

Reproduce for ec2:
mkdir foo
cd foo
yarn init # accept defaults
yarn add @aws-cdk/aws-ec2

Also repros for codepipeline-api and events.

Related to #84


ITAK

Are you thinking of making a version for an iTAK-Forwarder?
I'm trying to set up an entirely IOS dependant system and was pleased to see TAK made an IOS version.
If not is there a way to make this work for that application.

I don't have any plans for an iOS version at this time. I don't have a MacOS computer to do iOS development on. I'm also not sure if iTAK has plugins.

iTAK does not support plugins, and may never while it is part of the Apple App Store since that is disallowed.  If they removed the app from the store and deployed only through TestFlight then they could do it, but that is a step backward and probably won't.


dpsctl.py Python3 compatibility

I ran 2to3 on the dpsctl code.
Code is now compatible to python2 and python3.
I added some documentation in README.md.
Fix the build of emu/ and updated documentation

Currently I only tested the code with the emulator (haven't received the hardware yet).
I would like to do some more work on the control script:

Create proper python module, to easily script the power supply with python
Write more documentation and examples

Thanks! I will take it for a spin on HW next year, ie. later this week ;)

Creating a proper python module would be great for writing custom test scripts and dataloggers and the such


loose Jinja2 dependencies in Superset extra

resolves #6388
Checklist

[x] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[x] Links to related issues (if applicable)
[x] Tests for the changes have been added/updated (if applicable)
[x] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[x] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

@KulykDmytro - could you open this PR to the main https://github.com/datahub-project/datahub repo? This repo is just a fork of that one for our release process


Proposing @jonathanawesome as a maintainer

Living up to his username, @jonathanawesome has done an awesome job helping @thomasheyenbrock, and I would like him to be able to triage tickets and approve @thomasheyenbrock's PRs for the 2.0 rewrite branch, and if he would like, the ability to approve PRs to main.
Passionate about graphiql user experience, he has built fantastic prototypes, and like Thomas and others, has spent countless hours in the issue queues, chat histories and beyond to gain a deeper understanding of what a user-driven roadmap looks like for graphiql.
What's more, @thomasheyenbrock tells me he would be delighted to have him as a co-maintainer.

Thanks for all your hard work everyone!

Thanks! 🚀


Reduce the command injection attack space

Not really a major deal but I just feel it's best not to trust user input ever, even if it's not that much a big deal in a container and that the exploit will only be executable by the person starting the container.
All variable expansions provided by environment variables that can be set by the user have been wrapped in double quotes ensuring that no matter of the contents it will be treated as 1 argument where used (no less, no more).
Also makes the commands a little more resilient to failure if empty values sneak past the -z/-n checks. Where [ "${VAR}" ] has been used (without -z or -n) they will get past but I am fixing this in another code cleanup PR.

Merged


Support resuming user_medias fetch with stored cursor

Add a user_medias_paginated method (along with _gql and _v1 versions)
which take an optional end_cursor parameter and return the next
end_cursor along with the medias for that page.  This enables fetching of
media from a stored cursor.  The user_medias, user_medias_gql, and
user_medias_v1 are also adapted to use these paginated versions to do
the fetching one page at a time.

I will help you with the tests, please update the documentation


Hotfix: Changes overlay click behavior for iOS/BB

iOS was not dismissing the menu on click of the mega menu semi-opaque overlay.
BB was dismissing it too soon.
Changes

Adds cursor: pointer to the styles for the overlay so that iOS bubbles the click event up to the body, which dismisses the menu in the mega menu script.
Changes the event type to click instead of mousedown, so that a mouseup event is required to the dismiss the menu (hopefully this fixes the issue on BB).

Testing

gulp build
Load in iOS simulator (try iPhone 5)
Click hamburger menu, scroll down till the menu isn't visible, and click the overlay. Menu should close

Review

@jimmynotjim
@sebworks
@KimberlyMunoz

Works locally :+1:


Add unit tests for Crypto Internals

Added unit tests around BitMath.

sorry @bitcoinbrisbane for make you waiting, just saw you PR.
I see that BitMath is only used for IsPositivePowerOf2, is it not a better idea to remove the other methods instead of testing them ?

Its ok,
I’ve just run code analysis and look for classes with low code coverage.  As the method where public, I just thought to add unit tests.
Either remove or make them private?

On 2 Sep 2015, at 9:33 am, Nicolas Dorier<EMAIL_ADDRESS>wrote:
sorry @bitcoinbrisbane https://github.com/bitcoinbrisbane for make you waiting, just saw you PR.
I see that BitMath is only used for IsPositivePowerOf2, is it not a better idea to remove the other methods instead of testing them ?
—
Reply to this email directly or view it on GitHub https://github.com/MetacoSA/NBitcoin/pull/121#issuecomment-136891866.

Yes, let's remove the code if we can and keep the test for the one we use.

@bitcoinbrisbane BitMath is certainly better as internal and without the method we don't use, can you update this PR or I do it ?

Hi Nicolas,
I wont be able to until tomorrow night.

From: Nicolas Dorier<EMAIL_ADDRESS>Sent: Monday, 14 September 2015 2:27 AM
To: MetacoSA/NBitcoin
Cc: Lucas Cullen
Subject: Re: [NBitcoin] Add unit tests for Crypto Internals (#121)
@bitcoinbrisbanehttps://github.com/bitcoinbrisbane BitMath is certainly better as internal and without the method we don't use, can you update this PR or I do it ?

Reply to this email directly or view it on GitHubhttps://github.com/MetacoSA/NBitcoin/pull/121#issuecomment-139892605.

Sorry for the late response. I'm happy to change the access modifier to internal.

I'll accept the PR as now, however, I'll appreciate if you can remove unused code from NBitcoin like the methods in BitMath when you have time.

Should we deprecate them first?
Sent from my iPhone
On 24 Sep 2015, at 2:35 am, Nicolas Dorier <notifications<EMAIL_ADDRESS>wrote:
I'll accept the PR as now, however, I'll appreciate if you can remove unused code from NBitcoin like the methods in BitMath when you have time.

Reply to this email directly or view it on GitHubhttps://github.com/MetacoSA/NBitcoin/pull/121#issuecomment-142656172.

No, these are most likely not used, we can remove.


Handle payjoin errors according to BIP 78

"The receiver is allowed to return implementation specific errors which
may assist the sender to diagnose any issue.
However, it is important that error codes that are not well-known and
that the message do not appear on the sender's software user interface.
Such error codes or messages could be used maliciously to phish a non-
technical user. Instead those errors or messages can only appear in
debug logs.
It is advised to hard code the description of the well known error
codes into the sender's software."
See:
https://github.com/bitcoin/bips/blob/master/bip-0078.mediawiki#receivers-well-known-errors
This commit displays templates based on a particular ResponseError
and debugs the rest, according to this specification.
result:

needs rebase

I'm not sure why these platform-specific errors are happening in CI after rebase since they don't seem to relate to my changes. @benthecarman any hints?

Maybe just try rebasing? seems like a spurious failure

yep it was spurious


Optimize mybatis plugin by using method interceptor API v2.

Please answer these questions before submitting your issue.

Why do you submit this issue?
[ ] Question or discussion
[ ] Bug
[ ] Requirement
[x] Feature or performance improvement

Requirement or improvement
When #6915 is completed, I will optimize #6913.

Current issue is completed in #6937.


How to hide chat widget on page leave.

Is your feature request related to a problem? Please describe.
How can I hide / Destroy chat widget on page leave. I dont want to show chat widget on all pages. i have a support page and want to show chat only on support page.
Describe the solution you'd like
there should be an option to show/hide on widget.
Describe alternatives you've considered
i hide it via css.
Additional context
I am using angular with ionic framework. and i want to hide it using *ngIf or is there an directive available ?

Hello @Saqib92 👋
If you would like to hide the Chat Widget on particular pages you can use visibility="hidden" component property. Later if you would like to show it again, change the value to visibility="minimized" for example.
You can read about all available component properties here: https://github.com/livechat/chat-widget-adapters/tree/master/packages/widget-angular#config-data
In case you would like to destroy the widget completely, just use *ngIf on the <livechat-widget> element.

Setting it to false value would cause the ngOnDestroy to be called which destroys current widget instance.
Setting it to tru value would cause the ngOnInit to be called which initialize new widget instance.

dynamic visibility="hidden" is not working.
I tried Ionic Life cycle hook:
page.ts
myVisibilityVariable: string = 'minimized'; // by default;

ionViewDidEnter(){ // WillEnter workd when page loads.
    this.myVisibilityVariable = 'minimized';
}

ionViewDidLeave(){. // didLeave works when page changed.
    this.myVisibilityVariable = 'hidden'; // not working
}

page.html

[visibility]="myVisibilityVariable" // not working

visibility="{{myVisibilityVariable}}" // not working

Hi @Saqib92, I have tried it the same way, it is maximizing/minimizing/hiding the widget fine for me now.
I have added livechat-widget in app.component.html, so it is working as the model defined in app.component.ts.
But I am looking to find the solution to hide/maximize/minimize the widget from other component like Login.component/AccountDetails.component.
Do you have any solution for the same, much like we do in js as below, I need to implement in angular, please help me out.
LiveChatWidget.get(‘chat_data’)
LiveChatWidget.call(‘set_customer_name’, ‘Test’)
LiveChatWidget.call(‘set_customer_email’<EMAIL_ADDRESS>LiveChatWidget.call(‘maximize’)
LiveChatWidget.call(‘minimize’)
LiveChatWidget.call(‘hide’)
Please note that, LiveChatWidget is globally accessible in js, I should be able to do these where ever is required across the application in Angular.
@walaszczykm  please help me to proceed further.

Hi @Saqib92, I have tried it the same way, it is maximizing/minimizing/hiding the widget fine for me now. I have added livechat-widget in app.component.html, so it is working as the model defined in app.component.ts.
But I am looking to find the solution to hide/maximize/minimize the widget from other component like Login.component/AccountDetails.component.
Do you have any solution for the same, much like we do in js as below, I need to implement in angular, please help me out.
LiveChatWidget.get(‘chat_data’) LiveChatWidget.call(‘set_customer_name’, ‘Test’) LiveChatWidget.call(‘set_customer_email’<EMAIL_ADDRESS>LiveChatWidget.call(‘maximize’) LiveChatWidget.call(‘minimize’) LiveChatWidget.call(‘hide’)
Please note that, LiveChatWidget is globally accessible in js, I should be able to do these where ever is required across the application in Angular.
@walaszczykm please help me to proceed further.

you can do something like this: https://github.com/livechat/chat-widget-adapters/issues/40#issuecomment-1141771514
with the help you SubjectBehaviour Service.

@Saqib92 Thanks for the prompt reply, I have tried the BehaviourService too in other components.
I tried updating the properties using WidgetState, but the property values are not updating at the widget side.
I am not sure, may be these WidgetStateSubject, WidgetStateService are meant for read-only.

@walaszczykm  please help me to proceed further.


Scenario Caching

Closes #1595.
Summary of Changes
This PR makes a number of substantial changes to scenario builds:

Scenario build artifacts are now cached and built incrementally, meaning that subsequent builds (without the clean option) will only build the artifacts that depend on the changed DSL objects
If the map is a local file in the scenario directory (e.g. a map.net.xml or .xodr file), changes to the map file itself will also trigger a rebuild of artifacts that depend on the map
All build artifacts are now in a local build/ directory in each scenario's directory
A build.db SQLite database file is now created during builds to keep track of dependencies
The allow_offset_map option has been removed. This must now be set by setting shift_to_origin=True in a MapSpec object in the scenario.py file if this option is needed.
All scenarios must have a scenario.py, and must call gen_scenario(), rather than the individual gen_ functions. Those functions have been made private, and a deprecation warning will be raised if called from outside of gen_scenario().
Empty meshes are now skipped when generating the map GLB file to get rid of noisy trimesh warnings
Map GLB file is now only built once (unless the map changes)

Example
Here is the output from a terminal session that demonstrates what a typical iteration workflow looks like now:
# First, clean and build the scenario
$ time scl scenario build --clean scenarios/sumo/minicity/
build-scenario scenarios/sumo/minicity/
INFO:/home/saul/code/SMARTS/smarts/sstudio/genscenario.py:"map_glb" took: 32551.769018ms
INFO:/home/saul/code/SMARTS/smarts/sstudio/genscenario.py:"traffic" took: 8146.712780ms
INFO:/home/saul/code/SMARTS/smarts/sstudio/genscenario.py:"social_agent_missions" took: 5.387068ms
INFO:/home/saul/code/SMARTS/smarts/sstudio/genscenario.py:"bubbles" took: 3.000498ms

real	0m42.086s
user	0m42.094s
sys	0m2.561s

# Then, make a change to the traffic specification in scenario.py, and rebuild (without --clean)
$ time scl scenario build scenarios/sumo/minicity/
build-scenario scenarios/sumo/minicity/
INFO:/home/saul/code/SMARTS/smarts/sstudio/genscenario.py:"traffic" took: 7710.859299ms

real	0m9.068s
user	0m9.289s
sys	0m2.256s

Hi Saul, just minor, have you tested the this in the replay scenarios (e.g., NGSIM)? I went though your changes/descriptions and the current scenario.py files in the NGSIM folder, looks ok but just wanted to double check.
Also regarding the following point:
"The allow_offset_map option has been removed. This must now be set by setting shift_to_origin=True in a MapSpec object in the scenario.py file if this option is needed."
Should we check all the replay scenarios to see if we should add the shift_to_origin=True if we havent done yet? I can check them once this PR get merged.
Thanks!

@Gamenot  I need to do a second review specifically on the build changes.

Hi @qianyi-sun, getting to your comments just now.

Hi Saul, just minor, have you tested the this in the replay scenarios (e.g., NGSIM)? I went though your changes/descriptions and the current scenario.py files in the NGSIM folder, looks ok but just wanted to double check.

They all build correctly using the new system. I haven't run any examples with them, but there shouldn't be any change.

Also regarding the following point:
"The allow_offset_map option has been removed. This must now be set by setting shift_to_origin=True in a MapSpec object in the scenario.py file if this option is needed."
Should we check all the replay scenarios to see if we should add the shift_to_origin=True if we havent done yet? I can check them once this PR get merged.

Yes that's a good idea. There may be some scenarios that require that change that aren't covered by the tests. If you have some time to check them that would be helpful, thanks!

Should we put in a CI test to specifically test the scenario build caching feature?

Should we put in a CI test to specifically test the scenario build caching feature?

Good idea. I added a test in cli/tests/test_studio.py.


ReactiveDropdownField should have borderRadius field in constructor

Hi there,
In the ReactiveDropdownField widget, we can actually edit the source code to add a BorderRadius to the drop down list, like so:
DropdownButton<T>(
                  value: effectiveValue,
                  items: items,
                  borderRadius: BorderRadius.circular(10),

This gives a BorderRadius to the drop down list. However, the borderRadius field is not available to pass in to a ReactiveDropdownField and it should be, to add more customization. I also wonder is there a way to align the drop down list to the bottom of the input field?
Thank you!! :)

Hi @BenjaminFarquhar,
Thanks for the issue. I will take a look and include the missing properties to the ReactiveDropdownField

Hi again @BenjaminFarquhar,
COuld you please give more details? Where did you see the property borderRadius? Because in the definition of the class DropdownButton from Flutter, this property does not exist.

Hi @BenjaminFarquhar I did the PR #236 but there is no clear understanding how long will it take @joanpablo to release...
So I released a separate version of the field into standalone package.
https://pub.dev/packages/reactive_dropdown_field

Also try to use this package
It simplifies reactive_forms development
https://github.com/artflutter/reactive_forms_generator

@BenjaminFarquhar I do not recommend you to wait until release of the whole form engine.
The issue is that your props are available only form 2.5.* version of the Flutter.
This means that the whole reactive_forms should be bumped to use 2.5.* and this is a little bit complicated and will require new major release.
I'm not sure that this is a good idea...
Anyway. You have a separate package. You now what to do :-)

@vasilich6107 Thanks!

Also try to use this package It simplifies reactive_forms development https://github.com/artflutter/reactive_forms_generator

This seems very nifty, thanks. Had a little play and the form.model wasn't updating when I typed. I must have missed something. I will have a proper attempt again later. My text fields are custom and extend ReactiveFormField<String, String>, and they also usually use formControlName but I added formControl to them so not sure what the issue was but I'm fairly sure it will be with my code not the package.

@BenjaminFarquhar if you could provide a standalone git repo with reproduction sample I will try to help you

@BenjaminFarquhar it seems that your issue is fixed
Could you close it?

@vasilich6107 Thanks so much! I will get in touch if I have any issues when I get around to trying the generator again. Yes this is fixed now :)


Revisit update_tuple_item

What is wrong?
Per https://github.com/ethereum/py-ssz/pull/84#discussion_r312866552 and https://github.com/ethereum/py-ssz/pull/84/files/48ac9650557ca9591f6403c6146a237bea2b9396#r313158837 discussion.
https://github.com/ethereum/trinity/blob/2644c231f0d7c98128565ba4e5c8cf02e3145f35/eth2/_utils/tuple.py#L49
How can it be fixed

@ralexstokes:
the idea was to ignore any and all positional inputs to this function (and in this case just return some value to do an unconditional swap). i ran into a case where it was helpful to ignore all positional args but it would be worthwhile to revisit our usage of this idiom to see if we still need it.

a more complete solution is something like lambda *_, **__: new_value but we are taking a shortcut in that we don't pass any keyword args here (and expect no one else will either)

I don't think the *_ is necessary at runtime, because we don't pass any *args to update_tuple_item_with_fn, so the function isn't called with any additional parameters. But since we have a type hint that the function has to fulfill (in Trinity at least, not in py-ssz), we should keep it as it is as far as I can see.
On that note, I think the type hint is wrong: Callable[[VType, Any], VType] means that the function takes two arguments, not an arbitrary number. Unfortunately, to my knowledge the only way to fix this would be to use Callable[..., VType], so we'd lose the annotation for the first argument... But since mypy seems to not complain about this right now (which surprises me a little), I think we should just keep it as it is. So I'm closing this, but feel free to reopen if you disagree.

these are the only cleanups i see at the moment: https://github.com/ethereum/trinity/compare/master...ralexstokes:fix-use-of-tuple-varargs


Cloning doesn't overwrite identity

Fixes #8486

This is the same change that was reverted?

Yes, this is the change that I had mistakenly merged in 1.10.x


Generated picture size inaccurate

The generated image for width 720 actually has a width of 721 pixels. When the browser renders it at 720 pixels it looks blurry (it contains text).
Can you explain why this happens?
Is it a bad idea to provide image widths exactly as they will be rendered in the browser? Should I better provide images a little bit bigger (e.g. 800 instead of 720) so they never look blurry when rendered in smaller resolutions?

Huh...  I don't know why imagemagick would add a pixel. Will have to research this more. Nice catch.
As far as providing images exactly as they will be rendered, it's fine but make sure that you're aware of device pixel ratios. Basically different devices might have multiple physical pixels per CSS pixel, depending on the screen. They range all the way from 1 to ~3.5.

Probably ImageMagick calculates the image height differently than the implementation in generated_image.rb. The solution would be to just pass the width to ImageMagick and let it work out the height by itself. This solution works with the convert tool, not sure if the Ruby binding supports it though.
PS: Do you know of any statistics for "true" mobile screen resolutions? All statistics I could find e.g. statcounter.com don't seem to be considering device pixel ratios.

Yep, that works. I opened a pull request #126.


Command line options not formatted properly in gui

Here is a link showing what the command line options menu in the OSX client looks like. There is obviously some problem with tabs....
https://img.bi/#/xornNBZ!u0tLcFpJF5gUg9NpjvEf3UIvfzzoCuG8JQg7xjpi
This also shows that some lines are not being translated, even though transifex shows 100%. These lines don't appear to be in the files uploaded to transifex at all...

As of processing the latest pull request, builds of v<IP_ADDRESS> now have a changed look in both the Command Line options and About pages.
The changes I just added, where ones done over a month ago, that Cryptoslave had wanted.  Both he and I noticed that they had been lost, so I found them and added them back into the source code.  The 2 dialogs now have different images referenced, and some other small form & source code modifications were also made.  Think the font selection code may have also been changed to terminal for the Command line window display options.  I would like to now decrease the size of the font there, as well as edit a few small things on the About page.
New round of input here from your test results will be appreciated and welcome.
Plans are to start working on upgrading the command list this next week, without question we have some new ones, that need to be added.  Getting that list together and documenting one line strings that are useful to the user for that help is the first step.  Coding it to a standard for which language translations can be added is also the intention.

About Anoncoin: This looks fine. However, for the french translation, the text gets cut off on the right hand side of the window. Is it possible to increase the window width?
https://img.bi/#/4SWYgNQ!wymVR8EYXLtecp5WHXsz2E7DSqOD8VHoZQ5EbcWL
Command line window: There is no menu item for the command line window when using the french translations. However, when clicking on the "preferences" menu, the command line window opens erroneously. It is not possible to open the normal "preferences" pane in the french translation, making it impossible to modify the i2p, and coin control features.
https://img.bi/#/iVn0C2e!2nVx0g2T3Qz3H4bZCyQGPGU1CfpRXZvIpdyRcrcx

All this display correctly in anoncoin 9.4.4 windows French, also the command line windows has got slider which seems to lack on mac version in above picture.

I can increase the size of the "About Anoncoin" window manually, but this is not ideal.
Can you access the "preferences" in french on windows?


SCSS not compiling with "sourceMap": false

Hey there,
I don't need the source map file, so I naturally set sourceMap to false in my .brackets.json file. But as soon as I do so, my scss files won't be properly compiled anymore. Instead Brackets just gets stuck while compiling and doesn't output any css anymore.
There are some errors in the console:

Here's my .brackets.json file:
{
  "sass.enabled": false,
  "path": {
    "sass/style.scss": {
      "sass.enabled": true,
      "sass.options": {
        "outputDir": "..",
        "outputStyle": "nested",
        "sourceMap": false
      }
    }
  }
}

When I include the source map, everything compiles fine.

Nevermind, of course I have to put
"sourceMap": "false"

Accidentally closed a bit early. Problem is still there.

Can you test this branch?
But keep in mind that, without source maps, live preview won't be updated until you don't save the file and files won't be compiled when partials change.

@nicolo-ribaudo maybe this info should be added to the docs?

I think there should be a "database" in which save info about file relations, so that live preview and compilation-on-partials-changes don't depend on source maps.
Something like this:
[{
  "C:/projects/mySite/main.css": [
     "C:/projects/mySite/main.scss",
    "C:/projects/mySite/_config.scss"
   ]
}]

Sorry but how can I test the branch you referenced, @nicolo-ribaudo? I'm new to Brackets Extensions and don't know how to compile the source files accordingly.


Serious PEP8 review

Also, implement a make directive for automated flake8 checks with 100 max line length.

https://github.com/arthurpaulino/miraiml/commit/0793a999141f5b44deec01f0d98f0fc4e2a6dfe1


cmd/cgo: cgo-related compile errors yield poor error messages

What version of Go are you using (go version)?

go version go1.11.2 linux/amd64

Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
go env Output
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/martin/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/martin/go"
GOPROXY=""
GORACE=""
GOROOT="/usr/local/go"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build515058236=/tmp/go-build -gno-record-gcc-switches"

What did you do?
Natively compile a modularized Go project with CGO_ENABLED=1 on Linux.
What did you expect to see?
A descriptive error informing me which import-statement in which .go-file caused the problem.
What did you see instead?
The following mostly C-related error output, which requires me to debug the issue by selectively removing module dependencies and such until the problem is resolved to get closer to identifying the offending import/include:
# runtime/cgo
In file included from _cgo_export.c:3:0:
/usr/include/stdlib.h:25:10: fatal error: bits/libc-header-start.h: No such file or directory
 #include <bits/libc-header-start.h>
          ^~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.

While runtime/cgo is mentioned, this is certainly not being imported by me directly. Also great would be to report common missing dependencies in case of known failures. (I.e. answer the question "What am I missing which contains the missing file?")

It's a hard problem.  The code that failed to compile is in fact in the standard library's runtime/cgo package.  That package does a #include <stdlib.h>.  It appears that on your system that does not work.  So there seems to be something deeply broken in your C compiler installation.  Since we didn't write or install the C compiler, it's very hard for us to give you a useful error report, or to answer a question like what you are missing.
The first result on a Google search for the C compiler error message suggested
sudo apt-get install gcc-multilib g++-multilib

so I would try that.

I don't really see what we can do better here, so I'm going to close this issue.  Please comment if you have a suggestion.


Enhance session view

Is your feature request related to a problem? Please describe.
My request is to improve my productivity and experience monitoring running queries.
When I am running a big query, I do not know how complete it is or where in the query plan is currently executing/processing at.  Many times, I have terminated my long running query to only discover that is it "almost" done running resulting in lost work.
Describe the solution you'd like
I would like a tool that can display which segment of the query  execution plan is being acted on like in Oracle's SQL Developer tool (Real time SQL Monitor).  This tool displays all the running sessions and when it is clicked on it display the execution plan with a spinning icon to indicate with step is currently executing.  See attached image:

Describe alternatives you've considered
Oracle's approach is good and we should adopt it.
Additional context
N/A

Thank you for the idea.


Severity tag missing

I'm not seeing the 'severity' (e.g. DEBUG, INFO, ERROR, etc) tag being included in logspout-logstash messages.  Is there a configuration option I'm missing or is this not supported yet?

I’m not to sure about this as I’m not currently using the project myself. Hopefully someone else that is can answer the question.


Create a thin docker-compose for air-gapped systems

Pull requests:

https://github.com/Senzing/docker-compose-demo/pull/183
https://github.com/Senzing/docker-compose-demo/pull/184
https://github.com/Senzing/docker-compose-demo/pull/185


Salve amulets don't work at revenants

title says it all

This works properly. Be sure its equipped in your wildy setup and salve amulet only works if training melee, you need the imbued version for mage or range.


Explore passing workers into P2P merges

see #557
We should be able to do this more elegantly as soon as we materialise the graph on the scheduler to avoid dropping the results.

So this is blocked by https://github.com/dask-contrib/dask-expr/issues/14


github action failing for existing code base

pre-commit run -a
shed.....................................................................Passed
flake8...................................................................Failed
- hook id: flake8
- exit code: 1

cubedash/summary/_stores.py:1570:6: B019 Use of `functools.lru_cache` or `functools.cache` on methods can lead to memory leaks. The cache may retain instance references, preventing garbage collection.
cubedash/summary/_summarise.py:232:6: B019 Use of `functools.lru_cache` or `functools.cache` on methods can lead to memory leaks. The cache may retain instance references, preventing garbage collection.

flake8-markdown..........................................................Passed
curlylint................................................................Failed
- hook id: curlylint
- exit code: 1

Traceback (most recent call last):
  File "/root/.cache/pre-commit/repo00x8vp8z/py_env-python3/bin/curlylint", line 8, in <module>
    sys.exit(patched_main())
  File "/root/.cache/pre-commit/repo00x8vp8z/py_env-python3/lib/python3.8/site-packages/curlylint/cli.py", line 301, in patched_main
    patch_click()
  File "/root/.cache/pre-commit/repo00x8vp8z/py_env-python3/lib/python3.8/site-packages/curlylint/cli.py", line 291, in patch_click
    from click import _unicodefun  # type: ignore
ImportError: cannot import name '_unicodefun' from 'click' (/root/.cache/pre-commit/repo00x8vp8z/py_env-python3/lib/python3.8/site-packages/click/__init__.py)
Traceback (most recent call last):
  File "/root/.cache/pre-commit/repo00x8vp8z/py_env-python3/bin/curlylint", line 8, in <module>
    sys.exit(patched_main())
  File "/root/.cache/pre-commit/repo00x8vp8z/py_env-python3/lib/python3.8/site-packages/curlylint/cli.py", line 301, in patched_main
    patch_click()
  File "/root/.cache/pre-commit/repo00x8vp8z/py_env-python3/lib/python3.8/site-packages/curlylint/cli.py", line 291, in patch_click
    from click import _unicodefun  # type: ignore
ImportError: cannot import name '_unicodefun' from 'click' (/root/.cache/pre-commit/repo00x8vp8z/py_env-python3/lib/python3.8/site-packages/click/__init__.py)
Traceback (most recent call last):
  File "/root/.cache/pre-commit/repo00x8vp8z/py_env-python3/bin/curlylint", line 8, in <module>
    sys.exit(patched_main())
  File "/root/.cache/pre-commit/repo00x8vp8z/py_env-python3/lib/python3.8/site-packages/curlylint/cli.py", line 301, in patched_main
    patch_click()
  File "/root/.cache/pre-commit/repo00x8vp8z/py_env-python3/lib/python3.8/site-packages/curlylint/cli.py", line 291, in patch_click
    from click import _unicodefun  # type: ignore
ImportError: cannot import name '_unicodefun' from 'click' (/root/.cache/pre-commit/repo00x8vp8z/py_env-python3/lib/python3.8/site-packages/click/__init__.py)
Traceback (most recent call last):
  File "/root/.cache/pre-commit/repo00x8vp8z/py_env-python3/bin/curlylint", line 8, in <module>
    sys.exit(patched_main())
  File "/root/.cache/pre-commit/repo00x8vp8z/py_env-python3/lib/python3.8/site-packages/curlylint/cli.py", line 301, in patched_main
    patch_click()
  File "/root/.cache/pre-commit/repo00x8vp8z/py_env-python3/lib/python3.8/site-packages/curlylint/cli.py", line 291, in patch_click
    from click import _unicodefun  # type: ignore
ImportError: cannot import name '_unicodefun' from 'click' (/root/.cache/pre-commit/repo00x8vp8z/py_env-python3/lib/python3.8/site-packages/click/__init__.py)

yamllint.................................................................Passed
check for added large files..............................................Passed

Fixed in the PR, closing.


Citadel issue cert without SAN field for VM in mesh expansion

When the node agent on the VM has been running for a while, it is starting to output errors to the log file. Turns out that the node_agent will occasionally write out a certificate with a missing SAN field! (This must be coming from the Citadel server.) And once that broken certificate is written out, node_agent is unable to get certificate updates

Possibly same issue as #11078 .

Yes, it's a dup of #11078, closing this one.

I will work on this issue. Closing this one and use #11078 to track.


1424 edit patient detials

IMPORTANT: Where possible all PRs must be linked to a Github issue
Fixes #1424
Checklist

[x] I have written Unit tests for any new feature(s) and edge cases for bug fixes
[ ] I have added any strings visible on UI components to the strings.xml file
[ ] I have updated the  CHANGELOG.md file for any notable changes to the codebase
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the project's style guide
[ ] I have built and run the fhircore app to verify my change fixes the issue and/or does not break the app

closing this, same ticket update can be track now on this https://github.com/opensrp/fhircore/pull/1713


From stock invalidation of include pathes.

Cannot compile it because include pathes are different and they are confusing somehow, like Zydis is Zycore || Zycore is Zydis so how about to improve some readme about adding coldhook in to projects ?

Attempt to redefine all of the include pathes will resuld in LNK2001 error, unresolved external symbols.


[docs] pnpm nx apidoc docs clean repo check issues

Currently pnpm nx apidoc docs gives this error partway through, even if run from a clean repo:
[err]
git repository not clean prior to regenerating docs, bailing.

The state of the repo at this point will have a modified docs/content/api.json file and a bunch of deleted docs/content/api/ files.
This can be avoided with DIRTY_OK=true pnpm nx apidoc docs. The check is also turned off in CI.
None of this is necessarily bad, but we should give a couple minutes thought to whether there's a smoother way to do this.

(From discord): We can remove this check, it was meant for an earlier version of the system where apidoc didn't run typedoc.


Release 1.1.1

This release targets Swift 2.0 / Xcode 7.0.
Bug fixes

Fix DecodeError.MissingKeyPath handling thrown from decode() (#53)

:ship:


Fix tests

They use a Db which is null...

Would be nice to upgrade to using angel_test
... Which doesn't exist yet


Loading coop_box_pushing game as turn-based flattens observation tensor

Basically, the bug (or the thing I don't understand) is this discrepancy:
pyspiel.load_game("coop_box_pushing", dict(fully_observable=True)).observation_tensor_shape()
# returns [11, 8, 8]
pyspiel.load_game_as_turn_based("coop_box_pushing", dict(fully_observable=True)).observation_tensor_shape()
# returns [708]

Hmm.... ah yes, see here: https://github.com/deepmind/open_spiel/blob/efa004d8c5f5088224e49fdc198c5d74b6b600d0/open_spiel/game_transforms/turn_based_simultaneous_game.h#L107
Wow, we wrote that a long time ago, I haven't seen it in a while. I think the reasoning behind that choice went something like: when you represent a simultaneous game as turn-based, you need to encode the intermediate steps as observations/information states to each player, and the player who is observing it should know who they are. So you add this information to the tensor (and there's no easy way to do it unless you flatten it). This avoids collisions in the observations across players, which could be important if you're using one table to represent a joint policy (as e.g. CFR does) or one neural network for both players.
In retrospect, there are times where this does not make sense-- like (presumably) your setting-- and we should make it optional (and maybe not even the default, at least not for the tensor functions which are normally used in RL).  And we can implement the old behavior easily via a flag sent to the game transform itself.
Do you feel up to implementing it? If not, I'd be happy to do it, but it may take time because I'm quite busy at the moment.

Ah, good to know it is intended behavior, not a bug.
I understand the addition of the player onehot encoding.
However, I think just flattening everything with no good way recover the information is not optimal.
In the case of 2D environments (3D tensors), the spatial relationship between the different grid items usually can be exploited, by e.g. convolutional networks. To use one-hot encodings with convolutional networks, I have seen them implemented as additional image planes filled with either zeros or ones. In other words, the channel dimension of the image tensor is extended.
Using multiple observation tensors (in this case the 2D grid as 3D tensor, and a 1D onehot) would be another way of dealing with this, but I think that requires careful work to ensure API compatibility.
This way the downstream implementation can decide how to handle it.


"Basic Javascript: Local scope and functions" tests throws a regex error in firefox

Describe your problem and how to reproduce it:
Using Firefox and Firefox Developer edition and doing the challenge "Basic JavaScript: Local Scope and Functions", when completing the challenge the console throws the error:
// running tests You should add a local myVar variable. // tests completed // console output SyntaxError: invalid regular expression flag s
When the local var is well defined, tested in Chrome and worked
Add a Link to the page with the problem:
https://www.freecodecamp.org/learn/javascript-algorithms-and-data-structures/basic-javascript/local-scope-and-functions
Tell us about your browser and operating system:

Browser Name: Not working in Firefox and Firefox Developer edition (worked in Chrome)
Browser Version: Firefox developer edition v77.0b3(64-bit)
Operating System: Ubuntu 18.04.4 LTS

If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box):

Thank you, for bringing this up. We are aware of the issue. This is expected to be fixed by #38762.


PSD Bulk export of ANIM group

Changes

PSD bulk export has small modification when exports ANIM group in BG Layout extractor

ANIM group is not exported as one output but similar to export to compositing where it's subgroups are exported as separated outputs and are stored the same way

:black_flag:
Pype 2.x PR

pype
https://github.com/pypeclub/pype/pull/965

Issue was in plugin CollectContext where were modified variables and in extension check where ".psd" was checked instead of "psd".


Upgrade node to latest LTS

Node 12 has been out of support for some time and there are some security concerns in my org for using a tool on an old runtime that isn't seeing security patches.
https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/

Hopefully this is as simple as updating the runtime in the action file, but I created this for discussion in case there are known issues around doing so.

👋 Thanks for the heads-up, it should be fairly simple, I'll take a look later today!


add comment for webpack async chunks

@thejameskyle also I faced a problem that babel plugin doesn't process require.ensure, so I had to add that functionality in my fork and use it in my project. do you want me send pr or you think this functionality is redundant?

See https://github.com/thejameskyle/react-loadable/pull/67


Nullify library symbols

Maybe, implementing https://github.com/raviqqe/til/tree/main/dag-encoder and uniquifying strings might be easier.


Integrated dropdown to utilization card

Pull Request Test Coverage Report for Build 1614

32 of 33 (96.97%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.05%) to 84.055%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/components/StorageOverview/Utilization/Utilization.js
29
30
96.67%

Totals

Change from base Build 1600:
0.05%

Covered Lines:
3980

Relevant Lines:
4538

💛  - Coveralls

@rawagner Please review.

web-ui-component PR - https://github.com/kubevirt/web-ui/pull/383

@rawagner Incorporated all the review comments. Please review.


Previous version default channel order was BGR but v3.0 RGB?

New MMEngine data preprocessor supports bgr_to_rgb
It seems mmdet has changed default channel order to RGB:
https://github.com/open-mmlab/mmdetection/blob/main/configs/_base_/models/fast-rcnn_r50_fpn.py#L8
Isn't this a problem with previous weights? Have all weights for these models been recomputed or I am misinterpreting something?

@mmeendez8 It is the same as the original, please check carefully


[r2dbc] Support ClickHouseDialect

ClickHouse provides an r2dbc driver. Building a project with Testcontainers fails because of the following exception:
Caused by: Caused by: org.springframework.data.r2dbc.dialect.DialectResolver$NoDialectException: Cannot determine a dialect for ClickHouse using ConnectionPool[ClickHouse]; Please provide a Dialect

Add ClickHouse to the list https://github.com/spring-projects/spring-data-relational/blob/main/spring-data-r2dbc/src/main/java/org/springframework/data/r2dbc/dialect/DialectResolver.java#L113-L120
I had to add my own provider because of https://github.com/spring-projects/spring-framework/issues/33947 and register it in spring.factories to reach out to this issue.

We currently don't have the capacity to add and support additional dialects.
But you can register your own Dialect using a DialectProvider
Related: https://github.com/spring-projects/spring-data-relational/pull/1770


Miner displays incorrect value of mined blocks

Miner displays incorrect value of mined blocks, suddenly starts counting blocks ever second.
GPU AMD R290 / Windowsx64

I believe that the same bug which causes this is also causing the miner to crash on occasion. It's our top priority right now.


--train_steps flag not working

Description
So I am running tensor2tensor framework with my own model, and I see that --train_steps flag not working., it keeps on training past 1000 that I have defined and stops above 200000 (sorry I could not track exactly when model stopped, but it does go beyond the set steps by a large margin)
	t2t-trainer \
	  --t2t_usr_dir=$USR_DIR \
	  --data_dir=$DATA_DIR \
	  --problem=$PROBLEM \
	  --model=$MODEL \
	  --hparams_set=$HPARAMS \
	  --output_dir=$TRAIN_DIR
	  --train_steps=1000 \
	  --eval_steps=100

Is there any hint why this could not be working?
Also, what is the flag to train with early stopping?
...
Environment information
OS: <your answer here>
Linux 16
$ pip freeze | grep tensor
# your output here

mesh-tensorflow==0.0.5
tensor2tensor==1.13.0
tensorboard==1.13.1
tensorflow-datasets==1.0.1
tensorflow-estimator==1.13.0
tensorflow-gpu==1.13.1
tensorflow-metadata==0.13.0
tensorflow-probability==0.6.0
tensorflow-tensorboard==1.5.1

$ python -V
# your output here

Python 2.7.15 :: Anaconda, Inc.
For bugs: reproduction and error logs
# Steps to reproduce:
...

# Error logs:
...

I am having similar problem, unable to modify train_steps , eval_steps and eval_after_n_training _steps values. Any help?


Introducing better dependency management

Description
This PR includes the following proposed change(s):

Adding a bom file

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Refactoring / Documentation

if your change is a breaking change, please add breaking change label to this PR
How Has This Been Tested?
Running build locally
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have made corresponding changes to the documentation
[x] New and existing unit tests pass locally with my changes

Do we need to update documentation for these changes? It doesn't break the git action, which is nice. :)


Magellan GraphQL comparison

Does anyone know of the differences between this repo and the Magellan GraphQL repo? I was thinking of switching over to that repo, there seems to be more documentation there and I'm curious to know if anyone has any experience with both repos

Hey @saviodo5591, i've been looking at it but it seems that they are both different packages, i was looking for subscriptions in graphql, i will try it out and in any case let you know how it goes, thanks for sharing it!


Avoid warning after setting creator or modifier when opening in excel

Excel shows up a warning when xsi:type="dcterms:W3CDTF" is set in docProps/core.xml on dc:creator or cp:lastModifiedBy.

Can't even think why those particular nodes were set to StringNodeW3C (which are for dates). Probably a typo.


Add Polygon.center method

Center the polygon on a given point or {x: 0, y: 0}

I've renamed the method to .centerAt(), since we might want a .center getter that returns the centre of the bounding box, rather than the centroid. Otherwise looks good!


Remove 6.2 upgrade testing from nightly builds

Description
6.2.x fails to install the robotest app due to a busted hook init container,
described in #2243. Because 6.2.x won't eventually be supported to
7.1.x (when k8s 1.19 merges #2028), it doesn't hurt to remove
upgrade testing from it now.
Nightlies have been failing due to this bug since the cache fixes (#2233)
resulted in actually testing 6.2 upgrades.
Type of change

Internal change
This change has a user-facing impact

Linked tickets and other PRs

Contributes to/avoids #2243
Fixes nightly failures surfaced by #2233

TODOs

[x] Self-review the change
[x] Perform manual testing
[ ] Address review feedback

Testing done
Installed 6.2.5 manually, looked at the pods:
walt-70-upgrade-node-0:/$ kubectl get pods
NAME                              READY   STATUS                  RESTARTS   AGE
echoserver-77f9c59fc5-j9r9z       1/1     Running                 0          3m16s
echoserver-77f9c59fc5-k2tf2       1/1     Running                 0          3m16s
echoserver-install-606670-q96lh   0/1     Completed               0          3m25s
echoserver-status-23edee-c2g8p    0/1     Completed               0          3m15s
echoserver-status-ece370-5h6j5    0/1     Init:CrashLoopBackOff   4          2m43s
walt-70-upgrade-node-0:/$ kubectl logs -c init echoserver-status-ece370-5h6j5

connected to https://gravity-site.kube-system.svc.cluster.local:3009 adminagent@graciousbassi9271
[ERROR]: Get https://gravity-site.kube-system.svc.cluster.local:3009/pack/v1/repositories/gravitational.io/packages/robotest/6.2.5/envelope: stat /etc/kubernetes/kubectl.kubeconfig: no such
file or directory, failed to create kubernetes client from /etc/kubernetes/kubectl.kubeconfig

I also checked 6.3.18, it does not suffer from the same behavior:
walt-70-upgrade-node-0:/$ kubectl get pods
NAME                              READY   STATUS      RESTARTS   AGE
echoserver-77f9c59fc5-7hd25       1/1     Running     0          12m
echoserver-77f9c59fc5-rnctc       1/1     Running     0          12m
echoserver-install-efa6de-s6t8s   0/1     Completed   0          12m
echoserver-status-a93bb6-dhr8m    0/1     Completed   0          12m

I believe this is due to resolver relying on POD_IP and the PR that was not backported to 6.2.x.


oc sa get-token is deprecated

oc sa get-token is deprecated and no longer works on openshift 4.14 and newer, replace with current oc create token command

Thank you for the PR. Do you know what the default duration oc a token is when running oc create token is? I believe it's 1h, but I am far from certain. In either case is 1h sufficient for what this script is doing or should we extend the life with --duration flag?

Yeah I am pretty sure it is an hour, and this probably not long enough. I have run into the token timing out while I was testing stuff. Let me increase the timeout to 24 hours or something like that.


Fix specular water

Specular highlight code for water was in place but it was not working. This PR fixes specular highlight for water, and makes reflectivity and shine damper attributes editable on water components.
https://user-images.githubusercontent.com/28971753/167267431-421854dc-8cac-440f-8146-df35e53aa451.mp4

Looks amazing!!


yoasobi - idol

Song Name
Idol
Artist Name
Yoasobi
Source
Youtube
Youtube Link
No response
Direct File Link
No response
Song ID
1218996
Start Offset [ms]
No response
Extra
No response

reject
no link given


Allow specifying tree node padding increment

Since I'm rendering custom nodes with icons in my tree, the default left padding of 16px per level doesn't look good. I'd like to be able to customise this to be based on rem units, ideally using CSS, but a Vue property would work too.
ElementUI version
1.1.6
OS/Browers version
Linux/Chrome 54 (Electron)
Vue version
2.1.10

Added an indent property: https://github.com/ElemeFE/element/pull/2713/files#diff-17d1001f344d2451e2df4708e8623fecR765


Fix pyBQM.add_linear and pyBQM.add_quadratic

Codecov Report

Merging #908 (d092e92) into main (92cb108) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #908      +/-   ##
==========================================
+ Coverage   82.82%   82.83%   +0.01%
==========================================
  Files          79       79
  Lines        6666     6672       +6
==========================================
+ Hits         5521     5527       +6
  Misses       1145     1145

Impacted Files
Coverage Δ

dimod/binary/pybqm.py
96.72% <100.00%> (+0.08%)
:arrow_up:

Continue to review full report at Codecov.

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


YOURLS on HTTP/2 Server

I did all the best I can, but it only gives me
Not Found
The requested URL /1 was not found on this server.
Apache/2.4.18 (Ubuntu) Server at kalyebooks.com Port 443
When I was in hostgator, I was able to run, install and shrink urls and successfully redirect them. Though very hard.
Now that I moved to a vps, with HTTP/2 & rewrite enabled.
Been wondering if it's config file that need to change http into https
Or allow over ride all.
Searched all I can.
After being so glad, putting your name and photo, and the contributor name and link their github on my credit page, I just fall apart right now, after finding saving grace from all shortener, suddenly it doesn't work anymore, and I will not find any other shortener anymore, than this one. It's the best, but I can't figure it out anymore.
Is this compatible for http/2? I'm so frustrated.
PLEASE HELP ME.
Before any bug report:

[ ] Check you are using the LATEST release or the development branch.
[ ] Make sure you have SEARCHED closed issues first.
[ ] Read the GUIDELINES linked in the yellow notice box above.
[ ] COMPLETE this issue template.
[ ] DELETE these first lines.

Thanks you!
Technical details

YOURLS version:
PHP version:
Any other useful information depending on context (server version, mysql version, browser version, OS version...)

Reproducible Bug Summary

This is the first step
second step
etc... METHODOLOGY

This is a bug because... DESCRIPTION

Apache latest version, PHP 7, mysql 5.6, ubuntu 15.10 with ssl enabled.