Find and Replace Window

Hi,
I was trying to migrate v2 to v3 and noticed "find and replaced" window is not implemented in v3. I understand that is simple to do, but I was wondering if you have any plan migrating that window to v3 in any future soon?
Regards,

See Issue #7.


Question: Future of react-calendar-timeline

Hi
What is the plan with this library?
Will there be releases in the future?

Hi,
There will be future releases. As for a plan, there is no clear plan for now.


Add damage type and instigator to death screen

Contains
This cleans up the changes made in #2736, which adds the damage type and instigator that caused the death to the death screen. The data is retrieved from DoDestroyEvent, so to ensure that accurate death information is displayed in the death screen in the future, different classes/modules have to specify the correct instigator and damage type there.
How to test
You can kill the character directly through the kill command or through fall damage with the hjump command.

Hooray Jenkins reported success with all tests good!

Thanks @iojw and @VaibhavBajaj :-)
After some workspace cleanup I could validate this in single player or as host+client in multiplayer, but I crash on a pure client in multiplayer with the following error:
23:51:59.737 [main] INFO  o.t.network.internal.ServerImpl - Destroying entity: EntityRef{id = 12724, netId = 1280, prefab = 'Potions:PoisonPotion'}
23:51:59.741 [main] ERROR o.t.e.event.internal.EventSystemImpl - Failed to invoke event
java.lang.NullPointerException: null
	at org.terasology.logic.players.MenuControlSystem.loadDeathDetails(MenuControlSystem.java:89)
	at org.terasology.logic.players.MenuControlSystemMethodAccess.invoke(Unknown Source)
	at org.terasology.entitySystem.event.internal.EventSystemImpl$ByteCodeEventHandlerInfo.invoke(EventSystemImpl.java:506)
	at org.terasology.entitySystem.event.internal.EventSystemImpl.sendStandardEvent(EventSystemImpl.java:269)
	at org.terasology.entitySystem.event.internal.EventSystemImpl.send(EventSystemImpl.java:260)
	at org.terasology.entitySystem.entity.internal.BaseEntityRef.send(BaseEntityRef.java:145)
	at org.terasology.network.internal.ServerImpl.processEvent(ServerImpl.java:244)
	at org.terasology.network.internal.ServerImpl.processMessages(ServerImpl.java:276)
	at org.terasology.network.internal.ServerImpl.update(ServerImpl.java:187)
	at org.terasology.network.internal.NetworkSystemImpl.update(NetworkSystemImpl.java:317)
	at org.terasology.engine.subsystem.common.NetworkSubsystem.preUpdate(NetworkSubsystem.java:46)
	at org.terasology.engine.TerasologyEngine.mainLoop(TerasologyEngine.java:406)
	at org.terasology.engine.TerasologyEngine.run(TerasologyEngine.java:368)
	at org.terasology.engine.Terasology.main(Terasology.java:152)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at com.intellij.rt.execution.application.AppMain.main(AppMain.java:147)

It seems that nuiManager.getScreen("engine:deathScreen") on the pure client might be resolving as null - or my quick debugging session didn't work right.
Either of you able to confirm and take a look if it seems a valid problem?

Note to self / when somebody merges this: I've been meaning to enter a bite-sized issue for getting a console command that does damage of type x to the player (for debugging stuff like this exact sort of functionality, or testing resist potions). Figured it would be best to do so after this is merged.
Oddly I could've sworn I already submitted such an issue, but I can't readily find it. So .. odd.
Also: @VaibhavBajaj & @iojw ping! :-) Want to merge this already!

Hooray Jenkins reported success with all tests good!

Hooray Jenkins reported success with all tests good!

Sorry this took a while, I had to go through a few iterations to fix the issue with pure clients.
The issue as a pure client is that all the damage and death logic is handled on the server, with the character entity on the server being linked to the server client entity. However, the death screen is displayed by the local client entity, and this local client entity has no reference to the character entity on the server at all. As I had initially stored the details in a component on the character for replication, the local client entity could not retrieve the relevant details to display on the death screen locally.
Thankfully I realised I could just store the details in the event itself - finally works and it's pretty short too. This should fix all the issues.

Hooray Jenkins reported success with all tests good!

ping @iojw, still around on this one? I think we are 1 or 2 iterations away from merging 👍

@oniatus: yup, will get to it in a few days!

Hooray Jenkins reported success with all tests good!

Hooray Jenkins reported success with all tests good!

Hooray Jenkins reported success with all tests good!

Hooray Jenkins reported success with all tests good!

Thanks @oniatus! Ping to @Cervator to maybe add the issue regarding the console command to inflict damage of a specific type since this has finally been merged.


Change the default cursor

I've tried to change the default cursor shape by editing the config.h file as
follows:
/*
 * Default shape of cursor
 * 2: Block ("█")
 * 4: Underline ("_")
 * 6: Bar ("|")
 * 7: Snowman ("☃")
 */
static unsigned int cursorshape = 6;

Then, I've ran make && sudo make install but seems like the cursor didn't change as expected.
Is there something I should watch out for?

Shell: zsh

Probably your shell overwriting it. What does your .zshrc look like?

It worked for me https://github.com/LukeSmithxyz/st/issues/121, because I don't do make install, but:
$ make
$ ln -s /opt/stterm/st  /usr/local/bin/st

So any new changes always point to my newest binary.
Maybe you need to check:

run ./st inside your newest compilation, if this work that means all the make install process links still point to the old binary.
run st without any of your "additional" bash config. as @decantr suggested

@decantr @azzamsa
It really seems like the shell is overwriting it.
After running make install, I've tried
# Does not read .zshrc
$ st -e zsh -f

Now the cursor has changed appropriately.
Thanks for the suggestions!
Now I know where to look into for the solution


Feat/start materialize

Codecov Report

Merging #5 (64bb3d8) into main (7dc5343) will decrease coverage by 75.00%.
The diff coverage is 0.00%.

:exclamation: Current head 64bb3d8 differs from pull request most recent head 76563ea. Consider uploading reports for the commit 76563ea to get more accurate results

@@             Coverage Diff              @@
##              main       #5       +/-   ##
============================================
- Coverage   100.00%   25.00%   -75.00%
============================================
  Files            4        7        +3
  Lines            8       32       +24
============================================
  Hits             8        8
- Misses           0       24       +24

Impacted Files
Coverage Δ

lib/blog/posts.ex
0.00% <0.00%> (ø)

lib/blog/posts/post.ex
0.00% <0.00%> (ø)

lib/blog_web/controllers/post_controller.ex
0.00% <0.00%> (ø)

lib/blog_web/router.ex
100.00% <ø> (ø)

Continue to review full report at Codecov.

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


How to access Embedding Tables?

I study DLRM and embedding table features on pytorch environment and using kaggle dataset.
I can't solve some questions by myself.
so I ask here.
First, (in create_emb and apply_emb method.)
I found EmbeddingBag(n,m,mode="sum") creates [nxm] matrices.
and EE, emb_l store EmbeddingBag modules in create_emb.
After making tables, apply_emb use them(emb_l ..) by this code
E = emb_l[k] , V = E(index, offset,per_sample_weights)
I found offset is a list which is consist of batchsize (0~batch-1).
What does index mean? Is this just random value?
and What is the n,m meaning?
Pytorch explains n is the number of embedding and m is an embedding dimension.
But, I heard that the embedding table is composed of user IDs and Items. and It is factorized separately.
n is user ID, m is item. right? I'm confused.
Second,
I want to match the concepts between papers and codes. so I just extract the data with print().
I can't find sparse indices on dlrm_s_pytorch.py
Sparse indices for accessing to embedding tables are made by dlrm_data_pytorch.py?
I'm not good at English. So, If you have the unclear sentences, please ask me. then I'll edit it by using translator.
Thanks.

What does index mean? Is this just random value?

Each index represents a categorical value. Before training, all of the unique categorical values are given an index. You can find this preprocessing step in the code here:https://github.com/facebookresearch/dlrm/blob/1302c71624fa9dbe7f0c75fea719d5e58d33e059/data_utils.py#L876

What is the n,m meaning?

N is the number of rows in the table. Each categorical value is assigned to a row. For example we could have the categorical values:
{Red, Green, Blue}
Which would give us N = 3. (Often in practice we make N < number of unique categorical values. We can use the hashing trick to decrease table size)
M is the dimension of an individual embedding. The embedding table learns M-dimensional vectors for each categorical value. Increasing M can help increase model quality.
In this model user ids and item ids would each get their own table with their own embeddings. To make a prediction for a <user, item> pair we look up the embedding for the user and the embedding for the item.

Sparse indices for accessing to embedding tables are made by dlrm_data_pytorch.py?

Yep. They are read from whatever dataset you're using. For Criteo-Kaggle they are produced here: https://github.com/facebookresearch/dlrm/blob/1302c71624fa9dbe7f0c75fea719d5e58d33e059/dlrm_data_pytorch.py#L284

What does index mean? Is this just random value?

Each index represents a categorical value. Before training, all of the unique categorical values are given an index. You can find this preprocessing step in the code here:https://github.com/facebookresearch/dlrm/blob/1302c71624fa9dbe7f0c75fea719d5e58d33e059/data_utils.py#L876

What is the n,m meaning?

N is the number of rows in the table. Each categorical value is assigned to a row. For example we could have the categorical values:
{Red, Green, Blue}
Which would give us N = 3. (Often in practice we make N < number of unique categorical values. We can use the hashing trick to decrease table size)
M is the dimension of an individual embedding. The embedding table learns M-dimensional vectors for each categorical value. Increasing M can help increase model quality.
In this model user ids and item ids would each get their own table with their own embeddings. To make a prediction for a <user, item> pair we look up the embedding for the user and the embedding for the item.

Sparse indices for accessing to embedding tables are made by dlrm_data_pytorch.py?

Yep. They are read from whatever dataset you're using. For Criteo-Kaggle they are produced here: https://github.com/facebookresearch/dlrm/blob/1302c71624fa9dbe7f0c75fea719d5e58d33e059/dlrm_data_pytorch.py#L284

