How to give Prompt to trained RETRO Model?

I am following the instructions on the RETRO-pytorch GItHub repo. After training my model, how do I go about using it to generate responses?
retro = RETRO(
    chunk_size = 64,                         # the chunk size that is indexed and retrieved (needed for proper relative positions as well as causal chunked cross attention)
    max_seq_len = 2048,                      # max sequence length
    enc_dim = 896,                           # encoder model dim
    enc_depth = 2,                           # encoder depth
    dec_dim = 796,                           # decoder model dim
    dec_depth = 12,                          # decoder depth
    dec_cross_attn_layers = (3, 6, 9, 12),   # decoder cross attention layers (with causal chunk cross attention)
    heads = 8,                               # attention heads
    dim_head = 64,                           # dimension per head
    dec_attn_dropout = 0.25,                 # decoder attention dropout
    dec_ff_dropout = 0.25,                   # decoder feedforward dropout
    use_deepnet = True                       # turn on post-normalization with DeepNet residual scaling and initialization, for scaling to 1000 layers
)

seq = torch.randint(0, 20000, (2, 2048 + 1))      # plus one since it is split into input and labels for training
retrieved = torch.randint(0, 20000, (2, 32, 2, 128)) # retrieved tokens - (batch, num chunks, num retrieved neighbors, retrieved chunk with continuation)

loss = retro(seq, retrieved, return_loss = True)
loss.backward()

wrapper = TrainingWrapper(
    retro = retro,                                 # path to retro instance
    knn = 2,                                       # knn (2 in paper was sufficient)
    chunk_size = 64,                               # chunk size (64 in paper)
    documents_path = './retro_training_set/',              # path to folder of text
    glob = '**/*.txt',                             # text glob
    chunks_memmap_path = './train.chunks.dat',     # path to chunks
    seqs_memmap_path = './train.seq.dat',          # path to sequence data
    doc_ids_memmap_path = './train.doc_ids.dat',   # path to document ids per chunk (used for filtering neighbors belonging to same document)
    max_chunks = 1_000_000,                        # maximum cap to chunks
    max_seqs = 100_000,                            # maximum seqs
    knn_extra_neighbors = 100,                     # num extra neighbors to fetch
    max_index_memory_usage = '100m',
    current_memory_available = '1G'
)

Now when I want to give this model a text input (any prompt), how would I go about doing that? Which method or function would I use? Which model/tokenizer should I use to encode the input prompt and then decode the model output tensor? Is there a method for that?
Example Prompt:
"The movie Dune was released in"

https://github.com/lucidrains/RETRO-pytorch/issues/23 contains a notebook with a good example.
I think putting it together with the README instructions looks like this:
import torch
from retro_pytorch import RETRO, TrainingWrapper

# instantiate RETRO, fit it into the TrainingWrapper with correct settings

retro = RETRO(
    max_seq_len = 2048,                      # max sequence length
    enc_dim = 896,                           # encoder model dimension
    enc_depth = 3,                           # encoder depth
    dec_dim = 768,                           # decoder model dimensions
    dec_depth = 12,                          # decoder depth
    dec_cross_attn_layers = (1, 3, 6, 9),    # decoder cross attention layers (with causal chunk cross attention)
    heads = 8,                               # attention heads
    dim_head = 64,                           # dimension per head
    dec_attn_dropout = 0.25,                 # decoder attention dropout
    dec_ff_dropout = 0.25                    # decoder feedforward dropout
).cuda()

wrapper = TrainingWrapper(
    retro = retro,                                 # path to retro instance
    knn = 2,                                       # knn (2 in paper was sufficient)
    chunk_size = 64,                               # chunk size (64 in paper)
    documents_path = './text_folder',              # path to folder of text
    glob = '**/*.txt',                             # text glob
    chunks_memmap_path = './train.chunks.dat',     # path to chunks
    seqs_memmap_path = './train.seq.dat',          # path to sequence data
    doc_ids_memmap_path = './train.doc_ids.dat',   # path to document ids per chunk (used for filtering neighbors belonging to same document)
    max_chunks = 1_000_000,                        # maximum cap to chunks
    max_seqs = 100_000,                            # maximum seqs
    knn_extra_neighbors = 100,                     # num extra neighbors to fetch
    max_index_memory_usage = '100m',
    current_memory_available = '1G'
)

# get the dataloader and optimizer (AdamW with all the correct settings)

train_dl = iter(wrapper.get_dataloader(batch_size = 2, shuffle = True))
optim = wrapper.get_optimizer(lr = 3e-4, wd = 0.01)

# now do your training
# ex. one gradient step

seq, retrieved = map(lambda t: t.cuda(), next(train_dl))

# seq       - (2, 2049)         - 1 extra token since split by seq[:, :-1], seq[:, 1:]
# retrieved - (2, 32, 2, 128)   - 128 since chunk + continuation, each 64 tokens

loss = retro(
    seq,
    retrieved,
    return_loss = True
)

# one gradient step

loss.backward()
optim.step()
optim.zero_grad()

# do above for many steps, then ...

# encode prompt
from transformers import AutoTokenizer

tokenizer = AutoTokenizer.from_pretrained("bert-base-cased")

prompt_str = "The movie Dune was released in"

prompt_ids = tokenizer(prompt_str)['input_ids'][1:-1]

prompt = torch.tensor([prompt_ids])

sampled = wrapper.generate(prompt, filter_thres = 0.9, temperature = 1.0)

# decode sample
decoded = tokenizer.decode(sampled.tolist()[0])

print(decoded)

#23 contains a notebook with a good example.
I think putting it together with the README instructions looks like this:
import torch
from retro_pytorch import RETRO, TrainingWrapper

# instantiate RETRO, fit it into the TrainingWrapper with correct settings

retro = RETRO(
    max_seq_len = 2048,                      # max sequence length
    enc_dim = 896,                           # encoder model dimension
    enc_depth = 3,                           # encoder depth
    dec_dim = 768,                           # decoder model dimensions
    dec_depth = 12,                          # decoder depth
    dec_cross_attn_layers = (1, 3, 6, 9),    # decoder cross attention layers (with causal chunk cross attention)
    heads = 8,                               # attention heads
    dim_head = 64,                           # dimension per head
    dec_attn_dropout = 0.25,                 # decoder attention dropout
    dec_ff_dropout = 0.25                    # decoder feedforward dropout
).cuda()

wrapper = TrainingWrapper(
    retro = retro,                                 # path to retro instance
    knn = 2,                                       # knn (2 in paper was sufficient)
    chunk_size = 64,                               # chunk size (64 in paper)
    documents_path = './text_folder',              # path to folder of text
    glob = '**/*.txt',                             # text glob
    chunks_memmap_path = './train.chunks.dat',     # path to chunks
    seqs_memmap_path = './train.seq.dat',          # path to sequence data
    doc_ids_memmap_path = './train.doc_ids.dat',   # path to document ids per chunk (used for filtering neighbors belonging to same document)
    max_chunks = 1_000_000,                        # maximum cap to chunks
    max_seqs = 100_000,                            # maximum seqs
    knn_extra_neighbors = 100,                     # num extra neighbors to fetch
    max_index_memory_usage = '100m',
    current_memory_available = '1G'
)

# get the dataloader and optimizer (AdamW with all the correct settings)

train_dl = iter(wrapper.get_dataloader(batch_size = 2, shuffle = True))
optim = wrapper.get_optimizer(lr = 3e-4, wd = 0.01)

# now do your training
# ex. one gradient step

seq, retrieved = map(lambda t: t.cuda(), next(train_dl))

# seq       - (2, 2049)         - 1 extra token since split by seq[:, :-1], seq[:, 1:]
# retrieved - (2, 32, 2, 128)   - 128 since chunk + continuation, each 64 tokens

loss = retro(
    seq,
    retrieved,
    return_loss = True
)

# one gradient step

loss.backward()
optim.step()
optim.zero_grad()

# do above for many steps, then ...

# encode prompt
from transformers import AutoTokenizer

tokenizer = AutoTokenizer.from_pretrained("bert-base-cased")

prompt_str = "The movie Dune was released in"

prompt_ids = tokenizer(prompt_str)['input_ids'][1:-1]

prompt = torch.tensor([prompt_ids])

sampled = wrapper.generate(prompt, filter_thres = 0.9, temperature = 1.0)

# decode sample
decoded = tokenizer.decode(sampled.tolist()[0])

print(decoded)

The code in the notebook for training several times is probably needed for good results though.

@filipesilva Can you please share notebook which you are referencing, its not accessible. or if you can share code for training multiple epochs, will be really very helpful. Thanks

@aakashgoel12 looks like the notebook that was in #23 is not there anymore. I don't have a copy of it, unfortunately. All the code I have is what I put in the comment.

@aakashgoel12 looks like the notebook that was in #23 is not there anymore. I don't have a copy of it, unfortunately. All the code I have is what I put in the comment.

Thanks @filipesilva. Can you please check if what I have written below is correct or need some modification. Thanks in advance.
num_epochs=3
train_dl = wrapper.get_dataloader(batch_size = 4, shuffle = True)
for epoch in range(num_epochs):
counter=0
for batch in tqdm(train_dl):
seq, retrieved = map(lambda t: t.cuda(), batch)
loss = retro(
seq,
retrieved,
return_loss = True)
# one gradient step
loss.backward()
optim.step()
optim.zero_grad()
if counter%10==0:
print("Epoch:{}, BatchNo:{}, Loss:{}".format(epoch, counter, loss))
counter+=1
print("After epoch - {}, loss: {}".format(epoch,loss))

I really can't tell 😅 I only played around with this a couple of months ago and never really tried again.

hello
documents_path = './text_folder',              # path to folder of text
glob = '**/*.txt',                             # text glob
chunks_memmap_path = './train.chunks.dat',     # path to chunks
seqs_memmap_path = './train.seq.dat',          # path to sequence data
doc_ids_memmap_path = './train.doc_ids.dat',
Is this in the path retro? Or what dataset is it?


Resources list table width is out of card (UI)

Make a scroll zone in the card to be not limited by the width of elements in the table.

Done and merged in dev branch


WindowCommands Control missing from NuGet Package

Hello,
I've notice that samples and other code that reference the WindowCommands tool, which I assume adds the close, maximize, and minimize buttons to the form. I believe this is present in the latest Master branch on GitHub, but not in the NuGet package? Is this a known issue, or am I missing more here?
Thanks,
Patrick

I just noticed that WindowCommands was added to the control toolkit after the latest version was pushed to NuGet. I'm going to go ahead and close this issue and pull from the current build.
My apologies.

Yeah we're a little behind on getting a new release out. If you don't need UIAutomation, I'd recommend the second for the head at the moment (https://github.com/MahApps/MahApps.Metro/commit/4396c096ff96a96437f2e94f662b3b8bfcf13748) so that you get combobox styling


Amortized bo implementation

Implements amortized BO. Note: checkout the last commit: I've removed a couple of imports that get annoying when running the solver in a different environment.

