[Android] Run callback on caller / background thread

Would it be possible to add an option to disable the runOnUiThread behaviour?
I have a usecase where I need to fetch some data over HTTP from an IntentService, which runs on a background thread. Most of the logic is in the fuel callback, and that should preferably also run on the background thread.
I'm making do with wrapping stuff in thread {}now, but it would be nice to have the option in this library.

I think the Callback style was design with the ease-of-use in mind so that there was no configuration support from the beginning.
Can you try the synchronous version instead? If you use the synchronous version, it will just run on the caller thread hence in your use-case, it will be on the bg thread?

I think the default behaviour is great. It does what you expect it to in 90% of the cases.
The sync option is a good tip. My calls can run on both the UI and a backend thread though, depending on the context.
I suppose wrapping the callback in thread on these cases is not the end of the world.

Ok closing this issue then


package:discover prints an incorrect list of packages

This fixes #21538.
This updates the cached $manifest array after it's built, so that the console command can print the correct list of packages.

You need to describe clearly how to recreate.


Analysis of game logs

PDBot records game logs.  Do we want to do something with them?

I messaged /u/mtgratingtester on reddit who used to do this for mtg goldfish before wotc asked them to stop. They obviously won’t care about pd. Would be good to start with something that exists. I’ll also try Josh Silvestri who published similar stuff on cfb for a while. We know each other by sight (but no more) so that might be an “in”.

I messaged Josh. No reply yet from either.
@silasary – Can you attach some examples?

Things that can be analyzed here:

When you play card X do you win or lose?
When card X is played with card Y do you win or lose?
When you play card X and your opponent plays card Y do you win or lose?

I guess basically we would store {match_id,card,deck_id} for every card ever played. Then you can run whatever queries you want.

I remembered the bot could give me these. Some examples:
192353549.txt
192298930.txt
190352623.txt

Basic plan is probably:

When posting results from PDBot send the match id too.
Store the match_id in match
Look for /(.) plays [(.)]/ and $1 = mtgo username, $2 = card.

A more complex problem is to tie existing logs to matches but that shouldn't be too hard given that we know the result and both opponents and which match came before which in each place. We may have to reject some due to ambiguity.

More thinking.
I think we store an order number (what number action in the game this was) and (chat|action) enum so we can elide chat unless you were in the game or are an admin. (Or should all chat be considered public because the game was public? I think it's weird if people can view it retroactively when the only "person" there besides your opponent was pdbot.)

Features:

A match page that shows everything that happened.
Analysis of your results when you play any given card.
Analysis of archetype results when any give card is played.
Common phrases that occur in wins? (P1 activates CARD targeting CARD)

Everything modo bot needs to do has been done.
The analysis should be happening based on logsite's side.

pdlogs.match.id == decksite.match.mtgo_id for when we want to tie it up to people. Also mtgo_username I guess.

I linked the matches off /person/{id}/matches/. Step one.

The most basic unit we're interested in here is "{mtgo_username} casts [{cardname}]". I'm just gonna process those to start with.

Improvement: record the turn against each play.

There's a tiny amount of data where we have the match id but not the log. Not important.
mysql> SELECT COUNT(*) FROM `match` WHERE mtgo_id AND mtgo_id NOT IN (SELECT match_id FROM logsite.game);
+----------+
| COUNT(*) |
+----------+
|       91 |
+----------+
1 row in set (0.10 sec)

mysql> SELECT COUNT(*) FROM `match` WHERE mtgo_id AND mtgo_id IN (SELECT match_id FROM logsite.game);
+----------+
| COUNT(*) |
+----------+
|    49015 |
+----------+
1 row in set (0.20 sec)

We have none of the reverse.
mysql> SELECT COUNT(*) FROM logsite.game WHERE match_id NOT IN (SELECT mtgo_id FROM `match`);
+----------+
| COUNT(*) |
+----------+
|        0 |
+----------+
1 row in set (0.24 sec)

This is a pretty large amounts of data.
Naïve consolidation queries on small amounts of it take minutes.
[logsite.game]

game_id,
log

=>

[logsite._game_card_person]

game_id,
name,
mtgo_username

=>

[decksite._played_card_person_stats]

name,
person_id,
season_id,
wins,
draws,
losses

Building _played_card_person_stats is super expensive. Indexes might help.
Will have to think of something smarter or do everything piecemeal.

mysql>  SELECT
    ->             *
    ->         FROM
    ->             logsite._game_card_person AS gcp
    ->         INNER JOIN
    ->             person AS p ON gcp.mtgo_username = p.mtgo_username
    ->         INNER JOIN
    ->             logsite.game AS g ON g.id = gcp.game_id
    ->         INNER JOIN
    ->             `match` AS m ON m.mtgo_id = g.match_id
    -> ;
Empty set (4 min 38.79 sec)

Possible other interesting stats: average number of turns per game. If we can link it to the decks, which we can now, we can crunch those averages by archetype or any other combination similar to the matchups page.

I think the first thing to do would be to structure the data, so when PDBot sends the lines to the DB have a processor that can extract structured info like turn number, card played or action taken, so they can be aggregated later by those keys.
Answering questions like: what's the average number of times someone has activated Rofellos? Or what's my win record when I cast Chain Lightning twice in a game?
All in all I think we could build a pretty complex an interesting system for analysis.


Multi-GPU train error

Hi I am facing a multi-gpu training problem and I have went through this repo and the original maskrcnn-benchmark repo and still can't solve it.
My problem is that when I train with One GPU, every thing is fine
But when I use multi-GPU(2,3,4), I got this error

How does this happen during multi-gpu training?  It seems to be a problem with the boxlist_op.py but I I don't think there's any fault in my code because it went GREAT when using one gpu.
Thanks A Lot !!!

What's your running command? It seems that the problem is in the inference time. Our codes only support one image for one gpu in the inference time.


Some strings have and lower dash in it, is it normal? Should it be included in the translation?

I wante to help translate in italian the HDT however I'm in doubt regarding some strings that have a "_" in it. Should it be included both when it appears at the beginning of the string and also when it is in the middle of it like "P_aladin" or "Wa_rrior"?

This is used for Alt+{key} menu controls. Pressing e.g. Alt+{N} in HDT will open the "New" menu item.
Whether you want to implement this in Italian is up to you, I don't think it's gets much use anyway.

Thanks for the clarification =) Didn't knew they were for the shortcuts.

I'm confused, I started translating a lot of lines using the previously suggested tool ResX but now I see in that the project moved to Crowdin. Should I trash the work done so far? How can I re-start translating in Crowdin?

Alright, uploaded it. You can sign up and request access here: https://translate.hsdecktracker.net/project/hearthstone-deck-tracker/it#

Will do!


Can't we use images to display in the table columns ?

You can use images inside a table cell. You just have to use a custom component for that.
With custom components, you can use display images, lists, or whatever else you want.
Please, take a lot at custom components in the documentation


Fix Coveralls Integration

