Redis initial implementation

Initial simple implementation of redis provider.

Hi, Tomasz. Please check my changes.

Thanks for resuming work on adding Redis support.
I have left few minor comments.
It would have been better if you rebased your changed on top of latest master. Without it, I see my recent commits as part of you PR. Also, the history would be more linear. If it would be easy enough for you I suggest to try to rebase and/or cherry pick.
I will need a Signed-off-by: First Last <email@domain> on every of your commits prior I merge. Check this out. On top of that, it would be good if you squash all your commits into one ;)

@zarusz Maybe it's better to create new pull request with clearing history? I'm affraid I've dubbed the commits.

And second question: in which branch should I do pull request? Master or Develop?

@diseks let's not create another pull request. Not sure how much you are git ninja, but you should be able to rename your branch locally to something else, branch from develop again, call the branch name as you have right now, cherry pick your changes (or simply copy over the new redid work), optionally squash your commits into one adding the signed off statement in comments and finally push force :)
Doing force push will make the branch point at your new commits and will be reflected on this PR.
I see you want to merge to Master, but my preference is to merge this PR to develop - you should be able to edit this on GitHub. Remember to branch your changes from develop.

@zarusz Thanks for your advice! I've done as you described.

@diseks looks really good. Thanks for basing your work on latest develop - history is more clean now.
Early next week I plan to test this with my Redis install. This should be the final check.

@diseks I did some testing with a dockerized Redis (v4.0.11 arm64). It seems that the request-response are not fully implemented. If you run the SimpleConsole sample or the RedisMessageBusIt test the responses for all requests are not being consumed at all. Check the related code comment I have left for you with a hint what I believe is missing.

Ok. I'll check and fix it.

Hey @diseks, are you still working on this? I am not sure what the next steps are.
Do you want me to add the missing request-response parts for Redis?

Hi @zarusz. Sorry for delay. Was busy. To the end of week I try to fix it.

Hi @diseks, do you still plan to finish this PR?
In few weeks I will want to build the Redis transport provider.

Hi @zarusz! Sorry for delay. Unfortunately I haven't enough much time to carefully fix issues.
Please help me to understand where missing request-response parts I should placed?

Hi @diseks, I haven't heard back and assumed you lost interest in this PR, so I went ahead and this weekend I created a first release for redis (including resquest-response):
https://www.nuget.org/packages/SlimMessageBus.Host.Redis
More testing is still needed before a stable release.
Also, since your initial contribution a year ago there was a series of refactoring done to the core runtime which cause conflicts with this PR. It was easier for me to create the transport provider from scratch based on the new runtime.
Anyways, we now have the redis provider. Feel free to close this PR or contribute based on the redis branch.


gltfpack: -kn and -mi conflict

It would seem that keep names and mesh instancing don't work together as collapsing nodes into instances would cause names to be missing.
What happens?

using -mi switch together with -kn has no effect

How to fix it?

Removing -kn produces proper instances (but nodes loose their names)

What should happen?

with both -mi -kn present i would expect instances to still be created within either node of generic name or props inherited from first instance node

That's correct and somewhat expected though. There's unfortunately no "official" way for saving instance names, and the point of -kn is to preserve named nodes so that the application can modify their transforms at will. As such I'm not sure how to improve this.

It makes sense. I also don't see solution that wouldn't complicate things. I think this issue is a sufficient documentation so closing it now.


CI3 register_shutdown_function prevents other shutdown functions from being called