@miguelgondu: How should we handle solvers that require a different environment? Amortized BO requires tensorflow for example.


use with different geographic regions

how would i use this gem with different geographic zones that have different holidays? is there a way that is already built in?

@Goltergaul, we just released a gem called biz that allows multiple schedule configurations, which should fulfill your use case. Check it out and feel free to shoot me an email or open an issue if you have any questions.


Add MDX compiler option overrides

Issue: #11
What Changed
This is an alternative to #12. Rather than provide a single option, provide an object mdxCompileOptions that gets directly to the MDX compiler.
In addition:

[x] Update docs
[x] Update example to test/demo

How to test
yarn storybook

Change Type

[ ] maintenance
[ ] documentation
[ ] patch
[ ] minor
[ ] major

📦 Published PR as canary version: 0.0.4-canary.27.b47ca0a.0

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

Self-merging @ndelangen


New variable for submissionMethodDetails

New variable for submissionMethodDetails
This variable can be specified as a command line argument, e.g. -v submissionMethodDetails:"quick". Currently supported values are listed here. If the variable is not set, the submissionMethodDetails field of the initial tender data falls back to its default value – quick.
Examples
belowThreshold with no-auction
bin/op_tests -s openProcedure -A robot_tests_arguments/single_item_tender.txt -v submissionMethodDetails:"quick(mode:no-auction)"

belowThreshold with fast-forward
bin/op_tests -s openProcedure -A robot_tests_arguments/single_item_tender.txt -v submissionMethodDetails:"quick(mode:fast-forward)"

aboveThresholdUA
bin/op_tests -s openProcedure -A robot_tests_arguments/openua_testing.txt -v submissionMethodDetails:"quick(mode:fast-forward)"

aboveThresholdEU
bin/op_tests -s openProcedure -A robot_tests_arguments/openeu_testing.txt -v submissionMethodDetails:"quick(mode:fast-forward)"

competitiveDialogueUA
bin/op_tests -s openProcedure -A robot_tests_arguments/competitive_dialogue_simple.txt -v DIALOGUE_TYPE:UA -v submissionMethodDetails:"quick(mode:no-auction)"

competitiveDialogueEU
bin/op_tests -s openProcedure -A robot_tests_arguments/competitive_dialogue_simple.txt -v submissionMethodDetails:"quick(mode:no-auction)"

I did not run the whole suite to check whether the auction was actually skipped or ran in the fast-forward mode, but I think that having the proper values in the submissionMethodDetails field is sufficient.
The test_tender_data_limited function was not modified because it ignores submissionMethodDetails.
+@kosaniak, +@lesiavl, please review.

This change is

Review status: 0 of 3 files reviewed at latest revision, 1 unresolved discussion.

op_robot_tests/tests_files/initial_data.py, line 143 at r1 (raw file):

def test_tender_data_limited(params):
    data = test_tender_data(params)

тут не потрібно другий параметр задавати при виклику  test_tender_data ?

Comments from Reviewable

Reviewed 2 of 2 files at r1, 3 of 3 files at r2.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

op_robot_tests/tests_files/initial_data.py, line 143 at r1 (raw file):
Previously, kosaniak wrote…
тут не потрібно другий параметр задавати при виклику  test_tender_data ?

 :)

Comments from Reviewable

я не уважна :(

Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

op_robot_tests/tests_files/initial_data.py, line 143 at r1 (raw file):
Previously, selurvedu wrote…
 :)

Done.

Comments from Reviewable


link in project description should be http

Thanks, I was trying to get an SSL certificate setup but it's not working the way I thought it would.

Oh, wow... github added the CNAME file when I set it up in the settings.


CustomizerDefault.options ElementAdder_Tier for Endgame utility + life&mana flasks

Split "other" section in two to support ElementAdder_Tier for both Utility and Life&Mana. General standardization & cleanup.

As per ReadMe:
FILTER-QUERY-ID: Unique name that identifies this query. Will be sometimes referred as 'filter-rule-name' Having the same name for different rules is not allowed. Changing the ID of a rule after initial release is also not recommended. This value is used for the save&load system and therefore very critical.
Even simple changes like "High Level Life/Mana Flasks - iLvl 82" -> "High Level Life & Mana Flasks - iLvl 82" are enough to cause problems here. Because this string is the main method to find the right filter rule to edit for the saveLoad system. Meaning if you change this name here, all existing changes to these rules will be invalidated. You will have to use the Title parameter to change displayNames here:
https://github.com/NeverSinkDev/FilterBlade-Public-Assets/blob/main/README_OptionFile.md#-title
I'll update the ReadMe with this info.
Then in line 1305 you wrote $sdh instead of $shd.
While the actual order of the "SHD" string value does not matter, $shd here is a variable, meaning you need to be exact when accessing this variable.
If you're feeling fancy you can also add some default appearance lines to the base conditional rule. Such as size or color. But since those are just simple flasks, we can live without any here.
Otherwise a very solid first attempt at working with option files :P

Updated with fixes to the above.
I couldn't think of any default styling for flasks that would add value.

Nice!
The other function also has the $shd typo, and @NeverSinkDev do we need rarity and arealevel filtering by default in these new custom rules? I think it's gonna catch uniques and maybe leveling if we dont, right?

Doh. Fixed the typo and add default Rarity Normal Magic. None of the other flask rules have AreaLevel set, is that just for custom rules?

Looks good to me, thanks!


Added explicit call to filesystem mount()

Added an explicit call to fs->mount() in the init() function.
FileSystem does not require a block device to be passed to it:
https://github.com/ARMmbed/mbed-os-example-filesystem/blob/master/main.cpp#L39
If you do not pass a block device to the constructor, it will not automatically do a mount. By calling mount() explicitly, we support constructors without this block device.
Tested with LittleFileSystem and FATFileSystem.
Without this fix, if you do not pass a block device to your file system constructor, the FCC looks for an unmounted file system, thus your credentials will be written over and you will receive a new device identity.

@MarceloSalazar @janjongboom Can you please review?

Thanks for the fix!
@bridadan you may want to review

Its been tested with both constructors:
FATFileSystem("name", &blockdevice)
FATFileSystem("name")
Thus works even if mount is called twice. That said, It's probably worth looking further into any consequences. Here is the line that would be called twice: https://github.com/ARMmbed/mbed-os/blob/a3d8c8fe161e27a99876ab9db1bd1adf6a20e49a/features/filesystem/fat/FATFileSystem.cpp#L290
Let me confirm there are no implications.

Fixed based on @bridadan suggestion to handle this in the error checking.
Checks if fcc_status is FCC_STATUS_KCM_STORAGE_ERROR then attempts to mount storage and retries

I see we're duplicating the logic, which may solve this problem but adds complexity for maintenance.
Is there anything we could do to simplify this?

Can move this line out of the ifdef-
fcc_status = fcc_verify_device_configured_4mbed_cloud();
This was also duplicated there.. but it doesn't make sense to duplicate it. Thus we can change the mount logic also to be outside of the ifdef.


Merge pull request #1 from pnp/dev

Catch up.
Before creating a pull request, make sure that you have read the contribution file located at
https://github.com/pnp/powerShell/blob/dev/CONTRIBUTING.md
Type

[ ] Bug Fix
[ ] New Feature
[ ] Sample

Related Issues?
Fixes #X, partially fixes #Y, mentioned in #Z, etc.
What is in this Pull Request ?
Please describe the changes in the PR.
Guidance

You can delete this section when you are submitting the pull request.*
Please update this PR information accordingly. We use this as part of our release notes in monthly communications.
Please target your PR to Dev branch. If you do not target the Dev branch we will not accept this PR.

Catch up.

Hey @ToddKlindt - is this a work in progress ? Was it created by mistake, I don't see any files changed :)  ?

Closing PR as no files in PR

Boo. I PRed in the wrong direction. I was trying to get my local repo back into sync. Nothing to see here.  Just an IT Pro screwing up GitHub. :)


Fix error in NumPy fancy indexing

NumPy arrays of integers have an __index__ method -- it's meant to apply to scalars, and it raises an error if you actually try to run it on an array.
This ruined most cases where NumPy fancy indexing was supposed to work, because it ended up in the case for things with an __index__ instead of things that are iterable.
If we move the is_iterable check first, we won't encounter this error.

Codecov Report

Merging #51 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #51   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines         144    144
  Branches       32     32
=====================================
  Hits          144    144


I am trying to deploy the Python web app quickstart to App Service and this is what I got

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az webapp up
Errors:
Expecting value: line 1 column 1 (char 0)
Traceback (most recent call last):
python3.7/site-packages/knack/cli.py, ln 206, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 603, in execute
    raise ex
cli/core/commands/__init__.py, ln 661, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
cli/core/commands/__init__.py, ln 652, in _run_job
    cmd_copy.exception_handler(ex)
...
lib/python3.7/json/__init__.py, ln 348, in loads
    return _default_decoder.decode(s)
lib/python3.7/json/decoder.py, ln 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
lib/python3.7/json/decoder.py, ln 355, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az webapp up -n {}

Expected Behavior
Environment Summary
Darwin-19.0.0-x86_64-i386-64bit
Python 3.7.4
Shell: bash

azure-cli 2.0.75

Additional Context

cc @AzureAppServiceCLI @antcp

Please use version 2.0.77 also please make sure you run this command from the context of where the directory where the code is.


