Update docs to use newer version of esptool.py

Apparently, esptool.py now supports python3. See https://docs.micropython.org/en/latest/esp8266/esp8266/tutorial/intro.html#deploying-the-firmware.
This will simplify our instructions - we will now only need python3. Yeah!

In the PyPi, it says that python 3 is supported starting in esptool 1.3. I think we should just recommend python 3 for everything. In addition to the main doc, need to change the summary of required software in the readme, etc.


docs(readme): fixed typo

Current coverage is 100.00%

Merging #6 into master will not affect coverage as of 697e157

@@            master      #6   diff @@
======================================
  Files           12      12
  Stmts           45      45
  Branches         0       0
  Methods          0       0
======================================
  Hit             45      45
  Partial          0       0
  Missed           0       0

Review entire Coverage Diff as of 697e157

Powered by Codecov. Updated on successful CI builds.

:+1:


Skip Malformed Rows (those missing an end_date)

Expected
The Charges endpoint should still return data, even if a portion of charges have corrupted data.
How it happens
Charges can become corrupted if Teslamate does not detect that a charge ever ended, such as if the car or Teslamate goes offline when the charging ends.
Note
The charging_processes table's most recent row may be missing an end_date if the charge is in progress. These charges were already being skipped in the chargingDetails endpoint. I am assuming it is safe to skip them here, since they essentially have no data until the charge finishes.
Please Test

Please test that this still builds as I am not entirely familiar with Go and am unsure how to test this on my docker container.

@tobiasehlert Did you have any thoughts about my newer fix to the problem?


Add Windows support

Currently things seem pretty unhappy on Windows, and it'd be really awesome if support was added. fsnotify already supports Windows, which from a brief scan of the code looks like the most platform-specific thing.

Windows 10
Go 1.9.2
Bazel 0.8.1

Attempting bazel build //ibazel from cmd leads to:
λ bazel build //ibazel
WARNING: C:/users/ben/appdata/local/temp/_bazel_ben/fa1bxbsu/external/com_github_bazelbuild_bazel_integration_testing/WORKSPACE:1: Workspace name in C:/users/ben/appdata/local/temp/_bazel_ben/fa1bxbsu/external/com_github_bazelbuild_bazel_integration_testing/WORKSPACE (@build_bazel_integration_testing) does not match the name given in the repository's definition (@com_github_bazelbuild_bazel_integration_testing); this will cause a build error in future versions
ERROR: D:/tools/bazel-watcher/ibazel/BUILD:30:1: no such package '@com_github_fsnotify_fsnotify//': failed to fetch com_github_fsnotify_fsnotify: 2017/12/19 18:02:56 --remote should be used with the --vcs flag. If this is an import path, use --importpath instead.
 and referenced by '//ibazel:go_default_library'
ERROR: Analysis of target '//ibazel:ibazel' failed; build aborted: no such package '@com_github_fsnotify_fsnotify//': failed to fetch com_github_fsnotify_fsnotify: 2017/12/19 18:02:56 --remote should be used with the --vcs flag. If this is an import path, use --importpath instead.
INFO: Elapsed time: 1.763s
FAILED: Build did NOT complete successfully (19 packages loaded)

This may be a go_repository issue.
If running from the msys bash the above doesn't trigger but instead nothing happens:
$ bazel build //ibazel
WARNING: C:/users/ben/appdata/local/temp/_bazel_ben/fa1bxbsu/external/com_github_bazelbuild_bazel_integration_testing/WORKSPACE:1: Workspace name in C:/users/ben/appdata/local/temp/_bazel_ben/fa1bxbsu/external/com_github_bazelbuild_bazel_integration_testing/WORKSPACE (@build_bazel_integration_testing) does not match the name given in the repository's definition (@com_github_bazelbuild_bazel_integration_testing); this will cause a build error in future versions
INFO: Analysed 0 targets (0 packages loaded).
INFO: Found 0 targets...
INFO: Elapsed time: 0.345s, Critical Path: 0.00s
INFO: Build completed successfully, 1 total action

Interestingly a git prompt window will pop up for a split second, which looks like it's running the workplace_status.sh script.
I'm not sure what issues may lie beyond those. I am but a simple C++ developer and the world of go packaging frightens and confuses me :)

Ben, thanks for taking a look!
Let's deal with the errors first since they are the most fun.
WRT your first compiler error. I think that is an issue with rules_go (probably worth reporting, they are really nice). I believe the issue is that when the helper that fetches code for dependencies during the fetch stage is unable to find the git CLI, but I also wouldn't be surprised if that weren't the case. It's definitely something to do with this tool. If you want to go spelunking you could try running the build again with the -s option which will print out the command that Bazel is about to run and the directory it will do it in before running it. You could then invoke the command yourself from the rules_go repo but with some debugging statements.
To the second one, it looks like //ibazel isn't a target in that repo. That's weird. What do you get as the output of bazel query //ibazel:*? If you could reply with that it would be appreciated.

Interestingly a git prompt window will pop up for a split second, which looks like it's running the workplace_status.sh script.

Yep, that's workplace_status.sh trying to do its magic. That's how we set the version string during linking.

Currently things seem pretty unhappy on Windows, and it'd be really awesome if support was added. fsnotify already supports Windows, which from a brief scan of the code looks like the most platform-specific thing.

Interestingly, in my simple experiments of just getting it to compile for Windows, fsnotify has not been a problem. That's not to say it won't be a problem in the future, but right now it seems okay. The primary issue is this single line of code. I treat the subprocesses as a process group (I think they are AKA ProcessTrees in Windows) and when I kill the group. In POSIX land this means I kill all children and descendants + the parent process that I directly invoked. I know next to nothing about how processes work in Windows and I don't know how to do that in Windows. I just did a little looking around and there might be a library that does this.
If you wanted to take on the task of figuring that out, it shouldn't be extremely difficult once you get ibazel compiling in Windows. So let's focus on that first. In the meantime, I will try and block out some time to put a stub in place to show how to get different code running in Linux vs Windows but I need to land some of the PR that are in limbo right now.

I'm not sure what issues may lie beyond those.

Thanks for the response!
There's some definite weirdness happening under the msys bash - I've never seen issues like it, but perhaps it's new bazel changes. I'm willing to write msys off - I think bazel's general guidance under Windows is to use native cmd, I was just grasping at straws :)
As for the -s, unfortunately no additional output comes out - my guess would be because the errors are happening prior to the execution of any actions. Running a bazel fetch //ibazel also fails but for all repos at once.
RE the .sh, I wonder what the bazel recommend thing to do here is. Bazel doesn't require bash on Windows, which means that the .sh is running at all is just some artifact of my machine configuration (probably because I had my git install add itself to path and register some extensions). In genrules/etc the bazel guidance has been to provide both .sh and .cmd files, but I don't know a good way to switch which one is run from a bazel.rc flag. When I was doing some extra_actions work I ended up using python instead of bash scripts to bypass the whole issue (and avoid writing bat files ;)
I've dug into the rules_go stuff a bit and there's definitely some weirdness here that's not ibazel specific. So besides the real issue you mentioned with the processes and the .sh we can probably ignore the busted go rules. I'll try to file some bugs there, as I should be able to repro them with some trivial repos :)

What happens if you run the commands again after commenting out
https://github.com/bazelbuild/bazel-watcher/blob/1c4b5146f1c81cf2969423ef2d4f9a3e45b5a20d/tools/bazel.rc#L1

Still the death with the other errors, unfortunately. I did add some random prints and changes to how env_execute works inside of rules_go and made some progress, then noticed that there was a commit in rules_go relating to Windows today. I changed the ibazel-watcher workspace to point at the latest master commit 8968f4061342f30836ffc9a88d7e423039ce0435 from bazelbuild/rules_go#1169 with those changes.
Now there's some new interesting errors, which again look like rules_go badness - I'll poke them over there and see what they say - that commit message claims the tests should pass, though, so either there's a missing test or my environment is borked :)

same here windows10

any news ?

Thanks for the reminders @djleonskennedy I'll make sure this goes on our hotlist with the Bazel team.

/cc @dslomov

If I do the same I get:

So this is still outstanding, no go? I'm not a go developer so I unfortunately couldn't contribute much (much of any value anyway) but it seems like based on experience with other languages and technologies, seems like we should be able to just gather info about environment and have conditional calls for this sort of thing.

I will give it a go and try to implement it. Right now we do it in conjunction with Bash for Windows. It works but it is not ideal.
@ronnyek it is indeed the same as I was thinking, using conditional calls.

@ronnyek @GeertVL, the traditional way of making code target a platform in Go is to construct an interface that is shared between various OSes and then use build tags to include the environment specific code. If you try to use conditional calls you will discover that the golang libraries expose different interfaces on different OSes and as a result your code won't compile on any of them.
The way I would do it would be in a few phases:

Think about what a good interface for interacting with an inode watch is. This interface may be different than what is exposed by any OS.
Extract the existing code into the interface
Write the code for the Windows implementation and validate it using e2e tests (fortunately we have these already)

If you want help with anything in there, feel free to ping the thread with questions and I'll be as helpful as I can!

Whoops, I didn't notice this issue.
I made a quick attempt at porting this to Windows just to see what it would entail. I got some basic functionality working, though unit tests do not pass on Windows and I have not touched e2e tests at all. It'll probably be fairly tricky to get this all working right on Windows, but I'd be happy to try to get my pull request to a useful state.
If anyone else is still working on this, I suppose it would be wise for us to merge our efforts.
For now, I will post the PR and hopefully we can get it in the right shape to merge. Sorry if I duplicated anyone's efforts, I didn't actually expect for my effort to lead to a working binary so quickly :)

@jchv awesome, thanks!!!

Looks like #213 was closed. Should bazel-watcher work with windows for any of the recent releases? I just installed via npm and get the error:
FATAL: Your platform/architecture combination NaN is not yet supported.

Same issue here, installed the latest version (marked as windows compatible) from npm and I get this error message as well

It may not yet be available via npm at this point, but you can download a Windows binary from the latest GitHub release: https://github.com/bazelbuild/bazel-watcher/releases/tag/v0.10.0

@ocombe, Did you attempt this through NPM? If so, I don't have a windows box to test my launcher script on so I was flying by the seat of my pants. launcher is really quite simple and if you wanted to take a stab at fixing it. The code is right here https://github.com/bazelbuild/bazel-watcher/blob/master/release/npm/index.js
It turns out I was subtracting string from string instead of delimiting. If you replace the index.js file with that it will give you a much more helpful error message at the least. Could you either make that change to the file locally, or replace it with the one that is on master and tell me the new error output?