Codeigniter's register_shutdown_function() is preventing my shutdown function from being called because the register function in core/Codeignter.php exits at the very end of _error_handler in Common:php line 636. Not sure if there is an intended way for custom shutdown function to be added but I add mine as a pre_system hook which isn't added to the shutdown queue until after Codeigniter's.
From php doc (http://php.net/manual/en/function.register-shutdown-function.php):

Multiple calls to register_shutdown_function() can be made, and each will be called in the same order >as they were registered. If you call exit() within one registered shutdown function, processing will stop >completely and no other registered shutdown functions will be called.

_shutdown_function() calls _error_handler(), which only exits when a fatal error is detected ... just as PHP normally behaves.
Are you saying that you want your own shutdown handler to execute something in that particular case? I'm not sure if that's a safe thing to do.

Yes, my shutdown function logs the fatal error and sends an email report.

Fatal error are already being logged ... you can either extend CI_Exception::log_exception() to send an email with that, or just register your shutdown function in index.php.


surface taskReconciliationStartedAt in SingularityState object

Explicit reconciliation can sometimes cause delays in Mesos offers. This PR adds a taskReconciliationStartedAt field to SingularityState, which makes it easier to understand when a reconciliation run is taking place.

Reconciliation stats have been reporting fine in stable, merging this


Run test case/suite in clean environment

Hey, 👋🏻
thank you for this cool project! It looks really promising. I can't wait to try it out very soon. The documentation is quite limited but I just skimmed through all source files.
One thing I'm missing is how I can ensure that each test case or suite runs in a clean NeoVim environment.
I tried to find out how other projects (partially using vusted) do this. It kinda looks like they use a before hook to reset as much as possible. Doing this correctly seems like a ton of work. There are so many things to keep care of (I don't even dare to start a listing here). So I guess it would make sense to provide such an overall reset function for everyone or even inject this globally into busted.
A more invasive but also very complete solution would be to run each test case or suite in its own NeoVim instance. I actually think this is how plenary.nvim does it. But I like vusted it's approach more...
I would love to discuss this topic and maybe contribute with a PR. Looking forward to get feedback.

Thanks!
I think this function is need.

So I guess it would make sense to provide such an overall reset function for everyone or even inject this globally into busted.

That makes sense.
I would like to provide this as vusted.helper function.
I think that simple unit tests without vim state should avoid doing too many cleanup.
So user should call the cleanup function explicitly.

The --clean flag is worth looking into. According to man nvim:
     --clean     Mimic a fresh install of Nvim. Skip loading non-builtin plugins and shada (viminfo) file.

Opening Neovim using the current VUSTED_ARGS adds the third-party plugin directory and config directory to the runtime path, while for --clean flag it does not.

nvim -u NONE -i NONE -n / nvim --clean
:echo &runtimepath

Also, I don't know which version was the flag added, so we might have to look into that as well.

The --clean flag is worth looking into.

Thanks!
Though this seems to be related another problem, that sounds good.


symlink support

Hi @blang,
This PR adds support for the Symlink method to vfs. This is pretty straight forward on all Filesystem implementations except for MemFS, of course. In MemFS, it's probably still not a perfect analog of an OS filesystem, but I think it's a close enough start.
Thanks,
d#
CC: @goutamtadi1 @larham

@dsharp-pivotal It's a bit late, but this code looks great to me. I would really appreciate it if you could make a pull request on my fork.

hmm this would be nice :)


NFAuth script error

Bug report
Your Environment

Netflix add-on version: <1.12.1>
Operating system version/name: <xbian current k5.4.85 -->
Device model:  <RasPi4 4GB-->

Used Operating system:

[ ] Android
[ ] iOS
[X] Linux
[ ] OSX
[ ] Raspberry-Pi
[ ] Windows

Describe the bug

my lubuntu desktop will not recognize chrome or brave to run the NFAuth python script.
both are current and have web dev tools installed
Expected behavior

an Auth code (file?) to login to netflix via this addon
Actual behavior

python script fails...no code/file is produced...angry people in my house...
Steps to reproduce the behavior

1 Follow directions in the wiki.
2.try again, on a windows machine
3.come here for help
Possible fix

on the windows machine, the script tried to open chrome in incognito.
and for brave/chrome on my lubuntu desktop, is it because brave has widevine installed?
Is it possible I can maybe login with the old way if I change to v1.13 of the addon, assuming my port of kodi is close enough to v19 in xbian?  I haven't tried that yet, but I can only assume that since the dev of xbian-kodi is still calling it v18.9, it's not quite the full v19 release, so it may or may not work
Debug log

The debug log can be found here:

I will attach a log file in the morning
Additional context or screenshots (if appropriate)
Other information

Screenshots

see screenshot in issue 1028

have you edited the script?
when you run the script the browser will be opened or not?

have you edited the script?
when you run the script the browser will be opened or not?

I tried with Brave first, after editing the script to point to it, then I reverted back and tried with chrome. Same error both times. And it didn’t work with Chrome on windows 10 either.

Sent from nowhere near my ready room with my new tricorder

On Jan 15, 2021, at 10:27, Stefano Gottardo<EMAIL_ADDRESS>wrote:

have you edited the script?
when you run the script the browser will be opened or not?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I tried with Brave first, after editing the script to point to it, then I reverted back and tried with chrome. Same error both times. And it didn’t work with Chrome on windows 10 either.

Sent from nowhere near my ready room with my new tricorder

On Jan 15, 2021, at 10:27, Stefano Gottardo<EMAIL_ADDRESS>wrote:

have you edited the script?
when you run the script the browser will be opened or not?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Well
i have installed Lubuntu 20.04 + python 3, after
i have installed Chrome by downloading package from Chrome website
then i have run the script, without edit it, after opened Chrome has generated the key file without problems
after this,
i have tried to install Brave browser as website instructions,
i have add the browser path in the script file /usr/bin/brave-browser
and have generated the key file without problems also with Brave
you have some type of problem in your Lubuntu
or you still doing something wrong

Well
i have installed Lubuntu 20.04 + python 3, after
i have installed Chrome by downloading package from Chrome website
then i have run the script, without edit it, after opened Chrome has generated the key file without problems
after this,
i have tried to install Brave browser as website instructions,
i have add the browser path in the script file /usr/bin/brave-browser
and have generated the key file without problems also with Brave
you have some type of problem in your Lubuntu
or you still doing something wrong

That’s...above and beyond my expectations...and some bad news. Lol.
Would you mind sharing what host machine you’ve done this on, spec-wise?  My desktop is Intel Atom - based, which might be a significant part of the problem.
But that doesn’t explain why it failed on a win10 laptop as well.
Sent from nowhere near my ready room with my new tricorder

On Jan 18, 2021, at 10:00, Stefano Gottardo<EMAIL_ADDRESS>wrote:

Well
i have installed Lubuntu 20.04 + python 3, after
i have installed Chrome by downloading package from Chrome website
then i have run the script, without edit it, after opened Chrome has generated the key file without problems
after this,
i have tried to install Brave browser as website instructions,
i have add the browser path in the script file /usr/bin/brave-browser
and have generated the key file without problems also with Brave
you have some type of problem in your Lubuntu
or you still doing something wrong
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

That’s...above and beyond my expectations...and some bad news. Lol.
Would you mind sharing what host machine you’ve done this on, spec-wise?  My desktop is Intel Atom - based, which might be a significant part of the problem.
But that doesn’t explain why it failed on a win10 laptop as well.
Sent from nowhere near my ready room with my new tricorder

On Jan 18, 2021, at 10:00, Stefano Gottardo<EMAIL_ADDRESS>wrote:

Well
i have installed Lubuntu 20.04 + python 3, after
i have installed Chrome by downloading package from Chrome website
then i have run the script, without edit it, after opened Chrome has generated the key file without problems
after this,
i have tried to install Brave browser as website instructions,
i have add the browser path in the script file /usr/bin/brave-browser
and have generated the key file without problems also with Brave
you have some type of problem in your Lubuntu
or you still doing something wrong
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

mmh sound strange that is architecture depends...
the machine have AMD cpu
on Windows is another different story, need to know/see what happen but is difficult debug because is not like a script

mmh sound strange that is architecture depends...
the machine have AMD cpu
on Windows is another different story, need to know/see what happen but is difficult debug because is not like a script

what do you plan to do with this Issue thread?
do you want continue or close it?

what do you plan to do with this Issue thread?
do you want continue or close it?


Data is not able to capture in event on hovering on series line

Version
Latest version
Link to Minimal Reproduction
No response
Steps to Reproduce

Set Trigger Line event to the chart
On hovering the series line, capture the data in mouseover event
Data regarding, (x axis & y axis) are not coming

Current Behavior

Expected Behavior
Expected the data (including x axis point & y axis point). If possible, expecting the nearest data point value also.
Environment
- OS:
- Browser:
- Framework:

Any additional comments?
No response

seems a duplicate of #18249
see Demo Code

Thanks for the replies.
I am closing this issue.
The main issue/ feature is in #18262 ticket. Can you please help me to get the nearest data point data on hovering on the series.


New Field - Vehicle Info

Stacey has asked for fields related to vehicles:
Vehicle Make/Model
License Plate Number

Where would this go?

This wasn't specified. I think for now just putting it in the main client section would make the most sense.

Channel Square Apartments also asks for the color and the make/model year of the vehicle.

Also need:

License plate state
Vehicle Identification Number

Via Stacey, we need several more fields on vehicles. In sum:

Vehicle make/model
License plate number
License plate state
Vehicle Identification Number
Driver's license number
Driver's license state
Driver's license expiration date (probably?)


redis connection bug deletes development database?

This is on Database cleaner 1.5.1
When I run 'bx rspec spec/../some_spec.rb' the development database ended up getting deleted.
It only happens when I use

DatabaseCleaner[:redis, connection: "redis://.."]

in spec/spec_helper.rb.   If I remove this code change below, then 'bx rspec ..' behaves normally - only cleans the test db.  Otherwise it cleans the development db.  Seems like a big bug, why would it suddenly act like it's operating on development environment?
+  # Get our Redis connection details:
+  redis_connection = "redis://#{YAML.load(ERB.new(File.read('config/resque.yml')).result(binding))[Rails.env]}"
+
   # Database cleaner strategy
   config.before(:suite) do
     DatabaseCleaner.strategy = :transaction
-    DatabaseCleaner[:redis].strategy = :truncation
+    DatabaseCleaner[:redis, connection: redis_connection].strategy = :truncation
     DatabaseCleaner.clean_with(:truncation)
   end

   config.before(:each) do
-    DatabaseCleaner[:redis].start
+    DatabaseCleaner[:redis, connection: redis_connection].start
     Rails.cache.clear
     DatabaseCleaner.start
   end

   config.after(:each) do
-    DatabaseCleaner[:redis].clean
+    DatabaseCleaner[:redis, connection: redis_connection].clean
     DatabaseCleaner.clean
   end

database.yml
default: &default
  adapter:  mysql2
  encoding: utf8
  host:     <IP_ADDRESS>
  database: development
  username: blah
  password:

development:
  <<: *default

# Warning: The database defined as "test" will be erased and
# re-generated from your development database when you run "rake".
# Do not set this db to the same as development or production.
test:
  <<: *default
  host: localhost
  database: test
  username: root
  password:

Nevermind.  Need to do this:  http://stackoverflow.com/a/28086122


Is there a way to get the AMI ID that was created?

Would be great if there were ENV vars set with the artifacts that were created.

i've thought about this too, how would want to see it set? what about multiple AMIs, etc. that are created?

Maybe an array of named AMIs?

I'm currently doing this with jenkins by having a downstream job, pulling out the consoleText of my packer job, extracting the artifacts "id" field and creating a properties file for each REGION/AMI pair
trigger_0.properties
REGION=us-east-1
AMI=ami-123456

trigger_1.properties:
REGION=us-west-1
AMI=ami-654321

Then using the parameterized trigger plugin with parameter factory "For Every properties file, invoke one build" to trigger what happens next for each (packer) artifact
The only unfortunate thing is the parameterized trigger plugin factories only work in the "build step" so has to be a separate job from the packer build (as a post-build step) - but having the properties files already created as (jenkins) artifacts would mean you could use the "copy artifact" plugin rather than log parsing which is somewhat cleaner.
So my pipeline is currently:

Packer Build (packer)

Runs Packer
trigger...

Packer Artifact Trigger

extracts artifact id from consoleText
builds properties files
for each property file, trigger...

Setup Integration Test

So we get two of these in the above example, with AMI + REGION parameters
updates my ec2 cloud plugin config to point to the AMI to test
which triggers...

Integration Test

spins up our application build to run on the new AMI (NodeLabel Parameter Plugin)
updates my our default ec2 image if successful

This seems to work quite well so far.
In conclusion:
I feel the best output would be a properties file for each Region/AMI created allowing a simple tie in with other existing jenkins plugins (primarily the copy artifact + parameterized trigger plugin)
If packer were a "build" job not a post-build then that would simplify the 2 jobs to one

Just wanted to note that Packer's "manifest" post-processor outputs machine-readable JSON which includes the AMI ID(s).  You will want Packer v0.12.1 or later.


runtime: make the scavenger more prompt

Currently, there's a lag of at least five minutes (and potentially much longer) between the Go heap size shrinking and the process' RSS shrinking. This is done to amortize the cost of releasing and re-acquiring system memory, but has several downsides:

It's bad for memory-constrained environments, whether that constraint is 10MB or 100GB, since it makes it basically impossible to respond to resource changes by shrinking the heap. This is particularly noticeable in server environments that allow temporarily exceeding a soft limit, since it turns this soft limit into a hard limit.
It prevents using memory for other useful purposes. There's no such thing as "unused memory". Memory not used by an application will be used by the OS for the buffer cache or the zero page cache, and free memory gives the OS the flexibility to perform optimization like transparent huge pages or NUMA migration. In the mobile sphere, this is particularly noticeable to the user, since it means less space for application and resource caching, which directly affects user experience.
Finally, it's confusing. It's clear from questions on golang-nuts and elsewhere that people expect their RSS to track their heap size and, in particular, their heap profiles.

I believe we should make the scavenger release memory promptly and aggressively.
I believe we can do this without significant overhead by improving the design of the scavenger. Currently the scavenger is careful to recycle spans that have been unused for more than five minutes. This is largely wasted effort because virtual memory is fungible: there's a slight TLB locality boost to retaining very recently used memory (on the order of a few megabytes), but beyond this it doesn't matter what unused pages we return to the OS. The inflexibility of the scavenger has several downsides. Primarily, it requires many system calls to release sparse unused regions of memory, and these system calls have a very high per-call cost because the OS needs to do remote TLB invalidation. This cost also grows with the number of CPUs. It can also needlessly delay freeing memory because coalescing two neighboring free spans takes the most recent "used" time of the two.
I propose separating the concerns of how many pages to release from which pages to release.
Which pages to release should be based on minimizing the number of sysUnused calls (madvise on Linux). Roughly, releasing n pages should attempt to release an n page span and if no spans n pages or larger are on the free list, it should release as few smaller spans as possible. There are many algorithms that satisfy this outline, and I suspect the details don't matter. It would be reasonable to adopt the exact algorithm used by tcmalloc, since that has been field-tested. We should preferentially release spans from the end of the free lists, since those have been least recently used and have the least preference for being reused.
How many pages to release is a harder question, but by separating these concerns we have the flexibility to choose a good answer. For example, the current policy can be expressed as retaining max(HeapInUse over the past 5 minutes). The "right" answer depends on the costs of releasing and re-acquiring memory, the future size of the heap, and the relative CPU versus memory costs we're willing to incur. Because of the "sawtooth" behavior of GC, we do know something about the future size of the heap, at least: the heap will grow to the heap goal in the near future, and often that heap goal is in a rough steady state. Hence, as a simple heuristic, I propose we use the heap goal times a "steady state variance" factor, with some hysteresis to retain cost amortization in case garbage collections are happening rapidly:
retain = C * max(current heap goal, max({heap goals of GCs over the past T seconds}))
C = 1 + ε = 1.1
T = 10 seconds

/cc @matloob @RLH

ping @aclements
Is there any chance we can get this into 1.8? If this is a simple change, it would make a huge difference. If it's not a simple change, it's probably too late for 1.8.

Not a simple change, I'm afraid. Moving to 1.9.

Closely related: https://github.com/golang/go/issues/14045#issuecomment-307487786

An interesting report of this problem in the field: https://medium.com/samsara-engineering/running-go-on-low-memory-devices-536e1ca2fe8f

@mknyszek, should this issue be closed now?

Yes, I think so.


ICO image - unknown image format

Hey!
Currently, we try to load ICO file and we get error: unknown image format
We using official Heroku Docker build so I think it should be supported.
https://docs.imgproxy.net/#/image_formats_support?id=ico-support
Thanks for this amazing project!  😊

Hi!
Could you give me the ICO that causes the issue?

Hey,
Sure! Attaching as ZIP, because I cannot attach ICO files here.
favicon.ico.zip

Well, that's interesting. Looks like this ICO contains an image that imgproxy can't recognize. Keeping digging.

We found a lot more ICO images that are not supported, I think most of them cannot be processed. I think they will be related somehow. 🙂

Oh, I think, I got it. ICO stores BMP with an incomplete header. That's why ICO with PNG works fine, but ICO with BMP causes an error. I'll try to solve this.

Looks like I solved this. Docker image build is still in progress, the new image will be available in an hour (yep, DockerHub's autobuild is pretty slow).

nice! thanks for all your hard work! 🎉

@DarthSim it's working! thank you!


Should not block completions when let a on the previous line

Bug Report
🔎 Search Terms
no let variable completions suggestions
⏯ Playground Link

Playground link with relevant code
💻 Code

let superLongVariable

// No completions here

🙁 Actual behavior
No completions
🙂 Expected behavior
Should not block completions

Note: it is different from https://github.com/microsoft/TypeScript/issues/54654 as it is not regression, TypeScript was always blocking it as I remember, but I do not understand why e.g. we can have something like this:
let superLongVariable
superLongVariable = 5

but I do not understand why e.g. we can have something like this:
let superLongVariable
superLongVariable = 5

Because of ASI this is essentially:
let superLongVariable;
superLongVariable = 5;

And this is valid code.

@MartinJohns I think that’s what OP meant - it is valid code, so TS should offer a completion for superLongVariable there, but doesn’t.

TypeScript was always blocking it as I remember, but I do not understand why [this is the case when] we can have something like this:

The problem is that you could be writing
let superLongVariable
= 4;

and we have logic that basically says if you're possibly about to write that =, we shouldn't offer completions.
Seems tweakable though

Actually it looks like this does already work?

Actually it looks like this does already work?

That looks like a "word completion", not a TS completion.

Same issue happens with this code:
const obj = {
  name: '',
} as const

// no completions of `obj`


[svplugins/edi_register_db] Respond to Emart EDI service update

Transfer xlrd to openpyxl
# pip uninstall xlrd
# pip install openpyxl

Facilitate backward compatibility


PREFLIGHT FAIL: NO MAG SENSOR 0

Hi all,
I just got a new Intel Aero RTF drone. Upon updating the flight controller firmware to 1.6.5, I get a preflight fail error in QGroundControl while calibrating the controller. I've tried updating the firware to the stable releases 1.7.3 and 1.8.1 but still I get the same error.
The flight controller used to work out of the box without any updates, so I am pretty sure the hardware is fine.
I found out by looking into other threads on this forum that this might happen because of the firmware version and 1.6.5 was concluded to be a stable version which had everything working fine. But, it doesnt work for me.
Kindly help!

Can you ensure your external GPS/mag is connected?

Yes, I haven't touched the hardware.
It was working before updating the firmware (All sensors were fine, I even operated it using a RC)  and just after updating it, while running calibration in QGroundControl, it throws this error.

@Addy0265 is this still a problem with 1.9?

@Addy0265 is this still a problem with 1.9?

Hi, the issue was fixed.
There was sme connection issue with the GPS/MAG module.
I tried re-soldering the connections but it still didn't work.
So I replaced it wit another GPS module of the same make and I was able to calibrate it.


Adding users to environments isn't clear

When I am in a project, and I click on "access control" it allows me to add a user. My assumption was that adding a new github user would add them to the project I was working in, but it seems that it adds them generally to the system, and creates a private project for them.
To add users to a specific project, I have to go to the manage projects page, and then add users to that project.

We designed it so that if you add a user to the project (now environment), they'd automatically be added to the site. it would basically grant them permissions so the site.
@hibooboo2 please correct me if i'm incorrect.

@deniseschannon you are correct. Just add them to the environment.

This is also why the list of users for access control is now hidden by default under the Customize button, because you probably don't need to change it.  But some extra text there could make it more obvious what you're doing affects access to the system, not to the environment.

With the various changes, I don't believe this has made it more explicitly clear on the environments side (adding members to environments adds them to the site) as well as default would require you to customize who has access to the site.
v0.51.0


Implement HTTP Zapier rule datasource

┆Issue is synchronized with this Jira Task by Unito

Work has started on this one.

Done in #107


http: superfluous response.WriteHeader call from go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp.(*respWriterWrapper).WriteHeader (wrap.go:98" msg=)

What did you do?
Multiple errors of ts=2024-02-29T16:40:47.378Z caller=stdlib.go:105 level=error component=web caller="http: superfluous response.WriteHeader call from go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp.(*respWriterWrapper).WriteHeader (wrap.go:98" msg=) in the logs - during normal operation.
What did you expect to see?
No errors.
What did you see instead? Under which circumstances?
No response
System information
Linux 4.15.0-142-generic x86_64
Prometheus version
prometheus, version 2.47.2 (branch: HEAD, revision: 3f3172cde1ee37f1c7b3a5f3d9b031190509b3ad)
  build user:       root@79f2ad339b75
  build date:       20231012-16:07:10
  go version:       go1.21.3
  platform:         linux/amd64
  tags:             netgo,builtinassets,stringlabels

Prometheus configuration file
  scrape_interval: 1m
  scrape_timeout: 50s
  evaluation_interval: 1m
  external_labels:
    thanos_distinct: lalala
rule_files:
- /etc/config/recording_rules.yml
- /etc/config/alerting_rules.yml
scrape_configs:
- job_name: pushgateway-lalala
  honor_labels: true
  honor_timestamps: true
  params:
    owner:
    - an_owner
    service:
    - serviceA
  scrape_interval: 1m
  scrape_timeout: 50s
  metrics_path: /metrics
  scheme: http
  follow_redirects: true
  enable_http2: true
  relabel_configs:
  - separator: ;
    regex: (.*)
    target_label: owner
    replacement: devx
    action: replace
  - source_labels: [__address__]
    separator: ;
    regex: (.*)
    modulus: 11
    target_label: __tmp_hash
    replacement: $1
    action: hashmod
  - source_labels: [__tmp_hash]
    separator: ;
    regex: ^10$
    replacement: $1
    action: keep
  metric_relabel_configs:
  - source_labels: [tmp]
    separator: ;
    regex: .*
    target_label: tmp
    action: replace
  - source_labels: [instance]
    separator: ;
    regex: (.*)
    target_label: exported_instance
    replacement: $1
    action: replace
  - source_labels: [owner]
    separator: ;
    regex: (.*)
    target_label: exported_owner
    replacement: $1
    action: replace
  - source_labels: [instance]
    separator: ;
    regex: ^$
    target_label: instance
    replacement: some_replacement
    action: replace
  static_configs:
  - targets:
    - sometarget.svc.cluster.local:9091
    labels:
      owner: devx
- job_name: fed-target
  honor_labels: true
  honor_timestamps: true
  params:
    match[]:
    - some_metric
  scrape_interval: 1m
  scrape_timeout: 50s
  metrics_path: /federate
  scheme: http
  follow_redirects: true
  enable_http2: true
  relabel_configs:
  - source_labels: [__address__]
    separator: ;
    regex: (.*)
    modulus: 11
    target_label: __tmp_hash
    replacement: $1
    action: hashmod
  - source_labels: [__tmp_hash]
    separator: ;
    regex: ^10$
    replacement: $1
    action: keep
  metric_relabel_configs:
  - separator: ;
    regex: ^thanos_distinct$
    replacement: $1
    action: labeldrop
  consul_sd_configs:
  - server: sometarget.svc.cluster.local:8500
    datacenter: chidc2
    tag_separator: ','
    scheme: http
    allow_stale: true
    refresh_interval: 1m
    services:
    - serviceA
    - serviceB
    follow_redirects: true
    enable_http2: true

Alertmanager version
No response
Alertmanager configuration file
No response
Logs
ts=2024-02-29T19:48:50.391Z caller=stdlib.go:105 level=error component=web caller="http: superfluous response.WriteHeader call from go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp.(*respWriterWrapper).WriteHeader (wrap.go:98" msg=)
ts=2024-02-29T19:49:15.226Z caller=stdlib.go:105 level=error component=web caller="http: superfluous response.WriteHeader call from go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp.(*respWriterWrapper).WriteHeader (wrap.go:98" msg=)
ts=2024-02-29T19:49:58.233Z caller=stdlib.go:105 level=error component=web caller="http: superfluous response.WriteHeader call from go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp.(*respWriterWrapper).WriteHeader (wrap.go:98" msg=)
ts=2024-02-29T20:08:02.707Z caller=stdlib.go:105 level=error component=web caller="http: superfluous response.WriteHeader call from go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp.(*respWriterWrapper).WriteHeader (wrap.go:98" msg=)