[Don't Review]

Used as reference to discuss intervaljoin performance improvements

Closing the PR; since it is just a reference it doesn't need to be open.


Added hide of search suggestions when suggestion clicked  [#94690442]

This hides the search suggestions when an suggested item is clicked, when the enter key is pressed and when any element outside of the search suggestions other than the search input is clicked.
My edit replaced the tab on line 364.

Thanks! The cucumber tests will fail because of some timing issues we haven't resolved yet.


"Convert URL to PDF" button prevents dismiss "cookie policy" on webpages & appears on final PDF conversion blocking content

Summary
The red "Convert URL to PDF" banner/button covers cookie policy pop-ups on websites, preventing their dismissal and causing them to appear in the final PDF output, often blocking content.
Description
Issue

The red "Convert URL to PDF" banner/button covers the cookie policy pop-up on websites, preventing dismissal. This results in the cookie policy being visible in the converted PDF, often cutting off or blocking content at the bottom of the page.

TL;DR

The red "Convert URL to PDF" button blocks the dismissal area of cookie policy pop-ups, making them visible in the PDF and cutting off content behind them.

Suggested Solution

Provide the ability to temporarily hide the red "Convert URL to PDF" button to allow for the dismissal of cookie policies before conversion.

Thank you, I will try to fix them as soon as possible.


Update License

Review current license for the project and update as necessary.

@joeywwwu @hchqin @juliaeveritt I realized that we never actually ended up discussing whether we needed to update/chnage our license or not. Personally, I do think that the MIT license suits our needs since it's relatively flexible and favours permissibility, but I also don't have a strong preference (say between MIT vs GNU or other licenses). What do you guys think?

@monazhu Thanks for bringing this up! I don't have a strong preference either. I think MIT license work well in our project, could be a good choice as our license.

Agreed! MIT seems like a good fit to me too. Thanks for remembering this!

Thanks! I think the MIT license would be a better fit here as well

Okay sounds good! Then we'll stick with the MIT license we currently have :)


Multiple order on query

Summary
Collections can be sorted by multiple fields but with only one option for order.
There isn't a option to order multiple fields in a different way.
Basic example
query {
  allStoreItens(
    sort: [{ by: "date" }, { by: "title" }]
    order: { "date": "DESC" }, { "title": "ASC"}
   ) {
    ...
  }
}

Motivation
With this option, we be able to order all items by more recent date and the items that have same date will sorted alphabetically descending.

You can set an order with the sort argument like this:
query {
  allStoreItens(sort: [{ by: "date", order: DESC }, { by: "title", order: ASC }]) {
    ...
  }
}

We have probably forgotten to mention it in the docs... :)

Thanks @hjvedvik!
I send a PR to fix that.


SharePoint list result size is limited to 500 items when using the API in PowerApps.

SharePoint Lists have a maximum return size of 500 results using the API, this means I only see 500 Teams and so can only select Teams listed in the first 500 results. Generally, schools who need this PowerApp have large amounts of Teams so the 500 limit isn'y usable.
Is there a way to use a Flow to return the list of Teams from the SharePoint list? That would get around the 500 limit.

We are having the same issue. We have around 5000 Teams as we are looking to use this app for a group schools. During the presentation it said it would be able to accept 100,00 Teams. Each school has around 700 Teams

We are having the same issue. We have around 5000 Teams as we are looking to use this app for a group schools. During the presentation it said it would be able to accept 100,00 Teams. Each school has around 700 Teams

I'm hoping once they add the SDS documentation for this product it will solve the limitation issue.

I'm hoping once they add the SDS documentation for this product it will solve the limitation issue.

Hi guys,
You can up the limit to 2000 within the Powerapps app.  If you go into the Classroom Drop-in Powerapp and settings, you can set the limit there, from 500-2000.  Hope this helps.
Joe.


/camera/imu is black

I followed the GitHub link for the experience the real-sense camera d435i : https://github.com/IntelRealSense/realsense-ros [ros-development]
I am able to read the linear acceleration and angular velocity form /camera/accel/sample and /camera/gyro/sample but i want that to publish in one topic
i already use the unite_imu_method: linear-acceleration and copy parameter too but it is not publishing any thing.
Please reply soon..

Hi @shivangi-futuristicbots  If you are using the ROS2 ros2-development wrapper then the unite_imu_method instruction is defined in a different way.  Instead of using unite_imu_method:=linear_interpolation, the linear_interpolation setting is simply defined with the value '2'.
unite_imu_method:=2

Thank for your support @MartyG-RealSense
I have one queries is real-sense camera would be preferred for the AVG and IMU reading of the Real-sense camera and imu reading of other imu sensors [lord microstraine], which is better?

If you are asking whether an external high quality, high cost IMU sensor will provide better results than the IMU that is built into the RealSense camera, the answer is likely yes.

Hi @shivangi-futuristicbots  Do you require further assistance with this case, please?  Thanks!

Not Yet , may be in future Thanks @MartyG-RealSense  for the help

Okay, thanks very much @shivangi-futuristicbots for the update.  I will close this issue as you do not require further assistance.  Plese feel free to re-open it at a future date or create a new issue if you need to do so.  Thanks again!


Postgres 1.2.0-build00028 exception when appending to stream

Was testing the latest version of Postgres (1.2.0-build00028) but I am getting a "Object reference not set" when appendToStream. Reading from stream did not give an exception.

Using this store initializer:
using (var store = new PostgresStreamStore(new PostgresStreamStoreSettings(Settings.Default.ESConnectionString) { Schema = "EventStore".ToLower() }))
Also tried with no schema provided to the PostgresStreamStoreSettings. (I also ran the await store.CreateSchema() again.
Also downgraded both packages for SqlStreamStore and SqlStreamStore.Postgres to version 16 => same error.
using docker  image 10.5 of postgres.
I have a public schema and a eventstore schema in postgres db

Ok after debugging the source I saw that the _settings.GetUtcNow() is mandatory...
Now it is working properly.

I check the guards, it default to SystemClock.Default but if you set explicitly set it to null, it will throw internally.

I used this initializer: using (var store = new PostgresStreamStore(new PostgresStreamStoreSettings(Settings.Default.ESConnectionString) { Schema = "EventStore".ToLower() }))


Rc/#762 fixes file upload hanging on UI

https://github.com/h2oai/h2ogpt/issues/762

Manually tested putting multiple things in file upload and url and paste and all seem to work correctly with this minor change


No available formula with the name "titouanvanbelle/xctesthtmlreport/xchtmlreport"

@chrisballinger I can't seem to be able to install xchtmlreport through brew. Any idea what's happening there?
~[19:12:09]
$ brew install TitouanVanBelle/XCTestHTMLReport/xchtmlreport
==> Tapping titouanvanbelle/xctesthtmlreport
Cloning into '/usr/local/Homebrew/Library/Taps/titouanvanbelle/homebrew-xctesthtmlreport'...
warning: You appear to have cloned an empty repository.
Tapped 0 formulae (3 files, 1.9KB)
Error: No available formula with the name "titouanvanbelle/xctesthtmlreport/xchtmlreport"
==> Searching for a previously deleted formula (in the last month)...
Error: No previously deleted formula found.
==> Searching for similarly named formulae...
==> Searching local taps...
Error: No similarly named formulae found.

~[19:12:25]
$ brew tap
fatal: ambiguous argument 'refs/remotes/origin/master': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:
'git <command> [<revision>...] -- [<file>...]'
homebrew/cask
homebrew/cask-versions
homebrew/core
titouanvanbelle/xctesthtmlreport

~[19:12:39]
$ brew untap titouanvanbelle/xctesthtmlreport
Untapping titouanvanbelle/xctesthtmlreport... (5 files, 2.9KB)
Untapped 0 formulae

~[19:12:44]
$ brew tap
homebrew/cask
homebrew/cask-versions
homebrew/core

Same for me. Can't install this tool using Homebrew

I am having this problem as well. Are there any plans for a fix? Better, are there any plans to include the formula in Homebrew directly?

Homebrew won't merge the formula unless this repo has more than 50 stars
On Fri, Jun 22, 2018 at 2:50 PM Keith Bertelsen<EMAIL_ADDRESS>wrote:

I am having this problem as well. Are there any plans for a fix? Better,
are there any plans to include the formula in Homebrew directly?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/TitouanVanBelle/XCTestHTMLReport/issues/86#issuecomment-399592186,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAfqH1ZMFShYJ6GPmUl0YWKPK0RgOYKLks5t_Wa3gaJpZM4UVa3M
.

50 stars? That's doable. Already integrated this tool on my project and the results is really satisfying! Gave my star to it.

We got them now :)

@chrisballinger Any update on this?

The pull request in question is here: https://github.com/Homebrew/homebrew-core/pull/26977
It was rejected because the notoriety requirements are:

20+ forks
20+ watchers
50+ stars

I don't suppose there's a way we could at least create a tap for this, until it meets the notoriety requirements? That would make it much easier to use with Brewfile.

I've been trying to spread the word around as best I can. I'll keep trying, though! :)

Repo now meet the requirements with 20 watchers. Can we re-open PR?

As suggested by #182 I created a Brew tap repository and brew one-liner installation now works again!. README has been updated to reflect the change.
BTW, if you use Mint it is even easier...
I am closing this issue since I think it is already fixed. If you have problems reopen please :)

Mint
Mint is the easiest way...
Install latest stable version
mint install XCTestHTMLReport/XCTestHTMLReport

Install latest from develop branch
mint install XCTestHTMLReport/XCTestHTMLReport@develop

Homebrew
Install latest stable version
brew install XCTestHtmlReport/xchtmlreport/xchtmlreport

Install latest from develop branch
$ wget https://raw.githubusercontent.com/XCTestHTMLReport/XCTestHTMLReport/develop/xchtmlreport.rb
$ brew install --HEAD --build-from-source xchtmlreport.rb


Performance: Prisma generate freezes with large schema with dbml generator

there seems to be another issue that isn't caught by the tests, because when generating in a reasonably sized project it's frozen, when removing the dbml generator provider it does generate correctly
Originally posted by @PabloSzx in https://github.com/notiz-dev/prisma-dbml-generator/issues/37#issuecomment-1174157553

@PabloSzx let's track the performance issue here. We should setup a large dummy/test schema to find out the bottleneck of this generator. How large (lines of code) is the prisma.schema where it frooze?

Is ~650 lines

Think I have another instance of this, my schema is ~400 lines long, with many enums peppered all over which I think is affecting graph complexity.

@yourbuddyconner is it freezing or just taking for a long time to generate?
Are thus enums with a few or a lot of entries?
I guess will need to create a test schema with lots of entries, relationships and enums to test it out.

I feel like it was hanging, I profiled the process and it wasn't using a considerable amount of CPU or memory, so my conclusion was a weird hang of some kind but that wasn't based on a deep dive or anything.
I am using a schema generated by Prisma, which includes an average of ~1 enum across 20+ models with a web of relations across them. The enums have <10 options, averaging 2-4, and idk if that affects performance at all.
Anecdotally, I have been having difficulty generating ERDs with this schema across a variety of tools, with the only reliable one being PGAdmin.


Load/Capture specific timestamps of video

Be sure to search for your issue before opening a new one.
Current Behavior

Expected Behavior
I've got a video in a folder which is not being played, but I want to load frames from specific timestamps/frame intervals (every 30 seconds or every 200th frame). I want to use this these frames as thumbnails.
Steps to Reproduce
How should I go about doing it?
Environment

URL attempting to play: mp4
Browser: Safari
Operating system: MacOS Mojave 10.14
jsFiddle example: https://jsfiddle.net/sv5x3ug1

Other Information

This would require some kind of static file processing. Way beyond the scope of this project.
This might get you started: https://stackoverflow.com/a/29806483/1250405


Convert mapview to kotlin (2)

Migrate MapView to kotlin
In order to use all of the v10 api, we need to convert some part of the java codebase to kotlin. This PR is aimed at converting the MapView class to kotlin (using the Convert Java File to Kotlin File tool from Android Studio).
I had to convert a bunch of other class to kotlin (Callout, MarkerView, PointAnnotation, Image, NativeUserLocation, Light, AbstractSouceConsumer, Source and LocationManager) to avoid unresolved supertypes error.

This looks great, thanks for all the effort!!
I made a few comments, the most important of which is just to match the new JS API.


Integrate in the iCub Telemetry Server the network ping command via RPC port

We shall integrate in the iCub Telemetry Server the network ping command to be triggered via RPC port.