(fixes #38)

Coverage increased (+100.0%) to 100.0% when pulling 4dcc55c253e044e31b074270ef529f1b5c5e6406 on coverage into f18ad744da7bc1dfa0f5cbf7234e33c83785e6a7 on master.


Lets discuss in the dev meeting tomorrow

wrong branch. meant to send to development


⚠ Bofrost.fr has degraded performance

In bd45e41, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 989 ms

Resolved: Bofrost.fr performance has improved in e88f5c1 after 13 minutes.


Improve markdown highlighting

Open some markdown file and use :TSCaptureUnderCursor to get missing highlights.

Markdown looks better now


Added test for multiple connections

Related Issue(s)
Closes #44
Proposed Changes

Change 1: Added a new test class for 'Server' named 'ServerTest'.
Change 2: Added a new test method testMultipleConnections in the ServerTest class.
Change 3: Introduced a loop to simulate multiple client connections and assert their successful connection status.
Change 4: Utilized an ExecutorService for concurrent execution of client connection tasks.
Change 5: Added a stop method in 'Server'.

Description
This test (testMultipleConnections) is designed to check the server's capability to handle multiple clients connecting simultaneously. It creates a set number of Client instances, each attempting to connect to the server concurrently. The test asserts that each client successfully establishes a connection. This setup simulates a scenario where multiple clients connect to the server concurrently, testing the server's concurrency and connection handling capabilities.

:rocket: This PR description was created by Ellipsis for commit 58a699369a1eeb102f306f1a8a0c904de5f3eb69.

Summary:
This PR adds a new test for the 'Server' class to validate its ability to handle multiple concurrent client connections, updates the 'Client' and 'Server' classes with new methods to support this test, and utilizes an ExecutorService to concurrently execute client connection tasks.
Key points:

Added a new test class 'ServerTest' with a method testMultipleConnections to test server's ability to handle multiple connections.
Updated 'Client' class with a connectToServer method.
Updated 'Server' class with a stop method.
Utilized an ExecutorService for concurrent execution of client connection tasks.
The test asserts successful connection of each client.

Generated with :heart: by ellipsis.dev

Looks good overall to me, but it could be beneficial to add some logging statements in the test class to provide more information in case of failures. For example, logging the start and completion of each client connection attempt.


FM Receiver example

Motivation
See how FutureSDR can be applied on simple example such as an FM Receiver.
Also testing the AudioSink as stated in #4
Solution
While we add discussion on Discord and other alternatives implementation, we come up with the idea that an Apply block with a mutable closure would be sufficient.

Hi, I hope you don't mind. But since I didn't want this to be stalled, I just added a very minimal FM receiver example to the project: https://github.com/FutureSDR/FutureSDR/blob/master/examples/fm-receiver/src/main.rs


trivy BOMs and vuln diffs

This is some work following the move from Syft to Trivy for SBOM generation in https://github.com/thepwagner-org/actions/pull/14
Since Trivy's CycloneDX SBOM format contains vulnerabilities, my goal was to modify the report to include any .vulnerabilties[] difference between the two SBOMs.
I ran into an issue in that the library I was using for CycloneDX didn't support the .vulnerabilities[] path. I started adding it in https://github.com/CycloneDX/cyclonedx-javascript-library/pull/163 , but that is more involved than I thought it would be. 😿
🕵 I'm going to evaluate generating my own types from the CycloneDX JSON schema. Given what I want (.components[].purl and .vulnerabilites[].id), I don't feel bad ignoring the other 99% of the BOM.

I'm going to evaluate generating my own types from the CycloneDX JSON schema

That worked out OK, :shipit: .
Comments look like:


Improvements to ansible-runner docker image

Used COPY instead of ADD as recommended in docker best practices,
clubbed RUN commands to reduce number of layers,
added comments
fixes #204
Ran both the images and used find <dir> -type f for a couple of directories and compared files to verify.
Signed-off-by: darshkpatel<EMAIL_ADDRESS>

Made mistake Signing the commit. Closing.


New version: dail8859.NotepadNext version 0.6.1

Pull request has been created with WinGet Releaser v2 :rocket:
Microsoft Reviewers: Open in CodeFlow

For some reason, this application does not remain running during automated validation. But it runs fine in a Windows 11 VM.

/AzurePipelines run

@wingetbot waivers Add Validation-Executable-Error


Feature: Special event is activated only when the application is launched during a certain time interval (for Christmas, this is from 20.12 00:00:00 to 10.01 23.59.59 inclusive)

Fixed by 9bac1be commit


etcd discover

When I try to bring up a cluster with this discover tech, I got two question as follow:

The first step is to determine the cluster size with curl command: curl -X put .....
It seems this command use the 80 port as default. But when I tried this with my localhost, which
report with connection refused when a etcd server is actually running in my host.  Is there any
other config against this?

And this command use http, what if I use a https scheme? Can this still comply with?

Or Can I just ask for short, could this etcd discover service be used for https cluster?

As far as I can tell that the discovery mechanism requires a existing etcd cluster. The etc member will anticipate the cluster with curl from the port 80. As I found in the doc following lines.
If http://example.com hosts an etcd cluster for discovery service, a full URL to discovery keyspace will be http://example.com/v2/keys/_etcd/registry.
But when I start a cluster in local, I cannot get access to port 80 with curl.

It seems the port should be assigned, 80 is the default one.
curl -X PUT http://example.com/v2/keys/_etcd/registry/${UUID}/${member_id}?prevExist=false -d value="${member_name}=${member_peer_url_1}&${member_name}=${member_peer_url_2}"
what does this member_id means here?

The first step is to determine the cluster size with curl command: curl -X put .....
It seems this command use the 80 port as default. But when I tried this with my localhost, which
report with connection refused when a etcd server is actually running in my host. Is there any
other config against this?

@WandyLau I think you are focusing on the port 80 example too much.
#!/bin/sh

UUID=$(uuidgen)
cluster_size=3

curl -X PUT http://<IP_ADDRESS>:2379/v2/keys/_etcd/registry/${UUID}/_config/size -d value=${cluster_size}

maybe this example will help you.

And this command use http, what if I use a https scheme? Can this still comply with?

Sure as far as curl is concerned curl -X PUT https://<IP_ADDRESS>:2379/..... you may need additional flags to connect to the cluster depending on your setup.
https://github.com/coreos/etcd/blob/master/Documentation/v2/security.md

Also why not use public-discovery-service?

@hexfusion Thanks for your reply. We are trying to build up a production solution, so this public service may not be proper in security. We have to reduce the dependency on the third party.

yes, I can make it.  BTW, what does this v2 mean in the link? I thought it was the version of etcd. So
I could not get the result with v3.
curl -X PUT http://<IP_ADDRESS>:2379/v2/keys/_etcd/registry/${UUID}/_config/size -d
value=${cluster_size}

And what does this member_id mean in below link? How can I get it?
curl -X PUT http://example.com/v2/keys/_etcd/registry/${UUID}/${member_id}?prevExist=false -d
value="${member_name}=${member_peer_url_1}&${member_name}=${member_peer_url_2}"

When I try to register the first node, I got some confused
curl -X PUT http://example.com/v2/keys/_etcd/registry/${UUID}/${member_id}?prevExist=false -d
value="${member_name}=${member_peer_url_1}&${member_name}=${member_peer_url_2}"

what does the member_id means?

what does the member_id means? And after I register a etcd, how can I unregister it if I may find some mistake arguments that I set.

member_id is a unique identifier for each node. For example etcdctl member list would display the member_id of each node in the cluster.
@WandyLau I am not that familiar with manual node management of v2 cluster using REST API but assuming your using etcdctl.
cluster-reconfiguration-operations

And after I register a etcd, how can I unregister it if I may find some mistake arguments that I set.

from the link above
etcdctl member remove a8266ecf031671f3
Removed member a8266ecf031671f3 from cluster

You might need to review the --endpoints flag for etcdctl.
I hope that helps.

@hexfusion yes, that's helpful.
One more question on DNS discovery, how the DNS SRV records are created? They are created by the etcd automatically or manually? I found no explanation on this. Could you please help me with this?

One more question on DNS discovery, how the DNS SRV records are created? They are created by the etcd automatically or manually?

@WandyLau you would need to add these records manually. This is a pretty good example of what is expected from the docs. create-dns-srv-records

@hexfusion Ah, I got it. Thanks. I think I can close this.


Hangup a channel does not work

hi all
I have created a channel in a.go :
key := ari.Key{Kind: ari.ChannelKey, Node: "caller", App: "xx"}
org := ari.OriginateRequest{Endpoint: "SIP/x", App: "xx", CallerID: "test" , Timeout: 300}
t1, err := cl.Channel().StageOriginate(&key, org)
	if err != nil {
		fmt.Print("Cannot create channel : ", err)
	}
t1.Exec()

now in b.go I want to destroy the channel so I do the following :
key := ari.Key{ari.ChannelKey, "01d5sgk517qws4rsa91v5tzf3r-ch" ,"caller", " ", App: "xx"}
cl.Channel.Hangup(&key, "for test")

but it does not Hangup the channel
is there any way to destroy a channel like :
curl -s -u ehsan:esx -X  DELETE "http://localhost:8088/ari/channels/01d5sgk517qws4rsa91v5tzf3r-ch"

c.Hangup() should be calling exactly that HTTP request, under the hood.
If you look at channel.go, you will see that it is using the delete operation on the x/channels/{channelID} URI.

In general, you would not want to manually construct the ari.Keys.  This is what Handles (such as Channel Handles) are for.  You can pass channel handles around in local code, and they contain references to the client, the app, subscriptions, etc.  You can pass ari.Keys around over a network, either as JSON or as protobufs, and they contain everything needed for another client to reconstruct the Handle.  See the example applications for common usage.

Also, in your example, please note that your original Key will NOT match the key returned from the Originate.  The Key which is passed to Originate (or StageOriginate) is a reference Key.  It is completely optional and only used for matching the new channel to an existing channel, in terms of Node (the Asterisk box on which the channel should reside), and communication parameters, such as codecs.

Also reference: https://github.com/CyCoreSystems/ari#handles.  The non-standard Hangup code you are using may be causing the problem.

thank you
yes I should use channel handles. I use channel.Get() to get the ChannelHandle of the channel
and it works
key := ari.Key{"channel", "01d5sn4kqckph8gj33t7qk5fee-ch", "", "", "xx"}
tt := cl.Channel().Get(&key)
tt.Hangup()

but I wonder why this method works for bridge
bkey := ari.Key{ari.BridgeKey, bid, "", "", "ss"}
cl.Bridge.Delete(&bkey)

also works by BridgeHandle
bkey := ari.Key{ari.BridgeKey, bid, "", "", "ss"}
b := cl.Bridge.Get(&bkey)
b.Delete()

Asterisk is picky about the hangup codes.  When you use the Hangup() from the ChannelHandle, it uses a standard hangup code.  The one you are using in your example is not a standard code.  I would guess that Asterisk is ignoring it, or otherwise not processing it properly.  It may be translating it into something which your SIP stack or endpoint is not recognizing.  It's hard to say without more information.

yes, when I use ChannelHandles and BridgeHandles to hangup a channel or delete a bridge it works correctly.
but for Bridges, it also works even when I do not use BridgeHandle

Bridges are just deleted, not hung up.  There are no hangup codes to be transmitted for a bridge, which is why they work for you in this manner.

Yeah I got it.
Thank you Ulexus 🙏


Fix android build when API<23 and test, update build doc and pipeline

Now the all tests have passed on android.

/azp run

@pranavsharma Windows CI failed due to network error.
BTW, could you please enable Android CI? The yaml is tools/ci_build/github/azure-pipelines/android-arm64-crosscompile-ci-pipeline.yml

/azp run

@snnn looks like windows gpu pipeline fails due to network issue again

/azp run

@linkerzhang The windows cpu ci failed due to insufficient disk space 😂

/azp run

@linkerzhang nGraph CI fails due to an unrelated error

/azp run


Update the SubscriptionUpdate to contain both IDs and states of updated Entities

In this PR we change the SubscriptionUpdate message model:
- repeated google.protobuf.Any updates;
+ repeated EntityStateUpdate entity_state_updates;

Here EntityStateUpdate is the message containing both Entity ID and entity state.
The updates field has been deleted and its number has been marked as reserved for the serialized format compatibility.
This change is required to make the subscribers aware of the ID of the received updates.

@armiol, PTAL.

@armiol, PTAL.


Hierarchical Optimization: Respect Boundaries

Sometimes, one wants to have hierarchically optimized parameters (e.g. sigmas), but also confine them to some box constraints [lb ub]. Therefore, having computed the optimal parameters in the inner loop, check if they stay in the box (if one was passed), and set them to the left or right boundary if not.

keep in mind we then need to calculate and use the derivative of J with respect to sigma, since it is not 0 anymore

We have already computed the derivatives dJ/dsigma, dJ/dc etc., so this should theoretically be no problem, and practically at least for forward sensitivities not too difficult to implement. But it might be not the most urgent issue ...


mkdir not working in all cases

Describe the bug
mkdir has many possible cases. It is known to not work in all cases
To Reproduce

mkdir d1/d2/d3/d4
mkdir d1/d2/d5/d6
mkdir d1/d2
you can only see d5

mkdir with root path is not working in all times

Expected behavior

mkdir should work in all cases

Screenshots

None

Additional context

generally, the support for directory operation wiith root is not working.

it's invalid in parent directory does not exist, checks are added, parent directories must exists. closing


UI - Rancher does not allow you to edit the number of nodes in an already created EKS cluster.

It would be good if the EKS driver edit screen allows you to change the number of nodes in an existing EKS cluster. Thus, the user would not need to make this change through the AWS console.
Pull requests

https://github.com/rancher/kontainer-engine/pull/231
https://github.com/rancher/ui/pull/4072

As we are moving to the new EKS v2 support in Rancher v2.5, we encourage all users to upgrade and use the new provisioner which allows being able to change the number of nodes in EKS.
Thank you for your PR! @ribeiro-rodrigo


v11: add more cross-links to v10

The problem
Many users don't realize the main carbon website is now showing only v11 content. We need to integrate more context linking to v10 so user's can quickly understand they are on v11 and understand where to go to get v10. We need to account for user's by-passing the homepage through saved bookmarks and links from other pages or sites. If they miss the homepage how do they know they are looking at v11.
Possible solutions

Add persistent banner to the shell header (just homepage or across all pages) that links back to v10
Add notification to all component pages that are new or have been update in v11. Link page to v10 pages?
Add notification to token pages like color>usage?? (is this necessary?)

Hey team! Please add your planning poker estimate with ZenHub @laurenmrice @thyhmdo


Is it possible to either run a node or a miner?

Is it possible to either run a node or a miner?

Dogecoin reference implementation is generic. It contains all functional blocks: full node, miner and wallet.  It can be built with or without GUI\wallet, but not without miner.
CoinGreen was created from dogecoin reference implementation. So the CoinGreen application that I have is also generic and includes everything.   It can be built with or without GUI\wallet, but not without miner.
There are other applications that adhere to dogecoin protocal and implement different subsets of functions.
For example:

Fully functional wallet application (just one of !!!) - [multidoge] (https://multidoge.org/)
Stand alone dogecoin miner (just one of !!!) - DDD
other applications, such as sites that show blockchain history


cess of possibly undefined property i sAttribute through a reference with static type QName.

I got this error when building from the source
[java] D:\Repo\ApacheRoyaleASJS\royaleasjs-sdk\frameworks\projects\XML\src
main\royale\XML.as(1175): col: 12 Error: Access of possibly undefined property i
sAttribute through a reference with static type QName.
[java]
[java]                     if(name.isAttribute)
[java]                             ^
[java]
[java] D:\Repo\ApacheRoyaleASJS\royaleasjs-sdk\frameworks\projects\XML\src
main\royale\XML.as(1951): col: 6 Error: Property prefix is read-only.
[java]
[java]                                     ns.prefix = attr.split(":")[1];
[java]                                     ^
[java]
[java] D:\Repo\ApacheRoyaleASJS\royaleasjs-sdk\frameworks\projects\XML\src
main\royale\XML.as(2216): col: 47 Error: Access of possibly undefined property p
refix through a reference with static type QName.
[java]
[java]                     _name = getQName(nameRef.localName,nameRef.prefi
x,nameRef.uri,nameRef.isAttribute);
[java]                                                                ^
[java]
[java] D:\Repo\ApacheRoyaleASJS\royaleasjs-sdk\frameworks\projects\XML\src
main\royale\XML.as(2216): col: 74 Error: Access of possibly undefined property i
sAttribute through a reference with static type QName.
[java]
[java]                     _name = getQName(nameRef.localName,nameRef.prefi
x,nameRef.uri,nameRef.isAttribute);

No error like this when building XML, now.  Maybe this can be closed?

@estanglerbm this is very old thread , I don't even remember the specific version I cloned.
Did you have valid proof that this is no longer happening?
If so, just mention which Royale version solves this problem and I'm happy to close it.

@blackjyn Well, if you build develop right now, there are no compile errors in XML.  Doesn't that prove it?

I'm no longer updated my local repo since then and prefer the binary dist instead.
I assume you already build latest version and the problem has gone so I'm gonna close it with peace


Module 2 exercises [Moemedi Mbakile]

Please "drag & drop" exercise 1 pdf file below.
Cattle post proximity map.pdf

Please "drag & drop" exercise 2 pdf file below.
Map 2.pdf

Please "drag & drop" exercise 3 pdf file below.
Module 2 Ex3.pdf

Please "drag & drop" exercise 4 pdf file below.

Please UPDATE THE TITLE of this "Issue" in the title bar above with your name.

If you haven't done so, please go back to Module 2 and make sure to take the post-training survey. Participating in surveys is required for receiving the "GALUP Certificate" by the University of Florida.

Well done, Moemedi. You clearly grasped the material outlined in Module 2 and successfully applied it to these exercises. Thank you for your participation!


allow OSBA to run under managed identity rather than providing service principal

RIght now it appears I need both managed identities and service principals to manage my kubernetes cluster. It would be nice if I can merge everything to use just managed identities since these show up in the resource group and not in the AAD.
Let me know what you think of this idea. Is this a good idea, or am I nuts.
The enhancement request is to allow managed identities object ids as inputs for the azure.clientId  parameter

I'd love to be able to use it with aad-pod-identity. It'd make it much easier to manage identity cleanup on cluster removal.

@norshtein is there any update on this?


Add backend changes from publish branch

@dchiquit My assumptions are that this has the backend for sync and lock, and those have both already been QA-ed separately and merged into the publish branch. If this is all true, then this PR should have also passed QA already and be ok to merge, and in that case I'll approve it.


Incompatibility with cordova-plugin-googlemaps

Actual Behaviour
Hello,
When both  cordova-plugin-googlemaps and phonegap-plugin-push are used in the app's, the app crash just after the slash screen appear.
Platform and Version
"android": "5.2.2"
Device used
Asus Zenfone 2 and Sony experia M2
Cordova CLI version and cordova platform version
cordova --version  6.3.0
cordova platform version android "android": "5.2.2"

Plugin version
phonegap-plugin-push  1.8.3
cordova-plugin-googlemaps 1.3.9

The code I run :
 var options = {
            android: {
                'senderID': '330401114899',
                // 'icon': 'images/app/logo.png',
                // 'iconColor': 'rgb(235, 102, 41)',
                'vibrate': true,
                'sound': true,
                'forceShow': true
            }
        };

        var push = PushNotification.init(options);

        push.on('registration', function(data) {
            console.log('----  ---- registration ---- ---- ', data);
            console.log('$window', $window);
        });

        push.on('notification', function(data) {
            console.log(data.message);
            console.log(data.title);
            console.log(data.count);
            console.log(data.sound);
            console.log(data.image);
            console.log(data.additionalData);
        });

What's more, If I remove the cordova-plugin-googlemaps plugin, The app runs fine.
Thank you,

@noam1610 same root cause as #1325. The resolution will be in the upcoming 1.9.0 release which will use Cocoapods to include the GCM libraries. Then the two plugins will be able to include the same libs without this error.

Thank you !

I have got this issue too after I upgrade push plugin to version 1.8.3/1.8.4.
It doesn't happen with push plugin version 1.8.2

@noam1610 can you make sure you have CocoaPods installed then try the latest code from master to see if your bug is resolved. If it isn't I may need to actually send a PR to cordova-plugin-googlemaps.

I don't use cocoapods.
I will try ..

Thank you,
It seems to work...


Windows ia64 build

Problem description:
This may be an odd question, but how would I build an eclipse temurin build for Windows for Itanium? Specifically Windows Server 2008 R2. I know that openjdk 8 has source code for Windows ia64, I just can't find a "simple" way to compile it. Unless I'm missing something, it doesn't look like eclipse temurin supports it.

Wow there's a question I didn't expect! The project does not have any Itanium hardware, now any easy way to obtain it. In principle I would expect that it wouldn't be too hard if the prerequisite packages were available - which may be easier if you've got the software to let you run x64-32/ia32 code on your system. It would need cygwin or similar to be available, and while they did have a 32-bit version last time I checked it was marked as "not recommended" :-)
Obviously, given the above, we are unable to offer any particular support for it and the demand would realistically be too low for us to do so. What problems did you come across when trying? In principle I would expect that if you had a suitable compiler and the code is still in the JDK8 codebase you should be able to build it as normal.
If you want to try it yourself, our wrapper script makejdk-any-platform.sh probably won't work (Although it might...) but take a look at the log in https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk8u/job/jdk8u-windows-x64-hotspot/1227/consoleFull for an example of what an x64 build looks like. In particular the downloading of freetype and the ./configure line (you probably don't need all of the options we include). It's roughly speaking a normal "configure; make" process once you've got all the bits in place. You need an existing JDK7 or JDK8 as a "boot JDK" in order to build the JDK.

