feature: GraphQL execution for @defer support

This PR is to merge the @defer required changes to the GraphQL executer into main. There is already some of the @defer work in the main branch but we chose to build the executor changes on a separate branch in isolation not to cause any instability to main.

@calvincestari
I'd like to provide feedback about the the way I would expect to be able to leverage the @Deferred property.
The ability to introspect the status of pending vs final is useful, however as an end user I would like to be able to await on a pending state and eventually receive the final result.
pseudo example taken from here
      expect(animal.fragments.$deferredGenus).to(equal(.pending))

      let genus = try await animal.fragments.$deferredGenus

      expect(animal.fragments.$deferredGenus).to(equal(.fulfilled)) //its now finished
      expect(animal.fragments.deferredGenus).toBeNonNil() //its non nil
      expect(animal.fragments.deferredGenus).to(equal(genus))

NOTE: I'd expect something similar with @stream where I could receive an AsyncStream

Hi @john-twigg-ck, thanks for the feedback.

The ability to introspect the status of pending vs fulfilled is useful, however as an end user I would like to be able to await on a pending state and eventually receive the final result.

The implementation of @defer is built on the existing network stack which unfortunately does not have any async/await support. So awaiting on the status won't be possible in 1.x. @AnthonyMDev is currently working on Swift 6 compatibility which will be a major version bump, and will include proper support for async/await too so it's possible that in the future we could get that working but it's still got some challenges though due to the way that the models and data is handled, i.e.: models and their data are not persisted so there is nothing to 'update'.

NOTE: I'd expect something similar with @stream where I could receive an AsyncStream

There's only bad news here I'm afraid as unfortunately there is no existing support for @stream, and it's not in this implementation either. I don't think any of the Apollo clients support @stream nor does the Apollo Router. Eventually when the @defer/@stream proposal is merged into the GraphQL spec we'll have to provide support for it.


Error creating index when db does not exist

Bit of an edge case this, but...
Calling ensureIndex on a Datastore gives an error when all the following are true:

The Datastore is not in-memory
The Datastore has not yet been loaded (loadDatabase may have been called, but the executor has not yet got round to carrying it out)
The file for the database does not exist on the filesystem
The directory of the database file does not exist

The error comes about because calling ensureIndex results in a (synchronous) call to Persistence.prototype.persistNewState, which in turn calls fs.appendFile, all before Persistence.prototype.loadDatabase has been called. This may ordinarily go unnoticed because if 4. above is not true then fs.appendFile will actually create the db file.
I've spent a while thinking of a solution to this, but I can't come up with anything that keeps  ensureIndex synchronous (short of synchronously checking that the db file exists every time it is called).

fyi, this is my workaround by running fs.mkdir once if the flag local.swgg.modeNedbInitialized is not set
        local.swgg.collectionCreate = function (schemaName) {
        /*
         * this function will create a persistent nedb-collection from schemaName
         */
            if (!local.swgg.cacheDict.collection[schemaName]) {
                if (local.modeJs === 'node' && !local.swgg.modeNedbInitialized) {
                    // init nedb dir
                    local.fs.mkdir('tmp', local.utility2.nop);
                    local.fs.mkdir('tmp/nedb.collection', local.utility2.nop);
                }
                local.swgg.modeNedbInitialized = true;
                local.swgg.cacheDict.collection[schemaName] = new local.swgg.Nedb({
                    autoload: true,
                    filename: 'tmp/nedb.collection/' + schemaName,
                    timestampData: true
                });
            }
            return local.swgg.cacheDict.collection[schemaName];
        };


Update dependencies to use last version of inappwebview

Update

Flutter 3.24.3
archethic_lib_dart
web: ^1.0.0
web_socket_channel: ^3.0.1
test: ^1.25.8

done in version: 2.1.6


Use common naming scheme for RST snippet files