[x] Create the child process using the spawn() method from the Node.JS child_process module (Display the returned data and errors on the terminal). => https://github.com/dic-iit/yarp-openmct/issues/13#issuecomment-909603639
[x] Create an RPC interface for triggering the ping command (the intent is to enable the ON/OFF toggling of the ping through a button on the web client control console). => https://github.com/dic-iit/yarp-openmct/issues/13#issuecomment-909613782 and https://github.com/ami-iit/yarp-openmct/issues/13#issuecomment-912637378
[x] Generate asynchronously the Telemetry samples from the returned data (round-trip ping delay in milliseconds), ready to be sent to the OpenMCT visualizer at request. => https://github.com/ami-iit/yarp-openmct/issues/13#issuecomment-912645059

Thanks!


Sync with upstream

Sync with upstream, make Run take an *analysis.Pass argument instead. This is necessary due to upstream changes.
I've tested golangci-lint with this version of unconvert, and AFAICT it works same as before. However, make test_linters in the golangci-lint repo fails, so apparently something has changed.

Hello,
I have done the sync, with a minimal changes approach, you and I did globally the same things.
Thank you anyway.


Add a work around for spread

I'm sorry, I created this PR by mistake.
We wanted to use this change for compiling dftatom through LFortran.


Dialogflow cx

Hi,
Thank you for the module and the gateway, i have successfully communicate with diagflow agent (using dialogflow essential).
But when i try using dialogflow cx, it appear an error
root@debian:/usr/src/drachtio-dialogflow-phone-gateway# node app {"level":30,"time":1642661963767,"pid":548,"hostname":"debian","msg":"connected to sip on tcp/<IP_ADDRESS>:5060,udp/<IP_ADDRESS>:5060","v":1} {"level":30,"time":1642661996791,"pid":548,"hostname":"debian","msg":"call connected, starting dialogflow agent dialogcs-proj using lang en-US","v":1} {"level":30,"time":1642661996863,"pid":548,"hostname":"debian","msg":"res: undefined","v":1} {"level":30,"time":1642662003584,"pid":548,"hostname":"debian","msg":"received error","code":5,"v":1} {"level":50,"time":1642662003584,"pid":548,"hostname":"debian","msg":"got error: {\"msg\":\"com.google.apps.framework.request.NotFoundException: No DesignTimeAgent found for project 'dialogcs-proj'.\",\"code\":5}","v":1} {"level":30,"time":1642662360674,"pid":548,"hostname":"debian","msg":"call ended","v":1}
Do you have any advice? it seem drachtion could find dialogflow agent.
Thank you

I have only tested this module with dialogflow es, not sure it works with cx

Thank you, for your confirmation Dave. Is there any plan to update the API so can support using dialogflow cx?


Choice of personalization

Proposed changes

Ticket: https://ogre-d4g.atlassian.net/browse/D4G-178
Types of changes

What types of changes does your code introduce to ogre ?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, renaming)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation Update (if none of the other choices apply)
[ ] Other (please describe):

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.
offseason_ogre/blob/master/CONTRIBUTING.md) doc

[x] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation and modify the README.md (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules
[ ] If the feature changes current behavior, reasons why your solution is better.

What is the new behavior?

Add page to choose between personalization of the profile and predefined persona

@Baboo7 les modifs sont faites


Use new Setup defintion

To remove this warning:

@gep13 this requires v0.12.0 which has the needed bits in Mono code generation too.

@devlead sorry, not sure I follow?!?

Only the obsoleted method is avail on mono, the new method wasn't avail on mono. It's sorted in the newly released version just have to make sure we run it.
Perhaps we should introduce some logic in bootstrapper to ensure all's updated, as we use package.config for Cake it wont update if package is already installed.

@devlead ah, I see what you mean.  I have updated the packages.config file to install 0.12.0, however, you are right, if it is already in the tools folder, then the latest version won't be installed.

however, you are right, if it is already in the tools folder, then the latest version won't be installed.

@gep13 that's why you should alter your bootstrapper to check for md5sum on packages.config...
here's what I'm using:
$PACKAGES_CONFIG_MD5 = Join-Path $TOOLS_DIR "packages.config.md5sum"

...

# Restore tools from NuGet?
if(-Not $SkipToolPackageRestore.IsPresent)
{
    # Restore packages from NuGet.
    Push-Location
    Set-Location $TOOLS_DIR

    # Check for changes in packages.config
    if(Test-Path $PACKAGES_CONFIG_MD5)
    {
        if((Get-FileHash -Path $PACKAGES_CONFIG -Algorithm MD5).Hash -ne (Get-Content $PACKAGES_CONFIG_MD5))
        {
            Write-Verbose -Message "MD5 Sum changed. Removing nuget packages."
            Remove-Item * -Recurse -Exclude packages.config,nuget.exe
        }
    }
    else
    {
        Write-Verbose -Message "MD5 Sum not present. Removing nuget packages."
        Remove-Item * -Recurse -Exclude packages.config,nuget.exe
    }

    Write-Verbose -Message "Restoring tools from NuGet..."
    $NuGetOutput = Invoke-Expression "&`"$NUGET_EXE`" install -ExcludeVersion -OutputDirectory `"$TOOLS_DIR`""
    Write-Verbose -Message ($NuGetOutput | out-string)

    (Get-FileHash -Path $PACKAGES_CONFIG -Algorithm MD5).Hash | Out-File $PACKAGES_CONFIG_MD5 -Encoding "ASCII"

    Pop-Location
    if ($LASTEXITCODE -ne 0)
    {
        exit $LASTEXITCODE
    }
}

@mholo65 interesting.
@devlead @patriksvensson what are your thoughts on the above?  Should we add this also to the default bootstrapper file?

@mholo65 @devlead @gep13 Yes, I think this would make an excellent addition to the bootstrapper.

@mholo65 so, now the next question...
Have you got a sample of doing the same thing within the build.sh file?

@gep13 nope, but it shouldn't be too hard. I can implement both ps1 and sh and test on Windows + Linux if someone else can verify on Mac..

@mholo65 I can verify on OS X.

One crazy idea that would be xplat and not require hash files, would be to use a Cake script and use it's exit code if "purge" should be done or not.
Such a script could look something like this
FilePath packagesConfig = File("./tools/packages.config");
if (!FileExists(packagesConfig))
{
    throw new FileNotFoundException("Package config was not found at ./tools/package.config");
}

var packageVersion = new Version(XmlPeek(packagesConfig, "/packages/package[@id='Cake']/@version") + ".0");

Information("Package: {0}", packageVersion);

var version = Context.GetType().Assembly.GetName().Version;
Information("Assembly: {0}", version);

var compare = version.CompareTo(packageVersion);

Information("Compare: {0}", compare);

if (compare!=0)
{
    throw new Exception("Version mismatch");
}
else
{
    Information("Versions match!");
}

On success it would output something like
> .\tools\Cake\Cake.exe .\validatepackageconfig.cake
Package: <IP_ADDRESS>
Assembly: <IP_ADDRESS>
Compare: 0
Versions match!

> $LASTEXITCODE
0

On version mismatch
> .\tools\Cake\Cake.exe .\validatepackageconfig.cake
Package: <IP_ADDRESS>
Assembly: <IP_ADDRESS>
Compare: -1
Error: Version mismatch
> $LASTEXITCODE
1

@devlead @gep13 @mholo65 Or we can just do like I've done in the Nancy project: https://github.com/patriksvensson/Nancy/blob/7299ce5be4d5e2839a64ba693a1d396322f06308/cake-build.ps1#L20

@devlead @gep13 @mholo65 But I think that @mholo65's solution would be a great contribution to the cake-build/resources repository :)

@devlead @gep13 @mholo65 Or we can just do like I've done in the Nancy project: https://github.com/patriksvensson/Nancy/blob/7299ce5be4d5e2839a64ba693a1d396322f06308/cake-build.ps1#L20

@patriksvensson but that would imply skipping package.config, which personally I wouldn't mind ;)

Drawback would be that you'll endup with loads of Cake versions after a while.


[PostAgent] Make it possible to override Content-Type?

Would it be possible to make it so that Content-Type can be overridden?
Setting Content_Type: json works perfectly in most cases, but what if I want to send as json while defining my own Content-Type line?
Would it be possible to add a check which respects that the Content-Type header is set, even if Content_Type is set to json?
The reason for this request is that I need to remove the "; charset=utf-8" part which gets added to the Content-Type when Content_Type is set to json.

That's easy:
...
        "headers": {
          "Content-type": "application/json"
        }
...

  "headers": {
    "Content-type": "application/json"
  }

without setting  "content_type": "json" does indeed work partially. The issue is that it then wont format the payload as json.
To make post agent format as json it seem "content_type": "json" is needed which overrides the headers you've set yourself, and therein lies the issue for me.
Another possible fix for me would be to be able to declare the payload as json without setting content_type to json I guess?

I had a similar problem to yours, and setting "content_type" to "json" and adding a specific "Content-type" header fixed it.

Sadly that doesn't fix it, as stated earlier. Setting content_type to json overrides my "Content-type" header.
The config:
{
  "post_url": "https://example.com",
  "method": "post",
  "content_type": "json",
  "headers": {
    "Content-type": "application/json"
  },
  "emit_events": "true",
  "no_merge": "true",
  "output_mode": "clean",
  "payload": {
    "Test": "test"
  }
}

The headers that end up being sent:
{
  "content-length": "14",
  "accept": "*/*",
  "user-agent": "Huginn - https://github.com/huginn/huginn",
  "content-type": "application/json; charset=utf-8",
  "accept-encoding": "gzip,deflate"
}

I'd like for my own defined Content-type header to be overriding the header which comes from picking Content_Type Json


Add remote_data on all tests that require network

This PR implements a first step towards the goal of running the tests of astroplan without network access. I'm using the decorator @pytest.mark.remote_data from the package pytest-remotedata, which is currently in the dependencies. In the long term, I would like to check one by one the marked tests and see if the network access is actually required or if it is an
accident, typically by using Observer.at_site

Neat, thanks! We used to prevent problems due to internet access this with mocks like this:
https://github.com/astropy/astroplan/blob/7880daad50733fc4d5c6df0a6fd077bac3fefe39/docs/conf.py#L159-L160
Could you add the --remote-data flag to the test matrix, e.g. here:
https://github.com/astropy/astroplan/blob/f1bd5036534667e81339dc8f582c7ca0a03bdca2/tox.ini#L76

I have added a factor -remote that enables remote data in all tests. The tests without -remote use only local data. If you prefer that -cov tests are run always with remote data, it can be done also.

Thank you very much for smoking out the remote data tests, @sergiopasra ! I took some liberty to push follow up commits here. Hope you don't mind.


fix data race in history code (#405)

Done ! :)

I've moved the locking to the start of the functions.


pull request for e2e test at 02/05/2017 08:56:08

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


deprecated solution

[Enter feedback here]
according to this website https://learn.microsoft.com/en-us/azure/aks/static-ip
the solution to specify loadbalancerIP is deprecated

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

ID: ada58160-dc50-b977-d3bc-65f92406e16d
Version Independent ID: 5c596e04-fddb-effe-3b72-575f4df42d4e
Content: Use TLS with an ingress controller on Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/ingress-tls.md
Service: azure-kubernetes-service
Sub-service: aks-networking
GitHub Login: @asudbring
Microsoft Alias: allensu

@paulfinol
Thanks for your feedback! We will investigate and update as appropriate.

@asudbring
Can you please check and add your comments on this doc update request as applicable.

@paulfinol
Thanks for your feedback! We will investigate and update as appropriate.

#reassign:Nickomang

@paulfinol, Thanks for pointing this out. I understand that service.spec.loadbalancerIP has been deprected by upstream Kubernetes, and eventually Cloud Provider Azure will follow suit, but for the time being usage remains viable and services using the field should function without issue. I've added a follow up item to our backlog to change this article to use the new service annotations at such time as service.spec.loadbalancerIP is no longer valid.
Closing this for now, feel free to reopen if you'd like to continue the discussion further, and thanks again for bringing this up.
#please-close


Extract docs during static site generation

Addresses an issue introduced in 6a8200df95d3c5a732fc1f9f104f83556645954f. While that commit successfully avoids extracting the docs while developing locally, it neglected to handle the case when the site was being built for deployment. As a result this broke CI.
Options are now optional as they should be, and the site generation script will ensure docs are built during deployment.

Test Plan:
I ran generate.js and confirmed that the script ran to completion. When extractDocs is set to true, all HTML files for docs, apis, components are present in the build dir. if false, only the homepage, blog, showcase, help page are built.


Incomplete crawls dues to race-condition on crawl()

If the combination of number of items and fetch time per item is high enough (when compared to the crawler's crawl interval), the crawler can stop crawling before processing the discovery phase.
A simple example of this problem would be:

Set a low enough crawl interval (say, 1ms)
Give the initial url to the crawler
Watch the crawler stop crawling (normally) even though there are still items to be discovered

What is happening behind the scenes:
The crawler (i.e. lib/crawler.js) maintains an array and a counter (_openRequests and _openListeners, respectively) to know when it is safe to stop crawling (note: _openRequests was previously a counter as well).
Since the fetchcomplete event is triggered (and the array/counter updated) before reaching the discovery phase, the crawler can reach the next crawl step (i.e. the next setInterval tick) before having new items added to the queue (since the discovery phase is yet to run, even though the flags have already been updated). When this happens, the crawler (unexpectedly) stops normally.

Submitted a PR with a fix in line with the current behavior for _openRequests and _openListeners. Let me know if there are any problems.

@HugoMFernandes Hrm, this is certainly unexpected — since simplecrawler's own discovery process is synchronous and triggered synchronously after emitting fetchcomplete, and, unless I'm misunderstanding something relatively fundamental (which, admittedly, is not impossible!) it should not be susceptible to race conditions. Are you using your own asynchronous discovery process?
In order to support asynchronous discovery or holding the crawler open while client code checked in with an async service, a hook (crawler.wait()) was added, which as you have observed, is what _openListeners corresponds to. Unless there's a bug with this code, I think it should support your use case: https://github.com/cgiffard/node-simplecrawler#waiting-for-asynchronous-event-listeners
If this code isn't working, or doesn't satisfy your case, I'd like to explore fixing it rather than adding a parallel mechanism for tracking.
I really appreciate the effort you've gone to to clearly list the steps for reproduction, but just so I can be absolutely certain we're on the same page with this, do you have a code sample which reliably reproduces the problem?

Thanks for filing an issue, btw! :)

Hey @cgiffard ,
No problem. Basically, the code is not synchronous if the crawler needs to deflate/unzip the response content (in order to discover new items).
Here's a simple scenario reproducing the problem: https://github.com/HugoMFernandes/node-simplecrawler-i298

Wanted to thank you sincerely for filing such a comprehensive test case — first time I've seen one that complete! Apologies for the delay in my response — I'll get a chance to look at this on the weekend I think.

No problem, cheers :)