Mainly the issue I'm having is getting to a buildable state. Unless I'm missing something, compiling for ia64 windows can only be done with Visual Studio 2010, which openjdk supposedly supports. However, I can't seem to build out an sln file.
It's entirely possible I'm misreading the requirements though, but it was my understanding that to build openjdk, you need cmake. cmake doesn't support ia64 windows from what I can find. There are ia64 files present in openjdk 7 and 8, so it should be possible right?

Openjdk itself should not require cmake to build (Curious as to where you saw that - was it in our docs or upstream openjdk?)
We've only been building on VS2013 and later recently so I can't confirm if that really does still work. (The OpenJ9 project now uses cmake by default, but the hotspot builds do not need it)

Oh, maybe it was OpenJ9 I was seeing then. I know it's preferable to build on the platform you're deploying on, but ia64 doesn't actually have compilers (for windows) that run on the device. You need to cross-compile from an x86 devices, which may be where I'm running into issues.
Since there's no cross-compile support in mingw for ia64, the docker build script wouldn't function right?


FunBlocks Update - Converge UI and add code formatting

Changes:

The blocks UI now has the same look as the text CodeWorld UI. This includes the removal of bootstrap
Code is now formatted according to a simple style. ifs, case and so on get indented.
Code generation is also somewhat rewritten, as a result most blocks don't need to be hard coded. Each block is converted to a AST using its .functionName value and the list of inputs.

Sorry for the delay!  I was out of town for the weekend.  I'll merge and push this now.


NIFI-2756 - Add displayName to maven archetypes

@alopresto Would you mind having a look on this PR?
I am not familiar with maven archetypes but I noticed that the ones we have seem to result in code lacking some of the present practices such as having displayName, including * imports, unused classes, etc.
This PR tries to address that.
Cheers

@bbende mind sharing the steps you took? Keen to try to reproduce and fix.

@trixpan sure...

mvn clean install on this branch

In some other directory:

mvn archetype:generate -DarchetypeGroupId=org.apache.nifi -DarchetypeArtifactId=nifi-processor-bundle-archetype -DarchetypeVersion=1.1.0-SNAPSHOT -DnifiVersion=1.1.0-SNAPSHOT -DarchetypeCatalog=local

Fill in something like groupId org.apache.nifi, artifactId displaytest-bundle, basename displaytest

Try to build the generated project using mvn clean install -Pcontrib-check

Repeat with service archetype:

mvn archetype:generate -DarchetypeGroupId=org.apache.nifi -DarchetypeArtifactId=nifi-service-bundle-archetype -DarchetypeVersion=1.1.0-SNAPSHOT -DnifiVersion=1.1.0-SNAPSHOT -DarchetypeCatalog=local
For additional reference:
https://cwiki.apache.org/confluence/display/NIFI/Maven+Projects+for+Extensions

@bbende hopefully the second commit should address your feedback.
May I ask you to test and if ok, merge into master?
Kind regards


Respect isolate_session_state in config when ClusterSpec propagation is enabled

Currently, the isolate_session_state  is hard coded to be true when ClusterSpec propagation is enabled.
However, here comes the large-scale distributed training case where workers are running on fragile machines. In this case:

It is common to see worker N failed and then is restarted on another machine by the infrastructure like Yarn. Then the ClusterSpec held by PS servers are out-of-dated as the IP of worker N has changed.
It is reasonable that in some time period, the inbound training data increased heavily in size and we want to increase the worker number without restarting the whole job including PS servers.
It can be helpful to manually replace slow workers detected with new ones on a healthy machine without interrupting the existing training job.

With code change in this pull request, users can decide whether to share state of his Session like Variable values with others through the isolate_session_state switch in the session configuration.
In industry, the resource isolation among multiple distributed-training jobs is usually ensured by infrastructure using something like Cgroup or Docker. Hence it is definitely safe and popular to turn off the session isolation for workers inside the same job.

I signed it!

@saeta @jaingaurav Is there any update or feedback? Thanks a lot!

@saeta @mrry Any update or comment? Thanks a lot.