Same errors spotted with v2.50.1 as well.

Same error with v2.48.0

I'm looking into this.

Could you provide a simple procedure for reproducing this log message? I've tried accessing various web server paths, and am so far unable to trigger the bug.

Unfortunately now, but it seems to be happening when CPU usage is high.
I'll try to see if I can narrow it down to specific circumstances.

Thank you. From reading the code in question (wrap.go from the log message), I see they try to protect against superfluous calls to WriteHeader, so the bug isn't obvious.

@zeusal do you have the same experience, i.e. that it's non-trivial to reproduce?

We faced this error on v2.53.0 as well. No anomaly spotted in the logs and memory usage was high at this point. I almost thought the pod was OOM killed but the actual reason was this error (shown up a lot of times) followed by a segmentation fault.
 I could probably write up a small repro, but I hope this description is enough to go on based on what I've seen.
Many tutorials out there show instrumenting the individual route:
handleFunc := func(pattern string, handlerFunc func(http.ResponseWriter, *http.Request)) {
	handler := otelhttp.WithRouteTag(pattern, http.HandlerFunc(handlerFunc))
	mux.Handle(pattern, handler)
}

handleFunc("/spinwheel", spinwheel)

as well as installing a server-wide handler:
// Add HTTP instrumentation for the whole server
handler := otelhttp.NewHandler(mux, "/")

This works fine under normal operations. But if there is a problem, like possibly the side effects from high memory usage mentioned above, I see this error message. It appears when http.Error() is used, the first Error() does is write the status code in the header. But http.Error also takes a string to return. So in the Error() code, it writes the header, then writes to the ResponseWriter.
In the RespWriterWrapper, I'm seeing the writeHeader call happen twice for this error. The first time I believe is from the direct header write from http.Error(), then the second one from the write of the string content.
I think there is a bug in RespWriterWrapper's logic for writeHeader(). In the code it checks if !w.wroteHeader. If it hadn't written the header it will set the flag to true. But the problem here is that regardless of whether the flag is true or false it will still fall through and attempt to call WriteHeader at the end.
I believe this is where the original issue error message is coming from. Walking through the code execution, on the second call to RespWriterWrapper.writeHeader() the flag is correctly set to true, but it still tries to set the header anyway.

Looks like the behavior I was seeing in the last comment is fixed in the latest release: https://github.com/open-telemetry/opentelemetry-go-contrib/pull/6055


Some sm:next arrows are not visualized

See which arrows are missing:

Here is relevant script where it occurs:
statistics-extraction.sms.ttl.gz
### Tasks
- [ ] https://github.com/kbss-cvut/s-pipes-editor/issues/25

@rodionnv please investigate the issue and let me know if you get stuck

