[rom_ext,csrng] Fix rom_ext for pending csrng interrupt

Description
The test //sw/device/tests/autogen:plic_all_irqs_test_0 fails for both fpga_cw310_sival_rom_ext and silicon_owner_sival_rom_ext.
The failure is for peripheral 19 (kTopEarlgreyPlicPeripheralAdcCtrlAon): rather than 19 the test gets peripheral 27 (kTopEarlgreyPlicPeripheralCsrng), and interrupt 174 (kTopEarlgreyPlicIrqIdCsrngCsEntropyReq).
Apparently the interrupt happens right after peripherals_irqs_enable https://cs.opensource.google/opentitan/opentitan/+/master:sw/device/tests/autogen/plic_all_irqs_test.c;drc=e5b074b89ee185a45a1008415cc6839a14a033ad;l=2111 , since a call to LOG_INFO after that and before peripheral_irqs_trigger is not run. It seems there is a pending interrupt from csrng upon test entry, so I suspect the rom_ext leaves it pending.
ADC is the first peripheral in the test, so it makes sense it would be the one to fail, and the fact that both fpga_cw310 and silicon running with rom_ext fail the same way suggests it is a rom_ext issue.

Can you check if #22525 fixes this issue? I am not entirely sure it's related but it's worth a try.

The root cause for this bug is most likely #22877 (similar as for #22819).

I just re-ran the test on the FPGA //sw/device/tests/autogen:plic_all_irqs_test_0_fpga_cw310_sival_rom_ext. It's working both on master (622018b25190bc6129f6495edc98dec642b6c037) and earlgrey_es_sival (7f1ef1c112fbdb0e5b3dab8030044a4c21d2c946). It seems that the keymgr related fixes linked in the previous post indeed solve this issue.
I am closing this issue.

FYI: @nasahlpa also check the test on silicon where it's working now as well :-)


[SSH] Enable SSH connect inside

Coverage decreased (-0.02%) to 51.631% when pulling 317ceb528b02c2cc8875a2977d6eed376b1b78b0 on canwan/enable-ssh-inside into bdf828003537909dfdeefbe30b766239b9bcece3 on master.

Coverage decreased (-0.02%) to 51.631% when pulling 317ceb528b02c2cc8875a2977d6eed376b1b78b0 on canwan/enable-ssh-inside into bdf828003537909dfdeefbe30b766239b9bcece3 on master.


Fix/router migration

Full changelog

removes a duplicate migrations
fixes #2028

Baffled that request_host could even be an empty string...


Patch fails with tensorflow v1.2.0

compile_arm fails while patching when running with tensorflow v1.2.0.
Changing the WORKSPACE patch content to:
diff --git a/WORKSPACE b/WORKSPACE
index e0931512f..5b7152a67 100644
--- a/WORKSPACE
+++ b/WORKSPACE
@@ -7,6 +7,12 @@ closure_repositories()

 load("//tensorflow:workspace.bzl", "tf_workspace")

+new_local_repository(
+    name = "arm_compiler",
+    path = "%%CT_ROOT_DIR%%",
+    build_file = "BUILD.arm_compiler",
+)
+
 # Uncomment and update the paths in these entries to build the Android demo.
 #android_sdk_repository(
 #    name = "androidsdk",

temporarily fixes the issue.

Thank you for taking the time to troubleshoot the issue and providing a patch, I'm looking into this and will provide some new binaries soon !


Grok detects when pattern appears but not when it disappears

We have a (special) log file per module which is cleared on restart. Grok is configured to look for a specific pattern in those files. From the moment a module starts and the pattern appears, grok flags it, good. However, if after a restart the pattern is no longer there, grok fails somehow to refresh itself and still indicates the existence of the pattern. We've tried setting readall: true to no avail. The weird thing is that the detection always works when the pattern shows up, but it doesn't detect when it goes away. The flow is always the same; module (re)starts, a log file is written, log file contains the pattern or not. Grok is running in docker (latest) and the modules producing the log file are also running in containers, the log files are bind mounted. We have now tried to set the poll interval to see if this might solve the case (which seems unlikely as grok seems to detect changes when the pattern appears...). Is there somewhere a log file or other means how we can debug this behavior?

Hi Koen, usually users do not want a metric to disappear if the logfile becomes empty. A typical example is a metric counting the number of ERROR messages. After a while, logrotate might rename the current log file, and the application continues logging to a new empty file. However, in that case users would not expect the ERROR counter to reset to zero. Rather, the counter should remain as it is and increase when the next ERROR is logged.
That being said, there is a retention configuration for metrics where you can configure that metrics expire after some time if the pattern is not seen again. So if you configure retention to 1h, the metric will disappear after 1h if there was no more matching log line. Note that if you set retention to a value lower than 1m (1 minute), you also need to configure the retention_check_interval because by default retention is only checked every 53s (53 seconds).

This seems to have solved it, thanks for explaining.