Thank you for your quick reply. It's helpful to me. :)
More questions.
Please check this flow I've understood is correct
1.Datas are preprocessed by data_utils.py and dlrm_data_pytorch.py
(It makes input data. e.g categorical, continuous features, sparse indices from source-train.txt)

2. Dense features are processed by bottom MLP (create_mlp, apply_mlp)
Embedding Tables are made by dlrm_s_pytorch.py using sparse indices, categorical features. (create_emb)

3. Access to each embedding table with user_id_index, item_id_index (apply_emb)

4. Concatenate in interaction layer and get CTR using top_mlp

If this is correct, 3 is difficult to me.
Code says
E = emb_l[k] = EmbeddingBag(n,m,...) (k : 0~25)
V = E(sparse_index,sparse_offsets)
The number of sparse_index is 26.(they are consisted of various integer numbers),(continuous feature : 13)
Half of data is about users and the other half is about items?
Does those are preprocessed by 1, used in 3 and need more operations to get complete embedding vector?
If E=emb_l[1] is EmbeddingBag(1460,16),    (EmbeddingBag is consist of 1460 data {red,green,purple ...} )
V = E(ind,off) is getting EmbeddingBag(1460,16) data? (getting purple embedding vector)
then ind is index about user or item that i want to access?

Thank you for your quick reply. It's helpful to me. :)
More questions.
Please check this flow I've understood is correct
1.Datas are preprocessed by data_utils.py and dlrm_data_pytorch.py
(It makes input data. e.g categorical, continuous features, sparse indices from source-train.txt)

2. Dense features are processed by bottom MLP (create_mlp, apply_mlp)
Embedding Tables are made by dlrm_s_pytorch.py using sparse indices, categorical features. (create_emb)

3. Access to each embedding table with user_id_index, item_id_index (apply_emb)

4. Concatenate in interaction layer and get CTR using top_mlp

If this is correct, 3 is difficult to me.
Code says
E = emb_l[k] = EmbeddingBag(n,m,...) (k : 0~25)
V = E(sparse_index,sparse_offsets)
The number of sparse_index is 26.(they are consisted of various integer numbers),(continuous feature : 13)
Half of data is about users and the other half is about items?
Does those are preprocessed by 1, used in 3 and need more operations to get complete embedding vector?
If E=emb_l[1] is EmbeddingBag(1460,16),    (EmbeddingBag is consist of 1460 data {red,green,purple ...} )
V = E(ind,off) is getting EmbeddingBag(1460,16) data? (getting purple embedding vector)
then ind is index about user or item that i want to access?

Half of data is about users and the other half is about items?

The split does not need to be 50/50. This model supports many categorical features. The Criteo dataset is unlabelled, but if you take a look at the Avazu dataset you can get an idea of what those categorical features may be.
For example we could have an embedding table for each of the following categorical variables:
User ID
Favorite color,
Location
Item type
...
you can imagine many more categorical features that would describe either the user or context related to the prediction

then ind is index about user or item that i want to access?

The index is for the given categorical feature. In your example this would be "color"
Chapter 8 of this book may be helpful.

Half of data is about users and the other half is about items?

The split does not need to be 50/50. This model supports many categorical features. The Criteo dataset is unlabelled, but if you take a look at the Avazu dataset you can get an idea of what those categorical features may be.
For example we could have an embedding table for each of the following categorical variables:
User ID
Favorite color,
Location
Item type
...
you can imagine many more categorical features that would describe either the user or context related to the prediction

then ind is index about user or item that i want to access?

The index is for the given categorical feature. In your example this would be "color"
Chapter 8 of this book may be helpful.

Thank you for your detailed reply!
I think i need to learn more
If I have any other questions, I will open this issue again. :)

Thank you for your detailed reply!
I think i need to learn more
If I have any other questions, I will open this issue again. :)


Support for Debian Linux bullseye

I tried to run this Tmux setting on Debian Linux Bullseye but with no success.
Could you or someone clarify for me what I need to do for it to work?

Hello @jean-bonilha 👋
I'm not sure what it means when you say "with no success".
I just tried using Oh my tmux! on a fresh Debian Bullseye box and it works fine for me.

Hello @jean-bonilha 👋
Any news?


New package: eQuilibrator v0.2.0

Registering package: eQuilibrator
Repository: https://github.com/stelmo/eQuilibrator.jl
Created by: @stelmo
Version: v0.2.0
Commit: 767de0e31184d22f8238efd0979470472785cec4
Reviewed by: @stelmo
Reference: https://github.com/stelmo/eQuilibrator.jl/commit/767de0e31184d22f8238efd0979470472785cec4#commitcomment-51518941

Hi, it seems my package is does not meet the requirements for auto-merging due to its name (eQuilibrator.jl). However, this package wraps an external library that uses the non-standard name format: eQuilibrator (see here: https://equilibrator.weizmann.ac.il/). The packaging guidelines indicates that this should be okay. If so, could you please manually merge my package? Thanks!

Hi @giordano,
Could you manually merge my package into the registry please? With regard to the naming issue: the "original" package this is an interface to has the strange spelling, hence my decision to pick this name...
Ps. if you are not the right person to ask about this, please let me know :)


[LevelUP - Advanced 5th Edition} Minor Bug fixes and some sheet improvements

Submission Checklist
Required

[Yes] The pull request title clearly contains the name of the sheet I am editing.
[Yes] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[Yes] The pull request makes changes to files in only one sub-folder.
[Yes] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: <   >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: <   > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <   > (i.e. Dungeons & Dragons, FATE)

[ ] I have followed the Character Sheets Standards when building this sheet.

[ ] This sheet has been made on behalf of, or by, the game's publisher.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

Added new Dex(Basic) initiative option
Added Bloodied HP on to the NPC sheet
Added all the missing words into the translate
Added Standard / Advantage / Disadvantage logic for initiative, saves, stat rolls

Character Sheet Info Roll20 Internal Use only.


claim_list should return uniform data structure

claim_list currently returns a different data signature based on what type of claim it is (channel vs. stream).
claim_xxx API functions promise that they return claims. Claims must have a singular, uniform definition. Fields that are not shared across claim types should either be returned as null or these values should be in a sub-property (e.g. claims could contain stream and channel sub-properties that contain either null or a map of type-specific properties).
Internal Use
Acceptance Criteria

Definition of Done

[ ]  Tested against acceptance criteria
[ ] Tested against the assumptions of the user story
[ ] The project builds without errors
[ ] Unit tests are written and passing
[ ] Tests on devices/browsers listed in the issue have passed
[ ] QA performed & issues resolved
[ ] Refactoring completed
[ ] Any configuration or build changes documented
[ ] Documentation updated
[ ] Peer Code Review performed

All of the local commands (which query sqlite db) return uniform data structure for tx and txos. Only exception right now is data coming from lbryumx. Re-assigning this to @shyba for further consideration.

If I understood it correctly, this relates closely to #1483 but I think it has a broader scope. The issue with missing claim data is being handled at 1483, but maybe we can say this issue is to make the response of claim_xxx commands fixed on a template right?

It's consistent between channels/claims now, but not consistent with other api commands that hit the wallet server. This will be covered by https://github.com/lbryio/lbry-sdk/issues/2259


Declare a capability namespace that indicates a bundle contains annotations

PDE / Tycho have a quite convenient feature that makes OSGi annotations automatically available to the user, that way no special configuration is required to use them.
This is currently archived by maintaining a fixed set of known items what of course not works for custom ones or needs to be extend once there are new items specified.
Instead of this, it would be good to have a "annotation capability" a bundle can define that includes attributes / directives of:

the package names that contain annotations
the retention of these annotations (e.g. class or runtime)

based on this an IDE/Tool can scan the target/repositories/... for items it wants to make available automatically to the user.
To make this work across tools / IDEs there should be a standard namespace defined to allow wider adoption (and maybe even predefined annotations).

Sorry @laeubi, I did not get it. Could you help me with a user story.

based on this an IDE/Tool can scan the target/repositories/... for items it wants to make available automatically to the user.

your branch https://github.com/osgi/osgi/tree/requirements/696


[stable/external-dns] upgrade external-dns image

Signed-off-by: David J. M. Karlsen<EMAIL_ADDRESS>

/lgtm


gateway is not reachable

Set-HcsIpAddress returns "gateway is not reachable" on an Azure VM. In fact, gateway doesn't reply to ping, but exists and works fine. What should be done in this case?

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

ID: 9b908ad2-a6e3-c1c4-469c-d8d03caeb3c6
Version Independent ID: 3b0a4572-bdfe-3443-abce-4004ab349d7e
Content: Provision StorSimple Virtual Array in Hyper-V
Content Source: articles/storsimple/storsimple-virtual-array-deploy2-provision-hyperv.md
Service: storsimple
GitHub Login: @alkohli
Microsoft Alias: alkohli

@mcurko Thanks for the Comment. We are actively investigating and will get back to you soon.

@mcurko, Could you give us the more information related to logs to help better on this?

In fact, as I'm testing on Azure, tihs is a problem with nested virtualization. I solved this by creating an internal NAT on the host OS. Here comes another problem, the setup needs a domain controller and I'm again in the field of network connectivity issues for the guest OS. I suggest you amend the article with a step-by-step tutorial on how to deploy the hyper-v image on hyper-v on Azure.

@mcurko , Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate

