User management: Allow extraction of the student registration number from a part of the SAML2 attribute value

Checklist
General

[ ] I tested all changes and their related features with all corresponding user types on a test server.
(Not possible, as SAML2 authentication is not configured on the test servers.)
[x] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Server

[x] I followed the coding and design guidelines.
[x] I added multiple integration tests (Spring) related to the features (with a high test coverage).
[x] I implemented the changes with a good performance and prevented too many database calls.
[x] I documented the Java code using JavaDoc style.

Motivation and Context
Our SAML2 provider changed their configuration in a way such that the registration number now additionally includes some prefix instead of only the actual number. The admin cannot change this on their end. Therefore, the value of interest has to be extracted from the whole attribute value when received from Artemis.
Description
This PR introduces a new optional configuration parameter saml2.value-extraction-patterns. It contains a list of key-value pairs of the format:

key: The name of the SAML2 attribute for which the received value needs some additional extraction.
value-pattern: This is a regular expression that has to match the whole attribute value. It has to contain a named capture group. Only the part of the received value that is matched by this group is stored in Artemis.

If either the key pattern or the value pattern do not match the received attribute, then no transformation is applied to the value. Instead, the value is used as received from the authentication provider.
Steps for Testing
This cannot be tested on the test servers. Instead, start your local Artemis with the saml2 profile.

Do not enable any of the extraction key/value pairs in src/main/resources/config/application-saml2.yml. Artemis should start normally.
Enable just one of the two elements of a pair. Artemis should now fail to start.
Enable both key and value_pattern. Artemis should now start again.
Modify the value of a value_pattern such that it no longer contains a named capture group with the name value. Artemis should fail to start.
Fix the capture group name. Enable an additional key/value_pattern pair with the same key. Artemis should fail to start.

The actual functionality is hard to test without a proper SAML2 provider set up. If you have:

Not enabling the two parameters should not change the behaviour when authenticating via SAML2.
Choose a key/value pair where you can extract some part of the value using a regex. Check that only the part of the value that is matched by the capture group is used by Artemis.

Review Progress
Code Review

[ ] Review 1
[ ] Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Test Coverage

Class
Branch
Line

SAML2Service
75%
93%

Code looks fine, although I would suggest allowing to configure a set of pairs [(key1, pattern1), (key2, pattern2),...] whereas key is a SAML2 attribute and pattern is the extraction pattern (first group extracts the information). This makes this PR apply to more use cases.

Code looks fine, although I would suggest allowing to configure a set of pairs [(key1, pattern1), (key2, pattern2),...] whereas key is a SAML2 attribute and pattern is the extraction pattern (first group extracts the information). This makes this PR apply to more use cases.

@LDAP Very good suggestion :+1: , I implemented it in the latest commit.


Adding async operation support.

This PR adds the option to pass-in open, destroy, read, write, writev, and final as async function instead of a callback.

This gets really tricky if someone makes a function that takes a callback and returns a promise. We've struggled with this same issue in async-done and I don't think the complexity of the API is worth it. It almost seems like it should just be a wrapper of some sort.

I’m leaning towards just making a major that impls the internal _read/write etc with promises instead of cbs. Luckily we don’t have any public apis with cbs exposes on the stream itself so interop is trivial

@mafintosh have you done any profiling on creating so many intermediate promises? I've not, so I'm just curious.

This gets really tricky if someone makes a function that takes a callback and returns a promise.

It needs to be mutually exclusive. In the implementation as-is I relied on the skill of the programmer (either pass-in a regular fn or a fn returning a promise). I see that this might be a bit naive, so how about adding a guard for regular function to exit with an error or show a warning if a regular function returns a promise (wrong API use).


Add theme support for edxapp's LMS

Purpose
disclaimer this PR is a replacement for #105

Adding a custom theme to an Open edX site must be done upon deployment and not in the common
Docker image because it is specific to each customer.

Proposal

[x] add edxapp new settings for theme url/tag
[x] make the LMS BuildConfig more versatile to checkout and build custom theme
[x] add custom theme example for patient0-development

Closes #105

Superb!


MMHDT 614 adjust accomack county gbloc

MMHDT-614
Integration PR
Summary
All ACCOMACK county GBLOCs must be set to be "BGNC". Currently they are split between two GBLOCS of "BGNC" and "BGAC"

was this already applied in prod?

conflicts

was this already applied in prod?

yes, this is just the migration script because it was meant for all environments. Helps with the git history


Navigation Collection Cleared after first SaveChanges

Hi,
I am having an issue when updating a collection of an existing entity.  Everything is OK when I call _dbContext.SaveChanges() the first time.
Before calling SaveChanges(), all items in collection are marked as EntityState.Added which is Ok, and when I call SaveChanges() the items are inserted into the database.
But after that, the navigation collection is cleared and its items are marked as EntityState.Deleted, and if I call SaveChanges() again they are deleted from the database.
See structure below (simplified):
public class Parent
{

    public Parent()
    {
        this.Children = new List<ParentChild>();
    }

    public int Id { get; set; }

    public string Name { get; set; }

    public virtual IList<ParentChild> Children { get; set; }
}

public class Child
{
    public int Id { get; set; }

    public string Name { get; set; }
}

public class ParentChild
{
    public int ParentId { get; set; }

    public int ChildId { get; set; }

    public int SortOrder { get; set; }

    public virtual Parent Parent { get; set; }

    public virtual Child Child { get; set; }
}

The update method is as follows:
int parentId = 100;

Parent entity = _dbContext.Parents
    .Include(x => x.Children)
    .Where(x => x.Id == parentId)
    .FirstOrDefault()
    ;

entity.Children = new List<ParentChild>
{
    new ParentChild
    {
        ParentId = parentId,
        ChildId = 20,
        SortOrder = 1
    },
    new ParentChild
    {
        ParentId = parentId,
        ChildId = 25,
        SortOrder = 2
    },
    new ParentChild
    {
        ParentId = parentId,
        ChildId = 30,
        SortOrder = 3
    }
};

_dbContext.SaveChanges();

//some extra updates here

_dbContext.SaveChanges();

Am I doing something wrong?

@fromberg100 The update method does this:

Loads all existing children
Removes all existing children from navigation (because the collection is replaced) and replaces them with three new children
SaveChanges will now delete all the existing children and insert all new children

If that's not your intention, then that might be the issue. Otherwise, please post a small, runnable project/solution or complete code listing that demonstrates the behavior you are seeing.

I was not clearing the collection, however, I tried and also got the same results.
I missed to include that I override Equals:
public class ParentChild
{
    public int ParentId { get; set; }

    public int ChildId { get; set; }

    public int SortOrder { get; set; }

    public virtual Parent Parent { get; set; }

    public virtual Child Child { get; set; }

    public override bool Equals(object obj)
    {
        if (obj == null)
        {
            return false;
        }

        var other = (ParentChild)obj;

        if (!Object.Equals(this.ParentId, other.ParentId))
            return false;
        if (!Object.Equals(this.ChildId, other.ChildId))
            return false;

        return true;
    }
}

When I do not override Equals, then it works ok.  I believe this is related to issue #16161

@fromberg100 Please post a small, runnable project/solution or complete code listing that demonstrates the behavior you are seeing.

please find solution enclosed.  Please read carefully the comments I wrote specifically on top of method SetSameChildren on file Program.cs (line no. 52)
If there is anything unclear with, please let me know.
UpdateCollection.zip

@fromberg100 Confirmed this is a duplicate of #16161. However, note that defining Equals and GetHashCode in this way goes against best practice for .NET code since it depends on mutable state which makes the hashing unstable. See https://github.com/aspnet/EntityFrameworkCore/issues/15687#issuecomment-492280606


Fix creation of 5 svg:paths for 1 connection, should be 1:1

Better connection/point event emission, now is 2:1


CG Demo: enabling/disabling certain functionality

@bhargavsk1077 @ChartistDev We can pick these changes tomorrow. Let's target to get this branch merged in the next few days.


Add common package for basic java primitives Oak specialization

This is a preliminary step to reduce the footprint of future API changes.

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

This is regarding your main comment. I hope you are reviewing only "liran-funaro:pr-common-primitives" commit. You shouldn't see any of  "Anastasia's PR". The reason is that this PR is based on Anastasia's branch (not yet committed) for easier future merge.


Modified the test_rancher_cli.py file to use only docker compose files

Removed files which are not relevant
Removed rancher-compose files from directory resources/rccmds/v2
Added comments to catalog list testcase

@sangeethah : Please review. For the tests failing, issues have been filed


Problem with inline admin integration

I seem to have a problem with the HTMLField inside a custom plugin with Inlines created inside Django CMS.
When I enter inside the modal in the front-end to edit the plugin, all the inlines with HTMLField works, but when I press the add-row button, the next inline starts with a Textfield instead of the Ckeditor field.
The function use to call the CKeditor remains like this : initCMSCKEditorid_slide_image___prefix___text(); inside the new Inline created.

+1

+1

sorry for the late delay @nicolas8448, would you be able to provide a screenshot so we understand the problem better?

please re-open if its still relevant to you. A screenshot or screencast would be welcome to understand the problem.


Drop EF Core content from the Blazor DI topic

Drop EF Core content after https://github.com/dotnet/AspNetCore.Docs/pull/18981 merges.
@JeremyLikness Please confirm my plan here. After we merge and take your PR content live, I'll circle back around and work this issue to delete the EF Core section from the DI topic ...
https://docs.microsoft.com/en-us/aspnet/core/blazor/fundamentals/dependency-injection?view=aspnetcore-3.1#use-of-entity-framework-dbcontext-from-di

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

ID: 7683690c-e3aa-4a00-152a-e44cfd7fc789
Version Independent ID: d9aabe95-69e9-0a03-81ff-429914137f9d
Content: ASP.NET Core Blazor dependency injection
Content Source: aspnetcore/blazor/fundamentals/dependency-injection.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

@guardrex That is correct - confirmed. /cc @danroth27 and @javiercn


[Addition]: Yet another cluster of self-witnessing "Deepers" in Hamburg, Germany

Hotspot b58 Addresses
11C2tfxFmJf3P6CQkwASrdBS15HcTQk7m3gy1Z91tdJk5ECi6YP
112nNwmSKdbEawL8ZrWdypuq8HtVGgALAWA2WUxxiaJxTZ6hicn2
112aBUz9JntSQpyrQTtuV3X3sf3Enqq72cB2C2JiPVo91kkLukZM
11rc8HvFqxTk52inzGuY1Lb8jveTTh3hKbuFThLuLkzxtGAvWua
11JSJ3K1X321bewyjdMwHUtoRpdnZ2iK63cTpw2FkwY2kLwHVwY
112k7HGT1WmQynRGXsEDhmMYuFKVXSQuX2m6xB4emCnckycyQGxj
1125yUT388AdNKUnXp4SnEZyJi7ZWrda1MCA8RMSeNWgFiYtDw1d
Discord Handle
georgi#0751
Reason(s)
I saw one of those hotspots in new my hex today and was wondering why my hotspot can't witness it. Then I saw he witness only few other hotspots with impressive RSSI. I checked them and found seven self-witnessed hotspots. There is no communication with any other hotspots.
11C2tfxFmJf3P6CQkwASrdBS15HcTQk7m3gy1Z91tdJk5ECi6YP - Docile Lava Cow
112nNwmSKdbEawL8ZrWdypuq8HtVGgALAWA2WUxxiaJxTZ6hicn2 - Sunny Emerald Badger
112aBUz9JntSQpyrQTtuV3X3sf3Enqq72cB2C2JiPVo91kkLukZM - Passive Pearl Ape
11rc8HvFqxTk52inzGuY1Lb8jveTTh3hKbuFThLuLkzxtGAvWua - Mammoth Neon Bobcat
11JSJ3K1X321bewyjdMwHUtoRpdnZ2iK63cTpw2FkwY2kLwHVwY - Old Blood Cobra
112k7HGT1WmQynRGXsEDhmMYuFKVXSQuX2m6xB4emCnckycyQGxj - Melodic Arctic Lobster
1125yUT388AdNKUnXp4SnEZyJi7ZWrda1MCA8RMSeNWgFiYtDw1d - Smooth Iron Ape