@HugoMFernandes thanks a lot for reporting this! :pray: The issue seemed to be that the queue item would be marked as fetched before the unzipping was done, so rather than maintaining an _openDiscoveries counter, I opted for a solution where the queue item is marked as fetched only after it's been unzipped.
I used your test case to debug the issue and could confirm that my changes indeed fixed the problem there!
I'll be sure to thank you in the 1.0.0 release notes as well :)

@fredrikekelund That's fine. I only added the _openDiscoveries counter because I assumed you wanted to keep fetching and discovering as different concepts (I hadn't even looked at master at this point / just 0.7). I prefer it this way as well.
Thanks for looking into this :)

@HugoMFernandes Thanks again for going to the trouble of submitting such a comprehensive test case!


Add backwards-compatible test matrix

Building off of https://github.com/fedimint/fedimint/pull/3982
Progress on https://github.com/fedimint/fedimint/issues/3962
Adds backwards-compatibility testing for fedimintd, gatewayd, fedimint-cli, and gateway-cli defined as a Cartesian product of the binaries and versions defined in scripts/tests/cross-test-ci-all.sh (just v0.2.1 for now). This approach won't scale well, however this is useful to begin identifying breaking changes in master, which already exist.
In CI, this step takes ~30 minutes so I've decided to run on each push instead of only in the merge queue. I think it's a good future improvement to parallelize the runs and to consider running only on merge.
Backwards-compatibility matrix is logged at the end of the run.
Backwards-compatibility tests summary:
fed_version  client_version  gateway_version  exit_code
v0.2.1       v0.2.1          current          1
v0.2.1       current         v0.2.1           1
v0.2.1       current         current          1
current      v0.2.1          v0.2.1           1
current      v0.2.1          current          1
current      current         v0.2.1           1

If any combination fails, this step will fail CI.

I'm fine with landing, but I think the first most needed improvement would be running these tests in parallel.

Should we also test old/new gateway-cli with old/new gateway? I think we currently test some combinations by accident, at least I'm getting the following error, which is caused by #3880:
00:01:27 Error: polling gateway connect-fed after 60 retries
00:01:27
00:01:27 Caused by:
00:01:27     0: command: /nix/store/k6x11hjg5frcc8frqhadylh66yirz6lk-gateway-cli/bin/gateway-cli --rpcpassword=theresnosecondbest -a ht
tp://<IP_ADDRESS>:11434 connect-fed fed11qgqpw9thwvaz7te3xgmjuvpwxqhrzw3j8qmrsvp0qqqjpvtxswx3mg6s3dfg8xsc0jjfjg86zs75nvtgxpzqvk4na2rpcv3kg
xhfny
00:01:27     1: exit status: 1
00:01:27        stdout:
00:01:27
00:01:27        stderr:
00:01:27        Error: error decoding response body: missing field `balance_msat` at line 1 column 59549
00:01:27
00:01:27        Caused by:
00:01:27            missing field `balance_msat` at line 1 column 59549

Should we also test old/new gateway-cli with old/new gateway? I think we currently test some combinations by accident, at least I'm getting the following error, which is caused by https://github.com/fedimint/fedimint/pull/3880

I agree it's useful to test old/new gateway-cli with old/new gatewayd. It's already intentionally included in the test matrix for this PR :smile:
https://github.com/fedimint/fedimint/pull/4010#discussion_r1443873703
I've previously reverted https://github.com/fedimint/fedimint/pull/3880 locally and verified the balance_msat error is resolved in the backwards compatibility tests.

Good stuff

I agree it's useful to test old/new gateway-cli with old/new gatewayd. It's already intentionally included in the test matrix for this PR 😄

Sorry :see_no_evil: I went by the table and didn't check the code :sweat_smile:

Sorry 🙈 I went by https://github.com/fedimint/fedimint/pull/4010#issue-2066005232 and didn't check the code 😅

No worries!
Also, for visibility to folks following this PR (https://github.com/fedimint/fedimint/pull/4050), I'm currently debugging some issues in CI in a separate PR to avoid cluttering this one. The GH actions runner started running out of disk space before finishing the backwards-compatibility run.

The latest change moves the backwards-compatibility tests to a separate github action workflow that isn't required to pass.

Doesn't add any additional time to the required Build on linux workflow
Gives a clear pass/fail of backwards-compatibility in CI if we wait for CI to fnish
Doesn't block merging if backwards-compatibility doesn't finish or fails

Rebasing to include the fix from https://github.com/fedimint/fedimint/pull/4104


Show boost criteria in "Get X Boost" CS quests

My proposal is to simply add the inspiration/eureka for the corresponding tech boost - for example, "$CS wants the Eureka for Feudalism" -> "$CS wants the Eureka for Feudalism (Create 6 farms)"

@brianschamel @chaorace I will look into this one later next week. This has also been bothering me.

@Remolten Any news?

@chaorace None yet. I'll look into it when I get time (maybe next week).


Wrong type for firmware-component-activation-date

The firmware-component-status/firmware-component-activation-date attribute has a wrong type. It is of type string but should be yang:date-and-time.

Based on decision at the DMIP call on 1st of April, we immediately fixed the issue and provided an updated YANG.


prism mock exits with fatal Token error message

Context

I was hoping to try out the mock behavior, but the lack of actionable error messages frustrated me and made me look elsewhere
Current Behavior

I used the docker getting started guide copying the command and providing the openapi URL:
$ docker run --init -p 4010:4010 stoplight/prism:4 mock -h <IP_ADDRESS> https://raw.githubusercontent.com/okta/okta-sdk-java/okta-sdk-root-10.0.0/src/swagger/api.yaml
[2:23:10 AM] › [CLI] …  awaiting  Starting Prism…
[2:23:13 AM] › [CLI] ✖  fatal     Token "definitions" does not exist.

I am completely open to that file being invalid OpenAPI according to some definition of invalid, but the tool does not tell me that. Also, I made sure that a copy of Insomnia could actually import it, meaning it was valid for some definition
Expected Behavior

Of course, the ultimate expected behavior is that it parses the provided file and generates stubs for its endpoints, but this specific bug is because

Token "definitions" does not exist.

does not offer the user any action to take; there are 14 mentions of definitions in that file, and the software is currently angry at one of them, but does not say which one
Possible Workaround/Solution

Steps to Reproduce

Execute the console command above
Observe the error

Environment

Version used: "stoplight/prism@sha256:0f582c33a702bbdfc1d47b43f9856ae98b943b2fe54d2265b1cfb37291e5435e" which is the current :4 as of this bug

Environment name and version (e.g. Chrome 39, node.js 5.4): the image has "NODE_VERSION=16.17.1" in its environment

Operating System and version (desktop or mobile): desktop

Link to your environment/workspace/project: see above

I can confirm that with the spec.yaml you linked to the docker command works; do you want this left open to track the bad error message, or closed because you're going to look into communicating the generic linting error better?

Let's leave it open while I discuss it with the team. Glad you're off to the races again! :)