@YASWANTHM-MSFT This is not a doc enhancement. This is product question. The VHD that is provided is meant to be deployed on an on-premises server (that's what the product is designed for) and not in Azure which is what the customer is asking. Please reassign this to the product team. Thanks!

@mcurko , I reached out to internal product team. The scenario which you are trying is currently not supported(AD would not work in Azure) . If you wish you may leave your feedback here. All the feedback you share in these forums will be monitored and reviewed by the Microsoft engineering teams responsible for building Azure.

OK, in the end I made it all work. I understand that StorSimple Virtual Array is meant to be installed on-prem, but here I wanted only to check it out and see it work for testing purposes. So I installed another hyper-v guest, installed AD DS, promoted it to DC, created necessary records in DNS and it worked just fine!

@mcurko, Glad to know that your issue is resolved.We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Automated cherry pick of #2702: Bugfix: SNAT is not performed on the +est packets.

Cherry pick of #2702 on release-1.1.
#2702: Bugfix: SNAT is not performed on the +est packets.
For details on the cherry pick process, see the cherry pick requests page.

Codecov Report

Merging #2733 (766680b) into release-1.1 (abbfa6d) will decrease coverage by 20.40%.
The diff coverage is n/a.

@@               Coverage Diff                @@
##           release-1.1    #2733       +/-   ##
================================================
- Coverage        61.91%   41.50%   -20.41%
================================================
  Files              278      139      -139
  Lines            21232    17278     -3954
================================================
- Hits             13145     7171     -5974
- Misses            6718     9472     +2754
+ Partials          1369      635      -734

Flag
Coverage Δ

kind-e2e-tests
?

unit-tests
41.50% <ø> (ø)

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

Impacted Files
Coverage Δ

pkg/util/k8s/name.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/agent/agent_linux.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/ovs/openflow/default.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/agent/config/node_config.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/apis/controlplane/helper.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/agent/cniserver/pod_configuration_linux.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/apis/controlplane/v1beta1/register.go
0.00% <0.00%> (-92.86%)
:arrow_down:

pkg/ovs/openflow/logs.go
9.52% <0.00%> (-90.48%)
:arrow_down:

pkg/apis/controlplane/register.go
0.00% <0.00%> (-90.00%)
:arrow_down:

pkg/agent/nodeportlocal/k8s/annotations.go
0.00% <0.00%> (-83.88%)
:arrow_down:

... and 235 more

/test-all
/skip-ipv6-all
/skip-ipv6-only-all
Since the changes are only focusing on Windows, skip IPv6 tests.

It seems that "jenkins-windows-ovs" actually succeeded. "jenkins-windows-networkpolicy" is known to be broken at the moment.


Use django url patterns in angular

First, you have to merge: #1465
Fixes #1453

@normanjaeckel rebased. Please review and merge

@normanjaeckel please review once more

Please rebase (but dont set #1453 to fixed).

done


Set up some sort of blogging system on rec/rec

Requirements:

development of articles in separate branches
comments from users
some reasonable preview workflow

Nice to have:

markdown/uses mkdocs
embedded execution of Python examples
automatic publication of stories on a calendar or timetable

I went with Pelican, which so far has been very little work to get going.


error downloading from github

Hey sir, I got this error while trying. I know it does not make any difference in nix but I tried this on a fresh install (the very first boot entry).
I liked how your config looked and I wanted to learn. Found it on youtube and tried it.
Gratz on the great work!
Cheers

Sir, I am so sorry, I actually found it on reddit, I think. Anyway, nice work there, all good sir, god bless


Extending options to allow customizing fetch options / added request retry to make plugin more failure resistant to temporary network failure, server timeouts, etc...

Description
This small PR adds two options to the plugin:

fetchOptions: This allows the user of the plugin to customize options passed to node fetch, e.g. to specify an increased timeout etc...
retryOptions: Allows the user to pass in options for p-retry, enabling for making the plugin more failure resistant to temporary network failure, server timeouts, etc...

In addition, it wraps the fetch call to the GrapQL API in a p-retry, so that the user can specify a retry behavior with the above mentioned  retryOptions option.
Note: the default value for retryOptions was set to { retries: 1 } to minimize change in behaviour to the plugin.
Related issues
https://github.com/craftcms/gatsby-source-craft/issues/67

@andris-sevcenko thanks for the Feedback! Sure, I've just updated the my branch


Unable to configure extension when using a template deployment

I suspect I've got the JSON format wrong for configuring the extension, but I've not been able to find any specific documentation for what I'm trying to do. I'm bringing up a number of VMs and trying to configure Docker on them using a resource-group based templated deployment.
The relevant part of my template is:
{
    "apiVersion": "2015-05-01-preview",
    "dependsOn": [
        "Microsoft.Compute/virtualMachines/VMNAME"
    ],
    "location": "[resourceGroup().location]",
    "name": "VMNAME/dockerExtension",
    "properties": {
        "autoUpgradeMinorVersion": true,
        "publisher": "Microsoft.Azure.Extensions",
        "settings": {
            "certs": {
                "ca": "...",
                "cert": "...",
                "key": "..."
            },
            "login": {
                "email": "...",
                "password": "...",
                "username": "..."
            },
            "docker": {
                "options": [
                    "-D",
                    "--dns=<IP_ADDRESS>",
                    "--storage-driver=aufs",
                    "--graph=/mnt"
                ],
                "port": "2376"
            }
        },
        "type": "DockerExtension",
        "typeHandlerVersion": "1.0"
    },
    "type": "Microsoft.Compute/virtualMachines/extensions"
}

And I'm receiving this error in /var/log/azure/Microsoft.Azure.Extensions.DockerExtension/1.0.1507151643/docker-extension.log
[DockerExtension] 2015/07/21 14:14:15 ----------------------------------------
[DockerExtension] 2015/07/21 14:14:15 Extension handler launch args: "./bin/docker-extension enable"
[DockerExtension] 2015/07/21 14:14:15 distro info: Ubuntu 14.04
[DockerExtension] 2015/07/21 14:14:15 using distro driver: driver.UbuntuUpstartDriver
[DockerExtension] 2015/07/21 14:14:15 user: root uid:0 gid:0
[DockerExtension] 2015/07/21 14:14:15 env['PATH'] = /usr/local/sbin:/usr/local/bin:/usr/bin:/usr/sbin:/sbin:/bin
[DockerExtension] 2015/07/21 14:14:15 + starting: 'enable'
[DockerExtension] 2015/07/21 14:14:15 ERROR: error parsing handler settings: error deserializing public settings for handler: json: cannot unmarshal number into Go value of type string

Any hints as to what I'm doing wrong, or even better, an addition to the README to describe using the extension from a templated deployment would be gratefully received.
Thanks
Stephen

Hi Stephen,
The error looks obscure. Can you please add your /var/lib/waagent/[extension-name]/config/0.settings file (by redacting your certs/password) as well.
Meanwhile, you should move  certs  and login objects into "protectedSettings" key in your template (not "settings") as they are secrets. Otherwise they will be discarded. This is a nice reminder we should probably add a full fledged ARM template example as well.

Thanks @ahmetalpbalkan. I've now moved my certs and login as you suggest. The contents of 0.settings are as follows...
{
  "runtimeSettings": [
    {
      "handlerSettings": {
        "publicSettings": {"docker":{"certs":{"ca":"...","cert":"...","key":"..."},"login":{"email":"...","password":"...","username":"..."},"options":["-D","--dns=<IP_ADDRESS>","--storage-driver=aufs","--graph=/mnt"],"port":2376}}
      }
    }
  ]
}

Just a guess, but I'm assuming the port number being an integer rather than a string is the problem? It's a string in my deployment template :)

@stevenewey Hmm, port number is my #1 suspicion at this point. It should be string at all times. So was it a string your template and you found it converted to int here somehow or was it string all along?

It was a string in my deployment template. I've actually just tried removing port altogether from the template and re-running the deployment. And I actually get the same error and see the same data in 0.settings (and the file was definitely updated to a newer modified time).

Hmm what do you mean you see the same data in 0.settings, was it unchanged although you updates the template?
Are you reusing the VM between deployments of te DockerExtension? If so I believe you need to first uninstall the extension and add it to the VM again. Is it possible to start from a clean VM for you in this case?

@ahmetalpbalkan The data was unchanged, though the modified time on the file on disk had updated. So I tried your advice and just deleted the VM and things magically started working when I reran the deploy! I then deleted and re-created again, this time with the port back in (as a string) and it also worked, so the issue was I'd used a integer at some previous attempt.
Sorry about that! And thanks for you help.

Not a problem at all sorry for the confusion. I think there's problems with the adding/removing extensions to the VM in the ARM template experience overall. It all sounds complicating to me as well.


python311Packages.pydeck: 0.8.0 -> 0.9.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.pydeck is: Large-scale interactive data visualization in Python
meta.homepage for python311Packages.pydeck is: https://github.com/visgl/deck.gl/tree/master/bindings/pydeck
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.9.0 with grep in /nix/store/n8zl9bqnc557ql97pai3ia7h4x6bcixq-python3.11-pydeck-0.9.0
found 0.9.0 in filename of file in /nix/store/n8zl9bqnc557ql97pai3ia7h4x6bcixq-python3.11-pydeck-0.9.0

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

4 package rebuild(s)

First fifty rebuilds by attrpath

python311Packages.pydeck
python311Packages.streamlit
streamlit

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/n8zl9bqnc557ql97pai3ia7h4x6bcixq-python3.11-pydeck-0.9.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python311Packages.pydeck https://github.com/r-ryantm/nixpkgs/archive/1322f22ef18ad4659e71a48385f47ed2cc3aa5f1.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/1322f22ef18ad4659e71a48385f47ed2cc3aa5f1#python311Packages.pydeck

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/n8zl9bqnc557ql97pai3ia7h4x6bcixq-python3.11-pydeck-0.9.0
ls -la /nix/store/n8zl9bqnc557ql97pai3ia7h4x6bcixq-python3.11-pydeck-0.9.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

4 packages built:

python311Packages.pydeck
python311Packages.pydeck.dist
streamlit (python311Packages.streamlit)
streamlit.dist (python311Packages.streamlit.dist)

Maintainer pings
cc @Creator54 for testing.

LGTM


Create an exec wrapper class

Create a class to wrap the exec function
Use dependency injection so that the command wrapper can use it.
Update unit tests.

#21


[Enhancement] Support bitbucket

This is an awesome tool. However, github private repos are a pay-to-play service. Bitbucket offers private repos free of charge so it would be beneficial to have this tool support bitbucket repos as well.

Yes, I have considered this: #7, for now there's no api to commit code to bitbucket so it seems impossible.
For a workaround, you could use secret gist instead of repo.

Actually, Atlassian recently added support for commits:
https://developer.atlassian.com/bitbucket/api/2/reference/resource/repositories/{username}/{repo_slug}/src#post

That's great!
I'll take some time to look at this, thanks for the information.

Cool!
I was thinking of forking your code and attempting it myself but I couldn't make sense of how the code was structured. It looks like you have a custom build process in place as the files don't appear to be laid out like a typical chrome extension.
In any case, this would be the perfect opportunity for you to use ES6 classes to abstract out a common interface and have separate github and bitbucket implementations of that interface. Let me know when you get started on this, I'd like to chip wherever possible.

A common interface will be a good idea, but maybe a little difficult to cover gist since it's something different from repos.
and actually I didn't use some build process, the javascript files under src is just the same as the work in the extension as content-script.

Noticed that you've added the enhancement tag...and that you've started to work on a common interface...so is it safe to assume that you're actively working on this feature?

Yes, I'm working on this, should be shipped out in next few weeks.
On 2017年9月9日(土) 4:37 DimuDesigns<EMAIL_ADDRESS>wrote:

Noticed that you've added the enhancement tag...and that you've started to
work on a common interface...so is it safe to assume that you're actively
working on this feature?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/leonhartX/gas-github/issues/31#issuecomment-328195219,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADzvrrvCYnfR1CEWF-QwhmTmN5K5W9ihks5sgZeEgaJpZM4PI1a5
.

@DimuDesigns Hi, the bitbucket support is almost completed, but still need test to insure that really works.
I will make a release with the .crx file before upload to the store, so do you like to try that and help to test?

Awesome!
It would be my pleasure to test your latest release!

Loaded up the CRX file locally and here are my findings.
Unfortunately I was not able to create a repo from the App Script editor but I was able to push content to an existing repo.
Also you need to make the dialogs more context-sensitive by updating their texts; some of them reference github when they should be referencing bitbucket.
Great work so far! The only major issue I came across was not being able to create a repo. Fix that and make the dialogs context-sensitive and you're good to go!

Thanks! Can you describe the create repo issue in detail since I can not
reproduce.
On 2017年9月23日(土) 7:48 DimuDesigns<EMAIL_ADDRESS>wrote:

Awesome!
It would be my pleasure to test your latest release!
Loaded up the CRX file locally and here are my findings.
Unfortunately I was not able to create a repo from the App Script editor
but I was able to push content to an existing repo.
Also you need to make the dialogs more context-sensitive by updating their
texts; some of them reference github when they should be referencing
bitbucket.
Great work so far! The only major issue I came across was not being able
to create a repo. Fix that and make the dialogs context-sensitive and
you're good to go!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/leonhartX/gas-github/issues/31#issuecomment-331577813,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADzvrve2GZb6mD3OoZJEZef0NXVoaAQWks5slDlPgaJpZM4PI1a5
.

Sure. Here are the steps I took:

login using the bitbucket sign-in
navigate to the repo drop-down
select "Create new repo"
a dialog opens titled "Create Github repository" (should be bitbucket)
I filled out the form with Repository Name, Description and selected Type (tested both public and private) from the dropdown
I receive the error message "Failed to create new repository"

@DimuDesigns What is the repo name you created? Did you filled with user/org name like leonhartX/some-repo?

@leonhartX I tried creating repos with and without a username prefix; neither worked :( .
Here's are the strings I tried:

dimudesigns/testRepo25
testRepo25
DimuDesigns/testRepo25

Um, that's weird. It should work with the simple repo name without username (the login form's username is used), can you check your login name to make sure it's correct?
Since the username has nothing to do with login so typo won't be noticed, but the name will be used after login like create repo, etc.
Also, if you can open the chrome's developer tools' network panel to see which request get an error, that's will be much helpful!
Anyway, I'm trying to add full name repo creating support so you can create repo with single repo name(some-repo), it will be created under your own username by default, or a full name like some-org/some-repo to create it under the organization/username.

@DimuDesigns Hi, I have made some changes, now you can select the repo's owner(user or team) and just enter the repo name to create(also works with github orgs).
The crx file is updated in the pre-release, so please try this and give for your feedback.
Thanks!

supported in v3.0.0


An error occurs after executing "Execute Function Now..." action for the .NET 8 function app which deployed the .NET 7/6 Isolated project

OS: Win10
Build Version: 20230910.1
Regression: Not a regression
Repro Steps:

Create a .NET 8 function app.
Create a .NET 7 Isolated project -> Deploy it to the .NET 8 function app.
Expand the function app -> Functions -> Right click the function -> Click "Execute Function Now..." -> Type the default value -> Enter.
Check whether no error occurs.

Expect:
No error occurs.
Actual:
An error occurs.

Details in Output:
6:48:17 AM: Error:

:( Application Error

If you are the application administrator, you can access the diagnostic resources.

More Info:

This issue also reproduces for the .NET 8 function app which deployed the .NET 6 Isolated project.
This issue doesn't reproduce for the .NET 8 function app which deployed the .NET 6 (LST) project.

Verified this issue with the build 20230919.5, it still reproduces for now. So I reopen it.

Could you verify that the same issue occurs when deploying through the func CLI?
Here are instructions on how to do that: https://learn.microsoft.com/en-us/azure/azure-functions/functions-core-tools-reference?tabs=v2#func-azure-functionapp-publish
We want to validate if this is a service issue or a VS Code issue.

@nturinski  This issue doesn't reproduce if the .NET 7/6 Isolated project is deployed to function app using the func CLI command.


Feature/okn 2179 - Add header to styleguide

@janeirikh @cristain7 Is this outdated and can be deleted?


Border radius shorthand not supported in react native

Binaries:

Node: 8.12.0 - /usr/local/opt/node@8/bin/node
Yarn: 1.9.4 - /usr/local/bin/yarn
npm: 6.4.1 - /usr/local/opt/node@8/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman

npmPackages:

styled-components: 3.4.9 => 3.4.9

Reproduction
I'm using React Native and border radius to only have a radius on two corners
styled.View`
  border-radius: 4px 4px 0 0;
`;

I get an error saying Invalid prop `borderRadius` of type `string` supplied to `StyleSheet generated`, expected `number`.
Works fine if I only use one unit: border-radius: 4px; but that's not what I want to obtain.
The solution for my code is to use
border-top-right-radius: 4px;
border-top-left-radius: 4px;

but I think SC should be smart enough to convert multiple values in border radius to the corresponding individual definition.

https://github.com/styled-components/css-to-react-native/issues/11

I didn't quite get what that issue is about 😅

Basically we disabled exploding the shorthand for this particular property because not all RN elements can receive the style, namely Image.

I'm really having difficulty to use border-radius on styled-components.... Did someone figure out a way out? I'm reading everywhere and it's not working.


Unwanted space between dfn and comma in auto-added attribute-info

When you dfn an attribute or dict-member in a dt, Bikeshed automatically adds an attribute-info span to it.  This ends up generating a comma, followed by the type. But if the dfn is followed by whitespace (like the dt is unclosed, so it gets the linebreak and indent of the next line), then there's an unwanted bit of whitespace between the term and the comma, which looks bad.
See https://drafts.csswg.org/css-counter-styles/#the-csscounterstylerule-interface for an example

Fixed in 367426f


How to import it into the eclipse?

Hi, I'm new to the BLE and android.Unfortunately, I have got into trouble with importing the project into the eclipse. When I import it as usual, something wrong, there is a red cross under the res folder and no files under the src folder. But I import some similar projects, nothing happened. So what's wrong? Maybe I need to do something special for this project?Thanks a lot.

Hi @zlc111, we recommend you use Android Studio to build this sample as it provides all the necessary tools built-in and configured correctly (such as Gradle for building) in order to get you up and running as fast as possible. You can download Android Studio from the official website. Let me know if you require any additional assistance getting set up with Android Studio.


Add unit tests for input scanner

Added unittest for inputscanner file referring #1446.
Check whether it is ok @bitwiseman

@ganes1410 Failing due to jshint failures.  Runmake js, fix errors, and resubmit.

Is your local branch up-to-date? Are you running from the project root?   What happens when you run make without any parameters?

@ganes1410
Did you give up?  Is there any way we can help?

@bitwiseman  No man.I have not been home for some days.Thats why i could not work on it.
When i try to run make without any parameters , it shows make: *** No targets specified and no makefile found.  Stop.

Travis CI blew up on build 1976.4 but it does not appear to be caused by my code. Looks like the issue lies with Travis CI itself.


I prefer a different push notification

Hello,
Awesome hook! Would this work if I prefer to use a different push notification service?
Like:
pushcrew.com
sendpulse.com
Thank you

you are free to hack and make it work for other notification providers, but not everything will work out of the box!

Thank you. And you can make with this providers.


Use gcloud app instead of gcloud preview app

We should use gcloud app instead of gcloud preview app which will go away soon.

@tmatsuo mind resolving conflicts and then I'll re-review?

Closing this - it seems like this has already been taken care of.


My Emails is Sorting Oldest to Newest why?

RainLoop version, browser, OS:
1.16.0
Expected behavior and actual behavior:
email date order from new to old
Steps to reproduce the problem:
not possible
Logs or screenshots:
I have more than one account, only one is affected. (Domain icloud.com)

After Update, I have this problem.


Refactor / move class should resolve embed source paths

I just moved a class with embedded source paths, which didn't get resolved:
[Embed(source="../../../../assets/images/thumbnails/thumbnail1.jpg")]
static public var Thumbnail1		:Class;

Cheers,
Dave

WIP


Merge v3 dev branch to master

Description
Only one commit actually.
Test Options

[ ] fast test
[ ] full test - HPO
[ ] full test - NAS
[ ] full test - compression

Checklist

[ ] test case
[ ] doc

How to test

should we merge to dev branch first?

Suggest postponing this merge unless otherwise you need merging back and forth...


Throw parse exception first

[x] Have you signed the Contributor License Agreement?
[x] Are you working against an Issue?

Resolves https://github.com/microsoft/winget-create/issues/464

Microsoft Reviewers: Open in CodeFlow

/azp run


Fake missing sources referenced from shell expansions

Sources referenced from shell expansions are effectively equivalent to sources included using the %include directive, for instance %include %{SOURCE1} does the same thing as %(cat %{SOURCE1}).
Treat such sources that are not available the same way as missing included sources.
Related to #122.

Actually, this is unnecessary, as missing sources referenced from shell expansions don't cause parsing to fail, RPM just emits a warning.


Hyper2 Pay site?

Hi, sorry, not really an issue, just wanted to ask if this site for hyper2 is legit: https://zeit.co/blog/hyper2 or fake?
Thanks!

Yup! Zeit is the main organization that hosts this repo - that is their blog post announcing features and changes for v2 of Hyper.

@tempusthales What do you mean as "Pay" site.


Parameter Injection

When we developing a model and trying to give it a shot with lm-evaluation-harness, I have an issue with the file path that includes an equal sign, even if that will be legal on the Linux File System.
Here's the error we got:
File "/usr/share/bin/lm_eval/utils.py", line 92, in <dictcomp>
    k: handle_arg_string(v) for k, v in [arg.split("=") for arg in arg_list]
ValueError: too many values to unpack (expected 2)

Currently, we solve it by renaming the file path without the '=' sign

Here's an example that causes errors.
lm_eval --model hf --model_arg "pretrained=$WORK/mistral-7b-v0.1_tokenizer=v2|data=m|step=34500|epoch=18975" --tasks "tmmluplus"

Hi! This is due to the simple parsing utility we use to split the CLI strings into key-value pairs. Where the value part contains additional equal signs, the parser fails to correctly split these into just two elements, leading to the error.
https://github.com/EleutherAI/lm-evaluation-harness/blob/488759d2c9a55ad3ed55d55417b7733e95e37667/lm_eval/utils.py#L81-L94

Hi! This is due to the simple parsing utility we use to split the CLI strings into key-value pairs. Where the value part contains additional equal signs, the parser fails to correctly split these into just two elements, leading to the error.

Thanks for responding. I know this issue exists because the argument parser uses the equal sign to separate.
But I'm wondering if there are any plans to solve this problem.
And to be honest, it's not necessary to solve it.
I just want to submit this problem to you guys.

It may be worth considering allowing for = signs within a parameter name if it appears within "...", as one thing that's been requested is the ability to pass dicts and lists into the library as arg values via the CLI.
However, this might lead to annoying bugs if it means that we end up silently failing when a user typos their kwargs and this no longer errors immediately.

It may be worth considering allowing for = signs within a parameter name if it appears within "...", as one thing that's been requested is the ability to pass dicts and lists into the library as arg values via the CLI.

How about a yaml config? Would be easier to maintain compared to creating all these increasingly complicated parsing rules.

Maybe yeah. There may be a point at which it's easier to just have people write it out in a python script by passing the correct kwargs direct to the LM

As discussed in #1412 with @Am1n3e , we may want to provide another launcher aside from the CLI that makes it easier for users to provide any arbitrary argument combos to simple_evaluate.


OSQP does not compile if PROFILING flag is disabled

When I try to compile osqp with the PROFILING flag disabled I get the following error
/home/gromualdi/robot-code/osqp/src/util.c:127:15: error: ‘OSQPSettings {aka struct <anonymous>}’ has no member named ‘time_limit’
   if (settings->time_limit) c_print("time_limit: %.2e sec\n", settings->time_limit);
               ^~
/home/gromualdi/robot-code/osqp/src/util.c:127:71: error: ‘OSQPSettings {aka struct <anonymous>}’ has no member named ‘time_limit’
 if (settings->time_limit) c_print("time_limit: %.2e sec\n", settings->time_limit);
                                                                     ^~
CMakeFiles/osqpstatic.dir/build.make:159: recipe for target 'CMakeFiles/osqpstatic.dir/src/util.c.o' failed
make[2]: *** [CMakeFiles/osqpstatic.dir/src/util.c.o] Error 1
CMakeFiles/Makefile2:304: recipe for target 'CMakeFiles/osqpstatic.dir/all' failed
make[1]: *** [CMakeFiles/osqpstatic.dir/all] Error 2
Makefile:148: recipe for target 'all'

osqp version: da403d4b41e86b7dc00237047ea4f00354d902ed
This patch fixes the problem
diff --git a/src/util.c b/src/util.c
index a97ec7eb..5bdc5b89 100644
--- a/src/util.c
+++ b/src/util.c
@@ -108,8 +108,8 @@ void print_setup_header(const OSQPWorkspace *work) {
   else c_print("          check_termination: off,\n");

 # ifdef PROFILING
-  if (settings->time_limit) c_print("          time_limit: %.2e sec,\n",
-                                    settings->time_limit);
+  if (settings->time_limit) c_print("time_limit: %.2e sec\n", settings->time_limit);
+  else c_print("time_limit: off\n");
 # endif /* ifdef PROFILING */

   if (settings->scaling) c_print("          scaling: on, ");
@@ -124,9 +124,6 @@ void print_setup_header(const OSQPWorkspace *work) {
   if (settings->polish) c_print("polish: on, ");
   else c_print("polish: off, ");

-  if (settings->time_limit) c_print("time_limit: %.2e sec\n", settings->time_limit);
-  else c_print("time_limit: off\n");
-
   c_print("\n");
 }

Let me know If you are interested in a PR

Maybe the same as #157?
If so, then no PR is necessary (although the offer is appreciated).

Hi @goulart-paul I don't know which branch should I use, since the one indicated by @gbanjac in https://github.com/oxfordcontrol/osqp/issues/157#issuecomment-502143228 does not exist anymore

I think that fix will have ended up in the current 0.X development branch, which is mostly bug fixes.
Try here: https://github.com/oxfordcontrol/osqp/tree/develop-0.x

https://github.com/oxfordcontrol/osqp/tree/develop-0.x works. Thank you very much.


fix: rename ports vars to uppercase

Changes

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1485  👈 (View in Graphite)
#1482
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


[ECR-4395] Fix sonar bugs

Overview

See: https://jira.bf.local/browse/ECR-4395
Definition of Done

[ ] There are no TODOs left in the code
[ ] Change is covered by automated tests
[ ] The coding guidelines are followed
[ ] Public API has Javadoc
[ ] Method preconditions are checked and documented in the Javadoc of the method
[ ] Changelog is updated if needed (in case of notable or breaking changes)
[ ] The continuous integration build passes

Coverage decreased (-0.06%) to 84.927% when pulling 262c3368a09a5ccd25ab4e5693d4336630b10488 on ecr-4395 into 34083c6dbf1d2fdbce7a238692ae18acbe97aa24 on master.


Gpu support

This PR will add a docker file and instruction to run Smart Social Distancing on x86 nodes with GPU.

Nice PR. I tested this on a GTX 1070 and it seems to work. I get logs like
Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 7036 MB memory) -> physical GPU (device: 0, name: GeForce GTX 1070 ...
However I am not getting any speedups compared to running it without GPU. Have you seen speedups with these models and another GPU?

Thank you @mats-claassen for mentioning this, I had tested the GPU support for SSD detector only and not for OpenPifPaf. Now it is fixed for OpenPifPaf too, can you test it one more time please?

Works much better now. I can process multiple frames per second with OpenPifPaf.
What about Mobilenet? if that is not supported for GPU we could remove it from the comment in the .ini file

Both Mobilenet and OpenPifPaf are currently supported for GPU.


WIP: Teach Indexify about Raft

This is majorly a work in progress, as I'm learning the details of the Raft algorithm as I go :-)
Here is the status:
Done:

Add in-memory interface for OpenRaft
Add Raft management and communication APIs
Add Raft RPC protocol layer
Integrate coordinator state into Raft state machine
Integrate coordinator api layer

TODO:

Test
Fix merge conflicts, Clean up, document, make pretty

Future PR:

Sled integration

Raft is already in!


urllib2.HTTPError: HTTP Error 503: Service Unavailable

I'm getting this error on my gcloud app which runs python 2.7.16
Same error occurs on python 3.7.1 when executing the command line demo of the documentation
>>> amazon.lookup(ItemId='B07MFZY2F2') Traceback (most recent call last):   File "<stdin>", line 1, in <module>   File "C:\Python27\lib\site-packages\amazon\api.py", line 174, in lookup     response = self.api.ItemLookup(ResponseGroup=ResponseGroup, **kwargs)   File "C:\Python27\lib\site-packages\bottlenose\api.py", line 274, in __call__     {'api_url': api_url, 'cache_url': cache_url})   File "C:\Python27\lib\site-packages\bottlenose\api.py", line 235, in _call_api     return urllib2.urlopen(api_request, timeout=self.Timeout)   File "C:\Python27\lib\urllib2.py", line 154, in urlopen     return opener.open(url, data, timeout)   File "C:\Python27\lib\urllib2.py", line 435, in open     response = meth(req, response)   File "C:\Python27\lib\urllib2.py", line 548, in http_response     'http', request, response, code, msg, hdrs)   File "C:\Python27\lib\urllib2.py", line 473, in error     return self._call_chain(*args)   File "C:\Python27\lib\urllib2.py", line 407, in _call_chain     result = func(*args)   File "C:\Python27\lib\urllib2.py", line 556, in http_error_default     raise HTTPError(req.get_full_url(), code, msg, hdrs, fp) urllib2.HTTPError: HTTP Error 503: Service Unavailable

Same Error.
Were you able to fix this?

@Karlheinzniebuhr - I found the issue. You need to pass region too. See the code snippet below.
amazon = AmazonAPI(AMAZON_ACCESS_KEY, AMAZON_SECRET_KEY, AMAZON_ASSOC_TAG, region='IN')
By default, it's US. If your affiliateId belongs to some other region, pass into it.
Check this file.

@Karlheinzniebuhr - I found the issue. You need to pass region too. See the code snippet below.
amazon = AmazonAPI(AMAZON_ACCESS_KEY, AMAZON_SECRET_KEY, AMAZON_ASSOC_TAG, region='IN')
By default, it's US. If your affiliateId belongs to some other region, pass into it.
Check this file.

Still raises the same exception.


[FAQ] RocketChat and website availability after conference

From Dan,
I think we should announce about the rocketchat and also make and announce a decision about the rest of the website. (i.e. something like "The ACL2020 website will remain up for ACL attendees to rewatch any of the conference videos, although the RocketChat and zoom links will be removed  [?on DATE? Friday? Monday?].  The videos will also be uploaded to the ACL Anthology over the next few months"

@hao-fang  Do you know the exact details of when will do we plan to remove the links? I can then add the FAQ accordingly.

@raosudha89 not yet ... it will take some time to remove these links. maybe we say by July 25 because our contract with RocketChat ends on that date.

@hao-fang  Do you think we should add some version of this FAQ in our tonight's release? I know that Joel said they don't have a definite answer as yet. But if you think many people are asking this question, then we can say something without specifying exact dates.

@raosudha89 Let's wait for tomorrow probably, just to make sure we do not communicate the wrong information to people.
If they ask, our volunteers will be able to give some answers.
And we can update the FAQ tomorrow around noon once PCs have made an official decision, and we can then update the FAQ & make an announcement.

Sounds good. Thanks.

Close this as we already make the announcement and the conference is almost over.


Use a Retina Image for Higher Resolution Displays

Describe your problem and - if possible - how to reproduce it
The 'Use a Retina Image for Higher Resolution Displays' challenge gives no context whatsoever about what a retina image is, what a retina display is, or what this challenge is really teaching you to do. I completed the challenge, but really had no idea what I'd just done or how that would effect a webpage I built and what benefits it would really provide. This platform caters to people of all levels of knowledge and I feel that this assumes a fairly high level of knowledge about screens/displays.
Add a Link to the page with the problem
https://learn.freecodecamp.org/responsive-web-design/responsive-web-design-principles/use-a-retina-image-for-higher-resolution-displays

Agreed. There needs to be a brief description of what a “retina” display is, as not everyone pays attention to Apple’s marketing. (Do other manufacturers have similar terms for their high-density displays?)


Use Nephio annotation

Fixes #7
Seems like we might not want to use string checks but some sort of booling check?

I don't have permission to merge here, but LGTM


Data Factory Package - Microsoft.Azure.Management.DataFactory is depreciated

Nuget Package Microsoft.Azure.Management.DataFactory is depreciated, the document needs to be updated with the alternate package and corresponding examples.

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

ID: 1ebc0c44-b401-1f1f-2ed5-7928918e49b4
Version Independent ID: 52517770-de52-18b3-7bf6-078c18b70fe6
Content: Create Azure Data Factory using .NET SDK - Azure Data Factory
Content Source: articles/data-factory/quickstart-create-data-factory-dot-net.md
Service: data-factory
Sub-service: data-movement
GitHub Login: @jianleishen
Microsoft Alias: jianleishen

@vinayakmoh
I've delegated this to @jianleishen , a content author, to review and share their valuable insights.

Thanks @vinayakmoh for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. The requested updates have not been made since the creation of this issue, and the timeline for resolution may vary based on resourcing, so we've created an internal work item to incorporate your suggestions. We are closing this issue for now, but feel free to comment here as necessary. #please-close


Added option to allow validation of empty values by custom validators

This feature is implemented as a new optional feature - it is not a breaking feature
You set it by calling the below method passing true:
govalidator.SetEmptyValueAllowed(true)

New feature, or refactor of existing code: master/develop branch

Q
A

Documentation
yes

Bugfix
no

BC Break
no

New Feature
yes

RFC
yes

QA
yes

Description
This change is necessary because the user of this package should be able to validate empty values.

How do you reproduce it?
Try and validate any empty value

What did you expect to happen?
for this package to call our custom validator even if its an empty value

What actually happened?
it does not call our custom validator - it returns false

Are you adding documentation?
Yes I have updated the readme

Are you providing a QA improvement (additional tests, CS fixes, etc.) that
does not change behavior?
I have added 2 additional tests. This feature is implemented as a new optional feature - it is not a breaking feature

Explain why the changes are necessary:
Because we should be able to validate empty values if we want to

TARGET THE master BRANCH
Yes

This seems like a good non-breaking implementation for the issue lots of people are having. How can we get some reviews here?

This seems like a good non-breaking implementation for the issue lots of people are having. How can we get some reviews here?


Update widget npm package

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Koa2 and Controller Actions -> this is undefined

in this example is this undefined
class Root {
	async index (ctx) {
		console.log(this) // <-- is undefined
		ctx.body "Root"
	}
}

const app = new Koa()
const root = new Root()

app.use(root.index)
app.listen(3000)

i have to explicity do this
// ...
app.use(root.index.bind(root))
// ...

That is uncool xD

This is how JavaScript works. Another way is to bind in the constructor:
class Root {
    constructor () {
        this.index = this.index.bind(this);
    }
    async index (ctx) {
        console.log(this) // <-- is undefined
        ctx.body "Root"
    }
}

what @targos said


IPADDRESS and IP_ADDRESS discrepancy

When setting up Spring Cloud Consul JVM environment variables in Docker the following variables
-e SPRING_CLOUD_CONSUL_DISCOVERY_PREFERIPADDRESS={value}
-e SPRING_CLOUD_CONSUL_DISCOVERY_IP_ADDRESS={value}
are propagated to
spring.cloud.consul.discovery.preferIpAddress and
spring.cloud.consul.discovery.ipAddress
respectively.
In the former case there is no underscore between IP and ADDRESS. In the latter, there is an underscore.

I'm afraid this is a spring boot issue. Not much we can do about it.

Ok, I thought maybe it was related to the Spring Cloud bootstrap context initialization.
If it's not, I will then check if the normal relaxed binding works (which should be ...DISCOVERY_PREFER_IP_ADDRESS) and post correct environment variable names here.

Yes, for some reason only preferIpAddress is bound from both and PREFERIPADDRESS and PREFER_IP_ADDRESS. IP address will not be bound bound to property if typed without underscore.
The following keys work with relaxed binding rules for Spring Boot:
-e SPRING_CLOUD_CONSUL_DISCOVERY_PREFER_IP_ADDRESS={value}
-e SPRING_CLOUD_CONSUL_DISCOVERY_IP_ADDRESS={value}
-e SPRING_CLOUD_CONSUL_DISCOVERY_HOSTNAME={value}


Include invalid sigV4 authType in error message

This PR adds the authType to the error message when it is invalid for SigV4.
This came up in this issue report and it would have been nice if the authType showed up in the logs there.

Hiya @grafana/grafana-operator-experience-squad, is this alright for you as codeowners?

Thanks!


Statistical metric for data amount produced

We need a metric to quantify the data amount produced. Propose:
variable year production rate
or
variable year curation rate

ET favours variable year curation rate .


zig build test fail on zig 0.13.0 src/minilib/TimerQueue.zig

Step to re-procedure
git fetch origin zig-0.13
zig build test
same error on master branch

You seems to be running on 0.14.0

Maybe cache issue? Or using wrong branch?

This way more simpler, "git fetch" only retrieve data from the branch; but it doesn't change the branch you're on.
git checkout zig-0.13

A simple checkout could fix your issue...


Cannot read properties of undefined reading "totalProgess"

Hey, this error occurs when clearing a line in any game mode
I'm on the 1.3.5 release

my settings
settings.zip

Yep it was a small bug where an animation that already finished was being cleared, fixed now


Punkt: expose custom variables on object instances

I was very excited to discover this JS implementation of Punkt. Thanks for building it!
Using your library I'd like to do the following, but it seems the current implementation won't allow configuring custom variables:
# In Python...
trainer = PunktTrainer()
trainer.INCLUDE_ALL_COLLOCS = True
trainer.train(text)

I think a good way to conform to the original python API would be to expose these on the instance objects in the constructor. I've taken a shot at this in the PR. Let me know what you think.

Thanks @mikehenrty. This is not my preferred solution but it's good enough so I think I will merge. My original intention was to make all this more JavaScripty in feel but I did not find the time.

Do you need a release on npm?

Do you need a release on npm?

Yes please :)
Thanks again @Yomguithereal !

v0.21.0 is now published.

cheers


who know google oauth2 set

GOOGLE_CLIENT_ID:
GOOGLE_CLIENT_SECRET:
GOOGLE_OAUTH2_REDIRECT_URL: http://ec2-xx-xx-xx-xx.compute-1.amazonaws.com:3000/auth/google_oauth2/callback
i can't  post
https://accounts.google.com/o/oauth2/auth?access_type=offline&client_id=1044454023387-emhp5vmfnp5cf1n9juk09g8gcckrova7.apps.googleusercontent.com&redirect_uri=http%3A%2F%2Fexchange.stardustcoin.org%3A3000%2Fauth%2Fgoogle_oauth2%2Fcallback&response_type=code&scope=email+profile&state=feebda67c3a7c30a8bef0ab551b1ebd6ebee705bcab2c669

There are 2 GOOGLE_CLIENT_ID setting in the same file. You might have changed it in one location (production) and may have missed it for development or viceversa.


investigate tensorflow

https://jalammar.github.io/Supercharging-android-apps-using-tensorflow/ etc

https://github.com/ruslo/hunter/issues/667


Refactor ContentUploadModal.jsx from class based to functional component

Ref: #4460
About This PR
This PR refactored

packages/volto/src/components/manage/Contents/ContentsUploadModal.jsx

from class based to functional component.

Before:
https://github.com/user-attachments/assets/eabaddbb-629d-43f0-9107-776840fab8f8
After:
https://github.com/user-attachments/assets/7088d68a-bc22-41bc-97e7-f54e1828411c

@antedotee you need to sign the Plone Contributor Agreement to merge this pull request.
Learn about the Plone Contributor Agreement: https://plone.org/foundation/contributors-agreement

@antedotee you need to sign the Plone Contributor Agreement to merge this pull request.
Learn about the Plone Contributor Agreement: https://plone.org/foundation/contributors-agreement

@antedotee you need to sign the Plone Contributor Agreement to merge this pull request.
Learn about the Plone Contributor Agreement: https://plone.org/foundation/contributors-agreement

I have signed the contributor agreement still I am getting the error. Any suggestion?

@antedotee from https://plone.org/foundation/contributors-agreement

I signed the Agreement, but haven't been given access yet?
Please allow a week for a response. Volunteers process the Contributor Agreements. If you have issues, such as making a spelling error in your email or GitHub username, please contact<EMAIL_ADDRESS>
You can also add the email you used in the agreement to your GitHub account.

This PR may duplicate work in https://github.com/plone/volto/pull/6178.
Closing because the contributor has not followed up.


daemon: added function to check if kubernetes option is set

By having this function allows cilium to detect if it should connect to
kubernetes and give the proper status report for the kubernetes side.
Signed-off-by: André Martins<EMAIL_ADDRESS>

Nice!


Support for SMARTS environment

Hello,
Are there any plans for supporting the SMARTS platform in the foreseeable future?
Thanks.

Will support in the future but not too soon.


quick options in taskbar icon

I think will be very usefull if in the taskbar icon shows a menu  or a keyboard shortcut where we can quickly  enable or disable different characteristics of the program, like fancyzones.

@ledemon Please add your suggestions to #6751


Type="number" not showing error when letters are entered

Hi and thanks for the plugin, it's really great!
I don't quite understand though, I have an input type="number" but when I put letters (a,b,c...) it does not show an error (it doesn't validate either). Is it normal behavior?