@rodionnv
I believe cd s-pipes-modules; git pull would solve the issue.

Ok, then I don't know then ... I just hope you did cd s-pipes/../s-pipes-modules instead of  cd s-pipes/s-pipes-modules.

The issue with the missing modules should be resolved by adding an import statement
owl:imports <http://onto.fel.cvut.cz/ontologies/lib/sm-module-adapter> ;
in the ontology s-pipes-modules/s-pipes.ttl.
@blcham not sure if we should pushed to s-pipes-modules

Task https://github.com/kbss-cvut/s-pipes-editor/issues/25 is resolved in https://github.com/kbss-cvut/s-pipes-editor/pull/26

@kostobog we should not need sm-module-adapter ontology. The ontology is useful only to show the pipeline in topbraid composer, which we will not use soon anymore.
What axiom from sm-module-adapter ontology you need?

sml:BindWithConstant rdfs:subClassOf sm:Modules .

@blcham need to refactor sm-module-adapter ontology to finish this issue.

I finished refactoring of RDF ontologies in s-pipes-modules and now it works correctly.

@rodionnv make sure that it works for you and close this ticket.

@blcham Yes, it works for me.


integrate folders into profile screen

This is me making this work simply. We should talk about making this more robust (like if favorites updates, making sure your profile updates too). I think maybe we should talk about sagas soon

@keybase/react-hackers this is ready to look at

@keybase/react-hackers bump

:+1:

👍
I updated the types too

:mag_right: The commits 41cdcebb0791...c8154d8d99f1 introduced some visual changes:

Changed: unlock_folders-Success (before) (after)

👍


How do I run RPNSD on a single audio file with the pretrained model?

Hi,
Thanks for making RPNSD available.   I was wondering, what if I don't want to run the experiment?  Say I have a long audio file, file.wav, can I just run RPNSD on it and get the diarization result?  For example, something like:
  ./diarize.py  --pretrained_model modelbest.pth.tar file.wav

Thanks

Hi,
Thanks for making RPNSD available. I was wondering, what if I don't want to run the experiment? Say I have a long audio file, file.wav, can I just run RPNSD on it and get the diarization result? For example, something like:
  ./diarize.py  --pretrained_model modelbest.pth.tar file.wav

Thanks

Hi, I think it is a good idea to support this and it is not very difficult based on the inference code. I think I will take a look this week. (sorry I am overwhelmed recently)
Thanks!

Hi, just checking..


Retire LicenceFinder app

Licence finder has been replaced by the Find a Licence specialist finder, so remove from K8s configuration.
https://trello.com/c/1VXQW5I1/1620-epic-retire-original-licence-finder-application

I double-checked anyway and it looks good, has received zero requests in prod since 2023-07-19 14:45 👍


Rename stack/unstack to melt/unmelt

My colleague and I have been using Deedle for a few years but never realized the existence of stack/unstack function. The normal terms I would look for in intellisense is pivot/melt that's common in pandas and Excel domain.
https://pandas.pydata.org/pandas-docs/stable/generated/pandas.DataFrame.pivot.html
https://pandas.pydata.org/pandas-docs/stable/generated/pandas.DataFrame.melt.html
Similar ideas was proposed in #217
It would be ideal to rename them to be the same as pandas.

Just realized there is a pivotTable function. Apparently I didn't go through Titanic tutorial carefully. :(
Unstack was just the reverse of Stack, a generic case of pivotTable. I would just rename stack/unstack with melt/unmelt and leave pivotTable as is.

I'm happy to rename stack/unstack to melt/unmelt. Consistency with pandas is probably a good idea!

Closed as it's addressed in #436


SqlServiceAccount: Updated Get-ServiceObject to find SSIS service

Pull Request (PR) description
Get-ServiceObject calls the Get-SqlServiceName function which searches the registry for the specified service type.  For SSIS, it correctly finds the service and returns MsDtsServer.  Get-ServiceObject then searches the $managedComputer.Services collection comparing the Name property with the returned value and fails as the collection contains MsDtsServer130.  Changed the operator from -eq to -like with an * after $serviceNameFilter.  This should still function correctly with instances as well.
This Pull Request (PR) fixes the following issues
None - found bug, but fixed it instead of submitting it.
Task list

[x] Added an entry under the Unreleased section of the change log in the CHANGELOG.md.
Entry should say what was changed, and how that affects users (if applicable).
[x] Resource documentation added/updated in README.md.
[x] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help.
[x] Comment-based help added/updated.
[ ] Localization strings added/updated in all localization files as appropriate.
[ ] Examples appropriately added/updated.
[x] Unit tests added/updated. See DSC Resource Testing Guidelines.
[ ] Integration tests added/updated (where possible). See DSC Resource Testing Guidelines.
[x] New/changed code adheres to DSC Resource Style Guidelines and Best Practices.

This change is

Codecov Report

Merging #1246 into dev will not change coverage.
The diff coverage is 100%.

@@         Coverage Diff          @@
##            dev   #1246   +/-   ##
====================================
  Coverage    98%     98%
====================================
  Files        34      34
  Lines      4171    4171
====================================
  Hits       4113    4113
  Misses       58      58

What is the protocol when the build fails, but it's not related to a change I made?  Am I to fix it?

Please let me know if there are any outstanding tasks I need to do.

Let me look into the helper function.

@twerthi Great work on this! When you have time, can you look at the lint errors here https://ci.appveyor.com/project/PowerShell/sqlserverdsc/builds/19832432?fullLog=true#L2172

@johlju The build is failing stating that there were commands missed in code coverage.  I'm not familiar with what it's trying to tell me, would you be able to provide some explanation?  I'll gladly fix it, just not sure what it's saying :)

That last commit isn't right, not validating password on disabled accounts.  I see a need where someone would want to change a password of a disabled account such as sa, which is often disabled.  I will fix that.

AppVeyor has been unable to build this project.  I have it hooked up to my repository and each failure is something different.  Mostly it's out of memory exceptions, but it will sometimes not be able to install SQL so the tests themselves fail.  Is there a way to have AppVeyor give the containers more RAM?

Default it seems the containers use as much memory as they need. https://docs.docker.com/engine/reference/run/#user-memory-constraints
Maybe the constraint is the build worker. The build worker do install several instances in parallel (the integration tests) when the containers are running unit tests. The instances (except one I think) is stopped by the integration test as a final test. Maybe before That they ett to much memory of the build worker (or the build worker does not dynamically allocate more memory fast enough).
If removing all integration tests does the containers finish successfully?
If they do, maybe we have to look how to optimize or remove integration tests.

Removed integration tests and trying on my repository ... which I think is tied to this PR so that commit is in there too.  Please don't merge until we are able to resolve.  Should I close the PR?

Hmm, looks like it still fails.  I'm a bit out of my element here, any advice?

Going to close PR and run a build just on my repository to see if the two running at the same time is causing issues.

It's hit or miss as to whether it works.  I'll reopen.

I havent had the time to look into this (help out) deeper as I have been on vacation.
Let me know your result working on the separate branch in your fork.
I will try to help out as soon as I can.

No worries, go have fun on your vacation!
I was able to finally get a build to execute to completion on my repository.  I have one outstanding issue in the Unit Tests, then I'll put the Integration Tests back and see if it all works together.

Okay, my fork is now building successfully.  Reverted my commit that removed the integration tests and letting build run.

Finally, build is succeeding.

Please let me know if there is anything more I need to do for this PR.

@twerthi I will continue the review of this PR this week. Sorry for not repsonding sooner, I have been away on vacation and the day job kept me busy. 🙂

No worries!  Hope you had a nice vacation :)

@johlju thought I'd check in on this?

No worries!  Thanks for getting back to me :)

Thanks for reviewing!  I'll go through the requested changes and try to get them done as quickly as possible.

@twerthi No problem! 😄 Sorry it took so long. I just merged another PR so you have to rebase again.

LOL!  Thankfully there haven't been any merge conflicts so it's no big deal :)

There should be merge conflicts. If none, then you haven't moved the change log entries back under the Unreleased section (they are probably under the 12.1 section or something now). 😉

@johlju, I have refactored the function searching exceptions for the SqlLogin issue to your suggested method.  I submitted a build to AppVeyor which is reporting that it is failing, but also says that all tests passed.  I reviewed the log and an unable to find the reason for the failure, would you be able to point me in the right direction?

The remaining issues are a comment I left in Reviewable and the changing of VersionNumber to Version, which I think is related to the Reviewable discussion.


State changes when audio finished playing

If the audio has finished playing the needle of the sound wave will stay at the end but the slider/needle of the markers section will disappear. Also the play/pause button does not change it's state when the audio has finished.
Also when the audio is paused the slider/needle above the markers does not change its position when seek the audio file by clicking on the sound wave, but only when playback is resumed.

@TS-Stefan could you have a look. Seems like the wavesurfer event for "end" does not stop the timeline "needle"

@WolfgangDrescher we have released a new version with a fix for your reported situation. Could you retest if it is working now as expected for you?


Fix a small warning on param info access

Vst3Parameeter::create used to do
if (info->module && info->module[0])
but clap_param_info_t defines the module a `char module[CLAP_PATH_SIZE] so it is always a realized pointer. This made macos throw the warning
address of array 'info->module' will always evaluate to 'true' [-Wpointer-bool-conversion]
correctly. So just fix it!

Just a leftover from defensive  programming strategies. :)

yeah! good instinct! I checked if the compiler was correct 3 times before changing :)


typo: acccess -> access

@stevewhims The PR passed the acceptance checklist. Please evaluate further and approve/complete the PR if appropriate. Thanks!


move Highcharts based chart generation to a separate project

Highsoft is willing to sell us an OEM licence, but we will have to ship it with a close source licence.
As a consequence, we will move the Highcharts based chart generation into a pluggable separate project, so that core Gatling can still be Apache 2 licenced.

To do so, I'll create an assembly that will generate a tar.gz with the same structure as gatling.
Users will only have to unzip the file in gatling's installation folder.
We will also provide a bundle.


Error email

When I put "node main.js" in / server / and place a custom email in the constans.js, I get the following error:
SSL Proxy listening on port 443 mx resolved: rest::getJSON localhost:200 Error on connectMx for:  { exchange: 'mail.olivos.com.pe', priority: 0 } { Error: connect ETIMEDOUT <IP_ADDRESS>:25     at Object.exports._errnoException (util.js:1026:11)     at exports._exceptionWithHostPort (util.js:1049:20)     at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1136:14)   code: 'ETIMEDOUT',   errno: 'ETIMEDOUT',   syscall: 'connect',   address: '<IP_ADDRESS>',   port: 25 } error on connectMx Error: can not connect to any SMTP server     at tryConnect (/root/CryptoVen/node_modules/sendmail/sendmail.js:88:49)     at Socket.<anonymous> (/root/CryptoVen/node_modules/sendmail/sendmail.js:94:13)     at emitOne (events.js:115:13)     at Socket.emit (events.js:210:7)     at emitErrorNT (internal/streams/destroy.js:62:8)     at _combinedTickCallback (internal/process/next_tick.js:102:11)     at process._tickCallback (internal/process/next_tick.js:161:9) Error with your mail server: can not connect to any SMTP server

When you left the mail configuration by default, the following message comes out: ```
SSL Proxy listening on port 443
mx resolved:
rest::getJSON
localhost:200
I'm missing something?