@mdaniel , Prism 5.0.1 gives a nicer error message:
% docker run --init -p 4010:4010 stoplight/prism:5.0.1 mock -h <IP_ADDRESS> https://raw.githubusercontent.com/okta/okta-sdk-java/okta-sdk-root-10.0.0/src/swagger/api.yaml
[4:33:42 PM] › [CLI] …  awaiting  Starting Prism…
[4:33:45 PM] › [CLI] ✖  fatal     at "#/components/examples/AppUserSchemaResponse/value/properties/profile/allOf/0", token "definitions" in "#/definitions/base" does not exist


chore: copy across cat pool spec from celestia-core

Overview
Given that it is only the specification of celestia-app that is published it makes more sense to include it there rather than in the directory of the cat pool itself
Checklist

[ ] New and updated code has appropriate documentation
[ ] New and updated code has new and/or updated testing
[ ] Required CI checks are passing
[ ] Visual proof for any user facing features like CLI or documentation updates
[ ] Linked issues closed with keywords

Codecov Report

Merging #2207 (7937bc1) into main (772d0b8) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2207   +/-   ##
=======================================
  Coverage   24.89%   24.89%
=======================================
  Files         127      127
  Lines       14335    14335
=======================================
  Hits         3568     3568
  Misses      10400    10400
  Partials      367      367

In the past there was a push for specs to live in the same repo

alternatively we could simply link to the catpool spec in core. all that matters imo is that we have a single link to start with

I tend to agree that they should live in their own repo but its a bit strange when it comes to forks. I would also prefer that from the outside Celestia is presented as a single cohesive spec

it may as well make sense to have a separate website dedicated to the core related specs

Ideally I'd like to have a single site that contains all the specification of the celestia network (including celestia-node as well). For now, to make it more visible, I think it's better here and once we develop a better system we can remove it and just have a spec directory in core


/help/: Restyle docs.

This is a series of commits to restyle the /help/ page docs to be more readable and polished looking.

A few notes:

Can you style "MAIN INDEX" more like the headings in the slideout?  It seems too big.  Maybe style it like "Guides" but without the expander chevron?
Don't expand "account basics" by default; I think it clutters the sidebar.
Maybe when a section is expanded, the chevron should change to point right or something?  Not sure what's standard here, but I think it'd help make the sidebar behavior clearer if the "Guides" one we're starting expanded looks different.
Did the sidebar font size get bigger?  Tons of things line-wrap.

Have you tested on mobile?  I didn't.

All of them except for:

Maybe when a section is expanded, the chevron should change to point right or something? Not sure what's standard here, but I think it'd help make the sidebar behavior clearer if the "Guides" one we're starting expanded looks different.

Are updated. We can come up with the best idea when in office together perhaps?

@brockwhittaker looks like you didn't push?

This is up to date.

Merged, thanks @brockwhittaker!
@synicalsyntax FYI, I'd definitely appreciate any thoughts you have on further revisions!

I like the facelift, but can we bring back the green/white theme for the left sidebar? It stood out visually and promoted our branding colors better. We could also change the indigo to a blacker/grayer shade (#333?) so that it can show better contrast as well.
Would be willing to discuss this further on the #design stream :)

I'm personally OK with bringing back a version of the green, but probably worth discussing a bit as a group (you may want to just prototype a color you like to anchor the discussion before posting on #design).


Technical debt

Here's a list of things to clean up, preferably before a release.  Perhaps we can all get together in Berlin and hack on these.

mtiLib: Add tests.  Import from Monotype spec repo.
otlLib: Finish moving code from mtiLib / feaLib here, and review / polish the API.
mtiLib & feaLib: Compare API and harmonize.
mtiLib & feaLib: Add main.
varLib: Refactor and add main.

Improve test coverage.

Figure out what to do re new pens / move symfont into proper place.

mtiLib error reporting: print line number as well as input line.

what's the status of this?
maybe is it too general and should be split up into separate issues?


rapidjson accessing memory it doesn't own

In FindMember(const Ch* name), `name` is accessed directly without regards to
the length of `name` itself. `name` is provided by the user, so we can't assume
anything about it.

This is not causing bugs in production, but I think it's still bad practice.
Enabling Guard Malloc
(https://developer.apple.com/library/mac/documentation/performance/conceptual/ma
nagingmemory/articles/MallocDebug.html) on Mac OS X is catching this.

Also, I'm not sure what this access is looking for. If the `name` is not
0-terminated, it that such a bad thing?

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 21 Apr 2014 at 5:09

Forgot to specify : document.h line 271.

Original comment by<EMAIL_ADDRESS>on 21 Apr 2014 at 5:10

Thanks for reporting this issue.

I agree that this is a bug and can indeed lead to problems in production, e.g.
in cases where 'name' is dynamically allocated and (significantly) shorter than
the name of the (longest) member element in the JSON object.

Since the existing member names may be coming from the outside world, e.g. when
using 'FindMember' to validate the contents of the received JSON object, this
could at least lead to a DoS issue in some cases.

You can find a patch in my personal/unofficial fork at GitHub at
  https://github.com/pah/rapidjson/commits/fixes/108
  https://github.com/pah/rapidjson/commit/6558b8fa
(append .patch or .diff to download the raw contents)

Original comment by<EMAIL_ADDRESS>on 23 Apr 2014 at 3:55

Fixed in
https://github.com/miloyip/rapidjson/commit/02673bec74967dcd108be1a77d9e8841b38d
1aa0

Original comment by<EMAIL_ADDRESS>on 20 Jun 2014 at 11:17

Changed state: Fixed

Issue 109 has been merged into this issue.

Original comment by<EMAIL_ADDRESS>on 20 Jun 2014 at 11:17


Getting "type object argument after ** must be a mapping" error in image generation

Hi, @lucidrains,
We have trained the model using OpenAIs pretrained VAE. We have use the train_dalle.py file for training.
python train_dalle.py --image_text_folder=./dataset_images
After training, we have tried to generate the result using the saved model by following command.
python generate.py --dalle_path=/content/DALLE-pytorch/dalle-final.pt --text='bird has wings that are brown and has a red crown' --num_images=20
On running the above command we are facing the error:
Traceback (most recent call last):   File "generate.py", line 55, in <module>     vae = DiscreteVAE(**vae_params) TypeError: type object argument after ** must be a mapping, not NoneType

Thank you for the quick response @lucidrains ,
We have already tried the changes suggested by you, in which facing below error :
Traceback (most recent call last):   File "generate.py", line 65, in <module>     dalle = DALLE(vae = vae, **dalle_params).cuda() TypeError: type object got multiple values for keyword argument 'vae'

@pragnakalpdev7 removing this line should do the trick. The vae will get passed in properly later on.
https://github.com/lucidrains/DALLE-pytorch/blob/720a6060084e27a99f343802d3799a9fa8797ec6/train_dalle.py#L78

Again, can't verify myself but I believe if you're using a pretrained VAE (OpenAIDiscreteVAE, for instance), then I believe that the value for 'vae_params' will actually be none. The dictionary **dalle_params may contain the vae definition. As such, you can just remove that argument from your call to DALLE:
dalle = DALLE(**dalle_params).cuda()

Does that work?

Hello @afiaka87,
We have already tried this trick, but not getting good results. We think maybe we are missing something.

Hello @afiaka87,
We have already tried this trick, but not getting good results. We think maybe we are missing something.

@pragnakalpdev7 one more patch! https://github.com/lucidrains/DALLE-pytorch/commit/f2b02bab902f4acd435b0eadb4442e2788c0fcae


HHH-16436 Fix Numeric Overflow Exception

If we set type "Integer" for NUMBER(10,0), Numeric Overflow Exception occurs when some data stored in db (eg:<PHONE_NUMBER>) is bigger than Integer.MAX_VALUE.

(And $2^{31}-1$ is a 10-digit number, so this is correct.)

But some datas like '3210987654' can be saved in column NUMBER(10,0), which is bigger than Integer.MAX_VALUE.

But some datas like '3210987654' can be saved in column NUMBER(10,0), which is bigger than Integer.MAX_VALUE.

That's correct. But Oracle doesn't give us a way to create a numeric column with exactly 31 binary digits so this is the best we can possibly do. [The contrary option would be to use NUMERIC(9,0) to store integers, and then some int values would not fit. Which is obviously much worse.]
Of course in practice one is extremely unlikely to encounter numeric values like<PHONE_NUMBER> in a NUMBER(10,0) column, since everyone sensible declares column types with at least one decimal digit of overhead. So this "problem" is entirely hypothetical.


datetime-local (and probably others) don't play well with autosave=True

I have trouble editing datetime-local fields in Chrome with autosave=True. The problem seems to be due to the fact that autoform is setting the .value of the input I'm editing with a value that's different than the current value.
For example, when typing into an input type="datetime-local", I see the following on https://github.com/aldeed/meteor-autoform/blob/master/inputTypes/datetime-local/datetime-local.js#L5
> val
Wed Apr 30 3 23:59:00 GMT-0700 (PDT)
> AutoForm.Utility.dateToNormalizedLocalDateAndTimeString(val, atts.timezoneId)
"0003-04-30T23:59:00.000"

This eventually results in the following if statement running in attrs.js:
var ValueHandler = AttributeHandler.extend({
  update: function (element, oldValue, value) {
    if (value !== element.value)
      element.value = value;
  }
});

> element
<input type="datetime-local" name="registrationOpenDate" id="uiENhKmFuDygkivAc" data-schema-key="registrationOpenDate" class="form-control">
> element.value
"0003-04-30T23:59"
> value
"0003-04-30T23:59:00.000"

This if statement shouldn't run because the values are really the same, but AutoForm.Utility.dateToNormalizedLocalDateAndTimeString formats the time string with seconds, which screws this all up (for chrome at least, I don't know how other browsers behave).

This is kind of related to #774.

@jankapunkt, just curious: why did you close this?

Hi @jfly the issue is 6 years old and I closed all before 2020. We did a major update to v7. If this issue persists in v7 feel free to reopen and I will dedicate some time to resolve it.


Limitation of 200 pod identities per Cluster

I know that there was initial a limit of 50 pod identities per Cluster, which has been increased to 200. We're now facing those limits as our plan is to share the AKS cluster for multiple applications, which are sitting in their dedicated namespaces. As some of the applications have multiple microservices, its not unusual to have 10-15 identities used for a single application.
Where does the limitation of 200 come from? Is it NMI level limitations (altough 200 Nodes per AKS Cluster is supported and NMI runs as a daemon set)? Any chance to increase the limit?

Thanks for reaching out to us , but at this time it does not look like you are referring to a document but to a specific scenario . If that's the case we request you to please post your question on MicrosoftQ&A and other forums , we have our engineers and others who are monitoring them and will be happy to help .
If you have a document which you are following and if you think it needs  an update , please let us know about that here .


There is a problem about import Pilmoji