If you are talking on Chrome and Firefox it should work, can you show the code please? Also its important to know what version are you using.

This is now in the documentation. Nothing we can do about that... See #874
Thanks


Changed Request message to Shunt and added Trigger and Emit debug helper...

Please review

It seems that the coral semantic is modified to fit some specific use case of the cassandraActor. Why?
What are those use cases? The current semantic of the Coral actor is to process data. This pull request introduces a new semantic of actors passing each others commands, in some case without data.
Currently coral actors are passing each other data, hence they build up a dataflow. These changes transform the data flow in a collection of RPC objects. This is a very different paradigm, and I would be strongly discourage, since it stimulated the use of actors as remote procedure agents.

The CoralActor semantics are not modified, the existing semantics are left intact.
The Trigger and Emit messages are introduced to facilitate testing of the trigger and emit functions. The only case where you ever want to use the Trigger and Emit messages is in test classes.
The emit function always takes a json object as input, but in the current usage, it is almost never used. This is because there are only a few exceptions in which you want to work with the original json in the emit function.
I do not see how the changes transform any data flow, and I do not understand the connection with RPC objects. There is also no paradigm shift.


Creating two collections with the same title is currently impossible

Pending fix: https://github.com/Atlantic18/DoctrineExtensions/pull/1648

Works with Zikula 1.4.3-rc3.