Replace mergeInto(LibraryManager.library... for addToLibrary(...

Since emscripten 3.1.45, it is possible to use addToLibrary instead of mergeInto and having to pass LibraryManager.library as the first parameter.

The function used to add symbols the JS library has been renamed from mergeInto, to the more specific addToLibrary.  This new function does not require the passing of LibraryManager.library as a first argument.  The old mergeInto continues to exist for backwards compat.

Note to the merge team
As Godot currently builds with 3.1.39, this PR is to be merged only when emscripten will be >= 3.1.45.

We usually aim to preserve support for older compilers, so I would only add this conditionally (falling back to the previous method) if there are significant gains. Otherwise it's better IMO to keep the code compatible with earlier 3.1.x releases.

Closing as per @akien-mga's previous comment.


Update webhint

This patch updates webhint to enable Code Fixes and address a few dependency related security issues.

we also need to update CHANGELOG.md to reflect the webhint update

we also need to update CHANGELOG.md to reflect the webhint update

This patch is just for updating webhint. Not the release patch for the vscode extension itself.


Support a built-in --version option

Similar to--help, and activated when a new keyword param version is passed to a command in addition to name. I did not find something like that in the docs, so please provide a pointer if it's in there already.

It's already there.
http://click.pocoo.org/3/api/#click.version_option


Results of text exercises are shown weirdly

Describe the bug
The results of text exercises are shown weirdly.
To Reproduce
Just view the result of a text exercise.
Screenshots

Environment

 - OS: Windows 10
 - Browser: chrome
 - Version: Google Chrome Version 74.0.3729.169 (Offizieller Build) (64-Bit)

Duplicate of #444 ?

True @max-OS1


[DEVOPS-317] Windows installer: build cert synchronously

It's possible for the Windows installer to "finish" before the build
certificates script has finished. This change may fix it, although it
hasn't yet been tested.
Before merging test the resulting CI build manually.

I manually tested this twice and it has the desired effect of ensuring the installer doesn't claim to be done before the cmd window closes. cc @fersel

Looks good to me @jmitchell

@jmitchell oh I just noticed, cardano-sl-0.6-release has been merged into cardano-sl-0.6 so PR needs to be just merged there (you can pass CI since it was proven it works now)


set extension and plugin version from tag

@rmzs0711 didn't you forget to update build.gradle.kts?


Meeting

Lately there has been a lot UX discussion and changes, for me some make sense some rather less, but that's just my opinion. We should probably sit together to create a plan how we proceed further, to avoid unnecessary work and do not create irregularities.
The "weekly meeting" tomorrow could be suitable for this
CC: @teolemon @stephanegigandet @monsieurtanuki

yes, it would be great.normally, it's at 5PM, Paris time.

@monsieurtanuki @M123-dev @stephanegigandet here's the link for the meeting as a reminder: https://play.workadventu.re/@/open-food-facts/open-food-facts/open-food-facts-office

Actually, their link seems down. Here's a Google Meet for 5PM instead:  https://meet.google.com/mxe-tfnu-nod

@monsieurtanuki starting right now @ https://meet.google.com/mxe-tfnu-nod

As far as I’m concerned I don’t believe in meetings, but I hope you guys had a good one and found new ideas and solutions.

@monsieurtanuki I moved some discussions to https://github.com/openfoodfacts/smooth-app/discussions
that way, we can refine and discuss designs before implementation


folder_branch_ops: some tlfs can have an FBO without a head

If there's a local journal for it, for example.
As an added measure, don't fast-forward a folder if it wasn't ever
cleared out in the first place.
Also, this reverts the previous revert of fast-forward forcing.  New code for review is in the latest commit.
Issue: KBFS-1923

Can you make a regression test to capture this? Or would that be too onerous?


An existing LES should not need to re-calculate when viewed

This is a general issue.
It takes a long time to load a LES while the calculation is already done.
We could save the calculation to the database in order to save time. Don't quiet know what the limitations are database wise and when it comes to $.ajax. So I'll have to look into that first.

[ ] Researching options for storing large amounts of data to the database
[ ] Researching limitations of browsers

If we were to store it in the database. The thing would be, to store the 35040 points per end-point in MEDIUMBLOB type columns. Thing is though, if you have 200 end-points - it will still be slow.
The other options are:

Redis (using ENDPOINT_KEY_NAME => [ 35040 values ])
Writing to file (this seems really slow mentally but maybe in practice it's really fast)

I put some thought into this recently, but I will start by saying these ideas are off the top of my head. I have some experience of building simple caching systems, but this task will be tricky because of the amount of data we need to store at once, and the way in which we need to access it.
Requirements

We need to store at least two versions of the computed network: the “blue-line” and “green-line” loads.

The blue-line and green-line loads should be separate, since the blue-line load rarely needs to be recalculated. The green-line load, however, may be frequently recalculated as a user plays with the “technology behaviour” settings in the front-end.

The cache should be available with a Ruby API to fetch the data. For example:
cache = NetworkCache.for(TestingGround.first, strategies)
cache.node('HV Network’).load

This will allow us to extend the API at a later date with the nice-to-haves.

Ideally the API should load the minimal amount of data needed to fulfil the request. The previous example should load the “HV Network” load only. This probably means storing each node load in a separate file/cache-key/etc.

It should be possible to disable the cache for development and testing purposes, and get a fully-recalculated network each time.

Nice-to-have
Not needed in the initial version, but considerations for the future:

Store the loads of individual technologies to be shown in a chart.

Store the “storage” amounts of individual technologies, so that we may visualise how much is stored in a battery or heat-tank at each moment in time.

Cache Expiry

Green-line loads need to be expired and recalculated when a user changes the technology behaviour settings.

Blue- and green-line loads need to be expired when the user changes the technology mix.

Editing a topology should expire all caches for all LES’ using that topology.

All caches should be expired when performing a deploy to staging or production.

Storage
Storing the data in MySQL is probably not the best approach. Dumping everything into a DB column will be quite limiting, and RDBMS’ were’t really designed for this use-case.
Possibilities I see are:

Store computed networks in Memcache. Memcache settings will need to be customised to increase the available space, and special care will be needed so as not to exceed the 1MB-per-object limit (I believe it may be possible to change this?).

Store computed networks in Redis.

Save the loads to file(s) on disk.

My favourite of these options is the third. We would serialise each node load (which is just an array of numbers) using MessagePack. MessagePack was used to perform caching of large dataset objects in ETEngine, and is extremely fast (a quick test suggests we can serialise a load curve to MsgPack format and save to disk in less than 0.1 ms). The OS and file-system will cache the contents of frequently-accessed files in memory. Perhaps there is a suitable hybrid approach of serialising with MessagePack and storing in Memcache? Saving to disk is definitely easier though.
--
It would also be nice for the front-end to also have a nice API for dealing with the networks:
network = new Network(id, strategies);

network.loadFor("HV Network")
  .success(function(loads) {
    // Goes to the server to retrieve the load, or fetches it from
    // the "network" object if it has been loaded previously.
  })
  .error(function(err) {
    // ...
  });

The less we have to deal with arrays and hashes and raw HTTP requests, the better.
--
I think a useful first step would be to test and benchmark the various options. How quickly can each one serialise and de-serialise a single load curve? How about 30 curves?
Then you would document your results in this issue and propose your preferred solution. I don't want to burden you with too much bureaucracy, but caching is deceptively hard, so it would be good to design and discuss a solution before getting too deep into the implementation. :laughing:

I think a useful first step would be to test and benchmark the various options. How quickly can each one serialise and de-serialise a single load curve? How about 30 curves?

I'll write a tiny script that will do this. Benchmark is my friend.

so it would be good to design and discuss a solution before getting too deep into the implementation.

I agree :+1:

The OS and file-system will cache the contents of frequently-accessed files in memory. Perhaps there is a suitable hybrid approach of serialising with MessagePack and storing in Memcache?

Perhaps there is. I'll look into it with the script.

Benchmark is my friend.

evanphx/benchmark-ips is pretty sweet.

evanphx/benchmark-ips is pretty sweet.

Nice! :smile:

Store the loads of individual technologies to be shown in a chart.

Yes, and it would also be very nice to store loads of stakeholders at various parts in the network to be shown in a chart

Editing a topology should expire all caches for all LES’ using that topology.

Doesn't this depend on the operation? Loads will not change if names or costs are changed for instance. Only if capacities are updated should it be necessary to re-calculate right?

Results of benchmarking
Contents of script
           big-redis      0.102  (± 0.0%) i/s -      1.000  in   9.766620s
       big-memcached      0.068  (± 0.0%) i/s -      1.000  in  14.622987s
            big-file      0.100  (± 0.0%) i/s -      1.000  in  10.028063s
        mp-big-redis      0.360  (± 0.0%) i/s -      2.000  in   5.563542s
    mp-big-memcached      0.362  (± 0.0%) i/s -      2.000  in   5.520575s
         mp-big-file      0.310  (± 0.0%) i/s -      2.000  in   6.449823s
        bp-big-redis      0.349  (± 0.0%) i/s -      2.000  in   5.731799s
    bp-big-memcached      0.352  (± 0.0%) i/s -      2.000  in   5.685069s
         bp-big-file      0.299  (± 0.0%) i/s -      2.000  in   6.699207s

Comparison:
    mp-big-memcached:        0.4 i/s
        mp-big-redis:        0.4 i/s - 1.01x slower
    bp-big-memcached:        0.4 i/s - 1.03x slower
        bp-big-redis:        0.3 i/s - 1.04x slower
         mp-big-file:        0.3 i/s - 1.17x slower
         bp-big-file:        0.3 i/s - 1.21x slower
           big-redis:        0.1 i/s - 3.54x slower
            big-file:        0.1 i/s - 3.63x slower
       big-memcached:        0.1 i/s - 5.30x slower

It's a bit of tie between MessagePack-memcache or MessagePack-redis.

Yes, and it would also be very nice to store loads of stakeholders at various parts in the network to be shown in a chart

I'm not sure if we need to cache that; we might be able to pull all the individual loads out of the cache, recreate the network (without having to recalculate) and compute the stakeholder loads. We should only cache

Doesn't this depend on the operation? Loads will not change if names or costs are changed for instance. Only if capacities are updated should it be necessary to re-calculate right?

Indeed. Changes to the capacity, layout, or component names are the only operations which would warrant a cache expiry (and the last two may be prohibited, per #511).
However, for a first version expiring on any topology change is okay. We can concern ourselves with the little optimisations when we have the basics nailed. :smiley:

Results of benchmarking

Can you explain the naming conventions? I'm not sure of the difference between mp-big-memcache and bp-big-memcached, and big-memcache, ...

I'm not sure of the difference between mp-big-memcache and bp-big-memcached, and big-memcache, ...

Oh yes - sorry
big - means 200 curves
mp - means to_msgpack / MessagePack.unpack
bp - means pack("d*") / unpack("d*")

Could you put the script in a Gist? 2.5 seconds to store 200 curves seems a bit disappointing; still way better than recalculating, but I was hoping it would be faster. :disappointed:

Yes: https://gist.github.com/grdw/5c03be7090a9b65c5c76

Yes: https://gist.github.com/grdw/5c03be7090a9b65c5c76

Thanks. I made some changes.

With benchmark/ips you don't need to wrap the contents of each report block with n.times. Whereas the normal benchmark library tells us “it takes n seconds to execute this block”, benchmark/ips reports “you can execute this block n times per second (reported as ‘i/s’)”.

The raw file and Redis versions (big-file and big-redis) simply returned a string without casting the values back to Array[Float]. The non-MessagePack version looks very uncompetitive for our use-case after casting the values back to floats.

When benchmarking, you want to be testing the minimal amount of code possible. The original version created a new load curve in each benchmark iteration. If creating the random curve takes much longer than storing it in a cache, the numbers end up looking similar to one another. Instead, one random curve is created before the benchmark and reused in each test.

Test
Before
After

memcached+msg
57.1 i/s
308.9 i/s

redis+msg:
56.9 i/s
293.4 i/s

file+msg:
54.5 i/s
291.7 i/s

redis:
17.0 i/s
23.2 i/s

file:
16.9 i/s
22.6 i/s

memcached:
11.9 i/s
14.1 i/s

I separated the read and write benchmarks to be separate. Putting them together is fair if we expect their use to be split 50/50, but it's likely that reads will be far more frequent.

Write Benchmarks
                   redis     12.473  (± 8.0%) i/s -     63.000
               memcached     21.524  (± 4.6%) i/s -    108.000
                    file     24.040  (± 4.2%) i/s -    120.000
               redis+msg    772.103  (± 4.3%) i/s -      3.900k
           memcached+msg    777.646  (± 2.8%) i/s -      3.952k
                file+msg    760.444  (±39.6%) i/s -      2.646k

    Comparison:
           memcached+msg:      777.6 i/s
               redis+msg:      772.1 i/s - 1.01x slower
                file+msg:      760.4 i/s - 1.02x slower
                    file:       24.0 i/s - 32.35x slower
               memcached:       21.5 i/s - 36.13x slower
                   redis:       12.5 i/s - 62.35x slower

Read Benchmarks
                   redis      5.055  (± 0.0%) i/s -     26.000
               memcached     43.282  (± 4.6%) i/s -    220.000
                    file     42.003  (± 4.8%) i/s -    210.000
               redis+msg    427.486  (±18.2%) i/s -      1.978k
           memcached+msg    499.135  (± 6.2%) i/s -      2.499k
                file+msg    642.729  (± 6.1%) i/s -      3.250k

    Comparison:
                file+msg:      642.7 i/s
           memcached+msg:      499.1 i/s - 1.29x slower
               redis+msg:      427.5 i/s - 1.50x slower
               memcached:       43.3 i/s - 14.85x slower
                    file:       42.0 i/s - 15.30x slower
                   redis:        5.1 i/s - 127.14x slower

The data from the benchmarking +

but it's likely that reads will be far more frequent.

Concludes that file+msg is a winner.

Can this be closed @grdw?

Yes


Correct namespace for FileUtilz constant

Corrects namespace to match change spree/spree@1a2fbb40d1927668f42868984c6f03c6fa35a5f3

Thanks for this patch! :heart:


Upgrade snap base to core20

Continuing the discussion from the closed issue #15.
It seems the main hesitation remaining is dropping support for i386, however:

Linux kernel dropped support for this architecture nearly 10 years ago
There are only 8,439 images available for i386 compared to the 5,319,128 images for x64 which shows the lack of popularity for this platform
The last Ubuntu OS to support i386 was 16.04, which lost standard support last year
Intel discontinued this platform since 2007, so it's been 15 years since this was even manufactured.

There doesn't seem to be much compelling evidence to continue supporting this platform.
core22 will be released at some point this year as well, but likely not a target for this snap for some time.

I'd also love to start collecting a list of compelling reasons to update -- beyond just "less duplication of bases" what are the benefits, especially while core18 is still actively supported?

On the i386 point, we have a full order of magnitude more (metrics reporting) i386 users than either of ppc64el or s390x (which are both double digits and i386 is triple).

Hi @tianon :wave:
Regarding reasons to update the base, I have a use case would seem to require it.  I would like to add NVIDIA support.  Please see issue, conversation and initial PR here:

https://github.com/docker-snap/docker-snap/issues/91
https://github.com/docker-snap/docker-snap/pull/90

I had a go at switching to the core22 base.  The actual snapcraft changes changes required seem fairly minimal.  Please see this diff for the specifics:

https://github.com/jocado/docker-snap/compare/nvidia-support...jocado:docker-snap:core22-nvidia-support

What do you think ?

I'm going to cross my fingers and hope @xnox and/or @zhsj have some opinions/thoughts they're willing to share (probably a lot of overlap with what xnox has already put into the other threads but maybe slightly more specific about switching the base here with some of my concerns above and perhaps better ideas about how to address them or better context about whether they even matter). :bow: :+1:

i feel like creating a track for the current version of the snap, and leave it frozen at current revisions, and effectively remain as is, with base:core18 and all the legacy tooling in place.
The latest track should move on with times, and upgrade to core22 base, drop i386, gain riscv64, and move on with times.
This will include dropping upstream obsolete tooling.
In terms of users, the fast majority are on amd64 and the are roughly split between core20 and core22 users anyway based on other system atributes visible in the store. The snap users on Ubuntu Core 16 and 18 are a minority.
W.r.t. testing multi-arch support, testing arm64 things is trivial for most people (i have arm64 ubuntu thinkpad laptop, whilst general public can access free arm64 instances in oracle cloud, or use arm64 instances in azure /  gcp / aws as well). i386 is legacy, arm64 & riscv64 are the future.
And yes, I do not wish to introduce CUDA support on core18 based snaps, like at all.
This will be painful, thus I will need to further consult with people about this.

request for a legacy core18 track https://forum.snapcraft.io/t/new-legacy-track-for-docker-snap-core18/34266

core18 track has current releases out, so we can start to merge upgrades to core22 base, and enabling new things.

Great!  This is just to switch bases to start with.
https://github.com/docker-snap/docker-snap/pull/93


Support scss files for Ember apps

Hey there 👋!
Thanks for this amazing project. I'm looking forward to use it in our applications and libraries.
I was testing out the Ember CLI integration and it seems that it doesn't support .scss files. Is there intention to support it? If so, how would one import sass files (with variables) etc ?
Thanks.

Update on this 🎉
We now have preprocessor support for @css-blocks/ember-cli in master released in v0.20.0-beta.3.
You can now provide your own Block preprocessors in ember-cli-build.js like so (eyeglass optional, but encouraged):
var sass = require("node-sass");
var eyeglass = require("eyeglass");

function scssPreprocessor(file, data, _configuration, _sourceMap) {
  return new Promise((resolve, reject) => {
    const sassOptions = {
      file,
      data,
      outputStyle: "expanded",
      sourceMap: true,
      outFile: file,
      eyeglass: {},
    };
    sass.render(eyeglass(sassOptions), (err, res) => {
      if (err) {
        reject(err)
      } else {
        resolve({
          content: res.css.toString(),
          sourceMap: res.map.toString(),
          dependencies: [],
        })
      };
    });
  })
}

module.exports  = function(defaults){
 const app = new EmberApp(defaults, {
    'css-blocks': {
      parserOpts: {
        preprocessors: {
          scss: scssPreprocessor
        }
      }
    }
  });
  return app.toTree();
}

I would love to come up with a system were modules can provide auto-discoverable CSS Blocks preprocessors that can simplify the preprocessor integration story for consumers. But, thats for another RFC 🙂

Note: Going to leave this ticket open until the features are out of beta and documented 👍


Improving Documentation

This issue will accompany us for 2-3 milestones - will try improving the following documentation:

[ ] ANYWAY Products (+ Overview Diagram)
[ ] ANYWAY Teams
[ ] BackEnd Current Architecture
[ ] DB Tables documentation
[ ] New flash flow (insertion, classification and location extraction)
[ ] Infographics data creation flow

Nowadays we have some documentation docs dir - and some in github wikis
Consider either using github wikis OR use gitbook instead
@BarVolunteering @ziv17 @assafdayan what else are we missing on that list? Any thoughts?

If we use github wikis then the changes in the docs won't be in the pull request of the changed code.
Other then that it seems good.

Development environment Vs. Production Vs testing.
Flows of adding data
The static folder.


Production version

I wanna use it in a production environment, how far are we for that?

I'm not sure it will ever be production ready. The main problem is that you need to give sudo-access to the www-data user, you potentially open up an array of security leaks. It really depends on the rest of the configuration of your system whether you can safely use this.
I personally use it in a small and unimportant production environment and I've never experienced any security or other problems. But that's no guarantee.


[unittests] Send Mouse events via DisplayServer instead of push_input

Currently Unittests simplify mouse-events by just pushing them to Viewports. For dealing with mouse-screen-coordinates (caused by the introduction of multiple native Windows) it becomes necessary to extend the DisplayServer functionality for unittests.
This PR introduces DisplayServerMock based on DisplayServerHeadless, which additionally supports basic Mouse-Input handling.
This functionality is required for:

#67531
#71768

Updated 2023-01-29: Renamed DisplayServerTests to DisplayServerMock.

Haven't reviewed yet, but is there any reason why we wouldn't want to implement this mock support directly in DisplayServerHeadless?

I also thought about this but took this safe approach with a new class, because I am not familiar with where DisplayServerHeadless is used and the relevant requirements there.
Let me know if I should move the changes to DisplayServerHeadless.

Thanks!


grpc gen proto for php not working, protc-gen-php not found (I installed grpc,protobuf pecl ext)

I am using ubuntu 17.10 + php 7.1 + installed pecl grpc , protobuf extension,
And i also installed protobuf-compiler protobuf-compiler-grpc
$ which protoc
/usr/bin/protoc

$ which grpc_php_plugin
/usr/bin/grpc_php_plugin

protoc  --plugin=protoc-gen-grpc=/usr/bin/grpc_php_plugin  --php_out=src proto/blob.proto
protoc-gen-php: program not found or is not executable
--php_out: protoc-gen-php: Plugin failed with status code 1.

I am not using composer, i do not using it .
i just want to compile proto file to php. by your tools. but seems they do not working

For comparison, If you using apache thrift, you can compile thrift to php very easy
thrift -r --gen php o.thrift

protoc version 3.0.0 does not have the --php_out plugin. You will need at least 3.3.0.

hello. i installed 3.4.1   but the same result
获取 Outlook for Android
On Wed, Sep 27, 2017 at 2:03 AM +0800, "Stanley Cheung"<EMAIL_ADDRESS>wrote:
protoc version 3.0.0 does not have the --php_out plugin. You will need at least 3.3.0.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I think you are missing grpc_php_plugin file.
If you build grpc from source, grpc_php_plugin is installed as part of it under grpc/bins/opt/.
If you only want grpc_php_plugin, you can clone grpc repo and then
cd grpc
git submodule update --init
make grpc_php_plugin

Since there was no further update, I'm going to close that issue for now.


can not create directory on server!

can not create directory on server when i click FTPSync->upload!
but when i upload a file,it works.
is there anything wrong in my config file?
"default": {
	"upload_on_save": false, // set *false* if you do not want to upload on save!

	// "port": 21,
	// "tls": false,
	// "timeout": 30, // [seconds]

	// "passive": true,
	// "download_on_open": false,
	// "overwrite_newer_prevention": true,
	 "default_folder_permissions": "777",
	// "default_upload_permissions": null, // null = no action taken
	// "time_offset": 0, // [seconds]
	// "always_sync_local_permissions": true,

	// Value "auto" = use UTF-8 if availible (FEAT: UTF8), otherwise use local
	// "encoding": "auto",

	// Trade small performance impact for more stable and secure transfer (old file is intact until the download is finished)
	// "use_tempfile": true,

	// Regular expression, recommending using \b in general and /.../ for folders to avoid matching substrings
	// "ignore": "",

	// Can be used for increase of performance or to allow build scripts to finish
	// "upload_delay": 0, // [seconds]

	// Only if the server has MFMT extension installed
	// "set_remote_lastmodified": true,

	// Chmod value for files newly downloaded by FTPSync
	//   "auto" = same as on server
	//   null = no action taken
	//   "0644" = example for direct value
	// "default_local_permissions": "auto",

	// List of lists with pathnames and filenames to folders to be watched for change in between delay (upload_delay)
	//   example: after_save_watch: [ [ "code/assets/css", "*.css" ], [ "code/assets/", "*.jpg, *.png, *.gif" ] ]
	//   used only in conjunction with upload_on_save and upload_delay **
	// For more info see https://github.com/NoxArt/SublimeText2-FTPSync/wiki/Why-and-how-to-use-afterwatch
	// "after_save_watch": [],

}

Hi, the config seems OK. What Sublime version do you have?
So you mean when you upload a folder it does not get created? It should
Is there anything interesting in the console when you do it?
Thank you

version 3，it works now..


Port Cifar Augmented pipeline and its nodes

Fixes #242

cc @ericmjonas @Vaishaal
Some notes:

This lacks any unit tests right now. I wanted to get this out early to get some feedback on interfaces. I will work on unit tests soon

To do augmentation I was forced to use FunctionNode -- I don't think we have a better way to do this ? Also not being able to chain function nodes in the middle of a pipeline is a pain.

It'll be good to see if some of these APIs like RandomFlipper can be generalized like @ericmjonas mentioned in the other thread.

Alright - code-wise, this mostly looks good. The higher level question is how we want to represent this pipeline conceptually. Is it a flatmap with some key attached to the features that we can later group on (this is messy because it assumes labels and features have identical partitioning, etc.) or would we rather think of this in more general terms with some kind of ensembling. I think this is one of those situations where a whiteboard and an example pipeline might be useful, then we can decide if we have sufficient tools to represent that thing or not.

Cool - We can chat about this on Monday. I don't think we have the right machinery to do this kind of augment-ensembles cleanly yet, but we can try to use this pipeline to design things

I've fixed all the comments other the discussion about ensembling. I also introduced a new class RandomImageTransformer instead of the RandomFlipper per @ericmjonas 's request.

Alright added unit tests for the two patcher nodes and one for flipHorizontal. I'm going to run the entire pipeline on the local cluster to make sure we get the same results we got for the paper. @etrain Let me know if you have any other comments on this.

Other than the performance problem with repeated convolutions, the test error was fine. I got
16/03/21 01:05:04 INFO RandomPatchCifarAugmented: Test error is: 0.1833

from using 5120 filters (i.e. 40k features)

@etrain the latest commit https://github.com/amplab/keystone/pull/247/commits/202317e2ee045f12373c00a18b12541602575b9f updates the other CIFAR / Mnist pipeline to also use the style described in #258 -- Tests pass, but it would still be good to get another pair of eyes on this commit

This LGTM, merging. Thanks @shivaram !


Help with running multi workers

Hi all,
I trying to have a lot of agents that run frequently(about 6000 agents per 5 mins for one round).
I was follow https://github.com/cantino/huginn/wiki/Running-Many-Job-Workers install guide
After running multi job workers(three workers),
I found there are some agents have been done more than twice.
Is there a setting I have not noticed?
BR,
Austin

Hi @bluesea1001,
I have not tried the instructions in the wiki myself but I think they are missing the -i argument to give the additional workers a unique identifier. In my tests this Procfile configuration worked.

I believe that we will be removing those wiki instructions and replacing
them with @dsander's setup instructions in the repo soon too.
On Wednesday, August 19, 2015, Dominik Sander<EMAIL_ADDRESS>wrote:

Hi @bluesea1001 https://github.com/bluesea1001,
I have not tried the instructions in the wiki myself but I think they are
missing the -i argument to give the additional workers a unique
identifier. In my tests this Procfile
https://github.com/dsander/huginn/blob/97ceff62b3ce3e3d892c35c876a09a64bd77b1dd/Procfile#L42-L50
configuration worked.
—
Reply to this email directly or view it on GitHub
https://github.com/cantino/huginn/issues/977#issuecomment-132493917.

Hi @dsander @cantino ,
I was using bundle execute rake jobs:work in Procfile then set currency to add works with foreman. It is somehow like write multi lines of delay_jobs and work fine until we set lot of jobs run at the same time.
I'm simulate  the case we receive lot of events and jobs. So I add 2000 PostAgents that will send request to 2000 WebhookAgents each create event and has 2000 TestAgents receive and create new event to check to rundown is correct.
But when the test running, we found there will be some case a event propagate twice, means there will be two delay_jobs records and the agent do receive twice while we expect it should run just once. And we do not sure if there has the chance an delay_jobs will be call twice while we has huge amount agent in db
We wonder if anyone has relate experience or any advice.
Thanks.

For me the foreman concurrency option was not working at all, did you try the Procfile setup I linked earlier?

Yes, I have done as you say earlier.
But,  there are still some agents have been done more than twice.

You might need to show us an example of an Event propagating twice.  That should only be possible if you're using a non-transactional DB or there is a bug in delayed_job.

In my TestAgent's receive method , I create event with id of incoming event in payload and run for a while then check the new events, there will be some event has same incoming event id in it's payload.
I'm using MySQL.

Can you share the scenario JSON of your test setup?

I was able to reproduce it using the following script, it create 2000 Post- and WebHookAgents which are triggered by one ManualEventAgent:
trigger = User.find(1).agents.create!(type: "Agents::ManualEventAgent", name: "PostTrigger")

2000.times do |i|
  options = {"secret"=>"supersecretstring", "expected_receive_period_in_days"=>1, "payload_path"=>"payload"}
  webhook = User.find(1).agents.create!(type: 'Agents::WebhookAgent', name: "Webhook #{i}", options: options)
  options = {"post_url"=>"http://localhost/users/1/web_requests/#{webhook.id}/supersecretstring", "expected_receive_period_in_days"=>"1", "content_type"=>"form", "method"=>"post", "payload"=>{"payload"=>{"agent"=>"#{i}"}}, "headers"=>{}}
  webhook = User.find(1).agents.new(type: 'Agents::PostAgent', name: "Post #{i}", options: options)
  webhook.sources << trigger
  webhook.save!
end

After everything finished I would expect 2001 Events but most of the time get a few duplicates like @bluesea1001 mentioned.
The problem seems to be some of the Jobs can not be deleted:
   Mysql2::Error: Deadlock found when trying to get lock; try restarting transaction: DELETE FROM `delayed_jobs` WHERE `delayed_jobs`.`id` = 7767
/home/huginn/huginn/vendor/bundle/ruby/2.2.0/gems/activerecord-4.2.2/lib/active_record/connection_adapters/abstract_mysql_adapter.rb:299:in `query'
/home/huginn/huginn/vendor/bundle/ruby/2.2.0/gems/activerecord-4.2.2/lib/active_record/connection_adapters/abstract_mysql_adapter.rb:299:in `block in execute'
/home/huginn/huginn/vendor/bundle/ruby/2.2.0/gems/activerecord-4.2.2/lib/active_record/connection_adapters/abstract_adapter.rb:473:in `block in log'
/home/huginn/huginn/vendor/bundle/ruby/2.2.0/gems/activesupport-4.2.2/lib/active_support/notifications/instrumenter.rb:20:in `instrument'
/home/huginn/huginn/vendor/bundle/ruby/2.2.0/gems/activerecord-4.2.2/lib/active_record/connection_adapters/abstract_adapter.rb:467:in `log'

DJ rescues the exception and re-runs the job, because our code already ran the event is created twice. Seems to be a known but rare bug of DelayedJob which happens when you rapidly process a lot of jobs.
Not sure what we can do about it.

Oh wow, we totally ran into this last week at my day job when we upgraded to Rails 4.
We had deadlocks in DJ until we forked delayed_job_active_record and set it to use the older, less deadlock-y sql scope provided by this PR: https://github.com/collectiveidea/delayed_job_active_record/pull/111
To use it, we added this to our delayed_job initializer:
Delayed::Backend::ActiveRecord.configure do |config|
  config.reserve_sql_strategy = :default_sql
end

Could you try adding this to the Gemfile and using the above initializer and seeing it it fixes it?
gem 'delayed_job_active_record', :git => 'https://github.com/mavenlink/delayed_job_active_record', :branch => 'mavenlink'

If it does, I'll fork delayed_job_active_record and merge in that PR so that we're not reliant on Mavenlink's fork.

Got some strange invalid multibyte char (US-ASCII) errors in the debian test VM caused by the umlaut in the Gemspec, not sure if that was caused by me fiddling with Gemfile and Gemfile.lock on the production setup but we should verify it does not happen normally.
The good new is that seems to have fixes the issue, ran the test 5 times and it always worked. It's noticeably slower but i guess that is not really an issue.

Thanks for testing it.  I just forked the gem and created https://github.com/cantino/delayed_job_active_record with branch configurable-reserve-sql-strategy.  Can you point at that and see if the encoding issue is fixed?  If it is, mind making a PR?

@bluesea1001 Can you update your huginn instance and check if the issue is fixed?

Yes, We use the configuration and resolve our original problem.
But after using this solution, we can run from 6000 agents to just 3000 agents.

How many delayed job workers are you running?
On Sunday, August 23, 2015, codapie<EMAIL_ADDRESS>wrote:

Yes, We use the configuration and resolve our original problem.
But after using this solution, we can run from 6000 agents to just 3000
agents.
—
Reply to this email directly or view it on GitHub
https://github.com/cantino/huginn/issues/977#issuecomment-134054807.

Hi @cantino , It's running three DJ workers.

3000 agents is a lot.  What are you using it for, if I may ask?

We want to build lite IFTTT like services.

It should be possible to get Huginn working with this many Agents, but it will likely take some optimization on your part.  I've personally never pushed it that far.  Running more DJ workers, maybe switching to Postgresql, maybe considering using Resque instead of delayed_job, would all be options.  Is this going to be a paid service?

Thanks for your suggestion. We will try it out!
It's not paid service for user right now, we will charge money from hardware provider(I think XD).
But for now, it is totally free.

That's great, I'd love for you to be building on top of Huginn. Any attempt
to productionalize will probably require some development time on your
part, and I hope that you can send contributions back to us! :)
On Wednesday, August 26, 2015, codapie<EMAIL_ADDRESS>wrote:

Thanks for your suggestion. We will try it out!
It's not paid service for user right now, we will charge money from
hardware provider(I think XD).
But for now, it is totally free.
—
Reply to this email directly or view it on GitHub
https://github.com/cantino/huginn/issues/977#issuecomment-134899924.

I'm going to close this issue.  Please reopen if you want to discuss further!


Need to define classes for land-cover aggregation from Tiff

Description
Need to go from ~20 classes down to ~5 for regionalization.

@Zeitsperre Is the dataset the NALCMS? With 19 classes? Depending on the discretization (There is another set with over 60 classes) I can provide a classification to agregate similar use classes.

@richardarsenault Yes, the raster we ended up going forward with is the CEC NALCMS (2010). From the metadata:
The following list describes the display of land cover classification in the .img file:
Value 1, Temperate or sub-polar needleleaf forest, RGB 0 0.24 0;
Value 2, Sub-polar taiga needleleaf forest, RGB 0.58 0.61 0.44;
Value 3, Tropical or sub-tropical broadleaf evergreen forest, RGB 0 0.39 0;
Value 4, Tropical or sub-tropical broadleaf deciduous forest, RGB 0.12 0.67 0.02;
Value 5, Temperate or sub-polar broadleaf deciduous forest, RGB 0.08 0.55 0.24;
Value 6,  Mixed forest, RGB 0.36 0.46 0.17;
Value 7,  Tropical or sub-tropical shrubland, RGB 0.7 0.62 0.18;
Value 8,  Temperate or sub-polar shrubland, RGB 0.7 0.54 0.2;
Value 9,  Tropical or sub-tropical grassland, RGB 0.91 0.86 0.37;
Value 10, Temperate or sub-polar grassland, RGB 0.88 0.81 0.54;
Value 11, Sub-polar or polar shrubland-lichen-moss, RGB 0.61 0.46 0.33;
Value 12, Sub-polar or polar grassland-lichen-moss, RGB 0.73 0.83 0.56;
Value 13, Sub-polar or polar barren-lichen-moss, RGB 0.25 0.54 0.45;
Value 14, Wetland, RGB 0.42 0.64 0.54;
Value 15, Cropland, RGB 0.9 0.68 0.4;
Value 16, Barren lands, RGB 0.66 0.67 0.68;
Value 17, Urban, RGB 0.86 0.13 0.15;
Value 18, Water, RGB 0.3 0.44 0.64;
Value 19, Snow and Ice, RGB 1 0.98 1.

If you can provide suggested groupings, I can go about creating a definition that can be referenced to reclassify pixels on-the-fly and then replace the category names within the JSON or GeoJSON outputs. I should have some time to put towards this in the next week.

We can also consider integrating the MODIS Annual Land Cover imagery (500m resolution) https://lpdaac.usgs.gov/products/mcd12q1v006/. After all, 2010 was a while ago now :sweat_smile:.
Classifications for those are in either 17 (IGBP) or 15 (UMD) categories. It probably wouldn't be too hard to integrate in the future. Ideally, a 2020 version of the CEC NALCMS should be available soon, but I have no idea if this is in the works.

That would be for the maintenance phase.

OK great. Here is a list of suggested groupings:
Group 1: Forest
Value 1, Temperate or sub-polar needleleaf forest, RGB 0 0.24 0;
Value 2, Sub-polar taiga needleleaf forest, RGB 0.58 0.61 0.44;
Value 3, Tropical or sub-tropical broadleaf evergreen forest, RGB 0 0.39 0;
Value 4, Tropical or sub-tropical broadleaf deciduous forest, RGB 0.12 0.67 0.02;
Value 5, Temperate or sub-polar broadleaf deciduous forest, RGB 0.08 0.55 0.24;
Value 6,  Mixed forest, RGB 0.36 0.46 0.17;
Group 2: Shrubs
Value 7,  Tropical or sub-tropical shrubland, RGB 0.7 0.62 0.18;
Value 8,  Temperate or sub-polar shrubland, RGB 0.7 0.54 0.2;
Value 11, Sub-polar or polar shrubland-lichen-moss, RGB 0.61 0.46 0.33;
Group 3: Grass (low vegetation)
Value 9,  Tropical or sub-tropical grassland, RGB 0.91 0.86 0.37;
Value 10, Temperate or sub-polar grassland, RGB 0.88 0.81 0.54;
Value 12, Sub-polar or polar grassland-lichen-moss, RGB 0.73 0.83 0.56;
Value 13, Sub-polar or polar barren-lichen-moss, RGB 0.25 0.54 0.45;
Value 16, Barren lands, RGB 0.66 0.67 0.68;
Group 4: Wetland
Value 14, Wetland, RGB 0.42 0.64 0.54;
Group 5: Crops
Value 15, Cropland, RGB 0.9 0.68 0.4;
Group 6 : Impervious/water
Value 17, Urban, RGB 0.86 0.13 0.15;
Value 18, Water, RGB 0.3 0.44 0.64;
Value 19, Snow and Ice, RGB 1 0.98 1.
I think that would make sense from a hydrological perspective.

Working on this today. I should have updates this afternoon.


Python 3.12 compatibility

Is there a plan to make a package compatible with Python 3.12?

You might track https://github.com/lextudio/pysnmp/issues/10

Hello
stumbling on this issue, there could be an easy way to work around this by using the pyasyncore Pypi package. I don't use pysnmp myself so I did not try it out for you, sorry.

@gpatel-fr The question is whether people should rely on that package.
Nobody is actually maintaining that pyasyncore (except dumping the old files into it), so whatever bad happens the fire is on others, not you, right?

You could ask yourself another question: is answering like this to someone trying to help an efficient way of showing you as a thoroughly unpleasant character ?

@gpatel-fr You don't actually help, but copied some random information here. Don't overreact when people like me pointed out the fact that you don't like.


[PLACEHOLDER] - Place a descriptive title here

Hello there,
I am getting below error while creating resource group through Bicep script. How to resolve this error?

@alpesh-exigotech thanks for reporting the issue.
The deployment that is failing for you is trying to grant AVD enterprise application access to your pool compute and service objects, this is used by the scaling plan and start VM on connect features.
Please make sure that your Microsoft Entra ID tenant has the following app ID:

Name: could be Azure Virtual Desktop or Windows Virtual Desktop
AppID: 9cdead84-a844-4324-93f2-b2e6bb768d07

If you don't have it on you Entra ID tenant, please check if the Microsoft.DesktopVirtualization resource provider has been registered in a subscription connected to the tenant.

Thank you @danycontre for your guidance. I found that Microsoft.DesktopVirtualization was not registered in my subscription.
After registering it I start getting below error with few successful resource deployments. Can you please let me know what I am missing here.

@alpesh-exigotech please go inside the AVD-MGMT-Plane* deployment and share the details of the internal deployment that failed.

@danycontre
Are you referring this one ?

@alpesh-exigotech checking if you are still having this issue?
Did you select a group to be granted access to AVD resources?

Hi @danycontre
Thanks for follow up. Yes, I selected group access and it's working. However, I am facing some issues with Private DNS, and I believe that's it related to open issue https://github.com/Azure/bicep/issues/9839 with this. Please correct me if I am referring to older or different issues. For now, I am closing this issue and will open another one for Private DNS.


Begin docs

Begins adding the structure for the documentation.
@sdarwin when you have a chance can you please enable automated doc builds, and provide the link? Thanks.

please enable automated doc builds

OK. I am almost ready with a newer upgraded jenkins server and methodology.  Within the next couple days.  crypt can be the first service.   Go ahead and check in the docs to both master and develop branches, at your convenience.  That is necessary as a prerequisite to be able to work on the previews.


autotranslation for scala structured data

see https://gist.github.com/jarutis/f8dbcc1e41e7889572decf98d1141dcb

Time to figure how/whether ScalaEvaluator.setupAutoTranslation can be made to work?


&IF, &THEN, &ELSEIF, &ELSE, and &ENDIF preprocessor directives get parsed as errors

For example,
&IF LOOKUP("{&OPSYS}","MSDOS,WIN32":U) > 0 &THEN
    &SCOPED-DEFINE SLASH ~~~\
&ELSE
    &SCOPED-DEFINE SLASH /
&ENDIF

Gets parsed as this:

Registered: https://github.com/usagi-coffee/tree-sitter-abl/issues/88

I noticed
&then
changes to
&
then

That is probably related to this

@ccecvb yes, currently, the parser we use incorrectly understands preprocessor directives, so we are waiting for the fix on it.


Create a Cloud Service to Exchange data with Robots

Feature: Develop the cloud service Bridge
In the third generation of Lego Mindstorms,
it is possible to install a complete Linux distribution and a complete Oracle JRE.
This generation, the bricks features to connect to internet
It is necessary to develop a cloud service to exchange data with robots

    Scenario: Develop a Publish/Subscribe service to read/write topics
        Given EV3Dev robot
        And the Cloud service `Bridge`
        Then provide a way to publish/subscribe topics

Duplicated.
https://github.com/ev3dev-lang-java/bridge/issues


REPLY_COM structure has address missing

According specification it should be:
case REPLY_COM:
buf[len++] = reply->id;
buf[len++] = p->address;
buf[len++] = byte_0(p->baud_rate);
buf[len++] = byte_1(p->baud_rate);
buf[len++] = byte_2(p->baud_rate);
buf[len++] = byte_3(p->baud_rate);
break;

@z8a, Thanks for reporting.


Create Cold Start Performance benchmark

Can we skip most of the grammar checks given a flag?
Implications of grammar serialization on cold start performance?
Could the lookahead functions be generated in advance?

Could we serialize some of the results related to Lexer optimizations?

See: https://github.com/bd82/regexp-to-ast/issues/17

We do not need anything fancy, Just add a similar code snippet
like this: https://github.com/SAP/chevrotain/pull/995/files#diff-82f87be4123976abd596f21427a33109
to the lexer init phase during the last version vs Master benchmark


Update tapir-core, tapir-http4s-server, ... to 0.19.0-M16

Updates

com.softwaremill.sttp.tapir:tapir-core
com.softwaremill.sttp.tapir:tapir-http4s-server
com.softwaremill.sttp.tapir:tapir-json-circe
com.softwaremill.sttp.tapir:tapir-newtype
com.softwaremill.sttp.tapir:tapir-openapi-circe-yaml
com.softwaremill.sttp.tapir:tapir-openapi-docs
com.softwaremill.sttp.tapir:tapir-openapi-model

from 0.19.0-M4 to 0.19.0-M16.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.softwaremill.sttp.tapir" } ]

labels: library-update, early-semver-pre-release, semver-spec-pre-release

Superseded by #150.


Using standard nose coverage instead of xcover.

We realized we didn't need xcover recently in a PR over on oauth2client.
H/T to @jonparrott

LGTM


Multiple Ghost blogs?

What if you want to create 1000s of ghost blogs would you deploy this lib 1000s of times or
is there a more prudent way with only 1 or a few installations?
thanks!
i guess pretty much replicating Ghost(Pro)'s Business plan.

You would probably run a virtual machine (or better yet, a cluster) - and not use this at all. Azure Web Apps are really meant for people that don't want to run a lot of something ;-)
In that case, I'd probably just recommend a Docker Cluster in one your favorite flavors.


issue creating subdirectory

e.g., creating a "subDir" within the configDir
Path subDir = Paths.get(getConfigDir() + "/subDir");
subDirFile = new File(subDir.toFile(), "fileName.conf");

Basically, Path specifies the path of the directory, and File creates an instance of that directory

e.g., creating a "subDir" within the configDir
Path subDir = Paths.get(getConfigDir() + "/subDir");
subDirFile = new File(subDir.toFile(), "fileName.conf");

Basically, Path specifies the path of the directory, and File creates an instance of that directory

To create a subdirectory arenas within the existing directory configDir:
arenaDir = getConfigDir().resolve("arenas");
Files.createDirectory(arenaDir);


Changed the default for sync-interval from 10 seconds to 1.

The docs still stated that old behavior where the sync-interval defaulted to 10 seconds when a config file was used.

Thanks for cathing that!


[10.x] Fix Str::isUuid() returning false when given a Str::uuid()

Currently, Str::isUuid(Str::uuid()) returns false, because isUuid() only supports strings, and Str::uuid() returns a UuidInterface.
This fixes a bug described here: https://github.com/laravel/framework/pull/28412, because the uuid validation rule uses Str::isUuid().

I don't feel like this is a bug. The Str class should only work on strings, and nothing else.

The Str class should only work on strings, and nothing else.

I agree with this, but Str::isUuid(Str::uuid()) returning false definitely isn't right

I would say differently, Str::uuid() should return a string, simply because it's evident from the naming and usage form. Even considering UuidInterface has __toString method. For me changing Str::uuid() return looks better.

I dunno - this is the Str class. So, it feels like it should be focused on strings. Not sure I really want to introduce a breaking change to make it accept objects too.


About car models

is this the only cars that will [Latrax version]
[Tamiya version]. cause I have a car of my own I would like to know if it would work

I'm currently using a different car and it has been a hassle. I've managed to get everything to work up until the road following portion. It seems like a software issue though so I believe it's possible but you will most likely run into some compatibility issues. That being said, there are certain requirements that your hardware needs to meet to make sure it will work properly.

@rodsspt,
It should be possible to adapt the road following to another platform, but it would take a bit of work.  You could implement a class similar to this to support your car.
That said, it has been a while since we've run through the examples, so they may not work out-of-the-box, given the updated software dependencies.
@coltonwestern do you mind sharing the issue you faced?
Best,
John


[BUG] WireGuard Package not present after upgrade to 21.05

Previously running 21.02.2 on SG-3100 with Wireguard v0.1.2 Installed via Package Manager GUI.
Before upgrading, removed repo using the command from the wiki page. Did not remove WireGuard.
Upgraded to 21.05.
After upgrade completed, went to Package Manager GUI to upgrade WireGuard to v0.1.2_1. The WireGuard was not present either under the Installed Packages or the Available Packages.
Wireguard is still present and running on the system.

See #103
pkg remove pfSense-pkg-WireGuard, remove-repo, re-add repo, install package.

I don't have SG-3100. But nothing has changed with the builds. the way the pfSense gui works, that it won't show packages that are installed. when you do an upgrade the it remove ths patch the we do to show unofficial packages to the "Installed Packages" GUI. so remove/re-add repo should sort that out.

@gshaffer1  This is bit of a limitation of the pfSense package manager UI, its not really setup for a package occurring in multiple repos. for now the above work around w/ work.

@gshaffer1 I have updated the scripts.

remove package
remove repo
install new repo
install package should install new package if not run pkg install -r mced pfSense-pkg-WireGuard
GUI should now keep that package updated.

Again we are learning how best to do this since the pfSense gui doesn't support multiple repos, short of patching the whole UI this is bit harder.

@tcquade you can look here. https://github.com/theonemcdonald/pfSense-pkg-WireGuard/wiki/Installation but if you are not comfortable with running those commands or recovering when something goes wrong, please wait for the latest packages to make its way into the netgate repos or for 2.5.2 to be released.

Thanks!
That is exactly what I wanted.
Ted
From: vajonam @.>
Sent: 8-Jun-21 16:04
To: theonemcdonald/pfSense-pkg-WireGuard @.>
Cc: tcquade @.>; Mention @.>
Subject: Re: [theonemcdonald/pfSense-pkg-WireGuard] [BUG] WireGuard Package not present after upgrade to 21.05 (#107)
@tcquade https://github.com/tcquade  you can look here. https://github.com/theonemcdonald/pfSense-pkg-WireGuard/wiki/Installation but if you are not comfortable with running those commands or recovering when something goes wrong, please wait for the latest packages to make its way into the netgate repos or for 2.5.2 to be released.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/theonemcdonald/pfSense-pkg-WireGuard/issues/107#issuecomment-857204134 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AUBFL4REJWXNYGKVZGTWI23TR2HWLANCNFSM46E6L3WQ .  https://github.com/notifications/beacon/AUBFL4Q7WP3R6FB3LFZEVF3TR2HWLA5CNFSM46E6L3W2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOGML6LJQ.gif


input_number almost impossible to use in box mode

Home Assistant release (hass --version): 0.55
Python release (python3 --version): 3.5.3
Component/platform: input_number
Description of problem:
input_number doesn't respect step if set to 0.5 in "box" mode. It works in slider mode
Expected:
input_number should use the specified step

Additionally on Chrome Android there is no up/down arrow so it's almost impossible to change a value.

Another things, since the value is constraint by min and max, it's impossible to type a value because the field is automatically correct to respect min or max
example : min = 15, max = 30
at start value is set to 15 (min)
I would like to set 19, so I press "del" to remove "5" but the field is set to "15" again before I can type the "9"

Home Assistant release ( hass --version ): 0.55
Python release ( python3 --version ): 3.5.3
Component/platform: input_number
Description of problem:

Unable to edit value by using backspace to remove and replace.   backspace would not remove the value.
Expected:
hit end key, use backspace to remove value, then enter new value
Use of up/down arrow to adjust value creates an entry in the logbook for every press of the up/down arrows.  starting value is 5, press up arrow for 6, press up arrow again for 7, press up arrow again for 8.  Three logbook entries will be created showing change to 6, change to 7, and change to 8.  haven't checked to see if three change events are fired.
Expected:
One entry in the logbook showing value changed from 5 to 8.  Would expect to get just one event for value change from 5 to 8.

I made a pull request to add support for the step, min and max settings and to support decimal numbers.
https://github.com/home-assistant/home-assistant-polymer/pull/451

The PR https://github.com/home-assistant/home-assistant-polymer/pull/451 has been merged but the problem is still here in release 0.56.1
Now I can type 19,5 but using up/down arrow doesn't respect the step 0.5
And we still can type a value because the box auto set value to min or max during typing
Example :
min : 15
max : 30
step : 0.5
current value : 20
I want to type 19, so I click in the box to set the focus and I press backspace in order to delete 0 and 2 before typing 1 and 9 but when 0 is deleted the box is automatically set to 15 (min value)
The box should wait during typing, for example 500ms between the last keystroke and the auto correct.

The box should wait during typing, for example 500ms between the last keystroke and the auto correct.
OR just wait for 'enter' for validation.

Still an (annoying) issue.

Installed Home Assistant version 0.79.1 a few days ago on Armbian on Cubieboard2.
Problem with input_number in box mode still exists.
Issue: Input field not visible at all.
Expected: To see an input field to input data.

I agree, same problem with 0.79.3

I think that the major factor is how the browser interprets the web page presented.
When I tried different browsers I got different results
What browsers are you using and can you try a different one to see what happens ?

The same on Chrome 69 (win), Edge 42 (win) and Safari 12.0 (iPhone).

On a pc ? On Android ? On a mac ?
Can you access another browser ? On the same machine /different machine (what is the extent of the problem ?)
If it is a browser rendering problem, there's not a lot HA can do about it.
Having said that; Chrome is not a bit player in this market so should be supported to accurately render

Hi All,
I've just got back to look at this in detail and I must apologise for assuming it was a browser issue.
I (once upon a time) had issues with getting at the spinner controls of the input_number, which was fixed so I assumed that this was a similar issue.
Having reviewed it on ALL the browsers I have access to on ALL the platforms I have access to, I find the symptoms identical and the box just completely inaccessible. Please see the attached photo.

I've reviewed the notes on the increment changes through from; about 79.0 (I'm not 100% sure when it stopped working) and can find no mention of changes to web content code/rendering.
But something has changed ... and NOT for the better.
Mutt

So... it seems that in the file app-084b7564.js there is this piece of html code on line 5319.
Interestingly the tag starts with  but is closed with . I can believe that in the browser the code is being "cleaned" so the ha-slider seems to be closed in browser.
Unfortunately, this cleaning encloses the  tag in the hidden  tag.
So, if the word ha-slider will be exchanged with paper-slider in the line 5319, maybe it will work ?
I changed the line in my code, but I see no difference in the final output (in the browser).
Do I have to trigger recompilation somehow ? or remove some cached files like PYC or JS.MAP ?

@yunnanpl what do you mean by "exchanged with "?
The line allready reads "".
Do you mean "[has to be ]changed to "?
You should probably add a pull request if this really fixes it.
If you are right then this PR from 2 weeks ago may have introduced the problem in the first place: https://github.com/home-assistant/home-assistant-polymer/commit/3961eff3729d239ac5bc3526251d6c4aad1035f9#diff-ac832a113130c8082287f3037b457f02
What do you think @jeradM @balloob ?

I think !!! it's fixed.
Well it appears to me to be so on 0.81.0
Let them know if anyone out there is still having problems with it.
Mutt

ok for me on 0.81.0, thanks !

Fixed in home-assistant/home-assistant-polymer#1861 which was part of 0.81. Please open a new issue if you still see issues 👍


Fix API descriptions for ClusterQueue

Signed-off-by: tenzen-y<EMAIL_ADDRESS>
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
I fixed API descriptions for ClusterQueue.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

/ok-to-test

@alculquicondor @ahg-g @kerthcet BTW, I'd like to join the Kubernetes org. So could you be my sponsor?

Sponsored by 2 reviewers. Note the following requirements for sponsors:
Sponsors must have close interactions with the prospective member - e.g. code/design/proposal review, coordinating on issues, etc.
Sponsors must be reviewers or approvers in at least one OWNERS file within one of the Kubernetes GitHub organizations*.
Sponsors must be from multiple member companies to demonstrate integration across community.

Sure, please cc me then.

Sure, please cc me then.

Thanks for sponsoring me!

/lgtm
/approve

@alculquicondor @ahg-g @kerthcet BTW, I'd like to join the Kubernetes org. So could you be my sponsor?

Sponsored by 2 reviewers. Note the following requirements for sponsors:
Sponsors must have close interactions with the prospective member - e.g. code/design/proposal review, coordinating on issues, etc.
Sponsors must be reviewers or approvers in at least one OWNERS file within one of the Kubernetes GitHub organizations*.
Sponsors must be from multiple member companies to demonstrate integration across community.

@alculquicondor It seems that I need two people to sponsor me to join Kubernetes org.
Could you be my sponsor?

Yes, please go ahead and create the issue.


building micro/patch versions

Hello,
I am wondering if there is a way to build a devcontainer using a specific micro/patch version.  Heroku deployments require specifying the patch number, at least for python, and devcontainer seems to pull the latest, which can cause the dev container to become out of sync with the repo. I'm wondering if there is a way to override this, maybe with a dockerfile ARG?  Apologies if i am missing something obvious.

merci mon dieu
Le lun. 16 janv. 2023 à 08:10, Tyler Schlichenmeyer <
@.***> a écrit :

Hello,
I am wondering if there is a way to build a devcontainer using a specific
micro/patch version. Heroku deployments require specifying the patch
number, and devcontainer seems to pull the latest, which can cause the dev
container to become out of sync with the repo. I'm wondering if there is a
way to override this, maybe with a dockerfile ARG? Apologies if i am
missing something obvious.
—
Reply to this email directly, view it on GitHub
https://github.com/devcontainers/images/issues/330, or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5EKERLPEDH4QG5Z3ZPI3Y3WSTX65ANCNFSM6AAAAAAT4MPYSY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hi @schlich 👋
If you are using dev container images, then you can pin upto a minor version (eg. python 3.11 / 3.10). However, not to a patch version.
Have you looked at devcontainer features? With it, you could install (eg. python 3.11.1) as follows 👇
"features": {
    "ghcr.io/devcontainers/features/python:1": {
         "version": "3.11.1"
     }
}

That's a great point I hadn't considered. I've use features to pin versions of other things but for some reason hadn't considered using it for the base.  Should I use that feature on top of a node devcontainer image then to get the same result? What are the upsides and downsides of installing the feature on top of, say, a node devcontainer vs a python devcontainer?  Any caching, install time, or container size considerations?

Should I use the Python feature on top of a node devcontainer image then to get the same result?

You could, if you don't have the same pinning to patch version issue. Else, you could simply use a base:ubuntu/debian image & then use Features to install node & python.

What are the upsides and downsides of installing the feature on top of, say, a node or base linux devcontainer vs a python devcontainer?

Downsides

If you use a python image, then the time it takes to pull an image will be a little less as compared to building from source with the python Feature. We are planning to update the Python Feature to use a prebuilt binary but haven't made progress yet - https://github.com/devcontainers/features/issues/75
Python image uses the common-utils Feature which helps set the user, add themes, install common toolings, etc which would be missed with the Feature. However, using a base image or adding this Feature would help.

Upsides - You get more flexibility with versioning, and can make use of Feature options to customize the devcontainer (eg, insta)

Any caching, install time, or container size considerations?

Images & Features are both cached.
As Python Feature currently builds from source, the time needed would be little more compared to an image
I think size is almost the same, as both the image & Feature builds Python from source. Hence, shouldn't have a major difference.

an excellent and thorough response, thank you!


Test jupyter notebooks

Add Jupyter notebook to docs and test them

Awesome. Will any notebook in /docs automatically be built? Will this slow down making readthedocs? For example, should I not put large example runs in there?

Good questions @bw4sz. The documents are not run on readthedocs. They are just processed into HTML.
However, we test the notebooks to see if they are actually working here.
So we can skip the files that are heavy for GitHub actions since that is where we are doing the testing.
Right now, it tests any notebook in the docs using the wildcard. We can actually select the files that we want to test and skip the ones that may overload the testing platform.
pytest --nbmake docs/*.ipynb
or
Something like this
pytest --nbmake docs/notebooks_test_on_action*.ipynb


Add CMSG macros for unix/bsd and unix/notbsd

This adds the "Ancillary Data Object Macros" from RFC 2292. My C's weak but I think I've translated them faithfully. Tested on macOS and Linux/glibc.

r? @alexcrichton
(rust_highfive has picked a reviewer for you, use r? to override)

Any feedback?

Thanks for the PR and sorry for the delay! These look good to me though, thanks!
@bors: r+

:pushpin: Commit 2701f620d29bcd45c3d61b6f52bc4b86d17d2a68 has been approved by alexcrichton

:hourglass: Testing commit 2701f620d29bcd45c3d61b6f52bc4b86d17d2a68 with merge c2f6c44e549553645abdd7749e7ccc3a5875802e...

:boom: Test timed out

@bors: r+

@gnzlbg: :key: Insufficient privileges: Not in reviewers

r? me

@bors: retry

:hourglass: Testing commit 2701f620d29bcd45c3d61b6f52bc4b86d17d2a68 with merge 04a13d1b979e12a999fcef99437eabc0e6c806c6...

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 04a13d1b979e12a999fcef99437eabc0e6c806c6 to master...


Collecting dataset - total_instances

https://github.com/princeton-nlp/SWE-bench/blob/5583b1b3a0fc78c495a4041b907f1c61ea6b6a7d/collect/build_dataset.py#L143
total_instances is not total instances (len(prs)) but ix+1

Hi @tillwenke can you provide more context for this issue? Are you running into a bug or error message?total_instances refers to the number of candidate task instances that have been considered from all PRs so far.

No error. I just find the log confusing. It looks like this:
2023-11-07 09:03:34,209 - build_dataset - INFO - [corona-warn-app/cwa-server] ( 0 / 1 ) 0 valid, 0 with tests.
2023-11-07 09:03:43,671 - build_dataset - INFO - [corona-warn-app/cwa-server] ( 100 / 101 ) 6 valid, 4 with tests.
2023-11-07 09:04:03,965 - build_dataset - INFO - [corona-warn-app/cwa-server] ( 200 / 201 ) 17 valid, 11 with tests.
2023-11-07 09:04:47,337 - build_dataset - INFO - [corona-warn-app/cwa-server] ( 300 / 301 ) 40 valid, 28 with tests.
2023-11-07 09:05:13,544 - build_dataset - INFO - [corona-warn-app/cwa-server] ( 400 / 401 ) 53 valid, 35 with tests.
2023-11-07 09:05:31,691 - build_dataset - INFO - [corona-warn-app/cwa-server] ( 500 / 501 ) 61 valid, 40 with tests.
2023-11-07 09:05:33,634 - build_dataset - INFO - [corona-warn-app/cwa-server] ( 600 / 601 ) 62 valid, 41 with tests.
2023-11-07 09:05:33,655 - build_dataset - INFO - [corona-warn-app/cwa-server] ( 700 / 701 ) 62 valid, 41 with tests.
2023-11-07 09:05:35,242 - build_dataset - INFO - [corona-warn-app/cwa-server] ( 800 / 801 ) 63 valid, 42 with tests.
2023-11-07 09:05:35,261 - build_dataset - INFO - [corona-warn-app/cwa-server] ( 900 / 901 ) 63 valid, 42 with tests.
2023-11-07 09:05:35,282 - build_dataset - INFO - [corona-warn-app/cwa-server] ( 1000 / 1001 ) 63 valid, 42 with tests.
2023-11-07 09:05:35,302 - build_dataset - INFO - [corona-warn-app/cwa-server] ( 1100 / 1101 ) 63 valid, 42 with tests.
2023-11-07 09:05:35,322 - build_dataset - INFO - [corona-warn-app/cwa-server] ( 1200 / 1201 ) 63 valid, 42 with tests.
2023-11-07 09:05:35,343 - build_dataset - INFO - [corona-warn-app/cwa-server] ( 1300 / 1301 ) 63 valid, 42 with tests.
2023-11-07 09:05:36,985 - build_dataset - INFO - [corona-warn-app/cwa-server] ( 1400 / 1401 ) 64 valid, 42 with tests.
2023-11-07 09:05:37,005 - build_dataset - INFO - [corona-warn-app/cwa-server] ( 1500 / 1501 ) 64 valid, 42 with tests.
2023-11-07 09:05:37,029 - build_dataset - INFO - [corona-warn-app/cwa-server] ( 1600 / 1601 ) 64 valid, 42 with tests.
2023-11-07 09:05:38,864 - build_dataset - INFO - [corona-warn-app/cwa-server] ( 1700 / 1701 ) 65 valid, 43 with tests.```

For `x / y` I would expect `x / #number_of_all_instances ~ 1700`. As of now e.g. `100/101` is kinda misleading.
Thanks

Gotcha, I just updated the logging message to be a bit more straightforward ("Up to {idx} checked") instead of using total_instances. This should be more clear, idx will refer to how many PRs have been checked out so far for whether it could serve as a usable task instance.

Closing this for now, @tillwenke feel free to re-open if you feel the problem can be better addressed!


Windows support

Discussion for solutions and fixes to occur here.

we got windows support see #116

merged in 7e4c8884c1669d8de76f3d0e140a655428c147a3


Install fails

https://github.com/Stability-AI/StableSwarmUI/blame/0635d14b65f49a65d1585359bc4ba55fbdc1006e/src/Backends/NetworkBackendUtils.cs#L465
During install, this error occurs:
16:32:30.269 [Init] [Installer] Enabling ComfyUI...
16:32:30.270 [Init] [Installer] Downloading model from 'https://huggingface.co/stabilityai/stable-diffusion-xl-base-1.0/resolve/main/sd_xl_base_1.0.safetensors'... please wait...
16:32:30.270 [Init] Initializing backend #0 - ComfyUI Self-Starting...
Unhandled exception. System.InvalidOperationException: Process must exit before requested information can be determined.
at System.Diagnostics.Process.EnsureState(State state)
at System.Diagnostics.Process.get_ExitCode()
at StableSwarmUI.Backends.NetworkBackendUtils.<>c__DisplayClass13_0.g__MonitorErrLoop|3() in /storage/projects/stable-swarm-webui/StableSwarmUI/src/Backends/NetworkBackendUtils.cs:line 465
at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location ---
at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)

That error doesn't break the install, it just makes a mess in logs -- above commit fixes it to handle the edge case there cleanly


symfony ckeditor plugin integration

i downloaded my ckeditor from http://ckeditor.com/builder and i extract it into "web/bundles/sonataformatter" folder.
Any one find a solution for the problem please help ????this my code:
`
ivory_ck_editor:

    default_config:  news
    configs:
        news:

            extraPlugins: "codesnippet"
            language: %locale%
            toolbar: standard
            stylesSet: "my_styles"
            uiColor: "#FCFCFC" #Couleur de fond de l'interface
            height: '600px' #Hauteur par défaut
            contentsCss: ['bundles/app/css/style.css']
    plugins:
        codesnippet:
            path:       "/bundles/sonataformatter/vendor/ckeditor/plugins/codesnippet/"
            filename:   "plugin.js"

`
and i got this js error message :
http://localhost/sandbox-build-2.4/web/bundles/sonataformatter/vendor/ckeditor/plugins/codesnippet/plugin.js?t=E4KA".

Hum, If I understand correctly, you just want to add a custom plugin which is not shipped by default with the bundle. Am I right?
If yes, the error message seems truncated. Anyway, I think I know why it's not working. Basically, when you install a plugin, you must also install its dependencies and the dependencies of the dependencies and so on. Regarding the code snippet plugin, it requires the Widget & Dialog plugins which you have not install/configured.

I see you ping everyone on all related issues :) If you have build a custom ckeditor package with the online builder, you should have everything packaged with the dependencies I guess. Am I right? If yes, then, you can use a custom CKEditor version, just need to follow this documentation: http://symfony.com/doc/current/bundles/IvoryCKEditorBundle/usage/version.html

@famas23 Does my recommendations help you?

Closing as there is no feedbacks and I just regive a try to https://github.com/symfony/symfony-standard/compare/master...egeloen:ckeditor-plugin and it works as expected.


fix: clear particular notifier state when disabled

avoids outdated notification data if players are disabling and enabling notifiers (while plugin remains enabled)
for example: disable level notifier, gain a level, later enable level notifier, get single xp drop for same skill => previously dink would trigger a level notif (tho level up occurred much earlier)

✅ tested in-game


[BUG] Rebuild function never deletes the previous database backup

Version
5.0.18, 5.0.19
Describe the bug
Every time the LiteEngine.Rebuild method is called the database's contents are copied in a brand new backup file named with an incremental numerical suffix. The previous backup file is never deleted and all the copies just keep accumulating on disk.
Code to Reproduce
using var db = new LiteDatabase("Filename=TestDB"); db.Rebuild();
Expected behavior
There should always be only one backup copy of the database on disk while the previous one should be deleted during the Rebuild process, otherwise all these redundant copies will end up occupying precious disk storage, especially if the database size is significant.
Screenshots/Stacktrace

Additional context
I believe this happens because of the way the backupFilename is assigned in the RebuildService.Rebuild method:
var backupFilename = FileHelper.GetSuffixFile(_settings.Filename, "-backup", true);
The checkIfExists parameter sohuld not be set to true, as it will trigger the suffix to increment. Instead a check on the existence of the backup and its potential deletion could be performed before creating the new one. This, at least for me, seemed to solve the issue:
FileHelper.Exec(5, () =>
{
    if (File.Exists(backupFilename))
    {
        File.Delete(backupFilename);
    }
    File.Move(_settings.Filename, backupFilename);
});

I am faced with the same problem. Using 5.0.20

same problem, i prefer to stay in version 5.0.17

@michelebastione
did you only replace the code with
FileHelper.Exec(5, () =>
{
if (File.Exists(backupFilename))
{
File.Delete(backupFilename);
}
File.Move(_settings.Filename, backupFilename);
});
or also set this to false?
var backupFilename = FileHelper.GetSuffixFile(_settings.Filename, "-backup", true);

@mw911
Yes, you need to set the checkIfExists parameter to false or the function will keep adding increasing numerical suffixes to backupName, that's the root of the problem. The second part is to ensure that the backup file doesn't already exists when we rename the original file, or an IOException is thrown. Though you can avoid to check if the file already exists before calling File.Delete(backupFilename), as it doesn't raise any exception when you do so.

is this issue resolved in latest build 5.0.21

I'm still getting it in 5.0.21.


Services needs to be released

The FriendlyOperations service is not released but it is referenced in several released web maps.
id: e377b17ba01241ea80e74e75323e1776
The HosileOperations service is not released but it is referenced in several released web maps.
id: e51a4c093b644a7f9cd12faf7df95da6

@lfunkhouser - these are ready for your review.

@jrweakland Both items are released under owner DemoMilitary

Verified and closing


RedBlueDoors not trainable with the actual max_step

Hi Maxime,
I wasn't able to make the agent learn RedBlueDoors and I thought it was because my implementation of agent's memory was buggy but, in fact, it is because the max_step is too low.
In this line, I tried max_steps=20*size*size instead of max_steps=10*size*size and it works now.

But what is 10sizesize here? 360? It seems it really should be trainable with such a high max_steps. Possibly, a different set of hyper-parameters would work.

I don't know if it should be trainable or not with such max_step. The only thing I can do is to experiment. And actually, I can only say that I wasn't able to make it learn...
If you know hyper-parameters that work, let me know. But I didn't find.
It would be great to know that the environment proposed in your repo is learnable (without curriculum).

The question I would ask is: how often does an untrained agent succeed (percentage of episodes) in getting a reward. It has to be possible to succeed by chance within 360 steps. If the agent succeeds at all, then some learning should ideally take place, but the current hyper parameters likely don't work when the agent doesn't succeed often enough.

You are right, it is possible to get reward within 360 steps by acting randomly. However, it can't make it learn, even by testing other hyperparameters.

I feel like that's kind of a flaw with PPO, but ok, let's increase max_steps.


Python: No module named 'semantic_kernel.core_skills'

#Describe the bug
When attempting to import TimeSkill from the semantic_kernel.core_skills module, the interpreter throws a ModuleNotFoundError. This error suggests that either the module does not exist within the package or the package structure is not recognized by the interpreter.
##To Reproduce
Steps to reproduce the behavior:
Set up a new environment and install the semantic-kernel package.
Create a new Python file and insert the provided code snippet.
Run the file with the Python interpreter.
The ModuleNotFoundError occurs, stating: "No module named 'semantic_kernel.core_skills'".
##Expected behavior
The expected behavior would be for the TimeSkill class to be successfully imported from semantic_kernel.core_skills, allowing the script to run and execute the main function without import errors.
Platform
OS: Windows 10
IDE: VS Code
Language: Python

Hi @Sarfaraz021, which version of the Python package are you using? As of 0.4.6.dev0, we've renamed all skill instances to plugin.

Please upgrade to the latest package, for example, 0.9.0b1. Closing this as a non-issue. If you need further help, feel free to re-open.

@moonbox3 Thanks for the updates, am using python-3.11.5


Can I use launch_persistent_context in scrapy-playwright with headless=False?

Is it possible to use persistent context with headless browser false?
I'm using this setup, but the browser does not run.
PLAYWRIGHT_LAUNCH_OPTIONS = {
    "headless": False,
    "devtools": False,
    "args": [
        "--disable-extensions-except=/home/user/metamask-chrome/metamask",
        "--load-extension=/home/user/metamask-chrome/metamask"
    ]
}

PLAYWRIGHT_CONTEXTS = {"persistent": {"user_data_dir": str(Path.home() / "playwright-persistent-context")}}

I have the exact same problem when I set a user_data_dir in the playwright context even though headless is False the browser window does not open/show. When I do not set the user_data_dir the browser shows without any difficulties. Anyone has an idea how to change the configuration so it works in headed mode with a user data dir?

I had a closer look at the code to figure out what was going on. It looks like when using persistent mode, the PLAYWRIGHT_LAUNCH_OPTIONS setting is completely ignored. Instead, those options need to be added to the persistent context.
I used the following:
"PLAYWRIGHT_CONTEXTS": {
    "persistent": {
        "headless": False,
        "devtools": False,
        "user_data_dir": str(Path.home() / "playwright-persistent-context"),
        "args": [
            "--disable-extensions-except=/home/user/metamask-chrome/metamask",
            "--load-extension=/home/user/metamask-chrome/metamask"
        ],
    }
},

With this setting, the browser shows up including the extension. Maybe this is something to mention in the docs for how to configure a persistent context.

That's correct, persistent contexts are launched directly from the BrowserType (BrowserType.launch_persistent_context) without launching a browser instance first. I'm treating this as a documentation request then, to clarify this behavior in the README. Thank you for the research @sliverc.

78282b56ea745f4dc6c20c33cea35acb98a78054


[Feature] Add Dockerfile

Add Dockfile, the builded image can be attached in the dockerhub.

So how to use the dockerfile ?

So how to use the dockerfile ?

Install docker, and then  use the Dockerfile to build mmcls image
docker build -t mmcls:${Image_Name}

this web docker-gitbook show you how to install and use docker.


Adding sagemaker:AddTags permissions for SM-Gitlab-Example-Launch-Policy

Issue #, if available: Instantiating a new project from the gitlab template currently has a permission error:
User: arn:aws:sts::XXXXX:assumed-role/SM-Gitlab-Example-Launch-Role-XXXX/servicecatalog is not authorized to perform: sagemaker:AddTags on resource: arn:aws:sagemaker:XXX:XXXX:code-repository/sm-model-deploy-XXX because no identity-based policy allows the sagemaker:AddTags action (Service: AmazonSageMaker; Status Code: 400; Error Code: AccessDeniedException; Request ID: xxxxxxxx; Proxy: null)
Description of changes: Adding sagemaker:AddTags permissions to SM-Gitlab-Example-Launch-Policy
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@mttanke we have also just started to run into this. All new sagemaker domains are failing to create due to this missing permission. Did AWS mistakenly add this as a required permission when creating a domain?


Launcher icon updated

@RitwikSingh28 Sir the issue #6 has been resolved. Libralink logo has been added as the launcher icon of the app.

@RitwikSingh28 Sir the changes have been done. App name updated to LibraLink


ski_centers csv file for Ski

Hi guys,
Thank you for sharing your work!
I have a question, how did you get the ski_centers.csv file? I have the SkiPose-PTZ dataset, but I don't find any file with that name.
Thanks!

Please contact the owner to get it. Thanks!


onFlush and postFlush Entity Listeners

Simply check if an onFlush or postFlush entity listener is subscribed
for the current class in the entity manager during dispatch.
This is a rebased followup to #5822, per @lcobucci's review.

@alexgurrola you can rebase the branch and keep the same PR (github plays well with that)... my comments are now on the other PR and you forgot to apply them. Please apply the changes I've asked together with the ones @phansys said.
Also there was a question to @Ocramius regarding the performance impact of these changes.

@alexgurrola, I see both your pull request are closed without any merge, I am not sure about the current progress of your improvements, please can you give me an update?

@mwansinck I set aside a bit of time this morning to address the issues that were present.  A new Pull Request will be available momentarily.


Multi-World Loading

On my Genisys server, whenever it starts, it tries enabling SimplePortals, but if there are portals in worlds other than "world", then it crashes the plugin and disables it, causing my portals to not work.

SimplePortals doesn't support Genisys.