from pilmoji import Pilmoji
File "/usr/local/lib/python3.8/dist-packages/pilmoji/init.py", line 1, in
from . import helpers, source
File "/usr/local/lib/python3.8/dist-packages/pilmoji/helpers.py", line 7, in
from emoji import EMOJI_UNICODE
ImportError: cannot import name 'EMOJI_UNICODE' from 'emoji' (/usr/local/lib/python3.8/dist-packages/emoji/init.py)
My system is Ubuntu 20.04 ,pilmoji version: 2.0.1
What can I do?

Being fixed by #13

The fix seems to have been made, but when will it be published to pypi?


Upgrade dependencies

Just upgrade the dependencies to the latest version. Left out typescript deps because they will be removed soon anyways. Also didn't touch react-native because tests began to fail after I upgraded to v0.56.0.

Okay so there seems to be an issue with node 6 and jest-styled-components 6. Do we want to keep running travis with node 6 or should switch to just running with 8 and 10 instead?

@alexandernanberg We can do 8 + 10 in the 4.0 branch yeah


Sliderbreak doesn't appear

Describe the bug
On this replay of my friend, he didn't fc but Rewind says he did. Looking more into it, there was a sliderbreak at 760 combo but in Rewind it doesn't show up and continues the combo.
Expected behavior
Having the sliderbreak show up on Rewind like in the game
Screenshots
idk why i can't attach files so ig i'm using imgur
in game
on Rewind
Desktop (please complete the following information):

OS: Windows
Version 11

Additional context
Please also provide the following if they are mentioned and related:
Beatmap : https://osu.ppy.sh/beatmapsets/1816835#osu/3902948
Replay : https://osu.ppy.sh/scores/osu/4508226732
Skin : Any

this app seems dead


Icons

For example for the reload button. Maybe Google  or something else.

Will be resolved by including PrimeNG via #17.


Update offline-backup-overview.md

Remove preview tag for Data Box for MARS, add DPM-A, MABS as supported.

#sign-off

#sign-off


⚠ PrivateHD has degraded performance

In 68ee7a2, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 2324 ms

Resolved: PrivateHD performance has improved in e9263f7.


Added missing patternops

Signed-off-by: ashish<EMAIL_ADDRESS>Description
This PR fixes #
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

@Revolyssup is this more or less a copy/paste from other adapters?

@Revolyssup is this more or less a copy/paste from other adapters?

yyupsss, copy paste was done way before. This is just some correction

@Revolyssup is this more or less a copy/paste from other adapters?

yyupsss, copy paste was done way before. This is just some correction

That's great. 👍


🐛 [FIREBASE_MESSAGING] FirebaseMessaging.onMessageOpenedApp does not work

Bug report
Describe the bug
i'm testing the official flutterfire  example for firebase_messaging, https://github.com/FirebaseExtended/flutterfire/tree/master/packages/firebase_messaging/firebase_messaging/example the FCM notifications works in fore and background, i'm using the latest plugins:
firebase_messaging: ^9.1.2
is supposed that the message screen must be show when the notification is tapped, but nothing happens, I don't even have the debug messages.
Steps to reproduce
run the example from github, when you tap on notification, the event FirebaseMessaging.onMessageOpenedApp is not triggered
Steps to reproduce the behavior:

run the example
click on arrived notification
expect to open messages screen
nothings happens

Expected behavior
the app must show the message screen

Flutter doctor
Run flutter doctor and paste the output below:
flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 2.0.4, on Microsoft Windows [VersiÃ³n 10.0.19042.928], locale es-419)
[√] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[√] Chrome - develop for the web
[√] Android Studio (version 4.1.0)
[√] Connected device (4 available)

• No issues found!

Hi @alexlovar
If you run the official example on android the notification will be presented in the foreground using flutter_local_notifications.
This is why the firebase messaging stream will not be fired when tapping the notification.
Foreground notifications are just a limitation on android so you will need to take a look at how the flutter_local_notifications official example does things if you want to use tapping on the notification for interaction.
Thank you

so strange because the flutter_local_notifications plugin is include in the sample, but the example does not work correctly

Hi, On foreground onMessageOpenedApp didnt work anyone got solution for this?