Update README files to generate sdk in azure-sdk-for-net repo.

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
If updating an existing version, please select the specific langauge SDKs and CLIs that must be refreshed after the swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[x] No refresh required for updates in this PR
No refresh is needed, we hadn't  published any SDKs.

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ ] Check this box if any of the following apply to the PR so that label “WaitForARMFeedback” will be added automatically to begin ARM API Review. Failure to comply may result in delays to the manifest.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits.

[x] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[x] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If any of the following scenarios apply to the PR, request approval from the Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in a stable version
[ ] Removing properties in a stable version
[ ] Removing API version(s) in a stable version
[ ] Updating API in a stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

/azp run

@changlong-liu , there is incorrect update in the PR description which added ARM review label, we updated the same. Can you please review the changes now? There are no swagger changes. This is purely SDK related changes.

@changlong-liu , there is incorrect update in the PR description which added ARM review label, we updated the same. Can you please review the changes now? There is no swagger changes. This is purley SDK related changes.

/azp run

Wrongly I have marked ARM checked. we dont need this. I have removed now, could you pls review it.

/azp run


Fix: umidiplay -> umidiparser

Only fixed example in README, though there are other legacy references elsewhere that may need to be fixed. BTW thank you for this library! Super helpful for a project I'm working on 😄

Thank you very much!