All hotspots have been included in
https://github.com/helium/denylist/pull/2471
--> You can close

Thank you. Closed.


Add an icon of the current connected DB client somewhere in the query-browser page

This way we know which are the current DB client and which SQL language we can use.

Closed on https://github.com/sqlectron/sqlectron-gui/commit/f0284a92ec484da1d8ec0dd37b3271c40493d8fd


[vtk] enables openvr module when openvr is listed

This PR sets the VTK_MODULE_ENABLE_VTK_RenderingOpenVR flag when openvr is in the feature list. Previously, adding this feature did nothing other than changing the dependencies.

Hi @veggiesaurus
Thanks for your PR.
Have you tested this feature openvr ?

Hi @veggiesaurus
Thanks for your PR.
Have you tested this feature openvr ?

Yes, I'm using it now. When compiling with this feature, the correct headers and libs now get copied to the install directory

LGTM now.

Thanks for the improvement!


fix issues to be compatible with latest peft

What does this PR do?

Fixes the issues https://github.com/huggingface/peft/issues/286 and https://github.com/huggingface/peft/issues/317
Adds Callback to be used with HF Trainer to make sure intermediate checkpoints are saving only peft adapters, solving https://github.com/huggingface/peft/issues/286#issuecomment-1506850018
Adds a debug_mode arg to quickly test out the fine-tune script on tiny subset of dataset

Your pull request isn't working. Crashed when it tries to save a checkpoint, was training on 8x RTX 3090.

@lksysML if you're training in 8bit, there's a separate bug with bitsandbytes latest version that causes a massive VRAM spike when saving files. Can bypass it by backdating: pip install bitsandbytes==0.37.2

Yea, I ran into that issue yesterday and fixed it already. I think that has something to do with peft, I rolled back to an old version and it didn't crash.
WandB report of crash with your pull request: https://wandb.ai/lksy/huggingface/runs/hxq051bg/log
Also, debug_mode doesn't work. I set it to True and it continued with the whole training set instead of the 1024 examples as it supposed to.

@pacman100
I'm a bit confused about the intended functionality here.
I've that, when attempting to train a new adapter after loading the weights of the old one, that I get bad results (mainly because of my small dataset size, which is a requirement for my implementation).
I was able to get good results by continuing the training of a previous adapter instead (and which was the functionality that broke).
Now, we save the adapter model in a new folder

We delete the pytorch_model.bin (which is the full weights, not something the OJ code ever needed, which makes sense. It was saving adapter_model.bin as pytorch_model.bin before)

But then the resume_from_checkpoint code is still searching for pytorch_model.bin

I can move the adapter out of the folder and rename it, in an attempt to get the previous functionality, but then

It's not actually fixed. And the conflicting nature of this PR's code tells me that this functionality wasn't tested anyways.

@ElleLeonne
I did test and it does work
Continuing from the ckpt of the above run via:
python finetune.py     --base_model 'path/to/it'     --data_path 'yahma/alpaca-cleaned'     --output_dir './lora-alpaca-2' --debug_mode --num_epochs=3 --cutoff_len=512 --lora_target_modules="[q_proj,k_proj, v_proj, o_proj]" --lora_r=16 --micro_batch_size=8 --resume_from_checkpoint ./lora-alpaca/checkpoint-20/adapter_model/

results in it being correctly loaded:

In the above run, the eval loss at the end was 1.70. In the run here using above command: https://wandb.ai/smangrul/huggingface/runs/18ux1bhz?workspace=user-smangrul, the eval loss starts at 1.65 and ends at 1.48

@ElleLeonne
I did test and it does work

@pacman100
I realize this isn't your repo, so I appreciate you taking the time to investigate this and help improve our little project here.
Using his method will cause the code to perform the same functionality as loading the adapter, then starting a new training run. This is because this block

will never execute, now that we're removing the pytorch_model.bin. (previously, there was a binding script that would save the lora_adapter's state dict as pytorch_model.bin. It was the same file, just named differently.) I can attempt to preserve this functionality by moving the adapter to the main folder and renaming it, but then I encounter the previous error.
As a result,

the trainer will literally never try to load the state of the old training session. It will instantiate a new training session, using a instance of warmup steps, new training loss calculations, and a new starting point for the adapter. It functions more akin to merging the weights with the model and starting a new training run, if I understand the process correctly
Which I guess is just to say that this PR doesn't restore the old functionality of the code. It just cuts out half of what the old code used to do. It would be really nice to be able to load the trainer's state when continuing training again.

Er... the logic of the code there is:

if pytorch_model.bin exists, then:

Load the checkpoint file and use it
resume_from_checkpoint is True, telling trainer.Train to load the file again (erroneously loaded twice in pre-existing code)

if it does not exist, then:

set resume_from_checkpoint to False to tell trainer.train that it should not load any file
check if adapter_model.bin exists, if it does:

load the adapter_model file and use it
train based on that

else:

train a new model

the torch.load + set_peft_model_state_dict lines are what enable resuming a prior checkpoint. the resume_from_checkpoint being sent to trainer.Train is a pre-existing code mistake.
For reference, train's documentation for the resume_from_checkpoint arg:

resume_from_checkpoint (str or bool, optional):
If a str, local path to a saved checkpoint as saved by a previous instance of [Trainer]. If a
bool and equals True, load the last checkpoint in args.output_dir as saved by a previous instance
of [Trainer]. If present, training will resume from the model/optimizer/scheduler states loaded here.

resume_from_checkpoint is True, telling trainer.Train to load the file again (erroneously loaded twice in pre-existing code)

I don't think this is true. There was a very clear marked difference in quality with the old, working version of resume_from_checkpoint, and the current version of resuming from just the adapter. It wasn't just a coding mistake.

The loading of adapter_model.bin (ie the correct file that is intended to be used) means resuming from prior saves (of adapter_model.bin) does in fact work.

I know it works. My point was that this is only half the required solution. I was able to get this solution working myself without the PR. The results are just bad.
https://github.com/tloen/alpaca-lora/pull/154#discussion_r1147950135
See the original PR for the update in question, too.

@ElleLeonne,
this gist https://gist.github.com/pacman100/8e7a6eedabf34e1a88dd74a96c3b619f should exhibit the behaviour that you are looking for.
But, it doesn't make much sense to me, could you provide a concrete example of how the existing code with previous peft was being used and how this PR is failing to do that with some concrete metrics?

due to linear decay, the final ckpt  of existing lora ckpt will have learning rate close to 0 and so what you were earlier doing was essentially training with very low lr
IF you were adding the new small dataset to the initial dataset to increase the number of steps and loading the previous ckpt for continuation, then it would have been randomly shuffled and the number of steps till the previous ckpt would have been skipped which most likely would contain many samples from the new small dataset.

Both the above points are a weird way of continuing training.