@mrry @saeta @jaingaurav @yuefengz Any update or comment? Thanks a lot.

@saeta I have addressed your requested changes for days.
Could you please help proceed this pull request? Thanks a lot~

@mrry @josh11b Kindly ping... Appreciate your comments.

@mrry @saeta
After a new commit is pushed, your reviews are dismissed in this PR by GitHub.
Could you help review it again? Thanks a lot.

@josh11b Kindly ping~

@mrry @saeta
After a new commit is pushed, your reviews are dismissed in this PR by GitHub.
Could you help review it again? Thanks a lot.

@josh11b Kindly ping~

@josh11b Kindly ping~

@yuefengz Hi, Yuefeng, could you please help invite @josh11b to continue this PR? One week has past TT

@rthadur Could you please help ping reviewers to continue this PR? Thanks a lot!

@josh11b Kindly ping~

@josh11b Kindly ping~

@josh11b Kindly ping~

@josh11b Kindly ping~

My last comment was:
"Okay, I'm fine with this description assuming the other reviewers agree that this description actually matches what the code does."
@saeta @mrry  ?

@josh11b @saeta Thanks a lot for your comments!
Has this PR been accepted after these changes or is there anything I can do to push forward this PR?

@josh11b @saeta @mrry Kindly ping~

Hi @shishaochen Just to let you know, over the past few days, we've been working this PR though the machinery. It's getting closer. Sorry again for the massive delay.


allow unrating on media pages. fixes #595

:speech_balloon: :arrow_up:

Should this be in the rating widget instead?

I think it might be better to create some media mixins for this sort of functionality between manga/anime instead of all the duplicate code.
This fits with how it is handled on the library page currently.

I was thinking more like making the Awesome Rating widget return null if it's the same value

We would need to change it up a fair bit as it's used on the onboarding pages before library entries are created.

Can't we compare the clicked value with the displayed value, and not have it refer at all to the library entry?

:100: That could work!

:bulb: bump.
Looking forward to seeing how the library handles on 1.13 & glimmer instead of react.

LGTM! 🚀


Change ports

5000 -> 37220
5105 -> 37221
Tied to: https://github.com/stratisproject/Breeze/pull/242
Resolves: https://github.com/stratisproject/Breeze/issues/240

Please ping me on merge @nopara73, I need to change that in the UI as well.

@dev0tion  I did change the UI as well: https://github.com/stratisproject/Breeze/pull/242

Lets try to get this in asap

that's cool, thanks @nopara73


Merge release/lavida (4.5.0) branch into master

PR Details
Description

Related GitHub Issue

Related Jira Key

Motivation and Context

How Has This Been Tested?

Types of changes

[ ] Defect fix (non-breaking change that fixes an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that cause existing functionality to change)

Checklist:

[ ] I have added any new packages to the sonar-scanner.properties file
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

@MishkaMN

@MishkaMN  Please resolve merge conflicts.

@MishkaMN Please resolve merge conflicts.

Looks like the base was incorrect. I switched to carma-master and approved


GPU install

I am trying to use 6 msi rx 570s on miner.  At what point during installation should I connect GPUs to the motherboard with PCIe risers? I’m having trouble booting into Ubuntu when I connect one GPU.

The installation works with 1 GPU or more GPUs. Don't forget to turn on the "BIOS" in the "Above 4G decoding" and "PCI Gen 2" settings.


test: adding manual step for deploying goldpinger

Fixes #
Description
Used to manually test node to node + pod to pod connectivity on a given CNI configuration
How was this change tested?
Used the make command to validate CNI without overlay and various other CNI flavors.
Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[ ] No

Release Note

As this doesn't touch any of the go code and CI is failing on ci-test, im bypassing it.


Connecting to multiple vcenters

HI,
I want to run three scheduled tasks against 3 virtual centers. I have been trying to figure out the best way to do this. All three can use the same credentials.
In creating a credential file using set-vicredentialstoreitem, I noticed that I must use the host name to create this stored credential. This means I will have three different cred files. I am tying to do this as efficiently as possible.
The only way I have seen to do this is to use three globalvariables files and use the -job parameter, with the vcheck script?
Thanks to anyone who can help!

If the account you are running against is AD integrated, you could run your scheduled task as that user, therefore not requiring the stored credential to be passed into the script manually.
Just remember to update the $Server variable in "00 Connection Plugin for vCenter.ps1" for each copy of vcheck you run.
I run one copy of vcheck via a scheduled task for each vcenter under my remit this way and it works well
It does mean maintaining the plugins separately for each of them, but that is kind of necessary anyway because my settings and exclusions are different for each anyway.

Hello legroucha,
I would like the same thing that you
You say to add "[ValidateSet("server1","server2","server1-A","server1-B","server2-A","server2-B")][string]$server" in param section
You want say that i must create a new function like this  in "vcheck.ps1" ?
function NameFunction
{
param
(
"[ValidateSet("server1","server2","server1-A","server1-B","server2-A","server2-B")][string]$server"
)
}
thank you for your feedback ;)
i'm also french and my english is not perfect ^^

my vcheck.ps1 with new line in param

as you can see i've remove the validateset
My Launch-all-vcenter.ps1

regards

Hi Legroucha,
thank you for your feedback.
I will do this and will be back to you if needed ;)

Using given suggestion getting below error:
:\Scripts\PS-Mandeep\vCheck-vSphere-master\vCheck.ps1 : Missing an argument
or parameter 'server'. Specify a parameter of type 'System.String' and try
gain.
t line:1 char:60
& "D:\Scripts\PS-Mandeep\vCheck-vSphere-master\vCheck.ps1" -server
~~~~~~~

CategoryInfo          : InvalidArgument: (:) [vCheck.ps1], ParameterBind
ingException
FullyQualifiedErrorId : MissingArgument,vCheck.ps1

:\Scripts\PS-Mandeep\vCheck-vSphere-master\vCheck.ps1 : Missing an argument
or parameter 'server'. Specify a parameter of type 'System.String' and try
gain.
t line:1 char:60
& "D:\Scripts\PS-Mandeep\vCheck-vSphere-master\vCheck.ps1" -server
~~~~~~~

CategoryInfo          : InvalidArgument: (:) [vCheck.ps1], ParameterBind
ingException
FullyQualifiedErrorId : MissingArgument,vCheck.ps1


Problems accessing server from outside my network on OSX

Everything is installed and working perfectly as far as the server goes. Scanning like a dream.
My issue is a Mac OSX (Yosemite 10.10.5) issue. I can't seem to access my map from my IP:5000/#.
I tried enabling sharing and still nothing. Anyone else run into these issues?
I was running the old pogo maps fine (the ones that "enabled sharing" and provided a URL), but I can't seem to get this one working outside my network.

You either need to port forward or use ngrok (https://ngrok.com/)

Worked perfectly. Thanks @Mickey32111!


README: correct link to blog post

Thanks for this contribution. Seems like the URL has changed over time.


Add coverage for base and db_check modules.

Also fix a few bugs related to db file passing and a minor CI tweak.

Codecov Report

Merging #47 (4de1682) into main (afcce5e) will increase coverage by 6.41%.
The diff coverage is 100.00%.

:exclamation: Current head 4de1682 differs from pull request most recent head 717e227. Consider uploading reports for the commit 717e227 to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main      #47      +/-   ##
==========================================
+ Coverage   90.51%   96.92%   +6.41%
==========================================
  Files          10       10
  Lines         601      586      -15
==========================================
+ Hits          544      568      +24
+ Misses         57       18      -39

Impacted Files
Coverage Δ

sss/base.py
100.00% <100.00%> (+28.12%)
:arrow_up:

sss/city.py
96.77% <100.00%> (-0.11%)
:arrow_down:

sss/db_check.py
100.00% <100.00%> (+40.00%)
:arrow_up:

sss/geoid.py
82.92% <100.00%> (-0.41%)
:arrow_down:

sss/puma.py
98.92% <100.00%> (-0.02%)
:arrow_down:

sss/report.py
100.00% <100.00%> (ø)

sss/sss_table.py
97.29% <100.00%> (-0.04%)
:arrow_down:

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

this looks great, thank you!


Can we get a trial_id in trial code?

What would you like to be added:
If we want to dump some models in trial code, we would like to know the trial_id in a different trial.
Does nni have any API could get trial_id in trial code?
Why is this needed:
Without this feature, how does current nni work：
Components that may involve changes:
Brief description of your proposal if any:

added in this pr https://github.com/microsoft/nni/pull/1353


[Suggestion] Add animations what can be seen in thirdperson or at other players.

I dont know is this possible to you . But what about  to add animations such running , wallrunning, jumping and more.

it's not possible with current codebase and I have no plans to do something about this


fixed typo

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

Correction confirmed. Thanks.


Bump nom to v7

Is it possible to apply this to a version 0.9.x? I don't want to figure out how to refactor my lettre 0.9 usage to 0.10. I currently have the following compiler warning:
warning: the following packages contain code that will be rejected by a future version of Rust: nom v4.2.3


NgFor with GridLayout just work for first element

From @davidjonz2005 on August 9, 2017 15:18
Did you verify this is a real problem by searching Stack Overflow and the other open issues in this repo?
yeah , there have been the same problem with "StackLayout" based on this link
Tell us about the problem
my code is below
<StackLayout>
  <ng-template ngFor let-index [ngForOf]="[1,2,3,4]">
    <GridLayout>
       <Label [text]=" 'This is Label' + index "></Label>
    </GridLayout>
  </ng-template>
</StackLayout>

yield only This is Label 1, where as
            "This is Label 1"
            "This is Label 2"
            "This is Label 3"
            "This is Label 4"

must be the result
Which platform(s) does your issue occur on?
Android,i dont know about ios
Please provide the following version numbers that your issue occurs with:

CLI: 3.2.0-2017-07-31-9530
Cross-platform modules<EMAIL_ADDRESS>Runtime(s): 3.2.0-next-2017-06-28-1901
Plugin(s):

"dependencies": {
		"@angular/animations": "~4.1.0",
		"@angular/common": "~4.1.0",
		"@angular/compiler": "~4.1.0",
		"@angular/core": "~4.1.0",
		"@angular/forms": "~4.1.0",
		"@angular/http": "~4.1.0",
		"@angular/platform-browser": "~4.1.0",
		"@angular/router": "~4.1.0",
		"nativescript-angular": "~3.1.0",
		"nativescript-telerik-ui": "^3.0.4",
		"nativescript-theme-core": "~1.0.2",
		"nativescript-toast": "^1.4.6",
		"reflect-metadata": "~0.1.8",
		"rxjs": "~5.3.0",
		"tns-core-modules": "~3.1.0",
		"zone.js": "~0.8.2"
	},
	"devDependencies": {
		"babel-traverse": "6.4.5",
		"babel-types": "6.4.5",
		"babylon": "6.4.5",
		"lazy": "1.0.11",
		"nativescript-dev-typescript": "~0.5.0",
		"typescript": "~2.2.1"
	}

Please tell us how to recreate the issue in as much detail as possible.
just run above code in simple
tns create yourapp --ng
Is there code involved? If so, please share the minimal amount of code needed to recreate the problem.
no
Copied from original issue: NativeScript/NativeScript#4680

Hi @davidjonz2005,
I reviewed your scenario with latest NativeScript CLI 3.1.3 and runtime 3.1.0 and the ngFor seems to work as expected. For your convenience, I am attaching a sample project. Also, bear in mind that you should specify the height of the GridLayout,
Archive.zip


improve connection migration

There's a comprehensive PR on connection migration for the draft (https://github.com/quicwg/base-drafts/pull/732/files). While we're not quite there yet (missing too many frame types, among others), there are two things we can improve over the current state:

[ ] enable connection migration for the server (currently only clients can migrate)
[ ] only do the connection migration if the packet causing the migration is not a reordered packet (i.e. has a higher packet number than the highest received so far)

Related: https://datatracker.ietf.org/meeting/100/materials/slides-100-quic-sessb-connection-migration/

Obsolete, since #963 was merged.


Add Failed Pod - Container Creating Test

Which problem is this PR solving?
Add e2e nrql tests for pod dashboard for container creating.
New Tests?
Please describe the new tests that were added (if applicable).

[x] This change requires changes in testing:

[x] e2e tests

Checklist:

[ ] Add changelog entry following the contributing guide
[x] Tests have been added
[ ] Documentation has been updated

LGTM. I've also ran this nrql tests locally and they pass.


Remove oneVPL sample source

Existing Sample Changes
Description
Remove oneVPL sample code, which will be stored in oneVPL dispatcher repo instead.  Following oneDNN pattern.
Fixes Issue# VPL7-1769
External Dependencies
None
Type of change
Please delete options that are not relevant. Add a 'X' to the one that is applicable.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Implement fixes for ONSAM Jiras

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x] Command Line
[x] oneapi-cli
[ ] Visual Studio
[ ] Eclipse IDE
[x] VSCode
[x] When compiling the compliler flag "-Wall -Wformat-security -Werror=format-security" was used