bring back 'get started' link and pre-graduation hero text

→ https://deploy-preview-1417--fluxcd.netlify.app/


Duplicate Submits

I've been running Z-NOMP for solo mining. I've seen a number of cases where the private pool is submitting the same solution to the node for a block from 2 different miners. So, this is probably happening at the share level, as well. Anyone else notice this? It happens almost every day, and sometimes twice. I only see it at the block level, since the share data is meaningless to me, but if it's happening at the block level, it may be happening much more frequently at the share level. I'm running the master repo, not the last release.

^^ I should add that I have the log files that cover examples of this, if you need them.

Duplicate shares only happen with low diff shares because of their low nonce range, but z-nomp should reject duplicates from miners. 2 miners submitting the block at near the same time can happen and would actually happen since miners are supposed to find that nonce.

I would be interested in the log though

When I see the next occurrence I’ll provide you with the relevant node, Z-NOMP, and individual miner logs. Even if this is just 2 miners individually finding a solution for the same block, at the same time, one would have the “best” solution, and the pool should not report it as a “Block found” twice. Perhaps that’s just a matter of testing for that condition before the “Block found” is issued to the log. Node.js/javascript is just gooblygook to me (being an old assembler/“C” guy, myself), so I have not looked at the code.

On Feb 13, 2017, at 10:26 PM, Procrastinator<EMAIL_ADDRESS>wrote:
I would be interested in the log though
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/z-classic/z-nomp/issues/86#issuecomment-279611830, or mute the thread https://github.com/notifications/unsubscribe-auth/AOexPTpd1EJHgNak65FGL2kgGu4yIC3dks5rcTsTgaJpZM4MAD_T.