Same here.

Did you solve it ?
ty

What about the topic at the top? Is it not help? https://github.com/3s3s/opentrade/issues/319


Error reading adduct table

hello team,
While executing Lipidex jar file I get this error "Error reading adduct table".
Can you please let me know 'What I am missing here?'
Thanks
Prasad

Hi Prasad,
Thanks for giving LipiDex a try.  Could you send me the adduct table .csv found in your src/libraries folder for the library you were trying to load?
-Paul

On May 2, 2018, at 7:14 AM, prasadpb<EMAIL_ADDRESS>wrote:
hello team,
While executing Lipidex jar file I get this error "Error reading adduct table".
Can you please let me know 'What I am missing here?'
Thanks
Prasad
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks Paul for quick response.
I am not sure if I got that far; as this is the first error I got before
opening the LipiDex
[image: Screen Shot 2018-05-02 at 14.42.15.png]
On Wed, 2 May 2018 at 14:37 Paul Hutchins<EMAIL_ADDRESS>wrote:

Hi Prasad,
Thanks for giving LipiDex a try. Could you send me the adduct table .csv
found in your src/libraries folder for the library you were trying to load?
-Paul

On May 2, 2018, at 7:14 AM, prasadpb<EMAIL_ADDRESS>wrote:
hello team,
While executing Lipidex jar file I get this error "Error reading adduct
table".
Can you please let me know 'What I am missing here?'
Thanks
Prasad
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/coongroup/LipiDex/issues/1#issuecomment-385963214,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJyGMjkHe-AFmDl456Bm8b5vk-l2qWtdks5tuaijgaJpZM4TvWVa
.

Hi Prasad,
I cannot view the screenshot.  Could you try emailing it directly to<EMAIL_ADDRESS>Thanks,
Paul

Hi Prasad,
I downloaded the 1.0.1 release of LipiDex onto a different computer and did not receive this error.  Did you access LipiDex by cloning it, downloading the release, or downloading the repository? I want to make sure I am tracing exactly what you did.
Thanks,
Paul

Hi Paul,
Same here, it worked on my Mac, but when I tried it few time times on Windows (Virtual Machine) getting same error repeatedly. Is it issue with Windows or VM?
Also, do you have step-by-step simple instructions on performing data analysis from QE+ raw data files?
Is compound discoverer processing required? can I use an output from Progenesis software?
It will be further interesting to see if LipiDex is useful for shotgun/ direct infusion data where we have just m/z and MS/MS data without chromatography.
Sorry for too many questions at once :)
Thanks
Prasad

Hi Prasad,
Currently, LipiDex has been tested on Mac, Windows, and Windows Server.  I haven't tested it on a virtual machine and without knowing the specifics of your VM setup, I might not be able to replicate the problem.
For data analysis instructions, I would refer you to the LipiDex wiki or the LipiDex Testing Procedures.
LipiDex currently accepts feature tables from either Compound Discoverer or MZmine 2.
Currently, we do not have plans to incorporate shotgun/direct infusion capabilities.  Although the Spectrum Searcher utility can match MS/MS spectra, it does not include the advanced functionality of programs like LipidXplorer for multi-dimensional shotgun lipidomics.
-Paul


Update pxt.json

@pelikhan is this good to be merged? I fixed the comma

Need to test that the translations are ok

Can we merge and test it?

Easiest to test locally first