I appreciate that. I'll play around with it and see how everything works out.
The idea was basically to introduce the new data as a new "epoch", which helped include the relevant data without needing to fully retrain the adapter every night (which takes about 7 hours for me). Then I can delay needing to spin the whole adapter retraining up for a longer period of time, while still showing noticeable improvements day-to-day. It also fixed a major problem where sometimes training would fail in the middle of the night, and I'd need to use the old adapter until I could fix the problem.
I would increase the epoch from (let's say) 5 to 10, so the learning rate decay would still exist, but not be 0.

I dont know if this is the RIGHT way, but this simple modification produces a adapter_model.bin with the right size:
- model.save_pretrained(output_dir)
+ model.save_pretrained(output_dir, state_dict=old_state_dict)


Add to Django Packages -> Markdown Category

Please add this package to the relevant Markdown Category in the Django Packages web site.
It should be the only one working with Django-2.2 and Python-3.7 out of the box, although #76 reports adversely.

Thanks, I've added it to that grid now.


Prevent very large reverts based on finality certificates

Lotus is not designed to handle only a few large reverts (for sure not 900+ epochs). Introduce a way to prevent that in Lotus and notify the operator.

Turns out we already handle this case: https://github.com/filecoin-project/lotus/blob/02a8b972db82eee0ff109366f57122caf7790128/chain/store/store.go#L813-L816 (although for the wrong reasons... I'll update the comment).

Ah... nvm. That refuses to go that far back into the past, but will still fork if the common ancestor is that far back.

Actually, that's fine? I don't want to go back 900 epochs, but if 2/3rds of the power does that and then makes process by finalizing tipsets < 900 epochs ago on the fork... I might as well switch. So I think we're good as-is.

Turns out we already handle this case: https://github.com/filecoin-project/lotus/blob/02a8b972db82eee0ff109366f57122caf7790128/chain/store/store.go#L813-L816

Is it this case? I don't think it is, the 900 is how many blocks can get reverted (so it is head.Epoch - commonAncestor.Epoch), this only says that you cannot set a checkpoint that is too far in the past

See my later comments. But given F3's 100 epoch max-per-instance, we could end up reverting 1000 epochs, but no more.
Yeah, we should probably fix this regardless.

Hm.... well, that would be a very very special case. Basically, worst-case is that we could revert 900 epochs plus the worst-case natural revert (most likely 903-ish).

Eh, I'm just going to fully fix this so it's easy to reason about.


Allow both time and Time as column names

I kept getting the error "Must have a time column" when I was loading a timeseries. but I did have a time column!  The problem was I called it Time rather than time. I think the code should be more forgiving to users and allow time and Time as legal time variable names.

Would you be okay if the column is called "time" but the code accepts "Time" and changes it to "time"? We need to have a single name for the column.

Yes that is ok. to a biologist time and Time are the same thing.
H
On Sat, Jul 25, 2020 at 11:59 AM Joseph Hellerstein <
<EMAIL_ADDRESS>wrote:

Would you be okay if the column is called "time" but the code accepts
"Time" and changes it to "time"? We need to have a single name for the
column.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sys-bio/teUtils/issues/3#issuecomment-663889219, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIBSDRT3XLJBB3HTIJ6YATR5MTRDANCNFSM4PHR4CZQ
.

--
Herbert Sauro, Professor
University of Washington, Bioengineering
206-685-2119, www.sys-bio.org
<EMAIL_ADDRESS>Books: http://books.analogmachine.org/

The new implementation allows time to be in any mix of cases. So TimE and tiME also work.


Fix #3305: Restore .gitattributes and fix baselines.

Fixes #3305

@Cellule could you look at the new set of files fixed? Since my original PR, it looks like some files were fixed and some new incorrect line endings were added. Local, VSO, and GitHub CI are all showing no problems now.

hum it's all wasm stuff... I think my clone is probably messed up. I will reclone after you merge this.
Looks good though

Unfortunately auth is having an outage so I can't use the signoff website right now. Any luck on your end?


Image is not rendering on ios 14 when serving from remote url.

To Reproduce

use Xcode -beta for ios 14
Serve image from backend via url
use FastImage
Image is not displaying

Expected behavior
The image should render on ios 14 platform
Dependency versions
react-native-cli: 2.0.1
react-native: 0.59.10
"react-native-fast-image": "^7.0.2"

Probably related to https://github.com/SDWebImage/SDWebImage/issues/3040

@omprakashnitdgp just update SDWebImage pod to 5.8.3 and it should work

@t3chnoboy How can I update SDWebImage pod?
I didn't install SDWebImage just use fastImage

@sionel pod update SDWebImage or you can remove Podfile.lock and run pod install --repo-update

thanks @t3chnoboy Had the same issue, that fixed it for me
pod update SDWebImage and the done a reinstall of my pods

Does anyone has a clue if it's possible to update SDWebImage for a project that's not using pods? We're locked in RN < 0.60 and we don't manage our dependencies with pods.
This is causing react-native-fast-image to not be able to render images in iOS 14.

I'm having the same issue. Also on RN 0.59

I'm having the same issue on RN 0.59.9. So, we also don't use pods. Anyone has a solution on how to update SDWebImage on RN 0.59 ?

Solution for RN 0.59

cmd + shift + f in xCode
search for CALayerDelegate
use this fix https://github.com/facebook/react-native/issues/29279#issuecomment-658244428

I found displayLayer function in path


Use device without delay

Hi,
this crate has great features and runs nicely on my embedded controller (stm32f103).
However, I cannot use it, because it borrows a delay object. However, on my embedded controller  I only have one delay object and using it with the mpu6050 struct prevents me from using simple delay_ms, even though I only want to use e.g. get_acc_avg.
It would be great if the functions that don't require a timer can be executed without a delay object.
(But maybe it's only me not knowing the trick to share a delay...)
Greetings!

this crate has great features and runs nicely on my embedded controller (stm32f103)
great to hear!

my mpu6050 was faulty, so at the moment I cannot test it or change it :/ Maybe i'll get a hold on one during the christmas time.
I honestly have never thought about your issue. I only use the delay object once internally during startup (wait 100ms, according to data sheet). In the example I provide, I should probably use it during the loop to limit the rate, as well.
Could you educate me further?

why do you need a separate delay object and can't reuse your existing one? I haven't read too deep into the embeddel_hal docs regarding this

Hi and thanks for your reply. As already mentioned, I do not have very good insights into the embedded hal either, but I'll try

why do you need a separate delay object and can't reuse your existing one? I haven't read too deep into the embeddel_hal docs regarding this

Let's assume I have the following code:
// ...
// Create delay object. This can only be done once due to hardware restrictions
let mut delay = Delay::new(cp.SYST, clocks);
// use delay
delay.delay_ms(100 as u32);
// initialize and use mpu
let mut mpu = Mpu6050::new(i2c_bus, delay);
let acc = mpu.get_acc().unwrap();

// later on use the delay again. For whatever reason
delay.delay_ms(100 as u32);

This does not compile, because the mpu object now "owns" the delay object:
error[E0382]: borrow of moved value: `delay`
   --> src/main.rs:186:5
    |
101 |     let mut delay = Delay::new(cp.SYST, clocks);
    |         --------- move occurs because `delay` has type `stm32f1xx_hal::delay::Delay`, which does not implement the `Copy` trait
...
173 |     let mut mpu = Mpu6050::new(i2c_bus, delay);
    |                                         ----- value moved here
...
186 |     delay.delay_ms(100 as u32);
    |     ^^^^^ value borrowed here after move

On a microcontroller, I only have one delay object. This is probably due to the inherent seqential nature of microcontroller code without an OS.
So I cannot use this crate in my project, because I also need access to the delay. AFAIK, there is no module, that provides "sub-delays" (yet). Maybe something like RTFM does this.
I could imagine, a solution would be to pass the delay as a function parameter to the init function and return it after the initialization, so that ownership can be transferred back to the caller.
BTW: you wait twice in a row in your wake function (probably by accident): Here: https://github.com/juliangaal/mpu6050/blob/39f441d4dec903919f4dea026f54d38514ebe9ae/src/lib.rs#L353 and here:  https://github.com/juliangaal/mpu6050/blob/39f441d4dec903919f4dea026f54d38514ebe9ae/src/lib.rs#L553
Best Regards!

This does not compile, because the mpu object now "owns" the delay object

The borrow checker :D  I understand. I looked at this crate for bigger brother of the chip and you can see that they only use a mutable reference &mut delay in the self::init_mpu(...) method.

https://github.com/juliangaal/mpu6050/blob/39f441d4dec903919f4dea026f54d38514ebe9ae/src/lib.rs#L553
Actually, this wasn't a mistake. It was recommended to put in a delay after writing in the datasheet. But I guess it has to work without it as well, and isn't entirely my responsibility :D

As I said I don't have access to the chip right now. Would you be willing to send a PR?
https://github.com/juliangaal/mpu6050/blob/39f441d4dec903919f4dea026f54d38514ebe9ae/src/lib.rs#L327
would have to be changed to pub fn new(i2c: I, &mut delay: D) but can't be sideffect free anymore, because
https://github.com/juliangaal/mpu6050/blob/39f441d4dec903919f4dea026f54d38514ebe9ae/src/lib.rs#L358
must be called and pass the &mut delay to
https://github.com/juliangaal/mpu6050/blob/39f441d4dec903919f4dea026f54d38514ebe9ae/src/lib.rs#L351

any update, thoughts? ;)

I hope I find some time this weekend. Then I'll try to create a PR

Related: https://github.com/stm32-rs/stm32f1xx-hal/issues/131

I started to implement this feature, but somehow I have problems on the embedded board. One reason might be the absence of a fpu on the uC... But I don't know when I can find enough spare time soon to finish it.

I'll order a new unit before christmas ;)

I couldn't test the updated version on branch non-owning-delay bcs of family and other things. Could you checkout this branch? You can add the branch to your Cargo.toml, for example
[dependencies]
rand = { git = "https://github.com/rust-lang-nursery/rand", branch = "next" }

Have a look at the updated example for the API change. I think I'll get rid of the side effect new(...) constructor in the future, but for now, would you mind testing like shown in the example?

Hey, I can confirm that the code in the non-owning-delay branch works.

perfect, thanks. I'll merge into master then


upgrade log, remove kv-log-macro

Removes kv-log-macro, upgrades the log crate. Kv logging is now allowed by the macros so this gets rid of an extra dependency. Thanks!

LGTM, I'm merging.


Added new post about design and front-end development

Hey @aktuskey. 👋
I just saw this guest post. You shared a great perspective... thanks for writing it up!
Currently this PR is submitted to your fork of the repo. In order for me to merge it in, we'll need to resubmit another PR to the main (base fork: bryanbraun/bryanbraun.github.io) repo.
I was going to ask you to do it, but apparently I have the ability to do it too (since they are both public projects), so I went ahead and created it here: https://github.com/bryanbraun/bryanbraun.github.io/pull/23
That's the one that I'll merge when the post gets published.
Thanks again for contributing!


MusicBrainz::Recording Not Defined

The Readme states that MusicBrainz::Track is deprecated and to use MusicBrainz::Recording, but MusicBrainz::Recording is not defined in version 0.7.7 (latest).
MusicBrainz::Track.search returns nil, even when the search is valid, every time as well.

This was an issue with rubygems.org (I'm guessing?), fixed with gem 'musicbrainz', git: 'https://github.com/localhots/musicbrainz.git' in my Gemfile.


No Authorized Tessels Found

When I try to get connected using "t2 root", I got the following error
C:\Users\I008123\tessel-code\ble>t2 root
INFO Looking for your Tessel...
WARN No Authorized Tessels Found.
Even thought, we have lan connection:
C:\Users\I008123\tessel-code\ble>t2 wifi list
INFO Looking for your Tessel...
INFO Connected to Setu.
INFO Connected to "iPhone de Jose"
INFO IP Address: <IP_ADDRESS>
INFO Signal Strength: (63/70)
INFO Bitrate: 1mbps
When I list the devices
C:\Users\I008123\tessel-code\ble>t2 list
INFO Searching for nearby Tessels...
USB     Setu
I only got the USB connection, and the "t2 provision" had run before
C:\Users\I008123\tessel-code\ble>t2 provision
INFO Looking for your Tessel...
INFO Connected to Setu.
INFO Authenticating Tessel with public key...
INFO Tessel is already authenticated with this computer.

Can you try deploying something simple, but explicitly via lan? This should be all that's needed:
mkdir t2-simple && cd t2-simple && t2 init && t2 run index.js --lan

Here's my output:
$ mkdir t2-simple && cd t2-simple && t2 init && t2 run index.js --lan
INFO Initializing new Tessel project for JavaScript...
INFO Created ".npmrc".
INFO Created ".tesselinclude".
INFO Created "index.js"
INFO Created "package.json".
INFO Looking for your Tessel...
INFO Connected to bishop.
INFO Building project.
INFO Writing project to RAM on bishop (3.072 kB)...
INFO Deployed.
INFO Running index.js...
I'm blinking! (Press CTRL + C to stop)

@i008128 you could try to toggle the firmware version of the tessel:
t2 version 0.0.16 and in a second step t2 version 0.1.0
After that you try things @rwaldron suggested again (maybe something was broken within the tessel).


fix: fix queryParams in C# when they contain a  json array (box/box-codegen#470)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

16 of 16 (100.0%)  changed or added relevant lines in 1 file are covered.
4 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.8%) to 77.034%

Files with Coverage Reduction
New Missed Lines
%

Box.Sdk.Gen/Internal/OneOfT3.cs
1
22.22%

Box.Sdk.Gen/Schemas/FileFullOrFolderMiniOrWebLink.cs
3
31.58%

Totals

Change from base Build<PHONE_NUMBER>:
0.8%

Covered Lines:
6671

Relevant Lines:
8509

💛  - Coveralls


Questions about release 1.0

What needs to be done before ? I can help.

It'd be great to get polka to 1.0. What can be done to help? Is there a list of things you want to get done before 1.0?

@lukeed Was just wondering about this too. Is it time to move @next to 1.0? Is anything missing?

@lukeed Is there anything community could help with to push the 1.0.0 over the finish line?


Each supported external content type is now advertised in

Accept-Post response header.
Resolves: https://jira.duraspace.org/browse/FCREPO-2585
To test this feature:

Fire up the one-click
curl -v -XOPTIONS http://localhost:8080/rest/
Verify that the Accept-Post header containers the following option:
message/external-body; access-type=URL

Interested parties
Tag (@ mention) interested parties or, if unsure, @fcrepo4/committers

> curl -ufedoraAdmin:fedoraAdmin -v -XOPTIONS http://localhost:8080/rest
*   Trying ::1...
* TCP_NODELAY set
* Connected to localhost (::1) port 8080 (#0)
* Server auth using Basic with user 'fedoraAdmin'
> OPTIONS /rest HTTP/1.1
> Host: localhost:8080
> Authorization: Basic ZmVkb3JhQWRtaW46ZmVkb3JhQWRtaW4=
> User-Agent: curl/7.54.0
> Accept: */*
>
< HTTP/1.1 200 OK
< Date: Thu, 09 Nov 2017 17:05:24 GMT
< Link: <http://localhost:8080/static/constraints/ContainerConstraints.rdf>; rel="http://www.w3.org/ns/ldp#constrainedBy"
< Accept-Patch: application/sparql-update
< Accept-Post: text/turtle,text/rdf+n3,text/n3,application/rdf+xml,application/n-triples,application/ld+json,multipart/form-data,application/sparql-update,message/external-body; access-type=URL
< Allow: MOVE,COPY,DELETE,POST,HEAD,GET,PUT,PATCH,OPTIONS
< Content-Length: 0
< Server: Jetty(9.3.1.v20150714)
<
* Connection #0 to host localhost left intact

👍


[BUG]: Clicking RESET in Wizard mode does not visually change

Scenario: Model Edit
Step: 7.2
Link to the project: https://app.staging.terarium.ai/projects/c5a3cb4b-7a37-4037-9f7b-f893cf206e6a/workflow/947fe5c6-a918-4d7c-88c5-c50d0e7e7bc7?operator=be85ed07-9058-4a05-905f-94466b49f6b7

After following the steps through to 7.1, 7.2 asks one to press RESET, if one WIZARD the reset does not visually seem to happen, if on NOTEBOOK it does

Remove the model-edit test


OpenRCT2 Launcher says it can't connect to internet even though I have internet connection

see title. On my Surface Pro 3 I can't download OpenRCT2 though the launcher because it can't connect to internet. I'm on Windows 10 Pro. I have internet connection through Wi-Fi network.

Is google reachable? It pings Google.com to see if an internet connection is available.

Op 31-aug.-2015 om 19:11 heeft Luke Davies<EMAIL_ADDRESS>het volgende geschreven:
see title. On my Surface Pro 3 I can't download OpenRCT2 though the launcher because it can't connect to internet. I'm on Windows 10 Pro. I have internet connection through Wi-Fi network.
—
Reply to this email directly or view it on GitHub.

It's reachable. However, I can access google from Microsoft Edge, Chrome, Firefox, etc. I don't know why the launcher wouldn't connect to the internet.

Does it say it doesn't work, or does it really not work. The massage should only restrict the fancy images, the updater itself should still work.

Op 31-aug.-2015 om 19:19 heeft Luke Davies<EMAIL_ADDRESS>het volgende geschreven:
It's reachable. However, I can access google from Microsoft Edge, Chrome, Firefox, etc. I don't know why the launcher wouldn't connect to the internet.
—
Reply to this email directly or view it on GitHub.

The Updater didn't download OpenRCT2 on it's own. I just says "Downloading" at the click of "Launch OpenRCT2" (does nothing) and at the launcher main Splash (white splash) it says "offline mode: no internet connection found."

This will no longer be an issue in the new launcher.


UofT-DSI | Production - Assignment 2

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
I have populated the assignment 2 notebook.
What did you learn from the changes you have made?
Working with column transformer, pipeline, train-test split, cross-validation, prediction and prediction probabilities, and different metrics.
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
No, the instruction was very clear.
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
No.
How were these changes tested?
Running the code tests.
A reference to a related issue in your repository (if applicable)
Checklist

[x] I can confirm that my changes are working as intended

A2 complete


Gravity not resetting when switching hero

In GitLab by @MaxwellJung on Oct 1, 2023, 11:28

In GitLab by @ghost1 on Oct 1, 2023, 11:42
:tada: This issue has been resolved in version 0.10.3 :tada:
The release is available on GitLab release.
Your semantic-release bot :package: :rocket:

mentioned in commit 12a81383c4d60933e4b479962a65c03490cdf9b0

created branch 449-gravity-not-resetting-when-switching-hero to address this issue


Incorrect thumbprint does not result in failure

Affected Puppet, Ruby, OS and module versions/distributions

Puppet: 4.10.4
Ruby:  2.3.1p112 (2016-04-26) [x86_64-linux-gnu]
Distribution: Ubuntu 16.04
Module version: 2.2.0

How to reproduce (e.g Puppet code you use)
sslcertificate { "Install MyCert" :
name       => 'MyCert.crt',
location   => 'C:\temp',
store_dir  => 'Root',
root_store => 'LocalMachine',
password   => '',
thumbprint => 'FOOBAR',
require    => File['C:\temp\MyCert.crt'],
}
MyCert.crt contains a single root level certificate with no intermediates. The thumbprint value does not match this certificate.
What are you seeing
MyCert gets installed into CA rather than Root. The puppet run terminates without an error.
What behaviour did you expect instead
MyCert would not get installed and the puppet run would display an error, "Thumbprint 'FOOBAR' was not found n MyCert.crt."
Output log
Notice: /Stage[main]/win_instrument::Config/Sslcertificate[Install MyCert]/File[inspect-MyCert.crt-certificate.ps1]/content:
Info: Computing checksum on file C:/temp/inspect-MyCert.crt.ps1
Info: /Stage[main]/win_instrument::Config/Sslcertificate[Install MyCert]/File[inspect-MyCert.crt-certificate.ps1]: Filebucketed C:/temp/inspect-MyCert.crt.ps1 to puppet with sum 5079a791910bf353a5206d74cbb21bde
Notice: /Stage[main]/win_instrument::Config/Sslcertificate[Install MyCert]/File[inspect-MyCert.crt-certificate.ps1]/content: content changed '{md5}5079a791910bf353a5206d74cbb21bde' to '{md5}5b0ce7c056d180dcd99b3004282e96bf'
Notice: /Stage[main]/win_instrument::Config/Sslcertificate[Install MyCert]/File[import-MyCert.crt-certificate.ps1]/content:
Info: Computing checksum on file C:/temp/import-MyCert.crt.ps1
Info: /Stage[main]/win_instrument::Config/Sslcertificate[Install MyCert]/File[import-MyCert.crt-certificate.ps1]: Filebucketed C:/temp/import-MyCert.crt.ps1 to puppet with sum 577c9b403e7cfda694853dd4d4240140
Notice: /Stage[main]/win_instrument::Config/Sslcertificate[Install MyCert]/File[import-MyCert.crt-certificate.ps1]/content: content changed '{md5}577c9b403e7cfda694853dd4d4240140' to '{md5}8eab18b06d4642ca8bc6b4afde4fd47c'
Notice: /Stage[main]/win_instrument::Config/Sslcertificate[Install MyCert]/Exec[Install-MyCert.crt-SSLCert]/returns: executed successfully
Any additional information you'd like to impart

Hi,
based on a discussion in https://groups.io/g/voxpupuli/message/449 we decided to archive this repository. I'm going to close all issues and PRs. If you're interested in maintaining the module, please respond to our mailinglist.


gerrit-357849-2

Thank you!

One failing?

@zoranzoki21 there were some issue with testLinkMethods, but it could be caused by changes on wikipedia site, as we don't have proper unittests and we check some functionality using live sites.
I'm closing this one here, as it sees that 3rd patch for this gerrit has been merged.


Check and fix newer Intel compilers

[x] check if compilation works with newer Intel compilers

Works on sigma/tetralith with Intel 2023 after minor update: https://github.com/tdep-developers/tdep/commit/dd17f197c6689d40f5b71d24cfcfe0a038ec6391
Closing.


Failed to run WA3

Hi WA team,
I was using WA2 successfully. Now I'm switching to the WA3 dev branch 'next'.
I followed the instructions in the README.rst file and installed WA3 but when I try to run it I get:
Traceback (most recent call last):
File "/usr/local/bin/wa", line 16, in
from wa.framework.entrypoint import main
File "/usr/local/lib/python2.7/dist-packages/wa/init.py", line 13, in
from wa.framework.output import RunOutput, discover_wa_outputs
File "/usr/local/lib/python2.7/dist-packages/wa/framework/output.py", line 8, in
from wa.framework.configuration.execution import CombinedConfig
File "/usr/local/lib/python2.7/dist-packages/wa/framework/configuration/execution.py", line 7, in
from wa.framework.configuration.plugin_cache import PluginCache
File "/usr/local/lib/python2.7/dist-packages/wa/framework/configuration/plugin_cache.py", line 24, in
from wa.framework.target.descriptor import list_target_descriptions
File "/usr/local/lib/python2.7/dist-packages/wa/framework/target/descriptor.py", line 4, in
from devlib import (LinuxTarget, AndroidTarget, LocalLinuxTarget,
ImportError: cannot import name ChromeOsTarget
Is it a known issue ?

Please make sure that you've installed the latest delvib master: https://github.com/ARM-software/devlib

Thanks for your quick answer.
I installed the latest version of devlib (4.0.0) and that fixed the issue.


What does this package add to pyenv + virtualenvwrapper?

Hello - I'm a longtime user and fan of pyenv and virtualenvwrapper.
With this setup I can already run commands like $ mkvirtualenv myvenv, $ mkproject someproject, $ workon someproject, etc using the various python versions installed with pyenv.
When I want to make a new virtual env, I do $ pyenv shell 3.6.1 (or whatever version I want if not default), then $ mkproject foo.  Everything sticks and works fine with the pyenv shims.
I've read the readme, but I'm still trying to figure out what pyenv-virtualenvwrapper adds to this setup or changes.  I'm happy to put together a PR if it would help make this clearer to others as well.

@blueyed Could you help clear up some of my confusion around usage of this package?

pyenv-virtualenvwrapper is a wrapper for virtualenvwrapper to make it little easier to deal with Pythons built by pyenv. Because of the shim architecture of pyenv, it actually is not actually compatible with scripts which need to be source'd into the user's shell, like virtualenv and virtualenvwrapper.
In reality, this script is doing only few things - installing virtualenvwrapper into current Python version in pyenv, then call source to setup virtualenvwrapper into current user's shell.
https://github.com/pyenv/pyenv-virtualenvwrapper/blob/v20140609/bin/pyenv-virtualenvwrapper#L93
As you're doing by yourself, all these steps actually don't need to be implemented as a pyenv plugin; it can be done by yourself if you prefer to do so. The initialization of virtualenvwrapper is just some command line like source ~/.pyenv/versions/2.7.13/bin/virtualenvwrapper.sh. This plugin changes the initialization step to a command line of pyenv virtualenvwrapper, that's all.


Version is not set in pyproject.toml

The problem
Hi! The pyproject.toml of this project declares the version of this project as 0, which is wrong.
The correct version of the python project needs to be declared there (see relevant poetry documentation)
Operating system
Arch Linux
Python version
3.10
Problem-relevant code
https://github.com/ludeeus/awesomeversion/blob/736f1fce7a80042124c71100fc4bcdaff6d5543d/pyproject.toml#L21
Traceback/Error logs
No response
Additional information
If the project's version is not set, sdist tarballs and wheels built from it will have wrong metadata set. This means that other projects will not be able to rely on it without the built package being patched beforehand (they will not be able to use the version information).
This is problematic in general version resolution as well as in system's packaging.

This is set during the release workflow https://github.com/ludeeus/awesomeversion/blob/main/.github/workflows/release.yml#L25-L28

Okay, but that allows noone to build this project from this git repository or a tarball created from a tag in this repository.
More specifically this allows noone to run tests, as those are not included in the sdist tarball on pypi.org
Please consider changing the workflow as it is limiting the use of this library and/or adding the tests to the sdist tarball.


You may want to include this base Docker image for Flask

I made a public (and open source) Docker image with all the bells and whistles that people can use to build a Python Flask web application.
It has uWSGI for running the application, Nginx to serve HTTP and Supervisord to control them, so people don't have to learn how to install and configure all those to build a Python Flask web app.
If there was a Docker image like this one before, I would have liked to find it in your page, so I share this with you in case you want to include it.
It seems that uWSGI and Nginx is one of the more robust and performant ways to deploy a Python web app (here the benchmarks): http://nichol.as/benchmark-of-python-web-servers.
There are even some template projects people can use to bootstrap their own. And also, they don't have to clone the full project or something, they can just use it as a base image.
Docker Hub: https://hub.docker.com/r/tiangolo/uwsgi-nginx-flask/
GitHub: https://github.com/tiangolo/uwsgi-nginx-flask-docker
And thanks for a great guiding site!

:+1: cool, looks good! I've added it to the Docker page and it should be live in a minute or so

Cool, thanks!


bluebird missing index.js

(This issue tracker is only for bug reports or feature requests, if this is neither, please choose appropriate channel from http://bluebirdjs.com/docs/support.html)
Please answer the questions the best you can:

What version of bluebird is the issue happening on?
3.5.0
What platform and version? (For example Node.js 0.12 or Google Chrome 32)
Node v6.9.4
Did this issue happen with earlier version of bluebird?
No
(Write description of your issue here, stack traces from errors and code that reproduces the issue are helpful)

We can download bluebird using npm i bluebird, but when it installs the module folder in ./node_modules/bluebird does not contain an index.js. Therefore node's module system can not properly require the bluebird module.

What? Can you give more details on your build system I cannot reproduce

This is just me trying to npm i on my desktop. I use nvm, but other than that I'm macOS 10.12.5.

Looks like this is an issue with our NPM proxy messing around with the package.json as it is coming in. It removed the "main" directive from the package.json.


Passing in list of database names to create in cluster does not work as expected

I see in your example for postgres you have a databases variable. A list of strings is passed in to the module to create a database for each item in the list.  This is not implemented anywhere.
The example that references this variable is here
I am happy to make a PR if you like, just curious if this was a future feature that has not been added in the master branch yet.
Thanks!

@mohsenSy I have a branch that fixes a few bugs.  Would be happy to make a PR if you allow my user to contribute.

@JohnDzialo thanks a lot for your contributions, in order to merge them you must create fork my repository and create a PR from your repository to mine and then I can accept it. I am looking forward to see your PR

https://github.com/mohsenSy/terraform-digitalocean-db/pull/2

Closed by #2 thanks @JohnDzialo


Error on px4io-v2/board_config.h ?

I am not sure, but think that there some error on  px4io-v2/board_config.h
The line:
#define GPIO_PPM_IN		(GPIO_ALT|GPIO_CNF_INPULLUP|GPIO_PORTE|GPIO_PIN9)
says the IO chip has PORT E, but the chip don't have such port.
Its seems that there are two location on the code that refer to this macro:

fmu.cpp, (correct if I am wrong) the io code doesn't use it
drv_hrt - that seems to configure wrong pin. but because GPIO_PPM is also defined, and initialized there is no issue.

I might be wrong, and its some kind of trick, and I am not sure if and how it effect the current code, and currently don't have PPM remote to check that.

@matanhavi
Thank you for reporting this.

Fixed! Thanks @matanhavi.


[QUESTION] RetryPolicy Timeout vs DataLakeFileClient.existsWithResponse Timeout

Query/Question
Assume we have the following method call
DataLakeFileClient.existsWithResponse(Duration.ofSeconds(60), Context.NONE).getValue();

failing with java.lang.IllegalStateException: Timeout on blocking read for<PHONE_NUMBER>0 NANOSECONDS
This means the request is timing out on the Mono Publisher blockingGet. I wanted to get some clarity on the timeout values in different areas of the SDK and if mu assumptions on how it works is correct
Let's say when creating the DataLakeServiceClient we have a RetryPolicy as follows
RequestRetryOptions retryOptions = new RequestRetryOptions(RetryPolicyType.EXPONENTIAL,
                                                                       5,
                                                                       (Duration) null,
                                                                       (Duration) null,
                                                                       (Duration) null,
                                                                       null);

Q1. Should the Timeout value given to the existsWithResponse method be larger to equal to number of trials * timeout duration to allow underlying http client to do more trials?
Q2. Should we also instead apply timeout values to the RequestRetryOptions?
What's the relation or difference between

Timeout given to the underlying HttpClient
Timeout given to (DataLake|Blob)ServiceClient through RequestRetryOptions
Timeout given to xxxWithResponse method

Scenario I would like to understand.
Assume we define Retry with 3 max retries and 30 seconds timeout and call existsWithResponse with timeout of 60.
If HttpClient sends request 1 and doesn't respond in 30 seconds does that mean it will timeout? and then request 2 should run. Id request 2 does respond in 30 seconds and timeouts.
Does that mean request 3 will not be executed because the publisher above from existsWithResponse has hard timeout of 60 seconds?
Why is this not a Bug or a feature Request?
This is a question for clarity of already implemented feature and code is assumed to be correct and behaving up to expectation so far.
Setup (please complete the following information if applicable):

Library/Libraries: [azureDataLakeStoreSdkVersion = '2.2.9', azureKeyvaultCoreVersion = '1.0.0', azureStorageVersion = '7.0.0', azureStorageFileDatalakeVersion = '12.11.0', azureIdentityVersion = '1.5.3']

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[X] Query Added
[X] Setup information Added

Hi @atemsah-incorta

What's the relation or difference between

Timeout given to the underlying HttpClient
Timeout given to (DataLake|Blob)ServiceClient through RequestRetryOptions
Timeout given to xxxWithResponse method

This is the timeout you'd think of at the lowest level. If the HttpClient doesn't receive a response after this timeout, it abandons it. While it is generally good practice to have some sort of timeout on an HttpClient to ensure it doesn't sit forever on lost requests, I find it not the most influential in a general retry strategy.
These are the settings for the retry policy, implemented through a Reactor timeout. This policy will apply the configured timeout to the rest of the pipeline plus the http roundtrip. A timeout here causes a reactive operation cancellation, a delay, followed by a retry. This is the prime focus of any timeout/retry policy.
This is an individual reactive timeout you can give to the entire end-to-end operation, including all retries. This is more useful for adjustment to individual operations. We don't generally expect its usage, but the option has been provided should it be needed.

If HttpClient sends request 1 and doesn't respond in 30 seconds does that mean it will timeout? and then request 2 should run. Id request 2 does respond in 30 seconds and timeouts.
Does that mean request 3 will not be executed because the publisher above from existsWithResponse has hard timeout of 60 seconds?

This is correct. Not all timings are perfect in an async environment, and so I imagine a race condition taking place with these numbers that line up exactly, but yes that is the general idea.

Hi @atemsah-incorta
Do you have any additional questions? If not, we can go ahead and close this issue. If you have any additional questions, please let us know, thanks!


Terraform style improvement

Currently the way Terraform code is written is a bit inconsistent. The style guide on offical website suggests to:
When multiple arguments with single-line values appear on consecutive lines at the same nesting level, align their equals signs:

Example:
This:
resource "aws_subnet" "ce-1c" {
  vpc_id = "${aws_vpc.CompilerExplorer.id}"
  cidr_block = "<IP_ADDRESS>/24"
  availability_zone = "us-east-1c"
  map_public_ip_on_launch = true

  tags {
    Name = "CompilerExplorer1c"
    Site = "CompilerExplorer"
  }
}

would become this:
resource "aws_subnet" "ce-1c" {
  vpc_id                  = "${aws_vpc.CompilerExplorer.id}"
  cidr_block              = "<IP_ADDRESS>/24"
  availability_zone       = "us-east-1c"
  map_public_ip_on_launch = true

  tags {
    Name = "CompilerExplorer1c"
    Site = "CompilerExplorer"
  }
}

I`m happy to submit a pull request if this is something desired.

The improved formatting can be found here: https://github.com/Cosaquee/compiler-explorer-image/blob/feature/improve-terraform-code-style/terraform/main.tf. This would be quite a big change in terms of line changes, but there is no new or changed logic.
I would like to improve couple more things, like:

using aws_iam_policy_document data source to make the IAM policies more readable
avoid hardcoding ACM certs by getting them with data source
using local variables

Thanks for the suggestions! Re: formatting, I'm using the built-in formatting of my IDE (IntelliJ) and am a huge fan of auto-format. If I can't configure that to change, we'll be stuck with it as is. (I'm also mildly against such lining-up, but that's a whole other conversation :D, and I prefer consistency over that).
I'd be all for merging a change. But it has to be automatable (either in-IDE, or at a push as a make target).
I'd love to see some of the other suggestions: as you can tell I'm a huge terraform noob! I just dumped a bunch more policies in, and will look at aws_iam_policy_document right now :). There's also some other hard-coded sources that came in with a recent change. I didn't use data for them as some had dire warnings about pulling and storing local secrets in plaintext (!).
Local variables: sounds great!
Thanks for taking the time to help improve CE!

I found I can align at equals in the IDE (yay)

Closed by https://github.com/mattgodbolt/compiler-explorer-image/commit/ad087c800e2d8aa141bdc8c36b4ae8b342ee76f2 and https://github.com/mattgodbolt/compiler-explorer-image/commit/24e42d319cc151ad40b7fbae9015d4d932343d16


There are wrong de-CH translations in package/flutter-localization

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
I found in MaterialLocalizationDe a translation "TBD" and some clear translation errors like there is no ß in de-CH
Expected results
No TBD translations in German and no ß letters in de-CH.
Actual results
There is a translation "TBD" which seems to be wrong. I also found wrong translations for de-CH (basically de-DE just in upper case) and some clear translation errors like there is no ß in de-CH
Code sample
Code sample
[Paste your code here]

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[Paste your output here]

FYI @HansMuller

Any updates here?
Might be not the right place here, but is a test required for fixing this?

Just FYI there are more such wrong translations. I just cannot translate those languages and I think it doesn't make sense to use some automatic translation tools.

% grep TBD packages/flutter_localizations/lib/src/l10n/*.arb | wc -l
65

Those "TBD" translations are from 2018

@HansMuller what is about my PR #136091 which corrects some translations? Is it worthless since I cannot edit the Google internal message IDs?

@rekire - your suggestions for translation updates in https://github.com/flutter/flutter/pull/136091 are useful. I'll need to route them through Google's localization team.


Set index defaults to (shards=2, replicas=0)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Index defaults to 2 Shards and 0 replicas

What is the current behavior? (You can also link to an open issue here)
Index defaults to 5 shard, 1 replicas. This is not an ideal default, as storage grows very large if the index has a replica, and fewer shards leads to better search speed.

What is the new behavior (if this is a feature change)?
Index defaults to 2 shard, 0 replicas

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
Yes, a minor one. If users programmatically create Marqo indexes, and don't explicitly set these values, they will get an unexpected index configuration.

Have unit tests been run against this PR? (Has there also been any additional testing?)

Unit tests:  https://github.com/marqo-ai/marqo/actions/runs/5560758700
API tests: https://github.com/marqo-ai/marqo/actions/runs/5560756634

Related Python client changes (link commit/PR here)
n/a

Related documentation changes (link commit/PR here)
TODO

Other info
This PR is associated with, and replaces, this PR due to issues if shards==1: https://github.com/marqo-ai/marqo/pull/536

Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes/features)
[ ] Docs have been added / updated (for bug fixes / features)

This has been replaced by https://github.com/marqo-ai/marqo/pull/540

With 2 shards vector search behaves unusually. For example, if we index 1k documents, and then do a search with limit=1k, then you end up getting less than 1k results, for example 956. This isn’t an issue if shards=3.


Fixed false spelling in README.md

changed "objected-oriented programming" to "object-oriented programming"

What does this PR do?
(Provide a description of what this PR does.)
Test Plan
(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)
Related PRs and Issues
(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)
Have you read the Contributing Guidelines?
(Write your answer here.)

Thanks for correcting the readme file.

Give a star if you liked our repo.

Sure I'll give a star! :)


Added feature to help resolve mocks based off same type search.

Accepts new mock format JVLmock_PrefabName__ChildName.
Finds an object that matches the type of the mock from the specified child of the prefab in the new format.
Cleans name of any object Mesh with " Instance" on the end of the name for finding mock replacement.

Since there are many unnamed Mesh objects used the current mocking system can not replace them. This is the required changes to get a functioning mesh mocking system, but should work for other types too.
Examples of usage:

Create a mocked mesh called JVLmock_Pickable_Thistle__default, this will find the first Mesh on the Pickable_Thistle prefab under the child "default" for a replacement.
Create a mocked mesh named JVLmock_TrollCave02__dirtfloor (11), this will find the first Mesh on the TrollCave02 prefab under the child "dirtfloor (11)" for a replacement.

Thanks, it looks really good. Especially using the ClassMember to find fields on the child, which makes it much more generic.
Using __ as a separator instead of a single underscore is also a good idea to keep the names intact and feels natural to use.
While testing with the VikingShip, a child name can be ambiguous, e.g. mast is used for both the mesh and the interactive child. The interactive child is nested shallower and therefore found first. Intermediate children could be a solution, so maybe JVLmock_VikingShip__ship__mast to match the ship child first and then the mast child.
With more complex matching, the warnings should also be more specific. For example, if a child wasn't found should print a different warning than if the type on the child wasn't found. If you don't want to worry about this, I can also do this later.
There are a few minor comments I will make on the code but this won't be a blocker.

Current minor suggestions have been implemented. I will look into making the pathing more dynamic to accept more nesting for cases like: JVLmock_VikingShip__ship__mast and work on better error logging for that case.

Testing this on my current mocked locations the startup time is significantly slower than my stand alone solution which had caching. Next improvement will be to add in the ability to cache all instances with the longer path finding names after finding them once.


Electron 11

This is an attempt to upgrade Electron to version 11.5.0 and other dependencies as a first step in modernizing this package.
Feedback and testing welcome!

Hi @StorytellerCZ , how can we install this version of your PR in my machine? or is there a test version on npmjs?

maybe this repo can be helpful

@diavrank as you can see the tests are failing, but that has been documented elsewhere as a failure of the tests. I will try to get a beta release out for testing anyhow sometime today or tomorrow. Feel free to bug me about this if it does not happen.

@StorytellerCZ hi, is there any update on this? :)

Looks like the entire testing suite needs to undergo severe upgrade. Done some limited testing myself, but would appreciate more feedback.


Remove all checkboxes from notifications modal

Everyone: think about how notifications should become seen (onmouseover, onclick, on notification display, etc.).

:-1:

The check icon there is awkward. A check normally means you've completed something. Thus sayeth Chris.

Moving this to #124


Memory Leak

There is a current memory leak I'm investigating. I believe queues are not being properly removed or destroyed. I'll be waiting for memory usage to go high and then take a heapdump to find the issue

This is a leak within the threads

This is a leak where songs are not being destroyed entirely properly

This is an issue with Prism-media when tracks are set to have inline volume

This will have a temporary fix in the next update where infinite threads spawn instead of having a set number based on cpu core count and having theoretically infinite uptime so long as queues are being added to keep the worker alive

Partial fix in https://github.com/AmandaDiscord/Volcano/commit/dd23c79307ca44bfd27dfde76f015a135efecf58

Totally fixed in https://github.com/AmandaDiscord/Volcano/commit/5dc49a434211d7b0817ae19f9b4f78995f0348c9
Removed dep on @discordjs/opus and swapped to opusscript. Issue resolved thanks to answer from https://github.com/discordjs/voice/issues/197


Updated code quality readmes and docs

Why are these changes needed?
Updated the README and docs to clearly state that the implementation of some of the metrics are borrowed from starcoder and codeparrot scripts
Related issue number (if any).

LGTM


provide cart object in processing signal

The processing signal provides the Order instance after it has been converted from a Cart instance. However, it is not possible (unless I missed something) to access the original Cart instance in the signal receiver, which would be helpful in some cases.
This pull requests depends on #132 (and is a superset of its commits). So don't merge this before #132 has been accepted :)

Simple enough. I never really got to finish writing signals for the shop, thanks for doing so!


Block-scoped declarations not yet supported outside of strict mode

I'm trying to run Jest on a React app that I have which uses Vue.js for some components, but I keep running into an error that says Block-scoped declarations not yet supported outside strict mode. I've looked into issues filed on this repository, along with on Stack Overflow, with the same error, but nothing that I tried from those sources has worked. One solution that was presented was to put use strict at the top of my files, but since my project contains hundreds of files, I'd rather not do that. I'm posting here in the hope that I can finally find a solution. I am using the latest version of Jest, I have tried re-installing my node_modules folder, and I have run Jest with the --no-cache option with no success. Attached are what I believe are the relevant files. The complete text of the error:
`    C:\Users\z003pudj\Documents\Project X\projectX-core>npm test
 ><EMAIL_ADDRESS>test C:\Users\z003pudj\Documents\Project X\projectX-core
 > jest --config=./__tests__/jest.config

 FAIL  __tests__\sum.test.js
 ? Test suite failed to run

 SyntaxError: Block-scoped declarations (let, const, function, class) not yet
   supported outside strict mode

      at exports.runInThisContext (vm.js:53:16)
      at Module._compile (module.js:373:25)
      at Object.Module._extensions..js (module.js:416:10)
      at Module.load (module.js:343:32)
      at Function.Module._load (module.js:300:12)
      at Module.require (module.js:353:17)
      at require (internal\module.js:12:17)

Test Suites: 1 failed, 1 total
Tests:       0 total
Snapshots:   0 total
Time:        5.053s
Ran all test suites.
 console.log src\js\config.js:8
   INCLUDE_TESTING

  console.info node_modules\vue\dist\vue.runtime.common.js:6111
    You are running Vue in development mode.
    Make sure to turn on production mode when deploying for production.
    See more tips at https://vuejs.org/guide/deployment.html

npm ERR! Test failed.  See above for more details.`

Apologies for the .txt extensions. Couldn't upload the files otherwise.
sum.js.txt
package.json.txt
.babelrc.txt
sum.test.js.txt
webpack.config.dev.js.txt
jest.config.txt

Looks like your Vue files are not properly transformed by Babel. Have you tried reaching help at jest-vue-preprocessor repo?

I just posted on their repo. Is there anything that you could recommend that would help me out?

Without an example repo I'm afraid I can't help much. But you can also check @kentaromiura's article on Vue with Jest<EMAIL_ADDRESS>

This is unlikely a Jest issue, as @thymikee probably an issue with the preprocessor or your code.

Alright, I'll try that then. Thanks!


Simplify code by using NSCache for the time zone cache dictionary

This is similar to #38, but uses an NSCache, which allows us to delete even more code. It's slightly slower than using the plain NSDictionary, but the difference is 0.001 sec on my machine, and it doesn't measure the benefits of not having to register for memory warnings. I'd go for that one.

I really want this PR to be merged, because it fixes thread-safety problem.
I have race condition (thread-safety) issue on v0.7.
Thread : Crashed: NSOperationQueue 0x1909cd80 :: NSOperation 0x191c9e10
0  CoreFoundation                 0x246eb138 -[__NSDictionaryM objectForKey:] + 195
1  CoreFoundation                 0x246eb0bd -[__NSDictionaryM objectForKey:] + 72
2  MayChannel                     0x001c0c27 -[ISO8601DateFormatter dateComponentsFromString:timeZone:range:fractionOfSecond:] (ISO8601DateFormatter.m:619)
3  MayChannel                     0x001c0f87 -[ISO8601DateFormatter dateFromString:timeZone:range:] (ISO8601DateFormatter.m:692)
4  MayChannel                     0x001c0f4b -[ISO8601DateFormatter dateFromString:timeZone:] (ISO8601DateFormatter.m:686)

28 CoreFoundation                 0x24731039 -[__NSCFCalendar dateFromComponents:] + 1208
29 CoreFoundation                 0x24730ab5 -[_NSCopyOnWriteCalendarWrapper dateFromComponents:] + 148
30 MayChannel                     0x001c0fc9 -[ISO8601DateFormatter dateFromString:timeZone:range:] (ISO8601DateFormatter.m:701)
31 MayChannel                     0x001c0f4b -[ISO8601DateFormatter dateFromString:timeZone:] (ISO8601DateFormatter.m:686)

Oh wow, I had no idea this wasn't merged yet. Doesn't look like the repo is still maintained.

Oh... Is this project abandoned? > @boredzo

It's not abandoned; I just have a to-do graph a mile wide. This project has still has a place in it, though.

Thank you!


Fixing byte[] as groupby value

Codecov Report

Merging #12 into develop will decrease coverage by 9.88%.
The diff coverage is 66.66%.

@@              Coverage Diff              @@
##             develop      #12      +/-   ##
=============================================
- Coverage      67.13%   57.24%   -9.89%
  Complexity         4        4
=============================================
  Files           1030     1030
  Lines          50780    50783       +3
  Branches        7100     7100
=============================================
- Hits           34092    29072    -5020
- Misses         14364    19554    +5190
+ Partials        2324     2157     -167

Impacted Files
Coverage Δ
Complexity Δ

...tion/groupby/DictionaryBasedGroupKeyGenerator.java
96.3% <66.66%> (-0.55%)
0 <0> (ø)

...e/pinot/core/operator/blocks/EmptyFilterBlock.java
0% <0%> (-100%)
0% <0%> (ø)

...lection/iterator/FloatSelectionColumnIterator.java
0% <0%> (-100%)
0% <0%> (ø)

...ot/minion/events/EventObserverFactoryRegistry.java
0% <0%> (-100%)
0% <0%> (ø)

...minion/executor/ConvertToRawIndexTaskExecutor.java
0% <0%> (-100%)
0% <0%> (ø)

...on/iterator/FloatArraySelectionColumnIterator.java
0% <0%> (-100%)
0% <0%> (ø)

...a/org/apache/pinot/minion/metrics/MinionMeter.java
0% <0%> (-100%)
0% <0%> (ø)

.../apache/pinot/minion/metrics/MinionQueryPhase.java
0% <0%> (-100%)
0% <0%> (ø)

...selection/iterator/IntSelectionColumnIterator.java
0% <0%> (-100%)
0% <0%> (ø)

...ection/iterator/DoubleSelectionColumnIterator.java
0% <0%> (-100%)
0% <0%> (ø)

... and 325 more

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 a46b692...ee96645. Read the comment docs.


Document if this SDK supports Flutter Web

Expected Behaviour
one ore more of the follwing

aepsdk flutter supports flutter web, flutter build as a web app and an example is provided
adobe does not support flutter web and documents it

Actual Behaviour
No documentation found
Reproduce Scenario (including but not limited to)
n/a
Steps to Reproduce
n/a
Platform and Version
n/a
Sample Code that illustrates the problem
n/a
Logs taken while reproducing problem
n/a

According to the Project readme, This repository is a monorepo. It contains a collection of Adobe Experience Platform Mobile SDK Flutter plugins listed below. These plugins can be found in the plugins directory.
The aepsdk_flutter supports mobile sdk Flutter plugins.
We don't specifically call out the non-supported platforms including Flutter Web in the documentation.

People coming here may be Flutter-only teams and not previous Adobe mobile users.
The README should point out that _no other Flutter platforms are supported or that Adobe metrics may be supported on one of the other 4 platforms using other SDKs
We had three people look at the docs without understanding what we needed to do for Adobe metrics for Flutter apps based on the platform.

May I know what flutter platform you are looking for Adobe Analytics Metrics?
We don't support Flutter Web in this repo.  This repo is for Mobile SDK only.
Are you a new user or you have been using adobe Flutter Mobile SDK previously for Analytics?

Add a note for supported platforms info to the main readme.
https://github.com/adobe/aepsdk_flutter?tab=readme-ov-file#aepsdk-flutter
The Flutter plugins within this repository are specifically designed to support the Android and iOS platforms only.


many modules, such as php, restart apache regardless of $service_ensure

modules are statically notifying Class[apache::service], which is causing Service[httpd] to be refreshed.
The easiest way to fix this is to have modules always notify the appropriate Apache::Mod[$mod_name]. This means every new module needs to call apache::mod { $mod_name: } at the least. Then we can use the spaceship syntax to subscribe Class[apache::service] to Apache::Mod<| |>, which is what we do with vhosts already. The result should be the cascading of notifications only when we ensure the service is enabled. Something like:
#manifests/service.pp
if $ensure_r {
  Apache::Mod<| |> ~> Class['apache::service']
}

Since we do something very similar with the vhosts, this should just work.
There's probably another way to do this, maybe even more elegant, so I'll think about it a bit more before committing to this change.


Fwd: lenkall

Fwd: lenkall created by<EMAIL_ADDRESS>on 2016-12-15 23:37:17

<EMAIL_ADDRESS>replied,
Begin forwarded message:

<EMAIL_ADDRESS>replied, E le ng kall ---because it is food.

<EMAIL_ADDRESS>replied, leng kall? is a question asked if the food are still ok, or if they are edible..

<EMAIL_ADDRESS>replied, Yep. Have to see it in a sentence to know. It could be "le ng kall" (is it
edible) or "ilengkall" (piled up).

<EMAIL_ADDRESS>replied, Piled up is ilengkai; aile ng kall-are these food/edible like Adelbai said.


Porting pneumatic spring plugin

Porting plugins from bitbucket.
Install the project using instructions in buoy_entrypoint and run
ign gazebo buoy_gazebo/worlds/mbari_wec.sdf

I ticketed a couple issues with ideas for follow-up PRs:

#3
#4


miniDuck placement

Enhancements:

[ ] make mini ducks move around the cursor more naturally, rather than just in the same direction, diagonal line.
[ ] when the mini ducks wander, make their movement more natural looking, instead of just teleporting randomly

Closing this for now. See issue #6 for other mini duck ideas.


Better Guidance for Creating New Mints

Currently, suggestions for creating new mints doesn't seem to take into the account that small denominations are required for most transactions. For example, when I had a 140 XZC balance, I was suggested to make 50 XZC + 2x 10 XZC mints. This isn't the optimal situation for ensuring users have the necessary denominations for any given spend.

This is much less relevant now that Sigma mints automatically. Closing.


PIN pattern export as image

Implemented in Version 0.4.0

@serrq If you want the pdf export feature, please open a separate issue for that :)

Possible. I would add an optional setting when saving an image to include the name.
Please open a new issue for that. :)


engsys: idempotent publishing for re-runs

Is there a general strategy we can make artifact publishing idempotent to job re-runs?
Example:
In publish-cli.yml (and anywhere else), when we have a pipeline artifact being published:
  - task: 1ES.PublishPipelineArtifact@1
    inputs:
      targetPath: release
      artifact: UploadedReleaseArtifacts

And in a subsequent step such as create-pull-request, fails due to a flaky issue. On a rerun, we will get an error:

##[error]Artifact UploadedReleaseArtifacts already exists for build 4153919.

Here's a recent build run demonstrating this issue.

In this particular case the assumption was made that the GitHub token would not be rate limited. A recent test of some automation broke this assumption but that should not be the case going forward.
A few ways to solve this would be to:

Use a token that's less likely to be subject to rate limits
Further subdivide jobs so that artifact uploads and tasks that may depend on those artifacts are separated and the dependent jobs can be retried
Remove some artifact publishing from publish-cli.yml in some cases because it's not strictly necessary (we shouldn't do this)
Skip the artifact publish task on failure (assume that previous run produced the right artifact and uploaded it)

I'm closing this since it was caused by a one-off event but if it happens again we'd probably start with ensuring that automation tests use a different token.

@danieljurek Is there a property for re-uploading in-place? I feel like this would be acceptable for a non-production artifact.
Btw: I hit this twice again today. Here's another instance: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=4172035&view=results
Another tactical thing: can we shift the PR add comment up? (haven't looked at the implementation here)

Reopen! This is still a problem let's do something about it.


added tracker response to provide status info

this adds a tracker response to provide status info to the tracker clients

removed "data" and "state" from the response wrapper


One of lawsets have two laws 3

Screenshots

ah that is intended, this one is NT default i think, its kinda like crewsimov, but with rank and job in mind

oh wait, just saw the numberin.... uh... hmmm
well that should be an easy enough fix, imma do it myself

ah that is intended, this one is NT default i think, its kinda like crewsimov, but with rank and job in mind

Nothing in rules written about non-ion laws with same number but different order... is AI free to choose and can break one over other at will?

nope, im just BLIND, and didnt notice the numbers on the side being wrong fixin it now


Bot Hosting

Suggest Best Vps for telegram bots
Ex :-
Mirror Bot, Leech bot, Url uploader, Rename bot

Sure 👍


Move omni_getcrowdsale amountraised field into verbose output and sort participants

Note: we could leave amount raised in the non-verbose output, but that would mean calculating all participants when non-verbose which is a slow and expensive call.

Note: we could leave amount raised in the non-verbose output, but that would mean calculating all participants when non-verbose which is a slow and expensive call.

It's probably okay to move it into the non-verbose output, because the whole SP, including database, is already deserialized at that point, and all values in memory.
A likely more signifiant improvement could be to replace the FormatMP() calls with FormatByType(int64_t amount, uint16_t propertyType) (where propertyType is stored once), because each time FormatMP is used, the SP is fetched over and over again from the DB to determine divisibility.

Sorry dude, I did mean to include your feedback.  You're quite correct, it's really poor performance to load the SP on every iteration of the recipients loop - once that's modified the performance is very quick so I agree the calculation of amountraised can be moved out of the verbose output.
The sorting methodology is also floored, since I used a map ordered by blockheight, block is also the key resulting in only one crowdsale purchase being stored per block - you can see this from checking the amountraised against the total amountsent.
$ src/omnicore-cli getcrowdsale_MP 3 | grep amountraised
    "amountraised" : "93249.21601551",

$ src/omnicore-cli getcrowdsale_MP 3 true | grep amountsent
            "amountsent" : "100.00000000",
            "amountsent" : "8.07394800",
            "amountsent" : "2.04500000",
            "amountsent" : "1000.00000000",
            "amountsent" : "110.00000000",
            "amountsent" : "189.55000000",
            "amountsent" : "5.00000000",
            "amountsent" : "50.00000000",
            "amountsent" : "1.99882495",
            "amountsent" : "7.00000000",
            "amountsent" : "175.07666050",
            "amountsent" : "0.20000000",
            "amountsent" : "305.51100000",
            "amountsent" : "100.00000000",
            "amountsent" : "6.19185495",
            "amountsent" : "7.00000000",
            "amountsent" : "50.00000000",
            "amountsent" : "70.44000000",
            "amountsent" : "45.00000000",
            "amountsent" : "0.80000000",
            "amountsent" : "100.00000000",
            "amountsent" : "20.00000000",
            "amountsent" : "20.00000000",
            "amountsent" : "40.00000000",

I've made the changes fixes here 8f5241699bce83b617226acb05bdfda7f2d0a748 :)

Sorry dude, I did mean to include your feedback.

Oh my bad, I'm really sorry for the rushed merge then!

The sorting methodology is also flawed ...

I see it now.


Implement ability for user to delete user profile in account settings

From @yfzwinnie on November 2, 2018 22:2
Copied from original issue: Microsoft/Access2Justice#754

@KellyeDC  - While this sounds like a standard thing that MS should have put in place, I don't think it is necessary for pilot.


add alerta service

Description
Add Alerta.io in Notification Services.
But the service does not show up, as shown on this number: https://github.com/louislam/uptime-kuma/issues/1186
Type of change
Please delete any options that are not relevant.

Alerta notification service
User interface (UI)
Translation update

Checklist

[X] My code follows the style guidelines of this project
[X] I ran ESLint and other linters for modified files
[X] I have performed a self-review of my own code and tested it
[X] I have commented my code, particularly in hard-to-understand areas
[X] My changes generate no new warnings
[ ] My code needed automated testing. I have added them (this is optional task)

Wait, if I made this PR, it's because I was asked to help me with my problem.

We had to make: "npm run test-with-build".
But it took me a long time to find. It is not explained anywhere that everything is downloaded, and therefore it does not take the sources into account.

I'm fixing the issues before resubmitting a working PR.

Sorry for your experience, but the contributing guide should be pretty clear on the procedure needed to start the development server, which should not use the pre-built frontend files.
I'd also recommend prefixing your new language keys with alerta as well.


_currentList will not updated when the controller is not attached to the view

Problem: When working with lazy loadings widgets and an new state of the list has already triggerd via dispatchNewList, it will still show the old state of the list.
Example:  Working with the BloC Pattern and a TabBarView. Usually  you get the new data of the list over a listener and fire dispatchNewList. But the works only for the current visible tab view. when switching to an other tab, you will still the the old state of the list.
Solution:
Update the _currentList even when the controller is not attached to an view, but do not perform any animation.
Version: 0.0.10
File: animaeted_list_dispatcher.dart
Method: dispatchNewList LOC: 60
original code:
if (!animatedListController.isAttached) return;
bug fix:
 if(!animatedListController.isAttached){       _oldList = currentList;       _currentList = newList;       return; }
If you want, I can create also a PR, but didn't found any hints how you wanna work with pr :D
Actually I just made the fix locale and it works now nice.
BTW: Thx, for the package. Before I had my own solution with Animation and Reorder but yeah it was not quite so good like yours. Had Problems with smooth scrolling during the drag&drop.

I have the same problem
just found this package, ran the example, and it worked as expected, but when I tried to use this in my views the widget was blank without throwing any exceptions. I'm using riverpod to get current list

Please, migrate to the new version (0.1.0) and let me know if the issue persists.


Chore/stability test sarif converter

[ ] Ready for review
[ ] Linked to Jira ticket

What does this PR do?
this adds a sanity test to the sarif converter and fixes an issue that looses issues

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fix(e2e): fixed some issues with e2e tests

Solace e2e: Updated the chart url
Prometheus metrics: increased timeout and used a unique name for the metric label

Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)

/run-e2e prometheus

/run-e2e

Oooh bloody auto-merge :)
Was going to re-run the tests again but let's see what main does :)


Knowledge discovered is shown as a colon-delimited string

Knowledge events currently describe the type of knowledge discovered as a colon-delimited string, not the English phrase displayed in dwarf fortress:
Current:

Preferred:

So far I see two options:
Option 1: I could edit HistoricalEventKnowledgeDiscovered.Html(c *Context) in events.go to compare x.Knowledge against a bunch of raw strings and use the corresponding English phrase, ex:
func (x *HistoricalEventKnowledgeDiscovered) Html(c *Context) string {
	knowledge := x.Knowledge
	switch x.Knowledge {
	case "astronomy:daylight variation with solar year":
		knowledge = "the variation of daylight with the season"
	case "astronomy:geocentric model":
		knowledge = "the theory that the sun moves around the world"
	//...
	}
	return c.hf(x.Hfid) + util.If(x.First, " was the very first to discover ", " independently discovered ") + knowledge
}

Option 2: Edit the ForceEnums in overwrites.json to include the discovered knowledge type, regenerate models.go to create a list of Knowledge types, and compare these in evens.go (rather than comparing with raw strings).
    "ForceEnum": {
        "df_world|historical_events|historical_event+HfDied|cause": true,
        "df_world|historical_events|historical_event+HfDied|death_cause": true,
        "df_world|historical_events|historical_event+KnowledgeDiscovered|knowledge": true
    },

type HistoricalEventKnowledgeDiscoveredKnowledge int

const (
	HistoricalEventKnowledgeDiscoveredKnowledge_Unknown HistoricalEventKnowledgeDiscoveredKnowledge = iota
	HistoricalEventKnowledgeDiscoveredKnowledge_AstronomydaylightVariationWithSolarYear
	HistoricalEventKnowledgeDiscoveredKnowledge_AstronomygeocentricModel
	//...
)

Option 2 seems closer to the pattern used elsewhere in LegendsBrowser2, but without the original legends.xml files, I'm having trouble doing this myself without breaking model.go and analyze.json. (Or perhaps I'm misunderstanding something about how to generate these)
TLDR; Would you mind updating overrides.json and generating these HistoricalEventKnowledgeDiscoveredKnowledge enums yourself? Or is there a way I can do it?

I've changed it to an enum and added an option to regenerate the analyzed data without a need for all xml files:
go run analyze.go -l=true -a ../inputs
-l will load the existing analyze.json and only add new findings from your xml files in ../inputs
You can then use go run analyze.go -g=true to regenerate the model code

Perfect, thanks! I'll put up a PR when I've gotten phrases for all of these written out. There are more than I realized...


App not detected by Crashlytics

I have followed the instructions:

Installed the Fabric plugin in the Android Studio, created personal account in fabrio.io.
Installed react-native-fabric through npm
Did the following:
3.1. Edited android/settings.gradle
3.2. Edited android/app/build.gradle
3.3. Edited MainApplication.java in the application class since my react native version is 0.41.2. Please look below:
@Override    public void onCreate() {  	super.onCreate();  	Fabric.with(this, new Crashlytics()); // Added this line to fix the error ""  	SoLoader.init(this, /* native exopackage */ false);    }