OK, got another, fresh, example. The pool log didn't show any other log entries for these 2 shares.
Z-NOMP Pool Log:
2017-02-15 17:03:03 [Pool]  [zclassic] (Thread 12) Submitted Block using submitblock successfully to daemon instance(s)
2017-02-15 17:03:03 [Pool]  [zclassic] (Thread 2) Submitted Block using submitblock successfully to daemon instance(s)
2017-02-15 17:03:03 [Pool]  [zclassic] (Thread 2) Block notification via blocknotify script
2017-02-15 17:03:03 [Pool]  [zclassic] (Thread 1) Block notification via blocknotify script
2017-02-15 17:03:03 [Pool]  [zclassic] (Thread 3) Block notification via blocknotify script
2017-02-15 17:03:03 [Pool]  [zclassic] (Thread 5) Block notification via blocknotify script
2017-02-15 17:03:03 [Pool]  [zclassic] (Thread 6) Block notification via blocknotify script
2017-02-15 17:03:03 [Pool]  [zclassic] (Thread 4) Block notification via blocknotify script
2017-02-15 17:03:03 [Pool]  [zclassic] (Thread 10) Block notification via blocknotify script
2017-02-15 17:03:03 [Pool]  [zclassic] (Thread 7) Block notification via blocknotify script
2017-02-15 17:03:03 [Pool]  [zclassic] (Thread 8) Block notification via blocknotify script
2017-02-15 17:03:03 [Pool]  [zclassic] (Thread 12) Block notification via blocknotify script
2017-02-15 17:03:03 [Pool]  [zclassic] (Thread 11) Block notification via blocknotify script
2017-02-15 17:03:03 [Pool]  [zclassic] (Thread 9) Block notification via blocknotify script
2017-02-15 17:03:03 [Pool]  [zclassic] (Thread 12) Block found: 00000006baa71a69ecefe8c00a731d1db261d687f71ab41cb760a165e403abde by node07
2017-02-15 17:03:03 [Pool]  [zclassic] (Thread 12) Share accepted at diff 32/77913.39737503 by node07 [::ffff:<IP_ADDRESS>]
2017-02-15 17:03:03 [Pool]  [zclassic] (Thread 2) Block found: 00000006baa71a69ecefe8c00a731d1db261d687f71ab41cb760a165e403abde by node09
2017-02-15 17:03:03 [Pool]  [zclassic] (Thread 2) Share accepted at diff 32/77913.39737503 by node09 [::ffff:<IP_ADDRESS>]
ZCL Node Log:
2017-02-16 00:03:03 UpdateTip: new best=00000006baa71a69ecefe8c00a731d1db261d687f71ab41cb760a165e403abde  height=58236  log2_work=43.020583  tx=149228  date=2017-02-16 00:02:24 progress=0.999991  cache=0.7MiB(745tx)
2017-02-16 00:03:03 AddToWallet e02f8c184287287cb4c76c6f143cc09f341d3914522b34f64289f6aa15e862cf  new
2017-02-16 00:03:03 keypool reserve 2
2017-02-16 00:03:03 CreateNewBlock(): total size 1000
2017-02-16 00:03:03 keypool return 2

Thanks, I have a small idea how to resolve it, does the duplicate block show up in the redis database twice?

Honestly, I don’t know. I don’t even have the Webui enabled, although redis is running (no choice). If you can tell me how to query redis from the command line, I can check it. For solo mining, I personally find little value is the pool stats, I just tail &grep the log file for “Block found”, since that’s all that matters for solo.

On Feb 15, 2017, at 5:53 PM, Procrastinator<EMAIL_ADDRESS>wrote:
Thanks, I have a small idea how to resolve it, does the duplicate block show up in the redis database twice?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/z-classic/z-nomp/issues/86#issuecomment-280193584, or mute the thread https://github.com/notifications/unsubscribe-auth/AOexPUxvBYemyN1vaVDHQP7J4TkG_U8zks5rc54DgaJpZM4MAD_T.

you can use redis-commander sudo npm install -g redis-commander, then you can just use redis-commander to start it

Is that a GUI or cli? The rig running the zcl node and z-nomp is headless. I can only access over ssh.

On Feb 15, 2017, at 6:00 PM, Procrastinator<EMAIL_ADDRESS>wrote:
you can use redis-commander sudo npm install -g redis-commander, then you can just use redis-commander to start it
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/z-classic/z-nomp/issues/86#issuecomment-280195039, or mute the thread https://github.com/notifications/unsubscribe-auth/AOexPRKwS2QagRsMrzGhZSxrzjaYNKzuks5rc5_AgaJpZM4MAD_T.

it launches a website served over port 8081

Great! Thanks!

On Feb 15, 2017, at 6:04 PM, Procrastinator<EMAIL_ADDRESS>wrote:
it launches a website served over port 8081
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/z-classic/z-nomp/issues/86#issuecomment-280195763, or mute the thread https://github.com/notifications/unsubscribe-auth/AOexPcYkodcbB04v9sFtlzEEhAn8zBjwks5rc6CngaJpZM4MAD_T.

