Any extra dataset prep needed?

I have followed the instructions from README. I have set up a TPU v3-8 machine which can be confirmed below:

I have hosted the ImageNet-1k (imagenet2012) in a separate bucket and it's structured like the below (following instructions from here):

While launching training, I am using the following command:
gcloud alpha compute tpus tpu-vm ssh $NAME --zone=$ZONE --worker=all --command "TFDS_DATA_DIR=gs://imagenet-1k/tensorflow_datasets bash big_vision/run_tpu.sh big_vision.train --config big_vision/configs/vit_s16_i1k.py  --workdir gs://$GS_BUCKET_NAME/big_vision/workdir/`date '+%m-%d_%H%M'`"

It results into the following:
SSH key found in project metadata; not updating instance.
SSH: Attempting to connect to worker 0...
2022-05-10 10:30:25.858388: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libcudart.so.11.0'; dlerror: libcudart.so.11.0: cannot open shared object file: No such file or directory
2022-05-10 10:30:27.319919: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libcuda.so.1'; dlerror: libcuda.so.1: cannot open shared object file: No such file or directory
2022-05-10 10:30:27.319952: W tensorflow/stream_executor/cuda/cuda_driver.cc:269] failed call to cuInit: UNKNOWN ERROR (303)
I0510 10:30:27.335715<PHONE_NUMBER>75488 xla_bridge.py:263] Unable to initialize backend 'tpu_driver': NOT_FOUND: Unable to find driver in registry given worker:
I0510 10:30:27.336199<PHONE_NUMBER>75488 xla_bridge.py:263] Unable to initialize backend 'gpu': NOT_FOUND: Could not find registered platform with name: "cuda". Available platform names are: Interpreter TPU Host
I0510 10:30:30.058175<PHONE_NUMBER>75488 train.py:65] Hello from process 0 holding 8/8 devices and writing to workdir gs://big_vision_exp/big_vision/workdir/05-10_1030.
I0510 10:30:30.568850<PHONE_NUMBER>75488 train.py:95] NOTE: Global batch size 1024 on 1 hosts results in 1024 local batch size. With 8 dev per host (8 dev total), that's a 128 per-device batch size.
I0510 10:30:30.570343<PHONE_NUMBER>75488 train.py:95] NOTE: Initializing train dataset...
I0510 10:30:31.039579<PHONE_NUMBER>75488 dataset_info.py:522] Load pre-computed DatasetInfo (eg: splits, num examples,...) from GCS: imagenet2012/5.1.0
I0510 10:30:31.303886<PHONE_NUMBER>75488 dataset_info.py:439] Load dataset info from /tmp/tmpggpl8znitfds
I0510 10:30:31.308489<PHONE_NUMBER>75488 dataset_info.py:492] Field info.description from disk and from code do not match. Keeping the one from code.
I0510 10:30:31.308714<PHONE_NUMBER>75488 dataset_info.py:492] Field info.release_notes from disk and from code do not match. Keeping the one from code.
I0510 10:30:31.308900<PHONE_NUMBER>75488 dataset_info.py:492] Field info.supervised_keys from disk and from code do not match. Keeping the one from code.
I0510 10:30:31.308959<PHONE_NUMBER>75488 dataset_info.py:492] Field info.module_name from disk and from code do not match. Keeping the one from code.
I0510 10:30:31.309248<PHONE_NUMBER>75488 logging_logger.py:44] Constructing tf.data.Dataset imagenet2012 for split _EvenSplit(split='train[:99%]', index=0, count=1, drop_remainder=False), from gs://imagenet-1k/tensorflow_datasets/imagenet2012/5.1.0
Traceback (most recent call last):
  File "/usr/lib/python3.8/runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.8/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/home/spsayakpaul/big_vision/train.py", line 372, in <module>
    app.run(main)
  File "/home/spsayakpaul/bv_venv/lib/python3.8/site-packages/absl/app.py", line 312, in run
    _run_main(main, args)
  File "/home/spsayakpaul/bv_venv/lib/python3.8/site-packages/absl/app.py", line 258, in _run_main
    sys.exit(main(argv))
  File "/home/spsayakpaul/big_vision/train.py", line 122, in main
    train_ds = input_pipeline.make_for_train(
  File "/home/spsayakpaul/big_vision/input_pipeline.py", line 69, in make_for_train
    data, _ = get_dataset_tfds(dataset=dataset, split=split,
  File "/home/spsayakpaul/big_vision/input_pipeline.py", line 53, in get_dataset_tfds
    return builder.as_dataset(
  File "/home/spsayakpaul/bv_venv/lib/python3.8/site-packages/tensorflow_datasets/core/logging/__init__.py", line 81, in decorator
    return function(*args, **kwargs)
  File "/home/spsayakpaul/bv_venv/lib/python3.8/site-packages/tensorflow_datasets/core/dataset_builder.py", line 565, in as_dataset
    raise AssertionError(
AssertionError: Dataset imagenet2012: could not find data in gs://imagenet-1k/tensorflow_datasets. Please make sure to call dataset_builder.download_and_prepare(), or pass download=True to tfds.load() before trying to access the tf.data.Dataset object.

Is there anything I'm missing out here?

The error in https://github.com/google-research/big_vision/issues/2#issuecomment-1122519532 is expected I think since
https://github.com/google-research/big_vision/blob/8ca9d84a82d40f3245b5ab2daac5c2405b223351/big_vision/tools/download_tfds_datasets.py#L39
data_dir is already set here.

Sure!
I am currently running this:
https://github.com/google-research/big_vision/issues/2#issuecomment-1122533163

Update.
This is the current error (I faced one regarding imagenet2012_real but was able to quickly resolve it):
11 01:50:04.274455<PHONE_NUMBER>39360 logging_logger.py:44] Constructing tf.data.Dataset imagenet_v2 for split _EvenSplit(split='test', index=0, count=1, drop_remainder=False), from gs://imagenet-1k/tensorflow_datasets/imagenet_v2/matched-frequency/3.0.0
Traceback (most recent call last):
  File "/usr/lib/python3.8/runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.8/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/home/spsayakpaul/big_vision/train.py", line 372, in <module>
    app.run(main)
  File "/home/spsayakpaul/bv_venv/lib/python3.8/site-packages/absl/app.py", line 312, in run
    _run_main(main, args)
  File "/home/spsayakpaul/bv_venv/lib/python3.8/site-packages/absl/app.py", line 258, in _run_main
    sys.exit(main(argv))
  File "/home/spsayakpaul/big_vision/train.py", line 270, in main
    evaluators = eval_common.from_config(
  File "/home/spsayakpaul/big_vision/evaluators/common.py", line 37, in from_config
    evaluator = module.Evaluator(model, **cfg)
  File "/home/spsayakpaul/big_vision/evaluators/classification.py", line 34, in __init__
    self.ds, self.steps = input_pipeline.make_for_inference(
  File "/home/spsayakpaul/big_vision/input_pipeline.py", line 97, in make_for_inference
    data, _ = get_dataset_tfds(dataset=dataset, split=split,
  File "/home/spsayakpaul/big_vision/input_pipeline.py", line 53, in get_dataset_tfds
    return builder.as_dataset(
  File "/home/spsayakpaul/bv_venv/lib/python3.8/site-packages/tensorflow_datasets/core/logging/__init__.py", line 81, in decorator
    return function(*args, **kwargs)
  File "/home/spsayakpaul/bv_venv/lib/python3.8/site-packages/tensorflow_datasets/core/dataset_builder.py", line 565, in as_dataset
    raise AssertionError(
AssertionError: Dataset imagenet_v2: could not find data in gs://imagenet-1k/tensorflow_datasets. Please make sure to call dataset_builder.download_and_prepare(), or pass download=True to tfds.load() before trying to access the tf.data.Dataset object.

Currently doing (after installing tfds-nightly):
import tensorflow_datasets as tfds

data_dir = "gs://imagenet-1k/tensorflow_datasets"
ds = tfds.load("imagenet_v2", data_dir=data_dir, download=True)

It seems to be taking more than expected but will keep on updating anyway. I am maintaining a log here:
https://gist.github.com/sayakpaul/9544d3ba935805bd47d71fd8596e7bc0 (not yet complete).

Looks like I was able to make things up and running:

I have also updated the gist I mentioned in https://github.com/google-research/big_vision/issues/2#issuecomment-1123162532.
Keeping it open until the training completes.


PHP 8.0 default Laravel build failing

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Getting a docker build failure for the default Laravel template:
#8 21.64          at /nix/store/lgcjj6s23v2203zyihsd0j26wh1saj0c-source/pkgs/build-support/trivial-builders/default.nix:88:7:
#8 21.64
#8 21.64            87|       inherit buildCommand name;
#8 21.64            88|       passAsFile = [ "buildCommand" ]
#8 21.64              |       ^
#8 21.64            89|         ++ (derivationArgs.passAsFile or []);
#8 21.64
#8 21.64        error: php80 has been dropped due to the lack of maintenance from upstream for future releases

To reproduce
Start a new project and select the Laravel template.
Expected behavior
Successful build
Environment
version v1.20.0 on railway

php 8.0 is not supported anymore (whether intentionally or not)
please use php 8.1

@brody192 Should I report this in another Railway repo? It's the image used by the Railway template itself. I didn't select 8.0.

yes that repo is out of date, you would need to change the php version used in the composer file within your cloned repo

Ah ok, thank you!


Additional include support

includes can be within other blocks
added tests based on jade-loader

Thanks for your contribution! I'll try it out ASAP


Windows machine needed

@tellison kindly offered to help.
@sxa555 and @smlambert can you add a description of the machines?
Shelley mentioned the build machines description might be enough.

Sorry I mis-spoke

That windows box is a bare metal server ...

The build machines build-win2012-1.adoptopenjdk.net and build-win2012-2.adoptopenjdk.net are bare metal machines.
I hadn't noticed that the referenced test machine was at Packet.

I also wouldn't say no to a second Windows 2012R2 for JCK testing if you're able to get one - our first one is at Joyent. It doesn't have to be anything large, 2CPU, 4GB sort of spec would likely be adequate/

I saw this on my notification feed. If the machine hasn't already been added, I would like to add it using the Jenkins Node Helper API. I have the jobs ready to be deployed, waiting on PR approvals (should be done soon).
Here's a link to the API repo openjdk-jenkins-helper

@VermaSh Sounds worth a shot. We haven't got the machine yet though

To clarify, for this project we would be requesting 2 more Windows machines, 1 for JCK testing, 1 more for non-JCK testing (preferably 8GB/8CPU/Windows Server 2012R2).

I am provisioning two additional Windows machines.  We are limited to hostnames of <= 15 chars, so they are named:

test-win2012-1.adoptopenjdk.net to be known to Jenkins as test-softlayer-x64-windows-2012r2-1 and,
jck-win2012-1.adoptopenjdk.net to be known to Jenkins as jck-softlayer-x64-windows-2012r2-1

Each one is 2 CPU, 8GB RAM, Windows 2012R2, 100Gb boot disk, 25 Gb data disk, 100 Mbs network link.
I will create a new issue for configuring these machines for project use.

@smlambert

I am not fussy on spec, can be small as @sxa555 mentioned, 2CPU, 4GB, or a duplicate of the test-packet-x64-windows-2012r2-1 (8GB, 8CPU running Windows Server 2012R2), this would improve test execution time, as we run some types of tests concurrently.

Let me know how you get on with the 2CPU 8GB test machine.  I can easily add more CPUs in situ if you get more fussy and decide you would like them :-)
@VermaSh Please see issue #337 for configuring these new machines with help from your scripts.
I'm closing this issue as completed.  Please open a new issue if there are further machine requests.


Support for custom HTTP routes when using gRPC

Hi,
In GrpcService routes are dynamically generated from the gRPC definition. When I try to use this HTTP client the url has to be /grpc.hello.HelloService/Hello which is not very friendly. It would be great addition if I can provide custom routes, example /Hello would make it easy for any HTTP Client.

I had discussed this briefly with @kowshikn and just want to clarify, this is for supporting unframed gRPC together with normal  - normal gRPC naturally needs to be at it's normal paths for use from gRPC clients.

Hi, @kowshikn and @anuraaga. Thanks for the feature request. Would it be enough to allow specifying a custom path for each gRPC service? e.g.
Server.builder()
      .service(GrpcService.builder()
                          .addService(fooService)
                          .addService(barService)
                          .enableUnframedRequests(false)
                          .build())
      .service(GrpcService.builder()
                          .addService("/foo", fooService)
                          .addService("/bar", barService)
                          .enableUnframedRequests(true)
                          .build())
      .build();

Yeah I think that's reasonable :) Another idea I had was setting a mapping of path to MethodDescriptor in GrpcService. Since we convert from mapped path to method descriptor based on the DSL right now might be simpler to just allow adding a way to do a custom mapping at that point.


Stopped updating?

Last update Saturday 0005, meaning Sunday/Monday updates haven't happened. Are you guys on this?

Hi @diplodata thanks for raising this issue. It stems from the world bank data bank API's SSL certificate failing verification. We are looking into where exactly the problem lies.

Hi, the issue has been resolved and the dataset has been updated.


List of UX issues

Patterns
SUSE theme?

Manage cf users

View users and their permissions
Change the permissions of a single user
Optional - Bulk change the permissions of multiple users (assign only? apply new permissions to all users?)

Endpoints list page (and possibly others): Top nav action icons - are they clear enough?

Should we have text? Better icons?

Navigation bar on left + tabs

Require some kind of depth indicator normally solved by breadcrumbs (in a space of an organisation of a cloud foundry)
Could this be solved via pseudo menu breadcrumbs in main left hand side nav bar?


Add author

The author shows as category header in PackMan on current 2019.4 and all 2020+.
Before this PR, it just read "Other".

Thank you for this PR, I've merged it.


Typo of homework 2 fixed

Corrected typo in iloc function, now it takes all the rows and not only first 10

Thanks a lot, you now have a place in the hall of fame: https://github.com/cafawo/FinancialDataAnalytics/graphs/contributors


Change LanesInUse to int64_t

LanesInUse is changed from size_t to int64_t to cater for
unknown case, this PR would align pldm changes to the new PDI
change.
Signed-off-by: Manojkiran Eda<EMAIL_ADDRESS>

failing to compile because it's dependent on the PDI change


Bootstrap Scrollspy

With the feature available in Version 3.3.6, is it already ported in Version 4.4.2. alpha or do I have to use jQuery to set it up?
I would like to know if its available for Bootstrap Alpha.

Scrollspy is present in v4: http://v4-alpha.getbootstrap.com/components/scrollspy/


Zend_Service_Amazon_S3: Can't debug

I'm using ZF 1.12.3 and putObject method stopped works from localhost (Eg: <IP_ADDRESS> or fakeDomain.tld), always returns bool(false), no error, notice or warning.
In online production server works fine. After hours, i captured error by Wireshark and discovery the error, its about "RequestTimeTooSkewed ". Is it possible display exception when putObject or other methods return false. Believe to be most useful to debug.
Thank you.

This is a good idea, but at this point we can not change the API (current behaviour or return type).
So I will close this issue.


Update webpack-chain version

New release have very useful feature - https://github.com/neutrinojs/webpack-chain/pull/84

This would be a breaking change to our public API. So you need to monkey patching its internal implementation for compatibility with earlier versions. Otherwise we'll have to wait until v4 to update it.


Using Grafana UI with standard generated prometheus rules

Hi.
First, thanks for all the good work you've made with this project.
I am trying to use your project, but i would like to keep Grafana for a UI as my users are used to it.
I also would like to stick with the standard generate rules (not the one with the --generic-rules options) as the resulting rules offer way more possibilities (like dynamic grouping).
I have been trying to rebuild dashboards similar to your interface inside Grafana and managed to get some first results as shown by the following screen captures:

However, I'm encountering some issues with missing information from the background Prometheus data source:

The calculation window
The availability target
The success or errors PromQL request
The total PromQL request

All this information is available from the API you provide and could probably be exploited through a JSON data source. However, to keeps things simple for my user, I would like to avoid doing it.
So I post this message here to know if you have some ideas or hints to help me achieve my goal ?
Regards,

Hey, I get what you want (and I want it myself), however, there are some limitations in the Grafana UI that will make this rather hard to implement (otherwise I would have).
For example, the grouping uses dynamic labels and we would need Grafana to support dynamic tooltips (based on queries) which is something they can't do right now. (I wanted to open an issue for a long time).
Until we can fix these things upstream in Grafana, I think the best idea is to use the generic rules  and for some SLOs that use grouping use the UI directly.

Hi, thanks for your answer.


Queries that involve string timestamps do not work properly unless the string is singled quoted

I have a table with the following time column schema:
  "dateTimeFieldSpecs": [
    {
      "dataType": "STRING",
      "format": "1:DAYS:SIMPLE_DATE_FORMAT:yyyy-MM-dd-HH",
      "granularity": "1:DAYS",
      "name": "time_stamp"
    }
  ],

When I run the following query that should yield non-null output I do get empty response with no failure:
"SELECT \"time_stamp\", avg(\"fifteen_minutes_freshness\") AS \"fifteen_minutes_freshness:avg\" FROM DailyPinotGlobalDataFreshnessTable WHERE ((\"time_stamp\" >= 2023-08-02-00 AND \"time_stamp\" <= 2023-08-31-23)) GROUP BY \"time_stamp\"  ORDER BY avg(\"fifteen_minutes_freshness\") DESC  LIMIT 30"
+------------+-------------------------------+
| time_stamp | fifteen_minutes_freshness:avg |
+------------+-------------------------------+
+------------+-------------------------------+

Adding single quotes around the timestamp values makes the query work properly:

"SELECT \"time_stamp\", avg(\"fifteen_minutes_freshness\") AS \"fifteen_minutes_freshness:avg\" FROM DailyPinotGlobalDataFreshnessTable WHERE ((\"time_stamp\" >= '2023-08-02-00' AND \"time_stamp\" <= '2023-08-31-23')) GROUP BY \"time_stamp\"  ORDER BY avg(\"fifteen_minutes_freshness\") DESC  LIMIT 30"
+---------------+-------------------------------+
|   time_stamp  | fifteen_minutes_freshness:avg |
+---------------+-------------------------------+
| 2023-08-29-00 | 99.28717948717949             |
| 2023-08-30-00 | 98.84102564102564             |
| 2023-08-25-00 | 98.5228426395939              |
+---------------+-------------------------------+

Further more using a single boundary does yield a result but not the proper one:
"SELECT \"time_stamp\", avg(\"fifteen_minutes_freshness\") AS \"fifteen_minutes_freshness:avg\" FROM DailyPinotGlobalDataFreshnessTable WHERE (**\"time_stamp\" >= 2023-08-29-00**) GROUP BY \"time_stamp\"  ORDER BY avg(\"fifteen_minutes_freshness\") DESC  LIMIT 30"
+---------------+-------------------------------+
|   time_stamp  | fifteen_minutes_freshness:avg |
+---------------+-------------------------------+
| 2023-08-29-00 | 99.28717948717949             |
| 2023-08-30-00 | 98.84102564102564             |
| **2023-08-25-00** | 98.5228426395939              |
+---------------+-------------------------------+

When you do use single quotes around the time stamp, it does yield the proper result:

"SELECT \"time_stamp\", avg(\"fifteen_minutes_freshness\") AS \"fifteen_minutes_freshness:avg\" FROM DailyPinotGlobalDataFreshnessTable WHERE (\"time_stamp\" >= '2023-08-29-00') GROUP BY \"time_stamp\"  ORDER BY avg(\"fifteen_minutes_freshness\") DESC  LIMIT 30"
+---------------+-------------------------------+
|   time_stamp  | fifteen_minutes_freshness:avg |
+---------------+-------------------------------+
| 2023-08-29-00 | 99.28717948717949             |
| 2023-08-30-00 | 98.84102564102564             |
+---------------+-------------------------------+

This is not a big deal for manual queries but we do have some tooling that autogenerates queries like these that do work well in other SQL interface dbs but they produce the wrong output in Pinot silently: i.e. there is no error reported but the query result is incorrect.
Desired Behavior: is that query figures that the passed timestamp is a string with the format in the schema and parsed as such. If not and error must be reported while parsing the query.

This is incorrect. 2023-08-31-23 will be parsed as 2023 - 8 - 31 - 23 (minus) ==> 1961. The query itself parses but is semantically wrong. String literal must be quoted

Uff that is a very misleading parsing, specially given the fact that the server knows that this field is a string: shouldn't the schema play a role here? If I am filtering on a string column using a numeral that should be an error right? or even better the field being used as filter should be interpreted as a string.

Uff that is a very misleading parsing, specially given the fact that the server knows that this field is a string: shouldn't the schema play a role here? If I am filtering on a string column using a numeral that should be an error right? or even better the field being used as filter should be interpreted as a string.

AFAIK, no DB can handle this because that is the standard way of parsing it. We can make pinot fail the query if it detects type mismatch (this might be the standard SQL behavior), but that is an incompatible change because I can imagine people might ingest numbers as string type, and filter string with number


chore: use AlgHom.restrictDomain

I previously didn't know the existence of AlgHom.restrictDomain and wrote many lemmas using a longer expression that is exactly the definition of AlgHom.restrictDomain; I'm now changing all these lemmas to use AlgHom.restrictDomain. There are more appearances of .comp (IsScalarTower.toAlgHom but either the file doesn't import AlgHom.restrictDomain, or using it breaks the proof, so I only replaced one appearance.

Another replacement can be made, and if reviewers think it's appropriate, I can certainly merge.

@alreadydone yes, please merge in the additional replacement.

bors d+


compile

how do you compile this?

Download build requirements
Open solution in Visual Studio
Hit Start
???
Profit


[[FIX]] Accept get and set as ID properties

get and set are both valid identifiers, so they may appear as PropertyDefinitions within a PropertyDefintionList.
(I moved some code around, so adding ?w=1 to the review URL will suppress white space changes and should make reviewing easier.)
Resolves gh-2288.

Coverage remained the same at 96.33% when pulling 2ad235c3ae3bdfd0d6d8e4874f9ad0df0be5bce9 on jugglinmike:2288-get-as-ident-prop into 12811e797a264f323b920b314db965b8adcb207e on jshint:master.


Add score for ECO code ECO:0000204 "Author statement"

We have some evidence that refer to ECO code http://purl.obolibrary.org/obo/ECO_0000204 "Author statement" but that is missing from the eco_scores.tsv file.
This ECO code needs to be added to the file, and an appropriate score modified decided upon.

We are no longer using ECO codes for scoring in the platform anymore.

We are no longer using ECO codes for scoring in the platform anymore.


_posts/0000-01-02-georgescreates.md

This is actually my initialisation on GitHub. I am completed this course for newbies and so far, I created a branch, created a file and made a commit, and opened a pull request

This is the comment for this commitment


How to run the entire current file?

Besides of %SnipRun

Besides :%SnipRun, aka the intended way ??
Not much, except the API (see API.md, run_range) while hacking a little for the end of the range, maybe putting -1 or just a large number will work, I haven't tested

I think you might appreciate my wrappers:
function SnipRunFlowSnippet()
  -- set up state to track if user cancels or accepts
  local canceled = false
  local accepted = false

  -- prompt for selection
  print "Select code to run and press ESC to cancel or Return to execute"

  -- enter visual lines
  vim.cmd "normal! V"
  vim.cmd "redraw"

  while not canceled and not accepted do
    -- loop to do the selection allowing enter, esc or v-block up and down
    local char = vim.fn.getchar()
    if char == 27 then -- escape
      canceled = true
    elseif char == 13 then -- return
      accepted = true
    elseif char == 106 then -- j
      vim.cmd "normal! j"
      vim.cmd "redraw"
    elseif char == 107 then -- k
      vim.cmd "normal! k"
      vim.cmd "redraw"
    end
  end

  if accepted then
    -- HACK: exit visual first to get selection to marks
    local esc = vim.api.nvim_replace_termcodes("<esc>", true, false, true)
    vim.api.nvim_feedkeys(esc, "x", false)
    -- run the snippet
    vim.cmd ":'<,'>SnipRun"
  end

  -- exit visual and confirm cancellation
  if canceled then
    print "Operation canceled"
    local esc = vim.api.nvim_replace_termcodes("<esc>", true, false, true)
    vim.api.nvim_feedkeys(esc, "x", false)
  end
end

function SnipRunFlowFile()
  vim.cmd "normal! ggVG$"
  local esc = vim.api.nvim_replace_termcodes("<esc>", true, false, true)
  vim.api.nvim_feedkeys(esc, "x", false)
  vim.cmd ":'<,'>SnipRun"
end

setting them from bindings in normal 1. in a table:
      "<cmd>lua SnipRunFlowSnippet()<CR>",
      "Execute SNIPPET with SnipRun",
      general_opts,
    },

    ["<leader>cX"] = {
      "<cmd>lua SnipRunFlowFile()<CR>",
      "Execute FILE with SnipRun",
      general_opts,
    },```

2. or native:
```vim.api.nvim_set_keymap('n', '<leader>cx', ':lua SnipRunFlowSnippet()<CR>', general_opts)
vim.api.nvim_set_keymap('n', '<leader>cX', ':lua SnipRunFlowFile()<CR>', general_opts)```

@tom-gora , I don't know if you were addressing me or Wangch29.
I personally don't see very much added value (plus, you might want your script to listen to up/down arrows as well as j/k, this caused me 30s of confusion). If you prefer "command then select" workflow, maybe the "operator mode" (see the friendly manual ) for vim motions will be of interest
Thanks for sharing your script

@michaelb Sorry, I was intending to reply to the op, as I came up with the solution and earlier I've seen this issue while researching. I agree the added value is small. I barely wrote this to be able to quickly trigger sniprun with leader based mappings without needing to type commands. This works for me and by all means I'm not a pro, I just wanted to help the person asking the question.

vim.api.nvim_set_keymap('n', '<leader><leader>r', '<cmd>%SnipRun<CR>', {silent = true})


Update bemserver-celery-settings.py

Avoid to repeat timezone
Fix BaseSchedule.__init__() got an unexpected keyword argument 'day'

Sorry. This got closed accidentally (and silently) when I renamed master into main.
You're right about day_of_month.
Would you like to resubmit against main branch?

No problem. #233 have just been opened.


Add umask option to package resource type

Use Case
When the puppet agent runs in a scheduled manner under the puppet service on a Linux server, it uses umask 022 when it installs python modules and gems via the package resource type. When forced to run by sudo puppet agent --test, the python and gem packages get installed so that only root can use the packages. This is frustrating as the package then needs to be manually uninstalled, and then reinstalled being sure to set umask 022. This assumes the team member running the command even noticed there was an issue in the first place. Generally the issue appears when a GitLab Runner process begins to fail or another process begins to fail.
Describe the Solution You Would Like
I would like to see one of the following:

Resource type package uses umask 022 by default in the background, maybe for:

gem
pip, pip2, pip3
puppet_gem
puppetserver_gem

Or

Add attribute umask to resource type package

Describe Alternatives You've Considered
We currently use npwalker-recursive_file_permissions in our various Puppet modules based on server type and whether or not we are installing a gem or a python module.
Additional Context
N/A

@bobnegri Thank you for opening this issue, would it be possible to check if this behavior is being caused by puppet or if it's being caused by running the python and gem commands? Thank you!

I have no idea how I missed the question. Sorry.
I know from experience that running the commands as 'root', one has to add 'umask 022' before executing the pip install or gem install commands, otherwise only the 'root' user can use the newly installed/updated module.
As I mentioned, when the puppet service kicks of the puppet agent run on a Linux server, things seems to install with the correct permissions. The issue is when we manually force the puppet agent to run due to limited change implementation windows. (At my company, the puppet agent runs hourly.)


cereal/SubMaster: encapsulate frequency management in FrequencyTracker

This PR refactors the frequency management in the SubMaster class by encapsulating it into a new FrequencyTracker class. This change simplifies the code, improves maintainability, and enhances efficiency by reducing redundant dictionary lookups. The FrequencyTracker class handles frequency calculations and checks, making the SubMaster class cleaner and more focused on its core responsibilities.

trigger-jenkins


Read source file instead of redirecting stdin

This PR adding file open feature, so our compiler doesn't need redirection anymore.
Also, I added some help information.
lee@ubuntu /U/l/d/VitaminC (read-input-file)> ./vitaminc --help
A simple C compiler.
Usage:
  ./vitaminc [options] file

  -o, --output <file>  Write output to <file> (default: test.ssa)
  -d, --dump           Dump the abstract syntax tree
  -h, --help           Display available options

I think we can add a feature to call qbe and gcc directly, so that users can generate executables directly.
Right now, our code generation flow requires calling qbe and gcc manually.

Additionally, I would like to rename the result of auto args = cmd_options.parse(argc, argv); as auto opts = ...;, and the result of auto inputs = args.unmatched(); as auto args = ...;. The rationale is that currently, inputs represents arguments, which are not actually optional. The command line option parser first parses the options, and those unmatched are the arguments. The usage can be viewed as follows:
Usage: ./Vitaminc [OPTIONS...] ARGUMENTS...

What are your thoughts on this?

Additionally, I would like to rename the result of auto args = cmd_options.parse(argc, argv); as auto opts = ...;, and the result of auto inputs = args.unmatched(); as auto args = ...;. The rationale is that currently, inputs represents arguments, which are not actually optional. The command line option parser first parses the options, and those unmatched are the arguments. The usage can be viewed as follows:
Usage: ./Vitaminc [OPTIONS...] ARGUMENTS...

What are your thoughts on this?

Yeah, I think renaming args to opts has better readability! 👍


Episodic effect

@dotzhou how about squashing all these commits?  I will show you how.  Otherwise looks good.


add archlinux support

This got cherry-picked from https://github.com/elastic/puppet-logstash/pull/334

Hi,
Thanks for the patch, but there's a philosophical problem here...
Arch Linux is not a supported operating system for Logstash, so it's weird for us (Elastic) to claim support in the Puppet module.
There's also the practial issue that we should aim to have acceptance testing for every operating system the module supports. Maintaining the tests can be just as much work as maintaining the module.
If I read your patch correctly, people who insist on running Logstash and this module on Arch Linux can just set manage_repo => false. Is that correct?

Hi,
at the moment it is enough to set manage_repo to false. Each module should provide working defaults for a operating system it wants to support. There are many modules on the forge that support operating systems that are not officially supported by the upstream vendor. I added lots of unit tests + improvements in https://github.com/elastic/puppet-logstash/pull/334, and I can add acceptance tests as well for Archlinux if https://github.com/elastic/puppet-logstash/pull/334 gets merged

Sorry, but there's no intention for this module to support Arch Linux. We consider the module a (small) part of the whole Elastic Stack of products and Arch is not a supported OS for the stack. If we imply support in this part, it could imply support in other parts that isn't there.


Added Google remarketing tag

Description
https://jira.mesosphere.com/browse/DCOS_SITE-210
Urgency

[ ] Blocker
[ ] High
[ ] Medium

Requirements

Test all commands and procedures.
Build content locally and test for formatting/links.
Add redirects to dcos-website/redirect-files.
Change all affected versions (e.g. 1.7, 1.8, and 1.9).
See the contribution guidelines.

@daveyheuser thanks for doing this on such short notice! The link checker is flagging the URL with a warning about a leading or trailing whitespace... Do you think it's possible to fix that without breaking the functionality of the tag? I'll ask tiffany about it too. Thanks!


False positive CVE-2017-9735

There is false positive reported on dependency jetty-setuid-java-1.0.3.jar:
https://github.com/eclipse/jetty.toolchain/tree/master/jetty-setuid/jetty-setuid-java
Maven coordinates:
<dependency>
    <groupId>org.eclipse.jetty.toolchain.setuid</groupId>
    <artifactId>jetty-setuid-java</artifactId>
    <version>1.0.3</version>
</dependency>

This dependency uses jetty only as API dependency:
https://github.com/eclipse/jetty.toolchain/issues/7

@volphy thanks for the report. I would highly recommend reading:

How does dependency-check work
How to read the report

In this case the false positive is not on the CVE, but rather the CPE (common platform enumeration). We can add a suppression to remove this FP.
In addition, if this truly is only an API dependency - it should likely have the scope set to provided.


[SPARK-50674][PYTHON] Fix check for ‘terminate’ method existence in UDTF evaluation

What changes were proposed in this pull request?
Fix check for ‘terminate’ method existence in UDTF evaluation
Why are the changes needed?
To ensure that UDTFs without a terminate method can still be used with partitioning without causing an AttributeError.
Previously, udtf with partitioning will raise an AttributeError if the terminate method is not defined, as shown below
>>> from pyspark.sql.functions import udtf
>>> from pyspark.sql import Row
>>>
>>> @udtf(returnType="a: int")
... class TestUDTF:
...     def eval(self, row: Row):
...             if row[0] > 5:
...                     yield row[0],
...
>>> df = spark.range(8)
>>> TestUDTF(df.asTable().partitionBy("id").orderBy("id")).show()
org.apache.spark.api.python.PythonException: Traceback (most recent call last):
...
  File "...pyspark/worker.py", line 1052, in eval
    if self._udtf.terminate is not None:
AttributeError: 'TestUDTF' object has no attribute 'terminate'

However, the terminate method is not required in such cases.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Existing tests.
Was this patch authored or co-authored using generative AI tooling?
No.

should we add a simple test?

Sounds good! Added a unit test.

Merged to master.


SMIRNOFF: Using  non-integer bond orders in interpolation

The spec  describes interpolating force constants between bond orders of integers, i.e. 1 and 2. It might be useful to allow interpolating between bond orders that aren't integers, i.e. having the force constant change with bond order with one slope between 1.0 and 1.5, but a different slope between 1.5 and 2.0. The spec does not disallow this, but if we wanted to use them, it would be useful to explicitly state that non-integer values are allows.
This was brought up at some point in the past few months by the team that was investigating WBOs - @SimonBoothroyd or @pavankum may be able to provide more context, including if this is still of scientific interest.

Yeah, parameters with a steep slope due to lack of diverse training data tend to produce unphysical k values when extrapolated below 1, if there is flexibility to include non-integer bond orders that would be great so that we can divide into ranges of super-steep and not-so-steep slopes. Another approach discussed was to use a flat value below 1 and interpolate afterwards, may be these can be named as a different scheme, instead of using the tag linear may be step-linear or something similar that conveys it explicitly.

This is interesting. Yes, I suggest explicitly allowing this.


Issue while excluding IP

I noticed Total Requests still increasing in the HTML report after excluding an IP, although the excluded IP is not being shown in the VISITOR HOSTNAMES AND IPS panel.
To run the goaccess I'm using the config file.
goaccess -f /var/log/nginx/access.log -p /etc/goaccess/custom_goaccess.conf

To exclude IP ip-add, added following to the custom_goaccess.conf:
exclude-ip ip-add

@allinurl am I missing something here? wrong configuration or something else..

Are you using the latest version v1.3?

Yes @allinurl
> goaccess --version
GoAccess - 1.3.
For more details visit: http://goaccess.io
Copyright (C) 2009-2016 by Gerardo Orellana

Build configure arguments:
  --enable-utf8
  --enable-geoip=mmdb
  --enable-tcb=btree


Bottom sheet is closing wrongly

Reason
the bottom sheet is closing while the user touch close to the learn more button in the offer sheet
Description
The bottom sheet is closing wrongly while the user touchs nearby the the learn more button in the offer sheet
Success Criteria

[ ] the behavior descripted should not happen

Additional Information
Problably the swipe function is closing the bottom sheet, because the div classname of the learn more button also is "title"

outdated


Travis-ci stage for deploy

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


Make sure expandMember works with a trailing slash

I ran across this issue when trying to build nushell. Their Cargo.toml has a trailing slash in the globbing pattern, which breaks naersk with the same error seen in #53
This fix addresses that specific issue, however the members field actually accepts a much more extensive globbing syntax, at least according to the docs.
I'm hoping that more involved cases are relatively rare in practice. If anyone has a more extensive/less error-prone way of handling this, please do!

Updated to use lib.hasSuffix and enabled github actions on the fork.

Hmm, I might be missing something(this is my first time using Github Actions), but it looks like there's a failure in the post action due to a missing environment variable:
/home/runner/work/naersk/naersk/.github/actions/nix/install-nix: line 71: CACHIX_SIGNING_KEY: unbound variable

Hmm, well I'm not sure why the checks are not showing up in this pull request, but the actions now pass on the fork. I had to make a small fix to the action to make sure CACHIX_SIGNING_KEY was set as well.


fix: downgrade semantic-release-action

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

v1.1.11
GitHub release

Your semantic-release bot :package::rocket:


🐛 Bug: Error: could not determine executable to run

Bug Report Checklist

[X] I have tried restarting my IDE and the issue persists.
[X] I have pulled the latest main branch of the repository.
[X] I have searched for related issues and found none that matched my issue.

Expected
When running npx package-json-validator --warnings --recommendations I expect to install temporarily this package and then run the validator.
Actual
I get the following error:
npm error could not determine executable to run

Additional Info
I believe that this is caused by the removal of the bin key in the package.json by #89.
Can we restore this key and see if that works?

Logs:
14 http fetch GET 200 https://registry.npmjs.org/package-json-validator 132ms (cache revalidated)
15 verbose stack Error: could not determine executable to run
15 verbose stack     at getBinFromManifest (/opt/homebrew/lib/node_modules/npm/node_modules/libnpmexec/lib/get-bin-from-manifest.js:17:23)
15 verbose stack     at exec (/opt/homebrew/lib/node_modules/npm/node_modules/libnpmexec/lib/index.js:202:15)
15 verbose stack     at async Npm.exec (/opt/homebrew/lib/node_modules/npm/lib/npm.js:207:9)
15 verbose stack     at async module.exports (/opt/homebrew/lib/node_modules/npm/lib/cli/entry.js:74:5)
16 verbose pkgid<EMAIL_ADDRESS>

Ran into the same thing just now.
I see a PR for this is up. Could you take a look into merging it @JoshuaKGoldberg, pretty please?

To other folks finding this issue: awaiting the PR getting merged, you can run @rgant’s fork with fix directly using the following command:
npx github:rgant/package-json-validator#add-package-bin

Oh! Sorry, I dropped this notification. Will fix now!

Thank you, kind Sir!


Add restart option to admin pulldown menu

Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find a similar feature request

Problem statement
would it be possible to add a restart service option to the admin/user pull down maneu in te upper right corner
Suggested solution
im writing templates and it seems like a cash issue. since after 12 uploads of a template it n longer takes the template but falls back to an older template version. restarting the pc solves this issue but i guess reatrting the service would do the same trick, just don't know how to.
Describe alternatives you've considered
restarting the machine but that is rediculous
Examples of other systems
no where found
Do you want to develop this?

[ ] I want to develop this.

This is a very heavy hammer for a specific (but difficult) problem. Adding a general "restart server" button is not something I want to support - it introduces a lot of security concerns and also we cannot know the specifics of any given installation.
The template cache issue does need addressing, though! I have been stumped on this one for a while now.

sorry not restart server but service

I am not sure we can restart the service from within and adding permissions to do so externally without creating huge security problems is hard. I would caution against quick shots in this regard.


strictNullChecks with control flow in multiple loops throws TypeError

TypeScript Version:
nightly (1.9.0-dev.20160502)
Code
function test() {
	let x: number | undefined;
	while (y()) {
		while (y()) {
			while (y()) {
				x = foo(x);
			}
		}
		x = 1;
	}
}
function y(): boolean {
	return true;
}
function foo(x: number): number { return 1; }

Expected behavior:
Compile error reported on foo(x), as x can be undefined on that location.
Actual behavior:
Type Error:
$ ./node_modules/.bin/tsc --strictNullChecks test
node_modules/typescript/lib/tsc.js:35204
                throw e;
                ^

TypeError: Cannot read property 'flags' of undefined
    at isConstEnumObjectType (node_modules/typescript/lib/tsc.js:21310:24)
    at checkExpression (node_modules/typescript/lib/tsc.js:21721:17)
    at checkExpressionWithContextualType (node_modules/typescript/lib/tsc.js:21669:26)
    at checkApplicableSignature (node_modules/typescript/lib/tsc.js:20309:31)
    at chooseOverload (node_modules/typescript/lib/tsc.js:20606:30)
    at resolveCall (node_modules/typescript/lib/tsc.js:20530:26)
    at resolveCallExpression (node_modules/typescript/lib/tsc.js:20669:20)
    at resolveSignature (node_modules/typescript/lib/tsc.js:20791:28)
    at getResolvedSignature (node_modules/typescript/lib/tsc.js:20808:26)
    at checkCallExpression (node_modules/typescript/lib/tsc.js:20824:29)

Good catch. Surprising we haven't caught this one already. An even simpler repro is:
function test() {
    let x: number | undefined;
    x = 1;
    while (y()) {
        while (y()) {
            x = foo(x);
        }
    }
}

Fix is pretty easy, I will have it up soon.

Thanks for the quick fix! This shows once again that the nightly build can be very useful to find bugs 😃


Feature Request - gradle support

At the moment, all sample projects (e.g. the springboot pet-store) are maven examples.
Is there a plan to add Gradle examples? If so, when will it be available?

Hi @JulianKahnert, I'll look at adding a gradle build file to the applications in the samples folder. Would that be enough?

Hi @sapessi,
thank you for your quick response. Agradle build file and the installation steps to build an AWS Lambda compatiable jar file would be great.

Hey @JulianKahnert, Just committed Gradle build files to the samples in the core branch of this repo. Next, I'll work to include Gradle build files in the archetypes.


Ability to require more than 1 thread for an action

With unsafeExtraThread you can acquire additional threads while running. It might be useful to have a withThreads equivalent that allows more threads to be used by a single rule, e.g. when running make -j as a subaction. See https://github.com/snowleopard/hadrian/issues/289 for an example.

We're using shake to build bootable Linux images, not unlike what buildroot does. There are enough packages to build so that building individual packages with -j1 and letting shake take care of doing things in parallel usually gives pretty good CPU utilization.
One case where it doesn't work so well is for bigger packages like Mesa: When we change Mesa's configuration, the build system will rebuild Mesa and then the packages that depend on Mesa. That rebuild of Mesa will take quite a while, because it's using only one core...
So I wonder what a good API for withThreads would look like? I something that allows to say "I'd like to use at least m and up to n threads would be nice, and withThreads providing the actual number of cores acquired. This would allow that

when building big packages in parallel to smaller ones (like when doing an initial build), Mesa gets a reasonable share of the cores and
when Mesa is the only thing being built it gets all the cores.

Obviously m and n would be capped to the -j parameter given to shake itself.
Another thing that sprung to my mind is that it might make sense to delay building bigger packages until one or more smaller packages are finished, to give them some extra CPU horsepower. But maybe that's too tricky to get right... :-)

Normally when executing an Action you have a single thread. I can imagine two possible API's:
-- run an action with N threads
withThreads :: Int -> Action () -> Action ()

-- run an action with (N..M) threads, saying how many you got
withThreadsRange :: (Int, Int) -> (Int -> Action ()) -> Action ()

I imagine the semantics would be to come out of the thread pool, and add a new entry that requires N threads. Initially I'd probably require the inner Action () to be non-blocking (no calls to need), but that's potentially solveable. For the range, trying to reschedule things to optimise those targets would be infeasible. Having a range at all is possible, but complicates things quite a bit. Is a range really that useful? Why not just say Mesa always takes 4 CPU and have a table of "expensive" builds?

I had a look at implementing this and ran into #823
Relevant GHC issue is https://gitlab.haskell.org/ghc/ghc/-/issues/21136


Rider

Bug Description
I can install mirrord over the plugins in Rider, but it the icon isnt showing up like in Pycharm
Steps to Reproduce

Install mirrord plugin in Rider

Backtrace
No response
Relevant Logs
No response
Your operating system and version
Linux
Local process
no local process
Local process version
No response
Additional Info
No response

Thanks for reporting. Unfortunately, we don't have support for Rider at the moment.
I checked out how we can get this done, and it seems that the Rider SDK is a bit non existent so this might take time to solve. I hope we can prioritize it in few weeks time.

https://github.com/rebel-000/CommandlineArgumentsWCheckboxes/blob/main/src/main/kotlin/com/github/rebel000/cmdlineargs/extensions
might be useful


Adding ballooning support

Please describe the change you are making
Adding support for memory ballooning. It also changes the return of the MemoryPolicy() function as I discovered that it is always present.
Are you the owner of the code you are sending in, or do you have permission of the owner?
Yes
The code will be published under the BSD 3 clause license. Have you read and understood this license?
Yes

can we implement any tests for this ?

done


Improve tests

Coverage increased (+4.8%) to 79.441% when pulling 9868a12c98510af1e54113f17bfacccfc557a8c5 on thiagobustamante:master into 3e78e78e921e31a85791240f85f22f23f9b1b76e on Leanty:master.

Coverage increased (+4.8%) to 79.441% when pulling 9868a12c98510af1e54113f17bfacccfc557a8c5 on thiagobustamante:master into 3e78e78e921e31a85791240f85f22f23f9b1b76e on Leanty:master.


Fix issue 17: support DMD change that makes alias able to take any types, such as delegate types

lgtm


change splitting call to update

fixes #8453

@amaltaro, Alan please review

Thanks Alan for review. :-) Now the patch is tested.
One thing I am wondering is whether we should separate the update and set REST api. I am not sure whether there is the case we want to wipe out splitting parameters and just inject new one.


Possible bug in status code filter

First off, thank you for building this terrific extension!
I'm trying to tighten up my rule for testing for the absence of a CSP header and am running into issues. It was working as expected until I added a filter for status code. No matter what I put in here (single value, comma-delimited list), the check does not fire.
Here's what I've got set:

I haven't had the time to dig through the source yet but wanted to open an issue in the meantime.

Hi @leesoh
Sorry for the delay, I was on holidays.I will look and I tell you.
Thanks!

Solved in new version.
Thanks :)


typo fix in gdb part

Landed as 3a201199902511326a8f4c0e69ee2f3933183203. Thanks for the contribution, and my apologies for my tardiness in incorporating this! This will be released in the next version of bug-clinic, out shortly.


Android game stuck at boot splash screen for OnePlus One A0001

Godot version:
Godot Engine v3.2.stable.mono.official
OS/device including version:
Oneplus One - A0001 / Android Version 9 / LineageOS Bacon
Issue description:
I developed my first game with Godot 3.2. I tested the game (GLES 3) with OnePlus 7T Pro. It works without any problem. All shaders are working fine too.
When i tried older phone like One Plus One A0001, It loaded boot splash screen and stuck there.
I switched from GLES 3.0 to GLES 2.0, it worked except some shaders with TextureLod which is expected.
To be able to test the behaviour on google play with multiple APKs feature:
I created one apk for Armeabi-v 7a and GLES 3.0 higher versionCode
Another apk for Armeabi-v 7a and GLES 2.0 lower versionCode
I created Alpha release and publish both APKs.
When I try to install the game from google play for OnePlus One A0001, it uses apk with GLES 3.0.
Because the phone has GLES 3.0, but it does not work.
Steps to reproduce:
1 - Export the project to android by using the GLES3 renderer
Godot debugging and Logcat do not show any error.
Godot Debugging Output:
Installing to device (please wait...): Oneplus A0001
--- Device API >= 21; debugging over USB ---
--- DEVICE API >= 21; DEBUGGING OVER USB ---
Reverse result: 0
--- Debugging process started ---
Loading Android module: org/godotengine/godot/Firebase
Loading Android module: org/godotengine/godot/SQLBridge
Loading Android module: org/godotengine/godot/GodotPaymentV3
Loading Android module: org/godotengine/godot/GodotShare
Godot Engine v3.2.stable.mono.official - https://godotengine.org
OpenGL ES 3.0 Renderer: Adreno (TM) 330
Logcat output:
2020-02-17 10:24:07.640 652-3584/system_process I/ActivityManager: START u0 {act=android.intent.action.MAIN flg=0x10000000 cmp=[app.package.name]/com.godot.game.GodotApp} from uid 2000
2020-02-17 10:24:07.721 652-1260/system_process I/ActivityManager: Start proc 10804:[app.package.name]/u0a112 for activity [app.package.name]/com.godot.game.GodotApp
2020-02-17 10:24:08.848 10804-10845/[app.package.name] I/godot: Loading Android module: org/godotengine/godot/Firebase
2020-02-17 10:24:08.863 10804-10845/[app.package.name] I/godot: Loading Android module: org/godotengine/godot/SQLBridge
2020-02-17 10:24:08.868 10804-10845/[app.package.name] D/godot: SQLBridge:Initialized
2020-02-17 10:24:08.870 10804-10845/[app.package.name] I/godot: Loading Android module: org/godotengine/godot/GodotPaymentV3
2020-02-17 10:24:08.875 10804-10845/[app.package.name] I/godot: Loading Android module: org/godotengine/godot/GodotShare
2020-02-17 10:24:08.936 10804-10845/[app.package.name] I/godot: Godot Engine v3.2.stable.mono.official - https://godotengine.org
2020-02-17 10:24:08.958 10804-10845/[app.package.name] I/godot: OpenGL ES 3.0 Renderer: Adreno (TM) 330
2020-02-17 10:24:09.039 10804-10845/[app.package.name] I/godot:
2020-02-17 10:24:09.431 10804-10845/[app.package.name] I/godot: Mono: Logfile is: /data/data/[app.package.name]/files/mono/mono_logs/2020_02_17 10.24.09 (10804).txt
2020-02-17 10:24:10.800 10804-10804/[app.package.name] D/FrogSquare: Reading File: res://godot-firebase-config.json
2020-02-17 10:24:12.620 652-1257/system_process I/ActivityManager: Displayed [app.package.name]/com.godot.game.GodotApp: +4s991ms

Because the phone has GLES 3.0, but it does not work.

Many mobile GPUs claim to support OpenGL ES 3.0, but so do very poorly in practice. This is the case of the Adreno 330 GPUs, so I don't think we can do anything about this.

Thanks for reply.
For this type of case, multiAPK feature does not work too. I will try to change my game into GLES2, i will lose some graphics and shader features but it is better than nothing.


Feature/dnssec

Implement DNSSEC in erl-dns.

Closing this PR as this approach is dead. https://github.com/aetrion/erl-dns/pull/59 is the PR that I will merge.


Switched to double quotes

This makes it work in Windows cmd.exe, where single quotes don't escape.

Changes Unknown when pulling 198b5e26cd0770f96b17fed1d919bbfd1f218bc9 on dapetcu21:master into ** on tuvistavie:master**.

Oh, it's been a while I have not been using Windows, so I did not notice at all.
Thanks for the PR, I will bump the package patch version and publish to npm.


Performance issue

First of all, thanks for writing this library :)
We're running into a performance issue that I wanted to alert you to.
We're loading a fairly large (~900KB) JSON schema file, parsing it, and running the resulting JavaScript object through this library's dereference() function.  The dereference() call takes around 68 seconds to complete.
We've disabled all loading of local or external files, so I don't think that any of the time spent in dereference() is due to loading external resources.
Here's basically what the code looks like:
const rawSchema = fetch(...);
const schema = JSON.parse(rawSchema);

if (typeof (schema) !== 'object')
	throw `The schema was not an object.`;

const options = {
	parse: {
		json: false,
		yaml: false,
		text: false,
		binary: false
	},
	resolve: {
		external: false,
		file: false,
		http: false
	},
	continueOnError: false,
	dereference: {
		circular: true
	}
};

console.log(performance.now());

$SchemaRefParser
	.dereference(schema, options)
	.then(dereferencedSchema => {
		console.log(performance.now());
	});

The schema does contain circular references.

I have similar issues with a 4MB json schema file (FHIR).

any updates for this defect ?
I'm also facing it

I encounter this performance problem in case of non direct circular reference
In this simple case, I enter 680 times in the dereference$Ref function
"interface-.ts-69348-69886-.ts-0-209362": {
      "type": "object",
      "properties": {
        "byteLength": {
          "type": "number"
        },
        "slice": {
          "type": "function",
          "arguments": {
            "begin": {
              "type": "number"
            },
            "end": {
              "type": "number"
            }
          },
          "return": {
            "$ref": "#/definitions/interface-.ts-69348-69886-.ts-0-209362"
          },
          "required": [
            "begin"
          ]
        }
      },
      "required": [
        "byteLength",
        "slice"
      ],
      "additionalProperties": false
    },

Can you take a look at this problem ? this is really annoying

Sorry bad example, I didn't pass so much time inside this one.
Try to find a better example

found a fix
It's available in my PR: https://github.com/APIDevTools/json-schema-ref-parser/pull/195
Now performances are really good


Build long prefix package using conda-build 2.x

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

shouldn't this be reported to conda smithy ? next time i'll rerender your changes will be gone

shouldn't this be reported to conda smithy ? next time i'll rerender your changes will be gone

Yes, rerendering will remove these changes, but that is the idea.  Before moving all of conda-forge to using conda build 2.x, packages with short binary prefixes, like swig, need to be rebuild to support long prefixes.  This is being done manually so as to not disrupt the rest of the organization until the switch can be made.
The plan is once most (all?) of the short binary prefix packages have been rebuilt then conda-forge-build-setup will be updated to use conda build 2.x.  Until then conda build will be kept pinned at 1.x
More details can be found in conda-forge/conda-forge-build-setup-feedstock#40.

As Jonathan has already said, this is meant to be a temporary change as part of prepping for switching to conda-build version 2.0.x.
I'm ok with seeing this merged.
Any other concerns @jschueller ? If not, would you like to do the honors?

Thanks all, glad this build made in into conda-forge!


Ford Racing 3: problem with analog triggers and calibration

I'm having an issue with this game on Windows 8.1, 10 and 11:
AFAIK this is a dinput 8 game, so dinput8.dll has to be used in this case. Now, when using dinput8.dll, vibration works fine but triggers won't:
In the game's mapping menu I assign left trigger to break and right trigger to accelerate. The game will recognize Left trigger as "RX Axis" and Right trigger as "RY Axis". So far, so good, cause they're 2 different axes indeed (thanks to "ExtendedGamepad" value in Xidi.ini file). Once you're racing, though, triggers behave as follows:
Right trigger
0 to halfway pressed = break/reverse
Halfway to fully pressed= accelerate
Left trigger
Does absolutely nothing despite being mapped.
I guess all it needs is calibrating triggers and sticks in the game's calibration menu, but here's the issue: the game will crash when exiting calibration menu, with the following message:

JOYSTICK:1574: Failed to set device properties (0x80004001)

Problem when this happens is that changes are not saved and therefore the game will use the triggers as described. When using dinput.dll, though, the game won't crash when exiting calibration menu, but both vibration and analog triggers won't work.
The only setting I've found so far that will kinda fix the issue is changing to "XInputNative" value in Xidi.ini file, and it's fine for the most part: right trigger will accelerate and left trigger will break, just as intended. The only issue is the game recognizes both as part of the same axis, so I can't use both at the same time. I know, I know, who would want to accelerate and break at the same time, right? Sometimes I like to try a few things and also I feel I am in full control of the car, so to speak. It may not be important in this game, but some more technical games/simulators might as well make good use of it.
Do you think this coul be addressed?
I will run as many tests as you require.
Thanks a lot for such incredible tool.

Could you please reproduce the issue with logging enabled and then upload a log file from Xidi? Add this to your Xidi.ini:
[Log]
Enabled                             = yes
Level                               = 4

Also, if you created a custom mapper configuration, could you please share that too?

XInputNative should be exposing separate axes as well (Z and ZRot) for LT and RT, so it's confusing why it would behave that way in FR3.
This may not be your issue, but it is worth noting some old games for whatever reason don't like gas and brake being on discrete axes, and some outright don't allow it (Midnight Club II). Sometimes the merged triggers of XInputSharedTriggers are desirable in this case. It's better to have working controls without the benefit of separated gas and brake, than a car that accelerates on its own or whatever madness occurs.
Also, I'm not aware of a game that REQUIRES calibration to get input to functionally work. It's more of a legacy feature for fine-tuning a gamepad's response in a wild-west era of gamepads. You'd think minimum was minimum and maximum was maximum, and it'd just read it normally, unless it's getting confused about the virtual controller itself.

Another thing about XInputNative and ExtendedGamepad that will probably really confuse old games is that games expect "not pressed" to be neutral, whereas both of these mappers use "not pressed" as extreme negative.
I'm not sure how this game handles separate vs shared axis, but one option for separating them that might be less confusing to the game:
TriggerLT           = Axis(RotX, +)
TriggerRT           = Axis(RotY, +)

What this will do is set LT and RT to report neutral (instead of extreme negative) when not pressed and extreme positive when pressed.


test: add wasm-speed tests

Signed-off-by: Aakash Shukla<EMAIL_ADDRESS>Attached is the run results:

@bianpengyuan @mandarjog could you guys take a look a this and see if it covers your needs?

LGTM @bianpengyuan  ?

I don't think there is any change after my last comment?

Note: we are waiting on long string benchmarks.

added test for large strings and updated the pr to include results of the run

So it's a factor of 10-15x slowdown for large strings?

@Aakash2017 can you paste in text instead of a picture please?

pasted in text

V8 deserialize json is 2x faster than null.

V8 deserialize json is 2x faster than null.

I suspected that that this is an optimization that occurs due to the use of the globally defined google::protobuf::struct args that was filled in the json_serialize_test(). Added additional test for deserializing an empty configuration, and serializing and then deserializing a protobuf. Both of the added tests imply that v8 deserialize json is around 5x slower.

So it's a factor of 10-15x slowdown for large strings?

Looking into the dissembled code for small string and large strings tests to see how compiler optimizations is affecting our benchmarking. I believe that optimizations of the string creation is impacting performance and the xDoNotRemove variable does not necessarily account for this. I added in an intermediate test small_string_compiler_test to show a possible discrepancy, and will update all the string tests to make sure that they are representative of small/large string modification.

Let's add an issue to look into some of these and get this merged.


Security contact information

Hello! Is there a contact address for reporting security issues? I saw that previous concerns were just opened as issues on GitHub, is that the intended venue for reporting?
I also posted this question on the forum, but figure there may be a better chance of reaching a security contact here!
Also, love the keyboards! I have two :)

Please open a ticket here: https://daskeyboard.mojohelpdesk.com/
Thank you!


[WIP] Support multiple views

Here is the initial approach behind allowing multiple views.
I am not happy with the ways I have to avoid making borrowck angry, so that will need to be cleaned up. I think this should be a good general direction though.
This is also waiting on cursor OT.

I think there is an argument to be made about storing the id vs passing the id to different places. I think the less state we store, the better off we will be in the future. I could see storing the view_id on Editor causing some hard to track down bugs.

yea, I do appreciate that general philosophy. I don't have really strong feelings either way; the main thing that influences me is how we have, so far, done some of this state-stashing (around cursor pos, scrolling, and applying edits) so this sort of seems to fall in line with that. 🤷♂

Not sure if I will have the time to continue working on this in the near future.


Not working in Microsoft Edge

It's not working in microsoft edge. Also, sometimes create issues in chrome, safari. Please, help regarding this how I can make it's AJAX request compatible with all browsers. Thanks

I've just tested the demo locally using Edge 16 and it seems to work as expected.
Could you provide more details as to how it's not working? What version of Edge? Are there any errors in the development console?


Merge Work Separating Conda_Steps

As Chris noted in #392, we need to solve this in order to support building environments in parallel.
This PR solves 1) and 3) as noted in that issue, which is to simply:

Build the environment solve

Install the environment

We do this utilizing conda-lock.
The new methods that were introduced (corresponding to this problem) are solve_lock_environment and build_lock_environment, which are called in the larger build_conda_environment function call.
cc: @costrouc

@anirrudh this looks good these two steps. I don't see anything I would request that needs changing. Next step would be to add the downloads of packages

Updates On https://github.com/Quansight/conda-store/issues/392
I was able to get the packages downloading and extracting using filelock. For more information, please consult the changes to build.py.
Currently, the conda prefix is hard-coded inside the function. I looked around the conda module and wasn't able to find a way to get a prefix outside of the activated environment it was launching from while using a python instance, so I'm not sure if there's a better way to get this information. Perhaps I am missing something. Other than that, the function behaves as it should in accordance with the lockfile API.
cc: @costrouc

Cool! Seems like I just missed a couple things - will update and resolve!

Currently, the lock files are left even after exiting the context manager for filelock. I have a few ideas on how to resolve and remove them cleanly, including putting them into a tempdir while we are using this function. That way, they disappear when we exit context.
https://github.com/tox-dev/py-filelock/issues/76 is the best example of this.

Currently, the lock files are left even after exiting the context manager for filelock

Yeup this is expected. https://github.com/tox-dev/py-filelock/blob/main/src/filelock/_unix.py#L44-L46 says there are some race condition issues. I'm completely fine with us leaving these files behind.

Awesome, are we good for merge then or should I also give implement the libarchive conda format as a part of this PR?

@anirrudh I made a few changes. Mainly making sure that we are leveraging the work that the conda folks have been doing with

https://github.com/conda/conda-package-streaming
https://github.com/conda/conda-package-handling


Batarang - TypeError: Cannot read property 'apply' of undefined

opbeat-angular.js:2062Uncaught TypeError: Cannot read property 'apply' of undefined
at ZoneDelegate.invoke (http://localhost:5555/build/js/lib.js:66:10675)
at Zone.run (http://localhost:5555/build/js/lib.js:66:6529)
at ZoneService.runInOpbeatZone (http://localhost:5555/build/js/lib.js:68:30622)
at Object. (http://localhost:5555/build/js/lib.js:67:5697)
at HTMLDocument.maybeBootstrap (chrome-extension://ighdmehidhipcmcojjgiloacoafjmpfk/dist/hint.js:569:11)

@timhc22 ,
Thanks for reporting the issue.
I can see that you have AngularJS Batarang Chrome extension enabled. Unfortunately opbeat-angular has some issues with Batarang.
I will make sure to raise this issue in our planning sessions.
However, Disabling the extension would resolve the issue, if that's possible.

We have this issue too; would really appreciate the conflict being removed.  Especially considering that this would mean that any real users of our product who happen to have Batarang installed won't be able to use our product!

@jdotjdot ,
Thanks for reaching out.
We're working to fix this issue. I will post updates about our progress here.
Also, Just so we're on the same page , This issue should only happen if the The extension is explicitly enabled for your domain name, so even if the user has the extension installed on their browser the issue will not happen until they go to DevTools and enable it for your domain.
Are you experiencing this issue even if the extension is not enabled?

I've still had this issue with the extension installed and enabled globally
in chrome but not enabled for that specific domain name / tab / page.
On Tuesday, November 15, 2016, jahtalab<EMAIL_ADDRESS>wrote:

@jdotjdot https://github.com/jdotjdot ,
Thanks for reaching out.
We're working to fix this issue. I will post updates about our progress
here.
Also, Just so we're on the same page , This issue should only happen if
the The extension is explicitly enabled for your domain name, so even if
the user has the extension installed on their browser the issue will not
happen until they go to DevTools and enable it for your domain.
Are you experiencing this issue even if the extension is not enabled?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/opbeat/opbeat-angular/issues/1#issuecomment-260665509,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABRHQeMJbRZmI7ighg4IHlUAGr3S_oQQks5q-cpDgaJpZM4Kla9Q
.

@jdotjdot ,
Are you seeing the same Error and stack trace as posted here?

Yes
On Tue, Nov 15, 2016 at 10:19 AM, jahtalab<EMAIL_ADDRESS>wrote:

@jdotjdot https://github.com/jdotjdot ,
Are you seeing the same Error and stack trace as posted here?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/opbeat/opbeat-angular/issues/1#issuecomment-260669387,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABRHQaXCjpOQ6Gs1GXNLl_5dunTEw3atks5q-c1qgaJpZM4Kla9Q
.

Thanks!
I will look into it

Any idea on the timeline of this?  It turns out not being able to use batarang is actually really killing my team's productivity, so I'm probably going to have to remove $opbeat until this is fixed.

@jdotjdot ,
We have prioritized this issue.
We probably will have a fix released this week or beginning of the next week.
Cheers

I have just released<EMAIL_ADDRESS>which fixes this issue.
Let me know how it works for you.

looks like it did. thanks!

amazing thanks!!
On 17 November 2016 at 15:01, Morgan Intrator<EMAIL_ADDRESS>wrote:

looks like it did. thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/opbeat/opbeat-angular/issues/1#issuecomment-261353224,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADxPCTRWMDAvxMy5ecrAKex_otyeMTH2ks5q_LKNgaJpZM4Kla9Q
.

--
Unegma LTD
09791695

Amazing thanks :D


I always used npm version to update my semver, so each of my version change has its own commit and a proper tag, which can be proved by  git log, however, when I generate a auto-changelog, all the commits are smashed together, there's no date, no semver, just a long list of commits, what did I do wrong?
I didn't pass in complicated options, all I tried was -r, -p, etc.
The repo in question is https://github.com/ZYinMD/sepia
In fact, it had worked properly in this repo before, for a while, but one day it suddenly changed behavior.
Thanks!

Hey @ZYinMD. Have you recently pulled the repo down to another machine? It looks like you haven't pushed any of your version tags from the machine where you have been running npm version: https://github.com/ZYinMD/sepia/tags
Running git push --tags on your original machine will push all the version tags up. Then git fetch on the new machine will pull them down, and auto-changelog should then work correctly.

@ZYinMD I had a quick look and it looks like your tags are all a bit messed up.
For example, your 1.0.0 commit in the master branch is https://github.com/ZYinMD/sepia/commit/a5b54bba3091e4fa7a4afd75be311b94d3684e9b whereas the v1.0.0 tag is https://github.com/ZYinMD/sepia/commit/49a0aaea2adff377b98150b50d0c9c8ea0958803
This means auto-changelog doesn't see any tags, and so doesn't know where to split the versions.
If you wanted to fix it, you would have to update all of your tags to match the right commits from master, then force push all tags to update them on the remote.
# overwrite v1.0.0 tag with correct commit
git tag v1.0.0 a5b54bb -f

# do the same for all other version commits

# overwrite all tags on remote
git push --tags -f

# re-run auto-changelog
auto-changelog

Ahh! I see what happened!
I thought I was being clever, and I did "postversion": "auto-changelog && git add CHANGELOG.md && git commit --amend --no-edit", so that the new CHANGELOG.md can be published together with the new version, and doesn't create a separate commit.
That of course changed the SHA. *face palm

I fixed it by this:
"postversion": "auto-changelog && git commit -a --amend --no-edit && git tag -f $(git tag | tail -1) HEAD"
Achieved what I wanted, feeling clever again~


New list resource tab does not do pagination and refresh on parent resource refresh

@davidjumani please address for the new list resource tab used in vmsnapshots/backups tabs for VM instance:

Add a watch on resource, refresh the view if parent resource changes (i.e. refresh button)
Add pagination option for the API (see rick account in primate-QA which has over 35 backups for testing)

@blueorangutan package


Reuse code in MiddlewarePipe from Next

I had refactor step by step (commit by commit) so you can reviewing this pr by commits.
By first step I remove duplication of Next and MiddlewarePipe - Next can be deprecated.
Then I replace SplQueue to array - IMO they are fasters however I didn't any benchmarks.
I last change I move Next logic into Handle method - for improved readability.

@weierophinney Following your suggestions I rollback and improve my main change: reuse Next in MiddlewarePipe.
Build fail because __clone is not covered now (Next care about clone queue). We can delete this implementation now, however it can be BC.

@weierophinney I'm waiting for your response here :)


Proof of concept for Firebase support in Flutter framework

Demonstration of basic Firebase support on iOS and Android. Not ready to merge yet.

Just want to say: AWESOME!!

LGTM


FIFO version of SlotMap

According to this issue by @orlp, FIFO version of freelist is a desired enhancement.
I simply implement FIFO by adding a "tail" pointer.
Some unit test and benchmark might be needed, but I'm not sure what we want, so I just leave it here.

I have decided to not go ahead with this for now.

I have decided to not go ahead with this for now.


corrected error in calculation of neutral density from lyman-alpha

Small update in Lya_to_neut_dens which contained the wrong expression for N1 using the adas ADF15 rate coefficients.
Previous expression
N1 = emiss_prof / E_21 / (ne * pec_exc + ni * pec_recomb)
change to
N1 = ((emiss_prof / E_21)- ni * ne*pec_recomb) / (ne * pec_exc)
The overall effect of this error is small except in situations near the wall or diverter.

Thanks, @aarmros ! Great catch.


docs: add a note on byob

fixes #59

let me know if there's any more changes you'd like to see.

Should be good after this


fix about using generic type

Hi @mirromutth,
When I try to compile to resolve issue, I got some errors about generic type.
At first, I tried like T t = (T)options.getValue(option); but still failed with warnings found and -Werror specified. So I suggest this fix. Please let me know if any other ideas you have.

In my opinion, best is ConnectionFactoryOptions supports generic. getValue() returns Object for now.

It was moved and merged as #202 for fix confilicts.


conflict with ppt-slides

@rultor release, tag is 0.4.2

@rultor release, tag is 0.4.2

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 0.4.2

@yegor256 Done! FYI, the full log is here (took me 4min)


reb2sac build failure with linux

reb2sac fails to make from source files, stating undefined reference to "acos" (should be found in libc6). Updating and reinstalling libc6 has no effect.

Further details::
after running: ./configure
whilst running: make
during section:
gcc -g -O2 -o reb2sac absolute_activation_inhibition_generation_method.o absolute_inhibition_generation_method.o (etc etc)
issue in: kinetic_law_constants_simplifier.o: undefined reference to symbol 'acos@@GLIBC_2.2.5'
issue reads:
/usr/bin/ld: kinetic_law_constants_simplifier.o: undefined reference to symbol 'acos@@GLIBC_2.2.5'
//lib/x86_64-linux-gnu/libm.so.6: error adding symbols: DSO missing from command line
collect2: error: ld returned 1 exit status
Makefile:608: recipe for target 'reb2sac' failed

Sounds like "-lm" is missing from the compile line.  I'm not sure why this was not in the generated makefile though.

Thanks for looking into this!

Sorry for delay on this.  I cannot reproduce this.  Can you provide more details?  Which linux distribution and version are you running?  Can you send the entire output log for the configure and make commands?


Next release notes (3.0.1 + 2.7.1)

When you have fixed the 2.7.1 -> 2.8.0 change, you can merge this branch. I have rebase-merged the akka 2.5.6 PR already, so maybe you should remove that commit. (or maybe it will work automatically, not sure)


Sometimes notes are missing location even when enabled

Location tracking on notes works about half the time on two of my devices, when it's missing there is a console error of:
Non-fatal location error Position acquisition timed out
Bumping the timeout to 6000 ms appears to fix this.
https://github.com/open-nomie/nomie6-oss/blob/12f22dbcc32bf19c8674f5588fb0b45ba7df1005/src/modules/locate/locate.ts#L34
Is anyone else experiencing this?
EDIT: Actually on my Android phone 6000 gets a fix ~80% of the time, 9000+ seems to work better from a cold start. I'll try pre-fetching location on initial load as well

I hadn't noticed this issue before. Will try to reproduce ...

I'm able to reproduce this now. The downside of increasing the timeout is that the note potentially waits that long before saving, which makes things feel quite laggy.
It would be nice to save the note immediately and save the location asynchronously, in the background, if necessary. However, I need to get more familiar with the storage implementations before attempting it.

That is true, the many seconds of delay is annoying.

It would be nice to save the note immediately and save the location asynchronously, in the background, if necessary. However, I need to get more familiar with the storage implementations before attempting it.

This is the right way IMO. In the meantime I'm trying a fix where the location is fetched (and cached) on load. So far it's working OK.

Nice, that should reduce the occurrence of the problem.

@qcasey How has the fix been working out for you since - mind creating a PR?

It's been working well for the most part. Sometimes (maybe 1 in 20) the location isn't fetched on load so I end up waiting the full 9 second timeout I configured like before. Still an improvement though.
The image I'm using is on ghcr.io/qcasey/nomie6-oss:fix-location-timeout-longer. I'll create a PR


Multibuffer error after a few hundred messages

Here is the repository that shows this bug: https://github.com/jasonpfi/Boost-Multibuffer-Bug
Please check README.md for more info. I'm using Boost 1.66.0.
Essentially I'm trying to send a few hundred messages very quickly using some python code. The problem is that the multibuffer will simply contain the wrong value at exactly the same point every time.
When I switch my implementation to using flatbuffers, this error does not occur.
I have isolated the error to this very line:
const char* buf = boost::asio::buffer_cast<const char*>(boost::beast::buffers_front(buffer));
but I don't know what's wrong with it. Any insight would be appreciated!

You are looping over 50 bytes, but there is no guarantee that the first buffer in the multi_buffer input area will have 50 bytes. It could be 49. Or 1. Only the flat_buffer guarantees that the input area will be a single buffer. So this code can produce undefined behavior:
    for (int i = 0; i < 50; i++)

You need to check the size of the buffer returned by buffers_front and only loop over those bytes. If the multi_buffer split your message up into more than one buffer, then using buffers_front might not be sufficient to access all the bytes. If you want to access all the bytes you can use the iterator adaptor from Asio:
http://www.boost.org/doc/libs/1_66_0/doc/html/boost_asio/reference/buffers_iterator.html
The multi_buffer is designed for better performance when appending many pieces of small data, and for when the caller doesn't mind that the data is split up into multiple buffers. The flat_buffer is designed for when the caller prefers to have the data in a single contiguous memory buffer, and doesn't mind the additional overhead of the allocate/copy/free cycle caused by appends.

Very cool, that makes a lot of sense. Thanks for the insight.
Closing!


Fixes and refactoring for adding team to a case

Related to: https://trello.com/c/uxlRUoKo/559-5-let-case-owners-give-view-only-rights-to-teams
Description
This is the first refactoring step needed before we can implement view-only collaborations on a case. It does not implement any new functionality yet but it is now possible to add this relatively easily.

Refactored the code to use a form object rather than model validations in order to be consistent with our established patterns related to the use of form/service objects. This allowed for removal of the include_message attribute which was only used for form validation.
Renamed the service to be consistent with naming of other services.
Improved the interface to the service to be more object-oriented
Removed the redundant NotifyTeamAddedToCaseJob which wasn't adding value and was not tested, to improve consistency with other service implementations.
Fixed a bug where a team could be added to a case more than once.
Fixed bugs which would allow multiple case owners.
Added more test coverage.

In a follow-up PR I will address some issues with EditInvestigationCollaboratorForm.

Checklist:

[x] Have you documented your changes in the pull request description?
[ ] Has acceptance criteria been tested by a peer?

I'm testing the bug fixes, please wait till I confirm.


Installing the components

Hi all
I tried to install the MarkDownHelpViewer component like described in the Wiki but an error appears:
FrameViewer10_2;   is required but could not be found.
Can you give me a hint please, what my mistake is?
Thanks a lot
Michael

Wich Delphi version are you using?

Sorry, I forgot
Delphi 10.2 Enterprise

Did you have installed HTMLViewer in your IDE? I've notice a difference on package names and I'll fix it in the next version as soon as possible...

Fixed in 2.3.2 version: please read the updated installation steps here

Thank you very much for yout help. It works now.


illegal instruction

Please make sure that this is a build/installation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:build_template
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):
TensorFlow version:
Python version:
Installed using virtualenv? pip? conda?:
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:

Describe the problem
Provide the exact sequence of commands / steps that you executed before running into the problem
Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

import tensorflow as tf causes illegal instruction. on docker

@faizulhz ,
Can you please provide all the details mentioned in the standard template?Thanks!

@faizulhz if you build tensorflow on latest platform with latest instruction such as AVX512. Then use this version on a older platform without AVX instruction . You may get this error.
Please refer to this link: https://github.com/tensorflow/tensorflow/issues/17411

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


MenuButton not accessible on Safari browser

MenuButton is not accessible on Safari browser with keyboard's Tab key. On Chrome, Firefox & Edge(on MacOS) it can be accessed.
Same issue exists with Button component
https://user-images.githubusercontent.com/18233426/131891074-5d609d00-43eb-423d-8177-0724dee8b36e.mov
https://user-images.githubusercontent.com/18233426/131891102-637565e9-8afe-491b-93ad-452da1b565ca.mov

Hi @gsaran
https://corehelpcenter.bqe.com/hc/en-us/articles/115003481673-Tab-key-is-not-working-on-Mac-Safari
You can resolve this issue by taking these steps:

Open the Safari browser on your Mac.
Select Safari > Preferences > Advanced.
To make it easier to navigate links and form elements, check the option 'Press Tab to highlight each item on a webpage.'


RSS feed for blog

Calling this a "bug" is a bit borderline, but it's a regression from the previous version so I think it's appropriate.  The blog should have an RSS feed, preferably at the same URL (https://typelevel.org/blog/feed.rss) as the old one.
It isn't clear offhand where the old feed came from.  Was it generated in post-processing?  Did it somehow come out of the server?

The old feed is just a Jekyll generated file that's pretty straightforward:
https://github.com/typelevel/typelevel.github.com/blob/9c2be08b3217ca6c1a9a65e3832ab47131e8f5fb/blog/feed.rss
Alternatively, you could use a plugin. The following looks nice, but note that it generates Atom feeds, and the old one was RSS 2.0:
https://github.com/jekyll/jekyll-feed


bits.Clz, bits.Ctz functions are slower than equivalents in Go 1.9 math/bits package

Since Prometheus is typically built using Go 1.9, and it is (so far) the primary user of this (tsdb) package, it would make sense to use the faster LeadingZeros64 and TrailingZeros64 functions from math/bits.
A benchmark shows them to be roughly 7 times faster.
$ go test -test.bench=.
goos: linux
goarch: amd64
pkg: github.com/dgryski/go-bits
BenchmarkClz-8<PHONE_NUMBER>	         2.14 ns/op
BenchmarkCtz-8<PHONE_NUMBER>	         2.14 ns/op
BenchmarkLeadingZeros64-8<PHONE_NUMBER>	         0.31 ns/op
BenchmarkTrailingZeros64-8<PHONE_NUMBER>	         0.31 ns/op
PASS
ok  	github.com/dgryski/go-bits	6.042s

I'd be fine with that. Wanna send a PR?

PR #198


Enhancement: Add remBADGER and remDIGG to Boost

The remediation assets remBADGER and remDIGG need to be added to boost.
1 remBADGER = 1 * remBADGER.ppfs() in boost value
1 remDIGG = 1 DIGG

remBADGER Address: 0x6af7377b5009d7d154f36fe9e235ae1da27aea22
It is confirmed deployed on registry: https://etherscan.io/tx/0x0bae0ccda349e08661c29c3307e667455d34043b8b5b0291fe47a84b97293558

closed by #557 for remBADGER, will open separate issue for remDIGG


server public ip attach problem

Been a server admin since early 2000s (started CS in 1999) and, for whatever reason, the CS2 binary binds to <IP_ADDRESS> on UDP instead of the specific -ip address specified in launch options. This worked with CS 1.6 but does not work in CS2.
udp 0 0 <IP_ADDRESS>:27015 <IP_ADDRESS>:* 1156703/./cs2
how can i fix this problem?

If you aren't using host network mode (--net=host). Then you can pass the IP to the publish parameter -p:

If you don't specify an IP address (i.e., -p 80:80 instead of -p <IP_ADDRESS>:80:80) when publishing a container's ports, Docker publishes the port on all interfaces (address <IP_ADDRESS>) by default.

Source
This will have the same result as using the start parameter on the binary.


GUI  Input Area of a LCD Config is to small for bigger Displays

## Problem summary ##
GUI Area for LCD Diplays is to small for a 40x4 Display
## MF Version ##
7.10.xx
## Steps to reproduce ##

Create a LCD Display as Device.   ( USer said up to 40x2 is possible in the Library and also in the MF alowed Settings

Create a Config and check the INPUT Area we can insert the LCD Data….. Its not wide enough to show all 40 Positions

## Expected result ##
The GUI should support the upper limitation of the Firmware.
## Further details ##
I would say a enlarge (whatever imporvisional)  should be done a.s.a.p. within the next hotfix/release.   Not care about good optics.  We can perfectly solve this with the planed GUI rework in the future.
Alternate:  If its not possible ( or not in your opinion) then we should limit the LCD Size in Firmware a little smaller to fit the GUI current Maximum.
Related Forum Topic
[Issue created by Pizman: 2021-01-27]

we don't support 40x4... LCDs can only have max 80 characters


[WIP] fleetd: register heartbeat only for the first attempt

When beating the Heart for the 1st time, call Heart.Register() to avoid such a case of registering machine with the same ID. Starting from the next heartbeat, however, call Heart.Beat() to allow registration with the same ID. That way fleetd can handle the machine presence in a graceful way.
Suggested-by: @wuqixuan
Fixes: https://github.com/coreos/fleet/issues/750
This PR depends on #1561.
NOTE: this is still experimental. do not merge it yet please!

@dongsupark this sounds wrong to me: doesn't it mean that the followup attempts will "steal" the ID from another instance if there is a conflict?...
(Haven't really thought it through in detail though -- so I might be something here...)

A bugfix for a regression in fleetd was already included in #1561. Anyway this PR #1563 does not seem to correctly fix the original issue.
Closing.


🐛Update "capture" attribute to enumerated values

The capture attribute can be both a boolean and enumerate.  This work adds type CameraCaptureEnum = "user" | "environment";
and amends the capture property of EventListenerOptions
docs:
https://www.w3.org/TR/html-media-capture/#the-capture-attribute

Fixes #

Files from lib folder are generated from src/lib folder. Some files from src/lib are generated from TSJS-lib-generator repository.
Read Contributing lib.d.ts fixes for more info.

Silly me, read guidlines


Extract sort case insensitive code

Automated message from Dropbox CLA bot
@borisyankov, it looks like you've already signed the Dropbox CLA. Thanks!


Implement buffer textures

See https://www.opengl.org/registry/specs/ARB/texture_buffer_object.txt

Creating a shader resource view of a buffer gives us exactly this. Marking as done.


Sum not being returned in profiles by default

General Information:

OS:  MacOS Cataline 10.15.7
Python version: 3.7.10
Library version: 0.5.1

Describe the bug:
sum is not being returned by default for numerical columns. I'd understand if it were for performance reasons, but it appears that it is still being calculated since it has an entry in the times dictionary.
To Reproduce:
 import pandas as pd
import dataprofiler
from dataprofiler import Profiler, ProfilerOptions

df = pd.DataFrame([1,2,3], columns=["a"])

profile_options = ProfilerOptions()

profile_options.set({
    "data_labeler.is_enabled": False,
    "histogram_and_quantiles.is_enabled": False,
})

profile = Profiler(data=df, options=profile_options)

report = profile.report()

assert "sum" in report["data_stats"]["a"]["statistics"]

Expected behavior:
Numeric columns should return sum by default in reports.

PR merged.


Feature request: propose usage of imapM

Could HLint detect cases like
map f $ zip [0..] list
and propose to use imap(M) instead?

I am not a fan of pointing people at the lens library unless they were already using the lens library before. Personally I'd use zipWithFrom f 0 list there which is a lot more monomorphic than the imap is.

Such a suggestion is fine as well.
Although it doesn't hide that indices start at zero.

I'm not sure I should make either, since I generally try to avoid going from a situation where you have to import an extra package - and both those suggestions would do that. (I personally think that making the start index explicit is useful, but it's more personal preference.)


fix: video playback duration value glitching, fix VideoPlayExample

⚠ I might be doing it as I'm unsure if _hasUpdatedTexture = false; removal has no side-effects (I didn't get any)
The Duration was glitching alternating -1 and correct Duration, when the Update was called twice during the same frame (this happens when video file is using <60fps)

I also adjusted few stuffs in PlayVideoExample, it now works correctly at least with files found in Resources\common\test-footage

Good catch. I frequently check the context.LocalTime to test if the operators has already been updated in the current frame.  The _hasTextureUpdated was an attempt to only execute some compute shaders if the texture actually changes. This is relevant when apply stuff like motion vector extraction to videos with framerates like 25fps or if the current display rate is not 60 fps (like note books with 144 FPS refresh rate).


Purchase a Premium Licence from a distributor

Hi,
We are an MS Partner located in Algeria and we wanted to purchase a Power BI Premium Licence from our local Microsoft Distributor however the Distributor indicates that the SKU is not on his price list; What is the solution ?
Regards
Mehdi HAMMADI

Hi @HappymediaDz,
Please ask your local Microsoft Distributor to get in touch with our marketing Power BI product marketing manager Megan Tomlin.
As this is a documentation channel and your question isn't related to documentation, I'm closing the GitHub issue.
Thanks,
Kesem


enqueue/enqueue-bundle minimum-stability issue

Hello,
Tried to install job queue pack but it fails due to composer minimum-stability.
I think we can resolve this with replace enqueue/enqueue-bundle's version range with ^0.9  instead of 0.9.x-dev version.
composer req enqueue/enqueue-bundle enqueue/job-queue-pack
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Installation request for enqueue/job-queue-pack ^0.9.0 -> satisfiable by enqueue/job-queue-pack[0.9.0].
    - enqueue/job-queue-pack 0.9.0 requires enqueue/enqueue-bundle 0.9.x-dev -> satisfiable by enqueue/enqueue-bundle[0.9.x-dev] but these conflict with your requirements or minimum-stability.

Yes, please.


Worker not getting invoked after scheduling a request

Blocks https://github.com/google/ground-android/issues/1080

@gino-m Assigning this bug to you as you're already working on the related piece of code.
https://github.com/google/ground-android/pull/1616


Bug issue

@Ankur0812 Closing this PR as it is not up-to-date with master and in favor of https://github.com/plausible/wordpress/pull/77


Server App & Much Changes

Summary by CodeRabbit

New Features

Introduced a new API for newsletter subscriptions.
Added a new API for contact submissions.
Enhanced the prerendering of multiple pages for improved performance.

Bug Fixes

Corrected typographical errors in component names.

Improvements

Simplified error handling and feature flag checks across several components.
Updated component loading behavior for better performance.

Documentation

Updated error handling logic to provide clearer user redirection in case of feature flag issues.

@coderabbitai summary


Add hint about browser compatibility

There isn't a documentation about the browser compatibility. What I expect:

From which browser and version is the plugin compatible?
Is the plugin able to save notes when using it from a webpage with the file:///-protocol?

Added browser compatibility in README.md.
https://github.com/txusko/PostItAll/commit/fce6082a61d58797a6e9b18fb2a53f5ef2036500
It seems that don't work on Opera, and I have don't tested yet on Explorer and Edge.


fix(pwm): Limit pwm re-export to unproven

The SAMD51 and SAME54 HALs re-export the pwm module to avoid breaking
the previous, public API. The pwm module is currently unproven, but
the re-export occurred regardless, creating an error when building
without the unproven feature. Add a cfg attribute to limit the
re-export to builds with the unproven feature.

Could we add a build configuration to CI to catch this from happening again?

I think that's a good idea, but I don't know the CI system. I also don't have any experience with Rust CI yet. I think it would be a lot easier and faster if one of the maintainers made that change.

I think we probably could, but I'm not sure it's worthwhile at this point. embedded-hal==1.0.0 gets rid of the unproven feature altogether.


silences: avoid deadlock

Calling gossip.GossipBroadcast() will cause a deadlock if
there is a currently executing OnBroadcast* function.
See #982

LGTM


Add max_file_lines option back

Not sure if this was removed on accident or on purpose, but it is still referenced in the README.

Can you change the comment in the readme to make it clear that the default value is nil? It looks like the default is 1000

@p00f #61


WHAT DO WE CALL THIS THING?

I think having an org from the start is a better idea. I have all of these

@compiled/css-in-js 👈 fav atm. it's literally what it is
@compiled/css
@puck-pkg/css
@compiled-css/core

settling on @compiled/css-in-js