@jchv strangely, i cannot run the installator of release 0.10.0 on windows 10 insider.

@achew22 how is it going with getting a google windows laptop? I'll inter-office you one (never joking)

I was able to execute the ibazel executable listed as a release no problem. This isnt a windows insider build, and it did the usual "this software vendor is not recognized as trusted" or whatever that little message is, but you can choose to execute anyway... if I deleted ibazel from my npm home in app data, and replaced with this executable renamed... it at least executes without further error.
I'm diving into testing the actual functionality itself now. Thanks for the tips!

Since this issue is still open
https://github.com/bazelbuild/bazel-watcher/blob/master/release/npm/index.js#L33
that doesn't seem correct.
https://nodejs.org/docs/latest-v4.x/api/os.html#os_os_platform

@rerion I would love to get a PR to address this. I'm happy to release if you send the fix in

I think this issue is solved. If you run into more issues please feel free to open new issues.


number of simulated points - multiple relaizations - bibliografy

Hi,
I discovered today you're nice package for Hawkes process and I'm trying to use it. However, for my purposes, I need to simulate and estimate very long time series, e.g. with a baseline = 1 , branching ratio almost critical = 0.99 and simulation interval of order of 2e6 (or even grater if possible...). I founded that, on simulation interval smaller than the ones I'm interested in and in the case of power law kernel, you're ibrary performs better (in terms of simulationtime) than tick (https://x-datainitiative.github.io/tick/index.html). However, it seems that there's an upper bound on the number of points that can be simulated, 1e6 + 1 I would say. Is there any way to change it?
Moreover, I was wondering whther in both the simulation part and in the estimaiton part you used or not the approximation of the power law kernel introduced in https://link.springer.com/article/10.1140/epjb/e2013-40107-3 .
Finally, I wanted to ask whther is possible to simulate and fit multiple instances from the same system and which papers, more or less, you referred to in the library.
Thank you so much

Hi

You can change the maximum number of events in a single simulation by changing the value of N_MAX at line 118 in /Hawkes/model.py (https://github.com/omitakahiro/Hawkes/blob/master/Hawkes/model.py#L118).

The intensity function of the model with a power-law kernel is calculated for simulation and inference using a technique introduced in  https://doi.org/10.1029/93GL02142.

The implementation of multivariate Hawkes process is in progress, but it will take a while.

Thank you so much for your answer!
WIth respect to point 3, I was not asking for multivariate Hawkes but for multiple realizations of the same one dimensional Hawkes process, in such a way to simulate the maximization of the ensamble average of the likelihood function. In tick for example you can use the SimuHawkesMulti class (https://x-datainitiative.github.io/tick/modules/generated/tick.hawkes.SimuHawkesMulti.html) .
Moreover, I'm facing som problema in the inference part as some times it takes very long time or seems not to converge. Is there a way to set some maximum number of iterations and/or some tolerance threshold? Or to plot the likelihood evaluated during the optimization?
Thank you in advance for your answers and however, you're tutorial is great!

In our package, parallel simulation of a Hawkes process is not implemented at this time.

The likelihood and the norm of the gradient during the optimization is printed when you pass an optional argument to the fit function:

h2.fit(T,itv,opt={'print':True})

The maximum number of iterations can be also set similary: (In the below example, the optimization algorithm stops at the 10th step of the iterations)

h2.fit(T,itv,opt={'print':True, 'stop':10})

At the default setting, the optimization algorithm stops if the norm of the gradient become less than 1e-5. You can adjust this value at https://github.com/omitakahiro/Hawkes/blob/master/Hawkes/tools/Quasi_Newton.py#L135.


feat: release, docker, deploy (#32)

Do I need to merge the PR, in order to see if everything is working? 🤔

yes I think so, but maybe we can wait until tomorrow, so I can be around/awake to check it all works - I am sure i missed something 😂

sure, no hurry 👍


Code of Conduct Button

Problem
As an organization,
we want to be MLH compliant
so we can be sponsored by MLH
Link to Asana Ticket
Solution
Added the code of conduct object to buttonProps.
Change summary:

added code of conduct button a6a383c

Steps to Verify:

Checkout branch and start local server
Verify that Our Code of Conduct button appears & that it takes you to correct link

Screenshots:

@tim-nguyen-cs thoughts?

It works for now - thanks for making that update! We should have a newer iteration of the site ou by this weekend anyway.

In order to fix some issues with mobile responsiveness, we have opted to put in two sets of buttons.
Issue with mobile

**Fix **


Fix travis

Although travis is green, both Python 3.6 and Python 3.7 are not run at all as the interpreter are missing.
https://travis-ci.org/github/morepath/morepath_cerebral_todomvc
Todo
Use a matrix, see https://github.com/morepath/morepath_static/pull/4 how to fix it.

Well @jugmac00 , I would like to work on it :)

I heard you have experience with it :-)
Thanks, please go for it!

can i work on the issue?

Hi @rajchordia14 - @Aniruddha-Shriwant was a bit faster on this one.
But there are many similar issues to go for. Have a look at any of those
https://github.com/search?q=org%3Amorepath+Update+supported+Python+versions&type=issues

Well @jugmac00 , I had done with my 4 PR's for the Hacktoberfest,
Thanks for guiding me from the beginning, will keep exploring new things as I did in this organization's repositories :)

fixed by #5

@Aniruddha-Shriwant Good luck for your future! :-)

Hey @jugmac00 ,
I think Hacktoberfest has new rules now,
PR won't count if the repo is not participated in Hacktoberfest and in order to participate in Hacktoberfest, the maintainers should add the Hacktoberfest topic to their repo

Hi @Aniruddha-Shriwant
no worries. The new rules only apply to PRs after Oct 3 - the PRs until then remain valid.


Notification permanently displayed

For a couple weeks this app shows a notification that cannot be swiped away. It just says the title of the last song I played. Clicking on it poems the app, but there is nothing to resolve. There is nothing in the cache. Only way to remove the notification is to restart my phone.

Opens, not poems.

I have the same problem. Samsung s6 with android 6.0.1, DSub https://github.com/daneren2005/Subsonic

#288

https://play.google.com/apps/testing/com.adam.aslfms


changed readme accordingly to responsive mechanism

Readme is a bit confusing, because accordingly to example it's possible to use data-grid props to set default mappings for individual items

Thanks!


Investigate how a Job can be stopped

Users would like to prevent a Job from consuming resources, deletion of Jobs is kind of a blunt weapon as the user might still want to investigate its output or logs. Research needs to be done in order to know how this might help.

One option would be to use the parallelism spec and set it to 0.
From the Kubernetes documentation:  "The requested parallelism (.spec.parallelism) can be set to any non-negative value. If it is unspecified, it defaults to 1. If it is specified as 0, then the Job is effectively paused until it is increased."

There is an interesting discussion here on how to use backoff and restart policies to stop a job: https://github.com/kubernetes/community/pull/583

Another option, as Job are supposed to be ran to completion, would be to keep the job template in the ~/.nerd directory.
The user could save a job template in multiple ways, when the user starts, stops or deletes a job. nerd job start|stop|delete <args> --save
The user could also list and describe templates and reuse them whenever they want. nerd template list|describe|delete (not sure about the necessity to create a template command)
Then nerd job start --template name.yaml
This could be interesting for users that want to switch to kubectl, and they would already have their own job manifests.

So after some tests, parallelism set to 0 is not really what we need. Indeed, it deletes the running pod, meaning that we loose its logs, events and statuses.
The backoffLimit and backoffSeconds could be really useful as it would let  users specify how many times they want their job to retry in case of failure. Only problem is that this feature is only available from Kubernetes v1.8

For the first milestone we will "stop" a job by reducing its parallelism to 0. This effectively removes the pod so no logs are available to the user but it will no longer consumer resources


fix: sync style

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Specify encoding when writing output file to avoid errors when defaul…

…t target encoding doesn't have all characters
Specify encoding when writing output file to avoid errors when default target encoding doesn't have all characters. utf8 seems like the most universal and supported encoding. Otherwise, the cli fails with encoding errors when input file contains unicode text (basically most files nowadays) and the target system has default encoding set to some one-byte charset like cp1252
Checklist:

[x] Commit Message Formatting: Commit titles and messages follow guidelines in the
conventional commits.
[x] Documentation has been updated, if necessary.
[x] Examples have been added, if necessary.
[x] Tests have been added, if necessary.

@cau-git @vagenas I propose flagging this as fix when merging.

thanks, I was going to propose something similar to fix an error I was having with UnicodeEncodeError: 'charmap' codec can't encode character '...' in position ...: character maps to <undefined>


More customisation for chat file intros

Can there be a flag --remove-intro-from-chat which will not save the intro (or system prompt) in the .txt chat history file? That makes the chats more readable from the users perspective. We could write something like:
pytgpt generate --intro="$intro"  --remove-intro-from-chat "Prompt"
which would USE the intro together with the chat history but just won't write the intro to the chat history file. To continue the chat one would then have to use the --intro flag each time.
This has a lot of use cases if one wants dynamic intro's, we could just pass a variable $intro at different times when calling in an LLM on the same chat.

Okay. This will be limited to newly created chats.

Let's keep this issue fresh to stop it from closing again.


apps/openssl: make index.txt errors more verbose

openssl ca was  a little bit too silent, when it couldnt parse index.txt ...
steps to reproduce: create index.txt with

echo "" > index.txt

which creates

hexdump index.txt
0000000 000a
0000001

-> Trivial change, tested with openssl ca to sign a csr
-> the ocsp and svr changes were only compile tested by me, but seemed to make sense. ?

Checklist

https://superuser.com/questions/1327848/openssl-ca-fails-after-password-without-error-message

Hey, this is a trivial thing that has bitten at least two people in the past.    Anyone up to merging this or redoing this? Should be a no-brainer.

I am sorry but unfortunately this is outside of what we usually consider as acceptable under CLA: trivial. Could you please submit a CLA? https://www.openssl.org/policies/cla.html

I just submitted my ICLA. Do I need to ammend the commit description?

You'll need to remove the CLA: trivial line from each commit.


Upload progress during push/publish

Do you want to request a feature or report a bug?
Feature
What is the current behavior?
You never quite sure how long things are going to take. Some form of progress indication would be great. This is a nice to have ofcourse.
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem.
Steps to reproduce the issue:

Execute push or publish
There's no real progress feadback to the user.
Not sure if the process has hung.

What is the expected behavior?
A percentage indicator of how far along the process is.
Additional environment details

OS version (ie Windows 10 build X, macOS Sierra 10.12.6, etc.)
MacOS
Output of amplify --version:
0.1.14
Did this work in previous versions?: Y/N
N

@jarrettj Currently we show the stack events streamed directly from Cloudformation. Does that information seem helpful to know whats going on behind the scenes?
It's a bit complex to predict the total time estimate beforehand because the provisioning of the various AWS resources varies on several parameters and not trivial to predict, but we can surely work on a naive solution to display which resources are provisioned and how many resources are remaining to be provisioned.
Thoughts?

Hi
Good day.
Yup that sounds awesome man. And it’s actually the upload of the files that I was talking about. The lambda function to be specific.
I saw the CF template output. But sometimes I have to kill the amplify process because it’s just not uploading the lambda function.
The internet my side of the world isn’t that great. So maybe it’s just me.
Thanks. For the update though.
Regards.

On 29 Aug 2018, at 18:56, Kaustav Ghosh<EMAIL_ADDRESS>wrote:
@jarrettj Currently we show the stack events streamed directly from Cloudformation. Does that information seem helpful to know whats going on behind the scenes?
It's a bit complex to predict the total time estimate beforehand because the provisioning of the various AWS resources varies on several parameters and not trivial to predict, but we can surely work on a naive solution to display which resources are provisioned and how many resources are remaining to be provisioned.
Thoughts?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Using the latest version. There's better info while pushing functions to the cloud.


Ps performance improvements

Changes to have Restlet use Apache HTTP Client.  Also ensure we release Response and ClientResource objects to allow the HTTP connection pool to recycle connections.

Coverage decreased (-0.6%) to 68.058% when pulling d97deb1c63e9b1c663f0d72ebb861a4a0298e1b1 on ps_performance_improvements into bcc691c2598fcde9dbcd14cbf4d2f7cd16c9b406 on master.

The code looks good. A few weeks ago I overheard a conversation about a convention that package names be singular, but I don't know if that was something the team had agreed upon or one person's opinion. (It's not the kind of think I care about.)

Looks good to me @psantos1113

Coverage decreased (-0.6%) to 68.023% when pulling 5d23747ef43180241ff6f58a29f68535392e042f on ps_performance_improvements into 60c80a2429fd1e566c19e3942d7973b91f983f03 on master.


Integrate bulk import with ingest operation model

Background
We'd like to make some changes to how compactions work to avoid the need for separate splitting compactions:

https://github.com/gchq/sleeper/issues/966

We have another issue for setting up a new interface to the state store for ingest:

https://github.com/gchq/sleeper/issues/1115

Description
We'd like to be able to use the new method signature for completing an ingest job in bulk import.

Closing as this is not needed for now.


[BUG]: PowerApp is not accessible to external users

Describe the bug
External users are not able to access canvas app if creator kit components are added into the app. Issue Reported by user: "You don't have access to this Your sign-in was successful, but you don't have permission to access this resource". Browser keeps asking to sign-in and showing below screenshot.
Expected behavior
External users should be able to access app.
Screenshots

Yes, post my investigation on this issue, it seems like this is happening due to conditional access policy.


A few questions

Thanks for the great work.
I have a few questions:

Is it possible to optimized with concurrency inferences?
Is it better to add sync def for FastAPI entry point?
How to scale up to 100 users using websockets with fasterwshiper.

others:

from openai import OpenAI, AsyncOpenAI does not support stream argument, which should be mentioned
it will be nice to have num_workers.

Not sure what you mean here.
Are you referring to functions handling web sockets?
Haven't tested with more than one connection myself.
From README.md: "Unlike OpenAI's API, faster-whisper-server also supports streaming transcriptions(and translations). ". It could probably be made clearer, but it's there.
You should be able to configure that using environment variables. From uvicorn docs

  --workers INTEGER               Number of worker processes. Defaults to the
                                  $WEB_CONCURRENCY environment variable if
                                  available, or 1. Not valid with --reload.

Thank you for your reply.

Not sure what you mean here.

As for concurrency inferences, I mean dynamic  batching  or static batching inference in  a single instance on a GPU. Batching inference speeds up the entire pipeline.

Are you referring to functions handling web sockets?

I have read your source code and I did not see asyn in front of  transcriptionsdef. Is this on purpose?

@mru4913 did you find any way to do concurrency?
In principle, submitting a batch of requests should be easy for the model to handle - but openai style requests won't handle that...
Dynamic batching would be more complex (something like vLLM) but definitely would be very interesting as an open source offering.
LMK if you find any hacks to get either working. Thanks.
And thanks @fedirz for making this repo, just started working with it today and things have being working without a hitch.


MessageCompiler fetches messages using wrong locale

This can be reproduced when used with config.messages = :i18n, may also apply to custom messages.
Minimal reproducible example
begin
  require 'bundler/inline'
rescue LoadError => e
  $stderr.puts 'Bundler version 1.10 or later is required'
  raise e
end

gemfile(true) do
  source 'https://rubygems.org'

  gem 'dry-validation', '0.11.1', require: false
  gem 'i18n', '0.8.6', require: false
  gem 'rspec', require: 'rspec/autorun'
end

require 'i18n'
require 'dry-validation'

UserForm = Dry::Validation.Schema do
  configure do
    config.messages = :i18n
    config.namespace = :user
  end

  required(:name).filled(:str?)
end

I18n.available_locales = %i[en ru]
I18n.locale = I18n.default_locale = :en
I18n.backend.store_translations(
  :en,
  errors: {
    rules: {
      user: {
        rules: {
          name: {
            filled?: 'must be filled'
          }
        }
      }
    }
  }
)
I18n.backend.store_translations(
  :ru,
  errors: {
    rules: {
      user: {
        rules: {
          name: {
            filled?: 'заполните это поле'
          }
        }
      }
    }
  }
)

RSpec.describe 'I18n validation messages' do
  subject(:result) { UserForm.call(name: nil) }

  around do |example|
    # Set Russian locale for all examples
    I18n.with_locale(:ru) { example.run }
  end

  EXPECTED = ['заполните это поле'].freeze

  context 'with new implicit options ({ hints: false })' do
    it { expect(result.errors[:name]).to eq(EXPECTED) }
  end

  context 'with explicit locale' do
    it { expect(result.messages(locale: I18n.locale)[:name]).to eq(EXPECTED) }
  end

  # Failing example!
  context 'when reusing default MessageCompiler instance' do
    it { expect(result.messages[:name]).to eq(EXPECTED) }
  end
end

# Monkey patch MessageCompiler to make the failing example pass:
#
# module PatchMessageCompiler
#   def default_lookup_options
#     { locale: messages.default_locale }
#   end
# end
#
# Dry::Validation::MessageCompiler.prepend PatchMessageCompiler

Expected behaviour
All tests pass.
Actual behaviour
One test fails.
Why?
After some debugging I've found out that messages are fetched in MessageCompiler#visit_predicate:
https://github.com/dry-rb/dry-validation/blob/806ce1adc14d4bd340d8d31f3c89d6c433f12303/lib/dry/validation/message_compiler.rb#L107-L113
msg_opts is a hash with options passed to I18n.t. It contains the :locale option deduced from default_lookup_options, which is set in the constructor using messages.default_locale (active locale at the moment):
https://github.com/dry-rb/dry-validation/blob/806ce1adc14d4bd340d8d31f3c89d6c433f12303/lib/dry/validation/message_compiler.rb#L13-L20
MessageCompiler is instantiated when Schema subclass is created. It can be instantiated one more time when validation error messages are retrieved via MessageCompiler#with, but only when provided with non-default (non-empty) options, otherwise the first created instance of MessageCompiler is reused:
https://github.com/dry-rb/dry-validation/blob/806ce1adc14d4bd340d8d31f3c89d6c433f12303/lib/dry/validation/message_compiler.rb#L30-L33
So we have the following problem:

MessageCompiler is instantiated when the schema subclass is created with the locale which was active at that moment;
The current locale is changed;
Schema is called to validate the input, validation error messages are retrieved using the Result#messages method (with empty options hash);
The first created instance of MessageCompiler is reused, so we get messages in the locale which was active when that instance was created, not in the current locale.

If some options are passed to Result#messages explicitly or implicitly (Result#errors passes { hints: false }, Result#hints passes { failures: false }), then MessageCompiler#with returns new instance that has the current locale as the :locale options in default_lookup_options, so we get messages in the current locale.
Possible workarounds

Make #default_lookup_options fetch locale each time it's invoked (see how I monkey-patched MessageCompiler in the example above).

Change MessageCompiler#with so it takes the current value of messages.default_locale into account when deciding whether to create a new instance:
def with(new_options)
  return self if new_options.empty? && locale == messages.default_locale
  self.class.new(messages, options.merge(new_options))
end

Don't pass :locale option to Messages::Abstract#call?

I arrived here from the trailblazer/reform-rails#57 where the dry-validation messages were always returning in a single language when using Reform, dry-validations, and i18n. The first possible workaround does not work for that case, but the second one does. I'm currently monkey-patching MessageCompiler#with with and it works as expected. Any possibility a fix for the i18n issues are coming soon?


Version 0.8.9

[x] Advanced error handling via the "error" event. If a custom handler is not added, runtime errors will be automatically logged in the console. If a handler is added, handling (and logging) is fully up to the user. If client.connected is False, handlers will NOT be used.
[x] Update documentation

Released in https://github.com/isaackogan/TikTokLive/releases/tag/v0.8.9


Support Ruby 1.9.3

Some tests are failing with Ruby 1.9.3. This seems to be the issue: http://www.redmine.org/issues/9774

Well, I guess we'll have to wait for moving to Ruby 1.9.3 until we move to Rails 3 because Rails 2.3 is no longer maintained: https://github.com/rails/rails/pull/3745 :(

This is fixed in RC4; https://github.com/radiant/radiant/commit/2cec77e4164d56828ba63a014faa6adb6e46ed77


Attachments lost on references with spaces

First off, thank you for making this importer available to the community!
I just did my first import and I noticed an issue. Any of my notes that had a space in them did not get their attachement correctly linked.
This is the file  /Users/jeff/Dropbox/bearExport2/DnD/PC/Hemlock Underleaf.md
# Hemlock Underleaf
![[attachments/Hemlock%20Underleaf/Untitled.jpg]]
(Craig) Male half-elf druid (circle of the moon)

#DnD/PC

Another import tool parsed the image correctly, but it used the reference format ![](attachments/Hemlock%20Underleaf/Untitled.jpg)
While this seems to be an Obsidian bug, looking at Markdown documentation, is seems that the proper format for image references is ![alt-text](path/to/imgage.png)

Hi @shepster, thanks for the report!
I've replicated the report and will work on a fix so this doesn't happen in future imports. The example helped a lot. :)
The issue here is that white space got replaced with a %20. A short term fix is just to manually remove the %20 and replace it with a proper white space.
From ![[attachments/Hemlock%20Underleaf/Untitled.jpg]] to ![[attachments/Hemlock Underleaf/Untitled.jpg]]

While this seems to be an Obsidian bug, looking at Markdown documentation, is seems that the proper format for image references is

Yeah, this is an Obsidian "quirk". The attachment format is in the form of: ![[path/to/image.jpg]]

@shepster the issue should be fixed, feel free to reopen this issue or create a new one if you run into a similar issue when re importing.
Thank you! :)