In my login screen I just wanted to test crash the app using Crashlytics.crash(); of course I also initialized it globally with:
var Fabric = require('react-native-fabric'); var { Crashlytics } = Fabric;
Now the problem comes in, I'm now in login screen and when I pressed the Login button I encountered similar problem found here. Yet from what I understood in your reply, I configured it properly. But I want to crash it, not prompted by red error. What I did was to make a release build, and it successfully crashed when I pressed the button. So does this mean, I can only test Crashlytics with the release build?
My analysis, correct me if I'm wrong:

Since the Fabric plugin is installed in the Android Studio and it is configured in the native code, the React Native layer (codes) are not detected by Fabric.

So my problems are:

Is it correct to assume that I can't use or test Crashlytics with Debug mode? If not, did I miss any configuration?
Fabric plugin in my Android Studio can't detect my app even though I have added android:name="io.fabric.ApiKey" in the AndroidManifest.xml. I'm stuck in the Launch your app! stage from the Fabric plugin.
Now if I go to https://fabric.io/dashboard, I'm stuck with the Launch your app screen, also. I will be redirected to https://fabric.io/onboard.

I was having the same issue. In my case adding
buildscript {       repositories {              maven { url 'https://maven.fabric.io/public' }           }        dependencies {             // The Fabric Gradle plugin uses an open ended version to react                      // quickly to Android tooling updates                             classpath 'io.fabric.tools:gradle:1.+'            }      }  apply plugin: 'io.fabric'
to build.gradle worked

@asachin16's solution is the correct answer, closing this

@asachin16's solution is documented on repo as setup for Android without Android Studio.
Can we clarify that these settings are still needed for Android Studio?

Having same issue, @asachin16 which build.gradle files, there are multiple ones?

my solution

android/app/build.gradle

  apply plugin: "com.android.application"
  apply plugin: 'io.fabric'

{ ... }

dependencies {
   { ... }
    compile('com.crashlytics.sdk.android:crashlytics:2.5.5@aar') {
        transitive = true;
    }
}

{ ... }

buildscript {
    repositories {
        maven { url 'https://maven.fabric.io/public' }
    }
    dependencies {
        classpath 'io.fabric.tools:gradle:1.+'
    }
}

2017-12-21 18:32:23,269 [se-915-b01]   INFO - ols.intellij.CrashlyticsPlugin - Crashlytics 4.1.1 initialized for Android Studio 3.0
2017-12-21 18:32:23,326 [d thread 2]   INFO - .openapi.application.Preloader - Finished preloading com.intellij.openapi.actionSystem.impl.ActionPreloader@7c341011
2017-12-21 18:32:24,694 [se-915-b01]  DEBUG -                    Crashlytics - -------------Crashlytics appending to Intellij Logger-------------
2017-12-21 18:32:24,728 [se-915-b01]   WARN -                    Crashlytics - Crashlytics was unable to completely start up.
com.crashlytics.api.storage.exception.UnsupportedVersionException
at com.crashlytics.api.storage.Hexifier.get(Hexifier.java:67)
at com.crashlytics.api.RestfulWebApiCache.loadFromEncodedCache(RestfulWebApiCache.java:255)
at com.crashlytics.api.RestfulWebApiCache.loadCachedOauthUser(RestfulWebApiCache.java:186)
at com.crashlytics.api.RestfulWebApi.(RestfulWebApi.java:180)
at com.crashlytics.tools.intellij.core.CoreBundleActivator.init(CoreBundleActivator.java:278)
at com.crashlytics.tools.intellij.core.CoreBundleActivator.start(CoreBundleActivator.java:140)
at org.apache.felix.framework.util.SecureAction.startActivator(SecureAction.java:645)
at org.apache.felix.framework.Felix.activateBundle(Felix.java:2154)
at org.apache.felix.framework.Felix.startBundle(Felix.java:2072)
at org.apache.felix.framework.BundleImpl.start(BundleImpl.java:976)
at org.apache.felix.framework.BundleImpl.start(BundleImpl.java:963)
at com.crashlytics.tools.bootstrap.osgi.OsgiBootstrapper.loadComponents(OsgiBootstrapper.java:155)
at com.crashlytics.tools.intellij.OSGiPluginServices.loadComponents(OSGiPluginServices.java:139)
at com.crashlytics.tools.intellij.CrashlyticsPlugin.initComponent(CrashlyticsPlugin.java:90)
at com.intellij.openapi.components.impl.ComponentManagerImpl$ComponentConfigComponentAdapter.getComponentInstance(ComponentManagerImpl.java:491)
at com.intellij.openapi.components.impl.ComponentManagerImpl.createComponents(ComponentManagerImpl.java:120)
at com.intellij.openapi.application.impl.ApplicationImpl.lambda$createComponents$8(ApplicationImpl.java:466)
at com.intellij.openapi.progress.impl.CoreProgressManager.lambda$runProcess$1(CoreProgressManager.java:176)
at com.intellij.openapi.progress.impl.CoreProgressManager.registerIndicatorAndRun(CoreProgressManager.java:556)
at com.intellij.openapi.progress.impl.CoreProgressManager.executeProcessUnderProgress(CoreProgressManager.java:501)
at com.intellij.openapi.progress.impl.ProgressManagerImpl.executeProcessUnderProgress(ProgressManagerImpl.java:66)
at com.intellij.openapi.progress.impl.CoreProgressManager.runProcess(CoreProgressManager.java:163)
at com.intellij.openapi.application.impl.ApplicationImpl.createComponents(ApplicationImpl.java:473)
at com.intellij.openapi.components.impl.ComponentManagerImpl.init(ComponentManagerImpl.java:104)
at com.intellij.openapi.application.impl.ApplicationImpl.load(ApplicationImpl.java:425)
at com.intellij.openapi.application.impl.ApplicationImpl.load(ApplicationImpl.java:411)
at com.intellij.idea.IdeaApplication.run(IdeaApplication.java:206)
at com.intellij.idea.MainImpl$1.lambda$null$0(MainImpl.java:49)
at java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:311)
at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:762)
at java.awt.EventQueue.access$500(EventQueue.java:98)
at java.awt.EventQueue$3.run(EventQueue.java:715)
at java.awt.EventQueue$3.run(EventQueue.java:709)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:80)
at java.awt.EventQueue.dispatchEvent(EventQueue.java:732)
at com.intellij.ide.IdeEventQueue.dispatchEvent(IdeEventQueue.java:343)
at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:201)
at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:116)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:105)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:93)
at java.awt.EventDispatchThread.run(EventDispatchThread.java:82)
Please give me solution for this problem i have stuck on it for last many days