guide: begin How To section

Extracted from #1581

JUST A DRAFT TO START THIS GOING — Please don't review yet.

Moves Updating Tracked Files into the new How To section
Creates the Best Practices how to guide (for #72) Extracted to #1748
How to undo dvc add (Fixes #625)

Ref: #899 and https://documentation.divio.com/how-to-guides/#how-to
AFTER MERGE

[ ] This doc can be expanded for stage outputs.

From https://github.com/iterative/dvc.org/pull/1705#issuecomment-695832489

@imhardikj please solve merge conflicts and continue this (:

So now that the Best Practices guide will be moved out of how-to, what's your plan for this PR @imhardikj ? Thanks

I assume this is ready for review?

Does this close #625 @imhardikj ? If so please link them with a special word in the description of this PR.

[ ] Oh, also! Let's link to this guide from the add reference. Please find a suitable place in the Description.

@jorgeorpinel I can't request a review on this PR. I guess this is because your the author of PR. Please review, I have made the  requested changes.

expand on this how to in the future, to encompass any tracked data (not just added) i.e. any outputs.
Yeah this can be doc can be expanded for outputs. Will look into it. Thanks

p.s. @shcheklein can you please approve this PR? I started it so I can't.

Actually @imhardikj please try to approve it, maybe you also can? Probably not but I'm not sure.

Failed test is OK (new page). Merging 🎉

This doc can be expanded for stage outputs.

Please do a separate PR with a proposal for that when you have some spare time @imhardikj, thanks.

Looks like we failed to update the old /doc/user-guide/updating-tracked-files links to the new location, /doc/user-guide/how-to/updated-tracked-files
But I remember reviewing the checks before merging... In fact they didn't fail 🤔 https://app.circleci.com/pipelines/github/iterative/dvc.org/5905/workflows/79da4dc3-e28f-4321-810c-89b0fb2afac1/jobs/5961 cc @shcheklein
Fixing in #1823

But I remember reviewing the checks before merging... In fact they didn't fail 🤔 https://app.circleci.com/pipelines/github/iterative/dvc.org/5905/workflows/79da4dc3-e28f-4321-810c-89b0fb2afac1/jobs/5961 cc @shcheklein

we don't use CircleCI to check links anymore. We migrated to the GH actions. I guess you should have merged the latest master?

Makes sense. Well, it's fixed now 🙂


Make port picking functions overridable

Instead of exposing these three functions as pointers and let users set each separately. I would suggest grouping them in a struct as usually users would set them together (rather than setting 1 and leaving 2 to be defaults...).
Maybe the gpr_set_allocation_functions is a good reference to look at.

[trickle] No significant performance differences

Done. Thanks for the review!

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

[microbenchmarks] No significant performance differences

Known failures:
Basic Tests Linux (opt): #11745
Tsan C: #11482


DLL library

Hi,
I work with VB.NET so it's not possible for me to compile a C# project, so I can't get DLL library.
Can you please compile the project into a DLL library so I can download ad use it ??
Many thanks
Sergio

Hello @sirjo66, this repo is no longer maintained.
Regards


Be able to print TickTock inside profiling

This can't really be done without significant changes and a much more detailed profile tree.
An inner profile might finish without the outer one being done. This causes problems, especially if the profiles have been triggered before.


Inference on MoE models

The inference command provided for MoE models gives errors on both the provided pre-trained MoE models and on the ones I have trained myself.
This is the command that I am trying to use:
DATA_PATH=/path/to/data-bin
MODEL_PATH=/path/to/model.pt
python -m fairseq_cli.eval_lm
$DATA_DIR
--path $MODEL_PATH
--gen-subset valid
--sample-break-mode none
--tokens-per-sample 2048
--batch-size 1
--fp16
--output-word-probs
--is-moe
--distributed-world-size 8
--model-overrides "{'world_size': 8, 'moe_eval_capacity_token_fraction': 0.05}"
I downloaded the pre-trained moe_15b model from the moe_lm README onto my machine, and it unzips into a directory that looks like this:
en_moe_lm_15b:
—model-rank-0.pt
—model-rank-1.pt
...
—model-rank-63.pt
—model-shared.pt
I try running the given command, setting MODEL_PATH=/path/to/en_moe_lm_15b/model.pt and DATA_PATH=/path/to/data-bin/wikitext-103. I get the following error:
Traceback (most recent call last):
File "/data/meenakshi/miniconda3/envs/fairseq_moe/lib/python3.9/site-packages/torch/multiprocessing/spawn.py", line 69, in _wrap
fn(i, *args)
File "/data/meenakshi/MoE/fairseq/fairseq/distributed/utils.py", line 335, in distributed_main
main(cfg, **kwargs)
File "/data/meenakshi/MoE/fairseq/fairseq_cli/eval_lm.py", line 384, in main
models, model_args, task = checkpoint_utils.load_model_ensemble_and_task(
File "/data/meenakshi/MoE/fairseq/fairseq/checkpoint_utils.py", line 478, in load_model_ensemble_and_task
model.load_state_dict(state["model"], strict=strict, model_cfg=cfg.model)
File "/data/meenakshi/MoE/fairseq/fairseq/models/fairseq_model.py", line 126, in load_state_dict
return super().load_state_dict(new_state_dict, strict)
File "/data/meenakshi/miniconda3/envs/fairseq_moe/lib/python3.9/site-packages/torch/nn/modules/module.py", line 2041, in load_state_dict
raise RuntimeError('Error(s) in loading state_dict for {}:\n\t{}'.format(
RuntimeError: Error(s) in loading state_dict for TransformerLanguageModel:
size mismatch for decoder.embed_tokens.weight: copying a param with shape torch.Size([51200, 768]) from checkpoint, the shape in current model is torch.Size([267744, 768]).
size mismatch for decoder.output_projection.weight: copying a param with shape torch.Size([51200, 768]) from checkpoint, the shape in current model is torch.Size([267744, 768]).
I understand that this is due to the dict.txt of the wikitext-103 dataset being a different size than the one used to train the moe-15b model, but how do I fix this? I cannot find any information about the dict.txt of the moe-15b model.
——————
I have also tried to train my own moe models using the following command:
NUM_EXPERTS=8
TOKENS_PER_SAMPLE=1024
fairseq-train --task language_modeling
data-bin/wikitext-103
--save-dir checkpoints/moe_wikitext-103
--tokens-per-sample $TOKENS_PER_SAMPLE
--ddp-backend fully_sharded --memory-efficient-fp16 --checkpoint-activations
--arch transformer_lm_gpt --share-decoder-input-output-embed
--decoder-layers 24 --decoder-embed-dim 1024 --decoder-ffn-embed-dim 4096
--decoder-attention-heads 16
--moe-expert-count $NUM_EXPERTS --moe-freq 2
--moe-gating-use-fp32 --moe-second-expert-policy all
--moe-normalize-expert-grad sqrt_world_size
--moe-eval-capacity-token-fraction -1.0
--max-sentences-valid 1 --num-workers-valid 0
--criterion moe_cross_entropy --moe-gate-loss-wt 0.01 --moe-gate-loss-combine-method sum
--optimizer adam --fp16 --adam-betas '(0.9, 0.98)' --clip-norm 0.0
--lr 0.0005 --warmup-updates 750
--dropout 0.2 --attention-dropout 0.2
--batch-size 2 --update-freq 2
--max-update 250 --disable-validation
--log-format json --log-interval 10
And after training I get a model directory that looks like this:
moe_wikitext-103:
—checkpoint_last-rank-0-shard0.pt
—checkpoint_last-rank-1-shard1.pt
...
—checkpoint_last-rank-7-shard7.pt
—checkpoint_last-shared-shard0.pt
...
—checkpoint_last-shared-shard7.pt
Running inference on this model using a similar command as above originally results in errors like this:
Model file not found: checkpoints/moe_wikitext-103/checkpoint_last-rank-7.pt
So I edited the eval_lm.py file to add "-shard{rank}" to the end of the files. After trying that I get this error:
Traceback (most recent call last):
File "/data/meenakshi/miniconda3/envs/fairseq_moe/lib/python3.9/site-packages/torch/multiprocessing/spawn.py", line 69, in _wrap
fn(i, *args)
File "/data/meenakshi/MoE/fairseq/fairseq/distributed/utils.py", line 335, in distributed_main
main(cfg, **kwargs)
File "/data/meenakshi/MoE/fairseq/fairseq_cli/eval_lm.py", line 384, in main
models, model_args, task = checkpoint_utils.load_model_ensemble_and_task(
File "/data/meenakshi/MoE/fairseq/fairseq/checkpoint_utils.py", line 478, in load_model_ensemble_and_task
model.load_state_dict(state["model"], strict=strict, model_cfg=cfg.model)
File "/data/meenakshi/MoE/fairseq/fairseq/models/fairseq_model.py", line 126, in load_state_dict
return super().load_state_dict(new_state_dict, strict)
File "/data/meenakshi/miniconda3/envs/fairseq_moe/lib/python3.9/site-packages/torch/nn/modules/module.py", line 2041, in load_state_dict
raise RuntimeError('Error(s) in loading state_dict for {}:\n\t{}'.format(
RuntimeError: Error(s) in loading state_dict for TransformerLanguageModel:
size mismatch for decoder.embed_tokens.weight: copying a param with shape torch.Size([50000, 1024]) from checkpoint, the shape in current model is torch.Size([267744, 1024]).
size mismatch for decoder.output_projection.weight: copying a param with shape torch.Size([50000, 1024]) from checkpoint, the shape in current model is torch.Size([267744, 1024]).
This is similar to the previous error but it makes no sense to me as I trained the model on the same dataset that I am trying to evaluate it on.
Environment details:

fairseq Version: moe branch
PyTorch Version: 2.0.1
OS: Linux
How you installed fairseq: source
Build command you used: pip install --editable ./
Python version: 3.9.19
CUDA version: 3.7
GPU models and configuration: 8 Tesla P100-PCIE-16GB GPUs

I met the same issue, any solutions?


feat: add typescript types

Let's get some types up in here :)
This PR adds a simple d.ts file which declares the type of the postal code object to be Record<string, string>
I considered setting up a literal type for the entire postal code object, so that you have an object that is indexed by a set of literals, not just Record<string, string>, but I'm not sure how much value it really adds.
Some devs may prefer not having to work with literals here, since in order to get a string out of the object you have to assert your string keys as index literals before indexing the postal code object:
    const postalCodes = (await import('norway-postal-codes')).default;

    let myPostalCode: string;

    const anyCity = postalCodes[myPostalCode]; // type is any

    if (myPostalCode in postalCodes) {
      const literalIndex = myPostalCode as keyof typeof postalCodes; // type is "1001" | "1003" | "1005" | etc.

      const stringCity = postalCodes[literalIndex]; // type is string
    }

On the other hand, such a type would certainly be preferable to Record<string, string> if your postal code is actually typed as a literal, not simply a string, for example if you have set up some kind of validation/sanitation pipeline.
I think making such a type would work if we add this to make.js:
	write(
		'index.d.ts',
		`declare module "norway-postal-codes" { export default ${JSON.stringify(
			ret2
		)}; }`
	);

What are your thoughts on typing this module?

I think Record<string, string> is good enough.

Thanks :)


Add hashmaps

Hello @rsdy
Can you review this PR?
1
Add hash table maps for both BPF programs and userspace programs.

PerCpuHashMap
LruHashMap
LruPerCpuHashMap

All structures provide similar methods.
For BPF programs, get, get_mut, set, delete, and get_val are implemented.
For userspace programs new, set, get, delete and iter methods are implemented.
2
Check Map is valid BPF map type (BPF_MAP_TYPE_HASH or BPF_MAP_TYPE_PERF_EVENT_ARRAY) when HashMap::new function is called.
3
Add derive Clone, Debug attribute to PerCpuValues.
4
Add hashmaps example program.

#[repr(align(8))] does not affect the item if the alignment of the item is greater than 8 bytes.
And it's okay that items have smaller alignments 1, 2, or 4 because those alignments are also aligned to 8 bytes.
So using that macro is not helpful.
I wrote a comment about the alignment at #150. I hope we have some documentation that states this limit  clearly.

I also considered raising compilation error if the alignment of the value is greater than 8 bytes.
But, isn't it too harsh?
I am not sure this behavior is desirable..

@rsdy
I am going to merge this. Thanks

I also considered raising compilation error if the alignment of the value is greater than 8 bytes.

I'd like to see what kind of false positives this might trigger. I think if we are certain that this locks in correct behaviour, I'd be inclined to make it a hard error with a good error message than propagating failure to the verifier.

@rsdy
I agree with you.
So I am going to add a feature of raising compilation error if the value has
a big alignment over 8 bytes.
This is the basis why the item whose alignment exceeds 8 bytes are banned.

In official Rust documents, it is stated that misaligned pointers or references
lead to undefined behavior.
In BPF programs, the pointer returned by bpf_map_lookup_elem is misaligned
because the value the pointer points to is merely 8 bytes aligned data.
Hence, creating reference of the value, or dereferencing the pointer incurs
undefined behavior.

@rhdxmr that sounds promising, and quite clear cut.


QUESTION: how can I add my custom chart ?

I want to add custom charts, how can I do it? is there is a feature like that?

Hello @mohafiz , what custom chart? If the chart is provided as a plugin, then read Plugins Section to see how you can include the files of the plugins on the package.

ok nice, i missed this section of the documentaion


0-infinite quantifier inside look behind capture group causes tremendous performance loss

Creating a syntax highlighter with the following code and running it on a single line file filled with thousands of the letter c will cause tremendous lag

{
	"name": "Lag Syntax",
	"scopeName": "source.redcmd.syntax.lag",
	"patterns": [
		{
			"match": "(?<=ab*)c",
			"name": "strong keyword.control"
		}
	]
}

Replacing the * with a + or changing the regex to (?<=ab+|a)c will remove all lag and allow my machine to easily interpret a million character single line file
But then a file filled with bc will lag instead
I cant seem to figure out how the engine matches lookbehinds
does it start with the c first then try to match the lookbehind afterwards?
or does it try to match the lookbehind first, then the c?
using {0,10000} instead of * reduces lag massively
which seems to suggest that the engine is trying to match b an infinite amount (or an internal limit) of times before giving up
but for some reason it only does so when its allowed to match 0 times??
maybe its an underflow error?
But then using the regex (?<=a(b+)?)c also causes an enormous amount of lag
which I don't get
unless.. (b+)? gets optmized to b*
or maybe cause the engine backtracks in the wrong direction and ends up endlessly rechecking b?
Noticed single line files were only getting partially highlighted

Dug around and found out it was caused by this regex:

(?<={\\d*)\\\\{2} is the part that was going over board on every single \\

This might be causing some of the serious performance issues in the C++ syntax as well.
Part of our preprocessing steps in the C++ grammar optimize (b+)? to become b* so that's probably not doing us any favors

I just looked through all c, cpp, cpp.embed and cuda-cpp grammar files and did not see this specfic issue once.

Oh, well thanks for saving me the trouble @RedCMD. I suppose it's just regular catastrophic backtracking then for C++

@jeff-hykin
This may be causing a lot of the C++ performance issues
I notice many "#inline_comment" includes under captures
https://github.com/microsoft/vscode-textmate/issues/167

Thank you for reaching out! I think the regex (?<=ab*)c might be running into catastrophic backtracking. Unfortunately, I don't know enough about the inner workings of a regex engine to help deeper than that. We are using oniguruma as the underlying regular expression engine. The author of oniguruma has been incredibly helpful and responsive in the past.
@kkos do you think there are optimization opportunities in oniguruma around such regular expressions?


Fix Changelog

Fixes #4829.
PR checklist

[X] Addresses an existing issue: fixes #4829
[ ] New feature, bugfix, or enhancement
[ ] Includes tests
[ ] Documentation update

Overview of change:
Fixed the name and link of the invalid-void rule in the changelog.
CHANGELOG.md entry:
[no-log] fix invalid-void changelog entry

Thanks for your interest in palantir/tslint, @BBosman! Before we can accept your pull request, you need to sign our contributor license agreement - just visit https://cla.palantir.com/ and follow the instructions. Once you sign, I'll automatically update this pull request.

Thanks 😄 good catch!

@JoshuaKGoldberg Signed the CLA.


Changes to new GP feature

Few changes in the UI page included to new GP feature

changes  UI page included to new GP feature works fine


Create ETH SF Bounties

I'm trying to secure a $1500 bounties budget for the ETHSF hackathon. I would really like to offer hack teams:

a $250 USD equivalent payout for integrating with the Ethereum Profiles API (3box-js). Must make use of set, get, remove. (More than just getProfile)
a few small bounties for testing the 3Box dapp and identifying functional bugs (not design/ui)

Any thought on this, how to make ^ better, or other ideas? Going to try to get them up tomorrow so developers can plan around them maybe.

Will be working on drafts in this doc and posting tonight or tomorrow morning hopefully


Setting overlap_threshold to .2 throws error

@prenc I also noticed that setting the overlap threshold to .2 (and maybe lower) and clusters to say 80 or so causes an error.

@prenc
Below is a screenshot

The input matrix has about 19,000 rows and ~350 columns (because it is a medical dataset unfortunately I can't provide the matrix itself).

@grant-roy, I figured out what the problem is. Even though you set the overlap threshold to 0.2, the rank list (stores the best chromosomes) has only 38 of them while you are looking for 80. In the next version, it will be fixed, and only as many as there are up to max_biclusters will be taken.
Also, most of the currently fixed parameters will be available as arguments, so I'd suggest using a bigger population and reproduction size for such a dataset.

@prenc  I'll try it in the next version. Thanks for having a look.
I will also try to optimize the parameters, I was going to ask about these, glad to hear they will be available as arguments. Let me know if you have any other suggestions, thanks again.

@grant-roy The next version is almost ready (together with a new reworked README). You can try it out from here. Let me know if there is still something you need. We're going to continue developing the algorithm with new features.

@prenc sounds great, thanks.  I'll try it out and let you know how it goes (and offer any suggestions for features if I have any).


Timestamps for detected words?

For transcriptions with a souce audio file, is there any way to return timestamps for words detected by the recognizer? (Eg. if it detected 'mistake' at 1m:03s of the audio file, then a function to return this timestamp.)
I'm trying to develop a lightweight speech editor that deletes repeated words, combining speech_recognition with pydub/librosa. If there are any workarounds fo this, or if this would be better addressed using another py package, do let me know.

Just looked at the source code... and there doesn't appear to be any support for timestamps. Theoretically, if the underlying speech recognition engine (Sphinx, Google Cloud Speech API, etc.) supported providing back timestamps, then SpeechRecognition could add it as a "special feature" sort of like it does for the Sphinx keywords.
(Note: For those curious about my mention of "special features"... I didn't see the Sphinx keyword special features documented anywhere... but it does appear in the source code and in the tests/test_special_features.py file)


Subtitles

Any way to get .srt and .sub subtitles to work?

Criminal Justice(2019) S01 E01 WebRip z -  mkvCinemas.mkv

Criminal Justice(2019) S01 E01 WebRip z -  mkvCinemas.mkv

Sorry?

Only support srt

Yes ended up converting the srt files to vtt :)
Usimg this
https://github.com/nwoltman/srt-to-vtt-cl
U should mention this so that same question isn't raised again 😅
Also 10bit video is black in the browser
Most probably because browsers don't support it right?

1.
x265 encoded video requires browser support


Issues noticed exported collection when exported as Postman Collection

I have checked the following:

[X] I use the newest version of bruno.
[X] I've searched existing issues and found nothing related to my issue.

Describe the bug
When we perform export from Bruno API client, using option as "Export" --> "Postman Collection" noticed following issues in exported file:

Sequence of API requests is not maintained as arranged in API client, rather its exported in alphabetical order which is not correct. As a part of chaining of requests we made need to run API call 1 first and then API call 2 due to dependency of data. If now post export - if API call 2 is documented before API call 1 then when run via Postman API client or Newman CLI, scripts will fail as sequence is not correct.

As per screen shot, "ping" is first request in collection/collection folder but when we perform "Export" ---> "Postman Collection" the "ping" API request has been sequenced in last due to alphabetical order.
Also, "auth" JSON object/ JSON parameter is missing from exported file- (i.e. exported using option "Export" - "Postman Collection") across all API request wherein Auth is selected as "Bearer Token" for each of API request.
Also, all issues mentioned / captured in below ticket are also reproducible in exported file (wherein file is exported as "Postman Collection" from Bruno API client)
https://github.com/usebruno/bruno/issues/2496

.bru file to reproduce the bug
NA
Screenshots/Live demo link
NA

Sequence of API requests is not maintained as arranged in API client, rather its exported in alphabetical order which is not correct. As a part of chaining of requests we made need to run API call 1 first and then API call 2 due to dependency of data. If now post export - if API call 2 is documented before API call 1 then when run via Postman API client or Newman CLI, scripts will fail as sequence is not correct.

Can confirm. Simply saving requests will also reorder them in alphabetical order for me. Version 1.18.1 does not have this issue iirc.
Greetings :blush:


Initial commit

Draft of changes for @electrum and @dain
Feedback on the general approach would be welcomed. I'm keeping the port of the kerberos auth_to_local rule parsing as close to the C code as possible, with the intention that it should be fairly self-evident that it does the same thing.

Code available here: http://web.mit.edu/kerberos/dist/index.html
aname_get_selstring is in src/lib/krb5/os/localauth_rule.c

Can you explain at a high level what this does?

@dain: This is going to implement Kerberos auth_to_local rules (http://web.mit.edu/kerberos/krb5-devel/doc/admin/conf_files/krb5_conf.html#realms) so you can use them for checkCanSetUser.
There is some interest in some subset of that functionality at a POC, and we figure re-using the existing de-facto standard rule syntax is probably a reasonable approach.

I didn't know that existed.  Looks nice to me.  @electrum what do you think?

The big open question (apart from "are you folks generally OK with this plan") is how to get the config into it. The rules don't lend themselves to being shoved into a properties file:

We'd like to keep this as close as possible to the krb5.conf syntax.
The order of the rules is important.
Basically all parts of the rules are optional, so cramming them all into one property and trying to split them correctly seems iffy.

Pending answering that, I'm going to write the code that parses the rules and tests for it, and we'll figure out how to get the rule text into the code later.

@dain and @electrum: I have to run, but I should have some time later this evening to answer questions.
Thanks for taking a look!


Alpine 3.8

https://alpinelinux.org/posts/Alpine-3.8.0-released.html

I am waiting for the Docker image: https://github.com/docker-library/official-images/pull/4501

@frol no rush and keep up the good work. Thanks.

https://github.com/frol/docker-alpine-glibc/commit/a562f7695834ff5223c1bcd75459e0288fbcbc89


Question: What is the relation between Pylance and Pylint?

Hello,
I know I'm supposed to report an issue not to post a question, but I can't find answer anywhere else.
Is Pylance supposed to replace Pylint?

Pylance is a logical linter. It provides functionality like diagnostics (errors/warnings), quick fixes (like add/remove import, refactoring), completions, semantic colourization, type checking and more. Pylint provides a subset of Pylance functionality plus some stylistic linting features (e.g. PEP-8 compliance diagnostics). You may not need Pylint if you are using Pylance and find that it provides all the features you need.


MudTreeView: Server side data not downloaded if the custom node template is used

Bug type
Component
Component name
MudTreeView
What happened?
We tried to connect server side data (https://mudblazor.com/components/treeview#server-side-data) and custom tree (https://mudblazor.com/components/treeview#custom-tree) in the code below. Unfortunately, the LoadServerData method doesn't run, and the children of the "Categories" node aren't displayed.
@inject ISnackbar Snackbar

<MudTreeView Items="TreeItems" ServerData="LoadServerData">
    <ItemTemplate>
        <MudTreeViewItem Value="@context" <EMAIL_ADDRESS>            <Content>
                <MudTreeViewItemToggleButton<EMAIL_ADDRESS><EMAIL_ADDRESS>/>
<EMAIL_ADDRESS>            </Content>
        </MudTreeViewItem>
    </ItemTemplate>
</MudTreeView>

@code {
    public class TreeItemData {
        public string                Title      { get; set; }
        public bool                  IsExpanded { get; set; } = false;
        public bool                  IsScanned  { get; set; } = false;
        public HashSet<TreeItemData> TreeItems  { get; set; }
        public bool HasChild => !IsScanned || (TreeItems != null && TreeItems.Count > 0);

        public TreeItemData(string title) { Title = title; }
    }

    HashSet<TreeItemData> TreeItems { get; set; } = new();

    protected override void OnInitialized() {
        TreeItems.Add(new("All Mail"));
        TreeItems.Add(new("Categories") {
            TreeItems = new() {
                new("Social"),
                new("Updates"),
            }
        });
        TreeItems.Add(new("History"));
    }

    async Task<HashSet<TreeItemData>> LoadServerData(TreeItemData parentNode) {
        Snackbar.Add(parentNode.Title, Severity.Warning);
        await Task.Delay(2000);
        parentNode.IsScanned = true;
        return parentNode.TreeItems;
    }
}

Expected behavior
After clicking on the expand icon on the left of the "Categories" node:
a) the snackbar should be displayed,
b) the loading icon on the left of the node should be displayed, and
c) the "Social" and "Updates" child nodes should be displayed.
Reproduction link
https://try.mudblazor.com/snippet/QumbPPFhJbSAoqRV
Reproduction steps

Click on the expand icon of the "Categories" node

Relevant log output
No response
Version (bug)
6.0.2
Version (working)
No response
What browsers are you seeing the problem on?
Chrome
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I am having the same issue, as soon as I am trying to use a custom item template, the serverdata method wont be called no matter how you set up the properties. Some time has passed since the last post, does anyone know a workaround or fix for this now?

Still an issue. :(

This https://github.com/MudBlazor/MudBlazor/pull/3831 should cover the issue above.


Google maps context menu

I have implemented the context menu to my google map, but for some reason it only works before the map has been loaded in.
here are my codes
HTML:
map.component.html (1).txt
Typescript:
map component typescript.txt
newb at both angular 2 and github here, sorry in advance

newbs are great!  It means more people joining the community.
Can you put a console.log in your onContextMenu function?  That we can know if that's even being called.

i placed a console.log in the function. the function is not being called when my map is loaded.

It looks like sebm-google-maps doesn't emit the default (contextmenu) event.  Instead it has an output of mapRightClick.

I don't think you need the *ngFor="item of items" (unless you're planning to have multiple maps on the page).

Try this in the html:
<sebm-google-map [latitude]="lat" [longitude]="lng" [zoom]="zoom" [disableDefaultUI]=false [zoomControl]="false" (mapClick)="mapClicked($event)"
         (mapRightClick)="onContextMenu($event)" >

The item value in onContextMenu will be undefined, but that's fine.  Let me know if your console.log starts sending messages to the console.

i have since edited my codes alot since i first posted this issue so here are the new codes, though i dont think i have edited anything related to the context menu
HTML:
map component.html.txt
Typescript:
map component.ts.txt
The component is being called and send the messsage but there are errors

Error in my console.log once edit my codes to output of mapRightClick:
Uncaught ViewWrappedError
EXCEPTION: Cannot read property 'document' of undefined
TypeError: Cannot read property 'document' of undefined
Uncaught TypeError: Cannot read property 'document' of undefined
P.S Thank you very much for spending time to help me with this.

Can you change your console.log to this:
console.log('ayy context menu', $event);

And send me the event object that is printed out.  It looks like it doesn't have some properties that I expect it to have.  And what browser are you using?

The new message in my console:
ayy context menu Object {coords: Object}

For my application im using google chrome

found this error in my IDE:
file: 'file:///c%3A/Users/L33902/Desktop/Projects/mapit/node_modules/angular2-contextmenu/tsconfig.json'
severity: 'Error'
message: 'File 'c:/Users/L33902/Desktop/Projects/mapit/node_modules/angular2-contextmenu/./typings/browser.d.ts' not found.'
at: '1,1'
source: 'ts'
could this be the problem?

The problem is that sebm-google-maps is not giving you an actual MouseEvent just the coords part of it.
I'd file an issue with them asking for the whole event to be emitted by the EventEmitter.  They could emit an additional event or sourceEvent property if they want to make sure not to make breaking changes.
You could theoretically create your own event but I have no idea how to calculate a browser x/y coordinate from their provided lattitude and longitude.  Here is how you could create your own event (with static x and y values) if you just want to see the context menu show up.
let ev = new MouseEvent('contextmenu', {
        'view': window,
        'clientX': 10,
        'clientY': 20
    });
this.contextMenuService.show.next({
      // Optional - if unspecified, all context menu components will open
      contextMenu: this.contextMenu,
      event: ev,
      item: item,
    });

The reason i wanted to do this was to allow for user to open a context menu at a point to drop a marker but from my understanding i cant do this anymore?
Well it cant be help angular 2 is still at a very early stage.
Thanks for the help. I guess this can be closed for now?

You can do it if you get the other library to pass the MouseEvent to you

otherwise no.

On 3/29/17 20:57, ikeashark wrote:

The reason i wanted to do this was to allow for user to open a context
menu at a point to drop a marker but from my understanding i cant do
this anymore?
Well it cant be help angular 2 is still at a very early stage.
Thanks for the help. I guess this can be closed for now?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/isaacplmann/angular2-contextmenu/issues/56#issuecomment-290270408,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0lQGcp1KMi4jvCFP60JKALr6DOHVnJks5rqv4AgaJpZM4MiKKU.


Support contextmenu events (aka right-click)

Can you run build.sh please?

@jfyne done

@delkopiso thanks


fix: full width mega menu displacement and content overflow

📑 Description
Fixes a displacement and overflow of the full-width mega-menu that happens due to the default space-x of the parent navigation element.

Current
Updated

Current ref.: https://flowbite-svelte.com/docs/components/mega-menu#Full_width_with_image

Status

[x] Completed

✅ Checks

[x] My pull request adheres to the code style of this project
[x] I have updated the documentation as required
[x] I have checked the page with https://validator.unl.edu/
[x] All the tests have passed
[x] My pull request is based on the latest commit (not the npm version).

ℹ Additional Information

Summary by CodeRabbit

Style

Updated the MegaMenu styling to improve layout when in full mode.

I don't see the problem.

I don't see the problem.

Thanks for the review @shinokada
It's in the image you shared. I'm adding annotations to the image to make it obvious. The "want-state" is visible in the Updated cell in the PR description.
The PR was opened and tested on linux. Now I retested on macOS in a Chromium based browser and also Firefox and can confirm the issue exists.

Thank you for your PR.


Clone, Push, and Pull

Is it possible to clone, push, and pull repositories using this framework?

Yes, I believe clone, push, and fetch are theoretically possible now. I'm not sure pull is since that would require rebase or merge.

+1 on this. I have to implement clone and/or pull as part of a project I'm working on... so you may see a pull request for me if nobody gets to it first.

Can we split this up into 3 separate issues, so they can be tracked separately? I'm on the path to implementing these, but they will probably be done with separate pull requests at separate times... might make it easier to track


replace None with Void

All uses of None should be changed to Void. This will work on both 0.3 and 0.4.

Done


Some tools cannot be upgraded on MacOS

Describe the bug
rtx upgrade fails with mkdir: cannot set permissions
To Reproduce

have older version of a tool installed (for me happens with node).
run rtx upgrade

Expected behavior
Latest version of the tool is installed
** Actual behaviour**
mkdir: cannot set permissions '/var/folders/ty/6r9cq6bs047_f23vx4dmlvcw0000gp/T': Permission denied
node-build: TMPDIR=/var/folders/ty/6r9cq6bs047_f23vx4dmlvcw0000gp/T is set to a non-accessible location
mkdir: cannot set permissions '/var/folders/ty/6r9cq6bs047_f23vx4dmlvcw0000gp/T': Permission denied
node-build: TMPDIR=/var/folders/ty/6r9cq6bs047_f23vx4dmlvcw0000gp/T is set to a non-accessible location
rtx<EMAIL_ADDRESS>[ERROR] /Users/czar/Library/Caches/rtx/node/node-build/bin/node-build exited wit ✗ 0s rtx failed to install<EMAIL_ADDRESS>
rtx doctor output
rtx version:
  2023.10.1 macos-arm64 (19220e5 2023-10-05)

build:
  Target: aarch64-apple-darwin
  Features: DEFAULT, NATIVE_TLS, OPENSSL, SELF_UPDATE
  Built: Thu, 5 Oct 2023 00:14:46 +0000
  Rust Version: rustc 1.72.1 (d5c2e9c34 2023-09-13)
  Profile: release

shell:
  /bin/zsh
  zsh 5.9 (x86_64-apple-darwin22.0)

rtx data directory:
  /Users/czar/.local/share/rtx

rtx environment variables:
  RTX_SHELL=zsh

settings:
  {"always_keep_download": "false", "always_keep_install": "false", "asdf_compat": "false", "disable_default_shorthands": "false", "disable_tools": "[]", "experimental": "true", "jobs": "4", "legacy_version_file": "true", "legacy_version_file_disable_tools": "[]", "log_level": "INFO", "missing_runtime_behavior": "warn", "plugin_autoupdate_last_check_duration": "10080", "raw": "false", "trusted_config_paths": "[]", "verbose": "false", "yes": "false"}

config files:
  /Users/czar/.config/rtx/config.toml
  /Users/czar/.rtx.toml

plugins:
  bun           (core)
  deno          (core)
  go            (core)
  gradle        https://github.com/rfrancis/asdf-gradle.git#9a2ca80
  java          (core)
  kubectx       https://gitlab.com/wt0f/asdf-kubectx.git#8c0b4aa
  node          (core)
  python        (core)
  ruby          (core)
  spring-boot   https://github.com/AlexCzar/asdf-spring-boot.git#ec2f7b6

toolset:
  spring-boot@latest, java@20, bun@latest, node@18, deno@latest

No problems found

Additional context
Platform: MacOS MacBook Pro M1
rtx install type: cargo
Other tools I have are upgraded successfully.

I have no idea what is going on with your machine but this isn't an rtx issue

If anyone else encounters this issue and sees this, the root cause appears to be on the node-build side:
https://github.com/nodenv/node-build/issues/839
https://github.com/nodenv/node-build/issues/840


Simple regex

Description
In order to validate user input, we first sanitize its value to mitigate for typos.
We are using the following steps:

change value to lowercase
remove all white spaces
remove non-alphabetical characters with the exception of . , ! , and - . (These characters are part of the <!--...--> and <!DOCTYPE> tags and must be allowed.)
change value of each tag to lowercase (to account for the all-caps !DOCTYPE tag)
finally, check if sanitized user input matches an element from the tags array. (Please note, this functionality was already in place. The code had to be refactored though to change each tag to lowercase before comparison.)

Note
One idea for a new issue: List every tag following convention. Right now, each input is listed will all letters in lowercase. HTML tags are not case-sensitive, so main, Main, MAIN, and other variations are all equivalent. The same applies to !DOCTYPE or !doctype. Since the convention is to use lowercase for HTML tags for consistency and readability, and all caps for document type, it would be a nice enhancement to add as well.
Related Issue
Closes #15
Acceptance Criteria (AC)

[x] Trimming unnecessary white spaces at the beginning and/or end of the input.
[x] Ensuring case-insensitive matching.
[x] Stripping off any inclusion of angle brackets, or non-alphabetic characters in general.

Note
As mentioned in the Description section, it was necessary to allow for . , ! , and -.
Type of Changes

Type

✓
:sparkles: Enhancement

Testing Steps / QA Criteria
Either one of these inputs should be accepted. I am using !DOCTYPE as an example, but testing with any other tag should also work.

!DOCTYPE
!doctype
!Doc  type
<!doctype>
<   !DOCTYPE   >

Well done @CorinaMurg 🥳


Support ingestion from public buckets with limited permissions

As discussed in https://github.com/rilldata/rill-developer/issues/1594, sometimes we are not able to resolve S3/GCS downloads from public buckets. Let's investigate this and see if it's possible to address.

My hypothesis is that the object store has different permissions for downloading a file and doing meta/head requests (to get size info). Maybe some public buckets only give people download permissions and not meta/head permissions. Our S3/GCS connectors might rely on being able to do meta requests.
If that hypothesis is true, maybe we should try downloading a file anyway if the meta requests return permission errors. (This would only be for single files, for globs we need ability to do listings.)


Terminus connection-mode showing success on SFTP switch in Test, Live

If you don't pass an env parameter via Terminus when calling the connection-mode function, it will return a list of all environments for that site, including Test and Live. This comes from the static function env():

public static function env( $args = array(), $key = 'env', $label = 'Choose environment', $choices = null) {
if (!$choices) {
$choices = array('dev','test','live');
}
if ( isset($args[$key]) ) {
return $args[$key];
}
return self::menu($choices, $default='dev', $label, true);

}
Which is totally fine 99.9% of the time for all the different functions that call env(). Just not with the connection_mode(). Also, when telling Terminus to switch those two environments from Git to SFTP, is shows success instead of failure.
$ terminus site connection-mode --site=peace-church-marinette --set=sftp

dev
test
live

Choose environment: 2
Success: Successfully changed connection mode to sftp
$ terminus site connection-mode --site=peace-church-marinette --env=test
Connection mode: Git
$
The takeaway is that the connection-mode() has some interesting not so edge case issues. I've done some initial code on my local version of Terminus to address this that seems to work fairly well.

I think a combination of #238 and #285 fixes this issue.


How to use alternative themes of material icons

Google announced a bunch of updates to Material Guidelines during io2018. One of the updates is themes of material icons - https://material.io/tools/icons/
It now has "Filled", "Outlined", "Rounded", "Two-Tone",  and "Sharp".
In Flutter, I know how to use regular icons which are "Filled" ones. I also know how I can add individual assets to the assets folder in Flutter app and use those in my code but I'd like if they were readily available so that you could use them like -
IconButton(
  icon: Icon(Icons.volume_up), // <=== Point of interest here
  tooltip: 'Increase volume by 10%',
  onPressed: () { setState(() { _volume *= 1.1; }); },
)

Looking at this repo I think it only supports default (Filled) right now but are there any plans to add other themes?
Probably
IconButton(
  icon: Icon(Icons.volume_up, theme: IconTheme.outlined), // <=== Example
  tooltip: 'Increase volume by 10%',
  onPressed: () { setState(() { _volume *= 1.1; }); },
)

Same question here.. ??

I've made a package that lets you use the the 'outline' theme only: https://pub.dartlang.org/packages/outline_material_icons

I think if I were to implement it, I would pass a parameter (enum most probably) to the Icon constructor rather that creating new classes like FilledIcons.XXX or OutlinedIcons.XXX. That way it is scalable if there were new styles to be added in future and it will also stay backward-compatible by defaulting the parameter value to .Filled which was default before multiple styles were announced.

To me it would make more sense to leave Icons. as is as it is more like a "set of icons" just by their identity and not adhering to a particular theme. To get the icons in different themes, I think it makes more sense to pass it as an enum parameter to the Icon class constructor instead of creating multiple FilledIcons, OutlinedIcons. It would definitely be nice to have those separately created but it does not scale if there were more themes to be added in future. Passing a param with default value set to .Filled would be ideal IMHO.

Ahem, it's september 2019, nothing is done about this???????

I think they are available now?

+1

Work is currently underway to bring this functionality and additional optimizations for icons. Can't give a release estimate so in the meanwhile, I'd continue to get the icons you need as png assets from material.io if you're blocked. But it's a current priority.

https://github.com/flutter/flutter/pull/54923 is last dependent PR. Then we will update the icon font and consts!


docs: document consensus rules from 4.5 Output Descriptions

Motivation
We need to document all consensus rules.
Specifications

Designs

Solution
Closes https://github.com/ZcashFoundation/zebra/issues/3217
Closes https://github.com/ZcashFoundation/zebra/issues/3225
Review
Not urgent. Anyone can review but I think it's best for @dconnolly to take a look
Reviewer Checklist

[ ] Code implements Specs and Designs
[ ] Tests for Expected Behaviour
[ ] Tests for Errors

Follow Up Work

@Mergifyio refresh


Eng traindata and tight text

There is an issue when you write "I'd" really close together and when you end your sentence with " '. " also really tight.

Do you know which font it is?
You could try fine-tune with a narrow font and see if there is any
improvement
On 03-Oct-2017 3:58 PM, "Daniel"<EMAIL_ADDRESS>wrote:

There is an issue when you write "I'd" really close together and when you
end your sentence with " '. " also really tight.
[image: example1]
https://user-images.githubusercontent.com/1424084/31120816-25062e2a-a836-11e7-9c4c-d3e6304210dd.png
[image: example2]
https://user-images.githubusercontent.com/1424084/31120814-24b86bd6-a836-11e7-95ce-5a6abad666cd.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tesseract-ocr/tessdata_best/issues/7, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AE2_owKdl00ETA5S9ThvQ_5oO4eMYY7Sks5sogwxgaJpZM4Pr5gs
.

I get this text:
% tesseract https://user-images.githubusercontent.com/1424084/31120816-25062e2a-a836-11e7-9c4c-d3e6304210dd.png -
Estimating resolution as 734
In a ground-breaking chapter, Clive Leach and Suzy Green make a case for the integration of
coaching and positive psychology within educational settings. The evidence-base for the use
of positive education is surveyed and practical suggestions for the use of coaching and posi-
tive psychology are made. This chapter is supported by two case studies of the use of
integrated approaches in schools: ‘A strategic approach to enhance wellbeing’ and ‘A strength-

based coaching programme:
% tesseract https://user-images.githubusercontent.com/1424084/31120814-24b86bd6-a836-11e7-95ce-5a6abad666cd.png -
Estimating resolution as 753
Id like to start by thanking everyone involved with the West Midlands Coaching Pool,
particularly Samantha Darby and Rebecca Davies. This is where I developed my passion for
coaching and the impact it can have within professional contexts.

That looks pretty good. Only a superfluous linefeed was added.


question: How to get an ID of a (inserted/updated/deleted) record using Query Builder?

Could you please help me?
DB::connection('mongodb')->collection('apiRequests')->insert($entry);
How to get that record ID?
PS: DB::getPdo()->lastInsertId(); doesn't works :(

In laravel is this in the model of Eloquent (->getKey())

namespace App\Models\MongoDB\Cases;
use Jenssegers\Mongodb\Model as Eloquent;
class MyModel extends Eloquent {
protected $collection = 'xxxtableMongo';
protected $connection = 'mongodb';
}
in controller
$myModel->save()
$myModel->getKey();


Add graph for hierarchical agent teams

Add graphical displays for specific scenes to better represent the current state.

Great!


Added Announcement Block Feature

@ujjwaleee26 STOP SPAMMING! Do not open and close pull requests for the same issue. See https://6.docs.plone.org/contributing/first-time.html#things-not-to-do, Item 3.


Relative Font size update for tooltip class

Is your feature request related to a problem? Please describe.
We are using this plugin on our websites, but on a scan of the pages for accessibility, we are getting flagged for using font measurements in pixel instead of relative font measurements for font-size in particular, which is the expected norm for accessibility requirements for our websites.
Describe the solution you'd like
Need an update of the font-size in the scss (line no. 371) , https://github.com/wakirin/Litepicker/blob/master/src/scss/main.scss
classname : &__tooltip
font-size: 11px; to corresponding relative measurement, namely 0.6875em (or) 68.75%
Describe alternatives you've considered
Currently no known alternatives available on our side.
Additional context
Add any other context or screenshots about the feature request here.

Hi team, Can we get an update on this issue / can this item be prioritized ?

Hi Team ,Can we get can status on this issue and when it be fixed  /can this item be  prioritized ?


pd.get_dummies(s) with s being a logical test

Hi,
I am using Pandas more and more for preparing my dataset before feeding it to other statistical software.
I believe something really useful and, somehow, missing from the current version of Pandas is a simple way to generate dummy variables based on a logical test.
Dummy variable coding (0-1) is everywhere in the academic literature (I have a phd in economics) and are a key component of many statistical tests.
Something like
 pd.get_dummies(df.A > df.C) that would generate a dummy that is one if A>C and zero otherwise, allowing the user to

include/exclude missing variables
specify what happens when columns have mixed types (see (https://stackoverflow.com/questions/36373681/how-to-create-dummy-variables-in-pandas-when-columns-can-have-mixed-types)

Of course, we can always generate the dummies above manually, but the experience shows that this simply dummy variable creation can be, at times, tricky.
 This is important. For instance, in Stata missing values are evaluated as +infinity. So something like 2> NaN would be False. In Sas, this is the opposite. We need control over this.

similar logic would of course apply for logical tests based on dates comparisons (useful for time-series regressions)

actually, it would be even more precise to define the following options

comparing string to numeric = return NA or zero
comparing numeric to na = return NA, zero (implicitly assuming NA=infinity) or 1 (implicitly assuming NA=-infinity)

last: this also applies to string comparison. say master.A=="jreback"

So, ignoring all the dtype issues, your main desire is a

simple way to generate dummy variables based on a logical test.

I'd say this is done with (df.B > df.C).astype(int), right? Again, assuming all the types are OK.
Bringing back the dtype issues, makes thing tricker from a library perspective, and a bit more difficult for the user.

comparing numeric to string= return nan or 0

I think we've been moving towards being stricter about this recently. It's better to put the burden on the user rather than guess at what they want. So that means you'd need.
master['B'] = pd.to_numeric(master['B'], errors='coerce')
In [41]: (master['B'] > master['C']).astype(int)
Out[41]:
0    1
1    1
2    0
3    1
4    0
5    0
dtype: int64

to fill the NaNs. Or create a new column if you need to keep the original around.

comparing numeric to na = return nan OR zero (implicitly assumingnan=+ infinity) OR 1 (implicitly assuming nan=- infinity)

This goes against python / numpy / IEEE behavior of all numeric : NaN comparisons evaluating false.
If you want the nan to be like -inf, then (master['B'] > master['C']) | pd.isnull(master).any(1).

of after @TomAugspurger coercion step.
In [6]: pd.get_dummies(master.loc[master['B'] > master['C'], 'B'].astype('category'))
Out[6]:
   1.0
0  1.0
1  1.0
3  1.0

@randomgambit pandas is all about building pipelines of simple operations and combining them, rather than trying to do everything with lots of options in a single function (not mentioning R here :)

@randomgambit you might want to read working with text data and indexing where these topics are covered quite a bit.

fair enough.
I agree with you that my suggestions above may be too 'greedy' from a Pandas philosophical point of view. After all, what is really important here is predictability and control.
My point is that it is somewhat frustrating to have a powerfull dummy variable creation tool such as get_dummies, that can in a single coding line generate legit dummy variables out of columns looking like.
Out[6]:
0  "jreback randomgambit"
1  "tom randomgambit"
3  "jreback"

but I have to rely on the two step process to_numeric followed by get_dummies and astype. Consider how many lines I need to write if I create dummies based on multiple columns (which happens, well, every time).
Maybe the simplest solution to my suggestion would be to come up with

pd.get_dummiesc(logical condition involving several columns, errors='coerce')

which give zeros every time there is an exception.

@randomgambit
the point of pandas is to clean your data, there are tremendous variations. I am not sure I understand why any of this is a problem. A simple pipeline is all that is needed. How do I know that [16] is NOT legititmate? it looks fine to me, but you may disagree, so this requires the user to do something with it. pandas CANNOT just guess at this.
In [14]: s = Series(["jreback randomgambit", "tom randomgambit", "jreback"])

In [15]: s
Out[15]:
0    jreback randomgambit
1        tom randomgambit
2                 jreback
dtype: object

In [16]:

In [16]: (s.str
   ....:    .extract('(\w+)', expand=False)
   ....:    .pipe(pd.get_dummies)
   ....: )
Out[16]:
   jreback  tom
0      1.0  0.0
1      0.0  1.0
2      1.0  0.0

hi @jreback thanks for your feedback. I think you may have misunderstood my point. sorry if I was not clear.
all im saying is that it is somehow frustrating to see that get_dummies is able to generate dummies out of complex string columns as you describe, but is unable to get dummies out of basic columns comparisons.
now, i totally agree with you that if we agree to disagree, the choice should be left to the final user. :)

@randomgambit oh I get your point. But we are not going to be adding lots of options to a method, where there is already a clear way of doing this. If you have a convincing example pls show it.

@randomgambit thanks.
I think we'll close this one for now, with the understanding that a dedicated method for this kind of comparison isn't a good fit for pandas. Rather, we'd encourage separately handling the dtype issues, the comparisons, and then the get_dummies (but I think .astype(int) suffices here).
But please do raise issues as you discover more usability concerns. It makes me happy to see others with an economics background using pandas :)

ok, i gave full hearths to both of you :D
thanks!


The occupation standards import template should support the organization its registered with

I'm asking Megan to create an import file for rapidskills that'll have all of hcap's standards and all of urban's.
https://hcapinc.org/national-library
and
https://www.urban.org/policy-centers/center-labor-human-services-and-population/projects/competency-based-occupational-frameworks-registered-apprenticeships
None of urban's are registered but all of HCAPs, and they're registered with different DOLs / states / etc. Can we update the import to support giving the registration organization name so we can use that to build the registration?
For example, here's what we get on HCAP's site:

You can see the tags "State" and "PA" means they're registered with "Pennsylvania Department of Labor and Industry" and associated with the state of Pennsylvania. For the other example above, it's "AK" and "Federal". These should all be "U.S. Department of Labor" but associated with the state of Alaska.

@Pcushing just want to make sure I understand what is needed for this task.  For the current import, we assume that the organization listed in the organization_title must exist already in our database.  For this task, if the organization does not exist, then it should be created, rather than returning an error, correct?

@littleforest correct.


[core] Compilation issue due to variable inlining

This example produces a compilation issue after decompiling.
import java.util.HashMap;
import java.util.Map;

public class JadxTest {
  public String test() {
    Map<String, String> map = new HashMap<>();
    return map.get("test");
  }
}

This is the decompilation result (tested with openjdk 8, latest git, java-input and java-convert)
import java.util.HashMap;

public class JadxTest {
    public String test() {
        return new HashMap<>().get("test");
    }
}

In my opinion named variables (visible in smali) should not be inlined. Their names might help to understand the context.
This example can be used for unit tests.

@nitram84 I commit a change to reject inline of named variables, so this issue is resolved.
But I don't check for missing generics (i.e. new HashMap<>()), because it is only possible if type with generics applied for variable, but name got rejected. Not sure if this needed, so maybe I will fix this later.
Also, during fix I encounter many small issues in different parts of jadx and these changes can cause new regressions, hope not so many :sweat_smile:

@skylot Thank you for fixing this issues! I haven't seen regressions due this fix so far. I think this issue can be closed. For possible regressions or issues with generics I think the best way is to open a new issue if there is a failing case.

For possible regressions or issues with generics I think the best way is to open a new issue if there is a failing case

@nitram84 sure, closing


#30 Add mobile-friendly elements to frontend NavBar

Description of Proposed Changes
Rssolves #30
Adds mobile navbar for FE.
Type of Change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

Testing
Added unit tests.
Checklists
GitHub

[x] Related issue appears at beginning of pull request title with pound sign, and title describes the changes being proposed.
[x] Related issue is linked in pull request description using closing keywords.
[x] The appropriate label has been chosen for this pull request.
[x] The correct project has been selected for this pull request.
[x] The milestone corresponding with the issue has been selected for this pull request.
[x] All commits in this branch, including merge commits, begin with the issue number and a pound sign.

Code Standards

[x] Code follows the style guidelines of this project.
[x] Code comments have been added, particularly in hard-to-understand areas.
[x] I have performed a self-review of my code.
[x] Any dependent changes have been merged and published in downstream modules.
[x] These changes generate no new warnings.

Documentation and Testing

[x] New and existing unit tests pass locally with these changes.
[x] Corresponding changes to the documentation have been made.
[x] Tests have been added, where appropriate; or, no tests are relevant for this pull request.
[x] This pull request contains UI changes, and screenshots of those UI images appear below; or, this pull request contains no UI changes.

Closing this for now, will re-do the story when having a mobile nav is warranted.


[Storage] Support Locationusage and WORM (Immutability Policy) and upgrade API version to 2018-02-01

Description

This is target to release on 5/7 build.
Since the tight schedule, please help to review ASAP.
The private PR pass check: https://github.com/Azure/azure-sdk-for-net-pr/pull/833
The unit test is modified by removing disable Storage Account Encryption test. Since server doesn't allow disable encryption now.
The swagger PR:
WORM: https://github.com/Azure/azure-rest-api-specs/pull/2937
Location Usage: https://github.com/Azure/azure-rest-api-specs/pull/3014 (in review)
The Rest Doc:
WORM: https://microsoft.sharepoint.com/teams/AzureStorage/Shared Documents/XLocation/SRP Service and Container Management.docx?web=1
Location Usage:
https://microsoft.sharepoint.com/:w:/t/AzureStorage/EYFU1AAlf3BBqV6C28ajBB0BpOXZDKOfZYnoQnZOU6wivA?e=uP50YO (See get location usage API section)

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] Please add REST spec PR link to the SDK PR
[x] I have read the contribution guidelines.
[x] The pull request does not introduce breaking changes.

General Guidelines

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[x] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[x] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code.
[x] The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK.

@dsgouda
I have rebase my code on the latest change on "psSdkJson6" branch and run msbuild build.proj /t:Build /p:Scope=SDKs\Storage, but don't see anything change.
I have created the generate.ps1 file and add to the PR.


Document that "F" (and "FF", etc) removes the preceding period if it doesn't print anything

Document that "F" (and "FF", etc) removes the preceding period if it doesn't print anything.
From the documentation:
DateTime date1 = new DateTime(2008, 8, 29, 19, 27, 15, 18);
CultureInfo ci = CultureInfo.InvariantCulture;

Console.WriteLine(date1.ToString("hh:mm:ss.F", ci));
// Displays 07:27:15

Indeed, there is code which explicitly removes this decimal point.
Would it be worth documenting this behaviour?

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

ID: dda93f14-f1b5-fde6-e441-302cdac261ab
Version Independent ID: 96abc7c4-3719-82da-e271-881d62f0640f
Content: Custom date and time format strings
Content Source: docs/standard/base-types/custom-date-and-time-format-strings.md
Product: dotnet-fundamentals
GitHub Login: @adegeo
Microsoft Alias: adegeo

@canton7 Thanks for the suggestion, the article update is in PR #22498

@canton7 Thanks for the suggestion, the article update is in PR #22498


Update for SMAPI 3.0

Update for compatibility with SMAPI 3.0 and SDV 1.4

Thanks for the help! I added it to my open PR in the original repo.


Digital Bitbox support

Hello. Thank you for your greatly useful wallet.
Stemming from numerous customer requests, we would like to add support for the Digital Bitbox hardware wallet (digitalbitbox.com) to MyEtherWallet. We are happy to provide a device for testing purposes.
As a reference for anyone reading, our latest firmware v2.2.2 is recently released and adds support for a MyEtherWallet integration (website, github). A copy of this PR branch is running at https://digitalbitbox.com/mew.
With respect to the referral link, for your information, details of our affiliate program can be found at https://digitalbitbox.com/policies#affiliate

Just shot you an email. 🎉

(Oh and don't worry about conflicts at the moment. We have a couple other updates so best to resolve right before we're ready to pull in. I can assist with this)

Thanks! We'll send test devices soon.

Just got them. Will take a look tomorrow and start testing! Thank you!

@douglasbakkum Please resolve conflicts :)

Also, please remove changes to chrome-extension/js/etherwallet-master.js and dist/js/etherwallet-master.js

Will do! Cheers.

@douglasbakkum I think i got it -- will you test? im on the road: https://kvhnuke.github.io/etherwallet/#send-transaction

@tayvano  Thanks!
Sorry - I missed this notification somehow. There is a problem connecting to the Digital Bitbox on this MEW version (our old local copy still works). I will try to fix it hopefully by end of tomorrow.
Cheers.

can you send me a BitBox for testing?

@tayvano I found the problem - it is on our end and will require a few extra days to fix (in firmware). The good news is that the Digital Bitbox will then only work with https://www.myetherwallet.com (and https://digitalbitbox.com) for Ether use cases. This will eliminate fake MEW site attacks if the Digital Bitbox is used.
@Zwilla If you would like to discuss dev work, please contact<EMAIL_ADDRESS>Note that we will only provide MyEtherWallet support to the official MyEtherWallet team.

@douglasbakkum does this mean that your stick will not work for users on cold places?

@Zwilla If you mean on an airgapped computer, then yes. Note that a hardware wallet IS a cold place (cold storage).

@tayvano I updated the firmware and fixed (and tested) the problem. Sorry for the delay -- it is important to take our time and be careful when signing and releasing firmware.
The new firmware is available at digitalbitbox.com/firmware
I will make a new PR to fix some minor bugs.
Thanks!


FundNode is not a signed operation.

#92  👈
main

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

Join @zhenlu and the rest of your teammates on  Graphite

Merge activity

Mar 28, 12:16 AM EDT: @zhenlu merged this pull request with Graphite.


fix: PG instances anti-affinity configuration

This patch is fixing the PodAntiAffinity configuration for PostgreSQL Pods,
allowing PostgreSQL and PgBouncer to coexist on the same node when
the anti-affinity configuration is set to required.
Closes: #4293

This PR is a combination of the work of @ceriath https://github.com/cloudnative-pg/cloudnative-pg/pull/3346 and of @CiraciNicolo https://github.com/cloudnative-pg/cloudnative-pg/pull/4294, with some nits changed by me.
It supersedes the two PRs above.

E2e tests: https://github.com/EnterpriseDB/cloudnative-pg/actions/runs/10093452879
I used the EDB branch to spare some OSS worker.

The operator upgrade tests are failing. Time to understand why

I was chasing ghosts. The online upgrade cannot work because PG Pods will change (the new anti-affinity section), and the new version of the operator will trigger a rollout.
I guess we should have a discussion and reach an agreement on what we want to do here.
We could add a note to the release notes explaining that upgrading the operator will trigger a rollout of the PG Pods and why this is happening, but this can be painful.

Another E2e run: https://github.com/EnterpriseDB/cloudnative-pg/actions/runs/10107122120

I think that a managed rolling restart of the pods should be acceptable given this fix; considering that in any kind of update - version, spec, ... - is already triggered.
I agree with you that a notice in the documentation is needed, probably will be enough for the majority of the user; meticulous user already have the habit to read carefully the release notes.

Upgrade E2e tests are green.
I started another full run: https://github.com/EnterpriseDB/cloudnative-pg/actions/runs/10108836516

Please don't merge on Friday 😢

E2e tests are green

/ok-to-merge


Proposed tweaks to printer style from HashiCorp internal discussions

Hey @fatih - amazing work on this! You've definitely taken a top-N side project off the list of many HashiCorp employees and community members. :smile:
We have an internal thread where we were trying to converge on a consistent documented style for HCL. Here's where the general consensus was headed:
//                     v brace MUST appear at end of first line of block
resource "type" "name" {
  // <-- indents MUST be two spaces
  an_attribute      = "value"
  another_attribute = "another_value"
  third_attribute   = "three"
  //                ^ equals for adjacent attributes SHOULD be aligned

  // ^ empty lines MAY be used to separate attributes into logical groups
  subblock {
    one   = "foo"
    two   = "bar"
    three = "qux"
    // ^^ attribute ordering not dictated, users MAY choose alpha or any other semantic ordering
  }
}

Seems like just the equals alignment and tabs-vs-2-spaces were the differences I could see on first glance of the printer output config.
Let me know what you think! Happy to dig into the implementation to make these tweaks as a PR.

Thanks for your work @fatih !
Just chiming in to +1 @phinze comments.
While the main goal is consistency and not the 'styling' itself, we happen to try and use the approach described above. It also happens to be the HashiCorp way it seems.
Would love to see this printer formatting in.

following this out of <3

Hi @phinze
I'm really happy to see you folks like it very much! This is an awesome feeling :)
I'm more than happy to tweak or adding additional settings to the print config. The good thing is I've created the packages and architect the whole systems so each component can be configured without affecting each other. hclfmt is using the printer package here, and the printer package has a way to receive configuration options. So let me try answer your questions:

//                     v brace MUST appear at end of first line of block

Can you give more detail about this, I didn't get this detail

// <-- indents MUST be two spaces

This is doable already. With the https://github.com/fatih/hcl/blob/master/printer/printer.go#L17 option. This will use spaces instead of tabs. Right now hclfmt is not supporting this however I'm planning to add an option to it or make it default and give people an option to use tabs. Such as :
hclfmt -usetabs

I'll change it, noted!

^ equals for adjacent attributes SHOULD be aligned

Good call! I'm doing it already for comments but not for values of object items (a.k.a attributes). Noted.

// ^ empty lines MAY be used to separate attributes into logical groups

I've thought about this a lot actually and eventually tried to enforce it. Why? Because I thought it would make an overall hcl file much more easier to read. For example gofmt does this with two and more declarations. Even though you add newlines, it groups them explicitly. Thoughts one that?

// ^^ attribute ordering not dictated, users MAY choose alpha or any other semantic ordering

So you mean the key names should be not ordered, only the object items (a.k.a attributes)? Can you give more detail?
--
Beside that, I'm thinking to add another mode called NoComments, which when enabled will strip out any comments. I think this might be useful for cases where comments are not needed and a more smaller, compact file can be used.
Thanks again for the kind words! :)

Hi @fatih
I'd like to echo @phinze's thanks :smile:

// v brace MUST appear at end of first line of block

The "v" is actually a downward arrow pointing to the opening brace on the next line. In other words:

```javascript
resource "foo" { // <- valid
}

resource "foo"
{ // <- invalid
}

// ^^ attribute ordering not dictated, users MAY choose alpha or any other semantic ordering

I think the idea here is that we should let the human decide how they want to order things. Think about gofmt - it doesn't care about the order of the fields in a struct, it only cares that they are aligned and have a trailing comma.

The "v" is actually a downward arrow pointing to the opening brace on the next line. In other words:

Aha now I get it :) That's the case already:

I think the idea here is that we should let the human decide how they want to order things. Think about gofmt - it doesn't care about the order of the fields in a struct, it only cares that they are aligned and have a trailing comma.

You are right. The printer doesn't care too about the ordering, it only aligns the comments right now and doesn't touch the order.
Btw just a question (also to @phinze too), right now we only align comments. What should be for the case below:
foo = "Fatih Arslan" # yoo 1
developer = "Phinze" # yoo 2
barbar = "Seth" # yoo 2

Right now we do it like:
foo = "Fatih Arslan" # yoo 1
developer = "Phinze" # yoo 2
barbar = "Seth"      # yoo 2

Once we change it to align according to the attribute, we have two ways doing it:
# only attributes are aligned, not comments

foo       = "Fatih Arslan" # yoo 1
developer = "Phinze" # yoo 2
barbar    = "Seth" # yoo 2

# both attributes and comments are aligned

foo       = "Fatih Arslan" # yoo 1
developer = "Phinze"       # yoo 2
barbar    = "Seth"         # yoo 2

Which do you think is more appropriate?

Which do you think is more appropriate?

Other opinions are welcome, but for my part, I think the spirit of "aligning for readability" is most captured in the final example w/ both = and # aligned.

Two cents on the alignment question: go fmt's desire to align keys and values in map/struct literals is constantly causing merge conflicts for me. While I do like how it looks once it's applied, it is pretty inconvenient that adding or changing one attribute counts as a diff for an entire block.
For Go I just suck it up because that decision was made long ago and anyway compiling/testing usually catches the times when I accidentally resolve the merge incorrectly (which is almost every time!), but I worry that in Terraform I'm much less likely to notice if I accidentally dropped an attribute because terraform plan diff output is rather noisy anyway, and so it's easy to miss things.
So I guess you could consider this a counter-plea to instead standardize on an hcl style where the layout of one line does not in any way depend on the layout of lines around it, so that when we add one line it really only adds one line, and doesn't instead tinker with the entire resource block. I don't consider the non-aligned version to be so unreadable as to warrant the inconvenience and risk of resolving unnecessary merge conflicts.

Other opinions are welcome, but for my part, I think the spirit of "aligning for readability" is most captured in the final example w/ both = and # aligned.

Agreed.
I also see @apparentlymart's point, but I don't think a VCS system should dictate a language choice. That being said, I have not once experienced merge conflicts as a result of aligning struct fields or Go formatting. As long as you are always formatting, I think it's fine. And because the output is so human-readable, merge conflicts would be easy to decipher.
You can always ask git to ignore whitespace changes on the CLI, or you can add ?w=1 to the URL on GitHub.

@apparentlymart I can understand your concerns, but hclfmt is optional as you know. So even though you can use it, you can always fix the lines yourself after the formatting is done. Another thing is, that it's hard to impossible to please everyone. I know this from gofmt already. However gofmt excelled because of that attitude. And as per @sethvargo, I did not encountered merge conflicts too.

@sethvargo I humbly submit that this is because you're not maintaining/rebasing old pull request branches for months on end as the master branch continues to evolve. :)
But I would also concede that the likelihood of long-living branches on an internal Terraform config is less than in code contributed to an open source project, and really the merge request problem only arises in the case of a long-lived branch. So, retracted.


chore: fix tester image copy pattern

Benchmark results
Metrics with a significant change:

circuit_witness_generation_time_in_ms (root-parity): 201 (-21%)
circuit_witness_generation_time_in_ms (merge-rollup): 145 (-43%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
64 txs

l1_rollup_calldata_size_in_bytes
740
740
740

l1_rollup_calldata_gas
6,680
6,692
6,680

l1_rollup_execution_gas
586,812
586,824
586,812

l2_block_processing_time_in_ms
1,357
5,073
10,122 (+1%)

l2_block_building_time_in_ms
22,979
90,426
179,722

l2_block_rollup_simulation_time_in_ms
22,836
89,937
178,779

l2_block_public_tx_process_time_in_ms
9,022
35,543
70,461 (-1%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
15,452 (-1%)
28,517 (-1%)

node_database_size_in_bytes
22,921,296
39,587,920

pxe_database_size_in_bytes
29,868
59,425

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes
circuit_witness_generation_time_in_ms
circuit_proving_time_in_ms
circuit_proof_size_in_bytes

private-kernel-init
112
16,772
33,766
N/A
N/A
N/A

private-kernel-inner
314 (+1%)
57,607
33,766
N/A
N/A
N/A

private-kernel-reset
562 (+1%)
200,856
33,766
N/A
N/A
N/A

private-kernel-ordering
323 (+1%)
53,688
48,421
N/A
N/A
N/A

base-parity
7.58
128
64.0
279
13,484
2,208

root-parity
48.8
27,064
64.0
:warning: 201 (-21%)
181,471 (-4%)
2,720

base-rollup
657
111,094
925
7,476 (-2%)
160,800 (-3%)
3,104

root-rollup
64.0 (+1%)
11,454
789
194 (+5%)
16,478 (-2%)
3,424

public-kernel-app-logic
122
61,017
51,751
292 (+1%)
434 (+4%)
65,344

public-kernel-tail
440 (+1%)
166,458
7,691
463
421 (-1%)
10,112

public-kernel-setup
101 (+1%)
61,017
51,751
290 (-1%)
397 (-3%)
65,344

public-kernel-teardown
104 (+2%)
61,017
51,751
273 (-10%)
340 (-2%)
65,344

merge-rollup
6.36 (-2%)
2,696
925
:warning: 145 (-43%)
3,733 (-4%)
3,104

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
512 leaves
1024 leaves
2048 leaves
4096 leaves
32 leaves

batch_insert_into_append_only_tree_16_depth_ms
11.0
17.9
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.7
31.8
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.642
0.549
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
52.0 (-1%)
80.9
263
504 (-1%)
988
1,958
N/A

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
95.9
159
543
1,055
2,079
4,127
N/A

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.532 (-1%)
0.499
0.479 (+1%)
0.471 (-1%)
0.468
0.467
N/A

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
61.4
120 (+1%)
379 (+1%)
740 (-1%)
1,474
2,958 (+1%)
N/A

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
106
208
692
1,363
2,707
5,395
N/A

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.536
0.538 (+1%)
0.517 (+1%)
0.509
0.511
0.515 (+1%)
N/A

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
67.1

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
108

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
0.588

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes
1 registered classes

tx_size_in_bytes
51,016
1,046,607

Transaction size based on fee payment method
| Metric |  |
| - |  |
Transaction processing duration by data writes.

Metric
0 new note hashes
1 new note hashes
2 new note hashes

tx_pxe_processing_time_ms
2,372 (+1%)
2,127
5,320 (+3%)

Metric
0 public data writes
1 public data writes
2 public data writes
4 public data writes
8 public data writes

tx_sequencer_processing_time_ms
657
894
961 (+1%)
1,758 (+1%)
1,151 (+1%)


Unterminated String JSON Error when using Structured Output

I can't find a solution to this issue. After debugging, I think the problem occurs when I send a prominent prompt and expect a more extensive response, and it runs off tokens. How can I instruct OpenAI to adjust the answer based on the remaining tickets?
The error occurs in this function:
 const matchingAnalysis = createStructuredOutputChainFromZod(
      JobDescriptionMatchAnalysisSchema,
      {
        prompt: promptForMatching,
        llm: modelForAnalysis,
      }
    );

My model is configured like this:
const modelForAnalysis = new ChatOpenAI({
    openAIApiKey: process.env.OPENAI_API_KEY,
    modelName: "gpt-3.5-turbo",
    maxTokens: -1,
    // streaming: true,
    temperature: 0.4,
  });

Full error below:
SyntaxError: Unterminated string in JSON at position 9145
    at JSON.parse (<anonymous>)
    at FunctionCallStructuredOutputParser.parseResult (file:///Users/ddeisadze/Documents/GitHub/resume-writer-aid/node_modules/langchain/dist/chains/openai_functions/structured_output.js:38:35)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async LLMChain._getFinalOutput (file:///Users/ddeisadze/Documents/GitHub/resume-writer-aid/node_modules/langchain/dist/chains/llm_chain.js:90:31)
    at async LLMChain._call (file:///Users/ddeisadze/Documents/GitHub/resume-writer-aid/node_modules/langchain/dist/chains/llm_chain.js:120:31)
    at async LLMChain.call (file:///Users/ddeisadze/Documents/GitHub/resume-writer-aid/node_modules/langchain/dist/chains/base.js:95:28)
    at async AnalyzeMatchBetweenResumeAndJobDescription (webpack-internal:///(api)/./src/analysis/matchAnalysis.tsx:85:47)
    at async handler (webpack-internal:///(api)/./src/pages/api/analyzeMatch.tsx:14:48)
    at async Object.apiResolver (/Users/ddeisadze/Documents/GitHub/resume-writer-aid/node_modules/next/dist/server/api-utils/node.js:366:9)
    at async DevServer.runApi (/Users/ddeisadze/Documents/GitHub/resume-writer-aid/node_modules/next/dist/server/next-server.js:481:9)

I'm having the same problem.
even configuring the maxToken continues, You commented to verify that the json was not finished, is there no way to do this verification in the code or improve the error log?


Upgrade Rails to <IP_ADDRESS>

This is to resolve an issue I was experiencing installing the dependencies related to mimemagic. I imagine this isn't currently affecting production because we run on Ubuntu which will have those files.
Fetching mimemagic 0.4.3
Installing mimemagic 0.4.3 with native extensions
Gem::Ext::BuildError: ERROR: Failed to build gem native extension.

rake aborted!
Could not find MIME type database in the following locations:
["/usr/local/share/mime/packages/freedesktop.org.xml",
"/opt/homebrew/share/mime/packages/freedesktop.org.xml",
"/opt/local/share/mime/packages/freedesktop.org.xml", "/usr/share/mime/packages/freedesktop.org.xml"]

It turned into a bigger upgrade than I was expecting for a patch update!

Seems okay to me, unsure on why the change from whitelist to allowlist but I'm assuming that's just a requirements hiccup.

Seems okay to me, unsure on why the change from whitelist to allowlist but I'm assuming that's just a requirements hiccup.

It was due to this change https://github.com/carrierwaveuploader/carrierwave/commit/4c3cac75f3a473e941045c23ebb781f61af67d79 that we started getting deprecation warnings, so I thought I may as well fix them.


Attaching an event handler to a cursor makes it loose every other item

This was the weirdest debugging session, I was almost pulling my hairs out trying to figure out what's happening here. In the end I have found out that adding an event handler to a feed cursor makes the cursor behave weird. Example:
r.table('test').changes().then((cursor) => {
  cursor.once('error', (err) => { console.log(err) });
  cursor.eachAsync(item => console.log(item));
  r.table('test').insert({ num: 1 }).then(()=>{});
  r.table('test').insert({ num: 2 }).then(()=>{});
  r.table('test').insert({ num: 3 }).then(()=>{});
  r.table('test').insert({ num: 4 }).then(()=>{});
});

The above will print out to the console new_val changes with items 2 and 4.
Removing the line cursor.once(...) fixes the problem.
No error, no nothing.

OK, I found the culprit. It seems the problem is somewhere here: (cursor.js).
Once I've commented out this part:
        setImmediate(function() {
          self._each(self._eachCb.bind(self), function() {
            if (self._emittedEnd === false) {
              self._emittedEnd = true;
              self._eventEmitter.emit('end');
            }
          });

Everything works fine.

Your script is throwing:
Unhandled rejection ReqlDriverError: You cannot call `eachAsync` once you have bound listeners on the Feed.
    at Cursor.eachAsync (/home/michel/projects/rethinkdb-all/rethinkdbdash/lib/cursor.js:308:11)
    at /home/michel/projects/rethinkdb-all/rethinkdbdash/test.js:4:10
    at tryCatcher (/home/michel/projects/rethinkdb-all/rethinkdbdash/node_modules/bluebird/js/release/util.js:11:23)
    at Promise._settlePromiseFromHandler (/home/michel/projects/rethinkdb-all/rethinkdbdash/node_modules/bluebird/js/release/promise.js:488:31)
    at Promise._settlePromise (/home/michel/projects/rethinkdb-all/rethinkdbdash/node_modules/bluebird/js/release/promise.js:545:18)
    at Promise._settlePromise0 (/home/michel/projects/rethinkdb-all/rethinkdbdash/node_modules/bluebird/js/release/promise.js:590:10)
    at Promise._settlePromises (/home/michel/projects/rethinkdb-all/rethinkdbdash/node_modules/bluebird/js/release/promise.js:673:18)
    at Async._drainQueue (/home/michel/projects/rethinkdb-all/rethinkdbdash/node_modules/bluebird/js/release/async.js:125:16)
    at Async._drainQueues (/home/michel/projects/rethinkdb-all/rethinkdbdash/node_modules/bluebird/js/release/async.js:135:10)
    at Immediate.Async.drainQueues [as _onImmediate] (/home/michel/projects/rethinkdb-all/rethinkdbdash/node_modules/bluebird/js/release/async.js:16:14)
    at processImmediate [as _immediateCallback] (timers.js:383:17)

This is working as intended. You cannot mix both event emitter and next/each/etc.
Simply because if you do on('data', console.log), the cursor will fetch data as fast as possible to emit the results. In this case we don't want to keep the results in a buffer in case you would like to call next/each.
If I'm not mistaken, the official driver behaves the same way.

Well I made a mistake with the example, the order of 'once' and eachAsync was incorrect.
r.table('test').changes().then((cursor) => {
  cursor.eachAsync(item => console.log(item));
  cursor.once('error', (err) => { console.log(err) });
  r.table('test').insert({ num: 1 }).then(()=>{});
  r.table('test').insert({ num: 2 }).then(()=>{});
  r.table('test').insert({ num: 3 }).then(()=>{});
  r.table('test').insert({ num: 4 }).then(()=>{});
});

This code should execute to prove what I meant. Even if what you say is true, in that you can't have both an emitter and use eachAsync or each, there should be an error thrown when trying to use an emitter after one of the each or other commands were used. Otherwise unexpected behavior happens as I described above (loosing every 2nd item) and the developer is left to get to the bottom of this problem on his own.

Hum, you are supposed to bind the listener for errors before calling each/eachAsync. if you don't you could emit an error before the listener is bound which eventually crash your process.
We could add some safety though. You're welcome to send a pull request.


Questions for clean- and poison-data in poison_data directory

Hello again,
I have a few questions about the data under the ./poison_data/ directory and hope you could kindly assist:

Would you please help me understand the columns in files such as train-clean.csv, test-poison.csv? The column names in those files are marked as 0, 1 and I wasn't able to find what they represent, some clarification would be appreciated.

I'm trying to use my own poison data by uploading my train-/dev-/test-clean, and train-/dev-/test-poison data, and by setting load: true in the config file. However, I'm a bit confused by how the test-eval.csv and test-detect.csv were created in the 'eval' and 'detect' modes. would you please explain?

Additionally, I thought the test-poison.csv file contained only poison test data, but then the test-poison.csv file seems to change and becomes larger during the experiment, even if I set load:true. What changes may have been made to this file in the process?

Thank you very much for your timely response!
Best,
Wencong

Hello,
I've figured it out, lol...
Closing the ticket now.
Best,
Wencong


[stdlib] using static method dispatch instead of failable casts

Overriding AnySequence.dropFirst and AnySequence.prefix to delegate
these operations to an underlying sequence, thus simplifying default
implementations in Sequence.

@gribozavr please have a look.

Updated.

The change LGTM, but because of the signature change in AnySequence.init(_:), this requires writing a proposal.

Will do. Thanks for reviewing it.

Rebased and ready to go.
Removing the 'swift-evolution-pinding-discussion' label as it has recently been accepted here.

Needs splitting.


layout issue: Mobile view need to be fixed

Layout of profile page is needed to be fixed

@Satvik1769 can you express what changes you are thinking of ? with ss

this!!!

Can you also work on each page and make every page responsive according to the resolution??

other pages don't have issue this page is having a bit of issue. It can be done . I was thinking to let others take this issue as it is a great good first issue.


Presistence Scrroll Save

As the extension is using LocalStorage API of Browser If a user reset all settings or uninstall a browser most likely all the saved scrolls and pages will be lost. So there will be no way he can get back his data back.

I would like to propose the this as Solution

Let user Import and Export all the saved scrolls to as a JSON file which he can later import to another browser he can import and have all the scrolls which he added in past.

An alternative would be using some kind of authentication and save the scrolls in Database once the same user logs in he can have all the Scrolls back in the Browser.

I like the import or export idea for now, we can create a database in future.

I would love to work on this feature.
Can you assign this ito me ?

Hi, @ashutosh4336 are you still working on this? I would love to help in some part of this enhancement.