Summary
RST snippets are used to reduce repetition of the same messaging (e.g., call to action for Slack's #support channel). Use common naming convention for these files.
Changes

[x] Update file name and extension for the RST snippets

Testing
Locally verified that the documentation builds successfully. Pure docs update, so no other test needed.
Tickets
NA

closing as not urgent. what we have works just fine.


Unable to clean.bash on darwin.

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go version go1.7 darwin/amd64
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/mike/workplace/GOPATH"
GORACE=""
GOROOT="/Users/mike/workplace/go"
GOTOOLDIR="/Users/mike/workplace/go/pkg/tool/darwin_amd64"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/z4/vplvdvkx6fsfv7z6w8pzfqnr0000gn/T/go-build489990318=/tmp/go-build -gno-record-gcc-switches -fno-common"
CXX="clang++"
CGO_ENABLED="1"

What did you do?
If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
A link on play.golang.org is best.
run clean.bash in $GOORT/src
What did you expect to see?
I want to use source code in my darwin to build on centos, I want to run clean.bash to make a clean directory to build a new go binary. (For some cgo headers can only be built on Linux).
What did you see instead?
go: open /Users/mike/workplace/go/src/runtime/internal/sys/zversion.go: no such file or directory

You do not need to use clean.bash. Use make.bash
On Fri, 9 Sep 2016, 14:56 Peng Gao<EMAIL_ADDRESS>wrote:

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go version go1.7 darwin/amd64
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/mike/workplace/GOPATH"
GORACE=""
GOROOT="/Users/mike/workplace/go"
GOTOOLDIR="/Users/mike/workplace/go/pkg/tool/darwin_amd64"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/z4/vplvdvkx6fsfv7z6w8pzfqnr0000gn/T/go-build489990318=/tmp/go-build -gno-record-gcc-switches -fno-common"
CXX="clang++"
CGO_ENABLED="1"
What did you do?
If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
A link on play.golang.org is best.
run clean.bash in $GOORT/src
What did you expect to see?
I want to use source code in my darwin to build on centos, I want to run
clean.bash to make a clean directory to build a new go binary. (For some
cgo headers can only be built on Linux).
What did you see instead?
go: open /Users/mike/workplace/go/src/runtime/internal/sys/zversion.go: no
such file or directory
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/17035, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAcA2828aOpy4c7wI8by1X1OT5E6iiCks5qoOcMgaJpZM4J4s2p
.

Yes, I got this error first, that's why I try to run clean.bash

I'm pretty sure you working directory contains a mixture of go versions. I
recommend deleting your checkout and starting from a clean checkout.
On Fri, 9 Sep 2016, 15:04 Peng Gao<EMAIL_ADDRESS>wrote:

[image: image]
https://cloud.githubusercontent.com/assets/4769989/18376012/9d9ceba2-768d-11e6-8fed-0657aaa9a5a4.png
Yes, I got this error first, that's why I try to run clean.bash
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/17035#issuecomment-245820328, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAAcA7C6R5_vouogQ1eXWM4gzAJb5aXIks5qoOjpgaJpZM4J4s2p
.

I download a go1.5 linux binary for GOROOT_BOOTSTRAP, and run GOROOT_BOOTSTRAP=go1.5path ./make.bash
I use git to clean the directory git checkout . , but still get this message.

I use git to clean the directory git checkout . , but still get this message. (On centos)

Your working directory is not clean. Check git status and be aware that some files may be hidden by .gitignore. I recommend you delete your checkout and make a fresh checkout.

Let's move this to a mailing list.


Document custom Kafka client metrics on otel.io

Context

New Kafka metrics design is outlined in KIP-714
It's not OTel-compatible and uses different names/attributes/instruments than otel
Metrics are intended to be collected by default, serialized to OTLP and sent to broker from which they can be collected by users - impl in java

Effectively, Kafka project does not plan to follow OTel semconv (@AndrewJSchofield to confirm)
OTel provides several kafka instrumentation components:

Some of them are based on monkey-patching/byte-code rewriting and can emit otel-compatible metrics and traces
Others (such as collector kafkareceiver or .NET Aspire Kafka integration library)  scrape metrics that broker provide

The problem:
Group 1 (monkey-patched instrumentations) might still want to emit kafka-specific metrics/traces. We'll need to keep them in otel-semconv repo so they are consistent across languages/clients.
Group 2 (instrumentations that report what's available) have more difficult problems:
There are multiple ways to scrape different sets of metrics from Kafka:

Java uses Kafka JMX metrics
collector uses Kafka client library APIs to get stats
.NET Aspire component uses metrics available through underlying librdkafka
Once KIP-714 is implemented in different langauges, there will be yet another way

These metrics in most cases can't be converted to OTel ones (use different instruments, don't support histograms, don't report the same attributes, etc).
As  a result, we're going to end up with each language SIG (plus external components) defining their own set of custom metrics for Kafka based on what they have.
What we can do on otel semconv side:

recommend a default way to scrape metrics from kafka
recommend naming approach (e.g. Java and collector start with kafka and not messaging.kafka as we do in OTel semconv)
provide a single place to list all existing non-otel-compatible kafka-related metrics (e.g. docs in opentelemetry.io) similar to JMX, but for all languages

Given we removed them from conventions here open-telemetry/semantic-conventions#338, do we really need to do anything here?

@joaopgrassi we still want to document them and the semconv WG decision was to have an informative section on opentelemetry.io, so I transferred issue

Thanks for transferring this issue @lmolkova . Since this is a first instance of something like that being documented, we need to figure out where and how to put this within the docs. To be honest right now I am not sure what the best place will be, any suggestions?

@svrnm I wonder if we can add a page under Semantic Conventions, something like "External conventions" where we would be able to provide documentation about non-otel-authored/compliant signals Otel collector/instrumentation libraries emit.
E.g.
Semantic Conventions
External Conventions
Kafka
I believe there are more candidates to be in that folder (looking into collector receivers, there are plenty of scrapers (Redis, RabbitMQ, ...) that don't always document metrics.  Ideally, we want them to at least add a link to external documentation.
As an alternative, we could consider adding a section under "Collector" since most of this external conventions will come through it and then, in rare cases they are needed outside of the collector (like in java-instrumentation), we could just link the section in the Collector.
WDYT?

Why aren't they following semconv?

I would prefer this:

add a page under Semantic Conventions, something like "External conventions"

Since it's consistent with where we keep naming for common components.

Why aren't they following semconv?

Kafka specific ones we want to find home for are legacy ones from pre-otel world (which Kafka owners AFAIK want to preserve for the time being).

Why aren't they following semconv?

Kafka specific ones we want to find home for are legacy ones from pre-otel world (which Kafka owners AFAIK want to preserve for the time being).

Ah, ok.

Kafka specific ones we want to find home for are legacy ones from pre-otel world (which Kafka owners AFAIK want to preserve for the time being).

Is there a discussion that we can reference for that? Or, asked differently: have we (opentelemetry community) actively engaged in a conversation with them (kafka community) if this is they right way forward? Not that we can tell them what to do but we can at least help (if wanted) to make an inform decision


Remove src_dir from .coveralls.yml

The src_dir option is no longer available:
https://github.com/satooshi/php-coveralls/pull/136
Before this PR, php-coveralls throws an exception:
https://travis-ci.org/chinpei215/cakephp-eager-loader/jobs/101469368#L296 (Before)
https://travis-ci.org/chinpei215/cakephp-eager-loader/jobs/101470617#L297 (After)

Thx, I had the same issues.

Thanks.


Multistep Multivariate Time Series

Hi,
I am building a forecast model by using LSTM through DLPY framework. My model is built, however, when it comes to extending the forecast beyond test data, it is throwing an error.
I am using this statement "forecast_tbl = model1.forecast( horizon=400)"
Does it mean that I need to always provide a dataset to get the extended forecast in case of multivariate situation? (I get an output if I provide a test dataset in model1.forecast, however, I am not able to extend the forecast beyond the index of test_data)

Can you share the notebook if possible? It should work with multivariate if you follow all the steps from https://github.com/sassoftware/python-dlpy/blob/master/examples/time_series_forecasting/Multisteps_vs_Onestep_ahead_Forecasting.ipynb to prepare the data and train the model.

Can you share the notebook if possible? It should work with multivariate if you follow all the steps from https://github.com/sassoftware/python-dlpy/blob/master/examples/time_series_forecasting/Multisteps_vs_Onestep_ahead_Forecasting.ipynb to prepare the data and train the model.

Infact, I tried replicating the example shared in the link. However, if i try to have an extended forecast beyond the length of test data, it throws an error. (i.e. model.forecast(horizon=400) ) in case of multivariate forecasting. Please find the notebook
Test.zip

OK. I checked the source codes as well as the API doc:
test_table : string or :class:CASTable, optional
Specifies the test table. If test_table=None, the model cannot have
additional static covariates or predictor timeseries, and can only
be a autoregressive model. In this case, the forecast extends the
timeseries from the last timestamp found in the training/validation set.
If the model contains additional static covariates or predictor
timeseries (that are available for predicting the target timeseries),
the test table has to be provided, and the forecast starts from the
first timestamp in the test data. If the model is autoregressive, and
the test data columns do not include all the required preceeding
time points of the target series (the lagged target variables),
the forecast will be extended from the last time timestamp in
training/validation set and only use the static covariates or
predictor timeseries information from the test data if they are
available for the corresponding time points.
Default : None
Yes, you have to specify test_table and it can only predict when the table contains the input features for all your time steps.

OK. I checked the source codes as well as the API doc:
test_table : string or :class:CASTable, optional
Specifies the test table. If test_table=None, the model cannot have
additional static covariates or predictor timeseries, and can only
be a autoregressive model. In this case, the forecast extends the
timeseries from the last timestamp found in the training/validation set.
If the model contains additional static covariates or predictor timeseries (that are available for predicting the target timeseries), the test table has to be provided, and the forecast starts from the
first timestamp in the test data. If the model is autoregressive, and
the test data columns do not include all the required preceeding
time points of the target series (the lagged target variables),
the forecast will be extended from the last time timestamp in
training/validation set and only use the static covariates or
predictor timeseries information from the test data if they are
available for the corresponding time points.
Default : None
Yes, you have to specify test_table and it can only predict when the table contains the input features for all your time steps.
Thanks for the response.
Does it mean that I can not extend the forecast horizon beyond the length of the test_table?

No, you cannot extend the horizon beyond the length of your test_table. The input features have to be computed first and stored in the test_table.

I am closing this issue, @pynyah please ping us back if you have further questions. thanks


adding dashboard config content

@asheet Document looks good to me. Could you please move this section after custom notebook and before the model serving?

@asheet Document looks good to me. Could you please move this section after custom notebook and before the model serving?

updated


Define new BaseAreaManager and create custom grid version of it

In order to keep chunks smaller, i'll add them in such small updated.
For now I focused on supporting the bare minimum, and things will only be added if there is no better way of doing it.
I decided to transition grids from lists to dictionaries, because it is now impossible to have two grid files for the same CRS.
the biggest issue I have is how to name the column with eopatch names in the grid representation. I was considering just fixing it to eopatch_name so that the rest of the code does not need to use self.area_manager.NAMES_COLUMN

Codecov Report
Base: 84.45% // Head: 84.72% // Increases project coverage by +0.26% :tada:

Coverage data is based on head (1860e36) compared to base (04dad2f).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@                   Coverage Diff                   @@
##           area-manager-rework     #148      +/-   ##
=======================================================
+ Coverage                84.45%   84.72%   +0.26%
=======================================================
  Files                       49       49
  Lines                     4039     4097      +58
=======================================================
+ Hits                      3411     3471      +60
+ Misses                     628      626       -2

Impacted Files
Coverage Δ

eogrow/core/area/base.py
98.06% <100.00%> (+0.56%)
:arrow_up:

eogrow/core/area/custom_grid.py
98.38% <100.00%> (+0.95%)
:arrow_up:

eogrow/core/logging.py
98.65% <0.00%> (+1.34%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


⚠ Fetchers has degraded performance

In 43269f7, Fetchers ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 136 ms

Resolved: Fetchers performance has improved in bc14c5c.


Style/ArgumentsForwarding is detecting false positives

I am getting false positives, which if autocorrected break the code
Here is an example:
def before_transition(options = {}, &block)
  add_callback(callback_type: :before, callback_class: Callback, from: options[:from], to: options[:to], &block)
end

when I run rubocop I get the warning:
Style/ArgumentsForwarding: Use shorthand syntax ... for arguments forwarding.

Which seemed incorrect to me, but just in case I was missing something I tried running autocorrect which produced:
def before_transition(...)
  add_callback(...)
end

Which is completely broken and a change of behaviour.

Expected behavior
No warning raised
Actual behavior
Raised a false positive
Steps to reproduce the problem
As above
RuboCop version
$ [bundle exec] rubocop -V
1.60.0 (using Parser <IP_ADDRESS>, rubocop-ast 1.30.0, running on ruby 3.2.1) [arm64-darwin22]
  - rubocop-performance 1.20.2
  - rubocop-rspec 2.26.1

I have kind of similar problem:
def view(path = nil, options = {}, &block)
  cache = options.delete(:cache)
  cache = config[:environment] == 'production' if cache.nil?
  template = Flame::Render.new(
    self,
    (path || caller_locations(1, 1)[0].label.to_sym),
    options
  )
  template.render(cache: cache, &block)
end

lib/flame/controller.rb:128:12: C: [Correctable] Style/ArgumentsForwarding: Use shorthand syntax ... for arguments forwarding.
		def view(path = nil, options = {}, &block)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
lib/flame/controller.rb:136:20: C: [Correctable] Style/ArgumentsForwarding: Use shorthand syntax ... for arguments forwarding.
			template.render(cache: cache, &block)
                   ^^^^^^^^^^^^^^^^^^^^

How should I replace there something via ...?
RuboCop v1.60, Target Ruby version is 3.0.

The issue is caused by the same reason as #12618 and is planned to be resolved in #12624.


Node can't start normally

select : 3) Create ChainDB from own NKN node and host on the SAME server
After the task is completed, the node is in the OFF state and cannot be started.

My install of a node and generating a ChainDB went without problem on a aarch64, ubuntu 20.10 install on AWS. It started no problem.
Please write "arch" without "" in your terminal and let me know the output.
Please post your log with error

heh interesting... are you sure you're putting the right IP into nstatus? hahah

try another browser and incognito for nstatus.org and try again

also did the node work before you tried Creating the ChainDB?

The node worked properly before ChainDB was created


New feature - side bar

Add option to create a collapsible sidebar to XOT template projects. The sidebar would contain various buttons which would normally appear on the footer bar:

Table of Contents
Accessibility Options
Glossary
Help
Project Introduction
Page Introduction

Alternatively, the sidebar could contain a table of contents for the project where you can see the structure of the project with ticks for viewed pages.

@FayCross sounds good and I'll reply to the separate email discussion but adding a few comments here first...
Perhaps one or more custom buttons would also be useful e.g. text box content with option to choose icon and label and perhaps url and target to be able to display other things in addition to what you have already listed.
Option to have the sidebar appear right or left and perhaps even top or bottom too.
Option to replace the footer and so automatically include the navigation buttons in the sidebar or not.
We would obviously need to add relevant styling for the different bundled themes and defaults and guidance for those with custom themes.

@ronm123 yes - I have the right & left option already working! I like the idea of adding custom buttons but I can't work out the best way to do them in the editor as ideally you'd be able to add as many as you like and have different options for what happens when you click them - text, URL, PDF. I guess using the datagrid in some way with a row for each custom button - but it's complicated because you might want them to upload files etc. rather than just have text box to enter content. There's lots to think about

@FayCross yes I can see that's a bit complicated. As there would be a limit to how many buttons are practical anyway what about just a fixed set of perhaps 4 or 5 optional buttons that don't appear if not used?
Also is the sidebar permanently displayed or pops out on hover etc?
I also wondered if the contents might change as you navigate so buttons that appear/change on a page by page basis like Transcript for example.

I've committed the first version of this but there's still work to be done. Including:

[ ] Get working on all themes (only tested on default theme so far)
[ ] Text /button width issues if buttons are set to show label and the label is quite long
[ ] Look at whether z-index should be higher than lightboxes so the side bar controls are still visible when project intro etc. are open


Launcher firmware: OTA update

It works for now, having a better method for dealing with root certificate changes is something we have to think about for the future.


Blue when selected

Histograms and categories become blue when being selected. This is expected to improve the speed on detecting that a filter is being applied.

@javierarce @xavijam please CR
@javisantana you can also do it just for fun.

Tests failing :)

@xavijam it's lint! they pass all in local. What should I do?

Run in the terminal, grunt test, it will warn you about linting and tests problems.

tests are passing now

:+1:

If you want me to :trollface:, I miss a spec 💃

waiting for @javierarce while I add the spec

Fine by me, what do you think @javisantana?

:es:

zzzzZZZZzzzz, can you guys merge this?

hey, you merge your PR


k8s-reporter: Stop dumping unreachable container's info

What this PR does / why we need it:
In order to keep to logs cleaner and more informative,
refrain from printing the same error again again,
and also save time if the container / pod aren't
reachable (instead of running more commands).
Additionally - Fix logic of checking if the virt-handler's compute container is healthy,
not just exist.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #7799
Special notes for your reviewer:
Release note:
None

/cc @phoracek @qinqon

/lgtm

Addressed comments by checking that the pod / container are healthy,
fixed another place that checked only that the compute container exists, but didn't check it is healthy

/retest

Rebased

Is this ready?

Is this ready?

Basically yes, i want to do few more tests, once it will be ready i will undraft

Update inner pod variable to not override the outer one, so if there is a Get error, the outer won't be affected,
as it is still being used

/approve

Thanks Alona
@qinqon can you please review again ?

/retest

/cc @EdDev

/lgtm

/retest


AttributeError: module 'asyncio' has no attribute 'coroutine'

With update to HA to 2023.6, there was a breaking change introduced that may be related to python.   See the issue here in r-renato's repository.  I'm posting this here as this fork is more actively maintained.  Thanks!
https://github.com/r-renato/ha-climacell-weather/issues/73

unfortunately this repo doesn't seem to be any more active than the original

With update to HA to 2023.6, there was a breaking change introduced that may be related to python. See the issue here in r-renato's repository. I'm posting this here as this fork is more actively maintained. Thanks!
r-renato#73

Can you test the latest release? I dont really use this integration anymore.


Unit test: Unfunded accounts can now issue transactions

@sczembor Ready for another review.

@sczembor I have added the support where unfunded account is able to issue txn in group txn. I have unit tests for both normal group txn and inner txn as well.


Release/v1.0.2

@KarishmaBothara don't forget to create the same PR for main (back-merge) and for azalea

I've recreated all deployment branches that include the changes in v1.0.2, so good to close this


[ConstraintSystem] Add a new stat to be used for expression type chec…

…ker performance testing.
This counts the number of leaf scopes we reach while solving the
constraint sytem, and is a much better measure of the growth of
unnecessary work than the total number of scopes opened.
There were two tests where I had a difficult time getting scale-test
to fit the curve even after adjusting some of the parameters, so I've
left those to use the old stat for now.

@swift-ci Please smoke test

@graydon I'll take another look at the always-on counters. It wasn't clear to me if these got reported out when doing scale-test, so I did the lazy-programmer approach of copying what was already happening for the total scope count.

@graydon Oh yes, and on decreasing counts - are these being sampled within a constraint system?
If not, then I don't think there should be a problem here since they should always be increasing from system to system.

@rudkx All stats get sampled on every construction/destruction of any FrontendStatsTracer.

@swift-ci please smoke test Linux platform


Invalid Record tab title when editing a record

Description
A record tab title stays "New record" even in edit mode. It should say be "Edit record".

Fixed while working on navigation (3db90a6dec861feadf90a1abe82bf99ad7de1e8a).


Neo2 Fixes

I would like to use karabiner to configure neo2 for macOS sierra. Unfortunately i can not find the config options for the json. http://wiki.neo-layout.org/wiki/Neo auf dem Apple Macintosh einrichten
Any help would be highly appreciated.

I am trying to port the old neo2 configuration to json too.
Unfortunately there are still some features missing here.
It can map from one key to another atm.
This is not enough to map the 4th and 6th layer.

Really would like to see better and more complex examples and/or a reference to make more complex configs.

@neominik I tried this config and it does nothing for me. how did you installed it

This works fine for me:
{
    "profiles": [
        {
            "name": "Neo Layout",
            "selected": true,
            "simple_modifications": {
                "caps_lock": "left_option",
                "non_us_pound": "right_option"
            }
        }
    ]
}

I hope that there will be a solution for Layer 4/6. I think we have to wait for final Version of Karabiner Elements.

Level 4 isn't working yet, is it? :(
Or has anyone managed to make it work? Is there a workaround or another tool to achieve this?

Nope, not to my knowledge. I (painfully) rolled back to El Capitan via Time
Machine to get Neo again.
This is really quite an important issue for Neo users on Mac OS and should
be discussed in the Neo project. Unfortunately their website and issue
tracker seems somewhat abandoned, at least to me (being used to fairly
quick reaction times on GitHub).
It definitely would be a good idea to put a warning on the Neo website/wiki
to warn users not to upgrade (though it might be too late by now).

+1

+1 - I would like to support this somehow, preferably with money. Can I donate somewhere to give this bug priority?

I'm working on a implementation of the Neo2 layout over here. Actually I'm mostly finished on the implementation side, I just have to do some more testing and add some installation instructions. Unfortunately, you will have to compile (and sign) that version yourself right now because I don't have an Apple Developer account that would allow me to create a redistributable package.

@jgosmann That is very awesome! Will this be merged back into the main repo? How can I help testing this?

I have no plans at the moment to get this in a state where it would be merged back into the main repo. Many of the layer 4 keys are hard coded which would not be acceptable in the main release.
As for testing: Install it and see if the layout including layer 4/6 works? (I haven't written the compile and install documentation yet, hopefully I manage this weekend.)

@jgosmann Could you ping this bug once you have outlined the installation process somewhere? I have for example no experience to sign a kernel extension, so I would not know how to install this.
Also, could you write a few lines of what cleanup would be needed to get this merged back into master? It would be a shame if you effectively fork this and people in the future would need to choose between your Neo port (which probably did not receive latest bug fixes since you moved on) and upstream. Not suggesting that you should do the cleanups (you did plenty by making Neo work!), but only you know what would be required. So writing something down gives somebody else a chance to do the work.

Could you ping this bug once you have outlined the installation process somewhere?

Will do.

Also, could you write a few lines of what cleanup would be needed to get this merged back into master?

I think the main part would be a way to to allow to configure remappings from a key + a set of modifier key to a different key + a different set of modifier keys. Then the Neo2 remappings need to be moved from the source code to that config file. Probably is is also necessary to do some cleanup of the actual code doing the remapping; it is a little bit hacked in now (partially because I'm not really familiar with the code base).

Cool, thanks. I will wait for your installation guidance and then test this.

I've put the installation and build instructions online. Please test them and let me know if anything doesn't work or I skipped something.

well, for building this you need an apple developer license wich costs 99 $ per year, don't you? can anyone here maybe upload a compiled version?

You don't need a developer license if you build and run it on your own Mac. You only need the developer license if you want to distribute the build package and have it run on other people's Macs. That's why I can't distribute a compiled version at the moment.

@jgosmann Thank you for putting the documentation up. I tried following the build instructions, but in step 3

Click the “Create” buttons next to “Developer ID Application” and “Developer ID Installer”

Those buttons are greyed out for me. The only things I can "create" is iOs Development and Mac development. Is this expected? How can I change this?
Otherwise, the build seems to have been successful, beside the signing of course. Full output is in this gist.

I might have remembered incorrectly which buttons I clicked. Can you try the “create” button next to “Mac development”?

Otherwise, the build seems to have been successful, beside the signing of course.

Unfortunately, I don't think the required kernel module will be loaded without being signed.

Partial success here. It seems that it properly codesigns the binaries, but not the package:
code sign Library/Application Support/org.pqrs/Karabiner-Elements/updater/Karabiner-Elements.app/Contents/Frameworks/Sparkle.framework/Versions/A/Resources/Autoupdate.app
code sign Library/Application Support/org.pqrs/Karabiner-Elements/updater/Karabiner-Elements.app
code sign Library/Application Support/org.pqrs/Karabiner-Elements/bin/karabiner_grabber
code sign Library/Application Support/org.pqrs/Karabiner-Elements/bin/karabiner_event_dispatcher
code sign Library/Application Support/org.pqrs/Karabiner-Elements/bin/karabiner_console_user_server
code sign Applications/Karabiner-EventViewer.app
code sign Applications/Karabiner-Elements.app

but later
Sign with Developer ID
productsign: error: Could not find appropriate signing identity for “D497CAAC851FC0B463D7D33B87D09CF169C6B3CD”.
Make Archive

Full output: https://gist.github.com/SirVer/ddcd707e255f9b1409e4413a118b87a3
I will try to install this tomorrow. @jgosmann, Any more ideas on the package codesign error?

There will remain some elements that cannot be signed which is fine. However, in your case nothing seems to be signed as there is the following error multiple times:
D497CAAC851FC0B463D7D33B87D09CF169C6B3CD: no identity found

I think you missed step 4 of the build instructions because the ID is my the one of my key.

@jgosmann My apologizes, I relinked the first gist again. I am not having my best day today :/. The correct one is https://gist.github.com/SirVer/1be4e5c16284b8b53931ac92eba8ffc4.

I think that might be fine. It seems only the installer could not be signed which should be ok (as long as you don't want to distribute it to other people). So I'd suggest to install it and see if it works.

The package did install without any problems and the other layers are working on my laptop again. I will upgrade my Mac now from El Capitan and see if it still works. I will report any further issues in this bug.
@jgosmann Your build instructions and installation information are correct and working. Thanks again for putting this together.
@christophebeling I can confirm that no payed developer license is needed for the instructions.
@tekezo Neo2 is a popular keymap in Germany and having it supported will benefit karabiner-elements popularity. Could you offer your opinions on upstreaming Jan's changes into the master branch?

@jgosmann Your build instructions and installation information are correct and working. Thanks again for putting this together.

Just to verify: Did you end up clicking the “Create” button next to “Mac development”? Also, it just occurred to me that the other buttons might have been greyed out because the corresponding signing keys were already generated.

I can confirm that it builds fine using a key created with ”Mac Development” (the other option was greyed out for me too — possibly due to this, but it doesn’t seem important).
After trying it for a few minutes, the only odd thing I notice is Home/Pos 1 & End (i.e., Mod4+U/E (Neo) = Mod4+A/F (Qwertz labels)), which seems to behave differently in different contexts/applications (e.g., in this text box, it scrolls up and down). I’ll open an issue for that.
(Also: re-adjust the key repeat settings to something sane like 250 / 50.)
Thank you very much @jgosmann! Maybe I’ll dare to update to Sierra after all. ;)

@jgosmann First issues I encountered:

CapsLock/Right Cmd + Qwerty-A used to jump to the beginning of the line, but now instead scrolls up.
CapsLock/Right Cmd + qwerty-asdf are the cursor keys. But adding shift to select text does not work, but did before.

I think it would be best to post such issues to the jgosmann/Karabiner-Elements-Neo issue tracker and discuss them there. I just noticed that the issues tab was deactivated, but I enabled it now. I also added the problems mentioned as jgosmann/Karabiner-Elements-Neo#1 and jgosmann/Karabiner-Elements-Neo#2. I will fix them as soon as I have time.


Add in the individual units tables

The tables should be in pretty good shape, and I even got a rudimentary version of the click-to-expand functionality we have on the current site—but I'm sure we'll want to replace that with something more modular/accessible. This is proof-of-concept stage…

I can't tell if this is somehow environmental for me, but it seems like the listing page is broken? It just returns a 404.


Support for html tags in translations

Hi. I've updated recently react-i18nify from version 1 to latest and I can see that support for dangerousHTML was dropped. Also I didn't find a way to render html tags in translations. Sample translation:
{
  'sample': 'I want to make one word bold here'
}

This translation is not rendered, so I can see I want to make one word bold here on the page. Is this feature on the roadmap? Thanks.

You can just do this yourself, for example like this:


Can't find a changelog that documents breaking changes

I've just updated form alpha.2 to alpha.4 and noticed that .breadcrumb stopped working because it now requires a .breadcrumb-item on each <li>. Is there a history where I can see all the breaking changes and new features?

you may be able to glean most of the information by looking at the various ship lists https://github.com/twbs/bootstrap/issues?utf8=✓&q=is%3Aissue label%3A"ship list"  (but nothing as involved as, say, the migration guide from 3 to 4 http://v4-alpha.getbootstrap.com/migration/)


Acronyms using python

Acronyms are shortened forms of word  or phrases. They assist in making manuscripts easy to read and understand. Additionally, they help in meeting the strict word-count targets, avoiding the repetition of words, thereby making the text easy to read.

Assign me this issue

@Laraibnehal Assigned to you


Add description of refactoring for chooseImpl()

Explain how to refactor chooseImpl()
This should resolve #66

Codecov Report

Merging #106 into lab3 will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##            lab3    #106   +/-   ##
=====================================
  Coverage   69.7%   69.7%
=====================================
  Files        107     107
  Lines       7334    7334
  Branches    1388    1388
=====================================
  Hits        5112    5112
  Misses      1796    1796
  Partials     426     426

Continue to review full report at Codecov.

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


Added two procedures to populate the tables employee and department

So I have added two SQL Stored Procedures to populate the tables Employee and Department.
This will use REST API to get random user data from randomuser.me/api.
Please change the nationality at the API request to populate the names based on your country. I've currently set it to India (IN)
Note: I've been using this repo to present demo to my clients to use BOB and VS Code for GitHub based development. So having the company system application with valid data is a necessity for all my demos.

Hi @worksofliam please take a look at this PR and let me know if you need any more information from my side?

Hi @worksofliam , I have modified the code as per your suggestion and did a commit. Please take a look.


gravity forms set to display none

my forms have conditional logic on them, which adds a display: none inline tag to the forms. the display: none tag remains on the form as wp-footer() is not loading (i think). How can i ensure wp-footer() function is loaded?

Hi there! Sorry to hear you're having issues (but glad you're giving Timber a try!) We try to keep GitHub clear for just bugs with the Timber code itself. For questions like this, please post to the StackOverflow channel:
http://stackoverflow.com/questions/tagged/timber
... I'll try to follow-up there if there's an answer I can provide


Add a hosted/web interface to interact with the servers

It would be great to be able to visualise the grpc endpoints of the Miden-Node in a playground like these Ethereum endpoints:
https://ethereum.github.io/execution-apis/api-documentation/
This would make them more readable and enable more people to play with them.
I believe that this would potentially involve creating a website to host this playground for all users or could be added as a local playground to run.

related to: https://github.com/0xPolygonMiden/miden-node/issues/95


Allow queries without authentication key

It would be nice if pdc-client could do queries without insisting on having an auth key.
Authentication is only really needed for updating PDC data.
Version
pdc-client-1.8.0-18.fc32
Steps to Reproduce:
$ pdc -s https://pdc.fedoraproject.org rpm list
Actual results:
Traceback (most recent call last):
File "/usr/bin/pdc", line 22, in
runner.run()
File "/usr/lib/python3.8/site-packages/pdc_client/runner.py", line 183, in run
self.client = pdc_client.PDCClientWithPage(self.args.server, page_size=self.args.page_size, ssl_verify=ssl_verify, page=self.args.page)
File "/usr/lib/python3.8/site-packages/pdc_client/init.py", line 407, in init
super(PDCClientWithPage, self).init(server, token, develop, ssl_verify, page_size)
File "/usr/lib/python3.8/site-packages/pdc_client/init.py", line 238, in init
token = self.obtain_token()
File "/usr/lib/python3.8/site-packages/pdc_client/init.py", line 256, in obtain_token
raise Exception('Could not obtain token from any known URL.')
Exception: Could not obtain token from any known URL.
Expected results:
Read some results without auth key.

I wasn't aware that the pdc client and server are still used. I thought fpdc (with fpdc-client) is the new cool thing.
cc @lubomir

Ah, so I wasn't aware of fpdc...


ComfyUI surpport?

Hi,thanks for you excellent work,do you have a plan about ComfyUI version? Just like SVD model or AnimateDiff,that can be used on ComfyUI?

Thanks for your interest. I'm trying to support ComfyUI this week.

That's great!

I'm not very familiar with ComfyUI, so I need more time to deploy it. If you are familiar with ComfyUI, you are welcome to help me deploy ComfyUI.

I'm not very familiar with ComfyUI, so I need more time to deploy it. If you are familiar with ComfyUI, you are welcome to help me deploy ComfyUI.

Hi, the absolute best place to find out anything you need to know to help is banodoco discord. There are comfy devs there who are expert in porting git projects to comfy who would gladly help.

Thanks~

ad_comfyui channel especially

Okay！


how to embed  flutter into an existed SwiftUI project

I already have a swiftUI project, how can i embed my flutter module into this SwiftUI project?

Hi @kevin-zhaotk, Please refer the IOS section of the official docs https://docs.flutter.dev/development/add-to-app.
Closing as this is neither a bug nor a feature request.

@maheshmnj  that doc only describes  how to integrate flutter into a iOS app with UIKit & Swift, not about SwiftUI;
so, can you share some docs about integrating flutter to SwiftUI?

reopening based on the above comment.

I am quite not sure if the existing add to app docs are applicable for swiftUI project. Labeling this issue for further insights from the team.
cc: @jmagman


Optimize spill performance by buffering spill data write

Optimize disk spilling performance for hash build and hash join by
buffering the serialized spill data before persist to disk. With 1MB
buffering per operator, for a given Meta internal production shadowing
query, the spilling performance has been improved a lot. The hash
build cpu time drops from ~10 hrs to <1hr (>10x). The hash probe
cpu time drops from ~1.5hr to ~10 mins (~10x). The spilling stats show
the performance improvement is due to the reduced number of remote
ios. spillDiskWrites of hash build drops from 4M to 100K and spillWriteTime
drops from >7hrs to <3mins.
This PR adds query config to allow user to set per-query write buffer size.
The follow will add the corresponding native session property at Presto to
allow user to configure in Prestissimo.

@xiaoxmeng Thank you for the optimization. Overall looks good. Some minor comments.

@mbasmanova thanks for prompt review!

@xiaoxmeng There are CI failures. Would you take a look?


Fongo.getDB(String) should return com.mongodb.DB but returning com.mongodb.FongoDB

Getting the following error while calling getDB method using fongo 1.6.7:
java.lang.NoSuchMethodError: com.github.fakemongo.Fongo.getDB(Ljava/lang/String;)Lcom/mongodb/DB;
The signature of Fongo.getDB(String) in 1.6.7 is
 public FongoDB getDB(String dbname)
which should be
 public DB getDB(String dbname)
The versions before this(<1.6.7) has correct signature.

Hi,
can you provide a test class ?
because FongoDB extends from DB and I'm not sure to understand where this can be a problem ?

Sorry for the inconvenience. This is not an issue. I had the class compiled with a different Fongo version and executed with a different version.


Custom Format Class

Since we cannot use JSON (external dependency), we can only create our own file format.
This is a very simple text-based format.
name
last modified
---
"income" OR "expense"|amount|category|date|notes

Example
Jane Roe
2024-01-01T04:11:25+08:00
---
income|5.00|your mom|2024-01-01|none
expense|1.99|expense test 1|2024-01-01|none

Notes

make sure to deny user data with the pipe character in it as we use it for delimitation.
use LocalDate for expense records
use LocalDateTime for last modified
ignore lines with parse errors

To Do

[x] implement deserialization into objects
[x] implement serialization into files

partially resolved in e1899535576259ea4207749ffc687a53e19c4c0e, which implements parsing, which should handle record errors gracefully.

note: treat rows with state like "loans" for the income/expense field as invalid. the row should be either income or expense.

serialization is implemented.

7bcc57fff22bb271351b8966a19986e830b9424b


Fix langlib deviations

Purpose
$title.
Fixes https://github.com/ballerina-platform/ballerina-lang/issues/33423
Fixes https://github.com/ballerina-platform/ballerina-lang/issues/33410
Fixes https://github.com/ballerina-platform/ballerina-lang/issues/33409
Fixes https://github.com/ballerina-platform/ballerina-lang/issues/33419
Fixes https://github.com/ballerina-platform/ballerina-lang/issues/31814
Fixes https://github.com/ballerina-platform/ballerina-lang/issues/33395
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Need to remove the langlib-fixes branch from the workflows before merging this PR.


PHP 7.1 Cannot open source device in ... CAS-1.3.5/CAS/Client.php:931

Hi Folks,
One of the wordpress plugins I use (Authorizer) uses phpCAS and is throwing this error when it tries to start the session:
PHP Fatal error:  Uncaught Exception: Cannot open source device in plugins/authorizer/vendor/CAS-1.3.5/CAS/Client.php:931
Not sure what's going on here.  What sort of debug can I get you that would help?
Thanks!

Yeah... this was an altogether issue.  My Apache chroot needed the /dev/random and /dev/urandom nodes.  Duh.  Close this issue... silly me.


Error on bing searches

I get this error message when it gets to the part of bing searches:
Traceback (most recent call last):
File "C:\Users\xxxx\Documents\clothings\Microsoft-Rewards-Farmer-master\main.py", line 24, in main
executeBot(currentAccount, notifier, args)
File "C:\Users\xxxx\Documents\clothings\Microsoft-Rewards-Farmer-master\main.py", line 141, in executeBot
accountPointsCounter = Searches(desktopBrowser).bingSearches(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\xxxx\Documents\clothings\Microsoft-Rewards-Farmer-master\src\searches.py", line 57, in bingSearches
search_terms = self.getGoogleTrends(numberOfSearches)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\xxxx\Documents\clothings\Microsoft-Rewards-Farmer-master\src\searches.py", line 27, in getGoogleTrends
trends = json.loads(r.text[6:])
^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\xxxx\AppData\Local\Programs\Python\Python312\Lib\json_init_.py", line 346, in loads
return _default_decoder.decode(s)
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\xxxx\AppData\Local\Programs\Python\Python312\Lib\json\decoder.py", line 337, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\xxxx\AppData\Local\Programs\Python\Python312\Lib\json\decoder.py", line 355, in raw_decode
raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

hi! ive experienced id say every bug imaginable. today when trying too startup, i got a new bug! my fix too so far every bug ive had is the following:
(also i love your \Documents\clothings. gotta keep the bot safe hahaha.) anyways:
1: go to Windows Settings
2: Type Reset in search.
3: Factory reset your computer, make sure you choose the option too only remove apps and NOT User files. itll uninstall all your apps (this includes python and all the modules, plus all other PIP, Visual studio code issues that may cause your bug)
4: when finished, download MSwebdriver that matches your edge, same with chrome. have these in the directory that your bot is in.
5: install Python 3.11. DO NOT install 3.12 as it removed some stuff from earlier python versions which is needed for this bot.
6:  When in account.json, if you removed the proxy crap, it should look something like this:
[
{
"username": "riogrghu",
"password": "wqetwryeturytkuly"
}
]
not this:
[
{
"username": "riogrghu",
"password": "wqetwryeturytkuly",
}
]
(if you noticed, after password field, you do not want the , after it. this wont work.)
umm i have beginner friendly fixes. ill add them as i think of more, i hope these help!
(fyi i am new too python, this project as with other microsoft rewards farming repos has been my learning experiance. keep me in touch! id love too learn and or help)


add mgr support (install, gatherkeys, daemon creation)

@alfredodeza i think we should cut the jewel branch before mgr changes, Not sure the code will move to jewel but looking at other bluestore changes i believe we could push the stable jewel version to pypi and also cut a branch here.

Tests now pass!
The gatherkeys changes should not effect jewel (or older) installs.  However, the install task now tries to install the ceph-mgr package, which doesn't exist for jewel.  I'm not sure how to handle that... suggestions?  Maybe a --no-mgr flag that can be passed for legacy versions?  Or separate it into a different install step and warn instead of error out if it can't be found?

(Either way, we should make sure that master still is able to install kraken and jewel!)

I think the easier approach would be not to install mgr by default for all and then install only using --install-mgr flag which can be done easily with 'components' list.  It would require user to run one more command.
Nicer way would be to detect 'jewel', 'kraken' etc but that can be done only after install or we could simplify that by using a flag in ~/.ceph_deploy.conf which was discussed during hammer time.
Also I believe it should be fine to branch out for 'jewel' and modify tests to use jewel branch for jewel tests, I dont there there will be lot of back ports for that branch, but if we are taking bluestore changes for jewel then it doesn't make sense to branch out.

this seems reasonable for now; after luminous is out we can switch teh
default.  although even then we'll still need a
--no-install-mgr flag... so we may as well add that now too?

Sure, I guess it makes sense to add the flag now.

Confirmed that install works with both jewel and master for rpm and deb.  I think this is ready to merge?

retest this please

using 'ceph' metapackage was nice :+1:

http://pulpito.ceph.com/vasu-2017-03-17_17:00:23-ceph-deploy-jewel---basic-vps/

Thanks, Vasu!


paper

what is your paper title

Hello @loaynabi I didn't release a paper for this project, however if you would like to cite my work you can use :
@software{Narjis_Diet_recommendation_system_2023,
author = {Narjis, Zakaria},
license = {Apache-2.0},
month = jan,
title = {{Diet recommendation system}},
url = {https://github.com/zakaria-narjis/Diet-Recommendation-System},
version = {1.0.0},
year = {2023}
}

I will when i finish my work cause it is with the same dataset you used but
i need to find a paper that used this dataset
On Sun, 23 Jun 2024 at 3:21 PM Zakaria Narjis @.***>
wrote:

Hello @loaynabi https://github.com/loaynabi I didn't release a paper
for this project, however if you would like to cite my work you can use :
@software{Narjis_Diet_recommendation_system_2023,
author = {Narjis, Zakaria},
license = {Apache-2.0},
month = jan,
title = {{Diet recommendation system}},
url = {https://github.com/zakaria-narjis/Diet-Recommendation-System},
version = {1.0.0},
year = {2023}
}
—
Reply to this email directly, view it on GitHub
https://github.com/zakaria-narjis/Diet-Recommendation-System/issues/7#issuecomment-2184967496,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5JDPWN5ZKQYSMSLAH3XKK3ZI24VRAVCNFSM6AAAAABJYFGT42VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBUHE3DONBZGY
.
You are receiving this because you were mentioned.Message ID:
@.***>

Can you please send me the machine learning notebook you used? I will try
to improve what i can so i can put your project in my reference of my paper
that will be published in the conference soon
On Sun, 23 Jun 2024 at 3:23 PM Loay Nabil @.***> wrote:

I will when i finish my work cause it is with the same dataset you used
but i need to find a paper that used this dataset
On Sun, 23 Jun 2024 at 3:21 PM Zakaria Narjis @.***>
wrote:

Hello @loaynabi https://github.com/loaynabi I didn't release a paper
for this project, however if you would like to cite my work you can use :
@software{Narjis_Diet_recommendation_system_2023,
author = {Narjis, Zakaria},
license = {Apache-2.0},
month = jan,
title = {{Diet recommendation system}},
url = {https://github.com/zakaria-narjis/Diet-Recommendation-System},
version = {1.0.0},
year = {2023}
}
—
Reply to this email directly, view it on GitHub
https://github.com/zakaria-narjis/Diet-Recommendation-System/issues/7#issuecomment-2184967496,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5JDPWN5ZKQYSMSLAH3XKK3ZI24VRAVCNFSM6AAAAABJYFGT42VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBUHE3DONBZGY
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

https://github.com/zakaria-narjis/Diet-Recommendation-System/blob/main/food-recommendation-system.ipynb. I would assume you are asking for this one.

the problem that there is a problem that the user needs an amount of
calories to reach his goal but i dont understand how can machine learning
solve this problem while the dataset only has the calories of the food not
the user needs i dont understand how you did it nor your data preparation,
you did a huge and a well work but i dont understand it if you can clarify
to me anything i will be thankful for you
On Tue, 2 Jul 2024 at 17:54, Zakaria Narjis @.***>
wrote:

https://github.com/zakaria-narjis/Diet-Recommendation-System/blob/main/food-recommendation-system.ipynb
http://url. I would assume you are asking for this one.
—
Reply to this email directly, view it on GitHub
https://github.com/zakaria-narjis/Diet-Recommendation-System/issues/7#issuecomment-2203438160,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5JDPWL3JJHJQJOJ6XFTKLLZKK5LZAVCNFSM6AAAAABJYFGT42VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMBTGQZTQMJWGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

in addition the data does not have the feature to know if is the food
dinner or breakfast or lunch i added this feature to the dataset myself
On Tue, 2 Jul 2024 at 18:18, Loay Nabil @.***> wrote:

the problem that there is a problem that the user needs an amount of
calories to reach his goal but i dont understand how can machine learning
solve this problem while the dataset only has the calories of the food not
the user needs i dont understand how you did it nor your data preparation,
you did a huge and a well work but i dont understand it if you can clarify
to me anything i will be thankful for you
On Tue, 2 Jul 2024 at 17:54, Zakaria Narjis @.***>
wrote:

https://github.com/zakaria-narjis/Diet-Recommendation-System/blob/main/food-recommendation-system.ipynb
http://url. I would assume you are asking for this one.
—
Reply to this email directly, view it on GitHub
https://github.com/zakaria-narjis/Diet-Recommendation-System/issues/7#issuecomment-2203438160,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5JDPWL3JJHJQJOJ6XFTKLLZKK5LZAVCNFSM6AAAAABJYFGT42VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMBTGQZTQMJWGA
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

There's no dinner or breakfast or lunch in the dataset, so what I did is I take your recommended daily calorie intake. then I divide it into breakfast lunch dinner like (60% dinner 20% breakfast 20% lunch or idk u can play with it and u can find this % in my code I forgot exactly how I distributed this %) , for example your daily recommended calorie intake is 2000 calories, therefore for dinner you need 60% of 2000 which is 1200. So using KNN the system recommends to you a dish with calories around 1200 and with enough proteins fats and stuff recommended by health expert( for this part you would like to do some research on what are the recommended protions/ grams for each food component (which are our features).

I did that research and in my interface I calculated the needed protein
fats and carbs but i dont understand how can machine learning do this task
when the feature is the calories of the food and the given input is the
calories of the user
On Tue, 2 Jul 2024 at 6:38 PM Zakaria Narjis @.***>
wrote:

There's no dinner or breakfast or lunch in the dataset, so what I did is I
take your recommended daily calorie intake. then I divide it into breakfast
lunch dinner like (60% dinner 20% breakfast 20% lunch or idk u can play
with it and u can find this % in my code I forgot exactly how I distributed
this %) , for example your daily recommended calorie intake is 2000
calories, therefore for dinner you need 60% of 2000 which is 1200. So using
KNN the system recommends to you a dish with calories around 1200 and with
enough proteins fats and stuff recommended by health expert( for this part
you would like to do some research on what are the recommended protions/
grams for each food component (which are our features).
—
Reply to this email directly, view it on GitHub
https://github.com/zakaria-narjis/Diet-Recommendation-System/issues/7#issuecomment-2203567464,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5JDPWNRNDPTDZBLXLUZNIDZKLCNRAVCNFSM6AAAAABJYFGT42VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMBTGU3DONBWGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Do you have a whatapp we can chat on it better than email if you dont mind
On Tue, 2 Jul 2024 at 6:40 PM Loay Nabil @.***> wrote:

I did that research and in my interface I calculated the needed protein
fats and carbs but i dont understand how can machine learning do this task
when the feature is the calories of the food and the given input is the
calories of the user
On Tue, 2 Jul 2024 at 6:38 PM Zakaria Narjis @.***>
wrote:

There's no dinner or breakfast or lunch in the dataset, so what I did is
I take your recommended daily calorie intake. then I divide it into
breakfast lunch dinner like (60% dinner 20% breakfast 20% lunch or idk u
can play with it and u can find this % in my code I forgot exactly how I
distributed this %) , for example your daily recommended calorie intake is
2000 calories, therefore for dinner you need 60% of 2000 which is 1200. So
using KNN the system recommends to you a dish with calories around 1200 and
with enough proteins fats and stuff recommended by health expert( for this
part you would like to do some research on what are the recommended
protions/ grams for each food component (which are our features).
—
Reply to this email directly, view it on GitHub
https://github.com/zakaria-narjis/Diet-Recommendation-System/issues/7#issuecomment-2203567464,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5JDPWNRNDPTDZBLXLUZNIDZKLCNRAVCNFSM6AAAAABJYFGT42VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMBTGU3DONBWGQ
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

how to run this git repo in my vs code


Created a new project (Random Password Generator)

Created a new project (Random Password Generator) and added it to the JavaScript directory present in know-your-code branch.

Great work @JhndaCoder!


Suggestions to implement image-goal based navigation

Hi, I'm trying to extend the current tasks to include image-based target-driven navigation. I've been looking at the simulator and api. So far, my understanding is that there are sensors attached to agents and these are passed to the renderer for getting images. I see that there is a method to render based on cameras alone (instead of a sensor tied to an agent).
https://github.com/facebookresearch/habitat-sim/blob/b11d6e7f3c948cbff3a40672ff213bd51c625fe4/src/esp/gfx/Renderer.cpp#L132
I'm not sure how this can be used in the python API to get renderings of a target-location though. Is there a way to define a task sensor that takes in a pose of the world and provides the image from that pose?

The functionality needed to do this exists, but it is hidden away in c++ right now (we are working on that).  Currently, I think the easiest way to do this would be to temporarily move the agent with
https://github.com/facebookresearch/habitat-sim/blob/master/src/esp/bindings/bindings.cpp#L581
get the observations via
https://github.com/facebookresearch/habitat-sim/blob/master/habitat_sim/simulator.py#L69
and then restore the agent to its previous location.

@srama2512 what a scenes dataset are you working with?
Habitat-API was designed specifically for task definitions. You can check:

Navigation task definition.
Embodied Question and Answering definition.

@erikwijmans - Ah, got it. Moving the agent temporarily seems like a solution for now. Thanks!
@mathfac - I'm currently trying to use both MatterPort and SUNCG. If I understand correctly, I would just have to create ImageNavigationTask and create a TargetSensor right? I wasn't sure how I would render images from the TargetSensor though.
In House3D, I was able to create an extra camera (one that is static throughout the episode) and obtain the same information. Is there a functionality to do that at the moment?

In House3D, I was able to create an extra camera (one that is static throughout the episode) and obtain the same information. Is there a functionality to do that at the moment?

This functionality does exist, and it is somewhat exposed into python (although it is currently exposed in a dangerous way, there are some caveats about memory ownership that are quite different than normal memory ownership in python, we are working on fix this).  In #6 you can see how to create a new node in the scene graph: https://github.com/facebookresearch/habitat-sim/pull/6/files#diff-8a755bab394eaa22725cd457449d7108R62, create a PinholeCamera that uses the node for rendering: https://github.com/facebookresearch/habitat-sim/pull/6/files#diff-142edc24b058301997a8b0d8ab887f6dR134, and finally how to move that camera to an arbitrary location: https://github.com/facebookresearch/habitat-sim/pull/6/files#diff-142edc24b058301997a8b0d8ab887f6dR236
If the camera is attached to a child of the root node instead of a child of the agent, it won't move with the agent.
The renderer will happily take a PinholeCamera object and render from that view (although you will then have to read the result into an array like we do here: https://github.com/facebookresearch/habitat-sim/blob/master/habitat_sim/simulator.py#L121)

Thank you for the detailed instructions! I will close this issue now.


Spawns Python processes that hog CPU resources

When working with python files, the package will after a few minutes have spawned 3-5 Python processes that steal all my CPU resources. Atom and the funcionalities of the tool seem to work fine, but doing anything else simultaneously is impossible. Killing the troublesome Python processes don't seem to give any obvious side effects, but after 10+ minutes or so they will have spawned again and started eating up all my CPU resources.

This sounds like an issue that existed a long time ago. Are you still experiencing this?

Deactivated the app since, so haven't tried lately. Will enable it again and see if I experience the same.

I am experiencing this issue (2017-11-06). This starts only after editing python file(s) in Atom with python-tools running. After killing the process, Python-Tools no longer works.
(Windows Server 2016 Standard 64 bit; 2 Xeon processors; 32 GB RAM)

@travistennies could you find the parameters for that executable? It's possible that some other program is giving you this problem. Admittadly there are situations where this could happen but it should be rare (I believe its when the jedi package finds itself in a circular dependency). Do you have any other atom python packages installed like python-autocomplete?

@MichaelAquilina I have not tested if the problem still exists, but I do have python-autocomplete installed (and did have when the problem occurred).

@MichaelAquilina I am no longer having an issue because I stopped using python-autocomplete. And I ran it, without python-tools running, and got the same result/issue. So...false alarm.

Just installed python-tools, tried to use it, and had my whole system lock up because it used up all my RAM. I checked htop and there it was python-tools using up over 10GB of RAM (and swap).


166 percentile blend

Rebased to update TRAVIS environment to ensure tests pass.
Replaces #220 (which replaced #212).

I've had a look at the data from the new CLI test. It's hard to know what the expected output should be but it looks like it could be reasonable. Link to ipython notebook on Jira


fix: ensure insteadOf gets picked up for submodules

Hi @itsdalmo. I know you just changed this for the e2e test, but I'm opening it back up for feedback.
We have some private repos that must use git instead of https. Unless the git config is applied globally, I have found that I am unable to get the insteadOf directive to work on the submodules. I've tried a number of variations on it, including setting GIT_CONFIG and supplying a file identical to the global config. Unfortunately, that doesn't supplement the .git/config, but rather simply overrides it.
I'm wondering if you or anyone else may have suggestions on how to fix? Do the e2e tests break, or does it just leave your local test environment in a bad state?

Hi! I'm a bit confused by this @jhosteny:

We have some private repos that must use git instead of https.

I thought the current configuration for insteadOf would replace  git:// and<EMAIL_ADDRESS>with https://, and the entire point was to use https:// to clone submodules since we only pass a token to the resource (no deploy key)?
On that subject, shouldn't g.command("git", "config", "url.https://.insteadOf", "git://") actually be g.command("git", "config"<EMAIL_ADDRESS>"git://")? I.e. it needs to include x-oauth-basic? 🤔
Have you tried doing something similar to the comment below this Stackoverflow response? i.e. git -c url.git@github.com:.insteadOf=https://github.com/ submodule update --init --remote --recursive?

I'm sorry, that wasn't very clear of me. I mean that our default configuration of the subrepo should use<EMAIL_ADDRESS>not https, since developers want to use their SSH keys to perform operations. In CI, however, we use insteadOf to replace that with access via https and access tokens. Your explanation is exactly what we want to do.

Hi! I'm a bit confused by this @jhosteny:

We have some private repos that must use git instead of https.

I thought the current configuration for insteadOf would replace git:// and<EMAIL_ADDRESS>with https://, and the entire point was to use https:// to clone submodules since we only pass a token to the resource (no deploy key)?

Yes, you are correct - we use<EMAIL_ADDRESS>not git, so I forgot this case. I will fix (assuming the below does not work).

On that subject, shouldn't g.command("git", "config", "url.https://.insteadOf", "git://") actually be g.command("git", "config"<EMAIL_ADDRESS>"git://")? I.e. it needs to include x-oauth-basic? thinking

Hmm - I thought I pretty exhaustively tried all other options, but I don't recall now if I tried this. I will check, and amend the PR if this works.

Have you tried doing something similar to the comment below this Stackoverflow response? i.e. git -c url.git@github.com:.insteadOf=https://github.com/ submodule update --init --remote --recursive?

@itsdalmo actually, I recall now why the git: protocol did not have the x-oauth-basic token specified - you can only clone a public repo with the git: protocol at GH, since git: offers no security. Thus, it could probably just be removed entirely.

@itsdalmo the method you referenced appears to be working properly (though it is a bit more complicated). I am going to soak this in our environment for awhile with a local version of the resource. Is it possible that the subrepo in your e2e test can be made private to ensure this works correctly?

Great to hear @jhosteny!

Is it possible that the subrepo in your e2e test can be made private to ensure this works correctly?

Would that not make it hard for others to run the E2E tests? 🤔

I am going to soak this in our environment for awhile with a local version of the resource.

Roger that, let me know when you feel ready for a review!

Would that not make it hard for others to run the E2E tests? thinking

Yeah - I'm not sure how to handle this case. Is it worth having an extra tagged case that is not run normally, but can run against a private repo? Or just skip it? I suppose if the git config were wrong, git operations would fail on submodules, so there is value as-is.

@itsdalmo this has been running successfully for us over the last month or so, if you would like to review. I am not sure how to add tests unless there is tagged set that only get run manually against private sub repos.

@itsdalmo is there anything I can do on this one to help get it merged? We are running a fork that has this, along with #186, #189 and #224. We're running a number of buildroot builds with numerous nested private submodules, and it has been working well now.

@jhosteny I'll have look this week, thank you for contribution, I'll investigate making selective test-case for E2E targeting a private repo, which can be optionally overridden locally.

Note that the latest change also fixes the issue observed in #234

I got the same issue being unable to fetch a repository with submodules (I got an authentication error), I tried this pr and it fixes my issue too.

yes, :shipit: please! :smile:

Is there any chance we can get this merged?

Is there any chance we can get this merged?

Is this still going in at some point?

We are also running into issues cloning a repository with submodules.
If this is approved, can we merge and release it @itsdalmo @rickardl?

@itsdalmo gentle ping on this!


Encoding in dataset.py:61 json.load()

When starting training, I got the following error:
UnicodeDecodeError: 'charmap' codec can't decode byte 0x8d in position 238339: character maps to <undefined>
By adding an encoding parameter, the error was resolved:
D = json.load(open(os.path.join(self.output_path, f'{split}.json'), encoding="utf-8"))

Although I did not have this encoding issue on my end, it is great that this simple fix worked for you. Thanks for posting it!


docs: regenerate api documentation

generated docs were outdated that's why I regenerated docs using npm run gen-docs.
I think all docs should be generated automatically during deployment so we don't have to manually do that periodically.

Covered by #593, but thanks for the heads up.


:sparkles: Update external provider modules

With the current external provider module names, we cannot import these providers into other projects, such as kantra. Updating the module names to the path inside of the main repo should fix this issue.

Assuming we can get tests to pass, is there any reason to hold this PR @eemcmullan ? It would really be to my benefit to merge this.


Fix prefix to fetch values from testResourcesConfig

Fixes https://github.com/micronaut-projects/micronaut-test-resources/issues/663

e.g this is what I get when running the Kafka tests

The test-resources prefix is "missing" as expected.


Incorrect 'local variable is assigned to but never used' warning with del in if block

Issue Report Checklist

[x] Searched the issues page for similar reports
[x] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[x] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
I've noticed that the warning "local variable 'blah' is assigned to but never used' will be triggered on the edge case of a deleted variable inside an if statement. This may be a pyflakes thing, but I thought I would mention it here first.
What steps reproduce the problem?
Consider the following function:
def test():

    if True:
        color = 1
        del color

The color variable on line 4 is highlighted as "local variable 'color' is assigned to but never used"
What is the expected output? What do you see instead?
The warning should not be produced. If the del statement is replaced with print no warning is issued. Also, the warning is not issued if the if block is removed.
Versions

Spyder version: 3.3.1
Python version: 2.7.14
Qt version: 5.6.2
PyQt version: 5.6
Operating System name/version: Windows

Dependencies

IPython >=4.0;<6.0:  5.4.1 (OK)
cython >=0.21     :  0.26.1 (OK)
jedi >=0.9.0      :  0.10.2 (OK)
matplotlib >=2.0.0:  2.0.2 (OK)
nbconvert >=4.0   :  5.3.1 (OK)
numpy >=1.7       :  1.13.3 (OK)
pandas >=0.13.1   :  0.20.3 (OK)
pycodestyle >=2.3 :  2.3.1 (OK)
pyflakes >=0.5.0  :  1.6.0 (OK)
pygments >=2.0    :  2.2.0 (OK)
pylint >=0.25     :  1.7.4 (OK)
qtconsole >=4.2.0 :  4.3.1 (OK)
rope >=0.9.4      :  0.10.5 (OK)
sphinx >=0.6.6    :  1.6.3 (OK)
sympy >=0.7.3     :  1.1.1 (OK)

This may be a pyflakes thing, but I thought I would mention it here first.

Thanls for reporting, but you're right, this a Pyflakes thing. We only the warnings/errors reported by it.


Remove padding_left setting to ensure all the plots have the same width

This PR is to fix plot layout problem #2. In my environment, all plots will be displayed at the same width as the screenshot below.

@timdiller
Thanks for your advice! I adjusted the white space between the plots with https://github.com/timdiller/dynamic_plot_example/pull/4/commits/39e964216f88a63649441eed154883e0afd09a71.

As far as I have tried, it seems that if the padding size of each plot is varied, the inner width of themselves also changes (this is a bit counter-intuitive for me). Therefore, I set the plot's padding to zero and the padding on the HPlotContainer itself. The disadvantage of this method is that the background color of the value_axis is not white (I set HPlotContainer's bgcolor to "white", but it does not seem to apply to the padding.)

@timdiller
Thank you so much!


Crash when trying to open the electric pump advanced

crash-2024-12-25_17.26.14-client.txt

Mek has made some code changes in the new version, and I will fix this issue later. During this period, you can lower the version of Mek.

Mek has made some code changes in the new version, and I will fix this issue later. During this period, you can lower the version of Mek.


Automatically enable GitHub Actions workflow(s) for forked repositores

Description
Having to manually enable GitHub Actions workflows is an annoyance. Sometimes I forget to enable them before pushing commits.
Screenshot

Example URL
https://github.com/refined-github/refined-github/fork

I think in generally one doesn't need to use CI on the fork because it may depend on secrets. This could be useful for simple CI tests that can run on the fork before they are approved in the PR on the upstream repo, but in reality these results might not be visible in that workflow.
So in practice it'd end up wasting CI or confusing users with failing CIs that are failing only due to missing secrets.


Safe subset of TypeScript -- For plugins/add-ons to TypeScript applications

I am considering the scenario of letting the users a hosted application upload their own front-end plugins, and get them served together with the rest of the application.
Today, frontend addons are usually realized by embedding Iframes (Zendesk uses this strategy AFAIK), or only making it possible if the application is self-hosted (Microsoft Sharepoint uses this strategy AFAIK).
Doing straight embedding of a user-provided code is of course a clear no-go in the JavaScript world, due to the risk of cross site scripting attacks. But what the code was first run through some kind of security audit -- then the model might be reasonable.
This would be a tedious and close to impossible thing with classical JavaScript, but with a TypeScript program which does not utilize 'any' type (explicitly or implicitly), and does only use a restricted set of imports, it might actually be feasible to make either an automatic or semi-automatic security audit of a plugin submission.
A project with similar goals is "Safe Haskell" (https://downloads.haskell.org/~ghc/7.8.4/docs/html/users_guide/safe-haskell.html) but I have not heard of any such thing in TypeScript.
Would it be a hard task to add a flag to tsc or tslint/tslint-immutable to enforce a safe subset of TypeScript. Or to tweak a static analysis tool like https://github.com/SonarSource/SonarTS to perform a trustable security audit of a TypeScript program, and what steps would this involve?

I too am interested in this. I have looked at SonarTS but I am unsure if something like this would be within the scope of the intended use case of that tool. Hopefully one of the devs could outline the steps involved.

You may be interested in https://github.com/tc39/proposal-realms.

There are so many ways that a program can trick static analysis to gain access to unsafe APIs. It is not just about typed or untyped.
Providing provably-secure-code guarantees is something that outside the scope of this project.

Great, thank you very much for your answers 👍 If you want, you can close this now, or we can let it stay open for a week, if maybe others have something to add.
I know @saevarb personally, and I know he needs a programming language-related project as part as his bachelor Computer Science degree, and I have suggested the investigation of this topic to him -- the creation of this issue is motivated by this.
If you agree that there are no solutions out there right now, and nothing planned, maybe this could be investigated for a few months by Sævar.
If any one can help us elaborate more in depth with the following "There are so many ways that a program can trick static analysis to gain access to unsafe APIs. It is not just about typed or untyped."  that would be great -- then one could choose maybe one single subject for Sævars study activity.

callEval(this);

function callEval(o: object) {
     Object.getOwnPropertyDescriptor(o, "eval").value.call(undefined, "console.log('eval')");
}

there are no anys here, there are no unsafe casts, there are no direct use of unsafe functions like eval.. yet you can call eval.
I do not think static analysis is worth while in this domain. every time you enforce a pattern a malicious actor can find another. ultimately you want whatever tool you write to be usable..
I think something along the lines of https://developers.google.com/caja/ or the realms proposal as noted by @andy-ms is more promising.

I'd like to point out that the above does use any, and the https://palantir.github.io/tslint/rules/no-unsafe-any/ line rule correctly detects it. But @mhegazy is right that we don't catch everything.

Just an update -- We had a meeting today with Professor in practical & automatic proofs of software termination and correctness http://www.imada.sdu.dk/~petersk/ -- We will continue with the project for at least 3 months! Please post anything that can help us -- We will also keep you updated ;-) This is gonna be great!


Generate CDS/AppCDS cache on first launch

https://github.com/ev3dev-lang-java/openjdk-ev3/issues/16
CDS: https://docs.oracle.com/javase/8/docs/technotes/guides/vm/class-data-sharing.html
AppCDS: http://openjdk.java.net/jeps/310

Maybe we should install in the EV3Dev session for user robot, a .bashrc or .bash_profile file to add that statement:
java -Xshare:dump

What do you think?

No! :D
That would probably mean a major slowdown of the login process.


Run watchFs tests without isolated daemons

Isolated daemons should not be necessary when file system watching is enabled.
Also bumps native-platform.
Fixes #13366.

Windows and Linux passed now, and macOS has many failures :(

When using native platform 0.22-milestone-4 the macOs tests pass as well...

Fired up three builds:

Linux: https://builds.gradle.org/buildConfiguration/Gradle_Check_WatchFs_36_Trigger/36797880
macOS: https://builds.gradle.org/buildConfiguration/Gradle_Check_WatchFs_32_Trigger/36797932
Windows: https://builds.gradle.org/buildConfiguration/Gradle_Check_WatchFs_38_Trigger/36797984

The Linux and Windows builds pass, the macOS build seems to fail :( https://builds.gradle.org/buildConfiguration/Gradle_Check_WatchFs_32_Trigger/36797932
I re-triggered it, though it seems to fail again: https://builds.gradle.org/buildConfiguration/Gradle_Check_WatchFs_32_Trigger/36799774

FTR, the problem was that the continuous build tests don't work on JDK14 on macOS. They do work on JDK8:
https://builds.gradle.org/buildConfiguration/Gradle_Check_WatchFs_37_Trigger/36809952
We're all good! \o/

@bot-gradle test QF please

OK, I've already triggered QuickFeedback build for you.

@bot-gradle test RfM please.

OK, I've already triggered ReadyForMerge build for you.

The performance test failure seems to be unrelated to the changes on this branch and is due to the changes on master which need to be included in this branch. I'll merge the PR now.


DBZ-1583 Propagate Replicator exception to Kafka Connect

https://issues.jboss.org/browse/DBZ-1583

Thx!


wakatime: 1.53.4 -> 1.54.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/wakatime/wakatime-cli/releases.
meta.description for wakatime is: "WakaTime command line interface"
meta.homepage for wakatime is: "https://github.com/wakatime/wakatime-cli"
meta.changelog for wakatime is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.54.0 in filename of file in /nix/store/8yl2x03s6wzzsiwyryrlys6p7hc9426s-wakatime-1.54.0

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
wakatime

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/8yl2x03s6wzzsiwyryrlys6p7hc9426s-wakatime-1.54.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/8yl2x03s6wzzsiwyryrlys6p7hc9426s-wakatime-1.54.0
ls -la /nix/store/8yl2x03s6wzzsiwyryrlys6p7hc9426s-wakatime-1.54.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.


Skewed assignment of alternatives when using traffic_fraction parameter

I used the following curl command on 1000 client_ids to check A/B traffic using traffic_fraction as 0.5 but got unexpected result which was skewed distribution of red and blue alternatives. I got red around 250 and blue around 750.  Can you look into this bug.
curl "http://localhost:5000/participate?experiment=myexperiment&alternatives=red&alternatives=blue&client_id=<provided 1000 client_ids>&traffic_fraction=0.50"

If you set traffic_fraction to 0.5, half of the traffic is omitted from the experiment is gets group A by default. The other half has a 50% change of being in group B. Therefore I would say that this is working as expected in your case.
If you want half of your clients to be in group B then you need to set traffic_fraction=1.


update license to apache 2

This is a request to change the license to apache 2, as discussed with Wim Provoost.

You shouldn't simply overwrite the (c) year in the per-file header ;-)

I wanted to indicate that this license was applied in 2015 ...
How should it be done? Just leave the years as is, or change it to (C) x, 2015 ?
p.s. I planned on pinging you once this got in, but apparently you're still subscribed to this repo :-)

Updated according to @NicolasT s feedback.
In the meantime I have been given the power (and permission) to merge this in myself ... so will merge now.


Update README.md

Add "It is fast" to summary given results of recent CBOR optimizations in Milestone v1.3.
Update Usage section and Limitations section to have the same warning and recommend using io.LimitReader with very large messages.
Add "Recent activities" under Current Status  and use shorter descriptions like this: "add struct to/from CBOR map with int keys. Simplifies using COSE data (RFC 8152)."

Codecov Report

Merging #35 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #35   +/-   ##
=======================================
  Coverage   96.98%   96.98%
=======================================
  Files           6        6
  Lines        1294     1294
=======================================
  Hits         1255     1255
  Misses         26       26
  Partials       13       13

Continue to review full report at Codecov.

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


Following Docker instructions causes error with snakemake

I followed instructions to build Docker image and then finally create an volume causes an error with snakemake.
Not sure how to report but, I do my best:
Commit hash: 260f318
same as page source: https://github.com/coderefinery/reproducible-research/blob/gh-pages/_episodes/07-docker.md
When executing step with instructions:
"
$ docker run -v <path-on-hostmachine/results_directory>:/opt/word_count/results word_count:0.1 snakemake -s Snakefile_all
"
I get following error:
"
docker run -v /home/itsme/Documents/Learning/word-count/results:/opt/word_count/results word_count:0.1 snakemake -s Snakefile_all
Traceback (most recent call last):
File "/usr/local/bin/snakemake", line 5, in
from snakemake import main
File "/usr/local/lib/python3.5/dist-packages/snakemake/init.py", line 21, in
from snakemake.workflow import Workflow
File "/usr/local/lib/python3.5/dist-packages/snakemake/workflow.py", line 23, in
from snakemake.rules import Rule, Ruleorder, RuleProxy
File "/usr/local/lib/python3.5/dist-packages/snakemake/rules.py", line 16, in
from snakemake.io import (
File "/usr/local/lib/python3.5/dist-packages/snakemake/io.py", line 1223
f"{_name}() cannot be used; attribute name reserved"
^
SyntaxError: invalid syntax
"

This most likely is not relevant issue. I have not used Docker before, and after raising this Issue, I noticed that I had to execute twicethe 'exit'-command to return to my normal shell. I do not know if it is possible to be inside of nested docker images such as: normal_shell_env->docker_container1->docker_container2? If the reported issue cannot be replicated, this can be closed.

Thanks for reporting and sorry this never got a follow-up. The lesson has now been changed to critically discuss existing definition files in the context of reproducibility rather than building containers. So this is not relevant anymore but I still very appreciate raising it.