From: Abhijith Chatra<EMAIL_ADDRESS>Sent: Friday, May 29, 2020 3:16:48 PM
To: microsoft/pxt-neopixel<EMAIL_ADDRESS>Cc: Peli de Halleux<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [microsoft/pxt-neopixel] Update pxt.json (#50)
Can we merge and test it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fpxt-neopixel%2Fpull%2F50%23issuecomment-636217156&data=02|01|jhalleux%40microsoft.com|bbafbe3c6bb744ca1d6008d8041dfae8|72f988bf86f141af91ab2d7cd011db47|1|0|637263874095570184&sdata=288xtMMNF7h96KUXy%2Fg2lpD4%2F6KzfONTSlRfjGKg%2FTc%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAA73QKPZXRYUUBLWBRV4PILRUAX5BANCNFSM4K4HKOCQ&data=02|01|jhalleux%40microsoft.com|bbafbe3c6bb744ca1d6008d8041dfae8|72f988bf86f141af91ab2d7cd011db47|1|0|637263874095580141&sdata=%2FlY7OoeWVYgNQNmYvSfURNfkR0L2yixYfZvhyplMagM%3D&reserved=0.

I tested it by coping the strings. Merging.


Suggest element ID instead of name for variable suggestions

Is your feature request related to a problem? Please describe.
In Camunda, expressions have to use element ID (specifically decision ID) in order to reference variables.
Currently, the editor suggests element name which is contrary to the engine implementation, even though it's the way it's specified in DMN.
Describe the solution you'd like
Variable suggestions should use element ID. They can optionally provide element name in description.
Describe alternatives you've considered
Change the engine implementation.
Additional context

https://docs.camunda.io/docs/next/components/modeler/dmn/decision-requirements-graph/#required-decisions
https://camunda.slack.com/archives/CKGH9LR40/p1698044735402279?thread_ts=1657098884.820659&cid=CKGH9LR40

Conclusion: We are following up with this as for now.


Mills cannot claim tasks due to failed Computation that is accidentally enqueued

Describe the bug
During the stress test in halo-cmm-dev, the issue has been found when multiple mills running and the mill lock duration is short.
From the log, all mills were trying to claim task but none of them made it. Because of that, the entire duchy was halted and no progress could be made.
Steps to reproduce

In k8s configuration, set the mill lock duration to a small value, e.g., 30 second.
Create more than one replica of Mills in a duchy.
Deploy in cloud environment and create a LLv2 R/F measurement.

Component(s) affected
Duchy
Version
HEAD, v0.5.x, v0.4.x
Environment
halo-cmm-dev
Additional context
This bug can stop all mills to claim tasks so that the entire system is halted.
Root cause:

When the stage attempts exceeds the limit, mill should return immediately after failComputation() instead of processing it one more time.
The enqueueComputation should not enqueue Computations with ending stage.

Explanation:
Without the return, the mill will keep processing the stage even though failComputation has been called. The processing will eventually fail because of the token version mismatch(failComputation has updated the token to ending stage). During handleExcaption for the mismatch, the mill will get the latest token and use it to handle exception. (Note, the logic here is a bit off because the exception is caused by the previous token but the latest token might have different state.) Function handleException checks the number of attempts. However, this latest token is in ending stage forwarded by failComputation previously. So handleException sees the attempts is 1 instead of exceeding the limit. It then enqueues the Computation of ending stage and all mills will try to claim it. Claiming the failed computation is an abyss and the system will never get out without manual intervention.
Diagnosis:
Run SQL query against the duchy's database
SELECT *
FROM Computations
WHERE ComputationStage=13 AND LockExpirationTime IS NOT NULL

If the query returns any result, the issue happens.
Manually Solution:
Run SQL query against the duchy's database
UPDATE Computations
SET LockExpirationTime = NULL
WHERE (Protocol=1 AND ComputationStage=13 AND LockExpirationTime IS NOT NULL)
   OR (Protocol=2 AND ComputationStage=9 AND LockExpirationTime IS NOT NULL)

This query helps to resolve the abnormal state.

Fixed by #1626


Rename the Stories.

The proposed changes in this PR are straightforward, involving the renaming of the story titles.

PR looks good to me but can you please move the alert banner component from unstable to stable and merge to latest main code. Thanks.


Fix server selection bug in CentovaCast module

Resolves #904 from BoxBilling repo as far as I can tell.
Weird issue, seems the extension actually works perfectly fine (at least for me on PHP 8.0.19), the macros weren't included in the template and caused an issue as a result when configuring the product. (issue mentioned something about it working on an older version of PHP, this might just be a compatibility patch?)
Also, location to the API client was wrong and caused an error in one instance, this has been fixed. Didn't seem to cause any catastrophic failures on my end, but enough to throw an error.

What a weird issue. Just to check, the API class is still working with their current version?

What a weird issue. Just to check, the API class is still working with their current version?

Absolutely fine! Their "castdemo.centova.com" server disabled API access at some point. But I span up a trial instance and everything seems to work as expected.


Grow MicroProfile community

On of my suggestion https://github.com/eclipse/microprofile-marketing/issues/708

From the recent JakartaOne Livestream Chinese, a few Chinese companies either evaluating MicroProfile and using MicroProfile technologies. Two MicroProfile talks were among the JakartaOne Livestream Chinese talks.
I have introduced them to join MicroProfile Working Group as a memeber. They are evluating and discussing internally. They will reach out to me when they are going to proceed with joining the group.

Regarding to this subject: Grow MicroProfile community, under the Release MicroProfile 6.0, there is an initiative about multilingual community-outreach using blog post in multiple languages, what is WIP and it's accomplished some of its first milestones, related  issues and final outcomes are going to be listed as follow:

[ ] German  https://github.com/eclipse/microprofile-marketing/issues/737  -
- Blog Post: WIP
- Tweet: WIP
[ ] Russian  https://github.com/eclipse/microprofile-marketing/issues/736
- Blog Post: WIP
- Tweet: WIP
[x] Spanish https://github.com/eclipse/microprofile-marketing/issues/738
- Blog Post: here
- Tweet: here
[ ] Portuguese  https://github.com/eclipse/microprofile-marketing/issues/739
- Blog Post: WIP
- Tweet: WIP
[ ] Japanese https://github.com/eclipse/microprofile-marketing/issues/773
- Blog Post: here
- Tweet: WIP


Fix createmany test

this is a nice refactoring 👍
note though that this was not the cause of the failing tests. The error was coming from the createMany in the Group model. I'll make the change.


Update TrainBDT.py

Not finished but attempted to adapt script to handle multiclass BDT ROC curves via a separate plotting function

Closing since Raghav integrated changes in a separate PR


tab in terminal spits out new line

hi, when im tabbing for file/command completion in the terminal, on every tab a new line is generated, is that a desired behavior? see example attached.

Yes. If there are multiple completion suggestions it will list them there.


Modifying state from decorator component does't work

 Issue by ritz078
Friday Jun 24, 2016 at 10:18 GMT
Originally opened as https://github.com/storybooks/react-storybook-addon-info/issues/18

I am using a decorator Container that accepts the children. The children are cloned inside container so that they use the value from the container's state.
But the moment I use this plugin, it doesn't work because Story component comes in between Container and children and somehow children can't access the props passed by the Container. It just renders with the default props.
Container.js
export default class Container extends Component {
  constructor (props) {
    super(props);
    this.state = {
      value: props.value
    };
    this.handleChange = this.handleChange.bind(this);
  }

  handleChange (data) {
    this.setState({
      value: data.value
    }, () => {
      this.props.action('data changed')(data);
    });
  }

  render () {
    const self = this;
    const children = React.Children.map(this.props.children, (child) => {
      return React.cloneElement(child, {
        value: self.state.value,
        onChange: self.handleChange
      });
    });

    return (
      <div className={this.props.className}>
        {children}
      </div>
    );
  }
}

Story.js
import React  from 'react';
import { storiesOf, action } from '@kadira/storybook';
import { Switch } from '../components';
import Container from './Container';

storiesOf('Switch Button')
  .addDecorator((story) => <Container action={action} value={false}>{story()}</Container>)
  .addWithInfo('Basic', 'hello',  () => (
    <Switch name='basic' />
  ))

Now when I click on Switch, its state doesn't change but when I replace addWithInfo with add, it works.

Will be fixed with #1147


Clickhouse instance node selector

When I set custom node selector, for clickhouse instance, they are not being passed to the chi pods that are generated. But its working fine when it comes to clickhouse operator
Please look and it and let me know if I am missing something
Here is my config
clickhouse:
    installCustomStorageClass: true
    nodeSelector:
      component: stateful
    tolerations:
      - effect: NoSchedule
        key: component
        operator: Equal
        value: stateful
    persistence:
      size: 15Gi
    resources:
      requests:
        cpu: 1000m
        memory: 1000Mi
        limits:
          cpu: 2000m
          memory: 4Gi
    settings:
      prometheus/endpoint: /metrics
      prometheus/port: 9363
      prometheus/metrics: true
      prometheus/events: true
      prometheus/asynchronous_metrics: true

    zookeeper:
      nodeSelector:
        component: stateful
      tolerations:
        - effect: NoSchedule
          key: component
          operator: Equal
          value: stateful

    clickhouseOperator:
      nodeSelector:
        component: stateful
      tolerations:
        - effect: NoSchedule
          key: component
          operator: Equal
          value: stateful

@sarsony1 Looks like the ClickHouse Installation is not passing the node selector to the CHI pods.
Let me look into this.
Meanwhile, can you use the affinity and podDistribution (in case of multiple shards) instead?
clickhouse:
  affinity:
      nodeAffinity:
        requiredDuringSchedulingIgnoredDuringExecution:
          nodeSelectorTerms:
          - matchExpressions:
            - key: component
              operator: In
              values:
              - stateful
    podDistribution:
      - type: ReplicaAntiAffinity
        topologyKey: kubernetes.io/hostname
      - type: MaxNumberPerNode
        maxNumber: 1
        topologyKey: kubernetes.io/hostname

Even nodeAffinity is not being passed to the pods. I can see that in the ClickhouseInstance resource when I dry run but the same is not being reflecting in the pods.

@sarsony1 We have been using the nodeAffinity and podDistribution, which it works fine for us.
Can you check if the pods are running on the desired pods?


[bug] Error: could not detect network (event="noNetwork", code=NETWORK_ERROR, version=providers/5.7.2)

Is there an existing issue for this?

[X] I have searched the existing issues

RainbowKit Version
0.7.3
wagmi Version
0.7.5
Current Behavior
I'm getting this error from a few days back when using Alchemy:
When Error: could not detect network (event="noNetwork", code=NETWORK_ERROR, version=providers/5.7.2)
Even just using the example from scratch, I'm getting it:

Seems to be something to do with Alchemy.
Expected Behavior
Not fail.
Steps To Reproduce
npm init @rainbow-me/rainbowkit@latest
cd my-rainbowkit-app
npm run dev
Link to Minimal Reproducible Example (CodeSandbox, StackBlitz, etc.)
https://codesandbox.io/s/zen-browser-c5mh21?file=/pages/_app.tsx
Anything else?
No response

Same for me.

Same for me.

this comes from the websocket provider right now, removed it and stopped getting this error but thats not the best solution

@soccerarenadev @brossetti1 Alchemy changed connection url, and Ethers library cannon handle it properly, so e.g AlchemyRpcProvider() won't work if you are using it. You would need to use JsonRpcProvider() or WebSocketProvider and pass full connection string. It would then work just fine ;)
Examples:
    return new providers.WebSocketProvider(
      `wss://eth-mainnet.g.alchemy.com/v2/${alchemyId}`,
      ETHEREUM_MAINNET_CHAIN_ID,
    );

    return new providers.JsonRpcProvider(
      `https://eth-mainnet.g.alchemy.com/v2/${alchemyId}`,
      ETHEREUM_MAINNET_CHAIN_ID,
    );

@soccerarenadev @brossetti1 Alchemy changed connection url, and Ethers library cannon handle it properly, so e.g AlchemyRpcProvider() won't work if you are using it. You would need to use JsonRpcProvider() or WebSocketProvider and pass full connection string. It would then work just fine ;) Examples:
    return new providers.WebSocketProvider(
      `wss://eth-mainnet.g.alchemy.com/v2/${alchemyId}`,
      ETHEREUM_MAINNET_CHAIN_ID,
    );

    return new providers.JsonRpcProvider(
      `https://eth-mainnet.g.alchemy.com/v2/${alchemyId}`,
      ETHEREUM_MAINNET_CHAIN_ID,
    );

Still, it shouldn't behave like this. It should catch the error and fallback to the next provider is given.

I am also running into this issue on a fresh build npm init @rainbow-me/rainbowkit@latest

@soccerarenadev @brossetti1 Alchemy changed connection url, and Ethers library cannon handle it properly, so e.g AlchemyRpcProvider() won't work if you are using it. You would need to use JsonRpcProvider() or WebSocketProvider and pass full connection string. It would then work just fine ;) Examples:
    return new providers.WebSocketProvider(
      `wss://eth-mainnet.g.alchemy.com/v2/${alchemyId}`,
      ETHEREUM_MAINNET_CHAIN_ID,
    );

    return new providers.JsonRpcProvider(
      `https://eth-mainnet.g.alchemy.com/v2/${alchemyId}`,
      ETHEREUM_MAINNET_CHAIN_ID,
    );

How is this configured in practice in rainbowkit
_app.js

ETHEREUM_MAINNET_CHAIN_ID is really = 1


fix: Fix importing datasource with same name but different plugin issue

Description

Earlier we were not doing the plugin check while importing the application in the destination org, this resulted in action getting connected to datasource with same name but different plugin type, which resulted in action failure.

Steps To Reproduce

Create a postgres datasource named dummy_datasource
Attach an action for this datasource
Export the application
Create new org with mongo datasource named dummy_datasource
Try to import the application in newly created org
Check the action failure as the datasource is not configured correctly

Fixes #12527 #12536
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

JUnit TC
Manual testing

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

/ok-to-test sha=d6c8d0e

/ok-to-test sha=dda6d58

/ok-to-test sha=5981c12


12/01 PM Publish

Docs Build status updates of commit 9e8d2e3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-conceptual/azps-5.1.0/migrate-from-azurerm-to-az.md
:white_check_mark:Succeeded
View (azps-5.1.0)

docs-conceptual/azps-5.1.0/new-azureps-module-az.md
:white_check_mark:Succeeded
View (azps-5.1.0)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


⚠ gorzdrav.org has degraded performance

In d621b3e, gorzdrav.org (https://gorzdrav.org) experienced degraded performance:

HTTP code: 200
Response time: 18096 ms

Resolved: gorzdrav.org performance has improved in 64d7d60 after 4 minutes.


fix: guards against scenes with no player set or loaded

Fixes https://github.com/godot-escoria/escoria-issues/issues/110

Tested this by loading room 13 which has no player scene configured and debug set to TRACE. I'm not seeing the messages I'm expecting to see based on the source code changes. Am I testing it incorrectly @BHSDuncan ?

Actually, fair point: I'm not 100% sure how this can be tested unless we have a UI that doesn't extend ESCGame. The "fix" was already in for the 9-verbs UI; however, what I did applies to the base ESCGame and, as such, is unlikely to be hit.

Ok on the principle.
To fix @balloonpopper, the only way is to explicitely call the overriden methods using:
.left_click_on_bg(position)

But this has to be done by gamedev.


Stable version tag for Docker

Description
Would it be possible to have a stable version tag? Something like an alias called 1-stable that will always return the latest but stable build for version 1?

We could. This has already been discussed to also change what 1 and 1.11 tag means.
Actually 1 and 1.11 mean latest 1.11.x tag (what I would call stable) and that the default comportement of the drone plugin we use.
From my point of view that would add another pipeline configuration for such case which add complexity to a file that is already complex. I think that people that would use this tag could easely build the image locally from the last commit themself.
To be clear, this is only my point of view and why I am not in favor but if someone want to do it and it get 2 validation I wouldn't block because maybe and simply doesn't see the full need.

Ah, in that case 1 is what I'm looking for and I'm happy with it. :+1:


1, remove apiserver from agent

2, sse events as message passing chan
Signed-off-by: kevin xu<EMAIL_ADDRESS>

Codecov Report

Merging #481 into master will increase coverage by 0.05%.
The diff coverage is 26.77%.

@@            Coverage Diff             @@
##           master     #481      +/-   ##
==========================================
+ Coverage   42.63%   42.69%   +0.05%
==========================================
  Files          50       48       -2
  Lines       12613    12448     -165
==========================================
- Hits         5378     5315      -63
+ Misses       5953     5832     -121
- Partials     1282     1301      +19

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 0c4c056...3f6a4c2. Read the comment docs.


Add thunkify function

This PR adds a new, simple thunkify function. It takes an fn function and returns a new function that asks for arguments before returning a third function that, when called, invokes fn with the provided arguments.
const thunkify = fn => (...args) => () => fn(...args);

Original PR idea came from @CrossEye at a Stackoverflow answer.
Motivation

While very simple, wrapping functions to delay their invocations is a really common pattern.
Ramda itself does not provide any out-of-the-box tool to help with this in a straight forward manner.
It is specially useful when arguments to the thunkified function are not known until a later execution stage.
It helps with creating composable functions that get arguments from composed chains.
The fn function could be variadic and can be called by the thunk with any number of arguments.

Example
This has been adapted from a real world application I'm currently working on.
function throwErrorWith(message, data) {
  throw Object.assign(new Error(message), data);
}

const thenThrowErrorWith = R.thunkify(throwErrorWith);

// ...later

const throwIfBadFormat = unless(
  validFormat,
  thenThrowErrorWith(
    'Invalid response format: expected one of <result></result> or <error></error>'
  )
);

Some considerations:

Not very keen on the thunkify name, to be honest.
This is my first PR on the project. My knowledge of the internal functions is rather limited, so I'm far from sure if this is the current "Ramda" approach to writing the solution.
There might be better ways and/or other internals I should be using. Looking forward to your input!
I'm not very familiar with Haskell-like signatures so wasn't quite sure how to describe thunkify. Help appreciated.

Not very keen on the thunkify name, to be honest.

sounds fine to me, but enthunk would be ok as well.

I came up with pointfree version if you are interested:
const getThunk = useWith(o(always), [apply]);
const enthunk = useWith(unapply, [getThunk]);

See derivation process: https://goo.gl/Y2oEme

@tommmyy Very nice. Good job! That's similar to what I previously came up with (and mentioned in original SO question). But yours actually handles variadic functions.


Scrolling with trackpad not smooth on Chromium-based browsers on Ubuntu

Hello,
I've noticed a problem when scrolling on my laptop in Chromium-based browsers.  It's hard to describe, but basically, when scrolling a document, the scroll bar will move slowly and kind-of "vibrates" back and forth, making it quite hard to actually scroll up or down. This does not happen when clicking on and dragging the scroll bar, or when using a scroll wheel on a mouse.  I'm guessing this has something to do with horizontal scrolling messing things up?
I am on Ubuntu 17.04 with Unity.  I had the problem with 16.10 too.  I also tried on Firefox, but the issue only shows up on Chrome/Chromium.  I don't know which version of CodeMirror causes this, I noticed the bug on the software Abricotine and on the online CodeMirror demos (e.g. https://codemirror.net/demo/complete.html).
Thanks for your help :)

I'm not seeing this on Chrome 57 on Debian—but it may well be that this is dependent on the events generated by the trackpad driver, in which case it'd be tricky to reproduce on different hardware.

Experiencing the same, Dell XPS 13 with Ubuntu 17.04. The bug is visible in the official demo, chromium 60.0.3112.113. Firefox 55.0.2 runs fine. @xlambein , do you by any chance have the same laptop (or same touchpad)?
xinput --list gives
⎡ Virtual core pointer                    	id=2	[master pointer  (3)]
⎜   ↳ Virtual core XTEST pointer              	id=4	[slave  pointer  (2)]
⎜   ↳ ELAN Touchscreen                        	id=10	[slave  pointer  (2)]
⎜   ↳ DLL0704:01 06CB:76AE Touchpad           	id=12	[slave  pointer  (2)]

Yup I have that one exactly.

I can confirm that this is related to horizontal two finger scroll. Deactivating it via synclient HorizTwoFingerScroll=0 leads to smooth scrolling in the codemirror demo


RFM - gcc-arm-embedded

Current Maintainer

[ ] I am the maintainer of the package and wish to pass it to someone else;

I DON'T Want To Become The Maintainer

[x] I have followed the Package Triage Process and I do NOT want to become maintainer of the package;
[x] There is no existing open maintainer request for this package;

Checklist

[x] Issue title starts with 'RFM - '

Existing Package Details
Package URL: https://chocolatey.org/packages/gcc-arm-embedded
Package source URL:
The maintainer of this package doesn't allow contact and there's no package source repository.
Please note that there's a need to have 7-2018-q2-update as well as the latest 8-2018-q4-major versions of this package.
I do not wish to maintain this package, however I attach the ones I created as starting point.
gcc-arm-embedded.<IP_ADDRESS>80627.zip
gcc-arm-embedded.<IP_ADDRESS>81220.zip

@bencefr I have looked your package. Could you tell me how you determine if an executable should be shimmed or not (for example cc1.exe is not shimmed, why and how do you determine them)?
Thanks in advance!

@pauby , I don't know what it means to shim an exe file. I simply copied the existing package and updated the contents.

@bencefr I think that last comment was intended for @chtof
Shimming is a process of making an executable available on the %path% without changing the %path% itself. So the executable can sit inside the package tools folder and still be available to run directly from the command line.
I think @chtof was asking which executables should be available on the commands line directly.

Sorry @pauby , I misaddressed my previous comment.
@chtof , I am a Unix dev and I expect availability of all tools without major inconvenience of all the awkwardness of Windows environment. I am also not the user of these packages, therefore I have no stong opinions what should or should not be available in the path, so I trust the original package.
However light-heartedly speaking, if I install something with choco and then I would need to dig the drive where something gets installed and then set my paths in weird ways, then I'd question the usefulness of the package manager.

This have now been picked up by a new maintainer.
@n3rd4i

Package has been updated with:

gcc-arm-embedded.<IP_ADDRESS>80627.zip

As for:
2. gcc-arm-embedded.<IP_ADDRESS>81220.zip

The GNU ARM Embedded Toolchain for Windows, version 8-2018-q4-major has a critical bug and should not be used.

Based on above comment this [particular] update will be deferred.

So, this ticket could be closed?

yup it sure can. Thank you @n3rd4i for stepping up and taking over the maintenance of the package.


Relics - Check if using #r is better than #ff0000 in relic descriptions (from Discord's GK)

It is.


Add requirement for String:CRC32 to README.md

The GUI silently fails, but running the script manually revealed that
String:CRC32 was an expected dependency. This is using the most recent
version of Strawberry Perl installed via chocolatey.
Installing String:CRC32 resolved the error and allowed the script to run
successfully.

Thank you for your contribution!


Point out that the token must have write scope

In case of an HTTP 403, inform the user that their token might not have the right scope for the action.
As things are today, the HF Hub API only returns 'X-Error-Message': "You don't have the rights to create a model under this namespace" in that case so we have to rely on the HTTP error.
I think a better solution would be for the API to return a proper X-Error-Code in case of entity creation/update + a read-only scoped token but that requires server-side changes first.
Fixes #1653

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Add support for ExtendedSocketOptions TCP_KEEP*

Currently in Netty with Epoll one can configure TCP_KEEP* options.
With Java 11 one can specify the options below when NIO is in use:
https://docs.oracle.com/en/java/javase/11/docs/api/jdk.net/jdk/net/ExtendedSocketOptions.html
TCP_KEEPCOUNT
TCP_KEEPIDLE
TCP_KEEPINTERVAL

This feature request is for supporting these options for NIO transport.

I think you can already do this by using: NioChannelOption.of(...) or I am missing something ?

Thanks a lot @normanmaurer, it works perfect :bowing_woman:


Tips for being Memory Efficient

This library is amazing! Do you have any tips for being memory efficient?
In my case I augment an image set of abut 300 images which after augmenation turns into about 21K images but all 21k are stored in memory. Is there a way to do this without it being stored in memory but still able to visualize the augmentations and use down stream for modelling purposes?

Not sure what you mean here with visualization?
Usually the library expects you to augment images batch by batch, so that your memory only has to be able to handle one batch of images at a time. That 21k number sounds like you are pre-generating lots of augmented images before starting the training. That is bound to consume lots of RAM and the library can't really do anything about it. Your options would be to change your code to augment every batch on the fly or to save the 21k images and dynamically load them per batch from the filesystem (this will consume lots of space and potentially be slow) or to compress the 21k images to JPEG in memory and then decompress them per batch (this will introduce JPEG compression artifacts and might not save much space for small images).

Awesome that makes sense. I am indeed doing this

sounds like you are pre-generating lots of augmented images before starting the training

and that is causing the issue. I am using the augment_images command and doing it multiple times until I get to the 20K number. When you say "augment every batch on the fly, how to do this"? With a generator or something else?

When you say "augment every batch on the fly, how to do this"? With a generator or something else?

Sample a batch of e.g. 64 images from your dataset, then apply augment_images() to these 64 images, then train (one forward-backward pass) on the 64 augmented images, then repeat from the start.

what I mean by visualization I pass the augmented array to ia.draw_grid(img_array,row,col)

You feed the whole 20k images into draw_grid()?

I am looking at the readme and I am not sure if my question is answered or not. I have a list of 16 augmenters, and I want all sample images to be augmented by them at least twice during training process or more. Right now I am looping over the list and using augmenter.augment_images() command twice for each augenter in my list and creating a new array of augmented images. This is super memory intensive so I am trying to change my process. All of the readme's uses ia.sequential but I dont want to do sequnetial augmentation as that would be too strong. Any thoughts?

If your only aim here is to avoid too strong augmentations by reducing the number of applied augmenters per image, then you can simply use SomeOf(N, augmenters) instead of Sequential(). E.g. something like SomeOf(2, [Affine(...), AdditiveGaussianNoise(...), Dropout(...), ...]) will always pick exactly two of the provided augmenters and apply them to the image batch, while the other augmenters are ignored. You can also set N to (1,3) to pick at least one and up to three augmenters per batch.

You feed the whole 20k images into draw_grid()?
No I want clear. Just a sample of those images.

If your only aim here is to avoid too strong augmentations by reducing the number of applied augmenters per image, then you can simply use SomeOf(N, augmenters) instead of Sequential()

Yes!! I discovered SomeOf yesterday and have been fiddling with it ever since. I am training with it and and even toying with throwing it in a generator. Will keep you posted on results. AWESOME library.

Closing for now. Thanks for your help as usual.

I have memory error issue even with a batch size of 32.
I'm trying to setup an input pipeline for yolo object detector. For every batch I have an image_batch numpy array (32,416,416,3). I set up a augumentation pipeline of resize, flip, noise...etc. (note: I am not setting up a seq(list) in advance and using it. I am updating the list for every batch) When i run this, i get an error:
Memory Error: np.zeros(output_shape, images.dtype) at Resize()
How can i solve this?

 So lower your mini-batch size (by powers of 2 so try
16, 8, 4, 2) or decrease the resolution of your images. It sounds like you
dont even have enough RAM to deal with your images.
Steps:

Determine how much RAM you have
Create a dataset of  x = np.zeros(1,416,416,3)
Compute bytes by x.nbytes. Divide 32*x.nbytes/1024/1024/1024 to convert
to gigabytes and compare that to the ram you have. If this number exceeds
your RAM then lower the 32 in the formula until it is below your ram

On Wed, Mar 13, 2019 at 1:46 AM Abarajithan<EMAIL_ADDRESS>wrote:

I have memory error issue even with a batch size of 32.
I'm trying to setup an input pipeline for yolo object detector. For every
batch I have an image_batch numpy array (32,416,416,3). I set up a
augumentation pipeline of resize, flip, noise...etc. (note: I am not
setting up a seq(list) in advance and using it. I am updating the list for
every batch) When i run this, i get an error:
Memory Error: np.zeros(output_shape, images.dtype) at Resize()
How can i solve this?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/aleju/imgaug/issues/83#issuecomment-472288402, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ANU-SvCbykd8pcGMF-ndBTfXN0W6Jl1Sks5vWJCugaJpZM4RApUv
.

Memory Error: np.zeros(output_shape, images.dtype) at Resize()

Please post the full stack trace.
One copy of a batch of 32 416x416x3 images in uint8 should require about 16MByte of RAM. That will be needed a few more times if you don't use Sequential (at least one copy per augmenter). How much RAM do you have? Is it possible that your RAM is already mostly used up before even starting the augmentation process? Are you preloading the whole dataset into RAM?

I have 8gb ram. I use tfrecords read through a generator to read one
example at a time.
On Fri, Mar 15, 2019, 9:49 PM Alexander Jung<EMAIL_ADDRESS>wrote:

Memory Error: np.zeros(output_shape, images.dtype) at Resize()
Please post the full stack trace.
One copy of a batch of 32 416x416x3 images in uint8 should require about
16MByte of RAM. That will be needed a few more times if you don't use
Sequential (at least one copy per augmenter). How much RAM do you have? Is
it possible that your RAM is already mostly used up before even starting
the augmentation process? Are you preloading the whole dataset into RAM?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/aleju/imgaug/issues/83#issuecomment-473350055, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AZJnpVyXpGgwK30_37ggYkTqStOFingeks5vW8f3gaJpZM4RApUv
.


Add BGSAVE command.

https://redis.io/docs/latest/commands/bgsave/

I split implementation into 2 commands (note - different docs) and split IT.
The timing question in IT is still open.

Suspended - no desicion on how should we test that and whether GLIDE client needs that command


Trouble installing

hi there! really interested in your plugin, but I'm having trouble with installing. I've unzipped the folder and copied it to my plugin folder, but I receive an error when trying to enable it. I also can't find main.js... sorry if this is a dumb question, I'm still new to obsidian :)