yeah is a firebase bug, look this link: https://github.com/FirebaseExtended/flutterfire/issues/6011
just put the  FirebaseMessaging.onMessage.listen((RemoteMessage message) just after gettoken:


Add zarf tools yq (or similar) for easily parsing yaml files

Is your feature request related to a problem? Please describe.
As Ezra I want to be able to use yq with Zarf so that I can quickly pull information out of YAML files / Kubernetes Manifests within actions.
Describe the solution you'd like

Given I have a manifest I want to pull information from
When I run zarf tools yq
Then I have access to the yq CLI: https://github.com/mikefarah/yq

Describe alternatives you've considered
We could explore other libraries but the main yq cli is already written in go - it could also be put behind another layer rather than at the root of tools (to pave the way for potentially more tools like jq)
Additional context
This comes from us wanting to reduce dependencies within Software Factory and open up more options when using UDS Run.

Some notes:
// SPDX-License-Identifier: Apache-2.0
// SPDX-FileCopyrightText: 2021-Present The Zarf Authors

// Package tools contains the CLI commands for Zarf.
package tools

import (
	yqCLI "github.com/mikefarah/yq/v4/cmd"
)

func init() {
	yqCmd := yqCLI.New()
	yqCmd.Use = "yq"

	toolsCmd.AddCommand(yqCmd)
}

Successfully embeds yq under zarf tools yq. zarf tools yq --version works properly.
Some things I wasn't able to grok about https://github.com/mikefarah/yq/blob/master/yq.go that may cause issues w/ usage. More research required.

Thanks! I had already gotten that far locally and then sidetracked trying to track down why it made the package so much larger (tldr I wasn't building on Linux). Back to trying to put in doc generation and test things. Will need to try to account for making sure per your note that there aren't issues with certain commands running.

Super interested in this hitting main. Happy to test once ready.


chore(backend): remove sqp from google search results

Pull Request
Purpose

looking at options to remove SQP from the google search results
Approach

Adding <meta name="robots" content="noindex, nofollow" /> to SQP markdown files, looking at options to create a robots.txt
Open Questions and Pre-Merge TODOs

[ ] looking at the best options to solve this issue
[ ] create a robots.txt to deny google from indexing the SQP pages

Requirements

[ ] These changes meet the standards for contributing.
[ ] I have read the code of conduct.

according THIS info, these changes should work


Support custom source and resource folders

I've got a custom configuration of source and resource folders:
android {
    …
    sourceSets {
        main {
            manifest.srcFile 'AndroidManifest.xml'
            java.srcDirs = ['src']
            res.srcDirs = ['res']
            assets.srcDirs = ['assets']
            jniLibs.srcDirs = ['libs']
        }
        test {
            java.srcDirs = ['test']
            resources.srcDirs = ['test']
        }
    }
    …
}

This isn't honored by gradle-android-eclipse. It seems to expect the standard src/main/java etc. hierarchy.
Workaround: Edit the .classpath manually after running gradle eclipse.

Feel free to issue a pull-request.


Update adobe-acrobat-pro to 2018

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

There are significant differences between these files, the new one seems to be a "update" package that's half the size of the current download.
Also, this was updated September 2017 from 2015 to 2017 and inspecting the current download shows it was modified Feb 2018.
Thanks for the PR nonetheless!

i found the real, full download URL for Adobe Acrobat 2018 (DC) now:
https://trials3.adobe.com/AdobeProducts/APRO/18/osx10/Acrobat_DC_Web_WWMUI.dmg
(the won't work in browser because of the cookie requirement)
should i submit another pull request?


t.throws() doesn't seem to be working

What am I doing wrong here? t.throws() doesn't seem to be working (not failing the test when a function is not throwing any exceptions).
Here's a sample test, that passes successfully (instead of failing as expected):
'use strict'

var test = require('tape')

function emptyFunction () {
  // nothing happens here
}

test('This test should fail', function (t) {
  t.plan(1)
  t.throws(emptyFunction())
})

I'm on Node 5.0, tape version 4.4.0.

@dmitrizagidulin throws takes a single function - you're passing it undefined (the return value of a function)

Ohh I see. Is there a way to pass parameters to that function? (and why does it count undefined as a successful throws() test?)

t.throws(myFunction.bind(null, 'arg1', 'arg2'))

t.throws(function () {
  myFunction('arg1', 'arg2')
})

@dmitrizagidulin indeed that seems like a bug - assert.throws(undefined) throws with "block must be a function".

Thanks! :) Appreciate the guidance and the fix.

np - I'll be releasing a patch version for v1, v2, v3, and v4 shortly that will include this fix.


[Map] Scroll behaviour inside ScrollView

Does the map support the feature that scrolling from an outbounding ScrollView is disabled while scrolling inside the MapView?
Google and Apple Maps support this out of the box. If you move around the map the ScrollView is fixed. The HMS map you cannot move if it is inside a ScrollView. Very little and laggy from left to right but not up and down.
Is there a setting I am missing? Or do I have to build this by my own checking the "onCameraMove" callback?

The only solution I can think of is disabling ScrollView while interacting with the map. For instance the code below worked for me
export default function App() {
  const [mapActive, setMapActive] = useState(false);
  return (
    <>
      <ScrollView scrollEnabled={!mapActive} style={{padding: 16}}>
        <View style={{padding: 16, backgroundColor: "yellow", marginBottom: 8}}>
          <Text>This is a View</Text>
        </View>
        <View style={{padding: 16, backgroundColor: "yellow", marginBottom: 8}}>
          <Text>This is a View</Text>
        </View>
        <MapView
          mapType={1}
          style={{height: 200, marginBottom: 8}}
          onCameraIdle={() => {
            console.log("Idle");
            setMapActive(false);
          }}
          onCameraMove={() => {
            console.log("Move");
            setMapActive(true);
          }}
        />
        <View style={{padding: 16, backgroundColor: "yellow", marginBottom: 8}}>
          <Text>This is a View</Text>
        </View>
        <View style={{padding: 16, backgroundColor: "yellow", marginBottom: 8}}>
          <Text>This is a View</Text>
        </View>
      </ScrollView>
    </>
  );
}

Thanks, this is what I meant by building it by my own.

Is there any way to speed up the idle time event? If I moved the map it takes about 7 seconds until it gets fired. So the app seems stucked for 7 sec.
Maybe it is possible to set it as an attribute here HuaweiMap.OnCameraIdleListener?

This should be a feature request, pls reopen it @Mike-mei , as the Author said Google and Apple Maps (React Native Map) support this out of the box.

export default function App() {
  const [mapActive, setMapActive] = useState(false);
  return (
    <>
      <ScrollView scrollEnabled={!mapActive} style={{padding: 16}}>
        <View style={{padding: 16, backgroundColor: "yellow", marginBottom: 8}}>
          <Text>This is a View</Text>
        </View>
        <View style={{padding: 16, backgroundColor: "yellow", marginBottom: 8}}>
          <Text>This is a View</Text>
        </View>
        <MapView
          mapType={1}
          style={{height: 200, marginBottom: 8}}
          onCameraIdle={() => {
            console.log("Idle");
            setMapActive(false);
          }}
          onCameraMove={() => {
            console.log("Move");
            setMapActive(true);
          }}
        />
        <View style={{padding: 16, backgroundColor: "yellow", marginBottom: 8}}>
          <Text>This is a View</Text>
        </View>
        <View style={{padding: 16, backgroundColor: "yellow", marginBottom: 8}}>
          <Text>This is a View</Text>
        </View>
      </ScrollView>
    </>
  );
}

tried this in the latest @hmscore/react-native-hms-map,  the map is difficult to be scrolled in the ScrollView, sad for Huawei User haha, I had to deliver the app in coming weeks, this will delivery bad UX to the Huawei User, Please take caution on this.

@aktug is there any progress on this issue? Setting scrollEnabled={false} every time the camera is Idle is really not a good solution, especially since slowly moving the map will send multiple Move -> Idle calls. Also, starting to move the map vertically (in case of vertical ScrollView) doesn't work most of the time, giving a really poor UX.

Hi All,
We has any update for scrollEnabled={false}, as @anyamiletic mentioned, every time the camera is Idle is really not a good solution, it is not smooth from my side


Bad includes from CLI Generator when renaming outputs with GenerateName

OS: Windows
Target: MSVC
Brief Description
I was using a the GenerateName property of Driver.Options to provide a function that modified the names of my output files (adding "Wpr" to the end of the name in this case) to make it easier to identify the automatically generated wrappers. This worked until I tried to generate a wrapper for a class that made use of another wrapped class. The cpp file for the dependent wrapper tried to #include the other wrapper on it's original rather than modified name.
Please let me know if you need more information to reproduce this.

Thanks for the bug report.
@ddobrev This one should be easy to fix, please assign this to whoever is available.

@tritao will do. Mohit should be done with another user's bug tomorrow so I'll have him take care.


[hotfix][table-planner] Correct StreamExecLookupJoin from SingleTransformationTranslator to MultipleTransformationTranslator

The StreamExecLookupJoin node adds support for upsertMaterialize since 1.16 and it should be a MultipleTransformationTranslator. For the BatchExecLookupJoin, it does not change because no upsertMaterialize support in batch mode.
This hotfix aims to correct the class signature.

@flinkbot run azure

@flinkbot run azure


Added a Hangman Game Project

Hangman is a word game where one player guesses letters to solve a word chosen by another player. If the player guessing makes a wrong guess, the other player draws a part of a person being hanged. The game is over when the person being hanged is complete.

mention the issue no

@rimmon1234 mentioned the issue number

Issue No #222

@rimmon1234 It's merged. Thanks for your contribution.

Hello I Am a bit concerned that my this contribution or pr is not shown in the gssoc leaderboard

Please Give The Necessary Labels For Gssoc @iamrahulmahato @Tapabrata-Goswami


Yandex browser

clappr player not work on yandex browser ,   only work on OS - Windows 10 , can anyone fix so it can play on Windows 7 too ?

@Kreeig did you notice any console error? what kind vide were you playing? do you have any url to test?

yes , for example - http://sportlive.ws/s/euro2 , you see there no error , just not play

hey @Kreeig I rewrite the title (for SEO) and I'll let this issue open so anyone can pick an try to fix it. Thanks.

I have that browser and it works on Fedora 23 + Yandex Browser. The video is flash version

I've tested here with MacOS Yosemite, it's working fine. @Kreeig can you still reproduce this?


Add functions for computing basic lens properties such as entrance pupil, f#, focal length, etc.

The current system doesn't have these basic functions. They are not very important if you are only doing simulations/scattering of an existing lens system but they become important for optimization, which we will add in a future release.

These functions seem less important in modern lens design, with off axis or free form surfaces where the concepts don't necessarily even make sense. Closing the issue.


Redis: Change MaxValue for the port 65535

Changed MaxValue for the port from 65536 to 65535.
Not sure if PortField would be better suited here, as both IntegerField and PortField are used in other places. It's just that the current max of 65536 is out of the valid port range...
While highly unlikely anyone would try this, I just did and it indeed let me set the port to 65536 .

@fvanroie I think you accidentally pushed a bit more than intended (SwaggerUI doesn't look like redis ;) )

Better use PortField when already there (and a new branch) ;)

Sorry I think I messed up this PR. I only meant to push out e92e219.
I'll resubmit the PR?


Add NetworkPolicy handler to block cross-project network traffic (#456)

This change adds a new handler function that creates NetworkPolicies for each AppInstance. It will always create one NetworkPolicy to allow ingress only from other apps in the same project. It will additionally create another NetworkPolicy for each published port in the Acornfile, and this port will be accessible from anywhere inside the cluster (as well as outside, thanks to the Ingress resource that also gets created).
This also adds a new configuration option to acorn install: --disable-network-policies (false by default).
re: #456
Checklist

[x] The title of this PR would make a good line in Acorn's Release Note's Changelog
[x] The title of this PR ends with a link to the main issue being address in paranthesis, like: This is a title (#1216). Here's an example
[x] All relevant issues are referenced in the PR description. NOTE: don't use GitHub keyworkds that auto-close issues
[x] Commits follow contributing guidance
[x] Automated tests added to cover the changes. If tests couldn't be added, an explanation is provided in the Verification and Testing section
[x] Changes to user-facing functionality, API, CLI, and upgrade impacts are clearly called out in PR description

Marking this as draft again. The last thing I need to figure out here is handling published TCP ports, which have klipper-backed LoadBalancer services created for them.

I'm closing this and going to open a new PR. golangci-lint is failing for no discernable reason, since it doesn't fail for me locally, even when I spin up a new environment inside of a container and run it there. Not sure what weird stuff is happening with this...


JSONPath not correctly translated to JSONPatch paths

Description:
Consider the following fragment of the Envoy Proxy bootstrap yaml:
layered_runtime:
  layers:
  - name: global_config
    static_layer:
      re2.max_program_size.error_level:<PHONE_NUMBER>
      re2.max_program_size.warn_level: 1000
  - name: some_other_layer
     static_layer:
        key.entry: value

I would like to add the key envoy.some.key.or.other.made.up into the static_layer of the layer called global_config.
A JSONPatch that works looks like this:
- op: add
  path: /layered_runtime/layers/0/static_layer/envoy.some.key.or.other.made.up
  value: a value

But this patch assumes that the layer I want to add the key to is always the first.
I tried to use a JSONPath instead, with the following example:
- op: add
  jsonPath<EMAIL_ADDRESS>  value: a value

I have to use the bracket syntax (['envoy.some.key.or.other.made.up']) because the dots must not be treated as one of the JSONPath separators. This is valid JSONPath according to the RFC:

JSONPath expressions move further down the hierarchy using dot notation ($.store.book[0].title) or the bracket notation ($['store']['book'][0]['title']); both replace XPath's / within query expressions, where dot notation serves as a lightweight but limited syntax while bracket notation is a heavyweight but more general syntax.

This results in the following key added to the correct hash:
layers:
   - name: global_config
     static_layer:
      '[''envoy.some.key.or.other.made.up'']': a value
       envoy.restart_features.use_eds_cache_for_ads: false
       envoy.something.completely.made.up: arbitrary string
       re2.max_program_size.error_level:<PHONE_NUMBER>

I get the same result for this JSONPath string as well:
- op: add
  jsonPath<EMAIL_ADDRESS>  value: a value

The translation from JSONPath to a Path used by the JSONPatch should not have the brackets as part of the result.
<EMAIL_ADDRESS>should translate to /layered_runtime/layers/0/static_layer/envoy.some.key.or.other.made.up and not to /layered_runtime/layers/0/static_layer/'[''envoy.some.key.or.other.made.up'']'

cc @denniskniep

@liorokman can you try this:
- op: add
  jsonPath<EMAIL_ADDRESS>  path: envoy.some.key.or.other.made.up
  value: a value

@denniskniep I tried this, and this works.
Don't you think that it's a little counter-intuitive relative to how regular JSONPatch add operations are specified? When JSONPath is not being used, the path used also points to a place that doesn't exist yet in the document.
For example:
- op: add
  path: /layered_runtime/layers/0/static_layer/envoy.some.key.or.other.made.up
  value: a value

In this case, the envoy.some.key.or.other.made.up value doesn't exist, yet the JSONPatch RFC says that this is how it should be specified in order to add a new key.
At the very least, the documentation should be clear that the value of the path attribute is appended to the calculated values of the jsonPath attribute. I couldn't find this information in the current documentation.

Hello @liorokman
Are you using patch via EnvoyPatchPolicy CRD ?

@eshepelyuk I'm actually using it for modifying the bootstrap configuration in EnvoyProxy

@eshepelyuk I'm actually using it for modifying the bootstrap configuration in EnvoyProxy

Thank for confirming, i realized that jsonPath support for EnvoyPathcPolicy is not released yet and there is no docs regarding it

@eshepelyuk I'm actually using it for modifying the bootstrap configuration in EnvoyProxy

Thank for confirming, i realized that jsonPath support for EnvoyPathcPolicy is not released yet and there is no docs regarding it

It's not released yet, but you can see the documentation here

@denniskniep

- op: add
  jsonPath<EMAIL_ADDRESS>  path: envoy.some.key.or.other.made.up
  value: a value

Edit: Reason is, that JsonPath could only select what is currently in the document. If you want to introduce a new property you have to append it via path property.

If it's an error to specify a JSONPath that doesn't exist in the document, then that error should be returned.
If it's not an error, then the generated Path that is returned should correctly contain the decoded path.
The current behavior is that:

No error is returned
The generated path contains an invalid value '[''envoy.some.key.or.other.made.up'']'

The result is invalid configuration without any error message.

@liorokman  I completely agree that this should be better documented.
Furthermore I also agree to return the error and that we need to fix the jsonPointer generation, if special characters are used.
Thanks for highlighting these points, I will address them soon


[Polymer-Cli] Security  Vulnerabilities with 1.8.0

Our small project is using "polymer-cli": "^1.8.0" and after we run the npm audit, we got the  npm audit security report  which contains two HIGH level warning.
 High            Denial of Service

  Package         https-proxy-agent

  Patched in      >=2.2.0

  Dependency of   polymer-cli [dev]

  Path            polymer-cli > github > https-proxy-agent

  More info       https://nodesecurity.io/advisories/593

according to https://www.npmjs.com/package/github, this package has been renamed.
and
 High            Regular Expression Denial of Service

  Package         fresh

  Patched in      >= 0.5.2

  Dependency of   polymer-cli [dev]

  Path            polymer-cli > web-component-tester > send > fresh

  More info       https://nodesecurity.io/advisories/526

Can we update those?

This is a duplicate of #497. We are working on extracting the problematic packages from WCT. Hopefully we can finish this work soon to resolve all outstanding audit issues.