Team based boost mod

Description
Added a new gameMode that allows users to change the max amount of boost players can use. The mod allows team specific settings to allow asymmetrical gameplay (champs vs scrubs).
Changes proposed in this pull request

Added two new files: BoostMod.cpp, BoostMod.h.
Changed the solution to handle them.
Edited RocketPlugin.cpp to include the new gamemode.

Things add later on:

Allow options per player on each team instead of only teamwide.
Allow changing boost type (unlimited, recharge etc).
Possibly other boost related modifiers...

Attention!
These changes have been tested in a local lobby by myself since I couldn't get my friends to join. It might be a good idea to test them on a lobby with more players before shipping.

Rewrote it for the next update to get the style more inline with the rest of the project.


Chickneas/deprecate

added note in readme and link to new sample

Should we also formally mark it as deprecated in the setting of this repository?

yes, would I do that on this branch or after I merge?

@jchick-bentley @zachzhu2016 I'll archive it once this PR merges.


APK Can't be install in android

I have source dir like this:
pages
  pagelogin
      login.py
      login.kv
  pagehome
      home.py
      home.kv
assets
  .png
kivymd
main.py
root.kv

buildozer.spec:
`[app]

# (str) Title of your application
title = Importir Jam Tangan

# (str) Package name
package.name = importir

# (str) Package domain (needed for android/ios packaging)
package.domain = com.bs

# (str) Source code where the main.py live
source.dir = .

# (list) Source files to include (let empty to include all the files)
source.include_exts = py,pyc,png,jpg,jpeg,ttf,md,kv,json,atlas

# (list) List of inclusions using pattern matching
source.include_patterns = assets/*

#source.exclude_dirs=pages

# (str) Application versioning (method 2)
#version.regex = __version__ = ['"](.*)['"]
#version.filename = %(source.dir)s/kivymd/__init__.py

version = 11

# (list) Application requirements
# comma separated e.g. requirements = sqlite3,kivy
requirements = python3,kivy==1.11.1,requests,pillow,pygments,git+https://github.com/HeaTTheatR/KivyMD.git

# (str) Custom source folders for requirementss
# Sets custom source for any requirements with recipes
requirements.source.kivymd = kivymd

# (str) Presplash of the application
presplash.filename = %(source.dir)s/assets/splash.png

# (str) Icon of the application
icon.filename = %(source.dir)s/assets/logo.png

# (str) Supported orientation (one of landscape, sensorLandscape, portrait or all)
orientation = portrait

# (bool) Indicate if the application should be fullscreen or not
fullscreen = 0

# (string) Presplash background color (for new android toolchain)
android.presplash_color = #FFFFFF

# (list) Permissions
android.permissions = INTERNET,WRITE_EXTERNAL_STORAGE

# (int) Target Android API, should be as high as possible.
android.api = 28

# (int) Minimum API your APK will support.
android.minapi = 21

# (str) Android NDK version to use
android.ndk = 19b

p4a.branch=develop

# (bool) If True, then skip trying to update the Android sdk
# This can be useful to avoid excess Internet downloads or save time
# when an update is due and you just want to test/build your package
android.skip_update = False

# (bool) If True, then automatically accept SDK license
# agreements. This is intended for automation only. If set to False,
# the default, you will be shown the license when first running
# buildozer.
android.accept_sdk_license = True

# (str) Android logcat filters to use
android.logcat_filters = *:S python:D

# (str) The Android arch to build for, choices: armeabi-v7a, arm64-v8a, x86, x86_64
android.arch = armeabi-v7a

[buildozer]

# (int) Log level (0 = error only, 1 = info, 2 = debug (with command output))
log_level = 2

# (int) Display warning if buildozer is run as root (0 = False, 1 = True)
warn_on_root = 0

# (str) Path to build artifact storage, absolute or relative to spec file
build_dir = ./.buildozer

# (str) Path to build output (i.e. .apk, .ipa) storage
bin_dir = ./bin

# (list) Android additional libraries to copy into libs/armeabi
#android.add_libs_armeabi = libs/android/*.so
#android.add_libs_armeabi_v7a = libs/android-v7/*.so
android.add_libs_arm64_v8a = libs/android-v8/*.so
#android.add_libs_x86 = libs/android-x86/*.so
#android.add_libs_mips = libs/android-mips/*.so
`