OK, so if I’m understanding what I’m looking at, the DB has only 1 share for that round, belonging to the second of the two rigs (node09).

On Feb 15, 2017, at 6:04 PM, Procrastinator<EMAIL_ADDRESS>wrote:
it launches a website served over port 8081
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/z-classic/z-nomp/issues/86#issuecomment-280195763, or mute the thread https://github.com/notifications/unsubscribe-auth/AOexPcYkodcbB04v9sFtlzEEhAn8zBjwks5rc6CngaJpZM4MAD_T.

I've been looking into this issue also. You will have two pending block in "blocksPending" with the same height but different hash and only 1 of them is valid. The invalid block will show up in "getchaintips" RPC call.
I have created a "work-around" in paymentsProcessor.js to detect this invalid pending block and kick the invalid pending block to prevent payout errors. This is only sanity protection for the payment system and not a solution to the problem.
https://github.com/hellcatz/z-nomp/blob/hellcatz/libs/paymentProcessor.js#L508-L552
The issue is with node-stratum-pool. I have detected the problem here...it happens when a new block is submitted while the previous block is still being processed by the daemon. The new block is processed at the same block height. Read my commit notes...
https://github.com/hellcatz/node-stratum-pool/commit/4f60132e3be23c71b9ef021034a0f0b4956954c3
I'm still testing on the zcash testnet (not zclassic).  I have a couple ideas to possibly solve the issue, I have not been able to test them yet...

Hi,
I have same problem, you have found a solution?

Database redis blocksPending (2) with two different codes for the same block, it is now in pending (we surpassed 100 confirmations) but remains pending! Solution? Thanks
https://i.gyazo.com/1bc6a3a86d3cb228c98953ace2964e56.png

should be fixed with 86778a2ae27cb327e9db468627891c37a94bcc30

It is with my latest "major update 2" pull request.
Tested and proven to fix payments glitches from invalid duplicate blocks on zcash_testnet.
Invalid duplicate blocks are moved to "blocksDuplicate" in redis before causing payment issues.
Unfortunately, these blocks are truly invalid and must be marked as duplicates.
See Screenshow Below:

Should be fixed now with v1.0.3 I hope


cache API rework

The current cache API needs a deep rework as came out in #50136.
The problem
The cache operations must be quick, optimized and possibly inlined. The current API is clunky, functions are not inlined and passing parameters around that are basically always known at compile time.
See include/zephyr/cache.h
The proposal
We extend the set of functions so that we have:

cache_{data,instr}_enable()
cache_{data,instr}_disable()
cache_{data,instr}_line_size_get()
cache_{data,instr}_flush_all()
cache_{data,instr}_flush_range(addr, size)
cache_{data,instr}_invd_all()
cache_{data,instr}_invd_range(addr, size)
cache_{data,instr}_flush_and_invd_all(addr, size)
cache_{data,instr}_cache_data_flush_and_invd_range(addr, size)

This should allow us to get rid of useless parameters and make inlining easier.
The requirements
We still have to keep supporting external cache controllers besides arch operations (see for example drivers/cache/cache_aspeed.c ). This is currently done acting on CONFIG_HAS_EXTERNAL_CACHE / CONFIG_HAS_ARCH_CACHE and this is going to be kept also in the new API.
Open questions

Do we want to keep supporting syscall wrappings for this API?
Do we need other operations besides flush, invalidate and flush and invalidate? We have of course a scaling problem now that we got rid of parameters.

@andyross @lgirdwood @lyakh @nashif @carlescufi @stephanosio

Would creating a userspace friendly wrapper around DMA buffers (that does the work of flushing/invalidating at the right times perhaps) allow for these to not be syscalls?
Is there another use case that requires them to be syscalls?

Would creating a userspace friendly wrapper around DMA buffers similar to but not exactly like Linux's dma_buf (that does the work of flushing/invalidating at the right times perhaps) allow for these to not be syscalls?

Possibly but we cannot exclude other possible uses. Quoting @andyross from discord: Whether cache control instructions are priviledged or not is going to be arch-specific.  It's probably safe to assume that usermode code will never care, but at the same time we should probably support the API wherever we can.

Is there another use case that requires them to be syscalls?

I'm thinking for example user-space apps using OpenAMP. In that case OpenAMP is using cache operations so everything comes down to whether the arch supports operations in low-privilege threads or not.

@carlocaione can we also have
cache_{data,instr}_lock()
cache_{data,instr}_unlock()

This will allow us some more fine grained control over cache when needed for performance or debug reasons. Thanks

@lgirdwood  What does the lock/unlock entail, semantically?  Pinning cache lines (which I'm guessing is what you mean?) is a thing, but it tends to be highly hardware-specific, and may even have things like different interfaces for different levels of cache.  You need to worry about aliasing between ways (i.e. you can only have N lines pinned with an address modulo the way stride), etc...  This just seems like a hard API to have at the generic layer, though I'm not an expert.

And regarding the syscall discussion from last week: I guess my feeling is that we should have some kind of support for userspace here[1].  I don't necessarily know that it needs to be exactly the same API.  Something like the old API could be left as the "high level" API, since it has fewer entry points to make into syscalls, and obviously the concern about inlinability needed by the kernel and drivers isn't going to be a factor.
[1] Actually something like future SOF is a good use case.  Imagine two tightly-coupled userspace modules talking over shared memory in the incoherent cache; we don't necessarily want to have to modify the SOF framework or the Zephyr kernel to handle the communication, it would be nice for them to just map the shared region and do their thing.  But on Xtensa that requires cache management.

in this case there is an NXP platform with an ARM Cortex-M4 and a cache. According to the M4 spec there should not be a cache but NXP has added a custom one onto the chip

Wow, that's quite unusual. Are both functional at the same time? If they do, this is currently not supported (I mean, that would be very hard to support in a general way: you have to flush and invalidate twice every time?)

in this case there is an NXP platform with an ARM Cortex-M4 and a cache. According to the M4 spec there should not be a cache but NXP has added a custom one onto the chip

Wow, that's quite unusual. Are both functional at the same time? If they do, this is currently not supported (I mean, that would be very hard to support in a general way: you have to flush and invalidate twice every time?)

I think maybe we have a misunderstanding here, what I mean is there is only a non-standard cache. ARM Cortex M4 has no cache supported by it's specification, NXP basically added their own custom one despite it not being in the M4 specification. So this has caused problems for me because a lot of the ARM code that is conditionally compiled due to Kconfigs when an ARM core has a cache assumes that the core is an M7, because that is basically the only ARM v7-m core with a cache, when this is not the case. So I tried to get around the problem using the external cache API. You can see what I am describing in my PR here: #50871

ARM Cortex M4 has no cache supported by it's specification, NXP basically added their own custom one despite it not being in the M4 specification.

Ok, got it. In that case it's correct that you introduced an external cache controller driver


[WIP] Build call for weighted pool initialization:

Function "buildInitCall"
addLiquidityInitWeighted tests;

I think we could go a bit further on the idea of separating init from add liquidity and create a separate entity called InitPool, with the same structure as CreatePool. The way it's currently implemented, is still feels part of the add liquidity flow. I'm not completely sure on the impact of this, but I'd like to know what do you think of this idea. Do you think it's worth evaluating that change? Ps: I added a few small comments on the PR that would be nice to update as well.

My first version of the Add Liquidity Init was a different class, but I noticed that was possible to reuse parseAddLiquidityArgs and encodeUserData, so I moved the functionality to AddLiquidity, but we can also do the other way around, that is bring the 2 reusable functions to outside the AddLiquidity model, no problem, we just cannot have duplicated code.


Extensibletype's test_perfecthashing sometimes fails

There's a rare failure of perfecthashing unit test that showed up a few times in our CI builds. The error message can be slightly different, but I suspect the cause is the same. To reproduce:
$ for i in `seq 1000`; do python test_perfecthashing.py || break; done

test_methodtable (__main__.TestInterning) ...
1 table building took 0.000236988067627 seconds.
101 table building took 0.00116181373596 seconds.
201 table building took 0.00199198722839 seconds.
301 table building took 0.00313305854797 seconds.
401 table building took 0.00398087501526 seconds.
501 table building took 0.00597405433655 seconds.
601 table building took 0.00692415237427 seconds.
701 table building took 0.00718903541565 seconds.
801 table building took 0.00795102119446 seconds.
901 table building took 0.0124530792236 seconds.
1001 table building took 0.0133080482483 seconds.
1101 table building took 0.0155501365662 seconds.
1201 table building took 0.0147659778595 seconds.
1301 table building took 0.0160248279572 seconds.
1401 table building took 0.0165131092072 seconds.
1501 table building took 0.0170249938965 seconds.
1601 table building took 0.017324924469 seconds.
1701 table building took 0.0172760486603 seconds.
1801 table building took 0.0175180435181 seconds.
1901 table building took 0.0176138877869 seconds.
2001 table building took 0.0380890369415 seconds.
2101 table building took 0.0419509410858 seconds.
2201 table building took 0.0426738262177 seconds.
Error: No suitable table found
ERROR

======================================================================
ERROR: test_methodtable (__main__.TestInterning)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "test_perfecthashing.py", line 71, in test_methodtable
    build_and_verify_methodtable(ids[:i], flags[:i], funcs[:i])
  File "test_perfecthashing.py", line 22, in build_and_verify_methodtable
    table.generate_table(len(ids), ids, flags, funcs)
  File "methodtable.pyx", line 130, in extensibletype.methodtable.PerfectHashMethodTable.generate_table (build/objs/athena/src/numba/deps/pyextensibletype/extensibletype/methodtable.c:2682)
HashingError: Unable to create perfect hash table for table: PerfectHashMethodTable(
<snip>

This issue is obsolete as Extensibletype doesn't exist anymore, closing.


Essentially the geometry code is only robust to real-world size differences such as say 1 m over 1000 km, i.e. O(1e-6) differences. The CI was working down to O(1e-15) which will not give reliable results. So I have limited the precision of the geometry tests.