Thanks for your interest! Sounds like you're downloading the code from the master branch rather than the release page.

To manually install the plugin correctly, go to the latest release--which you can find here--and download main.js and manifest.json from there, no zip file needed. When you've got everything in place, it should look like this (data.json will be created automatically when you enable the plugin and tweak the settings):

If you still have trouble with manually installing it, Obsidian BRAT can install plugins that aren't in the community plugins list yet[^1]. If you need more info on this option, there should be plenty of help available on Obsidian's forums.
[^1]: Quick Tagger has been submitted and is awaiting approval 🤞

got it!! thank you so much!


Webapi exception

Describe the bug
The following exception is thrown after the web api is started and the first page is loading. Azure Ad authentication is enabled.
System.InvalidOperationException: Collection was modified; enumeration operation may not execute.
at System.Collections.Generic.List`1.Enumerator.MoveNext()
at Microsoft.SemanticKernel.KernelExtensions.Build(IKernelBuilder builder)
at CopilotChat.WebApi.Services.SemanticKernelProvider.GetCompletionKernel() in C:\Repos\chat-copilot\webapi\Services\SemanticKernelProvider.cs:line 28
at CopilotChat.WebApi.Extensions.SemanticKernelExtensions.<>c.b__2_0(IServiceProvider sp) in C:\Repos\chat-copilot\webapi\Extensions\SemanticKernelExtensions.cs:line 55
at ResolveService(ILEmitResolverBuilderRuntimeContext, ServiceProviderEngineScope)
at Microsoft.Extensions.DependencyInjection.ServiceLookup.ServiceProviderEngineScope.GetService(Type serviceType)
at lambda_method110(Closure, Object, HttpContext, IServiceProvider)
at Microsoft.AspNetCore.Authorization.AuthorizationMiddleware.Invoke(HttpContext context)
at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware.Invoke(HttpContext context)
at Microsoft.AspNetCore.Diagnostics.DeveloperExceptionPageMiddlewareImpl.Invoke(HttpContext context)
To Reproduce
Steps to reproduce the behavior:

Start the web api project.
Open browser and navigate to https://localhost:3000/.
See error

Expected behavior
It doesn't happen all the time, just occasionally.

I'm seeing this same behavior. Also seems to only happen on first attempt to hit API from the webapp.

I'm experiencing the same issue.

Fixed.


Merge rc/1.18 into master.

Conflicts in analysis-cpp.md resolved in favour of 1.18, which added a change note.
Similar to https://github.com/Semmle/ql/pull/160.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 10 committers have signed the CLA.:x: hvitved:x: esben-semmle:x: jbj:x: nickrolfe:x: xiemaisi:x: geoffw0:x: calumgrant:x: asger-semmle:x: dave-bartolomeo:x: adityasharadYou have signed the CLA already but the status is still pending? Let us recheck it.


Add ability to report posts/comments

Context
With the incoming feature of comments it will be useful to have a way to report spam or abusive comments
Expected Behavior
When reporting a post/comment a new page with a link to the post/comment should appear where you can add details of the abuse.
Current Behavior
It doesn't exist

I'd like some input on places who are doing this well so please add some suggestions to this field so we can flesh out this issue.

I gave this a go today. Didnt use any packages Niall :) Pretty happy with it, but not sure what to do with the report/data. I was thinking something like Nodemailer to send the report to an admin, but im open to suggestions?
Also looking for feedback on the styling. I found the figma file, but I mostly just winged it.
https://youtu.be/tIwlSFcrehI

Here is perfect! You've been on fire. 🔥
This looks great and I think you should find somewhere I'm already sending emails in the codebase so there should be a utility. If not we can make that into an issue first and create that.
What data will you send via the email?
I was originally thinking something like a report page but this is a great simple solution that makes sure we have a reporting mechanism (ya know, before we have millions of users 💅 )

You've been on fire.
Im unemployed and learning. Open source is the next step. Already learned a ton this week. prisma, tailwind, trpc, and docker are all new to me and Git is fine is theory, but we only learn with practical application. When you get tired of me hogging the issues let me know. :)

I'm already sending emails in the codebase so there should be a utility.
Cool. Ill look into it.

What data will you send via the email?
Just id/username of reporter and reportee, along with comment and article/post id.
ya know, before we have millions of users.
Its coming. There is already a good base of contributors. No doubt you have ideas for the site. Bring me in ill help make them happen.

Think I have the email working, but cant test it as it uses Amazon SES.
Just need to add an ADMIN_EMAIL to the process.env for the recipient who will recieve the reports.
I hardcoded the link to the codo.cu/articles/slug Hopefully it will work.
The email should look something this.

I'm going to close this because we have more specific issues open and this is all WIP by @Fishamble


Could not find a declaration file for module 'webpack'

Does this mean that I need manually install the peer dependency @types/webpack?
 TS7016: Could not find a declaration file for module 'webpack'. '/my_project/node_modules/webpack/lib/webpack.js' implicitly has an 'any' type.
      Try `npm install @types/webpack` if it exists or add a new declaration (.d.ts) file containing `declare module 'webpack';`

Have you tried?

Yes, install the missing type will work.


Add Inventory Item template as an option when defining Device Types

Environment

Python version: 3.8.5
Nautobot version: 1.0.0b2

Proposed Functionality
Add Inventory component in the Device Type.
Use Case
This is for device templating, inventory for similar devices may be necessary.
Database Changes
Unknown
External Dependencies
Unknown

@vinross32 can you expand on the use case a bit here?

@vinross32 can you expand on the use case a bit here?

As I understand it, when you are in a Device Type detail view and wish to add a component, Inventory Items are the only "Device Component" that cannot be added (templated) in this way:

:
So @vinross32 is asking that we bring add Inventory Items as an option for Device Types similarly how they are already an option for individual Devices.

Ack, thanks for clarifying.

Hello, is this feature going to be added anytime soon? It would be a great addition to the product to model chassis switches with multiple linecards, supervisors, etc. Thanks!

This is reduced in importance now that we have module types and modules (which are templateable) as an alternative to inventory items, but it is still a feature gap.


Write a minimum unit test for SWU hash to curve

At minimum this should define a simple curve with ab != 0 y^2 = x^3 + a*x + b and map whole field to the curve.

at 89e69a1 tryng to resolve this error:
    error[E0038]: the trait `map_to_curve_hasher::HashToField` cannot be made into an object
   --> ec/src/hashing/tests.rs:131:35
    |
131 | ..._hasher: MapToCurveBasedHasher::<GroupAffine<TestSWUMapToCurveParams>, HashToField<F127>, SWUMap<TestSWUMapToCurveParams>>;
    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ `map_to_curve_hasher::HashToField` cannot be made into an object
    |
note: for a trait to be "object safe" it needs to allow building a vtable to allow the call to be resolvable dynamically; for more information visit <https://doc.rust-lang.org/reference/items/traits.html#object-safety>
   --> ec/src/hashing/map_to_curve_hasher.rs:21:35
    |
21  | pub trait HashToField<F: Field> : Sized {
    |           -----------             ^^^^^ ...because it requires `Self: Sized`
    |           |
    |           this trait cannot be made into an object...

this^ was resloved by 8a9c0f6. now it fails because it can't construct proper field elements using field_new macro.

is basically done in 0075ba3


Remove query parameters from browse button

Fixes #9484
Removed queryStringSuffix for directory rendering since we should only base it off the input path field

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4699/
Test PASSed.

@calvinjia hmm i'm not sure what you mean. the fileview page is still needs browse's other state/props fields

@tieujason330 I think there is some overlap but I'm not convinced that the overlap vs difference is large enough to use the same views. Perhaps we could further modularize the overlapping portion? In the backend code, these two go through two separate if blocks.

For this PR, could you resolve the conflicts?

@calvinjia conflicts are resolved.
Hm, yeah I agree with modularizing. I'm looking into commonalities between several components right now to see what i can extract (Logs, Browse, BlockInfo)

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4747/
Test PASSed.

alluxio-bot, merge this please.

@tieujason330 Thanks for the fix! Could you cherry-pick this to branch-2.0?