if i using command -> buildozer android debug deploy run logcat my android launcher error
i used emulator but nothing error

@purnama11 Formulate the question so that it is understandable.

please tell me why apk can't install?

@purnama11 https://github.com/kivy/kivy/issues

Use and install android.ndk=19c (it work for me)
https://developer.android.com/ndk/downloads/older_releases


Update Identify-Your-AD-FS-Deployment-Goals.md

Fixed typo in artilce title.

@doctordns : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Unable to add Opensprinkler to an automation

I would like to turn on my sprinklers at a certain time but there is no way to add the sprinklers to an automation or a scene.

You have to use another App like Controller: https://apps.apple.com/de/app/controller-für-homekit/id1198176727


Enable search in discussions

Feature Description
I'd like to easily find text in discussions/notes. Currently, to find any previous thread, I usually have to toggle a couple of nodes and then use normal search /. This is fine if there are just a couple threads, but it gets really inconvenient in actual MRs with tens of threads.
Even if I can imagine an actual search functionality that would make it possible to also search in collapsed nodes, take me to the node with a match and expand it for me, a simple "expand_all_nodes" method would be sufficient to provide the basic functionality that I'm lacking.
Then it would of course make sense to have "collapse_all_nodes", to clean up the view. Actually, something like "toggle_all_nodes" would be best, to be able to only use one keybinding for both, but I'm not sure how the plugin would determine whether to expand or collapse nodes if some were collapsed and some expanded...

For now, I'm using this workaround for collapsing/expanding all nodes (I'm using i as my toggle_node mapping). It's good enough for the basic functionality (being able to search all nodes' texts), but it does not keep the cursor position. And it's vimscript :) I was not able to figure out how to run lua vim.cmd.normal on a specific line (and this seemed necessary because I had to iterate the lines from the end of the file to the beginning, since a simple g/^/normal i on closed nodes didn't work, as it changed the number of lines and the command only expanded the nodes that were within the original line range.
scriptencoding utf-8
let g:gitlab_nvim_nodes_expanded = 0
function! ExpandAllNodes()
    for i in range(line('$'), 1, -1)
        if match(getline(i), '^') > -1
            execute i . 'normal i'
        endif
    endfor
    let g:gitlab_nvim_nodes_expanded = 1
endfunction
nnoremap <silent> <buffer> <leader><tab> :silent call ExpandAllNodes()<cr>

function! CollapseAllNodes()
    g/^/normal i
    let g:gitlab_nvim_nodes_expanded = 0
endfunction
nnoremap <silent> <buffer> <leader><S-tab> :silent call CollapseAllNodes()<cr>

function! ToggleAllNodes()
    if g:gitlab_nvim_nodes_expanded
        silent call CollapseAllNodes()
    else
        silent call ExpandAllNodes()
    endif
endfunction
nnoremap <silent> <buffer> <tab> :silent call ToggleAllNodes()<cr>

I'm going to close this since normal / and ? search will be supported fine once you merge in the expand/collapse MR.


Update sensitivity-labels-teams-groups-sites.md

Missing space.

#sign-off for this missing space typo


Add StringOfLength assert

This is a nice little assert that allows us to replace the repeated:
validate(foo, {
  foo: [is.string(), is.ofLength({ max, min })]
});

// or

validate(foo, {
  foo: [is.notBlank(), is.ofLength({ max, min })]
});

with the simpler alternative:
validate(foo, {
  foo: is.stringOfLength({ max, min })
});

Since the vast majority of strings need to be validated for size anyway, this helps us avoid duplication for an extremely recurring pattern.
NullOrString also accepts max and min, but allows null values, so we can't always use it. Length is also not enough, because it accepts other objects that respond to length, such as arrays. String doesn't allow us to specify length boundaries. Hence the need for a new assert.

@ruimarinho Those look different, since a string can't be greater than anything, only its length can. Also, given that example, we could argue that the current asserts would already be missing options like LengthGreaterThan or LengthLessThan.
Anyway, this PR being stalled for so long makes me think it's no longer that relevant, so probably the best option is to simply close it until someone else feels the need for an assert like this.


use proj4 string for ST_transform

ref: #65
This fix #65 but introduce a 2ms slowdown, so we need a better SQL logic cc @bitner

latest commit fix most of the performance BUT it made it worse for non-epsg based CRS
here is what really needs to be reviewed
https://github.com/developmentseed/timvt/blob/a0794ac1b498658c9856d42433e6e17e0e1fcd79/timvt/layer.py#L128-L181

To be honest I can't think of a better solution to support non-epsg CRS. I think I'll add a note about the performance implication of using such CRS.


Merge master into major

This includes https://github.com/donejs/donejs/pull/769

Coverage remained the same at 96.396% when pulling cceaee1060f76996081e634104ab6ce2cb52568b on update-major into 162fbe72a34c7d8c4eb53e12b6a2c37def24e320 on major.


Added DeviceInfo to group sensors and switch under 1 device

Added DeviceInfo to group sensors and switch under 1 device.
Removed the enity_category from the switch so that it would show up under in the Controls section instead of in the Config section.

LGTM, thanks for improving the code base! Merged :)


🎄

                                          🎁 🎁 🎁
                                             🎁 🎁

Feel free to place your gift at the foot of the Christmas tree.

merry christmas

merry christmas :)

Merry Christmas to all Foundation related Github users :)

Happy Hanukkah!

Closed until next year ;)


Cannot set field value to NULL when editing Table's "Data" tab