Hi Jeff,
Please submit a new PR to the development branch but to the oneapi-src:master.
Thanks
Praveen


Memoize count for AR when calling total_count - Increases performance for large datasets

Doing count on RDBMS can be painfully slow for larger datasets. Memoizing takes care of the problem.
Memoizing doesn't seem to have a negative effect since we should only care about how many records there were in the DB when we did the initial count query.

@sarmiena Thank you for the patch. I wasn't aware that it's possibly slow.
@Empact Lovely. Took your advice and pushed it as 0a36c1793b16dad403186650dfa83c7b8fc06c3d. Thanks!


[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: Hostname mismatch, certificate is not valid for 'www.quest.dcs.shef.ac.uk'. (_ssl.c:1147)

Could not get the file at http://www.quest.dcs.shef.ac.uk/wmt16_files_mmt/training.tar.gz. [RequestException] None.
vocab_src, vocab_tgt = show_example(load_vocab, args=[spacy_de, spacy_en])
how can i solve this problem ,please help me!

Exception: Could not get the file at http://www.quest.dcs.shef.ac.uk/wmt16_files_mmt/training.tar.gz. [RequestException] None. - I am experiencing same issue; how to resolve it? when running make html - get this error: SSLCertVerificationError for the above url


Create SpellCheck.yml

Coverage: 91.378%. Remained the same when pulling 0564d2d23ea1d7ed1c14e2360f87c79e71235834 on ranocha-patch-1 into f0b37d29e1fd9ae27b7acc97b38adc85a12960cd on main.


fix: resolves first failing bitcoin-core test

[x] resolves #247
[x] follows contribution guide
[ ] ~code change includes tests~

Earlier, the first test in the .tests/script_tests_failing.json was failing due to it receiving the wrong type of error message. the expected error was SCRIPT_ERR_SIG_DER but it got SIG_HASHTYPE error instead. This PR fixes that issue.

Hi @b-j-roberts, please review the PR

Have you tried running the core test script and make sure no new tests are failing? @SoarinSkySagar

you should run bash ./tests/run-core-tests.sh to update the script_tests_failing.json and script_tests_passing.json

Have you tried running the core test script and make sure no new tests are failing? @SoarinSkySagar

Yes, I made sure of that @bloomingpeach

done! @bloomingpeach

you should run bash ./tests/run-core-tests.sh to update the script_tests_failing.json and script_tests_passing.json

done @bloomingpeach!

Nice! You have fixed ten more tests, really good job!

LGTM, waiting for @b-j-roberts' review

@bloomingpeach Thank you for helping review!


New Package: JetBrains.*

List the PackageIdentifier of packages to be added to automation

Add all JetBrains Products to automation.

Mentioning @jedieaston since he mostly updates JetBrains products.

@jedieaston All packages (except a few like Rider, ReSharper which I will add when I get time) are now automatically updated. I will look at https://github.com/aaronparker/evergreen/ for more packages that can be added to automation.

Please do not replace the host of the URLs. The links on download.jetbrains.com can be automatically redirected to the optimal CDN, while those on download-cdn.jetbrains.com cannot.


Could not find or access '../yum/stop.yml'

Expected Behavior
The role be install successfully on centos/7
Actual Behavior
Keep getting the error:
fatal: [ci-server]: FAILED! => {"reason": "Unable to retrieve file contents\nCould not find or access '/{project_path}/yum/stop.yml'"}

Steps to Reproduce the Problem

Try to install on centos/7
set the var jenkins_install_via: "yum"

I'm pretty sure that https://github.com/emmetog/ansible-jenkins/pull/39 fixed this.


attribute stuff broke compilation on Windows

Last changes broke the Windows build. I added an #ifdef to the attribute stuff.

Thanks, I have only tested on Linux/OS X


PR into prereleased branch

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Test complete.


Score adds non-numeric values

Values from Department list gets added.
Any question with a choice that has "item000" in it will be added to the score.

Just tested the fix and it seems to work well. Merging to master.


[TASK] Improving Card Recognition Robustness

Description
Grabbing blank frames may cause the program to crash as it tries to perform a calculation or other function at non-existent data (e.g. an empty cv::Mat variable).
In several parts of the processing pipeline, checks have been implemented to catch potential errors (e.g. verifying that there are non empty arrays, or verifying that there are enough cards to detect).
Improvements can be made as in some areas this may have been omitted.

Added try/catch statements to avoid exceptions crashing the program.


Add init command to create default config file

Description
This PR is in context with Issue #84. Implemented a init command to create default config file for 1build.
Fixes/Implements: #84
New dependencies introduced?
Please list all decencies introduced in this PR if any with reason.

mock - In order to make the tests compatible with Travis which uses Python 2.7. Will remove this once the travis build starts executing on 3.7.

Checklist:

[x] My code follows the style guidelines – pep8 of this project
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have updated the changelog draft for corresponding release
[ ] I have update README (If needed)

Hello @Ritabrata-TW! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file onebuild/commands/perform_command.py:

Line 26:1: W293 blank line contains whitespace

Also, can we handle the case with a more specific error message:
command 1build --init
Current output
Traceback (most recent call last):
  File "1build", line 8, in <module>
    sys.exit(main())
  File "/Users/gopinathlangote/Tech/rito/1build/onebuild/main.py", line 25, in main
    run(BUILD_FILE_NAME, sys.argv[1:])
  File "/Users/gopinathlangote/Tech/rito/1build/onebuild/main.py", line 18, in run
    .execute(arg_parser, arguments, build_file_name, command_name)
  File "/Users/gopinathlangote/Tech/rito/1build/onebuild/commands/init_command.py", line 8, in execute
    write_default_config_file(arguments[1])
IndexError: list index out of range


Cannot start the example-webpack locally

I'm on Ubuntu 18.04.3 LTS
I've cloned the repository and am trying to run the example-webpack locally.
yarn install && yarn run start
Gets me this error:
yarn run v1.17.3
$ yarn webpack --mode production
$ webpack --config ./webpackConfig.js --color --progress --display-chunks --mode production
/home/nmummau/Code/docx-templates/node_modules/webpack-cli/bin/config-yargs.js:89
                                describe: optionsSchema.definitions.output.properties.path.description,
                                                                           ^

TypeError: Cannot read property 'properties' of undefined
    at module.exports (/home/nmummau/Code/docx-templates/node_modules/webpack-cli/bin/config-yargs.js:89:48)
    at /home/nmummau/Code/docx-templates/node_modules/webpack-cli/bin/webpack.js:60:27
    at Object.<anonymous> (/home/nmummau/Code/docx-templates/node_modules/webpack-cli/bin/webpack.js:515:3)
    at Module._compile (internal/modules/cjs/loader.js:778:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:789:10)
    at Module.load (internal/modules/cjs/loader.js:653:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:593:12)
    at Function.Module._load (internal/modules/cjs/loader.js:585:3)
    at Module.require (internal/modules/cjs/loader.js:692:17)
    at require (internal/modules/cjs/helpers.js:25:18)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

"webpack-cli": "^3.1.1"
in package.json

Thanks @weidtn ! That solves the problem. I've updated the example code to reflect this. Seems to be an issue in the Webpack ecosystem.


Pessimistic Locking produces ArgumentError

I get "ArgumentError - wrong number of arguments (1 for 0)" @ https://github.com/lynndylanhurley/devise_token_auth/blob/v0.1.32.beta5/app/controllers/devise_token_auth/concerns/set_user_by_token.rb#L57
Anyone has an idea why? When I comment out the lock everything works fine. I don't get this.
I had to use the beta because of https://github.com/lynndylanhurley/devise_token_auth/issues/134 and https://github.com/lynndylanhurley/devise_token_auth/issues/101

That's strange - what version of Rails are you using?

I'm sorry... it was some code written by another developer, which does "lock" accounts (using a method called "lock!"). I should have noticed that earlier.


Weird indentation in test output

$ prove -v
t/smoke.t ..
             1..1
                 # File: gzip.info,  Node: Top,  Next: Overview,  Up: (dir)
#
# GNU Gzip: General file (de)compression
# **************************************
#
# This manual is for GNU Gzip (version 1.12), and documents commands for
# compressing and decompressing data.
#
#    Copyright © 1998–1999, 2001–2002, 2006–2007, 2009–2022 Free Software
# Foundation, Inc.
ok 1
ok
All tests successful.
Files=1, Tests=1,  0 wallclock secs ( 0.02 usr  0.00 sys +  0.06 cusr  0.00 csys =  0.08 CPU)
Result: PASS

Why are lines 2-3 of the output indented so oddly?
This doesn't happen every time, and it seemingly never happens when run under strace -f. Is there's a race condition somewhere?

Fixed in 47cf0cb4d52a9810734c3aa0c7300679d8fefd58.


[Update] - add lite-sora for useful resource

Pull Request Description
Please read CONTRIBUTING manual before submitting your PR.
Title
[Update] - add lite-sora for useful resource
Summary
This pull request adds information about the project lite-sora to the README file. XXX is xxx video diffusion model for xxx task.
Changes

Added a section in the README with links to the lite-sora GitHub repository
Ensured that the links are functional and the information is up-to-date.

Review Notes

Please review the added content for accuracy and formatting.
Check that the links provided are correct and lead to the intended resources.
Please check if both the Chinese and English READMEs have been updated simultaneously.

Related Issues

Closes #333

Additional Comments

The addition of lite-sora to our README will provide valuable resources for users interested in efficient video generation.

I am so sorry that I have a wrong merge, and now everything is right


Script not rerunning

I have this npm script:
   "dev": "cross-env NODE_ENV=development npm run build:i18n:dev && vite",
I want this script to stop and rerun if my src/i18n dir has a file change:
I tried:
  "watch": {
    "dev": "{src/i18n}/*.ts"
  },

but when I change src/i18n/en.ts I don't get what I want

Seeing this too. Even when watching just one file. In my case it is a file in node_modules

Same behaviour for me, no error when running the watch script, but no rerun of my script

My bad, i finally made it work !

This tool uses nodemon under the hood and nodemon, by default, only triggers on  .js, .mjs, .coffee, .litcoffee, and .json.  My package.json configuration is this:
{
   ...
  "watch": {
    "run_dev": {
      "patterns": [
        "src",
        "public"
      ],
      "extensions": [
        "css",
        "d",
        "js",
        "json",
        "png",
        "svg",
        "ts"
      ]
    }
}

I got the list of extensions for my public and src directory like this:
ls src/* public/* | cut -d. -f2 | sort -u

Slightly out of date as to what the defaults of nodemon are, but please set the patterns and extensions keys in order to watch non-default file types

If you need to watch files with extensions other than those that nodemon watches by default (.js, .coffee, .litcoffee), you can set the value to an object with patterns and extensions keys. You can also add an ignore key (a list or a string) to ignore specific files. Finally, you can add a quiet flag to hide the script name in any output on stdout or stderr, or you can use the inherit flag to preserve the original's process stdout or stderr. You can enable nodemon legacy watch and specify the restart delay in milliseconds with the corresponding flags.


fix: update common sense link

PR Summary:
PR simply fixes the broken hyperlink to the Common Sense review.

Thanks @emmanuel-ferdman, great catch!


2.2 release checklist

[x] Add planned downtime message to the production website
[x] Test in all browsers
[x] Create new release, update version in footer and push master
[x] Add Github release
[x] apt-get update and upgrade
[x] Install Ruby 2.1
[x] Install sendmail
[x] Update the database
[x] Deploy
[x] Add 2.2 + changelog news post
[x] Add tag to google analytics
[x] Create public branch based on master
[x] Delete 2.2 feature branches
[x] Publish code in github.com #354
[x] Email Alessandro about the fixed colors
[x] Email Simon about the IL issue

Original issue by @bmesuere on Mon Dec 23 2013 at 15:26.
Closed by @bmesuere on Sat Jun 07 2014 at 16:28.

Install ruby 2.1:
sudo apt-get remove ruby1.9
curl -L https://get.rvm.io | bash -s stable
rvm install 2.1
gem install passenger
passenger-install-apache2-module

Original comment by @bmesuere on Sat May 03 2014 at 16:53.

Database update statements:
CREATE TABLE IF NOT EXISTS `unipept`.`users` (
  `id` INT NOT NULL AUTO_INCREMENT,
  `username` VARCHAR(8) NOT NULL,
  `admin` TINYINT NOT NULL DEFAULT 0,
  PRIMARY KEY (`id`))
ENGINE = InnoDB;

INSERT INTO `unipept`.`users` (`id`, `username`, `admin`) VALUES (1, 'bmesuere', 1);
INSERT INTO `unipept`.`users` (`id`, `username`, `admin`) VALUES (2, 'pdawyndt', 1);
INSERT INTO `unipept`.`users` (`id`, `username`, `admin`) VALUES (3, 'guest', 0);

INSERT INTO `unipept`.`lineages` (`taxon_id`) VALUES (1);

Original comment by @bmesuere on Tue Jun 03 2014 at 22:47.

changelog message:
Unipept 2.2 includes these updates:

added

Peptidome clustering

added

myGenomes to peptidome analysis

added

Fixed color option to the sunburst graph

added

The option to disable individual lines in the unique peptide finder

added

Full screen support for Internet Explorer 11

added

Links to the bioproject pages where appropriate

added

Invalidation "metagenome" NCBI taxa

improved

The design of the unique peptide finder: abbreviations for organisms names, use icons where possible, better help texts, ...

improved

The loading of the unique sequences in the unique peptide finder

fixed

A bug where the genome size was not shown by default

fixed

A bug causing the LCA to be root in some cases in the unique peptide finder

fixed

A bug where matching children weren't shown while filtering on the multi-peptide results page

fixed

A bug where the custom drag cursors weren't shown in Internet Explorer

fixed

A crash when using autosort without genomes

fixed

An issue with the full screen view of the peptide finder graph

fixed

An issue where the download dialog didn't disappear by pressing the escape key

updated

Rails to 4.1.1

updated

Ruby to 2.1

updated

Bootstrap to 3.1.1

updated

D3.js to 3.4.6

Original comment by @bmesuere on Wed Jun 04 2014 at 21:07.

There were some serious issues when trying to deploy to production:

The capistrano assets precompilation failed.
-> I created a custom recipe to do the precompilation
This created new issues with the bootstrap icon fonts: apparently rails 4 doesn't precompile assets with all filetypes anymore.
-> I added these filetypes to the list of assets to compile
This showed there were also issues with other assets: rails 4 stopped generating assets without digest in the filename. This caused the workers and fonts to 404.
-> I created a custom rake task to copy these files to the assets directory
I figured out the initial capistrano problem was due to the roles being parsed incorrectly (strings were not converted to symbols.
-> I fixed this and disabled my custom recipe
The default assets precompilation generated the assets in an incorrect location causing everything to fail again
-> I again disabled the default precompilation and used my recipe

After almost 10 hours of debugging, everything seems to work now.
Original comment by @bmesuere on Sat Jun 07 2014 at 16:09.


Follow code reuse format

I adapted a solution from a forum, just needed to change the way I acknowledge the source in the code to comply with the course's code reuse policy.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files with missing lines
Coverage Δ
Complexity Δ

src/main/java/seedu/address/ui/MainWindow.java
0.00% <ø> (ø)
0.00 <0.00> (ø)

... and 38 files with indirect coverage changes


Eden Litespot

Eden Litespot
Summary
My name is Tyrell Gerald. I plan to build lite helium hotspots mainly for the Canadian market at a reduced price and much faster and more reliable delivery. I understand the importance of credibility and experience and I know that all the other applications will be from big companies with years of formal experience I may not have all the experience as the other but I do have the drive and determination to provide a excellent product and to help grow the helium network and ensure that the Canadian market can play a big role in that. I don’t think that it’s right for the people who may have learned about helium a bit too late to not have the opportunity to be apart of something this spectacular and therefore If given the opportunity I will do everything in my power to make this a reality and help spread the word that is helium. Thank you in advance for the consideration and please let me know what additional information is needed
Company Information
Business network home automation data systems have been around computers and all electronics for years. I have built multiple computers and have put together functioning home automation systems.What brought me to the helium network is the idea of creating a more reliable coverage network. And rewarding its users for providing a service
Product Information
So just like all the other units on the market I will be using a sbc like the  Raspberry Pi and then will be adapting the Sx1302 LoRa card which will provide 10 programmable parallel demodulation paths, an 8 x 8 channel LoRa packet detector, 8 x SF5-SF12 LoRa demodulators and 8 x SF5-SF10 LoRa demodulators. It is capable of detecting an uninterrupted combination of packets at 8 different spreading factors and 10 channels with continuous demodulation of up to 16 packets. For software I will adapt everything to run on the helium app and We will provide ota updates. that way it’s directly on the network and we don’t have to worry about security risks my goal would be to produce as many units as required with the order numbers I would be able to have the units assembled and shipped within 3-5 days of receiving the order my goal price point would be 450-500$ cad until we can get the price for the components down and at that point 300$ a unit. I will provide prototype photos and videos at a later date as I would like to see if I have a chance before starting to build it
Customer Service
I will provide email customer support and have a solid amount of people ready to get to the customers and resolve their issues I have years of experience working in customer service and technical support for companies such as Telus bell mobility ect. for repairs and replacements I will have the customer send the unit back in to the headquarters inspect and repair and send them back out until it will be possible to have a location in every province where they will be able to bring the units in person and have them repaired on site or replaced if needed each unit will have a 1 year replacement warranty which will cover any hardware issue
Hardware security
The swarm keys will be stored on a secured EMMC storage card
Manufacturing Information
Have built data centres and multiple electronic systems from computers to smart home panels phone systems intercoms ect. I have personally created similar units years ago before helium was a thing and am very familiar with the technology.
Proof of Identity
To be provided in private message .
Budget & Capital
The plan is to fund the initial development personally and I am confident that the profits and pre orders will be able to more then cover any expense I will also plan to have a large amount of units already in stock and ready to be shipped out upon order I will ensure to update the availability of the units accordingly and will only allow pre orders of up to a week delay to avoid having customers pay for a product and not receive it for months
Risks & Challenges
The only challenges I see would be the shipping of the units but to address this issue I plan to use multiple shipping companies and providing local shipping service
Other information
Desired Discord support channel name -
Twitter profile -Eden.Litespot
Facebook profile --Eden.Litespot
Other social profiles --Eden.Litespot
Website --EdenLitespot.io or .com or .ca
Payment methods available - debit credit crypto
Regions covered / shipped to - Canada
Your Name
Tyrell Gerald, upon approval I plan to structure a formal company and I will be the ceo.

Hi Tyrell, we emailed you instructions on how to get this application into the right place.

Hi
Thank you so much I was able to put the application in the list I’m almost certain there’s going to be some information  missing or that I would need to elaborate on and the KYC really looking forward to working with you. All to get this application approved
Thank you again
Regards
Tyrell

On May 30, 2021, at 12:58 PM, Sophi Kravitz @.***> wrote:

Hi Tyrell, we emailed you instructions on how to get this application into the right place.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Thanks for your submission. The manufacturing review committee will be in touch next week.


Android does not continue loading scripts without Speedkit layer

On a reduced bandwidth connection, Android does not finish loading scripts unless you add the speedkit-layer component and manually click the "nuxt-speedkit-button-init-app" button.
Other devices/browsers I've tested (Desktop: Firefox/Chrome/Safari, iOS: Chrome, Safari) will finish loading all the scripts without the speedkit-layer being in the code.
Is this intentional for the Lighthouse tests?
These are my speed kit settings:
speedkit: {       detection: {         performance: true,         browserSupport: true,       },       performanceMetrics: {         device: {           hardwareConcurrency: { min: 2, max: 48 },           deviceMemory: { min: 2 },         },         timing: {           fcp: 1200,           dcl: 1200,         },       },
I'm using version 2.2.8
Any help is appreciated.

I can see this in the module code which looks like it's responsible for forcing the page to load if the SpeedKit layer isn't there.
Cannot see a reason why it's not working in Android though.
`window.addEventListener('load', function () {
if (!document.getElementById('nuxt-speedkit-layer')) {
initApp(forceInit);
} else {
observeSpeedkitButton('nuxt-speedkit-button-init-reduced-view', initReducedView);
observeSpeedkitButton('nuxt-speedkit-button-init-app', () => initApp(true));

setup(<%= options.performanceMetrics %>);

if(('__NUXT_SPEEDKIT_AUTO_INIT__' in window && window.__NUXT_SPEEDKIT_AUTO_INIT__) || ((<%= !options.ignorePerformance %> && hasSufficientPerformance()) && supportedBrowser)) {
  initApp();
} else {
  setupSpeedkitLayer(supportedBrowser)
}

}
});`

Hello @readio ,
sounds strange, do you have a device, which Android version with which browser?
Try to store the global variable window.__NUXT_SPEEDKIT_FORCE_INIT__ = true in the head, if set it should be initialized. But this does not solve the problem...
Do you have animations in the initial viewport, SVG with filter etc.?
Even without SpeedkitLayer it is checked whether the device is fast enough, i.e. a comparison is made between FPS and IDLE duration.
This is checked in a period of 1000ms and FPS and IDLE callback should not be too far apart.
The threshold value is 0.65, i.e. anything below this is blocked. Lighthouse falls under this.
You can adjust this threshold value in the Nuxt Config, but be careful, this is adjusted with lighthouse ;)
{
  speedkit: {
    runOptions: { maxTime: 1000, threshold: 0.65 }
  }
}

Basically, such a symptom can also be triggered purely by a CSS animation 🙃
To test, you could also reduce the FPS on the device itself. For example, Pixel 6 can run at 60 fps or 90 fps for smoother scrolling...

Hi @ThornWalli
I have tested this on a real device - Samsung S20 fe using Chrome. I've also tested using an emulator via Android Studio. I get the same results with both.
I tried setting window.__NUXT_SPEEDKIT_FORCE_INIT__ = true, but it appeared inside entry.mjs this is still undefined. I have tested console logging this in chrome dev tools and can see the variable is set correctly so not sure why this is happening.
I have hard-coded the value to true in entry.mjs , and this forces Android to continue loading without the SpeedkitLayer.
However, this impacts the Lighthouse mobile score - it drops by 16 points.
 Also, there is an auto-play video within the initial viewport.
I'll test some different values for the runOptions and removing elements etc.
I think my main confusion was why is this only an issue on Android devices.

Hi @ThornWalli, I have the same problem with an old Android gadgets (9, 10 version). I still haven’t figured out how to leave the boost from Nuxt-speed-kit and make the application work on old Android devices. Any ideas on how to fix this?


Collapse lanes horizontally

Difficult to scroll horizontally when there are a lot of lanes
In complex applications, we often have a large number of lanes, and sometimes it's error-prone that we drag and drop to another lane. Right now, we can collapse lanes vertically. But, It'll be better if we collapse lanes horizontally, as we can accomodate more lanes in the viewport

@harshaktg you can achieve that using CSS and some conditional rendering using a custom lane header, according to my requirement.

Yes @SaiPavanManda , we can achieve them. But it will be much easier if we had an additional prop in that component to achieve the same.


Update g-introduction-core.md

All I've done is remove the paragraph before 'Browser requirements' as I feel it more confuses than helps,

Resolved otherwise around the same - but thanks for the PR:-)
Please feel free to submit more (before you make changes, synch your main branch with the main branch here).


T5 conversion issue

Not sure where this issue belongs, but figured I'd put it here in case anyone else has the same issue
when running generate on a converted t5 model I got the following error:
AttributeError: 'Parameter' object has no attribute 'CB'
It turned out that T5ForConditionalGeneration.named_parameters() didn't iterate over lm_head, so I was able to fix it by changing
https://github.com/huggingface/transformers/blob/c8b6ae858d61e5bc10e388d095aa74f7690d1021/src/transformers/utils/bitsandbytes.py#L139-L142
    # otherwise they have an attached head
    list_modules = list(model.named_parameters())
    last_name = list_modules[-1][0]
    return last_name.split(".")[0]

to
return "lm_head"

not sure if it's just my version of torch or where to put the issue but env:
torch: 1.13.0a0+340c412
cuda: 11.7
bnb: 0.31.8
transformers: 4.22.0.dev0

Hi @chessgecko ,
Great catch ! This should be accessible when this PR gets merged: https://github.com/huggingface/transformers/pull/18579
If you want to already run it on colab I made a colab demo that we have planned to make it public next week but I'll share it here: https://colab.research.google.com/drive/1YORPWx4okIHXnjW7MSAidXN29mPVNT7F?usp=sharing
Hope this helps and thanks a lot!

Closing the issue as https://github.com/huggingface/transformers/pull/18579 has been merged

Should this work for all T5 models or just t5-3b-sharded?


Streaming: Convert crypto APIs to net/http

Follow-up from #6461
Converts the high-level crypto APIs to net/http, enabling the handling of data with streams which significantly reduces memory usage (although these APIs are still less efficient than the gRPC ones, as the data is still buffered in memory once)

Closing as per discussion on the call


Delete CODEOWNERS

Change-type: patch
Signed-off-by: Kyle Harding<EMAIL_ADDRESS>

@balena-ci I self-certify!


do not pass --with-intree-gmp to configure when system gmp is used

@snowleopard this PR fixes issue of ghc-cabal configuring integer-gmp with intree gmp even when user prefers to use system supplied gmp.

@kgardas :+1:
Some comments:

Let's stick to lowerCamelCase naming convention as in the rest of the build system: rename the variables to gmpIncludeDir and gmpLibDir.
import Settings.Builders.Common seems to be redundant?
I think this can be simplified as follows using operator ?:

    let includeGmp = "-I" ++ gmpBuildPath -/- "include"
    gmpIncludeDir <- getSetting GmpIncludeDir
    gmpLibDir     <- getSetting GmpLibDir
    mconcat [ builder GhcCabal ? mconcat
              [ (null gmpIncludeDir && null gmpLibDir) ?
                arg "--configure-option=--with-intree-gmp"
              , appendSub "--configure-option=CFLAGS" [includeGmp]
              , appendSub "--gcc-options"             [includeGmp] ]

            , builderGcc ? arg includeGmp ]

@snowleopard Interesting, if ? does what it looks so, then indeed. W.r.t names, will do. W.r.t. import, I'll verify this. Thanks for the review!

@kgardas Here is how ? works:
true  ? args == args
false ? args == mempty

where true and false can be of type Bool, Action Bool or Predicate. Note, Args is a Monoid, with mempty being equivalent to a no-op. Furthermore, arg appends a single argument to Args, append appends a list of arguments to Args, remove removes arguments from Args if they are present, etc.

@snowleopard thanks for explanation, it looks natural, but underlying tech is still over my head. Anyway, code should be fixed based on your review. Retested on Solaris.

Many thanks. It also passed all CI checks. Merged now.


webelement with type="checkbox" isdisplayed check error

The problem
when the element attribute type=checkbox, check element visible fail, but other type attribute is available
Details
Element '//input[@ah-sin-wipe="dnsServer" and @type="checkbox"]' not visible after 5 seconds.
call is_displayed() return false with chrome browser but the element is practical visible in web page, do the same test firefox is successful
Tell us about your environment

**Selenium standalone server Version: 3.11.0
**Chrome Version: Google Chrome 69.0.3497.81
**Browser Driver Version: ChromeDriver 2.43.600233
**Selenium Version: 3.14.0
**OS Version: Linux fendora28

Code To Reproduce Issue [ Good To Have ]
Please remember that with sample code it's easier to reproduce the bug.

@jyfang821 Can you please attach also html page or better part of page related to element in time of check?
It's common practice to hide input element for checkboxes and leave clickable only parent div of it.

@jyfang821 Can you please attach also html page or better part of page related to element in time of check?

 			<input name="services-field-1" class="ace-switch ace-switch-7" type="checkbox" data-dojo-attach-point="enabled" ah-sin-wipe="dnsServer">
Use DNS server to resolve domain names to IP addresses. Aerohive devices that are DHCP clients can receive a domain name and DNS server IP address through DHCP, although any DNS settings that you enter here override those that are dynamically applied.

It's common practice to hide input element for checkboxes and leave clickable only parent div of it.

@jyfang821 Can you please also provide css style for this element?

@jyfang821 Based on your provided code user click on the span, that is bootstrap switch and input is hidden by css.

@jyfang821 Can you please show your code? Which method do you call? Selenide hasn't method is_displayed().

closing due to inactivity


set testEnvironment: node

I was getting inconsistent test breakages on my machine. Not sure why, perhaps a pkg version mismatch. This seems to fix it

Can you try a clean dependency install?
It hasn't failed for me or on CI so it might be something on your end.
It shouldn't be necessary to add that because we're using<EMAIL_ADDRESS>which defaults to using a Node.js environment.
From their blogpost:

For this reason, we are changing the default test environment from "jsdom" to "node".

Ah, didn't realise it had been updated. I blew out node_modules and reinstalled, seems to have fixed, thanks!


First tranche of clean up

First set of clean up work.

Much of the other unresolved clean up content is covered in the general editing issue

There are still some issues (please see the remaining comments), but they can be addressed in a separate pull request.


Namespace per block.

RFC: https://github.com/linkedin/css-blocks/issues/332
CSS Changes

[x] The class selector will now be defined as selecting an attribute named class in the namespace of the block. (conceptual change, not requiring any stylesheet parsing changes at this time).
[x] The default namespace for the attribute selector will be dom attributes in the namespace of the block. Therefore the state| namespace in the state attribute selector is no longer needed.
[x] Global states are accessed via the attribute namespace instead of a fake tag name.
[ ] The :scope selector will select an element with the attribute of scope that is in the namespace of the block.
[x] States named "scope" will no longer be allowed.
[x] non-block namespaces will be handled by registering common names of html, svg, and math and making them forbidden names for blocks.
[ ] Other non-block namespaces can be registered through configuration.

Handlebars Template Syntax Changes

[ ] The namespace identifier for the default block associated with the template is block.
[ ] If no element has an attribute of block:scope then it is automatically assigned to the template's root element.
[ ] Instead of class="foo other-block.bar" we would now write block:class="foo" other-block:class="bar".
[ ] The class attribute in html namespace can be forbidden or allowed based on configuration. If allowed, those classes can be used for styling with selectors defined outside of the scope of css blocks.

Documentation Changes

[ ] Document namespace concept in tutorials/readmes.
[ ] Website syntax updates.

One more item to add to the TODO list: Make the word block a reserved word (if it's not already... I don't think it is).
Because you reference the "default" exported block as block, the compiler should throw if you try to @export (whatever as block ) from "wherever.block.css".

Alt: You can just change the keyword default to block for the import/export parser.
This may be confusing for those coming from JavaScript world though – would not recommend.


No longer showing all classes covered.

Having just run a full set of unit tests I get this:

despite the fact that .sfdx/tools/testresults/apex/test-result-7071k00001sGoEH-codecoverage.json contains the full results
test-result-7071k00001sGoEH-codecoverage.txt

that file was renamed to .txt so I could upload it to github.

Hey @codefriar I'm about to release a new version of this extension and was looking at the pending issues. I know this issue was reported long time ago, and I think I solved this issue on one of the releases. Are you still experiencing this error? Thanks


Move metadata fields to the database

We should create a database table that has the following fields:
photo_hash (this is the foreign key)
aperture
datetime
exposure
flash
focal_length
gps
height
iso
lens
make
model
orientation
width
Once this is done we wont need the sidecars for anything until we start adding editing features.

Some metadata should be user editable; I still like the idea of sidecars for this (eg. title, copyright, etc.).
Although, for the purpose of searchability it definitely makes sense to have it in the DB too. Tricky one.


want to add the diff between customer and visitor and if any customer and has made for it's payment or not

please assign this to me i can implement it both frontend and backend

please assign this to me i can implement it both frontend and backend

Hello @sakshamgumber , Go for it
All the Best

Can you please assign this issue to me if nobody is working on it? @Trisha-tech Please add vsoc24 labels too so I can get it counted


SyntaxArena: thread safety

Use mutex to guard concurrent mutation to the arena.
But make a "single thread" mode so that we don't regress performance too much when we know the arena is not used by other threads.
(non Darwin/Glibc environment e.g. Windows implementation is currently a dummy)
https://github.com/apple/swift-syntax/issues/785
rdar://99812330

@swift-ci Please test

@swift-ci Please test

Thank you, this certainly helps, although TSan now reports an access race in the guard code:

WARNING: ThreadSanitizer: Swift access race (pid=42514)
  Modifying access of Swift variable at 0x0001148c22b0 by thread T25:
    #0 ScopeGuard.withGuard<A>(body:) <null>:82026728 (SwiftLintFrameworkTests:arm64+0x126a334)
    #1 SyntaxArena.withGuard<A>(body:) <null>:82026728 (SwiftLintFrameworkTests:arm64+0x126b504)
    #2 SyntaxArena.allocateRawSyntaxBuffer(count:) <null>:82026728 (SwiftLintFrameworkTests:arm64+0x126c2fc)
    #3 static RawSyntax.makeLayout(kind:uninitializedCount:arena:initializingWith:) <null>:82026728 (SwiftLintFrameworkTests:arm64+0xffcd14)
    #4 RawSyntaxLayoutView.replacingChild(at:with:arena:) <null>:82026728 (SwiftLintFrameworkTests:arm64+0x10036c8)
    #5 SyntaxData.replacingChild(_:at:) <null>:82026728 (SwiftLintFrameworkTests:arm64+0x1285864)

Full report: https://buildkite.com/swiftlint/swiftlint/builds/3411#018354b5-95eb-4c14-ab21-f7d59113f241

@jpsim I confirmed the failure locally. And the modified implementation resolves it. Could you check the new implementation?

@swift-ci Please test


The "log" function, is not log in base 10

I think that '' log '' does not work well, because when I use it it does the same as "ln" that is;
ln (6) = 1.791759469228055
log (6) = 1.791759469228055
This should not be the case, if ln is logarithm to base e that is fine, but log I understand that it is logarithm to base 10 and therefore it should be;
ln (6) = 1.791759469228055
log (6) = 0.77815125038364363250876679797961
Therefore "ln" works perfectly but "log" does not.
Maybe I do not use it well but I am using it as I put in the example and that way it works well for me in all calculators.
If I am not using it well I would like you to explain how to do it.

Thanks for your interest in this library!
I am aware of (and personally like) the convention to use log for base-10 logarithms and ln for base-e logarithms that you're referring to, but the more general convention is to use log to mean base-e logarithms. Many math libraries — including dart:math — use log to mean the base-e logarithm, which is why I stuck to this convention. I only included support for ln because it is a common alias for log.
If you'd like to create a base-10 log function-tree, use the two-parameter function log(base, x) pattern. For example:
final myLog10 = 'log(10, x)'.toSingleVariableFunction();

I hope that helps!

Thanks, that helps a lot.
The problem I see is that I am Spanish and from what I see in other countries it is said differently, because I am Spanish and here "log" refers to a logarithm in base 10 and "ln" to a logarithm in base e, no They are the same. Also tell you thanks for the example, but the problem I have is that I want to use it that way inside a string and then use the .interpret () function, would that be possible in any way?

I have tried that example, which you tell me and it does not let me put the variable x, that is, if I put:
final myLog10 = 'log (10, x)'. toSingleVariableFunction ();
print (myLog10 (10));
the result of print is:
Exception caught by gesture ═══════════════════════════════════════════
The following _Exception was thrown while handling a gesture:
Exception: Bad expression: '10x' ...
When the exception was thrown, this was the stack
0 _parseString
package: calculateme_app /… / src / interpreter.dart: 198
1 _parseString
package: calculateme_app /… / src / interpreter.dart: 92
2 _parseString
package: calculateme_app /… / src / interpreter.dart: 106
3 new SingleVariableFunction
package: calculateme_app /… / src / trees.dart: 54
4 StringMethods.toSingleVariableFunction
package: calculateme_app /… / src / extensions.dart: 22
...
Handler: "onTap"
Recognizer: TapGestureRecognizer # 7a53a
debugOwner: GestureDetector
state: possible
won sand
finalPosition: Offset (250.0, 763.0)
finalLocalPosition: Offset (45.8, 33.5)
button: 1
sent tap down
══════════════════════════════════════════════════ ════════════════

Hello again, I've been looking at your package, and I've found the _functionMap, if I'm not mistaken that's where you say what "log" refers to within the string when using the .interpret () function.
I mean this map:
final Map <String, Function> _functionMap = {
  'abs': (num x) => x.abs (),
  'accos': accos
  'asin': asin,
  'atan': atan,
  'ceil': (num x) => x.ceil (),
  'cos': cos,
  'cosh': (num x) => (pow (e, x) + pow (e, -x)) / 2,
  'cot': (num x) => 1 / tan (x),
  'coth': (num x) => (pow (e, x) + pow (e, -x)) / (pow (e, x) - pow (e, -x)),
  'csc': (num x) => 1 / sin (x),
  'csch': (num x) => 2 / (pow (e, x) - pow (e, -x)),
  'exp': exp,
  'floor': (num x) => x.floor (),
  'ln': log,
  'log': log,
  'round': (num x) => x.round (),
  'sec': (num x) => 1 / cos (x),
  'sech': (num x) => 2 / (pow (e, x) + pow (e, -x)),
  'sin': sin,
  'sinh': (num x) => (pow (e, x) - pow (e, -x)) / 2,
  'sqrt': sqrt,
  'so so,
  'tanh': (num x) => (pow (e, x) - pow (e, -x)) / (pow (e, x) + pow (e, -x))
};

I know that to get the logarithm in any base it can be done like this;
log (x) / log (y)
where x = the number of which you want to take the logarithm
and y = the base of the logarithm, in this case 10 which is what I want.
I have tried changing the map where "log" is defined on my own like this:
'ln': log,
  'log': (num x) => log (x) / log (10),
  'round': (num x) => x.round (),

but it has not worked for me, the result is the same as if I put
print ('log (6)'. interpret ()); // 1.791759469228055
I have even tried this:
'ln': log,
  'log': log,
  'log10': (num x) => log (x) / log (10),
  'round': (num x) => x.round (),

but it doesn't work either.
I know that something like this could be added to make it work well, but since I don't know exactly how the package is created I can't help with that. Even so I wish you could add it, that would be very helpful for me, as well as make the package even more complete than it already is.

Hi Hekhy,
I won't say anything about your style choice to put spaces before function parentheses. :)
It's strange: I can't reproduce your error. When I copy-and-paste your code,
final myLog10 = 'log (10, x)'. toSingleVariableFunction ();
print (myLog10 (10));

my output is:
1.0

Also, you're right about the organization of the function-mapping in defs.dart. If I change the map in defs.dart to:
.
.
.
  'ln': log,
  'log': (num x) => log (x) / log (10),
  'round': (num x) => x.round (),
.
.
.

then log behaves like a base-10 log function, as you expected. For example,
print('log(10)'.interpret());

outputs
1.0

It must be something on your end. From the error messages, it looks as though the comma is possibly being lost somehow: is your app manipulating the string before feeding it to the function-tree interpreter maybe?

Hi @ram6ler ,
Do not take into account the spaces before the parentheses, I do not do that like that, but when I translate my answer in the google translator automatically he adds spaces, but I need to translate it because I don't know much English.
Beyond that, I don't know what could be happening when I put this code:
final myVariable = 'log(10, 10)'.toSingleVariableFunction();
print(myVariable);

and my output is is an error that says this: Instance of 'SingleVariableFunction'
I don't know what could be happening but it doesn't work well for me ...
And if I put this other code:
final myVariable = 'log(10, 10)'.interpret();
print(myVariable);

my output is: 6,917705609835305
Which is also wrong ...
I do not know what I'm doing wrong.


Add /WX (warnings as errors) compile flag when compiling with MSVC.

This PR fixes #922. When compiling with MSVC (Microsoft Visual C++) warnings are now treated as errors for all exercises.

@vaeng, I created this PR for #922.

I assume you replaced the lines for all exercises and did not add anything else? I scrolled through the changes and read your explanation in the issue. Thanks for the write-up that helps a lot to speed things up.
I would love to have a single space for those files and have each exercise pull from that folder, but currently, we have to change it everywhere.
It is a good change, if all tests pass, we should merge this.


✨ decouple kcp and switch to space framework in the container scripts

Summary
Decouple kcp and introduce space framework to the core container scripts.
Related issue(s)
Fixes #

/hold

Replaced by PR #1218


V2 Image Duplication

Description
I initially mentioned this in #360 but this needed to be broken out into a separate issue. If you save an image in n sheets, each sheet will have n copies of that image in the cell.

Steps to reproduce the issue:

Add an image to multiple sheets and save it
Try moving the image in the saved document

Describe the results you received:
The image is duplicated (stacked on top of itself in place)
Describe the results you expected:
The image should be there only once
Output of go version:
go version go1.12 darwin/amd64

Excelize version or commit ID:
b2c12d784e94bfb14da800962c769f5a0f6f783e

Here is a gist with a small program to reproduce this: https://gist.github.com/mlh758/1141d5fd521b4e2aed23c7e920df5ac8
You'll need a logo.png and have to update the replace directive in go.mod to wherever you have this library saved on your system.

Thanks for the quick fix. I pulled this down and verified it fixed the issue.