Description
The right click context menu has no option to set a field value to NULL when editing table data using the table's "Data" tab view editor.
I realise the reason is because the field I'm trying to set to NULL is a non-nullable field, however it's also GENERATED ALWAYS so it must be NULL when inserted...
My strong opinion is that this option should never be excluded from being selected, regardless if the field is non-nullable or not. If the user NULLs a field accidentally and tries to save the edits, they'll get an error. We can then go back and fix it, rather than not being able to fix it due to lacking visibilility logic.

DBeaver Version
Community Version <IP_ADDRESS>406231636
Operating System
Windows 10
Database and driver
Postgre 15
Steps to reproduce

Create table with field set to non-nullable with a generated expression.
Insert a row
Duplicate the row, try to NULL the non-nullable generated field value

Additional context
No response

What's the column type?

It's been a while since no update here. If the issue is still actual please let me know and provide additional information.


Propragate darwin go

Motivation for this change
Better fix instead of https://github.com/NixOS/nixpkgs/pull/83099
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg build geoipupdate

@GrahamcOfBorg build hugo


Update license header and remove UTF-8 header

As IAG for science code, I can attest to the fact that the license header needn't be so elaborate.
Furthermore, marking files with a UTF-8 header was more relevant in older versions of Python, specifically before Python 3.0, to ensure correct interpretation of non-ASCII characters.  I can't think of a reason why this is included in IMPROVER headers.
(noticed this while adding new files as part of https://github.com/metoppv/improver/pull/2000)
Issues

https://github.com/MetOffice/improver_suite/issues/2214

...should we be using upper case? Not too worried, but others might be.

Indeed.  I wont get away with this 😋

You've not modified the bin/improver or bin/improver-tests files, which have slightly different license headers...

Ah, thanks.  I'll take a look.

This documentation doc/source/Code-Style-Guide.rst still lists the old header...

👍
Thanks for the feedback @bayliffe


Refactor: Reduce API calls, use built-in WP postmeta function calls

API calls might be affecting performance (with all the asynch stuff going on); maybe refactor that -- not a pressing thing, something down the line

Where can't I do this:

figures
modals


[Diary] Installing Fractal locally (process backend)

I went through a local installation of fractal on my mac on macos Ventura 13.1 once with a process runner and I could successfully run through the installation, setting up a client, collecting the actual tasks and running example 01. Great to see that everything is running in this setup!
I'll briefly document here what I did, then follow it up with some high level discussion points where we can improve and some minor issues that I observed.
Server installation
I installed a server environment by using the installation script here: https://github.com/fractal-analytics-platform/fractal-demos/tree/main/examples/server
. ./install_script.sh
I then started the server using a modified version of the server_config_and_start.sh: I changed the FRACTAL_RUNNER_BACKEND to process and removing the FRACTAL_SLURM_CONFIG_FILE=config_uzh.json from the env variable being set. The rest worked with the defaults.
. ./server_config_and_start.sh
(run this in a screen to have it running in the background)
Client installation
I installed the client from the 01_cardio_tiny_dataset example helper script. Everything executed below was in that environment:
. ./install_client.sh
I then ran the preliminary setup:
. ./preliminary_setup.sh
(and checked that the tasks would be collected successfully with fractal task check-collection 1)
Running an example experiment
I ran the 01_cardio_tiny_dataset example which has its test data on zenodo. For that to work, one needs to install the zenodo_get library first into the client env:
pip install zenodo_get
Then:
. ./fetch_test_data_from_zenodo.sh
One can then either go through the project creation, dataset creation, workflow creation & submission one by one. Or run it all at once by running:
. ./run_example.sh
This should complete fairly quickly (submitting the script to it being finished took 30s on my machine). One can check the status with fractal job show ID (where the ID is the job ID of the submitted workflow, 1 for the first workflow submitted. This is shown when submitting the workflow) [syntax is changing to this soon, at the moment, it still was fractal job status]
Open questions
In the preliminary setup, why do I need to provide a SLURM_USER when using a process runner? And what is the SLURM_USER variable used for in that case?
Learnings

We should have a global "How to get started" with Fractal that is not per repo, e.g. on the splash page here based on those demo scripts. If we add short READMEs to the demo scripts with what may need to be adapted, then the "How to get started" page could point to the server folder and e.g. the 01 example. And refer to the documentation for further details.
We should make sure to cover the option to use the process backend to run things locally, not just a slurm backend. The slurm backend is what we'll obviously use for deployments that need to scale, but for testing and to introduce people to Fractal, the local process runner that does not require a slurm setup is super helpful. The documentation can currently be slightly misleading here. i.e. the server Install and deploy page currently states having a SLURM client node as a requirement, while one can actually test Fractal without.

Minor observation
a) Some links in the documentation are currently broken. e.g. the links to computational backends and Configuration are broken on this page: https://fractal-analytics-platform.github.io/fractal-server/install_and_deploy/. It does not seem to affect links to external resources like pypi though. They point to https://fractal-analytics-platform.github.io/configuration/ instead of https://fractal-analytics-platform.github.io/fractal-server/configuration/ (missing /fractal-server inbetween). Not sure whether this affects other parts as well.

I have now updated the splash page with a "How to Get Started" section and added READMEs containing the relevant information contained here on how to run the example with the process backend.

A few answers will follow..

One can check the status with fractal job show ID (where the ID is the job ID of the submitted workflow, 1 for the first workflow submitted.

Note that ID is the job ID, not the workflow one. A job (technically called an ApplyWorkflow object, in the database) is created each time the workflow is applied, and for instance it also includes information on the I/O datasets.

This is shown when submitting the workflow) [syntax is changing to this soon, at the moment, it still was fractal job status]

Updated in `fractal-client=1.0.0rc1

In the preliminary setup, why do I need to provide a SLURM_USER when using a process runner? And what is the SLURM_USER variable used for in that case?

SLURM_USER is indeed optional on the server side, as it should be. There is a glitch on the client side, see https://github.com/fractal-analytics-platform/fractal/issues/404.

a) Some links in the documentation are currently broken.

Thanks.

Another question I had about the process runner:
Given that I removed the FRACTAL_SLURM_CONFIG_FILE variable and thus didn't specify any executors anymore, how does the process runner handle "submitting" all the things locally? Will it always run all wells in parallel? Or always run sequentially? I'm wondering whether a power user could set up Fractal on a powerful workstation using the process executor and how larger experiments would then be processed.

FRACTAL_SLURM_CONFIG_FILE only matters for the SLURM backend, as the variable name says ;)
The process backend directly submits all tasks (in the correct order) to a concurrent.futures.ThreadPoolExecutor. Tasks that have a parallelization_level attribute are run in parallel on multiple threads.
Important: this is multi-thread parallelization, but probably to get mid-size workflows running on a powerful workstation we would need multi-process one. We would then need to play around with (concurrent.futures.ProcessPoolExecutor)[https://docs.python.org/3/library/concurrent.futures.html#concurrent.futures.ProcessPoolExecutor]. Let's re-discuss about it.

Another question I had about the process runner:
Also, it appears to just be ignoring the executors I specified (or at least not complain that it doesn't have them specified).

This is expected.

Executors are always optional (even on the SLURM backend, where a default will be used if none is provided).
Executors only matter for the SLURM backend, while the process has no additional options (see above).

I have now updated the splash page with a "How to Get Started" section and added READMEs containing the relevant information contained here on how to run the example with the process backend.

Great, thanks! I'll review it as soon as I have a bit of time ;)

Thanks for the answers above @tcompa ! Makes sense!
I have no strong opinions on 1 vs 2 conda environments.
It may be easier to have a single folder with server & client code for the example. Benefit of the current setup is that we're anyway maintaining the server example folder and the 01 example. Running them in process runner mode takes almost no new changes, which is very convenient! So I'm also fine with them staying apart and maybe be a bit easier to maintain going forward.

Some small updates on the server (https://github.com/fractal-analytics-platform/fractal-server/pull/445 and others):

Clarified a bit that some requirements are optional, in https://fractal-analytics-platform.github.io/fractal-server/install_and_deploy/#preliminary-requirements.
Fixed a few broken links (let me know if you spot others).
General docs improvements here and there.

Ok, I think all the stumbling blocks mentioned here are now covered in the documentation & the READMEs :)


Why wont my background work when I go into Dms or servers?

This started happening after I restarted my PC and it has been working for days before I just restarted my PC like How you normally would and then it got screwed up. If anyone has a fix to this as soon as possible I would be very happy.

It's your theme, not BetterDiscord. Go bug the theme author regarding it.

same thing for me too, different theme tho

I've got the same issue too so I think it might be better discord that's the problem

It's not better discord, it's the theme, no matter what theme it is, when discord update it often break themes, just wait for a theme update


add DPR reader

For now, Pygaggle is for reranking only right?
As we are adding DPR reader, I am thinking about adding a new module for Reader.
So make pygaggle as a mix of reranker and reader.
@KaiSun314
Basically, what pygaggle doing rn is use a Reranker to rerank a list of Text for a Query. https://github.com/castorini/pygaggle/blob/master/pygaggle/rerank/base.py
To implement the Reader module, we want a Reader to read from a list of Context for a Question and gives an Answer, similar to what we did in bertserini (https://github.com/rsvp-ai/bertserini/blob/development/bertserini/reader/base.py)
I guess here, we can use the Text class as Context and Query for Question
so concretely, we need to define a data structure class for Answer and an abstract class Reader. The DPRReader should implement the Reader.
When implementing the DPRReader, we can use the API from transformers directly, https://huggingface.co/transformers/model_doc/dpr.html#dprreader, which will give us the logits for start/end.
To get the answer span from the logits, we can learn from how the dpr repo did here
CC: @lintool @ronakice

Yes, and after this add some standard QA metrics like EM/F1 post. We should start with the TextAnswer class representing the start/end indices plus the string but ideally, it should also be capable of handling Yes/No type answers (BooleanAnswer) and NULL type answers (NullAnswer). So this Answer class  will be of type Union[TextAnswer, BooleanAnswer, NullAnswer]

Yes, and after this add some standard QA metrics like EM/F1 post. We should start with the TextAnswer class representing the start/end indices plus the string but ideally, it should also be capable of handling Yes/No type answers (BooleanAnswer) and NULL type answers (NullAnswer). So this Answer class  will be of type Union[TextAnswer, BooleanAnswer, NullAnswer]

All sounds good to me at a high level. Let's look at draft PRs to discuss further.

All sounds good to me at a high level. Let's look at draft PRs to discuss further.

On it - reading DPR paper and familiarizing myself with the PyGaggle code, and then will implement a Reader module. ETA: probably a few days.

On it - reading DPR paper and familiarizing myself with the PyGaggle code, and then will implement a Reader module. ETA: probably a few days.

closed by https://github.com/castorini/pygaggle/pull/150


Archive versions below v0.67

Older React-Native versions docs do not receive many updated.
And those versions do not even appear on the version dropdown:

So it makes sense to me to declutter the RN website and remove older versions, moving them to a standalone immutable netlify deployment https://reactnative-archive-august-2023.netlify.app/ (created under Meta Open Source team)
My proposal also add the archived versions to the /versions page so that those older docs remain accessible forever:

Preview Url: https://deploy-preview-3818--react-native.netlify.app/versions
Benefits: smaller website, faster prod deployment.
Drawback: it becomes harder to edit any archived version docs, but it's probably not a problem IMHO.
This PR also simplifies some code (see DocsVersionDropdownNavbarItem).
Note: this archiving feature is not part of Docusaurus, it's just plain React code and you are free to change the versionsArchived.json data structure if you want extra attributes.
My recommendation is to archive some versions regularly, maybe at least once every 6 months, to avoid an ever increasing list of "live versions" that do not bring any real value.
Important: use a real standalone Netlify/Vercel site for each archived site. The "permalinks" on Netlify now have an expiration date.
Note: before merging this PR, please approve it. I'll delete the appropriate /versioned_docs and /versioned_sidebars files before merging. I tried to keep this PR clean and easy to review.

Thanks ;)
I wrote a RFC here with my proposals for a proper version archiving process that you could easily run on your own when needed: https://github.com/facebook/react-native-website/issues/3819
The goal of this PR is not to implement the perfect archiving system yet, but to be good enough to be merged as-is.
If someone at Meta can approve it fast, that would help me reduce the docs backport work needed in other PRs like this one: https://github.com/facebook/react-native-website/pull/3807
Once we all agree on a proper archiving process, it remains possible to rework this PR and migrate my current solution to the new cleaner solution.
The Netlify archive links in this PR could later be rewritten to become cleaner:
- "0.67": "https://reactnative-archive-august-2023.netlify.app/docs/0.67/getting-started",
+ "0.67": "https://archive.reactnative.dev/0.67/docs/getting-started",


touch raises misleading exception on non-existant directory

Hi,
I am using Snakemake v5.9.1, and the following error has been bugging me for quite some time.
In a workflow where certain rules have directory() as output, updating the state of the workflow with --touch fails if the respective directory does not exist. That alone seems like a bug, but the exception raised is an ImproperOutputException, presenting the wrong diagnosis for what went wrong (i.e., the output directory is correctly flagged with directory()). The following example reproduces the error:

rule master:
    input:
        'output/finish/testA/done.txt'

rule init:
    output:
        'output/init/{scenario}_foo.txt'
    run:
        with open(output[0], 'w') as foo:
            _ = foo.write('init\n')

rule intermediate:
    input:
        'output/init/{scenario}_foo.txt'
    output:
        folder = directory('output/intermediate/{scenario}/directory'),
        file = 'output/intermediate/{scenario}/complete.txt'
    run:
        os.makedirs(output.folder, exist_ok=True)

        with open(output.file, 'w') as dump:
            _ = dump.write('folder created')

rule finish:
    input:
        'output/intermediate/{scenario}/complete.txt'
    output:
        'output/finish/{scenario}/done.txt'
    run:
        with open(output[0], 'w') as dump:
            _ = dump.write('done\n')

After running the above, remove the intermediate directory (rm -rf output/intermediate/testA/directory), and the result (rm output/finish/testA/done.txt), and rerun with --touch. The following error message appears:
ImproperOutputException in line 14 of /<PATH>/smk_exception/Snakefile:
Outputs of incorrect type (directories when expecting files or vice versa). Output directories must be flagged with directory(). for rule intermediate:
output/intermediate/testA/directory
Shutting down, this might take some time.
Exiting because a job execution failed. Look above for error message

Best,
Peter

I experimented exactly the same problem with snakemake 5.20.1

Experiencing the same issue on 6.1.1

I see the same issue on v6.15.5

Same in 7.18.2

After all this time, the fix seems to be fairly straightforward. If folks could test the PR in their own environment, that would be helpful #2051


TEAMS.md is outdated

Context
Our organization has changed a bit in the past few months as new subteams have been created and new contributors have joined various subteams.
Proposal
The TEAMS.md document should be updated to include the following:
Ruby: rmoran, swigmore
Builders: fg-j, dumez-k, ekcasey
Stacks: dumez-k, mdelillo, martyspiewak

Resolved by #7


fix: TOB-ZKTRIE-18 get_account can read past the buffer

Verify the slice length before blindnly casting and transmuting.

To see the specific tasks where the Asana app for GitHub is being used, see below:

https://app.asana.com/0/0/1205044772140415

fix is incorrect.

Updated with the correct fix.

rebased on main.


I would like new versions of VS code to be installed automatically without any interaction.

I would like new versions of VS code to be installed automatically without any interaction. Currently when I open VS code and there is a new version, VS code asks if I want to install the update. If I click yes, VS code asks if I want to restart VS code. I don't want my workflow to be interrupted as soon as I start up VS code. I would like VS code to handle installing new versions without any interaction on all platforms, making the process transparent for the user.
How I imagine this would work:

I open VS code
I use VS code, VS code downloads the new version in the background and installs it (no notification)
I close VS code
The next time I open VS code, the new version is already installed

Are you using the System Setup?

The location is C:\Program Files\Microsoft VS Code, so I assume it was installed using the system setup.

Can you uninstall and reinstall the User Setup instead? This has a better update flow.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


RouterView component doesn't provide routeLocationKey

Reproduction
https://github.com/MichealPearce/vue-router-issue-123131
Steps to reproduce the bug
Was working on a tab system with vue router. Where each tab is a different RouterView with a different url. In my example, I have an array of strings which are urls I pass to router.resolve which is then passed to the RouterView component as a prop.
I found that the RouterView component does not provide the routeLocationKey which is what useRoute injects. Because of this, the result of useRoute is different than what the RouterView component is displaying and this causes issues. In my example, I have a page that is watching the route object and logs when it updates.

open app and add three tabs.
navigate to one of the different pages with one of the tabs being the "about" page.
select a tab that isn't the "about" page tab.
Navigate between the "home" page and the "contact" page.
See that the "about" page logs that the route has been changed when that tab hasn't been changed.

Expected behavior
The route provided as a prop to RouterView to be provided and used for useRoute
Actual behavior
RouterView component does not provide routeLocationKey so useRoute is not effected by it.
Additional information
This could be intentional but it seems like having useRoute return something that could be different from the route that the RouterView component is displaying could be an issue. At-least it was for my use case in building tabs like this.
This can also be worked around by creating a component that provides routeLocationKey yourself and wrapping it around the component like so.

This behavior is indeed expected. The inner behavior of RouterView is a bit more complex than just the route so there are other implications to changing this behavior.
Note you can always provide a custom route version for a custom tab system to work but this is outside of vue-router's scope. You could technically even override vue-router useRoute()


Update CODEOWNERS

Oops, forgot to change this to the frontend team before I left.

@avo I can't manually add reviewers, so mentioning you this way 🙂

@avo Since I don't have push access, you will need to merge these in for me.


Add better error handling around connection pulls (Already Piped, etc)

Errors are often popping up around https://github.com/multiformats/js-multistream-select/blob/master/src/dialer/index.js#L47. We need to ensure that we are catching errors at this level so that they can be passed up the callback chain instead of ending up as Uncaught Errors.
References:

https://github.com/ipfs/js-ipfs/issues/1446
https://github.com/libp2p/js-libp2p-switch/issues/235

Catch already piped errors on dialer.handle https://github.com/multiformats/js-multistream-select/pull/44

This is stale as of the 0.15 refactor


Reflections upgrade to 0.9.9

Reflection's 0.9.8 dependency javassist doen't properly support java8 and was throwing:
Error: java.io.IOException: invalid constant type: 18
Running some unit tests with PowerMockRunner caused this issue, so this fixes it for future tests that use it.

ops :)

changes looks good to me. Jenkins failures are unrelated. I am running some tests. will merge to master post that.


New processor-tests module

Bah, I keep working on it and failing to get it in! This adds a new module that includes a few simple processor tests and in addition it includes some dynamic tests generated based on what is in module-tests and samples. The dynamic tests take a while (since we are resolving dependencies and spinning up compilers) so they do not run on unless you specify -DgeneratedTests=true - you can specify that via the maven command line or you can set it in the launch config in eclipse.  The value here is that you can debug the processor running inside your IDE. Add your test to module-tests or to samples as normal, then run the processor-tests (with the flag set) - you'll get a new entry for your new test in the generated tests, then can re-run that in debug mode to test compilation of your code with the processor and test execution.

Merged in 20f96df


upgrade x/image dependency to support v4 and v5 bmp info headers

BITMAPV4HEADER and BITMAPV5HEADER support for image/bmp was introduced in https://go-review.googlesource.com/c/image/+/141799/ and solves the "bmp: unsupported BMP image" errors when trying to open BMP files with such headers: https://github.com/golang/go/issues/27767

Coverage remained the same at 100.0% when pulling 675e3c209ff3e9bbee22db0bffe990d3abace4ce on Hsn723:master into 9aab30e6aa535fe3337b489b76759ef97dfaf362 on disintegration:master.

Thank you!


[SPARK-14570][ML] Log instrumentation in Random forests

What changes were proposed in this pull request?
Added Instrumentation logging to DecisionTree{Classifier,Regressor} and RandomForest{Classifier,Regressor}
How was this patch tested?
No tests involved since it's logging related.

@thunterdb @jkbradley
I removed the redundant logging from RandomForest but I was curious as to whether all logging in RandomForest should go through Instrumentation, please advise.

Test build #56403 has started for PR 12536 at commit e2c69ca.

Test build #56403 has finished for PR 12536 at commit e2c69ca.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/56403/
Test PASSed.

@BenFradet thanks for the PR. I have one comment about performance.

Oh sorry, I had not realized that @jkbradley had asked you to remove the parameter. @jkbradley there is no performance issue doing that?

@thunterdb thanks for review, will fix the points you brought up.

Test build #56700 has started for PR 12536 at commit eaaafc4.

Test build #56700 has finished for PR 12536 at commit eaaafc4.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/56700/
Test PASSed.

Test build #57141 has started for PR 12536 at commit 85251e8.

Test build #57141 has finished for PR 12536 at commit 85251e8.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/57141/
Test FAILed.

@BenFradet it looks like there are some compilation issues

Test build #57158 has started for PR 12536 at commit 21e136e.

@thunterdb yup sorry, I just came back and saw this

Test build #57158 has finished for PR 12536 at commit 21e136e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/57158/
Test PASSed.

@BenFradet thanks!
@jkbradley can you make a final pass?

Looks good, except for that 1 comment

Test build #57369 has started for PR 12536 at commit 698a005.

LGTM pending tests
Thank you!

Test build #57369 has finished for PR 12536 at commit 698a005.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/57369/
Test PASSed.

Merging with master


Range-ify std.string.rightJustify() with rightJustifier()

100% unit test coverage
http://dlang.org/phobos/std_string.html#.rightJustify

Rest LGTM.

This unnecessary technical debt

It's necessary. Note that the original functions' implementations have been gutted and are now trivial one-liners. I'd like them to eventually be deprecated and removed, but that is very long term. The lazy range functions are the future of D.

Do you think consistency with #3224 is worth the trouble?


Content Creator: Display content status

As a Content Creator, I want to see the status of any piece of content so that I can make the best decision on what to work on
Feature: Display content status
  As a Content Creator,
  I want to see the status (e.g draft, waiting approval, approved) of any piece of content
  So that I can make the best decision on what to work on

  Scenario: Content table
    Given I will need to create or edit content
      And that I will want to know the contents status before working on it
     When I view a list of the content
     Then I will see the contents status

[x] Display content status / Content table

I believe this was resolved with https://github.com/punchcard-cms/punchcard/issues/236. Please check our running instance. If there is more to do here, please flesh out the story to describe in detail what the user expectation is.

Thanks, @Snugug. Yeah, looks like we are showing the status on the revisions screen now.
http://watson-cms.stage1.mybluemix.net/content/api/f82bca2a-ce61-4c01-bb87-e10abbbb2c89


build(dependencies): 🧱 update crates

Description
Update dependencies:
Bumps reqwest from 0.11.26 to 0.12.2. Updating aho-corasick v1.1.2 -> v1.1.3
Updating autocfg v1.1.0 -> v1.2.0 Updating backtrace v0.3.69 -> v0.3.71 Updating
bytes v1.5.0 -> v1.6.0 Updating clap v4.5.3 -> v4.5.4 Updating clap_derive
v4.5.3 -> v4.5.4 Updating fastrand v2.0.1 -> v2.0.2 Updating indexmap v2.2.5 ->
v2.2.6 Updating regex v1.10.3 -> v1.10.4 Updating rustix v0.38.31 -> v0.38.32
Updating smallvec v1.13.1 -> v1.13.2 Updating syn v2.0.53 -> v2.0.55 Updating
toml_edit v0.22.8 -> v0.22.9
Type of change

[x] Dependency update

How Has This Been Tested?

[x] cargo test run with all tests passing

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
[x] 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
[x] Any dependent changes have been merged and published in downstream modules

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 42.72%. Comparing base (0b22491) to head (14f8d71).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #24   +/-   ##
=======================================
  Coverage   42.72%   42.72%
=======================================
  Files           4        4
  Lines        1161     1161
=======================================
  Hits          496      496
  Misses        665      665

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


ImportError: cannot import name constants when upgrading from 2.4.1 to 2.5.2

My instances upgraded from 2.4.1 to 2.5.2 and importing boto fails as follows.  Rolling back to 2.4.1 fixes the issue.
from boto import connect_s3, connect_gs
  File "/home/something/.virtualenvs/something/lib/python2.7/site-packages/boto/__init__.py", line 1216, in <module>
    boto.plugin.load_plugins(config)
  File "/home/something/.virtualenvs/something/lib/python2.7/site-packages/boto/plugin.py", line 93, in load_plugins
    _import_module(file)
  File "/home/something/.virtualenvs/something/lib/python2.7/site-packages/boto/plugin.py", line 75, in _import_module
    return imp.load_module(name, file, filename, data)
  File "/usr/lib/python2.7/site-packages/google_compute_engine/boto/boto_config.py", line 30, in <module>
    from google_compute_engine import constants
ImportError: cannot import name constants

Are you installing the pypi package for google-compute-engine in the virtualenv? If so, you probably need to update it to that it includes the new constants file.

Please let us know if upgrading your virtual env package works. Re-open this issue if you run into problems.

Yes, updating the google-compute-engine and boto packages corrected the issue.  Thanks.


Tomcat output some error when shutdown

Some thread cannot close success:
二月 04, 2015 11:44:02 下午 org.apache.catalina.core.ApplicationContext log
信息: Closing Spring root WebApplicationContext
二月 04, 2015 11:44:02 下午 org.apache.catalina.loader.WebappClassLoader clearReferencesJdbc
严重: The web application [/book] registered the JDBC driver [com.mysql.jdbc.Driver] but failed to unregister it when the
web application was stopped. To prevent a memory leak, the JDBC Driver has been forcibly unregistered.
二月 04, 2015 11:44:02 下午 org.apache.catalina.loader.WebappClassLoader clearReferencesThreads
严重: The web application [/book] appears to have started a thread named [Abandoned connection cleanup thread] but has failed to stop it. This is very likely to create a memory leak.
二月 04, 2015 11:44:02 下午 org.apache.catalina.loader.WebappClassLoader clearReferencesThreads
严重: The web application [/book] appears to have started a thread named [ebean-mysql-periodic-1] but has failed to stop i
t. This is very likely to create a memory leak.
二月 04, 2015 11:44:02 下午 org.apache.catalina.loader.WebappClassLoader clearReferencesThreads
严重: The web application [/book] appears to have started a thread named [ebean-mysql1] but has failed to stop it. This is
 very likely to create a memory leak.

This is not a ebean bug.
You should close (shutdown - EbeanServer.shutdown) ebean in your web application - ServletContextListener.

On Sun, Feb 8, 2015 at 9:02 PM, Ryszard Trojnacki<EMAIL_ADDRESS>wrote:

You should close (shutdown - EbeanServer.shutdown) ebean in your web
application - ServletContextListener.

I didn't notice this method, I'll try it now, 3kx

I'll assume this can be closed.


Consider adding ArgumentNullException.ThrowIfNullOrEmpty(...)

i went with a different approach https://github.com/SimonCropp/Polyfill?tab=readme-ov-file#guard-1


"vite-svg-loader" resolved to an ESM file. ESM file cannot be loaded by "require" with v4.1.0+

env: vite4.5 + vue3 + node20

v4.0.0 works fine (for a long time).
v4.1.0+ (including latest v5.0.1/Vite5) is giving "vite-svg-loader" resolved to an ESM file. ESM file cannot be loaded by "require" error on dev.
It's weird since I am not using "require" at all.
More details of the error:
✘ [ERROR] "vite-svg-loader" resolved to an ESM file. ESM file cannot be loaded by `require`. See http://vitejs.dev/guide/troubleshooting.html#this-package-is-esm-only for more details. [plugin externalize-deps]

    node_modules/esbuild/lib/main.js:1373:27:
      1373 │         let result = await callback({
           ╵                            ^

  This error came from the "onResolve" callback registered here:

    node_modules/esbuild/lib/main.js:1292:20:
      1292 │       let promise = setup({
           ╵                     ^

  The plugin "externalize-deps" was triggered by this import

    vite.config.ts:8:22:
      8 │ import svgLoader from 'vite-svg-loader'
        ╵                       ~~~~~~~~~~~~~~~~~

my vite.config.ts is as follows:
import svgLoader from 'vite-svg-loader'

export default defineConfig(({ command, mode }) => {
  // handle modes
  return {
    plugins: [
      // more plugins
      svgLoader({
        svgoConfig: {
          // svgo configs
        },
      }),
    ]
  }
}

Yes, i've updated the module to use ESM, because Vite 5 deprecated the CJS Node API, see https://vitejs.dev/guide/migration#deprecate-cjs-node-api
You can fix it for now by setting "type": "module" in your package.json file, or use the .mjs extension, e.g. vite.config.mjs.
I'll look if I can create a hybrid version, so that CJS also will work for all users of Vite v4


Broken with angular >=1.3.0

Tried several versions of angular (1.3.0 - 1.3.14), and with all of them I got an error when connecting to google devs (after providing my googleApiKey).
Details:

Relevant info at the end of the "json" dump

Pastebin with full dump: http://pastebin.com/kUVWE6QE
PS: that one was generated when working with angular version 1.3.8 / notice the error was replicated with several other 1.3.X versions

This may be related to #36 #37
First of all, we have this project in production, along with angular 1.3.14
I tried to reproduce this, but have not figured out a way to do so.
According to the error, it seems as if there's a watcher, picking up every font as it's being added to the list (which is a lot when you're using google fonts). But i've no clue how this is possible..
Please set up a jsFiddle, illustrating the setup you're using. You can use my test-api key when you're worried about publishing yours AIzaSyCZtrpL_EVp0qm-jnMvXP4xfvrDcW61pXg.

I got the same error message when I was missing the webfontloader.js file. I had no clue this dependency was needed until I looked into the demo files. Do you think you could mention this in the readme?

Nice! I assume this was the issue. At least i could reproduce it by removing the webfontloader lib.
I updated the documentation and throw a readable Error if the library is not available.
I also updated the bower main section, its now including the library and the css.
Since this will be picked up by tools like wiredep, i released this as a minor update.
Thanks for the catch @brad-sobie !


[dv] Enable xcelium to include X for toggle coverage

Address #10332
Signed-off-by: Weicai Yang<EMAIL_ADDRESS>

Merging, given the private CI passed.


DANN with multiclass problems

It seems that the implemented DANN class does not work with multiclass, also if I provide a custom task network having as last layer Dense(n_classes, activation='softmax').

Hi @andreapi87,
Thank you for your interest in the ADAPT library!
What do you mean exactly by "it does not work"? What is the error message you get?
Can you please post the whole code you used to help us identify the problem?
Best,


README configuration Typescript errors: NodeJSConfiguration interface doesn't have metricExporter and metricInterval properties

NodeSJConfiguration interface does not have metricExporter and metricInterval properties causing TS errors using the README config. Im using latest nest-otel and latest versions of. all dependencies.
Im using the following combination of packages
        "@nestjs/common": "^9.0.0",
        "@nestjs/core": "^9.0.0",
        "nestjs-otel": "^5.0.0",
	"@opentelemetry/auto-instrumentations-node": "^0.36.3",
	"@opentelemetry/exporter-jaeger": "^1.9.1",
	"@opentelemetry/exporter-prometheus": "^0.35.1",
        "@opentelemetry/sdk-node": "^0.35.1",

with the following config copied from the README (note I aded a jaeger trace exporter)
const otelSDK = new NodeSDK({
	metricExporter: new PrometheusExporter(),
	metricInterval: 1000,
        traceExporter: new JaegerExporter(),
	spanProcessor: new BatchSpanProcessor(new JaegerExporter()),
	contextManager: new AsyncLocalStorageContextManager(),
	textMapPropagator: new CompositePropagator({
		propagators: [
			new JaegerPropagator(),
			new W3CTraceContextPropagator(),
			new W3CBaggagePropagator(),
			new B3Propagator(),
			new B3Propagator({
				injectEncoding: B3InjectEncoding.MULTI_HEADER,
			}),
		],
	}),
	instrumentations: [getNodeAutoInstrumentations()],
});

Neither metricExporter nor metricInterval are defined on the NodeSDKConfiguration interface passed to NodeSDK()
export interface NodeSDKConfiguration {
    autoDetectResources: boolean;
    contextManager: ContextManager;
    defaultAttributes: SpanAttributes;
    textMapPropagator: TextMapPropagator;
    metricReader: MetricReader;
    views: View[];
    instrumentations: InstrumentationOption[];
    resource: Resource;
    resourceDetectors: Detector[];
    sampler: Sampler;
    serviceName?: string;
    spanProcessor: SpanProcessor;
    traceExporter: SpanExporter;
    spanLimits: SpanLimits;
}

Aside from addressing the type incompatibility, how/where do i define my metric parameters if they don't exist on NodeSDKConfiguration? Is that possible with this configuration and package combination?
Thanks

@laurencefass I ran into the same issue. After playing around with the example apps in the README, I noticed the versions for @opentelemetry/sdk-node were different. For example, the config you've shared works in this example because the node sdk  version is "^0.24.0". However, in this other example your config would not work because the package version is "^0.34.0". I'm not sure what version changed the interface but I do know that metricExporter and metricInterval are no longer available options, as you pointed out.
The good news is that the metricReader field in the latest version will accept valid metric exporters (eg PrometheusExporter). I ended up using the setup from this example and things are working as expected.
Hope that helps!

@pragmaticivan do you have any context? Do dependencies need to be updated on this lib?


takeUntil optimized cache and network method and Gson parse model

Hey
Thank you for this awesome examples :)
I am trying to implement the optimized cache method using takeUntil in my app. I am using a Gson model to parse the JSON response  from the API .
I've been testing several methods but didn't succeed to get it right.am beginner in RxJava .
My code
https://gist.github.com/alouanemed/6b5f4d5d3623ae69331de25c12dd5b98
Thanks :) .

@alouanemed on the first look on your code, your implementation seems fine, anyway, I will check it more deeply and reply you.


"TypeError: from_buffer() cannot return the address of the raw string within a bytes or unicode object" when creating a GCSFileSystem object

What happened:
When I tried to load a GCSFileSystem object, I got the exception TypeError: from_buffer() cannot return the address of the raw string within a bytes or unicode object.
What you expected to happen:
I expect the GCSFileSystem object to instantiate and be usable. This has worked in the past. To my knowledge, nothing relevant on my machine has changed since it did. This works on a different machine using the same token file.
Minimal Complete Verifiable Example:
from gcsfs import GCSFileSystem

fs = GCSFileSystem(token="my-token.json")

I get the following exception.
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
/opt/anaconda3/envs/deepnlp/lib/python3.6/site-packages/gcsfs-0.7.1-py3.6.egg/gcsfs/core.py in _dict_to_credentials(self, token)
    326             token = service_account.Credentials.from_service_account_info(
--> 327                 token, scopes=[self.scope]
    328             )

/opt/anaconda3/envs/deepnlp/lib/python3.6/site-packages/google/oauth2/service_account.py in from_service_account_info(cls, info, **kwargs)
    210         signer = _service_account_info.from_dict(
--> 211             info, require=["client_email", "token_uri"]
    212         )

/opt/anaconda3/envs/deepnlp/lib/python3.6/site-packages/google/auth/_service_account_info.py in from_dict(data, require)
     54     # Create a signer.
---> 55     signer = crypt.RSASigner.from_service_account_info(data)
     56

/opt/anaconda3/envs/deepnlp/lib/python3.6/site-packages/google/auth/crypt/base.py in from_service_account_info(cls, info)
    113         return cls.from_string(
--> 114             info[_JSON_FILE_PRIVATE_KEY], info.get(_JSON_FILE_PRIVATE_KEY_ID)
    115         )

/opt/anaconda3/envs/deepnlp/lib/python3.6/site-packages/google/auth/crypt/_cryptography_rsa.py in from_string(cls, key, key_id)
    133         private_key = serialization.load_pem_private_key(
--> 134             key, password=None, backend=_BACKEND
    135         )

/opt/anaconda3/envs/deepnlp/lib/python3.6/site-packages/cryptography-3.1-py3.6-macosx-10.9-x86_64.egg/cryptography/hazmat/primitives/serialization/base.py in load_pem_private_key(data, password, backend)
     17     backend = _get_backend(backend)
---> 18     return backend.load_pem_private_key(data, password)
     19

/opt/anaconda3/envs/deepnlp/lib/python3.6/site-packages/cryptography-3.1-py3.6-macosx-10.9-x86_64.egg/cryptography/hazmat/backends/openssl/backend.py in load_pem_private_key(self, data, password)
   1248             data,
-> 1249             password,
   1250         )

/opt/anaconda3/envs/deepnlp/lib/python3.6/site-packages/cryptography-3.1-py3.6-macosx-10.9-x86_64.egg/cryptography/hazmat/backends/openssl/backend.py in _load_key(self, openssl_read_func, convert_func, data, password)
   1440     def _load_key(self, openssl_read_func, convert_func, data, password):
-> 1441         mem_bio = self._bytes_to_bio(data)
   1442

/opt/anaconda3/envs/deepnlp/lib/python3.6/site-packages/cryptography-3.1-py3.6-macosx-10.9-x86_64.egg/cryptography/hazmat/backends/openssl/backend.py in _bytes_to_bio(self, data)
    663         """
--> 664         data_ptr = self._ffi.from_buffer(data)
    665         bio = self._lib.BIO_new_mem_buf(data_ptr, len(data))

TypeError: from_buffer() cannot return the address of the raw string within a bytes or unicode object

my-token.json contains a dictionary with keys like type, project_id, private_key, etc.
When I step into the code I see that the value of data in _bytes_to_bio is of type bytes and is equal to the private_key value in my JSON token file.
Anything else we need to know?:
I see the same bug if I pass a dictionary to the GCSFileSystem constructor instead of a path to a token file.
This failing function is being called by GCSFileSystem._dict_to_credentials. There is a try/except block which  tries to read the credential information directly from the token in the except block. The except block is failing with a KeyError for me because my token does not contain a refresh_token field. So it may be the case that other people are hitting this bug, but aren't realizing it because their token does contain a refresh_token field.
Environment:

Dask version: dask-glm==0.2.0, dask-ml==1.7.0, gcsfs==0.7.1, argon2-cffi==20.1.0, cffi==1.7.0, cryptography==3.1
Python version: 3.6.12
Operating System: OS X, 10.15.7
Install method (conda, pip, source): conda

This has worked in the past. To my knowledge, nothing relevant on my machine has changed since it did.

Do you know which version this was? That particular piece of the code hasn't changed. I would also appreciate, if you have the time, if you would try agains the current main branch. I'm note sure how much we can do about something that apparently is happening deep within pycryptography.

I put all the version numbers I thought were relevant at the bottom of the original report. Is there another one you need?
I'll try installing from master when I get a chance.

You said that the same thing used to work on gcsfs, so the version for which that was true. Also the set of google packages google-auth, google-auth-oauthlib, google-api-core, google-api-python-client


Formula for GNU zgrep

BSD grep is slow and I assume BSD zgrep also is. It would be great to have a fast and convenient way to grep directories of compressed log files.

We don't do formula requests, sorry. If this isn't already part of the grep formulae we'd consider accepting a PR to add it to that.


geen idd

Thank you 🙇♀ for wanting to create an issue in this repository. Before you do, please ensure you are filing the issue in the right place. Issues should only be opened on if the issue relates to code in this repository.

If you have found a security issue please submit it here
If you have questions about writing workflows or action files, then please visit the GitHub Community Forum's Actions Board
If you are having an issue or question about GitHub Actions then please contact customer support

If your issue is relevant to this repository, please delete this text and continue to create this issue. Thank you in advance.

Hi
İş this mail personal or you sent it ,to all
13 dek 2024, C., 21:05 tarixində jommeke22f @.***>
yazdı:

Thank you 🙇♀ for wanting to create an issue in this repository. Before
you do, please ensure you are filing the issue in the right place. Issues
should only be opened on if the issue relates to code in this repository
.

If you have found a security issue please submit it here
https://hackerone.com/github
If you have questions about writing workflows or action files, then
please visit the GitHub Community Forum's Actions Board
https://github.community/c/code-to-cloud/github-actions
If you are having an issue or question about GitHub Actions then
please contact customer support
https://help.github.com/en/articles/about-github-actions#contacting-support

If your issue is relevant to this repository, please delete this text and
continue to create this issue. Thank you in advance.
—
Reply to this email directly, view it on GitHub
https://github.com/actions/cache/issues/1519, or unsubscribe
https://github.com/notifications/unsubscribe-auth/BL5ISFL5NIF4PBJ53GJXLLT2FMHVTAVCNFSM6AAAAABTSMD5DKVHI2DSMVQWIX3LMV43ASLTON2WKOZSG4ZTQOBSGUZTINA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Dependencies

This PR is attempting to fix all the warning messages displayed when you run "yarn install".
@Thivesshj  @RobynHancock  @RyanBroemer  @jademichellepeche can everyone please pull this branch and run yarn install in the Gym-King Directory
Run the front-end afterwards to test if its still working

Should this be merged Dylan?
Will this fix all the dependencies?

Should this be merged Dylan? Will this fix all the dependencies?

let me first figure out how we revert PR merges just incase

@RyanBroemer do you have any idea why that test is failing for the typeorm ?


Keywords file update to show all Methods

Hi, can you please update the "keywords.txt" file (with the below entries), so that all the Class Method colours adjust accordingly in the IDE.
Thanks
Tony
#######################################
Methods and Functions (KEYWORD2)
#######################################
begin	KEYWORD2
setPort	KEYWORD2
setSamplingInterval	KEYWORD2
connectToWifi	KEYWORD2
monitorTaskStatic	KEYWORD2

Thanks it's now added, will be included in the next release


Add import for checkbox group

This change is

Note: added to ES module version too, for consistency.